Hi all, This review takes into account the previous review recommendations: - change args.get with default values - change list() so that it returns - default the environment file related to the instance (/etc/sysconfig/dirsrv-<serverid> or $HOME/.dirsrv/dirsrv-<serverid>) - all the environment files if called with 'list(all=True)' - change exists() to return 'len(self.list()) == 1' - start() rebinds to the instance so that its final state is DIRSRV_STATE_ONLINE https://fedorahosted.org/389/attachment/ticket/47625/0002-Ticket-47625-CI-lib389-DirSrv-not-conform-to-the-des.patch |
-- 389-devel mailing list 389-devel@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/389-devel