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

Fix upstream to tag #32

Merged
merged 2 commits into from
Nov 7, 2023
Merged

Fix upstream to tag #32

merged 2 commits into from
Nov 7, 2023

Conversation

ocobles
Copy link
Contributor

@ocobles ocobles commented Nov 6, 2023

The upstream submodule was initially added using the last commit, but the upgrade-provider pulumi tool needs the SHA of a tag. I have manually checkout upstream to v1.19.0 and updated the go dependencies to that version as well

closed #31 in favor of this PR

Signed-off-by: ocobleseqx <[email protected]>
Signed-off-by: ocobleseqx <[email protected]>
Copy link

github-actions bot commented Nov 6, 2023

Does the PR have any schema changes?

Does the PR have any schema changes?

Found 2 breaking changes:

Resources

  • "equinix:fabric/cloudRouter:CloudRouter":
    • 🟡 inputs: "projects" missing
    • 🟡 properties: "projects" missing output "projects"
      No new resources/functions.

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

@aayushrangwala
Copy link

Just wondering what would be the way to verify if the SHA is right for the upstream-provider

@ocobles
Copy link
Contributor Author

ocobles commented Nov 6, 2023

@aayushrangwala I found it was failing by running https://github.com/pulumi/upgrade-provider locally. That's the tool the https://github.com/equinix/pulumi-equinix/blob/main/.github/workflows/upgrade-provider.yml action will use once it is configured. E2E test was failing here https://github.com/equinix/pulumi-equinix/actions/runs/6773373904/job/18408078544#step:9:57 after updating the upstream you can see here that test are green again

@aayushrangwala
Copy link

/approve

@aayushrangwala
Copy link

/lgtm

@displague displague merged commit c73136e into main Nov 7, 2023
15 checks passed
@displague displague deleted the fix-upstream-to-tag branch November 7, 2023 13:44
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.

3 participants