Skip to content

Conversation

@weyfonk
Copy link
Contributor

@weyfonk weyfonk commented Sep 26, 2024

This adds the fleet_ prefix to Kubernetes events generated by the gitjob controller, to clarify their context.

It also lowers verbosity of log messages indicating creation, deletion or reconciliation of GitRepos, making them visible when debug logging is disabled.

@weyfonk weyfonk requested a review from a team as a code owner September 26, 2024 14:11
@weyfonk weyfonk force-pushed the events-add-fleet-prefix branch from 4c38163 to 35a5d8c Compare September 27, 2024 11:51
This will help users find events produced by Fleet, whether through the
Rancher UI or `kubectl` for instance.
This makes reconciliation, resource creation, deletion and status update
errors for `GitRepo`s more visible, ensuring that they appear in logs
even when debug logging is disabled.
This will help map logs on `GitRepo`s, produced by the `gitjob`
reconciler, with those on corresponding bundles from their reconciler.
This will help map logs referring to a bundle with those referring to
its `GitRepo`, across the `fleet` and `gitjob` controllers.
This clarifies what events are about, as concisely as possible,.
@weyfonk weyfonk force-pushed the events-add-fleet-prefix branch from 9db6999 to e9ba3a2 Compare September 27, 2024 13:34
@manno manno merged commit 6000349 into rancher:main Sep 27, 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