https://pantsbuild.org/ logo
Powered by
#development
  • e

    enough-analyst-54434

    07/20/2018, 8:15 PM
    Jesus - ok
  • w

    witty-crayon-22786

    07/20/2018, 8:15 PM
    the tool is the equivalent of 
    git clone $SHA
    for a snapshot
  • w

    witty-crayon-22786

    07/20/2018, 8:16 PM
    remoting workers use it to check out a snapshot before executing
  • e

    enough-analyst-54434

    07/20/2018, 8:16 PM
    Much like the PATH issue for hermetic, doing all work for every master/stable branch is a hack that barely works
  • e

    enough-analyst-54434

    07/20/2018, 8:16 PM
    Things like this expose the flaw in the hack
  • e

    enough-analyst-54434

    07/20/2018, 8:16 PM
    Its not generally tenable
  • e

    enough-analyst-54434

    07/20/2018, 8:16 PM
    I'll introduce extra shards then to keep throughput up.
  • w

    witty-crayon-22786

    07/20/2018, 8:17 PM
    it's a question of what is a public API vs a private API
  • e

    enough-analyst-54434

    07/20/2018, 8:17 PM
    and public releases vs back door
  • w

    witty-crayon-22786

    07/20/2018, 8:18 PM
    these are public.
  • w

    witty-crayon-22786

    07/20/2018, 8:18 PM
    in that anyone may consume a nightly
  • w

    witty-crayon-22786

    07/20/2018, 8:19 PM
    supporting that has a cost, and yes, twitter is the only one consuming them. but any company that is actually trying to maintain internal compatibility with HEAD should be as well
  • e

    enough-analyst-54434

    07/20/2018, 8:20 PM
    I'm saying I disagree with that, we can't afford - in general - to support per-commit - not even nightly - binary releases
  • e

    enough-analyst-54434

    07/20/2018, 8:21 PM
    A Twitter of Foursquare should be able to have an imac and build extras if needed
  • e

    enough-analyst-54434

    07/20/2018, 8:21 PM
    1 imac
  • w

    witty-crayon-22786

    07/20/2018, 8:21 PM
    everything other than this utility is necessary for pants' own CI
  • e

    enough-analyst-54434

    07/20/2018, 8:22 PM
    yup
  • e

    enough-analyst-54434

    07/20/2018, 8:22 PM
    exactly
  • w

    witty-crayon-22786

    07/20/2018, 8:22 PM
    so if we added an integration test to pants' core ci for this, it would be necessary for CI
  • e

    enough-analyst-54434

    07/20/2018, 8:22 PM
    yes
  • w

    witty-crayon-22786

    07/20/2018, 8:22 PM
    it is a public API without integration tests. it is no less a public API
  • e

    enough-analyst-54434

    07/20/2018, 8:23 PM
    That is not the case today and I stare at optimizing CI and see an extra 2 minutes burned
  • e

    enough-analyst-54434

    07/20/2018, 8:23 PM
    per-shard
  • e

    enough-analyst-54434

    07/20/2018, 8:23 PM
    At any rate, I'll move these off to side-shards. I think my uncomfotableness with the way we support per-sha releases is well known
  • w

    witty-crayon-22786

    07/20/2018, 8:23 PM
    @enough-analyst-54434: one option would be to move it to a before_deploy of the rust-tests shards.
  • e

    enough-analyst-54434

    07/20/2018, 8:27 PM
    Agreed. Plenty of ways to solve, primary frustration is finding this compilation in a spot that is surprising.
  • w

    witty-crayon-22786

    07/20/2018, 8:28 PM
    yea, understood. Daniel will have a better idea of next steps there. it doesn't need to stay the way it is now indefinitely, but certainly through the next month or so
  • w

    witty-crayon-22786

    07/20/2018, 8:31 PM
    and thank you for your work here... it's much appreciated.
  • w

    witty-crayon-22786

    07/20/2018, 8:32 PM
    i would like to re-discuss the cost of HEAD releases at some point though, as they have proven absolutely invaluable
  • w

    witty-crayon-22786

    07/20/2018, 8:32 PM
    (and i really don't know how much the S3 storage costs vs the CPU time)
1...566567568...906Latest