edgelesssys / contrast

Deploy and manage confidential containers on Kubernetes
https://docs.edgeless.systems/contrast
GNU Affero General Public License v3.0
161 stars 6 forks source link

coordinator: move manifestGeneration gauge into authority #538

Closed burgerdev closed 3 weeks ago

burgerdev commented 3 weeks ago

The UserAPI server has no concept of a "current manifest generation", so it should not be recording the corresponding metric. Authoritative here is, well, the Authority, which is where I'm moving the gauge.

The other reason why we're calling GetManifests in a SetManifest call is that we need to get hold of the CA instance. This is going to change with a future PR for RFC004, when the CA will be returned by SetManifest directly. This is also when I'm going to add tests for Authority.