-
Notifications
You must be signed in to change notification settings - Fork 356
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 RBAC and Tagging Support to Ansible Repositories #3513
Add RBAC and Tagging Support to Ansible Repositories #3513
Conversation
@miq-bot add_label automation/ansible, blocker, enhancement, gaprindashvili/yes, pending core |
4865240
to
b703a15
Compare
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1526217 Add Policy toolbar button to enable tagging in Automation > Ansible > Repositories.
Add also tag_edit_form_field_changed for proper rendering of tagging screen.
Add calling tag method to button method in ansible repository controller and also small refactoring of button method, for tagging support of Ansible Repositories.
Add displaying tags in the table in summary screen of a chosen repository, in Automation > Ansible > Repositories.
b703a15
to
f0c82db
Compare
Add Policy toolbar button to enable tagging of any Ansible Repository from its summary page.
Checked commits hstastna/manageiq-ui-classic@8fce5f1~...f7ca15a with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@h-kataria @dclarizio @martinpovolny Adding RBAC and Tagging Support to Ansible Repos done so now I would need some review. Thanks so much! :) |
looks good, ready to merge once Travis is green. |
@himdel can you look at Javascript test failures. |
Add RBAC and Tagging Support to Ansible Repositories (cherry picked from commit 4ee1690) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1553396
Gaprindashvili backport details:
|
@hstastna Travis is failing in gaprindashvili branch. Please take a look.
|
@simaishi If is failing because I've written spec tests for master branch, for the newest code and also for the parts of the code I haven't written. The older code is different, it looks like it still uses compressed ids which were, I think, removed from the newest code. And that's not related to this PR. Anyway, I will try to fix that failing spec test for gaprindashvili branch |
Yes, I realized that code has changed between master/gaprindashvili. But since that failing test was added in this PR, I've added the info here 😄 Thanks for fixing! |
Caused by backporting ManageIQ#3513
@miq-bot add_label test |
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1526217
Needs to merge with: [SOLVED/merged]
ManageIQ/manageiq#17083 (tagging support to main MIQ repo)
ManageIQ/manageiq#17091 (RBAC support)
Done:
(Policy > Edit Tags)
tagging_edit
andtag_edit_form_field_changed
to routes to enable tagging for Ansible Repos or for proper rendering of tagging screentag
method tobutton
method in ansible credential controller and also small refactoring ofbutton
method, in that controllerbutton
method in ansible repository controllerBefore:
![repos_before](https://user-images.githubusercontent.com/13417815/36999873-ce8ab174-20c1-11e8-8e46-40eef4d1377c.png)
![repository_summary_no_tag](https://user-images.githubusercontent.com/13417815/37093517-e2cf4326-220f-11e8-9efc-e03731df3c30.png)
After:
![repos_after](https://user-images.githubusercontent.com/13417815/36999363-2cdda198-20c0-11e8-9e6c-9db8a8fdbf71.png)
![repos_tag_page](https://user-images.githubusercontent.com/13417815/36999796-8ae307aa-20c1-11e8-9e62-c56740b242af.png)
![tag_success](https://user-images.githubusercontent.com/13417815/36999810-916039d6-20c1-11e8-9661-d0c7638a73eb.png)
![repository_summary](https://user-images.githubusercontent.com/13417815/37028761-7aa562e2-2135-11e8-87ad-28ef31f6c621.png)
Note:
How to check RBAC support (one of possible scenarios):
remember those tags