jmsperu commented on PR #137: URL: https://github.com/apache/cloudstack-go/pull/137#issuecomment-4757230003
Confirmed now against a live CloudStack **4.22** deployment (raw `response=json` output): | API | actual response key | |---|---| | `listDbMetrics` | `dbMetrics` | | `listUsageServerMetrics` | `usageMetrics` | | `listHostsMetrics` | `host` | | `listClustersMetrics` | `cluster` | So `host` / `cluster` (and `volume` / `zone`) are correct as in this PR. The two to fix: - `DbMetrics`: keep `json:"dbMetrics"` (the original) — `db` returns nothing. - `UsageServerMetrics`: use `json:"usageMetrics"`, not `usageserver`. With those two tweaked, this is good to merge. 👍 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
