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

ca: replace ca.PrimaryIntermediateProviders #10476

Merged
merged 1 commit into from
Jun 24, 2021

Conversation

dnephin
Copy link
Contributor

@dnephin dnephin commented Jun 23, 2021

With an optional interface that providers can use to indicate if they
use an intermediate cert in the primary DC.

This removes the need to look up the provider config when renewing the
intermediate.

With an optional interface that providers can use to indicate if they
use an intermediate cert in the primary DC.

This removes the need to look up the provider config when renewing the
intermediate.
@dnephin dnephin added theme/connect Anything related to Consul Connect, Service Mesh, Side Car Proxies theme/certificates Related to creating, distributing, and rotating certificates in Consul labels Jun 23, 2021
@dnephin dnephin requested a review from a team June 23, 2021 19:49
@github-actions github-actions bot removed the theme/certificates Related to creating, distributing, and rotating certificates in Consul label Jun 23, 2021
@dnephin dnephin added the pr/no-changelog PR does not need a corresponding .changelog entry label Jun 23, 2021
Copy link
Member

@rboyer rboyer left a comment

Choose a reason for hiding this comment

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

LGTM

@dnephin dnephin merged commit bb37c4d into master Jun 24, 2021
@dnephin dnephin deleted the dnephin/ca-primary-uses-intermediate branch June 24, 2021 18:05
@hc-github-team-consul-core
Copy link
Contributor

🍒 If backport labels were added before merging, cherry-picking will start automatically.

To retroactively trigger a backport after merging, add backport labels and re-run https://circleci.com/gh/hashicorp/consul/396909.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-changelog PR does not need a corresponding .changelog entry theme/connect Anything related to Consul Connect, Service Mesh, Side Car Proxies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants