Skip to content

[Tigron]: x509 helper #4138

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

Merged
merged 2 commits into from
Apr 21, 2025

Conversation

apostasie
Copy link
Contributor

On top of #4137.
Broken out of #4129.

x509 certificates manipulation is a very common test use-case, and in that context it makes sense to provide a helper for it as part of Tigron, leveraging DataTemp for automated resource cleanup, saving the need for the test author to carry over cleanup methods.
Note that the helper is focused first on generating server certificates, though it also provides an escape hatch allowing any x509 template to be signed for peeps who have custom scenarios or not happy about the defaults.

Second commit does remove the existing nerdtest/ca helper and does clean-up code a little bit to adapt to the new utils.Cert.

@apostasie apostasie changed the title [Tigron]: x509 helper [Tigron]: x509 helper Apr 19, 2025
@apostasie
Copy link
Contributor Author

Failure is #4106

@apostasie apostasie force-pushed the tigron-2025-04-x509-helper branch from 1060802 to 5961a9c Compare April 21, 2025 04:36
@AkihiroSuda AkihiroSuda added this to the v2.0.5 milestone Apr 21, 2025
@AkihiroSuda AkihiroSuda added the area/ci e.g., CI failure label Apr 21, 2025
Signed-off-by: apostasie <spam_blackhole@farcloser.world>
Signed-off-by: apostasie <spam_blackhole@farcloser.world>
@apostasie apostasie force-pushed the tigron-2025-04-x509-helper branch from 5961a9c to 15bb28e Compare April 21, 2025 15:35
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit 4f54d76 into containerd:main Apr 21, 2025
54 of 55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci e.g., CI failure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants