cloud-init-dev team mailing list archive

[Joshua Powers <josh.powers@xxxxxxxxxxxxx>]

Review: Approve

LGTM one comment below.

Diff comments:

> diff --git a/tests/cloud_tests/releases.yaml b/tests/cloud_tests/releases.yaml
> index 45deb58..c8dd142 100644
> --- a/tests/cloud_tests/releases.yaml
> +++ b/tests/cloud_tests/releases.yaml
> @@ -143,19 +143,6 @@ releases:
>              alias: zesty
>              setup_overrides: null
>              override_templates: false
> -    yakkety:
> -        # EOL: Jul 2017
> -        default:
> -            enabled: true

If you wanted to leave this in you could just mark enabled to false. That way it could stay in here and someone could just flip the bit if they wanted to run tests, however, I don't see any reason to keep this in other than for posterity.

> -            feature_groups:
> -                - base
> -                - debian_base
> -                - ubuntu_specific
> -        lxd:
> -            sstreams_server: https://cloud-images.ubuntu.com/daily
> -            alias: yakkety
> -            setup_overrides: null
> -            override_templates: false
>      xenial:
>          # EOL: Apr 2021
>          default:


-- 
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/328098
Your team cloud-init commiters is requested to review the proposed merge of ~smoser/cloud-init:fix/disable-yakkety-tests into cloud-init:master.