-
Notifications
You must be signed in to change notification settings - Fork 226
doc(self-managed): clarify usage of keycloak images for bitnamilegacy… #7474
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.
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR clarifies the documentation for Keycloak images used in Camunda's air-gapped installations, addressing confusion about multiple image sources (bitnamilegacy vs vendor-ee).
Key changes:
- Updated Keycloak image references to use
camunda/keycloak(open-source) andregistry.camunda.cloud/keycloak-ee/keycloak(enterprise) - Added comprehensive documentation section explaining Camunda's custom Keycloak images, including variants, registries, tag prefixes, and availability
- Included notes about backward compatibility and usage recommendations
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
versioned_docs/version-8.8/self-managed/deployment/helm/configure/registry-and-images/air-gapped-installation.md |
Updates Keycloak image references and adds detailed documentation section for version 8.8 |
docs/self-managed/deployment/helm/configure/registry-and-images/air-gapped-installation.md |
Identical changes applied to current documentation version for consistency |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@giorgionaps please take a look at this for @leiicamundi as a priority 🙏 |
marcel-dias
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
...on-8.8/self-managed/deployment/helm/configure/registry-and-images/air-gapped-installation.md
Outdated
Show resolved
Hide resolved
giorgionaps
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor comment from me
198113d
|
@giorgionaps I've fixed the comment and enabled-auto merge, could you re-approve? Thank you |
giorgionaps
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved on my side
|
🧹 Preview environment for this PR has been torn down. |
Description
This PR is related to the question https://camunda.slack.com/archives/C03UR0V2R2M/p1764857316964609
It improves the description of the keycloak images as those have multiple sources legacy / vendor-ee
When should this change go live?
bugorsupportlabel)available & undocumentedlabel)holdlabel)low priolabel)PR Checklist
{type}(scope): {description}commit message(s)/docsdirectory (version 8.9)./versioned_docsdirectory.@camunda/tech-writersunless working with an embedded writer.