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

[v0.10] Add integration test checks for CA bundle secret #2863

Merged

Conversation

weyfonk
Copy link
Contributor

@weyfonk weyfonk commented Sep 17, 2024

Backport of #2846 to release/v0.10.
Refers to #2829.

When a git job is created for a GitRepo with a configured CA bundle, integration tests now validate that a secret is created with the contents of that bundle and validate its contents against the GitRepo spec.

This also validates through integration tests that RBAC resources and the CA bundle secret are owned by their GitRepo and will therefore be deleted as soon as the GitRepo is.
Validation does not actually delete anything, due to a limitation 1 of the test framework.

When a git job is created for a `GitRepo` with a configured CA bundle,
integration tests now validate that a secret is created with the
contents of that bundle and validate its contents against the `GitRepo`
spec.

This also validates through integration tests that RBAC resources and
the CA bundle secret are owned by their `GitRepo` and will therefore
be deleted as soon as the `GitRepo` is.
Validation does not actually delete anything, due to a limitation [1] of
the test framework.

[1]: https://book.kubebuilder.io/reference/envtest#testing-considerations
@weyfonk weyfonk requested a review from a team as a code owner September 17, 2024 14:20
Copy link
Contributor

@0xavi0 0xavi0 left a comment

Choose a reason for hiding this comment

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

lgtm

@weyfonk weyfonk merged commit 9d6e8ad into rancher:release/v0.10 Sep 18, 2024
12 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