Skip to content
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

Use newer API version for Azure Cost Management client #5158

Merged
merged 4 commits into from
Jun 10, 2024

Conversation

samdoran
Copy link
Contributor

@samdoran samdoran commented Jun 7, 2024

Jira Ticket

COST-5113

Description

The SDK defaults to using 2022-10-01 and there is a bug with the Azure API not properly returning data for V2 cost exports. Hopefully using the newer Azure API will work aruond the problem while Microsoft fixes the bug on their end.

Testing

  1. Checkout Branch
  2. Restart Koku
  3. Hit endpoint or launch shell
    1. You should see ...
  4. Do more things...

Release Notes

  • proposed release note
* [COST-5113](https://issues.redhat.com/browse/COST-5113) Use version 2023-07-01-preview for the Azure Cost Management client

The SDK defaults to using 2022-10-01 and there is a bug with the Azure API not
properly returning data for V2 cost exports. Hopefully using the newer Azure API
will work aruond the problem while Microsoft fixes the bug on their end.
@samdoran samdoran added the azure-smoke-tests pr_check will build the image and run azure + ocp on azure smoke tests label Jun 7, 2024
Copy link

codecov bot commented Jun 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.1%. Comparing base (318dc3b) to head (42697c4).

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #5158     +/-   ##
=======================================
- Coverage   94.1%   94.1%   -0.0%     
=======================================
  Files        375     375             
  Lines      31193   31194      +1     
  Branches    3727    3727             
=======================================
- Hits       29361   29360      -1     
- Misses      1167    1168      +1     
- Partials     665     666      +1     

@lcouzens lcouzens marked this pull request as ready for review June 10, 2024 07:55
@lcouzens lcouzens requested review from a team as code owners June 10, 2024 07:55
@samdoran samdoran merged commit e2f0934 into main Jun 10, 2024
11 checks passed
@samdoran samdoran deleted the azure-specify-api-version branch June 10, 2024 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
azure-smoke-tests pr_check will build the image and run azure + ocp on azure smoke tests smokes-required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants