kitchen-docker 3.3.3 → 3.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f64096aa934f3502dd1427249a5d15db1371855534522ecb20b4ea4e928f7a5b
-  data.tar.gz: 013667f1b55015399185c490cae0c3cd5e899e9fb8678af3fcb42d74c5c9e5d8
+  metadata.gz: be40fecdd59251c4d7b1a20de831bd939fd7018ce19d19a2a2f205ac2464f4fb
+  data.tar.gz: 04b4579a2569e8be9f943d77eca1d4fd9f5e72e2aefc2ad3009855bf1e565935
 SHA512:
-  metadata.gz: 414439ff16eeffb9f0288925cd6579da33f60968efe4ead0fa85d6454d98499f634dfed7bfe90ff2131c2a519c2b5f109fdefd7cb854a917f7ff756f539a7f24
-  data.tar.gz: 72446a19d4951a3b1b87b66a3c604f13f67cac6395588e1698c4a9dc4ab7cd8f9f3edaa31c1c5ecc7ed5477a26a0e40639d217290ae48c74fb368daadc555b45
+  metadata.gz: d14d1fcddd8c823cc51231351311d2130bb9e9dd8355bb09e0551635a982a38b8abc52fca32cb03b5813d484bc256a688574ad8da6ef44a3d20525bdeac8fb21
+  data.tar.gz: 93f7faf2b089f727c6c78808b6c3e4d3fc9234e22a4defa21a0b463b61992324d8ecd26fb5bb9bfbc7da415d00234c67815d7bc20e889c563da50324c76c1040

data/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "3.3.3"
+  ".": "3.3.4"
 }

data/CHANGELOG.md

@@ -6,6 +6,13 @@ Future CHANGELOG notes will be in GitHub release notes
 
 * Docs: document the last four options and split contributor docs ([#463](https://github.com/test-kitchen/kitchen-docker/pull/463)) ([4818bce](https://github.com/test-kitchen/kitchen-docker/commit/4818bce))
 
+## [3.3.4](https://github.com/test-kitchen/kitchen-docker/compare/v3.3.3...v3.3.4) (2026-08-23)
+
+
+### Bug Fixes
+
+* read the container IP from NetworkSettings.Networks ([#479](https://github.com/test-kitchen/kitchen-docker/issues/479)) ([98b3337](https://github.com/test-kitchen/kitchen-docker/commit/98b3337d9c03e7cdde6d6d28d9e311b87f37c64f))
+
 ## [3.3.3](https://github.com/test-kitchen/kitchen-docker/compare/v3.3.2...v3.3.3) (2026-08-23)
 
 

data/lib/kitchen/docker/docker_version.rb

@@ -18,6 +18,6 @@ module Kitchen
   # Everything belonging to the kitchen-docker plugin.
   module Docker
     # The version of the kitchen-docker gem.
-    DOCKER_VERSION = "3.3.3".freeze
+    DOCKER_VERSION = "3.3.4".freeze
   end
 end

data/lib/kitchen/docker/helpers/container_helper.rb

@@ -12,6 +12,7 @@
 # limitations under the License.
 
 require "erb" unless defined?(Erb)
+require "ipaddr" unless defined?(IPAddr)
 require "json" unless defined?(JSON)
 require "shellwords" unless defined?(Shellwords)
 require "tempfile" unless defined?(Tempfile)
@@ -253,15 +254,47 @@ module Kitchen
           parse_container_id(output)
         end
 
+        # The container's address on the Docker network.
+        #
+        # Read from +NetworkSettings.Networks+ rather than the top-level
+        # +NetworkSettings.IPAddress+. That field was only ever populated for
+        # the default bridge, and Docker 29 removed it altogether -- asking for
+        # it there fails the whole `docker inspect` with "map has no entry for
+        # key \"IPAddress\"", which took +use_internal_docker_network+ with it.
+        # +Networks+ has been present since Docker 1.9, so reading it works on
+        # both.
+        #
+        # A container attached to several networks has an address on each; the
+        # first is used, which is the only one for the single-network case this
+        # option is for.
+        #
         # @param state [Hash] instance state naming the container
         # @return [String] the container's address on the Docker network
         # @raise [Kitchen::ActionFailed] if it cannot be determined
         def container_ip_address(state)
-          cmd = "inspect --format '{{ .NetworkSettings.IPAddress }}'"
+          cmd = "inspect --format '{{range .NetworkSettings.Networks}}{{.IPAddress}} {{end}}'"
           cmd << " #{state[:container_id]}"
-          docker_command(cmd).strip
-        rescue
-          raise ActionFailed, "Error getting internal IP of Docker container"
+          output = docker_command(cmd, suppress_output: !logger.debug?)
+
+          # Picked by parsing rather than by taking the first word, so that a
+          # warning docker writes to stderr is not returned as an address.
+          address = output.split(/\s+/).find { |token| ip_address?(token) }
+          raise ActionFailed, "Docker reports no IP address for the container" if address.nil?
+
+          address
+        rescue => e
+          raise ActionFailed, "Error getting internal IP of Docker container. #{e}"
+        end
+
+        # @param token [String] a candidate address
+        # @return [Boolean] whether it parses as an IPv4 or IPv6 address
+        def ip_address?(token)
+          return false if token.nil? || token.empty?
+
+          IPAddr.new(token)
+          true
+        rescue IPAddr::Error
+          false
         end
 
         # Stops and removes the container.

data/spec/container_helper_spec.rb

@@ -270,6 +270,58 @@ describe Kitchen::Docker::Helpers::ContainerHelper do
     end
   end
 
+  describe "#container_ip_address" do
+    def helper_inspecting(output)
+      h = helper
+      @asked = nil
+      allow(h).to receive(:docker_command) { |cmd, _opts = {}| @asked = cmd; output }
+      h
+    end
+
+    let(:state) { { container_id: "abc123abc123" } }
+
+    it "returns the address docker reports" do
+      expect(helper_inspecting("172.17.0.7 \n").container_ip_address(state)).to eq "172.17.0.7"
+    end
+
+    it "reads Networks rather than the removed top-level IPAddress" do
+      # Docker 29 dropped NetworkSettings.IPAddress. Asking for it does not
+      # return empty -- it fails the whole inspect with a template error, which
+      # took use_internal_docker_network down with it.
+      helper_inspecting("172.17.0.7\n").container_ip_address(state)
+      expect(@asked).to include(".NetworkSettings.Networks")
+      expect(@asked).not_to include(".NetworkSettings.IPAddress")
+    end
+
+    it "takes the first address when the container is on several networks" do
+      expect(helper_inspecting("172.17.0.7 172.19.0.2 \n").container_ip_address(state))
+        .to eq "172.17.0.7"
+    end
+
+    it "handles an IPv6 address" do
+      expect(helper_inspecting("2001:db8::2 \n").container_ip_address(state)).to eq "2001:db8::2"
+    end
+
+    it "ignores a warning docker wrote to stderr" do
+      expect(helper_inspecting("WARNING: something happened\n172.17.0.7 \n").container_ip_address(state))
+        .to eq "172.17.0.7"
+    end
+
+    it "fails loudly when docker reports no address" do
+      # Returning "" here would be recorded as the instance hostname, and the
+      # connection would fail somewhere far from the cause.
+      expect { helper_inspecting(" \n").container_ip_address(state) }
+        .to raise_error(Kitchen::ActionFailed, /no IP address/)
+    end
+
+    it "fails loudly when the inspect itself fails" do
+      h = helper
+      allow(h).to receive(:docker_command).and_raise(Kitchen::ShellOut::ShellCommandFailed, "boom")
+      expect { h.container_ip_address(state) }
+        .to raise_error(Kitchen::ActionFailed, /Error getting internal IP/)
+    end
+  end
+
   describe "#remove_container" do
     it "stops the container before removing it" do
       # `docker rm` refuses to remove a running container, so the order matters.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-docker
 version: !ruby/object:Gem::Version
-  version: 3.3.3
+  version: 3.3.4
 platform: ruby
 authors:
 - Sean Porter