astefanutti commented on code in PR #3571:
URL: https://github.com/apache/camel-k/pull/3571#discussion_r969243892
##########
pkg/controller/integration/monitor.go:
##########
@@ -57,11 +57,28 @@ func (action *monitorAction) Name() string {
}
func (action *monitorAction) CanHandle(integration *v1.Integration) bool {
+ // When in InitializationFailed condition a kit is not available for
the integration
+ // so the monitor action is not able to handle it.
+ if isInInitializationFailed(integration.Status) {
Review Comment:
How is the integration going to move from error to running phase if it gets
updated to fix the issue? My understanding is that the `monitor` action should
still be applied to check whether the digest has changed? If that understanding
is correct, the `isInInitializationFailed` check should be moved in the
`Handle` method.
--
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: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
