-
-
Notifications
You must be signed in to change notification settings - Fork 101
unixPB: Add UBI10 static dockerfile #4044
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
Conversation
|
Im going to run this pr in the static container deployment task in AWX. If it runs without error its good to go |
8069529 to
cf7ca70
Compare
Signed-off-by: Stewart X Addison <[email protected]>
cf7ca70 to
def94b0
Compare
|
|
@Haroon-Khel Please pick up the latest version as the JDK extract had the wrong |
|
Also noting that this should not be made live with a |
|
Latest awx run |
|
The awx job deploys the container fine, but the container does not stay up Maybe try it without Also our other containers use |
Hmm it's a default if you don't specify a command when you run the container. That woulld suggest to me that perhaps the ssh startup isn't working - which I haven't been testing. I have noticed that some of the containers didn't have the default entry point so I've had to use |
Signed-off-by: Stewart X Addison <[email protected]>
dockerhost-osuosl-ubuntu2404-aarch64-1 |
|
Fixed by latest commit which removes the |
|
Yep now it stays up |
Great - thanks for testing! So now I just have to confirm from the AQA_Test_Pipeline runs whether either the installation of a JRE vs JDK for the agent, and the absence of |
Signed-off-by: Stewart X Addison <[email protected]>
Signed-off-by: Stewart X Addison <[email protected]>
|
I don't know why but on my laptop with |
|
Need an extra commit to create /tmp/.X11-unix otherwise weston starts but won't allow things to connect (Ideally we'd move that location ..) |
Signed-off-by: Stewart X Addison <[email protected]>
|
Taking this out of draft. I'm still not sure what the issue is with my x64 podman system, but it's reproducible with ubi10 with the minimal dockerfile below. Switch the first line to reference ubi9 and it works fine. @Haroon-Khel Can you do one final check with this PR in AWX and confirm that you're still happy with it please? |
Signed-off-by: Stewart X Addison <[email protected]>
steelhead31
left a comment
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.
Looks good.
|
Another option from the which still force a shutdown of the process holding the lock |
|
Latest check https://awx.adoptium.net/#/jobs/playbook/3019/output |
|
@sxa The latest awx check ran successfully and the container is up and running |
Part of #3868
Relative to earlier versions this also implements points 1, 2, 4, 5 of #4043 (comment) (Albeit with point 1 using JDK21 instead of 25 for now until it GAs), plus some moving about of the control stuff like
ENTRYPOINTto the end.Log of running this Dockerfile to show time stamps - JDK download/install is under four seconds
The eagle-eyed among you will note that this has had
x64andx86-64switched toaarch64for this test ;-)Most of the time is spent installing
perland the other prereqs required for the testing.An AQA_Test_Pipeline run with JDK8+25 excluding
extended.openjdkand the system tests is at https://ci.adoptium.net/job/AQA_Test_Pipeline/457 (One last hurrah for our Equinix aarch64 machines!)Checklist