ack

You can do except (PermissionError, IOError) as e: rather than duplicating the code.

Unless I'm mistaken. we can run as different users as well. It could be rather confusing. Can we have this error dynamic or not mention user names at all? (same below, of course)

@mhonek That's a good tip to prevent the duplication.

We can run as different users, but it's really unlikely (unless in containers). The thing is though, that if we are running "not as root/dirsrv" and the operation works ... then we are running as the right use. If we aren't root/dirsrv and it fails, it probably is wrong .... It's a really tricky balance here because if we say nothing abotu the user, we are missing good hints to fix the problem. If we say too much, we then risk confusion. I think that mentioning this is reasonable to me since they are the two most common users related to admin on rhel/fedora/suse.

Fair enough. Only, I would change wording then to something like ... Are you running as correct user (usually root or dirsrv)?

Besides that, ACK from me. Thanks.

OHHH I think you didn't communicate that intent very well ... perhaps an example message next time. Hahha. That's actually a reasonable improvement. I understood it as you trying to remove all references to dirsrv/root.

@mhonek Tweak to the message.

:thumbsup: Merge please.

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

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.