On Sun, Jun 20, 2021 at 10:45 AM Miro HronĨok <mhroncok(a)redhat.com> wrote:
I think this is a good idea. This particular bot has a history of misbehavior
and rather than banning all the well behaving bots (that be definition we don't
even know about, because they behave good), we should disable this particular one.
Rather than "no bots allowed" policy, we might need a "bots that violate
our
policies and guidelines or have a tendency to break things will be disabled
until fixed" policy.
Yeah, that works for me too. Though I wouldn't want to make this a
special case and create an actual policy for this instead, that we
could point to when something like this happens.
I also think that I probably was not clear in my original message.
Any builds that are triggered by an actual human action, like
- scripted (mass) rebuilds with no *Version* changes,
- automatic builds after human-approved PRs,
- etc.
are of course exempt, because the action of an actual human being
triggered them.
The only thing I *don't* want is: Bots submitting builds for new
*Versions*, without human interaction.
Regarding Zbyszek's point:
Second, I think the guideline is simply wrong. As counterexamples,
we
currently have python3.10beta2 in rawhide, systemd-249-rc1, and
kernel-5.13.0-0.rc6. Pushing pre-release vesions of low-level packages
is a crucial part of development of the distro and collaboration with
upstream projects and language ecosystems.
There's actually already an exemption in the Updates Policy for those cases.
And I don't have any problem with those, because those builds are
prepared, built, tested, and shepherded by actual humans, instead of
created by a bot that just throws them at the wall to see what sticks
and what doesn't.
If you look at bodhi updates for rhcontainerbot it's pretty obvious
that nobody even looks at the updates that are created for those
builds:
https://bodhi.fedoraproject.org/updates/?search=&status=testing&u...
It looks like any build that receives -1 karma or fails gating tests
will just be stuck in "testing" until obsoleted by the next automated
build for that package.
Fabio