chore: re-enable tests in verify-tls-config-with-external-azure-db.spec.ts

[christoph-jerolimov]

This is a follow-up PR on #3839 to re-enable tests.

This e2e-test should pass at least 3 (more?) times with 100% success ratio to get merged.

/assign @redhat-developer/rhdh-cope
/hold

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign psrna for approval. For more information see the Code Review Process.

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

Details

Needs approval from an approver in each of these files:

• e2e-tests/OWNERS

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

[rhdh-qodo-merge]

You are nearing your monthly Qodo Merge usage quota. For more information, please visit here.

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🔒 No security concerns identified

⚡ Recommended focus areas for review Flakiness Risk Re-enabling the suite with .serial can increase runtime and hides concurrency issues; ensure any timing-dependent steps or external Azure DB dependencies are stable and that proper waits/retries/logging are in place to diagnose intermittent failures. .serial("Verify TLS configuration with Azure Database for PostgreSQL health check", () => { const namespace = process.env.NAME_SPACE_RUNTIME || "showcase-runtime"; const job: string = process.env.JOB_NAME || ""; let deploymentName = process.env.RELEASE_NAME + "-developer-hub"; if (job.includes("operator")) { deploymentName = "backstage-" + process.env.RELEASE_NAME; Env Var Assumptions The test depends on RELEASE_NAME, JOB_NAME, and NAME_SPACE_RUNTIME; consider validating or defaulting these to avoid accidental false negatives when env vars are missing or malformed. const namespace = process.env.NAME_SPACE_RUNTIME || "showcase-runtime"; const job: string = process.env.JOB_NAME || ""; let deploymentName = process.env.RELEASE_NAME + "-developer-hub"; if (job.includes("operator")) { deploymentName = "backstage-" + process.env.RELEASE_NAME;

📄 References No matching references available

[rhdh-qodo-merge]

You are nearing your monthly Qodo Merge usage quota. For more information, please visit here.

PR Type

Tests

---

Description

• Re-enable previously skipped e2e test for Azure Database TLS configuration

• Change test execution mode from skip to serial for proper test sequencing

---

File Walkthrough

Relevant files

Tests

verify-tls-config-with-external-azure-db.spec.ts Re-enable test with serial execution mode                                e2e-tests/playwright/e2e/external-database/verify-tls-config-with-external-azure-db.spec.ts Changed test.describe.skip() to test.describe.serial() to re-enable the test Allows test to run sequentially instead of being skipped Maintains existing test structure and configuration +1/-1

---

[rhdh-qodo-merge]

You are nearing your monthly Qodo Merge usage quota. For more information, please visit here.

PR Code Suggestions ✨

No code suggestions found for the PR.

[github-actions]

The image is available at:

• quay.io/rhdh-community/rhdh:pr-3856
• quay.io/rhdh-community/rhdh:pr-3856-0e158b94

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

The image is available at:

• quay.io/rhdh-community/rhdh:pr-3856
• quay.io/rhdh-community/rhdh:pr-3856-b41046fe

[christoph-jerolimov]

/test e2e-ocp-helm

[rhdh-qodo-merge]

You are above your monthly Qodo Merge usage quota. If you are a paying user, please link your GitHub/GitLab/Bitbucket account with your qodo account here to claim your seat. To allow usage organization-wide without linking, please reach to Qodo.

[christoph-jerolimov]

/test e2e-ocp-helm

[rhdh-qodo-merge]

You are above your monthly Qodo Merge usage quota. If you are a paying user, please link your GitHub/GitLab/Bitbucket account with your qodo account here to claim your seat. To allow usage organization-wide without linking, please reach to Qodo.

[openshift-ci]

@christoph-jerolimov: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-ocp-helm	b41046f	link	true	/test e2e-ocp-helm

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[zdrapela]

Incorporated in #3830
/close

[openshift-ci]

@zdrapela: Closed this PR.

Details

In response to this:

Incorporated in #3830
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.