-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[Key Vault] Custom polling method for admin backup client #19204
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
Merged
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
a11223e
rehydration, but no .result()
laiapat dd32026
poll-able rehydration
laiapat 520ba1b
passing a/sync backup rehydration
laiapat 79e20c6
cleaned up; rehydrated and regular poller success
laiapat e27d134
base64 encoded cont tokens
laiapat dc742f9
cleaned up
laiapat afbf7ea
poller testing
laiapat b85fbe3
resolve flaky test
laiapat d945cd3
live-only sync poller test
laiapat 1efc09e
remove private core refs
laiapat 19a00f3
override public run()
laiapat 9e0fa34
pylint; remove test flakiness
laiapat 284d5af
pylint; clean request_status
laiapat ea4f4c1
tests passing after rebase
laiapat 2512b42
fix super() calls
laiapat a131385
get pipeline response w/ cont token
laiapat ef47092
refine tests, add examples
laiapat 0eff710
examples in docstrings
laiapat 20a300c
doc and pylint fixes
laiapat cf25535
use *_status; update tests
laiapat 2ba1ea2
Address feedback
laiapat 86efeb6
use poller.continuation_token()
laiapat 78ccc8d
Thanks, Charles!
laiapat 54f6575
appease pylint
laiapat 86b4e41
get status on rehydration
laiapat ef75b1a
simplify set_initial_status
laiapat b728a58
don't depend on _async_url
laiapat File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
12 changes: 12 additions & 0 deletions
12
...lt/azure-keyvault-administration/azure/keyvault/administration/_internal/async_polling.py
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| # ------------------------------------ | ||
| # Copyright (c) Microsoft Corporation. | ||
| # Licensed under the MIT License. | ||
| # ------------------------------------ | ||
| import base64 | ||
|
|
||
| from azure.core.polling.async_base_polling import AsyncLROBasePolling | ||
|
|
||
|
|
||
| class KeyVaultAsyncBackupClientPollingMethod(AsyncLROBasePolling): | ||
| def get_continuation_token(self) -> str: | ||
| return base64.b64encode(self._operation.get_polling_url().encode()).decode("ascii") |
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.