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

1.7.2 docs #211

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

1.7.2 docs #211

wants to merge 1 commit into from

Conversation

ABNER-1
Copy link
Member

@ABNER-1 ABNER-1 commented Sep 23, 2024

prepare 1.7.2 docs, introduction of specified-pod is only added into current(wait review)

@ABNER-1 ABNER-1 changed the title .7.2 docs 1.7.2 docs Sep 23, 2024
docs/user-manuals/advancedstatefulset.md Outdated Show resolved Hide resolved
docs/user-manuals/advancedstatefulset.md Outdated Show resolved Hide resolved
# ...
```

When the controller receives the above Pod update, it will prioritize the deletion process of Pods with the specified deletion label and ensure that the `maxUnavailable` limit is not exceeded.
Copy link
Member

Choose a reason for hiding this comment

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

it will prioritize the deletion process of Pods with the specified deletion label
-> it will trigger the deletion process of the pod with specified deletion label .

plz also comment that the pod will be rebuild by the workload if the replicas is not matched after deletion.

Copy link
Member Author

Choose a reason for hiding this comment

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

I emphasize prioritizing deletion over updates because I want to highlight that deletion takes precedence.

I think "the pod will be rebuilt by the workload if the replicas are not matched after deletion" should perhaps be "The pod will be rebuilt by the workload if the ordinal is not reserved."

This is because even if the replicas do not match after deletion, the pod will still be rebuilt, and the pod with the highest ordinal will be deleted.

…current(wait review)

Signed-off-by: Abner-1 <yuanyuxing.yyx@alibaba-inc.com>
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.

2 participants