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

Failed GH Action reports as skipped, bypassing merge checks #7126

Closed
mhofman opened this issue Mar 6, 2023 · 2 comments · Fixed by #7127 or #8267
Closed

Failed GH Action reports as skipped, bypassing merge checks #7126

mhofman opened this issue Mar 6, 2023 · 2 comments · Fixed by #7127 or #8267
Assignees
Labels
bug Something isn't working tooling repo-wide infrastructure vaults_triage DO NOT USE

Comments

@mhofman
Copy link
Member

mhofman commented Mar 6, 2023

Describe the bug

#7000 landed even though the deployment integrations tests failed. Similarly #6816 has the same failure (bug triggered by #7000), but managed to merge.

It seems that the deployment GH action failed with an out of disk runner error, and the overall job failed, but the relevant deployment-test check is simply marked as skipped, which is an acceptable state to continue merging. I am not sure how to update the mergify rules to handle this situation.

To Reproduce

Expected behavior

PRs not merging

Platform Environment

GH Actions / CI

Screenshots

Error seen on the action run details:

System.IO.IOException: No space left on device : '/home/runner/runners/2.302.1/_diag/Worker_20230305-232305-utc.log'
   at System.IO.RandomAccess.WriteAtOffset(SafeFileHandle handle, ReadOnlySpan`1 buffer, Int64 fileOffset)
   at System.IO.Strategies.BufferedFileStreamStrategy.FlushWrite()
   at System.IO.StreamWriter.Flush(Boolean flushStream, Boolean flushEncoder)
   at System.Diagnostics.TextWriterTraceListener.Flush()
   at GitHub.Runner.Common.HostTraceListener.WriteHeader(String source, TraceEventType eventType, Int32 id)
   at GitHub.Runner.Common.HostTraceListener.TraceEvent(TraceEventCache eventCache, String source, TraceEventType eventType, Int32 id, String message)
   at System.Diagnostics.TraceSource.TraceEvent(TraceEventType eventType, Int32 id, String message)
   at GitHub.Runner.Worker.Worker.RunAsync(String pipeIn, String pipeOut)
   at GitHub.Runner.Worker.Program.MainAsync(IHostContext context, String[] args)
System.IO.IOException: No space left on device : '/home/runner/runners/2.302.1/_diag/Worker_20230305-232305-utc.log'
   at System.IO.RandomAccess.WriteAtOffset(SafeFileHandle handle, ReadOnlySpan`1 buffer, Int64 fileOffset)
   at System.IO.Strategies.BufferedFileStreamStrategy.FlushWrite()
   at System.IO.StreamWriter.Flush(Boolean flushStream, Boolean flushEncoder)
   at System.Diagnostics.TextWriterTraceListener.Flush()
   at GitHub.Runner.Common.HostTraceListener.WriteHeader(String source, TraceEventType eventType, Int32 id)
   at GitHub.Runner.Common.HostTraceListener.TraceEvent(TraceEventCache eventCache, String source, TraceEventType eventType, Int32 id, String message)
   at System.Diagnostics.TraceSource.TraceEvent(TraceEventType eventType, Int32 id, String message)
   at GitHub.Runner.Common.Tracing.Error(Exception exception)
   at GitHub.Runner.Worker.Program.MainAsync(IHostContext context, String[] args)
Unhandled exception. System.IO.IOException: No space left on device : '/home/runner/runners/2.302.1/_diag/Worker_20230305-232305-utc.log'
   at System.IO.RandomAccess.WriteAtOffset(SafeFileHandle handle, ReadOnlySpan`1 buffer, Int64 fileOffset)
   at System.IO.Strategies.BufferedFileStreamStrategy.FlushWrite()
   at System.IO.StreamWriter.Flush(Boolean flushStream, Boolean flushEncoder)
   at System.Diagnostics.TextWriterTraceListener.Flush()
   at System.Diagnostics.TraceSource.Flush()
   at GitHub.Runner.Common.TraceManager.Dispose(Boolean disposing)
   at GitHub.Runner.Common.TraceManager.Dispose()
   at GitHub.Runner.Common.HostContext.Dispose(Boolean disposing)
   at GitHub.Runner.Common.HostContext.Dispose()
   at GitHub.Runner.Worker.Program.Main(String[] args)
@mhofman mhofman added bug Something isn't working tooling repo-wide infrastructure labels Mar 6, 2023
@mhofman
Copy link
Member Author

mhofman commented Mar 7, 2023

The condition match in #7127 doesn't seem to help

@mhofman mhofman reopened this Mar 7, 2023
@ivanlei ivanlei added the vaults_triage DO NOT USE label Mar 20, 2023
@mhofman mhofman linked a pull request Aug 29, 2023 that will close this issue
@mhofman
Copy link
Member Author

mhofman commented Aug 29, 2023

This should be properly fixed by #8267

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tooling repo-wide infrastructure vaults_triage DO NOT USE
Projects
None yet
3 participants