[browser][MT] enable more tests#97391
Conversation
pavelsavara
added
arch-wasm
|
Tagging subscribers to 'arch-wasm': @lewing Issue Detailsnull
|
|
/azp run runtime-wasm |
|
Azure Pipelines successfully started running 1 pipeline(s). |
ilonatommy
left a comment
ilonatommy
left a comment
There was a problem hiding this comment.
Looks good, I only have concerns about change in runtime-extra-platforms-wasm.yml
There was a problem hiding this comment.
Shouldn't it be in a future PR, where we don't simultaneously enable any new tests, just change the warning to error? Are we 100% sure about the enabled tests to not fail?
There was a problem hiding this comment.
We need to start moving quicker now. This is the way how it will get noticed, rather than ignored.
There was a problem hiding this comment.
In the last 7 days we had 7 PR that are not connected with threading that had failures on '_Threading' lanes (both timeouts and BadExists). There is 100% probability that this change will introduce a mess
There was a problem hiding this comment.
that's a goal. To get people to start working the [ActiveIssue] chore, instead of ignoring like for the last 6 months.
There was a problem hiding this comment.
cc @lewing is it good time to start moving on this ?
There was a problem hiding this comment.
Yes, we need to start standing up the tests and marking the failing ones asap. I'll let people know.
There was a problem hiding this comment.
I'm working to make it as green as possible before we merge this.
|
/azp run runtime-wasm |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run runtime-wasm |
|
Azure Pipelines successfully started running 1 pipeline(s). |
pavelsavara
force-pushed
the
browser_mt_enable_more_tests
branch
from
d40aee7 to
767a4c3
Compare
|
/azp run runtime-wasm |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run runtime-wasm |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run runtime-wasm |
|
Azure Pipelines successfully started running 1 pipeline(s). |
pavelsavara
requested review from
MichalStrehovsky,
marek-safar and
thaystg
as code owners
|
/azp run runtime-wasm |
|
Azure Pipelines successfully started running 1 pipeline(s). |
pavelsavara
force-pushed
the
browser_mt_enable_more_tests
branch
from
dc9eb39 to
dc13b06
Compare
|
/azp run runtime-wasm |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
I filled #97518, the rest is green or known |
4 participants
No description provided.