Skip to content

ROX-11101: Remove deprecated resources from QA tests#3116

Merged
rhybrillou merged 7 commits intomasterfrom
yann/ROX-11101-qa_remove_old_resource
Nov 7, 2022
Merged

ROX-11101: Remove deprecated resources from QA tests#3116
rhybrillou merged 7 commits intomasterfrom
yann/ROX-11101-qa_remove_old_resource

Conversation

@rhybrillou
Copy link
Contributor

@rhybrillou rhybrillou commented Sep 16, 2022

Description

The set resources used for access control is being simplified. A number of resources has been deprecated.

The goal here is to get rid of the deprecated resources, both in the database and in the codebase (UI, QA, central, DB). This PR focuses on the QA test codebase removal. Full PR chain:
#3036
#3115
#3116
#3428

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 Sep 16, 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 changed the base branch from master to yann/ROX-11101-ui_remove_old_resources September 16, 2022 15:27
@rhybrillou
Copy link
Contributor Author

/test all

@openshift-ci
Copy link

openshift-ci bot commented Sep 16, 2022

@rhybrillou: No presubmit jobs available for stackrox/stackrox@yann/ROX-11101-ui_remove_old_resources

Details

In response to this:

/test all

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 rhybrillou requested review from a team and rukletsov September 19, 2022 15:55
@rhybrillou rhybrillou force-pushed the yann/ROX-11101-ui_remove_old_resources branch from 0d026e3 to f137334 Compare September 19, 2022 16:10
@rhybrillou rhybrillou force-pushed the yann/ROX-11101-qa_remove_old_resource branch from 0c5e706 to 9eee493 Compare September 19, 2022 16:11
@rhybrillou rhybrillou force-pushed the yann/ROX-11101-qa_remove_old_resource branch 2 times, most recently from 9309005 to dd62b13 Compare September 21, 2022 19:29
@rhybrillou rhybrillou force-pushed the yann/ROX-11101-ui_remove_old_resources branch from 3e5ff8d to 9311186 Compare September 23, 2022 07:43
@rhybrillou rhybrillou force-pushed the yann/ROX-11101-qa_remove_old_resource branch from dd62b13 to 50cc118 Compare September 23, 2022 07:44
Base automatically changed from yann/ROX-11101-ui_remove_old_resources to master September 23, 2022 13:55
@rhybrillou rhybrillou marked this pull request as ready for review September 23, 2022 13:56
@ghost
Copy link

ghost commented Sep 23, 2022

Images are ready for the commit at c827cad.

To use with deploy scripts, first export MAIN_IMAGE_TAG=3.72.x-524-gc827cad97e.

@rhybrillou rhybrillou force-pushed the yann/ROX-11101-qa_remove_old_resource branch from 50cc118 to 5df5b1c Compare September 23, 2022 21:09
@rhybrillou rhybrillou force-pushed the yann/ROX-11101-qa_remove_old_resource branch 2 times, most recently from 1c95c7d to c30a767 Compare October 17, 2022 10:16
@rhybrillou rhybrillou force-pushed the yann/ROX-11101-qa_remove_old_resource branch from 90fbdef to c827cad Compare November 6, 2022 18:50
@rhybrillou rhybrillou merged commit f77c711 into master Nov 7, 2022
@rhybrillou rhybrillou deleted the yann/ROX-11101-qa_remove_old_resource branch November 7, 2022 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants