search

Azure / azure-sdk-for-net

This repository is for active development of the Azure SDK for .NET. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/dotnet/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-net.
MIT License
5.49k stars 4.81k forks source link

DigitalTwins CI pipeline is broken by 39753 #39787

Closed lmolkova closed 1 year ago

lmolkova commented 1 year ago

Library name

Azure.ResourceManager

Please describe the feature.

DigitalTwin CI pipeline is broken by #39753 - https://dev.azure.com/azure-sdk/public/_build/results?buildId=3239008&view=results

A couple of tests fails with following serialization issue

  Failed CreateDigitalTwinWithSystemAssignedIdentityAndEndpoint() [3 s]
  Error Message:
   System.Text.Json.JsonException : The object or value could not be serialized. Path: $.
  ----> System.InvalidOperationException : Cannot write a JSON value within an object without a property name. Current token type is 'StartObject'.
  Stack Trace:

Previous commit before #39753 passes - https://dev.azure.com/azure-sdk/public/_build/results?buildId=3239017&view=results

lmolkova commented 1 year ago

@archerzz @ArcturusZhang @ArthurMa1978 could someone please take a look?

joseharriaga commented 1 year ago

The "net - storage - ci" pipeline appears to be having the same issue: 🔗 https://dev.azure.com/azure-sdk/public/_build/results?buildId=3241642&view=results

amnguye commented 1 year ago

Thanks @joseharriaga !

Yeah this also started happening for the Azure.ResourceManager.Storage pipelines yesterday and we haven't made changes to this package for a while now.

This occurs for the tests that are Playback Only if that helps. It also does repo locally as well with the same error message.

archerzz commented 1 year ago

@lmolkova @joseharriaga @amnguye The issue should have been fixed. Could you please try again? Sorry for the breaking, we were in the middle of a fundamental library upgrade.

joseharriaga commented 1 year ago

Speaking for the storage CI pipeline, that one looks like it's fixed. Thank you, @archerzz !

lmolkova commented 1 year ago

all green now https://dev.azure.com/azure-sdk/public/_build?definitionId=1729&_a=summary, thank you @archerzz !