openstack/senlin
Code Issues Proposed changes

Remove 'limit' and 'marker' from cluster-policy list parameters

Browse Source 
We won't have a lot of policies attached to the same cluster. The limit
and marker parameters were mainly designed for long lists that exceed
the real estate of a page, for example. This patch removes them from the
db and api layer.
This commit is contained in:
tengqm
2015-03-04 17:16:10 +08:00
parent fa459ac953
commit 5821169f00
5 changed files with 3 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
senlin/db/sqlalchemy/api.py
View File

@@ -668,7 +668,6 @@ def cluster_policy_get(context, cluster_id, policy_id):
def cluster_policy_get_all(context, cluster_id, filters=None,
limit=None, marker=None,
sort_keys=None, sort_dir=None):
query = model_query(context, models.ClusterPolicies)
query = query.filter_by(cluster_id=cluster_id)
@@ -686,7 +685,6 @@ def cluster_policy_get_all(context, cluster_id, filters=None,
query = db_filters.exact_filter(query, models.ClusterPolicies, filters)
return _paginate_query(context, query, models.ClusterPolicies,
limit=limit, marker=marker,
sort_keys=keys, sort_dir=sort_dir,
default_sort_keys=['priority']).all()