diff options
author | Brad P. Crochet <brad@redhat.com> | 2015-07-30 15:23:38 -0400 |
---|---|---|
committer | Brad P. Crochet <brad@redhat.com> | 2015-07-30 15:23:38 -0400 |
commit | 3742ba77a7765c51f2a431c85c482c960a9a7337 (patch) | |
tree | c9e782ddd1dea4864eceb7777991b947c57da330 | |
parent | 6bec389675d7717e9507dd9bf74a0f881c0b5675 (diff) | |
download | tuskar-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.py | 6 |
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) |