Skip to content

Conversation

@rm3l
Copy link
Contributor

@rm3l rm3l commented Oct 31, 2025

Description of the change

As reported in #290, deploying the current Helm Chart might result in a Backstage pod not being able to start due to a config validation error. You can run this command from the main branch to reproduce the issue:

helm install my-backstage ./charts/backstage \
  --set backstage.appConfig.app.baseUrl=https://backstage.mydomain.com \
  --set backstage.appConfig.backend.baseUrl=https://backstage.mydomain.com

To fix this issue (and also prevent future similar issues), I thought it would make sense to include the default app-config.yaml file in the backstage-backend container command, just like it is currently done in the backstage Dockerfile.

Existing or Associated Issue(s)

Fixes #290

Additional Information

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
  • JSON Schema generated.
  • List tests pass for Chart using the Chart Testing tool and the ct lint command.

@rm3l rm3l requested a review from a team as a code owner October 31, 2025 14:43
@rm3l rm3l force-pushed the 290-error-in-logs-for-helm-chart-deployment branch from 289c155 to 8e067b3 Compare October 31, 2025 14:44
@rm3l rm3l changed the title fix(#290): Always include the default app-config.yaml from the Backstage image fix: Always include the default app-config.yaml from the Backstage image Oct 31, 2025
@rm3l rm3l changed the title fix: Always include the default app-config.yaml from the Backstage image fix: Always include the default app-config.yaml from the Backstage image in the container command Nov 2, 2025
@rm3l rm3l force-pushed the 290-error-in-logs-for-helm-chart-deployment branch from 8e067b3 to 21d5345 Compare November 9, 2025 17:55
@rm3l rm3l force-pushed the 290-error-in-logs-for-helm-chart-deployment branch from 21d5345 to f6c068d Compare November 9, 2025 17:58
@rm3l
Copy link
Contributor Author

rm3l commented Nov 12, 2025

/cc @ChrisJBurns Thoughts on this? This is confirmed to fix issues like #290
Thanks.

@ChrisJBurns
Copy link
Contributor

@vinzscam Wondering what your thoughts are on this? I always defer to yourself when it comes to some of the app-config changes 😄

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.

Error in logs for Helm Chart deployment

2 participants