lburgazzoli commented on code in PR #5246:
URL: https://github.com/apache/camel-k/pull/5246#discussion_r1524578978


##########
pkg/util/defaults/defaults.go:
##########
@@ -44,5 +44,10 @@ const (
        installDefaultKamelets = true
 )
 
+// Maps base image variants to their respective digests
+var baseImageVariantMapping = map[string]string{

Review Comment:
   This may hijack a little bit the issue but this comment about runtime lookup 
of the right base image make me thinking if we should do it for every 
architecture and having the `IntegrationPlatform` reflecting the images in its 
status.
   
   The reason for that is something we never have discussed but it should be 
possible to have a mixed kubernetes set-up where some nodes are amd64 and some 
arm64 so you may have some kits working for some nodes and not for some others.
    



-- 
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

Reply via email to