Conversation
flihp
left a comment
There was a problem hiding this comment.
These bits of metadata look like they'll be useful in access control as sort-of roles in the RBAC sense.
|
since ixi has merged the branch this work builds on please do rebase on main, also seems like this can graduate from a draft once rebased |
|
@flihp are we going to want a version number on |
|
Certainly will get this rebased. To @iximeow's question. I did wonder if I should be introducing these as |
|
Promoted to a real PR, and I'll put a note on the agenda for how much we want to lean in to a breaking API change (even though that it is clearly documented as unstable). |
This is part of propolis#1114 to add support for including the project and silo ids in the VM instance configuration data provided by attestations. These PRs are meant as discussion points, and not intended to be merged yet. This change allows upstream callers to make richer policy decisions as project and silo ids are likely to be much more stable than VM ids.
The actual changes here are only to the data carrying type.