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.

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

Roles are in need of annotations #11268

Closed
benjaminapetersen opened this issue Oct 7, 2016 · 2 comments
Closed

Roles are in need of annotations #11268

benjaminapetersen opened this issue Oct 7, 2016 · 2 comments

Comments

@benjaminapetersen
Copy link
Contributor

benjaminapetersen commented Oct 7, 2016

Per discussion w/@deads @enj @liggitt @jwforres
Re this PR in the web console and this Trello card.
Roles are in need of the following annotations:

authorization.openshift.io/infrastructure 
openshift.io/description
openshift.io/display-name

The filter we initially said was this:

      return _.filter(roles, function(item) {
        // image-puller & image-pusher ok, other system: no
        return (item.metadata.name === 'system:image-puller' ||
               item.metadata.name === 'system:image-pusher') ||
               ! _.contains(item.metadata.name, 'cluster-') &&
               ! _.contains(item.metadata.name, 'system:');
      });

and the roles that passed the test:

edit, registry-admin, registry-editor, system:image-pusher, self-access-reviewer, sudoer, system:image-puller, admin, registry-viewer, self-provisioner, basic-user, view
@benjaminapetersen
Copy link
Contributor Author

Gonna keep the discussion about the annotation going here so those who don't want to get notified about the UI stuff can ignore.

@jwforres pointed out roles like image puller are infrastructure, but still need to be in our filtered list.

Brain dump of other possibilities:

authorization.openshift.io/user-friendly
authorization.openshift.io/primary
authorization.openshift.io/fundamental
authorization.openshift.io/end-user
authorization.openshift.io/end-usable
authorization.openshift.io/end-user-should-know-about-maybe
authorization.openshift.io/voodoo

I'm leaning towards something like user-friendly at this point, it seems to fit intention of the category we are trying to create. Its not a hard line, more of a helpful suggestion.

@benjaminapetersen
Copy link
Contributor Author

Perhaps authorization.openshift.io/system-only (exclusive)

  • new roles will show up automatically
  • if a new role is created & the desire is to hide it, the annotation must be added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants