robur-coop / albatross

Albatross: orchestrate and manage MirageOS unikernels with Solo5
ISC License
141 stars 17 forks source link

rework albatross-stats #129

Closed hannesm closed 8 months ago

hannesm commented 1 year ago

There are some issues with albatross-stats:

A solution from this mornings brainstorming:

This would as well solve #55 by just not encoding and decoding this information ;)

hannesm commented 1 year ago

The protocol has been inverted (the stats daemon now connects to the albatross daemon) in #131.

What is left to do is the question: should stats and influx being merged? As outlined in #55 this would allow less marshalling work, though that feels like premature optimization while loosing some functionality.

hannesm commented 8 months ago

Since there's premature optimization mentioned in the comment above from a year back, closing this issue. When this becomes the trouble point, we can always open it.