Skip to content

Docs V1: Expand the Kubernetes plugin page - #7079

Open
rahulshendre wants to merge 6 commits into
pipe-cd:masterfrom
rahulshendre:docs/v1-kubernetes-plugin-page
Open

Docs V1: Expand the Kubernetes plugin page#7079
rahulshendre wants to merge 6 commits into
pipe-cd:masterfrom
rahulshendre:docs/v1-kubernetes-plugin-page

Conversation

@rahulshendre

Copy link
Copy Markdown
Contributor

What this PR does:

Expands the Kubernetes plugin page from the current stub into a full reference. It adds prerequisites, quick sync, pipeline sync, every pipeline stage, with their options, livestate and drift detection, plan preview, etc.
Every stage name, option, field, and default is taken from the plugin source - pkg/app/pipedv1/plugin/kubernetes

Why we need it:

Expands the Kubernetes plugin page to match the standard of the Kubernetes Multicluster plugin and ECS plugin.

Which issue(s) this PR fixes:

Part of #6679

Does this PR introduce a user-facing change?:

  • How are users affected by this change:
  • Is this breaking change:
  • How to migrate (if breaking change):

Signed-off-by: rahulshendre <rahulshendre789@gmail.com>
@rahulshendre

Copy link
Copy Markdown
Contributor Author

cc @khanhtc1202 @eeshaanSA @mohammedfirdouss

khanhtc1202
khanhtc1202 previously approved these changes Jul 26, 2026
Comment thread docs/content/en/docs-v1.0.x/plugins/kubernetes.md Outdated
Comment thread docs/content/en/docs-v1.0.x/plugins/kubernetes.md Outdated

@khanhtc1202 khanhtc1202 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.

Sorry, I missed click approval

Signed-off-by: rahulshendre <rahulshendre789@gmail.com>
@codecov

codecov Bot commented Jul 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.17%. Comparing base (8671bd5) to head (4c413ad).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7079      +/-   ##
==========================================
- Coverage   29.36%   29.17%   -0.20%     
==========================================
  Files         601      526      -75     
  Lines       64328    58492    -5836     
==========================================
- Hits        18891    17063    -1828     
+ Misses      43969    40071    -3898     
+ Partials     1468     1358     -110     
Flag Coverage Δ
. 23.12% <ø> (ø)
.-pkg-app-pipedv1-plugin-analysis ?
.-pkg-app-pipedv1-plugin-ecs ?
.-pkg-app-pipedv1-plugin-kubernetes 58.39% <ø> (ø)
.-pkg-app-pipedv1-plugin-kubernetes_multicluster 59.62% <ø> (ø)
.-pkg-app-pipedv1-plugin-scriptrun ?
.-pkg-app-pipedv1-plugin-terraform ?
.-pkg-app-pipedv1-plugin-wait ?
.-pkg-app-pipedv1-plugin-waitapproval 52.71% <ø> (ø)
.-pkg-plugin-sdk 50.14% <ø> (-0.20%) ⬇️
.-tool-actions-gh-release ?
.-tool-actions-plan-preview 25.51% <ø> (ø)
.-tool-codegen-protoc-gen-auth ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rahulshendre

Copy link
Copy Markdown
Contributor Author

@khanhtc1202 @eeshaanSA @mohammedfirdouss PTAL when you have time, thanks!

@eeshaanSA eeshaanSA left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Largely LGTM. Please do fix the nits. Also, as noticed, double check application configurations. Make sure they adhere to all the v1 changes.

# port / url / deployTargets ...
config:
chartRepositories:
- type: HTTP # ChartMuseum, Nexus, Artifactory, ...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

safe to remove the comments here, I suppose

@mohammedfirdouss mohammedfirdouss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM
@khanhtc1202 PTAL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants