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

pin down urllib3<2 since botocore doesn't support it #453

Merged
merged 1 commit into from
May 10, 2023

Conversation

fruch
Copy link
Contributor

@fruch fruch commented May 9, 2023

Look like new version of requests is pulling newer versions of urllib3 which are not comptible with
botocore.

Ref: boto/botocore#2926
Ref: psf/requests#6443
Ref: urllib3/urllib3#2168

Look like new version of requests is pulling newer
versions of urllib3 which are not comptible with
botocore.

Ref: boto/botocore#2926
Ref: psf/requests#6443
Ref: urllib3/urllib3#2168
@@ -42,7 +42,7 @@ jobs:

pip install -U pip setuptools

python setup.py install
pip install .
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how is this change related to the urllib3 problem?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought it was the issue at first.

Since there was a ton of new warning that setup.py install is going to be deprecated.

I might move it to it's own commit

@tchaikov
Copy link
Contributor

created #454 so we can undo this once the related issues on upstream (botocore and requests) are addressed.

@tchaikov
Copy link
Contributor

the test failed:

2023-05-09 14:35:02 INFO     ['./ccm', 'create', 'CCMCluster-cassandra', '-v', '3.11.4', '-n', '1']
2023-05-09 14:35:02 DEBUG    [stdout] Current cluster is now: CCMCluster-cassandra
2023-05-09 14:35:02 DEBUG    [stderr] 
2023-05-09 14:35:02 INFO     ['./ccm', 'updateconf', 'read_request_timeout_in_ms:[100](https://github.com/scylladb/scylla-ccm/actions/runs/4926279591/jobs/8803175292?pr=453#step:9:101)00', 'range_request_timeout_in_ms:10000', 'write_request_timeout_in_ms:10000', 'truncate_request_timeout_in_ms:10000', 'request_timeout_in_ms:10000']
2023-05-09 14:35:03 DEBUG    [stdout] 
2023-05-09 14:35:03 DEBUG    [stderr] 
2023-05-09 14:35:03 INFO     ['./ccm', 'start', '--wait-for-binary-proto']
2023-05-09 14:35:03 DEBUG    [stdout] [node1 ERROR] Unable to find java executable. Check JAVA_HOME and PATH environment variables.
2023-05-09 14:35:03 DEBUG    [stderr] Error starting nodes, see above for details (you can use --verbose for more information)
Error: -09 14:35:03 ERROR    [ERROR] Error starting nodes, see above for details (you can use --verbose for more information)

@fruch
Copy link
Contributor Author

fruch commented May 10, 2023

created #454 so we can undo this once the related issues on upstream (botocore and requests) are addressed.

Thanks, you are correct we shouldn't leave forever

@fruch
Copy link
Contributor Author

fruch commented May 10, 2023

the test failed:

2023-05-09 14:35:02 INFO     ['./ccm', 'create', 'CCMCluster-cassandra', '-v', '3.11.4', '-n', '1']
2023-05-09 14:35:02 DEBUG    [stdout] Current cluster is now: CCMCluster-cassandra
2023-05-09 14:35:02 DEBUG    [stderr] 
2023-05-09 14:35:02 INFO     ['./ccm', 'updateconf', 'read_request_timeout_in_ms:[100](https://github.com/scylladb/scylla-ccm/actions/runs/4926279591/jobs/8803175292?pr=453#step:9:101)00', 'range_request_timeout_in_ms:10000', 'write_request_timeout_in_ms:10000', 'truncate_request_timeout_in_ms:10000', 'request_timeout_in_ms:10000']
2023-05-09 14:35:03 DEBUG    [stdout] 
2023-05-09 14:35:03 DEBUG    [stderr] 
2023-05-09 14:35:03 INFO     ['./ccm', 'start', '--wait-for-binary-proto']
2023-05-09 14:35:03 DEBUG    [stdout] [node1 ERROR] Unable to find java executable. Check JAVA_HOME and PATH environment variables.
2023-05-09 14:35:03 DEBUG    [stderr] Error starting nodes, see above for details (you can use --verbose for more information)
Error: -09 14:35:03 ERROR    [ERROR] Error starting nodes, see above for details (you can use --verbose for more information)

Yeah, I didn't have time to look at it.

Doesn't seem to be related to the installation issue, maybe to the change of using java8, but still I would expect it to fail more than one test (or we are stopping at first failure 😭)

@tchaikov
Copy link
Contributor

the test failed:

2023-05-09 14:35:02 INFO     ['./ccm', 'create', 'CCMCluster-cassandra', '-v', '3.11.4', '-n', '1']
2023-05-09 14:35:02 DEBUG    [stdout] Current cluster is now: CCMCluster-cassandra
2023-05-09 14:35:02 DEBUG    [stderr] 
2023-05-09 14:35:02 INFO     ['./ccm', 'updateconf', 'read_request_timeout_in_ms:[100](https://github.com/scylladb/scylla-ccm/actions/runs/4926279591/jobs/8803175292?pr=453#step:9:101)00', 'range_request_timeout_in_ms:10000', 'write_request_timeout_in_ms:10000', 'truncate_request_timeout_in_ms:10000', 'request_timeout_in_ms:10000']
2023-05-09 14:35:03 DEBUG    [stdout] 
2023-05-09 14:35:03 DEBUG    [stderr] 
2023-05-09 14:35:03 INFO     ['./ccm', 'start', '--wait-for-binary-proto']
2023-05-09 14:35:03 DEBUG    [stdout] [node1 ERROR] Unable to find java executable. Check JAVA_HOME and PATH environment variables.
2023-05-09 14:35:03 DEBUG    [stderr] Error starting nodes, see above for details (you can use --verbose for more information)
Error: -09 14:35:03 ERROR    [ERROR] Error starting nodes, see above for details (you can use --verbose for more information)

Yeah, I didn't have time to look at it.

Doesn't seem to be related to the installation issue, maybe to the change of using java8, but still I would expect it to fail more than one test (or we are stopping at first failure sob)

hi @fruch , is it possible that we fail to detect the java runtime installed by

uses: actions/setup-java@v3
? as in 66e9b4c, we only check for the well-known directories. which are the directories used by the debian / fedora packages. but the workflow of "setup-java" installs the runtime into somewhere else: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/8.0.362-9/x64. see https://github.com/scylladb/scylla-ccm/actions/runs/4926279591/jobs/8803175292#step:5:30 . shall we add this directory to the list of
for java_path in pathlib.Path('/usr/lib/jvm/').iterdir():
?

@fruch
Copy link
Contributor Author

fruch commented May 10, 2023

the test failed:

2023-05-09 14:35:02 INFO     ['./ccm', 'create', 'CCMCluster-cassandra', '-v', '3.11.4', '-n', '1']
2023-05-09 14:35:02 DEBUG    [stdout] Current cluster is now: CCMCluster-cassandra
2023-05-09 14:35:02 DEBUG    [stderr] 
2023-05-09 14:35:02 INFO     ['./ccm', 'updateconf', 'read_request_timeout_in_ms:[100](https://github.com/scylladb/scylla-ccm/actions/runs/4926279591/jobs/8803175292?pr=453#step:9:101)00', 'range_request_timeout_in_ms:10000', 'write_request_timeout_in_ms:10000', 'truncate_request_timeout_in_ms:10000', 'request_timeout_in_ms:10000']
2023-05-09 14:35:03 DEBUG    [stdout] 
2023-05-09 14:35:03 DEBUG    [stderr] 
2023-05-09 14:35:03 INFO     ['./ccm', 'start', '--wait-for-binary-proto']
2023-05-09 14:35:03 DEBUG    [stdout] [node1 ERROR] Unable to find java executable. Check JAVA_HOME and PATH environment variables.
2023-05-09 14:35:03 DEBUG    [stderr] Error starting nodes, see above for details (you can use --verbose for more information)
Error: -09 14:35:03 ERROR    [ERROR] Error starting nodes, see above for details (you can use --verbose for more information)

Yeah, I didn't have time to look at it.

Doesn't seem to be related to the installation issue, maybe to the change of using java8, but still I would expect it to fail more than one test (or we are stopping at first failure sob)

hi @fruch , is it possible that we fail to detect the java runtime installed by

uses: actions/setup-java@v3
? as in 66e9b4c, we only check for the well-known directories. which are the directories used by the debian / fedora packages. but the workflow of "setup-java" installs the runtime into somewhere else: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/8.0.362-9/x64. see https://github.com/scylladb/scylla-ccm/actions/runs/4926279591/jobs/8803175292#step:5:30 . shall we add this directory to the list of
for java_path in pathlib.Path('/usr/lib/jvm/').iterdir():
?

The failing case trys to run cassandra, not scylla

@fruch
Copy link
Contributor Author

fruch commented May 10, 2023

the test failed:

2023-05-09 14:35:02 INFO     ['./ccm', 'create', 'CCMCluster-cassandra', '-v', '3.11.4', '-n', '1']
2023-05-09 14:35:02 DEBUG    [stdout] Current cluster is now: CCMCluster-cassandra
2023-05-09 14:35:02 DEBUG    [stderr] 
2023-05-09 14:35:02 INFO     ['./ccm', 'updateconf', 'read_request_timeout_in_ms:[100](https://github.com/scylladb/scylla-ccm/actions/runs/4926279591/jobs/8803175292?pr=453#step:9:101)00', 'range_request_timeout_in_ms:10000', 'write_request_timeout_in_ms:10000', 'truncate_request_timeout_in_ms:10000', 'request_timeout_in_ms:10000']
2023-05-09 14:35:03 DEBUG    [stdout] 
2023-05-09 14:35:03 DEBUG    [stderr] 
2023-05-09 14:35:03 INFO     ['./ccm', 'start', '--wait-for-binary-proto']
2023-05-09 14:35:03 DEBUG    [stdout] [node1 ERROR] Unable to find java executable. Check JAVA_HOME and PATH environment variables.
2023-05-09 14:35:03 DEBUG    [stderr] Error starting nodes, see above for details (you can use --verbose for more information)
Error: -09 14:35:03 ERROR    [ERROR] Error starting nodes, see above for details (you can use --verbose for more information)

Yeah, I didn't have time to look at it.

Doesn't seem to be related to the installation issue, maybe to the change of using java8, but still I would expect it to fail more than one test (or we are stopping at first failure sob)

hi @fruch , is it possible that we fail to detect the java runtime installed by

uses: actions/setup-java@v3
? as in 66e9b4c, we only check for the well-known directories. which are the directories used by the debian / fedora packages. but the workflow of "setup-java" installs the runtime into somewhere else: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/8.0.362-9/x64. see https://github.com/scylladb/scylla-ccm/actions/runs/4926279591/jobs/8803175292#step:5:30 . shall we add this directory to the list of
for java_path in pathlib.Path('/usr/lib/jvm/').iterdir():
?

The failing case trys to run cassandra, not scylla

And I've forgotten you convinced me to not use alternatives command line...

Yeah that is the root cause, and I didn't notice it.

@fruch
Copy link
Contributor Author

fruch commented May 10, 2023

I decided to merge this first, and then handle the CI breakage

@fruch fruch merged commit 512c708 into scylladb:next May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants