Skip to content

Fix .helmignore example pattern matching template directories#2005

Open
Copilot wants to merge 4 commits intomainfrom
copilot/fix-helm-ignore-file-example
Open

Fix .helmignore example pattern matching template directories#2005
Copilot wants to merge 4 commits intomainfrom
copilot/fix-helm-ignore-file-example

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 15, 2026

The example pattern */*/temp* matches any path segment containing "temp", including templates/. This breaks nested chart rendering when users copy the example.

Changes

  • Replace temp*draft* in glob pattern examples
  • Update comment: "Match draft files two levels deep (e.g., foo/bar/draft123)"
  • Applied to all doc versions and i18n (en, ko, uk, zh)

Example

Before:

# Match any file under subdir matching temp*
*/temp*

*/*/temp*
temp?

After:

# Match any file under subdir matching draft*
*/draft*

# Match draft files two levels deep (e.g., foo/bar/draft123)
*/*/draft*
draft?

The new pattern demonstrates the same glob matching behavior without colliding with Helm's templates/ directory.

Original prompt

This section details on the original issue you should resolve

<issue_title>Helm ignore file temp example</issue_title>
<issue_description>In the .helmignore file example (https://helm.sh/docs/chart_template_guide/helm_ignore_file/), there are a couple of lines in the example file that caused issues in our nested Helm charts:

`# Match any file under subdir matching temp*
/temp

//temp*`

These lines caused our nested Helm charts to not render correctly as the //temp* matched the subchart templates folder. Took quite a bit of head scratching to figure this out.

Thank you!</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Replace temp* patterns with draft* patterns to prevent inadvertently matching subchart template folders. The pattern */*/temp* could match paths like charts/mychart/templates/, causing nested Helm charts to fail to render correctly.

Changes applied to:
- Main documentation (docs/)
- Versioned documentation (version-3/)
- Internationalized versions (ko, uk, zh)

Fixes issue where */*/temp* pattern matched subchart templates folders

Co-authored-by: TerryHowe <104113+TerryHowe@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix helm ignore file example to prevent rendering issues Fix .helmignore example pattern matching template directories Feb 15, 2026
Copilot AI requested a review from TerryHowe February 15, 2026 13:20
Copy link
Copy Markdown
Contributor

@TerryHowe TerryHowe left a comment

Choose a reason for hiding this comment

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

/lgtm

I'm coauthor though.

@TerryHowe TerryHowe marked this pull request as ready for review February 15, 2026 15:39
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.

Helm ignore file temp example

2 participants