Hi Alkis, On Thu, Dec 01, 2011 at 06:14:42AM -0000, Alkis Georgopoulos wrote: > While the NBDROOT environment variable would allow for some flexibility, it > still wouldn't allow LTSP to use the nbd-client.initrd script, as some LTSP > users need to use the nbd-proxy hack. > Also, LTSP would need to initialize networking before nbd-client.initrd, to > get the necessary info from DHCP, so I think that providing a way to launch > an NBDCLIENT wrapper would be more flexible than the NBDROOT env var.
Right. > Furthermore, I believe it's best to have a common syntax for NBD, and the > host:port/path syntax that NFS uses > (http://www.rfc-editor.org/rfc/rfc2224.txt) is a good candidate. > > I'm attaching a proposed nbd-client.initrd script that should handled > all the use cases I've heard, while still being rather simple and while > maintaining the commas-based syntax. I also did some minor > optimizations, like replacing sed with case etc. It doesn't look too intrusive, and I would accept it if it worked, but there's a bug. Try running: a='[2001:6f8:32f::39]:1234' echo ${a%%:*} the result is: [2001 rather than [2001:6f8:32f::39] I'm not entirely sure how to fix this. In fact, the fact that IPv6 uses colons is the reason I chose commas rather than colons as hostname-port separator -- that way I didn't have to deal with this kind of mess :-) > Wouter, could you please consider it for inclusion? Thanks! > > ** Attachment added: "nbd-client.initrd" > > https://bugs.launchpad.net/ubuntu/+source/nbd/+bug/594595/+attachment/2614365/+files/nbd-client.initrd > > ** Patch removed: "Patch for debian/nbd-client.initrd" > > https://bugs.launchpad.net/ubuntu/+source/nbd/+bug/594595/+attachment/1427143/+files/nbd.patch > > ** Changed in: nbd (Ubuntu) > Status: Invalid => Triaged > > ** Changed in: nbd (Ubuntu) > Importance: Undecided => Wishlist > > -- > You received this bug notification because you are subscribed to nbd in > Ubuntu. > https://bugs.launchpad.net/bugs/594595 > > Title: > feature request: support colons in nbdroot kernel parameter > > To manage notifications about this bug go to: > https://bugs.launchpad.net/ubuntu/+source/nbd/+bug/594595/+subscriptions > -- The volume of a pizza of thickness a and radius z can be described by the following formula: pi zz a -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/594595 Title: feature request: support colons in nbdroot kernel parameter To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/nbd/+bug/594595/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs