From: Ismaël Bouya <ismael.bouya@normalesup.org>
Date: Fri, 9 Mar 2018 18:37:20 +0000 (+0100)
Subject: Change deploy script
X-Git-Url: https://git.immae.eu/?p=perso%2FImmae%2FProjets%2FPuppet.git;a=commitdiff_plain;h=bd263a98d542207c07b27dfb891e62b9e2c73c3d

Change deploy script
---

diff --git a/modules/role/files/cryptoportfolio/slack-notify.py b/modules/role/files/cryptoportfolio/slack-notify.py
index fe64fb6..dfced46 100644
--- a/modules/role/files/cryptoportfolio/slack-notify.py
+++ b/modules/role/files/cryptoportfolio/slack-notify.py
@@ -1,12 +1,18 @@
 #!/usr/bin/env python3
 
+import os
 import sys
 import json
 import urllib3
 
-project = sys.argv[1]
-url     = sys.argv[2]
-version = sys.argv[3]
+project = os.environ["P_PROJECT"]
+url     = os.environ["P_WEBHOOK"]
+version = os.environ["P_VERSION"]
+host    = os.environ["P_HOST"]
+if os.environ["P_HTTPS"] == "true":
+    scheme = "https://"
+else:
+    scheme = "http://"
 
 def post(url, data):
     urllib3.disable_warnings()
@@ -20,7 +26,12 @@ def post(url, data):
 
 data = {
         "icon_url": "https://learn.puppet.com/static/images/logos/Puppet-Logo-Mark-Amber.png",
-        "text": "Deployed {} on {}".format(version, project),
+        "text": "Deployed {} of {} on {}{}".format(
+            version,
+            project,
+            scheme,
+            host,
+            ),
         }
 
 json_data = json.dumps(data)
diff --git a/modules/role/manifests/cryptoportfolio.pp b/modules/role/manifests/cryptoportfolio.pp
index 67d88fb..bec247e 100644
--- a/modules/role/manifests/cryptoportfolio.pp
+++ b/modules/role/manifests/cryptoportfolio.pp
@@ -272,7 +272,14 @@ class role::cryptoportfolio {
     unless empty($cf_webhook_url) {
       exec { "bot-slack-notify":
         refreshonly => true,
-        command     => "/usr/local/bin/slack-notify Trader '${cf_webhook_url}' '${bot_version}'",
+        environment => [
+          "P_PROJECT=Trader",
+          "P_WEBHOOK=${cf_webhook_url}",
+          "P_VERSION=${bot_version}",
+          "P_HOST=${cf_front_app_host}",
+          "P_HTTPS=${cf_front_app_ssl}",
+        ],
+        command     => "/usr/local/bin/slack-notify",
         require     => File["/usr/local/bin/slack-notify"],
         subscribe   => Exec["py-cryptoportfolio-dependencies"],
       }
@@ -407,7 +414,14 @@ class role::cryptoportfolio {
     unless empty($cf_webhook_url) {
       exec { "front-slack-notify":
         refreshonly => true,
-        command     => "/usr/local/bin/slack-notify Front '${cf_webhook_url}' '${front_version}'",
+        environment => [
+          "P_PROJECT=Front",
+          "P_WEBHOOK=${cf_webhook_url}",
+          "P_VERSION=${front_version}",
+          "P_HOST=${cf_front_app_host}",
+          "P_HTTPS=${cf_front_app_ssl}",
+        ],
+        command     => "/usr/local/bin/slack-notify",
         require     => File["/usr/local/bin/slack-notify"],
         subscribe   => [Exec["go-cryptoportfolio-app"], Exec["web-cryptoportfolio-build"]],
       }