-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[7.17](backport #37158) [Azure] [Billing] Switch to Consumption API 1.1.0 with patches #37244
base: 7.17
Are you sure you want to change the base?
Conversation
* Switch to consumption 1.1.0 with patches The Consumption SDK does not URL encode the query parameters in the next link when the response requires more than one page to fetch all the usage details items. Our Azure SDK for Go fork contains a small patch to URL encode the next link value and avoid the 400 error. We also upgraded the azcore package to use the standard URL helper for the Azure SDK. * Update NOTICE.txt * Target tag v1.1.0-elastic instead of the commit Tidy up the go.mod file by targeting the tag `v1.1.0-elastic` on the Azure SDK fork instead of the commit. (cherry picked from commit 9de3307) # Conflicts: # NOTICE.txt # go.mod # go.sum
💔 Build Failed
Expand to view the summary
Build stats
Pipeline error
❕ Flaky test reportNo test was executed to be analysed. 🤖 GitHub commentsExpand to view the GitHub comments
To re-run your PR in the CI, just comment with:
|
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
14 similar comments
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
7 similar comments
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request is now in conflicts. Could you fix it? 🙏
|
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
2 similar comments
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
This pull request is now in conflicts. Could you fix it? 🙏
|
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
1 similar comment
This pull request has not been merged yet. Could you please review and merge it @zmoog? 🙏 |
@zmoog this one has been opened for a while, what is the plan with this PR? |
Yeah, other priorities are keeping this one behind. It's time to bring it to the finish line. |
This is an automatic backport of pull request #37158 done by Mergify.
Cherry-pick of 9de3307 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally
Mergify commands and options
More conditions and actions can be found in the documentation.
You can also trigger Mergify actions by commenting on this pull request:
@Mergifyio refresh
will re-evaluate the rules@Mergifyio rebase
will rebase this PR on its base branch@Mergifyio update
will merge the base branch into this PR@Mergifyio backport <destination>
will backport this PR on<destination>
branchAdditionally, on Mergify dashboard you can:
Finally, you can contact us on https://mergify.com