wpt-importer: Don't run git cl set-close when there are no try job results.

wpt-importer: Don't run git cl set-close when there are no try job results.

Originally in the do_auto_update method of the importer, I added
a section:

	if not try_results:
		_log.error('No try job results.')
		self.git_cl.run(['set-close']) return False

The purpose of this was to abort and indicate failure if something went
completely wrong with the CQ; I was expecting that try_results should
contain CQ bot try results like:

	{
		'builder_name': 'linux_chromium_rel_ng', 'status':
		'COMPLETED', 'result': 'SUCCESS',
	}, ...

But apparently this isn't the case, although I'm not sure why.
This CL replaces that section with just logging the try_results list
and continuing, which I believe should work as intened anyway and
will help us understand what try_results actually contains.

BUG=702761
Review-Url: https://codereview.chromium.org/2827823002
Cr-Commit-Position: refs/heads/master@{#465445}
Committed: https://chromium.googlesource.com/chromium/src/+/7865cc51cb1ebe2c072f62f7de98723e0f326c59

[qyearsley, 2017-04-18 22:37:24]

qyearsley@chromium.org changed reviewers:
+ jeffcarp@chromium.org

[qyearsley, 2017-04-18 22:42:27]

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

[commit-bot: I haz the power, 2017-04-18 22:43:16]

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

[jeffcarp, 2017-04-18 23:41:34]

lgtm

[qyearsley, 2017-04-18 23:42:12]

The CQ bit was unchecked by qyearsley@chromium.org

[qyearsley, 2017-04-18 23:42:14]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2017-04-18 23:42:32]

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

[qyearsley, 2017-04-19 00:15:11]

The CQ bit was unchecked by qyearsley@chromium.org

[qyearsley, 2017-04-19 00:15:13]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2017-04-19 00:15:40]

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-19 00:42:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492560913734760, "parent_rev":
"fef30214eb9d49ab66f9fe227199fb53e9c40892", "commit_rev":
"7865cc51cb1ebe2c072f62f7de98723e0f326c59"}

[commit-bot: I haz the power, 2017-04-19 00:42:10]

Description was changed from

==========
wpt-importer: Don't run git cl set-close when there are no try job results.

Originally in the do_auto_update method of the importer, I added
a section:

	if not try_results:
		_log.error('No try job results.')
		self.git_cl.run(['set-close']) return False

The purpose of this was to abort and indicate failure if something went
completely wrong with the CQ; I was expecting that try_results should
contain CQ bot try results like:

	{
		'builder_name': 'linux_chromium_rel_ng', 'status':
		'COMPLETED', 'result': 'SUCCESS',
	}, ...

But apparently this isn't the case, although I'm not sure why.
This CL replaces that section with just logging the try_results list
and continuing, which I believe should work as intened anyway and
will help us understand what try_results actually contains.

BUG=702761
==========

to

==========
wpt-importer: Don't run git cl set-close when there are no try job results.

Originally in the do_auto_update method of the importer, I added
a section:

	if not try_results:
		_log.error('No try job results.')
		self.git_cl.run(['set-close']) return False

The purpose of this was to abort and indicate failure if something went
completely wrong with the CQ; I was expecting that try_results should
contain CQ bot try results like:

	{
		'builder_name': 'linux_chromium_rel_ng', 'status':
		'COMPLETED', 'result': 'SUCCESS',
	}, ...

But apparently this isn't the case, although I'm not sure why.
This CL replaces that section with just logging the try_results list
and continuing, which I believe should work as intened anyway and
will help us understand what try_results actually contains.

BUG=702761

Review-Url: https://codereview.chromium.org/2827823002
Cr-Commit-Position: refs/heads/master@{#465445}
Committed:
https://chromium.googlesource.com/chromium/src/+/7865cc51cb1ebe2c072f62f7de98...
==========

[commit-bot: I haz the power, 2017-04-19 00:42:14]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/7865cc51cb1ebe2c072f62f7de98...