Merge lp:~ce-infrastructure/capomastro/capomastro-private-ppa-1387809 into lp:~ubuntuone-hackers/capomastro/charm

Proposed by Caio Begotti
Status: Merged
Merged at revision: 54
Proposed branch: lp:~ce-infrastructure/capomastro/capomastro-private-ppa-1387809
Merge into: lp:~ubuntuone-hackers/capomastro/charm
Diff against target: 39 lines (+6/-4)
3 files modified
config.yaml (+4/-1)
hooks/install (+1/-3)
inc/common (+1/-0)
To merge this branch: bzr merge lp:~ce-infrastructure/capomastro/capomastro-private-ppa-1387809
Reviewer Review Type Date Requested Status
Sheila Miguez (community) Approve
Review via email: mp+241643@code.launchpad.net

Description of the change

It is really annoying that apt-add-repository won't accept private PPAs so that was needed otherwise I would have to keep changing the install hooks of both Capomastro's and our forked Jenkins' charms. At least with this we can keep the private repository line with auth stuff in it separated in a config.yaml used for deployment only. This only affects Capomastro as the Jenkins changes were pushed to my forked (+junk) Jenkins charm already.

To post a comment you must log in.
Revision history for this message
Caio Begotti (caio1982) wrote :

Btw this is the original bug that this will fix, eventually: https://bugs.launchpad.net/capomastro/+bug/1387809

Revision history for this message
Sheila Miguez (codersquid) wrote :

+1 to the change.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config.yaml'
2--- config.yaml 2014-11-03 20:22:30 +0000
3+++ config.yaml 2014-11-13 02:55:22 +0000
4@@ -39,4 +39,7 @@
5 type: string
6 default: "capomastro@localhost"
7 description: "The superuser e-mail address for eventual notifications"
8-
9+ repository:
10+ type: string
11+ default: "ppa:ce-infrastructure/capomastro"
12+ description: "The PPA address or private repository line for apt-add-repository"
13
14=== modified file 'hooks/install'
15--- hooks/install 2014-11-04 11:14:37 +0000
16+++ hooks/install 2014-11-13 02:55:22 +0000
17@@ -4,9 +4,7 @@
18
19 . inc/common
20
21-PPA="ce-infrastructure/capomastro"
22-
23-apt-add-repository --yes ppa:${PPA} && apt-get update && apt-get install --assume-yes --force-yes --no-install-recommends capomastro
24+apt-add-repository --yes "${repository}" && apt-get update && apt-get install --assume-yes --force-yes capomastro
25
26 # https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=747270
27 apt-get --assume-yes --force-yes --purge remove python-librabbitmq
28
29=== modified file 'inc/common'
30--- inc/common 2014-11-04 19:35:30 +0000
31+++ inc/common 2014-11-13 02:55:22 +0000
32@@ -4,6 +4,7 @@
33
34 # development
35 debug=$(config-get debug)
36+repository=$(config-get repository)
37
38 # apache's site config
39 port=$(config-get port)

Subscribers

People subscribed via source and target branches