Skip to content

ROX-13326: Use UUID columns for roles, permissionsets and simpleaccessscopes in postgres mode. #3862

Merged
janisz merged 1 commit intorelease-3.73from
yann-patch-3.73/uuid_for_role_ids
Nov 21, 2022
Merged

ROX-13326: Use UUID columns for roles, permissionsets and simpleaccessscopes in postgres mode. #3862
janisz merged 1 commit intorelease-3.73from
yann-patch-3.73/uuid_for_role_ids

Conversation

@rhybrillou
Copy link
Contributor

@rhybrillou rhybrillou commented Nov 21, 2022

Description

Cherry pick of PR 3802 to release-3.73

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 run + manual steps of PR 3802

@rhybrillou rhybrillou requested review from janisz and mtodor November 21, 2022 14:47
@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

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 rhybrillou marked this pull request as ready for review November 21, 2022 14:47
@rhybrillou
Copy link
Contributor Author

rhybrillou commented Nov 21, 2022

/test

@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

@rhybrillou: The /test command needs one or more targets.
The following commands are available to trigger required jobs:

  • /test images
  • /test stackrox_branding-images

Use /test all to run all jobs.

Details

In response to this:

/test

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rhybrillou
Copy link
Contributor Author

rhybrillou commented Nov 21, 2022

/test go-unit-tests-release

@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

@rhybrillou: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

  • /test images
  • /test stackrox_branding-images

Use /test all to run all jobs.

Details

In response to this:

/test go-unit-tests-release

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rhybrillou
Copy link
Contributor Author

rhybrillou commented Nov 21, 2022

/test style-checks

@rhybrillou
Copy link
Contributor Author

rhybrillou commented Nov 21, 2022

/test ui-unit-tests

@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

@rhybrillou: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

  • /test images
  • /test stackrox_branding-images

Use /test all to run all jobs.

Details

In response to this:

/test style-checks

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

@rhybrillou: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

  • /test images
  • /test stackrox_branding-images

Use /test all to run all jobs.

Details

In response to this:

/test ui-unit-tests

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@janisz
Copy link
Contributor

janisz commented Nov 21, 2022

CI will run after openshift/release#34183 is merged

@rhybrillou
Copy link
Contributor Author

rhybrillou commented Nov 21, 2022

/test

@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

@rhybrillou: The /test command needs one or more targets.
The following commands are available to trigger required jobs:

  • /test images
  • /test stackrox_branding-images

Use /test all to run all jobs.

Details

In response to this:

/test

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rhybrillou
Copy link
Contributor Author

rhybrillou commented Nov 21, 2022

/test go-unit-tests-release

@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

@rhybrillou: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

  • /test images
  • /test stackrox_branding-images

Use /test all to run all jobs.

Details

In response to this:

/test go-unit-tests-release

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rhybrillou
Copy link
Contributor Author

rhybrillou commented Nov 21, 2022

/test style-checks

@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

@rhybrillou: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

  • /test images
  • /test stackrox_branding-images

Use /test all to run all jobs.

Details

In response to this:

/test style-checks

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rhybrillou
Copy link
Contributor Author

rhybrillou commented Nov 21, 2022

/test ui-unit-tests

@openshift-ci
Copy link

openshift-ci bot commented Nov 21, 2022

@rhybrillou: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

  • /test images
  • /test stackrox_branding-images

Use /test all to run all jobs.

Details

In response to this:

/test ui-unit-tests

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@janisz janisz force-pushed the yann-patch-3.73/uuid_for_role_ids branch from e648396 to 470e6a3 Compare November 21, 2022 18:16
@janisz janisz enabled auto-merge (squash) November 21, 2022 18:17
@janisz janisz merged commit 153e8a8 into release-3.73 Nov 21, 2022
@janisz janisz deleted the yann-patch-3.73/uuid_for_role_ids branch November 21, 2022 19:17
@rukletsov rukletsov mentioned this pull request Nov 22, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@janisz janisz janisz approved these changes

@mtodor mtodor Awaiting requested review from mtodor

Assignees

No one assigned

Labels

area/central area/postgres area/ui

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rhybrillou @janisz