Skip to content

CI: Tekton pipeline sync - #4346

Open
odh-devops-app[bot] wants to merge 1 commit into
mainfrom
ci-notebooks-1786385969
Open

CI: Tekton pipeline sync#4346
odh-devops-app[bot] wants to merge 1 commit into
mainfrom
ci-notebooks-1786385969

Conversation

@odh-devops-app

Copy link
Copy Markdown
Contributor

This PR updates Tekton pipelines for a CI build.

  • Pull pipelines → odh-pr
  • Push pipelines → odh-stable
  • PR target branch: main
  • Build branch:

- Pull pipelines use odh-pr tag
- Push pipelines use odh-stable tag
- Target branch: main

Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@openshift-ci
openshift-ci Bot requested review from dibryant and ysok August 10, 2026 18:19
@openshift-ci

openshift-ci Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign ysok for approval. For more information see the Code Review Process.

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

Details Needs approval from an approver in each of these files:

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

@github-actions github-actions Bot added the review-requested GitHub Bot creates notification on #pr-review-ai-ide-team slack channel label Aug 10, 2026
@openshift-ci openshift-ci Bot added the size/l label Aug 10, 2026
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Superseded by newer run: https://github.com/opendatahub-io/notebooks/actions/runs/31418501533

CI status [antigravity]

Run: Build Notebooks (push) #314185013403/3 complete · 3 skipped
Last updated: 2026-08-10T18:19:54Z

No workbench image jobs ran; all matrix jobs were skipped.

@openshift-ci

openshift-ci Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Hi @odh-devops-app[bot]. Thanks for your PR.

I'm waiting for a opendatahub-io member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Comment on lines +50 to +51
secret:
secretName: '{{ git_auth_secret }}'
Comment on lines +45 to +46
secret:
secretName: '{{ git_auth_secret }}'
Comment on lines +50 to +51
secret:
secretName: '{{ git_auth_secret }}'
Comment on lines +45 to +46
secret:
secretName: '{{ git_auth_secret }}'
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

📋 Review Summary

This PR updates Tekton pipeline definitions by adding new baseline CPU pipeline runs for CentOS Stream 9 (c9s) and adjusting test group configurations. The YAML files are syntactically sound and correctly structured for Konflux CI integration.

🔍 General Feedback

  • The added pipeline runs follow established Konflux conventions for pull request and push events.
  • Workspace and parameter mappings correctly target the multi-arch container build templates.

@github-actions

Copy link
Copy Markdown
Contributor

CI status [antigravity]

Run: Build Notebooks (pr) #314185015333/3 complete · 1 passed · 2 skipped
Last updated: 2026-08-10T18:20:30Z

No workbench image jobs ran; all matrix jobs were skipped.

@codecov-commenter

codecov-commenter commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.29%. Comparing base (34126ac) to head (c525a9e).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4346   +/-   ##
=======================================
  Coverage   44.29%   44.29%           
=======================================
  Files          46       46           
  Lines        5861     5861           
  Branches      977      977           
=======================================
  Hits         2596     2596           
  Misses       3045     3045           
  Partials      220      220           
Flag Coverage Δ
python 44.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 34126ac...c525a9e. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ysok

ysok commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Closing — superseded, please do not merge

This bot PR adds Tekton files we already have on main, from:

Those use the *-ubi9-* pipeline files with build-args-file and *-ci-push.yaml. This PR adds duplicate *-c9s-* files without build-args-file, which causes the Konflux error:

no FROM statement found

Merging would duplicate pipelines and break notebooks-group-test.yaml.

Next steps: close #4346, #4355, and #4425; mark the Jira Tekton step done using the PRs above; don’t re-run the onboarder until odh-konflux-central templates are fixed.

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

Labels

needs-ok-to-test review-requested GitHub Bot creates notification on #pr-review-ai-ide-team slack channel size/l

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants