Fix Http test failure because of Activity Id Format default change.

[tarekgh]

Fixes #38205

[ghost]

Tagging subscribers to this area: @tarekgh, @safern, @krwq
Notify danmosemsft if you want to be subscribed.

[ghost]

Tagging subscribers to this area: @dotnet/ncl
Notify danmosemsft if you want to be subscribed.

[tarekgh]

CC @stephentoub @noahfalk

[tarekgh]

/azp run runtime-libraries outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[stephentoub]

Should I be concerned about this as an indication that the activity change is a larger breaking change than anticipated?

[tarekgh]

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. transparent and tracestate fields). we have a config switch to get back the old behavior if necessary.

@noahfalk @shirhatti do you have any more input here?

[tarekgh]

/azp run runtime-libraries outerloop

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[noahfalk]

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 : )

[tarekgh]

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.