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.

Sign up for GitHub

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

Jump to bottom

Avoid materializing default namespace #328

Merged
merged 1 commit into from
Dec 2, 2019
Merged

Avoid materializing default namespace #328

merged 1 commit into from
Dec 2, 2019

Conversation

mkmik
Copy link
Collaborator

@mkmik mkmik commented Dec 2, 2019

Closes #323

@mkmik mkmik requested a review from atomatt December 2, 2019 11:03
@mkmik mkmik mentioned this pull request Dec 2, 2019
Closed
@mkmik
Copy link
Collaborator Author

mkmik commented Dec 2, 2019

bors r+

bors bot added a commit that referenced this pull request Dec 2, 2019
@bors
Merge #328
ad6e649 
328: Avoid materializing default namespace r=mkmik a=mkmik

Closes #323

Co-authored-by: Marko Mikulicic <mkm@bitnami.com>
@bors
Copy link
Contributor

bors bot commented Dec 2, 2019

Build succeeded

@bors bors bot merged commit 68ff559 into master Dec 2, 2019
@bors bors bot deleted the noimplicitnamespace2 branch December 2, 2019 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atomatt atomatt atomatt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Should not add namespace when sealing "cluster-wide" secret
2 participants
@mkmik @atomatt