squakez commented on code in PR #5200:
URL: https://github.com/apache/camel-k/pull/5200#discussion_r1507122645
##########
pkg/cmd/install.go:
##########
@@ -414,6 +415,24 @@ func (o *installCmdOptions) setupOperator(
platformName)
}
+ // Choose an architecture specific operator/base image when not already
given explicitly
+ //
+ arch, err := kubernetes.NodeArchLookup(o.Context, c)
Review Comment:
Adding any code it means maintenance burden in the long run to the people
maintaining the project. We must be careful to add only whatever is meant to be
supported. We have a procedure to install the ARM based architecture [1] and
beside that the final solution has been already identified and provided in a
comment above as well.
[1]
https://camel.apache.org/camel-k/2.2.x/installation/advanced/multi-architecture.html
--
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]
