Fix tests related to recent camel-k-runtime changes

[claudio4j]

The following PR brought knative from camel-quarkus
apache/camel-k-runtime#927

There is an additional merged PR in camel-k-runtime
apache/camel-k-runtime#933

Release Note

Kamelet: In Yaml dsl the `template` keyword was renamed to `routeTemplate`

[claudio4j]

The failing test TestRuntimeContainsEmbeddedArtifacts is occurring because camel-k-maven-plugin-1.16.0-SNAPSHOT artifact doesn't contain the fix. Probably next camel-k-runtime snapshot build will contain the fix.

[tadayosi]

@claudio4j There are a bit more places where they still use camel-k-knative in E2E tests:

Error: 4: > {"level":"error","ts":1669269776.4805422,"logger":"camel-k.maven.build","msg":"[ERROR] Some problems were encountered while processing the POMs:\n[ERROR] 'dependencies.dependency.version' for org.apache.camel.k:camel-k-knative:jar is missing. @ line 25, column 17\n @ ","stacktrace":"github.com/apache/camel-k/pkg/util/log.Logger.Error\n\tgithub.com/apache/camel-k/pkg/util/log/log.go:75\ngithub.com/apache/camel-k/pkg/util/maven.normalizeLog\n\tgithub.com/apache/camel-k/pkg/util/maven/maven_log.go:83\ngithub.com/apache/camel-k/pkg/util/maven.mavenLogHandler\n\tgithub.com/apache/camel-k/pkg/util/maven/maven_log.go:54\ngithub.com/apache/camel-k/pkg/util.scan\n\tgithub.com/apache/camel-k/pkg/util/command.go:77\ngithub.com/apache/camel-k/pkg/util.RunAndLog.func1\n\tgithub.com/apache/camel-k/pkg/util/command.go:56\ngolang.org/x/sync/errgroup.(*Group).Go.func1\n\tgolang.org/x/sync@v0.1.0/errgroup/errgroup.go:75"}

Like:

$ ag camel-k-knative e2e/
e2e/global/builder/build_test.go
54:			"camel-k-knative",

e2e/global/knative/knative_platform_test.go
50:		operatorID := "camel-k-knative"

[squakez]

I've restarted the checks. At this time, the new snapshot is released. For the future: we need to wait that the camel k runtime PR is merged into main and also wait that the build is run once that PR is merged.

[squakez]

@oscerd @claudio4j it seems something is failing when running in native mode.

[squakez]

Some other issue is related to the change in the yaml specification: https://camel.apache.org/manual/camel-3x-upgrade-guide-3_19.html#_camel_yaml_dsl

[claudio4j]

I am having a look at these tests, thanks @squakez @tadayosi

[tadayosi]

Thanks!

[tadayosi]

About this upgrade failure:
https://github.com/apache/camel-k/actions/runs/3545561937/jobs/5953803406

      cli_upgrade_test.go:94: 
          Timed out after 60.020s.
          Expected
              <string>: 
          to equal
              <string>: 1.11.0-SNAPSHOT

It could be that using the default timeout (60s) for this line is too short for snapshot dependencies:
https://github.com/apache/camel-k/blob/main/e2e/namespace/upgrade/cli_upgrade_test.go#L94

[squakez]

Just a comment. Could you try keeping the usage of template but as a single parameter instead of a list?

[claudio4j]

It could be that using the default timeout (60s) for this line is too short for snapshot dependencies: https://github.com/apache/camel-k/blob/main/e2e/namespace/upgrade/cli_upgrade_test.go#L94

Agree, let's merge and see next CI test results.

[oscerd]

Let's merge, so we can try to release this week.