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

move gcp local create_daily_archives return outside for loop #5112

Merged
merged 2 commits into from
May 20, 2024

Conversation

lcouzens
Copy link
Contributor

@lcouzens lcouzens commented May 16, 2024

Jira Ticket

COST-####

Description

This change will move the return outside the running loop within the local gcp create_daily_archives function

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-####](https://issues.redhat.com/browse/COST-####) Fix some things

@lcouzens lcouzens requested review from a team as code owners May 16, 2024 13:34
@lcouzens lcouzens added the hot-fix-smoke-tests pr_check label to run minimal smoke tests for fast moving bug-fix label May 16, 2024
Copy link

codecov bot commented May 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.2%. Comparing base (8c696cd) to head (a60979c).

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #5112     +/-   ##
=======================================
- Coverage   94.2%   94.2%   -0.0%     
=======================================
  Files        378     378             
  Lines      31624   31624             
  Branches    3756    3756             
=======================================
- Hits       29780   29778      -2     
- Misses      1176    1177      +1     
- Partials     668     669      +1     

@lcouzens lcouzens enabled auto-merge (squash) May 20, 2024 09:30
@lcouzens lcouzens merged commit edc3db6 into main May 20, 2024
11 checks passed
@lcouzens lcouzens deleted the Fix-local-gcp-downloader branch May 20, 2024 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hot-fix-smoke-tests pr_check label to run minimal smoke tests for fast moving bug-fix smokes-required
Projects
None yet
2 participants