1527610 - [mozlog] Library fails after python 3 update; TypeError: vars() argument must have _dict_ attribute

Status: RESOLVED FIXED

(Testing :: Mozbase, defect, P3)

Description

[AndreiH]

After installing the new version of mozlog library when running the following part of code:

import mozlog
mozlog.commandline.setup_logging('mozversion', None, {})

We receive the following error:

File "C:\Users\silviu.checherita\.virtualenvs\iris2-62hnA7_a\lib\site-packages\mozlog\commandline.py", line 263, in setup_logging
args = vars(args)
TypeError: vars() argument must have _dict_ attribute

The error is resulted from: https://searchfox.org/mozilla-central/source/testing/mozbase/mozlog/mozlog/commandline.py#263

Instead of args = vars(args) we can use args = args.__dict__

Comment 1

[Andrew Halberstadt [:ahal]]

Are you using mozlog with python 3 here?

I think the bug here is that dict's don't have iteritems in Python 3, that's a really bad way to test for a dict. We should change the condition to if not isinstance(args, dict).

Comment 2

[AndreiH]

(In reply to Andrew Halberstadt [:ahal] from comment #1)

Are you using mozlog with python 3 here?

I think the bug here is that dict's don't have iteritems in Python 3, that's a really bad way to test for a dict. We should change the condition to if not isinstance(args, dict).

Yes, we are running mozlog with Python 3, and it hits some code that was not hit by the unit tests when library was updated.
I currently don't know what is the fix here, but I am going to try with what you suggested, should work.

Comment 3

[AndreiH]

Attachment: Bug 1527610 - [mozlog] Library fails after python 3 update; TypeError: vars() argument must have _dict_ attribute

Comment 4

[AndreiH]

I have updated as you suggested, and now the bug did not reproduce anymore.
I have created a patch ready for review.
Thanks !

Comment 5

[shruthi0898]

Hi @jgraham,

Can you please tell if there is anything to be done to correct this bug?

Thanks and Regards,
Shruthi

Comment 6

[AndreiH]

(In reply to shruthi0898 from comment #5)

Hi @jgraham,

Can you please tell if there is anything to be done to correct this bug?

Thanks and Regards,
Shruthi

A patch was already created and accepted.
It just needs to be merged into central.

Comment 7

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:ahutusoru, could you have a look please?

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+1]]

This is more a question for Andrew.

Comment 9

[Andrew Halberstadt [:ahal]]

For future reference, if there's no needinfo or request on phabricator, best to assume I'm not going to see it. Bugmail/phabricator comments are very easy to miss.

Comment 10

[Andrew Halberstadt [:ahal]]

Sorry Andrei, looks like your patch bit-rotted. Could you rebase and push it to phabricator again?

Comment 11

[AndreiH]

Sorry for the late response.
Unfortunately I cannot update this on a daily basis.
If someone can take this and update it. Thank you !

Comment 12

[Nikki Sharpley [:nikkis]]

Hey Andrei & ahal. I'd be happy to update this one! As the version number will be updated to 4.1 with https://bugzilla.mozilla.org/show_bug.cgi?id=1550565, should I just stick with the changes, or update the versions to 4.1.1? I guess it depends on the order in which the patches both land?

Comment 13

[Nikki Sharpley [:nikkis]]

Attachment: Bug 1527610 - [mozlog] Library fails after python 3 update; TypeError: vars() argument must have _dict_ attribute

Updated from Differential D20458.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2180deaa2c68
[mozlog] Library fails after python 3 update; TypeError: vars() argument must have _dict_ attribute r=ahal

Comment 15

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/2180deaa2c68