Closed Bug 1405808 Opened 2 years ago Closed 2 years ago

AttributeError: 'NoneType' object has no attribute 'get' in mozlog


(Testing :: General, defect)

57 Branch
Not set


(firefox58 fixed)

Tracking Status
firefox58 --- fixed


(Reporter: kats, Assigned: ahal)



(1 file)

Lots of this output:

11:54:27     INFO -  mozlog.structuredlog: Failure calling log handler:
11:54:27     INFO -  Traceback (most recent call last):
11:54:27     INFO -    File "C:\slave\test\build\venv\lib\site-packages\mozlog\", line 237, in _handle_log
11:54:27     INFO -      handler(data)
11:54:27     INFO -    File "C:\slave\test\build\venv\lib\site-packages\mozlog\handlers\", line 94, in __call__
11:54:27     INFO -      formatted = self.formatter(data)
11:54:27     INFO -    File "C:\slave\test\build\venv\lib\site-packages\mozlog\handlers\", line 63, in __call__
11:54:27     INFO -      return self.inner(item)
11:54:27     INFO -    File "C:\slave\test\build\venv\lib\site-packages\mozlog\formatters\", line 17, in __call__
11:54:27     INFO -      rv = BaseFormatter.__call__(self, data)
11:54:27     INFO -    File "C:\slave\test\build\venv\lib\site-packages\mozlog\", line 71, in __call__
11:54:27     INFO -      return handler(data)
11:54:27     INFO -    File "C:\slave\test\build\venv\lib\site-packages\mozlog\formatters\", line 43, in test_status
11:54:27     INFO -      return self._output_test(item["test"], item["subtest"], item)
11:54:27     INFO -    File "C:\slave\test\build\venv\lib\site-packages\mozlog\formatters\", line 29, in _output_test
11:54:27     INFO -      "group": self.groups.get(test, ''),
11:54:27     INFO -  AttributeError: 'NoneType' object has no attribute 'get'
Interesting. I'll put up a patch to fix this particular error, but it looks like there might be another underlying issue. This can only happen if a test_status/test_end was logged without a suite_start message.

This is *supposed* to be prevented from happening by mozlog, but there is a bug which I haven't been able to land which means sometimes this check gets missed (bug 1372565). That said, all our suites should be logging suite_start. Is this some kind of new suite/mochitest flavor or something?
Assignee: nobody → ahalberstadt
Nope, it's a reftest run on windows, with webrender forced on. I did make a slight tweak to the reftest harness [1] but it shouldn't affect the suite_start message. I've done a lot of try pushes with these same changes before and this is the first time I'm seeing this. So maybe intermittent? I assumed it was a regression somewhere else in the code but I don't know if it's reproducible.

Oh there are some reftest intermittents where suite_start doesn't get logged, bug 1324961. I even think it spiked recently. So this bug is likely a byproduct of that, but might as well fix it to keep the logs clean.
Comment on attachment 8915578 [details]
Bug 1405808 - [mozlog] Default manifest groups to empty dict in errorsummary formatter,
Attachment #8915578 - Flags: review?(gbrown) → review+
Pushed by
[mozlog] Default manifest groups to empty dict in errorsummary formatter, r=gbrown
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.