-
Notifications
You must be signed in to change notification settings - Fork 994
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
Badge always incorrectly displayed as "failing" #1830
Comments
Hi @mihaigalos, |
We break our elasticsearch index on workflow run results. 😭 |
Sounds good, any test to guarantee this regression will never happen again? 😅 |
During the reproduction of the bug I saw that the problem go away, and than I saw a comment written by @TingluoHuang, which further explained everything. I'm going to close this issue, feel free to comment again or reopen it if this happens again. @mihaigalos thanks again for reporting this issue and @TingluoHuang thanks for quick response and explanation! |
@ruvceskistefan, the ELS breakage is a symptom and I don't understand the root cause. Would advise against closing this issue or at least guarding against future regressions with a test. What do you think? |
Makes sense, I'll reopen it and we're going to investivate the root cause. |
From what I understand, the ELS issue is outside the scope of the runner itself. I asked around to other teams and it looks like the root cause is an int32 overflow on the search index. 😭 @mihaigalos do you mind if I close this runner issue since the issue is truly out of scope for the runner? 🙇 You can post to https://github.community/c/code-to-cloud/github-actions/41 if you need more info or RCA about the ELS issue. |
This is still manifesting with the exact same conditions as in the initial bug report. |
Hi, any updates here? This succeeded, badge is red. |
Up. |
Same here, the github badges don't work for me. |
+1 |
I wish GitHub was 100% Free/Libre and open-source software, then skilled hackers from all around the world could fix this problem. |
Same here. |
actions/runner#1830 Signed-off-by: Joe <875022+moloch--@users.noreply.github.com>
Make sure to use the branch filter in your badge URL, by default, the badge is rendered for workflow run against your repository's default branch. |
Still red with explicit branch to
|
@mihaigalos Your |
Thanks for the clarification, this is true. I therefore need the default branch or some way of telling the badge to use Also, how do you recommend implementing this^? |
We have the same problem. Our actions are ran on tags, so when switching to tag branches, it passes. |
Please report this to /~https://github.com/orgs/community/discussions/categories/actions-and-packages since the runner (this repository) itself doesn't have knowledge about the badge. |
As a note, I just recently deleted a failed workflow run on |
Love how these update regardless of what you said. Both now show no status. |
Same problem. |
I removed all previous runs on |
@mihaigalos, I'm glad that it worked for you. I'll close this issue, if you are still getting the same error you can reopen it. |
Build a blank yml build badge and apply the correct code |
Works - deleted all previously failed runs. Thanks, @mihaigalos @wireless90 |
This badge is shown as "failing" but the tasks' statuses are all green.
Describe the bug
The badge was green since CD pipeline is prechecked before running, meaning it normally should never fail. The badge for it was green and it suddenly went to red - "failing", even though all constituent tasks are succeeding.
To Reproduce
See 1st paragraph.
Expected behavior
The badge is green when all tasks successful, red when at least one fails.
Runner Version and Platform
Full yaml snapshot available here.
Simplified yaml:
OS of the machine running the runner?
Linux
The text was updated successfully, but these errors were encountered: