Skip to content

Conversation

@fragglehunter
Copy link
Contributor

Updating the cert-manager task with proper trust anchor rotation instructions. Changed inspect_cert function to be able to read bundles

Copy link
Member

@kflynn kflynn left a comment

Choose a reason for hiding this comment

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

Looks really good. I left a few small nits (which should be copied to all the other versions), but mostly it's just embarrassing that I approved the version with the mistakes. 🙁

@kflynn
Copy link
Member

kflynn commented Dec 16, 2025

I forced DCO because @fragglehunter is a Buoyant employee.

Copy link
Member

@kflynn kflynn left a comment

Choose a reason for hiding this comment

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

Looks great, thanks! 🙂

Signed-off-by: Phil Henderson <philhenderson410@gmail.com>
Signed-off-by: Phil Henderson <philhenderson410@gmail.com>
Signed-off-by: Phil Henderson <philhenderson410@gmail.com>
Signed-off-by: Phil Henderson <philhenderson410@gmail.com>
Signed-off-by: Phil Henderson <philhenderson410@gmail.com>
Copy link
Member

@kflynn kflynn left a comment

Choose a reason for hiding this comment

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

Ship it! 🙂

@kflynn kflynn merged commit 0217689 into linkerd:main Dec 16, 2025
7 checks passed
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.

2 participants