feat: volume attachments

[choieastsea]

Overview

• attach/detach volume tool 추가

Key Changes

• compute_tools에 attached_volumes 정보 제공
• attach_volume, detach_volume 함수 추가
• action server에서 enum 파라미터 전달시 발생하는 오류 해결 (enum -> str)

Related Issues

• [FEAT] add tools about computes' volume attachment #61

Additional context

[halucinor]

LGTM 👍

[jja6312]

제가 봤을때 다 좋아보이는데,
create_volume_attachment 함수가 sdk코드(proxy)에서는 None이 아닌 VolumeAttachment를 반환하던데, 저라면 문서랑 최대한 같게끔 return타입을 만들어서 반환했을것같은데 None 반환은 어떤 의도이실지 궁금합니다

[choieastsea]

제가 봤을때 다 좋아보이는데, create_volume_attachment 함수가 sdk코드(proxy)에서는 None이 아닌 VolumeAttachment를 반환하던데, 저라면 문서랑 최대한 같게끔 return타입을 만들어서 반환했을것같은데 None 반환은 어떤 의도이실지 궁금합니다

저도 문서와 최대한 같은 정보를 제공하는 게 맞다고 생각합니다. 다만 해당 케이스에서는 두가지 이유가 있어 예외적으로 응답하였는데요.

1. 비동기 작업이라 volume attachment를 응답한 해당 시점에 실제 attach되지 않는데, 이는 mcp client 입장에서 혼란이 있을 수 있다고 생각하였습니다.
2. compute 입장에서 attachment를 별도의 자원으로 볼 필요가 없다고 생각하였습니다. (지극히 개인적인 의견입니다)

@jja6312 님 생각에 attachment로 응답하는 것이 좋다고 생각하는 이유가 있다면 알려주심 감사하겠습니다!

[jja6312]

@choieastsea 관련해서 찾아봤는데, Attachment로 따로 추가 제어할 필요가 없다면 None으로 반환하는 것도 괜찮아 보입니다. 다만 추가 제어에서 문제를겪으면 이부분을 유의하고있으면 좋을것같습니다