Merge lp:~fginther/pbuilderjenkins/add-proposed-hook into lp:pbuilderjenkins

Proposed by Francis Ginther
Status: Merged
Approved by: Francis Ginther
Approved revision: 139
Merged at revision: 139
Proposed branch: lp:~fginther/pbuilderjenkins/add-proposed-hook
Merge into: lp:pbuilderjenkins
Diff against target: 32 lines (+20/-0)
2 files modified
debian/changelog (+6/-0)
hooks/D09add_proposed (+14/-0)
To merge this branch: bzr merge lp:~fginther/pbuilderjenkins/add-proposed-hook
Reviewer Review Type Date Requested Status
Para Siva (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+281144@code.launchpad.net

Commit message

Add a hook script for adding the proposed pocket.

Description of the change

Add a hook script for adding the proposed pocket.

This was tested as a stand-alone hook here:
* http://s-jenkins.ubuntu-ci:8080/job/thumbnailer-devel-xenial-armhf-ci-fjg/1/consoleText
* http://s-jenkins.ubuntu-ci:8080/job/thumbnailer-devel-xenial-amd64-ci-fjg/2/consoleText

In both jobs, you can observer that some of the dependencies were satisfied by packages in 'xenial-proposed'.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Para Siva (psivaa) wrote :

This looks good to me. Just curious about the fact that this enables -proposed for all the archives. i.e. main, universe, restricted and multiverse. I also see another hook to enable universe, D09add_universe, although I dont see it as being in any of the configs in cupstream2distro.
Won't there be any other dependency issues when this D09add_proposed hook being used without the other?

review: Needs Information
Revision history for this message
Francis Ginther (fginther) wrote :

Psivaa, that's a valid point. This does assume that all these archives are intended to be enabled. But using all of these archives is how we've been setting up our chroots [1] in CI and recommending to others [2].

The fact that there is also a D09add_universe hook is a bit misleading. The CI build environment was more restrictive at one time, which is why it exists. For our current environment, it's now obsolete. If it comes up that some user needs a more restrictive version of this hook, they can use this as a template for building exactly what they need.

[1] - http://s-jenkins.ubuntu-ci:8080/job/create-debian-chroot/configure
[2] - https://wiki.canonical.com/InformationInfrastructure/Jenkaas/ExampleJobsDebian#Creating_the_debian_chroots

Revision history for this message
Para Siva (psivaa) wrote :

Thanks Francis, for the explanation. That makes sense. Since we have old and new documentation recommending all the archives, this should not confuse the users.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2015-07-28 20:03:22 +0000
3+++ debian/changelog 2015-12-21 20:06:42 +0000
4@@ -1,3 +1,9 @@
5+pbuilderjenkins (0.48) xenial; urgency=medium
6+
7+ * Add hook, D09add_proposed, for adding the proposed pocket.
8+
9+ -- Francis Ginther <francis.ginther@canonical.com> Mon, 21 Dec 2015 14:04:44 -0600
10+
11 pbuilderjenkins (0.47) wily; urgency=medium
12
13 * Support handling of multiple coverage xml files for cobertura.
14
15=== added file 'hooks/D09add_proposed'
16--- hooks/D09add_proposed 1970-01-01 00:00:00 +0000
17+++ hooks/D09add_proposed 2015-12-21 20:06:42 +0000
18@@ -0,0 +1,14 @@
19+#!/bin/bash
20+
21+set -e
22+
23+. /etc/lsb-release
24+
25+ARCH=$(dpkg --print-architecture)
26+if [ $ARCH == 'armhf' ]; then
27+ ARCHIVE_SOURCE=http://ports.ubuntu.com/ubuntu-ports/
28+else
29+ ARCHIVE_SOURCE=http://archive.ubuntu.com/ubuntu/
30+fi
31+
32+echo deb ${ARCHIVE_SOURCE} ${DISTRIB_CODENAME}-proposed main restricted universe multiverse >> /etc/apt/sources.list

Subscribers

People subscribed via source and target branches