From: "Alexander V. Tikhonov" <avtikhon@tarantool.org>
To: Sergey Bronnikov <sergeyb@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org
Subject: Re: [Tarantool-patches] [PATCH v1] Divide test box/net.box
Date: Mon, 1 Jun 2020 12:24:49 +0300 [thread overview]
Message-ID: <20200601092449.GA5590@hpalx> (raw)
In-Reply-To: <20200518100058.GC20053@pony.bronevichok.ru>
Hi Sergey, thanks for the suggestions, I've tried to use Tarantool
packages 1.6.8 version and older, but met a lot of issue with it's
installing. Also the main issue that it's not possible to rebuild
Tarantool of the old packages in the new OS. The only found available
package fot Ubuntu:14.04, but it has issues with installation
dependencies, anyway it's version is later than checking patches.
These issues were reported to out team and decided to avoid of checking
patches for fixes older than 1.7-1.9 versions.
On Mon, May 18, 2020 at 01:00:58PM +0300, Sergey Bronnikov wrote:
> Hi, Alexander
>
> On 20:04 Fri 15 May , Alexander V. Tikhonov wrote:
> > Hi Sergey, thanks a lot for the review. As we discussed the ability to
> > reproduce the issues on the fixing commits I've checked group of tests:
> > 10 tests as you asked in the issue and 3 that I've already checked.
> > Whithin these tests results were:
> > 3 tests: all reproduced the issue
> > 10 tests:
> > 4 - reproduced the issue
> > 1 - didn't show the issue
>
> could you reproduce issue using original test with source code without fix?
>
> > 5 - couldn't build and test too old fixes committed in 1.6 Tarantool
> > version.
>
> could you reproduce them using tarantool from packages?
>
> > To see the results with names please check the issue in github, where
> > I've created the table of results for the checked tests.
> >
> > On Thu, Apr 02, 2020 at 06:44:44PM +0300, Sergey Bronnikov wrote:
> > > Hello!
> > >
> > > before splitting test we had a net.box.skipcond file due to an issue
> > > https://github.com/tarantool/tarantool/issues/4271. I believe that file
> > > is useless now and we should add similar files for a new tests.
> > >
> > > I have looked on tests without postfix "gh-xxx" in a filenames.
> > > LGTM for them:
> > >
> > > net.box_connect_triggers.test.lua
> > > net.box_permissions.test.lua
> > > net.box_reconnect_after.test.lua
> > > net.box_get_connection_object.test.lua
> > >
> > > I propose to run tests with postfix "gh-xxx" in filenames on commits
> > > before fix where bug covered by test was added. Broken test would mean
> > > it still works.
> > >
> > > Sergey
> > >
>
> <snipped>
next prev parent reply other threads:[~2020-06-01 9:24 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-02 6:38 Alexander V. Tikhonov
2020-04-02 15:44 ` Sergey Bronnikov
2020-05-15 17:04 ` Alexander V. Tikhonov
2020-05-18 10:00 ` Sergey Bronnikov
2020-06-01 9:24 ` Alexander V. Tikhonov [this message]
2020-06-04 10:33 ` Sergey Bronnikov
2020-06-04 15:50 ` Alexander V. Tikhonov
2020-06-08 20:10 ` Sergey Bronnikov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200601092449.GA5590@hpalx \
--to=avtikhon@tarantool.org \
--cc=sergeyb@tarantool.org \
--cc=tarantool-patches@dev.tarantool.org \
--subject='Re: [Tarantool-patches] [PATCH v1] Divide test box/net.box' \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox