-
Notifications
You must be signed in to change notification settings - Fork 0
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
PLT-186: Debugging #18
Conversation
If you add this to your workflow file then GitHub will run it on each push, which is handy. (I did the same thing in https://github.com/CMSgov/ab2d-bcda-dpc-platform/pull/17/files#diff-67237c8e8d172516ba6771e48d67e8167dae05abbd9bc286dc90524c4c4ad754R6):
|
"Name=tag:Layer,Values=app" \ | ||
--query 'Subnets[0].SubnetId' \ | ||
--output text) | ||
echo "SUBNET_ID=$SUBNET_ID" >> "$GITHUB_ENV" |
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.
Is there no way to do this with data sources in the hcl file?
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.
Unfortunately it doesn't seem that way, Terraform includes the datasources for aws_vpc
and aws_subnet
, but those are mysteriously left out of the packer data sources documentation.
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.
Ah, another weird quirk of Packer! Thanks for checking.
🎫 Ticket
https://jira.cms.gov/browse/PLT-186
🛠 Changes
This PR contains fixes for the original implementation of the runner build workflow.
ℹ️ Context for reviewers
In order to test workflows, we must first merge them into the repo.
🔒 Security Implications
If any security implications apply, add Jason Ashbaugh (GitHub username: StewGoin) as a reviewer and do not merge this PR without his approval.