-
Notifications
You must be signed in to change notification settings - Fork 91
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
Create Docker Image directory which deploys to Dockerhub #464
Comments
This structure would be a departure from our SOP. Why not have the Containers to be run by the pod operator on airflow likely don't have anything intrinsically to do with airflow itself. This change seems similar to me to putting the There is the additional consideration of the reliability of docker images that needs to be addressed. Typically we mirror Dockerhub images via Jenkins to our internal container registry for production workflows. We would probably want a similar mechanism for containers used via pod operator, without necessarily mirroring via Jenkins, but this is tangential and something that can be considered separately from the main discussion. |
Thanks for the feedback @whd. We'll close this and figure out an alternative solution. |
Do we still need dockerhub for this use case? We could configure circleci to push images to gcr directly so they are readily available for podoperators. |
The PodOperator is great because it allows us to use specific environments for different jobs. The idea here is for us to store Airflow Job Dockerfiles in
telemetry-airflow
. These will be built alongside the CI. We can optionally build them in CI nightly, or as an Airflow step, if we need that.These images will live in
/images
, which does not need to be included in thetelemetry-airflow:latest
docker image.Each image will have it's own directory. The image will be pushed to the
telemetry-airflow
repository in Dockerhub, but with the tag$DIRNAME
. So for example:That image would be built and pushed to
telemetry-airflow:mozilla-schema-generator
. The PodOperator would use that tag to run the associated task.cc @whd, @haroldwoo for feedback
The text was updated successfully, but these errors were encountered: