Skip to content

Add OpenRAG v0.6.0 build script - #8711

Merged
rcarino5 merged 3 commits into
ppc64le:masterfrom
afsanjar:add/openrag-0.6.0-ubi10-ppc64le
Sep 2, 2026
Merged

Add OpenRAG v0.6.0 build script#8711
rcarino5 merged 3 commits into
ppc64le:masterfrom
afsanjar:add/openrag-0.6.0-ubi10-ppc64le

Conversation

@pjsharath28

Copy link
Copy Markdown
Contributor

Checklist

  • Have you checked and followed all the points mention in the CONTRIBUTING.MD
  • Have you validated script on UBI 9 container - validated on UBI 10 container instead
  • Did you run the script(s) on fresh container with set -e option enabled and observe success ?
  • Did you have Legal approvals for patch files ?

@pjsharath28

Copy link
Copy Markdown
Contributor Author

@cdeepali Could I please have a review on this
cc @rcarino5

Comment thread o/openrag/openrag_0.6.0_ubi_10.2.sh Outdated
# openrag requires Python >=3.13; UBI 10 only ships 3.12.
# altinstall keeps system python3/python3.12 intact.
cd /tmp
wget https://www.python.org/ftp/python/3.13.10/Python-3.13.10.tgz

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please do not build Python 3.13 from source here because wrapper script is designed to handle that. The build will happen on Python 3.13 as well through CI even though the script installs python3.12.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Resolved.

Along with removal of Python 3.13 source build I also had to remove the smoke test — OpenRAG requires a live OpenSearch instance at runtime, and the wheel enforces requires-python>=3.13 which would cause the install to fail under the UBI 10 default Python 3.12.

Requesting re-review.

@pjsharath28
pjsharath28 requested a review from cdeepali September 2, 2026 04:32
@cdeepali

cdeepali commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

@pjsharath28 pls add License file for the package.

@pjsharath28

Copy link
Copy Markdown
Contributor Author

@pjsharath28 pls add License file for the package.

Added

@rcarino5

rcarino5 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Approving workflow

@pjsharath28

Copy link
Copy Markdown
Contributor Author

All checks passed.
@cdeepali @rcarino5 can we get it merged?

@rcarino5
rcarino5 self-requested a review September 2, 2026 14:44

@rcarino5 rcarino5 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Checks are passing, too.

@rcarino5
rcarino5 merged commit de84ae2 into ppc64le:master Sep 2, 2026
17 checks passed
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