Merge lp://staging/~saviq/cloud-initramfs-tools/nfs-support into lp://staging/cloud-initramfs-tools

Proposed by Michał Sawicz
Status: Rejected
Rejected by: Scott Moser
Proposed branch: lp://staging/~saviq/cloud-initramfs-tools/nfs-support
Merge into: lp://staging/cloud-initramfs-tools
Diff against target: 79 lines (+44/-0)
2 files modified
overlayroot/etc/overlayroot.conf (+13/-0)
overlayroot/scripts/init-bottom/overlayroot (+31/-0)
To merge this branch: bzr merge lp://staging/~saviq/cloud-initramfs-tools/nfs-support
Reviewer Review Type Date Requested Status
cloud-initramfs-tools Pending
Review via email: mp+157542@code.staging.launchpad.net

Commit message

add NFS support to overlayroot

Description of the change

This adds support for NFS-backed overlayroot

Configuration:

overlayroot=nfs:host=10.0.0.1,path=/srv/nfs-overlay

This suffers from bug #1039402 unfortunately, so it's not really workable until the bug is fixed.

To post a comment you must log in.
Revision history for this message
Scott Moser (smoser) wrote :

Michał,
  Thanks for this. It generally looks fine to me. I'm OK to pull it into trunk, my only concern is bug 1039402.
  As I understand it, this is pretty much entirely broken without that bug fix in the kernel, right?

  If so, I'm just weary of pulling in code that is effectively "known broken". I'm not entirely opposed, especially if there is evidence that the kernel issue will be fixed. I'd definitely hold off on including it in ubuntu before raring release though.

  Thoughts?

Revision history for this message
Michał Sawicz (saviq) wrote :

W dniu 09.04.2013 16:05, Scott Moser pisze:
> Michał,
> Thanks for this. It generally looks fine to me. I'm OK to pull it into trunk, my only concern is bug 1039402.
> As I understand it, this is pretty much entirely broken without that bug fix in the kernel, right?

More or less, mounting with server-side locking supposedly helps, but
you can't actually mount it like that in initramfs, and I couldn't
remount later, so yeah.

> If so, I'm just weary of pulling in code that is effectively "known broken". I'm not entirely opposed, especially if there is evidence that the kernel issue will be fixed. I'd definitely hold off on including it in ubuntu before raring release though.
>
> Thoughts?

Perfectly agree. I mostly put it out there to get a feel whether I'm
getting anywhere with this. Holding off until the bug is fixed is perfect.

Thanks!
--
Michał Sawicz <email address hidden>
Canonical Services Ltd.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

On Tue, Apr 9, 2013 at 2:13 PM, Michał Sawicz
<email address hidden> wrote:
> W dniu 09.04.2013 16:05, Scott Moser pisze:
>> Michał,
>> Thanks for this. It generally looks fine to me. I'm OK to pull it into trunk, my only concern is bug 1039402.
>> As I understand it, this is pretty much entirely broken without that bug fix in the kernel, right?
>
> More or less, mounting with server-side locking supposedly helps, but
> you can't actually mount it like that in initramfs, and I couldn't
> remount later, so yeah.
>
>> If so, I'm just weary of pulling in code that is effectively "known broken". I'm not entirely opposed, especially if there is evidence that the kernel issue will be fixed. I'd definitely hold off on including it in ubuntu before raring release though.
>>
>> Thoughts?
>
> Perfectly agree. I mostly put it out there to get a feel whether I'm
> getting anywhere with this. Holding off until the bug is fixed is perfect.

I'm quite intrigued by the feature as well. Nice work. Cool idea. I
agree with publishing the branch and putting it into a holding state
until the kernel bug is fixed.

Cheers,
--
:-Dustin

Revision history for this message
Scott Moser (smoser) wrote :

Please excuse the harsh 'Rejected'.
I'm closing this right now because
a.) bug 1039402 makes it not really work
b.) i've moved code over to git.

If you want to re-submit, please feel free to do so to the git repo.

https://code.launchpad.net/~cloud-initramfs-tools/cloud-initramfs-tools/+git/cloud-initramfs-tools

Unmerged revisions

86. By Michał Sawicz

add NFS support to overlayroot

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches