rhtools / vminfo-parser

3 stars 3 forks source link

Granular breakdown by os does not respect `--sort-by-env` #73

Open stratus-ss opened 1 week ago

stratus-ss commented 1 week ago

There is a bug when using these flags

 "args": ["--file", "temp/dataset.csv", "--sort-by-env", "both","--prod-env-labels", "prod1,prod2", "--disk-space-by-granular-os", "--show-disk-space-by-os"]

Regardless of the sort flags passed in, this option returns the combine totals

csfreak commented 1 week ago

This actually has a deeper cause than what I originally thought.
The logic to handle environment sorting and the logic to handle split by version are currently mutually exclusive, this will require a more in-depth fix than I had hoped.