[openstack-dev] [Manila] ALLOWED_EXTRA_MISSING is cover.sh
Clinton.Knight at netapp.com
Wed Feb 10 18:30:21 UTC 2016
Hi, John. This is but one reason the coverage job doesn¹t vote; it has
other known issues. It is primarily a convenience tool that lets core
reviewers know if they should look more deeply into unit test coverage.
For a new driver such as yours, I typically pull the code and check
coverage for each new file in PyCharm rather than relying on the coverage
job. Feel free to propose enhancements to the job, though.
On 2/10/16, 1:02 PM, "John Spray" <jspray at redhat.com> wrote:
>I noticed that the coverage script is enforcing a hard limit of 4 on
>the number of extra missing lines introduced. We have a requirement
>that new drivers have 90% unit test coverage, which the ceph driver
>meets, but it's tripping up on that absolute 4 line limit.
>What do folks think about tweaking the script to do a different
>calculation, like identifying new files and permitting 10% of the line
>count of the new files to be missed? Otherwise I think the 90% target
>is going to continually conflict with the manila-coverage CI task.
>OpenStack Development Mailing List (not for usage questions)
>Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
More information about the OpenStack-dev