From 3742ba77a7765c51f2a431c85c482c960a9a7337 Mon Sep 17 00:00:00 2001 From: "Brad P. Crochet" Date: Thu, 30 Jul 2015 15:23:38 -0400 Subject: 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 --- tuskar_ui/infrastructure/parameters/forms.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) 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) -- cgit v1.2.1