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

Remove the namespace object or rename it victoriametrics #279

Open
stribb opened this issue Jul 12, 2021 · 2 comments
Open

Remove the namespace object or rename it victoriametrics #279

stribb opened this issue Jul 12, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@stribb
Copy link

stribb commented Jul 12, 2021

While it's convenient to create a namespace for the Victoriametrics operator, it has a massive downside in the case where the operator shares a namespace with other services.

In my setup, I (naturally) put Victoriametrics into the same NS as the operator, then went about adding several other services. This went fine until I had to delete Operator to re-create it. The most obvious way for me to delete it seemed that I should run kubectl delete -k kustomization.yaml. But, of course, since it also has the monitoring-system namespace, it deleted everything inside the namespace too.

My suggestion would be to remove the namespace object -- it's the safer way.

@f41gh7
Copy link
Collaborator

f41gh7 commented Jul 16, 2021

It make sense for me. May i ask to provide some example of usage or submit PR with needed changes?

@f41gh7 f41gh7 added the enhancement New feature or request label Jul 16, 2021
@flokli
Copy link
Contributor

flokli commented Oct 20, 2021

This only seems to be an issue with the kustomize bits in the repo. When deploying via the helm chart, there's no monitoring-system namespace created or deployed to at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants