python-openstackclient 7.3.1__tar.gz → 7.4.0__tar.gz

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/AUTHORS

@@ -112,6 +112,7 @@ Dmitriy Rabotyagov <noonedeadpunk@ya.ru>
 Dmitry Tantsur <dtantsur@protonmail.com>
 Dolph Mathews <dolph.mathews@gmail.com>
 Dongcan Ye <hellochosen@gmail.com>
+Doug Goldstein <cardoe@cardoe.com>
 Doug Hellmann <doug.hellmann@dreamhost.com>
 Doug Hellmann <doug@doughellmann.com>
 Doug Wiegley <dwiegley@salesforce.com>

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/ChangeLog

@@ -1,10 +1,18 @@
 CHANGES
 =======
 
+7.4.0
+-----
+
+* Fix credential creation
+
 7.3.1
 -----
 
 * [Neutron] Fix the "port show" command for trunk details
+* [Neutron] Support \`\`uplink-status-propagation-updatable\`\` extension
+* handle 'router create --flavor' option
+* [Neutron] "uplink-status-propagation" enabled by default
 
 7.3.0
 -----

{python_openstackclient-7.3.1/python_openstackclient.egg-info → python_openstackclient-7.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: python-openstackclient
-Version: 7.3.1
+Version: 7.4.0
 Summary: OpenStack Command-line Client
 Home-page: https://docs.openstack.org/python-openstackclient/latest/
 Author: OpenStack

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/openstackclient/identity/v3/credential.py

@@ -88,10 +88,10 @@ class CreateCredential(command.ShowOne):
         else:
             project = None
         credential = identity_client.create_credential(
-            user=user_id,
+            user_id=user_id,
             type=parsed_args.type,
             blob=parsed_args.data,
-            project=project,
+            project_id=project,
         )
 
         return _format_credential(credential)

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/openstackclient/network/v2/port.py

@@ -538,12 +538,12 @@ class CreatePort(command.ShowOne, common.NeutronCommandWithExtraArgs):
         uplink_status_group.add_argument(
             '--enable-uplink-status-propagation',
             action='store_true',
-            help=_("Enable uplink status propagate"),
+            help=_("Enable uplink status propagation (default)"),
         )
         uplink_status_group.add_argument(
             '--disable-uplink-status-propagation',
             action='store_true',
-            help=_("Disable uplink status propagate (default)"),
+            help=_("Disable uplink status propagation"),
         )
         parser.add_argument(
             '--project',
@@ -1103,6 +1103,18 @@ class SetPort(common.NeutronCommandWithExtraArgs):
                 "(requires data plane status extension)"
             ),
         )
+        uplink_status_group = parser.add_mutually_exclusive_group()
+        uplink_status_group.add_argument(
+            '--enable-uplink-status-propagation',
+            action='store_true',
+            help=_('Enable uplink status propagation'),
+        )
+        uplink_status_group.add_argument(
+            '--disable-uplink-status-propagation',
+            action='store_true',
+            help=_('Disable uplink status propagation'),
+        )
+
         _tag.add_tag_option_to_parser_for_set(parser, _('port'))
 
         return parser

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/openstackclient/network/v2/router.py

@@ -237,6 +237,9 @@ def _get_attrs(client_manager, parsed_args):
     if 'flavor_id' in parsed_args and parsed_args.flavor_id is not None:
         flavor = n_client.find_flavor(parsed_args.flavor_id)
         attrs['flavor_id'] = flavor.id
+    elif 'flavor' in parsed_args and parsed_args.flavor is not None:
+        flavor = n_client.find_flavor(parsed_args.flavor, ignore_missing=False)
+        attrs['flavor_id'] = flavor.id
 
     for attr in ('enable_default_route_bfd', 'enable_default_route_ecmp'):
         value = getattr(parsed_args, attr, None)

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/openstackclient/tests/unit/identity/v3/test_credential.py

@@ -73,10 +73,10 @@ class TestCredentialCreate(identity_fakes.TestIdentityv3):
         columns, data = self.cmd.take_action(parsed_args)
 
         kwargs = {
-            'user': self.credential.user_id,
+            'user_id': self.credential.user_id,
             'type': self.credential.type,
             'blob': self.credential.blob,
-            'project': None,
+            'project_id': None,
         }
         self.identity_sdk_client.create_credential.assert_called_once_with(
             **kwargs
@@ -105,10 +105,10 @@ class TestCredentialCreate(identity_fakes.TestIdentityv3):
         columns, data = self.cmd.take_action(parsed_args)
 
         kwargs = {
-            'user': self.credential.user_id,
+            'user_id': self.credential.user_id,
             'type': self.credential.type,
             'blob': self.credential.blob,
-            'project': self.credential.project_id,
+            'project_id': self.credential.project_id,
         }
         self.identity_sdk_client.create_credential.assert_called_once_with(
             **kwargs

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/openstackclient/tests/unit/network/v2/test_port.py

@@ -2633,6 +2633,35 @@ class TestSetPort(TestPort):
     def test_set_trusted_false(self):
         self._test_set_trusted_field(False)
 
+    def _test_set_uplink_status_propagation(self, uspropagation):
+        arglist = [self._port.id]
+        if uspropagation:
+            arglist += ['--enable-uplink-status-propagation']
+        else:
+            arglist += ['--disable-uplink-status-propagation']
+
+        verifylist = [
+            ('port', self._port.id),
+        ]
+        if uspropagation:
+            verifylist.append(('enable_uplink_status_propagation', True))
+        else:
+            verifylist.append(('enable_uplink_status_propagation', False))
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+        self.network_client.update_port.assert_called_once_with(
+            self._port, **{'propagate_uplink_status': uspropagation}
+        )
+        self.assertIsNone(result)
+
+    def test_set_uplink_status_propagation_true(self):
+        self._test_set_uplink_status_propagation(True)
+
+    def test_set_uplink_status_propagation_false(self):
+        self._test_set_uplink_status_propagation(False)
+
 
 class TestShowPort(TestPort):
     # The port to show.

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/openstackclient/tests/unit/network/v2/test_router.py

@@ -384,7 +384,7 @@ class TestCreateRouter(TestRouter):
     def test_create_with_no_tag(self):
         self._test_create_with_tag(add_tags=False)
 
-    def test_create_with_flavor_id_or_name(self):
+    def test_create_with_flavor_id_id(self):
         _flavor = network_fakes.create_one_network_flavor()
         self.network_client.find_flavor = mock.Mock(return_value=_flavor)
         arglist = [
@@ -392,7 +392,6 @@ class TestCreateRouter(TestRouter):
             '--flavor-id',
             _flavor.id,
         ]
-        arglist_with_name = [self.new_router.name, '--flavor-id', _flavor.name]
         verifylist = [
             ('name', self.new_router.name),
             ('enable', True),
@@ -412,18 +411,69 @@ class TestCreateRouter(TestRouter):
         self.assertEqual(self.columns, columns)
         self.assertCountEqual(self.data, data)
 
-        self.network_client.create_router.reset_mock()
-        verifylist_w_name = [
+    def test_create_with_flavor_id_name(self):
+        _flavor = network_fakes.create_one_network_flavor()
+        self.network_client.find_flavor = mock.Mock(return_value=_flavor)
+        arglist = [self.new_router.name, '--flavor-id', _flavor.name]
+        verifylist = [
             ('name', self.new_router.name),
             ('enable', True),
             ('distributed', False),
             ('ha', False),
             ('flavor_id', _flavor.name),
         ]
-        parsed_args_w_name = self.check_parser(
-            self.cmd, arglist_with_name, verifylist_w_name
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        columns, data = self.cmd.take_action(parsed_args)
+        self.network_client.create_router.assert_called_once_with(
+            **{
+                'admin_state_up': True,
+                'name': self.new_router.name,
+                'flavor_id': _flavor.id,
+            }
+        )
+        self.assertEqual(self.columns, columns)
+        self.assertCountEqual(self.data, data)
+
+    def test_create_with_flavor_id(self):
+        _flavor = network_fakes.create_one_network_flavor()
+        self.network_client.find_flavor = mock.Mock(return_value=_flavor)
+        arglist = [
+            self.new_router.name,
+            '--flavor',
+            _flavor.id,
+        ]
+        verifylist = [
+            ('name', self.new_router.name),
+            ('enable', True),
+            ('distributed', False),
+            ('ha', False),
+            ('flavor', _flavor.id),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        columns, data = self.cmd.take_action(parsed_args)
+        self.network_client.create_router.assert_called_once_with(
+            **{
+                'admin_state_up': True,
+                'name': self.new_router.name,
+                'flavor_id': _flavor.id,
+            }
         )
-        columns, data = self.cmd.take_action(parsed_args_w_name)
+        self.assertEqual(self.columns, columns)
+        self.assertCountEqual(self.data, data)
+
+    def test_create_with_flavor_name(self):
+        _flavor = network_fakes.create_one_network_flavor()
+        self.network_client.find_flavor = mock.Mock(return_value=_flavor)
+        arglist = [self.new_router.name, '--flavor', _flavor.name]
+        verifylist = [
+            ('name', self.new_router.name),
+            ('enable', True),
+            ('distributed', False),
+            ('ha', False),
+            ('flavor', _flavor.name),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        columns, data = self.cmd.take_action(parsed_args)
         self.network_client.create_router.assert_called_once_with(
             **{
                 'admin_state_up': True,

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0/python_openstackclient.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: python-openstackclient
-Version: 7.3.1
+Version: 7.4.0
 Summary: OpenStack Command-line Client
 Home-page: https://docs.openstack.org/python-openstackclient/latest/
 Author: OpenStack

{python_openstackclient-7.3.1 → python_openstackclient-7.4.0}/python_openstackclient.egg-info/SOURCES.txt

@@ -1287,6 +1287,7 @@ releasenotes/notes/osc4-volume-470422e5a453310e.yaml
 releasenotes/notes/pass_ssh_args-cf26a2ce26ccddaf.yaml
 releasenotes/notes/port-device-profile-4a3bf800da21c778.yaml
 releasenotes/notes/port-list-security-group-4af5d2e789174ff9.yaml
+releasenotes/notes/port_uplink_status_propagation_updatable-d1e155c19247b666.yaml
 releasenotes/notes/project-cleanup-skip-resource-option-4f80db0d8cf36fdb.yaml
 releasenotes/notes/properties-with-image-property-field.yaml-c51bf37c3106d6ff.yaml
 releasenotes/notes/qos-min-pps-rule-bfe22cea1966c4a0.yaml

python_openstackclient-7.4.0/python_openstackclient.egg-info/pbr.json

@@ -0,0 +1 @@
+{"git_version": "4b7e32ca", "is_release": true}

python_openstackclient-7.4.0/releasenotes/notes/port_uplink_status_propagation_updatable-d1e155c19247b666.yaml

@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    Add ``--enable-uplink-status-propagation`` option and
+    ``--disable-uplink-status-propagation`` option to ``port update`` command.

python_openstackclient-7.3.1/python_openstackclient.egg-info/pbr.json

@@ -1 +0,0 @@
-{"git_version": "0ba77e67", "is_release": true}