summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBrad P. Crochet <brad@redhat.com>2015-07-30 15:23:38 -0400
committerBrad P. Crochet <brad@redhat.com>2015-07-30 15:23:38 -0400
commit3742ba77a7765c51f2a431c85c482c960a9a7337 (patch)
treec9e782ddd1dea4864eceb7777991b947c57da330
parent6bec389675d7717e9507dd9bf74a0f881c0b5675 (diff)
downloadtuskar-ui-3742ba77a7765c51f2a431c85c482c960a9a7337.tar.gz
Do not sync parameters across roles
Wholesale syncing of the parameters across roles doesn't work. There are some parameters that can be different depending on the role. For one, NeutronPublicInterface might be different depending on the node's role. This patch comments out the sync. Future might involve creating a white/blacklist for which params should/should not be synced. Change-Id: I66be8126e0160aaa0af2f49b7d5fce705e720f52
-rw-r--r--tuskar_ui/infrastructure/parameters/forms.py6
1 files changed, 5 insertions, 1 deletions
diff --git a/tuskar_ui/infrastructure/parameters/forms.py b/tuskar_ui/infrastructure/parameters/forms.py
index cc31df57..e0c0609d 100644
--- a/tuskar_ui/infrastructure/parameters/forms.py
+++ b/tuskar_ui/infrastructure/parameters/forms.py
@@ -136,7 +136,11 @@ class AdvancedEditServiceConfig(ServiceConfig):
def handle(self, request, data):
plan = api.tuskar.Plan.get_the_plan(self.request)
- data = self._sync_common_params_across_roles(plan, data)
+
+ # TODO(bcrochet): Commenting this out.
+ # For advanced config, we should have a whitelist of which params
+ # must be synced across roles.
+ # data = self._sync_common_params_across_roles(plan, data)
try:
plan.patch(request, plan.uuid, data)