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

Add AuthKeyPair to RBAC #15359

Merged
merged 2 commits into from
Jun 14, 2017
Merged

Conversation

lpichler
Copy link
Contributor

AuthKeyPair is inherited from AuthPrivateKey and then from Authentication. (In CLASSES_THAT_PARTICIPATE_IN_RBAC is list of base classes)

In UI:
Compute -> Cloud -> Key Pairs

@miq-bot add_label bug, rbac

@miq-bot assign @gtanzillo

Links

https://bugzilla.redhat.com/show_bug.cgi?id=1441637

this model is inherited from AuthPrivateKey and then from
Authentication. (In CLASSES_THAT_PARTICIPATE_IN_RBAC is list of base classes)
@miq-bot
Copy link
Member

miq-bot commented Jun 13, 2017

Checked commits lpichler/manageiq@deb653a~...184297d with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@djberg96
Copy link
Contributor

djberg96 commented Jun 13, 2017

The whole CLASSES_THAT_PARTICIPATE_IN_RBAC approach feels weird to me. Wouldn't an inherited singleton.rbac? method, which subclasses could then override, be easier to hook into?

(Note that this comment may or may not be pertinent to the PR at hand. Just thought I would poke my nose in here)

Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM

@gtanzillo gtanzillo added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 14, 2017
@gtanzillo gtanzillo merged commit 4e2e528 into ManageIQ:master Jun 14, 2017
@lpichler lpichler deleted the add_auth_key_pair_rbac branch June 15, 2017 06:30
@lpichler
Copy link
Contributor Author

@djberg96 you are right it is weird, I'll think about it. Maybe there is 'technical reason ' why to model should participate to rbac (that it is using acts_as_miq_taggable)

thank you for your comment 👍

@simaishi
Copy link
Contributor

Backported to Fine via #16865

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

Successfully merging this pull request may close these issues.

5 participants