Skip to content

Commit 0c805de

Browse files
adrientheboroboquat
authored andcommitted
[installer-tests] fixup werft major phase naming
1 parent ae6dfaa commit 0c805de

File tree

1 file changed

+13
-4
lines changed

1 file changed

+13
-4
lines changed

.werft/installer-tests.ts

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -276,11 +276,20 @@ installerTests(TEST_CONFIGURATIONS[testConfig]).catch((err) => {
276276

277277
export async function installerTests(config: TestConfig) {
278278
console.log(config.DESCRIPTION);
279+
279280
// these phases sets up or clean up the infrastructure
280-
// If the cloud variable is not set, we have a cleanup job in hand
281-
const majorPhase: string = cloud == "" ? "cleanup-infra" : `create-${cloud}-infra`;
281+
let majorPhase: string;
282+
let phaseMessage: string;
283+
if (cloud == "") {
284+
// If the cloud variable is not set, we have a cleanup job in hand
285+
majorPhase = "cleanup-infra";
286+
phaseMessage = "Cleanup preview and test environment infrastructure";
287+
} else {
288+
majorPhase = `create-${cloud}-infra`;
289+
phaseMessage = `${op} Gitpod infrastructure on ${cloud}`;
290+
}
282291

283-
werft.phase(majorPhase, `Manage the infrastructure in ${cloud}`);
292+
werft.phase(majorPhase, phaseMessage);
284293
for (let phase of config.PHASES) {
285294
const phaseSteps = INFRA_PHASES[phase];
286295
const ret = callMakeTargets(phaseSteps.phase, phaseSteps.description, phaseSteps.makeTarget);
@@ -292,7 +301,7 @@ export async function installerTests(config: TestConfig) {
292301

293302
await sendFailureSlackAlert(phaseSteps.description, err, slackHook.get("self-hosted-jobs"));
294303

295-
werft.fail(`create-${cloud}-infra`, err.message);
304+
werft.fail(majorPhase, err.message);
296305

297306
return;
298307
}

0 commit comments

Comments
 (0)