Skip to content

Commit bad5569

Browse files
authored
Don't remove the runner, we are replacing it.
1 parent 40f9a7c commit bad5569

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

github-runner-installer

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -283,8 +283,13 @@ configure() {
283283
# Remove
284284
# @TODO: If runner has a job, wait for it to complete.
285285
cd $RUNNER_FOLDER
286-
echo ./config.sh remove --token ${RUNNER_CONFIG_TOKEN}
287-
./config.sh remove --token ${RUNNER_CONFIG_TOKEN} || say "config remove didn't work."
286+
287+
# @TODO: determine if config remove is needed. Seems like --replace should work?
288+
# echo ./config.sh remove --token ${RUNNER_CONFIG_TOKEN}
289+
# config.sh remove can fail because:
290+
# 1. the service is still installed.
291+
# 2. TBD.
292+
# ./config.sh remove --token ${RUNNER_CONFIG_TOKEN} || say "config remove didn't work."
288293

289294
rm -rf .credentials .runner
290295
line

0 commit comments

Comments
 (0)