Merge ~smoser/cloud-init:fix/disable-yakkety-tests into cloud-init:master

Proposed by Scott Moser on 2017-07-26
Status: Merged
Approved by: Scott Moser on 2017-07-26
Approved revision: a31eaa3c2bffddbe97fd2d3139e9dce4de8f38b6
Merged at revision: a31eaa3c2bffddbe97fd2d3139e9dce4de8f38b6
Proposed branch: ~smoser/cloud-init:fix/disable-yakkety-tests
Merge into: cloud-init:master
Diff against target: 24 lines (+0/-13)
1 file modified
tests/cloud_tests/releases.yaml (+0/-13)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Needs Fixing on 2017-07-26
Joshua Powers (community) Approve on 2017-07-26
cloud-init commiters 2017-07-26 Pending
Review via email: mp+328098@code.launchpad.net

Commit Message

tests: remove 'yakkety' from releases as it is EOL.

This is not strictly necessary, but since yakkety is no longer supported
we will remove it from the releases.yaml file.

To post a comment you must log in.
Joshua Powers (powersj) wrote :

LGTM one comment below.

review: Approve

FAILED: Continuous integration, rev:9df66c069866f8e9df6224b3a2944637b5d0252a
https://jenkins.ubuntu.com/server/job/cloud-init-ci/95/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Unit & Style Tests
    SUCCESS: Ubuntu LTS: Build
    SUCCESS: Ubuntu LTS: Integration
    FAILED: CentOS 6 & 7: Build & Test

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

review: Needs Fixing (continuous-integration)
Scott Moser (smoser) wrote :

i think best to pull the 'release' out so we dont' have inadvertant bit-rot on a hunk of text that we're not ever going to support again.
Ie, if its 'feature groups' needed change it would increase future burden to keep it here.

Joshua Powers (powersj) wrote :

Dropping sounds good to me then.

Scott Moser (smoser) wrote :

I'm ignoring the failure above as that is now fixed on trunk.
8a5296c41db45be3a172862f324ad44e732a2250
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/328101

so, pulling this. thanks for review.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tests/cloud_tests/releases.yaml b/tests/cloud_tests/releases.yaml
2index 45deb58..c8dd142 100644
3--- a/tests/cloud_tests/releases.yaml
4+++ b/tests/cloud_tests/releases.yaml
5@@ -143,19 +143,6 @@ releases:
6 alias: zesty
7 setup_overrides: null
8 override_templates: false
9- yakkety:
10- # EOL: Jul 2017
11- default:
12- enabled: true
13- feature_groups:
14- - base
15- - debian_base
16- - ubuntu_specific
17- lxd:
18- sstreams_server: https://cloud-images.ubuntu.com/daily
19- alias: yakkety
20- setup_overrides: null
21- override_templates: false
22 xenial:
23 # EOL: Apr 2021
24 default:

Subscribers

People subscribed via source and target branches

to all changes: