New issue
Advanced search Search tips

Issue 684070 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Mark successful test run on Swarming leaving temporary files behind as failed

Project Member Reported by mar...@chromium.org, Jan 23 2017

Issue description

Goal:
Make sure no successful[1] test execution leaves temporary file behind. While Swarming bots (mostly) handle this problem just fine, it's still a pain for developers and just poor hygiene.

[1] This issue isn't about failing tests. There can't be guarantee about a failing test; this is out of scope here.


Background:
Previous efforts already made sure that all tests are runnable from a read only directory ( issue 116251 ) and do not depend on full path. This is an continuation of the test execution self-containment effort. Swarming bots already report the number of temporary files in their TMP directory for maintenance purpose.


Action Items:
- Add TMP files detection in run_isolated.py after a task completes successfully. Report it by injecting this condition in the stdout.
- File bugs and fix any issue discovered.
- Once all problems are detected and fixed, enforce this by also setting exit:255.
 
Status: Available (was: Untriaged)
Project Member

Comment 2 by sheriffbot@chromium.org, Jun 20 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by mar...@chromium.org, Jun 20 2018

Labels: -Hotlist-Recharge-Cold
Status: WontFix (was: Untriaged)
We use a temporary tmp dir. That has been working good enough for cooperating tasks.

Sign in to add a comment