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.

Sign up for GitHub

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

Jump to bottom

Default build failing on bridged providers #961

Closed
69 tasks
guineveresaenger opened this issue Jun 6, 2024 · 3 comments · Fixed by #962
Closed
69 tasks

Default build failing on bridged providers #961

guineveresaenger opened this issue Jun 6, 2024 · 3 comments · Fixed by #962
Assignees
@guineveresaenger
Labels
kind/bug Some behavior is incorrect or out of spec p1 A bug severe enough to be the next item assigned to an engineer resolution/fixed This issue was fixed

Comments

@guineveresaenger
Copy link
Contributor

guineveresaenger commented Jun 6, 2024 •
edited
Loading

#943 removed JAVAVERSION as an explicit env var but pulumi-package-publisher inherits it.

Path to resolution:

  1. Update package-publisher to require an explicit input
  2. Test on a provider
  3. Release package-publisher
  4. Update package-publisher on all providers
  5. Trigger build of Update Workflows on all bridged providers

See also: #857

Tier 1

Tier 2
@guineveresaenger guineveresaenger added the p1 A bug severe enough to be the next item assigned to an engineer label Jun 6, 2024
@guineveresaenger guineveresaenger self-assigned this Jun 6, 2024
@VenelinMartinov
Copy link
Contributor

SCM is missing from the template and has a hanging update: pulumi/pulumi-scm#22

This was referenced Jun 6, 2024
Closed
Merged
guineveresaenger added a commit to pulumi/pulumi-civo that referenced this issue Jun 6, 2024
@guineveresaenger
Use explicit inputs for publish-provider-action (#515)
a6c5443 
This is a test for
pulumi/pulumi-package-publisher#28 in support of
resolving pulumi/ci-mgmt#961.

Merging this will not break default any more than it currently is.
@guineveresaenger guineveresaenger mentioned this issue Jun 6, 2024
Merged
guineveresaenger added a commit that referenced this issue Jun 6, 2024
@guineveresaenger
Bump package publisher version and update inputs (#962)
18d44a5 
Part of #961.

We can close #961 when all Workflows have updated to this publisher.
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Jun 6, 2024
@pulumi-bot pulumi-bot reopened this Jun 6, 2024
@pulumi-bot
Copy link

Cannot close issue:

  • does not have required labels: kind/

Please fix these problems and try again.

@guineveresaenger guineveresaenger added kind/bug Some behavior is incorrect or out of spec and removed resolution/fixed This issue was fixed labels Jun 6, 2024
@guineveresaenger guineveresaenger mentioned this issue Jun 7, 2024
Closed
@guineveresaenger guineveresaenger added the resolution/fixed This issue was fixed label Jun 7, 2024
@guineveresaenger
Copy link
Contributor Author

All related p1s have been resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@guineveresaenger guineveresaenger

Labels
kind/bug Some behavior is incorrect or out of spec p1 A bug severe enough to be the next item assigned to an engineer resolution/fixed This issue was fixed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bump package publisher version and update inputs pulumi/ci-mgmt
3 participants
@guineveresaenger @VenelinMartinov @pulumi-bot