-
-
Notifications
You must be signed in to change notification settings - Fork 101
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 Centos 6 docker build to centos7_docker_image_updater jenkins job #2753
Comments
NOTE: Any updates to this should be covered in a corresponding doc update to https://github.com/adoptium/infrastructure/blob/master/FAQ.md#what-about-the-builds-that-use-the-dockerbuild-tag |
@Haroon-Khel @steelhead31 it looks like we have consistent failures on the job after the related PR was merged on the CentOS6 part. Can one of you take a look at this please as we need to be sure that when PRs are merged we verify that they don't cause failures. |
I'll take a look |
Error log on latest build https://ci.adoptopenjdk.net/view/Failing%20Temurin%20jobs/job/centos7_docker_image_updater/331/execution/node/212/log/
Got a replica jenkins job which is just running the centos6 build at the moment https://ci.adoptopenjdk.net/job/centos7_image_update_haroon_test/8/console |
Might be related to the fact that the first build and push requires the image to be tagged https://stackoverflow.com/questions/41984399/denied-requested-access-to-the-resource-is-denied-docker Rerunning my replica jenkins job with a tag stage |
Still fails https://ci.adoptopenjdk.net/job/centos7_image_update_haroon_test/10/console I'll try to replicate the build in an ssh session on the machine itself |
While I am in the middle of finding the
|
Even after logging into dockerhub I still get the same error
|
Retagging the image does not solve the error, neither does building and pushing the image from the other x64 dockerhost machine dockerhost-equinix-ubuntu2004-x64-1 |
@Haroon-Khel , as mentioned on Friday :) , for both this one and the Risc-V push, I've ready a few articles on this issue, and it appears that this process sequence may work ...
|
Update: Scott's solution unfortunately did not work. Same I suspect the error is to do with how the |
Noting that while centos7_build_image was pushed by the George, do you know the history of what's happened to this repository that might be causing these failures? And do you have an account that can log into it? Doesn't look like |
@sxa Do you think removing the latest centos6_build_image image in dockerhub and then re-pushing an image would help? The dockerhub credentials I have can login to docker in the cli but not into hub.docker.com else I would have tested this theory |
Adding to this, it might be possible to view a permissions config for each repo in the adoptopenjdk dockerhub. Perhaps something changed a year ago without notifying us? Again I cannot login to docker hub using the cli credentials so if someone who can login to docker could check? |
The username and password worked for me this time, that's strange. I'll do some digging |
The adoptopenjdkuser account, used for automated pushing, is of the automated team not the owners team. I assume members of the owners team have greater permissions, @gdams's account is of the owners team so @sxa might be right in thinking the adoptopenjdkuser user might not be able to override an image published by a user of the owners account |
The adoptopenjdk user looks like it does not have permissions to delete an image/tag. I think only owners can do that, so @gdams @tellison and @karianna . I recommend one of the owners delete the latest tag of centos6_build_image and then we can see if the adoptopenjdkuser account is able to push a new centos6 image |
Though we should first confirm if an owner account can push an image to centos6_build_image before deleting the latest image |
Will wait for further info |
The issue with that is because we are storing SHAs of docker files in the SBoMs, deleting the SHAs would invalidate the values in there, so we should avoid deleting existing ones if at all possible. |
Image rebuilt and updated on DockerHub |
@tellison added a permission to the centos6 repo to allow the adoptopenjdkuser user to push images to the centos6 repo. It seems that the automated user needs a specific rule to allow it to push to the repo. Tim also created the |
WIth this solved I think its good to put the centos6 build back into https://ci.adoptium.net/job/centos7_docker_image_updater/ |
Closing this now that #3296 is merged and the centos6 part of https://ci.adoptium.net/job/centos7_docker_image_updater/ passes |
Currently our centos6 docker image is built and pushed from a github workflow,
infrastructure/.github/workflows/build.yml
Line 20 in 76d51f1
It would make sense to add this to our centos7_docker_image_updater jenkins job as part of the docker images it builds
The text was updated successfully, but these errors were encountered: