Merge pull request #1714 from nbgoodall/main

Make environment variables optional for tagged hosts

Author: djmb

lib/kamal/configuration/role.rb

@@ -36,7 +36,7 @@ def hosts
   end
 
   def env_tags(host)
-    tagged_hosts.fetch(host).collect { |tag| config.env_tag(tag) }
+    tagged_hosts.fetch(host).collect { |tag| config.env_tag(tag) }.compact
   end
 
   def cmd

test/configuration/role_test.rb

@@ -29,6 +29,13 @@ class ConfigurationRoleTest < ActiveSupport::TestCase
     assert_equal [ "1.1.1.3", "1.1.1.4" ], config_with_roles.role(:workers).hosts
   end
 
+  test "missing env tag is ignored" do
+    @deploy_with_roles[:servers]["workers"]["hosts"] = [ { "1.1.1.3" => [ "job" ] } ]
+
+    role = Kamal::Configuration.new(@deploy_with_roles).role(:workers)
+    assert_equal "redis://a/b", role.env("1.1.1.3").clear["REDIS_URL"]
+  end
+
   test "cmd" do
     assert_nil config.role(:web).cmd
     assert_equal "bin/jobs", config_with_roles.role(:workers).cmd