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 DockerHub description upon deployment #492

Merged
merged 1 commit into from
Jan 30, 2020

Conversation

ArturKlauser
Copy link
Contributor

It irked me that moving the docker image build from DockerHub auto-build to Github Actions had lost the automatic update of the info (aka. repository description) on DockerHub's web UI. So this PR puts it back, using the DockerHub API.

When docker images have been successfully deployed and tested, update the
DockerHub description with the current version of the README.md from github.
This is similar to what DockerHub Autobuild does, if it were used.
Copy link
Owner

@mvdan mvdan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, you really went the extra mile here :)

@ArturKlauser
Copy link
Contributor Author

I wanted it for one of my own repos anyway ;)

@mvdan mvdan merged commit 9834d31 into mvdan:master Jan 30, 2020
@ArturKlauser
Copy link
Contributor Author

Oh-oh, it looks like it didn't work on your setup. Seems like the DOCKER_PASSWORD you set in Github Actions is some sort of access token, not the actual password? DockerHub doesn't seem to allow updating the repository description with that. Not sure if you can choose the permissions for that token somewhere on DockerHub UI (or where ever you created that token).

From Github Actions log:
(node:17914) UnhandledPromiseRejectionWarning: Error: {"detail":"access to the resource is forbidden with personal access token"}

@mvdan
Copy link
Owner

mvdan commented Jan 30, 2020

Yeah, I did set up a token. Not too happy about having to use my real password, but it's a generated one anyway, so I'll change it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants