Merge ~cjwatson/launchpad-mojo-specs:codeimport-subordinates into launchpad-mojo-specs:master

Proposed by Colin Watson
Status: Merged
Merged at revision: d53c3068008bac16f4b5c3acb8f3f29d406f62fb
Proposed branch: ~cjwatson/launchpad-mojo-specs:codeimport-subordinates
Merge into: launchpad-mojo-specs:master
Diff against target: 103 lines (+1/-43)
4 files modified
dev/null (+0/-36)
lp-codeimport/collect-charms (+0/-1)
lp-codeimport/configs/custom-secgroups-production.yaml (+1/-2)
lp-codeimport/services (+0/-4)
Reviewer Review Type Date Requested Status
Cristian Gonzalez (community) Approve
Review via email: mp+404075@code.launchpad.net

Commit message

codeimport: Remove golive

To post a comment you must log in.
Revision history for this message
Cristian Gonzalez (cristiangsp) wrote :

Looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lp-codeimport/collect-charms b/lp-codeimport/collect-charms
2index a0a9fa0..724a5fa 100644
3--- a/lp-codeimport/collect-charms
4+++ b/lp-codeimport/collect-charms
5@@ -3,7 +3,6 @@ lp-codeimport git+https://git.launchpad.net/~launchpad/lp-codeimport/+git/char
6
7 # Subordinates
8 nrpe cs:nrpe-52
9-telegraf cs:telegraf-27
10
11 {%- if stage_name != "devel" %}
12 # Basenode
13diff --git a/lp-codeimport/configs/custom-secgroups-production.yaml b/lp-codeimport/configs/custom-secgroups-production.yaml
14index 3f5a829..4547841 100644
15--- a/lp-codeimport/configs/custom-secgroups-production.yaml
16+++ b/lp-codeimport/configs/custom-secgroups-production.yaml
17@@ -1,9 +1,8 @@
18 applications:
19 nrpe:
20 - nagios-monitored
21- telegraf:
22- - metrics
23 lp-codeimport:
24+ - metrics
25 - rsync-logs
26 rules:
27 metrics:
28diff --git a/lp-codeimport/golive/collect b/lp-codeimport/golive/collect
29deleted file mode 100644
30index c33df5e..0000000
31--- a/lp-codeimport/golive/collect
32+++ /dev/null
33@@ -1,5 +0,0 @@
34-# These are unpinned on the assumption that they should always be the latest when golive
35-# is added.
36-canonical-livepatch cs:canonical-livepatch
37-landscape-client cs:landscape-client
38-userdir-ldap cs:userdir-ldap
39diff --git a/lp-codeimport/golive/manifest b/lp-codeimport/golive/manifest
40deleted file mode 100644
41index 93c9f76..0000000
42--- a/lp-codeimport/golive/manifest
43+++ /dev/null
44@@ -1,3 +0,0 @@
45-collect config=golive/collect
46-deploy config=golive/services target=golive
47-include config=manifests/verify
48diff --git a/lp-codeimport/golive/services b/lp-codeimport/golive/services
49deleted file mode 100644
50index 1f15f42..0000000
51--- a/lp-codeimport/golive/services
52+++ /dev/null
53@@ -1,36 +0,0 @@
54-{%- if stage_name == "production" %}
55-{%- set environment_tag = "devops-production" %}
56-{%- elif stage_name in ("staging", "qastaging") %}
57-{%- set environment_tag = "devops-staging" %}
58-{%- else %}
59-{%- set environment_tag = "devops-unknown" %}
60-{%- endif -%}
61-golive:
62- series: {{ series }}
63- services:
64- lp-codeimport:
65- charm: lp-codeimport
66- nrpe:
67- charm: nrpe
68- landscape:
69- charm: landscape-client
70- options:
71- url: https://landscape.is.canonical.com/message-system
72- ping-url: http://landscape.is.canonical.com/ping
73- account-name: standalone
74- tags: juju-managed, devops-instance, {{environment_tag}}
75- registration-key: include-file://{{local_dir}}/canonical-is-landscape.key
76- canonical-livepatch:
77- charm: canonical-livepatch
78- options:
79- livepatch_key: include-file://{{local_dir}}/canonical-is-livepatch.key
80- livepatch_proxy: http://squid.internal:3128
81- userdir-ldap:
82- charm: userdir-ldap
83- options:
84- apt-repo-spec: "deb http://archive.admin.canonical.com/ubuntu {{ series }}-cat main"
85- relations:
86- - ["canonical-livepatch", "lp-codeimport"]
87- - ["canonical-livepatch", "nrpe"]
88- - ["landscape", "lp-codeimport"]
89- - ["userdir-ldap", "lp-codeimport"]
90diff --git a/lp-codeimport/services b/lp-codeimport/services
91index 8872e01..3cf072e 100644
92--- a/lp-codeimport/services
93+++ b/lp-codeimport/services
94@@ -109,9 +109,5 @@ lp-codeimport:
95 export_nagios_definitions: true
96 nagios_master: "{{ nagios_master }}"
97 {%- endif %}
98- telegraf:
99- charm: telegraf
100- expose: true
101 relations:
102 - ["nrpe", "lp-codeimport"]
103- - ["telegraf:juju-info", "lp-codeimport"]

Subscribers

People subscribed via source and target branches