lburgazzoli commented on code in PR #4003:
URL: https://github.com/apache/camel-k/pull/4003#discussion_r1084938611
##########
e2e/namespace/upgrade/cli_upgrade_test.go:
##########
@@ -51,7 +51,7 @@ func TestOperatorUpgrade(t *testing.T) {
Expect(os.Setenv("KAMEL_BIN", kamel)).To(Succeed())
// Should both install the CRDs and kamel in the given namespace
- Expect(Kamel("install", "-n", ns, "--olm=false",
"--force").Execute()).To(Succeed())
+ Expect(Kamel("install", "-n", ns, "--olm=false", "--force",
"--base-image", defaults.BaseImage()).Execute()).To(Succeed())
Review Comment:
I may have missed something but my understanding is that it is needed
because the installation installs the previous version of the operator which
uses adoptopnjdk as base image, then it performs an upgrade.
--
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]
