Skip to content

Adding documentation for resource quota enforcement feature#835

Merged
karmada-bot merged 1 commit into
karmada-io:mainfrom
mszacillo:main
May 21, 2025
Merged

Adding documentation for resource quota enforcement feature#835
karmada-bot merged 1 commit into
karmada-io:mainfrom
mszacillo:main

Conversation

@mszacillo

@mszacillo mszacillo commented May 17, 2025

Copy link
Copy Markdown
Member

What type of PR is this?
/kind documentation

What this PR does / why we need it:

Part of #6350, adds documentation for the resource quota enforcement feature we had added to the FederatedResourceQuota.

@karmada-bot karmada-bot added the kind/documentation Categorizes issue or PR as related to documentation. label May 17, 2025
@karmada-bot karmada-bot requested review from Poor12 and Tingtal May 17, 2025 20:56
@karmada-bot karmada-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label May 17, 2025
@zhzhuang-zju

Copy link
Copy Markdown
Contributor

@mszacillo The CI failed during the Netlify Build phase, and it doesn't seem to be a code issue.

Could you resubmit your PR to trigger the CI pipeline again?

@RainbowMango

Copy link
Copy Markdown
Member

Let me take a look, another PR is blocked by this issue.

@RainbowMango

Copy link
Copy Markdown
Member

Sent #836 to resolve the failing test.

@RainbowMango

Copy link
Copy Markdown
Member

@mszacillo #836 gets merged, and this should be good after the rebase.

@mszacillo

Copy link
Copy Markdown
Member Author

@RainbowMango Awesome! I just rebased my branch.

@RainbowMango RainbowMango left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/assign

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds documentation for the resource quota enforcement feature for FederatedResourceQuota. It updates the best practices guide with additional sections on ResourceQuota Management and control plane-level quota enforcement, and introduces a new usage example for enforcing overall resource limits.

Comment thread docs/userguide/bestpractices/federated-resource-quota.md Outdated
Comment thread docs/userguide/bestpractices/federated-resource-quota.md

@RainbowMango RainbowMango left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! I like this style, simple、 straightforward enough but has everything important.

Just one comment from Copilot needs to take a look?

Signed-off-by: mszacillo <mszacillo@bloomberg.net>
@mszacillo

Copy link
Copy Markdown
Member Author

Just one comment from Copilot needs to take a look?

Just resolved! Thanks again for the review

@RainbowMango RainbowMango left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@karmada-bot karmada-bot added the lgtm Indicates that a PR is ready to be merged. label May 21, 2025
@karmada-bot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: RainbowMango

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

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

@karmada-bot karmada-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 21, 2025
@karmada-bot karmada-bot merged commit 8eb8a04 into karmada-io:main May 21, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/documentation Categorizes issue or PR as related to documentation. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants