Re: ansible-review is run on pull requests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Thu, 14 May 2020 at 19:52, Nils Philippsen <nils@xxxxxxxxxx> wrote:
Hi everybody,

since a couple of hours, ansible-review is run on changes submitted
through pull requests in our Ansible repository on Pagure.

This means: when someone submits a PR to the ansible repo or pushes
changes into its source branch, a Zuul job is started which, when
finished, reports whether or not ansible-review finds problems in
changed playbooks/roles/... as a detailed comment (linking to test
results) and as a flag in the side bar (see [1] as an example).

Currently this is merely informational, i.e. if the Zuul job hasn't yet
finished or fails, this won't stop reviewers from merging the change
into the repository regardless. On the other hand, because we've set up
ansible-review to just process the changes submitted in the PR, this
takes relatively little time (think about 2 minutes from push to the
report coming in), so it should come in early enough unless you're in a
real rush. ;)

I'd like to thank Pingou, who did the preliminary work and structure
and especially Fabien Boucher who debugged the trouble we initially had
with our deployed version of Zuul[2] and contributed a workaround[3,4].

This is really cool, thanks to everyone involved 🎉🎉
 

Ciao,
Nils

[1]: https://pagure.io/fedora-infra/ansible/pull-request/62
[2]: https://pagure.io/fedora-infra/ansible/pull-request/54
[3]: https://pagure.io/fedora-infra/ansible/pull-request/60
[4]: https://pagure.io/fedora-zuul-jobs/pull-request/60
--
Nils Philippsen    "Those who would give up Essential Liberty to
Software Engineer   purchase a little Temporary Safety, deserve neither
Red Hat             Liberty nor Safety."  --  Benjamin Franklin, 1759
PGP fingerprint:  D0C1 1576 CDA6 5B6E BBAE  95B2 7D53 7FCA E9F6 395D
            old:  C4A8 9474 5C4C ADE3 2B8F  656D 47D8 9B65 6951 3011
_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux