Skip to content

Ensure default service account gets namespaces role#67

Merged
mnasiadka merged 1 commit intomasterfrom
fix-kube-nodepool
Apr 17, 2026
Merged

Ensure default service account gets namespaces role#67
mnasiadka merged 1 commit intomasterfrom
fix-kube-nodepool

Conversation

@Alex-Welsh
Copy link
Copy Markdown
Member

Error from kubernetes nodepool launcher log:

"Failure", "message":"namespaces is forbidden: User \"system:serviceaccount:zuul:default\" cannot list resource \"namespaces\" in API group \"\" at the cluster scope"

Error from kubernetes nodepool launcher log:
"Failure", "message":"namespaces is forbidden: User
\"system:serviceaccount:zuul:default\" cannot list resource
\"namespaces\" in API group \"\" at the cluster scope"
@Alex-Welsh Alex-Welsh requested a review from mnasiadka April 17, 2026 13:40
@mnasiadka
Copy link
Copy Markdown
Member

Hmm, why are we trying to deploy anything in the default namespace? Shouldn't that be configured to use some dedicated namespace?

@Alex-Welsh
Copy link
Copy Markdown
Member Author

Hmm, why are we trying to deploy anything in the default namespace? Shouldn't that be configured to use some dedicated namespace?

https://zuul-ci.org/docs/nodepool/latest/kubernetes.html#attr-providers.[kubernetes].pools.name
Not sure if i'm reading this right, but it sounds like it's creating a new namespace for each pool, which would explain why it's trying to list namespaces

@mnasiadka
Copy link
Copy Markdown
Member

Hmm, why are we trying to deploy anything in the default namespace? Shouldn't that be configured to use some dedicated namespace?

https://zuul-ci.org/docs/nodepool/latest/kubernetes.html#attr-providers.[kubernetes].pools.name Not sure if i'm reading this right, but it sounds like it's creating a new namespace for each pool, which would explain why it's trying to list namespaces

Fine by me - we should move to zuul-launcher soon (that means when I get time to add support for zuul-launcher to zuul-operator) - so that's fine for now.

@mnasiadka mnasiadka merged commit 9134f93 into master Apr 17, 2026
1 check passed
@mnasiadka mnasiadka deleted the fix-kube-nodepool branch April 24, 2026 10:48
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

Successfully merging this pull request may close these issues.

2 participants