I'm abandoning the docker image idea. I realized the varnishstat comm…

…and ofc is not available from the host system. Add docker notes to the readme. Closes #10
jonnenauha · Nov 26, 2016 · 0f0e3e2 · 0f0e3e2
1 parent c911f06
commit 0f0e3e2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -8,12 +8,9 @@ language: go
 go:
   - 1.7.3
 
-env:
-  global:
-    - DOCKER_REPO=jonnenauha/prometheus_varnish_exporter
-
 before_script:
   - sudo apt-get install varnish -y
+  - varnishstat -V
 
 script:
   - go build

diff --git a/README.md b/README.md
@@ -48,3 +48,9 @@ I'd be interested in Grafana dahsboard .json exports or Prometheus queries you m
 # Build
 
 Use `go build` or `./build.sh` for cross compilation.
+
+# Docker notes
+
+Building a docker image would not be useful for this application. It operates by executing the `varnishstat` tool. This executable is of course not available inside the exporters container. We could install and run Varnish inside the same container, but this would make little sense and be harder to manage.
+
+I do not know of a way to get the varnish stats over the network (--net="host" could be used to talk to it). If you have more info on this, please open an issue.