8000 Add docker image by jetexe · Pull Request #243 · cruft/cruft · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add docker image #243

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 5 commits into from
Apr 13, 2023
Merged

Add docker image #243

merged 5 commits into from
Apr 13, 2023

Conversation

jetexe
Copy link
Contributor
@jetexe jetexe commented Apr 7, 2023

Hi there!

Adding docker images with cruft. This workflow automatically builds docker images with cruft installed and pushes them to github packages and dockerhub registries.

You need to create a dockerhub account, create a repository there and set DOCKER_HUB_USERNAME and DOCKER_HUB_ACCESS_TOKEN secrets in the settings page.

Note: arm/v6 arch commented out since python build issue docker-library/python#823

Test images was pushed from https://github.com/jetexe/cruft/pull/2 PR:

- Dockerhub

Usage example:

# remember to run as the current user to have access to the file system.
docker run --rm -it -u"$(id -u):$(id -g)" -v"${PWD}:/app" cruft/cruft:2.14.0 check

Closes: #97

@codecov
Copy link
codecov bot commented Apr 7, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (47c2745) 99.69% compared to head (faff38e) 99.69%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #243   +/-   ##
=======================================
  Coverage   99.69%   99.69%           
=======================================
  Files          21       21           
  Lines         979      979           
=======================================
  Hits          976      976           
  Misses          3        3           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sambhav
Copy link
Member
sambhav commented Apr 7, 2023

@jetexe Thanks for this PR. I would prefer if we only published to ghcr. Would you be okay removing the dockerhub related logic?

@@ -0,0 +1,28 @@
FROM python:3.11.3-alpine
Copy link
Member

Choose a reason for hiding this comment

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

Could we switch to the chainguard python images here? https://github.com/chainguard-images/images/tree/main/images/python#usage

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can't see why a dockerized CLI application needs chainguard images, but I can try to switch. I need more time to experiment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@samj1912 there is no git inside cgr.dev/chainguard/python image, but it's required. So what about using cgr.dev/chainguard/python-dev version (since cruft it is a dev tool)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

also need to mention: linux/arm/v6, linux/arm/v7 and linux/386 are not supported by cgr.dev/chainguard/python image

@jetexe
Copy link
Contributor Author
jetexe commented Apr 7, 2023

Thanks for this PR. I would prefer if we only published to ghcr. Would you be okay removing the dockerhub related logic?

Not a problem. Removed.

@jetexe jetexe requested a review from sambhav April 12, 2023 12:20
Copy link
Member
@sambhav sambhav left a comment

Choose a reason for hiding this comment

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

Thanks for your hard work on this!

@sambhav sambhav merged commit 56eedcb into cruft:master Apr 13, 2023
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.

Feature Request: provide docker image
2 participants
0