Skip to content

Conversation

@rayaisaiah
Copy link
Contributor

Reason for Change:
Backporting #3161, #3316, #3333, and #3337 for v1.5.41 release of NPM.

Issue Fixed:

Requirements:

Notes:

* test: fix windows unit test for policymanager

Signed-off-by: Hunter Gregory <[email protected]>

* fix(test): flip bool

Signed-off-by: Hunter Gregory <[email protected]>

---------

Signed-off-by: Hunter Gregory <[email protected]>
fix: close telemetry handler before crashing

Signed-off-by: Hunter Gregory <[email protected]>
* fix: time waiting for appinsights to close was unbounded

Signed-off-by: Hunter Gregory <[email protected]>

* fix: close timer in case it hasn't fired yet

Signed-off-by: Hunter Gregory <[email protected]>

---------

Signed-off-by: Hunter Gregory <[email protected]>
@rayaisaiah rayaisaiah requested review from a team as code owners January 14, 2025 00:57
@rayaisaiah
Copy link
Contributor Author

/azp list

@azure-pipelines
Copy link

@rayaisaiah
Copy link
Contributor Author

/azp run Azure Container Networking PR

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rayaisaiah
Copy link
Contributor Author

/azp run NPM Conformance Tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rayaisaiah
Copy link
Contributor Author

/azp run NPM Scale Test

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rayaisaiah
Copy link
Contributor Author

/azp run NPM Conformance Tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rayaisaiah rayaisaiah added this pull request to the merge queue Jan 15, 2025
Merged via the queue into release/v1.5 with commit b84dbfd Jan 15, 2025
33 checks passed
@rayaisaiah rayaisaiah deleted the backportNPMChangesForV1.5.41 branch January 15, 2025 20:02
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