-
Notifications
You must be signed in to change notification settings - Fork 33
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
New scheduled workflow to test using the azcore backend nightly #3590
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
Contributor
thomas11
commented
Sep 25, 2024
- Initial sketch of using azcore client
- Adjust to recent AzureClient API changes
- Initial implementations of CanCreate, Head, Patch, Post
- Use new azcore client in provider
- TokenCredential bridge: use correct environment
- Use the existing auth stack in the azcore client
- Fix relative URLs and missing API version in Location header for polling
- Fix YAML test program capitalization
- Handle empty PUT body and async polling for completion. TestApi passes.
- Port handling of 404 when polling for deletion
- Upgrade azcore to 1.14
- Fix older bug in CanCreate
- Share code between PUT and PATCH
- Send correct provider version with requests
- Use the correct cloud/environment
- Beginnings of unit testing the new azcore client's HTTP request/response handling
- Add escape hatch to revert to Autorest
- Set Content-Type json header
- Fix some polling-related comments
- Bug fix and tests for CanCreate URL paths
- Set polling frequency to 30s like autorest
- Port handleResponseError
- Verbose request/response logging when log level >= 9
- Configure retries to be more like in Autorest
- Comment on relative Location URLs
- Correctly handle query params
- Remove usage of autorest RequestError in provider
- Propagate azCoreClient creation error; don't create a preliminary AzureClient
- Switch default Azure client to Autorest and add test
- Extend error handling test coverage
- Unit test for HTTP request headers
- Unit tests for CanCreate
- Test that CanCreate uses resource id and path
- Test polling for PUT and PATCH
- Test coverage and fixes for DELETE polling
- Round out test coverage
- comment cleanup
- Prevent double-escaped URL query params
- Some preliminary cleanup
- WIP custom static website, probably using the wrong azblob
- WIP custom static website, using the right azblob, wired up and compiles but untested
- WIP failed attempt to address the 'wrong audience' issue in TestStaticWebsiteDestroyTs
- Revert "WIP failed attempt to address the 'wrong audience' issue in TestStaticWebsiteDestroyTs"
- Beginnings of azidentity auth, CLI so far
- Add some more auth types, beginnings of separate auth config getter
- Check that autorest is enabled for legacy auth
- WIP chained to single credential, parse certs
- New scheduled workflow to test using the azcore backend nightly
Set user agent
…cWebsiteDestroyTs
…estStaticWebsiteDestroyTs" This reverts commit c3520d3.
thomas11
force-pushed
the
tkappler/azcore-cron
branch
from
September 25, 2024 14:37
2e84a13
to
21a2924
Compare
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.