Verify IPv6 Connectivity Between Rook-Ceph-Exporter and OCP Prometheus #11480
Conversation
OdedViner
force-pushed
the
ipv6_gss_bz
branch
from
2db5ac0 to
6459fb0
Compare
There was a problem hiding this comment.
PR validation on existing cluster
Cluster Name: oviner23-feb
Cluster Configuration: conf/deployment/vsphere/upi_1az_rhcos_vsan_3m_3w_ipv6.yaml
PR Test Suite: tier1
PR Test Path: tests/functional/monitoring/prometheus/metrics/test_monitoring_defaults.py::test_monitoring_ipv6
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master
Job UNSTABLE (some or all tests failed).
There was a problem hiding this comment.
PR validation on existing cluster
Cluster Name: oviner23-feb
Cluster Configuration: conf/deployment/vsphere/upi_1az_rhcos_vsan_3m_3w_ipv6.yaml
PR Test Suite: tier1
PR Test Path: tests/functional/monitoring/prometheus/metrics/test_monitoring_defaults.py::test_monitoring_ipv6
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master
Job UNSTABLE (some or all tests failed).
OdedViner
force-pushed
the
ipv6_gss_bz
branch
from
8fbe363 to
2ae40f3
Compare
| @tier1 | ||
| @bugzilla("2297285") | ||
| @pytest.mark.polarion_id("OCS-XXX") | ||
| def test_monitoring_ipv6(threading_lock): |
There was a problem hiding this comment.
no rook-ceph-exporter on External mode clusters because rook does not manage ceph in this case.
I think this test is irrelevant to External mode and we may add "skip_if" decorator
| @tier1 | ||
| @bugzilla("2297285") | ||
| @pytest.mark.polarion_id("OCS-XXX") | ||
| def test_monitoring_ipv6(threading_lock): |
There was a problem hiding this comment.
should we check if ipv6 feature enabled on deployment before running this test? if not we may want to skip.
we'll probably need to add marker for ipv6_enabled
There was a problem hiding this comment.
I added a condition for IPv4 so that the test now supports both IPv6 and IPv4.
There was a problem hiding this comment.
if it in most cases will check on ipv4 (our default environment now) should not we rename test?
test_monitoring_ipv6 -> test_monitoring_ip_version , or something similar
Signed-off-by: Oded Viner <[email protected]>
Signed-off-by: Oded Viner <[email protected]>
OdedViner
force-pushed
the
ipv6_gss_bz
branch
from
2ae40f3 to
7d160fd
Compare
Signed-off-by: Oded Viner <[email protected]>
OdedViner
force-pushed
the
ipv6_gss_bz
branch
from
630bacc to
cd16c50
Compare
Signed-off-by: Oded Viner <[email protected]>
There was a problem hiding this comment.
PR validation on existing cluster
Cluster Name: oviner23-feb
Cluster Configuration: conf/deployment/vsphere/upi_1az_rhcos_vsan_3m_3w_ipv6.yaml
PR Test Suite: tier1
PR Test Path: tests/functional/monitoring/prometheus/metrics/test_monitoring_defaults.py::test_monitoring_ipv6
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: DanielOsypenko, OdedViner 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 |
Description:
This PR introduces a test to verify the connectivity of
rook-ceph-exporterpods to OpenShift Prometheus over IPv6.Test Procedure:
rook-ceph-exporterpods.prometheus-k8spod in the openshift-monitoring namespace.curlfrom within theprometheus-k8spod to access the/metricsendpoint of each exporter via IPv6.ceph_AsyncMessenger_Worker_msgr_connection.