Open Stack

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

On 12/08/14 17:12, Henry Gessau wrote:
> On 8/12/2014 10:27 AM, Ihar Hrachyshka wrote:
>> as per [1], Cisco Nexus ML2 plugin requires a patched version of 
>> ncclient from github. I wonder:
>> 
>> - - whether this information is still current;
> 
> Please see: https://review.openstack.org/112175
> 
> But we need to do backports before updating the wiki.

Thanks for the link!

> 
>> - - why don't we depend on ncclient thru our requirements.txt
>> file.
> 
> Do we want to have requirements on things that are only used by a
> specific vendor plugin? So far it has worked by vendor-specific
> documentation instructing to manually install the requirement, or
> vendor-tailored deployment tools/scripts.
> 

In downstream, it's hard to maintain all plugin dependencies if they
are not explicitly mentioned in e.g. requirements.txt. Red Hat ships
those plugins (with no commercial support or testing done on our
side), and we didn't know that to make the plugin actually useable, we
need to install that ncclient module until a person from Cisco
reported the issue to us. We don't usually monitor random wiki pages
to get an idea what we need to package and depend on. :)

I think we should have every third party module that we directly use
in requirements.txt. We have code in the tree that imports ncclient
(btw is it unit tested?), so I think it's enough to make that
dependency explicit.

Now, maybe putting the module into requirements.txt is an overkill
(though I doubt it). In that case, we could be interested in getting
the info in some other centralized way.

/Ihar
-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.22 (Darwin)

iQEcBAEBCgAGBQJT6lSAAAoJEC5aWaUY1u57rk8IAKWBqBAJ+DChAkcU/hzs70o5
dqTKm1y5dtLpebSckjLuTb568nd1ShghCaqEQbck4U01g6aDg1hWyWzm2wF2FUyG
PtkYHZRSnKlqyAN7J2PU/Ak7uvTr51UfVKFzqc1hfLujY+SGSlzIjKeucXgjatts
TYIq53xz69y9+9GE/XxX0NpD1ROeaOwaj884WFUI5sIwKWvTjur929o58grym1Hb
bncQUc3wSY1Mtp6OdvwxZJ0MEmlC3t8ukykAUSkv1fBU6xSYo/nLmpGYeHn3o3GQ
icNJXTZbJ/z3oAktbTol1DCxHkKKKruMBqCZcxmxniAdV+l1yNSkZUlAqYwuy3E=
=nI7E
-----END PGP SIGNATURE-----