Fix limit list --service passing object instead of ID

In ListLimit.take_action(), find_service_sdk() returns a Service
object but the result is assigned directly to kwargs["service_id"]
without extracting the .id attribute. This causes the SDK to
serialize all object attributes as individual query parameters,
resulting in a 400 BadRequest from Keystone.

The fix appends .id to match the pattern used in CreateLimit and
region_id in the same file.

Also adds a unit test for the --service filter path.

Closes-Bug: 2161546

Change-Id: I916a33822f812ac095242017e94d945efd71f157
Signed-off-by: Amit Mahajan <Amit.Mahajan@windriver.com>

Author: Amit Mahajan

openstackclient/identity/v3/limit.py

@@ -171,7 +171,7 @@ def take_action(
         if parsed_args.service:
             kwargs["service_id"] = common_utils.find_service_sdk(
                 identity_client, parsed_args.service
-            )
+            ).id
 
         if parsed_args.region:
             kwargs["region_id"] = identity_client.get_region(

openstackclient/tests/unit/identity/v3/test_limit.py

@@ -450,3 +450,44 @@ def test_limit_list(self):
             ),
         )
         self.assertEqual(datalist, tuple(data))
+
+    def test_limit_list_with_service_filter(self):
+        self.identity_sdk_client.find_service.return_value = self.service
+
+        arglist = [
+            '--service',
+            self.service.name,
+        ]
+        verifylist = [
+            ('service', self.service.name),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.identity_sdk_client.limits.assert_called_with(
+            service_id=self.service.id,
+        )
+
+        collist = (
+            'ID',
+            'Project ID',
+            'Service ID',
+            'Resource Name',
+            'Resource Limit',
+            'Description',
+            'Region ID',
+        )
+        self.assertEqual(collist, columns)
+        datalist = (
+            (
+                self.limit.id,
+                self.project.id,
+                self.service.id,
+                self.limit.resource_name,
+                self.limit.resource_limit,
+                None,
+                None,
+            ),
+        )
+        self.assertEqual(datalist, tuple(data))