[recipes.py] move remote arg parsing to its module

[recipes.py] move remote arg parsing to its module

R=dnj@chromium.org, phajdan.jr@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2839353003
Committed: https://github.com/luci/recipes-py/commit/0b1247f8506739b0c6d6adc1f8145cb4d0d3cda8

[iannucci, 2017-04-27 00:40:56]

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

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

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-04-27 00:43:10]

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

[commit-bot: I haz the power, 2017-04-27 00:43:11]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/35c5ad38a80be910)

[iannucci, 2017-04-27 02:38:58]

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

[commit-bot: I haz the power, 2017-04-27 02:39:02]

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-04-27 02:46:31]

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

[commit-bot: I haz the power, 2017-04-27 02:46:31]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/35c61864c4578410)

[iannucci, 2017-04-27 03:29:05]

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

[commit-bot: I haz the power, 2017-04-27 03:29:14]

Dry run: CQ is trying da patch.

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

[iannucci, 2017-04-27 03:34:19]

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

[commit-bot: I haz the power, 2017-04-27 03:34:30]

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-04-27 03:41:19]

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

[commit-bot: I haz the power, 2017-04-27 03:41:20]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/35c64de9e1b40110)

[iannucci, 2017-04-27 04:21:27]

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

[commit-bot: I haz the power, 2017-04-27 04:21:35]

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-04-27 04:25:46]

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

[commit-bot: I haz the power, 2017-04-27 04:25:47]

Dry run: This issue passed the CQ dry run.

[dnj, 2017-04-27 16:38:40]

lgtm w/ comment

https://codereview.chromium.org/2839353003/diff/60002/recipes.py
File recipes.py (right):

https://codereview.chromium.org/2839353003/diff/60002/recipes.py#newcode350
recipes.py:350: bare_command=False,  # don't call postprocess_func, don't use
package_deps
add a TODO here, remove this when "remote" command is removed.

https://codereview.chromium.org/2839353003/diff/60002/recipes.py#newcode383
recipes.py:383: from recipe_engine import remote
Since you're adding a lot of optional flags, can we document "func",
"postprocess_func", "bare_command", etc. here?

[iannucci, 2017-04-29 06:29:06]

documented in a later cl :)

[iannucci, 2017-04-29 06:30:32]

The CQ bit was checked by iannucci@chromium.org

[commit-bot: I haz the power, 2017-04-29 06:30:34]

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-04-29 06:33:20]

CQ is committing da patch.

Bot data: {"patchset_id": 60002, "attempt_start_ts": 1493447432327500,
"parent_rev": "3cc22baad1f5cad990a9b0f7428ccf3627063e5f", "commit_rev":
"0b1247f8506739b0c6d6adc1f8145cb4d0d3cda8"}

[commit-bot: I haz the power, 2017-04-29 06:33:24]

Description was changed from

==========
[recipes.py] move remote arg parsing to its module

R=dnj@chromium.org, phajdan.jr@chromium.org
BUG=
==========

to

==========
[recipes.py] move remote arg parsing to its module

R=dnj@chromium.org, phajdan.jr@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2839353003
Committed:
https://github.com/luci/recipes-py/commit/0b1247f8506739b0c6d6adc1f8145cb4d0d...
==========

[commit-bot: I haz the power, 2017-04-29 06:33:25]

Committed patchset #5 (id:60002) as
https://github.com/luci/recipes-py/commit/0b1247f8506739b0c6d6adc1f8145cb4d0d...