-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
client: interpolate meta blocks with task environment #10876
Conversation
aa10d20
to
47e8f0b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the services version of this, we interpolate both the key and the value of a meta
block. Do we want to do the same here?
https://github.com/hashicorp/nomad/blob/main/client/taskenv/services.go#L165
goes to
nomad/client/taskenv/services.go
Line 70 in 72f431f
m[taskEnv.ReplaceEnv(k)] = taskEnv.ReplaceEnv(v) |
Adds missing interpolation step to the `meta` blocks when building the task environment. Also fixes incorrect parameter order in the test assertion and adds diagnostics to the test.
47e8f0b
to
1cc1bda
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Adds missing interpolation step to the `meta` blocks when building the task environment. Also fixes incorrect parameter order in the test assertion and adds diagnostics to the test.
Adds missing interpolation step to the `meta` blocks when building the task environment. Also fixes incorrect parameter order in the test assertion and adds diagnostics to the test.
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |
Fixes #6506 so our
meta
docs are correct.Adds missing interpolation step to the
meta
blocks when building the taskenvironment. Also fixes incorrect parameter order in the test assertion and
adds diagnostics to the test.