#145 Various updates and enhancements for the Package Update Guide
Merged by oturpe. Opened by adamwill.
fedora-docs/ adamwill/package-maintainer-docs update-guide-gating-tweaks  into  main

Download 145.patch

This is mainly about emphasizing the use of side tag builds and
updating the information on automated tests and waivers.

I tried to reflow the sections in a way that makes a bit more
sense.

I also threw out the whole long complicated bit on waiverdb-cli
because it should rarely be used any more, and it's a bit of a
footgun because it doesn't handle scenarios properly. It was
written back when we first tried to enable universal gating for
Fedora CI tests, and we did not have the waive button in Bodhi
at that time.

Signed-off-by: Adam Williamson awilliam@redhat.com

Thank you, this looks good. I'd just like to ask for a couple of small changes:

Tests shown with a red background failed.
Tests shown with a green background passed.
Tests shown with a blue background and an "i" icon had some minor issue, but effectively passed.
Tests shown with a blue background and an hourglass icon are pending.

This would be easier to read if it was a list.

but please check with the Fedora CI and/or Quality team

Contact method with a link would be great here, so that the reader would not have to hunt around for the contact method if they decide they actually need to do this.

Please do not waive failures of openQA tests -
those whose names start with update. -
without checking with the Fedora Quality team.

Same comment about contact method here.

Other than those, great update! This can be merged after the above comments have be considered and, if you agree with them, fixed.

rebased onto 6d02c750124883267104430ab18c385cce6b3253

Updated, thanks. I hope I got the xref syntax right; it's intended to point to https://docs.fedoraproject.org/en-US/ci/#_contact .

rebased onto 886ccf71d14458dcd276e739ae51bffa3f9ecec3

Pull-Request has been merged by oturpe

I tested the link, it does work. Merged. This is a large update, thank you for the effort!

According to the policy in README, commit access is granted to everybody with a merged pull request. Therefore I have added you to package-maintainer-docs group. You can now directly commit trivial fixes if you prefer that. Pull requests are also accepted for any kind of change, and strongly preferred for anything nontrivial.

Metadata