Users/v vdharmaraj/merge from main to beta4 #31616
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixes #23329 - Added SHR PoP token support to @azure/core-auth - Added SHR PoP token support to `InteractiveBrowserCredential` Native Broker Scenarios via @azure/identity and @azure/identity-broker
### Packages impacted by this PR [monitor-opentelemetry] ### Describe the problem that is addressed by this PR See: https://github.com/aep-health-and-standards/Telemetry-Collection-Spec/blob/main/ApplicationInsights/livemetrics.md#process-metrics ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? For CPU, looked at [.net](https://github.com/Azure/azure-sdk-for-net/blob/7c8f16deec3a1150bfa04ec372d3abe5b238dd18/sdk/monitor/Azure.Monitor.OpenTelemetry.AspNetCore/src/LiveMetrics/Manager.Metrics.cs#L290) & [java](https://github.com/Azure/azure-sdk-for-java/blob/43bdc30cb296915ec22aa46ba1e7d885e124f6b4/sdk/monitor/azure-monitor-opentelemetry-exporter/src/main/java/com/azure/monitor/opentelemetry/exporter/implementation/quickpulse/QuickPulseDataCollector.java#L358) implementations, along with this [stack overflow](https://stackoverflow.com/questions/74776323/trying-to-get-normalized-cpu-usage-for-node-process#:~:text=I'm%20trying%20to%20calculate%20the%20normalized%20cpu%20percentage%20for%20my). They all seem to be following the same rough calculation & multiplying by 100 in the end for a percentage. ### Are there test cases added in this PR? _(If not, why?)_ Modified an existing test to assert that the body of the post request includes old/new counter names with the process counter values. ### Checklists - [x] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [x] Added a changelog (if necessary)
Address remaining CODEOWNERS issues - for areas with no good owners I have removed them, same with folks who I contacted to set themselves public but did not respond.
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9147 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Patrick Hallisey <[email protected]>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9202 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Ben Broderick Phillips <[email protected]>
…1438) Sync .github/workflows directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9199 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Ben Broderick Phillips <[email protected]>
…ath (#31361) playing with the idea of exposing features under the `./experimental` subpath and add a dev-tool command to report error if we consume experimental Apis in a GA package. [ServiceBus] Add the experimental diagnostic feature omitMessageBody via an augmented `PeekMessagesOptions` interface under `./experimental` subpath export. After migrating to ESM, we no longer use rolled-up dts files thus lose the ability to trim `@beta` Apis from public type definition files. This PR explores an approach to expose Apis from the `./experimental` export.
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9179 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Scott Beddall <[email protected]>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9177 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Daniel Jurek <[email protected]>
### Packages impacted by this PR ` @azure/microsoft-playwright-testing` ### Issues associated with this PR added RUNNAME field in playwright service config. ### Describe the problem that is addressed by this PR ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary) --------- Co-authored-by: Siddharth Singha Roy <[email protected]>
### Packages impacted by this PR ` @azure/microsoft-playwright-testing` ### Issues associated with this PR ### Describe the problem that is addressed by this PR ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
Post release automated changes for azure-keyvault-common
### Packages impacted by this PR @azure-rest/ai-vision-image-analysis ### Issues associated with this PR N/A ### Describe the problem that is addressed by this PR Stable Release ### Are there test cases added in this PR? _(If not, why?)_ No changes.
Post release automated changes for azure-identity
* save-package-properties.yml only activates PR diff mechanism if the service directory is "auto" --------- Co-authored-by: Scott Beddall <[email protected]>
### Packages impacted by this PR @azure/microsoft-playwright-testing ### Issues associated with this PR ### Describe the problem that is addressed by this PR ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9212 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Ben Broderick Phillips <[email protected]>
…31451) Closes #27781 Add subscription property for AzureCliCredentialOptions --------- Co-authored-by: Charles Lowell <[email protected]>
…31466) - replace version string with ~8.10.0 - rush update - [monitor-opentelemetry-exporter] fixing linting error of unnecessary `undefine` type for the optional parameter
Drive traffic to the new credential chains doc on Learn. Also, eliminate duplication in the `DefaultAzureCredential` API docs. --------- Co-authored-by: Karishma Ghiya <[email protected]>
Post release automated changes for azure-arm-redisenterprisecache
…soft-playwright-testing (#31477) ### Packages impacted by this PR @azure/microsoft-playwright-testing ### Issues associated with this PR ### Describe the problem that is addressed by this PR ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
Post release automated changes for azure-microsoft-playwright-testing
<p dir="auto">This is an automatic PR generated weekly with changes from running the command rush update --full</p>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9216 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Wes Haggard <[email protected]>
### Packages impacted by this PR No code changes. Only sample and readme ### Issues associated with this PR Sample has scope incorrect for connection ### Describe the problem that is addressed by this PR Sample has scope incorrect for connection ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
### Packages impacted by this PR - `@typespec/ts-http-runtime` ### Describe the problem that is addressed by this PR We have had issues with the unbranded Core package becoming out of sync with the Azure packages. This PR adds a report which is a diff between the Azure Core packages and the unbranded package and causes the lint step to fail if there are any differences between Azure and unbranded that do not appear in the report. Running the `lint:fix` script will update the diff report. This will mean that: - Making an Azure Core change without a corresponding change to unbranded package, either in the source code or in the diff report, will result in a CI failure - Changes to the diff report can be reviewed so that any differences between Azure and unbranded implementations of a feature can be easily compared Why use the lint script (and lint:fix)? This is already run by CI, so by using it we don't need to make any changes to pipeline steps to get this working. This script is a one-off and so I just want to do the bare minimum to get things working 😄
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9229 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Wes Haggard <[email protected]>
…1487) ### Packages impacted by this PR - `@typespec/ts-http-runtime` ### Describe the problem that is addressed by this PR Updating the unbranded Core package with some recent changes in Core that didn't make it over.
### Packages impacted by this PR @azure/attestation ### Issues associated with this PR #31338 ### Describe the problem that is addressed by this PR Migrates `@azure/attestation` to ESM and tshy. Co-authored-by: Deyaaeldeen Almahallawi <[email protected]>
v-vdharmaraj
requested review from
AikoBB,
maximrytych-ms,
mjafferi-msft,
miguhern,
whisper6284,
gfeitosa-msft,
phermanov-msft,
ilyapaliakou-msft,
anujissarMS,
minnieliu,
paolamvhz,
shwali-msft,
allchiang-msft,
danielav7,
marche0133,
yogeshmo,
LuChen-Microsoft,
akania,
wiboris,
wanghoppe,
dpwatrous,
cRui861,
skapur12,
LarryOsterman,
anilba06,
gkostal,
HarshaNalluru,
minhanh-phan and
maorleger
as code owners
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe the problem that is addressed by this PR
Merging from main branch to beta4 branch