Skip to content

Commit 1de3e35

Browse files
committed
Remove jira ref form code
Signed-off-by: Milind waykole <mwaykole@redhat.com>
1 parent b28f0ed commit 1de3e35

File tree

3 files changed

+0
-19
lines changed

3 files changed

+0
-19
lines changed

tests/model_serving/model_server/kserve/authentication/conftest.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
from ocp_resources.secret import Secret
1414
from ocp_resources.service_account import ServiceAccount
1515
from ocp_resources.serving_runtime import ServingRuntime
16-
from simple_logger.logger import get_logger
1716

1817
from utilities.constants import (
1918
Annotations,
@@ -27,14 +26,10 @@
2726
from utilities.infra import (
2827
create_inference_token,
2928
create_isvc_view_role,
30-
get_pods_by_isvc_label,
3129
)
32-
from utilities.jira import is_jira_open
3330
from utilities.logger import RedactedString
3431
from utilities.serving_runtime import ServingRuntimeFromTemplate
3532

36-
LOGGER = get_logger(name=__name__)
37-
3833

3934
# HTTP/REST model serving
4035
@pytest.fixture(scope="class")
@@ -77,15 +72,9 @@ def http_raw_inference_token(model_service_account: ServiceAccount, http_raw_rol
7772

7873
@pytest.fixture()
7974
def patched_remove_raw_authentication_isvc(
80-
admin_client: DynamicClient,
8175
unprivileged_client: DynamicClient,
8276
http_s3_ovms_raw_inference_service: InferenceService,
8377
) -> Generator[InferenceService, Any, Any]:
84-
predictor_pod = get_pods_by_isvc_label(
85-
client=unprivileged_client,
86-
isvc=http_s3_ovms_raw_inference_service,
87-
)[0]
88-
8978
with ResourceEditor(
9079
patches={
9180
http_s3_ovms_raw_inference_service: {
@@ -95,10 +84,6 @@ def patched_remove_raw_authentication_isvc(
9584
}
9685
}
9786
):
98-
if is_jira_open(jira_id="RHOAIENG-52129", admin_client=admin_client):
99-
LOGGER.info("RHOAIENG-52129 is open; waiting for predictor pod rollout after auth toggle")
100-
predictor_pod.wait_deleted()
101-
10287
yield http_s3_ovms_raw_inference_service
10388

10489

tests/model_serving/model_server/kserve/authentication/test_kserve_token_authentication_raw.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
from utilities.constants import Annotations, Protocols
66
from utilities.inference_utils import Inference
77
from utilities.infra import check_pod_status_in_time, get_pods_by_isvc_label
8-
from utilities.jira import is_jira_issue_open
98
from utilities.manifests.onnx import ONNX_INFERENCE_CONFIG
109

1110
pytestmark = pytest.mark.usefixtures("valid_aws_config")
@@ -50,7 +49,6 @@ def test_disabled_raw_model_authentication(self, patched_remove_raw_authenticati
5049
)
5150

5251
@pytest.mark.smoke
53-
@pytest.mark.xfail(condition=is_jira_issue_open(jira_id="RHOAIENG-52129"), reason="RHOAIENG-52129", run=False)
5452
def test_raw_disable_enable_authentication_no_pod_rollout(self, http_s3_ovms_raw_inference_service):
5553
"""Verify no pod rollout when disabling and enabling authentication"""
5654
pod = get_pods_by_isvc_label(

tests/model_serving/model_server/kserve/storage/oci/test_oci_image.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ def test_model_car_no_restarts(self, model_car_inference_service):
5555

5656
@pytest.mark.tier1
5757
@pytest.mark.ocp_interop
58-
@pytest.mark.skip(reason="Skipping due to RHOAIENG-12306")
5958
def test_model_car_using_rest(self, model_car_inference_service):
6059
"""Verify model query with token using REST"""
6160
verify_inference_response(
@@ -68,7 +67,6 @@ def test_model_car_using_rest(self, model_car_inference_service):
6867

6968
@pytest.mark.tier1
7069
@pytest.mark.ocp_interop
71-
@pytest.mark.skip(reason="Skipping due to RHOAIENG-38674")
7270
def test_model_status_loaded(self, model_car_inference_service):
7371
"""Verify model status on the InferenceService resource is in a valid state."""
7472
model_status = model_car_inference_service.instance.status.modelStatus

0 commit comments

Comments
 (0)