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 019F19ECCFE; Thu, 7 Mar 2024 18:26:06 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 019F19ECCFE DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1709825166; bh=v5fi37989Oz1dyjeBFm6K/w939h4xWv/sl41478VgeY=; 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=Ki0GOK4OWYnNriKLW6wUa5fkF3QeS74Ln9YHdH6hPr37RPGBlBE0gOA00e+GuS5Zd Yb8yVCm2vVFkoP35e9BmCYlY8IoccZbMqJbDGafYSNDLz0KG8gEEduRDkBh6GW+0qa O4/AzZBWOiTEVCxfxOcYsFHfEXiPfia2NWjorBZI= Received: from smtp3.i.mail.ru (smtp3.i.mail.ru [95.163.41.67]) (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 B98EC3B29AA for ; Thu, 7 Mar 2024 18:26:04 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org B98EC3B29AA Received: by smtp3.i.mail.ru with esmtpa (envelope-from ) id 1riFdE-0000000AOWi-08LE; Thu, 07 Mar 2024 18:26:04 +0300 Date: Thu, 7 Mar 2024 18:22:05 +0300 To: Maxim Kokryashkin Message-ID: References: <20240306214724.32017-1-m.kokryashkin@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20240306214724.32017-1-m.kokryashkin@tarantool.org> X-Mailru-Src: smtp X-4EC0790: 10 X-7564579A: EEAE043A70213CC8 X-77F55803: 4F1203BC0FB41BD93B4404EFBC5FECACE5C610B1725ED3C661FD8B55A48E2EE5182A05F5380850404C228DA9ACA6FE27469FB80C6A9397A5C591814E25D11F9FBA2DB138D809966E3C1473A85485602194B48DAF27F3F842 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7D9C4478D0B876341EA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F79006375121C8F3070B83748638F802B75D45FF36EB9D2243A4F8B5A6FCA7DBDB1FC311F39EFFDF887939037866D6147AF826D83E168325B709779010D887A3B102FE9D1DAD9D2D4D242B7FCC7F00164DA146DAFE8445B8C89999728AA50765F7900637DCE3DBD6F8E38AFD389733CBF5DBD5E9C8A9BA7A39EFB766F5D81C698A659EA7CC7F00164DA146DA9985D098DBDEAEC8B861051D4BA689FCF6B57BC7E6449061A352F6E88A58FB86F5D81C698A659EA7E827F84554CEF5019E625A9149C048EE33AC447995A7AD182BEBFE083D3B9BA73A03B725D353964B0B7D0EA88DDEDAC722CA9DD8327EE4930A3850AC1BE2E7356C9A9530EBF72002C4224003CC83647689D4C264860C145E X-C1DE0DAB: 0D63561A33F958A52A70EB58C34793D45002B1117B3ED6967F1D6927E61186FFE772F934B9BCD185823CB91A9FED034534781492E4B8EEADF5E532225D4D775BBDAD6C7F3747799A X-C8649E89: 1C3962B70DF3F0ADE00A9FD3E00BEEDF3FED46C3ACD6F73ED3581295AF09D3DF87807E0823442EA2ED31085941D9CD0AF7F820E7B07EA4CFAD6A4626C2D22E05C71084EEC242C48B3301A6E0DA81898505EE618E89200269C69CBF34C6AF1FF5A3565A1AB386769012DE5A974F517B77ACB49E95F0EA9C720A7774F37BE69C2EC226CC413062362A913E6812662D5F2A5EAB5682573093F7837F15F2B5E4A70B33F2C28C22F508233FCF178C6DD14203 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojH2M812Ji8Af9hI9afdxAmQ== X-DA7885C5: 7636B2EA32259CFDF255D290C0D534F9192E8FB6CEE90BC76D7F5C23D5C55C514BD348A12E23F8405B1A4C17EAA7BC4BEF2421ABFA55128DAF83EF9164C44C7E X-Mailru-Sender: 689FA8AB762F7393590D8C940224AE33EAEAAD96B5C3266AE5FFBF58410CA5008F3280F28D65175EE49D44BB4BD9522A059A1ED8796F048DB274557F927329BE89D5A3BC2B10C37545BD1C3CC395C826B4A721A3011E896F X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH luajit v3] ci: extend tarantool integration testing 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 Kaplun via Tarantool-patches Reply-To: Sergey Kaplun Cc: tarantool-patches@dev.tarantool.org Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" Hi, Max! Thanks for the fixes! Please, consider my general comments below. 1) Now we have 6 types of names (please correct me if I forgot something): * Build test ... / LuaJIT ... * Exotic builds testing / LuaJIT ... * Sanitizers testing / LuaJIT ... * Static analysis / LuaJIT linters * Testing / LuaJIT ... * Testing / Tarantool ... As part of the patch: I) "Testing / Tarantool ..." is renamed to "Tarantool exotic builds testing / Tarantool ...". I do not agree that these are exotic builds -- these are ordinary builds, where we run the integration of LuaJIT tests under Tarantool. Plus, there is duplicate information that this is Tarantool-related testing. Perhaps we should not change the name of the jobs, meaning that by all "Testing" we mean LuaJIT tests. II) We bring all sorts of other things from Tarantool: * Testing / test-tarantool-static_build / static_build (push) * Tarantool ecosystem integration testing / test-tarantool-integration / tarantool-python / run_tests (push) Here, we have a lot of names that are mixed together and don’t look consistent with the older ones. I would suggest at least starting to fit into the overall picture by renaming it like this, using the following naming (if I understand correctly that we cannot change the job names at the end): * "Integration / Tarantool static_build" or "Integration / Tarantool / static_build" * "Integration / Tarantool ecosystem / ..." or "Integration / Tarantool / ecosystem / ..." I like the first mentioned options more than the second. 2) About all "Integration" workflows mentioned above: I suggest file naming like integration-tarantool-ecosystem.yml etc. for consistency. 3) It would be nice to have an option to skip integration testing. This can be done later. I just thought that sometimes it would be useful to specify the "-no-integration" branch suffix and then not run integration tests with Tarantool (namely "Integration", not our tests under Tarantool) - useful if you are debugging only our tests. On 07.03.24, Maxim Kokryashkin wrote: > > Workflow name |+/-| Reason > ----------------------------------------------------------------- > codeql | - | Not relevant to LuaJIT. > coverage | + | Long tests for profilers. > coverity | - | Cron workflow. > debug | + | Tarantool debug build. > debug_aarch64 | + | Tarantool debug build. > debug_asan_clang | + | Tarantool debug build. > default_gcc_centos_7 | + | gcc version (7) Minor: Old gcc version? > freebsd | - | Nightly build. > fuzzing | - | Impossible to bump LuaJIT. > integration | + | Tarantool ecosystem. > jepsen-cluster-txm | - | Manual workflow. > jepsen-cluster | - | Manual workflow. > jepsen-single-instance-txm | - | Cron workflow. > jepsen-single-instance | - | Cron workflow. > lango-stale-reviews | - | Cron workflow. > lint | - | LuaJIT has its own lint. > luajit-integration | + | Exotic LuaJIT options. I suggest the following reasoning: "Run tests under Tarantool" or something like that. > memtx_allocator_based_on_malloc | - | Not relevant to LuaJIT. > osx | - | Nightly build. > out_of_source | + | Out of source build. > packaging | - | No LuaJIT-relevant variety. > perf_cbench | - | Not enabled for PRs. > perf_linkbench_ssd | - | Not enabled for PRs. > perf_micro | - | Not relevant to LuaJIT. > perf_nosqlbench_hash | - | Not enabled for PRs. > perf_nosqlbench_tree | - | Not enabled for PRs. > perf_sysbench | - | Not enabled for PRs. > perf_tpcc | - | Not enabled for PRs. > perf_tpch | - | Not enabled for PRs. > perf_ycsb_hash | - | Not enabled for PRs. > perf_ycsb_tree | - | Not enabled for PRs. > publish-module-api-doc | - | No Doxygen in LuaJIT. > release | + | Tarantool release build. > release_asan_clang | + | Tarantool release build. > release_clang | + | Tarantool release build. > release_lto | + | Tarantool release build. > release_lto_clang | + | Tarantool release build. > reusable_build | - | Utility for integration. > source | - | Not enabled for PRs. > static_build | + | Tarantool static build. > static_build_cmake_linux | - | Just an OOS static build. > static_build_pack_test_deploy | - | Utility for packaging. > submodule_update | - | Not enabled for PRs. > > [1]: https://docs.github.com/en/actions/using-workflows/reusing-workflows#limitations > [2]: https://github.com/orgs/community/discussions/45342#discussioncomment-4779360 > --- > Changes in v3: > - Fixed comments as per review by Sergey Kaplun > - Fixed comments as per review by Sergey Bronnikov > > .github/workflows/tarantool-ecosystem.yml | 39 ++++++++ > .github/workflows/tarantool-exotic.yml | 67 +++++++++++++ > .github/workflows/testing.yml | 117 ++++++++++++++++------ -- Best regards, Sergey Kaplun