Skip to content

Fix syntax error in CRI-O drop-in file on Switching to EventedPLEG page#55696

Open
namanraofullstack wants to merge 2 commits intokubernetes:mainfrom
namanraofullstack:fix-docs-eventedpleg-crio-syntax
Open

Fix syntax error in CRI-O drop-in file on Switching to EventedPLEG page#55696
namanraofullstack wants to merge 2 commits intokubernetes:mainfrom
namanraofullstack:fix-docs-eventedpleg-crio-syntax

Conversation

@namanraofullstack
Copy link
Copy Markdown

Description

Fixes a syntax error in the CRI-O drop-in configuration example for enabling the EventedPLEG feature.

The current documentation suggests the following configuration:

[crio.runtime]
enable_pod_events: true

However, this results in a CRI-O configuration parsing error:

unable to decode configuration ... expected '.' or '=', but got ':' instead

CRI-O configuration uses TOML syntax, where key-value assignments must use = instead of :.

This PR updates the example to:

[crio.runtime]
enable_pod_events = true

This ensures the configuration is valid and prevents runtime errors when users attempt to enable EventedPLEG.

This change applies to both English and zh-cn translations.

Issue

Closes: #55670

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented May 6, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

Welcome @namanraofullstack!

It looks like this is your first PR to kubernetes/website 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval.

You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation.

You can also check if kubernetes/website has its own contribution guidelines.

You may want to refer to our testing guide if you run into trouble with your tests not passing.

If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs!

Thank you, and welcome to Kubernetes. 😃

@k8s-ci-robot k8s-ci-robot requested review from reylejano and tengqm May 6, 2026 18:52
@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. area/localization General issues or PRs related to localization language/en Issues or PRs related to English language language/zh Issues or PRs related to Chinese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels May 6, 2026
@namanraofullstack namanraofullstack force-pushed the fix-docs-eventedpleg-crio-syntax branch from feb998e to b6d60d3 Compare May 6, 2026 18:55
@netlify
Copy link
Copy Markdown

netlify Bot commented May 6, 2026

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit feb998e
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/69fb8deadb155d0007eabdde
😎 Deploy Preview https://deploy-preview-55696--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.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 6, 2026

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 91019f0
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/69fce7940eeb0d000840e5c8
😎 Deploy Preview https://deploy-preview-55696--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.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels May 6, 2026
@windsonsea
Copy link
Copy Markdown
Member

Hi @namanraofullstack, please only update the english version, and leave the other language for different localization teams.

/hold

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. and removed cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels May 7, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign lmktfy for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@namanraofullstack namanraofullstack force-pushed the fix-docs-eventedpleg-crio-syntax branch from 4dbbef0 to 91019f0 Compare May 7, 2026 19:27
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels May 7, 2026
@namanraofullstack
Copy link
Copy Markdown
Author

Hi @namanraofullstack, please only update the english version, and leave the other language for different localization teams.

/hold

I have reverted the zh-cn changes and only updated the english version.

@windsonsea
Copy link
Copy Markdown
Member

Thanks, please squash your commits into a single one.

/unhold

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 8, 2026
@windsonsea
Copy link
Copy Markdown
Member

/remove-language zh
/remove-area localization

@k8s-ci-robot k8s-ci-robot removed the language/zh Issues or PRs related to Chinese language label May 8, 2026
@k8s-ci-robot k8s-ci-robot removed the area/localization General issues or PRs related to localization label May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation: syntax error in CRI-O drop-in file on "Switching to EventedPLEG" page

3 participants