-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nvme_gw: implicitly create transports #18
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggested this a while ago.
ce03e44
to
139b1f2
Compare
139b1f2
to
e26ad66
Compare
Signed-off-by: Mykola Golub <mykola.golub@clyso.com>
e26ad66
to
d1465b2
Compare
60f48ed
to
d9ec721
Compare
Instead of providing create_transport command, implicitly create transports that are specified in spdk/transports config option (space separated values). Also support transport_{name}_options config param in spdk section to specify transport specific options (json encoded string). Fixes: ceph#12 Signed-off-by: Mykola Golub <mykola.golub@clyso.com>
d9ec721
to
8fb7d4c
Compare
@idryomov Thanks. Updated. |
Looks good! @PepperJo I think we are on the same page now, but I'd rather not override your "Changes requested". Please ack and I'll merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Mykola. LGTM
Instead of providing create_transport command, implicitly create
transports that are specified in spdk/transports config option
(space separated values).
Additionally one may use spdk/transport_{name}_options to
specify transport specific options (space separated key=val)
Fixes: #12
Signed-off-by: Mykola Golub mykola.golub@clyso.com