Closed stephanhorsthemke closed 1 year ago
The Kubernetes project currently lacks enough contributors to adequately respond to all issues.
This bot triages un-triaged issues according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/remove-lifecycle stale
/close
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues.
This bot triages un-triaged issues according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/remove-lifecycle rotten
/close
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle rotten
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.
This bot triages issues according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/reopen
/remove-lifecycle rotten
Please send feedback to sig-contributor-experience at kubernetes/community.
/close not-planned
@k8s-triage-robot: Closing this issue, marking it as "Not Planned".
What happened: I am not sure if it is a bug or a feature but I realised that if Preflight fails,
Reconciled
is gonna run which is at least unexpected considering the name "Reconciled" which should be only running if Reconcile ran which did not happen if Preflight fails.Ultimately the problem is that objects is only populated after Preflight so that parameter passed into
Reconciled
is nil in this case. I believe this happened to be also withkStatusAggregated
but I am experiencing this now with our custom Reconciled funtion.What you expected to happen: If Preflight fails then Reconciled is not executed
How to reproduce it (as minimally and precisely as possible): Fail Preflight and use a Reconciled Function which e.g. loops through the objects without checking for nil.
Anything else we need to know?: I believe an easy fix (an idea) would be to move the defer to run Reconciled after the Preflight