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
add image creation jobs for aws and azure #306
add image creation jobs for aws and azure #306
Conversation
hack/aws-image-job.yaml
Outdated
value: "t2.small" # comment | ||
- name: IMAGE_NAME | ||
value: "peer-pod-ami" | ||
# - name: AWS_REGION |
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.
Let's take this from the secret/configmap
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.
IMAGE_NAME is not used by peerpods, hence this parameter needs to be set specifically for the image creation, I'll do export IMAGE_NAME=${IMAGE_NAME:-peer-pod-ami} instead
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.
@snir911 can you please remove the AWS_REGION
, VPC_ID
and SUBNET_ID
parameters from the manifest. This is coming from the configmap/secret, so let's avoid mentioning it here.
b1398b0
to
7f0bea0
Compare
88e6360
to
d9a50ce
Compare
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.
/lgtm
once the commented out AWS_REGION, VPC_ID and SUBNET_ID parameters are removed from the manifest
cloud providers Signed-off-by: Snir Sheriber <ssheribe@redhat.com>
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.
/lgtm
apply scripts to create image in AWS/Azure