Skip to content

Blog for container restart policy #51745

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Aug 10, 2025
Merged

Blog for container restart policy #51745

merged 2 commits into from
Aug 10, 2025

Conversation

yuanwang04
Copy link
Contributor

Description

Add a blog post about the new alpha feature ContainerRestartRules in 1.34. The blog post explains how it works and provides some useful examples and set up.

Issue

Related KEP: kubernetes/enhancements#5307

@k8s-ci-robot k8s-ci-robot added this to the 1.34 milestone Jul 30, 2025
@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jul 30, 2025
@k8s-ci-robot k8s-ci-robot requested a review from lmktfy July 30, 2025 23:28
@k8s-ci-robot k8s-ci-robot added the area/blog Issues or PRs related to the Kubernetes Blog subproject label Jul 30, 2025
@k8s-ci-robot k8s-ci-robot added the language/en Issues or PRs related to English language label Jul 30, 2025
@netlify Netlify
Copy link

netlify bot commented Jul 30, 2025 •
edited
Loading

👷 Deploy Preview for kubernetes-io-vnext-staging processing.

Name Link
🔨 Latest commit 2ebe15a
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-vnext-staging/deploys/688bfd66462c7a0007efada0

@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jul 30, 2025
@netlify Netlify
Copy link

netlify bot commented Jul 30, 2025 •
edited
Loading

✅ Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit a223cea
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/689913fadfb9d90008aca3ef
😎 Deploy Preview https://deploy-preview-51745--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@aibarbetta
Copy link
Member

/sig release

@k8s-ci-robot k8s-ci-robot added the sig/release Categorizes an issue or PR as relevant to SIG Release. label Jul 31, 2025
aibarbetta
aibarbetta reviewed Jul 31, 2025
View reviewed changes
Copy link
Member

@aibarbetta aibarbetta left a comment

Choose a reason for hiding this comment

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

Please update the PR to merge into kubernetes:main (not kubernetes:dev.1-34)

@yuanwang04 yuanwang04 force-pushed the container_restart_blog branch 3 times, most recently from 81347ff to 2ebe15a Compare July 31, 2025 23:33
@k8s-ci-robot k8s-ci-robot added area/localization General issues or PRs related to localization language/ja Issues or PRs related to Japanese language language/pt Issues or PRs related to Portuguese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Jul 31, 2025
@yuanwang04 yuanwang04 changed the base branch from dev-1.34 to main July 31, 2025 23:34
@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jul 31, 2025
@yuanwang04 yuanwang04 force-pushed the container_restart_blog branch from 2ebe15a to cb11e1e Compare July 31, 2025 23:36
@yuanwang04 yuanwang04 marked this pull request as ready for review July 31, 2025 23:39
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 31, 2025
@k8s-ci-robot k8s-ci-robot requested review from deads2k and enj July 31, 2025 23:39
@aibarbetta
Copy link
Member

@yuanwang04 I see come unrelated commits in the diff, can you remove them and leave the blog content only?

@SergeyKanzhelev SergeyKanzhelev mentioned this pull request Aug 1, 2025
Open
12 tasks
@yuanwang04 yuanwang04 force-pushed the container_restart_blog branch from cb11e1e to 16780f9 Compare August 1, 2025 20:30
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Aug 1, 2025
@yuanwang04
Copy link
Contributor Author

@yuanwang04 I see come unrelated commits in the diff, can you remove them and leave the blog content only?

Done. Sorry didn't notice there was a messy rebase from dev-1.34 to main. It's cleaned up now.

@t-inu
Copy link
Member

t-inu commented Aug 2, 2025

/remove-language ja pt
/remove-area localization

@k8s-ci-robot k8s-ci-robot removed language/ja Issues or PRs related to Japanese language language/pt Issues or PRs related to Portuguese language area/localization General issues or PRs related to localization labels Aug 2, 2025
Serenity611
Serenity611 reviewed Aug 5, 2025
View reviewed changes
Copy link
Contributor

@Serenity611 Serenity611 left a comment

Choose a reason for hiding this comment

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

Reviewed for general proofreading/formatting, looks great! Added a few suggestions for switching headings to sentence case to align with the style guide :)

@yuanwang04 yuanwang04 force-pushed the container_restart_blog branch from 16780f9 to 899e0db Compare August 6, 2025 05:23
lmktfy
lmktfy reviewed Aug 6, 2025
View reviewed changes
Copy link
Contributor

@lmktfy lmktfy left a comment

Choose a reason for hiding this comment

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

Watch out for "we"; it can confuse, and we like to be clear.

content/en/blog/_posts/2025-0X-XX-Per-Container-Restart-Policy.md Outdated

### Example 1: Restarting on specific exit codes

In this example, we want to restart the container if and only if it fails with a
Copy link
Contributor

Choose a reason for hiding this comment

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

Avoid "we" when an article only has 1 author. Can you reword for clarity?

content/en/blog/_posts/2025-0X-XX-Per-Container-Restart-Policy.md Outdated

## We want your feedback!

This is an alpha feature, and we'd love to hear your feedback. Please try it out
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This is an alpha feature, and we'd love to hear your feedback. Please try it out
This is an alpha feature, and the Kubernetes project would love to hear your feedback.
Please try it out

@yuanwang04 yuanwang04 force-pushed the container_restart_blog branch from 899e0db to 041e363 Compare August 8, 2025 21:24
lmktfy
lmktfy reviewed Aug 10, 2025
View reviewed changes
@lmktfy
Ensure article is marked as draft
a223cea 
The article must be draft until the Kubernetes v1.34 release has happened, and also
must stay draft until it has a (future) publication date set.
lmktfy
lmktfy reviewed Aug 10, 2025
View reviewed changes
Copy link
Contributor

@lmktfy lmktfy left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 10, 2025
@k8s-ci-robot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 71a7670abe87d131eb7b124a513706bb53603046

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lmktfy

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

The pull request process is described here

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 10, 2025
@k8s-ci-robot k8s-ci-robot merged commit 1c09dc5 into kubernetes:main Aug 10, 2025
6 checks passed
@aibarbetta aibarbetta mentioned this pull request Aug 27, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lmktfy lmktfy lmktfy left review comments

@nate-double-u nate-double-u Awaiting requested review from nate-double-u

@deads2k deads2k Awaiting requested review from deads2k

@enj enj Awaiting requested review from enj

+2 more reviewers

@aibarbetta aibarbetta aibarbetta left review comments

@Serenity611 Serenity611 Serenity611 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@lmktfy lmktfy

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/blog Issues or PRs related to the Kubernetes Blog subproject cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. sig/release Categorizes an issue or PR as relevant to SIG Release. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

1.34

Development

Successfully merging this pull request may close these issues.

6 participants

@yuanwang04 @aibarbetta @t-inu @k8s-ci-robot @Serenity611 @lmktfy