Added CreateNamespace RPC

[harishp8889]

Signed-off-by: Harish P [email protected]

[codecov]

Codecov Report

Merging #45 (df651d5) into main (6f9332a) will increase coverage by 3.33%.
The diff coverage is 48.48%.

❗ Current head df651d5 differs from pull request most recent head 99a36b8. Consider uploading reports for the commit 99a36b8 to get more accurate results

@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   31.57%   34.91%   +3.33%     
==========================================
  Files           1        1              
  Lines         171      169       -2     
==========================================
+ Hits           54       59       +5     
+ Misses        112      106       -6     
+ Partials        5        4       -1     

Impacted Files	Coverage Δ
goopicsi.go	34.91% <48.48%> (+3.33%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[glimchb]

NVMeControllerConnect , NVMeControllerConnect and CreateNVMeNamespace should not return protobuf message as a return value. instead just string/int/error maybe

otherwise implementation and user are tightly coupled and forces user to know about protobufs. if library changes in the future to support REST, for example, the user of the library should not be affected by it...

[harishp8889]

NVMeControllerConnect , NVMeControllerConnect and CreateNVMeNamespace should not return protobuf message as a return value. instead just string/int/error maybe

otherwise implementation and user are tightly coupled and forces user to know about protobufs. if library changes in the future to support REST, for example, the user of the library should not be affected by it...

Fixed

[glimchb]

lgtm