stackabletech / druid-operator

An Operator for Apache Druid for Stackable Data Platform
Other
10 stars 0 forks source link

fix: JVM truststore & keystore location #564

Closed labrenbe closed 6 months ago

labrenbe commented 6 months ago

Description

The TLS secret containing truststore and keystore is mounted at /stackable/tls which is already used in Druid-specific properties (e.g. druid.auth.basic.ssl.trustStorePath). The pac4J extension uses the JVM config to configure the trust store.

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
- [x] Code contains useful comments
- [x] 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
- [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
sbernauer commented 6 months ago

I have to admit I don't understand the description, could you please try to make it a bit more clear what exactly the bug is? In any case please also add a changelog entry

sbernauer commented 6 months ago

Closing because there are multiple different keystores, the current implementation is correct