travis: Post result to irc on failure #11912

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1712-travisIRCfail changing 1 files +11 −7

MarcoFalke commented at 7:37 PM on December 15, 2017: member

Replaces #11598 by @TheBlueMatt

I am not entirely happy with my solution, as it will spam the irc with each failed job individually, instead one notice per travis build. However, assuming that very rarely more than one job per build fails, this might be acceptable.
travis: Post result to irc on failure fa5798b207
MarcoFalke added the label Tests on Dec 15, 2017
TheBlueMatt commented at 7:41 PM on December 15, 2017: member

Cool!
MarcoFalke commented at 9:26 PM on December 15, 2017: member

Heh, please note that no notification is sent when the job times out.
promag commented at 2:29 AM on December 17, 2017: member

Concept ACK.

Where is irccat2.py?
laanwj commented at 9:52 AM on December 20, 2017: member

Concept ACK

However, assuming that very rarely more than one job per build fails, this might be acceptable.

If there's a bug, that will usually (unless it's platform specific) make all the jobs fails, or am I missing something?

Also: just to be clear, this is for the repository only, not for PRs I hope? (the "$TRAVIS_REPO_SLUG" = "bitcoin/bitcoin" check)
MarcoFalke commented at 2:04 PM on December 20, 2017: member
Yes,
- this will only trigger on branches in the bitcoin/bitcoin repo, that is, pushes to branches or cron jobs running on branches.
- if there is a bug (let's say silent merge conflict in init.cpp), then all jobs will fail and all jobs will report error on irc. But merging such an obvious bug should happen never to rarely.

in .travis.yml:88 in fa5798b207

  82 | @@ -83,3 +83,7 @@ script:
  83 |  after_script:
  84 |      - echo $TRAVIS_COMMIT_RANGE
  85 |      - echo $TRAVIS_COMMIT_LOG
  86 | +after_failure:
  87 | +    - pip3 install irc --user
  88 | +    - if [[ "$TRAVIS_REPO_SLUG" = "bitcoin/bitcoin" && ( "$TRAVIS_EVENT_TYPE" = "push" || "$TRAVIS_EVENT_TYPE" = "cron" ) ]]; then wget "https://raw.githubusercontent.com/jaraco/irc/60f747a578157b03c952c050c765e2924d4fdc00/scripts/irccat2.py"; fi

laanwj commented at 9:24 AM on December 21, 2017:

Repeating this condition is a bit ugly. I know every - is invoked as a separate script, but as this is YAML, isn't it possible to spread a script over multiple lines?

MarcoFalke commented at 8:40 PM on January 2, 2018:

Jup, that is possible

MarcoFalke commented at 8:40 PM on January 2, 2018: member

Closing, since it appears to be no longer working on travis, for me.
MarcoFalke closed this on Jan 2, 2018
MarcoFalke deleted the branch on Jan 2, 2018
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#11598 Make travis complain on #bitcoin-core-dev when builds fail