Skip to content
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

fix: adjust rollout document. #616

Merged
merged 6 commits into from
Nov 15, 2023

Conversation

ryanzhang-oss
Copy link
Contributor

Description of your changes

adjust rollout document.

Fixes #

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

docs/howtos/crp.md Outdated Show resolved Hide resolved
Co-authored-by: Britania Rodriguez Reyes <[email protected]>
docs/howtos/crp.md Outdated Show resolved Hide resolved
docs/howtos/crp.md Outdated Show resolved Hide resolved
Arvindthiru
Arvindthiru previously approved these changes Nov 15, 2023
britaniar
britaniar previously approved these changes Nov 15, 2023
@ryanzhang-oss ryanzhang-oss dismissed stale reviews from britaniar and Arvindthiru via 6ad2542 November 15, 2023 22:16
docs/howtos/crp.md Outdated Show resolved Hide resolved
Co-authored-by: Britania Rodriguez Reyes <[email protected]>
if there are too many newly picked clusters and Fleet places resources on them at the same time,
your backend may get overloaded. The exact pattern of interruption may vary,
depending on the set of resources you place using Fleet.
Most outcomes can lead to service interruptions. Apps running on member clusters may temporarily become
Copy link
Contributor

Choose a reason for hiding this comment

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

Most outcomes listed above can lead to service interruptions

@ryanzhang-oss ryanzhang-oss merged commit bfd82c9 into Azure:main Nov 15, 2023
5 checks passed
@ryanzhang-oss ryanzhang-oss deleted the fix-rollout-desc branch February 12, 2024 19:04
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