stackabletech / zookeeper-operator

A tool that can be used to deploy and manager Apache ZooKeeper clusters/ensembles
Other
26 stars 8 forks source link

Fix the ZNode test #825

Closed siegfriedweber closed 2 months ago

siegfriedweber commented 2 months ago

Description

The numbers of the test steps were renamed in #802. The change in #799 relied on the old test step numbers. The ZNode test failed after the merge:

    logger.go:42: 07:38:10 | znode_zookeeper-latest-3.9.2_openshift-false/1-set-znode-override | starting test step 1-set-znode-override
    logger.go:42: 07:38:10 | znode_zookeeper-latest-3.9.2_openshift-false/1-set-znode-override | running command: [sh -c kubectl get zookeeperznode/test-znode -o json -n $NAMESPACE | jq '.status.znodePath = "/znode-override"' | kubectl replace -f- --subresource=status]
    logger.go:42: 07:38:11 | znode_zookeeper-latest-3.9.2_openshift-false/1-set-znode-override | Error from server (NotFound): zookeeperznodes.zookeeper.stackable.tech "test-znode" not found
    logger.go:42: 07:43:12 | znode_zookeeper-latest-3.9.2_openshift-false/1-set-znode-override | test step failed 1-set-znode-override

This pull request fixes the order of the test steps.

Definition of Done Checklist

# Author
# Reviewer
- [ ] Code contains useful comments
- [ ] Code contains useful logging statements
- [ ] (Integration-)Test cases added
- [ ] Documentation added or updated. Follows the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs-style-guide).
- [ ] Changelog updated
- [ ] Cargo.toml only contains references to git tags (not specific commits or branches)
# Acceptance
- [ ] Feature Tracker has been updated
- [ ] Proper release label has been added
- [ ] [Roadmap](https://github.com/orgs/stackabletech/projects/25/views/1) has been updated