Skip to content

e2e: add TLS tests#4266

Open
shajmakh wants to merge 3 commits into
openshift-kni:release-4.22from
shajmakh:tls-e2e-test-bp
Open

e2e: add TLS tests#4266
shajmakh wants to merge 3 commits into
openshift-kni:release-4.22from
shajmakh:tls-e2e-test-bp

Conversation

@shajmakh
Copy link
Copy Markdown
Member

manual cherry-picks to get e2e TLS tests to run in u/s prow CI.

shajmakh and others added 3 commits May 29, 2026 17:04
Add basic verification tests that ensure the scheduler deployment is adherent to the expected values.

CHERRYPICK NOTE: the commit message is updated to better reflect the actual
tests goal.

Assisted-by: Cursor v2.3.37
AI-Attribution: AIA HAb Hin R Cursor v2.3.37 model:claude-4.6-opus-high

Signed-off-by: Shereen Haj <shajmakh@redhat.com>
(cherry picked from commit 617dce3)
Add binary-e2e-tls and build-e2e-tls targets, include the TLS test binary in
binary-e2e-all, and run e2e-nrop-tls.test after the scheduler install suite in
hack/run-test-e2e.sh. Skip the TLS suite in BeforeSuite when the OpenShift
APIServer API is unavailable so kind-based CI is not broken.

Assisted-by: Cursor 3.1.7
AI-attribution: AIA PAI Hin R Cursor 3.1.7 v1.0
Signed-off-by: Shereen Haj <shajmakh@redhat.com>
(cherry picked from commit 6dfbade)
Add unit and e2e tests that verify RTE DaemonSet metrics TLS flags
(min version and cipher suites) match the cluster TLS profile, including
the modern TLS 1.3 configuration. Consolidate TLS e2e cases under the
tls suite and serial configuration tests.
Add E2E test to check tls args passed to rte container.

AI Attribution: AIA Human-AI blend, New content, Human-initiated, Reviewed, Cursor 3.0.16 v1.0

Signed-off-by: Niranjan M.R <mniranja@redhat.com>
(cherry picked from commit 0f3dfbe)
@openshift-ci openshift-ci Bot requested review from Tal-or and mrniranjan May 29, 2026 14:10
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 29, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: shajmakh

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:

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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 29, 2026
@shajmakh
Copy link
Copy Markdown
Member Author

/cc @mrniranjan

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 29, 2026

Important

Review skipped

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

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: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: b3bd4624-b1a5-4705-bf5f-49796972db1f

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@yanirq
Copy link
Copy Markdown
Member

yanirq commented May 31, 2026

/hold

pre GA hold

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants