Merge branch 'main' into several_detectors

Author: kpunwatk

.coderabbit.yaml

@@ -7,7 +7,7 @@ reviews:
   request_changes_workflow: false
   high_level_summary: true
   high_level_summary_placeholder: '@coderabbitai summary'
-  high_level_summary_in_walkthrough: true
+  high_level_summary_in_walkthrough: false
   auto_title_placeholder: '@coderabbitai'
   auto_title_instructions: ''
   review_status: true
@@ -25,7 +25,7 @@ reviews:
   auto_assign_reviewers: false
   poem: false
   labeling_instructions: []
-  path_filters: []
+  path_filters: ["!.github/**"]
   path_instructions: []
   abort_on_close: true
   disable_cache: false

.github/CODEOWNERS

@@ -9,3 +9,10 @@
 
 # Global rule:
 *       @opendatahub-io/opendatahub-tests-contributors
+
+# Test directory specific rules
+tests/model_explainability/       @adolfo-ab @sheltoncyril
+tests/model_registry/             @dbasunag @lugi0 @fege
+tests/model_serving/              @mwaykole @Raghul-M
+tests/rag/                        @jgarciao @jiripetrlik
+tests/workbenches                 @jstourac @andyatmiami

.pre-commit-config.yaml

@@ -6,7 +6,7 @@ default_language_version:
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: "v5.0.0"
+    rev: "v6.0.0"
     hooks:
       - id: check-merge-conflict
       - id: debug-statements
@@ -36,7 +36,7 @@ repos:
         exclude: .*/__snapshots__/.*|.*-input\.json$
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.12.7
+    rev: v0.12.8
     hooks:
       - id: ruff
       - id: ruff-format

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.13
+FROM fedora:42
 
 ARG USER=odh
 ARG HOME=/home/$USER
@@ -11,18 +11,17 @@ ENV UV_NO_CACHE=1
 ENV BIN_DIR="$HOME_DIR/.local/bin"
 ENV PATH="$PATH:$BIN_DIR"
 
-RUN apt-get update \
-    && apt-get install -y ssh gnupg software-properties-common curl gpg wget vim apache2-utils rsync\
-    && apt-get clean autoclean \
-    && apt-get autoremove --yes \
-    && rm -rf /var/lib/{apt,dpkg,cache,log}/
+# Install Python 3.13 and other dependencies using dnf
+RUN dnf update -y \
+    && dnf install -y python3.13 python3.13-pip ssh gnupg curl gpg wget vim httpd-tools rsync openssl openssl-devel\
+    && dnf clean all \
+    && rm -rf /var/cache/dnf
 
 # Install grpcurl
-RUN curl -sSL "https://github.com/fullstorydev/grpcurl/releases/download/v1.9.2/grpcurl_1.9.2_linux_x86_64.tar.gz"  --output /tmp/grpcurl_1.2.tar.gz \
+RUN curl -sSL "https://github.com/fullstorydev/grpcurl/releases/download/v1.9.2/grpcurl_1.9.2_linux_x86_64.tar.gz" --output /tmp/grpcurl_1.2.tar.gz \
     && tar xvf /tmp/grpcurl_1.2.tar.gz --no-same-owner \
     && mv grpcurl /usr/bin/grpcurl
 
-
 RUN useradd -ms /bin/bash $USER
 USER $USER
 WORKDIR $HOME_DIR

docs/DEVELOPER_GUIDE.md

@@ -66,6 +66,12 @@ The project utilizes [openshift-python-wrapper](https://github.com/RedHatQE/open
 Please refer to the [documentation](https://github.com/RedHatQE/openshift-python-wrapper/blob/main/README.md)  
 and the [examples](https://github.com/RedHatQE/openshift-python-wrapper/tree/main/examples) for more information.
 
+For any missing resources, please generate a new resource using
+[class_generator tool](https://github.com/RedHatQE/openshift-python-wrapper/blob/main/class_generator/README.md) and
+create a PR against wrapper. Calls to cluster resources from tests, utils and fixtures must always use
+openshift-python-wrapper resource or oc command
+(when wrapper resource is not relevant. e.g. must-gather generation)
+
 
 ## Conftest
 - Top level [conftest.py](../conftest.py) contains pytest native fixtures.

renovate.json

@@ -1,7 +1,8 @@
 {
   "$schema": "https://docs.renovatebot.com/renovate-schema.json",
   "extends": [
-    ":dependencyDashboard",
+    "config:recommended",
+    ":dependencyDashboardApproval",
     ":maintainLockFilesWeekly",
     ":prHourlyLimitNone"
   ],
@@ -10,10 +11,29 @@
     "enabled": true
   },
   "baseBranches": ["main"],
+  "recreateWhen": "never",
+  "timezone": "America/New_York",
   "packageRules": [
     {
-      "matchPackagePatterns": ["*"],
-      "groupName": "python-dependencies"
-    }
+      "groupName": "Python dependencies (minor and patch)",
+      "matchUpdateTypes": ["minor", "patch"],
+      "matchFileNames": ["pyproject.toml"],
+      "schedule": ["before 7am on monday"],
+      "prPriority": 7
+    },
+    {
+      "groupName": "Python dependencies (major)",
+      "matchUpdateTypes": ["major"],
+      "matchFileNames": ["pyproject.toml"],
+      "schedule": ["after 7pm on thursday"],
+      "prPriority": 10
+    },
+    {
+        "groupName": "GitHub Actions",
+        "matchManagers": ["github-actions"],
+        "matchFileNames": [".github/workflows/**"],
+        "schedule": ["before 6pm on wednesday"],
+        "prPriority": 5
+      }
   ]
 }

tests/conftest.py

@@ -416,18 +416,6 @@ def dsc_resource(admin_client: DynamicClient) -> DataScienceCluster:
     return DataScienceCluster(client=admin_client, name=py_config["dsc_name"], ensure_exists=True)
 
 
-@pytest.fixture(scope="module")
-def updated_dsc_component_state(
-    request: FixtureRequest,
-    dsc_resource: DataScienceCluster,
-) -> Generator[DataScienceCluster, Any, Any]:
-    with update_components_in_dsc(
-        dsc=dsc_resource,
-        components={request.param["component_name"]: request.param["desired_state"]},
-    ) as dsc:
-        yield dsc
-
-
 @pytest.fixture(scope="package")
 def enabled_modelmesh_in_dsc(
     dsc_resource: DataScienceCluster,