[oe] [meta-webserver][PATCH] nginx.service: sleep 0.1 after nginx start
Changqing Li
changqing.li at windriver.com
Wed Feb 26 01:48:08 UTC 2020
On 2/25/20 12:49 PM, Khem Raj wrote:
>
>
> On 2/24/20 7:53 PM, Changqing Li wrote:
>>
>> On 2/25/20 12:40 AM, Khem Raj wrote:
>>> On Mon, Feb 24, 2020 at 3:41 AM<changqing.li at windriver.com> wrote:
>>>> From: Changqing Li<changqing.li at windriver.com>
>>>>
>>>> Fix error info:
>>>> nginx.service: failed to parse pid from file /run/nginx/nginx.pid:
>>>> invalid argument
>>>>
>>>> Signed-off-by: Changqing Li<changqing.li at windriver.com>
>>>> ---
>>>> meta-webserver/recipes-httpd/nginx/files/nginx.service | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/meta-webserver/recipes-httpd/nginx/files/nginx.service
>>>> b/meta-webserver/recipes-httpd/nginx/files/nginx.service
>>>> index 9a6ca96..65c7752 100644
>>>> --- a/meta-webserver/recipes-httpd/nginx/files/nginx.service
>>>> +++ b/meta-webserver/recipes-httpd/nginx/files/nginx.service
>>>> @@ -7,6 +7,7 @@ Type=forking
>>>> PIDFile=/run/nginx/nginx.pid
>>>> ExecStartPre=@SBINDIR@/nginx -t
>>>> ExecStart=@SBINDIR@/nginx
>>>> +ExecStartPost=/bin/sleep 0.1
>>> I dont thiink we should be adding sleeps instead try to lock the
>>> dependencies
>>> so it happens correctly. or perhaps change the logic in code itself if
>>> dependencies
>>> can not handle it.
>>
>> error info: "nginx.service: failed to parse pid from file
>> /run/nginx/nginx.pid: invalid argument" is caused by race condition
>> between
>>
>> systemd and nginx. And this error will not caused problem except
>> scary log
>>
>> This should be a known problem:
>> https://bugs.launchpad.net/ubuntu/+source/nginx/+bug/1581864
>>
>> other distro like ubuntu, fedora, should also have this problem, I
>> can google people report this error on different distro.
>
> I do not question the problem but the solution, there have been
> different solutions posted to ml e.g.
>
> https://mailman.nginx.org/pipermail/nginx-devel/2016-February/007961.html
>
> or ubuntu fix
>
> https://github.com/aroth-arsoft/pkg-nginx/blob/master/debian/patches/nginx-fix-pidfile.patch
>
>
> perhaps you should try one of these patches
Hi, Khem
I found that both V1 and V2 are on master-next now, please remember to
drop the V1 when merge to master, thanks.
>
>>
>> ubuntu 16.04 use this as a workaround.
>>
>>>> ExecReload=@SBINDIR@/nginx -s reload
>>>> ExecStop=@BASE_BINDIR@/kill -s QUIT $MAINPID
>>>> PrivateTmp=true
>>>> --
>>>> 2.7.4
>>>>
>>>> --
>>>> _______________________________________________
>>>> Openembedded-devel mailing list
>>>> Openembedded-devel at lists.openembedded.org
>>>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>
More information about the Openembedded-devel
mailing list