https://pantsbuild.org/ logo
Powered by
#general
  • a

    ancient-stone-50795

    02/26/2016, 6:17 PM
    Is this a show stopper? after merging to the latest master I can't run this integration test by hand:
    Copy code
    $ ./pants test tests/python/pants_test/reporting:reporting_integration
...
13:15:22 00:01         [chroot]============== test session starts ===============
                     platform darwin -- Python 2.7.10 -- py-1.4.31 -- pytest-2.6.4
                     plugins: cov, timeout
                     collected 10 items 
                     
                     tests/python/pants_test/reporting/test_reporting_integration.py .........
FAILURE: Timeout of 60 seconds reached


13:16:22 01:01   [complete]
               FAILURE
  • a

    ancient-stone-50795

    02/26/2016, 6:19 PM
    (This was working for me this morning before I rebased.)
  • w

    witty-crayon-22786

    02/26/2016, 6:19 PM
    @ancient-stone-50795: you should bump up the timeout on that test
  • w

    witty-crayon-22786

    02/26/2016, 6:19 PM
    unless it is unexpected for it to be taking more than 60 seconds
  • w

    witty-crayon-22786

    02/26/2016, 6:19 PM
    test timeouts are now actually enabled, and the default is 60 seconds in pants.ini
  • a

    ancient-stone-50795

    02/26/2016, 6:19 PM
    =========== 10 passed in 58.45 seconds ===========
  • w

    witty-crayon-22786

    02/26/2016, 6:19 PM
    per target
  • w

    witty-crayon-22786

    02/26/2016, 6:20 PM
    ...and it includes the startup time =P
  • a

    ancient-stone-50795

    02/26/2016, 6:20 PM
    That run just passed, AFAIK, this is normal for some of our integration tests.
  • w

    witty-crayon-22786

    02/26/2016, 6:20 PM
    @ancient-stone-50795: yea, should bump that one up to 120 seconds or something then probably
  • w

    witty-crayon-22786

    02/26/2016, 6:21 PM
  • a

    ancient-stone-50795

    02/26/2016, 6:21 PM
    I found an example in 
    ./tests/python/pants_test/projects/BUILD
  • a

    ancient-stone-50795

    02/26/2016, 6:21 PM
    I'm glad we have per target timeouts now.
  • w

    witty-crayon-22786

    02/26/2016, 6:22 PM
    aye... thanks @limited-terabyte-34878
  • l

    limited-terabyte-34878

    02/26/2016, 6:23 PM
    😃 timeout time should not include the chroot time anymore
  • l

    limited-terabyte-34878

    02/26/2016, 6:23 PM
    if that’s what you mean by startup time
  • w

    witty-crayon-22786

    02/26/2016, 6:23 PM
    oh, neat.
  • l

    limited-terabyte-34878

    02/26/2016, 6:23 PM
    i believe i fixed that
  • a

    ancient-stone-50795

    02/26/2016, 6:25 PM
    @witty-crayon-22786: that change is updated and baking in CI. Travis started it right away so I should be able to merge it pretty soon.
  • w

    witty-crayon-22786

    02/26/2016, 6:26 PM
    ok, i can wait an hour or so
  • a

    ancient-stone-50795

    02/26/2016, 7:59 PM
    @witty-crayon-22786: it's in now
  • a

    ancient-stone-50795

    02/26/2016, 8:39 PM
    Created #C0P9B6BST for anyone that wants to discuss the upcoming summit. The agenda is here: https://docs.google.com/document/d/1rRarRlxhe_r8PiViV9zJtuNCuJQV2FQlgKUI-6bocjc/edit#
  • a

    ancient-stone-50795

    02/26/2016, 8:39 PM
    PantsSummitAgenda3March2016.txt
    PantsSummitAgenda3March2016.txt
  • w

    witty-crayon-22786

    02/26/2016, 10:07 PM
    whelp, looks like i'm going to be doing the release much, much later
  • w

    witty-crayon-22786

    02/26/2016, 10:08 PM
    need to get on the road
  • a

    ancient-stone-50795

    02/26/2016, 10:25 PM
    Hi folks, we've got a job opening at Square on our Build team. Working on Pants isn't the only thing that it would encompass, but that would be a significant part of it: http://jobs.jobvite.com/careers/square/job/olS30fw0
    👍 1
  • b

    bored-art-40741

    02/26/2016, 10:26 PM
    Twitter folks: how is your remote/shared artifact cache implemented?
  • b

    bored-art-40741

    02/26/2016, 10:27 PM
    I'm having a really hard time believing that this isn't a solved problem
  • f

    full-kilobyte-93429

    02/26/2016, 10:27 PM
    @bored-art-40741: not in the context of new engine right? just to be sure
  • b

    bored-art-40741

    02/26/2016, 10:28 PM
    Yeah
1...101810191020...2420Latest