CONSOLE-922: Support AppliedClusterResourceQuota for normal users

[rebeccaalpert]

List Pages

I added logic so the resource link knows what the current namespace is. We need to pass the user's current namespace to the resource link if it's an ACRQ because the ACRQ itself doesn't supply namespace metadata.

Scoped namespace:

All namespaces:

Support for search page (links work):

Support for API explorer page (instances links work):

Details Page

New details page:

New total used field (total used is cluster scope and used is scoped to the current namespace):

We show ACRQs in the Resource Quota card in the project overview:

I'm basing the meat of the list page part of the PR on some work Sam did a few years ago: https://github.com/spadgett/console/commits/applied-cluster-quota.

Fixes https://issues.redhat.com/browse/CONSOLE-922.

How to test:

1. Add new user (we are calling this user rebecca here) and login
2. Create a new project: oc new-project ralpert-test
3. Log in as admin
4. Create a new role for them: oc create role rqviewer --verb=get,list,watch --resource=resourcequota,appliedclusterresourcequota -n ralpert-test
5. Assign the role: oc adm policy add-role-to-user rqviewer rebecca --role-namespace=ralpert-test
6. Create a cluster resource quota:

oc create clusterquota for-user \
     --project-annotation-selector openshift.io/requester=rebecca \
     --hard pods=10 \
     --hard secrets=20

7. Change user to rebecca and run oc get appliedclusterresourcequota. If you can see an ACRQ, you should see it in the UI list.
8. The link should take you to a custom ACRQ details page.

[rebeccaalpert]

With suggested "always show ACRQ behavior":

Admin:

Regular user:

I addressed PR feedback where possible. @spadgett - I moved the ACRQ logic in the table row out into its own table row; let me know if you meant something else in your comment. Please also let me know if you have thoughts on the table component if you're against passing the namespace down.

I also added a test for the ACRQ ResourceUsageRow.

[spadgett]

I addressed PR feedback where possible. @spadgett - I moved the ACRQ logic in the table row out into its own table row; let me know if you meant something else in your comment. Please also let me know if you have thoughts on the table component if you're against passing the namespace down.

We definitely don't want to pass namespace down as a prop. Don't tables have a customData prop you can use?

An alternative that might be better is transitioning to the new table API: #8829

[spadgett]

@rebeccaalpert We should update the project overview page as well to show cluster quotas.

[rebeccaalpert]

All set @spadgett. I added ACRQs to the project dashboard.

[spadgett]

@rebeccaalpert Sorry, I wasn't clear. I was thinking of the ResourceQuotaCard on the project dashboard:

https://github.com/openshift/console/blob/22c6951efe7c4bca87f3f934063b9f4dcb0a4058/frontend/public/components/dashboard/project-dashboard/resource-quota-card.tsx

[rebeccaalpert]

This is all set @spadgett.

I defined an ACRQ list page and added ACRQs to the ResourceQuotaCard:

I updated the behavior for the ResourceQuotas list page. ClusterResourceQuotas are displayed on the ResourceQuotas list page if "all namespaces" is selected. Otherwise, ACRQs are shown. ACRQs now link to the relevant CRQ if users can list CRQs.

[spadgett]

@rebeccaalpert We should add some integration tests to cover this. It's an important function that we don't test manually often during development.

[rebeccaalpert]

Pushed changes from this morning; still needs integration tests.

[TheRealJon]

/lgtm

[rebeccaalpert]

/assign @opayne1
for docs approval

/assign @yapei
for QE approval

/assign @RickJWagner
for px approval

[openshift-bot]

/retest-required

Please review the full test history for this PR and help us cut down flakes.

[rebeccaalpert]

Putting on hold because the merge queue is blocked due to a test flake. This is ready for docs, product, and QE review though.

[openshift-bot]

/retest-required

Please review the full test history for this PR and help us cut down flakes.

[RickJWagner]

/label px-approved

[openshift-bot]

/retest-required

Please review the full test history for this PR and help us cut down flakes.

[opayne1]

/label docs-approved

[schituku]

/lgtm

[schituku]

/label qe-approved

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rebeccaalpert, schituku, TheRealJon, zherman0

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• frontend/OWNERS [TheRealJon]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-bot]

/retest-required

Please review the full test history for this PR and help us cut down flakes.