Skip to content

Commit

Permalink
ci: Fix appspot deployment failure (#4049)
Browse files Browse the repository at this point in the history
A couple of steps were in the wrong order.  We must compute tags and
subdomains before the "prep" step, which wipes out .git.
  • Loading branch information
joeyparrish committed Mar 22, 2022
1 parent 1399ec2 commit a4ddab3
Showing 1 changed file with 13 additions and 9 deletions.
22 changes: 13 additions & 9 deletions .github/workflows/release-please.yaml
Expand Up @@ -112,21 +112,13 @@ jobs:
node-version: 16
registry-url: 'https://registry.npmjs.org'

- uses: ./.github/workflows/custom-actions/prep-for-appspot

- uses: google-github-actions/auth@v0
with:
credentials_json: '${{ secrets.APPENGINE_DEPLOY_KEY }}'

- name: Compute appspot subdomain
- name: Compute appspot subdomain and promotion
run: |
# This is the same as the version tag, but with dots replaced by
# dashes. For example, v3.2.2 would have the subdomain v3-2-2.
APPSPOT_SUBDOMAIN=$( echo ${{ needs.release.outputs.tag_name }} | sed -e 's/\./-/' )
echo APPSPOT_SUBDOMAIN=$APPSPOT_SUBDOMAIN >> $GITHUB_ENV
- name: Compute appspot promotion
run: |
# "Promoting" an appspot deployment makes it the default which shows
# up on shaka-player-demo.appspot.com (no subdomain). This should be
# done for the latest release version from the latest release branch.
Expand All @@ -140,6 +132,18 @@ jobs:
echo APPSPOT_PROMOTE=false >> $GITHUB_ENV
fi
# Debug the decisions made here.
echo "Subdomain: $APPSPOT_SUBDOMAIN"
echo "Latest release: $LATEST_RELEASE"
echo "This release: $TAG_NAME"
echo "Promote: $APPSPOT_PROMOTE"
- uses: ./.github/workflows/custom-actions/prep-for-appspot

- uses: google-github-actions/auth@v0
with:
credentials_json: '${{ secrets.APPENGINE_DEPLOY_KEY }}'

- uses: google-github-actions/deploy-appengine@v0
with:
project_id: shaka-player-demo
Expand Down

0 comments on commit a4ddab3

Please sign in to comment.