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

One approach to solve null values for subscription_name #4789

Merged
merged 3 commits into from
Jan 15, 2024

Conversation

bacciotti
Copy link
Contributor

Jira Ticket

COST-4313

Description

This change will replace the null values with subscription_guid value.

Testing

  1. Checkout Branch
  2. Restart Koku
  3. Hit endpoint
    http://localhost:8000/api/cost-management/v1/reports/azure/costs/?group_by[subscription_guid]=*&filter[limit]=100
  4. Check if there are null values on some subscription_name keys

@bacciotti bacciotti added the smoke-tests pr_check will build the image and run minimal required smokes label Nov 17, 2023
koku/api/report/queries.py Outdated Show resolved Hide resolved
@bacciotti bacciotti force-pushed the COST-4313-subscription-name-null branch from 9cdf0de to 5dfac9a Compare November 20, 2023 11:56
@bacciotti bacciotti marked this pull request as ready for review November 20, 2023 11:57
@bacciotti bacciotti requested review from a team as code owners November 20, 2023 11:57
Copy link

codecov bot commented Nov 20, 2023

Codecov Report

Merging #4789 (2891109) into main (03d5006) will increase coverage by 0.0%.
The diff coverage is 100.0%.

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #4789   +/-   ##
=====================================
  Coverage   94.0%   94.0%           
=====================================
  Files        365     365           
  Lines      30299   30310   +11     
  Branches    3603    3607    +4     
=====================================
+ Hits       28478   28493   +15     
+ Misses      1158    1156    -2     
+ Partials     663     661    -2     

lcouzens
lcouzens previously approved these changes Nov 20, 2023
@lcouzens
Copy link
Contributor

Just need to bump that codecov if possible 👍

@bacciotti bacciotti force-pushed the COST-4313-subscription-name-null branch 3 times, most recently from 2fe5186 to af6f9c3 Compare November 22, 2023 11:51
@bacciotti bacciotti force-pushed the COST-4313-subscription-name-null branch from b8ecd1a to f86c0bb Compare December 13, 2023 16:40
@bacciotti
Copy link
Contributor Author

/retest

@bacciotti bacciotti force-pushed the COST-4313-subscription-name-null branch 2 times, most recently from 1ab17a2 to cd225b4 Compare December 14, 2023 13:19
@bacciotti bacciotti force-pushed the COST-4313-subscription-name-null branch from f9dbcf5 to bf3450a Compare December 14, 2023 14:05
Copy link

sonarcloud bot commented Dec 14, 2023

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

2 New issues
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@bacciotti bacciotti enabled auto-merge (squash) December 14, 2023 14:19
Copy link

sonarcloud bot commented Jan 15, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

2 New issues
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@bacciotti bacciotti merged commit 85e6f9d into main Jan 15, 2024
10 checks passed
@bacciotti bacciotti deleted the COST-4313-subscription-name-null branch January 15, 2024 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
smoke-tests pr_check will build the image and run minimal required smokes smokes-required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants