Quota UI: Adding Overall storage consumption for Super users panel page (PROJQUAY-3639) #1307
Conversation
Sunandadadi
commented
May 9, 2022
•
Sunandadadi
commented
…ge (PROJQUAY-3639)
| return; | ||
| } | ||
| let total = 0; | ||
| $scope.organizations.forEach(function (obj){ |
There was a problem hiding this comment.
Wondering if just doing a sum over the repositorysize table would be better. It would keep it to a single table operation, easier to cache, etc... as opposed to going through every single organization in the registry, which involves selecting from the tag and manifest tables.
There was a problem hiding this comment.
i.e have a superuser specific endpoint that just does this repositorysize table aggregation.
There was a problem hiding this comment.
We have the Quota Consumption data for every org under quota_report key. So, instead of making another API call, we are using the data the /organizations endpoint is fetching.
There was a problem hiding this comment.
Ah ok. I guess that panel is already fetching the complete list of organization.
|
/cherry-pick redhat-3.7 |
|
@HammerMeetNail: new pull request created: #1314 In response to this:
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. |
