Skip to content

fix: Revert openblas and other runtime dependency on ppc64le - #322

Merged
ntkathole merged 1 commit into
rhoai-3.5from
ibmp_numpy_issue
Aug 17, 2026
Merged

ntkathole merged 1 commit into
rhoai-3.5from
ibmp_numpy_issue

Conversation

@jyejare

@jyejare jyejare commented Aug 14, 2026

Copy link
Copy Markdown

What this PR does / why we need it:

Revert changes made in #321 to regain feast operator success failing due to missing openblas and other relevent dependency.

Which issue(s) this PR fixes:

Fixes RHOAIENG-83758

Checks

  • I've made sure the tests are passing.
  • My commits are signed off (git commit -s)
  • My PR title follows conventional commits format

Testing Strategy

  • Testing is not required for this change

Misc

This is a build-script-only change (shell script for ppc64le image builds). No code tests are affected.

@github-actions

Copy link
Copy Markdown

Post-Codefreeze Gatekeeper — FAILED (evaluation)

Advisory only — this check does not block merging. Please review the results before merging.

Target branch: rhoai-3.5
Required Jira version: 3.5 GA RHOAI RELEASE

❌ The following issues were found:

  • RHOAIENG-83758:
    • Release Blocker is Proposed, expected Approved

After resolving issues, push a commit, re-edit the PR description, or add the run-gatekeeper label to re-trigger this check.

@jyejare

jyejare commented Aug 14, 2026

Copy link
Copy Markdown
Author

/build-konflux feature-server

@puneetsharma21

Copy link
Copy Markdown

Hi @jyejare , this file is no longer being used on the 3.5 branch. the issue came because of this PR merged #321

changes on dockerfile https://github.com/red-hat-data-services/feast/pull/321/changes#diff-0f0355cec3ca85a127c21ee04fb1908b7d461518a300fcb9b15e170620e9987cR31

We added the following system packages to ensure Feast works correctly on Power
python3.12-devel postgresql-devel gcc openblas-devel

@puneetsharma21

Copy link
Copy Markdown

Also, please revert the re2-related changes. These changes are also specific to Power, it won't affect the other architectures.
https://github.com/red-hat-data-services/feast/pull/321/changes#diff-0f0355cec3ca85a127c21ee04fb1908b7d461518a300fcb9b15e170620e9987cL37

…for ibm-p

Fixes RHOAIENG-83758

Signed-off-by: Jitendra Yejare <11752425+jyejare@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Jitendra Yejare <11752425+jyejare@users.noreply.github.com>
@jyejare jyejare changed the title fix: Add openblas-openmp runtime dependency for NumPy on ppc64le fix: Revert openblas and other runtime dependency on ppc64le Aug 14, 2026
@jyejare

jyejare commented Aug 14, 2026

Copy link
Copy Markdown
Author

/build-konflux feature-server

@jyejare

jyejare commented Aug 14, 2026

Copy link
Copy Markdown
Author

@puneetsharma21 Thanks for bringing the real issue. Please re-review, i have reverted the required changes.

@puneetsharma21

Copy link
Copy Markdown

Thanks. LGTM

@ntkathole
ntkathole merged commit c533679 into rhoai-3.5 Aug 17, 2026
1 of 4 checks passed
@ntkathole

Copy link
Copy Markdown
Member

@jyejare please get the same done for main branch

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