Skip to content

Add VM inventory CSV export support#397

Merged
Dany9966 merged 3 commits into
cloudbase:masterfrom
mihaelabalutoiu:add-vm-inventory-csv-extraction
May 6, 2026
Merged

Add VM inventory CSV export support#397
Dany9966 merged 3 commits into
cloudbase:masterfrom
mihaelabalutoiu:add-vm-inventory-csv-extraction

Conversation

@mihaelabalutoiu
Copy link
Copy Markdown
Member

This PR includes the following changes:

  • Adds a PROVIDER_TYPE_ENDPOINT_INVENTORY_EXPORT capability type that
    supports exporting VM inventory as CSV;
  • Unit tests

Comment thread coriolis/api/v1/endpoint_inventory.py Outdated
@mihaelabalutoiu mihaelabalutoiu force-pushed the add-vm-inventory-csv-extraction branch 2 times, most recently from e348735 to 0cc2991 Compare March 27, 2026 09:26
Comment thread coriolis/api-refs/source/endpoint.inc Outdated
Comment thread coriolis/api-refs/source/endpoint.inc Outdated
Comment thread coriolis/api-refs/source/endpoint.inc Outdated
Comment thread coriolis/api-refs/source/endpoint.inc Outdated
Comment thread coriolis/api-refs/source/endpoint.inc Outdated
Comment thread coriolis/api/v1/endpoint_inventory.py Outdated
Comment thread coriolis/api/wsgi.py Outdated
Comment thread coriolis/api/v1/endpoint_inventory.py Outdated
Comment thread coriolis/api/wsgi.py Outdated
@mihaelabalutoiu mihaelabalutoiu force-pushed the add-vm-inventory-csv-extraction branch from 0cc2991 to 56313e3 Compare March 27, 2026 15:55
Copy link
Copy Markdown
Member Author

@mihaelabalutoiu mihaelabalutoiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I updated the code, thank you a lot

Comment thread coriolis/api/v1/endpoint_inventory.py Outdated
Comment thread coriolis/api-refs/source/endpoint.inc Outdated
Adds a `PROVIDER_TYPE_ENDPOINT_INVENTORY_EXPORT` capability type that
supports exporting VM inventory as CSV.

Signed-off-by: Mihaela Balutoiu <mbalutoiu@cloudbasesolutions.com>
Signed-off-by: Mihaela Balutoiu <mbalutoiu@cloudbasesolutions.com>
@mihaelabalutoiu mihaelabalutoiu force-pushed the add-vm-inventory-csv-extraction branch from 56313e3 to 61b8398 Compare May 6, 2026 11:08
Comment thread coriolis/providers/base.py Outdated
"""Export the full VM inventory as a CSV-formatted string.

Returns a standards-compliant CSV string with a header row and one
row per VM, sorted deterministically by VM ID.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also document each param of the method, and also what's expected to be returned.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, thank you!

Signed-off-by: Mihaela Balutoiu <mbalutoiu@cloudbasesolutions.com>
@mihaelabalutoiu mihaelabalutoiu force-pushed the add-vm-inventory-csv-extraction branch from 61b8398 to ba3717d Compare May 6, 2026 11:49
@Dany9966 Dany9966 merged commit c3cb2c1 into cloudbase:master May 6, 2026
5 checks passed
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