Merge pull request kubernetes#5776 from jonlangemak/fluentd-elasticse…

…arch-kibanafix Added API variables so Kibana could access ES through the API server
duda · Apr 3, 2015 · e4c91df · e4c91df
2 parents ab13d64 + 35038dc
commit e4c91df
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/cluster/addons/fluentd-elasticsearch/kibana-controller.yaml b/cluster/addons/fluentd-elasticsearch/kibana-controller.yaml
@@ -13,6 +13,11 @@ desiredState:
         containers:
           - name: kibana-logging
             image: gcr.io/google_containers/kibana:1.1
+            env: 
+              - name: "ES_SCHEME"
+                value: "https"
+              - name: "ES_HOST"
+                value: "\"+window.location.hostname+\"/api/v1beta1/proxy/services/elasticsearch-logging"
             ports:
               - name: kibana-port
                 containerPort: 80

diff --git a/cluster/addons/fluentd-elasticsearch/kibana-image/run_kibana_nginx.sh b/cluster/addons/fluentd-elasticsearch/kibana-image/run_kibana_nginx.sh
@@ -43,6 +43,11 @@ set -o errexit
 set -o nounset
 set -o pipefail
 
+#Report all environment variables containing 'elasticsearch' and ES related
+set | grep -i elasticsearch
+set | grep -i ES_SCHEME
+set | grep -i ES_HOST
+
 cat << EOF > /usr/share/nginx/html/config.js
 /** @scratch /configuration/config.js/1
  *
@@ -75,11 +80,7 @@ function (Settings) {
      *  +elasticsearch: {server: "http://localhost:9200", withCredentials: true}+
      *
      */
-
-
-    elasticsearch: "https://"+window.location.hostname+"/api/v1beta1/proxy/services/elasticsearch-logging",
-
-
+    elasticsearch: "${ES_SCHEME}://${ES_HOST}",
     /** @scratch /configuration/config.js/5
      *
      * ==== default_route