Fix Http test failure because of Activity Id Format default change.#38363
Fix Http test failure because of Activity Id Format default change.#38363tarekgh merged 2 commits intodotnet:masterfrom
Conversation
|
Tagging subscribers to this area: @dotnet/ncl |
|
/azp run runtime-libraries outerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Should I be concerned about this as an indication that the activity change is a larger breaking change than anticipated? |
From what I am seeing is now storing more stuff as W3C Id format became a default. so I expect the breaking wouldn't be big in such cases (I guess). NCL may talk about this scenario better. CC @karelz who may loop other guys can talk about that. to get @karelz up to speed, we have changed the Activity Id default Format from Heiraracal to W3C, and Http logs more info in the harder when having W3C format (e.g. @noahfalk @shirhatti do you have any more input here? |
|
/azp run runtime-libraries outerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Certainly there is some risk involved in making the breaking change, but this issue didn't change my assessment of it if that helps at all : ) |
|
The failures including the outerloop failures are not related. Also, this PR is addressed all Http test failures. I am going to merge this one. |
Fixes #38205