From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 8 Dec 2020 20:10:03 +0300 From: "Dmitry V. Levin" To: =?koi8-r?B?7s/XycvP1yDzxdLHxco=?= Message-ID: <20201208171002.GA30549@altlinux.org> References: <20201203181402.vvsliz3vxrshyahr@altlinux.org> <20201203182634.GC16060@altlinux.org> <20201203183000.tlu4x6kxoxfd6vyy@altlinux.org> <20201203184400.ybj33xx4jmf3o7lu@altlinux.org> <9244c0cc-d4ed-f264-7d33-6cca005f1065@basealt.ru> <20201205162922.GA14677@altlinux.org> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Cc: ALT Linux Team development discussions Subject: Re: [devel] Q: ON_QA 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, 08 Dec 2020 17:10:03 -0000 Archived-At: List-Archive: List-Post: On Mon, Dec 07, 2020 at 01:55:35PM +0300, Новиков Сергей wrote: > 05.12.2020 19:29, Dmitry V. Levin пишет: [...] > > Если стало легче, то я предлагаю завести новое состояние, например, ON_QA, > > и придумать более подходящий workflow, чем тот, который сложился сейчас, > > для тех репозиториев, в которых есть внешний QA. > Можно добавить следующие статусы: Давайте всё-таки отличать атрибуты и состояния: > 1. ON_MAINT_REVIEW - таск появился в списке --needs-approval=maint Мне кажется, что это классический EPERM. > 2. IN_QA_QUEUE - таск появился в списке --needs-approval=tester > 3. ON_QA_REVIEW - QA взяли таск в работу. С точки зрения сборочницы эти состояния (ожидание QA и собственно QA) неотличимы, поскольку происходят снаружи. Если бы сборочница обслуживала очередь QA, тогда разные состояния были бы естественны, а так непонятно, в чём разница, помимо атрибута, не влияющего ни на что, кроме внешнего вида задания. Например, если бы сборочница обслуживала какую-то очередь CI, то можно было бы предложить состояния вроде AWAITING_CI и ON_CI. > 4. READY_TO_COMMIT - получены апрувы от @maint и @tester На мой взгляд, это атрибут, а не состояние. Состояний у задания с этим атрибутом может быть несколько: PENDING, COMMITTING, а также AWAITING, BUILDING, и т.д. > 5. REJECTED_BY_MAINT - отклонен группой @maint Мне кажется, что это классический EPERM. Я пока не вижу, чем ON_MAINT_REVIEW и REJECTED_BY_MAINT отличаются как состояния (т.е. чем отличаются графы переходов из этих состояний). > 6. REJECTED_BY_QA - отклонен QA Это состояние задания я тоже пока не понимаю. В какие другие состояния возможен переход из этого состояния? -- ldv