ROX-18391: cleanup permission checker

[janisz]

Description

When we started PG migration project we were unsure in what direction it will go with permission checker. This resulted in a safe approach to support everything we could think of. After time we can cleanup permission checker interface and limit it to only two methods.

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

[janisz]

Current dependencies on/for this PR:

• master
  • PR ROX-18155: pg generic store: Exists #6766
    • PR ROX-18155: pg generic store: Count #6767
      • PR ROX-18155: pg generic store: Walk #6768
        • PR ROX-18155: pg generic store: GetAll #6769
          • PR ROX-18155: pg generic store: Get #6770
            • PR ROX-18155: pg generic store: GetByQuery #6771
              • PR ROX-18155: pg generic store: GetIDs #6772
                • PR ROX-18155: pg generic store: GetMany #6773
                  • PR ROX-18155: pg generic store: DeleteByQuery #6774
                  • PR ROX-18155: pg generic store: Delete #6775
                  • PR ROX-18155: pg generic store: DeleteMany #6776
                  • PR ROX-18155: pg generic store: regions #6777
                  • PR ROX-18155: pg generic store: cleanup #6778
                  • PR refactor(pg): remove blank lines #6881
                  • PR ROX-18155: pg generic store: Upsert #6882
                  • PR WIP: chore(deps): upgrade pgx to v5 #6915
                  • PR chore(deps): upgrade pgx to v5 #6916
                  • PR refactor(pg): preallocate memory when copy #6894
                  • PR ROX-18391: cleanup permission checker #6907 👈

This comment was auto-generated by Graphite.

[roxbot]

Images are ready for the commit at 0b3db22.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.1.x-412-g0b3db2286f.

[openshift-ci]

@janisz: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/gke-qa-e2e-tests	aede067	link	false	/test gke-qa-e2e-tests
ci/prow/ocp-4-10-qa-e2e-tests	aede067	link	false	/test ocp-4-10-qa-e2e-tests
ci/prow/ocp-4-13-core-bpf-qa-e2e-tests	aede067	link	false	/test ocp-4-13-core-bpf-qa-e2e-tests
ci/prow/ocp-4-13-qa-e2e-tests	aede067	link	false	/test ocp-4-13-qa-e2e-tests

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

[janisz]

/retest

[janisz]

Graphite rebased this pull request as part of a merge.

[janisz]

@janisz merged this pull request with Graphite.