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

When using templates, allow carriage returns (or offer the ability to remove them) #4802

Open
aronchick opened this issue Jan 8, 2025 · 1 comment
Labels
request/new Request: Indicates a new request that has been submitted and awaits initial triage type/enhancement Type: New features or enhancements to existing features

Comments

@aronchick
Copy link
Collaborator

aronchick commented Jan 8, 2025

E.g.

foo: {{ .bar }}
qaz: cous

If i do the following:

val="123
456"
bacalhau job run template_job.yaml --template-vars="bar=$val"

Bad things will happen :)

Error: the job provided is invalid: converting yaml to json: yaml: line 11: could not find expected ':'

IDEALLY we could just allow templating (not sure if this works).

In a slightly less ideal world, we could allow marking the mustache to eliminate the carriage return. E.g.

foo: {{- .bar -}}
qaz: cous

Or whatever.

@aronchick aronchick added type/enhancement Type: New features or enhancements to existing features request/new Request: Indicates a new request that has been submitted and awaits initial triage labels Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
request/new Request: Indicates a new request that has been submitted and awaits initial triage type/enhancement Type: New features or enhancements to existing features
Projects
None yet
Development

No branches or pull requests

1 participant