Disable API health check of downstream services

[tamirkamara]

Fixes #2048

What is being addressed

We want to disable the logic where the API pings downstream services to test an assumption that those cause timeouts in the system.
Preliminary tests showed that they sometimes those take a long time and some are probably blocking so making the server unresponsive to other requests.

How is this addressed

• Disable downstream services health check

[github-actions]

Unit Test Results

282 tests  +260   280 ✔️ +258   9s ⏱️ -5s
    1 suites ±    0       2 💤 +    2 
    1 files   ±    0       0 ❌ ±    0 

Results for commit 8afa81d. ± Comparison against base commit fa9e71a.

This pull request removes 22 and adds 282 tests. Note that renamed tests count towards both.

test_provisioned_health_api ‑ test_health
test_shared_service_templates ‑ test_get_shared_service_template[tre-shared-service-firewall]
test_shared_service_templates ‑ test_get_shared_service_template[tre-shared-service-gitea]
test_shared_service_templates ‑ test_get_shared_service_template[tre-shared-service-nexus]
test_shared_service_templates ‑ test_get_shared_service_templates[tre-shared-service-firewall]
test_shared_service_templates ‑ test_get_shared_service_templates[tre-shared-service-gitea]
test_shared_service_templates ‑ test_get_shared_service_templates[tre-shared-service-nexus]
test_workspace_service_templates ‑ test_create_workspace_service_templates
test_workspace_service_templates ‑ test_get_workspace_service_template[tre-service-azureml]
test_workspace_service_templates ‑ test_get_workspace_service_template[tre-service-devtestlabs]
…

tests_ma.test_api.test_errors.test_422_error ‑ test_frw_validation_error_format
tests_ma.test_api.test_errors.test_error ‑ test_frw_validation_error_format
tests_ma.test_api.test_routes.test_api_access.TestTemplateRoutesThatRequireAdminRights ‑ test_get_workspace_templates_requires_admin_rights
tests_ma.test_api.test_routes.test_api_access.TestTemplateRoutesThatRequireAdminRights ‑ test_post_user_resource_templates_requires_admin_rights
tests_ma.test_api.test_routes.test_api_access.TestTemplateRoutesThatRequireAdminRights ‑ test_post_workspace_service_templates_requires_admin_rights
tests_ma.test_api.test_routes.test_api_access.TestTemplateRoutesThatRequireAdminRights ‑ test_post_workspace_templates_requires_admin_rights
tests_ma.test_api.test_routes.test_api_access.TestUserResourcesOwnerOrResearcherRoutesAccess ‑ test_delete_user_resource_raises_403_if_user_is_not_workspace_owner_or_researcher
tests_ma.test_api.test_routes.test_api_access.TestUserResourcesOwnerOrResearcherRoutesAccess ‑ test_get_user_resource_raises_403_if_user_is_not_workspace_owner_or_researcher
tests_ma.test_api.test_routes.test_api_access.TestUserResourcesOwnerOrResearcherRoutesAccess ‑ test_get_user_resources_raises_403_if_user_is_not_researcher_or_owner_of_workspace
tests_ma.test_api.test_routes.test_api_access.TestUserResourcesOwnerOrResearcherRoutesAccess ‑ test_patch_user_resource_raises_403_if_user_is_not_workspace_owner_or_researcher
…

♻️ This comment has been updated with latest results.

[tamirkamara]

/test

[github-actions]

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/2509177006 (with refid b0ec2f70)

(in response to this comment from @tamirkamara)

[tamirkamara]

/test-force-approve
All tests passed (pr run and branch run) but for some reason the check wasn't marked so had to force it.

[github-actions]

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit b139341)

(in response to this comment from @tamirkamara)

[tamirkamara]

/test-force-approve

[github-actions]

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit b139341)

(in response to this comment from @tamirkamara)

[tamirkamara]

/test-force-approve

[github-actions]

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit 8afa81d)

(in response to this comment from @tamirkamara)