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

Add warning when generate_lease=true on PKI roles #16398

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

cipherboy
Copy link
Contributor

This option is known to cause problems with large numbers of issued
certificates. Ensure admins are warned about the impact of this field
and encourage them to disable it.

Signed-off-by: Alexander Scheel <alex.scheel@hashicorp.com>

@cipherboy cipherboy force-pushed the cipherboy-add-warning-generate-leases branch from ed3778d to 76a3b53 Compare July 22, 2022 15:11
@cipherboy cipherboy requested a review from a team July 22, 2022 17:37
@cipherboy cipherboy requested a review from sgmiller August 1, 2022 12:43
This option is known to cause problems with large numbers of issued
certificates. Ensure admins are warned about the impact of this field
and encourage them to disable it.

Signed-off-by: Alexander Scheel <alex.scheel@hashicorp.com>
@cipherboy cipherboy force-pushed the cipherboy-add-warning-generate-leases branch from 76a3b53 to 4e0acbe Compare August 3, 2022 21:11
@cipherboy cipherboy enabled auto-merge (squash) August 3, 2022 21:11
Copy link
Contributor

@kitography kitography left a comment

Choose a reason for hiding this comment

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

This looks like a helpful warning :)

@cipherboy cipherboy merged commit a736c12 into main Aug 8, 2022
@cipherboy cipherboy deleted the cipherboy-add-warning-generate-leases branch December 1, 2022 14:58
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.

None yet

3 participants