centurion 1.3.0 → 1.3.1

data/CONTRIBUTORS.md

@@ -9,6 +9,8 @@ Your name could be here!
  * [Tom Dooner][tdooner]
  * [Jonathan Chauncey][jchauncey]
  * [Matt Sanford][mzsanford]
+ * [Suren Karapetyan][skarap]
+ * [Jon Wood][jellybob]
 
 Pre-release
 -----------
@@ -67,3 +69,5 @@ Contributor                                   | Commits    | Additions | Deletio
 [tdooner]: https://github.com/tdooner
 [jchauncey]: https://github.com/jchauncey
 [mzsanford]: https://github.com/mzsanford
+[skarap]: https://github.com/skarap
+[jellybob]: https://github.com/jellybob

data/lib/centurion/deploy_dsl.rb

@@ -38,7 +38,7 @@ module Centurion::DeployDSL
     require_options_keys(options,  [ :container_port ])
 
     add_to_bindings(
-      options[:host_ip] || '0.0.0.0',
+      options[:host_ip],
       options[:container_port],
       port,
       options[:type] || 'tcp'
@@ -79,9 +79,10 @@ module Centurion::DeployDSL
 
   def add_to_bindings(host_ip, container_port, port, type='tcp')
     set(:port_bindings, fetch(:port_bindings, {}).tap do |bindings|
-      bindings["#{container_port.to_s}/#{type}"] = [
-        {'HostIp' => host_ip, 'HostPort' => port.to_s}
-      ]
+      binding = { 'HostPort' => port.to_s }.tap do |b|
+        b['HostIp'] = host_ip if host_ip
+      end
+      bindings["#{container_port.to_s}/#{type}"] = [ binding ]
       bindings
     end)
   end

data/lib/centurion/version.rb

@@ -1,3 +1,3 @@
 module Centurion
-  VERSION = '1.3.0'
+  VERSION = '1.3.1'
 end

data/spec/deploy_dsl_spec.rb

@@ -71,6 +71,17 @@ describe Centurion::DeployDSL do
       DeployDSLTest.set(:port_bindings, dummy_value)
       DeployDSLTest.host_port(999, container_port: 80)
 
+      expect(DeployDSLTest).to have_key_and_value(
+        :port_bindings,
+        dummy_value.merge('80/tcp' => [{ 'HostPort' => '999' }])
+      )
+    end
+
+    it 'adds new bind ports to the list with an IP binding when supplied' do
+      dummy_value = { '666/tcp' => ['value'] }
+      DeployDSLTest.set(:port_bindings, dummy_value)
+      DeployDSLTest.host_port(999, container_port: 80, host_ip: '0.0.0.0')
+
       expect(DeployDSLTest).to have_key_and_value(
         :port_bindings,
         dummy_value.merge('80/tcp' => [{ 'HostIp' => '0.0.0.0', 'HostPort' => '999' }])

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: centurion
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
   prerelease: 
 platform: ruby
 authors: