-
Notifications
You must be signed in to change notification settings - Fork 91
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Try to do the Docker build in CI #128
Conversation
} else { | ||
return TestPlatform.values(); | ||
} | ||
return TestPlatform.values(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trying to follow the decisions in #106 & #121. If Docker is not working but we are in CI (typically because we are running in Windows), native image generation will be skipped…as will this whole test:
durable-task-plugin/src/test/java/org/jenkinsci/plugins/durabletask/BourneShellScriptTest.java
Line 112 in 6b6a88e
assumeTrue("Docker is available", new Docker().isAvailable()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On maven-windows
:
[INFO] Running org.jenkinsci.plugins.durabletask.BourneShellScriptTest
[WARNING] Tests run: 1, Failures: 0, Errors: 0, Skipped: 1, Time elapsed: 0.011 s - in org.jenkinsci.plugins.durabletask.BourneShellScriptTest
As expected?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And mostly passing now on Linux, with just a couple skips, though BourneShellScriptTest
is really slow.
Hmm, Docker build is passing in CI:
So I wonder what is going on with my machine (Ubuntu): $ docker -v
Docker version 19.03.8, build afacb8b7f0 |
Now seems to be broken on Windows because we cannot even run shell scripts at all. |
Local failure was due to some sort of networking problem in Docker, since resolved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps it would be better to move the Go sources to another repo, say with Docker automatic builds, and consume them in this repo as a binary.
That is definitely the plan moving forward!
And mostly passing now on Linux, with just a couple skips, though BourneShellScriptTest is really slow.
It definitely is a beast. I'm not sure if the parameterized tests are overkill, but given all the different environments that it runs on and how any little change could affect some environment, maybe it's for the best?
Yeah I would not advocate reducing test coverage, this is critical, just wondering if there is some obvious inefficiency here. |
closing, since docker builds have now been moved to |
Trying to restore Docker builds lost after 17cac5e. Wondered whether the build is actually working for anyone else, since locally I am getting
Perhaps it would be better to move the Go sources to another repo, say with Docker automatic builds, and consume them in this repo as a binary.