swarming: Fix named cache support.

swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
  - Reshuffled the swarming.py trigger option grouping to differentiate between
    properties and request metadata.
- Add integration smoke test to ensure it actually work.
- Fix named cache bug, it didn't work: free_space wasn't initialized so all the
  named caches were deleted after use if min_free_space wasn't None.
- made min_free_space == 0 use the normal path.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
Review-Url: https://codereview.chromium.org/2636993002
Committed: https://github.com/luci/luci-py/commit/83e5bed8895f64f184dd994d721420835c9cbb64

[M-A Ruel, 2017-01-16 22:10:38]

Description was changed from

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
- Add integration smoke test to ensure it actually work.
- Fix bug where it didn't work.
- Add logging to help diagnose problems.

BUG=
==========

to

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
- Add integration smoke test to ensure it actually work.
- Fix bug where it didn't work.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
==========

[M-A Ruel, 2017-01-16 22:10:38]

maruel@chromium.org changed reviewers:
+ vadimsh@chromium.org

[M-A Ruel, 2017-01-16 22:12:38]

Patchset #1 (id:1) has been deleted

[M-A Ruel, 2017-01-16 22:14:42]

Description was changed from

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
- Add integration smoke test to ensure it actually work.
- Fix bug where it didn't work.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
==========

to

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
  - Reshuffled the swarming.py trigger option grouping to differentiate between
    properties and request metadata.
- Add integration smoke test to ensure it actually work.
- Fix bugs where it didn't work.
  - free_space wasn't initialized.
- made min_free_space == 0 use the normal path.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
==========

[M-A Ruel, 2017-01-16 22:15:54]

Description was changed from

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
  - Reshuffled the swarming.py trigger option grouping to differentiate between
    properties and request metadata.
- Add integration smoke test to ensure it actually work.
- Fix bugs where it didn't work.
  - free_space wasn't initialized.
- made min_free_space == 0 use the normal path.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
==========

to

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
  - Reshuffled the swarming.py trigger option grouping to differentiate between
    properties and request metadata.
- Add integration smoke test to ensure it actually work.
- Fix named cache bug, it didn't work: free_space wasn't initialized so all the
  named caches were deleted after use if min_free_space wasn't None.
- made min_free_space == 0 use the normal path.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
==========

[M-A Ruel, 2017-01-17 17:31:36]

Description was changed from

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
  - Reshuffled the swarming.py trigger option grouping to differentiate between
    properties and request metadata.
- Add integration smoke test to ensure it actually work.
- Fix named cache bug, it didn't work: free_space wasn't initialized so all the
  named caches were deleted after use if min_free_space wasn't None.
- made min_free_space == 0 use the normal path.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
==========

to

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
  - Reshuffled the swarming.py trigger option grouping to differentiate between
    properties and request metadata.
- Add integration smoke test to ensure it actually work.
- Fix named cache bug, it didn't work: free_space wasn't initialized so all the
  named caches were deleted after use if min_free_space wasn't None.
- made min_free_space == 0 use the normal path.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
==========

[Vadim Sh., 2017-01-17 23:20:23]

lgtm

[M-A Ruel, 2017-01-17 23:26:01]

The CQ bit was checked by maruel@chromium.org

[commit-bot: I haz the power, 2017-01-17 23:26:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-18 00:01:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-18 00:01:05]

Try jobs failed on following builders:
  Luci-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/33c7ae60edeb5010)

[M-A Ruel, 2017-01-18 00:43:46]

The CQ bit was checked by maruel@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-18 00:43:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-18 00:48:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-18 00:48:48]

Dry run: This issue passed the CQ dry run.

[M-A Ruel, 2017-01-18 00:51:12]

On 2017/01/18 00:01:05, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   Luci-py Presubmit on luci.infra.try (JOB_FAILED,
> https://luci-milo.appspot.com/swarming/task/33c7ae60edeb5010)

Did a small fix in unit test swarming_test.py

[M-A Ruel, 2017-01-18 00:51:16]

The CQ bit was checked by maruel@chromium.org

[M-A Ruel, 2017-01-18 00:51:17]

The patchset sent to the CQ was uploaded after l-g-t-m from vadimsh@chromium.org
Link to the patchset: https://codereview.chromium.org/2636993002/#ps40001
(title: "Fix swarming_test.py")

[commit-bot: I haz the power, 2017-01-18 00:51:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-18 00:56:04]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1484700676817990,
"parent_rev": "951dbc724ef339510031621026f97230720be925", "commit_rev":
"83e5bed8895f64f184dd994d721420835c9cbb64"}

[commit-bot: I haz the power, 2017-01-18 00:56:08]

Description was changed from

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
  - Reshuffled the swarming.py trigger option grouping to differentiate between
    properties and request metadata.
- Add integration smoke test to ensure it actually work.
- Fix named cache bug, it didn't work: free_space wasn't initialized so all the
  named caches were deleted after use if min_free_space wasn't None.
- made min_free_space == 0 use the normal path.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290
==========

to

==========
swarming: Fix named cache support.

- Add support for named cache in swarming.py so it can be used via the CLI.
  - Reshuffled the swarming.py trigger option grouping to differentiate between
    properties and request metadata.
- Add integration smoke test to ensure it actually work.
- Fix named cache bug, it didn't work: free_space wasn't initialized so all the
  named caches were deleted after use if min_free_space wasn't None.
- made min_free_space == 0 use the normal path.
- Add logging to help diagnose problems.

R=vadimsh@chromium.org
BUG=667290

Review-Url: https://codereview.chromium.org/2636993002
Committed:
https://github.com/luci/luci-py/commit/83e5bed8895f64f184dd994d721420835c9cbb64
==========

[commit-bot: I haz the power, 2017-01-18 00:56:09]

Committed patchset #2 (id:40001) as
https://github.com/luci/luci-py/commit/83e5bed8895f64f184dd994d721420835c9cbb64