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

helm-operator: does not match namespace #1293

Closed
endrec opened this issue Aug 17, 2018 · 4 comments
Closed

helm-operator: does not match namespace #1293

endrec opened this issue Aug 17, 2018 · 4 comments
Labels
bug helm UX In pursuit of a delightful user experience

Comments

@endrec
Copy link
Contributor

endrec commented Aug 17, 2018

In some charts (e.g. coreos/kube-prometheus) a few resource is created in the "kube-system" namespace, and not in the release namespace.
helm-operatot logs this error:

ts=2018-08-17T12:56:44.566927601Z caller=release.go:296 component=release output="secret \"alertmanager-monitoring\" annotated\n[...]servicemonitor \"monitoring-prometheus\" annotated\nthe namespace from the provided object \"kube-system\" does not match the namespace \"monitoring\". You must pass '--namespace=kube-system' to perform this operation.\nthe namespace from the provided object \"kube-system\" does not match the namespace \"monitoring\". You must pass '--namespace=kube-system' to perform this operation.\nthe namespace from the provided object \"kube-system\" does not match the namespace \"monitoring\". You must pass '--namespace=kube-system' to perform this operation.\n" err="exit status 1"

Despite of this error, the release seems working fine.

@ahmadiq
Copy link

ahmadiq commented Aug 17, 2018

+1 We're facing this issue as well.

@squaremo
Copy link
Member

That looks like the annotator -- which is a kind of secondary system that runs around marking all the things that were created by a release -- is the bit that's failing. It won't affect the release itself, but it's noisy, and may affect whether other systems (e.g., Weave Cloud) get the right idea about the resources.

The fix would be to make sure the kubectl annotate invocation always uses the namespace given by the resource.

@dholbach dholbach added the UX In pursuit of a delightful user experience label Oct 4, 2018
@samisq
Copy link

samisq commented Dec 11, 2018

+1 We're facing this exact issue too. Can this be safely ignored (for now)?

@nmaupu
Copy link

nmaupu commented Dec 18, 2018

Same issue here. We are using prometheus-operator and our charts create a servicemonitor object in the same namespace as the prometheus-operator itself along with the resources needed by the chart's release.
I confirm, everything is created as expected but the error is not very clear as this is the expected behavior and should be safely ignored or not...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug helm UX In pursuit of a delightful user experience
Projects
None yet
Development

No branches or pull requests

7 participants