| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
Codecov Report❌ Patch coverage is 37.96610% with 183 lines in your changes missing coverage. Please review. @@ Coverage Diff @@
## main #13236 +/- ##
============================================
+ Coverage 18.75% 18.77% +0.01%
- Complexity 17966 17983 +17
============================================
Files 6160 6163 +3
Lines 552578 552779 +201
Branches 67348 67365 +17
============================================
+ Hits 103660 103758 +98
- Misses 437512 437611 +99
- Partials 11406 11410 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness.
|
Sorry, something went wrong.
|
@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress. |
Sorry, something went wrong.
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 18042 |
Sorry, something went wrong.
|
@DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
Sorry, something went wrong.
|
[SF] Trillian test result (tid-16186)
|
Sorry, something went wrong.
|
@winterhazel , I can review, but you probably have someone more knowledgeable to look at this... |
Sorry, something went wrong.
@DaanHoogland I already asked some people that are familiar with Quota to review this one, but feel free to review as well if you want to. Your reviews would still be appreciated :) |
Sorry, something went wrong.
|
@winterhazel a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress. |
Sorry, something went wrong.
|
@winterhazel a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress. |
Sorry, something went wrong.
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 18196 |
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM, but I haven't been able to test this version yet. I'll try doing it soon.
Sorry, something went wrong.
|
@DaanHoogland @weizhouapache @sureshanaparti @Pearl1594 can we run the CI here? |
Sorry, something went wrong.
|
@Pearl1594 a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
Sorry, something went wrong.
There was a problem hiding this comment.
CLGMT, will test it locally.
Sorry, something went wrong.
|
[SF] Trillian test result (tid-16283)
|
Sorry, something went wrong.
|
@winterhazel a [SL] Jenkins job has been kicked to build packages. It will be bundled with no SystemVM templates. I'll keep you posted as I make progress. |
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM. Everything works fine, simulated regular API usages, and did some fuzzing as well.
Sorry, something went wrong.
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 18229 |
Sorry, something went wrong.
|
@Pearl1594 also asking to run the CI here one last time before the merge :) |
Sorry, something went wrong.
|
@Pearl1594 a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
Sorry, something went wrong.
There was a problem hiding this comment.
code lgtm, did not test it
Sorry, something went wrong.
|
[SF] Trillian test result (tid-16289)
|
Sorry, something went wrong.
|
Test failures are unrelated, so merging |
Sorry, something went wrong.
(cherry picked from commit 2fd83e1)
| Back | FazBrowse Home | New Git URL |
Description
This PR introduces the quotaResourceStatement API to the Quota plugin. This API allows viewing the Quota consumption associated with a specific resource. It will be used by the Quota UI rework in a separate PR.
In addition, some adjustments were made in the quotaStatement API in order to allow including the consumption of subdomains in the resulting statement.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
How Has This Been Tested?
I tested the API by fuzzing its parameters for root admin, domain admin, and user accounts. In the tests, I validated that the API returned the expected response while performing correct permission checking.
Environment configuration
My environment had two domains: ROOT and d1.
Each domain had a domain admin (dr for ROOT and d1 for d1) and a user account (ur for ROOT and u1 for d1). ROOT also had a project and the root admin.
Test 1 (user accounts)
id: returns the statement scoped to the owning account when the caller has access to it, and an error when it does not.
accountid: returns the statement scoped to the provided account if accessible, or an empty list otherwise.
projectid: returns the statement scoped to the provided project if accessible, or an exception otherwise.
isrecursive: ignored for user accounts.
Test 2 (domain admins)
id: returns the statement scoped to the caller's domain.
accountid: returns the statement scoped to the provided account if accessible, or an exception otherwise.
projectid: returns the statement scoped to the provided project if accessible, or an exception otherwise.
isrecursive: allows domain admins to generate the statement including usage records of subdomains.
Test 3 (root admins)
id: returns the statement of any resource without limiting.
accountid: returns the statement scoped to the provided account.
projectid: returns the statement scoped to the provided project.
isrecursive: allows admins to generate the statement including usage records of subdomains.