Skip to content

Commit

Permalink
Partial-Bug: #1567582 - SM Backend must prevent provision if domain n…
Browse files Browse the repository at this point in the history
…ot specified

We check if domain is specified at least for server or cluster.
If not, provision doesn't go through.
In hand with this, UI omits this check.

Change-Id: I8074df601b389971968eb6f35f7b932b73bab088
  • Loading branch information
nitishkrishna committed Apr 8, 2016
1 parent 7b3fa2e commit 3549f35
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/server_mgr_main.py
Expand Up @@ -4102,7 +4102,9 @@ def provision_server(self):
provision_params['domain'] = server['domain']
else:
provision_params['domain'] = cluster_params['domain']

if not provision_params['domain']:
msg = "No Domain specified either for server " + provision_params['server_id'] + " or cluster " + server["cluster_id"]
self.log_and_raise_exception(msg)
provision_params['rmq_master'] = role_ids['config'][0]
provision_params['uuid'] = cluster_params['uuid']
provision_params['smgr_ip'] = self._args.listen_ip_addr
Expand Down

0 comments on commit 3549f35

Please sign in to comment.