diff options
author | OpenShift Bot <eparis+openshiftbot@redhat.com> | 2017-05-01 18:35:19 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-05-01 18:35:19 -0500 |
commit | 4c2f899ac4f5ad183072e453ece4391ee456871f (patch) | |
tree | 7d3c861465917c3191f751812f1cb20e501a03d1 | |
parent | 5ba185676403be57764eaa669403a00db83a9a0e (diff) | |
parent | 977947f68584faec9957d6d38ea5cdd38c5ea1b0 (diff) | |
download | openshift-4c2f899ac4f5ad183072e453ece4391ee456871f.tar.gz openshift-4c2f899ac4f5ad183072e453ece4391ee456871f.tar.bz2 openshift-4c2f899ac4f5ad183072e453ece4391ee456871f.tar.xz openshift-4c2f899ac4f5ad183072e453ece4391ee456871f.zip |
Merge pull request #3865 from pilou-/fix_test_openssl_1.1.0
Merged by openshift-bot
-rw-r--r-- | roles/openshift_certificate_expiry/library/openshift_cert_expiry.py | 4 | ||||
-rw-r--r-- | roles/openshift_certificate_expiry/test/test_fakeopensslclasses.py | 3 |
2 files changed, 4 insertions, 3 deletions
diff --git a/roles/openshift_certificate_expiry/library/openshift_cert_expiry.py b/roles/openshift_certificate_expiry/library/openshift_cert_expiry.py index c204b5341..0242f5b43 100644 --- a/roles/openshift_certificate_expiry/library/openshift_cert_expiry.py +++ b/roles/openshift_certificate_expiry/library/openshift_cert_expiry.py @@ -135,7 +135,7 @@ platforms missing the Python OpenSSL library. continue elif l.startswith('Subject:'): - # O=system:nodes, CN=system:node:m01.example.com + # O = system:nodes, CN = system:node:m01.example.com self.subject = FakeOpenSSLCertificateSubjects(l.partition(': ')[-1]) def get_serial_number(self): @@ -202,7 +202,7 @@ object""" """ self.subjects = [] for s in subject_string.split(', '): - name, _, value = s.partition('=') + name, _, value = s.partition(' = ') self.subjects.append((name, value)) def get_components(self): diff --git a/roles/openshift_certificate_expiry/test/test_fakeopensslclasses.py b/roles/openshift_certificate_expiry/test/test_fakeopensslclasses.py index ccdd48fa8..8a521a765 100644 --- a/roles/openshift_certificate_expiry/test/test_fakeopensslclasses.py +++ b/roles/openshift_certificate_expiry/test/test_fakeopensslclasses.py @@ -17,7 +17,8 @@ from openshift_cert_expiry import FakeOpenSSLCertificate # noqa: E402 @pytest.fixture(scope='module') def fake_valid_cert(valid_cert): - cmd = ['openssl', 'x509', '-in', str(valid_cert['cert_file']), '-text'] + cmd = ['openssl', 'x509', '-in', str(valid_cert['cert_file']), '-text', + '-nameopt', 'oneline'] cert = subprocess.check_output(cmd) return FakeOpenSSLCertificate(cert.decode('utf8')) |