[GitHub] [camel-k] christophd commented on a diff in pull request #3223: chore: Log build duration metrics

[GitBox <gi...@apache.org>]

christophd commented on code in PR #3223:
URL: https://github.com/apache/camel-k/pull/3223#discussion_r853887956


##########
pkg/controller/build/action.go:
##########
@@ -61,3 +64,14 @@ func (action *baseAction) InjectLogger(log log.Logger) {
 func (action *baseAction) InjectRecorder(recorder record.EventRecorder) {
 	action.recorder = recorder
 }
+
+func (action *baseAction) findCreator(ctx context.Context, build *v1.Build) *corev1.ObjectReference {
+	var kit v1.IntegrationKit
+
+	// Fetch the IntegrationKit instance
+	if err := action.client.Get(ctx, types.NamespacedName{Name: build.Name, Namespace: build.Namespace}, &kit); err == nil {

Review Comment:
   For now I wanted to have a non-offensive change to not break the reconcile loop just for logging reasons. I mean the result of this is only used for logging reasons.



-- 
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-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org