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

feat: Enable sprig template function library to webhook template render #445

Merged
merged 1 commit into from
Jun 1, 2021

Conversation

toninis
Copy link
Contributor

@toninis toninis commented May 28, 2021

Description

With this PR we enable sprig functions template library to the Webhook Template.
This functionality is really helpful to our case as we are using the same configuration to multiple clusters with different versions and the node labels are not always the same.

i.e.

Instance Type:* `{{default (index .NodeLabels "node.kubernetes.io/instance-type") (index .NodeLabels "beta.kubernetes.io/instance-type") }}

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@bwagner5 bwagner5 self-requested a review May 28, 2021 15:52
@bwagner5
Copy link
Contributor

Hello! Thanks for contributing to NTH! I've glanced at the changes in the PR and everything looks pretty good, but please allow us some extra time to review since it does bring in several dependencies that we'll need to look into (we'll also be out until June 2nd for a US Holiday, Memorial Day). Thanks for your patience!

@imuqtadir
Copy link
Contributor

LGTM! Thanks for the PR once again

Copy link
Contributor

@bwagner5 bwagner5 left a comment

Choose a reason for hiding this comment

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

lgtm

@bwagner5
Copy link
Contributor

bwagner5 commented Jun 1, 2021

I'll merge this PR at the same time the license PR is merged: #446

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.

3 participants