openstack
/
trove
Code Issues Proposed changes

Merge "Display boolean module values"

This commit is contained in:
Jenkins 2017-01-19 06:22:14 +00:00 committed by Gerrit Code Review
parent f644fdcdb5 49d5558b1e
commit d6c2c418f2
2 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
releasenotes/notes/disply_module_bools_properly-571cca9a87f28339.yaml Normal file
View File

@ -0,0 +1,5 @@
---
fixes:
- |
Module list/show now returns boolean values as True/False instead of
1/0. Bug 1656398

10
trove/module/views.py
View File

@ -32,12 +32,12 @@ class ModuleView(object):
tenant_id=self.module.tenant_id,
datastore_id=self.module.datastore_id,
datastore_version_id=self.module.datastore_version_id,
auto_apply=self.module.auto_apply,
priority_apply=self.module.priority_apply,
auto_apply=bool(self.module.auto_apply),
priority_apply=bool(self.module.priority_apply),
apply_order=self.module.apply_order,
is_admin=self.module.is_admin,
is_admin=bool(self.module.is_admin),
md5=self.module.md5,
visible=self.module.visible,
visible=bool(self.module.visible),
created=self.module.created,
updated=self.module.updated)
# add extra data to make results more legible
@ -96,7 +96,7 @@ class DetailedModuleView(ModuleView):
def data(self, include_contents=False):
return_value = super(DetailedModuleView, self).data()
module_dict = return_value["module"]
module_dict["live_update"] = self.module.live_update
module_dict["live_update"] = bool(self.module.live_update)
if hasattr(self.module, 'instance_count'):
module_dict["instance_count"] = self.module.instance_count
if include_contents: