Add query for Redis by VSHN

appuio / appuio-cloud-reporting

Reporting for APPUiO Cloud

BSD 3-Clause "New" or "Revised" License

0 stars 1 forks source link

Add query for Redis by VSHN #129

Closed Kidswiss closed 1 year ago

Kidswiss commented 1 year ago

Summary

Add a query to get usage for Redis by VSHN instances
AppCat by VSHN services will all use almost the same query for billing. Added ability to render the changes via gotemplate.

Checklist

[x] Categorize the PR by setting a good title and adding one of the labels: bug, enhancement, documentation, change, breaking, dependency as they show up in the changelog
[x] Update tests.
[x] Link this PR to related issues.

Kidswiss commented 1 year ago

LGTM, I'm not sure though how much more logic should go into seeds before we properly do migrations of seeded data.

I was thinking the same... I feel like switching from just seeding to properly migrate the queries like any other schema change seems like the safer option.

bastjan commented 1 year ago

I was thinking the same... I feel like switching from just seeding to properly migrate the queries like any other schema change seems like the safer option.

It was never designed for this kinda flexibility. I think we might need to sit together and brainstorm what we would need from such a system.