Skip to content

Publish a Minimal Container Image to ECR Public #148

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

Closed

Conversation

thebeanogamer
Copy link

Running the current full-size image in a production deployment isn't really feasible, but the minimal container image already in this repo already has everything required. We're looking to move our Pyth Publisher into Kubernetes and this would make that change much easier.

To give you some idea of the size difference:

$ podman images --format "table {{.ID}} {{.Repository}} {{.Size}}" | grep -E "(pyth-network|6a18b7d)"
6a18b7d60f62  <none>                             136 MB
774cbbe966e0  public.ecr.aws/pyth-network/agent  5.83 GB

It'd probably also be worthwhile to upload a basic About and Icon to ECR Public to help users know they've found the correct page. Some guidance on doing that can be found on https://docs.aws.amazon.com/AmazonECR/latest/public/public-repository-catalog-data.html.

This is in addition to the existing full-size image
ali-behjati added a commit that referenced this pull request Jan 9, 2025
@ali-behjati
Copy link
Collaborator

ali-behjati commented Jan 9, 2025

Thanks a lot for your contribution.

I think the existing one is only useful for testing and we can stop publishing it. i created #149 to actually replace the main one with the minimal one and bumped the version.

@ali-behjati ali-behjati closed this Jan 9, 2025
ali-behjati added a commit that referenced this pull request Jan 9, 2025
* refactor(docker): use minimal docker for image publish

* update readme to add dockerfile

taken from #148

* add skipped dockerfile
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