camel-k icon indicating copy to clipboard operation
camel-k copied to clipboard

Add E2E workflow using Windows runner for testing Camel K CLI client and local run

Open tadayosi opened this issue 2 years ago • 4 comments

We release a Windows version of Camel K CLI, but there is no test coverage targeting Windows environment yet. This causes some Windows only issues with the CLI and kamel local commands unnoticed until users try the CLI on Windows, such as #3363.

Since GitHub provides Windows runners for workflows, it should be desirable to have an E2E workflow on Windows that runs part of our E2E suite, especially for the CLI client and the local testing.

tadayosi avatar Jun 16 '22 05:06 tadayosi

cc @phantomjinx

tadayosi avatar Jun 16 '22 05:06 tadayosi

Related issues: #213 #214 #3134

To reuse the existing e2e tests as-is, we are required to resolve those issues above first to make build work on Windows.

Another approach might be using the cross-compiled kamel.exe to run a separate e2e workflow with a dedicated set of Windows tests.

tadayosi avatar Jun 17 '22 04:06 tadayosi

Happy to take this on.

phantomjinx avatar Jun 20 '22 12:06 phantomjinx

This issue has been automatically marked as stale due to 90 days of inactivity. It will be closed if no further activity occurs within 15 days. If you think that’s incorrect or the issue should never stale, please simply write any comment. Thanks for your contributions!

github-actions[bot] avatar Sep 19 '22 00:09 github-actions[bot]

kind cannot be installed on Window runners at the moment.

see https://github.com/actions/runner-images/issues/5760 and https://github.com/microsoft/WSL/issues/6301

phantomjinx avatar Oct 24 '22 09:10 phantomjinx

One idea might be just run workflows that don't require a Kind cluster, such as build and local-it for the time being.

tadayosi avatar Oct 26 '22 04:10 tadayosi

One idea might be just run workflows that don't require a Kind cluster, such as build and local-it for the time being.

Yes. The build workflow is successful but local-it cannot be run at the moment as the base image used for the jdk runtime for the integrations are not available on windows either. I did try overriding the base image using the switch but could not find an image that contained all the correct components.

phantomjinx avatar Oct 26 '22 08:10 phantomjinx

Ah I see. Then maybe we can start small from only running build on Windows. The build runs unit tests so it should be still worth running on Windows.

tadayosi avatar Oct 26 '22 08:10 tadayosi

https://github.com/apache/camel-k/pull/3823 allows for running the build workflow. Does need to be switched on.

phantomjinx avatar Nov 18 '22 12:11 phantomjinx