search

fedora-copr / copr

RPM build system - upstream for https://copr.fedorainfracloud.org/
115 stars 59 forks source link

cron jobs must not print anything to stdout/stderr #1289

Open fedora-copr-github-bot opened 1 year ago

fedora-copr-github-bot commented 1 year ago

Original issue: https://pagure.io/copr/copr/issue/1289 Opened: 2020-02-25 07:26:49 Opened by: praiskup

If something is printed to stdout/stderr, whole infra team is notified via email.

That said, .. we should change the simple printing to logging, and keep stdout stderr clean.

praiskup commented at 2020-02-25 08:28:25:

No, the mail goes to root, which goes to root@fedoraproject.org (per previous discussion with @kevin).

praiskup commented at 2020-02-26 15:18:29:

For now applied this: https://infrastructure.fedoraproject.org/cgit/ansible.git/commit/?id=a6f03027a19a2b21571bf19762eae6aa4427f746

praiskup commented at 2021-04-19 10:27:15:

the ansible.git hashes were recalculated after push-force, so the link above doesn't work ... I've set MAILTO=praiskup@redhat.com to not bother fedora infra.

So the problem still exists, we should minimize the cron job output and perhaps start sending the leftovers to copr-team@.

praiskup commented 1 year ago

Changed to 'copr-team@' for now.