Yield429 #11
Draft
Yield429 #11
Conversation
This file contains hidden or 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
* implementation and some tests * wip * batch api tests * container api tests * wip * wrapping up helper tests * adding blob tests * adding datalake tests * fixing azurite failure * reverting parseNonIpUrl change * updating recordings and resolving errors * removing unecessary encode * addressing comments wip * wip * wip * addressing comments * adding onelake compatibility test * making options bag test more official * swapping onelake test to live only * removing onelake test * addressing last couple of comments
…olicy (Azure#46685) * removing override that forces token fetch on initial request * scope refactoring * removing unecessary elements in signatures * refactoring authorizeRequestOnChallenge overrides * refactoring and strengthening parsing logic * some tests * fixing failing tests * adding overrides back to prevent breaking change * Update sdk/storage/azure-storage-common/src/main/java/com/azure/storage/common/policy/StorageBearerTokenChallengeAuthorizationPolicy.java Co-authored-by: Copilot <[email protected]> * Update sdk/storage/azure-storage-common/src/test/java/com/azure/storage/common/policy/StorageBearerTokenChallengeAuthorizationPolicyTests.java Co-authored-by: Copilot <[email protected]> * using CoreUtils.parseAuthenticateHeader instead of custom util method * adding changelog entry * adding scope and tenant verification * wip * added comment about failure * removing warning * adding custom parsing back and adjusting tests * many many tests for the parsing logic * undoing new parsing changes * moving stuff around * adjusting tests * more test adjustment * formatting --------- Co-authored-by: Copilot <[email protected]>
…ent instance detection during test execution (Azure#47211) * Test changes to add leak detection * Update Configs.java * Update CosmosNettyLeakDetectorFactory.java * Update sdk/cosmos/azure-cosmos-encryption/src/test/java/com/azure/cosmos/encryption/CosmosNettyLeakDetectorFactory.java Co-authored-by: Copilot <[email protected]> * Update CosmosNettyLeakDetectorFactory.java * Fixes * Fixes * Update CosmosNettyLeakDetectorFactory.java * Update RxDocumentClientImpl.java * Fixes * Update CosmosNettyLeakDetectorFactory.java * Fixes * Fixes * Iterating on tests * Fixing build warning * Fixing memory leak * Reverting production changes * Iterating on test tools * Cleaning-up dummy QueryFeedRangeState properly * Update test-resources.json * Update sdk/cosmos/azure-cosmos-tests/src/test/java/com/azure/cosmos/CosmosDiagnosticsE2ETest.java Co-authored-by: Copilot <[email protected]> * [WIP] Fix Netty buffer and RxDocumentClientImpl leaks (Azure#47213) * Initial plan * Improve JavaDoc phrasing in RxDocumentClientImpl Co-authored-by: FabianMeiswinkel <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: FabianMeiswinkel <[email protected]> Co-authored-by: Fabian Meiswinkel <[email protected]> * NITs * Move static block to class level in cosmos-encryption TestSuiteBase (Azure#47216) * Initial plan * Move CosmosNettyLeakDetectorFactory.ingestIntoNetty() to class-level static block Co-authored-by: FabianMeiswinkel <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: FabianMeiswinkel <[email protected]> * Update SessionTest.java * Update tests.yml * Update CosmosNettyLeakDetectorFactory.java * Test config * Update CosmosNettyLeakDetectorFactory.java * Updating TestNG * Reverting TestNG to 7.9.0 (highest version still supporting Java8) * Switching back to TestNG 7.5.1 * Enabling leak detection in unit tests * Iterating on tests * Update pom.xml * Test changes (Azure#47233) * Update RntbdTransportClientTest.java * Updating netty leak detection system properties * Update CosmosNettyLeakDetectorFactory.java * Test changes * Prod memory leak fixes * Test fixes * Test fixes * Users/fabianm/portfixes (Azure#47252) * Update WebExceptionRetryPolicy.java * Update ThinClientStoreModel.java * Test fixes * Fix Netty ByteBuf leaks in StoreResponse and RetryContextOnDiagnosticTest (Azure#47266) * Initial plan * Improve logging for ByteBufInputStream close failures Change log level from debug to warn and catch Throwable instead of just IOException to make potential ByteBuf leak issues more visible. Co-authored-by: FabianMeiswinkel <[email protected]> * Fix ByteBuf leak in RetryContextOnDiagnosticTest Changed from Mono.just() to Mono.fromCallable() to defer StoreResponse creation, ensuring ByteBuf lifecycle is properly managed within each subscription rather than eagerly at mock setup time. Co-authored-by: FabianMeiswinkel <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: FabianMeiswinkel <[email protected]> * Update EncryptionAsyncApiCrudTest.java * Fix ByteBuf memory leak in TcpServerMock request decoders (Azure#47269) * Initial plan * Fix memory leak in ServerRntbdRequestDecoder and ServerRntbdContextRequestDecoder When overriding channelRead() in ByteToMessageDecoder and bypassing the parent's decode logic by calling context.fireChannelRead() directly, the ByteBuf reference must be explicitly released to prevent memory leaks. Added ReferenceCountUtil.release() in try-finally blocks to properly manage buffer lifecycle. Co-authored-by: FabianMeiswinkel <[email protected]> * Replace ReferenceCountUtil.release with safeRelease * Replace ReferenceCountUtil.release with safeRelease --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: FabianMeiswinkel <[email protected]> Co-authored-by: Fabian Meiswinkel <[email protected]> * Update ThinClientStoreModel.java * Fixing test issues * Disable netty leak detection in RetrycontextOnDiagnosticTest * Test and diagnostics improvements * Test fixes and more breadcrumbs * Test fixes * Test fixes * Test fixes * Update OrderbyDocumentQueryTest.java * Test fixes * Reverting too noisy logs * Test fixes * Fixing POMs * React to code review feedback * Update pom.xml * Addresses code review feedback * Update sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/directconnectivity/rntbd/RntbdRequestDecoder.java Co-authored-by: Annie Liang <[email protected]> --------- Co-authored-by: Copilot <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: FabianMeiswinkel <[email protected]> Co-authored-by: Annie Liang <[email protected]>
* feat(network): update api spec to package-2025-03-01 - Updated network resource manager API specification - Bumped package version from 2025-01-01 to 2025-03-01 - Maintained existing inner class configurations - Preserved sync stack disabled setting - Kept deprecation note for ApplicationGatewaySku * gulp codegen * release(network): update azure-resourcemanager-network to version 2.57.0 - Updated version in version_client.txt from 2.56.0 to 2.57.0 - Updated README.md to reflect new stable version 2.57.0 - Updated pom.xml to use stable version 2.57.0 instead of beta - Updated azure-resourcemanager dependency reference to 2.57.0 * chore(network): update api-version to 2025-03-01 - Updated `api-version` to `2025-03-01` in network changelog - Aligned resourcemanager dependency with latest api-version - Removed beta release notes and marked as stable release - Consolidated changelog entries under dependency updates section * chore(assets): update network manager asset tag - Updated asset tag from previous version to latest - Maintains consistency with azure resource manager network module - No functional changes, only metadata update * feat(versioning): add azure-resourcemanager-cosmos version entry - Added unreleased version entry for azure-resourcemanager-cosmos - Set version to 2.54.0 in version_client.txt - Maintained consistent formatting with other entries * chore(versioning): remove cosmos resource manager beta version entry - Removed unreleased Cosmos Resource Manager version entry - Cleaned up version client tracking file - Updated dependency version management list * chore(revapi): update revapi configuration to ignore removed fields - Ignore removal of ApplicationGatewayWafRuleSensitivityTypes.NONE - Ignore removal of SensitivityType.NONE - Mark removed fields as intentionally cleaned up in newer version * fix(revapi): remove unnecessary ignore flags for removed fields - Removed redundant 'ignore' flag for ApplicationGatewayWafRuleSensitivityTypes.NONE - Removed redundant 'ignore' flag for SensitivityType.NONE - Cleanup obsolete field removal justifications - Ensure proper detection of breaking changes in API evolution - Maintain accurate API compatibility checking configuration * fix(network): update justification for removed sensitivity fields - Update justification for ApplicationGatewayWafRuleSensitivityTypes.NONE removal - Update justification for SensitivityType.NONE removal - Both fields now specify removal due to DDoS ruleset changes instead of general cleanup
Bumps [cspell](https://github.com/streetsidesoftware/cspell/tree/HEAD/packages/cspell) from 9.2.1 to 9.3.2. - [Release notes](https://github.com/streetsidesoftware/cspell/releases) - [Changelog](https://github.com/streetsidesoftware/cspell/blob/main/packages/cspell/CHANGELOG.md) - [Commits](https://github.com/streetsidesoftware/cspell/commits/v9.3.2/packages/cspell) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* [VoiceLive] Fix MCP error * update code owner * update spell words * update format --------- Co-authored-by: Xiting Zhang <[email protected]>
) * Used another way to construct package info array * Filtered out empty string for PackageInfoFiles --------- Co-authored-by: ray chen <[email protected]>
…zure#47335) * Passed the package info for apireview and validate package pipeline * Added test pipeline variable to patch release pipeline * Fixed JSON conversion error on windows os * Used compile time variable format * Escaped forward slash in path * Used parameter instead * Escaped forward slash * replaced the backslash * Updated test pipeline input * Updated yml to use the array object * Dynamic loaded the artifact for set test pipeline version template * Apply suggestions from code review * Apply suggestions from code review * Added display name for testPipeline parameter * Reverted change of set-test-pipeline-version template reference
…xibleserver-2025-08-01 (Azure#47328) * [Automation] External Change * [Automation] Generate Fluent Lite from Swagger postgresql#package-flexibleserver-2025-08-01 * fix test after break * simplify changelog * nit, changelog --------- Co-authored-by: Weidong Xu <[email protected]>
) * Enforce an array for single element * Enforced an array for where filter --------- Co-authored-by: ray chen <[email protected]>
* Enables EndpointValidation * Update Configs.java * Fixing test failures * Update TestSuiteBase.java * Create InvalidHostnameTest.java * Update InvalidHostnameTest.java * Updating changelog * Updated changelogs * Adding tests for direct mode * Fixing test flakiness due to Client leak * Fixed changelog * Iterating on test fixes * Adding more memory related logs * Iterating on test fixes * Revert unnecessary changes * Update TestSuiteBase.java * Update TestSuiteBase.java * Update TestSuiteBase.java * Update log4j2-test.properties * Update RxDocumentClientImpl.java * Update InvalidHostnameTest.java * Update CosmosDiagnosticsTest.java
…manager/Microsoft.ServiceFabric/ServiceFabricManagedClusters/tspconfig.yaml', API Version: 2025-10-01-preview, SDK Release Type: beta, and CommitSHA: '833aeb9992144f6e04d99de1316a7f37a001ee94' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5554955 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release. (Azure#47219)
) * Include groupId in the pkg name when create api view request * Used the package info in the function signature --------- Co-authored-by: ray chen <[email protected]>
Co-authored-by: azure-sdk <[email protected]>
* prepare 2.57.0 * pom.xml * skip record for some CI failure tests * more recording skips * more recording skips * more recording skips
…e#47232) * Enable 410-1022 on Head requests to bail out. * Bail fast on barrier on reads. * Enhance tests to ensure primary is contacted and barrier request count doesn't exceed a certain threshold. * Enhance tests to ensure barrier post the QuorumSelected phase is invoked. * Code comments * Adding a way to run tests against a multi-region Strong account. * Adding a way to run tests against a multi-region Strong account. * Validate sub-status code too. * Code cleanup. * Add CHANGELOG.md entry. * Modify barrier hit criteria. * Modify barrier hit criteria. * Add tests for barrier bail out in Bounded Staleness consistency. * Add tests for barrier bail out in Bounded Staleness consistency. * Refactoring * Addressing code comments. * Verify write barrier criteria. * Verify barrier bail out criteria. * Verify barrier bail out criteria. * Managing merge. * Fix compilation errors. * Fix tests. * Fix tests. * Addressing comments. * Addressing comments. * Addressing review comments. * Refactoring. * Addressing review comments. * Addressing review comments. * Addressing review comments.
…into yield429 # Conflicts: # sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/directconnectivity/ConsistencyWriter.java # sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/directconnectivity/QuorumReader.java
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
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.
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines