Skip to content

Fix infinite Konflux update loop on release-0.19#3887

Merged
tpantelis merged 1 commit intosubmariner-io:release-0.19from
tpantelis:konflux_updates-0.19
Feb 13, 2026
Merged

Fix infinite Konflux update loop on release-0.19#3887
tpantelis merged 1 commit intosubmariner-io:release-0.19from
tpantelis:konflux_updates-0.19

Conversation

@tpantelis
Copy link
Contributor

@tpantelis tpantelis commented Feb 13, 2026

Summary by CodeRabbit

  • Chores
    • Updated CI/CD pipeline trigger conditions to optimize build execution patterns. Modified pipeline configuration to refine which code changes initiate automated processes.

Remove config/*** from Tekton trigger expressions to prevent
infinite build loop. The loop occurred because:

1. Konflux builds create new image with new SHA256
2. Renovate bot updates image digests in config/ files
3. config/*** trigger causes new build
4. Repeat indefinitely

This matches the configuration used in release-0.22 and newer
branches which do not experience this issue.

Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr3887/tpantelis/konflux_updates-0.19
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

Walkthrough

The pull request removes the config/*** pathChanged() condition from the trigger gates in two Tekton PipelineRun configurations (.tekton/submariner-operator-0-19-pull-request.yaml and .tekton/submariner-operator-0-19-push.yaml). Pipelines will no longer automatically trigger when only config directory files are modified.

Changes

Cohort / File(s) Summary
Tekton Trigger Conditions
.tekton/submariner-operator-0-19-pull-request.yaml, .tekton/submariner-operator-0-19-push.yaml
Removed config/*** from the pathChanged() OR-condition in on-cel-expression. Triggers now fire only for changes in .tekton YAML files, package/, internal/, or pkg/*** directories.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • Update Konflux references #3727: Modifies the same on-cel-expression gating in both .tekton/submariner-operator-0-19 YAML files with additional param and bundle updates.

Suggested labels

ready-to-test

Suggested reviewers

  • Oats87
  • skitt
  • sridhargaddam
  • vthapar
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: fixing an infinite Konflux update loop on release-0.19 by removing config/*** from Tekton trigger expressions.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into release-0.19
Actionable Comments Resolved ✅ Passed No unresolved actionable comments found in pull request. Only an informational automated message from submariner-bot exists, and code changes have been successfully applied without TODO/FIXME markers.

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


No actionable comments were generated in the recent review. 🎉


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

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

@tpantelis tpantelis added the ready-to-test When a PR is ready for full E2E testing label Feb 13, 2026
@tpantelis tpantelis merged commit 9e47b81 into submariner-io:release-0.19 Feb 13, 2026
44 of 45 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr3887/tpantelis/konflux_updates-0.19]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-test When a PR is ready for full E2E testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants