New issue
Advanced search Search tips

Issue 761196 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Delete swarming work dirs for devices that were removed long ago

Project Member Reported by bpastene@chromium.org, Aug 31 2017

Issue description

^
 
Labels: -Pri-3 Pri-2
This keeps triggering disk-full alerts (bug 765453). I should really fix this.
I wonder if cleanup_disk solution is applicable here: https://chrome-internal.googlesource.com/infra/infra_internal/+/master/go/src/infra_internal/tools/cleanup_disk/cmd/cleanup_disk/main.go#95

The folder's mtime is computed recursively as the latest update on any file.

I don't remember if it runs on swarming bots by default, but I imagine it doesn't hurt to do so.

Comment 3 by bpastene@google.com, Sep 18 2017

Still happening: bug 766094, bug 766125

Will work on fix sometime this week.
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 20 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/infra_internal/+/cda600e1103444712a84d3c46113582162b54fa2

commit cda600e1103444712a84d3c46113582162b54fa2
Author: Benjamin Pastene <bpastene@chromium.org>
Date: Wed Sep 20 18:47:18 2017

Project Member

Comment 5 by bugdroid1@chromium.org, Sep 20 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/6bc18aa21990f8c2e28f32ed75851d3401337cde

commit 6bc18aa21990f8c2e28f32ed75851d3401337cde
Author: Benjamin Pastene <bpastene@chromium.org>
Date: Wed Sep 20 23:48:28 2017

Project Member

Comment 6 by bugdroid1@chromium.org, Sep 22 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/b4e2494990d998b0f29f2f759590f1b3e2b96c8a

commit b4e2494990d998b0f29f2f759590f1b3e2b96c8a
Author: Benjamin Pastene <bpastene@chromium.org>
Date: Fri Sep 22 01:27:39 2017

Project Member

Comment 7 by bugdroid1@chromium.org, Sep 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/4a71824a0c1ab29de404803733fbe43ab7575052

commit 4a71824a0c1ab29de404803733fbe43ab7575052
Author: Benjamin Pastene <bpastene@chromium.org>
Date: Fri Sep 22 21:51:13 2017

android_docker: Make chrome-bot owner of container workdirs.

Bug:  761196 
Change-Id: Iabcf4342a1725fd0f38a68815eda81fd2f866289
Reviewed-on: https://chromium-review.googlesource.com/679277
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Benjamin Pastene <bpastene@chromium.org>

[modify] https://crrev.com/4a71824a0c1ab29de404803733fbe43ab7575052/infra/services/android_docker/containers.py
[modify] https://crrev.com/4a71824a0c1ab29de404803733fbe43ab7575052/infra/services/android_docker/test/containers_test.py

Project Member

Comment 8 by bugdroid1@chromium.org, Sep 25 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/63905248029c9674cfc7126752cce9adce5d1fb8

commit 63905248029c9674cfc7126752cce9adce5d1fb8
Author: Benjamin Pastene <bpastene@chromium.org>
Date: Mon Sep 25 21:20:27 2017

Project Member

Comment 9 by bugdroid1@chromium.org, Sep 25 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/puppet/+/67fb9a4a18d2279e3ff63f956e661f610ea86e1c

commit 67fb9a4a18d2279e3ff63f956e661f610ea86e1c
Author: Benjamin Pastene <bpastene@chromium.org>
Date: Mon Sep 25 21:29:33 2017

Status: Fixed (was: Assigned)
I think the cleanup_disk cron now removes them.

Sign in to add a comment