diff options
| author | Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com> | 2017-01-12 13:21:07 +0000 |
|---|---|---|
| committer | Rodolfo Alonso Hernandez <rodolfo.alonso.hernandez@intel.com> | 2017-01-12 13:21:07 +0000 |
| commit | 4b2355b3e3cdbdb4e1a395e313147decf6221bc4 (patch) | |
| tree | 10df9a43449bc1f7cfca3994990f5a3081712b17 /openstackclient/network/v2 | |
| parent | c8c29e8c2e4644b066cc2cb7e86b36664c4e094c (diff) | |
| download | python-openstackclient-4b2355b3e3cdbdb4e1a395e313147decf6221bc4.tar.gz | |
Fix Network QoS rule CLI attrs parsing
Allows to set zero values in the CLI parameters for
Network QoS rule.
Change-Id: Ie0e045ff4888615d68804fd739d5b995ca11e9a1
Closes-Bug: #1655947
Diffstat (limited to 'openstackclient/network/v2')
| -rw-r--r-- | openstackclient/network/v2/network_qos_rule.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/openstackclient/network/v2/network_qos_rule.py b/openstackclient/network/v2/network_qos_rule.py index 054d16b4..a662ca18 100644 --- a/openstackclient/network/v2/network_qos_rule.py +++ b/openstackclient/network/v2/network_qos_rule.py @@ -85,16 +85,16 @@ def _get_attrs(network_client, parsed_args, is_create=False): msg = _('"Create" rule command requires argument "type"') raise exceptions.CommandError(msg) rule_type = parsed_args.type - if parsed_args.max_kbps: + if parsed_args.max_kbps is not None: attrs['max_kbps'] = parsed_args.max_kbps - if parsed_args.max_burst_kbits: + if parsed_args.max_burst_kbits is not None: # NOTE(ralonsoh): this parameter must be changed in SDK and then in # Neutron API, from 'max_burst_kbps' to # 'max_burst_kbits' attrs['max_burst_kbps'] = parsed_args.max_burst_kbits - if parsed_args.dscp_mark: + if parsed_args.dscp_mark is not None: attrs['dscp_mark'] = parsed_args.dscp_mark - if parsed_args.min_kbps: + if parsed_args.min_kbps is not None: attrs['min_kbps'] = parsed_args.min_kbps if parsed_args.ingress: attrs['direction'] = 'ingress' |
