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

ROX-14398: Group Role permission with Access one #5848

Merged

Conversation

rhybrillou
Copy link
Contributor

@rhybrillou rhybrillou commented Apr 28, 2023

Description

This change is part of a series with the goal of making the configuration of user access control easier, and of limiting the risk of features not working because of missing permissions.

The goal is to bring the Role permission under the umbrella of the Access one.

Checklist

  • Investigated and inspected CI test results
    - [ ] Unit test and regression tests added
  • Evaluated and added CHANGELOG entry if required
    - [ ] Determined and documented upgrade steps
    - [ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.

Testing Performed

CI should be sufficient

@openshift-ci
Copy link

openshift-ci bot commented Apr 28, 2023

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@rhybrillou
Copy link
Contributor Author

rhybrillou commented Apr 28, 2023

@ghost
Copy link

ghost commented Apr 28, 2023

Images are ready for the commit at 1b13aac.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.0.x-439-g1b13aacd76.

@rhybrillou rhybrillou force-pushed the master-yann/ROX-14398-group_role_resource_with_access branch from bdbbecd to a4ab803 Compare May 10, 2023 23:34
@rhybrillou rhybrillou requested a review from a team May 10, 2023 23:35
@rhybrillou rhybrillou marked this pull request as ready for review May 11, 2023 07:35
@rhybrillou rhybrillou requested a review from a team as a code owner May 11, 2023 07:35
Copy link
Contributor

@parametalol parametalol left a comment

Choose a reason for hiding this comment

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

LGTM

@rhybrillou rhybrillou force-pushed the master-yann/ROX-14398-group_role_resource_with_access branch 3 times, most recently from 094b3dc to f06d40b Compare May 23, 2023 00:22
@rhybrillou
Copy link
Contributor Author

/retest

@rhybrillou rhybrillou force-pushed the master-yann/ROX-14398-group_role_resource_with_access branch from f06d40b to 1b13aac Compare May 25, 2023 21:07
@rhybrillou rhybrillou merged commit 52e521c into master May 26, 2023
@rhybrillou rhybrillou deleted the master-yann/ROX-14398-group_role_resource_with_access branch May 26, 2023 06:41
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.

2 participants