Merge lp:~pfalcon/linaro-license-protection/configs.capture-openid-groups into lp:linaro-license-protection/configs

Proposed by Paul Sokolovsky
Status: Merged
Merged at revision: 30
Proposed branch: lp:~pfalcon/linaro-license-protection/configs.capture-openid-groups
Merge into: lp:linaro-license-protection/configs
Diff against target: 80 lines (+19/-0)
11 files modified
README (+3/-0)
apache/security/all-private-hwpack-teams.txt (+5/-0)
apache/security/allowed-ops.txt (+1/-0)
apache/security/arm-private-hwpack-teams.txt (+1/-0)
apache/security/freescale-private-hwpack-teams.txt (+1/-0)
apache/security/linaro-android-teams.txt (+2/-0)
apache/security/linaro-teams.txt (+1/-0)
apache/security/mservices.linaro.org-teams.txt (+2/-0)
apache/security/samsung-private-hwpack-teams.txt (+1/-0)
apache/security/ste-private-hwpack-teams.txt (+1/-0)
apache/security/ti-private-hwpack-teams.txt (+1/-0)
To merge this branch: bzr merge lp:~pfalcon/linaro-license-protection/configs.capture-openid-groups
Reviewer Review Type Date Requested Status
Georgy Redkozubov Approve
Данило Шеган (community) Approve
Philip Colmer Pending
Review via email: mp+153859@code.launchpad.net

This proposal supersedes a proposal from 2013-03-18.

Description of the change

BP Ref: https://blueprints.launchpad.net/linaro-infrastructure-misc/+spec/prepare-snapshots-migration-ec2

Reviewing setup Philip did on ec2-54-242-19-2.compute-1.amazonaws.com, I see that we miss in our docs step of setting up files in /etc/apache2/security/ , which are referenced by site config files. So, this patch capture those files from snapshots.linaro.org host. I also compared it with what we have on staging.snapshots.linaro.org, and there, it's fewer files, and there're some textual differences:

$ diff -ur security security2
diff -ur security/allowed-ops.txt security2/allowed-ops.txt
--- security/allowed-ops.txt 2013-03-18 17:53:53.737577210 +0200
+++ security2/allowed-ops.txt 2013-03-18 17:50:06.000000000 +0200
@@ -1 +1 @@
-Ubuntu=https://login.ubuntu.com
+Ubuntu=https://login.launchpad.net
diff -ur security/all-private-hwpack-teams.txt security2/all-private-hwpack-teams.txt
--- security/all-private-hwpack-teams.txt 2013-03-18 17:53:53.849577329 +0200
+++ security2/all-private-hwpack-teams.txt 2013-03-18 17:50:07.000000000 +0200
@@ -1,4 +1,4 @@
-linaro-landing-team-freescale
+naro-landing-team-freescale
 linaro-landing-team-samsung
 linaro-landing-team-ste
 linaro-landing-team-ti
Only in security: linaro-android-teams.txt
Only in security: mservices.linaro.org-teams.txt

Note that last hunk looks like typo.

To post a comment you must log in.
Revision history for this message
Данило Шеган (danilo) wrote :

Paul, what's the status of our move to using exclusively BUILD-INFO.txt for OpenID protection? If that's done, we should instead look at removing the apache-based openid stuff (not in this branch: just file a bug about it when needed).

Otherwise, these changes are good and we should keep the configs reflect the reality, so +1 from me.

review: Approve
Revision history for this message
Georgy Redkozubov (gesha) wrote :

Looks good to reflect current state.

review: Approve
Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

Danilo: Is started with asking to migrate just as single subdir (as required for lp:1112620, and hopefully makes sense, as it's security related, and we first want to migrate subset and validate it), and there was no response from IS to https://rt.linaro.org/Ticket/Display.html?id=865 , so yes, IMHO we should not tie these 2 migrations together, at least not at this time, because it'll be just extra constraint to keep in mind.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'README'
2--- README 2013-02-21 17:10:15 +0000
3+++ README 2013-03-18 16:22:23 +0000
4@@ -174,6 +174,9 @@
5 /srv/staging.snapshots.linaro.org/configs/apache/staging.snapshots.linaro.org
6 files to /etc/apache2/sites-available
7
8+ Copy apache/security/ directory to /etc/apache2/security (it contains OpenID group
9+ mappings, etc).
10+
11 Add appropriate NameVirtualHost directive to
12
13 /etc/apache2/sites-enabled/000-default
14
15=== added directory 'apache/security'
16=== added file 'apache/security/all-private-hwpack-teams.txt'
17--- apache/security/all-private-hwpack-teams.txt 1970-01-01 00:00:00 +0000
18+++ apache/security/all-private-hwpack-teams.txt 2013-03-18 16:22:23 +0000
19@@ -0,0 +1,5 @@
20+linaro-landing-team-freescale
21+linaro-landing-team-samsung
22+linaro-landing-team-ste
23+linaro-landing-team-ti
24+linaro-landing-team-arm
25
26=== added file 'apache/security/allowed-ops.txt'
27--- apache/security/allowed-ops.txt 1970-01-01 00:00:00 +0000
28+++ apache/security/allowed-ops.txt 2013-03-18 16:22:23 +0000
29@@ -0,0 +1,1 @@
30+Ubuntu=https://login.ubuntu.com
31
32=== added file 'apache/security/arm-private-hwpack-teams.txt'
33--- apache/security/arm-private-hwpack-teams.txt 1970-01-01 00:00:00 +0000
34+++ apache/security/arm-private-hwpack-teams.txt 2013-03-18 16:22:23 +0000
35@@ -0,0 +1,1 @@
36+linaro-landing-team-arm
37
38=== added file 'apache/security/freescale-private-hwpack-teams.txt'
39--- apache/security/freescale-private-hwpack-teams.txt 1970-01-01 00:00:00 +0000
40+++ apache/security/freescale-private-hwpack-teams.txt 2013-03-18 16:22:23 +0000
41@@ -0,0 +1,1 @@
42+linaro-landing-team-freescale
43
44=== added file 'apache/security/linaro-android-teams.txt'
45--- apache/security/linaro-android-teams.txt 1970-01-01 00:00:00 +0000
46+++ apache/security/linaro-android-teams.txt 2013-03-18 16:22:23 +0000
47@@ -0,0 +1,2 @@
48+linaro
49+linaro-android-restricted
50
51=== added file 'apache/security/linaro-teams.txt'
52--- apache/security/linaro-teams.txt 1970-01-01 00:00:00 +0000
53+++ apache/security/linaro-teams.txt 2013-03-18 16:22:23 +0000
54@@ -0,0 +1,1 @@
55+linaro
56
57=== added file 'apache/security/mservices.linaro.org-teams.txt'
58--- apache/security/mservices.linaro.org-teams.txt 1970-01-01 00:00:00 +0000
59+++ apache/security/mservices.linaro.org-teams.txt 2013-03-18 16:22:23 +0000
60@@ -0,0 +1,2 @@
61+linaro-member-services-leadership
62+canonical-sysadmins
63
64=== added file 'apache/security/samsung-private-hwpack-teams.txt'
65--- apache/security/samsung-private-hwpack-teams.txt 1970-01-01 00:00:00 +0000
66+++ apache/security/samsung-private-hwpack-teams.txt 2013-03-18 16:22:23 +0000
67@@ -0,0 +1,1 @@
68+linaro-landing-team-samsung
69
70=== added file 'apache/security/ste-private-hwpack-teams.txt'
71--- apache/security/ste-private-hwpack-teams.txt 1970-01-01 00:00:00 +0000
72+++ apache/security/ste-private-hwpack-teams.txt 2013-03-18 16:22:23 +0000
73@@ -0,0 +1,1 @@
74+linaro-landing-team-ste
75
76=== added file 'apache/security/ti-private-hwpack-teams.txt'
77--- apache/security/ti-private-hwpack-teams.txt 1970-01-01 00:00:00 +0000
78+++ apache/security/ti-private-hwpack-teams.txt 2013-03-18 16:22:23 +0000
79@@ -0,0 +1,1 @@
80+linaro-landing-team-ti

Subscribers

People subscribed via source and target branches