XSF Discussion - 2019-08-30


  1. Alex

    is it a concern that Python 3.3 builds are failing on xmpp.org CI?

  2. jonas’

    yes

  3. jonas’

    they are the syntax check for the JSON files

  4. jonas’

    I recently learnt that one can put descriptive labels on travis jobs

  5. jonas’

    we should do that ;)

  6. jonas’

    Alex, so check the build logs, they (should) explain what’s wrong

  7. jonas’

    if it’s not clear to you, send me a link to the failed build and I’ll look into it

  8. ralphm

    Just commented on the ticket.

  9. ralphm

    Apparently, Travis can not / no longer download 3.3.

  10. ralphm

    Which is also ancient.

  11. ralphm

    (2012)

  12. Alex

    jonas’: its this PR: https://github.com/xsf/xmpp.org/pull/604

  13. ralphm

    Maybe we should run this job with 3.7 instead.

  14. jonas’

    oh dear

  15. jonas’

    yeah

  16. jonas’

    or 3.6 or something

  17. jonas’

    3.7 was tricky with travis, too

  18. ralphm

    Alex: since this change doesn't actually touch the files being linted with Python 3, I think you can just merge.

  19. jonas’

    ralphm, sure? I think the member list is linted

  20. ralphm

    jonas’: really? 3.7.0 was released over a year ago

  21. jonas’

    ah it’s not

  22. jonas’

    ralphm, yes, but something about their infrastructure

  23. jonas’

    you need to change more than just the python version

  24. ralphm

    Well, it didn't help that they canned a bunch of people after the acquisition.

  25. ralphm

    jonas’: shall I have a go?

  26. ralphm

    jonas’: meanwhile, you agree Alex can merge?

  27. jonas’

    yes, go for a merge

  28. linkmauve

    When I did https://github.com/xsf/xmpp.org/pull/595 I didn’t encounter any visible issue, all tests passed.

  29. Alex

    usually I do not merge my own PRs, but happy to do so ;-)

  30. jonas’

    done

  31. Alex

    (Y)

  32. ralphm

    linkmauve: could be it is not failing all the time

  33. jonas’

    https://github.com/xsf/xmpp.org/pull/605

  34. jonas’

    let’s see how that goes

  35. ralphm

    jonas’: why not PR 595?

  36. jonas’

    because I didn’t click the link

  37. ralphm

    Python 3.6 was last released in December

  38. jonas’

    it’s obviously better

  39. ralphm

    yay

  40. jonas’

    although a smoke test that it actually still runs the tests would be good

  41. jonas’

    (which I’ll do in a sec)

  42. ralphm

    I triggered a rebuild: https://travis-ci.org/xsf/xmpp.org/builds/564874726

  43. ralphm

    And 3.7 succeeds

  44. ralphm

    Let's merge it.

  45. jonas’

    wait

  46. jonas’

    only merge it if this build fails: https://travis-ci.org/xsf/xmpp.org/jobs/578703384

  47. ralphm

    I assumed you would merge anyway :-D

  48. ralphm

    but yay, it fails