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

Helm install should allow WATCH_NAMESPACE to be passed in #150

Closed
anishakj opened this issue Mar 26, 2020 · 0 comments · Fixed by #154
Closed

Helm install should allow WATCH_NAMESPACE to be passed in #150

anishakj opened this issue Mar 26, 2020 · 0 comments · Fixed by #154
Assignees
Labels
area/chart Issue with helm chart Pravega 0.8.0 Fix by pravega 0.8.0 Priority-P3

Comments

@anishakj
Copy link
Contributor

anishakj commented Mar 26, 2020

Description
Currently the Helm chart for the zookeeper Operator will ONLY watch for resources in the same namespace that the operator is installed:

    env:
    - name: WATCH_NAMESPACE
      valueFrom:
        fieldRef:
          fieldPath: metadata.namespace

https://github.com/pravega/zookeeper-operator/blob/master/charts/zookeeper-operator/templates/operator.yaml#L27-30

The helm chart should provide a way of overriding this so that the operator can watch ALL namespaces (by passing in "") or another namespace different to the one the operator is installed in.

Location
https://github.com/pravega/zookeeper-operator/blob/master/charts/zookeeper-operator/templates/operator.yaml#L27-30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/chart Issue with helm chart Pravega 0.8.0 Fix by pravega 0.8.0 Priority-P3
Projects
None yet
2 participants