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.

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

Allow further config of Management Center #386

Closed
wants to merge 3 commits into from

Conversation

plytro
Copy link

@plytro plytro commented Sep 17, 2023

These charts are nearly functionally identical to each other. This change fixes the formatting between the two so files can be compared more easily to see differences between them. Additionally, settings exist in the values.yaml on the hazelcast chart that were not consumed by the templates, like the license key. This corrects for that. The only functional difference between the two now should be the support for hazelcast persistence.

This also further adds init containers for mancetner and the option to configure Active Directory auth in backwards compatible way.

@devOpsHazelcast
Copy link

devOpsHazelcast commented Sep 17, 2023

CLA assistant check
All committers have signed the CLA.

@semihbkgr semihbkgr self-requested a review September 18, 2023 09:37
@plytro plytro changed the title Reconcile diffs between non-enterprise and enterprise charts Allow further config of Management Center Sep 18, 2023
@plytro
Copy link
Author

plytro commented Sep 22, 2023

@devOpsHazelcast any chance we can see this through to merge soon?

@devOpsHazelcast
Copy link

PR closed by Hazelcast automation as no activity (>3 months). Please reopen with comments, if necessary. Thank you for using Hazelcast and your valuable contributions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants