On Tue, Dec 12, 2017 at 11:04 AM Irit Goihman <igoihman@redhat.com> wrote:
Hi,
I pushed two patches fixing some of the issues:

https://gerrit.ovirt.org/#/c/85318/
https://gerrit.ovirt.org/#/c/85316/

Thanks for the quick response!

I sent this for the libvirt events, turns out the code is mostly correct, assuming
that libvirt does not add new arguments to the callback api, but pylint cannot
verify this.
https://gerrit.ovirt.org/#/c/85351/

With this we have only these errors:

************* Module vdsm.v2v
00:07:29.177 E:1368,27: Instance of 'closing' has no 'read' member (no-member)

This is handled by https://gerrit.ovirt.org/#/c/85318/
but I think we can do a better fix *without* disabling any warning, 
see the comment in:
https://gerrit.ovirt.org/#/c/85318/3/lib/vdsm/v2v.py@1369

00:07:29.182 ************* Module vdsm.tool.configurator
00:07:29.186 E:118,12: No value for argument 'action' in function call (no-value-for-parameter)
00:07:29.191 E:158,12: No value for argument 'action' in function call (no-value-for-parameter)
00:07:29.195 E:193,12: No value for argument 'action' in function call (no-value-for-parameter)
00:07:29.198 E:215,12: No value for argument 'action' in function call (no-value-for-parameter)

This is handled by https://gerrit.ovirt.org/#/c/85316/
but I don't understand the fix. The code seems fine
and the fix make it worse.

00:07:29.201 ************* Module vdsm.virt.vm_migrate_hook
00:07:29.204 E:199, 4: No value for argument 'domain' in function call (no-value-for-parameter)
00:07:29.206 E:199, 4: No value for argument 'event' in function call (no-value-for-parameter)
00:07:29.208 E:199, 4: No value for argument 'phase' in function call (no-value-for-parameter)

Francesco, can you look at these?

00:07:29.210 ************* Module vdsm.network.netlink.monitor
00:07:29.212 E:172,58: Instance of 'closing' has no 'poll' member (no-member)
 


On Tue, Dec 12, 2017 at 2:09 AM, Nir Soffer <nsoffer@redhat.com> wrote:
I'm trying to upgrade pylint to latest so we can enjoy latest fixes
like this:

Now we have some new errors, please check.

************* Module vdsm.clientIF
00:06:53.857 E:588,16: Possible unbalanced tuple unpacking with sequence: left side has 2 label(s), right side has 0 value(s) (unbalanced-tuple-unpacking)

587             if eventid == libvirt.VIR_DOMAIN_EVENT_ID_LIFECYCLE:
588                 event, detail = args[:-1]                                                                                                                                                 
589                 v.onLibvirtLifecycleEvent(event, detail, None)

Code is indeed incorrect, should be:

    event, detail = args[:2] 

00:06:53.859 E:593,16: Possible unbalanced tuple unpacking with sequence: left side has 1 label(s), right side has 0 value(s) (unbalanced-tuple-unpacking)
00:06:53.861 E:596,16: Possible unbalanced tuple unpacking with sequence: left side has 4 label(s), right side has 0 value(s) (unbalanced-tuple-unpacking)
00:06:53.863 E:599,16: Possible unbalanced tuple unpacking with sequence: left side has 5 label(s), right side has 0 value(s) (unbalanced-tuple-unpacking)
00:06:53.867 E:610,16: Possible unbalanced tuple unpacking with sequence: left side has 1 label(s), right side has 0 value(s) (unbalanced-tuple-unpacking)
00:06:53.869 E:615,16: Possible unbalanced tuple unpacking with sequence: left side has 1 label(s), right side has 0 value(s) (unbalanced-tuple-unpacking)
00:06:53.872 E:618,16: Possible unbalanced tuple unpacking with sequence: left side has 4 label(s), right side has 0 value(s) (unbalanced-tuple-unpacking)
00:06:53.873 ************* Module vdsm.v2v
00:06:53.874 E:1368,27: Instance of 'closing' has no 'read' member (no-member)
00:06:53.876 ************* Module vdsm.tool.configurator
00:06:53.878 E:118,12: No value for argument 'action' in function call (no-value-for-parameter)
00:06:53.880 E:158,12: No value for argument 'action' in function call (no-value-for-parameter)
00:06:53.881 E:193,12: No value for argument 'action' in function call (no-value-for-parameter)
00:06:53.883 E:215,12: No value for argument 'action' in function call (no-value-for-parameter)
00:06:53.885 ************* Module vdsm.virt.vm_migrate_hook
00:06:53.886 E:199, 4: No value for argument 'domain' in function call (no-value-for-parameter)
00:06:53.888 E:199, 4: No value for argument 'event' in function call (no-value-for-parameter)
00:06:53.890 E:199, 4: No value for argument 'phase' in function call (no-value-for-parameter)
00:06:53.892 ************* Module vdsm.network.netlink.monitor
00:06:53.893 E:172,58: Instance of 'closing' has no 'poll' member (no-member)

_______________________________________________
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel



--

IRIT GOIHMAN

SOFTWARE ENGINEER

EMEA VIRTUALIZATION R&D

Red Hat EMEA