infra_libs/logs: Check if log directory exists.

infra_libs/logs: Check if log directory exists.

Check if a log directory exists before probing it and attempting to
create a temporary file. This check prevents a condition being
encountered on Windows 10 where E:\ is mapped to a removable disk and,
consqeuently, the attempt to blindly create a file on it causes the
script to hang pending removable media insertion.

BUG=chromium:566094
TEST=local
  - Ran a test case on the afflicted system, hangs forever.
  - Applied fix, system works as expected.

Committed: https://chromium.googlesource.com/infra/infra/+/da4c7862a6d9f7c2b7ac439852440e9de240bf27

[dnj, 2015-12-14 23:26:07]

dnj@chromium.org changed reviewers:
+ agable@chromium.org, dsansome@chromium.org, kjellander@chromium.org

[dnj, 2015-12-14 23:26:08]

PTAL

[agable, 2015-12-14 23:46:10]

lgtm

[dnj, 2015-12-14 23:52:04]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2015-12-14 23:52:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1527843002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1527843002/1

[dnj, 2015-12-14 23:52:23]

Description was changed from

==========
infra_libs/logs: Check if log directory exists.

Check if a log directory exists before probing it and attempting to
create a temporary file. This check prevents a condition being
encountered on Windows 10 where E:\ is mapped to a removable disk and,
consqeuently, the attempt to blindly create a file on it causes the
script to hand pending removable media insertion.

BUG=chromium:566094
TEST=local
  - Ran a test case on the afflicted system, hangs forever.
  - Applied fix, system works as expected.
==========

to

==========
infra_libs/logs: Check if log directory exists.

Check if a log directory exists before probing it and attempting to
create a temporary file. This check prevents a condition being
encountered on Windows 10 where E:\ is mapped to a removable disk and,
consqeuently, the attempt to blindly create a file on it causes the
script to hang pending removable media insertion.

BUG=chromium:566094
TEST=local
  - Ran a test case on the afflicted system, hangs forever.
  - Applied fix, system works as expected.
==========

[dnj, 2015-12-14 23:57:51]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2015-12-14 23:58:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1527843002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1527843002/1

[commit-bot: I haz the power, 2015-12-14 23:59:16]

Description was changed from

==========
infra_libs/logs: Check if log directory exists.

Check if a log directory exists before probing it and attempting to
create a temporary file. This check prevents a condition being
encountered on Windows 10 where E:\ is mapped to a removable disk and,
consqeuently, the attempt to blindly create a file on it causes the
script to hang pending removable media insertion.

BUG=chromium:566094
TEST=local
  - Ran a test case on the afflicted system, hangs forever.
  - Applied fix, system works as expected.
==========

to

==========
infra_libs/logs: Check if log directory exists.

Check if a log directory exists before probing it and attempting to
create a temporary file. This check prevents a condition being
encountered on Windows 10 where E:\ is mapped to a removable disk and,
consqeuently, the attempt to blindly create a file on it causes the
script to hang pending removable media insertion.

BUG=chromium:566094
TEST=local
  - Ran a test case on the afflicted system, hangs forever.
  - Applied fix, system works as expected.

Committed:
https://chromium.googlesource.com/infra/infra/+/da4c7862a6d9f7c2b7ac439852440...
==========

[commit-bot: I haz the power, 2015-12-14 23:59:17]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/infra/infra/+/da4c7862a6d9f7c2b7ac439852440...

[kjellander_chromium, 2015-12-15 06:48:09]

LGTM! Thanks.