feat: updated ci that is faster and better #684
Conversation
jakeyheath
requested review from
abiju-czi,
alexlokshin-czi,
alldoami and
edulop91
| - uses: actions/checkout@v2 | ||
| - name: Set up Python 3.x | ||
| uses: actions/setup-python@v1 | ||
| - name: Generate token |
There was a problem hiding this comment.
generate a token using our github app to do sensitive things like clone repos instead of a SSH key
| # can use this filter to decide whether or not to run linters or tests. | ||
| - uses: dorny/[email protected] | ||
| token: {{`${{ steps.generate_token.outputs.token }}`}} | ||
| - name: Cache Fogg |
There was a problem hiding this comment.
utilize ~/.fogg/cache
| with: | ||
| add: -A | ||
| message: | | ||
| commit from fogg_ci -- ran fogg apply and pushed |
There was a problem hiding this comment.
if fogg apply has any changes, commit them back to the branch
| - name: {{ $component.Dir }} | ||
| if: {{ "${{ steps.filter.outputs.terraform == 'true' }}" }} | ||
| changed: | ||
| - added|modified: 'terraform/**' |
There was a problem hiding this comment.
find all changed files in terraform/* in the last commit
| changed: | ||
| - added|modified: 'terraform/**' | ||
| - uses: actions/github-script@v6 | ||
| id: changedDirs |
There was a problem hiding this comment.
take the changed files and find the directories associated with their env/account/module folder
| needs: find-changed-dirs | ||
| strategy: | ||
| matrix: | ||
| tfmodule: {{`${{ fromJson(needs.find-changed-dirs.outputs.allChanges) }}`}} |
There was a problem hiding this comment.
for each changed dir...
| - uses: actions/checkout@v3 | ||
| with: | ||
| token: {{`${{ steps.generate_token.outputs.token }}`}} | ||
| - name: fix terraform docs |
There was a problem hiding this comment.
fix the terraform docs and commit it back to branch
| ref: {{`${{ github.event.pull_request.head.ref }}`}} | ||
| git-commit-message: | | ||
| commit from fogg_ci -- ran terraform-docs and pushed | ||
| - name: fix terraform fmt |
There was a problem hiding this comment.
fix terraform fmt and commit it back to the branch
| add: -A | ||
| message: | | ||
| commit from fogg_ci -- ran terraform fmt and pushed | ||
| - name: tflint |
Summary
This PR updates the code that is generated to
fogg_ci.ymlfor github actions. It has been tested in a few iterations in shared-infra. The following new features include:fogg apply. should be the samefogg_ci.ymlevery timeTest Plan
References
In testing on shared-infra, this has been about 4 minutes faster than the previous iteration, per PR commit.