search

CDCgov / cfa-rt-postprocessing

Apache License 2.0
0 stars 0 forks source link

Super basic dockerfile #13

Closed natemcintosh closed 2 weeks ago

natemcintosh commented 2 weeks ago

Based on uv's docker integration docs.

This is the simplest version that they suggest.

natemcintosh commented 2 weeks ago

Note that actually attempting to build the image will fail, because there is not a pyproject.toml in this branch. Once #12 is merged, and this is merged, then it will be able to run the final RUN uv sync --frozen step.

natemcintosh commented 2 weeks ago

Next steps here could be to add an action that automatically builds this dockerfile. Could do that in this PR, or another; don't think it matters too much.

natemcintosh commented 2 weeks ago

Yea, that would be really great. For the CFA self-hosted runners, do we need to use those for everything? Or just the stuff that interacts with Azure?

amondal2 commented 2 weeks ago

I think just for the stuff that interacts with Azure as they are within the EXT subnet. The GH-provided ones should be fine for the other actions (test, pre-commit, etc)