#50815 Add missing ensure_list_str imports
Closed 3 years ago by spichugi. Opened 4 years ago by adamwill.
adamwill/389-ds-base ensureliststr-import  into  master

@@ -11,7 +11,7 @@ 

  import subprocess

  import logging

  from lib389.nss_ssl import NssSsl

- from lib389.utils import selinux_label_port, assert_c

+ from lib389.utils import selinux_label_port, assert_c, ensure_list_str

  

  

  ######################## WARNING #############################

@@ -32,6 +32,7 @@ 

  from lib389.utils import (

      assert_c,

      is_a_dn,

+     ensure_list_str,

      ensure_str,

      normalizeDN,

      socket_check_open,

lib389.instance.setup and lib389.instance.remove added uses of
ensure_list_str, but neglected to import it. This fixes that.

Signed-off-by: Adam Williamson awilliam@redhat.com

Pull-Request has been closed by adamwill

4 years ago

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

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

3 years ago