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

[COST-5135] Update Masu endpoint to handle new ocp on cloud flow #5277

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cgoodfred
Copy link
Contributor

Jira Ticket

COST-5135

Description

This change will update the masu endpoint to handle the new managed ocp on cloud flow if its enabled via unleash.

Testing

  1. Checkout Branch
  2. Restart Koku
  3. Create a test customer
  4. Load your ocp data
  5. Load your matching aws data
  6. Enable the processing unleash flag cost-management.backend.feature-cost-5129-ocp-cloud-processing.
    *you can also avoid masu and the unleash flags by making the processing check flag return True temporarily.
  7. Hit the masu endpoint to process OCP on Cloud data replacing your provider and dates as needed: /api/cost-management/v1/report/process/openshift_on_cloud/?schema=org1234567&provider_uuid=58c94173-027e-4ccf-a41e-5f2b169e09b5&start_date=2024-08-01
  8. Check that there is now information in the managed_aws_openshift_daily table and it matches your expectations.

Release Notes

  • proposed release note
* [COST-5135](https://issues.redhat.com/browse/COST-5135) Update masu endpoint to handle new OCP on Cloud processing flow.

@cgoodfred cgoodfred added the smoke-tests pr_check will build the image and run minimal required smokes label Aug 30, 2024
@cgoodfred cgoodfred self-assigned this Aug 30, 2024
Copy link

codecov bot commented Aug 30, 2024

Codecov Report

Attention: Patch coverage is 33.33333% with 8 lines in your changes missing coverage. Please review.

Project coverage is 94.1%. Comparing base (2160741) to head (b3c963f).

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #5277     +/-   ##
=======================================
- Coverage   94.1%   94.1%   -0.0%     
=======================================
  Files        375     375             
  Lines      31505   31516     +11     
  Branches    4642    4644      +2     
=======================================
+ Hits       29636   29641      +5     
- Misses      1194    1198      +4     
- Partials     675     677      +2     

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.

1 participant