8000 Migrate docker-hub to ECR using git-action by jo-bisonai · Pull Request #443 · Bisonai/orakl · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Migrate docker-hub to ECR using git-action #443

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

Merged
merged 33 commits into from
Mar 24, 2023
Merged

Conversation

jo-bisonai
Copy link
Member
@jo-bisonai jo-bisonai commented Mar 24, 2023

Description

This pull request (PR) is about migrating the repository from Docker Hub to AWS ECR. The reason for this change is that Docker Hub has a limitation for upload after a certain megabyte or call, which slows down the upload speed significantly. Therefore, I decided to migrate the repository to AWS ECR.

To connect to AWS ECR, I do not store any AWS secrets on Git-Action-Secrets. Instead, I use AWS web-identity to log in to ECR.

I have made the repository in ECR public for now. However, I plan to write a description about the organization later.

This PR is only triggered when you merge into the 'Master' branch containing each service changed. After merging, this will create using docker-compose in 'src/'. These images will then be pushed to ECR.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Deployment

  • Should publish npm package
  • Should publish Docker image

@jo-bisonai jo-bisonai self-assigned this Mar 24, 2023
@jo-bisonai jo-bisonai linked an issue Mar 24, 2023 that may be closed by this pull request
Copy link
Member
@martinkersner martinkersner left a comment

Choose a reason for hiding this comment

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

LGTM! We will have to synchronize docker image publishing with npm package publishing.

@jo-bisonai jo-bisonai merged commit 7df8358 into master Mar 24, 2023
@jo-bisonai jo-bisonai deleted the i-424/docker-hub-is-slow branch March 24, 2023 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker Hub is slow!
2 participants
0