diff options
author | Lena Novokshonova <enovokshonova@mirantis.com> | 2014-09-19 11:56:56 +0400 |
---|---|---|
committer | Lena Novokshonova <enovokshonova@mirantis.com> | 2014-09-19 13:52:53 +0400 |
commit | b42d1415d85d308f0ef7b5c6133d1573c9f68b1f (patch) | |
tree | d578e5735fe2fb1b385bd26cffceb697bca48398 /ceilometer/hardware/pollsters/util.py | |
parent | 1ac00d4a705d0a0adc761d7f9559c69bf3748f07 (diff) | |
download | ceilometer-b42d1415d85d308f0ef7b5c6133d1573c9f68b1f.tar.gz |
Use None instead of mutables in method params default values
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: Ib1498e68ab9cc770b237a324b733c85b4a5c6839
Diffstat (limited to 'ceilometer/hardware/pollsters/util.py')
-rw-r--r-- | ceilometer/hardware/pollsters/util.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ceilometer/hardware/pollsters/util.py b/ceilometer/hardware/pollsters/util.py index d2f807be..d3190d97 100644 --- a/ceilometer/hardware/pollsters/util.py +++ b/ceilometer/hardware/pollsters/util.py @@ -41,8 +41,9 @@ def make_resource_metadata(res_metadata=None, host_url=None): def make_sample_from_host(host_url, name, sample_type, unit, volume, project_id=None, user_id=None, resource_id=None, - res_metadata=None, extra={}): + res_metadata=None, extra=None): + extra = extra or {} resource_metadata = make_resource_metadata(res_metadata, host_url) resource_metadata.update(extra) |