Merge pull request #114 from base2Services/feature/alarm-tagging-fix

Fix tagging for multiple alarm actions
base2Services · Aug 17, 2023 · af2be86 · af2be86
2 parents 754ce5c + 63b63bd
commit af2be86
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/cfnguardian/tagger.rb b/lib/cfnguardian/tagger.rb
@@ -29,7 +29,11 @@ def tag_alarm(alarm, global_tags={})
         logger.debug "Updating tags on alarm #{alarm_arn}"
         new_tags.delete_if {|key, value| value.include?('?')}
         begin
-          @client.tag_resource({
+          alarm_severity = new_tags["guardian:alarm:severity"]
+          if alarm_severity.is_a?(Array)
+            new_tags["guardian:alarm:severity"] = new_tags["guardian:alarm:severity"].join("/")
+          end
+            @client.tag_resource({
             resource_arn: alarm_arn,
             tags: new_tags.map {|key,value| {key: key, value: value}}
           })

diff --git a/lib/cfnguardian/version.rb b/lib/cfnguardian/version.rb
@@ -1,4 +1,4 @@
 module CfnGuardian
-  VERSION = "0.11.4"
+  VERSION = "0.11.5"
   CHANGE_SET_VERSION = VERSION.gsub('.', '-').freeze
 end