[Docker driver] code coverage

Bug #1777423 reported by Michał Kostrzewa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Medium
Michał Kostrzewa
OpenContrail
Fix Committed
Undecided
Unassigned

Bug Description

There is no way to easily collect code coverage stats for docker driver.

Tags: windows
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/43912
Submitter: Michal Kostrzewa (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43914
Submitter: Michal Kostrzewa (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43912
Submitter: Michal Kostrzewa (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/43914
Submitter: Michal Kostrzewa (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/43914
Committed: http://github.com/Juniper/contrail-windows-docker-driver/commit/4d4a92f05979bbf648aeedf95679edbfecbf183a
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 4d4a92f05979bbf648aeedf95679edbfecbf183a
Author: Michal Kostrzewa <email address hidden>
Date: Mon Jun 18 13:23:04 2018 +0200

Add building script.

This change is being submitted as separate change, because adding build
tags will break Windows CI.

The order of operations is:

1. Merge this change.
2. Use ./Invoke-Build.ps1 in Windows CI
3. Merge the change that adds build tags and updates ./Invoke-Build.ps1

Change-Id: I534a4086ea3b11f1a09ce37e750e506468eb3943
Partial-Bug: #1777423

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/44050
Submitter: Michal Krasnoborski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/44080
Submitter: Michal Kostrzewa (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/44050
Committed: http://github.com/Juniper/contrail-windows-docker-driver/commit/8d11cd14a3e2870b5d1e895fb04797e32727aded
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 8d11cd14a3e2870b5d1e895fb04797e32727aded
Author: Michał Krasnoborski <email address hidden>
Date: Fri Jun 22 10:21:59 2018 +0200

Add missing licenses and update year

1. Add missing licenses
2. Update the year on licenses

Change-Id: I26d49f1793618b0ff2ea89c7b7ce5fa2a059eb49
Partial-Bug: #1777423

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/44080
Submitter: Michal Kostrzewa (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/44080
Committed: http://github.com/Juniper/contrail-windows-docker-driver/commit/ea385b6fa35e92a9ee6f29bf6c8adbf13f60a69a
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit ea385b6fa35e92a9ee6f29bf6c8adbf13f60a69a
Author: Michal Kostrzewa <email address hidden>
Date: Mon Jun 25 11:13:17 2018 +0200

Build MSI installer in Invoke-Build.ps1.

It's very hard to introduce Invoke-Build.ps1 to CI without also
providing a way to build the MSI. The best way to fix it is to introduce
a way to build the MSI directly in the docker driver repository.

We create wix_new.json file for now. We change the path specified inside
it. If we modify wix.json, CI fails. We can throw out wix.json and
rename wix_new.json to wix.json later - after we start using
./Invoke-Build.ps1 in CI.

Change-Id: I3dc380ede3ad52890e4e9763569fbcd8833915fd
Partial-Bug: #1777423

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/43912
Submitter: Michal Kostrzewa (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/43912
Committed: http://github.com/Juniper/contrail-windows-docker-driver/commit/ab523a6f5a7edbaf80a5fe43453086befcb22d75
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit ab523a6f5a7edbaf80a5fe43453086befcb22d75
Author: Michal Kostrzewa <email address hidden>
Date: Mon Jun 18 13:01:27 2018 +0200

Code coverage collection.

Implements powershell scripts that wrap go tools for code coverage
statistics.

Due to how these tools work, some changes to code had to be made:
* Write dummy empty files with "_test" suffix, so that respective
modules are taken into account when calculating total code coverage
* Add build tags for unit and integration tests, because they would be
potentially run at different moments in CI

Also, since the build tags were introduced, a build script was also
implemented that uses them.

Change-Id: Idbdc30c64e17b68d9d4b4558721cd5b245421d0f
Closes-Bug: #1777423

Changed in opencontrail:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.