=== ses is now known as Guest47761 | ||
mup | Bug #1556514 opened: Juju bootstrap does not assign and use floating IP on openstack <floating> <ip> <juju-core> <network> <openstack> <juju-core:New> <https://launchpad.net/bugs/1556514> | 00:34 |
---|---|---|
mup | Bug #1556514 changed: Juju bootstrap does not assign and use floating IP on openstack <floating> <ip> <juju-core> <network> <openstack> <juju-core:New> <https://launchpad.net/bugs/1556514> | 00:40 |
mup | Bug #1556514 opened: Juju bootstrap does not assign and use floating IP on openstack <floating> <ip> <juju-core> <network> <openstack> <juju-core:New> <https://launchpad.net/bugs/1556514> | 00:49 |
mup | Bug #1532841 changed: cannot add charm to storage <charm> <ci> <juju-core:Expired> <juju-core structured-metadata:Invalid> <https://launchpad.net/bugs/1532841> | 04:28 |
mup | Bug #1532841 opened: cannot add charm to storage <charm> <ci> <juju-core:Expired> <juju-core structured-metadata:Invalid> <https://launchpad.net/bugs/1532841> | 04:35 |
mup | Bug #1532841 changed: cannot add charm to storage <charm> <ci> <juju-core:Expired> <juju-core structured-metadata:Invalid> <https://launchpad.net/bugs/1532841> | 04:38 |
mup | Bug #1556630 opened: Timeout github.com/juju/juju/cmd/jujud/reboot <ci> <go1.5> <regression> <test-failure> <wily> <juju-core:Triaged> <https://launchpad.net/bugs/1556630> | 15:54 |
mup | Bug #1555291 changed: TestStartInstancePopulatesNetworkInfoWithoutAddressAllocation fails on centos <centos> <ci> <regression> <test-failure> <juju-core:Fix Released by sinzui> <https://launchpad.net/bugs/1555291> | 16:15 |
davecheney | thumper: https://github.com/juju/juju/pull/4706 | 20:46 |
davecheney | two things | 20:46 |
davecheney | 1, do you have lxc installed ? | 20:46 |
davecheney | i don't and the tests always pass reliably | 20:47 |
davecheney | but the test was ignoring basically all the error handling | 20:47 |
davecheney | so I added it back in | 20:47 |
davecheney | at least the test won't hang now | 20:47 |
thumper | yes I have lxc installed | 20:47 |
davecheney | thumper: PTAL https://github.com/juju/juju/pull/4706 | 20:48 |
davecheney | it reduced the test run time from 10s of seconds to milliseconds | 20:48 |
davecheney | i cannot get it to fail on my machine | 20:49 |
davecheney | ask you have the failing sample, can ypu try please | 20:49 |
thumper | I'll try | 20:49 |
thumper | tests pass here now | 21:05 |
thumper | davecheney: but I'm not sure why | 21:05 |
thumper | how does your branch fix it? | 21:05 |
=== _thumper_ is now known as thumper | ||
davecheney | thumper: i don't believe that this change will make the test pass where they previously failed | 22:50 |
davecheney | but previously the test didn't fail, it hung | 22:50 |
davecheney | with this change | 22:50 |
davecheney | you'll get an immediate error | 22:50 |
davecheney | which is better than the test hanging | 22:50 |
davecheney | then the test runner being killed and we get zero output | 22:50 |
thumper | right | 22:50 |
thumper | why did they hang? | 22:51 |
thumper | it isn't clear to me | 22:51 |
davecheney | no idea | 22:52 |
davecheney | i cannot reproduce the problem on my machine | 22:52 |
davecheney | and when we do it in CI | 22:52 |
davecheney | the test runner kills the job before we get any output | 22:52 |
thumper | well, even with your stress.sh it passes every time ow | 22:55 |
thumper | now | 22:55 |
davecheney | thumper: if you can make it hang on your machine without that CL | 22:56 |
davecheney | can you run the test with debug turned up to 11 pls | 22:56 |
* thumper tries | 22:56 | |
thumper | hmm... | 22:57 |
thumper | without your changes... | 22:57 |
thumper | it hangs first time | 22:57 |
davecheney | what output do yo get | 22:59 |
davecheney | are any of those log lines firing ? | 23:00 |
* thumper looks | 23:00 | |
* thumper added -check.vv | 23:00 | |
thumper | [LOG] 0:15.878 WARNING juju.cmd.jujud.reboot Waiting for containers to shutdown: [lxd:juju-c0518c0b-97b5-47ab-8c1b-541906fb2c73-machine-0] | 23:01 |
thumper | juju-c0518c0b-97b5-47ab-8c1b-541906fb2c73-machine-0 | STOPPED | | | PERSISTENT | 0 | | 23:01 |
davecheney | hmm, lxc-ls is being *ahem* patched | 23:02 |
davecheney | you should not see that | 23:02 |
thumper | this is lxd | 23:02 |
thumper | not lxc | 23:02 |
thumper | lxc list | 23:02 |
davecheney | we'd that would explain it | 23:02 |
davecheney | welp | 23:02 |
thumper | why did it pass with your patch then? | 23:03 |
* thumper is confused | 23:03 | |
davecheney | i don't know | 23:03 |
davecheney | i cannot get it to fail on my machine | 23:04 |
thumper | got any lxd containes? | 23:04 |
davecheney | i have to do this surgery via remote | 23:04 |
davecheney | nope, don't have lxd installed | 23:04 |
thumper | ah... | 23:04 |
thumper | that'll be why it is fine on your machine | 23:04 |
thumper | because lxd isn't initialized | 23:04 |
davecheney | that's some great test isolation we have | 23:04 |
thumper | fricken amazing | 23:04 |
* thumper heads to lunch | 23:05 | |
davecheney | cherylj: I don't have permission to close this bug https://github.com/juju/juju/issues/4705#issuecomment-195956659 | 23:16 |
davecheney | may I please have that permission | 23:16 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!