[14:11] <stokachu> anyone ever run into https://bugs.launchpad.net/juju-core/+bug/1380337?
[14:11] <mup> Bug #1380337: adding machine after destroying another fails <add-machine> <cloud-installer> <destroy-machine> <juju-core:New> <https://launchpad.net/bugs/1380337>
[17:08] <rogpeppe2> minor update to the juju errors package to use the new location for errgo: https://github.com/juju/errors/pull/8
[17:08] <rogpeppe2> reviews appreciated
[17:27] <TheMue> rogpeppe2: LGTM
[17:28] <rogpeppe2> TheMue: thanks!
[17:41] <TheMue> rogpeppe2: yw
[17:46] <rogpeppe2> anyone know anything about the bitbucket.org/kardianos/osext and bitbucket.org/kardianos/service dependencies?
[17:46] <rogpeppe2> they don't appear to be used by anything
[17:46] <jcw4> rogpeppe2: aren't they windows dependencies?
[17:46] <rogpeppe2> jcw4: possibly, but i haven't found anything that uses them
[17:47] <jcw4> rogpeppe2: I see... it's only in cmd/jujud/main_windows.go that it's even imported
[17:47] <jcw4> osext is a dependency of service
[17:48] <rogpeppe2> jcw4: ah, thanks
[17:48] <rogpeppe2> jcw4: i don't know why i didn't find that
[17:48] <jcw4> rogpeppe2: I just remember the second level dependency issue when I bumped into the same question recently
[17:49] <rogpeppe2> jcw4: it's a pity that the dependencies aren't findable by godeps. i should fix that.
[17:49] <jcw4> yeah. since it's only in the _windows.go files go needs a little prodding to find them I guess
[17:49] <jcw4> (when you're not on windows)
[17:50] <rogpeppe2> jcw4: yeah. godeps should try with different build flags.
[18:22] <bodie_> did someone ping me or is my IRC client highlighting this channel for no reason?  sigh
[20:44] <jcw4> rogpeppe2 I'm seeing this error in builds:
[20:44] <jcw4> Extant directories unknown:
[20:44] <jcw4>  gopkg.in/errgo.v1
[20:45] <jcw4> for example: http://juju-ci.vapour.ws:8080/job/github-merge-juju/914/console
[20:46] <jcw4> doesnt make sense to me because nothing in juju master refers to that yet, so it must be a transitive dependency
[20:50] <jcw4> Also, I can't find any transitive dependencies on that yet either.
[20:50] <jcw4> mgz: I think it's a build script issue... do you know why it's trying to pull in gopkg.in/errgo.v1 since that isn't in the dependencies.tsv, and doesn't seem to be a transitive dependency either
[20:52] <thumper> jcw4: is it from the juju/charms package?
[20:53] <rick_h_> jcw4: that was a branch that landed today
[20:53] <rick_h_> jcw4: make sure to update, I saw it go by in the email
[20:54] <jcw4> rick_h_: I am fully updated to juju master
[20:54] <jcw4> thumper: I'm trying to decipher what "it" is :)
[20:54] <rick_h_> jcw4: looking
[20:54] <jcw4> this error is when the build script is initially pulling down dependencies
[20:54]  * thumper sighs
[20:54] <thumper> found an intermittent test failure in worker/provisioner
[20:56] <jcw4> thumper: I've seen worker/provisioner fail a couple times intermittently, but I see worker/peergrouper fail fairly often locally
[20:56] <rick_h_> jcw4: https://github.com/juju/errors/pull/8 is the pr I was thinking of
[20:56]  * thumper sighs heavily
[20:56] <jcw4> rick_h_: yeah, that was why I pinged rogpeppe initially
[20:56]  * thumper is going to *fix* juju/errors
[20:57] <jcw4> YAY
[20:57] <jcw4> what baffles me is why the build script is even pulling it at all
[20:57] <jcw4> juju-core doesn't reference it yet
[20:57] <jcw4> and I couldn't find any dependencies of juju-core that referenced it either
[20:58] <rick_h_> like thumper said, probably juju/charm
[20:58] <thumper> I was told that errgo wasn't going to sneak into dependent packages
[20:58] <jcw4> thumper, rick_h_ I don't see how it could have
[20:58] <thumper> and it was only going to be used outside of juju/errors in the charmstore
[20:59] <rick_h_> thumper: jcw4 ok, justcccc ccbgjgvcveghhillduiklnblheujreenhtsnkubdrcjf
[20:59] <rick_h_> bah gueccccsccbgjgvscuneviintrdikfgbvegt
[20:59] <rick_h_> idhvdfvudkdkjb
[20:59] <rick_h_> double bah
[20:59] <thumper> wat?
[20:59] <rick_h_> called rick sitting in the hotel bumping his yubikey
[20:59] <thumper> rick_h_: you ok?
[20:59] <jcw4> haha
[20:59] <rick_h_> no, I'm still in brussels :P
[20:59] <thumper> rick_h_: you lucky fella
[21:00] <jcw4> rick_h_: did you go see the Atomium?
[21:00] <thumper> rick_h_: I was looking at the schedule for the first few days
[21:00] <thumper> $ git diff master | wc -l
[21:00] <thumper> 1731
[21:00] <thumper> hmm
[21:00] <thumper> this branch isn't going to break up well
[21:01] <urulama> thumper: errgo should stay within charmstore limits, yes.
[21:01]  * thumper puts hacking hat back on and ignores all y'all for a while
[21:03] <thumper> ah crap...
[21:04] <thumper> new struct from a merge has broken my work...
[21:04]  * thumper works around it
[21:04] <rick_h_> chase them down and make them eat pie
[21:05] <jcw4> wait, before committing to that course of action... which struct?
[21:05] <jcw4> rick_h_: isn't it LATE in Brussels?
[21:05] <jcw4> 11pm or so?
[21:06] <rick_h_> jcw4: yes
[21:06] <thumper> jcw4: not you
[21:06]  * jcw4 wonders if the initial pull for imports uses the HEAD revision of juju/errors to find child imports
[21:07] <urulama> thumper: btw, let me know if errgo somehow sneaks into the code ...
[21:07] <jcw4> thumper: whew
[21:07] <thumper> jcw4: was cmars's branch for login v2
[21:07] <thumper> urulama: ack
[21:07] <thumper> urulama: you home?
[21:07] <urulama> thumper: was having fun trying to QA all the "landed" branches from last week ... could have missed some though
[21:07] <urulama> thumper: i am
[21:08] <urulama> thumper: my flight was about 1h 20min ;)
[21:08] <thumper> urulama: I have a branch for juju/errors that needs updating based on our conversations in nuremberg
[21:08] <thumper> urulama: I got home yesterday just before lunch
[21:08] <thumper> urulama: working today/tomorrow taking thu/fri off in lieu
[21:08] <thumper> but today is finishing off what I started pre-sprint
[21:09] <urulama> thumper: enjoy :)
[21:09] <thumper> \o/
[21:09] <urulama> thumper: seen that errors branch, looks nice
[21:10] <thumper> I'll see if I can get to it tomorrow
[21:10] <thumper> or may this afternoon if I can fix all the test failures I caused
[21:10] <thumper> by changing a bucketload of API calls
[21:10] <urulama> that's always a good thing! :D
[21:10]  * thumper is full steam ahead on finishing identity bits
[21:11] <thumper> OOPS: 268 passed, 6 FAILED, 5 PANICKED
[21:11] <thumper> need to fix the failed and panicked
[21:11] <jcw4> yikes
[21:11] <thumper> :)
[21:11] <jcw4> :)
[21:11] <jcw4> where is the code that actually runs the CI build?  I dont' think its the jenkins-github-lander repo
[21:11] <urulama> don't panic
[21:12] <thumper> urulama: thanks for that :)
[21:12] <thumper> jcw4: not entirely sure sorry
[21:12] <jcw4> thanks thumper you can ignore me while you're dealing with your panic attacks
[21:12] <jcw4> :)
[21:13] <thumper> cheers buddy
[21:13]  * thumper goes back to it
[21:13] <jcw4> I bet its https://launchpad.net/juju-ci-tools
[21:17] <mgz> jcw4: rog borked it
[21:17] <jcw4> mgz: yay :)
[21:18] <mgz> with this change: https://github.com/juju/errors/pull/8
[21:18] <jcw4> mgz: it seems like the build script might be getting imports from the HEAD revisions of first level imports?
[21:19] <jcw4> mgz: right, but that's just on the errors repo, and juju-core wasn't updated to use that revision in the dependencies.tsv
[21:19] <mgz> yeah, something's up
[21:19] <mgz> godeps likely doesn't do exactly the right thing
[21:20] <jcw4> mgz: yeah, interesting
[21:21] <mgz> I'll have a poke around
[21:23] <thumper> mgz: just revert the merge for now
[21:23] <thumper> mgz: poke around later
[21:23] <mgz> jcw4: the repo is just getting lp:juju-release-tools and running `./make-release-tarball.bash master`
[21:24] <thumper> mgz: this is how we deal with broken builds
[21:24] <thumper> dvcs FTW
[21:24] <mgz> thumper: fair enough
[21:24] <jcw4> thumper: thanks!
[21:24] <jcw4> mgz: thanks!
[21:28] <jcw4> mgz: I see the problem I think
[21:28] <jcw4> compare_dependencies in check_dependencies.py doesn't seem to like any packages that are not accounted for in the dependencies.tsv file
[21:29] <jcw4> I bet godeps is just pulling down second level imports before updating first level imports to the specified revision
[21:30] <jcw4> which would bring gopkg.in/errgo.v1 into the pristine GOPATH, and then compare_dependencies complains
[21:58] <jcw4> mgz: thumper, rogpeppe2 -- I reverted https://github.com/juju/errors/pull/8 so that juju-core will build again.
[21:58] <jcw4> rogpeppe2: sorry! Hopefully we can figure out the build process so that we can land your change again
[21:58] <thumper> jcw4: kk