bind-charmers team mailing list archive

[Barry Price <barry.price@xxxxxxxxxxxxx>]

Barry Price has proposed merging ~barryprice/charm-k8s-bind/+git/charm-k8s-bind:wrapper into charm-k8s-bind:master.

Commit message:
Convert from using both ENTRYPOINT and CMD to a single wrapper script that sets up our config and then starts the bind service

Requested reviews:
  Canonical IS Reviewers (canonical-is-reviewers)
  Barry Price (barryprice)

For more details, see:
https://code.launchpad.net/~barryprice/charm-k8s-bind/+git/charm-k8s-bind/+merge/402966
-- 
Your team Bind Charmers is subscribed to branch charm-k8s-bind:master.

diff --git a/Makefile b/Makefile
index 35b100d..564aeb9 100644
--- a/Makefile
+++ b/Makefile
@@ -30,7 +30,7 @@ image-deps:
 
 image-lint: image-deps
 	@echo "Running shellcheck."
-	@shellcheck image-scripts/docker-entrypoint.sh
+	@shellcheck image-scripts/docker-wrapper.sh
 	@shellcheck image-scripts/dns-check.sh
 
 image-build: image-lint
diff --git a/dockerfile b/dockerfile
index 968d4fd..4215462 100644
--- a/dockerfile
+++ b/dockerfile
@@ -29,5 +29,4 @@ RUN chmod 0755 /usr/local/bin/dns-check.sh
 EXPOSE 53/udp
 EXPOSE 53/tcp
 
-ENTRYPOINT ["/usr/local/bin/docker-entrypoint.sh"]
-CMD /usr/sbin/named -g -u bind -c /etc/bind/named.conf
+CMD ["/usr/local/bin/docker-wrapper.sh"]
diff --git a/image-scripts/docker-entrypoint.sh b/image-scripts/docker-wrapper.sh
similarity index 97%
rename from image-scripts/docker-entrypoint.sh
rename to image-scripts/docker-wrapper.sh
index 2ed0857..4f807e1 100644
--- a/image-scripts/docker-entrypoint.sh
+++ b/image-scripts/docker-wrapper.sh
@@ -21,3 +21,5 @@ if [ -d "${BIND_CONFDIR}" ]; then
 else
 	echo "Something went wrong, ${BIND_CONFDIR} does not exist, not starting";
 fi
+
+/usr/sbin/named -g -u bind -c /etc/bind/named.conf