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

Stop publishing to heroku/heroku-private on Docker Hub #246

Merged
merged 1 commit into from
Feb 12, 2024

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Feb 12, 2024

The base images are currently double-published to Docker Hub - once to the public heroku/heroku repository, and then again to the private heroku/heroku-private repo.

Publishing to heroku/heroku-private was added in:
#130

...with the explanation that it might help internally with rollback use-cases.

However:

  1. We've never had cause to use this private Docker Hub repository for rollback in the 5 years since.
  2. We now have a robust image rollout and rollback process (which is new since the incident that prompted the above PR), which doesn't use these images, but instead Cheverny (for the S3 image releases) and versioned release image tags on the heroku/heroku repo for the Docker image side: https://github.com/heroku/cheverny/blob/main/docs/base-image-release.md#rollback-process
  3. The author of the PR that added the publishing of these images has confirmed that there isn't some other reason for keeping them: https://salesforce-internal.slack.com/archives/C01068P24S3/p1707346168976339

As such, we can stop publishing to this repository to simplify the publishing process (which is going to be getting a lot more complex for multi-arch support soon).

GUS-W-15018626.

The base images are currently double-published to Docker Hub - once to
the public `heroku/heroku` repository, and then again to the private
`heroku/heroku-private` repo.

Publishing to `heroku/heroku-private` was added in:
#130

...with the explanation that it might help internally with rollback use-cases.

However:
1. We've never had cause to use this private repository for rollback in
   the 5 years since.
2. We have a robust image rollout and rollback process, which doesn't use
   these images, but instead Cheverny (for the S3 image releases) and image
   tags on the `heroku/heroku` repo for the Docker image side:
   https://github.com/heroku/cheverny/blob/main/docs/base-image-release.md#rollback-process
3. The author of the PR that added the publishing of these images has
   confirmed that there isn't some other reason for keeping them:
   https://salesforce-internal.slack.com/archives/C01068P24S3/p1707346168976339

As such, we can stop publishing to this repository to simplify the
publishing process (which is going to be getting a lot more complex
for multi-arch support soon).

GUS-W-15018626.
@edmorley edmorley merged commit 69c5a0f into main Feb 12, 2024
3 checks passed
@edmorley edmorley deleted the edmorley/rm-heroku-private branch February 12, 2024 14:57
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