Skip to content
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

Add perl-IPC-Cmd to CentOS6 docker container #14930

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

jdekonin
Copy link
Contributor

Signed-off-by: Joe deKoning [email protected]

@jdekonin
Copy link
Contributor Author

jdekonin commented Apr 19, 2022

@pshipton I have modified the Build-Jenkins-Agent-Container to reference my fork with this single change.
https://openj9-jenkins.osuosl.org/view/Infrastructure/job/Build-Jenkins-Agent-Container/18/console

When this (successfully) completes, if you can update your PR (14900??) with this container reference change and respin that PR job for x86 we can see what the next failure point is (hopefully none). 16 changes to 18 for both JDK8 and JDK11.

8: 'eclipseopenj9/jenkins-agent-x86-centos6:16'

fyi @AdamBrousseau

Edit: autoconf failed, respun as build 18.

@jdekonin jdekonin force-pushed the master branch 3 times, most recently from e2a0533 to 90c4dc2 Compare April 19, 2022 21:01
@pshipton
Copy link
Member

Looks like it failed again due to a certificate problem. Let me know when a new container is available and I'll test it.

@jdekonin
Copy link
Contributor Author

RIght, seems there are plenty of certificates expired. Now on build 21. :(
https://openj9-jenkins.osuosl.org/view/Infrastructure/job/Build-Jenkins-Agent-Container/21/console

@AdamBrousseau
Copy link
Contributor

In build 21 ./autogen.sh seems to fail instantly, not sure why.

@jdekonin jdekonin force-pushed the master branch 3 times, most recently from 6df90b7 to 08371c9 Compare November 1, 2022 20:33
@jdekonin
Copy link
Contributor Author

jdekonin commented Nov 2, 2022

jenkins build docker x86 centos6

@jdekonin
Copy link
Contributor Author

jdekonin commented Nov 2, 2022

@pshipton based on work @vsebe performed previously for redhat6 containers.

https://openj9-jenkins.osuosl.org/job/Build-Jenkins-Agent-Container/24/console
eclipseopenj9/jenkins-agent-x86-centos6:24

Should likely test a few other platform/os combinations...open to suggestions.

@pshipton
Copy link
Member

pshipton commented Nov 2, 2022

We should test centos7 since it's affected by the change, and ubuntu16 since the wget change is common. I'd assume if ubuntu16 works then higher versions will as well. Not sure the platform matters.

@pshipton
Copy link
Member

pshipton commented Nov 2, 2022

@jdekonin the PR build failed

01:12:24  Step 1/28 : FROM nvidia/cuda:9.0-devel-ubuntu16.04 AS cuda-dev
01:12:35  Get "https://registry-1.docker.io/v2/": net/http: TLS handshake timeout

@pshipton
Copy link
Member

pshipton commented Nov 2, 2022

Testing of eclipseopenj9/jenkins-agent-x86-centos6:24 to build openssl 3.0.7
https://openj9-jenkins.osuosl.org/job/Pipeline-Build-Test-Personal/285

@pshipton
Copy link
Member

pshipton commented Nov 2, 2022

eclipseopenj9/jenkins-agent-x86-centos6:24 is working fine to build openssl 3.

@AdamBrousseau
Copy link
Contributor

Build may have that handshake fail because the OS is too old and doesn't have an updated cert store? I seem to recall a Let's Encrypt change a while back that broke curl on old OSs because they don't support new cert protocol or something like that.

@jdekonin
Copy link
Contributor Author

jdekonin commented Nov 2, 2022

jenkins build docker x86 ubuntu20

@jdekonin
Copy link
Contributor Author

jdekonin commented Nov 2, 2022

centos7 - Build-Jenkins-Agent-Container/25 - passes the creation of the container but fails uploading to docker hub
ubuntu16 - Build-Jenkins-Agent-Container/26 - fails with what appears to be missing repos...not related to what I modified. Maybe just out of service. Not sure we care really as we shouldn't be running this release any more anyways since its out of support.
ubuntu18 - Build-Jenkins-Agent-Container/27 - passes the creation of the container but fails uploading to docker hub

To add onto my todo list would be fixing any possible cert issues. Doesn't need to block this PR or use of eclipseopenj9/jenkins-agent-x86-centos6:24 though.

@jdekonin jdekonin marked this pull request as ready for review November 2, 2022 22:05
@jdekonin
Copy link
Contributor Author

jdekonin commented Nov 2, 2022

jenkins build docker x86 centos7

@pshipton pshipton requested a review from keithc-ca November 2, 2022 22:33
@pshipton
Copy link
Member

pshipton commented Nov 2, 2022

Pls create another PR, or modify this one, to update defaults.yml to use eclipseopenj9/jenkins-agent-x86-centos6:24.

buildenv/docker/mkdocker.sh Outdated Show resolved Hide resolved
buildenv/docker/mkdocker.sh Outdated Show resolved Hide resolved
buildenv/docker/mkdocker.sh Show resolved Hide resolved
buildenv/docker/mkdocker.sh Show resolved Hide resolved
buildenv/docker/mkdocker.sh Outdated Show resolved Hide resolved
buildenv/docker/mkdocker.sh Outdated Show resolved Hide resolved
* [skip ci]
* added for openssl3 support
* update wget with no certification option as most certificate have expired
* update build job to use latest container with these changes
* eclipse-openj9#13686

Signed-off-by: Joe deKoning <[email protected]>
@keithc-ca keithc-ca merged commit 93bcb9b into eclipse-openj9:master Nov 14, 2022
jdekonin added a commit to jdekonin/adoptium-infrastructure that referenced this pull request Dec 13, 2022
jdekonin added a commit to jdekonin/adoptium-infrastructure that referenced this pull request Dec 13, 2022
* compiling openssl v3 within container or system requires this package
* see eclipse-openj9/openj9#14930

Signed-off-by: Joe deKoning <[email protected]>
sxa pushed a commit to adoptium/infrastructure that referenced this pull request Jan 6, 2023
)

* compiling openssl v3 within container or system requires this package
* see eclipse-openj9/openj9#14930

Signed-off-by: Joe deKoning <[email protected]>

Signed-off-by: Joe deKoning <[email protected]>
pshipton added a commit to pshipton/infrastructure that referenced this pull request Apr 3, 2023
* compiling openssl v3 within container or system requires this package
* see eclipse-openj9/openj9#14930

These were added in adoptium#2855
but in the wrong place.

Signed-off-by: Peter Shipton <[email protected]>
pshipton added a commit to pshipton/infrastructure that referenced this pull request Apr 3, 2023
* compiling openssl v3 within container or system requires this package
* see eclipse-openj9/openj9#14930

These were added in adoptium#2855
but in the wrong place.

Signed-off-by: Peter Shipton <[email protected]>
pshipton added a commit to pshipton/infrastructure that referenced this pull request Apr 3, 2023
compile

* compiling openssl v3 within container or system requires this package
* see eclipse-openj9/openj9#14930

These were added in adoptium#2855
but in the wrong place.

Signed-off-by: Peter Shipton <[email protected]>
pshipton added a commit to pshipton/infrastructure that referenced this pull request Apr 3, 2023
compile

* compiling openssl v3 within container or system requires this package
* see eclipse-openj9/openj9#14930

These were added in adoptium#2855
but in the wrong place. perl-IPC-Cmd is required for all platforms.

Signed-off-by: Peter Shipton <[email protected]>
sxa pushed a commit to adoptium/infrastructure that referenced this pull request Apr 4, 2023
)

compile

* compiling openssl v3 within container or system requires this package
* see eclipse-openj9/openj9#14930

These were added in #2855
but in the wrong place. perl-IPC-Cmd is required for all platforms.

Signed-off-by: Peter Shipton <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants