Skip to content

Update bootstrap configuration for HCP Telemetry#122

Merged
chapmanc merged 4 commits into
mainfrom
chappie/bootstrapconfig/update
May 25, 2023
Merged

Update bootstrap configuration for HCP Telemetry#122
chapmanc merged 4 commits into
mainfrom
chappie/bootstrapconfig/update

Conversation

@chapmanc
Copy link
Copy Markdown
Contributor

Ran make copy-bootstrap-config to update the consul config

@chapmanc chapmanc requested a review from a team as a code owner May 25, 2023 17:03
@chapmanc chapmanc added the backport/1.1 Changes are backported to 1.1 label May 25, 2023
@david-yu
Copy link
Copy Markdown
Contributor

david-yu commented May 25, 2023

This may also need to be backported to 1.0.x? #90 was back ported to 1.0.x

@chapmanc chapmanc added backport/1.0 backport/1.1 Changes are backported to 1.1 and removed backport/1.0 backport/1.1 Changes are backported to 1.1 labels May 25, 2023
Copy link
Copy Markdown
Contributor

@freddygv freddygv left a comment

Choose a reason for hiding this comment

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

LGTM, one minor comment about the changelog entry

Comment thread .changelog/122.txt Outdated
@@ -0,0 +1,3 @@
```release-note:improvement
Update bootstrap configuration.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can this mention the rename, since there's a previous changelog entry about it?
https://github.com/hashicorp/consul-dataplane/blob/main/.changelog/90.txt

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure will do!

@chapmanc chapmanc merged commit 5076560 into main May 25, 2023
@chapmanc chapmanc deleted the chappie/bootstrapconfig/update branch May 25, 2023 19:54
@curtbushko curtbushko changed the title Update bootstrap configuration Update bootstrap configuration for HCP Telemetry May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/1.1 Changes are backported to 1.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants