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

Namespace cannot be overwritten with helm template #152

Closed
rgruchalski-klarrio opened this issue May 5, 2022 · 3 comments
Closed

Namespace cannot be overwritten with helm template #152

rgruchalski-klarrio opened this issue May 5, 2022 · 3 comments

Comments

@rgruchalski-klarrio
Copy link

rgruchalski-klarrio commented May 5, 2022

It would be awesome if the chart allowed overwriting the namespace of the:

  • deployment.apps/cert-manager-istio-csr
  • service/cert-manager-istio-csr
  • service/cert-manager-istio-csr-metrics
  • the istio-csr pod (replica set)
  • service account

Currently, when doing:

helm template --name-template=cert-manager-istio-csr \
    --namespace cert-manager \
    --set "app.tls.rootCAFile=/var/run/secrets/istio-csr/ca.pem" ...

into a kustomization, and kubectl apply -k of the resulting yaml, those resources end up in the default namespace.

kubectl get all -n default
NAME                                          READY   STATUS              RESTARTS   AGE
pod/cert-manager-istio-csr-5bc67d8f65-wt85l   0/1     ContainerCreating   0          16m

NAME                                     TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)    AGE
service/cert-manager-istio-csr           ClusterIP   10.96.58.72     <none>        443/TCP    16m
service/cert-manager-istio-csr-metrics   ClusterIP   10.96.139.220   <none>        9402/TCP   16m
service/kubernetes                       ClusterIP   10.96.0.1       <none>        443/TCP    17m

NAME                                     READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/cert-manager-istio-csr   0/1     1            0           16m

NAME                                                DESIRED   CURRENT   READY   AGE
replicaset.apps/cert-manager-istio-csr-5bc67d8f65   1         1         0       16m

Preferably the chart would define a way to set the namespace at the template time.

@caleygoff-invitae
Copy link

yes please. we install these components into a namespace called cert-manager.

@leotomas837
Copy link

Any news on this ? It has been 7 months...

knutejohKLP added a commit to knutejohKLP/istio-csr that referenced this issue Dec 14, 2022
…-manager#152. Signed-off-by: Knut-Erik Johnsen <knut-erik.johnsen@klp.no>
jetstack-bot added a commit that referenced this issue Feb 17, 2023
Add support for setting namespace for istio control plane. Fixes #152
knutejohKLP added a commit to knutejohKLP/istio-csr that referenced this issue Feb 19, 2023
…-manager#152. Signed-off-by: Knut-Erik Johnsen <knut-erik.johnsen@klp.no>

Signed-off-by: Knut-Erik Johnsen <knut-erik.johnsen@klp.no>
@rgruchalski-klarrio
Copy link
Author

👍🏻

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

No branches or pull requests

3 participants