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

Updated consul-template version #26863

Merged
merged 3 commits into from
May 7, 2024
Merged

Conversation

divyaac
Copy link
Contributor

@divyaac divyaac commented May 7, 2024

Thought about it, and didn't add a changelog because technically the change was in consul-template, not in Vault? There's a changelog there to reflect that.

If you think it's necessary, I can add a changelog here as well.

@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label May 7, 2024
@divyaac divyaac requested a review from VioletHynes May 7, 2024 16:46
@VioletHynes
Copy link
Contributor

I think it's necessary, since it affects the way Agent works, and some customers may be on the lookout for the bug fix in a Vault version. It's unlikely that Vault customers are looking at consul-template release notes, and they may not be aware that consul template powers Agent's template engine.

We might want to backport this to 1.16, too (since LTS)!

Copy link

github-actions bot commented May 7, 2024

CI Results:
All Go tests succeeded! ✅

Copy link

github-actions bot commented May 7, 2024

Build Results:
All builds succeeded! ✅

Copy link
Contributor

@VioletHynes VioletHynes left a comment

Choose a reason for hiding this comment

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

Looks good! I think we should remove the sentence about consul-template in the changelog but that's non-blocking and optional.

changelog/26863.txt Outdated Show resolved Hide resolved
Co-authored-by: Violet Hynes <[email protected]>
@divyaac divyaac enabled auto-merge (squash) May 7, 2024 17:07
@divyaac divyaac merged commit 57e6795 into main May 7, 2024
83 checks passed
@divyaac divyaac deleted the Update_Consul_Template_Version branch May 7, 2024 17:25
@peteski22 peteski22 added the backport/ent/1.16.x+ent Changes are backported to 1.16.x+ent label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/ent/1.16.x+ent Changes are backported to 1.16.x+ent hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed pr/no-milestone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants