On July 14, 2014, 1:26 p.m., Thomas Woerner wrote:
config/roles/databaseserver/role.py, line 65 http://reviewboard-fedoraserver.rhcloud.com/r/2/diff/1/?file=39#file39line65
Maybe it would be good to also have get_dbus_property in the roles for now with the comments to make the use of additional properties simple.
Sorry, could you give an example of what you mean? I'm not sure what properties have to do with decomission()
- Stephen
----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-fedoraserver.rhcloud.com/r/2/#review2 -----------------------------------------------------------
On July 14, 2014, 11:42 a.m., Stephen Gallagher wrote:
This is an automatically generated e-mail. To reply, visit: http://reviewboard-fedoraserver.rhcloud.com/r/2/
(Updated July 14, 2014, 11:42 a.m.)
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
Add placeholders for Domain Controller and Database roles
Diffs
config/Makefile.am 88f68818c877eeadf05e3c6bccea77e6693d428c config/roles/databaseserver/role.py PRE-CREATION config/roles/domaincontroller/role.py PRE-CREATION
Diff: http://reviewboard-fedoraserver.rhcloud.com/r/2/diff/
Testing
Started up rolekit and examined with d-feet that the new roles were available and functioned equivalently to the testrole.
Thanks,
Stephen Gallagher