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

Workflow failure: build #2004

Closed
pulumi-bot opened this issue Jan 21, 2025 · 4 comments · Fixed by #2003
Closed

Workflow failure: build #2004

pulumi-bot opened this issue Jan 21, 2025 · 4 comments · Fixed by #2003
Assignees
Labels
kind/engineering Work that is not visible to an external user resolution/fixed This issue was fixed

Comments

@pulumi-bot
Copy link
Contributor

Workflow Failure

build has failed. See the list of failures below:

@pulumi-bot pulumi-bot added kind/engineering Work that is not visible to an external user needs-triage Needs attention from the triage team p1 A bug severe enough to be the next item assigned to an engineer labels Jan 21, 2025
@flostadler flostadler self-assigned this Jan 21, 2025
@flostadler flostadler removed the p1 A bug severe enough to be the next item assigned to an engineer label Jan 21, 2025
@flostadler
Copy link
Contributor

There's issues with RoleCS selecting the correct language version. This might fix it: #2003

@flostadler
Copy link
Contributor

I'm not quite sure why this ever worked. Dotnet 6 does not support C# 11, but the test is currently set up that way (and seemed to have been passing).

@flostadler
Copy link
Contributor

We might need to upgrade native providers to dotnet 8: pulumi/ci-mgmt#1307

@flostadler
Copy link
Contributor

This most likely has to do with github upgrading the ubuntu-latest runner to 24.04.

Previously we first installed dotnet 8.0 and then tried installing dotnet 6.0 but it was already present:

dotnet-install: .NET Core SDK with version '6.0.428' is already installed.

Now dotnet 6.0 is no longer in the new runner image and we install it on top of dotnet 8.0. The v6 binary most likely taking precedence because it's on the path before the v8 one.

I think the big difference is that we were previously using dotnet 8.0 to compile for the dotnet 6.0 target whereas now we're compiling with dotnet 6.0 for the 6.0 target.

@flostadler flostadler removed the needs-triage Needs attention from the triage team label Jan 21, 2025
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/engineering Work that is not visible to an external user resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants