LGTM!

I wonder if we should specify RDN value there (as dsidm -b dc=example,dc=com organizationalunit list shows the list of RDN values so it's more convenient)

But it's probably not the PR where we should deal with it. So you have an ack from me too, BTW

Yeah, in the past I used the shortnams because most of the get/modify commands use the selector element rather than the dn. The dn in delete is more abotu being really sure about what you are deleting.

Saying that, the better answer may just be to have delete take the shortname instead.

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/4213

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.