-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/54/#review187
-----------------------------------------------------------
src/rolekit/server/rolebase.py
<
http://reviewboard-fedoraserver.rhcloud.com/r/54/#comment104>
Not entirely happy with this, but at this point it still seems borderline readable
enough, and adding a more descriptive API than just returning a dictionary would require a
rethink of how the systemdJobHandler() object is used away from the current
really-temporary-for-a-single-with-statement model. Ideas and fresh eyes welcome.
- Miloslav Trmac
On Čec. 30, 2014, 12:31 dop., Miloslav Trmac wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/54/
-----------------------------------------------------------
(Updated Čec. 30, 2014, 12:31 dop.)
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
Implement starting and stopping services, and use it in databaseserver.
Includes some work-in-progress edits to databaseserver, IMHO harmless enough but will
split them if asked.
Now rereading this, should the StartUnit/StopUnit D-Bus calls be non-blocking as well?
At this late hour I’m inclined to say that they are explicitly designed to be returning
quickly, so it isn’t really necessary.
Diffs
-----
TODO 053560af7e811ae9d26b7eafff92ac2786af5408
config/roles/databaseserver/role.py b632321de63c331b3b3d0445d134fe1626e9944c
src/rolekit/dbus_utils.py fb21cec2681f60ec2ad30372c4caf285e8ba13a7
src/rolekit/server/rolebase.py d53c745352183944e307cca5259a58f058f81e27
Diff:
http://reviewboard-fedoraserver.rhcloud.com/r/54/diff/
Testing
-------
starting/stopping postgres through d-feet.
Thanks,
Miloslav Trmac