Multiple 'atom_name' given to _cached_get
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
taskflow |
Fix Released
|
High
|
Joshua Harlow |
Bug Description
Seems like the formatter has a case where a duplicate argument name will be provided:
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
2015-09-16 19:04:46.672 9698 ERROR taskflow.
We should fix this:
Changed in taskflow: | |
importance: | Undecided → High |
assignee: | nobody → Joshua Harlow (harlowja) |
Changed in taskflow: | |
status: | New → In Progress |
Changed in taskflow: | |
milestone: | none → 1.22.0 |
status: | Fix Committed → Fix Released |
Reviewed: https:/ /review. openstack. org/224335 /git.openstack. org/cgit/ openstack/ taskflow/ commit/ ?id=f8624a136b7 7c68dbf3e75406f 2903586cf0762b
Committed: https:/
Submitter: Jenkins
Branch: master
commit f8624a136b77c68 dbf3e75406f2903 586cf0762b
Author: Joshua Harlow <email address hidden>
Date: Wed Sep 16 15:22:25 2015 -0700
Fix '_cache_get' multiple keyword argument name overlap
The argument 'atom_name' is also used by the fetch function
so when it is provided a conflict occurs and this ends badly.
To avoid this capture the needed variables used for fetching
a cached value into a functools.partial object and use that instead
of passing further arguments.
Closes-Bug: #1496608
Change-Id: Ic012f7687037bf 876d041c4bc62b3 f6606a8a845