Skip to content

Commit faae559

Browse files
committed
Revert "Remove reliance on bosh link named 'nats'"
Temporarily reverting these commits: * 4da790d * 3fc507f
1 parent 87b07e9 commit faae559

File tree

2 files changed

+25
-14
lines changed

2 files changed

+25
-14
lines changed

jobs/route_registrar/templates/registrar_settings.json.erb

Lines changed: 20 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,41 +1,51 @@
11
<%=
22
require 'json'
33

4-
nats_link_name = 'nats'
5-
if p('nats.tls.enabled')
6-
nats_link_name = 'nats-tls'
7-
end
8-
94
nats_machines = nil
105
if_p('nats.machines') do |ips|
116
nats_machines = ips.compact
127
end.else do
13-
link(nats_link_name).if_p("nats.hostname") do |hostname|
8+
link('nats').if_p("nats.hostname") do |hostname|
149
nats_machines = [hostname]
1510
end.else do
16-
nats_machines = link(nats_link_name).instances.map { |instance| instance.address }
11+
nats_machines = link('nats').instances.map { |instance| instance.address }
1712
end
1813
end
1914

2015
nats_port = nil
2116
if_p('nats.port') do |prop|
2217
nats_port = prop
2318
end.else do
24-
nats_port = link(nats_link_name).p("nats.port")
19+
nats_port = link('nats').p("nats.port")
2520
end
2621

2722
nats_user = nil
2823
if_p('nats.user') do |prop|
2924
nats_user = prop
3025
end.else do
31-
nats_user = link(nats_link_name).p("nats.user")
26+
nats_user = link('nats').p("nats.user")
3227
end
3328

3429
nats_password = nil
3530
if_p('nats.password') do |prop|
3631
nats_password = prop
3732
end.else do
38-
nats_password = link(nats_link_name).p("nats.password")
33+
nats_password = link('nats').p("nats.password")
34+
end
35+
36+
if_link('nats-tls') do |nats_link|
37+
if p('nats.tls.enabled')
38+
nats_machines = nats_link.instances.map { |instance| instance.address }
39+
nats_link.if_p("nats.hostname") do |hostname|
40+
nats_machines = [hostname]
41+
end
42+
if_p("nats.tls.hostname") do | prop |
43+
nats_machines = [prop]
44+
end
45+
nats_port = nats_link.p("nats.port")
46+
nats_user = nats_link.p("nats.user")
47+
nats_password = nats_link.p("nats.password")
48+
end
3949
end
4050

4151
message_bus_servers = nats_machines.map do |host|

spec/route_registar_templates_spec.rb

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@
227227
name: 'nats',
228228
properties: {
229229
'nats' => {
230-
'hostname' => 'nats-host', 'user' => 'nats-user', 'password' => 'nats-password', 'port' => 8080
230+
'host' => 'nats-host', 'user' => 'nats-user', 'password' => 'nats-password', 'port' => 8080
231231
}
232232
},
233233
instances: [Bosh::Template::Test::LinkInstance.new(address: 'my-nats-ip')]
@@ -236,7 +236,7 @@
236236
name: 'nats-tls',
237237
properties: {
238238
'nats' => {
239-
'hostname' => 'nats-tls-host', 'user' => 'nats-tls-user', 'password' => 'nats-tls-password', 'port' => 9090
239+
'host' => 'nats-tls-host', 'user' => 'nats-tls-user', 'password' => 'nats-tls-password', 'port' => 9090
240240
}
241241
},
242242
instances: [Bosh::Template::Test::LinkInstance.new(address: 'my-nats-tls-ip')]
@@ -247,11 +247,12 @@
247247
context 'when mTLS is enabled for NATS' do
248248
it 'renders with the nats-tls properties' do
249249
merged_manifest_properties['nats'] = { 'tls' => { 'enabled' => true } }
250+
merged_manifest_properties['nats']['tls']['hostname'] = 'my-nats-tls-hostname'
250251

251252
rendered_hash = JSON.parse(template.render(merged_manifest_properties, consumes: links))
252253
expect(rendered_hash['nats_mtls_config']['enabled']).to be true
253254
expect(rendered_hash['message_bus_servers'].length).to eq(1)
254-
expect(rendered_hash['message_bus_servers'][0]['host']).to eq('nats-tls-host:9090')
255+
expect(rendered_hash['message_bus_servers'][0]['host']).to eq('my-nats-tls-hostname:9090')
255256
expect(rendered_hash['message_bus_servers'][0]['user']).to eq('nats-tls-user')
256257
expect(rendered_hash['message_bus_servers'][0]['password']).to eq('nats-tls-password')
257258
end
@@ -262,7 +263,7 @@
262263
rendered_hash = JSON.parse(template.render(merged_manifest_properties, consumes: links))
263264
expect(rendered_hash['nats_mtls_config']['enabled']).to be false
264265
expect(rendered_hash['message_bus_servers'].length).to eq(1)
265-
expect(rendered_hash['message_bus_servers'][0]['host']).to eq('nats-host:8080')
266+
expect(rendered_hash['message_bus_servers'][0]['host']).to eq('my-nats-ip:8080')
266267
expect(rendered_hash['message_bus_servers'][0]['user']).to eq('nats-user')
267268
expect(rendered_hash['message_bus_servers'][0]['password']).to eq('nats-password')
268269
end

0 commit comments

Comments
 (0)