-
Notifications
You must be signed in to change notification settings - Fork 5
Enhance RFS volume support: add bandwidth/iops update handling #54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check the comments
🔴 Coverage decreased from [87.7845%] to [87.5569%] |
2 similar comments
🔴 Coverage decreased from [87.7845%] to [87.5569%] |
🔴 Coverage decreased from [87.7845%] to [87.5569%] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to setup review call and close on this @renukasingare
🔴 Coverage decreased from [87.7845%] to [87.1293%] |
🔴 Coverage decreased from [87.7845%] to [87.1328%] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change library and driver and test once.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check the comment once
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please put retry codes in utils.go and restore the retry logic
🔴 Coverage decreased from [87.7845%] to [87.7103%] |
🔴 Coverage decreased from [87.7845%] to [87.6879%] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check minor comments
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
Signed-off-by: Renuka <[email protected]>
1368a13
to
88e0ca2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM , lets do final review in master once I do maturity beta changes
🔴 Coverage decreased from [87.7845%] to [87.7414%] |
GHE -> https://github.ibm.com/alchemy-containers/armada-storage/issues/7698
ibmcloud-volume-interface -> IBM/ibmcloud-volume-interface#53
Add regional share support i.e rfs profile support in the library (please add support for bandwidth and iops)
curl logs (before making changes) -> https://github.ibm.com/orgs/alchemy-containers/projects/99/views/27?pane=issue&itemId=1649277&issue=alchemy-containers%7Carmada-storage%7C7658
curl logs(After code fix ) :-> https://github.ibm.com/orgs/alchemy-containers/projects/99/views/27?pane=issue&itemId=1723995&issue=alchemy-containers%7Carmada-storage%7C7795
samples log :-> https://github.ibm.com/orgs/alchemy-containers/projects/99/views/27?pane=issue&itemId=1724009&issue=alchemy-containers%7Carmada-storage%7C7796