Merge lp://staging/~allenap/gwacl/update-role into lp://staging/gwacl
Proposed by
Gavin Panella
Status: | Merged |
---|---|
Approved by: | Gavin Panella |
Approved revision: | 184 |
Merged at revision: | 184 |
Proposed branch: | lp://staging/~allenap/gwacl/update-role |
Merge into: | lp://staging/gwacl |
Diff against target: |
76 lines (+55/-0) 2 files modified
management_base.go (+30/-0) management_base_test.go (+25/-0) |
To merge this branch: | bzr merge lp://staging/~allenap/gwacl/update-role |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Raphaël Badin (community) | Approve | ||
Review via email:
|
Commit message
New API method UpdateRole.
To post a comment you must log in.
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"
:/