[travis] Update SDK_URL #8304

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1606-travisSDKURL changing 1 files +1 −1
  1. MarcoFalke commented at 3:26 PM on July 5, 2016: member

    @theuni @jonasschnelli Not sure on this one but the core sever can use whitelisting to avoid DOS and "other things".

  2. [travis] Update SDK_URL fa6ad56948
  3. MarcoFalke added the label Build system on Jul 5, 2016
  4. MarcoFalke added the label Tests on Jul 5, 2016
  5. luke-jr commented at 3:50 PM on July 5, 2016: member

    Wherever it is, please ensure all the Travis IPs get whitelisted. They're currently not, so Mac builds are failing for at least Knots.

  6. MarcoFalke commented at 3:55 PM on July 5, 2016: member

    Mac builds are failing for at least Knots.

    I am wondering if this was fixed for 0.13 by #8067.

  7. luke-jr commented at 4:02 PM on July 5, 2016: member

    I don't see why it would be?

  8. MarcoFalke commented at 4:08 PM on July 5, 2016: member

    I don't see why it never happened in the bitcoin core repo, or did I just miss it? As #8067 and previous pulls removed the special flags for the bitcoin repo, it should happen for us as well...

  9. luke-jr commented at 4:42 PM on July 5, 2016: member

    Not sure, maybe Travis reuses the same servers for repositories.

  10. jonasschnelli commented at 1:42 PM on July 6, 2016: contributor

    Travis seems happy now. I guess @theuni did update the whitelist. Does travis use different outbound IPs?

    I'll try now to re-trigger travis a couple of times to make sure it will work on different travis instances.

  11. luke-jr commented at 5:02 PM on July 6, 2016: member

    It seems to be reliably failing for the bitcoinknots/bitcoin repo at least as of a week ago. I just started a retry here: https://travis-ci.org/bitcoinknots/bitcoin/builds/142818285

    For reference, the full IP range is supposedly at: https://docs.travis-ci.com/user/ip-addresses/

  12. theuni commented at 5:29 PM on July 6, 2016: member

    I'm investigating this now. The ranges should be enabled, but I'm seeing failures that I can't pinpoint.

  13. theuni commented at 6:42 PM on July 6, 2016: member

    I believe I've tracked this down to a local server config issue. IPs weren't always presented correctly (hooray, Cloudflare!), so the filtering didn't work as expected. It should be pretty clear after a few travis builds if all is in order now.

  14. theuni commented at 6:43 PM on July 7, 2016: member

    ACK. This should be good to go now.

  15. jonasschnelli merged this on Jul 7, 2016
  16. jonasschnelli closed this on Jul 7, 2016
  17. jonasschnelli referenced this in commit 0cca2feb35 on Jul 7, 2016
  18. MarcoFalke deleted the branch on Jul 7, 2016
  19. codablock referenced this in commit 7639b45e7e on Sep 19, 2017
  20. codablock referenced this in commit e6b9863231 on Dec 27, 2017
  21. codablock referenced this in commit 5ff5474e47 on Dec 28, 2017
  22. andvgal referenced this in commit 224ef2ddfd on Jan 6, 2019
  23. DrahtBot locked this on Sep 8, 2021
Contributors
MarcoFalkeluke-jrjonasschnellitheuni
Labels
Build systemTests
github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-17 06:15 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me