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.

Sign up for GitHub

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

Jump to bottom

[AzureMonitorExporter] update demo project #34514

Merged
merged 3 commits into from
Feb 23, 2023
Merged

[AzureMonitorExporter] update demo project #34514

merged 3 commits into from
Feb 23, 2023

Conversation

TimothyMothra
Copy link
Contributor

@TimothyMothra TimothyMothra commented Feb 23, 2023 •
edited
Loading

Changes

@TimothyMothra TimothyMothra added the Monitor - Exporter Monitor OpenTelemetry Exporter label Feb 23, 2023
vishweshbankwar
vishweshbankwar approved these changes Feb 23, 2023
View reviewed changes
Copy link
Contributor

@vishweshbankwar vishweshbankwar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM: Should we update https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter#troubleshooting to point to demo as a hint?

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@TimothyMothra
Copy link
Contributor Author

LGTM: Should we update https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter#troubleshooting to point to demo as a hint?

Great callout! I'll update the Readme

@TimothyMothra TimothyMothra enabled auto-merge (squash) February 23, 2023 22:15
@TimothyMothra TimothyMothra merged commit 24959dd into main Feb 23, 2023
@TimothyMothra TimothyMothra deleted the tilee/demo branch February 23, 2023 22:38
Harshit-Shrivastava pushed a commit to Harshit-Shrivastava/azure-sdk-for-net that referenced this pull request Feb 24, 2023
@TimothyMothra @Harshit-Shrivastava
[AzureMonitorExporter] update demo project (Azure#34514)
b9e0a29 
* update demo project

* cleanup csproj

* update readme
Harshit-Shrivastava added a commit to Harshit-Shrivastava/azure-sdk-for-net that referenced this pull request Feb 24, 2023
@Harshit-Shrivastava @Jingshu923
@azure-sdk @seanmcc-msft @TimothyMothra @AlexanderSher @JoshLove-msft @mikeharder @bhargav-kansagara @BhKansag @NiveditJain @sjiherzig @minnieliu @richardcho-msft @huachuandeng @minwoolee-msft @yyw-msft @fengzhou-msft @vishweshbankwar @petrsvihlik @jsquire @christothes @joseharriaga @trrwilson @rajkumar-rangaraj @JialinXin @archerzz
Successfully built using tag 2023-04 (#1)
26cd1fe 
* [DataFactory]Added new features into 9.2.0 (Azure#34452)

* Increment package version after release of Azure.Monitor.Ingestion (Azure#34431)

* Disabled ApiCompact on Microsoft.Azure.WebJobs.Extensions.Storage (Azure#34478)

* [AzureMonitorExporter] misc cleanup (Azure#34434)

* misc cleanup

* add tests for EventSource

* more changelog cleanup

* Fix double dispose in BlobBatch (Azure#34427)

* Rename type (Azure#34480)

* [AzureMonitorExporter] fix nullables in more tests (Azure#34454)

* wip

* wip

* wip

* wip

* Delete duplicate tests.yml under sdk/communication (Azure#34448)

- Fixes Azure#34446

* Adding release date (Azure#34456)

Co-authored-by: Bhargav Kansagara <bhkansag@microsoft.com>

* Storage STG 87 (Azure#34455)

* Fixed comments by @tg-msft on APIView (Azure#34320)

* fixed comments by @tg-msft

* exportedAPIs

---------

Co-authored-by: Nivedit Jain <jain.22@iitj.ac.in>

* [digitaltwins] Add support for CP API Version 2023-01-31 (Azure#34437)

* Add support for stable API version 2023-01-31

* Add tests and recordings

* Revert decision to do a major version bump

* Add ACS Rooms Code Owners (Azure#34435)

Co-authored-by: Minnie Liu <peiliu@microsoft.com>

* Bumped default SAS version (Azure#34484)

* Feature/callautomation/callinvite (Azure#34337)

* add customContext to AddParticipant in ACS CallAutomation (Azure#33786)

* add customContext to AddParticipant in ACS CallAutomation

* revert: change on autorest.md to point to local file

* rerun autorest

* fix uri bug

* add Call Invite class (Azure#33929)

* fixing build with autorest

* fixing build by removing sip headers for now

* api file update

* make repeatabilityHeaders internal and generated automatically (Azure#33965)

* make repeatabilityHeaders internal and generated automatically

* remove repeatability in README

* fix tests

* Create call improvements (Azure#33932)

* add Call Invite class

* integrate call invite to create call

* fix tests

* add Call Invite class (Azure#33929)

* fixing build with autorest

* fixing build by removing sip headers for now

* api file update

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* remove comment

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* fix tests

* remove validation related unit tests

* fix tests

* remove comment

* resolve conflit

* fix merge conflicts

* make repeatabilityHeaders internal and generated automatically (Azure#33965)

* make repeatabilityHeaders internal and generated automatically

* remove repeatability in README

* fix tests

* integrate call invite to create call

* update auto gen

---------

Co-authored-by: Min Woo Lee 🧊 <77083090+minwoolee-ms@users.noreply.github.com>
Co-authored-by: Yingying Wu <98626395+yyw-msft@users.noreply.github.com>

* use call invite in redirect and fix tests (Azure#34037)

* use call invite in transfer (Azure#34083)

* use call invite in transfer

* adding unit tests for simple methods

* Remove call source for create call request (Azure#34039)

* add Call Invite class

* integrate call invite to create call

* fix tests

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* remove comment

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* remove comment

* resolve conflit

* fix merge conflicts

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* integrate call invite to create call

* remove comment

* fix merge conflicts

* update swagger endpoint

* generate code from swagger

* Merge branch 'richardcho/create-call' of https://github.com/richardcho-msft/azure-sdk-for-net into richardcho/create-call

* fix linter

* update autogen'

* fix auto gen

* update autogen

* merge conflict

* remove duplicate code

* use call invite in transfer (Azure#34083)

* use call invite in transfer

* adding unit tests for simple methods

* integrate call invite to create call

* fix tests

* integrate call invite to create call

* remove comment

* fix merge conflicts

* update swagger endpoint

* generate code from swagger

* Merge branch 'richardcho/create-call' of https://github.com/richardcho-msft/azure-sdk-for-net into richardcho/create-call

* fix linter

* update autogen'

* fix auto gen

* update autogen

* update swagger

* Revert "update swagger"

This reverts commit ac523d7.

* update swagger ref

* fix build

* remove call source

---------

Co-authored-by: Yingying Wu <98626395+yyw-msft@users.noreply.github.com>

* Update AddParticipant & RemoveParticipant (Azure#34155)

* update add participant logic

* generate code based on new swagger

* remove obsolete fils

* add CallInvite options

* fix build

* add custom context

* generate latest code

* update remove participant

* fix tests

* fix tests

* fix tests

* update api

* fix failed test

* use sipheaders and voipheaders for transfer (Azure#34190)

* update based on latest swagger (Azure#34197)

* update based on latest swagger

* api changes

* fix transfer logic (Azure#34263)

* fix uri (Azure#34267)

* Address CreateCall bugs (Azure#34273)

* fix transfer logic

* fix createCall issues

* updates (Azure#34281)

* add customContext to AddParticipant in ACS CallAutomation (Azure#33786)

* add customContext to AddParticipant in ACS CallAutomation

* revert: change on autorest.md to point to local file

* rerun autorest

* fix uri bug

* add Call Invite class (Azure#33929)

* fixing build with autorest

* fixing build by removing sip headers for now

* api file update

* make repeatabilityHeaders internal and generated automatically (Azure#33965)

* make repeatabilityHeaders internal and generated automatically

* remove repeatability in README

* fix tests

* Create call improvements (Azure#33932)

* add Call Invite class

* integrate call invite to create call

* fix tests

* add Call Invite class (Azure#33929)

* fixing build with autorest

* fixing build by removing sip headers for now

* api file update

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* remove comment

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* fix tests

* remove validation related unit tests

* fix tests

* remove comment

* resolve conflit

* fix merge conflicts

* make repeatabilityHeaders internal and generated automatically (Azure#33965)

* make repeatabilityHeaders internal and generated automatically

* remove repeatability in README

* fix tests

* integrate call invite to create call

* update auto gen

---------

Co-authored-by: Min Woo Lee 🧊 <77083090+minwoolee-ms@users.noreply.github.com>
Co-authored-by: Yingying Wu <98626395+yyw-msft@users.noreply.github.com>

* use call invite in redirect and fix tests (Azure#34037)

* use call invite in transfer (Azure#34083)

* use call invite in transfer

* adding unit tests for simple methods

* Remove call source for create call request (Azure#34039)

* add Call Invite class

* integrate call invite to create call

* fix tests

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* remove comment

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* fix tests

* remove comment

* resolve conflit

* fix merge conflicts

* integrate call invite to create call

* fix tests

* remove validation related unit tests

* integrate call invite to create call

* remove comment

* fix merge conflicts

* update swagger endpoint

* generate code from swagger

* Merge branch 'richardcho/create-call' of https://github.com/richardcho-msft/azure-sdk-for-net into richardcho/create-call

* fix linter

* update autogen'

* fix auto gen

* update autogen

* merge conflict

* remove duplicate code

* use call invite in transfer (Azure#34083)

* use call invite in transfer

* adding unit tests for simple methods

* integrate call invite to create call

* fix tests

* integrate call invite to create call

* remove comment

* fix merge conflicts

* update swagger endpoint

* generate code from swagger

* Merge branch 'richardcho/create-call' of https://github.com/richardcho-msft/azure-sdk-for-net into richardcho/create-call

* fix linter

* update autogen'

* fix auto gen

* update autogen

* update swagger

* Revert "update swagger"

This reverts commit ac523d7.

* update swagger ref

* fix build

* remove call source

---------

Co-authored-by: Yingying Wu <98626395+yyw-msft@users.noreply.github.com>

* Update AddParticipant & RemoveParticipant (Azure#34155)

* update add participant logic

* generate code based on new swagger

* remove obsolete fils

* add CallInvite options

* fix build

* add custom context

* generate latest code

* update remove participant

* fix tests

* fix tests

* fix tests

* update api

* fix failed test

* use sipheaders and voipheaders for transfer (Azure#34190)

* update based on latest swagger (Azure#34197)

* update based on latest swagger

* api changes

* fix transfer logic (Azure#34263)

* fix uri (Azure#34267)

* Address CreateCall bugs (Azure#34273)

* fix transfer logic

* fix createCall issues

* updates (Azure#34281)

* update call invite constructor (Azure#34334)

* update callInvite constructor

* create customcontext for callInvite by default

* James/update call invite (Azure#34341)

* modify customContextInternal Constructor

* generate code

* fix unit test

* fix codecheck

* Fixing automated tests (Azure#34348)

Co-authored-by: Min Woo Lee 🧊 <77083090+minwoolee-ms@users.noreply.github.com>

---------

Co-authored-by: huachuandeng <73561849+huachuandeng@users.noreply.github.com>
Co-authored-by: Min Woo Lee 🧊 <77083090+minwoolee-ms@users.noreply.github.com>
Co-authored-by: Yingying Wu <98626395+yyw-msft@users.noreply.github.com>
Co-authored-by: minwoolee-msft <77083090+minwoolee-msft@users.noreply.github.com>

* bump api-version

* Refactor statsbeat (Azure#34443)

* refactor statsbeat

* clean up

* fix statsbeat issue

* disable

* refactor transmitter

* rename

* resolve PR comments

* rmv using

* missed update

* Update sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/Statsbeat/Statsbeat.cs

Co-authored-by: Timothy Mothra <tilee@microsoft.com>

---------

Co-authored-by: Timothy Mothra <tilee@microsoft.com>

* Increment package version after release of Azure.Identity (Azure#34444)

* Fix typo "identifer" -> "identifier" (Azure#34498)

* Fix typo "identifer" -> "identifier"

* [Messaging Extensions] Release Prep (Feb 2023) (Azure#34477)

* [Messaging Extensions] Release Prep (Feb 2023)

The purpose of these changes is to prepare the Event Hubs and Service Bus
extensions packages for release.

* Removing project dependency temporarily

* Update sdk/eventhub/Microsoft.Azure.WebJobs.Extensions.EventHubs/CHANGELOG.md

Co-authored-by: Christopher Scott <chriscott@hotmail.com>

---------

Co-authored-by: Christopher Scott <chriscott@hotmail.com>

* Increment version for eventhub releases (Azure#34503)

Increment package version after release of Microsoft.Azure.WebJobs.Extensions.EventHubs

* Increment version for servicebus releases (Azure#34502)

Increment package version after release of Microsoft.Azure.WebJobs.Extensions.ServiceBus

* [Service Bus Extension] Restore project reference (Azure#34504)

The focus of these changes is to restore the project reference and the
in-flight work to support the isolated hosting model.

* [Text Analytics] Rename WellKnownFhirVersion enum to FhirVersion (Azure#34491)

* Azure OpenAI: fix streaming completions deserialization (Azure#34486)

* Azure OpenAI: fix streaming completions deserialization

* Refresh release date/changelog for retry after yesterday's pipeline infra issues

* Sync eng/common directory with azure-sdk-tools for PR 5431 Azure#2501

Co-authored-by: Konrad Jamrozik <kojamroz@microsoft.com>

* [AzureMonitorExporter] update demo project (Azure#34514)

* update demo project

* cleanup csproj

* update readme

* [Text Analytics] Disable checks that fail due to service issues (Azure#34521)

* [AzureMonitorOpenTelemetryDistro] AddAzureMonitorOpenTelemetry extension methods and AzureMonitorOpenTelemetryOptions (Azure#34198)

* Distro Public API

* PR feedback

* Update public API

* 4 Public APIs

* Update sdk/monitor/Azure.Monitor.OpenTelemetry/tests/Examples.AspNetCore/Examples.AspNetCore.csproj

* Fix build error for demo project.

* Fix error in sample

* Remove 461 packages

* 4 APIs in samples.

* Remove public api with parameters.

* PR feedback

* Update API

* Apply connectionstring from AzureMonitorOpenTelemetryOptions to AzureMonitorExporterOptions.

---------

Co-authored-by: Timothy Mothra <tilee@microsoft.com>

* Sync eng/common directory with azure-sdk-tools for PR 5562 (Azure#34522)

* Add todos to update packages to pick up the newest CODEOWNERS interpreter

* update

---------

Co-authored-by: Konrad Jamrozik <kojamroz@microsoft.com>

* [WebPubSub] upgrade to LTS net6.0 and GA (Azure#34461)

# Contributing to the Azure SDK

Please see our [CONTRIBUTING.md](https://github.com/Azure/azure-sdk-for-net/blob/main/CONTRIBUTING.md) if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see [this section](https://github.com/Azure/azure-sdk-for-net/blob/main/CONTRIBUTING.md#pull-request-etiquette-and-best-practices).

* Update AutoRest C# version to 3.0.0-beta.20230222.4 (Azure#34495)

Co-authored-by: archerzz <archerzz@gmail.com>

* Update AutoRest C# version to 3.0.0-beta.20230222.4 (Azure#34465)

* Update AutoRest C# version to 3.0.0-beta.20230223.4 (Azure#34524)

Co-authored-by: Mingzhe Huang <archerzz@users.noreply.github.com>

* Update AutoRest C# version to 3.0.0-beta.20230224.1 (Azure#34530)

* Successfully built using package 2023-04

---------

Co-authored-by: Jingshu923 <52914166+Jingshu923@users.noreply.github.com>
Co-authored-by: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com>
Co-authored-by: Sean McCullough <44180881+seanmcc-msft@users.noreply.github.com>
Co-authored-by: Timothy Mothra <tilee@microsoft.com>
Co-authored-by: Alexander Sher <AlexanderSher@users.noreply.github.com>
Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: Bhargav Kansagara <kansagara.bhargav1997@gmail.com>
Co-authored-by: Bhargav Kansagara <bhkansag@microsoft.com>
Co-authored-by: Nivedit Jain <niveditjain@microsoft.com>
Co-authored-by: Nivedit Jain <jain.22@iitj.ac.in>
Co-authored-by: Sebastian Herzig-Patel <seherzig@microsoft.com>
Co-authored-by: minnieliu <minnieliu96@hotmail.com>
Co-authored-by: Minnie Liu <peiliu@microsoft.com>
Co-authored-by: richardcho-msft <95773682+richardcho-msft@users.noreply.github.com>
Co-authored-by: huachuandeng <73561849+huachuandeng@users.noreply.github.com>
Co-authored-by: Min Woo Lee 🧊 <77083090+minwoolee-ms@users.noreply.github.com>
Co-authored-by: Yingying Wu <98626395+yyw-msft@users.noreply.github.com>
Co-authored-by: minwoolee-msft <77083090+minwoolee-msft@users.noreply.github.com>
Co-authored-by: Feng Zhou <fenzhou@microsoft.com>
Co-authored-by: Vishwesh Bankwar <vishweshbankwar@users.noreply.github.com>
Co-authored-by: Petr Švihlík <rocky.intel@gmail.com>
Co-authored-by: Jesse Squire <jsquire@microsoft.com>
Co-authored-by: Christopher Scott <chriscott@hotmail.com>
Co-authored-by: Jose Arriaga Maldonado <45773732+joseharriaga@users.noreply.github.com>
Co-authored-by: Travis Wilson <35748617+trrwilson@users.noreply.github.com>
Co-authored-by: Konrad Jamrozik <kojamroz@microsoft.com>
Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
Co-authored-by: JialinXin <jixin@microsoft.com>
Co-authored-by: archerzz <archerzz@gmail.com>
Co-authored-by: Mingzhe Huang <archerzz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vishweshbankwar vishweshbankwar vishweshbankwar approved these changes

@rajkumar-rangaraj rajkumar-rangaraj rajkumar-rangaraj approved these changes

@cijothomas cijothomas Awaiting requested review from cijothomas

@reyang reyang Awaiting requested review from reyang

Assignees
No one assigned
Labels
Monitor - Exporter Monitor OpenTelemetry Exporter
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@TimothyMothra @azure-sdk @rajkumar-rangaraj @vishweshbankwar