Skip to content

Commit ebbc0cd

Browse files
authored
Merge pull request #4319 from lsst-sqre/u/rra/production-tools
Disable production-tools on idfint
2 parents da6883f + e3ce25f commit ebbc0cd

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

environments/values-idfint.yaml

-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ applications:
2626
sasquatch: true
2727
sia: true
2828
ssotap: true
29-
production-tools: true
3029
sasquatch-backpack: true
3130
semaphore: true
3231
sqlproxy-cross-project: true

tests/config_test.py

+7-2
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,10 @@
1212
from phalanx.factory import Factory
1313
from phalanx.models.applications import Project
1414

15-
_ALLOW_NO_SECRETS = ("next-visit-fan-out",)
15+
_ALLOW_DISABLED = {"production-tools"}
16+
"""Temporary whitelist of applications not enabled anywhere."""
17+
18+
_ALLOW_NO_SECRETS = {"next-visit-fan-out"}
1619
"""Temporary whitelist of applications that haven't added secrets.yaml."""
1720

1821

@@ -168,7 +171,9 @@ def test_applications_enabled() -> None:
168171
for application in config.applications:
169172
name = application.name
170173
active = set(application.active_environments)
171-
assert active, f"Application {name} is not enabled in any environment"
174+
if not active and name not in _ALLOW_DISABLED:
175+
msg = f"Application {name} is not enabled in any environment"
176+
assert active, msg
172177
configured = set(config_storage.get_application_environments(name))
173178
for env in active:
174179
msg = f"Application {name} enabled for {env} but not configured"

0 commit comments

Comments
 (0)