feat(block): Add attachments list(#70)

- Add get_attchments tools
- Tests are updated and passing

Author: S0okJu

src/openstack_mcp_server/tools/block_storage_tools.py

@@ -3,6 +3,7 @@
 from .base import get_openstack_conn
 from .response.block_storage import (
     Attachment,
+    AttachmentSummary,
     ConnectionInfo,
     Volume,
     VolumeAttachment,
@@ -25,6 +26,7 @@ def register_tools(self, mcp: FastMCP):
         mcp.tool()(self.extend_volume)
 
         mcp.tool()(self.get_attachment_details)
+        mcp.tool()(self.get_attachments)
 
     def get_volumes(self) -> list[Volume]:
         """
@@ -225,3 +227,34 @@ def get_attachment_details(self, attachment_id: str) -> Attachment:
         }
 
         return Attachment(**params)
+
+    def get_attachments(
+        self,
+        volume_id: str | None = None,
+        instance: str | None = None,
+    ) -> list[Attachment]:
+        """
+        Get the list of attachments.
+
+        :return: A list of Attachment objects.
+        """
+        conn = get_openstack_conn()
+
+        filter = {}
+        if volume_id:
+            filter["volume_id"] = volume_id
+        if instance:
+            filter["instance"] = instance
+
+        attachments = []
+        for attachment in conn.block_storage.attachments(**filter):
+            attachments.append(
+                AttachmentSummary(
+                    id=attachment.id,
+                    instance=attachment.instance,
+                    volume_id=attachment.volume_id,
+                    status=attachment.status,
+                )
+            )
+
+        return attachments

src/openstack_mcp_server/tools/response/block_storage.py

@@ -42,3 +42,10 @@ class Attachment(BaseModel):
     attach_mode: str | None = None
     connection_info: ConnectionInfo | None = None
     connector: str | None = None
+
+
+class AttachmentSummary(BaseModel):
+    id: str
+    instance: str
+    volume_id: str
+    status: str

tests/tools/test_block_storage_tools.py

@@ -616,7 +616,7 @@ def test_register_tools(self):
         block_storage_tools.register_tools(mock_mcp)
 
         # Verify mcp.tool() was called for each method
-        assert mock_mcp.tool.call_count == 6
+        assert mock_mcp.tool.call_count == 7
 
         # Verify all methods were registered
         registered_methods = [
@@ -744,3 +744,36 @@ def test_get_attachment_details(
         mock_conn.block_storage.get_attachment.assert_called_once_with(
             "attach-123"
         )
+
+    def test_get_attachments(self, mock_get_openstack_conn_block_storage):
+        """Test getting attachments."""
+        mock_conn = mock_get_openstack_conn_block_storage
+
+        # Create mock attachment object
+        mock_attachment = Mock()
+        mock_attachment.id = "attach-123"
+        mock_attachment.instance = "server-123"
+        mock_attachment.volume_id = "vol-123"
+        mock_attachment.status = "attached"
+
+        mock_conn.block_storage.attachments.return_value = [mock_attachment]
+
+        # Test attachments
+        block_storage_tools = BlockStorageTools()
+
+        filter = {
+            "volume_id": "vol-123",
+            "instance": "server-123",
+        }
+        result = block_storage_tools.get_attachments(**filter)
+
+        # Verify the result
+        assert isinstance(result, list)
+        assert len(result) == 1
+        assert result[0].id == "attach-123"
+        assert result[0].instance == "server-123"
+        assert result[0].volume_id == "vol-123"
+        assert result[0].status == "attached"
+
+        # Verify the mock calls
+        mock_conn.block_storage.attachments.assert_called_once_with(**filter)