From 12d44b96570b83fb317df6455001ae6465f7bfdd Mon Sep 17 00:00:00 2001 From: Ameya Sathe Date: Fri, 19 Jun 2026 16:14:57 +0530 Subject: [PATCH] OSAC-1629: skip prow ci for all .github/ directory changes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Expands skip_if_only_changed patterns to cover entire .github/ directory instead of just .github/workflows/ subdirectory. This reduces unnecessary CI runs when PRs only modify GitHub metadata files (CODEOWNERS, issue templates, dependabot config, etc.). Changes: - fulfillment-service: ^\.github/workflows/ → ^\.github/ - osac-aap: ^\.github/workflows/.* → ^\.github/.* - osac-operator: Added ^\.github/.* to existing pattern - osac-test-infra: Added ^\.github/.* to existing pattern - osac-installer: No change (run_if_changed already excludes .github/) Expected impact: ~10-15% reduction in unnecessary CI runs. Assisted-by: Claude Code Signed-off-by: Ameya Sathe rh-pre-commit.version: 2.4.0 rh-pre-commit.check-secrets: ENABLED --- .../osac-project-fulfillment-service-main.yaml | 4 ++-- .../osac-project/osac-aap/osac-project-osac-aap-main.yaml | 2 +- .../osac-operator/osac-project-osac-operator-main.yaml | 2 +- .../osac-test-infra/osac-project-osac-test-infra-main.yaml | 2 +- .../osac-project-fulfillment-service-main-presubmits.yaml | 4 ++-- .../osac-aap/osac-project-osac-aap-main-presubmits.yaml | 2 +- .../osac-project-osac-operator-main-presubmits.yaml | 2 +- .../osac-project-osac-test-infra-main-presubmits.yaml | 2 +- 8 files changed, 10 insertions(+), 10 deletions(-) diff --git a/ci-operator/config/osac-project/fulfillment-service/osac-project-fulfillment-service-main.yaml b/ci-operator/config/osac-project/fulfillment-service/osac-project-fulfillment-service-main.yaml index e1d7ad46dd3e0..0baef1af9697c 100644 --- a/ci-operator/config/osac-project/fulfillment-service/osac-project-fulfillment-service-main.yaml +++ b/ci-operator/config/osac-project/fulfillment-service/osac-project-fulfillment-service-main.yaml @@ -74,11 +74,11 @@ tests: commands: echo "Test" container: from: src - skip_if_only_changed: ^.*\.(md|adoc)$|^LICENSE$|^charts/|^it/|^\.github/workflows/ + skip_if_only_changed: ^.*\.(md|adoc)$|^LICENSE$|^charts/|^it/|^\.github/ - as: e2e-vmaas capabilities: - intranet - skip_if_only_changed: ^.*\.(md|adoc)$|^LICENSE$|^\.github/workflows/ + skip_if_only_changed: ^.*\.(md|adoc)$|^LICENSE$|^\.github/ steps: dependencies: COMPONENT_IMAGE: fulfillment-service-pr diff --git a/ci-operator/config/osac-project/osac-aap/osac-project-osac-aap-main.yaml b/ci-operator/config/osac-project/osac-aap/osac-project-osac-aap-main.yaml index 937b2d27ab38a..c3c7c557f5514 100644 --- a/ci-operator/config/osac-project/osac-aap/osac-project-osac-aap-main.yaml +++ b/ci-operator/config/osac-project/osac-aap/osac-project-osac-aap-main.yaml @@ -83,7 +83,7 @@ tests: - as: e2e-vmaas capabilities: - intranet - skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|samples/.*|charts/.*|\.github/workflows/.*|\.ansible-lint.*|OWNERS|LICENSE)$ + skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|samples/.*|charts/.*|\.github/.*|\.ansible-lint.*|OWNERS|LICENSE)$ steps: dependencies: COMPONENT_IMAGE: osac-aap-pr diff --git a/ci-operator/config/osac-project/osac-operator/osac-project-osac-operator-main.yaml b/ci-operator/config/osac-project/osac-operator/osac-project-osac-operator-main.yaml index 65a132e016ba8..b53e7ab70453a 100644 --- a/ci-operator/config/osac-project/osac-operator/osac-project-osac-operator-main.yaml +++ b/ci-operator/config/osac-project/osac-operator/osac-project-osac-operator-main.yaml @@ -54,7 +54,7 @@ tests: - as: e2e-vmaas capabilities: - intranet - skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|\.ai-bot/.*|\.claude/rules/.*|\.golangci\.yml|\.yamllint\.yaml|\.pre-commit-config.*\.yaml|\.dockerignore|\.gitattributes|OWNERS|LICENSE|bin/.*)$ + skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|\.ai-bot/.*|\.claude/rules/.*|\.github/.*|\.golangci\.yml|\.yamllint\.yaml|\.pre-commit-config.*\.yaml|\.dockerignore|\.gitattributes|OWNERS|LICENSE|bin/.*)$ steps: dependencies: COMPONENT_IMAGE: osac-operator-pr diff --git a/ci-operator/config/osac-project/osac-test-infra/osac-project-osac-test-infra-main.yaml b/ci-operator/config/osac-project/osac-test-infra/osac-project-osac-test-infra-main.yaml index 2dc8a375db998..101e5b173dcb4 100644 --- a/ci-operator/config/osac-project/osac-test-infra/osac-project-osac-test-infra-main.yaml +++ b/ci-operator/config/osac-project/osac-test-infra/osac-project-osac-test-infra-main.yaml @@ -65,7 +65,7 @@ tests: - as: e2e-vmaas capabilities: - intranet - skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|\.ai-bot/.*|\.claude/rules/.*|\.golangci\.yml|\.yamllint\.yaml|\.pre-commit-config.*\.yaml|\.dockerignore|\.gitattributes|OWNERS|LICENSE|bin/.*)$ + skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|\.ai-bot/.*|\.claude/rules/.*|\.github/.*|\.golangci\.yml|\.yamllint\.yaml|\.pre-commit-config.*\.yaml|\.dockerignore|\.gitattributes|OWNERS|LICENSE|bin/.*)$ steps: workflow: osac-project-cluster-tool-vmaas - as: e2e-vmaas-periodic diff --git a/ci-operator/jobs/osac-project/fulfillment-service/osac-project-fulfillment-service-main-presubmits.yaml b/ci-operator/jobs/osac-project/fulfillment-service/osac-project-fulfillment-service-main-presubmits.yaml index 85c1bccb49973..5bad52b6bb80a 100644 --- a/ci-operator/jobs/osac-project/fulfillment-service/osac-project-fulfillment-service-main-presubmits.yaml +++ b/ci-operator/jobs/osac-project/fulfillment-service/osac-project-fulfillment-service-main-presubmits.yaml @@ -18,7 +18,7 @@ presubmits: pj-rehearse.openshift.io/can-be-rehearsed: "true" name: pull-ci-osac-project-fulfillment-service-main-e2e-vmaas rerun_command: /test e2e-vmaas - skip_if_only_changed: ^.*\.(md|adoc)$|^LICENSE$|^\.github/workflows/ + skip_if_only_changed: ^.*\.(md|adoc)$|^LICENSE$|^\.github/ spec: containers: - args: @@ -158,7 +158,7 @@ presubmits: pj-rehearse.openshift.io/can-be-rehearsed: "true" name: pull-ci-osac-project-fulfillment-service-main-unit rerun_command: /test unit - skip_if_only_changed: ^.*\.(md|adoc)$|^LICENSE$|^charts/|^it/|^\.github/workflows/ + skip_if_only_changed: ^.*\.(md|adoc)$|^LICENSE$|^charts/|^it/|^\.github/ spec: containers: - args: diff --git a/ci-operator/jobs/osac-project/osac-aap/osac-project-osac-aap-main-presubmits.yaml b/ci-operator/jobs/osac-project/osac-aap/osac-project-osac-aap-main-presubmits.yaml index 11816d6040780..d2313f8ca045e 100644 --- a/ci-operator/jobs/osac-project/osac-aap/osac-project-osac-aap-main-presubmits.yaml +++ b/ci-operator/jobs/osac-project/osac-aap/osac-project-osac-aap-main-presubmits.yaml @@ -17,7 +17,7 @@ presubmits: pj-rehearse.openshift.io/can-be-rehearsed: "true" name: pull-ci-osac-project-osac-aap-main-e2e-vmaas rerun_command: /test e2e-vmaas - skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|samples/.*|charts/.*|\.github/workflows/.*|\.ansible-lint.*|OWNERS|LICENSE)$ + skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|samples/.*|charts/.*|\.github/.*|\.ansible-lint.*|OWNERS|LICENSE)$ spec: containers: - args: diff --git a/ci-operator/jobs/osac-project/osac-operator/osac-project-osac-operator-main-presubmits.yaml b/ci-operator/jobs/osac-project/osac-operator/osac-project-osac-operator-main-presubmits.yaml index 3a9758f981415..17b9f5a0341bb 100644 --- a/ci-operator/jobs/osac-project/osac-operator/osac-project-osac-operator-main-presubmits.yaml +++ b/ci-operator/jobs/osac-project/osac-operator/osac-project-osac-operator-main-presubmits.yaml @@ -18,7 +18,7 @@ presubmits: pj-rehearse.openshift.io/can-be-rehearsed: "true" name: pull-ci-osac-project-osac-operator-main-e2e-vmaas rerun_command: /test e2e-vmaas - skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|\.ai-bot/.*|\.claude/rules/.*|\.golangci\.yml|\.yamllint\.yaml|\.pre-commit-config.*\.yaml|\.dockerignore|\.gitattributes|OWNERS|LICENSE|bin/.*)$ + skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|\.ai-bot/.*|\.claude/rules/.*|\.github/.*|\.golangci\.yml|\.yamllint\.yaml|\.pre-commit-config.*\.yaml|\.dockerignore|\.gitattributes|OWNERS|LICENSE|bin/.*)$ spec: containers: - args: diff --git a/ci-operator/jobs/osac-project/osac-test-infra/osac-project-osac-test-infra-main-presubmits.yaml b/ci-operator/jobs/osac-project/osac-test-infra/osac-project-osac-test-infra-main-presubmits.yaml index cbdbfd735eda8..a4a7fd8184bd1 100644 --- a/ci-operator/jobs/osac-project/osac-test-infra/osac-project-osac-test-infra-main-presubmits.yaml +++ b/ci-operator/jobs/osac-project/osac-test-infra/osac-project-osac-test-infra-main-presubmits.yaml @@ -18,7 +18,7 @@ presubmits: pj-rehearse.openshift.io/can-be-rehearsed: "true" name: pull-ci-osac-project-osac-test-infra-main-e2e-vmaas rerun_command: /test e2e-vmaas - skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|\.ai-bot/.*|\.claude/rules/.*|\.golangci\.yml|\.yamllint\.yaml|\.pre-commit-config.*\.yaml|\.dockerignore|\.gitattributes|OWNERS|LICENSE|bin/.*)$ + skip_if_only_changed: ^(README\.md|AGENTS\.md|CLAUDE\.md|docs/.*|\.ai-bot/.*|\.claude/rules/.*|\.github/.*|\.golangci\.yml|\.yamllint\.yaml|\.pre-commit-config.*\.yaml|\.dockerignore|\.gitattributes|OWNERS|LICENSE|bin/.*)$ spec: containers: - args: