Merge ~chad.smith/cloud-init:bug/1769754-salt-minion-bionic into cloud-init:master

Proposed by Chad Smith
Status: Merged
Approved by: Chad Smith
Approved revision: 6523126b97d5532cf127bb479800cdb43e62c003
Merge reported by: Chad Smith
Merged at revision: 27283c31f4bf85f40588cfa3b31389d70ec00243
Proposed branch: ~chad.smith/cloud-init:bug/1769754-salt-minion-bionic
Merge into: cloud-init:master
Diff against target: 27 lines (+12/-2)
1 file modified
tests/cloud_tests/testcases/modules/salt_minion.yaml (+12/-2)
Reviewer Review Type Date Requested Status
Joshua Powers (community) Approve
Server Team CI bot continuous-integration Approve
Review via email: mp+347856@code.launchpad.net

Commit message

tests: fix salt_minion integration test on bionic and later

In ubuntu, the salt-minion package version 2017.7.4+dfsg1-1 or later
automatically moves any seed keys from /etc/salt/pki/minion/ to
/var/lib/salt/pki/minion/. Fix integration tests to collect
either files in either /etc/salt/pki/minion/ or
/var/lib/salt/pki/minion/.

Description of the change

for series in artful bionic cosmic; do
  tox -e citest -- run --os-name $series --test modules/salt_minion --verbose;
done

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:6523126b97d5532cf127bb479800cdb43e62c003
https://jenkins.ubuntu.com/server/job/cloud-init-ci/78/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    SUCCESS: MAAS Compatability Testing
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/78/rebuild

review: Approve (continuous-integration)
Revision history for this message
Joshua Powers (powersj) :
review: Approve
Revision history for this message
Chad Smith (chad.smith) wrote :

An upstream commit landed for this bug.

To view that commit see the following URL:
https://git.launchpad.net/cloud-init/commit/?id=27283c31

Revision history for this message
Ryan Harper (raharper) :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tests/cloud_tests/testcases/modules/salt_minion.yaml b/tests/cloud_tests/testcases/modules/salt_minion.yaml
2index c24aa17..9227147 100644
3--- a/tests/cloud_tests/testcases/modules/salt_minion.yaml
4+++ b/tests/cloud_tests/testcases/modules/salt_minion.yaml
5@@ -28,10 +28,20 @@ collect_scripts:
6 cat /etc/salt/minion_id
7 minion.pem: |
8 #!/bin/bash
9- cat /etc/salt/pki/minion/minion.pem
10+ PRIV_KEYFILE=/etc/salt/pki/minion/minion.pem
11+ if [ ! -f $PRIV_KEYFILE ]; then
12+ # Bionic and later automatically moves /etc/salt/pki/minion/*
13+ PRIV_KEYFILE=/var/lib/salt/pki/minion/minion.pem
14+ fi
15+ cat $PRIV_KEYFILE
16 minion.pub: |
17 #!/bin/bash
18- cat /etc/salt/pki/minion/minion.pub
19+ PUB_KEYFILE=/etc/salt/pki/minion/minion.pub
20+ if [ ! -f $PUB_KEYFILE ]; then
21+ # Bionic and later automatically moves /etc/salt/pki/minion/*
22+ PUB_KEYFILE=/var/lib/salt/pki/minion/minion.pub
23+ fi
24+ cat $PUB_KEYFILE
25 grains: |
26 #!/bin/bash
27 cat /etc/salt/grains

Subscribers

People subscribed via source and target branches