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

Add option for larger runner on bridge workflows" #678

Merged
merged 3 commits into from
Oct 12, 2023

Conversation

guineveresaenger
Copy link
Contributor

Some bridge upgrades fail during make tfgen && make build SDKs due to the size of the provider.

We should be able to override the default runner with the buildSdk runner.

Additionally, this PR addresses a make bug that got added due to providers.json being file in provider-ci/providers that didn't directly respond to a provider name.
I'm happy to fix this in a different way; but moving the file out of the folder targeted by Make seemed the simplest approach.

  • Allow option to use buildSdk larger runner on bridge update workflow
  • Move providers.json out of providers folder to avoid a Make target confusion
  • Successfully generate test providers

@guineveresaenger guineveresaenger merged commit c3876d5 into master Oct 12, 2023
3 checks passed
@guineveresaenger guineveresaenger deleted the guin/upgrade-runner branch October 12, 2023 20:34
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