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

Update community-operator chart to support having a namespace override #277

Closed

Conversation

bbergquist0930
Copy link

Closes #276

We are building a top level "solution" helm chart that has several subchart dependencies, the community-operator helm chart being one. We want to have the subchart be installed within a specific namespace that we can specify in the top level solution chart.

Currently the community-operator helm chart has its namespace set to the Release namespace. This pull request updates the templates such that an "operator.namespaceOverride" value can be set which then installs the chart into that namespace. The proper namespaces for the database and service accounts are also configured if that value is set unless they are explicity set to something else such as the service account database namespace.

This has been tested as

  • a standalone installation of the chart with no namespace override
  • a standalone installation of the chart with operator.namespaceOverride set
  • a subchart with no namespace override
  • a subchart with operator.namespaceOverride set

When the operator.namespaceOverride is not set, there are no changes to the existing functionality.

All Submissions:

  • Have you opened an Issue before filing this PR?
  • Have you signed our CLA?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

This adds a conditional template for the namespace specified as the
operator override namespace.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Suppport namespace override in community-operator
1 participant