stackabletech / zookeeper-operator

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

docs: Document how to turn off quorum hostname verification #818

Closed sbernauer closed 2 months ago

sbernauer commented 3 months ago

Description

Please add a description here. This will become the commit message of the merge request later.

Definition of Done Checklist

# Author
- [ ] Changes are OpenShift compatible
- [ ] CRD changes approved
- [ ] CRD documentation for all fields, following the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs-style-guide).
- [ ] Helm chart can be installed and deployed operator works
- [ ] Integration tests passed (for non trivial changes)
- [ ] Changes need to be "offline" compatible
# 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
sbernauer commented 3 months ago

I just believe this should go into some kind of how to or troubleshooting section.

100% agreed. I didn't find where to locate it best :/

sbernauer commented 3 months ago

It's especially hard as I don't have an error message or similar. Guess I need to search the customer issue

sbernauer commented 3 months ago

I added a troubleshooting guide, happy about @fhennig comments

lfrancke commented 3 months ago

Thank you. I think this can wait for Felix to come back from vacation. Let's park it until next week.

fhennig commented 3 months ago

Good stuff! I like it. I think it's good where it is placed. Just resolve the discussion about the warning, I'd also like it to be more specific.

sbernauer commented 3 months ago

Moving to Track because

Preliminarily I'd rather just solve https://github.com/stackabletech/zookeeper-operator/issues/760 than document the workaround, unless we're running out of time for the release.

nightkr commented 2 months ago

So for 24.7 our only green version will be 3.9.2 where this issue is already "solved" by FIPS mode being enabled by default (see https://github.com/stackabletech/zookeeper-operator/issues/760#issuecomment-2178007599). I would propose closing this PR, or changing it to only mention enabling FIPS mode on 3.8.4.

sbernauer commented 2 months ago

Works for me