Skip to content

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