Merge ~nacc/git-ubuntu:apt-repo into git-ubuntu:master

Proposed by Nish Aravamudan
Status: Merged
Approved by: Nish Aravamudan
Approved revision: 83b80ef55316b9ebac16d20b77f7e764682cafb0
Merge reported by: Nish Aravamudan
Merged at revision: 83b80ef55316b9ebac16d20b77f7e764682cafb0
Proposed branch: ~nacc/git-ubuntu:apt-repo
Merge into: git-ubuntu:master
Prerequisite: ~racb/git-ubuntu:apt-repo
Diff against target: 99 lines (+63/-0)
2 files modified
gitubuntu/apt_repo_test.py (+4/-0)
snap/snapcraft.yaml (+59/-0)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
Robie Basak Pending
Review via email: mp+340069@code.launchpad.net

Description of the change

Make jenkins happy

To post a comment you must log in.
Revision history for this message
Nish Aravamudan (nacc) wrote :

I haven't yet verified this is perfect for still using the system gpg and not the snapped one, but I will do that tomorrow (or you can).

Revision history for this message
Nish Aravamudan (nacc) wrote :

But it does pass the self-test on my machine (and i found a quirk of the self-test, if you run it from a importer Git repository, the coverage check fails).

Revision history for this message
Nish Aravamudan (nacc) wrote :

On Tue, Feb 27, 2018 at 7:45 PM, Nish Aravamudan
<email address hidden> wrote:
> But it does pass the self-test on my machine (and i found a quirk of the self-test, if you run it from a importer Git repository, the coverage check fails).

I think this is because of --cov=gitubuntu and if . is also a
gitubuntu package, then it can't determine which you mean.

Revision history for this message
Server Team CI bot (server-team-bot) wrote :

FAILED: Continuous integration, rev:aa014cbc4d544ef8b872138535afb0437caf68c4
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/329/
Executed test runs:
    SUCCESS: VM Setup
    SUCCESS: Build
    FAILED: Unit Tests

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/329/rebuild

review: Needs Fixing (continuous-integration)
~nacc/git-ubuntu:apt-repo updated
83b80ef... by Nish Aravamudan

apt_repo_test: gpg should call the snapped gpg-agent

This ensures self-compatibility and ideally not affecting the running
user's gpg-agent.

The order of arguments to gpg matters, hence where this option is added.

Revision history for this message
Nish Aravamudan (nacc) wrote :

Just pushed a fixup commit for the gpg-agent stuff. Presuming this passes, I will land it after squashing the changes odwn.

Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:aa014cbc4d544ef8b872138535afb0437caf68c4
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/331/
Executed test runs:
    SUCCESS: VM Setup
    SUCCESS: Build
    SUCCESS: Unit Tests
    SUCCESS: Integration Tests
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/331/rebuild

review: Approve (continuous-integration)
Revision history for this message
Nish Aravamudan (nacc) wrote :

Merged after cleanup as 48249b21607fdfbb80af9d53e8d0b1375d8778c1

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/gitubuntu/apt_repo_test.py b/gitubuntu/apt_repo_test.py
2index 3488125..c17818f 100644
3--- a/gitubuntu/apt_repo_test.py
4+++ b/gitubuntu/apt_repo_test.py
5@@ -7,6 +7,7 @@ import urllib.error
6
7 import pkg_resources
8 import pytest
9+import shutil
10
11 import gitubuntu.apt_repo as target
12
13@@ -22,6 +23,9 @@ def trustedkeys():
14 'gpg',
15 '--no-default-keyring',
16 '--keyring', trustedkeys_path,
17+ '--agent-program', shutil.which(
18+ 'gpg-agent', path=os.getenv('PATH', os.defpath)
19+ ),
20 '--import',
21 pkg_resources.resource_filename(
22 'gitubuntu', 'apt_repo_test/pubkey.asc'
23diff --git a/snap/snapcraft.yaml b/snap/snapcraft.yaml
24index 9bf85a3..38ec2d2 100644
25--- a/snap/snapcraft.yaml
26+++ b/snap/snapcraft.yaml
27@@ -257,6 +257,57 @@ parts:
28 stage-packages:
29 - distro-info
30 after: [python3]
31+ libgpg-error:
32+ plugin: autotools
33+ source: https://gnupg.org/ftp/gcrypt/libgpg-error/libgpg-error-1.27.tar.bz2
34+ source-type: tar
35+ configflags: [--prefix=/usr]
36+ stage:
37+ - -usr/share/info
38+ libgcrypt:
39+ plugin: autotools
40+ source: https://gnupg.org/ftp/gcrypt/libgcrypt/libgcrypt-1.8.1.tar.bz2
41+ source-type: tar
42+ configflags: [--prefix=/usr]
43+ stage:
44+ - -usr/share/info
45+ libassuan:
46+ plugin: autotools
47+ source: https://gnupg.org/ftp/gcrypt/libassuan/libassuan-2.5.1.tar.bz2
48+ source-type: tar
49+ configflags: [--prefix=/usr]
50+ after: [libgpg-error]
51+ stage:
52+ - -usr/share/info
53+ libksba:
54+ plugin: autotools
55+ source: https://gnupg.org/ftp/gcrypt/libksba/libksba-1.3.5.tar.bz2
56+ source-type: tar
57+ configflags: [--prefix=/usr]
58+ stage:
59+ - -usr/share/info
60+ npth:
61+ plugin: autotools
62+ source: https://gnupg.org/ftp/gcrypt/npth/npth-1.5.tar.bz2
63+ source-type: tar
64+ configflags: [--prefix=/usr]
65+ stage:
66+ - -usr/share/info
67+ gnupg2:
68+ plugin: autotools
69+ source: https://gnupg.org/ftp/gcrypt/gnupg/gnupg-2.2.4.tar.bz2
70+ source-type: tar
71+ configflags:
72+ - --prefix=/usr
73+ - --enable-gpg2-is-gpg
74+ after:
75+ - libgpg-error
76+ - libgcrypt
77+ - libassuan
78+ - libksba
79+ - npth
80+ stage:
81+ - -usr/share/info
82 devscripts:
83 plugin: nil
84 stage-packages:
85@@ -310,6 +361,14 @@ parts:
86 - -lib/python3.6/site-packages/oauth*
87 - -lib/python3.6/site-packages/launchpadlib*
88 - -usr/lib/python3.6
89+ - -usr/bin/gpg-error*
90+ - -usr/share/aclocal/gpg-error.m4
91+ - -usr/bin/dumpsexp
92+ - -usr/bin/hmac256
93+ - -usr/bin/libgcrypt-config
94+ - -usr/bin/mpicalc
95+ - -usr/include/gcrypt.h
96+ - -usr/share/aclocal/libgcrypt.m4
97 prime:
98 - -usr/share/doc
99 install: |

Subscribers

People subscribed via source and target branches