#49510 Issue #35 - Add support for managing automember to dsconf
Closed by spichugi. Opened by alisha17.
alisha17/389-ds-base automembercli  into  master

Download 49510.patch

Bug Description: Add support for managing automember to dsconf

Fix Description: Added the dsconf support and tests for creating, removing, editing and listing
the automember definitions.

https://pagure.io/lib389/issue/35

Author: Alisha Aneja

Reviewed by: wibrown

PR for issue #35 - https://pagure.io/lib389/issue/35

rebased onto 435f86ac7b57e87bef481f9deae5b812d5a71e9b

rebased onto bc5d2c6b5e3d031b5044359515ecbe15b8c7ea57

This should be a mandatory field. Right now it's optional, which leads to:
Error: Attribute cn must not be None
And that error is from DSLdapObject itself

Filter should default to '(objectClass=*)' not ''

Because this can't be inferred, it has to be a mandatory argument like name

Because this can't be inferred this has to be a mandatory argument like name

This one is correct! :)

Probably don't need a default during edit because edit implies you want to change the value

rebased onto f374781315cd9ee4ae6fe19544334c3b852ad561

/rebuild

/rebuild

Ack from me. Can you rebase this to master, then I'll merge,

And edit reviewed by to "wibrown" as well please :)

rebased onto e16b7a442d5a79674e6e9f4bee97ec0d18070f9a

rebased onto 54a327359f6178db03417d996f0e4dd4c8542d19

Pull-Request has been closed by firstyear

PR merged manually to master

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/2569

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata