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

Move generation of CA Configuration into a method on the RuntimeConfig #8363

Merged
merged 1 commit into from
Jul 23, 2020

Conversation

mkeeler
Copy link
Member

@mkeeler mkeeler commented Jul 22, 2020

Previously this was done in the Agent.consulConfig function but we will need this in other places for Agent Auto Configuration so this functionality is being moved to a method on the RuntimeConfig

@mkeeler mkeeler requested a review from a team July 22, 2020 19:04
@mkeeler mkeeler force-pushed the refactor/protobuf branch 2 times, most recently from 48d6d6d to 6d5e062 Compare July 22, 2020 20:44
@freddygv
Copy link
Contributor

Should the commit from refactor/protobuf be here?

Copy link
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.

:shipit:

Copy link
Member

@hanshasselberg hanshasselberg left a comment

Choose a reason for hiding this comment

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

LGTM

agent/structs/connect_ca.go Show resolved Hide resolved
agent/config/runtime.go Outdated Show resolved Hide resolved
agent/config/runtime.go Outdated Show resolved Hide resolved
@mkeeler mkeeler force-pushed the refactor/protobuf branch 4 times, most recently from 6b86435 to aa3709b Compare July 23, 2020 14:33
Base automatically changed from refactor/protobuf to master July 23, 2020 15:24
…thod on the RuntimeConfig

This allows this to be reused elsewhere.
@mkeeler mkeeler merged commit 2ee9fe0 into master Jul 23, 2020
@mkeeler mkeeler deleted the refactor/ca-config-gen branch July 23, 2020 20:05
@hashicorp-ci
Copy link
Contributor

🍒✅ Cherry pick of commit 2ee9fe0 onto release/1.8.x succeeded!

hashicorp-ci pushed a commit that referenced this pull request Jul 23, 2020
…thod on the RuntimeConfig (#8363)

This allows this to be reused elsewhere.
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.

4 participants