Merge from main by hbelmiro · Pull Request #73 · opendatahub-io/pipelines-components

hbelmiro · 2026-05-15T14:22:38Z

Description of your changes:

Checklist:

Pre-Submission Checklist

All tests and CI checks pass
Pre-commit hooks pass without errors
You have signed off your commits
The title for your pull request (PR) should follow our title convention.
Learn more about the pull request title convention used in this repository.

Additional Checklist Items for New or Updated Components/Pipelines

metadata.yaml includes fresh lastVerified timestamp
All required files
are present and complete
OWNERS file lists appropriate maintainers
README provides clear documentation with usage examples
Component follows snake_case naming convention
No security vulnerabilities in dependencies
Containerfile included if using a custom base image

Signed-off-by: Mateusz Switala <mswitala@redhat.com>

Assisted by Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

…packages-list-in-pyproject-toml fix(package): add missing utils dir to the packages list in pyproject.toml

…ommand Signed-off-by: Helber Belmiro <helber.belmiro@gmail.com>

…ents and notebooks Assisted by Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

fix(components, automl): Use svg image as HTML <img> in AutoML notebooks

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

This reverts commit cd2437b. Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

fix NaNs in AutoML: regression

chore: update autogluon package versions to 1.5.0+rhaiv.2 in requirements and notebooks

Signed-off-by: Mateusz Switala <mswitala@redhat.com> Assisted-by: Claude Code

…th validation and error handling - Refactored the model download functions in classification, regression, and timeseries notebooks to enhance path validation. - Added checks to prevent invalid paths during model artifact downloads. - Improved error handling for SSL issues when accessing S3 resources. Asissted by Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com> # Conflicts: # components/training/automl/autogluon_models_training/notebook_templates/classification_notebook.ipynb # components/training/automl/autogluon_models_training/notebook_templates/regression_notebook.ipynb

Assisted by: Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

Signed-off-by: Mateusz Switala <mswitala@redhat.com> Assisted-by: Claude Code

Assisted by: Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

Assisted by Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

…timeseres_models_json_artifact_schema fix(components, AutoML): align model.json between tabular and timeseries

…fp-dependencies chore(deps): Use kfp v2.16.1

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com>

Signed-off-by: Nelesh Singla <117123879+nsingla@users.noreply.github.com>

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

…3 artifact lookup Allow CI to supply a pre-compiled pipeline YAML via RHOAI_COMPILED_PIPELINE_PATH instead of always compiling locally. This is needed when the cluster runtime image supports a different kfp version than the local environment. When the S3 artifact lookup fails (e.g. cluster-internal MinIO endpoint unreachable from Jenkins), derive the external MinIO route from RHOAI_URL and retry with verify=False. Co-authored-by: Cursor <cursoragent@cursor.com>

- Use descriptive variable names in _collect_artifact_keys (was h,n,p,r) - Use urlparse instead of greedy regex for external endpoint derivation - Respect KFP_VERIFY_SSL for retry S3 client instead of hardcoding verify=False - Re-raise exceptions when both primary and retry S3 lookups fail - Add RHOAI_COMPILED_PIPELINE_PATH to .env.example with description Co-authored-by: Cursor <cursoragent@cursor.com>

The primary s3_client fixture was missing verify=verify_ssl, causing SSL errors when connecting to MinIO with self-signed certs. The retry path already respected this flag but the initial client did not. Co-authored-by: Cursor <cursoragent@cursor.com>

…ooks templates Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com>

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

Signed-off-by: Wojciech Rebisz <147821486+Wojciech-Rebisz@users.noreply.github.com>

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com>

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

Co-authored-by: Cursor <cursoragent@cursor.com>

…-stack-to-ogx feat(components): Rebrand Llama Stack to OGX across autorag components, remove in-memory scenario, and update ai4rag to 0.6.1

…rride-and-artifact-retry fix(autorag): support pre-compiled pipeline YAML override and retry S3 artifact lookup

…variables fix: Rename variables in consts.py

adding build time dependencies for Cachi2 Signed-off-by: Nelesh Singla <117123879+nsingla@users.noreply.github.com>

ci: Adding support for Hermetic Builds

…h-main

coderabbitai · 2026-05-15T14:22:48Z

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (4)

main
master
incubation
rhoai

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

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

Review profile: CHILL

Plan: Enterprise

Run ID: 9969d1cb-603d-48d9-85dc-c40070f6a01e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger 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.}

openshift-ci · 2026-05-15T14:22:48Z

[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 nsingla 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:

OWNERS

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

Mateusz-Switala and others added 30 commits April 16, 2026 15:10

add mising util dir to packages list in pyproject.toml

c6ebd06

Signed-off-by: Mateusz Switala <mswitala@redhat.com>

fix(components, automl): Use svg image as HTML <img> in AutoML notebooks

b6e16e3

Assisted by Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

Merge pull request kubeflow#39 from Mateusz-Switala/fix-add-utils-to-…

bcc83b3

…packages-list-in-pyproject-toml fix(package): add missing utils dir to the packages list in pyproject.toml

fix: Remove redundant --extra test flag in Dockerfile's uv sync c…

50c4c80

…ommand Signed-off-by: Helber Belmiro <helber.belmiro@gmail.com>

chore: update autogluon package versions to 1.5.0+rhaiv.2 in requirem…

4d8f5a9

…ents and notebooks Assisted by Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

Merge pull request kubeflow#41 from DorotaDR/fix_automl_banners_odh

01d2b84

fix(components, automl): Use svg image as HTML <img> in AutoML notebooks

fix NaNs in AutoML: regression

68e18f5

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

duplicates and infs handling + unit tests

bed1121

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

pylint check fixes

830a06f

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

more pylint checks

71f50ed

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

clean frame after reading

350a869

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

unit tests added

16880e1

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

update docstrings

6b57fd7

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

PR comments submitted by claude addressed

3ce9e4c

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

update test configs based on review comments

246fbfe

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

keep cleansing in data loading component only

5f3ac73

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

ruff check fix

2b0d44f

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

Add extra-index-url for RH test index

0b8acb5

Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

update the inf handling from math to np

cd2437b

Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

Revert "update the inf handling from math to np"

eadbe00

This reverts commit cd2437b. Signed-off-by: Lukasz Cmielowski <lcmielow@redhat.com> Assisted-by: Cursor

Merge pull request kubeflow#47 from LukaszCmielowski/fix_automl_nans

91d7b65

fix NaNs in AutoML: regression

Merge pull request kubeflow#44 from DorotaDR/fix-autogluon-v150-rhaiv2

9b1ae78

chore: update autogluon package versions to 1.5.0+rhaiv.2 in requirements and notebooks

align model.json between tabular and timeseries

065e1df

Signed-off-by: Mateusz Switala <mswitala@redhat.com> Assisted-by: Claude Code

chore(automl): added upper limitation for resources

58f2409

Assisted by: Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

change field name to single

dc41f88

Signed-off-by: Mateusz Switala <mswitala@redhat.com> Assisted-by: Claude Code

chore(autorag, automl): added upper limitation for resources

1e3be86

Assisted by: Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

chore: ruff changes

73274f7

Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

refactor(pipelines): Defined constants for max cpus and memory

fe4396d

Assisted by Cursor Signed-off-by: Dorota Laczak <dlaczak@redhat.com>

Merge pull request kubeflow#49 from Mateusz-Switala/fix_align_tabula_…

08239e2

…timeseres_models_json_artifact_schema fix(components, AutoML): align model.json between tabular and timeseries

openshift-merge-bot Bot and others added 22 commits May 11, 2026 13:12

Merge pull request kubeflow#62 from Wojciech-Rebisz/dev-add-missing-k…

06379b9

…fp-dependencies chore(deps): Use kfp v2.16.1

Update metadata and .toml

4cdb465

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com>

Updat README.md

cde7abd

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com>

Adding support for Hermetic builds in midstream

9aefdf4

Signed-off-by: Nelesh Singla <117123879+nsingla@users.noreply.github.com>

Update for ogx 1.0.0 and ai4rag 0.6.1

3039cd1

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

Update metadata, validation script with allowed ~= operator and noteb…

8edb421

…ooks templates Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

Merge main

99e759a

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com>

Update after code rabbit review

40953db

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

Rename variables in consts.py

51e752e

Signed-off-by: Wojciech Rebisz <147821486+Wojciech-Rebisz@users.noreply.github.com>

Update missing param in patter.json

f8c1da6

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com>

Update after code rabbit review

5cb16fa

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

Update tests for new parameter constraints

faf7d62

Signed-off-by: Jakub Walaszczyk <jwalaszc@redhat.com> Assisted-by: Claude Code

style(autorag): fix ruff formatting in test_pipeline_integration.py

c799309

Co-authored-by: Cursor <cursoragent@cursor.com>

Merge pull request kubeflow#67 from jakub-walaszczyk/rebranding-llama…

9ab65b8

…-stack-to-ogx feat(components): Rebrand Llama Stack to OGX across autorag components, remove in-memory scenario, and update ai4rag to 0.6.1

Merge pull request kubeflow#71 from angaduom/fix/autorag-pipeline-ove…

41d5e31

…rride-and-artifact-retry fix(autorag): support pre-compiled pipeline YAML override and retry S3 artifact lookup

Merge pull request kubeflow#72 from Wojciech-Rebisz/dev-rename-autox-…

b0ca633

…variables fix: Rename variables in consts.py

implement pr suggestions

6df3cc2

adding build time dependencies for Cachi2 Signed-off-by: Nelesh Singla <117123879+nsingla@users.noreply.github.com>

Merge pull request kubeflow#68 from nsingla/RHOAIENG-59984-odh

66be6f9

ci: Adding support for Hermetic Builds

Merge remote-tracking branch 'opendatahub-io/main' into merge-from-od…

d2ba48e

…h-main

openshift-ci Bot requested review from DorotaDR and filip-komarzyniec May 15, 2026 14:22

hbelmiro marked this pull request as draft May 15, 2026 15:20

openshift-ci Bot added the do-not-merge/work-in-progress label May 15, 2026

hbelmiro closed this May 15, 2026

hbelmiro deleted the merge-from-odh-main branch May 15, 2026 15:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Merge from main#73

Merge from main#73
hbelmiro wants to merge 99 commits into
opendatahub-io:stablefrom
hbelmiro:merge-from-odh-main

hbelmiro commented May 15, 2026

Uh oh!

coderabbitai Bot commented May 15, 2026

Review skipped

Uh oh!

openshift-ci Bot commented May 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

Conversation

hbelmiro commented May 15, 2026

Pre-Submission Checklist

Additional Checklist Items for New or Updated Components/Pipelines

Uh oh!

coderabbitai Bot commented May 15, 2026

Review skipped

Uh oh!

openshift-ci Bot commented May 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants