Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Upgrade Zookeeper Operator dependency #271

Closed
HoustonPutman opened this issue May 20, 2021 · 3 comments · Fixed by #304
Closed

Upgrade Zookeeper Operator dependency #271

HoustonPutman opened this issue May 20, 2021 · 3 comments · Fixed by #304
Labels
dependency upgrades Upgrading dependencies of the Operator, such as Kubernetes, Kubebuilder, Zookeeper-Operator, etc. zookeeper Related to Zookeeper or the Zookeeper Operator
Milestone
v0.4.0

Comments

@HoustonPutman
Copy link
Contributor

The Zookeeper Operator v0.2.10 has been released: https://github.com/pravega/zookeeper-operator/releases/tag/v0.2.10

We should upgrade our dependency, and add support for new options.
@HoustonPutman HoustonPutman added the zookeeper Related to Zookeeper or the Zookeeper Operator label May 20, 2021
@HoustonPutman HoustonPutman added this to the main (v0.4.0) milestone May 20, 2021
@HoustonPutman
Copy link
Contributor Author

v0.2.11 is now available.

@HoustonPutman HoustonPutman changed the title Upgrade Zookeeper Operator dependency to v0.2.10 Upgrade Zookeeper Operator dependency to v0.2.11 Jul 14, 2021
@HoustonPutman HoustonPutman changed the title Upgrade Zookeeper Operator dependency to v0.2.11 Upgrade Zookeeper Operator dependency Jul 14, 2021
@HoustonPutman
Copy link
Contributor Author

v0.2.12 is now out and very recent, so we should target that. Looks like mostly bug fixes and a few additional config options.

@HoustonPutman
Copy link
Contributor Author

Opened a ticket related to the upgrade, pravega/zookeeper-operator#378.

Because we can use an empty value for the new spec.Probes ZK field, we will have to do zkCluster.WithDefaults() before submitting creating/updating/etc the object. This very much ties the Solr Operator down, making it essential that the version of the Zookeeper Operator that users are running is the same as the version the Solr Operator has as a dependency.

@HoustonPutman HoustonPutman mentioned this issue Aug 11, 2021
Merged
@HoustonPutman HoustonPutman added the dependency upgrades Upgrading dependencies of the Operator, such as Kubernetes, Kubebuilder, Zookeeper-Operator, etc. label Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dependency upgrades Upgrading dependencies of the Operator, such as Kubernetes, Kubebuilder, Zookeeper-Operator, etc. zookeeper Related to Zookeeper or the Zookeeper Operator
Projects
None yet
Milestone
v0.4.0
Development

Successfully merging a pull request may close this issue.

Upgrade Zookeeper Operator dependency to v0.2.12 HoustonPutman/solr-operator
1 participant
@HoustonPutman