-
Notifications
You must be signed in to change notification settings - Fork 12
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
block AWS cronjob #803
block AWS cronjob #803
Conversation
k8s/tools/block-aws/Dockerfile
Outdated
@@ -0,0 +1,27 @@ | |||
FROM python:2-slim |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you tried this with python 3?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was supposed to be python3! Great catch!
updated to python 3, currently running in Oregon-A. |
k8s/tools/block-aws/build.sh
Outdated
@@ -0,0 +1,5 @@ | |||
GIT_COMMIT_RAW=$(git rev-parse HEAD) | |||
GIT_COMMIT=${GIT_COMMIT_RAW:0:7} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be simplified to
GIT_COMMIT=$(git rev-parse --short HEAD)
k8s/tools/block-aws/build.sh
Outdated
GIT_COMMIT_RAW=$(git rev-parse HEAD) | ||
GIT_COMMIT=${GIT_COMMIT_RAW:0:7} | ||
|
||
docker build . -t quay.io/mozmar/blockaws:${GIT_COMMIT} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could also eliminate the GIT_COMMIT definition above entirely and declare and assign a default on the first use:
docker build . -t quay.io/mozmar/blockaws:${GIT_COMMIT:=$(git rev-parse --short HEAD)}
This has the added benefit of allowing it to be overridden with an environment variable.
TODO: add DMS + secret |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work @metadave!
No description provided.