Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jenkins jobs on GitHub are marked failed when at least one really fails and others succeed - this dismays #1082

Closed
kuvaldini opened this issue Jun 14, 2021 · 3 comments
Labels
iroha1 The legacy version of Iroha.

Comments

@kuvaldini
Copy link
Contributor

kuvaldini commented Jun 14, 2021

What we see

изображение

изображение

What should be

GitHub must show succeeded jobs as succeeded, failed as failed. Jobs statuses must correspond.


polite ping to @kamilsa @BulatSaif @lebdron

@kuvaldini kuvaldini changed the title Jenkins marks all jobs failed when at least one fails - this dismays Jenkins jobs on GitHub are marked as failed when at least one really fails and others succeed - this dismays Jun 14, 2021
@kuvaldini kuvaldini changed the title Jenkins jobs on GitHub are marked as failed when at least one really fails and others succeed - this dismays Jenkins jobs on GitHub are marked failed when at least one really fails and others succeed - this dismays Jun 14, 2021
@BulatSaif
Copy link
Contributor

The problem is in this line (https://github.com/hyperledger/iroha/blob/main/Jenkinsfile#L70), it is using the global variable currentBuild.currentResult, any build which runs in parallel(https://github.com/hyperledger/iroha/blob/main/Jenkinsfile#L488) can edit it.
possible fix:

  1. create a global map customBuildStatus and store the build results there, and use it to notify git:
    gitNotify ("Jenkins: " + build.name, customBuildStatus[build.name]:?currentBuild.currentResult, 'FAILURE')

or

  1. Remove all git Notify and left only one, since the results are the same

@kuvaldini kuvaldini pinned this issue Jun 15, 2021
@kuvaldini
Copy link
Contributor Author

kuvaldini commented Jun 16, 2021

And the other similar problem
there are only 2 tests failed, but GitHub reports 7 failed 2 succeeded and 3 started but never finished.

Screenshots were taken from #943 and https://jenkins.soramitsu.co.jp/blue/organizations/jenkins/iroha%2Firoha-hyperledger/detail/PR-943/78/pipeline

изображение

изображение

@BulatSaif
Copy link
Contributor

3 started but never finished.

This happens then GitHub is down or not available and it didn't receive notification about the finish

@appetrosyan appetrosyan unpinned this issue Oct 12, 2022
@appetrosyan appetrosyan closed this as not planned Won't fix, can't repro, duplicate, stale Oct 21, 2022
@nxsaken nxsaken added the iroha1 The legacy version of Iroha. label May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
iroha1 The legacy version of Iroha.
Projects
None yet
Development

No branches or pull requests

4 participants