Merge ~cjwatson/launchpad:charm-buildd-manager-authentication-timeout-option into launchpad:master

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 36ed703abed6d453c98dd028ebe376e208324df1
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:charm-buildd-manager-authentication-timeout-option
Merge into: launchpad:master
Diff against target: 30 lines (+7/-1)
2 files modified
charm/launchpad-buildd-manager/config.yaml (+6/-0)
charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf (+1/-1)
Reviewer Review Type Date Requested Status
Guruprasad Approve
Review via email: mp+454138@code.launchpad.net

Commit message

charm: Allow buildd-manager authentication timeout to be tuned

Description of the change

It looks like we may need to adjust this on production, so add a charm option for it.

To post a comment you must log in.
Revision history for this message
Guruprasad (lgp171188) wrote :

LGTM 👍

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/charm/launchpad-buildd-manager/config.yaml b/charm/launchpad-buildd-manager/config.yaml
2index ea622d3..58cbfe4 100644
3--- a/charm/launchpad-buildd-manager/config.yaml
4+++ b/charm/launchpad-buildd-manager/config.yaml
5@@ -9,6 +9,12 @@ options:
6 Credentials for reading from Artifactory repositories (formatted as
7 "user:token").
8 default: ""
9+ authentication_timeout:
10+ type: int
11+ description: >
12+ The time in seconds that the builddmaster will wait for a reply from
13+ the authserver.
14+ default: 60
15 builder_proxy_auth_api_admin_secret:
16 type: string
17 description: >
18diff --git a/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf b/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf
19index ddf9fbd..3641a19 100644
20--- a/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf
21+++ b/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf
22@@ -13,7 +13,7 @@ extends: ../launchpad-db-lazr.conf
23
24 [builddmaster]
25 authentication_endpoint: http://{{ domain_xmlrpc_private }}:{{ port_xmlrpc }}/authserver
26-authentication_timeout: 60
27+authentication_timeout: {{ authentication_timeout }}
28 {{- opt("builder_proxy_auth_api_admin_username", builder_proxy_auth_api_admin_username) }}
29 {{- opt("builder_proxy_auth_api_endpoint", builder_proxy_auth_api_endpoint) }}
30 {{- opt("builder_proxy_host", builder_proxy_host) }}

Subscribers

People subscribed via source and target branches

to status/vote changes: