Skip to content

Commit

Permalink
Remove the database adapter property
Browse files Browse the repository at this point in the history
Notifications only supports mysql
  • Loading branch information
dprotaso committed Apr 18, 2017
1 parent e134871 commit 717b771
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 0 additions & 2 deletions jobs/deploy-notifications/spec
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,6 @@ properties:
description: 'Admin client id of the UAA'
notifications.uaa.admin_client_secret:
description: 'Admin client secret of the UAA'
notifications.database.adapter:
description: 'Adapter (e.g. mysql or postgres) for the database connection'
notifications.database.username:
description: 'Username for the database connection'
notifications.database.password:
Expand Down
3 changes: 1 addition & 2 deletions jobs/deploy-notifications/templates/manifest.yml.erb
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
if_p("notifications.database.url") do |url|
database_url = url
end.else do
adapter = p("notifications.database.adapter")
username = p("notifications.database.username")
password = p("notifications.database.password")
port = p("notifications.database.port")
Expand All @@ -17,7 +16,7 @@
host = link("database").instances[0].address
end

database_url = "#{adapter}://#{username}:#{password}@#{host}:#{port}/#{database}"
database_url = "mysql://#{username}:#{password}@#{host}:#{port}/#{database}"
end
%>
applications:
Expand Down

0 comments on commit 717b771

Please sign in to comment.