-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Revert "[EventHub] Raise error when primary key is rotated" #37587
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
This reverts commit 3b2dfa4.
kashifkhan
approved these changes
Sep 26, 2024
API change check API changes are not detected in this pull request. |
* evaluator config * add a change log entry * fix changelog wording * fix another thing in changelog * fix typing issue * add to __init__ * fix some tests * fix a spell check issue * fix tests again * fix last failing test * clean up changelog
* fix livetest targeting * remove extra parameter
/azp run python - eventhub - tests |
Azure Pipelines successfully started running 1 pipeline(s). |
Co-authored-by: James Suplizio <[email protected]>
Setter and getter docstrings were reversed.
* Update prompty * use a non preview version of API * Remove odb * Remove prints * Update tests and fix the type conversion of response * Reformmated with tox run -e black -c ../../../eng/tox/tox.ini -- . * Update test --------- Co-authored-by: Nagkumar Arkalgud <[email protected]>
* encode the fact that azure-eventhub does not support pypy machines * add azure-sdk-tools test workflow * update cspell to account for new workflow * fix matrix generation when no direct packages are present * fix a test relying on folder ordering to NOT depend on that folder ordering
…csOnboarding (#37588) Co-authored-by: Daniel Jurek <[email protected]>
* fix statistics * remove useless code * remove useless env var * remove useless code
…SDK owner) (#37563) * code and test * update-testcase * Update CHANGELOG.md * Update CHANGELOG.md --------- Co-authored-by: azure-sdk <PythonSdkPipelines> Co-authored-by: ChenxiJiang333 <[email protected]> Co-authored-by: ChenxiJiang333 <[email protected]> Co-authored-by: Yuchao Yan <[email protected]>
* ensure the default isn't "scan the entire repo"
Examples were added to demonstrate how to authenticate with a user-assigned managed identity using and object ID and resource ID. Signed-off-by: Paul Van Eck <[email protected]>
* apply black and prepare for parsing the ci.yml so that we can understand if testproxy is enabled or disabled * The language specific cert trust in Language-Settings has been updated to no longer globally set environment variables SSL_CERT_DIR, SSL_CERT_FILE, and REQUEST_CA_BUNDLE. These are dynamically set in proxy_startup.py
…37619) * filter the trigger for the workflow to only changes that include azure-sdk-tools * extend the number of directories that can trigger the 'other' packages properly
* disable pypy running against azure-storage-file-share, azure-storage-file-datalake, azure-storage-queue
…tps://github.com/Azure/azure-sdk-for-python into revert-36245-swathipil/eh/rotate-primary-key-bug
swathipil
deleted the
revert-36245-swathipil/eh/rotate-primary-key-bug
branch
September 30, 2024 15:12
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.
Reverts #36245