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

Fix AttributeError: module 'ssl' has no attribute 'TLSVersion' #81

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

Alex-Burmak
Copy link
Member

The fix for:

2023-11-27 12:59:24,757 763834 [ERROR] max_latency: Error occurred while executing:
Traceback (most recent call last):
  File "/opt/yandex/clickhouse-tools/lib/python3.6/site-packages/ch_tools/monrun_checks_keeper/main.py", line 64, in wrapper
    result = ctx.invoke(cmd_callback, *a, **kw)
  File "/opt/yandex/clickhouse-tools/lib/python3.6/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/opt/yandex/clickhouse-tools/lib/python3.6/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/opt/yandex/clickhouse-tools/lib/python3.6/site-packages/ch_tools/monrun_checks_keeper/keeper_commands.py", line 64, in max_latency_command
    return Result(0, keeper_mntr(ctx)["zk_max_latency"])
  File "/opt/yandex/clickhouse-tools/lib/python3.6/site-packages/ch_tools/monrun_checks_keeper/keeper_commands.py", line 246, in keeper_mntr
    raise e
  File "/opt/yandex/clickhouse-tools/lib/python3.6/site-packages/ch_tools/monrun_checks_keeper/keeper_commands.py", line 233, in keeper_mntr
    not ctx.obj.get("no_verify_ssl_certs"),
  File "/opt/yandex/clickhouse-tools/lib/python3.6/site-packages/ch_tools/monrun_checks_keeper/keeper_commands.py", line 203, in keeper_command
    ssl_context.minimum_version = ssl.TLSVersion.TLSv1_2
AttributeError: module 'ssl' has no attribute 'TLSVersion'

Relates to #76

@Alex-Burmak Alex-Burmak requested review from Nuzhnov and myrrc November 27, 2023 10:02
@Alex-Burmak Alex-Burmak merged commit 2d80b02 into main Nov 27, 2023
16 of 18 checks passed
@Alex-Burmak Alex-Burmak deleted the alex-burmak branch November 27, 2023 12:33
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