Skip to content

Commit b1fca17

Browse files
committed
Fix route_registrar path logic in bpm template
- invoking .sub!() was replacing the string in-place, causing the data path to be added twice - Also brings back routing_api.auth_disabled flag to the routing_api bpm template because we accidentally forgot it [#175826416](https://www.pivotaltracker.com/story/show/175826416)
1 parent 4b49668 commit b1fca17

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

jobs/route_registrar/templates/bpm.yml.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ unless paths.empty?
3939

4040
paths.each do |path|
4141
unsafe['unsafe']['unrestricted_volumes'] << {"path" => path, "allow_executions" => true}
42-
unsafe['unsafe']['unrestricted_volumes'] << {"path" => path.sub!('jobs', 'data')}
42+
unsafe['unsafe']['unrestricted_volumes'] << {"path" => path.sub('jobs', 'data')}
4343
end
4444

4545
bpm['processes'][0].merge!(unsafe)

jobs/routing-api/templates/bpm.yml.erb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,10 @@ bpm = {
2323
]
2424
}
2525

26+
if p('routing_api.auth_disabled') == true
27+
bpm['processes'][0]['args'] << "-devMode"
28+
end
29+
2630
if p('golang.x509ignoreCN') == true
2731
bpm['processes'][0]['env'].merge!({"GODEBUG" => "x509ignoreCN=0"})
2832
end

0 commit comments

Comments
 (0)