[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2020-04-03 Thread Dan Streetman
> The swift package in Debian has already moved to systemd though, and they > went with a generated approach. To see what they did, pull-debian-source > swift and take a look at the debian/*init.in files and debian/rules > override_dh_installinit, override_dh_installsystemd. oh wow, it looks like

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2020-04-03 Thread Tytus Kurek
Dan / Corey, Thank you very much for looking into it. I don't think it's a priority any more, but still worth finalising and merging. Unfortunately, I have no more time to drive it, so I will have to leave it in your hands. Thank you Tytus -- You received this bug notification because you are

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2020-04-03 Thread Corey Bryant
Hi Tytus and Dan, Thanks for the patches Tytus. Always appreciate contributions! Generally for the core openstack packages they don't follow the normal sync from debian flow for a number of reasons and the packages tend to differ quite a bit. The swift package in Debian has already moved to

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2020-04-02 Thread Dan Streetman
Hello, generally, changes like this should be done in Debian first, however the 'cloud' packages can sometimes be special. @corey.bryant, do you have an opinion on changing the swift sysv init scripts over to normal systemd services (meaning, @tkurek doing the work with me assisting him)? If

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-06-13 Thread Tytus Kurek
Hi Mathew, Is there any chance to move this forward? Thanks -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1827340 Title: swift-container package is missing Upstart and System V files for the

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-25 Thread Mathew Hodson
** Changed in: swift (Ubuntu) Importance: Undecided => Low -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1827340 Title: swift-container package is missing Upstart and System V files for the

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-14 Thread Bryan Quigley
Hi Tytus, I did a once over and it looks good to me. I'm far from an expert though. I think they use git-ubuntu for this, but I'd check with one of the maintainers. Thanks, Bryan -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu.

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-14 Thread Tytus Kurek
Hi Bryan, I'm attaching an updated patch, however, as this is the first time I'm creating a patch for ubuntu package, I am not sure whether everything is correct. Please have a look and let me know if anything has to be changed. Regards Tytus ** Patch added: "lp1827340.patch"

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-14 Thread Tytus Kurek
** Changed in: swift (Ubuntu) Status: New => In Progress ** Changed in: swift (Ubuntu) Assignee: (unassigned) => Tytus Kurek (tkurek) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1827340

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-14 Thread Tytus Kurek
** Changed in: swift (Ubuntu) Status: In Progress => New ** Changed in: swift (Ubuntu) Assignee: Tytus Kurek (tkurek) => (unassigned) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1827340

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-13 Thread Bryan Quigley
Yea, I'd be for going systemd unit files only. They will always take precedence on 18.04+, so might as well only provide them. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1827340 Title:

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-13 Thread Tytus Kurek
I agree that it reveals a deeper issue. Would you suggest to remove all old Upstart / System V files (for all Swift services) and replace them with Systemd files? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu.

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-11 Thread Bryan Quigley
I don't see the point in enabling Upstart (or sysv init.d jobs) at this point - unless I really don't understand how this will be used Upstart jobs won't do anything on modern releases of Ubuntu. Instead perhaps using similar systemd jobs that Debian has

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-06 Thread Ubuntu Foundations Team Bug Bot
The attachment "lp1827340.patch" seems to be a debdiff. The ubuntu- sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-06 Thread Tytus Kurek
Patch attached ** Patch added: "lp1827340.patch" https://bugs.launchpad.net/ubuntu/+source/swift/+bug/1827340/+attachment/5261901/+files/lp1827340.patch ** Changed in: swift (Ubuntu) Status: New => In Progress -- You received this bug notification because you are a member of Ubuntu

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-06 Thread Tytus Kurek
** Changed in: swift (Ubuntu) Assignee: (unassigned) => Tytus Kurek (tkurek) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1827340 Title: swift-container package is missing Upstart and System

[Bug 1827340] Re: swift-container package is missing Upstart and System V files for the container-sharder service

2019-05-02 Thread Tytus Kurek
** Tags removed: field-medium -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1827340 Title: swift-container package is missing Upstart and System V files for the container-sharder service To