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

Add support for IBM Cloud Billing Converter #360

Merged

Conversation

chrisw-ibm
Copy link
Contributor

Support for using the converter to map IBM Cloud Billing Report CSV to the FOCUS specification.

  • An anonymized billing sample has been added for testing purposes

  • Unit test added for the billing csv

  • The data_loader was modified to support IBM Cloud's non-standard csv format which contains a header at the top of the CSV. This metadata is duplicated and concatenated to the billing line items.

  • progress and conversion rules export reports have been added

add ibm to progress readme

Signed-off-by: Chris Waddington <104161708+chrisw-ibm@users.noreply.github.com>
@chrisw-ibm chrisw-ibm force-pushed the add-ibm-cloud-billing-converter branch from e31a092 to aaac6b2 Compare August 26, 2024 16:15
@varunmittal91
Copy link
Collaborator

@chrisw-ibm thank you this is great. I approved run, it's builing as soon as done I can merge it back.

@chrisw-ibm
Copy link
Contributor Author

@varunmittal91 I pushed the fix for the linting error on the import order. Can you approve the run again :)

Signed-off-by: Chris Waddington <104161708+chrisw-ibm@users.noreply.github.com>
@chrisw-ibm chrisw-ibm force-pushed the add-ibm-cloud-billing-converter branch from 5452a07 to 23f1972 Compare August 26, 2024 16:32
@varunmittal91
Copy link
Collaborator

@chrisw-ibm sorry for the back and forth
here's the patch on the formatting error
formatting.patch

Signed-off-by: Chris Waddington <104161708+chrisw-ibm@users.noreply.github.com>
@chrisw-ibm
Copy link
Contributor Author

Thanks for the patch. All checks pass now!

@varunmittal91 varunmittal91 merged commit d97ceb0 into finopsfoundation:dev Aug 30, 2024
26 checks passed
@varunmittal91
Copy link
Collaborator

Thanks for the patch. All checks pass now!

Thank you so much @chrisw-ibm, this is awesome.

@varunmittal91
Copy link
Collaborator

Thanks for the patch. All checks pass now!

Thank you so much @chrisw-ibm, this is awesome.

Hey also realized we need to add IBM to supported list of cloud vendors on the README and add the piechart. I would love to connect and discuss more on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants