openstack
/
trove
Code Issues Proposed changes

Merge "restart_required cfg-param is bool and not string" into proposed/juno

This commit is contained in:
Jenkins 2014-10-15 08:54:47 +00:00 committed by Gerrit Code Review
parent 4a21bd9d24 08af8e029d
commit a4c38cc6dd
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
trove/configuration/views.py
View File

@ -118,8 +118,8 @@ class ConfigurationParameterView(object):
self.config = config
def data(self):
# v1 api expects this to be a 'true' or 'false' string instead of 1/0
restart_required = 'true' if self.config.restart_required else 'false'
# v1 api is to be a 'true' or 'false' json boolean instead of 1/0
restart_required = True if self.config.restart_required else False
ret = {
"name": self.config.name,
"datastore_version_id": self.config.datastore_version_id,

4
trove/extensions/mgmt/configuration/views.py
View File

@ -24,10 +24,12 @@ class MgmtConfigurationParameterView(object):
self.config = config
def data(self):
# v1 api is to be a 'true' or 'false' json boolean instead of 1/0
restart_required = True if self.config.restart_required else False
ret = {
"name": self.config.name,
"datastore_version_id": self.config.datastore_version_id,
"restart_required": self.config.restart_required,
"restart_required": restart_required,
"type": self.config.data_type,
"deleted": self.config.deleted,
"deleted_at": self.config.deleted_at,