← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] ~cjwatson/launchpad-layers:fix-set-role-after-connecting into launchpad-layers:main

 

Colin Watson has proposed merging ~cjwatson/launchpad-layers:fix-set-role-after-connecting into launchpad-layers:main.

Commit message:
Stop setting launchpad.dbuser

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad-layers/+git/launchpad-layers/+merge/439325

Commit 819dd81912f69d759d72b294936c084ac227c18b removed `db_user` from the config dictionary, since we can now log in using the user in the connection string and switch to the appropriate role using `SET ROLE`; but I forgot to remove the corresponding line from the template.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad-layers:fix-set-role-after-connecting into launchpad-layers:main.
diff --git a/launchpad-base/templates/launchpad-base-lazr.conf b/launchpad-base/templates/launchpad-base-lazr.conf
index 748aaa2..da168c7 100644
--- a/launchpad-base/templates/launchpad-base-lazr.conf
+++ b/launchpad-base/templates/launchpad-base-lazr.conf
@@ -63,7 +63,6 @@ bugs_domain: bugs.{{ domain }}
 {{- opt("candid_service_root", candid_service_root) }}
 code_domain: code.{{ domain }}
 config_overlay_dir: {{ secrets_dir }}
-dbuser: {{ db_user }}
 default_batch_size: {{ default_batch_size }}
 {{- opt("geoip_database", geoip_database) }}
 {{- opt("http_proxy", http_proxy) }}