fix(test): use Vector devel image for HTTP TLS e2e test - NO JIRA

[Clee2691]

Description

This PR fixes the HTTP TLS e2e test by using the vector devel image instead.

Should unblock the failing master e2e tests

/cc @vparfonov
/assign @jcantrill

Summary by CodeRabbit

• Bug Fixes
  • Corrected the TLS-enabled HTTP receiver deployment to use the appropriate receiver container image.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: openshift/cluster-logging-operator/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: f7ae4443-6081-4204-a76a-e40c19959dad

📥 Commits

Reviewing files that changed from the base of the PR and between dde56c0 and fff9287.

📒 Files selected for processing (1)

• test/framework/e2e/http_vector.go

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.

---

📝 Walkthrough

Walkthrough

Changes

TLS receiver image selection

Layer / File(s)	Summary
Update TLS receiver image constant test/framework/e2e/http_vector.go	The TLS receiver deployment now uses constants.VectorReceiverName for its container image.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: jcantrill

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly states that the HTTP TLS end-to-end test should use the Vector development image. It matches the main change.
Description check	✅ Passed	The description explains the change and its purpose, and it includes both a reviewer assignment and an approver assignment. The optional Links section is omitted.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
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.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

---

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

[qodo-for-rh-openshift]

PR Summary by Qodo

Use Vector devel receiver image for HTTP TLS e2e tests

🐞 Bug fix 🧪 Tests 🕐 Less than 5 minutes

AI Description

• Selects the dedicated Vector receiver image for TLS-enabled HTTP receiver deployments.
• Aligns TLS and non-TLS receiver setup, unblocking failing end-to-end tests.

High-Level Assessment

The current approach is optimal because it reuses the established VectorReceiverName component mapping and makes the TLS helper consistent with the non-TLS receiver helper. Directly reading the receiver image environment variable would bypass centralized image resolution without providing a benefit.

Files changed (1) +1 / -1

Bug fix (1) +1 / -1

http_vector.goUse receiver-specific Vector image for TLS tests +1/-1

Use receiver-specific Vector image for TLS tests

• Changes the TLS-enabled HTTP receiver deployment to resolve the dedicated Vector receiver image instead of the standard Vector component image. This ensures the test container includes the development tooling expected by the end-to-end setup.

test/framework/e2e/http_vector.go

[qodo-for-rh-openshift]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Tip of the day

💡 Did you know, you can add REVIEW.md to your repo root and Qodo follows it on every PR

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[vparfonov]

/lgtm

[jcantrill]

/approve
/label verified

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Clee2691, jcantrill

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 [jcantrill]

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

[Clee2691]

/retest

[openshift-merge-bot]

/retest-required

Remaining retests: 0 against base HEAD dde56c0 and 2 for PR HEAD fff9287 in total

[Clee2691]

/retest

[vparfonov]

/test e2e-using-bundle

[Clee2691]

/test e2e-using-bundle

[qodo-for-rh-openshift]

Important

The /test command by Qodo was deprecated on the 1st of September 2026 and is no longer available. We recommend switching to the latest Qodo review capabilities. Learn more

[Clee2691]

closed because #3458 includes the fix