-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Azure Monitor OpenTelemetry] Add Azure Functions Correlation Hook #26313
Conversation
API change check API changes are not detected in this pull request. |
this._functionsCoreModule = undefined; | ||
} | ||
|
||
private _addPreInvocationHook() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing wrong since I see this is the same way tracking context in Azure Functions in the beta works, but I'm just wondering for my understanding why we can drop the _addPostInvocationHook()
method when we move away from the legacy SDK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pre invocation is used to maintain correlation and Post Invocation is used to automatically generate telemetry, for auto generation of telemetry we need to have further discussions, that is why is not part of this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
[Hub Generated] Review request for Microsoft.ContainerService/aks to add version preview/2023-09-02-preview (Azure#26313) * Adds base for updating Microsoft.ContainerService/aks from version preview/2023-08-02-preview to version 2023-09-02-preview * Updates readme * Updates API version in new specs and examples * OSSKU supports WindowsAnnual option (Azure#26120) * add VirtualMachines agent pool type (Azure#26034) * GPU Profile and Properties for AKS Windows (and Linux) GPU Feature (Azure#26072) * switch file and fix typos * Update specification/containerservice/resource-manager/Microsoft.ContainerService/aks/preview/2023-09-02-preview/managedClusters.json Co-authored-by: Matthew Christopher <matthchr@users.noreply.github.com> --------- Co-authored-by: Matthew Christopher <matthchr@users.noreply.github.com> * add AgentPoolUpgradeSettings.nodeSoakDurationInMinutes (Azure#26046) * Update format for trustedaccess sourceResourceId (Azure#26123) * update readmes (Azure#26126) * Add NodeAutoProvisioning to AKS preview API (Azure#26047) * Add AIToolchainOperator to AKS preview API (Azure#26151) * Add AIToolchainOperator to AKS preview API Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com> * Remove custom name from the PR Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com> * prettier fix --------- Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com> Co-authored-by: Heba Elayoty <hebaelayoty@gmail.com> * Add ArtifactStreamingProfile and Enabled property (Azure#26144) * add ArtifactStreamingProfile to AKS API * reduce repetition * Update specification/containerservice/resource-manager/Microsoft.ContainerService/aks/preview/2023-09-02-preview/managedClusters.json --------- Co-authored-by: FumingZhang <81607949+FumingZhang@users.noreply.github.com> * fix typo (Azure#26152) * upgrade common types (Azure#26232) * rename enum to nodeOSUpgradeChannel (Azure#26366) --------- Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com> Co-authored-by: Junjie Zhang <109257558+junjiezhang1997@users.noreply.github.com> Co-authored-by: Qingqing <qizhe@microsoft.com> Co-authored-by: Chris Lopez <91150577+pineapplethevoyager@users.noreply.github.com> Co-authored-by: Matthew Christopher <matthchr@users.noreply.github.com> Co-authored-by: Max Horstmann <MaxHorstmann@users.noreply.github.com> Co-authored-by: Dong Liu <doliu@microsoft.com> Co-authored-by: Heba Elayoty <hebaelayoty@gmail.com> Co-authored-by: Ganeshkumar Ashokavardhanan <35557827+ganeshkumar5699@users.noreply.github.com>
Functionality for @azure/monitor-opentelemetry package
Adding Azure Fn code to handle auto correlation when running in that environment