barry-jin commented on a change in pull request #20138:
URL: https://github.com/apache/incubator-mxnet/pull/20138#discussion_r610044704
##########
File path: ci/Jenkinsfile_utils.groovy
##########
@@ -112,7 +112,18 @@ def get_git_commit_hash() {
}
def publish_test_coverage() {
- sh "curl -s https://codecov.io/bash | bash"
+ // CodeCovs auto detection has trouble with our CIs PR validation due the
merging strategy
+ git_commit_hash = get_git_commit_hash()
+
+ if (env.CHANGE_ID) {
+ // PR execution
+ codecovArgs = "-f '*.xml' -B ${env.CHANGE_TARGET} -P ${env.CHANGE_ID} -C
${git_commit_hash}"
Review comment:
It's not necessary. I will remove it.
The strange thing is that the codecoverage xml report was generated during
the test, but codecov script didn't find the xml file.
https://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-20138/2/pipeline/283
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
