-
-
Notifications
You must be signed in to change notification settings - Fork 230
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
DockerHub Description automated process is broken since 2 months (at least?) #777
Comments
I reached here to submit a issue to update the description and saw this, |
I don't have access to the jenkins docker hub account to check if the GitHub repository secrets are valid. Looking at the code of the GHA used to update description and short description on hub.docker.com at https://github.com/peter-evans/dockerhub-description/blob/main/src/dockerhub-helper.ts, I'm wondering if we shouldn't do these 2 API calls ourselves in a pipeline library function (for example) where Docker hub username and password are already used. That would avoid setting up and keep up to date GitHub secrets on this repository, and reuse this new function in other docker repositories in @jenkinsci & @jenkins-infra orgs. WDYT? |
That seems very reasonable to me. If the credentials are already used in the Pipeline job and if there is a relatively easy API call that will do the work, then let's call the API directly from the Pipeline. |
The reason it was done like this is because the delete permission is required and delete wasn’t wanted on the main credential |
What @timja said. Separating these credentials is key to safety (and avoid mistakes) |
I've fixed this by setting the right permissions on the agent image for robots (admin). inbound-agent already had the right permissions Yes the permissions aren't ideal but we've gone through this at length previously and it's a docker hub limitation. |
https://hub.docker.com/r/jenkins/agent now has bookworm references. |
Might be worth to update the description even more, there aren't any mentions of bookworm jdk21 (and maybe some reformatting in general) |
Each time there is a release, the automated process used to push the updated README to DockerHub is failing:
It's been at least 2 months with this failure:
I'm not sure why, and how to fix it yet (haven't looked at it) but opening issue for awareness
cc @lemeurherve @MarkEWaite @NotMyFault @gounthar @timja
The text was updated successfully, but these errors were encountered: