-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/107/#review331
-----------------------------------------------------------
src/rolectl
<
http://reviewboard-fedoraserver.rhcloud.com/r/107/#comment177>
Please add this to the manpage for rolectl as well.
src/rolekit/server/rolebase.py
<
http://reviewboard-fedoraserver.rhcloud.com/r/107/#comment178>
Let's make the prototype be:
__decommission_async(self, sender, force=False)
and do the same for do_decommission_async().
There may be cases where the role plugin itself may want to do something special on a
force-decommission.
- Stephen Gallagher
On Oct. 24, 2014, 12:39 p.m., Thomas Woerner wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/107/
-----------------------------------------------------------
(Updated Oct. 24, 2014, 12:39 p.m.)
Review request for RoleKit Mailing List, Miloslav Trmac, Stephen Gallagher, and Thomas
Woerner.
Repository: rolekit
Description
-------
With the force option it is possible to force a instance decommission even in the error
case.
Diffs
-----
src/rolectl ec11929bea45e60486a4d9000f0b30e66db665de
src/rolekit/client.py 5b117c4fe0a4af89cfe4cdb3898b5be29ae3e7ab
src/rolekit/server/rolebase.py e90414f57c896fff31f5264cb75c53f1c4be0f9e
Diff:
http://reviewboard-fedoraserver.rhcloud.com/r/107/diff/
Testing
-------
Thanks,
Thomas Woerner