fabrics: Fix bad UUID size introduced in recent UUID changes#538
fabrics: Fix bad UUID size introduced in recent UUID changes#538igaw merged 1 commit intolinux-nvme:masterfrom
Conversation
|
@igaw - I don't know why, but some of the checks are failing to build subproject/openssl. That has nothing to do with the one line change I made. |
The static builds will build the OpenSSL library which produces warnings (new?). With |
Codecov Report
@@ Coverage Diff @@
## master #538 +/- ##
==========================================
- Coverage 24.58% 24.31% -0.28%
==========================================
Files 32 32
Lines 6137 6133 -4
Branches 1290 1278 -12
==========================================
- Hits 1509 1491 -18
- Misses 4115 4128 +13
- Partials 513 514 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
71c25d1 ("util: Add simple UUID type") introduced a regression in nvmf_get_tel(). nvmf_get_tel() returns the lenght of the binary representation. Hence use NVME_UUID_LEN instead. Signed-off-by: Martin Belanger <martin.belanger@dell.com> [dwagner: massaged commit message] Signed-off-by: Daniel Wagner <dwagner@suse.de>
|
Rebased the commit on the current master with a build config change so that we can test if the builds are fixed now. While at also replaced the github reference in the commit message. The reason for this is that I want to avoid to have git commit message depending on an external service. We never know when we have to move the repo. |
3 participants
Fixes: #537
Signed-off-by: Martin Belanger martin.belanger@dell.com