Skip to content

Net 4451/windows dataplane fix#188

Merged
aahel merged 8 commits intomainfrom
NET-4451/windows-dataplane-fix
Jul 7, 2023
Merged

Net 4451/windows dataplane fix#188
aahel merged 8 commits intomainfrom
NET-4451/windows-dataplane-fix

Conversation

@aahel
Copy link
Contributor

@aahel aahel commented Jun 28, 2023

this pr is based on the changes from this pr.

  • Switch bootstrap config file from a named Unix pipe to a tempfile, since there shouldn't be anything sensitive in the bootstrap config file
  • Remove x-consul-token from the bootstrap config template since the token is injected into the gRPC stream by an interceptor

Testing method :-
Have tested this building building a new windows dataplane image and deploying consul along with dashboard and counting service in a gke windows cluster.

@aahel aahel requested a review from a team as a code owner June 28, 2023 08:42
@aahel aahel marked this pull request as draft June 28, 2023 08:44
@aahel aahel requested review from pglass and removed request for a team June 28, 2023 16:49
@aahel aahel marked this pull request as ready for review June 28, 2023 16:54
@aahel aahel changed the base branch from windows-preview to main June 28, 2023 17:07
@aahel aahel force-pushed the NET-4451/windows-dataplane-fix branch from 7e434e1 to 9bf1b28 Compare June 28, 2023 17:16
@aahel aahel force-pushed the NET-4451/windows-dataplane-fix branch from 9bf1b28 to e3e3dd3 Compare July 3, 2023 04:16
@aahel aahel requested a review from pglass July 3, 2023 04:28
Copy link

@pglass pglass left a comment

Choose a reason for hiding this comment

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

Approved. Feel free to address the changelog comment how you like.

Co-authored-by: Paul Glass <pglass@hashicorp.com>
@aahel aahel merged commit 2e4715d into main Jul 7, 2023
@aahel aahel deleted the NET-4451/windows-dataplane-fix branch July 7, 2023 17:46
@aahel aahel self-assigned this Dec 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants