Skip to content

Commit bbc5d12

Browse files
committed
further cleanup
1 parent 405fa86 commit bbc5d12

File tree

7 files changed

+14
-14
lines changed
  • services
    • clusters-keeper/src/simcore_service_clusters_keeper/core
    • director-v2/src/simcore_service_director_v2/modules/dynamic_sidecar/api_client
    • dynamic-sidecar/src/simcore_service_dynamic_sidecar/core
    • efs-guardian/src/simcore_service_efs_guardian/core
    • invitations/src/simcore_service_invitations/core
    • payments/src/simcore_service_payments/core
    • resource-usage-tracker/src/simcore_service_resource_usage_tracker/core

7 files changed

+14
-14
lines changed

services/clusters-keeper/src/simcore_service_clusters_keeper/core/application.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ def create_app(settings: ApplicationSettings) -> FastAPI:
6464
setup_redis(app)
6565
setup_clusters_management(app)
6666

67-
if app.state.settings.CLUSTERS_KEEPER_TRACING:
67+
if settings.CLUSTERS_KEEPER_TRACING:
6868
initialize_fastapi_app_tracing(
6969
app, tracing_data=get_tracing_data(app, settings.CLUSTERS_KEEPER_TRACING)
7070
)

services/director-v2/src/simcore_service_director_v2/modules/dynamic_sidecar/api_client/_thin.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
retry_on_errors,
1414
)
1515
from servicelib.fastapi.tracing import get_tracing_data
16-
from servicelib.tracing import TracingData
1716
from settings_library.tracing import TracingSettings
1817

1918
from ....core.dynamic_services_settings.scheduler import (
@@ -37,9 +36,7 @@ def __init__(self, app: FastAPI):
3736
tracing_settings: TracingSettings | None = (
3837
app.state.settings.DIRECTOR_V2_TRACING
3938
)
40-
tracing_data: TracingData | None = None
41-
if tracing_settings:
42-
tracing_data = get_tracing_data(app, settings.DIRECTOR_V2_TRACING)
39+
tracing_data = get_tracing_data(app, tracing_settings)
4340

4441
# timeouts
4542
self._health_request_timeout = Timeout(1.0, connect=1.0)

services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/application.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,10 @@ def create_app() -> FastAPI:
197197

198198
if application_settings.DYNAMIC_SIDECAR_TRACING:
199199
initialize_fastapi_app_tracing(
200-
app, tracing_data=get_tracing_data(app, settings.DYNAMIC_SIDECAR_TRACING)
200+
app,
201+
tracing_data=get_tracing_data(
202+
app, application_settings.DYNAMIC_SIDECAR_TRACING
203+
),
201204
)
202205

203206
# ERROR HANDLERS ------------

services/efs-guardian/src/simcore_service_efs_guardian/core/application.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,9 @@ def create_app(settings: ApplicationSettings | None = None) -> FastAPI:
6161

6262
setup_fire_and_forget(app)
6363

64-
if app.state.settings.EFS_GUARDIAN_TRACING:
64+
if app_settings.EFS_GUARDIAN_TRACING:
6565
initialize_fastapi_app_tracing(
66-
app, tracing_data=get_tracing_data(app, settings.EFS_GUARDIAN_TRACING)
66+
app, tracing_data=get_tracing_data(app, app_settings.EFS_GUARDIAN_TRACING)
6767
)
6868

6969
# EVENTS

services/invitations/src/simcore_service_invitations/core/application.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@ def create_app(settings: ApplicationSettings | None = None) -> FastAPI:
4848
if app.state.settings.INVITATIONS_PROMETHEUS_INSTRUMENTATION_ENABLED:
4949
setup_prometheus_instrumentation(app)
5050

51-
if app.state.settings.INVITATIONS_TRACING:
51+
if tracing_settings := app.state.settings.INVITATIONS_TRACING:
5252
initialize_fastapi_app_tracing(
53-
app, tracing_data=get_tracing_data(app, settings.INVITATIONS_TRACING)
53+
app, tracing_data=get_tracing_data(app, tracing_settings)
5454
)
5555

5656
# ERROR HANDLERS

services/payments/src/simcore_service_payments/core/application.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,9 @@ def create_app(settings: ApplicationSettings | None = None) -> FastAPI:
7878
if app.state.settings.PAYMENTS_PROMETHEUS_INSTRUMENTATION_ENABLED:
7979
setup_prometheus_instrumentation(app)
8080

81-
if app.state.settings.PAYMENTS_TRACING:
81+
if app_settings.PAYMENTS_TRACING:
8282
initialize_fastapi_app_tracing(
83-
app, tracing_data=get_tracing_data(app, settings.PAYMENTS_TRACING)
83+
app, tracing_data=get_tracing_data(app, app_settings.PAYMENTS_TRACING)
8484
)
8585

8686
# ERROR HANDLERS

services/resource-usage-tracker/src/simcore_service_resource_usage_tracker/core/application.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,10 +74,10 @@ def create_app(settings: ApplicationSettings) -> FastAPI:
7474

7575
setup_process_message_running_service(app) # Requires Rabbit
7676

77-
if app.state.settings.RESOURCE_USAGE_TRACKER_TRACING:
77+
if tracing_settings := app.state.settings.RESOURCE_USAGE_TRACKER_TRACING:
7878
initialize_fastapi_app_tracing(
7979
app,
80-
tracing_data=get_tracing_data(app, settings.RESOURCE_USAGE_TRACKER_TRACING),
80+
tracing_data=get_tracing_data(app, tracing_settings),
8181
)
8282

8383
# ERROR HANDLERS

0 commit comments

Comments
 (0)