edgexr / edge-cloud-platform

Apache License 2.0
1 stars 0 forks source link

shepherd: refactor `newMetric` function #193

Open levshvarts opened 1 year ago

levshvarts commented 1 year ago

This function is being used by both cluster and appinst metrics, and as a result the logic is pretty convoluted. See discussion

levshvarts commented 1 year ago

Furthermore - only network cluster metrics api works correctly(from envoy), cpu/mem/disk set reservedBy value instead. See output below:

> select * from "cluster-cpu" order by time desc limit 1
name: cluster-cpu
time                          cloudlet        cloudletorg cluster     clusterorg cpu
----                          --------        ----------- -------     ---------- ---
2023-05-23T22:28:19.50300002Z bonn-cloud-test edgexr      reservable0 EdgeXR-Dev 16.960662841443618
> select * from "cluster-network" order by time desc limit 1
name: cluster-network
time                           cloudlet        cloudletorg cluster     clusterorg   recvBytes sendBytes
----                           --------        ----------- -------     ----------   --------- ---------
2023-05-23T22:28:27.613788211Z bonn-cloud-test edgexr      reservable0 edgecloudorg 42015612  10807998