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

CMDLineV2 task not failing when executed program terminates due to error #14219

Closed
jackdeansmith opened this issue Jan 13, 2021 · 2 comments
Closed
Assignees
Labels
Area: ABTT Akvelon Build Tasks Team area of work bug Task: CmdLine

Comments

@jackdeansmith
Copy link

Question, Bug, or Feature?
Type: Bug

Enter Task Name: CMDLineV2

Environment

  • Server: Azure Pipelines
  • Account Name: SkyDrive
  • Team Project Name: CrossPlatform
  • Build definition name: iOS Run xPlat Unit Tests
  • Agent: Hosted Mac Internal Mojave

Issue Description

This pipeline is used as a PR gate in our repo to ensure that checked in code passes our unit tests. It consists of a few steps that create a test binary, then this "Run unit tests" step that executes the test binary:

Screen Shot 2021-01-12 at 5 31 48 PM

We recently noticed that this task succeeds in cases where the test executable terminates due to an error. In this case, a segfault:
Screen Shot 2021-01-12 at 5 42 24 PM

Because of this, code that does not pass our unit tests was allowed into our repo.

Minimal Repro

I created a minimal repro by creating a pipeline with three steps:

  1. Create an executable which exits with an error by segfaulting
    Screen Shot 2021-01-12 at 5 48 17 PM

  2. Run the executable via the CmdLineV2 Task
    Screen Shot 2021-01-12 at 5 48 50 PM

  3. Run the executable via the BashV3 Task
    Screen Shot 2021-01-12 at 5 49 11 PM

When I run this pipeline, task 2 succeeds while task 3 fails. Note that the printed script contents are not the same, for the CmdLineV2 task, the script is prepended with exec.
Screen Shot 2021-01-12 at 6 01 22 PM
Screen Shot 2021-01-12 at 6 01 36 PM

Based on this and the final few debug message, I believe this is related to this PR: #13573

Task logs

Production unit test pipeline logs:
logs_8835843.zip

Logs for minimal repro:
logs_8852883.zip

@github-actions github-actions bot added Area: ABTT Akvelon Build Tasks Team area of work bug labels Jan 13, 2021
@EzzhevNikita EzzhevNikita self-assigned this Jan 19, 2021
@EzzhevNikita EzzhevNikita added the awaiting deployment Related changes are waiting for deployment to be completed label Jan 27, 2021
@EzzhevNikita
Copy link
Contributor

@jackdeansmith The fix for has been deployed. Could you please check if it works for you now?

@EzzhevNikita EzzhevNikita removed the awaiting deployment Related changes are waiting for deployment to be completed label Feb 24, 2021
@jackdeansmith
Copy link
Author

@EzzhevNikita I just tried again and everything is working as expected. Thanks for getting to the bottom of this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: ABTT Akvelon Build Tasks Team area of work bug Task: CmdLine
Projects
None yet
Development

No branches or pull requests

3 participants