Skip to content
This repository has been archived by the owner on Jun 14, 2018. It is now read-only.

Add initial codecov integration #171

Merged
merged 7 commits into from
Feb 18, 2017
Merged

Conversation

ayj
Copy link
Contributor

@ayj ayj commented Feb 17, 2017

  • Initial integration with codecov.io and Jenkins. See https://codecov.io/gh/istio/manager for reports)
  • Race detection is temporarily disabled because istio.io/manage has some data races. Will file follow-up bugs for those.
  • Codecov is triggered on presubmit. This means the published codecov.io includes reports for the istio.io/manager/master as well as all of the pending PR. This is not a huge deal (yet) since we can differentiate reports on codecov.io's website based on commit summary.

@istio-testing
Copy link
Contributor

Jenkins job manager/presubmit passed

@istio-testing
Copy link
Contributor

Jenkins job manager/presubmit passed

@codecov-io
Copy link

codecov-io commented Feb 17, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@dcecb20). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #171   +/-   ##
=========================================
  Coverage          ?   31.47%           
=========================================
  Files             ?       19           
  Lines             ?     1757           
  Branches          ?        0           
=========================================
  Hits              ?      553           
  Misses            ?     1153           
  Partials          ?       51

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dcecb20...012271e. Read the comment docs.

@istio-testing
Copy link
Contributor

Jenkins job manager/presubmit passed

@ayj ayj changed the title [DONT_REVIEW_YET] initial code coverage integration with Jenkins Add initial codecov integration Feb 17, 2017
@ayj ayj requested a review from kyessenov February 17, 2017 23:54
@istio-testing
Copy link
Contributor

Jenkins job manager/presubmit passed

Copy link
Contributor

@kyessenov kyessenov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ayj ayj force-pushed the initial-codecov-integration branch from 70fdbe6 to 43a8a10 Compare February 18, 2017 00:00
@istio-testing
Copy link
Contributor

Jenkins job manager/presubmit passed

1 similar comment
@istio-testing
Copy link
Contributor

Jenkins job manager/presubmit passed

@ayj ayj merged commit be75ba6 into istio:master Feb 18, 2017
@ayj ayj deleted the initial-codecov-integration branch March 7, 2017 05:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants