-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/10/
-----------------------------------------------------------
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
- New PERSISTENT_STATES and TRANSITIONAL_STATES list
Diffs
-----
src/rolekit/config/__init__.py.in 4d8f53964b36feb25f97ce858cdf8eebdd0a3956
Diff: http://reviewboard-fedoraserver.rhcloud.com/r/10/diff/
Testing
-------
Thanks,
Thomas Woerner
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/5/
-----------------------------------------------------------
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
a = undefined_variable_name
is way too obvious for static analysis tools to detect; if we want
to raise an exception, use something rarer.
This doesn’t change behavior of the test code; the log in /tmp/log
handles both NameError and ZeroDivisionError in the same way.
Eventually, we might end up just with an explicit
raise SomeExceptionType
but that’s a less realistic test code, so let’s not do it just now.
Diffs
-----
src/rolekit/logger.py 9ad2464b00daa748e87cd137b7dade2d160c9313
Diff: http://reviewboard-fedoraserver.rhcloud.com/r/5/diff/
Testing
-------
Thanks,
Miloslav Trma?
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/6/
-----------------------------------------------------------
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
This function exists in rolekit.functions, but an import was missing.
Diffs
-----
src/rolekit/client.py 8f3d648a0c6b06a94d3562e5330ff0af08d5d14a
Diff: http://reviewboard-fedoraserver.rhcloud.com/r/6/diff/
Testing
-------
Thanks,
Miloslav Trma?
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/9/
-----------------------------------------------------------
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
Fix a copy/paste error, undefined symbol “x”
Diffs
-----
src/rolekit/server/rolebase.py 56ace5a0a309d179cbdea28a9bd945135c15b77e
Diff: http://reviewboard-fedoraserver.rhcloud.com/r/9/diff/
Testing
-------
Thanks,
Miloslav Trma?
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/8/
-----------------------------------------------------------
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
Fix a typo in Role[Kk]itError
Diffs
-----
src/rolekit/server/roled.py 0972e17ecc99bff403d6a107ca63c28fb0697d3d
Diff: http://reviewboard-fedoraserver.rhcloud.com/r/8/diff/
Testing
-------
Thanks,
Miloslav Trma?
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/7/
-----------------------------------------------------------
Review request for RoleKit Mailing List, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
Used in both DBusRole.Set and RoleBase.Set.
Diffs
-----
src/rolekit/errors.py 0402c457bdb1f4ff269a8a5011be9c1bc2bf37da
Diff: http://reviewboard-fedoraserver.rhcloud.com/r/7/diff/
Testing
-------
Thanks,
Miloslav Trma?
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-fedoraserver.rhcloud.com/r/2/
-----------------------------------------------------------
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
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://fedorahosted.org/reviewboard/r/310/
-----------------------------------------------------------
Review request for rolekit reviewers, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
- Added missing imports for dbus type conversion and possible method calls
- Dropped "backup_paths" setting from _DEFAULS
- Added own setting "myownsetting" altogether with an extended get_dbus_property method
- Added not covered "failonthis" setting to show error message in the log with get_dbus_property calls
Diffs
-----
config/roles/testrole/role.py 5a47874456eaa3a1de5793b86e96f7c01a6d641d
Diff: https://fedorahosted.org/reviewboard/r/310/diff/
Testing
-------
Thanks,
Thomas Woerner
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://fedorahosted.org/reviewboard/r/309/
-----------------------------------------------------------
Review request for rolekit reviewers, Stephen Gallagher and Thomas Woerner.
Repository: rolekit
Description
-------
- Fixed the name of the property method
- Use try to not trace back for unknown properties
Diffs
-----
src/rolekit/server/dbusrole.py 272bde8acf8793155c7c6323d0aa3ffda45ceb74
Diff: https://fedorahosted.org/reviewboard/r/309/diff/
Testing
-------
Thanks,
Thomas Woerner