Merge ~pelpsi/launchpad:launchpad-build-manager-fetch-service-support into launchpad:master

Proposed by Simone Pelosi
Status: Merged
Approved by: Simone Pelosi
Approved revision: 8e0fe4f6cc9cd7beb701675a86b399073156acf4
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~pelpsi/launchpad:launchpad-build-manager-fetch-service-support
Merge into: launchpad:master
Diff against target: 80 lines (+30/-4)
4 files modified
charm/launchpad-buildd-manager/config.yaml (+22/-0)
charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf (+4/-0)
charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-secrets-lazr.conf (+1/-1)
lib/lp/services/config/schema-lazr.conf (+3/-3)
Reviewer Review Type Date Requested Status
Ines Almeida Approve
Review via email: mp+462334@code.launchpad.net

Commit message

Update launchpad-buildd-manager to support Fetch Service

Add configs for fetch service into launchpad-buildd-manager charm.

To post a comment you must log in.
Revision history for this message
Ines Almeida (ines-almeida) :
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 58cbfe4..caac316 100644
3--- a/charm/launchpad-buildd-manager/config.yaml
4+++ b/charm/launchpad-buildd-manager/config.yaml
5@@ -52,6 +52,28 @@ options:
6 YAML-encoded dictionary mapping pillars to dictionaries of
7 configuration items to set for CI builds of those pillars.
8 default: ""
9+ fetch_service_control_admin_secret:
10+ type: string
11+ description: >
12+ Admin secret for requesting sessions from the fetch service.
13+ default: ""
14+ fetch_service_control_admin_username:
15+ type: string
16+ description: Admin username for fetch service.
17+ default: ""
18+ fetch_service_control_endpoint:
19+ type: string
20+ description: Endpoint for fetch service control service.
21+ default: ""
22+ fetch_service_host:
23+ type: string
24+ description: >
25+ Fetch service host, it could be either a single instance
26+ or a load balancer in front.
27+ default: ""
28+ fetch_service_port:
29+ type: int
30+ description: Fetch service port.
31 socket_timeout:
32 type: int
33 description: >
34diff --git a/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf b/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf
35index 3641a19..aed4cdf 100644
36--- a/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf
37+++ b/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-lazr.conf
38@@ -18,6 +18,10 @@ authentication_timeout: {{ authentication_timeout }}
39 {{- opt("builder_proxy_auth_api_endpoint", builder_proxy_auth_api_endpoint) }}
40 {{- opt("builder_proxy_host", builder_proxy_host) }}
41 {{- opt("builder_proxy_port", builder_proxy_port) }}
42+{{- opt("fetch_service_control_admin_username", fetch_service_control_admin_username) }}
43+{{- opt("fetch_service_control_endpoint", fetch_service_control_endpoint) }}
44+{{- opt("fetch_service_host", fetch_service_host) }}
45+{{- opt("fetch_service_port", fetch_service_port) }}
46 root: {{ buildd_manager_dir }}
47 socket_timeout: {{ socket_timeout }}
48 virtualized_socket_timeout: {{ virtualized_socket_timeout }}
49diff --git a/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-secrets-lazr.conf b/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-secrets-lazr.conf
50index af20066..7637fcb 100644
51--- a/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-secrets-lazr.conf
52+++ b/charm/launchpad-buildd-manager/templates/launchpad-buildd-manager-secrets-lazr.conf
53@@ -15,4 +15,4 @@
54
55 [builddmaster]
56 {{- opt("builder_proxy_auth_api_admin_secret", builder_proxy_auth_api_admin_secret) }}
57-
58+{{- opt("fetch_service_control_admin_secret", fetch_service_control_admin_secret) }}
59diff --git a/lib/lp/services/config/schema-lazr.conf b/lib/lp/services/config/schema-lazr.conf
60index bbfe8b8..fed991a 100644
61--- a/lib/lp/services/config/schema-lazr.conf
62+++ b/lib/lp/services/config/schema-lazr.conf
63@@ -1880,14 +1880,14 @@ fetch_service_control_admin_secret: none
64 # Admin username for fetch service.
65 fetch_service_control_admin_username: none
66
67-# Endpoint for fetch service authentication service
68+# Endpoint for fetch service control service.
69 fetch_service_control_endpoint: none
70
71 # Fetch service host, it could be either a single instance
72-# or a load balancer in front
73+# or a load balancer in front.
74 fetch_service_host: none
75
76-# Fetch service port
77+# Fetch service port.
78 fetch_service_port: none
79
80 # Optional sources.list entry to send to build workers when doing snap

Subscribers

People subscribed via source and target branches

to status/vote changes: