-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Unstable branding with rtm label and empty iteration #43483
base: release/9.0.1xx
Are you sure you want to change the base?
Conversation
@mmitche To confirm, we're keeping the branding unstable until a later date, correct? |
Correct. We flip on the 14th I think |
@dotnet/source-build-internal why is source build including a workload uninstall test? Not sure how source build figures out the version of wasm to uninstall. |
These aren't source-build specific tests, these are the Unified Build's scenarios tests which are run to validate source-build. This test is the Test. The core logic is here. It looks like it installs and uninstalls the workload. |
Who owns those tests then? uninstall is one of the least used workload commands (<1% of workloads customers) so I probably wouldn't cover it in these set of tests. I think this is probably related to workload sets not working well with no prerelease iteration. @dsplaisted you mentioned that -rtm feature bands were specially cased but something in the uninstall logic I think is trying to uninstall a baseline workload version. That error at least to me looks like WorkloadVersionRequestedNotFound. |
@richaverma1 had reached out to product teams to gather the tests scenarios. I believe she is the keeper of the tests but really it is the product teams that own their own scenarios. @mattscheffer implemented, and @mmitche has been involved as well. This is good feedback. |
@richaverma1 @mmitche @mattscheffer how can I remove the uninstall step from this particular PR for now? This likely only affects the unstable rtm builds and it's creating confusion since the public rtm branch is still rc2 branded. CC @dsplaisted in case he knows of a product fix we can make for uninstall long term. |
I moved Matthew Scheffer to another project and he is full time working on that. @mattscheffer do you have any suggestion on this though? |
You can't remove the uninstall step in this PR. It will have to be done in the scenario-tests repo (9.0 branch) then flow in. |
I made these tests quite a while ago but wasn't involved in bringing them to this repo, so unfortunately, I don't know if there is a quick way to skip them. I suspect that something with the branding is tripping up the checks to dotnet workload list. I'm guessing you'll need to remove/fix them in the scenario-tests repo, if they are actively being pulled from there. |
No description provided.