-
-
Notifications
You must be signed in to change notification settings - Fork 102
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
Support and use docker builds on Linux/s390x #2731
Support and use docker builds on Linux/s390x #2731
Comments
Base image is currently problematic as UBI is not currently a practical alternative due to missing prereqs in the default repositories such as the X11 packages. CentOS is not available on s390x. It May be more feasible when #2008 is finalised (allow us to run RHEL in the container) or use an alternate sysroot for the build. |
Our ROSI subscription allows me to create a suitable image, however we would likely not be able to push a RHEL image up to dockerhub. A suitable dockerfile for creating the image is in #2926 and requires a suitable Red Hat subscription credentials to be passed to it
|
Currently the build process cannot use the image that is on the two build machines as it expects to be able to perform a |
@andrew-m-leonard - I may need some expert assistance on this. I've tried using my branch that should skip the pull at https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-linux-s390x-temurin/178/console with my own repository (based on your new wiki page saying what to edit!) and the docker image set to |
@sxa Your pipeline has useAdoptBashScripts set to True, set this to False and it will generate the required USER_REMOTE_CONFIG correctly. |
Thanks - I thought I'd picked up on that and done a re-run but apparently not 👍🏻 |
https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-linux-s390x-temurin/181/console is the job that shows the problem I was describing where it seems to be skipping the pull despite (I think) being run with the correct options to pull in my |
It's still using AdoptBuildScripts=true, i've kicked off a new pipeline: |
Looks good:
|
Thanks - I'm getting true and false the the wrong way round because I'm too tired to read properly apparently :-) I'm re-running the build job at https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-linux-s390x-temurin/183/console with the docker image option to test the functionality I'm looking for. |
@andrew-m-leonard That job seems to have failed - could you take a look and see if there's an obvious reason for it as the error message is somewhat opaque. |
@sxa yeah, particularly opaque! I've started a new one to see if repeated: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-linux-s390x-temurin/186/ |
@sxa Looks like something is hanging in the container, can you add some debug? or can you try running it manually and see what is hanging? |
I previously tested this on the machines with the container without problems - it's only started being a problem when run from jenkins. Log from doing it again manually:
|
Hmmm may be related to #2834 which was an issue with the UID of jenkins in the container doesn't match that of the host. |
OK yeah this is a UID mismatch issue between the host and docker image - @Haroon-Khel do we have an issue to look at the options for resolving this permanently? I've run it with a patched docker image at https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk17u/job/jdk17u-linux-s390x-temurin/188/console and it seems to be running through as expected. |
Features that impact the whole project (e.g. Adding a new OpenJDK distribution)are made over at the adoptium.
Otherwise, please describe what enhancement you would like to see in the build
scripts:
In order to improve isolation of build environments, I'd like to try moving the s390x builds into docker containers. We now have a good range of machines that would be suitable for this as part of the work done in #2673 and therefore this is now more feasible. This will need:
The text was updated successfully, but these errors were encountered: