stackup 1.6.0 → 1.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 281a4cc419891f70b5a8251d52c080a678e21d2a76a57a0e715b478987c5d967
-  data.tar.gz: e1848f7cccad92a7a6883f3947daa68e70d1b72ad00bfa72727f0def2590ee89
+  metadata.gz: 60e50e255335b7924e71a6fed889c946c3b182c2b95277056f88e2e420915f79
+  data.tar.gz: 816b9a71283680668fe9d3cf35c54a3a1a2516a68f0251a1771e4bba17e2cce2
 SHA512:
-  metadata.gz: 59cd63007ce56b284e7176c82f354174bb78dec2fba84443e2b1a28465df5ed68cd0bc1ccd2474dab7572c61ebb85d9de328e12216395d0e68c8101863b0110a
-  data.tar.gz: 3acf8d1d83307ef2851001558c7880c4d43692f740254531369713e4cb7d38f3595398195cfd8f67460b10c1bd94cd7dca08a833fdf51371ef0a1c914addcffc
+  metadata.gz: be56a2ae96045ace217de36cd7f82592eb2588178b314f8d68d51e6b6d16a1e66c60557e78fa5f524cb710aa42f0d64372629fe4d6b9aad65c3ddec11fe051f4
+  data.tar.gz: 79974fadb7ac89a76a92fa424c4cc2ef63deeec8cfd55b6eda352e965ecd84e832322eddafaaf9eb57fd3640c3744646a484954664f15cfa9bbfeda4b0eb03c8

data/CHANGES.md

@@ -1,5 +1,9 @@
 # CHANGES
 
+## 1.7.0 (2020-11-25)
+
+* Feature: --no-fail-on-empty-change-set
+
 ## 1.6.0 (2020-11-19)
 
 * Feature: Support --service-role-arn on "change-set create"

data/README.md

@@ -218,6 +218,8 @@ The change-set name defaults to "pending", but can be overridden using `--name`.
 
 The `change-set create` subcommand, like the `up` command, supports `--service-role-arn` to specify a service role.
 
+It is impossible to create a change set with no changes. By default, stackup will only return successfully if a change set was actually created, and will otherwise fail. If the `--no-fail-on-empty-change-set` option is provided, stackup will return successfully if a change set was created _or_ if no change set was created because no changes were needed.
+
 ## Programmatic usage
 
 Get a handle to a `Stack` object as follows:

data/lib/stackup/change_set.rb

@@ -56,6 +56,7 @@ module Stackup
       options[:change_set_name] = name
       options[:change_set_type] = stack.exists? ? "UPDATE" : "CREATE"
       force = options.delete(:force)
+      allow_empty_change_set = options.delete(:allow_empty_change_set)
       options[:template_body] = MultiJson.dump(options.delete(:template)) if options[:template]
       # optionally override template_body with the original template to preserve formatting (& comments in YAML)
       template_orig = options.delete(:template_orig)
@@ -73,8 +74,12 @@ module Stackup
           when /COMPLETE/
             return current.status
           when "FAILED"
-            logger.error(current.status_reason)
-            raise StackUpdateError, "change-set creation failed" if status == "FAILED"
+            if allow_empty_change_set and current.status_reason == "The submitted information didn't contain changes. Submit different information to create a change set."
+              return current.status_reason
+            else
+              logger.error(current.status_reason)
+              raise StackUpdateError, "change-set creation failed" if status == "FAILED"
+            end
           end
           sleep(wait_poll_interval)
         end

data/lib/stackup/main_command.rb

@@ -280,6 +280,9 @@ module Stackup
         option ["--force"], :flag,
                "replace existing change-set of the same name"
 
+        option ["--no-fail-on-empty-change-set"], :flag, "don't fail on empty change-set",
+               :attribute_name => :allow_empty_change_set
+
         include HasParameters
 
         option "--tags", "FILE", "stack tags file",
@@ -314,6 +317,7 @@ module Stackup
           options[:role_arn] = service_role_arn if service_role_arn
           options[:use_previous_template] = use_previous_template?
           options[:force] = force?
+          options[:allow_empty_change_set] = allow_empty_change_set?
           options[:capabilities] = capability_list
           options[:preserve] = preserve_template_formatting?
           report_change do

data/lib/stackup/version.rb

@@ -1,5 +1,5 @@
 module Stackup
 
-  VERSION = "1.6.0".freeze
+  VERSION = "1.7.0".freeze
 
 end

data/spec/stackup/main_command_spec.rb

@@ -1,15 +1,19 @@
 require "stackup/main_command"
 
 describe Stackup::MainCommand do
-  context "change-set create --service-role-arn"
-    it "invokes stack.change_set.create with role arn passed through" do
-      mock_stackup = double()
-      mock_stack = double()
-      mock_change_set = double()
-      allow_any_instance_of(Stackup::MainCommand).to receive(:Stackup).and_return(mock_stackup)
-      allow(mock_stackup).to receive(:stack).and_return(mock_stack)
-      allow(mock_stack).to receive(:change_set).and_return(mock_change_set)
 
+  let(:mock_change_set) { double() }
+
+  before(:example) do
+    mock_stackup = double()
+    mock_stack = double()
+    allow_any_instance_of(Stackup::MainCommand).to receive(:Stackup).and_return(mock_stackup)
+    allow(mock_stackup).to receive(:stack).and_return(mock_stack)
+    allow(mock_stack).to receive(:change_set).and_return(mock_change_set)
+  end
+
+  context "change-set create --service-role-arn" do
+    it "invokes stack.change_set.create with role arn passed through" do
       expected_args = {
         role_arn: "arn:aws:iam::000000000000:role/example"
       }
@@ -20,4 +24,33 @@ describe Stackup::MainCommand do
         "--template", "examples/template.yml",
         "--service-role-arn", "arn:aws:iam::000000000000:role/example"])
     end
+  end
+
+  context "change-set create" do
+    it "invokes stack.change_set.create with allow_empty_change_set nil" do
+      expected_args = {
+        allow_empty_change_set: nil
+      }
+      expect(mock_change_set).to receive(:create).with(hash_including(expected_args))
+
+      Stackup::MainCommand.run("stackup", [
+        "STACK-NAME", "change-set", "create",
+        "--template", "examples/template.yml"])
+    end
+  end
+
+  context "change-set create --no-fail-on-empty-change-set" do
+    it "invokes stack.change_set.create with allow_empty_change_set true" do
+      expected_args = {
+        allow_empty_change_set: true
+      }
+      expect(mock_change_set).to receive(:create).with(hash_including(expected_args))
+
+      Stackup::MainCommand.run("stackup", [
+        "STACK-NAME", "change-set", "create",
+        "--template", "examples/template.yml",
+        "--no-fail-on-empty-change-set"])
+    end
+  end
+
 end

data/spec/stackup/stack_spec.rb

@@ -429,6 +429,38 @@ describe Stackup::Stack do
 
       end
 
+      context "when allow_empty_change_set is nil and there are no changes" do
+        it "raises an exception" do
+          cf_client.stub_responses(:describe_change_set, [{
+            status: "FAILED",
+            status_reason: "The submitted information didn't contain changes. Submit different information to create a change set."
+          }])
+          expect { create_change_set }.to raise_error(Stackup::StackUpdateError)
+        end
+      end
+
+      context "when allow_empty_change_set is true and there are no changes" do
+        it "does not raise an exception" do
+          cf_client.stub_responses(:describe_change_set, [{
+            status: "FAILED",
+            status_reason: "The submitted information didn't contain changes. Submit different information to create a change set."
+          }])
+          options[:allow_empty_change_set] = true
+          expect { create_change_set }.not_to raise_error
+        end
+      end
+
+      context "when allow_empty_change_set is true and there is some other failure" do
+        it "raises an exception" do
+          cf_client.stub_responses(:describe_change_set, [{
+            status: "FAILED",
+            status_reason: "some other failure message"
+          }])
+          options[:allow_empty_change_set] = true
+          expect { create_change_set }.to raise_error(Stackup::StackUpdateError)
+        end
+      end
+
     end
 
     describe "#change_set#execute" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stackup
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 1.7.0
 platform: ruby
 authors:
 - Mike Williams
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-11-19 00:00:00.000000000 Z
+date: 2020-11-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk-cloudformation