cluster load: Refresh prometheus_api connection in case of connection error #11327
Conversation
… error Signed-off-by: Elad Ben Aharon <[email protected]>
ocs_ci/ocs/cluster_load.py
Outdated
| )[0]["value"][1] | ||
| ) | ||
| except requests.exceptions.RequestException as e: | ||
| print(f"Encountered error: {e}. Refreshing connection and retrying...") |
There was a problem hiding this comment.
not critical, but better to replace print with log.error ?
Signed-off-by: Elad Ben Aharon <[email protected]>
pull-request-size
bot
added
size/M
Signed-off-by: Elad Ben Aharon <[email protected]>
pull-request-size
bot
added
size/S
There was a problem hiding this comment.
PR validation
Cluster Name:
Cluster Configuration:
PR Test Suite: tier1
PR Test Path: tests/
Additional Test Params: --io-in-bg --log-cluster-utilization
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master
Job FAILED (installation failed, tests not executed).
|
Triggered verification job here: |
|
Another verification here: |
There was a problem hiding this comment.
PR validation
Cluster Name:
Cluster Configuration:
PR Test Suite: tier1
PR Test Path: tests/
Additional Test Params: --io-in-bg --log-cluster-utilization
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master
Job UNSTABLE (some or all tests failed).
Signed-off-by: Elad Ben Aharon <[email protected]>
|
New changes are detected. LGTM label has been removed. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ebenahar The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
2 similar comments
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ebenahar The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ebenahar The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Triggered new validation: |
There was a problem hiding this comment.
PR validation
Cluster Name:
Cluster Configuration:
PR Test Suite: tier1
PR Test Path: tests/
Additional Test Params: --io-in-bg --log-cluster-utilization
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master
Job UNSTABLE (some or all tests failed).
Fixes #11328