Hi Dave,

Thank you for the debdiffs - they look good. A few questions:

Is this issue fixed in Vivid? I'm not sure if this is what you mean by
"Ceph upstream applied a new nofile limit(10x) to Giant+ releases (vivid
in Ubuntu repo)." - has this fix landed in Vivid, or is it just upstream
and expected to land in Vivid?

dep3 header in "Origin: http..." should probably say "Origin: upstream,
http..." to make it clear the patches are from upstream. Patches from
upstream carry more confidence in cherry-picking to Ubuntu, so we should
point this out for anybody trying to figure out the patches later. Don't
worry about redoing the debdiffs just for this - I can just tweak before
upload.

Changing the upstart script will presumably cause a conffile prompt if
the user has made local modifications to the file. We should point this
out in [Regression Potential] and make sure the SRU team know about it
in order to help make a decision. We'd like to avoid conffile prompts in
SRUs in general, but in this case it might make sense. I'll leave for
the SRU team to decide.

The Trusty version should be 0.80.8-0ubuntu0.14.04.2 but you got the
Utopic one right. I can tweak the Trusty version on upload. But there's
already a ceph SRU in flight, so an upload will have to wait until that
has gone through. I'll hold on reviewing the patch in detail and build
testing until then.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1420647

Title:
  Please apply nofile limit fix of ceph-osd to trusty, utopic

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ceph/+bug/1420647/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to