-
Notifications
You must be signed in to change notification settings - Fork 88
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
Additional namespaces flag #372
Conversation
57836a7
to
1009c9e
Compare
pkg/kotsadm/namespaces.go
Outdated
|
||
application := obj.(*kotsv1beta1.Application) | ||
|
||
if application.Spec.AdditionalNamespaces == nil { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't need this. loop works with nil arrays
} | ||
|
||
for _, additionalNamespace := range application.Spec.AdditionalNamespaces { | ||
// We support "*" for additional namespaces to handle pullsecret propagation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"We don't
support", right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do, but we don't create that. It (will be) handled in the kotsadm code to handle imagePulLSecret propagation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this then break on something like this:
- custom_ns
- worker*
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unsupported. only "*" and/or specific (exact) namespaces
No description provided.