Skip to content

Commit

Permalink
Revert "Fix CLI support for DVR"
Browse files Browse the repository at this point in the history
This reverts commit bfec80a.

This patch needs to be reverted until we resolve the issue with
patch 109180.

The previous change introduced a situation where we can only
override the default server side setting from the client side
for "--distributed" when the default is False. But we cannot
override when the default is True.

This is a requirement for DVR and at the same time we don't
want to introduce any new changes that would affect any current
plugins. So we will decide on it later and in the mean while I
will revert it.

Partial-Bug: #1346121
Partial-Bug: #1347960
Change-Id: I261f9e232b1f72ed0cb2f0c1ba218facaba9afcb
  • Loading branch information
Swaminathan Vasudevan committed Jul 25, 2014
1 parent e059143 commit c46bf95
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions neutronclient/neutron/v2_0/router.py
Original file line number Diff line number Diff line change
Expand Up @@ -70,15 +70,15 @@ def add_known_arguments(self, parser):
'name', metavar='NAME',
help=_('Name of router to create.'))
parser.add_argument(
'--distributed',
dest='distributed', action='store_true',
default=argparse.SUPPRESS,
'distributed', action='store_true',
help=_('Create a distributed router.'))

def args2body(self, parsed_args):
body = {self.resource: {'admin_state_up': parsed_args.admin_state}}
neutronV20.update_dict(parsed_args, body[self.resource],
['name', 'distributed', 'tenant_id'])
body = {'router': {
'name': parsed_args.name,
'admin_state_up': parsed_args.admin_state, }, }
if parsed_args.tenant_id:
body['router'].update({'tenant_id': parsed_args.tenant_id})
return body


Expand Down

0 comments on commit c46bf95

Please sign in to comment.