Bug #1327473 “Don't use mutables as default args” : Bugs : oslo.messaging

Revision history for this message

ChangBo Guo(gcb) (glongwave) wrote on 2014-06-07:

#1

This is fix for Nova:
https://review.openstack.org/#/c/98339/

Changed in nova:
assignee:	nobody → ChangBo Guo(gcb) (glongwave)
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-07: Fix proposed to oslo.messaging (master)

#2

Fix proposed to branch: master
Review: https://review.openstack.org/98569

Changed in oslo.messaging:
assignee:	nobody → ChangBo Guo(gcb) (glongwave)
status:	New → In Progress

Andrew Laski (alaski) on 2014-06-17

Changed in nova:
importance:	Undecided → Low

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-18: Fix merged to nova (master)

#3

Reviewed: https://review.openstack.org/98339
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=0bea84ac20fe498bd08f7212a0017196c8cb0812
Submitter: Jenkins
Branch: master

commit 0bea84ac20fe498bd08f7212a0017196c8cb0812
Author: ChangBo Guo(gcb) <email address hidden>
Date: Sat Jun 7 10:49:20 2014 +0800

Removes the use of mutables as default args

    Passing mutable objects as default args is a known Python pitfall.
    We'd better avoid this. This commit changes mutable default args with
    None, then use 'arg = arg or {}', 'arg = arg or []'. For unit code which
    doesn't use the args , just set with None. This commit also adds hacking
    check.

Closes-Bug: #1327473
Change-Id: I5a8492bf8ffef8e000b13b6bdfaef1968b96f816

Changed in nova:
status:	In Progress → Fix Committed

Mark McLoughlin (markmc) on 2014-06-20

Changed in oslo.messaging:
importance:	Undecided → Low

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-23: Fix merged to oslo.messaging (master)

#4

Reviewed: https://review.openstack.org/98569
Committed: https://git.openstack.org/cgit/openstack/oslo.messaging/commit/?id=821ee096a6b1dd897892af90b47738051d0fa8f4
Submitter: Jenkins
Branch: master

commit 821ee096a6b1dd897892af90b47738051d0fa8f4
Author: ChangBo Guo(gcb) <email address hidden>
Date: Sat Jun 7 12:06:08 2014 +0800

Removes the use of mutables as default args

Passing mutable objects as default args is a known Python pitfall.
We'd better avoid this.

Change-Id: I67cc0774a65886ef9fce0b72e52157b622248a85
Closes-Bug: #1327473

Changed in oslo.messaging:
status:	In Progress → Fix Committed

Liusheng (liusheng) on 2014-07-01

Changed in cinder:
assignee:	nobody → Liusheng (liusheng)
Changed in heat:
assignee:	nobody → Liusheng (liusheng)
Changed in neutron:
assignee:	nobody → Liusheng (liusheng)

OpenStack Infra (hudson-openstack) on 2014-07-01

Changed in neutron:
status:	New → In Progress
Changed in cinder:
status:	New → In Progress
Changed in heat:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-07-02: Fix merged to heat (master)

#5

Reviewed: https://review.openstack.org/103707
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=5541f4390333aae07879d9996c72f2095eda19a3
Submitter: Jenkins
Branch: master

commit 5541f4390333aae07879d9996c72f2095eda19a3
Author: liu-sheng <email address hidden>
Date: Tue Jul 1 10:58:22 2014 +0800

Use None instead of mutables in method params defaults

    Mutables in the method params defaults might cause errors and
    that's why it's anti-pattern in most of the cases and should be
    removed.

Change-Id: I8bc284f12ce72082a0482410ec2c20c2fc087a4b
Closes-Bug: #1327473

Changed in heat:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-07-03: Fix proposed to python-heatclient (master)

#6

Fix proposed to branch: master
Review: https://review.openstack.org/104423

Changed in python-heatclient:
assignee:	nobody → Liusheng (liusheng)
status:	New → In Progress

Kyle Mestery (mestery) on 2014-07-06

Changed in neutron:
milestone:	none → juno-2
importance:	Undecided → Low

Revision history for this message

Elena Ezhova (eezhova) wrote on 2014-07-07:

#7

Related bug in Tempest: https://bugs.launchpad.net/tempest/+bug/1330322

Revision history for this message

Mark McClain (markmcclain) wrote on 2014-07-07:

#8

I question the need for this . While mutable arguments can be a potential source for issues, an outright ban seems like a terrible idea. There are valid patterns (i.e. argument designed to intentionally accumulate or default read-only empty iterables). Reviewers should be be able to judge whether the usage of the the arguments is proper versus installing a blanket ban.

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-07-08: Change abandoned on neutron (master)

#9

Change abandoned by liusheng (<email address hidden>) on branch: master
Review: https://review.openstack.org/103713

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-07-08: Change abandoned on cinder (master)

#10

Change abandoned by liusheng (<email address hidden>) on branch: master
Review: https://review.openstack.org/103756

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-07-08: Change abandoned on python-heatclient (master)

#11

Change abandoned by liusheng (<email address hidden>) on branch: master
Review: https://review.openstack.org/104423

OpenStack Infra (hudson-openstack) on 2014-07-16

Changed in cinder:
assignee:	Liusheng (liusheng) → Ivan Kolodyazhny (e0ne)

Kyle Mestery (mestery) on 2014-07-16

Changed in neutron:
milestone:	juno-2 → none

Jay Bryant (jsbryant) on 2014-07-21

Changed in cinder:
importance:	Undecided → Low

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-07-22: Fix merged to cinder (master)

#12

Reviewed: https://review.openstack.org/106872
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=1cfb7da7ce3f93765ba2e267897b5b7825a0ba75
Submitter: Jenkins
Branch: master

commit 1cfb7da7ce3f93765ba2e267897b5b7825a0ba75
Author: Ivan Kolodyazhny <email address hidden>
Date: Tue Jul 15 00:13:54 2014 +0300

Use immutable default values for args

Default mutable values (e.g. arg1=[], arg2={}) could have side effects
in Python. So using None as a default value is safer.

Added hacking checks for default mutable args.

Closes: bug #1327473

Change-Id: I7055e534b91df794550de6c3b243324e582d4430

Changed in cinder:
status:	In Progress → Fix Committed

Russell Bryant (russellb) on 2014-07-23

Changed in nova:
milestone:	none → juno-2
status:	Fix Committed → Fix Released

Russell Bryant (russellb) on 2014-07-23

Changed in heat:
milestone:	none → juno-2
status:	Fix Committed → Fix Released

Russell Bryant (russellb) on 2014-07-23

Changed in oslo.messaging:
milestone:	none → juno-2
status:	Fix Committed → Fix Released

Russell Bryant (russellb) on 2014-07-24

Changed in cinder:
milestone:	none → juno-2
status:	Fix Committed → Fix Released

OpenStack Infra (hudson-openstack) on 2014-09-25

Changed in neutron:
assignee:	Liusheng (liusheng) → Kevin Benton (kevinbenton)

Thierry Carrez (ttx) on 2014-10-16

Changed in nova:
milestone:	juno-2 → 2014.2

Thierry Carrez (ttx) on 2014-10-16

Changed in cinder:
milestone:	juno-2 → 2014.2

Thierry Carrez (ttx) on 2014-10-16

Changed in heat:
milestone:	juno-2 → 2014.2

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-11-21: Fix merged to neutron (master)

#13

Reviewed: https://review.openstack.org/121403
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=bc61dd0e3493b0e5c5b2e0e1e9d16b65532c6c4a
Submitter: Jenkins
Branch: master

commit bc61dd0e3493b0e5c5b2e0e1e9d16b65532c6c4a
Author: Kevin Benton <email address hidden>
Date: Sun Sep 14 01:53:52 2014 -0700

Remove default dictionary from function def

Removes an unnecessary dictionary created in the default
parameters definition for the execute function in ip_lib.

Partial-Bug: #1327473
Change-Id: I9b60e87d1043aa67a14675dad2185395df940301

Kevin Benton (kevinbenton) on 2015-04-17

Changed in neutron:
assignee:	Kevin Benton (kevinbenton) → nobody

Jonathan Bell (jbell-f) on 2015-05-04

Changed in neutron:
assignee:	nobody → Jonathan Bell (jbell-f)

Steve Baker (steve-stevebaker) on 2015-05-11

Changed in python-heatclient:
importance:	Undecided → Low
status:	In Progress → Fix Released

Bertrand Lallau (bertrand-lallau) on 2015-10-30

Changed in octavia:
assignee:	nobody → Bertrand Lallau (bertrand-lallau)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-10-30: Fix proposed to octavia (master)

#14

Fix proposed to branch: master
Review: https://review.openstack.org/240455

Changed in octavia:
status:	New → In Progress

Revision history for this message

Reedip (reedip-banerjee-deactivatedaccount) wrote on 2015-11-02:

#15

Changing the user as there is no progress for a long time

Changed in neutron:
assignee:	Jonathan Bell (jbell-f) → Reedip (reedip-banerjee)

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-11-02: Fix proposed to neutron (master)

#16

Fix proposed to branch: master
Review: https://review.openstack.org/240748

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-11-02: Fix proposed to neutron-lbaas (master)

#17

Fix proposed to branch: master
Review: https://review.openstack.org/240756

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-11-02: Change abandoned on neutron (master)

#18

Change abandoned by Reedip (<email address hidden>) on branch: master
Review: https://review.openstack.org/240748

Revision history for this message

Reedip (reedip-banerjee-deactivatedaccount) wrote on 2015-11-02:

#19

Fixed in Neutron : https://review.openstack.org/240259

Changed in neutron:
assignee:	Reedip (reedip-banerjee) → nobody

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-11-05: Fix merged to neutron-lbaas (master)

#20

Reviewed: https://review.openstack.org/240756
Committed: https://git.openstack.org/cgit/openstack/neutron-lbaas/commit/?id=0aac2d029a7ed9bff903a00a4ab194e277dfe06c
Submitter: Jenkins
Branch: master

commit 0aac2d029a7ed9bff903a00a4ab194e277dfe06c
Author: Reedip Banerjee <email address hidden>
Date: Mon Nov 2 10:24:44 2015 +0530

Avoiding usage of mutables as default arguments

    Using Mutables objects as default arguments in function
    definition is a known Python pitfall. This patch fixes
    the issue in neutron-lbaas.

Change-Id: Idb1cccbebec5dfedde04ee474036e67569406caa
Closes-Bug: #1327473

Changed in neutron:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-11-06: Fix proposed to neutron-lbaas (stable/liberty)

#21

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/242534

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-11-09: Fix merged to octavia (master)

#22

Reviewed: https://review.openstack.org/240455
Committed: https://git.openstack.org/cgit/openstack/octavia/commit/?id=76e1e02e15bf83a7ac4ca01bfb2f6665975829c9
Submitter: Jenkins
Branch: master

commit 76e1e02e15bf83a7ac4ca01bfb2f6665975829c9
Author: Bertrand Lallau <email address hidden>
Date: Fri Oct 30 08:11:46 2015 +0100

Removes the use of mutables as default args

    Passing mutable objects as default args is a known Python pitfall.
    We'd better avoid this. This commit changes mutable default args with
    None, then use 'arg = arg or {}', 'arg = arg or []'. For unit code which
    doesn't use the args , just set with None. This commit also adds hacking
    check.

Change-Id: Ib4f195c9c00ca2c49579f9d62648bff7c8109fcf
Closes-Bug: #1327473

Changed in octavia:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-11-22: Fix merged to neutron-lbaas (stable/liberty)

#23

Reviewed: https://review.openstack.org/242534
Committed: https://git.openstack.org/cgit/openstack/neutron-lbaas/commit/?id=a9ad175f0be9b34287d41fdb76ee6a103e328bb6
Submitter: Jenkins
Branch: stable/liberty

commit a9ad175f0be9b34287d41fdb76ee6a103e328bb6
Author: Reedip Banerjee <email address hidden>
Date: Mon Nov 2 10:24:44 2015 +0530

Avoiding usage of mutables as default arguments

    Using Mutables objects as default arguments in function
    definition is a known Python pitfall. This patch fixes
    the issue in neutron-lbaas.

    Change-Id: Idb1cccbebec5dfedde04ee474036e67569406caa
    Closes-Bug: #1327473
    (cherry picked from commit 0aac2d029a7ed9bff903a00a4ab194e277dfe06c)

tags:

added: in-stable-liberty

Revision history for this message

Thierry Carrez (ttx) wrote on 2015-12-03: Fix included in openstack/neutron-lbaas 8.0.0.0b1

#24

This issue was fixed in the openstack/neutron-lbaas 8.0.0.0b1 development milestone.

Doug Hellmann (doug-hellmann) on 2015-12-03

Changed in neutron:
status:	Fix Committed → Fix Released

Revision history for this message

Doug Hellmann (doug-hellmann) wrote on 2015-12-07: Fix included in openstack/neutron-lbaas 7.0.1

#25

This issue was fixed in the openstack/neutron-lbaas 7.0.1 release.

Michael Johnson (johnsom) on 2016-03-01

Changed in octavia:
status:	Fix Committed → Fix Released

oslo.messaging

Don't use mutables as default args

Bug Description

Other bug subscribers

Remote bug watches

	Status	Importance	Assigned to	Milestone
Cinder	Fix Released	Low	Ivan Kolodyazhny	Cinder 2014.2 "juno"
OpenStack Compute (nova)	Fix Released	Low	ChangBo Guo(gcb)	OpenStack Compute (nova) 2014.2 "juno"
OpenStack Heat	Fix Released	Undecided	Liusheng	OpenStack Heat 2014.2 "juno"
neutron	Fix Released	Low	Unassigned
octavia	Fix Released	Undecided	Bertrand Lallau
oslo.messaging	Fix Released	Low	ChangBo Guo(gcb)	oslo.messaging juno-2 "j2"
python-heatclient	Fix Released	Low	Liusheng