Authentication-Results: mail-b.sr.ht; dkim=pass header.d=dvn.me header.i=@dvn.me Received: from q.meshwith.me (q.meshwith.me [91.121.161.13]) by mail-b.sr.ht (Postfix) with ESMTPS id 3660211EF38 for <~sircmpwn/sr.ht-discuss@lists.sr.ht>; Mon, 5 Apr 2021 22:15:53 +0000 (UTC) Date: Mon, 5 Apr 2021 23:15:45 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=dvn.me; s=q; t=1617660951; bh=YekGYC5GzBUJwTDZOL+6vsuhfnW6RAftft4SJbuRkF8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=HHNVTTOTyR7XFzJ/gxvuKLJpOzbJiTW1htqfDmwg1eLGqNFyy1mbBnjxEeQHGXslT xxrUiELjqFp9xYJfG8EgDf8PAbFtn0mjm0acUmk3cFQVRL2YiseIj4yfEuZqp72VVp r3dSUuQPZk6XvT/SweIFQJprFCrGRID0Opczn9FWs+1cTm9ejNozHWPPfkTiDYcU+W +x+TQOk+TC51YCKMHK9FXH2kD7wrxBDsAWqlTTjubhufFCe5Pp5VseAOMb3tkRM+L7 Blng+1Kdd4miiomLwfkCatnU28nQoDgN9nH2fN+Vp+UvuA/c2lLOgnxOh44Vlg4sdq fQNydqYXpFlyHxaRMWpxkoxnLAKHc/IwxRFO6uOxL2IXDra7g0KMmq1LBBIfsy+OLC jwIUreQ1Z2xU58fvvr3tDQsojFMHAglQb4BNd3VAgbfSjoSKRuBf13oNlTZnU9tRE5 ndwFy3UnYMHshFPPfXmSadxJ0edV3x2B9/MGGESBK7/ibzkLeomIuPKjDnnSsFFh5V Pxr6z/ZdAfJ2FXySqADxUYAvxFA3YpwJaIk5MT9nZxc6DTs7xOqJrv1Bsjn8i3cRon 7JlOnl2DZ5UswUYoQfSYeqKx+7NTmJbtLGoxa9uOVIMb8V/Vy510v7fkZVJfV3xnJ9 Ajuvf28eu2nv4tP2QdnVGzCY= From: Devan Carpenter To: Ecmel Berk =?utf-8?Q?Canl=C4=B1er?= Cc: Drew DeVault , ~sircmpwn/sr.ht-discuss@lists.sr.ht Subject: Re: [builds.sr.ht] Resubmitted builds are not associated with original build's project Message-ID: <20210405221545.vhbvbjnrlonbsu2u@iyo> References: <34f77392-19fa-5104-736f-10293d857642@ecmelberk.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Ecmel Berk Canlıer transcribed some bytes: > > You should commit a proper fix upstream if you want to associate it > > with, well, the upstream project. > > Well, the issue with the build[1] wasn't caused by an upstream change, > though. That's why I resubmitted the build instead of committing something > upstream. > > 1: see task "setup" line 3 Yea, the intention does not line up with the actual useage, from my point of view. You can see that other CI systems, like Gitlab CI, handle this the opposite way. If you re-run a build manually then it will supercede the previous build status in the UI. This is different than running an "ad-hoc" build by uploading an arbitrary manifest. It makes sense because of situations exactly like what Ecmel ran into. The *external environment* was the source of the error, not something in their repo. Re-running the build _is_ the proper fix in this case.