stackabletech / zookeeper-operator

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

Allow overriding ZNode path #799

Closed nightkr closed 2 months ago

nightkr commented 5 months ago

Description

Fixes https://github.com/stackabletech/zookeeper-operator/issues/681

Definition of Done Checklist

# Author
- [x] Changes are OpenShift compatible
- [x] CRD changes approved
- [x] CRD documentation for all fields, following the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs-style-guide).
- [x] Helm chart can be installed and deployed operator works
- [x] Integration tests passed (for non trivial changes)
- [x] Changes need to be "offline" compatible
# Reviewer
- [x] Code contains useful comments
- [x] Code contains useful logging statements
- [x] (Integration-)Test cases added
- [x] Documentation added or updated. Follows the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs-style-guide).
- [x] Changelog updated
- [x] 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
nightkr commented 5 months ago

Some integration tests are currently failing due to a missing spec.image, not sure if it's related to https://stackable-workspace.slack.com/archives/C02M1RE8S0Z/p1712274080585689.

nightkr commented 4 months ago

Turns out the test failures were due to stale beku work files. Clearing tests/_work made them pass.