squakez commented on code in PR #4793:
URL: https://github.com/apache/camel-k/pull/4793#discussion_r1347397830
##########
pkg/controller/integration/monitor.go:
##########
@@ -81,6 +81,12 @@ func (action *monitorAction) Handle(ctx context.Context,
integration *v1.Integra
integration.Status.IntegrationKit.Namespace,
integration.Status.IntegrationKit.Name, err)
}
+ // If integration is in error and its kit is also in error then
integration does not change
+ if isInIntegrationKitFailed(integration.Status) &&
Review Comment:
Yes. If there is no changes in the build, then, it must reuse the
Integration kit. Imagine you change the definition of the Java DSL because you
forgot a `;`. The IntegrationKit was good, but the runtime fails because the
syntax is wrong. You would amend it with such a `;` and it would not need to
rebuild a kit.
--
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
