spacebar/kamal
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #212 from basecamp/role-primary-hosts

Browse Source 
Pull the primary host from the role
This commit is contained in:
David Heinemeier Hansson
2023-04-12 14:09:38 +02:00
committed by GitHub
parent 76673c0c1b fb62f2e6e1
commit 2f1393cd92
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/mrsk/commander.rb
View File

@@ -36,7 +36,7 @@ class Mrsk::Commander
end end
def primary_host def primary_host
specific_hosts&.first || config.primary_web_host specific_hosts&.first || specific_roles&.first&.primary_host || config.primary_web_host
end end
def roles def roles

4
test/commander_test.rb
View File

@@ -47,8 +47,8 @@ class CommanderTest < ActiveSupport::TestCase
end end
test "primary_host with specific hosts via role" do test "primary_host with specific hosts via role" do
@mrsk.specific_roles = "web" @mrsk.specific_roles = "workers"
assert_equal "1.1.1.1", @mrsk.primary_host assert_equal "1.1.1.3", @mrsk.primary_host
end end
test "roles_on" do test "roles_on" do