From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 18 Feb 2020 17:12:56 +0300 (MSK) From: Ivan Zakharyaschev To: ALT Linux Team development discussions In-Reply-To: <20200218134107.GB29107@altlinux.org> Message-ID: References: <20200218134107.GB29107@altlinux.org> User-Agent: Alpine 2.20 (LFD 67 2015-01-07) MIME-Version: 1.0 Content-Type: multipart/mixed; BOUNDARY="1807885841-648360676-1582035176=:6363" Cc: darktemplar@altlinux.org Subject: Re: [devel] [APT PATCH] test/integration/framework/test-apt-method-http*: use nginx as a forking daemon X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 Feb 2020 14:12:56 -0000 Archived-At: List-Archive: List-Post: This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --1807885841-648360676-1582035176=:6363 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT On Tue, 18 Feb 2020, Dmitry V. Levin wrote: > On Tue, Feb 18, 2020 at 06:39:01AM +0300, Ivan Zakharyaschev wrote: > > This is needed not to send requests to nginx before it is ready to > > listen: when the main process forks and exits, nginx is considered to > > be ready. > > > > Without this change, I observed a failure in the http test, but not > > the https one, probably, because the delay in the https test was > > larger due to an extra package being built: > > > > Run Testcase (22/29) test-apt-method-http > > Building package: simple-package > > Test for successful execution of apt-get update … > > Err http://localhost x86_64 release > > Could not connect to localhost:8080 (127.0.0.1). - connect (111 Connection refused) > > > > However: > > > > Run Testcase (23/29) test-apt-method-https > > Building package: simple-package > > Building package: conflicting-package-one > > Test for successful execution of apt-get update … PASS > > Test that package(s) are not installed with rpm -q simple-package … PASS > > Test for successful execution of apt-get install simple-package … PASS > > Test that package(s) are installed with rpm -q simple-package … PASS > > Pinning invalid key in apt > > Test for failure in execution of apt-get update … PASS > > Test that package(s) are not installed with rpm -q conflicting-package-one … PASS > > Test for failure in execution of apt-get install conflicting-package-one … PASS > > Test that package(s) are not installed with rpm -q conflicting-package-one … PASS > > Apparently, this fix is insufficient, I experience sporadic failures > in these nginx-based tests. Something racy is still there. It's a bit difficult for me to help to find out the reason, because I don't know how to reproduce any other failures than those fixed by my patch... -- Best regards, Ivan --1807885841-648360676-1582035176=:6363--