fix(deps): update opi-api digest and api changes

[sandersms]

No description provided.

[artek-koltun]

we can just remove this test since there is no parent anymore

[sandersms]

Great - I deprecated it to start with but will remove it completely.

[artek-koltun]

we can remove this test

[codecov]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (457e5eb) 75.02% compared to head (1d6c83b) 74.77%.

Files	Patch %	Lines
pkg/kvm/nvme.go	50.00%	0 Missing and 1 partial ⚠️
pkg/utils/server.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #874      +/-   ##
==========================================
- Coverage   75.02%   74.77%   -0.25%     
==========================================
  Files          40       40              
  Lines        3699     3699              
==========================================
- Hits         2775     2766       -9     
- Misses        839      845       +6     
- Partials       85       88       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[artek-koltun]

Ok, I checked this patch thoroughly. We need to update some more lines:

diff --git a/pkg/backend/nvme_path.go b/pkg/backend/nvme_path.go
index e17f870..4aafceb 100644
--- a/pkg/backend/nvme_path.go
+++ b/pkg/backend/nvme_path.go
@@ -273,7 +273,7 @@ func (s *Server) StatsNvmePath(ctx context.Context, in *pb.StatsNvmePathRequest)
 }
 
 func (s *Server) opiTransportToSpdk(transport pb.NvmeTransportType) string {
-       return strings.ReplaceAll(transport.String(), "NVME_TRANSPORT_", "")
+       return strings.ReplaceAll(transport.String(), "NVME_TRANSPORT_TYPE_", "")
 }
 
 func (s *Server) opiMultipathToSpdk(multipath pb.NvmeMultipath) string {
diff --git a/pkg/utils/server.go b/pkg/utils/server.go
index b017426..96e4089 100644
--- a/pkg/utils/server.go
+++ b/pkg/utils/server.go
@@ -148,5 +148,5 @@ func OpiAdressFamilyToSpdk(adrfam pb.NvmeAddressFamily) string {
                return ""
        }
 
-       return strings.ReplaceAll(adrfam.String(), "NVME_ADRFAM_", "")
+       return strings.ReplaceAll(adrfam.String(), "NVME_ADDRESS_FAMILY_", "")
 }

and update godpu sha256 here to 4301eadbb4bd4803eb993a9fd40ae52f132e1c5489996af60777ab320765a8e0
sha256 was taken from here and the CI should pass

[sandersms]

Ok, I checked this patch thoroughly. We need to update some more lines:

diff --git a/pkg/backend/nvme_path.go b/pkg/backend/nvme_path.go
index e17f870..4aafceb 100644
--- a/pkg/backend/nvme_path.go
+++ b/pkg/backend/nvme_path.go
@@ -273,7 +273,7 @@ func (s *Server) StatsNvmePath(ctx context.Context, in *pb.StatsNvmePathRequest)
 }
 
 func (s *Server) opiTransportToSpdk(transport pb.NvmeTransportType) string {
-       return strings.ReplaceAll(transport.String(), "NVME_TRANSPORT_", "")
+       return strings.ReplaceAll(transport.String(), "NVME_TRANSPORT_TYPE_", "")
 }
 
 func (s *Server) opiMultipathToSpdk(multipath pb.NvmeMultipath) string {
diff --git a/pkg/utils/server.go b/pkg/utils/server.go
index b017426..96e4089 100644
--- a/pkg/utils/server.go
+++ b/pkg/utils/server.go
@@ -148,5 +148,5 @@ func OpiAdressFamilyToSpdk(adrfam pb.NvmeAddressFamily) string {
                return ""
        }
 
-       return strings.ReplaceAll(adrfam.String(), "NVME_ADRFAM_", "")
+       return strings.ReplaceAll(adrfam.String(), "NVME_ADDRESS_FAMILY_", "")
 }

and update godpu sha256 here to 4301eadbb4bd4803eb993a9fd40ae52f132e1c5489996af60777ab320765a8e0 sha256 was taken from here and the CI should pass

I also found a few updates needed in the README file and scripts for the same enums with an extensive search on the entire folder. I added those changes to the PR also.

[artek-koltun]

LGTM