Merge "Update metadata.yml with correct URL"
diff --git a/_modules/jenkins_common.py b/_modules/jenkins_common.py
index c7446fa..348aa09 100644
--- a/_modules/jenkins_common.py
+++ b/_modules/jenkins_common.py
@@ -69,7 +69,7 @@
 
     logger.debug("Calling Jenkins script API with URL: %s", jenkins_url)
     req = requests.post('%s/scriptText' % jenkins_url,
-                        auth=(jenkins_user, jenkins_password),
+                        auth=(jenkins_user, jenkins_password) if jenkins_user else None,
                         data=req_data)
     ret["code"] = req.status_code
     ret["msg"] = req.text
diff --git a/_states/jenkins_theme.py b/_states/jenkins_theme.py
index c27b0a2..2a91c10 100644
--- a/_states/jenkins_theme.py
+++ b/_states/jenkins_theme.py
@@ -10,6 +10,7 @@
             if(!pd.cssUrl.equals("${css_url}") || !pd.jsUrl.equals("${js_url}")){
                 pd.cssUrl = "${css_url}"
                 pd.jsUrl = "${js_url}"
+                pd.save()
                 state="SUCCESS"
             }else{
                 state="EXISTS"
diff --git a/jenkins/client/_job_template.sls b/jenkins/client/_job_template.sls
index de4ba21..adce826 100644
--- a/jenkins/client/_job_template.sls
+++ b/jenkins/client/_job_template.sls
@@ -2,7 +2,7 @@
   {#- Matrix way, simulating behavior of Jenkins job builder, not fully
       supported at the moment #}
   {%- for param_name, params in job_template.get('param', {}).iteritems() %}
-    {%- set replacer = "{{" + param_name + "}}" %}
+    {%- set replacer = client.replacer.open + param_name + client.replacer.close %}
     {%- for param in params %}
       {%- set job_name = job_template.name|replace(replacer, param) %}
       {%- set job = job_template.template|yaml|replace(replacer, param)|load_yaml %}
@@ -19,7 +19,7 @@
       {#- You may think WTF hack is this but we can't update variables in
           inner scope to replace all parameters. But we can abuse lists for
           this purpose }:-) #}
-      {%- set replacer = "{{" + key + "}}" %}
+      {%- set replacer = client.replacer.open + key + client.replacer.close %}
       {%- do _job_name.append(_job_name|last|replace(replacer, value)) %}
       {%- do _job.append(_job|last|yaml|replace(replacer, value)|load_yaml) %}
     {%- endfor %}
diff --git a/jenkins/client/job.sls b/jenkins/client/job.sls
index 9375ad0..f9b1541 100644
--- a/jenkins/client/job.sls
+++ b/jenkins/client/job.sls
@@ -14,7 +14,7 @@
   {%- for job_template_name, job_template in client.get('job_template', {}).iteritems() %}
     {%- if job_template.get('enabled', true) %}
       {%- for param_name, params in job_template.get('param', {}).iteritems() %}
-        {%- set replacer = "{{" + param_name + "}}" %}
+        {%- set replacer = client.replacer.open + param_name + client.replacer.close %}
         {%- for param in params %}
           {%- set job_name = job_template.name|replace(replacer, param) %}
           {%- do jobs.append(job_name) %}
@@ -24,7 +24,7 @@
       {%- for job_params in job_template.get('jobs', []) %}
         {%- set job_name = job_template.name %}
         {%- for key, value in job_params.iteritems() %}
-          {%- set replacer = "{{" + key + "}}" %}
+          {%- set replacer = client.replacer.open + key + client.replacer.close %}
           {%- set job_name = job_name|replace(replacer, value) %}
           {%- do jobs.append(job_name) %}
         {%- endfor %}
diff --git a/jenkins/files/slave/default b/jenkins/files/slave/default
index cbd0065..7673d5d 100644
--- a/jenkins/files/slave/default
+++ b/jenkins/files/slave/default
@@ -53,4 +53,4 @@
 MAXOPENFILES=8192
 
 # Arguments to pass to jenkins slave on startup
-JENKINS_ARGS="-jnlpUrl ${JENKINS_URL}/computer/$JENKINS_HOSTNAME/slave-agent.jnlp -jnlpCredentials ${JENKINS_LOGIN}:${JENKINS_PASSWORD}"
+JENKINS_ARGS="-jnlpUrl ${JENKINS_URL}/computer/$JENKINS_HOSTNAME/slave-agent.jnlp -jnlpCredentials ${JENKINS_LOGIN}:${JENKINS_PASSWORD} -slaveLog /var/log/jenkins/java-${NAME}.log"
diff --git a/jenkins/map.jinja b/jenkins/map.jinja
index 4ded07f..1530282 100644
--- a/jenkins/map.jinja
+++ b/jenkins/map.jinja
@@ -76,6 +76,10 @@
         'dir': {
             'jenkins_source_root': '/var/cache/salt/minion/jenkins/source',
             'jenkins_jobs_root': '/var/cache/salt/minion/jenkins/jobs',
+        },
+        'replacer': {
+            'open': '{{',
+            'close': '}}',
         }
     },
 }, merge=pillar.jenkins.get('client', {})) %}