Skip to content

[release/9.0-staging][wasm] Workaround incorrect mono restore when building WBT#110590

Merged
akoeplinger merged 2 commits into
dotnet:release/9.0-stagingfrom
lewing:wbt-no-mono
Dec 20, 2024
Merged

[release/9.0-staging][wasm] Workaround incorrect mono restore when building WBT#110590
akoeplinger merged 2 commits into
dotnet:release/9.0-stagingfrom
lewing:wbt-no-mono

Conversation

@lewing

@lewing lewing commented Dec 10, 2024

Copy link
Copy Markdown
Member

No description provided.

@ghost ghost added the area-Build-mono label Dec 10, 2024
@lewing lewing mentioned this pull request Dec 10, 2024
Merged
@carlossanlop

carlossanlop commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

Is staging correct? We don't want it to go straight to base?

@lewing

lewing commented Dec 10, 2024

Copy link
Copy Markdown
Member Author

test infrastructure only change

@carlossanlop carlossanlop added the Servicing-approved Approved for servicing release label Dec 10, 2024
@carlossanlop

carlossanlop commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

Tell mode: infra only. Approved.

@lewing

lewing commented Dec 10, 2024

Copy link
Copy Markdown
Member Author

Is staging correct? We don't want it to go straight to base?

I'm happy to retarget?

@carlossanlop

carlossanlop commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

I'm happy to retarget?

It depends on how urgent it is. Do we need it right now like the other PR you just merged?

@carlossanlop

carlossanlop commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

If not, staging is fine and we can monitor the builds to see if it worked or not.

@lewing

lewing commented Dec 10, 2024

Copy link
Copy Markdown
Member Author

If not, staging is fine and we can monitor the builds to see if it worked or not.

it should make the tests green but it isn't required for the product

@carlossanlop

carlossanlop commented Dec 10, 2024

Copy link
Copy Markdown
Contributor

Oh I missed that it was for tests, sorry. Carry on then. Staging it is.

@akoeplinger

akoeplinger commented Dec 11, 2024 •
edited
Loading

Copy link
Copy Markdown
Member

There's a different failure now though:

dotnet_install: Error: Could not find .NET Core SDK with version = 9.0.102

@lewing

lewing commented Dec 11, 2024

Copy link
Copy Markdown
Member Author

There's a different failure now though:

dotnet_install: Error: Could not find .NET Core SDK with version = 9.0.102

I think the failure is a result of #109304 updating to an unreleased sdk

@build-analysis build-analysis Bot mentioned this pull request Dec 12, 2024
Closed
@akoeplinger akoeplinger merged commit 8af4dec into dotnet:release/9.0-staging Dec 20, 2024
@akoeplinger akoeplinger mentioned this pull request Dec 20, 2024
Merged
@akoeplinger akoeplinger mentioned this pull request Jan 10, 2025
Merged
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@maraf maraf maraf approved these changes

@ilonatommy ilonatommy Awaiting requested review from ilonatommy ilonatommy is a code owner

@pavelsavara pavelsavara Awaiting requested review from pavelsavara pavelsavara is a code owner

@akoeplinger akoeplinger Awaiting requested review from akoeplinger

+1 more reviewer

@carlossanlop carlossanlop carlossanlop approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@lewing lewing

Labels

area-Build-mono Servicing-approved Approved for servicing release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@lewing @carlossanlop @akoeplinger @maraf