-
Notifications
You must be signed in to change notification settings - Fork 62.2k
Updates to AWS ECS deploy guide #3002
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
Conversation
Top-level environment variables cannot be referenced from jobs.<job_id>.steps[*].with using the direct environment variable syntax (e.g. $AWS_REGION)
Redirecting to $GITHUB_ENV does not populate the ${{ steps.[*].outputs.* }} context
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines. |
@thekaveman Thanks so much for opening a PR! I'll get this triaged for review ✨ |
This PR is stale because it has been open 7 days with no activity and will be automatically closed in 3 days. To keep this PR open, update the PR by adding a comment or pushing a commit. |
Please don't close! |
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.
Thanks so much for these fixes, @thekaveman! And also thanks for the awesome explanation and links to your workflow runs! 🙇♂️
I'll get this merged in 🚀
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. |
Why:
Following the steps in the guide, errors will occur running the workflow due to these syntax problems.
Example failed run with the guide syntax for environment variables before these changes: https://github.com/cal-itp/docker-django-aws-test/actions/runs/494684748
Example successful run with the updated environment variable syntax reflected in this PR: https://github.com/cal-itp/docker-django-aws-test/actions/runs/494699023
Example failed run with the output redirect to
$GITHUB_ENV
before these changes: https://github.com/cal-itp/docker-django-aws-test/runs/1724635263Example successful (step) with the updated ::set-ouput syntax reflected in this PR: https://github.com/cal-itp/docker-django-aws-test/runs/1724692920
What's being changed:
jobs.<job_id>.steps[*].with
, from$ENV_VAR
to${{ env.ENV_VAR }}
$GITHUB_ENV
to using::set-output name=variable::
Check off the following:
Direct link to staging deploy: https://docs-3002--aws-ecs-updates.herokuapp.com/en/actions/guides/deploying-to-amazon-elastic-container-service