Skip to content

chore: update tekton pipeline bundle digest and renovate config#16494

Open
ryankwilliams wants to merge 1 commit into
ansible:develfrom
ryankwilliams:tekton
Open

chore: update tekton pipeline bundle digest and renovate config#16494
ryankwilliams wants to merge 1 commit into
ansible:develfrom
ryankwilliams:tekton

Conversation

@ryankwilliams

@ryankwilliams ryankwilliams commented Jun 10, 2026

Copy link
Copy Markdown
Member
SUMMARY

This PR updates the run atf tekton pipeline bundle sha ahead of mint maker to unblock pipelines when deploying aap-dev. Also simplifies the renovate config to attempt to have mint maker open PRs to update the tekton pipeline bundle digest automatically.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change

Summary by CodeRabbit

  • Chores
    • Updated automated test pipeline to use a newer test bundle, improving test execution consistency and reliability.
    • Revised dependency-update behavior: Tekton updates are now enabled with auto-merging and run on a twice-daily business-day schedule to streamline maintenance.

@ryankwilliams

Copy link
Copy Markdown
Member Author

/run-atf-tests

@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: fab97581-26e6-45ca-b8df-ca3dc71c54b8

📥 Commits

Reviewing files that changed from the base of the PR and between 2217ea0 and 10a6175.

📒 Files selected for processing (2)
  • .tekton/run-atf-tests-pull-request.yaml
  • renovate.json
✅ Files skipped from review due to trivial changes (1)
  • .tekton/run-atf-tests-pull-request.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • renovate.json

📝 Walkthrough

Walkthrough

Updates Tekton PipelineRun bundle digest for aap-api-tests and revises Renovate's Tekton manager configuration: enables Tekton updates and automerge, changes the schedule to 0 */12 * * 1-5, and removes the prior package-specific rules.

Changes

Tekton and Renovate dependency management updates

Layer / File(s) Summary
Pipeline bundle image digest update
.tekton/run-atf-tests-pull-request.yaml
The aap-api-tests pipeline bundle sha256 digest is updated in the Tekton PipelineRun reference.
Renovate Tekton configuration simplification
renovate.json
Tekton manager configuration is updated with enabled and automerge set to true, schedule changed to 0 */12 * * 1-5, and the previous Tekton-specific packageRules block is removed.

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main changes in the changeset: updating the Tekton pipeline bundle digest and modifying the Renovate configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@renovate.json`:
- Around line 5-7: The top-level "automerge": true now applies to the tekton
manager and will merge all Tekton updates; either restore package-specific rules
or add safeguards. If you want package-scoped automerge, remove or set top-level
"automerge" to false and re-add a "packageRules" block with
"matchPackagePatterns" targeting "quay.io/aap-ci/tekton-catalog/pipeline/*" and
desired "automerge" settings; if you truly intend to automerge all Tekton deps,
keep "automerge": true but add protections such as "minimumReleaseAge" and
explicit exclusion rules in "packageRules" (or negative "matchPackagePatterns")
for critical pipelines to prevent immediate merging of risky updates.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 8c753230-2ee7-4af0-ad7c-18823e734e0b

📥 Commits

Reviewing files that changed from the base of the PR and between d5e5ea3 and 62ea910.

📒 Files selected for processing (2)
  • .tekton/run-atf-tests-pull-request.yaml
  • renovate.json

Comment thread renovate.json
@aap-pde-ci-bot

Copy link
Copy Markdown

❌ Test Results - FAILED

Summary

Metric Count
Total Tests 1334
✅ Passed 1251
❌ Failed 2
⚠️ Errors 20
⏭️ Skipped 61
⏱️ Duration 2873.68s

Pass Rate: 93.8%

❌ Failed Tests

Test Class
test_user_can_delete_inventories_with_role[Inventory Admin-202-Gateway] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_user_can_delete_inventories_with_role[Inventory Admin-202-Controller] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_user_can_delete_inventories_with_role[Organization Admin-202-Gateway] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_user_can_delete_inventories_with_role[Organization Admin-202-Controller] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_user_can_delete_inventories_with_role[Organization Inventory Admin-202-Gateway] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_user_can_delete_inventories_with_role[Organization Inventory Admin-202-Controller] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_team_can_delete_inventories_with_role[Inventory Admin-202-Gateway] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_team_can_delete_inventories_with_role[Inventory Admin-202-Controller] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_team_can_delete_inventories_with_role[Organization Inventory Admin-202-Gateway] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_team_can_delete_inventories_with_role[Organization Inventory Admin-202-Controller] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_delete_inventory_and_associated_rbac_assignment[Gateway] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_delete_inventory_and_associated_rbac_assignment[Controller] opt.test-suite.tests.rbac.test_inventories.TestControllerRBACInventories
test_team_can_test_notification_templates_with_role[NotificationTemplate Admin-202-Gateway] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_team_can_test_notification_templates_with_role[NotificationTemplate Admin-202-Controller] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_user_can_test_notification_templates_with_role[NotificationTemplate Admin-202-Gateway] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_team_can_test_notification_templates_with_role[Organization NotificationTemplate Admin-202-Gateway] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_user_can_test_notification_templates_with_role[NotificationTemplate Admin-202-Controller] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_team_can_test_notification_templates_with_role[Organization NotificationTemplate Admin-202-Controller] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_user_can_test_notification_templates_with_role[Organization Admin-202-Gateway] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_user_can_test_notification_templates_with_role[Organization Admin-202-Controller] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_user_can_test_notification_templates_with_role[Organization NotificationTemplate Admin-202-Gateway] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates
test_user_can_test_notification_templates_with_role[Organization NotificationTemplate Admin-202-Controller] opt.test-suite.tests.rbac.test_rbac_notification_templates.TestControllerRBACNotificationTemplates

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Ryan Williams <3375653+ryankwilliams@users.noreply.github.com>
@sonarqubecloud

Copy link
Copy Markdown

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants