commit 73a6ad9a13d4b6c2d9efe7d9ec142514fc6904f2
Author: Paul Michali <email address hidden>
Date: Tue Feb 16 15:45:54 2016 -0500
Use constraints on all targets.
The neutron-vpnaas repo already has converted to using constraints for
targets. However, the TC recommendation is to use constraints on the
original job names (pep8, py27,...), instead of using the constrained
names (e.g. pep8-constraints).
A separate commit (I34f0b355703fe0133f160afd4e32f2272d41ecaa) restored
the use of the non-constraint NAMED jobs for Liberty and newer.
This commit modifies those non-constraints targets so that they use
constraints. A manual check was made that each target used the same
package versions for both constraints and non-constraints targets.
Then, after both constraints and non-constraints jobs passed in Jenkins
(before and after the project-config commit was upstreamed), the constraints
based targets were removed from tox.ini.
Note: Found out that infra doesn't support constraints based cover and
releasenotes. Added TODO to these targets so that they can be updated,
if and when constraints are supported.
Note: In the gate hook, config_for_vpn_func_testing.sh calls Neutron's
config_for_func_testing.sh, which will setup files in the venv. To prevent
that script from modifying the VENV variable to have -constraints suffix
and store files in the wrong venv, we turn off USE_CONSTRAINT_ENV, prior
to calling the Neutron script.
Once this is upstreamed, we can backport this fix to Liberty to complete
the effort for this repo.
Reviewed: https:/ /review. openstack. org/280926 /git.openstack. org/cgit/ openstack/ neutron- vpnaas/ commit/ ?id=73a6ad9a13d 4b6c2d9efe7d9ec 142514fc6904f2
Committed: https:/
Submitter: Jenkins
Branch: master
commit 73a6ad9a13d4b6c 2d9efe7d9ec1425 14fc6904f2
Author: Paul Michali <email address hidden>
Date: Tue Feb 16 15:45:54 2016 -0500
Use constraints on all targets.
The neutron-vpnaas repo already has converted to using constraints for
targets. However, the TC recommendation is to use constraints on the
original job names (pep8, py27,...), instead of using the constrained
names (e.g. pep8-constraints).
A separate commit (I34f0b355703fe 0133f160afd4e32 f2272d41ecaa) restored
the use of the non-constraint NAMED jobs for Liberty and newer.
This commit modifies those non-constraints targets so that they use
constraints. A manual check was made that each target used the same
package versions for both constraints and non-constraints targets.
Then, after both constraints and non-constraints jobs passed in Jenkins
(before and after the project-config commit was upstreamed), the constraints
based targets were removed from tox.ini.
Note: Found out that infra doesn't support constraints based cover and
releasenotes. Added TODO to these targets so that they can be updated,
if and when constraints are supported.
Note: In the gate hook, config_ for_vpn_ func_testing. sh calls Neutron's for_func_ testing. sh, which will setup files in the venv. To prevent
config_
that script from modifying the VENV variable to have -constraints suffix
and store files in the wrong venv, we turn off USE_CONSTRAINT_ENV, prior
to calling the Neutron script.
Once this is upstreamed, we can backport this fix to Liberty to complete
the effort for this repo.
Change-Id: I4a2410ab80e601 8f1b65b2288ae66 312eb49cb27 133f160afd4e32f 2272d41ecaa
Depends-On: I34f0b355703fe0
Partial-Bug: #1522503