Skip to content
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

[Bug]: extraObjects accepts only string as templates #538

Open
oncipriani opened this issue Jan 22, 2024 · 1 comment
Open

[Bug]: extraObjects accepts only string as templates #538

oncipriani opened this issue Jan 22, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@oncipriani
Copy link

What happened?

I tried to use the "extraObjects" to create a Pod Disruption Budget object, but I could not find any way to add the proper labels, because apparently all templated values inside the "extraObjects" list must be quoted. That means that only simple string literals can be generated by the templates.

Even the example extrObject provided in the documentation does not work.

See #240 for more details.

Steps to reproduce

  1. Create a values.yml with at least one extraObject element containing at least one non-quoted template;
  2. Try to render the chart using helm template.

Expected behavior

The chart renders correctly, including the extra objects provided.

Relevant log output

No response

Screenshot

No response

Additional context

No response

Jaeger backend version

1.51.0

SDK

No response

Pipeline

No response

Stogage backend

No response

Operating system

No response

Deployment model

No response

Deployment configs

No response

@oncipriani oncipriani added the bug Something isn't working label Jan 22, 2024
@Tomy8s
Copy link

Tomy8s commented May 30, 2024

I made the same comment in #240, where you included an example, but this comment is more relevant to this issue.

You're seeing that error as values files must be valid YAML before being parsed by helm. The example in README.md was not valid YAML, and therefore did not work (I fixed this in #580). If you need to use helm templating in the variables file, you'll need to do something that still renders as valid YAML; sometimes no change is needed, sometimes adding quotes is enough, sometimes ugly hacks are necessary (as in your example in #240), possibly there are occasions where it is not possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants