← Back to team overview

cloud-init-dev team mailing list archive

Re: [Merge] ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master

 

thoughts ? disagree?

Diff comments:

> diff --git a/doc/rtd/topics/tests.rst b/doc/rtd/topics/tests.rst
> index 00c63c6..c487867 100644
> --- a/doc/rtd/topics/tests.rst
> +++ b/doc/rtd/topics/tests.rst
> @@ -182,6 +182,18 @@ Executing tests has three options:
>  * ``verify`` given a directory of test data, run the Python unit tests on
>    it to generate results.
>  
> +Dependencies
> +-------------------
> +In order to execute the integration tests the following must available:
> +
> +* tox
> +
> +or
> +
> +* python3
> +* lxd
> +* pylxd version 2.1.3

i dont care *too*  much about this... if i have to use a specific version of pylxd, i'm forced to virtualenv or *something* and thus "just use tox"

additionally, we have to remember to change this '2.1.3'' string whenever we update 'tox.ini'.

And I think in theory at least we wouldnt specifically depend on lxd at all (probably need some way to configure the lxd end point)

> +
>  Run
>  ---
>  The first example will provide a complete end-to-end run of data


-- 
https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/319617
Your team cloud init development team is requested to review the proposed merge of ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master.


References