Skip to content

Conversation

@ken2812221
Copy link
Contributor

From https://travis-ci.org/ken2812221/bitcoin-verify-commits/builds
I have run vecify-commits.py nightly on travis, as you can see that 30 minutes is not enough, it took 30-50 minutes to run the script with no extra options. So change it to 50 minutes would be better. Forgot to change this at #13066

From https://travis-ci.org/ken2812221/bitcoin-verify-commits/builds
I have run vecify-commits.py nightly on travis, as you can see that 30 minutes is not enough, it took 30-50 minutes to run the script with no extra options. So change it to 50 minutes would be better.
@fanquake fanquake added the Tests label Jun 12, 2018
@jnewbery
Copy link
Contributor

utACK 51ed05a , although I wonder if the nightly cron Travis build will time out due to verify-commits and the extended test suite.

@maflcko
Copy link
Member

maflcko commented Jun 12, 2018

@jnewbery This is the linter stage, which comes with a 50 minute quota on its own, independent from make and tests.

maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Jun 13, 2018
…g commits

51ed05a travis: Increase travis_wait time while verifying commits (Chun Kuan Lee)

Pull request description:

  From https://travis-ci.org/ken2812221/bitcoin-verify-commits/builds
  I have run vecify-commits.py nightly on travis, as you can see that 30 minutes is not enough, it took 30-50 minutes to run the script with no extra options. So change it to 50 minutes would be better. Forgot to change this at bitcoin#13066

Tree-SHA512: fced346edcb7dc626fa6e714d7db9a34d3e9f283ada64e19c30565ed214f72e26a57a905b4eb71ab24a16c97a1bd35e57e9a83dbb7dc1fff2db26505b810e61e
@maflcko maflcko merged commit 51ed05a into bitcoin:master Jun 13, 2018
@ken2812221 ken2812221 deleted the patch-1 branch June 13, 2018 13:31
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants