Skip to content

Commit 8b862af

Browse files
committed
workbench: remove cardano-world "perf" usage/references
Also makes setenv-defaults usable from other parts of the workbench
1 parent a7c4d03 commit 8b862af

File tree

7 files changed

+246
-164
lines changed

7 files changed

+246
-164
lines changed

Makefile

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -91,12 +91,14 @@ PROFILES_FORGE_STRESS_RTS := forge-stress-pre-rtsA4m forge-stress-pre-rtsA64m fo
9191
PROFILES_CHAINSYNC := chainsync-early-byron chainsync-early-byron-notracer chainsync-early-byron-oldtracing
9292
PROFILES_CHAINSYNC += chainsync-early-alonzo chainsync-early-alonzo-notracer chainsync-early-alonzo-oldtracing chainsync-early-alonzo-p2p
9393
PROFILES_VENDOR := dish dish-plutus dish-10M dish-10M-plutus
94-
# "qa" and "perf" namespaces for cardano world (world.dev.cardano.org) Nomad
95-
# Not all local profiles are compatible (yet) with a cloud run
94+
# Cardano World (world.dev.cardano.org) Nomad cluster's "qa" class nodes
9695
# Cloud version of "default", "ci-test" and "ci-bench"
97-
PROFILES_CW_QA := default-cw-qa ci-test-cw-qa ci-bench-cw-qa
98-
# The 52+explorer profile
99-
PROFILES_CW_PERF := default-cw-perf ci-test-cw-perf ci-bench-cw-perf cw-perf-value
96+
# Not all local profiles are compatible or tested (yet) with a cloud runs
97+
PROFILES_NOMAD_CW_QA := default-nomadcwqa ci-test-nomadcwqa ci-bench-nomadcwqa ci-test-nomadcwqa-mimicops
98+
# The dedicated P&T Nomad cluster on AWS
99+
# Cloud version of "default", "ci-test" and "ci-bench" plus value (52+explorer)
100+
# Not all local profiles are compatible or tested (yet) with a cloud runs
101+
PROFILES_NOMAD_PERF := default-nomadperf ci-test-nomadperf ci-bench-nomadperf value-nomadperf
100102

101103
LOCAL_PROFILES += $(PROFILES_BASE)
102104
LOCAL_PROFILES += $(PROFILES_FAST)
@@ -112,7 +114,7 @@ LOCAL_PROFILES += $(PROFILES_FORGE_STRESS_PRE)
112114
LOCAL_PROFILES += $(PROFILES_FORGE_STRESS_RTS)
113115
LOCAL_PROFILES += $(PROFILES_CHAINSYNC)
114116
LOCAL_PROFILES += $(PROFILES_VENDOR)
115-
CLOUD_PROFILES += $(PROFILES_CW_QA) $(PROFILES_CW_PERF)
117+
CLOUD_PROFILES += $(PROFILES_NOMAD_CW_QA) $(PROFILES_NOMAD_PERF)
116118

117119
## Note: to enable a shell for a profile, just add its name (one of names from 'make ps') to SHELL_PROFILES
118120

nix/workbench/backend/nomad-job.nix

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -273,12 +273,12 @@ let
273273
constraint = {
274274
attribute = "\${node.class}";
275275
operator = "=";
276-
# For cloud benchmarking, dedicated static machines in the "perf"
277-
# class are used. We mimic that for local/test runs.
276+
# Cloud jobs can run in the dedicated P&T Nomad cluster on AWS or in
277+
# Cardano World Nomad cluster's "qa" class nodes.
278278
# This default is just a precaution, like the top level namespace,
279-
# because there are also available "qa" Class nodes but usage of these
280-
# must be limited to short test and "infra" Class nodes are used for
281-
# HA jobs and must be avoided entirely.
279+
# because "qa" Class nodes usage must be limited to short test and
280+
# "infra" Class nodes, that are used for HA jobs, must be avoided
281+
# entirely.
282282
value = "perf";
283283
};
284284

@@ -412,9 +412,9 @@ let
412412
};
413413

414414
# Sensible defaults to run cloud version of "default", "ci-test" and
415-
# "ci-bench" in cardano-world qa class Nomad nodes.
416-
# For benchmarking dedicated static machines in the "perf" class are
417-
# used and this value should be updated accordingly.
415+
# "ci-bench" in Cardano World Nomad cluster's "qa" class nodes.
416+
# For benchmarking the dedicated P&T Nomad cluster on AWS is used and
417+
# this value should be updated accordingly.
418418
resources = {
419419
# Task can only ask for 'cpu' or 'cores' resource but not both.
420420
cores = 2; # cpu = 512;
@@ -450,10 +450,10 @@ let
450450
# address of an AWS EC2 instance set this to
451451
# ${attr.unique.platform.aws.public-ipv4}.
452452
address =
453-
# When using Cardano World (nomad.world.dev.cardano.org) "perf"
454-
# class nodes we use public IPs/routing, all the other cloud runs
455-
# are behind a VPC/firewall. Local runs just use 12.0.0.1.
456-
if lib.strings.hasInfix "cw-perf" profileData.profileName
453+
# When using the dedicated P&T Nomad cluster on AWS we use public
454+
# IPs/routing, all the other cloud runs are behind a VPC/firewall.
455+
# Local runs just use 12.0.0.1.
456+
if lib.strings.hasInfix "-nomadperf" profileData.profileName
457457
then "\${attr.unique.platform.aws.public-ipv4}"
458458
else ""
459459
;
@@ -1180,17 +1180,17 @@ let
11801180
# Port string from
11811181
''--port ${toString nodeSpec.port}''
11821182
]
1183-
# On cloud deployments to SRE-managed Nomad, that uses AWS, the hosts at
1184-
# Linux level may not be aware of the EIP public address they have so we
1185-
# can't bind to the public IP (that we can resolve to using templates).
1186-
# The only options available are to bind to the "all-weather" 0.0.0.0 or
1187-
# use the private IP provided by AWS. We use the latter in case the Nomad
1188-
# Client was not started with the correct `-network-interface XX`
1189-
# parameter.
1183+
# On cloud deployments to SRE-managed / dedicated P&T Nomad cluster, that
1184+
# uses AWS, the hosts at Linux level may not be aware of the EIP public
1185+
# address they have so we can't bind to the public IP (that we can resolve
1186+
# to using templates). The only options available are to bind to the
1187+
# "all-weather" 0.0.0.0 or use the private IP provided by AWS. We use the
1188+
# latter in case the Nomad Client was not started with the correct
1189+
# `-network-interface XX` parameter.
11901190
[
11911191
# Address string to
11921192
(
1193-
if lib.strings.hasInfix "cw-perf" profileData.profileName
1193+
if lib.strings.hasInfix "-nomadperf" profileData.profileName
11941194
then ''--host-addr {{ env "attr.unique.platform.aws.local-ipv4" }}''
11951195
else ''--host-addr 0.0.0.0''
11961196
)

0 commit comments

Comments
 (0)