Skip to content
This repository was archived by the owner on Feb 8, 2024. It is now read-only.

CORTX-32793: update rgw resource limit formula #131

Merged
merged 1 commit into from
Sep 13, 2022
Merged

CORTX-32793: update rgw resource limit formula #131

merged 1 commit into from
Sep 13, 2022

Conversation

@sjain09
Copy link
Contributor

@sjain09 sjain09 commented Sep 12, 2022

After fixing defect CORTX-34142, update needs to be made for rgw resource limit related parameters in-order fine tune the formula.

Signed-off-by: Saurabh Jain [email protected]

Problem Statement

  • Problem statement

Design

  • For Bug, Describe the fix here.
  • For Feature, Post the link for design

Coding

Checklist for Author

  • Coding conventions are followed and code is consistent

Testing

Checklist for Author

  • Unit and System Tests are added
  • Test Cases cover Happy Path, Non-Happy Path and Scalability
  • Testing was performed with RPM

Impact Analysis

Checklist for Author/Reviewer/GateKeeper

  • Interface change (if any) are documented
  • Side effects on other features (deployment/upgrade)
  • Dependencies on other component(s)

Review Checklist

Checklist for Author

  • JIRA number/GitHub Issue added to PR
  • PR is self reviewed
  • Jira and state/status is updated and JIRA is updated with PR link
  • Check if the description is clear and explained

Documentation

Checklist for Author

  • Changes done to WIKI / Confluence page / Quick Start Guide

@sjain09
CORTX-32793: update rgw resource limit formula
4e99029 
After fixing defect CORTX-34142, update needs to be made for rgw resource limit
related parameters in-order fine tune the formula.

Signed-off-by: Saurabh Jain <[email protected]>
@sjain09 sjain09 requested a review from cdeshmukh as a code owner September 12, 2022 14:28
@cla-bot cla-bot bot added the cla-signed label Sep 12, 2022
sachitanands
sachitanands approved these changes Sep 12, 2022
View reviewed changes
Copy link
Contributor

@sachitanands sachitanands left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@sjain09
Copy link
Contributor Author

sjain09 commented Sep 13, 2022

retest this please

@cdeshmukh
Copy link
Contributor

cdeshmukh commented Sep 13, 2022

@sjain09 please check why 1N deployment failing, meantime i will retrigger
https://eos-jenkins.colo.seagate.com/job/Cortx-PR-Build/job/Cortx-Deployment/job/Generic/job/rgw-integration/159/

@cdeshmukh
Copy link
Contributor

cdeshmukh commented Sep 13, 2022

retest this please

@cdeshmukh cdeshmukh merged commit 1123710 into Seagate:main Sep 13, 2022
@sjain09 sjain09 mentioned this pull request Sep 13, 2022
Merged
12 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

4 more reviewers

@cdeshmukh cdeshmukh cdeshmukh approved these changes

@priyanka25081999 priyanka25081999 priyanka25081999 approved these changes

@shriya-deshmukh shriya-deshmukh shriya-deshmukh approved these changes

@sachitanands sachitanands sachitanands approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sjain09 @cdeshmukh @priyanka25081999 @shriya-deshmukh @sachitanands