Skip to content

docs: add Helm chart installation - #599

Open
kkrypt0nn wants to merge 1 commit into
cloudnative-pg:mainfrom
kkrypt0nn:main
Open

docs: add Helm chart installation#599
kkrypt0nn wants to merge 1 commit into
cloudnative-pg:mainfrom
kkrypt0nn:main

Conversation

@kkrypt0nn

@kkrypt0nn kkrypt0nn commented Oct 15, 2025 •
edited
Loading

Copy link
Copy Markdown

The Helm chart has been added around 2 weeks a couple months ago.

Documented the versioned docs for:

  • 0.5.0
  • 0.7.0
  • 0.9.0
  • 0.10.0
  • 0.11.0
  • 0.12.0
  • 0.13.0
  • 0.14.0

Added the --version argument for the versioned docs with their version of the chart.

Closes #351

@kkrypt0nn
kkrypt0nn requested a review from a team October 15, 2025 08:15
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. documentation Improvements or additions to documentation labels Oct 15, 2025
@kkrypt0nn
kkrypt0nn force-pushed the main branch 4 times, most recently from a690a06 to 65c962f Compare October 15, 2025 08:37
@kkrypt0nn
kkrypt0nn force-pushed the main branch 4 times, most recently from 11c0990 to 4f83d5a Compare January 5, 2026 20:05
@raweber42

raweber42 commented Feb 3, 2026

Copy link
Copy Markdown

Can we merge this? I was looking for this all over the place 😅

@kkrypt0nn
kkrypt0nn force-pushed the main branch 2 times, most recently from 2f1ef31 to d769b2c Compare February 9, 2026 20:42
@kkrypt0nn
kkrypt0nn force-pushed the main branch 2 times, most recently from ff9b171 to 8e30764 Compare April 15, 2026 19:03
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Apr 15, 2026
@kkrypt0nn
kkrypt0nn force-pushed the main branch 2 times, most recently from 483f41b to bb09056 Compare July 1, 2026 06:35
@kkrypt0nn kkrypt0nn mentioned this pull request Jul 1, 2026
Open
@kkrypt0nn

kkrypt0nn commented Jul 1, 2026

Copy link
Copy Markdown
Author

Made its own installation snippet component so that it's simpler/cleaner

@ardentperf

ardentperf commented Jul 6, 2026

Copy link
Copy Markdown

do we really need to update all the old versions? if we just updated latest it would be much smaller PR and easier to review

@kkrypt0nn

kkrypt0nn commented Jul 6, 2026

Copy link
Copy Markdown
Author

It's essentially a change across all files so the review doesn't get more complex, maybe only checking the chart version & docs version.

I am however open to removing the old ones, it became bigger simply because the PR was opened when there were just 2 versions and I kept on updating it as new versions came..

@gbartolini
gbartolini requested a review from a team as a code owner August 6, 2026 22:49
@kkrypt0nn kkrypt0nn mentioned this pull request Aug 12, 2026
Closed
@danishedb danishedb mentioned this pull request Aug 24, 2026
Merged
@danishedb

danishedb commented Aug 24, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Hi @kkrypt0nn, I have reviewed the PR and Requesting you to have a look in the review patch here.

@kkrypt0nn @danishedb
docs: add Helm chart installation
5e923c9 
* Use built-in Docusaurus tabs for the Helm and kubectl install methods
* Rename the Helm release to plugin-barman-cloud, matching the chart's README and making the rollout verification command work
* Drop --create-namespace: the operator prerequisite guarantees the namespace exists
* Give each method its own verification step and example output

Signed-off-by: Krypton <root@krypton.ninja>
Signed-off-by: danishedb <danish.khan@enterprisedb.com>
Co-authored-by: danishedb <danish.khan@enterprisedb.com>
@kkrypt0nn

kkrypt0nn commented Aug 24, 2026

Copy link
Copy Markdown
Author

Hi @danishedb, thanks for the review! The changes made total sense therefore I went ahead and merged both as one co-authored commit, thanks again!

@Akay7 Akay7 mentioned this pull request Aug 25, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leonardoce leonardoce Awaiting requested review from leonardoce
@mnencia mnencia Awaiting requested review from mnencia
@gbartolini gbartolini Awaiting requested review from gbartolini
@fcanovai fcanovai Awaiting requested review from fcanovai
@armru armru Awaiting requested review from armru
@NiccoloFei NiccoloFei Awaiting requested review from NiccoloFei
1 more reviewer
@danishedb danishedb danishedb approved these changes
Reviewers whose approvals may not affect merge requirements

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Deploy as a Helm chart

4 participants

@kkrypt0nn @raweber42 @ardentperf @danishedb