ci: extend Konflux docs-only skip to match Prow skip rules

[mynhardtburger]

Extend the CEL expression in Tekton PipelineRun definitions to also skip Konflux builds when only .gitignore, OWNERS, PROJECT, LICENSE, .github/, or .tekton/ files are changed — matching the existing skip_if_only_changed regex used by the Prow ci-operator jobs.

An example of the existing prow ci-operator job skip condition:

skip_if_only_changed: ^docs/|\.md$|^(?:.*/)?(?:\.gitignore|OWNERS|PROJECT|LICENSE|CONTRIBUTING\.md)$|^\.github/|^\.tekton/

• https://github.com/openshift/release/blob/01591970ea5008bda7b05a49ce8a0905207182ee/ci-operator/jobs/opendatahub-io/models-as-a-service/opendatahub-io-models-as-a-service-main-presubmits.yaml#L21

Summary by CodeRabbit

Release Notes

• Chores
  • Updated pull request pipeline triggering conditions to exclude additional configuration files and CI/CD directories from initiating automated checks.

[openshift-ci]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[coderabbitai]

📝 Walkthrough

Walkthrough

Two Tekton Pipeline-as-Code configuration files updated to extend the CEL expression exclusion patterns for pull request triggers targeting the main branch. Previously excluded patterns (^docs/ and \.md$) are supplemented with additional exclusions for repository-level configuration files (.gitignore, OWNERS, PROJECT, LICENSE), .github/ directories, and .tekton/ directories. The predicate structure remains unchanged; only the regex list within files.all.all(...) is expanded across multiple lines.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: extending Konflux CI skip rules to match Prow behavior by excluding additional file patterns from triggering builds.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

.tekton/odh-maas-controller-pull-request.yaml (1)

12-14: Keep CEL skip regex in exact parity with the referenced Prow rule.

CONTRIBUTING\.md is not explicitly listed in this grouped alternative, so parity with the documented Prow regex is already drifting. Add it here to keep both rule sets synchronized if \.md$ is ever narrowed.

Suggested diff

-      || x.matches('^(.*/)?(\\.gitignore|OWNERS|PROJECT|LICENSE)$')
+      || x.matches('^(.*/)?(\\.gitignore|OWNERS|PROJECT|LICENSE|CONTRIBUTING\\.md)$')

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In @.tekton/odh-maas-controller-pull-request.yaml around lines 12 - 14, The CEL
skip-regex in the files.all.all(...) condition is missing CONTRIBUTING\.md and
thus drifts from the referenced Prow rule; update the grouped alternative
'^(.*/)?(\\.gitignore|OWNERS|PROJECT|LICENSE)$' inside that regex (the
expression used in the files.all.all(...) check) to include CONTRIBUTING\.md
(e.g., add '|CONTRIBUTING\\.md' into the list) so the YAML rule stays in exact
parity with the documented Prow regex.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In @.tekton/odh-maas-controller-pull-request.yaml:
- Around line 12-14: The CEL skip-regex in the files.all.all(...) condition is
missing CONTRIBUTING\.md and thus drifts from the referenced Prow rule; update
the grouped alternative '^(.*/)?(\\.gitignore|OWNERS|PROJECT|LICENSE)$' inside
that regex (the expression used in the files.all.all(...) check) to include
CONTRIBUTING\.md (e.g., add '|CONTRIBUTING\\.md' into the list) so the YAML rule
stays in exact parity with the documented Prow regex.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 926d037d-31a4-4f60-bdbb-9ba39d78f543

📥 Commits

Reviewing files that changed from the base of the PR and between 0b67ec8 and 87ff826.

📒 Files selected for processing (2)

• .tekton/odh-maas-api-pull-request.yaml
• .tekton/odh-maas-controller-pull-request.yaml

[chaitanya1731]

lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chaitanya1731, mynhardtburger

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [chaitanya1731,mynhardtburger]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment