-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix: Migrations script #800
Conversation
service_account_email: ${{ secrets.GOOGLE_SERVICE_ACCOUNT_EMAIL }} | ||
service_account_key: ${{ secrets.GOOGLE_APPLICATION_CREDENTIALS }} | ||
export_default_credentials: true | ||
- id: 'auth' |
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.
switching over to using identity provider as done in ci.yml
eng-pipes/.github/workflows/ci.yml
Line 125 in 04f8a2a
- id: 'auth' |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #800 +/- ##
==========================================
- Coverage 86.24% 85.05% -1.20%
==========================================
Files 117 117
Lines 3316 3372 +56
Branches 667 675 +8
==========================================
+ Hits 2860 2868 +8
- Misses 442 490 +48
Partials 14 14 ☔ View full report in Codecov by Sentry. |
5c11e21
to
1b1acef
Compare
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.
We should opt to use node 20 now that previous versions are deprecated on GitHub actions, but that can be done in a separate PR. Otherwise, LGTM.
@@ -34,13 +37,15 @@ jobs: | |||
with: | |||
node-version: '12' |
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.
We should probably update this to use node 20, which may require updating the rest of the actions in this file, but will help us ensure we are on the latest, most secure, version. This can be done in a separate PR.
Looks like this service account no longer exists. We've been using workload identity provider since #633 but we forgot to account for migrations.
https://github.com/getsentry/eng-pipes/actions/runs/9573265261/job/26394333363