[SudipSinha]

Description

Rename RELATED_IMAGE_ODH_TRUSTYAI_SERVICE_IMAGE to RELATED_IMAGE_ODH_TRUSTYAI_SERVICE_PY_IMAGE in the TrustyAI image param map to align with the upstream trustyai-service-operator.

Related:

1. fix(trustyai): update RELATED_IMAGE for odh-trustyai-service-py trustyai-explainability/trustyai-service-operator#887
2. https://redhat.atlassian.net/browse/RHAISTRAT-130

Release tracking

Release: rhoai-3.6-EA1

E2E test suite update requirement

• Skip requirement to update E2E test suite for this PR

E2E update requirement opt-out justification

This PR only renames an environment variable reference — no operator logic or behavior changes. No E2E test updates are required.

Summary by CodeRabbit

Bug Fixes

• Updated the TrustyAI service image configuration to use the correct Python service image reference.
• Added the image reference to the supported component configuration, improving deployment compatibility and ensuring the service uses the intended image.

[openshift-ci]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro Plus

Run ID: f5223186-4108-4c9f-a331-4ad5f3557e0b

📥 Commits

Reviewing files that changed from the base of the PR and between 6c27622 and ba8524b.

📒 Files selected for processing (1)

• component-params-env.yaml

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

---

📝 Walkthrough

Walkthrough

The TrustyAI service image mapping now uses RELATED_IMAGE_ODH_TRUSTYAI_SERVICE_PY_IMAGE. The variable is also listed in the validation exceptions with its RHOAI 3.6 EA1 tracking note.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to ba852

This PR only renames the TrustyAI service image environment-variable reference, with no stated operator behavior change; no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 10

✅ Passed checks (10 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the TrustyAI service image parameter rename, which is the main change in the pull request.
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.
Contribution Quality And Spam Detection	✅ Passed	The diff is a code-specific two-file image-variable rename linked to an upstream PR and Jira issue; it adds no security or validation logic, so no required security/code-quality signal is present.
No Hardcoded Secrets	✅ Passed	The PR changes only an environment-variable identifier and a YAML exception reason; added lines contain no credentials, secret assignments, private-key markers, credential URLs, or long base64 stri...
No Weak Cryptography	✅ Passed	The diff only renames a TrustyAI image environment variable and adds YAML metadata; it introduces no cryptographic primitive, custom crypto, or secret comparison.
No Injection Vectors	✅ Passed	The diff changes only a hardcoded image-key identifier and adds a static YAML exception; it introduces no SQL, shell execution, eval, unsafe YAML, or HTML sink (CWE-89, CWE-78, CWE-94, CWE-502, CWE...
No Privileged Containers	✅ Passed	The diff only renames an image parameter and adds a validation exception; it adds no manifest or Dockerfile security settings, so no privileged-container condition is introduced.
No Sensitive Data In Logs	✅ Passed	The diff changes only an image parameter mapping and YAML exception entry; it adds no logging calls, request bodies, credentials, tokens, or PII.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[MarianMacik]

Related image validation check is failing because it references RHOAI-Build-Config from 3.5 branch. Please add a RHOAI exception for it in component-params-env.yaml.

[MarianMacik]

@SudipSinha coincidentally #3984 merged so you can revert the last change, everything should work now.

[SudipSinha]

@MarianMacik I've reverted the change, but the check is still failing.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.84%. Comparing base (31f4a31) to head (a238edb).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4011   +/-   ##
=======================================
  Coverage   57.84%   57.84%           
=======================================
  Files         173      173           
  Lines       13090    13090           
=======================================
  Hits         7572     7572           
  Misses       4907     4907           
  Partials      611      611           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[MarianMacik]

Fixed now.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: MarianMacik

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [MarianMacik]

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