Skip to content

Add GVK to post-build error message#1547

Merged
matheuscscp merged 1 commit intofluxcd:mainfrom
maboehm:postbuild-errors
Nov 19, 2025
Merged

Add GVK to post-build error message#1547
matheuscscp merged 1 commit intofluxcd:mainfrom
maboehm:postbuild-errors

Conversation

@maboehm
Copy link
Copy Markdown
Contributor

@maboehm maboehm commented Nov 6, 2025

We recently had a syntax error in an env substitution, but all we saw in the cluster was

post build failed for '${NAME}': envsubst error: variable substitution failed: unable to parse variable name

because in the manifest we used templated the resource name:

metadata:
  name: ${NAME}

The error was quite confusing in that case. In general, the resource name itself might not be enough to pin-point an issue, since is is common to have the same name across e.g. a Deployment and a Service.

@stefanprodan
Copy link
Copy Markdown
Member

Signed-off-by: Marcel Boehm <marcel.boehm@inovex.de>
@matheuscscp matheuscscp added the backport:release/v1.7.x To be backported to release/v1.7.x label Nov 19, 2025
Copy link
Copy Markdown
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @maboehm

@matheuscscp matheuscscp merged commit 5cad388 into fluxcd:main Nov 19, 2025
6 checks passed
@fluxcdbot
Copy link
Copy Markdown
Member

Successfully created backport PR for release/v1.7.x:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:release/v1.7.x To be backported to release/v1.7.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants