Skip to content
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

Run make ci-mgmt to regen GHA workflows #1384

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Conversation

rquitales
Copy link
Member

Faster than waiting for the nightly sync to regen and fix the broken CI on master branch.

@rquitales rquitales requested a review from a team October 12, 2023 17:36
@github-actions
Copy link

Does the PR have any schema changes?

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Maintainer note: consult the runbook for dealing with any breaking changes.

@@ -135,7 +135,7 @@ jobs:
COVERAGE_OUTPUT_DIR: ${{ secrets.COVERAGE_OUTPUT_DIR }}
name: generate_coverage_data
needs: prerequisites
runs-on: pulumi-ubuntu-8core
runs-on: ubuntu-latest
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change reduces the runner size to a smaller machine. Is this what we want?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, generate_coverage_data was mistakenly given the larger runner instead of prerequisites for the master workflow.

@t0yv0 t0yv0 self-requested a review October 12, 2023 18:05
@rquitales rquitales enabled auto-merge (squash) October 12, 2023 18:06
@rquitales rquitales merged commit b4ab3a8 into master Oct 12, 2023
15 checks passed
@rquitales rquitales deleted the rquitales/fix-gha branch October 12, 2023 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants