opiproject / opi-api

Open Programmable Infrastructure API and Behavioral Model
Apache License 2.0
31 stars 39 forks source link

[storage] NvmeRemoteController* -> NvmeBackendController* #363

Open artek-koltun opened 10 months ago

artek-koltun commented 10 months ago

At the moment, we have an empty backend_nvme_pcie.proto and backend_nvme_tcp.proto. However, it is planned to have fabrics and pcie in one backend service. Probably, rename of backend_nvme_tcp.proto to backend_nvme.proto and deletion of backend_nvme_pcie.proto makes sense.

Also, since we can connect to local pcie nvme drives by the service in backend_nvme_tcp.proto it might make sense to rename messages: NvmeRemoteController* -> NvmeBackendController* NvmeRemoteNamespace* -> NvmeBackendNamespace*

Additionally, see https://github.com/opiproject/opi-api/pull/343

artek-koltun commented 9 months ago

Files are renamed/deleted

Message renames are left