HDDS-10612. Add Robot test to verify Container Balancer for RATIS containers - #6457
Conversation
adoroszlai
left a comment
There was a problem hiding this comment.
Thanks @afilpp for the patch. Overall looks good, added some minor comments.
It would be nice to create the environment as an add-on for ozone-ha instead of a completely separate one, but we can check if it's feasible in a follow-up task.
adoroszlai
left a comment
There was a problem hiding this comment.
Thanks @afilpp for updating the patch, LGTM.
myskov
left a comment
There was a problem hiding this comment.
LGTM, @siddhantsangwan please take a look
siddhantsangwan
left a comment
There was a problem hiding this comment.
The robot test logic LGTM.
|
Seems there is an intermittent failure on the acceptance test https://github.com/apache/ozone/actions/runs/8546730074/job/23418032793 @afilpp Could you take a look? Edit: Can refer to the comment in HDDS-10612 for possible root cause. |
…tainers (apache#6457) (cherry picked from commit 129cdc1)
…tainers (apache#6457) (cherry picked from commit 129cdc1)
…tainers (apache#6457) (cherry picked from commit 129cdc1)
What changes were proposed in this pull request?
HDDS-10612. Add Robot test to verify Container Balancer for RATIS containers
Currently there are only unit tests for Container Balancer and no acceptance tests at all. At a minimum, we should add a Robot test to verify Container Balancer for RATIS containers. And probably in the future we should add robot test for EC case.
Test case:
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10612
How was this patch tested?
Added Robot test