[Freeipa-devel] [PATCH] Adding SUDO Plugins and Test Suites

[JR Aquino]

Let me know how it looks now.


On Sep 27, 2010, at 1:17 PM, Rob Crittenden wrote:

A very good first try, just needs a little more work.

You need to create the 3 containers in
install/share/bootstrap-template.ldif so they get created when a server
is installed.

Done

In sudocmd.py you need to:
- remove imports for Flag, Bytes and base64
- Don't define a pre_callback if it isn't doing any work

Done

In sudocmdgroup.py you need to:
- Fix the doc example at the top. It starts with sudocmdgroup examples
then switches to group examples.
- Drop the Int import

Done

In sudorule.py you need to:
- Drop AccessTime and Password from imports

Done

- Should membercmd_sudocmd use the label "Sudo Commands" to be
consistent with Sudo Group Commands?

Done (yes, you are right)

- I'm not entirely sure what you are doing with cn in the dn, can you
add some documentation explaining what is going on?

It was cruft and has been removed.  Was originally there due to confusion about how the compat piece was going to work and wasn't sure if the dn would be defined by the cn or a different identifier.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Attached-are-the-Corrected-plugins-for-creating.patch
Type: application/octet-stream
Size: 51211 bytes
Desc: 0001-Attached-are-the-Corrected-plugins-for-creating.patch
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20100927/5700dd77/attachment.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20100927/5700dd77/attachment.htm>