Skip to content
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

SDK 3.36.0 : Fixes pipeline and test failures #4147

Closed

Conversation

sourabh1007
Copy link
Contributor

Description

As part of this PR, we are fixing below 2 things:

  1. Fixing preview release pipeline where it is find out that, this pipeline was never taking the passed variables.
  2. ClientConfig test is fixed on preview package. As distributedtracingId is also added.

Type of change

  • [] Bug fix (non-breaking change which fixes an issue)

sourabh1007 and others added 19 commits October 18, 2023 18:03
…4130)

* remove release variable

* revert build config variable change also
…Handling = Error on Json default settings when NotFound on Item operations. (#4125)

* issue 4115 initial checkin. need insight from issuer on reproducing this issue

* test refactoring and adding more coverage for other NotFound scenarios

* commit on some actionables

* setting JsonConvert.DefaultSettings to null so that other tests will not fail

* as requested, removed catches from test methods
* initial cleanup

* test updates - working

* cleanup

* more cleanup

* more

* whoops

* Add results to basline

* adding payload to xml

* some generics

* cleanup

* Add datamember serializer

* reorder functions and test fix

* tostring() update and add case

* fix payload

* fix datamembertest

* cleanup

* cleanup

* PR comment

---------

Co-authored-by: Matias Quaranta <[email protected]>
* Client Telemetry Release test fix

* get endpoint from env variable

* read client telemetry endpoint service from env

* updated yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants