| wallyworld | axw: hiya, i'd love a second lgtm on http://reviews.vapour.ws/r/302 if possible | 00:04 |
|---|---|---|
| fwereade | jw4, what was the last thing you saw me say? | 00:12 |
| fwereade | jw4, last thing I saw you say was (figuratively) | 00:12 |
| wallyworld | perrito666: ping | 00:18 |
| wallyworld | bug 1384001 was marked as Fix Committed in juju core, except it's not. It's only fix committed in core when the dependencies file is updated | 00:20 |
| mup | Bug #1384001: Juju doesn't retry hard enough when destroying MAAS environments <cloud-installer> <destroy-environment> <maas-provider> <Go MAAS API Library:Fix Committed by rvb> <juju-core:In Progress by wallyworld> <MAAS:Fix Committed by rvb> <https://launchpad.net/bugs/1384001> | 00:20 |
| jw4 | fwereade: I wandered off when your autocomplete quit working | 00:21 |
| jw4 | fwereade: :) | 00:21 |
| jw4 | fwereade: I guess I don't see any way around serializing sequence() if we want to ensure ordered sequence numbers | 00:22 |
| jw4 | fwiw, I found that the unit sequence number is using a similar but less stringent mechanism using 'unitseq' right on the service doc | 00:23 |
| wallyworld | perrito666: ignore me | 00:23 |
| wallyworld | my godeps failed | 00:23 |
| wallyworld | sorry | 00:23 |
| * jw4 eod-ish | 00:30 | |
| axw | wallyworld: will take a look in a sec | 01:01 |
| wallyworld | np, ty | 01:01 |
| === kadams54 is now known as kadams54-away | ||
| === kadams54-away is now known as kadams54 | ||
| cmars | of all the api client tests, which one is a good example of how to mock test an api client? I don't want to follow the wrong example. | 01:53 |
| === kadams54 is now known as kadams54-away | ||
| perrito666 | wallyworld: I just came back from dinner, I am pretty sure I committed that and juju bot thinks likewise | 02:16 |
| wallyworld | perrito666: yeah, forget i even mentioned it, my master was out of date and godeps didn;t work, so toally my mstake | 02:17 |
| perrito666 | that is why you changed the state of the bug I assume :) | 02:17 |
| perrito666 | ok then, going to sleep, see you all tomorrow | 02:18 |
| wallyworld | perrito666: see ya. i did change the bug back to waht it shold be | 02:19 |
| wallyworld | cmars: sadly, in juju/api/client, I am not aware of a test suite which doesn't run up the full stack. it's todo to not do that, but i don't thinks it's been done yet anywhere :-( | 02:20 |
| waigani | menn0: wired up the step: http://reviews.vapour.ws/r/310/ | 02:30 |
| menn0 | waigani: will take a look shortly | 02:30 |
| menn0 | waigani: np | 02:31 |
| menn0 | wallyworld: here's part of the upgrade steps rework: http://reviews.vapour.ws/r/311/ | 02:35 |
| menn0 | part 2 even | 02:35 |
| wallyworld | menn0: ok, ta. just about to go to lunch, will look when i get back | 02:37 |
| menn0 | wallyworld: np | 02:37 |
| menn0 | waigani: change looks good | 02:40 |
| waigani | menn0: thanks | 02:40 |
| wallyworld | menn0: one query in the review | 03:25 |
| menn0 | wallyworld: looking | 03:26 |
| menn0 | wallyworld: just responded | 03:35 |
| wallyworld | looking | 03:35 |
| wallyworld | menn0: maybe make it so that if Tag != "", then run steps for fromVersion? But if we are already fully on fromVersion, seems unnecessary to run steps again? | 03:37 |
| menn0 | wallyworld: that's how it works already I believe | 03:37 |
| wallyworld | hmmm, ok. if thats how it works now, then we shouldn't change the behaviour in this branch perhaps. but seems wrong | 03:38 |
| menn0 | wallyworld: this branch changes the behaviour in terms of the target version | 03:39 |
| menn0 | wallyworld: those 2 extra tests just confirm already existing behaviour | 03:39 |
| menn0 | wallyworld: it makes sense because 1.21-alpha/beta < 1.21 | 03:39 |
| wallyworld | yeah, that appears true. i was just surpised to see it running steps unnecessarily | 03:39 |
| wallyworld | yes when Tag != "", run the steps | 03:40 |
| menn0 | wallyworld: so if you're upgrading past 1.21 the steps for 1.21 should get run | 03:40 |
| wallyworld | yes, but not if you're upgrading from 1.21 | 03:40 |
| menn0 | wallyworld: I wouldn't say it's running steps unnecessarily. more steps may have been added between alpha/beta and final. | 03:40 |
| wallyworld | yes, and in those cases, Tag != "" | 03:41 |
| wallyworld | and we don't support upgrading from dev versions | 03:41 |
| wallyworld | but anyway, if it's existing behaviour, no need to hld up this branch | 03:41 |
| menn0 | wallyworld: ok | 03:41 |
| wallyworld | just surprised me is all | 03:41 |
| wallyworld | ie i don't agree with it :-) | 03:42 |
| menn0 | wallyworld: I think the behaviour is ok but whatever :) | 03:42 |
| menn0 | wallyworld: I'll make that test change you suggested and re-push | 03:42 |
| wallyworld | well, so long as the steps are idempotent, it diesn't matter | 03:42 |
| wallyworld | but there's a cost to running unnecessary steps | 03:42 |
| wallyworld | just added a ship it, thanks for extra 1.22 change | 03:43 |
| menn0 | wallyworld: the problem is I guess, that we don't know which steps were run when coming from a alpha/beta release | 03:46 |
| wallyworld | yes, but we don't officially support upgrading from alpha/beta | 03:46 |
| menn0 | wallyworld: I'm not adding 1.22 to the test data btw. I just changed the releases to be 1.20-alpha/beta to 1.21 to take advantage of what's already there. | 03:46 |
| wallyworld | yep, +1 | 03:46 |
| menn0 | wallyworld: thanks. will merge now. | 03:48 |
| wallyworld | thanks for fixing | 03:48 |
| menn0 | wallyworld: FYI fwereade has asked me to look at the relation scope issue for subordinate relations | 04:09 |
| menn0 | wallyworld: where they should be scoped to the machine, not the principal service | 04:09 |
| menn0 | wallyworld: apparently it's affecting Landscaope | 04:09 |
| menn0 | Landscape | 04:09 |
| menn0 | wallyworld: ring any bells? | 04:09 |
| wallyworld | menn0: vaguely | 04:10 |
| wallyworld | menn0: as yes | 04:10 |
| wallyworld | they want to add a single global chamr on a machine | 04:11 |
| menn0 | wallyworld: he asked me check with you regarding the priority of this | 04:11 |
| menn0 | wallyworld: that's the one | 04:11 |
| wallyworld | menn0: if you ask landscape, very high | 04:11 |
| wallyworld | we did rule it out of cope this cycle | 04:11 |
| wallyworld | but | 04:11 |
| wallyworld | if you had bandwidth to fix, that would be amazeballs. but it will be big | 04:11 |
| menn0 | wallyworld: he's asked me to look because i've just been in that code to do the env UUID upgrade | 04:12 |
| wallyworld | and not just landscape want it, wil be very useful | 04:12 |
| wallyworld | ok, can you then see if you can spec out what the changes will be, so it can be estimated and scoped? | 04:12 |
| menn0 | I don't strictly have the cycles. thumper wants me focused on the multi-environment work | 04:13 |
| wallyworld | ok | 04:13 |
| wallyworld | in that case, leave it, as we already told them it would not be in scope this cycle | 04:13 |
| menn0 | i'll do the initial thinking and scoping and see how it looks | 04:13 |
| wallyworld | ok, ta, anything would be good as ti will be more than we have now | 04:13 |
| jog | good afternoon wallyworld, FYI bug 1364410 has become consistently reproducible again in Juju CI | 04:18 |
| mup | Bug #1364410: API server fails to start with "address already in use" in MachineSuite tests <ci> <intermittent-failure> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1364410> | 04:18 |
| wallyworld | hi | 04:18 |
| wallyworld | i wasn't aware of that bug, thanks for letting us know | 04:19 |
| wallyworld | jog: sadly, looking at the bug report, if it is the issue i think it is, it will not be easy to fix | 04:20 |
| jog | it was a blocking CI regression bug in early September but sinzui demoted it after it became intermittent and we were able to get a blessed build revision | 04:20 |
| wallyworld | any fix will likely be a lot of work from what i now remember after reading the bug, as it changes the way tests are set up | 04:21 |
| wallyworld | so i'm not sure off hand how timely a fix can be | 04:21 |
| wallyworld | i'll take a look, just got to finish some wip | 04:23 |
| jog | wallyworld, ok currently the last 10 ppc64el unit test runs have failed | 04:23 |
| wallyworld | i thought ppc64el was non voting? | 04:23 |
| jog | it was non-voting for awhile but once the compiler test was worked around/disabled we set it back to voting | 04:24 |
| wallyworld | maybe not | 04:24 |
| wallyworld | ah ok | 04:24 |
| wallyworld | i guess a call needs to be made if we want to ship alpha3 this week | 04:24 |
| jog | maybe this test needs to be disabled as well, so we can at least gain coverage from the rest? Depending on how risky this issue is to users? | 04:25 |
| jog | if a fix is not coming soon, disabling the entire suite is not ideal | 04:26 |
| jog | i.e. disable == make it non-voting | 04:27 |
| wallyworld | if it's only on ppc64el, maybe we can skip to get alpha3 out, and then unskip once that happens | 04:28 |
| jog | wallyworld, unit tests are passing for other arches but there are a number of other non-unit test failures preventing a blessed revision | 04:37 |
| wallyworld | jog: just local upgrades though i think? | 04:38 |
| wallyworld | and a fix is currently going through CI now | 04:38 |
| jog | I also see a functional-ha-backup-restore and kvm-upgrade-trusty-amd64 failure... I'll go see if I can find out why | 04:39 |
| wallyworld | ok, ta | 04:41 |
| wallyworld | axw: a small one http://reviews.vapour.ws/r/313 | 04:41 |
| wallyworld | ty :-) | 04:41 |
| axw | wallyworld: done | 04:46 |
| wallyworld | ty | 04:46 |
| wallyworld | jog: i think i know why local upgrade tests are failing | 05:02 |
| wallyworld | juju --show-log set-env -e local-upgrade-trusty-amd64 tools-metadata-url= | 05:02 |
| wallyworld | tools-metadata-url is deprecated, replaced by agent-metadata-url | 05:03 |
| wallyworld | config will be migrated on start up, but old attr name is not valid in set-env | 05:03 |
| wallyworld | so test script needs to be tweaked | 05:03 |
| === kadams54 is now known as kadams54-away | ||
| jog | wallyworld, hmm... I thought we could not depreciate something like that and would need to support both tools-metadata-url and agent-metadata-url until the major version changes? | 05:11 |
| wallyworld | we do support both | 05:11 |
| wallyworld | in config | 05:11 |
| wallyworld | the approach is to print a warning when deploying an environment, and copy across the value for people | 05:12 |
| wallyworld | thus if you have an environments.yaml with the old attr, it will be migrated | 05:12 |
| wallyworld | when you deploy your environment | 05:12 |
| wallyworld | we've not supported set-env in that way before | 05:13 |
| jog | I see... you said that above... so it's just set-env you think we need to update in the test scripts | 05:14 |
| wallyworld | not sure if that's something (set-env) we need to do or not | 05:14 |
| wallyworld | just set-env yes | 05:14 |
| wallyworld | for end customers, tools-metadata-url is only ever read only | 05:14 |
| wallyworld | they could update it, but there's no reason to | 05:15 |
| wallyworld | if stakeholder strongly feel we need to add deprecated handling to set-env, we could do that | 05:15 |
| wallyworld | it just hasn't come up before as an issue | 05:16 |
| jog | I could see it being an issue if enterprise user scripts break. CI certainly has scripting that's setting tools-metadata-url | 05:18 |
| jog | those same scripts are used for <1.21, so we'll need to add some checking | 05:20 |
| axw | wallyworld: I'm going to take a look at #1387421, then I'll look at the scp bugs | 05:23 |
| mup | Bug #1387421: cmd/juju: add help on placement directives (zones, maas host name) <cmdline> <docs> <juju-core:Triaged by axwalk> <https://launchpad.net/bugs/1387421> | 05:23 |
| wallyworld | ok. i'd be surprised in any enterprise user set tools-metadata-url after bootstrapping. but other depecated attrs, maybe they do | 05:23 |
| wallyworld | axw: ok, sounds good. i'm about to look into the critical blocker to see if there's any palatable fix | 05:23 |
| axw | okey dokey | 05:24 |
| wallyworld | jog: if i'm not around later past my EOD, could you ask curtis about his opinion on the set-env handling of deprecated attributes. it's not something we've supported in the past but if we need to we can add it | 05:29 |
| jog | wallyworld, I think he's out Friday | 05:29 |
| wallyworld | ok, we can talk next week | 05:30 |
| wallyworld | maybe if i get time i can look to add it | 05:30 |
| jog | If I had to bet though, risk of breaking external customers in the same way this breaks CI will make it an issue :-/ | 05:31 |
| wallyworld | except that external customers will not set tools-metadata-url | 05:31 |
| wallyworld | but yeah, for other maybe; i'd have to check to see what ones have been deprecated | 05:32 |
| jog | I know we're doing things a bit different, so maybe I'm wrong | 05:32 |
| wallyworld | for customers tools-metadata-url is static | 05:32 |
| wallyworld | set once in yaml | 05:32 |
| wallyworld | main use is for private clouds, and those are i think almost always openstack, and suck customers would use keystronr catalog instead i would think | 05:33 |
| wallyworld | jog: i'm looking at http://juju-ci.vapour.ws:8080/job/run-unit-tests-trusty-ppc64el/ as linked in the bg - i can't see test runs with evidence of that same bug | 05:38 |
| wallyworld | i've looked at a few console logs, there's failures, but they are openstack reated | 05:38 |
| wallyworld | or upgrade related | 05:38 |
| wallyworld | not related to the api server port issue | 05:39 |
| jog | wallyworld, I could have mis-diagnosed it as the same... I initially opened bug https://bugs.launchpad.net/juju-core/+bug/1387936 and it has a log with the failure we're seeing | 05:42 |
| mup | Bug #1387936: Unit tests on ppc64el trusty failing with streams resources not found <ci> <ppc64el> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1387936> | 05:42 |
| wallyworld | jog: there errors look like the control hooks aren't running, which is indicative of a compiler issue | 05:42 |
| wallyworld | jog: that bug you just pasted isn't a bug | 05:43 |
| wallyworld | juju will look for simplstream data in several places, and with debug on, it logs the places it tries but is unsuccessful | 05:44 |
| wallyworld | this helps developers diagnose issues if people complain their system won't bootstrap | 05:44 |
| wallyworld | the compiler issue i mention above in the past has been a gccgo vs golang go issue | 05:45 |
| wallyworld | they handle stack frames differently | 05:45 |
| wallyworld | so some low level stuff fails on gccgo | 05:45 |
| wallyworld | jog: on the basis of the above, it would be great if we could unblock landings from bug 1364410 | 05:46 |
| mup | Bug #1364410: API server fails to start with "address already in use" in MachineSuite tests <ci> <intermittent-failure> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1364410> | 05:46 |
| jog | ok... I thought all the compiler issues had been worked around. mgz would working with various people on that... I can drop the severity to high for now. | 05:49 |
| wallyworld | jog: i have no answer for why the hooks aren't being run - in the past it's been a compiler issue | 05:50 |
| jog | ok, it's set back to high | 05:50 |
| wallyworld | ty :-) | 05:50 |
| wallyworld | jog: i'll start the work to handle deprecated settings in set-env, so when it lands you can keep the CI scripts unchanged | 05:59 |
| jog | wallyworld, that would be awesome and less risky, in case of possible customer scripting... It's 11pm here I'm about to turn in, anything else before I go? | 06:01 |
| wallyworld | jog: nah, thanks for your help | 06:01 |
| jog | thanks to you as well | 06:01 |
| jog | FYI, I moved the functional-ha-backup-restore test from HP to AWS and it passed, so possibly HP cloud flakiness | 06:02 |
| jog | wallyworld, would you please comment in bug https://bugs.launchpad.net/juju-core/+bug/1387936 with your suspicions about the compiler? | 06:09 |
| mup | Bug #1387936: Unit tests on ppc64el trusty failing <ci> <ppc64el> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1387936> | 06:09 |
| hazmat | finally re azure.. instance level public ips.. http://msdn.microsoft.com/library/azure/dn690118.aspx | 07:42 |
| hazmat | axw, fwiw re storage, gce just added its first local disk availability .. not by itype.. by api allocation at creation (81usd per month per dev) https://cloud.google.com/compute/docs/local-ssd | 07:45 |
| axw | hazmat: 81 USD??? | 07:47 |
| axw | sounds like a lot | 07:47 |
| axw | so they never had ephemeral storage before? | 07:47 |
| hazmat | axw, no | 07:54 |
| hazmat | axw, they always did net-storage afaics, they don't have sizing options (its per gb on local ssd) and color of black (model t ref) is 375gb | 07:56 |
| axw | so I see. reasonably hefty | 07:57 |
| mattyw | morning all | 08:15 |
| TheMue | morning | 09:03 |
| mattyw | TheMue, morning | 09:29 |
| voidspace | TheMue: dimitern: ok if I'm late to standup by ten minutes? | 10:01 |
| dimitern | voidspace, np | 10:02 |
| voidspace | thanks | 10:02 |
| voidspace | dimitern: TheMue: omw | 10:09 |
| hazmat | davecheney, ping | 10:16 |
| voidspace | gsamfira: hey | 10:27 |
| davecheney | hazmat: ack | 10:28 |
| voidspace | gsamfira: dimitern asked if reboot deals with debug-hooks in the same way it does juju-run? | 10:32 |
| mattyw | is fwereade around today or off still? | 11:04 |
| fwereade | mattyw, I'm here | 11:04 |
| fwereade | mattyw, what can I do for you? | 11:04 |
| perrito666 | morning all | 11:09 |
| wallyworld___ | axw: thanks for scp fixes | 12:10 |
| wallyworld___ | fwereade: since you are ocr, can i pretty please get a review on a critical bug fix for alpha3 http://reviews.vapour.ws/r/317/ ? | 12:11 |
| axw | wallyworld___: nps | 12:11 |
| fwereade | wallyworld___, just toasting a sandwich, on it in 5 | 12:12 |
| wallyworld___ | sure, thanks | 12:12 |
| fwereade | wallyworld___, LGTM, maybe one test tweak | 12:20 |
| wallyworld___ | fwereade: ty | 12:20 |
| perrito666 | man in this coutry its easier to get uranium that a port replicator for my lenovo model | 13:29 |
| natefinch | perrito666: sorry, we'll have to move our 1:1, I have to go catch an escaped chicken | 13:45 |
| perrito666 | natefinch: np | 13:45 |
| perrito666 | best reason ever, btw | 13:46 |
| natefinch | I live to amuse. | 13:55 |
| perrito666 | natefinch: nah, no pics nor vids of you chasing a chicken, thats not fun | 13:56 |
| natefinch | perrito666: closest I can come (my daughter, not me): https://fbcdn-sphotos-g-a.akamaihd.net/hphotos-ak-xpa1/v/t1.0-9/10303745_10152646984367028_7439232690755188047_n.jpg?oh=4e262e18b2e2723819b2bb8e2b250b53&oe=54E3659E&__gda__=1423914888_2c43b0a7691dfd8982b5deb1773a1595 | 14:00 |
| perrito666 | is that the runnaway chicken? | 14:01 |
| natefinch | nah, just one she picked up a while back. | 14:01 |
| natefinch | I have surprisingly few pictures of the chickens :) | 14:01 |
| perrito666 | natefinch: she smiles too much I dont think she knows what will happen to that chicken in the future | 14:02 |
| natefinch | perrito666: we are actually due for a "harvest" in a week or so | 14:02 |
| wwitzel3 | yum | 14:03 |
| perrito666 | a good things about chicken is that they loose the cute factor when they are ready to eat | 14:03 |
| natefinch | Yeah, the only hard part is the initial act of going from chicken the bird to chicken the meat. Really, once the feathers are off, they're not so much different than what you see in the grocery store. | 14:04 |
| natefinch | perrito666: moonstone? | 14:05 |
| perrito666 | well once you tags.Strip("<head>") they kind of no longer look like chickens anyway | 14:05 |
| natefinch | haha yeah | 14:06 |
| alexisb | natefinch, does your team have time to take a look at https://bugs.launchpad.net/juju-core/+bug/1388073 | 14:26 |
| mup | Bug #1388073: set-env no longer accepts empty values <ci> <compatibility> <regression> <set-env> <juju-core:Triaged> <https://launchpad.net/bugs/1388073> | 14:26 |
| alexisb | we need to get that resolved asap | 14:26 |
| natefinch | alexisb: ok | 14:27 |
| alexisb | thanks | 14:27 |
| tasdomas | natefinch, alexisb - I'm fixing #1388073 | 15:11 |
| mup | Bug #1388073: set-env no longer accepts empty values <ci> <compatibility> <regression> <set-env> <juju-core:Triaged> <https://launchpad.net/bugs/1388073> | 15:11 |
| alexisb | tasdomas, thanks! | 15:13 |
| mgz | tasdomas: okay, I want to yell at you | 15:22 |
| mgz | why cd17b6f3 | 15:23 |
| tasdomas | mgz, because we had that same piece of code pasted over 4 different places? | 15:24 |
| mgz | you wrote it wrong again! | 15:24 |
| mgz | straight up regresses bug 1379208 | 15:25 |
| mup | Bug #1379208: TestAddMetrics fails on gccgo <ci> <gccgo> <regression> <test-failure> <juju-core:Fix Released by gz> <https://launchpad.net/bugs/1379208> | 15:25 |
| perrito666 | Hey I keep having power outages, running to greener pastures now | 15:42 |
| mgz | tasdomas: see bug 1387936 | 15:51 |
| mup | Bug #1387936: Unit tests on ppc64el trusty failing <ci> <ppc64el> <regression> <juju-core:Triaged> <https://launchpad.net/bugs/1387936> | 15:51 |
| mgz | has link to one of the test runs | 15:51 |
| alexisb | mgz, in comment #3 of that bug, (2) Is Ian's change for a critical bug fix or is it something we can revert? | 15:55 |
| mgz | alexisb: I'm still trying to tease out if it's an accidental merge issue or an intended change, there are like 70 files touched, which makes a straight revert annoying | 15:57 |
| mgz | I'll add more details in a second | 15:58 |
| alexisb | mgz, ack, let core know how we can help | 15:58 |
| voidspace | yay, back online | 15:58 |
| voidspace | not sure if it's permanent or temporary | 15:58 |
| alexisb | and thanks mgz for triaging! | 15:58 |
| voidspace | they're still futzing with the line | 15:58 |
| alexisb | voidspace, are you one of those that gets nervous when they are disconnected | 15:59 |
| voidspace | alexisb: might have been the most productive hour of the day ;-) | 15:59 |
| voidspace | alexisb: although I compulsively click things when I'm thinking | 15:59 |
| alexisb | lol | 15:59 |
| voidspace | alexisb: and it's distracting when the clicks do nothing... | 15:59 |
| katco | fwereade: ping | 16:22 |
| fwereade | katco, pong | 16:23 |
| katco | fwereade: what i think is a reasonable draft of that spec is up | 16:24 |
| fwereade | katco, lovely, thanks | 16:24 |
| katco | fwereade: wallyworld didn't see anything too egregious, so i think we're ready for your +1 | 16:24 |
| katco | fwereade: (or not as the case may be) :) | 16:24 |
| fwereade | katco, brilliant, I *might* get to it tonight but I'm afraid I'm off to a halloween thing at laura's school imminently | 16:25 |
| katco | fwereade: how dare you care about your family! | 16:25 |
| fwereade | katco, ;p | 16:25 |
| katco | fwereade: totally understand :) have fun and HAPPY HALLOWEEN! :D | 16:25 |
| fwereade | katco, laura is dressed as a dinosaur :) | 16:26 |
| katco | fwereade: oh dear. pictures!! | 16:26 |
| katco | fwereade: mine is a ladybug :) | 16:26 |
| fwereade | aww :) | 16:26 |
| katco | alexisb: is yours going as a tractor or something similar? :) | 16:26 |
| alexisb | katco, he is going as a lion as he loves the "roar" | 16:27 |
| katco | alexisb: aw :) | 16:27 |
| natefinch | katco: have you seen the ladtbug girl books? | 16:28 |
| katco | natefinch: i have not! link please!! | 16:28 |
| natefinch | http://www.ladybuggirl.com/ | 16:28 |
| katco | natefinch: ok you are DIRECTLY responsible for me not being productive for the next bit of time. | 16:29 |
| natefinch | katco: great art and good morals and strong girl lead | 16:29 |
| katco | natefinch: love it | 16:29 |
| natefinch | katco: yep :) | 16:29 |
| natefinch | katco: Lily, our eldest: https://fbcdn-sphotos-d-a.akamaihd.net/hphotos-ak-xpf1/v/t1.0-9/p526x296/10624919_10152847646912028_8733903205136776490_n.jpg?oh=1ba16b1062ac2694854a3a2c57886f6a&oe=54E5B3A7&__gda__=1423915697_a6cde75ffa599528b4531dab195d58d5 | 16:32 |
| katco | natefinch: oh my she is DARLING! | 16:32 |
| katco | is that... a donut on a rope? haha | 16:33 |
| natefinch | haha.. halloween tradition, like bobbing for apples, but more sanitary | 16:33 |
| katco | lol | 16:33 |
| natefinch | our younger daughter, Zoë: https://scontent-a-iad.xx.fbcdn.net/hphotos-xap1/v/t1.0-9/10538654_10152847638537028_3233943882042977754_n.jpg?oh=8f5b74c34d729c2175c4a99e3895f8d7&oe=54EB41C1 | 16:34 |
| katco | natefinch: that smile is infectious :D | 16:35 |
| natefinch | the giggles even more so :) | 16:36 |
| katco | you have some cute ladies :D | 16:36 |
| natefinch | katco: thanks :) | 16:45 |
| alexisb | natefinch, love the picture of Zoe! | 16:52 |
| alexisb | Lily is also very cute, but Zoe's smile is awesome | 16:52 |
| tasdomas | natefinch, mgz, http://reviews.vapour.ws/r/320/ - should fix part of 1387936 and 1388073 | 17:11 |
| mgz | tasdomas: thanks! | 17:12 |
| * perrito666 is back | 17:22 | |
| alexisb | mgz, so is the last blocking the commit from Ian? | 17:23 |
| mgz | alexisb: I'm still not sure where it started breaking | 17:25 |
| mgz | ideally someone in the US can follow up and track down the point - it's confused by one of the failures being a variable segfault when running the tests | 17:26 |
| alexisb | natefinch, I know you have tasks today, perrito666, katco, ericsnow, tasdomas, cmars ^^ can any of you help out | 17:27 |
| alexisb | we need to get 1.21 released so this is top priority | 17:27 |
| alexisb | fwereade, ^^ | 17:27 |
| ericsnow | alexisb: I'll take a look but no one should let that stop them from looking too :) | 17:28 |
| mgz | ericsnow: basically, test with -compiler gccgo in provider/openstack is fubar | 17:29 |
| mgz | I'm not sure exactly which change regressed, it's very messy | 17:29 |
| ericsnow | mgz: that means I would need to set things up to test against an openstack provider? | 17:30 |
| mgz | ericsnow: nope just the unit tests | 17:31 |
| mgz | go test -compiler gccgo | 17:31 |
| ericsnow | mgz: k | 17:31 |
| tasdomas | could somebody take another look at http://reviews.vapour.ws/r/320/ ? | 17:34 |
| jrwren_ | natefinch: <3 ladybug girl. I read all of those to my duahgter. | 17:37 |
| ericsnow | mgz: so should I expect "go test -compiler gccgo" to have failed? (it didn't) | 17:37 |
| mgz | it fails in provider/openstack for me | 17:38 |
| perrito666 | well ubuntu upgrade plus power issues let my machine in a not so nice shape | 17:38 |
| ericsnow | mgz: I just did an "apt-get install gccgo" before running it (didn't have gccgo installed yet) | 17:39 |
| natefinch | jrwren_: Yeah, we got one as a gift, loved it, and have been getting all the rest of them. | 17:39 |
| perrito666 | mgz: go test -compiler gccgo for provider/openstack or all of it? | 17:39 |
| ericsnow | mgz: never mind (was in the state directory) | 17:40 |
| mgz | perrito666: provider/openstack is the interesting bit | 17:41 |
| mgz | ericsnow: you have the failures/segfault now? | 17:41 |
| ericsnow | mgz: running | 17:41 |
| * perrito666 runs all of provider just in case | 17:41 | |
| natefinch | ls | 17:45 |
| natefinch | heh | 17:45 |
| voidspace | agent cloudinit dependencies.tsv etc mongo rpc upgrades | 17:45 |
| natefinch | holy shit it worked | 17:46 |
| voidspace | :-) | 17:46 |
| ericsnow | mgz: aborted ("munmap of stack space failed: errno 22") in apiserver | 17:47 |
| perrito666 | mgz: ok github.com/juju/juju/provider/openstack51.623s | 17:47 |
| perrito666 | master | 17:47 |
| perrito666 | go test -compiler gccgo github.com/juju/juju/provider/... | 17:47 |
| ericsnow | mgz: (still running) | 17:48 |
| natefinch | on master I can't even get the test to compile in that directory | 17:48 |
| natefinch | /tmp/go-build382239753/github.com/juju/juju/provider/openstack/_test/_testmain.go:52:15: error: reference to undefined identifier ‘testing.MainStart’ | 17:48 |
| natefinch | m := testing.MainStart(matchString, tests, benchmarks, examples) | 17:48 |
| natefinch | ^ | 17:48 |
| natefinch | FAILgithub.com/juju/juju/provider/openstack [build failed] | 17:48 |
| natefinch | hm... I wonder if I screwed myself by upgrading to go 1.4 yesterday | 17:49 |
| mgz | hm... | 17:49 |
| perrito666 | mgz: I seem to be immune to these issues | 17:49 |
| natefinch | ahh, I think I know what it is... the new go tool running with the old gccgo ... funny | 17:50 |
| mgz | perrito666: gccgo version? | 17:51 |
| perrito666 | gccgo (Ubuntu 4.9.1-16ubuntu6) 4.9.1 | 17:52 |
| perrito666 | ubuntu 14.10 | 17:52 |
| ericsnow | mgz: segfault in github.com/juju/juju/environs/simplestreams and in github.com/juju/juju/provider/openstack | 17:53 |
| ericsnow | mgz: also, github.com/juju/juju/worker/uniter/jujuc failed | 17:53 |
| ericsnow | mgz: so more than just provider/openstack failed for me | 17:54 |
| mgz | the jujuc one tasdomas has branch up fixing | 17:54 |
| mgz | I think simplestreams/openstack is the same regression, but I have been failing to get it to pass by reverting the changes I think may be to blame | 17:55 |
| natefinch | I need one of these: http://goo.gl/OPhW6a | 17:56 |
| mgz | eheeh | 17:56 |
| ericsnow | mgz: the segfaults looked similar | 17:56 |
| ericsnow | mgz: "panic during panic" | 17:57 |
| mgz | ericsnow: you sometimes get other panics as well :) | 17:58 |
| natefinch | yay for gccgo | 17:58 |
| ericsnow | mgz: for the 2 segfaults: "/home/esnow/projects/go-workspace/src/github.com/juju/juju/environs/simplestreams/simplestreams_test.go:23" and /home/esnow/projects/go-workspace/src/github.com/juju/juju/provider/openstack/provider_test.go:29 | 17:59 |
| tasdomas | mgz, ericsnow, natefinch - landing my fix | 17:59 |
| ericsnow | mgz: for both a separate goroutine was sitting at the same spot in provider/dummy/environs.go, in case that matters | 18:00 |
| natefinch | tasdomas: awesome, thanks for getting that in, I know it's late on friday for you | 18:00 |
| ericsnow | natefinch, tasdomas: +1 | 18:01 |
| tasdomas | natefinch, it was my f-up to begin with | 18:01 |
| perrito666 | mgz: can I give you any more info? | 18:01 |
| mgz | perrito666: paste me a clean run with -v? | 18:02 |
| perrito666 | natefinch: no you dont | 18:03 |
| voidspace | wwitzel3: you need a review on 318 | 18:04 |
| voidspace | wwitzel3: juju-run | 18:05 |
| natefinch | perrito666: haha | 18:05 |
| perrito666 | I mean, really why would they print that, its not even a dynamic link | 18:05 |
| ericsnow | mgz: which issue are we looking at? | 18:06 |
| mgz | ericsnow: the segfaults | 18:06 |
| perrito666 | mgz: http://pastebin.ubuntu.com/8763738/ | 18:06 |
| perrito666 | mgz: hardly informative | 18:06 |
| ericsnow | mgz: I mean the launchpad issue | 18:07 |
| mgz | ericsnow: so, what I *tried* to do was back to a known good change to narrow down what the cause was | 18:07 |
| ericsnow | mgz: yeah, which would that be? | 18:07 |
| mgz | but I have been failing to get a clean run, even though the test on jenkins was blue as of 29th rev f2292c20 | 18:08 |
| mgz | I suspect it's either Ian's giant re-landing pr #988 or the later pr #996 but that seems to not be right? | 18:09 |
| ericsnow | mgz: same here (metrics-related compile failure) | 18:09 |
| ericsnow | mgz: at f2292c20 | 18:09 |
| mgz | ericsnow: run godeps there? | 18:10 |
| ericsnow | mgz: that's better (running now) | 18:11 |
| ericsnow | mgz: same panic at f2292c20 | 18:15 |
| mgz | yeah... hence my general confusion | 18:16 |
| wwitzel3 | voidspace: please :) | 18:16 |
| perrito666 | clearly its an issue with our toolchain versions | 18:16 |
| ericsnow | mgz: I get all sorts of failures (not panics) in the provider/openstack tests at ae4d5862a42c443e15f3fe49c2322010fecc83b3 | 18:22 |
| mgz | ericsnow: run multiple times as well, the results are not consistent on the same version | 18:23 |
| * ericsnow tries at a1a74d5ff07a30942c48b75ccd2fb1bbf8dad59a (where sinzui merged in from his alpha3 branch) | 18:24 | |
| ericsnow | mgz: ah, okay | 18:25 |
| voidspace | wwitzel3: hmmm... my EOD now | 18:28 |
| voidspace | dammit | 18:29 |
| voidspace | wwitzel3: it looks pretty straightforward though | 18:29 |
| ericsnow | mgz: same results (74 passed, 2 skipped, 5 FAILED, 1 PANICKED) at a1a74d5ff07a30942c48b75ccd2fb1bbf8dad59a | 18:29 |
| voidspace | wwitzel3: providing new flags and using those in NewRunContext to find the right context | 18:29 |
| voidspace | wwitzel3: there's at least one place returning a bare err that should maybe be errors.Trace(err) | 18:30 |
| voidspace | wwitzel3: I'll go jogging and if I get back & showered before our guests arrive I'll try and get a review in | 18:30 |
| mgz | ericsnow: right so, I think that's just the panic varying, sometimes it hard aborts the test run, sometimes it's recovered and you see the failures | 18:30 |
| ericsnow | mgz: ah | 18:31 |
| ericsnow | mgz: I'll try it a few times in a row then | 18:31 |
| wwitzel3 | voidspace: appreciate it, I'll double check all my error returns, and yep, that is a good summary of what is going on | 18:34 |
| tasdomas | mgz, natefinch, voidspace, ericsnow, my PR is in core now | 18:53 |
| === benji_ is now known as benji | ||
| ericsnow | mgz: yep, I get the full panic sometimes and the failed tests sometimes | 18:56 |
| natefinch | tasdomas: huzzah! nice work | 18:57 |
| natefinch | tasdomas: now get out of here. No one should be working that late on a Friday :) | 18:57 |
| ericsnow | mgz: no panic at c9630181b910dd96ac32868142fd9512442b7629 | 19:02 |
| mgz | ericsnow: thank you! | 19:05 |
| ericsnow | mgz: trying to narrow it down | 19:06 |
| mgz | yeqah, that's good for me too | 19:08 |
| mgz | wait... noly ran one test, what did I mess up | 19:09 |
| ericsnow | mgz: never mind, I still get intermittent panics at c9630181b910dd96ac32868142fd9512442b7629 | 19:14 |
| mgz | yeah, that's what I just saw | 19:14 |
| mgz | so... my guess is intermittent panics with gccgo on amd64 was already a thing | 19:15 |
| mgz | but that still doesn't help me with which change made the ppc test job fail reliably | 19:15 |
| ericsnow | mgz: yep, I get the same panic back to July | 19:19 |
| alexisb | ericsnow, mgz you not hitting known gccgo issues are you? | 19:29 |
| voidspace | I've gotta go | 19:29 |
| voidspace | guests arriving | 19:29 |
| voidspace | happy weekend everyone | 19:29 |
| ericsnow | alexisb, mgz: perhaps...regardless, it looks like the panics are an orthogonal issue | 19:30 |
| === fuzzy is now known as Ponyo | ||
| === kadams54 is now known as kadams54-away | ||
| natefinch | happy weekend all | 21:00 |
| perrito666 | ericsnow: care to explain? http://reviews.vapour.ws/r/298/#comment1932 | 21:27 |
| ericsnow | perrito666: I'll respond there. | 21:28 |
| perrito666 | tx | 21:29 |
| ericsnow | perrito666: done | 21:31 |
| perrito666 | tx a lot | 21:32 |
| * perrito666 keeps fixing | 21:32 | |
| ericsnow | perrito666: sorry I haven't been able to review more or otherwise take a look at that review request | 21:32 |
| perrito666 | ericsnow: np, I have my hands full, many of your comments have been useful either to fix things or to re-think why I wrote some things | 21:33 |
| perrito666 | many of the things I didn't like where just reactions to the re-write in backups and therer | 21:33 |
| perrito666 | fore not as nice as I would have liked at the moment | 21:33 |
| perrito666 | ericsnow: ah I see what you mean, well I would have expected that pattern not to exist, but for the sake of consistency I will make it follow it | 21:35 |
| ericsnow | perrito666: thanks! | 21:35 |
| perrito666 | man, it is specially hard to open files when your kb distribution has / only over the 7 | 21:36 |
| === lazyPower is now known as lazyWeekend | ||
| === kadams54 is now known as kadams54-away | ||
| === kadams54-away is now known as kadams54 | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
