-
Notifications
You must be signed in to change notification settings - Fork 12
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
fix: Rename Namespace MR to VaultNamespace #35
Conversation
Signed-off-by: Jordan Levin <[email protected]>
Signed-off-by: Jordan Levin <[email protected]>
…der-vault into jrl/fix-namespace
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.
Thanks @jaylevin for the PR. Left a few comments.
Signed-off-by: Jordan Levin <[email protected]>
0a3caca
to
4a792b7
Compare
Signed-off-by: Jordan Levin <[email protected]>
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.
Thanks @jaylevin LGTM!
Thanks @sergenyalcin! It looks like I don't have write permission to merge this pull request. Is it possible for you to merge this the next time you can get a chance? |
Description of your changes
Namespace
resource toVaultNamespace
in order to resolve naming conflict with the K8s nativeNamespace
resource.Fixes #12 (and the duplicate #31)
I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested
Tested in a kubernetes cluster on Crossplane v1.15.1 and kubernetes v1.26