x/tuskar-ui
Code Issues Proposed changes

Do not sync parameters across roles

Browse Source 
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
This commit is contained in:
Brad P. Crochet 2015-07-30 15:23:38 -04:00
parent 6bec389675
commit 3742ba77a7
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tuskar_ui/infrastructure/parameters/forms.py
View File

@ -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)