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

Re-enable blazor tests in 6.0 after 17.4 msbuild fix is in #44526

Closed
marcpopMSFT opened this issue Oct 10, 2022 · 8 comments
Closed

Re-enable blazor tests in 6.0 after 17.4 msbuild fix is in #44526

marcpopMSFT opened this issue Oct 10, 2022 · 8 comments
Labels
area-blazor Includes: Blazor, Razor Components Pillar: Technical Debt Priority:0 Work that we can't release without task test-failure

Comments

@marcpopMSFT
Copy link
Member

dotnet/sdk#28322 (comment)

@dotnet-issue-labeler dotnet-issue-labeler bot added area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework untriaged labels Oct 10, 2022
@TanayParikh
Copy link
Contributor

@marcpopMSFT @dsplaisted do we know when the 17.4 fix will be in? Is it just a matter of dependency flow, or do we need to take additional action?

fyi/ @javiercn

@marcpopMSFT
Copy link
Member Author

I think it needs to be in the helix images the tests run in. It kind of looks like from the history that this probably made it in for preview 3 of 17.4 but isn't in RC2 of the SDK. Since this issue broke because of the helix images, I think getting them updated to preview 3 may be sufficient.

@MattGal might know the schedule but I think it usually takes ~2 weeks from release to being adopted in the helix queues.

@TanayParikh TanayParikh transferred this issue from dotnet/sdk Oct 13, 2022
@TanayParikh TanayParikh added this to the 8.0-MQ milestone Oct 13, 2022
@mkArtakMSFT mkArtakMSFT modified the milestones: 8.0-MQ, .NET 8 Planning Feb 2, 2023
@ghost
Copy link

ghost commented Feb 2, 2023

Thanks for contacting us.

We're moving this issue to the .NET 8 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@MattGal
Copy link
Member

MattGal commented Feb 2, 2023

I think it needs to be in the helix images the tests run in. It kind of looks like from the history that this probably made it in for preview 3 of 17.4 but isn't in RC2 of the SDK. Since this issue broke because of the helix images, I think getting them updated to preview 3 may be sufficient.

@MattGal might know the schedule but I think it usually takes ~2 weeks from release to being adopted in the helix queues.

I missed this one back when I was originally tagged, but I would point out that the "regular" VS 2022 images are now at version 17.4.3, and you can see our upgrade schedule here

@mkArtakMSFT mkArtakMSFT added task area-blazor Includes: Blazor, Razor Components and removed area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework labels Feb 7, 2023
@mkArtakMSFT mkArtakMSFT added this to the .NET 8 Planning milestone Feb 7, 2023
@ghost
Copy link

ghost commented Feb 7, 2023

Thanks for contacting us.

We're moving this issue to the .NET 8 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@ghost
Copy link

ghost commented Oct 6, 2023

Thanks for contacting us.

We're moving this issue to the .NET 9 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@ghost
Copy link

ghost commented Dec 14, 2023

Thanks for contacting us.

We're moving this issue to the .NET 9 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@mkArtakMSFT mkArtakMSFT added the Priority:0 Work that we can't release without label Jan 3, 2024
@javiercn
Copy link
Member

Done as part of dotnet/sdk#38261

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components Pillar: Technical Debt Priority:0 Work that we can't release without task test-failure
Projects
None yet
Development

No branches or pull requests

5 participants