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

Script to copy reports access from group to role #18066

Merged

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Oct 5, 2018

Requested functionality:
Provide utility to duplicate customized report menu from some group to another group or role

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

Usage:
# help
bundle exec ruby ./tools/copy_reports_structure.rb --help
# copy reports menu from EvmGroup to SomeGroup
bundle exec ruby ./tools/copy_reports_structure.rb --source-group=EvmGroup --target-group=SomeGroup
# copy reports menu from EvmGroup to every group which has SomeRole
bundle exec ruby ./tools/copy_reports_structure.rb --source-group=EvmGroup --target-role=SomeRole
# reset to default reports menu for SomeGroup
bundle exec ruby ./tools/copy_reports_structure.rb --reset-group=SomeGroup

@miq-bot add-label enhancement

@yrudman yrudman force-pushed the script-to-copy-report-for-ll-groups-in-role branch from ec0a3df to 99afe76 Compare October 5, 2018 20:58
@miq-bot

This comment has been minimized.

@yrudman yrudman force-pushed the script-to-copy-report-for-ll-groups-in-role branch from 99afe76 to 47e664f Compare October 8, 2018 15:34
@yrudman yrudman changed the title [WIP] Script to copy reports access from group to role Script to copy reports access from group to role Oct 8, 2018
@yrudman
Copy link
Contributor Author

yrudman commented Oct 8, 2018

@miq-bot assign @gtanzillo

@yrudman
Copy link
Contributor Author

yrudman commented Oct 8, 2018

@miq-bot remove-label wip

@miq-bot miq-bot removed the wip label Oct 8, 2018
  - duplicate report structure from one group to another group
  - duplicate report structure from group to role, which means to each group having that role
  - remove cutomized report sructure from group
  - remove customized report structure from role, which means from each group having that role
EXAMPLE: bundle exec ruby ./tools/copy_reports_structure -s EvmGroup  -r SomeRole
         bundle exec ruby ./tools/copy_reports_structure -p SomeRole
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1634673
@yrudman yrudman force-pushed the script-to-copy-report-for-ll-groups-in-role branch from 47e664f to dc2495f Compare October 9, 2018 19:22
tools/copy_reports_structure.rb Outdated Show resolved Hide resolved
tools/copy_reports_structure.rb Outdated Show resolved Hide resolved
@yrudman yrudman force-pushed the script-to-copy-report-for-ll-groups-in-role branch from f0b0b49 to c675c6a Compare October 11, 2018 19:10
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.

Looks good! 👍

@gtanzillo gtanzillo added this to the Sprint 97 Ending Oct 22, 2018 milestone Oct 11, 2018
@gtanzillo gtanzillo merged commit a8f357d into ManageIQ:master Oct 11, 2018
@yrudman yrudman deleted the script-to-copy-report-for-ll-groups-in-role branch October 11, 2018 19:42
simaishi pushed a commit that referenced this pull request Oct 12, 2018
…roups-in-role

Script to copy reports access from group to role

(cherry picked from commit a8f357d)

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1634673
@simaishi
Copy link
Contributor

Hammer backport details:

$ git log -1
commit cf812e13e51ce3bada4819a5c319b6c6acb2e8a2
Author: Gregg Tanzillo <gtanzill@redhat.com>
Date:   Thu Oct 11 15:41:36 2018 -0400

    Merge pull request #18066 from yrudman/script-to-copy-report-for-ll-groups-in-role
    
    Script to copy reports access from group to role
    
    (cherry picked from commit a8f357dbe8af8fed28487d2c5af0f0730e2985af)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1634673

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.

4 participants