Adding elastic set to none #691

paigerube14 · 2024-09-05T19:03:57Z

When elastic search is set to false was getting the below error

2024-09-05 11:34:57,260 [INFO] telemetry collection disabled, skipping.
2024-09-05 11:34:57,260 [INFO] Alerts checking is enabled
Traceback (most recent call last):
  File "/Users/prubenda/PycharmProjects/kraken/run_kraken.py", line 658, in <module>
    retval = main(options.cfg)
  File "/Users/prubenda/PycharmProjects/kraken/run_kraken.py", line 514, in main
    elastic_search,
UnboundLocalError: local variable 'elastic_search' referenced before assignment

rh-pre-commit.version: 2.2.0 rh-pre-commit.check-secrets: ENABLED Signed-off-by: Auto User <auto@users.noreply.github.com>

kraken/prometheus/client.py

chaitanyaenr

LGTM except the minor typo. Thanks.

rh-pre-commit.version: 2.2.0 rh-pre-commit.check-secrets: ENABLED

adding elastic set to none

f350e79

rh-pre-commit.version: 2.2.0 rh-pre-commit.check-secrets: ENABLED Signed-off-by: Auto User <auto@users.noreply.github.com>

paigerube14 requested review from chaitanyaenr and tsebastiani September 5, 2024 19:03

chaitanyaenr reviewed Sep 5, 2024

View reviewed changes

kraken/prometheus/client.py Outdated Show resolved Hide resolved

chaitanyaenr approved these changes Sep 5, 2024

View reviewed changes

too many ls

2689321

rh-pre-commit.version: 2.2.0 rh-pre-commit.check-secrets: ENABLED

chaitanyaenr merged commit 26460a0 into krkn-chaos:main Sep 5, 2024
5 of 7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adding elastic set to none #691

Adding elastic set to none #691

paigerube14 commented Sep 5, 2024 •

edited

Loading

chaitanyaenr left a comment

Adding elastic set to none #691

Adding elastic set to none #691

Conversation

paigerube14 commented Sep 5, 2024 • edited Loading

chaitanyaenr left a comment

Choose a reason for hiding this comment

paigerube14 commented Sep 5, 2024 •

edited

Loading