Skip to content

GlobalResourceQuota doesn't exist yet#21

Merged
oliverbaehler merged 2 commits intoprojectcapsule:mainfrom
benlangfeld:patch-1
May 20, 2025
Merged

GlobalResourceQuota doesn't exist yet#21
oliverbaehler merged 2 commits intoprojectcapsule:mainfrom
benlangfeld:patch-1

Conversation

@benlangfeld
Copy link
Copy Markdown
Contributor

It's unclear why this was written the way it was. Any insight would be very helpful.

@prometherion
Copy link
Copy Markdown
Member

@oliverbaehler sounds to me just a typo: WDYT?

@oliverbaehler
Copy link
Copy Markdown
Contributor

hehe, i got ahead of myself. This is a feature from an upcoming release. We can remove it for now, sorre for the confusion

@benlangfeld
Copy link
Copy Markdown
Contributor Author

@oliverbaehler Is there an implementation anywhere, or is this just something someone is thinking of doing some time? I couldn't find any other references to this resource type.

@oliverbaehler
Copy link
Copy Markdown
Contributor

It will be landing in the next release. However the feature was redesigned. It will go into review this week

@benlangfeld
Copy link
Copy Markdown
Contributor Author

It will be landing in the next release. However the feature was redesigned. It will go into review this week

Do you have a link to a PR that I can look at? I couldn't find it.

It's unclear why this was written the way it was. Any insight would be very helpful.

Signed-off-by: Ben Langfeld <ben@langfeld.me>
Signed-off-by: Ben Langfeld <ben@langfeld.me>
@benlangfeld
Copy link
Copy Markdown
Contributor Author

@prometherion @oliverbaehler I think this somewhat reflects reality now (despite the proposed feature still having overloaded terminology). Could it be merged so as not to confuse the reader?

@oliverbaehler oliverbaehler merged commit 8aa726b into projectcapsule:main May 20, 2025
2 of 4 checks passed
@benlangfeld benlangfeld deleted the patch-1 branch May 21, 2025 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants