Alvaro Aleman

While the question of posting a comment on every failure is pretty controversial, I was wondering what ppl think about "Only include jobs that ran against the latest commit" in...

For posterity, one workaround for "I want my pod to rotate when config changes" is to put the config in a pod annotation and then use the downward api to...

> Wouldn’t it then be simpler to put the config into an environment variable? Yeah, but many applications do not support reading their config from an env var, the downward...

@DirectXMan12 So what I had in mind would be something like: * Add a new `pkg/manager/fake` that exposes a `NewFakeManager(obj ...runtime.Object)` * All methods on that `manager` except for `GetClient`...

For posterity, I did built something for this out-of tree some time ago that might be used as inspiration to add this to controller-runtime directly: https://github.com/kubernetes/test-infra/blob/682727078f0c86e556d9d766bd4ffbf3e2980a56/prow/tide/tide_test.go#L3836-L3923

It should also be mentioned that the "interact with cluster" part has been decoupled form the "run stuff" part and moved into pkg/cluster from pkg/manager in the meantime. So having...

Not really, the problem is that the reconcile.Request get put into a workqueue that does de-duplication, so custom information like that can not be preserved (or at least not with...

Just use watches and wrap the handler there, no reason to change the builder

> The TypeMeta is properly populated when getting objects from the client. It is? I thought that is only the case for metav1.PartialObject where we have a wrapper that re-adds...

> Does this need to run every time something gets the leader, or is it a "only one thing needs to run this, and it needs to be run before...