Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Current integ tests workflow fails on version incompatibility #3554

Closed
zelinh opened this issue May 24, 2023 · 7 comments · Fixed by #3559
Closed

[Bug]: Current integ tests workflow fails on version incompatibility #3554

zelinh opened this issue May 24, 2023 · 7 comments · Fixed by #3559
Assignees
Labels
bug Something isn't working release v2.8.0

Comments

@zelinh
Copy link
Member

zelinh commented May 24, 2023

Describe the bug

The current OS integ tests fails on importing module because urllib only supports OpenSSL 1.1.1+. Current ssl version is 1.0.2

Link: https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/4980/pipeline/74
Full context:

Running ./src/run_integ_test.py manifests/2.8.0/opensearch-2.8.0-test.yml --component job-scheduler --test-run-id 4980 --paths opensearch=/var/jenkins/workspace/integ-test/tar --base-path https://ci.opensearch.org/ci/dbc/integ-test/2.8.0/7879/linux/x64/tar ...

Traceback (most recent call last):

  File "./src/run_integ_test.py", line 13, in <module>

    from test_workflow.integ_test.integ_test_runners import IntegTestRunners

  File "/var/jenkins/workspace/integ-test/src/test_workflow/integ_test/integ_test_runners.py", line 11, in <module>

    from test_workflow.integ_test.integ_test_runner_opensearch import IntegTestRunnerOpenSearch

  File "/var/jenkins/workspace/integ-test/src/test_workflow/integ_test/integ_test_runner_opensearch.py", line 15, in <module>

    from test_workflow.integ_test.integ_test_suite_opensearch import IntegTestSuiteOpenSearch

  File "/var/jenkins/workspace/integ-test/src/test_workflow/integ_test/integ_test_suite_opensearch.py", line 22, in <module>

    from test_workflow.integ_test.topology import NodeEndpoint, Topology

  File "/var/jenkins/workspace/integ-test/src/test_workflow/integ_test/topology.py", line 16, in <module>

    from test_workflow.integ_test.local_test_cluster import LocalTestCluster

  File "/var/jenkins/workspace/integ-test/src/test_workflow/integ_test/local_test_cluster.py", line 13, in <module>

    from test_workflow.integ_test.service import Service

  File "/var/jenkins/workspace/integ-test/src/test_workflow/integ_test/service.py", line 14, in <module>

    import requests

  File "/usr/share/opensearch/.local/share/virtualenvs/integ-test-B5RGX0jA/lib/python3.7/site-packages/requests/__init__.py", line 43, in <module>

    import urllib3

  File "/usr/share/opensearch/.local/share/virtualenvs/integ-test-B5RGX0jA/lib/python3.7/site-packages/urllib3/__init__.py", line 39, in <module>

    "urllib3 v2.0 only supports OpenSSL 1.1.1+, currently "

ImportError: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with OpenSSL 1.0.2k-fips  26 Jan 2017. See: https://github.com/urllib3/urllib3/issues/2168

script returned exit code 1

To reproduce

https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/4980/pipeline/74

Expected behavior

No response

Screenshots

If applicable, add screenshots to help explain your problem.

Host / Environment

No response

Additional context

No response

Relevant log output

No response

@zelinh zelinh added bug Something isn't working untriaged Issues that have not yet been triaged labels May 24, 2023
@peterzhuamazon
Copy link
Member

This seems like an upgrade on urllib3 libs due to not supporting openssl version lower than 1.1.1+, where centos7 does not have now. We would need to speed up the process to upgrade to rockylinux8.

@zelinh
Copy link
Member Author

zelinh commented May 24, 2023

@peterzhuamazon
Copy link
Member

peterzhuamazon commented May 24, 2023

centos7:

openssl  1:1.0.2k-26.el7_9  

rockylinux8:

 openssl  1:1.1.1k-9.el8_7 

@peterzhuamazon
Copy link
Member

peterzhuamazon commented May 25, 2023

Related to this:

@peterzhuamazon
Copy link
Member

The newer version of requests cause a requirement update on openssl, from 1.1.0 to 1.1.1+, this means CentOS7 is not able to run newer version of request on any activities related to our python code.

@peterzhuamazon
Copy link
Member

Hi,

Option 1: Revert #3544 and keep using CentOS7 for this release
Option 2: Move all the changes to RockyLinux8 and remove CentOS7 support.

Consider our decision on the AL2023/NodeJS18/PythonUpgrade 3.9 has already made in this comment:#1563 (comment)

We will go ahead with option 1 and revert #3544.

cc: @bbarani @gaiksaya

Thanks.

@peterzhuamazon
Copy link
Member

Our test in #3544 is running in ubuntu-latest thus does not have the openssl issue in centos7, therefore, not caught.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working release v2.8.0
Projects
2 participants