squakez commented on code in PR #4942:
URL: https://github.com/apache/camel-k/pull/4942#discussion_r1410231061
##########
pkg/cmd/operator/operator.go:
##########
@@ -188,7 +188,6 @@ func Run(healthPort, monitoringPort int32, leaderElection
bool, leaderElectionID
selector := labels.NewSelector().Add(*hasIntegrationLabel)
selectors := map[ctrl.Object]cache.ByObject{
- &corev1.Pod{}: {Label: selector},
Review Comment:
Not an option. Altough in principle the idea is good there is a practical
problem with the possible status of the Pods at any time. For example, we are
correctly reported that a Pod is being rescheduled via the Deployment event
change, however, we don't catch correctly the final status, because it starts
`Running` (which we detect correctly during the Integration monitoring action)
but later can move to a different status (which we can only get watching the
Pods events).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
