summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2014-10-15 08:54:47 +0000
committerGerrit Code Review <review@openstack.org>2014-10-15 08:54:47 +0000
commita4c38cc6ddbfba2301766eced939229cccc31953 (patch)
tree38359bea911450f40d2f61dc2fb91d715e0ea643
parent4a21bd9d24a7e2cce219e2cbd6880309c975d2de (diff)
parent08af8e029d6e3d785f6387eef294eafad28ad3e1 (diff)
downloadtrove-a4c38cc6ddbfba2301766eced939229cccc31953.tar.gz
Merge "restart_required cfg-param is bool and not string" into proposed/juno2014.2.rc32014.2proposed/juno
-rw-r--r--trove/configuration/views.py4
-rw-r--r--trove/extensions/mgmt/configuration/views.py4
2 files changed, 5 insertions, 3 deletions
diff --git a/trove/configuration/views.py b/trove/configuration/views.py
index db8fa7fc..bde5dcce 100644
--- a/trove/configuration/views.py
+++ b/trove/configuration/views.py
@@ -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,
diff --git a/trove/extensions/mgmt/configuration/views.py b/trove/extensions/mgmt/configuration/views.py
index fa45e824..08b51563 100644
--- a/trove/extensions/mgmt/configuration/views.py
+++ b/trove/extensions/mgmt/configuration/views.py
@@ -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,