-
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
Do not run upgrade check for this provider #202
Conversation
Does the PR have any schema changes?Looking good! No breaking changes found. Maintainer note: consult the runbook for dealing with any breaking changes. |
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.
Seems fine to do this as a quick workaround, but I wonder if we couldn't make the upgrade job resilient to this case?
Makefile
Outdated
@@ -180,7 +180,6 @@ bin/pulumi-java-gen: .pulumi-java-gen.version | |||
# - Run make ci-mgmt to apply the change locally. | |||
# | |||
ci-mgmt: .ci-mgmt.yaml | |||
find .github/workflows/*.yml -type f ! -name "$(PACK)*.yml" -delete |
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.
I'm surprised that this changes with the checkUpstreamUpgrade flag change
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 an unrelated change from bumping ci-mgmt to latest. Since ci-mgmt isn't versioned, each time a change is made to .ci-mgmt.yaml
and then applied with make ci-mgmt
the latest version of ci-mgmt
is pulled.
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.
It was a recent change in ci-mgmt that had not done the nightly percolation. It's a legitimate change; I verified.
7509f22
to
3e012ee
Compare
This PR has been shipped in release v0.3.4. |
Because the upstream provider is pre-stable release, we run upgrades manually.
Fixes #200.