-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Introduce high privilege namespace (administrative namespace) #21215
Conversation
@@ -0,0 +1,4 @@ | |||
```release-note:change | |||
core/namespace (enterprise): Introduce the concept of high-privilege namespace (administrative 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.
Just wondering, as the code change universally uses the term "administrative namespace", maybe it would be clearer to stick with that terminology throughout rather than introducing "high-privilege namespace" as a synonym as well?
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.
LGTM
d2c23b5
to
ed9bd0d
Compare
8dfbfbd
to
e5cea2c
Compare
f151e24
to
dfb0d44
Compare
dfb0d44
to
6207e1c
Compare
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.
Looks good!
Full functionality only works in enterprise. This adds the core implementation pieces needed.