Skip to content

bug: fix helmfile parsing error for default env #68

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nvanthao
Copy link
Member

@nvanthao nvanthao commented May 22, 2025

What

As of now running task helm-install will show

in ./helmfile.yaml.gotmpl: error during helmfile.yaml.gotmpl.part.1 parsing: template: stringTemplate:5:25: executing "stringTemplate" at <.Values.username>: map has no entry for key "username"
task: Failed to run task "helm-install": exit status 1

This occurs because no username is configured for default environment.

Changes

  • Add username and password to default env

Before

environments:
  default:
    values:
      - chartSources:
          certManager: ./charts/cert-manager
          certManagerIssuers: ./charts/cert-manager-issuers

After

environments:
  default:
    values:
      - username: "test@example.com"
      - password: '{{env "REPLICATED_LICENSE_ID"}}'
      - chartSources:
          certManager: ./charts/cert-manager
          certManagerIssuers: ./charts/cert-manager-issuers

Testing

  • Run task helm-install works

Impact

This change fixes the template parsing error and ensures that the credentials are properly structured for use in the Helm templates. No changes to the actual functionality are required, only the structure of the configuration.

@nvanthao nvanthao force-pushed the gerard/b-missing-helmfile-values branch from 213a4da to 7c2111d Compare May 23, 2025 00:46
Copy link
Member

@0xJMart 0xJMart left a comment

Choose a reason for hiding this comment

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

@nvanthao Does this still happen? We're able to do the helm install with no issues.

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.

2 participants