openstack/deb-zaqar
Code Issues Proposed changes

Metadata is not optional (to storage).

Browse Source 
Test cases are not affected.

Change-Id: I8dd90839cb87fbccf57544f8068eece513a35bb2
This commit is contained in:
Zhihao Yuan 2013-03-18 15:50:45 -04:00
parent a23643ec3c
commit 938d3dd73d
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
marconi/storage/base.py
View File

@ -99,14 +99,14 @@ class QueueBase(ControllerBase):
pass
@abc.abstractmethod
def upsert(self, name, tenant=None, metadata=None):
def upsert(self, name, metadata, tenant=None):
"""
This methods handles both creates and updates
operations for queues.
:param name: The queue name
:param tenant: Tenant id
:param metadata: Arbitrary metadata
:param tenant: Tenant id
:returns: True if a queue was created and False
if it was updated.
"""

2
marconi/tests/storage/test_base.py
View File

@ -41,7 +41,7 @@ class QueueController(storage.QueueBase):
def get(self, name, tenant=None):
super(QueueController, self).get(name, tenant=tenant)
def upsert(self, name, tenant=None, metadata=None):
def upsert(self, name, metadata, tenant=None):
super(QueueController, self).upsert(name, tenant=tenant,
metadata=metadata)