Skip to content

Updated Helm Install command with create namespace flag#213

Open
ShubhamTatvamasi wants to merge 1 commit intoportainer:masterfrom
ShubhamTatvamasi:updated-chart-readme
Open

Updated Helm Install command with create namespace flag#213
ShubhamTatvamasi wants to merge 1 commit intoportainer:masterfrom
ShubhamTatvamasi:updated-chart-readme

Conversation

@ShubhamTatvamasi
Copy link
Copy Markdown
Member

@ShubhamTatvamasi ShubhamTatvamasi commented Mar 26, 2026

Updated Portainer chart docs to prefer Helm-native namespace creation (--create-namespace) and remove duplicate explicit kubectl create namespace command.

https://artifacthub.io/packages/helm/portainer/portainer

Copilot AI review requested due to automatic review settings March 26, 2026 08:49
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Portainer Helm chart README to streamline namespace setup by recommending Helm’s --create-namespace flag and reflecting it in install/upgrade examples.

Changes:

  • Replaced the “pre-create namespace” instructions with guidance to use --create-namespace (or create via kubectl).
  • Added --create-namespace to the testing and installation Helm commands.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


```bash
helm install --dry-run --debug portainer -n portainer deploy/helm/portainer
helm install --dry-run --debug --create-namespace -n portainer portainer deploy/helm/portainer
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

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

The dry-run example includes --create-namespace, but Helm does not create the namespace when --dry-run is set (it exits before the create-namespace logic). This can mislead readers into thinking the namespace will be created during testing; consider removing --create-namespace from the dry-run command or adding a note that it only takes effect on a real install/upgrade.

Suggested change
helm install --dry-run --debug --create-namespace -n portainer portainer deploy/helm/portainer
helm install --dry-run --debug -n portainer portainer deploy/helm/portainer

Copilot uses AI. Check for mistakes.
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