Skip to content

Commit 37f6d78

Browse files
committed
ci: Test if we're actually only cleaning up the expected Brancher instance
1 parent 7400356 commit 37f6d78

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

ci/test/magento/deploy_brancher.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
]);
1919

2020
$productionStage = $configuration->addStage('test', 'banaan.store');
21-
$productionStage->addBrancherServer('hndeployintegr8');
21+
$productionStage->addBrancherServer('hndeployintegr8')
22+
->setLabels(['gitref='.\getenv('GITHUB_SHA') ?: 'unknown']);
2223

2324
return $configuration;

ci/test/run-brancher.sh

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,11 @@ $DP jq .hostnames[0] deployment-report.json -r -e
7272
$DP jq .brancher_hypernodes[0] deployment-report.json -r -e
7373

7474
# Remove deployment report to make sure we can clean up using hnapi and labels matching
75+
BRANCHER_INSTANCE=$($DP jq .brancher_hypernodes[0] deployment-report.json -r -e)
7576
$DP rm -f deployment-report.json
7677

7778
# cleanup data
78-
$DP hypernode-deploy cleanup test -vvv
79+
$DP hypernode-deploy cleanup test -vvv | tee cleanup.log
80+
81+
# Run tests on cleanup
82+
grep "Stopping brancher Hypernode ${BRANCHER_INSTANCE}..." cleanup.log

0 commit comments

Comments
 (0)