Closed AKamyshnikova closed 9 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
28d1f69
) 85.12% compared to head (1159ce0
) 85.12%. Report is 3 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Run e2e tests locally and they passed
• [SLOW TEST:585.533 seconds]
Perform rolling restart on zk cluster
/root/zookeeper-operator/test/e2e/rolling_restart_test.go:20
Check rolling restart operation
/root/zookeeper-operator/test/e2e/rolling_restart_test.go:21
should perform rolling restart
/root/zookeeper-operator/test/e2e/rolling_restart_test.go:22
------------------------------
STEP: tearing down the test environment
Ran 8 of 8 Specs in 3469.747 seconds
SUCCESS! -- 8 Passed | 0 Failed | 0 Pending | 0 Skipped
--- PASS: TestAPIs (3469.75s)
PASS
ok github.com/pravega/zookeeper-operator/test/e2e 3469.779s
make undeploy
Run e2e tests locally and they passed
• [SLOW TEST:585.533 seconds] Perform rolling restart on zk cluster /root/zookeeper-operator/test/e2e/rolling_restart_test.go:20 Check rolling restart operation /root/zookeeper-operator/test/e2e/rolling_restart_test.go:21 should perform rolling restart /root/zookeeper-operator/test/e2e/rolling_restart_test.go:22 ------------------------------ STEP: tearing down the test environment Ran 8 of 8 Specs in 3469.747 seconds SUCCESS! -- 8 Passed | 0 Failed | 0 Pending | 0 Skipped --- PASS: TestAPIs (3469.75s) PASS ok github.com/pravega/zookeeper-operator/test/e2e 3469.779s make undeploy
@AKamyshnikova could you please check if there is change in CRD file generated? If there is change please check in the CRD file too
@anishakj Thanks for pointing this! Updated CRDs
@AKamyshnikova Could you please resolve conflicts?
Signed-off-by: Ann Taraday akamyshnikova@mirantis.com
Change-Id: Ia673486d8ebb3c76e86d1cc3af846cd890b4ffaa
Change log description
Updated usage for fake client - now required WithStatusSubresource(..)
Purpose of the change
Fixes: #567
What the code does
(Detailed description of the code changes)
How to verify it
All e2e tests should pass