fix: use tini as PID 1 from the start

This seems better practice and ensures safer shutdown if the container is killed while entrypoint scripts are running

Author: chadlwilson

buildSrc/src/main/resources/gocd-docker-agent/Dockerfile.agent.ftl

@@ -101,6 +101,6 @@ VOLUME /godata
   COPY --chown=root:root run-docker-daemon.sh /
 </#if>
 
-ENTRYPOINT ["/docker-entrypoint.sh"]
+ENTRYPOINT ["tini-static", "-g", "--", "/docker-entrypoint.sh"]
 
 USER go

buildSrc/src/main/resources/gocd-docker-agent/docker-entrypoint.sh

@@ -170,4 +170,4 @@ if [ "$1" = "${AGENT_WORK_DIR}/bin/go-agent" ]; then
   echo "set.AGENT_STARTUP_ARGS=%AGENT_STARTUP_ARGS_INTERNAL%" >> /go-agent/wrapper-config/wrapper-properties.conf
 fi
 
-try exec tini-static -g -- "$@"
+try exec "$@"

buildSrc/src/main/resources/gocd-docker-server/Dockerfile.server.ftl

@@ -92,6 +92,6 @@ RUN chown -R go:root /docker-entrypoint.d /go-working-dir /godata /docker-entryp
 VOLUME /go-working-dir
 VOLUME /godata
 
-ENTRYPOINT ["/docker-entrypoint.sh"]
+ENTRYPOINT ["tini-static", "-g", "--", "/docker-entrypoint.sh"]
 
 USER go

buildSrc/src/main/resources/gocd-docker-server/docker-entrypoint.sh

@@ -102,4 +102,4 @@ if [ "$1" = "${SERVER_WORK_DIR}/bin/go-server" ]; then
   done
 fi
 
-try exec tini-static -g -- "$@"
+try exec "$@"