chef-handler-status_notifier 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 25b94d5fb43b033b90f22b3172eea91b5c6a355e
-  data.tar.gz: 1ad7aa9146a72435bcf24e7d1ca99386138b8449
+  metadata.gz: f46ac6b803396196b68b6c4189fbfe5a558f610d
+  data.tar.gz: 2ef29efee61530660cfcf296896b690e06e18988
 SHA512:
-  metadata.gz: d6fc054b86f2cd59f33e7403e46ffb7da65dc45c576a787a9b4d55c338249c718b5a50e6261d7dce327fa3b9ba668a2698db5231904f23c7c67f2fa71c926751
-  data.tar.gz: 3d370da722ce30f628fce809b6e75f37d06524322859fce88274b22101720cb8f965a91ed3f066f67b00e51bfc345dfb77ea370951aeb2de2899eb36dcab942a
+  metadata.gz: c08b20496949c92206042d3104e5aabca5afa04952022e5466431dacfe8ba617f6a0884fc29bfbc8078d5a2deae44a1a529c8e0545af3c0987792a18a9771499
+  data.tar.gz: f3543c490274eb01ca9267a6c6fdd2536a8ebd8a4bdff2c89cc82739bbd882c8738424375f3aec8ae57f74f3ae644538668ffd76032e0baf13f81dd4446df054

data/chef-handler-status_notifier.gemspec

@@ -3,7 +3,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = "chef-handler-status_notifier"
-  spec.version       = "0.3.0"
+  spec.version       = "0.4.0"
   spec.authors       = ["Faizal Zakaria"]
   spec.email         = ["phaibusiness@gmail.com"]
   spec.summary       = %q{Chef status notifier handler}

data/lib/chef/handler/status_notifier.rb

@@ -25,12 +25,15 @@ class StatusNotifierHandler < Chef::Handler
   def report
     if run_status.failed?
       msg = "Failure on #{node.name}: #{run_status.formatted_exception}"
+      status = :failed
+      send_to_hipchat(msg)
     else
       msg = "Chef run succesfully on #{node.name}"
+      status = :success
     end
 
+    send_to_slack(status, msg)
     send_to_hipchat(msg)
-    send_to_slack(msg)
   end
 
   private
@@ -40,11 +43,11 @@ class StatusNotifierHandler < Chef::Handler
     hipchat[@hipchat_params[:room_name]].send(@hipchat_params[:username], msg, :notify => @hipchat_params[:notify])
   end
 
-  def send_to_slack(msg)
+  def send_to_slack(status, msg)
     return unless @slack_params[:enabled]
     slack.channel = @slack_params[:channel]
     slack.username = @slack_params[:username]
-    slack.ping msg
+    slack.ping '', attachments: [slack_attachment(status, msg)]
   end
 
   def hipchat
@@ -54,4 +57,15 @@ class StatusNotifierHandler < Chef::Handler
   def slack
     @slack ||= Slack::Notifier.new(@slack_params[:webhook_url])
   end
+
+  def slack_attachment(status, msg)
+    color = (status == :failed)? "#ff0000" : "#36a64f"
+    {
+      fallback: "Opsworks status, #{msg}",
+      color: "#{color}",
+      author_name: "OpsworksBot (",
+      title: "status: #{status}",
+      text: "#{msg}"
+    }
+  end
 end

data/spec/chef/handler.rb

@@ -1,7 +1,7 @@
 class Chef::Handler
   class RunStatus
     def failed?
-      true
+      false
     end
 
     def formatted_exception

data/spec/status_notifier_spec.rb

@@ -5,7 +5,7 @@ describe StatusNotifierHandler do
   let(:slack_params) do
     {
       enabled: true,
-      webhook_url: "https://slacktest.com",
+      webhook_url: "https://hooks.slack.com/services/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",
       channel: "#test",
       username: "FaiNow-Test"
     }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chef-handler-status_notifier
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Faizal Zakaria