Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/prometheus-operator] Ability to configure namespace #18837

Closed
arruzk opened this issue Nov 13, 2019 · 7 comments · Fixed by #22288
Closed

[stable/prometheus-operator] Ability to configure namespace #18837

arruzk opened this issue Nov 13, 2019 · 7 comments · Fixed by #22288
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.

Comments

@arruzk
Copy link
Contributor

arruzk commented Nov 13, 2019

I would like to create a chart which will depend on prometheus-operator and I want to install prometheus in a separate namespace.

dependencies:
  - name: prometheus-operator
    version: "7.2.0"
    repository: "@stable"

Currently it's not possible to specify different namespace for prometheus (namespace: {{ $.Release.Namespace }}).

Is it a better way than fork the chart and change it to something like namespace: {{ .Values.global.prometheusNamespace }}?

@vsliouniaev
Copy link
Collaborator

Look at the helm --namespace parameter

@arruzk
Copy link
Contributor Author

arruzk commented Nov 13, 2019

@vsliouniaev namespace from this parameter is used for deploying the main chart. I would like to install prometheus operator to the separate one.

@vsliouniaev
Copy link
Collaborator

You want to put portions of the chart into one namespace and other parts into another namespace? I don't really understand the reasoning for doing this

@arruzk
Copy link
Contributor Author

arruzk commented Nov 15, 2019

@vsliouniaev the idea is not to mix "service" pods\services\secrets (eg from prometheus operator) with my own applications.
I can install prometheus and my own applications as a different helm releases, but one of my application depends on prometheus. For this reason I included it as a dependency, but it's not very convenient to have this mix.

Here is an example of similar problem and solution #15202

@vsliouniaev
Copy link
Collaborator

Thanks for the explanation! Feel free to submit a PR into this chart to get this functionality.

@stale
Copy link

stale bot commented Dec 15, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

@stale stale bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 15, 2019
@stale
Copy link

stale bot commented Dec 29, 2019

This issue is being automatically closed due to inactivity.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants