Skip to content

Conversation

@LawnGnome
Copy link
Contributor

We used to update the Docker Hub description for
https://hub.docker.com/r/sourcegraph/src-batch-change-volume-workspace
on release, but doing so reliably has proven to be difficult, as Docker
Hub doesn't provide a reliable API to do this. When it fails, the entire
release shows up as red, which is obviously not ideal!

Since this description rarely changes, and the pushes themselves are
reliable, this removes the description update and makes it a manual
process.

We used to update the Docker Hub description for
https://hub.docker.com/r/sourcegraph/src-batch-change-volume-workspace
on release, but doing so reliably has proven to be difficult, as Docker
Hub doesn't provide a reliable API to do this. When it fails, the entire
release shows up as red, which is obviously not ideal!

Since this description rarely changes, and the pushes themselves are
reliable, this removes the description update and makes it a manual
process.
@LawnGnome LawnGnome requested a review from a team June 28, 2021 22:43
@LawnGnome LawnGnome self-assigned this Jun 28, 2021
Copy link
Member

@eseliger eseliger left a comment

Choose a reason for hiding this comment

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

That must've been fun 🙂

@LawnGnome
Copy link
Contributor Author

That must've been fun slightly_smiling_face

I tried to fix it a few weeks back, and it just won't go. I can take a hint.

@LawnGnome LawnGnome merged commit 0222062 into main Jun 28, 2021
@LawnGnome LawnGnome deleted the aharvey/remove-docker-publish branch June 28, 2021 22:55
scjohns pushed a commit that referenced this pull request Apr 24, 2023
We used to update the Docker Hub description for
https://hub.docker.com/r/sourcegraph/src-batch-change-volume-workspace
on release, but doing so reliably has proven to be difficult, as Docker
Hub doesn't provide a reliable API to do this. When it fails, the entire
release shows up as red, which is obviously not ideal!

Since this description rarely changes, and the pushes themselves are
reliable, this removes the description update and makes it a manual
process.
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.

3 participants