Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Implement CRUD API for managing Harbor per-project Quota #3090
base: topic/11-06-feat_implement_per-project_images_api_based_on_rbac
Are you sure you want to change the base?
feat: Implement CRUD API for managing Harbor per-project Quota #3090
Changes from 29 commits
8d1297c
9128ef7
c60bd90
af7596d
ed1a4b1
a1225c2
c8fece0
4f3448a
780a250
1a4bf70
63d5dbb
473b125
1e8ac1a
cfd21fc
a49f1ff
16b45d1
3128b0a
97a3022
1a03789
c118c7e
96fc374
a1f3c08
217a359
89ab7a7
896c44d
835d0c6
0a442c9
049dfe0
cae3fee
3d1c9cf
cb68069
1ec6c50
7ddcad7
90eaff7
79d85e2
15d396f
cd0d828
515d39d
b40f6e3
1c04e5a
fc3060c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fregataa Registry quota READ operation can be executed even if the user is not an admin.
However, it seems that GQL queries in the current SDK can only be executed through
Admin
.What do you think about adding this
query
,_query
functions toUser
as well?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fregataa If possible, instead of hardcoding the
scope_id
like this, I’d prefer to create aProjectScope
object and serialize it. However, since theProjectScope
type is currently located under themodels
directory, this violates the pants visibility rule.What do you think about moving the types related to
ScopeField
under thecommon
directory?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use PATCH, DELETE method? @fregataa
Check notice on line 703 in src/ai/backend/manager/api/schema.graphql
GitHub Actions / GraphQL Inspector
Field 'registry_quota' was added to object type 'GroupNode'
Check notice on line 1852 in src/ai/backend/manager/api/schema.graphql
GitHub Actions / GraphQL Inspector
Field 'create_container_registry_quota' was added to object type 'Mutations'
Check notice on line 1855 in src/ai/backend/manager/api/schema.graphql
GitHub Actions / GraphQL Inspector
Field 'update_container_registry_quota' was added to object type 'Mutations'
Check notice on line 1858 in src/ai/backend/manager/api/schema.graphql
GitHub Actions / GraphQL Inspector
Field 'delete_container_registry_quota' was added to object type 'Mutations'
Check notice on line 2586 in src/ai/backend/manager/api/schema.graphql
GitHub Actions / GraphQL Inspector
Type 'CreateContainerRegistryQuota' was added
Check notice on line 2592 in src/ai/backend/manager/api/schema.graphql
GitHub Actions / GraphQL Inspector
Type 'UpdateContainerRegistryQuota' was added
Check notice on line 2598 in src/ai/backend/manager/api/schema.graphql
GitHub Actions / GraphQL Inspector
Type 'DeleteContainerRegistryQuota' was added