-
Notifications
You must be signed in to change notification settings - Fork 940
Create staging release workflow #6279
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
|
Size Report 1Affected Products
Test Logs |
Size Analysis Report 1This report is too large (587,792 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.Test Logs |
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 Christina! Added a few comments.
NPM_TOKEN_API_DOCUMENTER: ${{ secrets.NPM_TOKEN_API_DOCUMENTER }} | ||
CI: true | ||
- name: Launch E2E tests workflow | ||
uses: peter-evans/repository-dispatch@v2 |
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.
This is a minor concern, but if we think there is a risk of passing tokens to a third party GHA, we could probably write our own action and host it in the same repo.
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.
Since it's pretty simple I changed it to just a curl.
OPERATOR=${{ github.actor }} | ||
curl -X POST -H "Content-Type:application/json" \ | ||
-d "{\"version\":\"$BASE_VERSION\",\"tag\":\"$STAGING_VERSION\",\"date\":\"$DATE\",\"operator\":\"$OPERATOR\"}" \ | ||
https://us-central1-feature-tracker-8ca2b.cloudfunctions.net/logStaging |
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.
hmm should we store the endpoint in a secret?
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.
Oh yeah, that's the way to do it. In fact I already did so for a couple of other workflows and forgot.
OPERATOR=${{ github.actor }} | ||
curl -X POST -H "Content-Type:application/json" \ | ||
-d "{\"version\":\"$BASE_VERSION\",\"tag\":\"$STAGING_VERSION\",\"date\":\"$DATE\",\"operator\":\"$OPERATOR\"}" \ | ||
https://us-central1-feature-tracker-8ca2b.cloudfunctions.net/logStaging |
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.
hmm should we store the endpoint in a secret?
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
Moving as many release tasks as possible out of local script and into Github Actions.
Try creating a workflow for staging first. If all works well, make one for production in a future PR.
repository_dispatch
trigger to make it easier to call from here)CDN steps will be run separately and manually in google3 after the NPM publish is done.