From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [87.239.111.99] (localhost [127.0.0.1]) by dev.tarantool.org (Postfix) with ESMTP id 874821BFCA1D; Thu, 30 Apr 2026 13:36:49 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 874821BFCA1D DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1777545409; bh=I3pWFJapJ8MO0BkIG+ooBWrp8U/i167vi3VQdV3ZKAc=; h=Date:To:References:In-Reply-To:Subject:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=bFt22lRXU12g7zTj+pHDWTKQeq8RrZl2RCe1J1/hn4EQ4YDEmtDG4npMUmU27qcGP hgWwRH+Pq2LVIYbKcrSayYlP3jR7J+QiV7xT68o5xSd699MpgpShz36s1pHFYShvG8 BeUn932djdbPmT99KsFjuITvkllQpEuIPmqIAkbE= Received: from send278.i.mail.ru (send278.i.mail.ru [95.163.59.117]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 825723AB05E for ; Thu, 30 Apr 2026 13:36:47 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 825723AB05E Received: by exim-smtp-6767fb646-2c7lp with esmtpa (envelope-from ) id 1wIOlC-00000000Cno-1C2A; Thu, 30 Apr 2026 13:36:46 +0300 Content-Type: multipart/alternative; boundary="------------LhxOv0YnniaISJ6Tmw1OFdGZ" Message-ID: <012c8041-6fb3-44b7-90fb-29d5aca9ef3b@tarantool.org> Date: Thu, 30 Apr 2026 13:36:45 +0300 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Content-Language: en-US To: Sergey Kaplun References: <20260423170806.9647-1-skaplun@tarantool.org> <398afbb0-66c6-4935-878f-ed01356fe740@tarantool.org> <2c608e1b-cb9e-42f0-8e51-1560d4c5f4b3@tarantool.org> In-Reply-To: X-Mailru-Src: smtp X-4EC0790: 10 X-7564579A: 646B95376F6C166E X-77F55803: 4F1203BC0FB41BD9DA546775E7411B7C07A03F9D84DEF93856E2E14D298482C900894C459B0CD1B939758AE0B44792600578E6996F38341322730F354E8E13CC2E9F0B2ADB1D41EDA43962510F43F2FA X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE73B44982FA5E78411EA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F7900637AC83A81C8FD4AD23D82A6BABE6F325AC2E85FA5F3EDFCBAA7353EFBB553375666AD5665AF5FCF4F0296C2BB6DD71D5CC107594415073230A8413F38D1B70C003389733CBF5DBD5E913377AFFFEAFD269176DF2183F8FC7C0A3E989B1926288338941B15DA834481FCF19DD082D7633A0EF3E4896CB9E6436389733CBF5DBD5E9D5E8D9A59859A8B64854413538E1713FCC7F00164DA146DA6F5DAA56C3B73B237318B6A418E8EAB86D1867E19FE14079C09775C1D3CA48CF3D321E7403792E342EB15956EA79C166A417C69337E82CC275ECD9A6C639B01B78DA827A17800CE7AEA1580DED4E70E3731C566533BA786AA5CC5B56E945C8DA X-C1DE0DAB: 0D63561A33F958A5D584DFED279BAB835002B1117B3ED6964BCD41ADE608316FB48B7A7F94616420823CB91A9FED034534781492E4B8EEAD09F854029C6BD0DABDAD6C7F3747799A X-C8649E89: 1C3962B70DF3F0AD73CAD6646DEDE1918E10F71CB4DF9F96AB70F9BE574AE9C625B6776AC983F447FC0B9F89525902EE6F57B2FD27647F25E66C117BDB76D65985E1413DDF159870E2E797AF2EDF70DC7B692FACCD7740F27D3C9A9147EC5016CDA60176D5938248B8341EE9D5BE9A0A0431FB670CD589461DC031AF1905D5728D2FB21C32BF2C7A6536EB022892E5344C41F94D744909CE2512F26BEC029E55448553D2254B8D95CD72808BE417F3B9E0E7457915DAA85F X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu53w8ahmwBjZKM/YPHZyZHvz5uv+WouB9+ObcCpyrx6l7KImUglyhkEat/+ysWwi0gdhEs0JGjl6ggRWTy1haxBpVdbIX1nthFXMZebaIdHP2ghjoIc/363UZI6Kf1ptIMVW+UUD87bVUobpthEai4xiI= X-DA7885C5: 442F087F843D719BF255D290C0D534F93C5A8A6D5D671E288F8129DACF02EA023A3199C3CD60DC205B1A4C17EAA7BC4BEF2421ABFA55128DAF83EF9164C44C7E X-Mailru-Sender: 689FA8AB762F7393520AF17B8A65FDE2A031E0A25EF66DF41927BCB719C90725BE8CA82D526A6A3BEF86D5F70DA33880E41E8EF7A07863ECB274557F927329BE2DDF8182D28ACDB545BD1C3CC395C826B4A721A3011E896F X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH luajit] ci: conditionally install lua-cjson for perf job X-BeenThere: tarantool-patches@dev.tarantool.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Sergey Bronnikov via Tarantool-patches Reply-To: Sergey Bronnikov Cc: tarantool-patches@dev.tarantool.org Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" This is a multi-part message in MIME format. --------------LhxOv0YnniaISJ6Tmw1OFdGZ Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hi, Sergey, thanks! LGTM (the job https://github.com/tarantool/luajit/actions/runs/25105723095 is failed, so I restarted it) Sergey On 4/29/26 14:15, Sergey Kaplun wrote: > Hi, Sergey! > > Updated the commit message to the following: > > | ci: conditionally install lua-cjson for perf job > | > | We have encountered the flakiness of the network on our CI runners. It > | happens only on lua-cjson installation and only in some particular job > | run. > | > | This patch works around it by installing the package only if it is not > | installed in the system. Also, it installs it only for Lua 5.1, which is > | compatible with LuaJIT, to avoid installation of modern Lua versions by > | default, since Lua 5.1 is not the default for most modern distros. > | > | Relates to tarantool/tarantool#12600 > > Branch is force-pushed. > > On 29.04.26, Sergey Bronnikov wrote: >> Hi, >> >> thanks for the comments. LGTM after updating a commit message. >> >> Sergey >> >> On 4/28/26 15:29, Sergey Kaplun wrote: >>> Hi, Sergey! >>> Thanks for the review! >>> Please consider my answers below. >>> >>> On 28.04.26, Sergey Bronnikov wrote: >>>> Hi, Sergey! >>>> >>>> A few things worry me: we still don't know the root cause of the problem. >>> It should be investigated and fixed in the scope of the 12600 [1]. For >>> LuaJIT's CI, this workaround works just fine. >> Probably it is worth to add to the issue that the problem is affected >> LuaJIT CI as well. > Added. > >>>> The proposed patch merely reduces the frequency of module installations. >>> Yes, it installs the module only once. >>> >>>> And if an installation is required, the installation will still break. >>> Not always, it usually passes successfully and fails only on the fourth >>> run. So, 1 successful installation is enough to prevent flakiness, which >>> is OK for our needs in LuaJIT. It is better to have some statistics >>> instead of having none, isn't it? >> Agree. >>>> See comments inline. >>>> >>>> Sergey >>>> >>>> On 4/23/26 20:08, Sergey Kaplun wrote: >>>>> We have encountered the flakiness of the network on our CI runners. It >>>>> happens only on lua-cjson installation and only in some particular job >>>>> run. >>>>> >>>>> This patch helps to deal with it by installing this package only if it >>>> s/helps to deal with it/helps to reduce a number of installations/ >>> Don't get the comment about the number of installations. Maybe rephrase >>> it like the following? >>> | This patch workarounds it by installing the package only if ... >> Ok, let's rephrase like you suggested. >> >> Also, it is worth referring to the issue #12600 ("See also #12600"). > Rephrased. > >>>>> is not installed in the system. Also, it installs it only for Lua 5.1, >>>>> which is compatible with LuaJIT. >>>>> --- >>>>> >>>>> Branch:https://github.com/tarantool/luajit/tree/skaplun/ci-perf-install-conditionally-lua-cjson >>>>> >>>>> .github/actions/setup-performance/action.yml | 4 +++- >>>>> 1 file changed, 3 insertions(+), 1 deletion(-) >>>>> >>>>> diff --git a/.github/actions/setup-performance/action.yml b/.github/actions/setup-performance/action.yml >>>>> index 4e0e1929..3c2a8230 100644 >>>>> --- a/.github/actions/setup-performance/action.yml >>>>> +++ b/.github/actions/setup-performance/action.yml >>>>> @@ -11,7 +11,9 @@ runs: >>>>> apt install -y curl luarocks util-linux >>>>> shell: bash >>>>> - name: Install Lua modules >>>>> - run: luarocks install lua-cjson >>>>> + run: > >>>>> + luarocks --lua-version=5.1 show lua-cjson || >>>>> + luarocks --lua-version=5.1 install lua-cjson >>>> Why do we need specifying Lua version now? Previously, it worked without >>>> setting exact version. >>> I've preferred to describe it specifically since the most modern distros >>> provide Lua 5.3+ by default. Should I add the comment in the commit >>> message? >> Yes, please. > Added. > >>>>> shell: bash >>>>> - name: Run script to setup Linux environment >>>>> run: sh ./perf/helpers/setup_env.sh >>> [1]:https://github.com/tarantool/tarantool/issues/12600 >>> --------------LhxOv0YnniaISJ6Tmw1OFdGZ Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 7bit

Hi, Sergey,

thanks! LGTM

(the job https://github.com/tarantool/luajit/actions/runs/25105723095 is failed,

so I restarted it)

Sergey

On 4/29/26 14:15, Sergey Kaplun wrote:
Hi, Sergey!

Updated the commit message to the following:

| ci: conditionally install lua-cjson for perf job
|
| We have encountered the flakiness of the network on our CI runners. It
| happens only on lua-cjson installation and only in some particular job
| run.
|
| This patch works around it by installing the package only if it is not
| installed in the system. Also, it installs it only for Lua 5.1, which is
| compatible with LuaJIT, to avoid installation of modern Lua versions by
| default, since Lua 5.1 is not the default for most modern distros.
|
| Relates to tarantool/tarantool#12600

Branch is force-pushed.

On 29.04.26, Sergey Bronnikov wrote:

Hi,

thanks for the comments. LGTM after updating a commit message.

Sergey

On 4/28/26 15:29, Sergey Kaplun wrote:

Hi, Sergey!
Thanks for the review!
Please consider my answers below.

On 28.04.26, Sergey Bronnikov wrote:

Hi, Sergey!

A few things worry me: we still don't know the root cause of the problem.

It should be investigated and fixed in the scope of the 12600 [1]. For
LuaJIT's CI, this workaround works just fine.

Probably it is worth to add to the issue that the problem is affected 
LuaJIT CI as well.

Added.



          

            
The proposed patch merely reduces the frequency of module installations.


          

          
Yes, it installs the module only once.



          

            
And if an installation is required, the installation will still break.


          

          
Not always, it usually passes successfully and fails only on the fourth
run. So, 1 successful installation is enough to prevent flakiness, which
is OK for our needs in LuaJIT. It is better to have some statistics
instead of having none, isn't it?


        

        
Agree.


        

          

            
See comments inline.

Sergey

On 4/23/26 20:08, Sergey Kaplun wrote:


            

              
We have encountered the flakiness of the network on our CI runners. It
happens only on lua-cjson installation and only in some particular job
run.

This patch helps to deal with it by installing this package only if it


            

            
s/helps to deal with it/helps to reduce a number of installations/


          

          
Don't get the comment about the number of installations. Maybe rephrase
it like the following?
| This patch workarounds it by installing the package only if ...


        

        
Ok, let's rephrase like you suggested.

Also, it is worth referring to the issue #12600 ("See also #12600").


      

      
Rephrased.



      

        

        

          

          

            

              
is not installed in the system. Also, it installs it only for Lua 5.1,
which is compatible with LuaJIT.
---

Branch:https://github.com/tarantool/luajit/tree/skaplun/ci-perf-install-conditionally-lua-cjson

   .github/actions/setup-performance/action.yml | 4 +++-
   1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/.github/actions/setup-performance/action.yml b/.github/actions/setup-performance/action.yml
index 4e0e1929..3c2a8230 100644
--- a/.github/actions/setup-performance/action.yml
+++ b/.github/actions/setup-performance/action.yml
@@ -11,7 +11,9 @@ runs:
           apt install -y curl luarocks util-linux
         shell: bash
       - name: Install Lua modules
-      run: luarocks install lua-cjson
+      run: >
+        luarocks --lua-version=5.1 show lua-cjson ||
+        luarocks --lua-version=5.1 install lua-cjson


            

            
Why do we need specifying Lua version now? Previously, it worked without
setting exact version.


          

          
I've preferred to describe it specifically since the most modern distros
provide Lua 5.3+ by default. Should I add the comment in the commit
message?


        

        
Yes, please.


      

      
Added.



      

        

          

          

            

              
         shell: bash
       - name: Run script to setup Linux environment
         run: sh ./perf/helpers/setup_env.sh


            

          

          
[1]:https://github.com/tarantool/tarantool/issues/12600



        

      

      

    

  
  


--------------LhxOv0YnniaISJ6Tmw1OFdGZ--