[Qa-jenkins-scm] [Git][qa/jenkins.debian.net][master] djm: properly initialise global variables (fixing a bug leading to time skew on future nodes)

[Holger Levsen (@holger)]

Holger Levsen pushed to branch master at Debian QA / jenkins.debian.net


Commits:
524453df by Holger Levsen at 2023-06-13T16:26:00+02:00
djm: properly initialise global variables (fixing a bug leading to time skew on future nodes)

Signed-off-by: Holger Levsen <holger at layer-acht.org>

- - - - -


1 changed file:

- bin/djm


Changes:

=====================================
bin/djm
=====================================
@@ -27,6 +27,9 @@ TARGET=
 ACTION=
 REASON=
 COMMAND=
+SHORTNODE=
+NODE=
+NTP_SERVER=
 LOCAL_LOGFILE=~/.djm.log
 UI_LOGFILE=~/.djm-jenkins-ui.log
 PARSER_CACHE=~/.djm-jenkins-parser.cache
@@ -545,7 +548,7 @@ djm_do() {
 		#
 		case $ACTION in
 			reboot)	( ssh $NODE "sudo reboot || ( echo press enter ; read a ) " || true ) & sleep 2
-				run_xterm2wait4node_comeback $NODE
+				run_xterm2wait4node_comeback
 				;;
 			powercycle)	case $SHORTNODE in
 						jenkins)	`dirname $0`/powercycle_x86_nodes.py jenkins
@@ -569,7 +572,7 @@ djm_do() {
 								exit 1
 								;;
 					esac
-					run_xterm2wait4node_comeback $NODE
+					run_xterm2wait4node_comeback
 					;;
 			shell)	xterm -T "$SHORTNODE / $ACTION" -class deploy-jenkins -bg $BG -fa 'DejaVuSansMono' -fs 10 -e "ssh $NODE" &
 				;;



View it on GitLab: https://salsa.debian.org/qa/jenkins.debian.net/-/commit/524453df9d9d4867f64e1eb34aa87532b1506d42

-- 
View it on GitLab: https://salsa.debian.org/qa/jenkins.debian.net/-/commit/524453df9d9d4867f64e1eb34aa87532b1506d42
You're receiving this email because of your account on salsa.debian.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/qa-jenkins-scm/attachments/20230613/24824c5a/attachment-0001.htm>