Code review comment for lp://staging/~allenap/gwacl/update-role

Revision history for this message
Raphaƫl Badin (rvb) wrote :

Looks good.

[0]

14 + RoleName string
15 + PersistentVMRole *PersistentVMRole

The PersistentVMRole object already contains the RoleName.

[1]

"alas, struct embedding is too clunky, so copy-n-paste it is"

:/

review: Approve

« Back to merge proposal