| oomichi | alex_xu: you have already done which was my request on https://review.openstack.org/#/c/172619/ | 00:00 |
|---|---|---|
| *** zhenguo has joined #openstack-nova | 00:01 | |
| *** achanda has quit IRC | 00:05 | |
| *** jamielennox|away is now known as jamielennox | 00:06 | |
| alex_xu | oomichi: yea, thanks for the review | 00:15 |
| oomichi | alex_xu: maybe we can restart it soon | 00:16 |
| alex_xu | oomichi: yea, The patch can begin to update before weekend, now we are in travel | 00:18 |
| *** baoli has joined #openstack-nova | 00:19 | |
| *** salv-orlando has quit IRC | 00:26 | |
| *** sdake has joined #openstack-nova | 00:30 | |
| *** sdake_ has quit IRC | 00:34 | |
| *** annashen has quit IRC | 00:38 | |
| *** VW_ has joined #openstack-nova | 00:39 | |
| *** tjones1 has joined #openstack-nova | 00:43 | |
| *** sdake_ has joined #openstack-nova | 00:46 | |
| *** tjones1 has quit IRC | 00:48 | |
| *** sdake has quit IRC | 00:49 | |
| *** sdake has joined #openstack-nova | 00:52 | |
| *** cfriesen has joined #openstack-nova | 00:54 | |
| *** sdake_ has quit IRC | 00:56 | |
| *** eliqiao has joined #openstack-nova | 01:00 | |
| *** igordcard_ has quit IRC | 01:04 | |
| *** isd has quit IRC | 01:04 | |
| *** Longgeek has joined #openstack-nova | 01:04 | |
| *** baoli has quit IRC | 01:07 | |
| *** unicell has quit IRC | 01:15 | |
| *** park_heijlong has joined #openstack-nova | 01:17 | |
| *** yamahata has joined #openstack-nova | 01:17 | |
| *** unicell has joined #openstack-nova | 01:17 | |
| *** dimsum_ has quit IRC | 01:19 | |
| *** achanda has joined #openstack-nova | 01:21 | |
| *** baoli has joined #openstack-nova | 01:21 | |
| *** liusheng has quit IRC | 01:24 | |
| *** shaohe_feng has joined #openstack-nova | 01:27 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix empty instance nw_info cache https://review.openstack.org/146036 | 01:28 |
| *** liusheng has joined #openstack-nova | 01:28 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix scheduler issue when multiple-create failed https://review.openstack.org/147048 | 01:28 |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix failure of stopping instances during init host https://review.openstack.org/167926 | 01:28 |
| *** dimsum_ has joined #openstack-nova | 01:28 | |
| *** mriedem has quit IRC | 01:30 | |
| *** mriedem has joined #openstack-nova | 01:31 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova-specs: Add the description about v2.1 API validation https://review.openstack.org/172771 | 01:31 |
| *** baoli has quit IRC | 01:35 | |
| *** dimsum_ has quit IRC | 01:35 | |
| *** otter768 has joined #openstack-nova | 01:35 | |
| *** VW_ has quit IRC | 01:36 | |
| *** yuntongjin has joined #openstack-nova | 01:37 | |
| *** wanghao_ has joined #openstack-nova | 01:40 | |
| *** VW_ has joined #openstack-nova | 01:48 | |
| *** Marga_ has quit IRC | 01:50 | |
| *** erkules_ has quit IRC | 01:53 | |
| *** popw has joined #openstack-nova | 01:53 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Delay STOPPED lifecycle event for all domains, not just Xen https://review.openstack.org/172775 | 01:54 |
| oomichi | eliqiao: hi | 01:56 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Delay STOPPED lifecycle event for all domains, not just Xen https://review.openstack.org/172775 | 01:56 |
| eliqiao | hi oomichi: | 01:57 |
| oomichi | eliqiao: thanks for reviewing https://review.openstack.org/172771 | 01:57 |
| oomichi | eliqiao: I guess your comment means we need some URLs for explaining the investigation, right? | 01:57 |
| *** mriedem has quit IRC | 01:58 | |
| eliqiao | oomichi: I mean, do you add a addition '''(quote) by mistake? | 01:58 |
| *** takedakn has joined #openstack-nova | 01:59 | |
| *** tjones1 has joined #openstack-nova | 02:00 | |
| eliqiao | oomichi: SDKs(fog, jclouds)' should be SDKs(fog, jclouds)'s or SDKs(fog, jclouds) ? | 02:00 |
| oomichi | eliqiao: ah, I got it. OK, will update soon. thanks :) | 02:00 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova-specs: Add the description about v2.1 API validation https://review.openstack.org/172771 | 02:01 |
| *** heyongli_ has joined #openstack-nova | 02:05 | |
| eliqiao | oomichi: :) thx | 02:06 |
| *** heyongli_ has left #openstack-nova | 02:06 | |
| *** heyongli_ has joined #openstack-nova | 02:06 | |
| *** VW_ has quit IRC | 02:06 | |
| oomichi | eliqiao: you too :) | 02:07 |
| *** sdake_ has joined #openstack-nova | 02:11 | |
| *** heyongli_1 has joined #openstack-nova | 02:13 | |
| *** heyongli_ has quit IRC | 02:14 | |
| *** sdake has quit IRC | 02:14 | |
| *** salv-orlando has joined #openstack-nova | 02:15 | |
| *** takedakn1 has joined #openstack-nova | 02:16 | |
| *** takedakn has quit IRC | 02:17 | |
| *** baoli has joined #openstack-nova | 02:18 | |
| *** salv-orl_ has joined #openstack-nova | 02:18 | |
| *** yamahata has quit IRC | 02:20 | |
| *** salv-orlando has quit IRC | 02:20 | |
| *** heyongli_ has joined #openstack-nova | 02:23 | |
| *** dsanders has quit IRC | 02:24 | |
| *** heyongli_1 has quit IRC | 02:25 | |
| *** dsanders has joined #openstack-nova | 02:26 | |
| *** dsanders has quit IRC | 02:27 | |
| *** shi has quit IRC | 02:29 | |
| *** takedakn1 has quit IRC | 02:29 | |
| *** shi has joined #openstack-nova | 02:29 | |
| *** takedakn has joined #openstack-nova | 02:29 | |
| *** dimsum_ has joined #openstack-nova | 02:35 | |
| *** takedakn has quit IRC | 02:37 | |
| *** Longgeek has quit IRC | 02:38 | |
| *** VW_ has joined #openstack-nova | 02:38 | |
| *** Longgeek has joined #openstack-nova | 02:40 | |
| *** dimsum_ has quit IRC | 02:41 | |
| *** achanda has quit IRC | 02:44 | |
| *** heyongli_ has quit IRC | 02:47 | |
| *** achanda has joined #openstack-nova | 02:48 | |
| *** heyongli_1 has joined #openstack-nova | 02:49 | |
| *** salv-orl_ has quit IRC | 02:49 | |
| *** oomichi has quit IRC | 02:55 | |
| *** achanda has quit IRC | 02:59 | |
| *** Marga_ has joined #openstack-nova | 03:11 | |
| *** Marga_ has quit IRC | 03:12 | |
| *** subscope_ has joined #openstack-nova | 03:12 | |
| *** Marga_ has joined #openstack-nova | 03:12 | |
| *** baoli has quit IRC | 03:15 | |
| *** garyk has quit IRC | 03:15 | |
| *** asettle is now known as asettle-lunch | 03:22 | |
| *** annashen has joined #openstack-nova | 03:23 | |
| *** sdake_ has quit IRC | 03:25 | |
| *** yuntongjin has quit IRC | 03:26 | |
| *** tojuvone has joined #openstack-nova | 03:28 | |
| *** unicell1 has joined #openstack-nova | 03:35 | |
| *** unicell has quit IRC | 03:37 | |
| *** yuntongjin has joined #openstack-nova | 03:37 | |
| *** salv-orlando has joined #openstack-nova | 03:39 | |
| *** tojuvone has quit IRC | 03:44 | |
| *** tojuvone has joined #openstack-nova | 03:44 | |
| *** erkules_ has joined #openstack-nova | 03:49 | |
| *** ruijing has joined #openstack-nova | 03:50 | |
| *** otter768 has quit IRC | 03:54 | |
| *** isd has joined #openstack-nova | 03:59 | |
| *** achanda has joined #openstack-nova | 03:59 | |
| *** annashen has quit IRC | 04:00 | |
| *** mdbooth has quit IRC | 04:01 | |
| *** salv-orlando has quit IRC | 04:02 | |
| *** achanda has quit IRC | 04:05 | |
| *** asettle-lunch is now known as asettle | 04:08 | |
| *** mdbooth has joined #openstack-nova | 04:08 | |
| *** cfriesen has quit IRC | 04:08 | |
| *** yuntongjin has quit IRC | 04:10 | |
| *** unicell1 has quit IRC | 04:11 | |
| *** unicell has joined #openstack-nova | 04:12 | |
| *** kaisers has joined #openstack-nova | 04:27 | |
| *** garyk has joined #openstack-nova | 04:28 | |
| *** annashen has joined #openstack-nova | 04:31 | |
| *** VW_ has quit IRC | 04:32 | |
| *** raies has joined #openstack-nova | 04:35 | |
| *** subscope_ has quit IRC | 04:43 | |
| *** annashen has quit IRC | 04:47 | |
| *** arnaud___ has joined #openstack-nova | 04:48 | |
| *** irenab has joined #openstack-nova | 04:49 | |
| *** tjones1 has quit IRC | 04:52 | |
| *** ishant has joined #openstack-nova | 04:53 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/nova: Omit RescheduledException in instance_fault.message https://review.openstack.org/165751 | 04:57 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova-specs: Add the description about v2.1 API validation https://review.openstack.org/172771 | 04:58 |
| *** annashen has joined #openstack-nova | 04:58 | |
| *** shaohe_feng has quit IRC | 05:01 | |
| *** _gryf|afk is now known as _gryf | 05:01 | |
| *** rushiagr_away is now known as rushiagr | 05:04 | |
| *** arnaud___ has quit IRC | 05:06 | |
| *** annashen has quit IRC | 05:06 | |
| *** liusheng has quit IRC | 05:06 | |
| *** liusheng has joined #openstack-nova | 05:06 | |
| *** haomaiwang has joined #openstack-nova | 05:09 | |
| *** shaohe_feng has joined #openstack-nova | 05:10 | |
| *** yuntongjin has joined #openstack-nova | 05:11 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: fill in instance metadata when launching instances https://review.openstack.org/141028 | 05:11 |
| openstackgerrit | garyk proposed openstack/nova: VMware: fill in instance metadata when resizing instances https://review.openstack.org/144436 | 05:12 |
| *** RuiChen_ has joined #openstack-nova | 05:14 | |
| *** irenab has quit IRC | 05:15 | |
| openstackgerrit | shihanzhang proposed openstack/nova: Add config option to override url for versions https://review.openstack.org/160266 | 05:16 |
| *** irenab has joined #openstack-nova | 05:20 | |
| *** shaohe_feng has quit IRC | 05:20 | |
| *** shaohe_feng has joined #openstack-nova | 05:21 | |
| *** tjones1 has joined #openstack-nova | 05:23 | |
| *** dulek has joined #openstack-nova | 05:25 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: enable configuring of console delay https://review.openstack.org/140669 | 05:26 |
| openstackgerrit | Alex Xu proposed openstack/nova-specs: API: Policy should be enforced at API layer where possible(final part) https://review.openstack.org/172619 | 05:27 |
| *** irenab has quit IRC | 05:29 | |
| openstackgerrit | wanghao proposed openstack/nova: fix 'Ignoring EndpointNotFound' error in init host https://review.openstack.org/147042 | 05:30 |
| *** moshele has joined #openstack-nova | 05:31 | |
| *** rushiagr is now known as rushiagr_away | 05:34 | |
| *** pradipta has joined #openstack-nova | 05:35 | |
| *** nkrinner has joined #openstack-nova | 05:35 | |
| *** shaohe_feng1 has joined #openstack-nova | 05:39 | |
| *** shaohe_feng has quit IRC | 05:41 | |
| *** shaohe_feng has joined #openstack-nova | 05:42 | |
| *** shaohe_feng1 has quit IRC | 05:43 | |
| *** ankit_ag has joined #openstack-nova | 05:44 | |
| *** annashen has joined #openstack-nova | 05:45 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge migrate server tests between v2 and v2.1 https://review.openstack.org/172792 | 05:48 |
| *** ajayaa has joined #openstack-nova | 05:49 | |
| *** RuiChen_ has quit IRC | 05:50 | |
| *** ildikov has quit IRC | 05:53 | |
| *** achanda has joined #openstack-nova | 05:53 | |
| *** otter768 has joined #openstack-nova | 05:55 | |
| *** shaohe_feng1 has joined #openstack-nova | 05:55 | |
| *** Maike has joined #openstack-nova | 05:57 | |
| *** shaohe_feng has quit IRC | 05:57 | |
| *** otter768 has quit IRC | 05:59 | |
| *** annashen has quit IRC | 06:00 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge multinic functional tests between v2 and v2.1 https://review.openstack.org/172796 | 06:01 |
| *** isd has quit IRC | 06:03 | |
| *** rushiagr_away is now known as rushiagr | 06:03 | |
| *** pkoniszewski has joined #openstack-nova | 06:05 | |
| *** annashen has joined #openstack-nova | 06:05 | |
| *** shaohe_feng has joined #openstack-nova | 06:08 | |
| *** aswadr has joined #openstack-nova | 06:08 | |
| *** afazekas has joined #openstack-nova | 06:08 | |
| *** shaohe_feng1 has quit IRC | 06:08 | |
| *** doron has joined #openstack-nova | 06:09 | |
| *** oro has quit IRC | 06:10 | |
| *** annashen has quit IRC | 06:13 | |
| *** rajesht has joined #openstack-nova | 06:13 | |
| *** doron_ has joined #openstack-nova | 06:14 | |
| *** Longgeek has quit IRC | 06:14 | |
| *** Longgeek has joined #openstack-nova | 06:14 | |
| *** doron has quit IRC | 06:14 | |
| *** doron_ is now known as doron_afk | 06:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/171943 | 06:18 |
| *** annashen has joined #openstack-nova | 06:19 | |
| *** annashen has quit IRC | 06:23 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Share multiple create server tests between v2 and v2.1 https://review.openstack.org/172801 | 06:24 |
| *** jcoufal has joined #openstack-nova | 06:26 | |
| *** dave-mccowan has joined #openstack-nova | 06:30 | |
| *** dave-mccowan has quit IRC | 06:34 | |
| *** rwsu has joined #openstack-nova | 06:34 | |
| *** Mike_D_laptop has joined #openstack-nova | 06:38 | |
| *** mpaolino has joined #openstack-nova | 06:41 | |
| *** yamahata has joined #openstack-nova | 06:42 | |
| *** shaohe_feng has quit IRC | 06:44 | |
| *** shaohe_feng has joined #openstack-nova | 06:45 | |
| *** apuimedo has joined #openstack-nova | 06:45 | |
| rajesht | hi cores, | 06:50 |
| rajesht | could you please give your opinion on this patch https://review.openstack.org/135202 | 06:50 |
| rajesht | thanks in advance | 06:50 |
| *** popw has quit IRC | 06:53 | |
| rajesht | jogo: you around ? | 06:53 |
| *** markus_z has joined #openstack-nova | 06:57 | |
| *** hdd has joined #openstack-nova | 06:58 | |
| openstackgerrit | Deepak C Shetty proposed openstack/nova: WIP: Nova part of volume snapshot improvements https://review.openstack.org/172813 | 07:00 |
| *** hdd has quit IRC | 07:01 | |
| *** tdurakov has quit IRC | 07:07 | |
| *** annashen has joined #openstack-nova | 07:09 | |
| *** jamielennox is now known as jamielennox|away | 07:11 | |
| *** ildikov has joined #openstack-nova | 07:14 | |
| *** eglynn has quit IRC | 07:20 | |
| *** jichen has joined #openstack-nova | 07:20 | |
| *** sahid has joined #openstack-nova | 07:20 | |
| *** achanda has quit IRC | 07:21 | |
| *** ihrachyshka has joined #openstack-nova | 07:22 | |
| *** achanda has joined #openstack-nova | 07:22 | |
| zigo | Could anyone tell me from where assertRaisesRegex is from? | 07:23 |
| *** jistr has joined #openstack-nova | 07:24 | |
| *** yamahata has quit IRC | 07:24 | |
| *** ihrachyshka has quit IRC | 07:28 | |
| bauzas | zigo: heya | 07:31 |
| bauzas | zigo: what's the context ? | 07:31 |
| *** mrda is now known as mrda-away | 07:32 | |
| *** Marga_ has quit IRC | 07:32 | |
| zigo | bauzas: In the context of running unit tests of nova when building my package, I get an error that this isn't defined. | 07:32 |
| bauzas | zigo: oh, that's normal | 07:33 |
| zigo | Example: http://paste.debian.net/166489/ | 07:33 |
| bauzas | zigo: that test method comes from the unittest package IIRC | 07:33 |
| zigo | bauzas: How come?!? | 07:33 |
| *** tjones1 has quit IRC | 07:33 | |
| zigo | bauzas: Oh, unittest2? | 07:33 |
| bauzas | zigo: lemme check | 07:33 |
| bauzas | zigo: I know there is an hacking rule for this | 07:34 |
| zigo | This actually was my question! :) | 07:34 |
| *** tjones1 has joined #openstack-nova | 07:34 | |
| bauzas | zigo: just to be clear assertRaisesRegexp is deprecated in favor of assertRaisesRegex AFAIK | 07:34 |
| *** Mike_D_laptop has quit IRC | 07:34 | |
| * zigo adds unittest2 as build-depends. | 07:35 | |
| zigo | bauzas: IMO, unittest2 must be in test-requirements.txt if nova uses it. | 07:35 |
| bauzas | zigo: sec, checking from which package it comes from | 07:36 |
| *** Nikolay_St has joined #openstack-nova | 07:36 | |
| *** markus_z has quit IRC | 07:37 | |
| *** mpavone has joined #openstack-nova | 07:37 | |
| zigo | bauzas: In case.py of unittest2. | 07:38 |
| * zigo restarted a package build ... | 07:38 | |
| *** annashen has quit IRC | 07:39 | |
| bauzas | zigo: I don't think so | 07:39 |
| *** mpaolino has quit IRC | 07:40 | |
| bauzas | zigo: I would rather bet on testtools, that's what I'm checking | 07:40 |
| zigo | bauzas: Already checked, and it wasn't there. | 07:40 |
| *** markus_z has joined #openstack-nova | 07:41 | |
| *** gszasz has joined #openstack-nova | 07:41 | |
| bauzas | zigo: mmm, yeah testtools can either import unittest2 or unittest depending on what's present | 07:42 |
| *** annashen has joined #openstack-nova | 07:43 | |
| bauzas | zigo: and indeed, a pdb trace shows me that's unittest2.assertRaisesRegex which is called | 07:43 |
| bauzas | zigo: and unittest2 is marked as dependency of testtools https://github.com/testing-cabal/testtools/blob/master/setup.py#L56, that's just because testtools is not pbr-ish | 07:44 |
| *** tdurakov has joined #openstack-nova | 07:44 | |
| tdurakov | hi everyone! | 07:45 |
| tdurakov | try to set up devstack env to work with block migration | 07:45 |
| tdurakov | there is error during the process: libvirtError: Unable to pre-create chardev file '/opt/stack/data/nova/instances/9ae675f8-fb28-4872-90eb-1431326756ae/console.log': No such file or directory | 07:46 |
| tdurakov | who dealt with such error? | 07:47 |
| *** jlanoux has joined #openstack-nova | 07:47 | |
| zigo | bauzas: Ah, thanks for finding-out the details. So moving forward, do you think unittest2 could be added in test-requirements.txt of Nova? | 07:49 |
| bauzas | zigo: no | 07:49 |
| bauzas | zigo: Nova requires testtools | 07:49 |
| bauzas | zigo: and testtools requires unittest2 so everything should be working | 07:49 |
| bauzas | zigo: but what confused you is that testtools silently backports to unittest if unittest2 is not available | 07:50 |
| bauzas | zigo: I don't know why it's designed that way, but it should not be related to Nova | 07:50 |
| bauzas | zigo: and on a side note, it's a general rule for all OpenStack projects - tho packages | 07:51 |
| zigo | bauzas: Adding unittest2 as a build-dependency didn't fix the issue, unfortunately! :( | 07:51 |
| *** doron_afk has quit IRC | 07:51 | |
| bauzas | zigo: the build dependency should be on testtools IMHO | 07:51 |
| zigo | bauzas: Do you think it'd be ok for me to patch and do assertRaisesRegex -> assertRaisesRegexp ? | 07:51 |
| bauzas | zigo: please don't, that's a hacking rule now | 07:52 |
| bauzas | zigo: N335 if I'm reading correctly | 07:52 |
| zigo | bauzas: I was meaning, in my *debian package*, as a Debian specific patch, not for Nova. | 07:52 |
| bauzas | zigo: og | 07:52 |
| bauzas | oh | 07:52 |
| StevenK | zigo: The unittest2 in Debian is too old | 07:52 |
| *** pixelb has joined #openstack-nova | 07:52 | |
| *** eglynn has joined #openstack-nova | 07:52 | |
| zigo | StevenK: Sure it is, but adding a build-depends on unittest2 didn't fix anything for me. | 07:52 |
| bauzas | zigo: well, I would rather do the necessary stuff elsewhere | 07:53 |
| zigo | Maybe because we have an older testtools. | 07:53 |
| bauzas | zigo: because it's for *all* OpenStack projects :) | 07:53 |
| StevenK | zigo: Which helps not at all, because the unittest2 in Debian doesn't even define assertRaisesRegex. | 07:53 |
| bauzas | wtf ?! | 07:53 |
| StevenK | bauzas: Because it's like 0.5 | 07:53 |
| StevenK | It's horribly outdated | 07:54 |
| bauzas | oh, | 07:54 |
| bauzas | then sounds like a dep refresh ? :) | 07:54 |
| *** doron has joined #openstack-nova | 07:54 | |
| zigo | bauzas: Sounds what I have to do, indeed! | 07:54 |
| * zigo will have a look into updating that unittest2 package in Debian Experimental then. | 07:54 | |
| * bauzas feels enough lucky to not deal with packaging stuff | 07:55 | |
| zigo | Oh, there's already version 0.8 in debian experimental! | 07:55 |
| zigo | Cool ! | 07:55 |
| zigo | Will try to use that one. | 07:55 |
| *** jay-lau-513 has joined #openstack-nova | 07:55 | |
| *** otter768 has joined #openstack-nova | 07:55 | |
| *** tjones1 has quit IRC | 07:56 | |
| bauzas | zigo: I don't know how you manage all of the os packages, but you'll have to do it for all of them, not only Nova | 07:56 |
| zigo | bauzas: If I upgrade unittest2, it's going to be helpful for all OpenStack packages, so nothing to worry about. | 07:56 |
| bauzas | cool | 07:56 |
| *** tjones1 has joined #openstack-nova | 07:57 | |
| *** eglynn has quit IRC | 07:57 | |
| *** doron_ has joined #openstack-nova | 07:59 | |
| *** doron_ is now known as doron_afk | 07:59 | |
| *** doron has quit IRC | 07:59 | |
| *** otter768 has quit IRC | 08:01 | |
| *** eglynn has joined #openstack-nova | 08:02 | |
| *** ijw has quit IRC | 08:03 | |
| *** deepthi has joined #openstack-nova | 08:05 | |
| *** lucasagomes has joined #openstack-nova | 08:09 | |
| *** andreykurilin has joined #openstack-nova | 08:11 | |
| zigo | bauzas: StevenK: Even with version 0.8 of unittest2, I still have the issue. | 08:11 |
| zigo | Do I need even version 1.0.1? | 08:12 |
| bauzas | [sbauza@sbauza nova]$ .venv/bin/pip freeze | grep unittest2 | 08:12 |
| bauzas | You are using pip version 6.0.8, however version 6.1.1 is available. | 08:12 |
| bauzas | You should consider upgrading via the 'pip install --upgrade pip' command. | 08:12 |
| bauzas | unittest2==1.0.0 | 08:12 |
| bauzas | zigo: ^ | 08:12 |
| zigo | bauzas: I'm not using pip *at all*! :) | 08:13 |
| StevenK | zigo: I'm not sure if 0.8 gives assertRaisesRegex -- if you have it installed, check :-) | 08:13 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Fix for live migration rollback when using shared storage https://review.openstack.org/172117 | 08:13 |
| zigo | Thanks. | 08:13 |
| zigo | Will do. | 08:13 |
| bauzas | zigo: yeah, but that just shows you my unittest2 version | 08:14 |
| zigo | Version 0.8 seems to have it... :( | 08:15 |
| *** eglynn has quit IRC | 08:15 | |
| zigo | So I'm not sure how I can fix this. | 08:15 |
| *** lsmola_ has joined #openstack-nova | 08:16 | |
| zigo | I think I'm going for the ugly renaming to the old version of the function. | 08:16 |
| zigo | Thanks to you guys for all the help. | 08:16 |
| *** popw has joined #openstack-nova | 08:18 | |
| *** achanda has quit IRC | 08:22 | |
| *** romainh has joined #openstack-nova | 08:23 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Neutron: Check port binding status https://review.openstack.org/156112 | 08:25 |
| *** Hao has quit IRC | 08:26 | |
| *** ujjain has joined #openstack-nova | 08:28 | |
| *** ujjain has quit IRC | 08:28 | |
| *** ndipanov has joined #openstack-nova | 08:28 | |
| *** derekh has joined #openstack-nova | 08:32 | |
| *** zz_johnthetubagu is now known as johnthetubaguy | 08:36 | |
| *** eglynn has joined #openstack-nova | 08:36 | |
| zigo | Now, I have only a single unit test failure: http://paste.debian.net/166496/ | 08:38 |
| zigo | bauzas: Any idea on that one? | 08:38 |
| zigo | Seems a trickly envetlet / greentread thing... :( | 08:39 |
| *** DazWorrall has joined #openstack-nova | 08:39 | |
| bauzas | zigo: I don't exactly know how eventlet is mocked for testing | 08:40 |
| *** tanee has quit IRC | 08:40 | |
| bauzas | alex_xu: around ? I have a question ? | 08:40 |
| bauzas | alex_xu: ^ | 08:40 |
| *** tanee has joined #openstack-nova | 08:41 | |
| alex_xu | bauzas: I'm in meeting, talk to you later, maybe 3 or 4 hours later after I back to hotel... | 08:42 |
| bauzas | alex_xu: np that's for a question about testing | 08:43 |
| bauzas | and oomichi seems not being up | 08:43 |
| heyongli_1 | ndipanov, hi | 08:43 |
| bauzas | alex_xu: any idea who could help zigo on http://paste.debian.net/166496/ ? | 08:43 |
| ndipanov | heyongli_1, hey | 08:44 |
| ndipanov | was just responding to you | 08:44 |
| openstackgerrit | zhangtralon proposed openstack/nova: DB:support for migration objects multi-statues https://review.openstack.org/169604 | 08:44 |
| ndipanov | gimme 5 minutes | 08:44 |
| heyongli_1 | thanks, ndipanov | 08:44 |
| *** haigang has joined #openstack-nova | 08:46 | |
| openstackgerrit | zhangtralon proposed openstack/nova: add support for muti_statuses_query from migration https://review.openstack.org/169631 | 08:47 |
| *** DazWorrall has quit IRC | 08:48 | |
| *** annashen has quit IRC | 08:51 | |
| *** jlanoux has quit IRC | 08:52 | |
| openstackgerrit | zhangtralon proposed openstack/nova: resize-revert can break anti-affinity policy https://review.openstack.org/160110 | 08:53 |
| *** dguitarbite has quit IRC | 08:56 | |
| *** jlanoux has joined #openstack-nova | 08:58 | |
| garyk | johnthetubaguy: can you please unblock https://review.openstack.org/141028 - L is now open and this was blocked way back in K | 09:02 |
| *** jamielennox|away is now known as jamielennox | 09:02 | |
| johnthetubaguy | garyk: we need the blueprint approved first | 09:02 |
| johnthetubaguy | garyk: what did we say about the blueprint in kilo, I don't remember seeing it | 09:03 |
| garyk | can you approve it? why not? | 09:03 |
| johnthetubaguy | garyk: the first question is does it need a spec, and well I would vote no | 09:03 |
| *** dguitarbite has joined #openstack-nova | 09:03 | |
| johnthetubaguy | garyk: technically we asked for these blueprints to be discussed in nova meetings, so we can approve them | 09:03 |
| *** ildikov has quit IRC | 09:04 | |
| *** salv-orlando has joined #openstack-nova | 09:04 | |
| johnthetubaguy | garyk: but I am cool just approving it to save us all a lot of time, did this get discussed for kilo already? | 09:04 |
| garyk | johnthetubaguy: this was a feature implemented by libvirt. so we are just doingt he same here. | 09:04 |
| garyk | johnthetubaguy: no, it was not discussed - it was just blocked for some reason unbeknownst to me | 09:04 |
| garyk | i do not think that we need a spec here. unless you think so. | 09:05 |
| johnthetubaguy | garyk: the -2 on the patches is because the blueprint was not approved, and we were during the freeze period, as I added this comment "sorry, your https://blueprints.launchpad.net/openstack/?searchtext=is not approved for kilo right now, please resubmit for L" | 09:06 |
| johnthetubaguy | garyk: we couldn't approve the blueprint during the priority only window in kilo | 09:07 |
| *** salv-orlando has quit IRC | 09:07 | |
| garyk | johnthetubaguy: this is the BP - https://blueprints.launchpad.net/nova/+spec/vmware-driver-domain-metadata - not sure what I need to submit | 09:07 |
| johnthetubaguy | garyk: it couldn't have been approved when the code was uploaded, but that involves putting it into the meeting, or just nudging me | 09:07 |
| johnthetubaguy | garyk: so the policy is basically add to the agenda for the next nova meeting | 09:08 |
| garyk | can you please let me know where i need to put this so that it can get the relevant eyes - it was actually +2'ed by dan pb | 09:08 |
| johnthetubaguy | garyk: let me just approve this though, as its clearly been slipping between every crack possible | 09:08 |
| garyk | johnthetubaguy: ok, thanks, where is that meeting agenda | 09:08 |
| *** belmoreira has joined #openstack-nova | 09:10 | |
| *** ztralon has joined #openstack-nova | 09:12 | |
| *** haigang has quit IRC | 09:13 | |
| *** tanee has quit IRC | 09:14 | |
| openstackgerrit | Deepak C Shetty proposed openstack/nova-specs: Volume snapshot improvements https://review.openstack.org/134517 | 09:14 |
| *** aswadr has quit IRC | 09:14 | |
| *** tanee has joined #openstack-nova | 09:14 | |
| *** haigang has joined #openstack-nova | 09:15 | |
| *** park_heijlong has quit IRC | 09:15 | |
| johnthetubaguy | garyk: so I have approved the BP, and unblocked those patches, because liberty is open, and its no longer blocked by feature freeze | 09:15 |
| garyk | johnthetubaguy: thanks | 09:15 |
| johnthetubaguy | garyk: just let me know about any other specs or blueprints that need approving | 09:16 |
| garyk | ok, thanks | 09:16 |
| johnthetubaguy | garyk: the agenda is in the standard location, over here, if thats useful in the future: https://wiki.openstack.org/wiki/Meetings/Nova | 09:16 |
| *** andymaier has joined #openstack-nova | 09:17 | |
| johnthetubaguy | I just updated it with this weeks dates, its meant to get done straight after the last meeting, but that doesn't always happen | 09:17 |
| *** Mike_D_laptop has joined #openstack-nova | 09:19 | |
| garyk | thanks | 09:20 |
| *** doude has quit IRC | 09:21 | |
| *** doude has joined #openstack-nova | 09:22 | |
| *** doude has quit IRC | 09:22 | |
| *** doude has joined #openstack-nova | 09:22 | |
| *** Mike_D_laptop has quit IRC | 09:23 | |
| raies | alex_xu: hi | 09:23 |
| *** aix has joined #openstack-nova | 09:23 | |
| *** Mike_D_laptop has joined #openstack-nova | 09:24 | |
| *** shaohe_feng has quit IRC | 09:24 | |
| *** eliqiao has quit IRC | 09:24 | |
| alex_xu | bauzas: emm...sorry, I'm not sure, I can help debug, but not sure those info enough to find something | 09:26 |
| alex_xu | raies: hi | 09:26 |
| *** pkoniszewski has quit IRC | 09:26 | |
| *** raies has left #openstack-nova | 09:26 | |
| *** haigang has quit IRC | 09:27 | |
| *** dheeraj-gupta-4 has joined #openstack-nova | 09:28 | |
| *** haigang has joined #openstack-nova | 09:28 | |
| * alex_xu have to leave now, will be back in hour. | 09:30 | |
| *** yuntongjin has quit IRC | 09:35 | |
| *** heyongli_1 has quit IRC | 09:35 | |
| *** heyongli_ has joined #openstack-nova | 09:35 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add min/max of API microversions to version API https://review.openstack.org/172856 | 09:35 |
| *** heyongli_1 has joined #openstack-nova | 09:39 | |
| *** heyongli_ has quit IRC | 09:39 | |
| *** heyongli_ has joined #openstack-nova | 09:43 | |
| *** ZZelle has quit IRC | 09:43 | |
| *** ZZelle has joined #openstack-nova | 09:44 | |
| *** danieru has quit IRC | 09:44 | |
| *** danieru has joined #openstack-nova | 09:45 | |
| *** heyongli_1 has quit IRC | 09:45 | |
| ndipanov | heyongli_, heyongli one of you :) | 09:48 |
| ndipanov | so I commented on the review | 09:48 |
| ndipanov | see if it makes sense to you | 09:48 |
| ndipanov | but refreshing pci devices like that is almost certainly the wrog thing to do and will result in more bugs going further | 09:49 |
| *** heyongli_ has quit IRC | 09:50 | |
| *** haigang has quit IRC | 09:54 | |
| *** haigang has joined #openstack-nova | 09:55 | |
| johnthetubaguy | alex_xu: thanks for the heads up, I have pushed that through so you can get going, all approved: https://blueprints.launchpad.net/nova/+spec/nova-api-policy-final-part | 09:55 |
| *** otter768 has joined #openstack-nova | 09:56 | |
| *** igordcard_ has joined #openstack-nova | 09:58 | |
| *** markvoelker has joined #openstack-nova | 09:59 | |
| *** popw1 has joined #openstack-nova | 09:59 | |
| *** haigang has quit IRC | 10:00 | |
| *** popw has quit IRC | 10:01 | |
| *** otter768 has quit IRC | 10:01 | |
| *** Maike_ has joined #openstack-nova | 10:03 | |
| *** Maike has quit IRC | 10:03 | |
| *** nithyag_ has joined #openstack-nova | 10:05 | |
| *** andrearo1 has joined #openstack-nova | 10:06 | |
| *** andrearosa has quit IRC | 10:06 | |
| *** nithyag__ has quit IRC | 10:07 | |
| kashyap | ndipanov: Do you have a pointer to multi-node CI job? I recall there was a discussion | 10:09 |
| *** dulek_ has joined #openstack-nova | 10:09 | |
| *** sdake has joined #openstack-nova | 10:09 | |
| kashyap | ndipanov: Oh, please disregard me, I just found it! | 10:10 |
| ndipanov | jogo, mentioned it | 10:10 |
| ndipanov | can you point me too it? | 10:10 |
| *** igordcard_ has quit IRC | 10:10 | |
| *** dulek has quit IRC | 10:11 | |
| kashyap | I just found the conf file it is using -- http://logs.openstack.org/20/169520/1/check/check-tempest-dsvm-aiopcpu-full/4a86955/logs/etc/nova/nova.conf.txt.gz | 10:11 |
| *** igordcard_ has joined #openstack-nova | 10:11 | |
| kashyap | What I was checking is, how is it doing live block-migrate without setting the relevant config attributes at all | 10:11 |
| *** doron_afk has quit IRC | 10:11 | |
| *** doron_afk has joined #openstack-nova | 10:11 | |
| *** jay-lau-513 has quit IRC | 10:13 | |
| *** IanGovett has joined #openstack-nova | 10:15 | |
| *** markvoelker has quit IRC | 10:16 | |
| *** markvoelker has joined #openstack-nova | 10:18 | |
| *** tjones1 has left #openstack-nova | 10:19 | |
| *** romainh has left #openstack-nova | 10:22 | |
| *** sdake_ has joined #openstack-nova | 10:28 | |
| *** markvoelker has quit IRC | 10:29 | |
| gillllliard | kashyap: What config needs to be set? | 10:29 |
| *** gillllliard is now known as gilliard | 10:29 | |
| kashyap | gilliard: These: block_migration_flag=VIR_MIGRATE_UNDEFINE_SOURCE, VIR_MIGRATE_PEER2PEER, VIR_MIGRATE_LIVE, VIR_MIGRATE_NON_SHARED_INC | 10:30 |
| kashyap | gilliard: Currently there's an upstream bug (just a flag setting) in live block migration code in Nova (patch in discussion) that enables libvirt TUNNELLED migration, for which there's no support yet. | 10:31 |
| openstackgerrit | Mh Raies proposed openstack/nova: Enable tenant/user specific instance filtering https://review.openstack.org/139025 | 10:31 |
| *** sdake has quit IRC | 10:31 | |
| gilliard | kashyap: which bug? | 10:31 |
| kashyap | gilliard: https://bugs.launchpad.net/nova/+bug/1441054 | 10:32 |
| openstack | Launchpad bug 1441054 in OpenStack Compute (nova) "live-migration --block-migrate fails with default libvirt flags" [Undecided,In progress] - Assigned to Mathieu Rohon (mathieu-rohon) | 10:32 |
| gilliard | looking at my notes, I always set VIR_MIGRATE_TUNNELLED too | 10:32 |
| gilliard | Thanks - looking. | 10:32 |
| *** igordcard_ has quit IRC | 10:32 | |
| *** igordcard_ has joined #openstack-nova | 10:33 | |
| *** ferest has joined #openstack-nova | 10:33 | |
| *** salv-orlando has joined #openstack-nova | 10:35 | |
| kashyap | gilliard: If you set it, it won't work for at-least block-migrate with current libvirt, QEMU and Nova. | 10:37 |
| *** ferest has quit IRC | 10:37 | |
| kashyap | See the rationale here, as a comment to Matt Riedemann -- https://review.openstack.org/#/c/171098/1/nova/virt/libvirt/driver.py | 10:37 |
| *** dimsum_ has joined #openstack-nova | 10:39 | |
| *** andreykurilin has quit IRC | 10:43 | |
| *** aix has quit IRC | 10:45 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Add numa_node field to PciDevicePool https://review.openstack.org/171304 | 10:45 |
| *** popw has joined #openstack-nova | 10:45 | |
| *** popw1 has quit IRC | 10:47 | |
| *** sdake_ has quit IRC | 10:53 | |
| openstackgerrit | sahid proposed openstack/nova: virt: remove small pages request https://review.openstack.org/172079 | 10:55 |
| *** mwagner_lap has quit IRC | 10:57 | |
| *** andrearo1 is now known as andrearosa | 10:58 | |
| openstackgerrit | Mh Raies proposed openstack/nova: Enable tenant/user specific instance filtering https://review.openstack.org/139025 | 10:59 |
| openstackgerrit | Mh Raies proposed openstack/nova: Enable tenant/user specific instance filtering https://review.openstack.org/139025 | 11:00 |
| *** dulek_ has quit IRC | 11:00 | |
| *** ishant has quit IRC | 11:01 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Add min/max of API microversions to version API https://review.openstack.org/172856 | 11:02 |
| *** Maike_ has quit IRC | 11:04 | |
| *** Maike_ has joined #openstack-nova | 11:04 | |
| *** ajayaa has quit IRC | 11:11 | |
| *** dulek_ has joined #openstack-nova | 11:12 | |
| *** Mike_D_laptop has quit IRC | 11:15 | |
| *** deepthi has quit IRC | 11:16 | |
| *** popw1 has joined #openstack-nova | 11:24 | |
| *** popw has quit IRC | 11:24 | |
| *** zzz_pczesno is now known as pczesno | 11:25 | |
| *** ajayaa has joined #openstack-nova | 11:25 | |
| *** ujjain has joined #openstack-nova | 11:34 | |
| *** erkules has joined #openstack-nova | 11:37 | |
| *** popw has joined #openstack-nova | 11:37 | |
| *** erkules_ has quit IRC | 11:38 | |
| *** tojuvone has quit IRC | 11:38 | |
| *** popw1 has quit IRC | 11:39 | |
| *** liusheng has quit IRC | 11:44 | |
| *** lucasagomes is now known as lucas-hungry | 11:47 | |
| *** ttx has quit IRC | 11:48 | |
| *** ttx has joined #openstack-nova | 11:48 | |
| *** ttx has quit IRC | 11:49 | |
| *** ttx has joined #openstack-nova | 11:49 | |
| *** ajayaa has quit IRC | 11:49 | |
| *** rushiagr is now known as rushiagr_away | 11:52 | |
| *** annashen has joined #openstack-nova | 11:52 | |
| *** dimsum_ has quit IRC | 11:52 | |
| *** annashen has quit IRC | 11:56 | |
| *** otter768 has joined #openstack-nova | 11:57 | |
| *** andreykurilin has joined #openstack-nova | 11:58 | |
| *** andreykurilin has quit IRC | 11:59 | |
| *** sdake has joined #openstack-nova | 12:00 | |
| *** andreykurilin has joined #openstack-nova | 12:00 | |
| *** romainh has joined #openstack-nova | 12:00 | |
| *** sdake_ has joined #openstack-nova | 12:01 | |
| *** Maike_ has quit IRC | 12:01 | |
| *** mwagner_lap has joined #openstack-nova | 12:02 | |
| *** otter768 has quit IRC | 12:02 | |
| *** Maike has joined #openstack-nova | 12:03 | |
| *** sdake has quit IRC | 12:04 | |
| *** Mike_D_laptop has joined #openstack-nova | 12:08 | |
| *** Mike_D_laptop has quit IRC | 12:08 | |
| *** dulek_ has quit IRC | 12:08 | |
| *** dulek_ has joined #openstack-nova | 12:08 | |
| *** dheeraj-gupta-4 has left #openstack-nova | 12:11 | |
| *** jamielennox is now known as jamielennox|away | 12:12 | |
| *** pradipta has quit IRC | 12:13 | |
| *** jecarey has quit IRC | 12:13 | |
| *** hdd has joined #openstack-nova | 12:15 | |
| *** maurosr is now known as maurosr_ | 12:16 | |
| *** dboik has quit IRC | 12:18 | |
| *** EmilienM|afk is now known as EmilienM | 12:19 | |
| *** rfolco has joined #openstack-nova | 12:19 | |
| *** salv-orl_ has joined #openstack-nova | 12:20 | |
| *** zz_jgrimm has quit IRC | 12:22 | |
| *** maurosr_ has quit IRC | 12:22 | |
| *** timello has quit IRC | 12:22 | |
| *** ekhugen has quit IRC | 12:22 | |
| *** ekhugen has joined #openstack-nova | 12:23 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: remove unused method https://review.openstack.org/172896 | 12:23 |
| *** zz_jgrimm has joined #openstack-nova | 12:24 | |
| *** timello has joined #openstack-nova | 12:24 | |
| *** salv-orlando has quit IRC | 12:24 | |
| *** sdake_ has quit IRC | 12:25 | |
| *** sdake has joined #openstack-nova | 12:25 | |
| *** maurosr_ has joined #openstack-nova | 12:25 | |
| *** maurosr_ is now known as maurosr | 12:26 | |
| *** dprince has joined #openstack-nova | 12:26 | |
| *** gszasz_ has joined #openstack-nova | 12:26 | |
| johnthetubaguy | bauzas: hey, just checking if you can raise the subject of blueprints at the next scheduler meeting | 12:27 |
| *** rushiagr_away is now known as rushiagr | 12:27 | |
| johnthetubaguy | bauzas: seems like there are a good few we can fast re-approve, if they get uploaded, happy to push them through, as before | 12:27 |
| *** sdake_ has joined #openstack-nova | 12:27 | |
| bauzas | johnthetubaguy: sure, I can do | 12:27 |
| bauzas | johnthetubaguy: I just had no time to work on them but sure I will | 12:27 |
| bauzas | johnthetubaguy: I just have something to work downstream and then I'll provide some BPs | 12:28 |
| johnthetubaguy | jogo: hey, just noticed there are a few procedural -2s in nova-specs you can probably lift, like this one: https://review.openstack.org/#/c/139272/ | 12:29 |
| *** sdake has quit IRC | 12:31 | |
| johnthetubaguy | bauzas: no worries, just trying to get things rolling into liberty where possible, so we are less "back loaded" | 12:32 |
| bauzas | johnthetubaguy: sure, np I understand | 12:32 |
| *** hdd has quit IRC | 12:32 | |
| openstackgerrit | sahid proposed openstack/nova-specs: process: add new section History https://review.openstack.org/172058 | 12:35 |
| *** adalbas has joined #openstack-nova | 12:37 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: quit early when mempages requested found https://review.openstack.org/171502 | 12:37 |
| *** romainh has quit IRC | 12:38 | |
| *** romainh has joined #openstack-nova | 12:38 | |
| *** gszasz_ has quit IRC | 12:38 | |
| *** andreykurilin has quit IRC | 12:39 | |
| *** Maike_ has joined #openstack-nova | 12:40 | |
| *** dboik has joined #openstack-nova | 12:40 | |
| *** Maike has quit IRC | 12:40 | |
| *** lucas-hungry is now known as lucasagomes | 12:41 | |
| *** kmartin has quit IRC | 12:43 | |
| *** pkoniszewski has joined #openstack-nova | 12:43 | |
| *** rdo has quit IRC | 12:45 | |
| *** gszasz_ has joined #openstack-nova | 12:45 | |
| *** ttx has quit IRC | 12:45 | |
| *** ttx has joined #openstack-nova | 12:45 | |
| sahid | ndipanov: i just have noticed your comment here https://review.openstack.org/#/c/172079/ | 12:45 |
| sahid | do you think we need a script to clean database and flavor? | 12:46 |
| *** ttx has quit IRC | 12:46 | |
| *** ttx has joined #openstack-nova | 12:46 | |
| *** rdo has joined #openstack-nova | 12:46 | |
| sahid | hum no this is not your point | 12:48 |
| *** eharney has joined #openstack-nova | 12:50 | |
| *** aysyd has joined #openstack-nova | 12:50 | |
| *** jroll has quit IRC | 12:50 | |
| *** jroll has joined #openstack-nova | 12:50 | |
| *** ivasev has joined #openstack-nova | 12:52 | |
| *** markvoelker has joined #openstack-nova | 12:52 | |
| *** dimsum_ has joined #openstack-nova | 12:53 | |
| *** ajayaa has joined #openstack-nova | 12:53 | |
| *** markvoelker_ has joined #openstack-nova | 12:54 | |
| *** aysyd has quit IRC | 12:55 | |
| *** aysyd has joined #openstack-nova | 12:55 | |
| openstackgerrit | sahid proposed openstack/nova: virt: remove small pages request https://review.openstack.org/172079 | 12:55 |
| *** baoli has joined #openstack-nova | 12:55 | |
| *** raildo has joined #openstack-nova | 12:57 | |
| *** markvoelker has quit IRC | 12:58 | |
| *** dimsum_ has quit IRC | 12:58 | |
| *** jichen has quit IRC | 12:59 | |
| *** openstackgerrit has quit IRC | 13:00 | |
| *** Maike_ has quit IRC | 13:00 | |
| *** femnad has joined #openstack-nova | 13:00 | |
| *** Maike_ has joined #openstack-nova | 13:01 | |
| *** toanster has quit IRC | 13:01 | |
| *** adrian_otto has joined #openstack-nova | 13:01 | |
| *** toan has joined #openstack-nova | 13:02 | |
| *** openstackgerrit has joined #openstack-nova | 13:03 | |
| *** krtaylor has quit IRC | 13:03 | |
| *** VW_ has joined #openstack-nova | 13:04 | |
| *** Marga_ has joined #openstack-nova | 13:05 | |
| *** Nikolay_St has quit IRC | 13:06 | |
| *** _gryf is now known as _gryf|afk | 13:10 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Invalidate AZ cache when the instance AZ information is different https://review.openstack.org/172906 | 13:10 |
| openstackgerrit | garyk proposed openstack/nova: VMware: fix log warning https://review.openstack.org/172910 | 13:15 |
| *** erlon has joined #openstack-nova | 13:15 | |
| *** Maike_ has quit IRC | 13:16 | |
| *** kolesovdv has joined #openstack-nova | 13:16 | |
| *** ozialien has joined #openstack-nova | 13:16 | |
| *** Maike has joined #openstack-nova | 13:17 | |
| *** popw has quit IRC | 13:18 | |
| *** popw has joined #openstack-nova | 13:19 | |
| *** smccully has joined #openstack-nova | 13:19 | |
| *** aix has joined #openstack-nova | 13:24 | |
| openstackgerrit | garyk proposed openstack/nova: libvirt: remove unnecesary quotes https://review.openstack.org/172913 | 13:25 |
| *** neelashah has joined #openstack-nova | 13:27 | |
| *** panbalag has joined #openstack-nova | 13:27 | |
| bauzas | garyk: re: https://review.openstack.org/#/c/172906/1 feel free to amend the commit msg if you're seeing that kind of mistake :) | 13:29 |
| bauzas | garyk: but no worries, I'll handle it | 13:30 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Invalidate AZ cache when the instance AZ information is different https://review.openstack.org/172906 | 13:30 |
| *** andreykurilin has joined #openstack-nova | 13:30 | |
| *** gszasz_ has quit IRC | 13:32 | |
| *** busterswt has joined #openstack-nova | 13:33 | |
| garyk | bauzas: thanks - i do not think that it is a good idea that i edit your commit messages | 13:34 |
| *** baoli has quit IRC | 13:34 | |
| bauzas | garyk: well, don't worry, I think it's even better to help the proposers by fixing such obvious mistakes | 13:35 |
| bauzas | garyk: amending a commit msg is very easy thanks to the Gerrit UI so I'm OK | 13:35 |
| bauzas | garyk: I even discussed with sdague and we agreed on helping proposers to fix trivial issues by uploading a new PS if so | 13:36 |
| bauzas | garyk: I think that makes the proposer happy to see we are now fencing him just for nitpicks | 13:37 |
| bauzas | s/now/not | 13:37 |
| *** baoli has joined #openstack-nova | 13:37 | |
| garyk | bauzas: i do not think that we should update commit messages - constructive inputs should encourage the owner to update the commit message | 13:37 |
| bauzas | but instead, we're helping hum | 13:37 |
| bauzas | him | 13:37 |
| * bauzas has fat fingers | 13:37 | |
| garyk | no, i do not think that it is helping - it is actually a little rude in my opinion :) | 13:37 |
| bauzas | why ? :) | 13:37 |
| bauzas | don't you think it's ruder to get a -1 for that kind of mistake ? | 13:38 |
| *** ferest has joined #openstack-nova | 13:38 | |
| *** jecarey has joined #openstack-nova | 13:38 | |
| *** ajayaa has quit IRC | 13:38 | |
| garyk | cause that person invested time and energy writing the commit message. and maybe due to fat fingers they made a mistake and would like to correct it in their own unique way. | 13:38 |
| garyk | a -1 is not rude. the purpose of -1 is to indicate that there is a problem or a suggestion | 13:38 |
| bauzas | garyk: but if the proposer doesn't like the modification, he can either rollback or provide another status | 13:38 |
| garyk | bauzas: if the proposer does not like to make modifications then this is not the correct environment for them to work in | 13:39 |
| garyk | seriously | 13:39 |
| sdague | garyk: well, I disagree. We have lots of patches trapped by trivial things like this, and moving the ball forward is fine. | 13:39 |
| *** adrian_otto has quit IRC | 13:39 | |
| garyk | sdague: you have the right to disagree. i find it sad that someone is not willing to update commit messages and needs a babysitter for that. | 13:40 |
| bauzas | garyk: I mean, imagine that my trivial fix is ready to be merged | 13:40 |
| bauzas | garyk: we couldn't merge it because of that stupid typo I made | 13:40 |
| garyk | bauzas: that was not a typo | 13:40 |
| garyk | a typo is ok - that did not allow the link to the bug. | 13:41 |
| bauzas | garyk: if we would leave the proposer fix that fat thing, then he could have to rework it again and getting again momentum, while if not, he could get 2x +2s quickly even in his absence | 13:41 |
| garyk | bauzas: someone reviewing the code should apply their discetion | 13:41 |
| garyk | bauzas: you are making a mountain out of an anthill | 13:41 |
| *** kolesovdv has quit IRC | 13:41 | |
| garyk | bauzas: you could have left the -1 and waited for a core to +2 it. | 13:42 |
| garyk | bauzas: did you have a +2. no. | 13:42 |
| bauzas | garyk: mmm, I was just opening a discussion for sharing thoughts - I just think it's nicer to help the proposer :) | 13:42 |
| *** ferest has quit IRC | 13:42 | |
| bauzas | garyk: I honestly discussed about my own patch because that's where you reviewed | 13:42 |
| garyk | and the inline commenty was not helping? i am lost | 13:42 |
| *** rushiagr is now known as rushiagr_away | 13:42 | |
| *** rushil has joined #openstack-nova | 13:43 | |
| bauzas | garyk: eh, I'm not saying you made a bad thing - I just consider it's worth giving you my opinion that it's OK to amend the commit msgs | 13:43 |
| garyk | bauzas: look at https://review.openstack.org/#/c/172633/ - here i had a number of issue and was not willing to block it on nits. | 13:43 |
| *** dulek_ has quit IRC | 13:43 | |
| bauzas | garyk: don't over react, look at my logs again :) | 13:44 |
| *** annegentle has joined #openstack-nova | 13:44 | |
| garyk | bauzas: sure, i just personally do not feel comfortable updating a commit message written by someone else. | 13:44 |
| bauzas | (15:29:57) bauzas: garyk: re: https://review.openstack.org/#/c/172906/1 feel free to amend the commit msg if you're seeing that kind of mistake :) | 13:44 |
| bauzas | (15:30:09) bauzas: garyk: but no worries, I'll handle it | 13:44 |
| bauzas | I was saying "feel free", ie. I'm OK with you amending that :) | 13:44 |
| *** rushiagr_away is now known as rushiagr | 13:45 | |
| garyk | whatever - i just do not feel that it was that urgent that i update something that you can address | 13:45 |
| *** markvoelker_ has quit IRC | 13:46 | |
| *** mjturek1 has joined #openstack-nova | 13:46 | |
| *** vladikr has joined #openstack-nova | 13:47 | |
| *** ajayaa has joined #openstack-nova | 13:50 | |
| *** mjturek1 has quit IRC | 13:51 | |
| *** mjturek1 has joined #openstack-nova | 13:51 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Return bandwidth usage after updating https://review.openstack.org/172524 | 13:51 |
| *** thomasem has joined #openstack-nova | 13:53 | |
| *** thomasem has quit IRC | 13:54 | |
| *** ferest has joined #openstack-nova | 13:54 | |
| *** Nikolay_St has joined #openstack-nova | 13:54 | |
| *** thomasem has joined #openstack-nova | 13:55 | |
| *** wverdugo500 has joined #openstack-nova | 13:57 | |
| *** loquacities has quit IRC | 13:57 | |
| *** popw has quit IRC | 13:57 | |
| *** otter768 has joined #openstack-nova | 13:58 | |
| openstackgerrit | Joel Coffman proposed openstack/nova-specs: Stop encrypted disk on instance suspend/power off https://review.openstack.org/172924 | 13:58 |
| *** ferest has quit IRC | 13:58 | |
| *** loquacities has joined #openstack-nova | 13:59 | |
| *** sdake has joined #openstack-nova | 13:59 | |
| *** artom has joined #openstack-nova | 14:00 | |
| *** artom has quit IRC | 14:01 | |
| *** artom has joined #openstack-nova | 14:01 | |
| *** superdan is now known as dansmith | 14:02 | |
| *** sdake_ has quit IRC | 14:02 | |
| *** otter768 has quit IRC | 14:03 | |
| *** Maike has quit IRC | 14:03 | |
| *** Maike has joined #openstack-nova | 14:04 | |
| *** wverdugo500 has left #openstack-nova | 14:04 | |
| *** dboik_ has joined #openstack-nova | 14:05 | |
| *** inhumanitas has joined #openstack-nova | 14:06 | |
| *** cfriesen has joined #openstack-nova | 14:06 | |
| *** thangp has joined #openstack-nova | 14:07 | |
| dansmith | sdague: wanna re-hit these ec2 objects patches? https://review.openstack.org/#/c/149746/15 | 14:08 |
| sdague | dansmith: sure | 14:08 |
| sdague | dansmith: are those going to be backport proposed? | 14:08 |
| sdague | or just liberty only | 14:08 |
| *** dboik has quit IRC | 14:08 | |
| *** dave-mccowan has joined #openstack-nova | 14:09 | |
| dansmith | sdague: liberty onl | 14:09 |
| sdague | ok, cool | 14:09 |
| *** lascii is now known as alaski | 14:09 | |
| *** dimsum_ has joined #openstack-nova | 14:10 | |
| *** wverdugo500 has joined #openstack-nova | 14:10 | |
| *** wverdugo500 has left #openstack-nova | 14:10 | |
| *** unicell1 has joined #openstack-nova | 14:10 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/nova: Ironic: do not destroy if node is in maintenance https://review.openstack.org/172930 | 14:11 |
| *** unicell has quit IRC | 14:11 | |
| *** rushiagr is now known as rushiagr_away | 14:12 | |
| *** nkrinner has quit IRC | 14:14 | |
| *** tellesnobrega has quit IRC | 14:17 | |
| *** sdake_ has joined #openstack-nova | 14:18 | |
| *** tellesnobrega has joined #openstack-nova | 14:19 | |
| *** krtaylor has joined #openstack-nova | 14:20 | |
| *** annashen has joined #openstack-nova | 14:20 | |
| *** BadCub_Hiding is now known as BadCub | 14:20 | |
| *** sdake has quit IRC | 14:21 | |
| *** jichen has joined #openstack-nova | 14:23 | |
| *** heyongli_ has joined #openstack-nova | 14:23 | |
| alex_xu | johnthetubaguy: thanks for the approve | 14:27 |
| heyongli_ | ndipanov: got you comments, will update according the comments. thanks | 14:27 |
| ndipanov | heyongli_, so they make sense :) - awesome | 14:28 |
| heyongli_ | ndipanov: really thanks to make i on track, :-D, rabitmq down on dev machine... | 14:29 |
| *** jistr is now known as jistr|mtg | 14:29 | |
| *** dboik_ has quit IRC | 14:30 | |
| *** dboik has joined #openstack-nova | 14:30 | |
| *** achanda has joined #openstack-nova | 14:31 | |
| *** rushiagr_away is now known as rushiagr | 14:35 | |
| *** dimsum_ is now known as dims | 14:39 | |
| *** doron_afk has quit IRC | 14:40 | |
| *** Hosam has joined #openstack-nova | 14:41 | |
| *** achanda has quit IRC | 14:41 | |
| *** heyongli_ has quit IRC | 14:44 | |
| *** Longgeek has quit IRC | 14:45 | |
| *** TobiasE has joined #openstack-nova | 14:45 | |
| *** Longgeek has joined #openstack-nova | 14:46 | |
| *** heyongli_ has joined #openstack-nova | 14:47 | |
| *** markvoelker has joined #openstack-nova | 14:47 | |
| *** sdake has joined #openstack-nova | 14:47 | |
| *** markvoelker_ has joined #openstack-nova | 14:48 | |
| ndipanov | sdague, in mriedem's absence would you look at: https://review.openstack.org/#/c/171304 | 14:49 |
| *** annashen has quit IRC | 14:50 | |
| *** sdake_ has quit IRC | 14:50 | |
| *** heyongli_1 has joined #openstack-nova | 14:51 | |
| *** markvoelker has quit IRC | 14:52 | |
| *** markvoelker has joined #openstack-nova | 14:52 | |
| *** markvoelker_ has quit IRC | 14:52 | |
| *** heyongli_ has quit IRC | 14:53 | |
| *** rwsu has quit IRC | 14:54 | |
| *** rwsu has joined #openstack-nova | 14:54 | |
| *** doron_afk has joined #openstack-nova | 14:55 | |
| *** doron_afk is now known as doron_ | 14:55 | |
| *** heyongli_1 has quit IRC | 14:55 | |
| *** mriedem has joined #openstack-nova | 14:55 | |
| *** heyongli_1 has joined #openstack-nova | 14:56 | |
| *** welldannit has joined #openstack-nova | 14:57 | |
| openstackgerrit | Chris Friesen proposed openstack/nova: Fix "down" nova-compute service spuriously marked as "up" https://review.openstack.org/163060 | 14:58 |
| *** markvoelker has quit IRC | 14:58 | |
| *** mriedem has quit IRC | 14:59 | |
| sdague | ndipanov: sure, looking now | 15:01 |
| *** hdd has joined #openstack-nova | 15:02 | |
| *** ankit_ag has quit IRC | 15:02 | |
| *** ganso_ has joined #openstack-nova | 15:03 | |
| *** dboik_ has joined #openstack-nova | 15:03 | |
| *** kmartin has joined #openstack-nova | 15:03 | |
| *** mriedem has joined #openstack-nova | 15:03 | |
| ndipanov | sdague, thanks | 15:04 |
| *** TobiasE has quit IRC | 15:05 | |
| *** dboik has quit IRC | 15:07 | |
| *** jistr|mtg is now known as jistr | 15:07 | |
| *** sdake_ has joined #openstack-nova | 15:07 | |
| *** zzzeek has joined #openstack-nova | 15:08 | |
| openstackgerrit | dave-mcnally proposed openstack/nova: rebuild_instance doesn't detach cinder volumes correctly https://review.openstack.org/172951 | 15:09 |
| sdague | ndipanov: hmmm... should there be a test that the field is dropped correctly in the old version of the object? | 15:10 |
| *** dave-mccowan has quit IRC | 15:10 | |
| *** hdd has quit IRC | 15:10 | |
| mriedem | yes, that was my -1 on friday | 15:11 |
| *** garyk has quit IRC | 15:12 | |
| pczesno | mriedem, sdague i'm adding this right now | 15:12 |
| *** whenry has quit IRC | 15:12 | |
| *** annegentle has quit IRC | 15:12 | |
| sdague | pczesno: ok, I'm -1 until we get that, +2 after that's posted | 15:12 |
| *** annegentle has joined #openstack-nova | 15:13 | |
| pczesno | sdague , thanks | 15:13 |
| mriedem | yeah, re-applied my -1 | 15:13 |
| mriedem | 'we don't test these elsewhere' isn't a good reason to not test them :) | 15:13 |
| *** sdake has quit IRC | 15:13 | |
| sdague | especially because it's a specific regression | 15:13 |
| mriedem | right, if a feature is completely busted, we're missing testing somewhere | 15:14 |
| mriedem | sahid: changing the default to False here would require a 6 month cycle I'd think https://review.openstack.org/#/c/159275/ | 15:15 |
| mriedem | since it's a change in default behavior | 15:15 |
| *** zz_jgrimm is now known as jgrimm | 15:15 | |
| mriedem | sahid: i didn't really want to say in the config option help that handling lifecycle events from the libvirt driver isn't tested well at scale | 15:16 |
| mriedem | sdague: thoughts on this? https://review.openstack.org/#/c/159275/ | 15:16 |
| sdague | mriedem: so.... I'm really not keen on dropping that entirely after that was supposed to take load off the old periodic job (and let you run it less often) | 15:18 |
| *** afazekas has quit IRC | 15:18 | |
| mriedem | so is there some history i'm missing here? pre-havana? | 15:18 |
| sahid | mriedem: we have 6 months to fix the issue here now so i do not know if that really make sense to have this option | 15:20 |
| sdague | yeh | 15:20 |
| mriedem | so this is the 'fix' https://review.openstack.org/#/c/172775/ | 15:21 |
| sdague | the theory, at one point, was that part of the reason people had 10 minute polls on the periodic job was to find downed guests in a sane amount of time | 15:21 |
| sdague | but if the event flowed back on it's own more realtime, that could reduce how often the periodic job had to run | 15:21 |
| mriedem | the default on the sync_instance_power_state periodic task is still 10 minutes | 15:22 |
| sdague | I think it's probably worth a DS session on how we want to handle this stuff in the future, because if it turns out that libvirt isn't reliable here, we're not saving ourselves anything | 15:22 |
| mriedem | so if the eventing stuff was the bees knees shouldn't that task default to run every hour or something? | 15:22 |
| sdague | mriedem: yeh, but it could be increased, and people were after this | 15:22 |
| sdague | mriedem: depends on the virt driver though, right? | 15:22 |
| mriedem | sdague: libvirt has been the only driver that implements this | 15:23 |
| mriedem | hyper-v has a change up to add it | 15:23 |
| *** annegentle has quit IRC | 15:23 | |
| sdague | mriedem: no, I mean - sync_instance_power_state | 15:23 |
| mriedem | it's known to be racy in reboot | 15:23 |
| sdague | that's a global | 15:23 |
| mriedem | i don't understand the question | 15:24 |
| mriedem | yes sync_instance_power_state is global | 15:24 |
| mriedem | well, for that compute process | 15:24 |
| mriedem | which should be 1:1 with a virt driver | 15:24 |
| mriedem | and libvirt is the only in-tree driver that implements it today | 15:24 |
| mriedem | the eventing stuff i mean | 15:25 |
| mriedem | it races with soft reboot, we don't test soft reboot well, at least at scale (there is an old tempest bug for the soft reboot test b/c it can't be reliably tested) | 15:25 |
| mriedem | idk, i guess i'm just surprised by the pushback on adding an option to disable a thing (not by default) if you want to b/c of it's dependence on external monitors doing things asynchronously with the compute service | 15:27 |
| mriedem | that just screams dangerous to me | 15:27 |
| ndipanov | mriedem, sdague I commented on that - imho it's irrelevant and a ton of cases misses that test - but I can add it | 15:29 |
| sdague | ndipanov: except this bug is *specifically* about fixing this edge case, so I don't think it's irrelevant | 15:29 |
| *** dave-mccowan has joined #openstack-nova | 15:30 | |
| *** yamahata has joined #openstack-nova | 15:30 | |
| *** russellb has quit IRC | 15:30 | |
| ndipanov | sdague, well it's about fixing from_dict, to_dict, not the backporting as such - so that's fair - we can add that test | 15:31 |
| *** jwcroppe has quit IRC | 15:31 | |
| openstackgerrit | Przemyslaw Czesnowicz proposed openstack/nova: Add numa_node field to PciDevicePool https://review.openstack.org/171304 | 15:32 |
| *** jwcroppe has joined #openstack-nova | 15:32 | |
| sdague | mriedem: well, honestly, I don't really want to deal with this as another bifurcation of how nova works. It seems we should figure out if it's possible to make reliable, or fully delete it. | 15:32 |
| *** mtanino has joined #openstack-nova | 15:32 | |
| *** tjones1 has joined #openstack-nova | 15:32 | |
| *** aysyd has quit IRC | 15:33 | |
| mriedem | i'm sure it's fine when a guest goes down and the user wasn't doing any operations on it that caused that, then you want it - and in most cases when you're doing operations on the instance and the events come, the instance.task_state is set so the events are ignored, | 15:33 |
| *** russellb has joined #openstack-nova | 15:33 | |
| mriedem | the problem is the delay when the compute API operation is complete and the events come in late | 15:34 |
| mriedem | i'm not sure how to shore those up beyond what's already been patched into kilo (and needs to be backported to stable) | 15:34 |
| mriedem | and the delay window is completely arbitrary | 15:34 |
| mriedem | we could test soft reboot in the large ops test but that uses the fake virt driver so doesn't help | 15:35 |
| *** ZZelle_ has joined #openstack-nova | 15:35 | |
| sdague | what other information do we have coming back in that event object? | 15:36 |
| sdague | I wonder if we can premptively know that an event was our own doing, and drop those | 15:37 |
| *** aysyd has joined #openstack-nova | 15:37 | |
| *** tjones1 has quit IRC | 15:37 | |
| mriedem | the instance uuid | 15:37 |
| mriedem | you still need a delay window | 15:38 |
| mriedem | from what i saw in logs from a recreate over the weekend, the stopped event came in after the reboot operation completed | 15:38 |
| mriedem | so the instance was active and task_state was None, then the stopped event came in and shuts down the instance | 15:38 |
| mriedem | 1 second later, the started event comes in but by then we're already stopping the instance and the started event is ignored | 15:38 |
| *** tjones1 has joined #openstack-nova | 15:39 | |
| *** jwcroppe_ has joined #openstack-nova | 15:39 | |
| sdague | does libvirt give us any other information about who requested an action? so we could figure out if this was lifecycle from a nova event? | 15:39 |
| mriedem | the fix in the libvirt driver today (for xen only w/o my patch) is for stopped events, there is a 15 second delay where if a started event comes in for the same instance, the stopped event is removed from the queue and not sent to the compute manager for processing | 15:40 |
| *** Nikolay_St has quit IRC | 15:40 | |
| mriedem | i'd have to dig into that | 15:40 |
| jichen | johnthetubaguy: hi, in case a pre-approved spec in Kilo (no spec) , just in launchpad, how can I make it 'approved' again in L?https://blueprints.launchpad.net/nova/+spec/add-instance-number-as-weight | 15:40 |
| sdague | mriedem: anyway, my pushback is just that it would be good to explore all the options here before pushing for a config file option which makes nova run in 2 different modes | 15:41 |
| johnthetubaguy | jichen: if you add it to liberty services, and liberty-1 it pops up in my searches, and I can get it approved for you | 15:41 |
| mriedem | sdague: yeah, that's fair | 15:41 |
| jichen | johnthetubaguy: you mean if I change to milestone target 'liberty-1' then it's ok? | 15:42 |
| *** claudiub has joined #openstack-nova | 15:42 | |
| *** jwcroppe has quit IRC | 15:42 | |
| *** tjones1 has quit IRC | 15:43 | |
| johnthetubaguy | jichen: no, I mean update the series goal to liberty as well | 15:43 |
| *** garyk has joined #openstack-nova | 15:43 | |
| *** jwcroppe_ has quit IRC | 15:43 | |
| johnthetubaguy | jichen: but since you asked so nicely, I just did that all for you, it should all be approved and unblocked now :) | 15:43 |
| jichen | ok, I saw the update just now , thanks a lot :) | 15:44 |
| johnthetubaguy | no problem | 15:44 |
| *** hemnafk is now known as hemna | 15:45 | |
| *** jistr has quit IRC | 15:45 | |
| *** yamahata has quit IRC | 15:49 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Delay STOPPED lifecycle event for all domains, not just Xen https://review.openstack.org/172775 | 15:49 |
| mriedem | sahid: sdague: i'd appreciate reviews on this in lieu of the config option since i'm going to be backporting it to juno ^ | 15:49 |
| *** tsekiyama has joined #openstack-nova | 15:50 | |
| *** rushil has quit IRC | 15:51 | |
| sahid | mriedem: yes sure i will do | 15:51 |
| *** tjones1 has joined #openstack-nova | 15:52 | |
| *** markvoelker_ has joined #openstack-nova | 15:53 | |
| *** jwcroppe has joined #openstack-nova | 15:54 | |
| *** cbader has joined #openstack-nova | 15:56 | |
| *** ttx has quit IRC | 15:57 | |
| *** tjones1 has quit IRC | 15:57 | |
| *** ttx has joined #openstack-nova | 15:57 | |
| *** igordcard_ has quit IRC | 15:57 | |
| *** markus_z has quit IRC | 15:57 | |
| *** otter768 has joined #openstack-nova | 15:59 | |
| cfriesen | sahid: what do you think about my suggestion to add a remotable function for reporting service state as part of objectifying the service/servicegroup stuff rather than as part of this bugfix? | 15:59 |
| *** markvoelker_ has quit IRC | 15:59 | |
| *** jwcroppe has quit IRC | 15:59 | |
| *** jwcroppe has joined #openstack-nova | 16:00 | |
| *** annegentle has joined #openstack-nova | 16:00 | |
| *** dboik_ has quit IRC | 16:01 | |
| *** dboik has joined #openstack-nova | 16:02 | |
| *** otter768 has quit IRC | 16:03 | |
| *** pkholkin has quit IRC | 16:03 | |
| *** jlanoux has quit IRC | 16:04 | |
| sahid | cfriesen: can you point me the review please ? | 16:04 |
| *** ttx has quit IRC | 16:05 | |
| *** ttx has joined #openstack-nova | 16:05 | |
| sahid | oh i see now | 16:08 |
| *** unicell1 has quit IRC | 16:09 | |
| *** pkholkin has joined #openstack-nova | 16:09 | |
| *** femnad has quit IRC | 16:09 | |
| *** afazekas has joined #openstack-nova | 16:10 | |
| bauzas | cfriesen: yeah, you got the idea, we first need to convert all calls to objects and then do the necessary stuff on objects | 16:10 |
| bauzas | cfriesen: I'm against any new conductor stuff, I'm talking about objects | 16:10 |
| *** romainh has left #openstack-nova | 16:11 | |
| heyongli_1 | ndipanov: i'm afraid can not finish revise today, so update you later. | 16:14 |
| *** jichen has quit IRC | 16:14 | |
| *** jichen has joined #openstack-nova | 16:15 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: testing ceph job with unskipped test_volume_boot_pattern https://review.openstack.org/172983 | 16:16 |
| cfriesen | sahid: my review is https://review.openstack.org/168418, Hans' proposed objectifying of service/servicegroup is https://review.openstack.org/133663 | 16:16 |
| *** smccully has quit IRC | 16:17 | |
| *** smccully has joined #openstack-nova | 16:17 | |
| *** apuimedo has quit IRC | 16:17 | |
| bauzas | dansmith: re: above, I think you would be interested in https://review.openstack.org/#/c/133663/ | 16:18 |
| cfriesen | bauzas: are you suggesting that this change should wait till the objectifying goes in? that work has been in progress since November, who knows how much longer it'll take? | 16:18 |
| *** Longgeek has quit IRC | 16:18 | |
| dansmith | bauzas: on a call, remind me in like 30 | 16:18 |
| *** belmoreira has quit IRC | 16:18 | |
| bauzas | cfriesen: I think hans's patch series is in a good shape for getting merged | 16:18 |
| bauzas | cfriesen: so yeah I think it's worth rebasing on top of it | 16:19 |
| bauzas | cfriesen: because if not, you will be ending up hacks like I mentioned | 16:20 |
| bauzas | ending up in * | 16:20 |
| cfriesen | it's only a _little_ bit hacky. :) | 16:21 |
| bauzas | yeah but errorprone :) | 16:22 |
| *** ozialien has quit IRC | 16:22 | |
| *** dboik has quit IRC | 16:22 | |
| ndipanov | heyongli_1, no worries | 16:24 |
| ndipanov | we have until end of the month | 16:24 |
| *** Sukhdev has joined #openstack-nova | 16:24 | |
| *** isd has joined #openstack-nova | 16:25 | |
| *** lsmola_ has quit IRC | 16:25 | |
| *** hdd has joined #openstack-nova | 16:26 | |
| *** Nic has joined #openstack-nova | 16:26 | |
| *** annashen has joined #openstack-nova | 16:26 | |
| cfriesen | bauzas: okay, I shall resign myself to more waiting. And go review Hans's series properly. :) | 16:26 |
| *** dboik has joined #openstack-nova | 16:26 | |
| *** adrian_otto has joined #openstack-nova | 16:30 | |
| *** afazekas has quit IRC | 16:30 | |
| *** baoli has quit IRC | 16:31 | |
| cfriesen | sdague: just replied to your comments on https://review.openstack.org/#/c/164762/ .... sorry it took so long, I just noticed that I had a draft comment that I hadn't actually published. | 16:31 |
| *** baoli_ has joined #openstack-nova | 16:33 | |
| cfriesen | sdague: as long as there is a delay between the scheduler placing the instance on a host and the instance being added as a group member, there is going to be the possibility of a race for the anti-affinity case. The proposed fix helps a lot in practice. | 16:34 |
| *** sandywalsh has quit IRC | 16:34 | |
| sahid | we have a fix here to prevent a race condition the code is "ready" since a long time now perhaps some cores can take a look ? | 16:35 |
| *** dougwig has quit IRC | 16:35 | |
| sahid | https://review.openstack.org/#/c/146036/ | 16:35 |
| *** sandywalsh has joined #openstack-nova | 16:35 | |
| *** dougwig has joined #openstack-nova | 16:35 | |
| *** patrickeast has joined #openstack-nova | 16:37 | |
| cfriesen | sdague/bauzas: could you remove your -1 from https://review.openstack.org/#/c/162746/9 now that the string freeze no longer applies? | 16:37 |
| *** irenab has joined #openstack-nova | 16:38 | |
| bauzas | cfriesen: just did :) | 16:38 |
| bauzas | cfriesen: could you please stop +1ing yourself btw. ? | 16:38 |
| bauzas | cfriesen: that's not necessary and I hope you're not disagreeing yourself :) | 16:39 |
| *** aysyd has quit IRC | 16:39 | |
| sdague | cfriesen: yeh, and bonus +2 because I liked this patch before | 16:39 |
| *** sahid has quit IRC | 16:40 | |
| *** eglynn has quit IRC | 16:41 | |
| *** subscope_ has joined #openstack-nova | 16:41 | |
| *** FL1SK has joined #openstack-nova | 16:41 | |
| *** aysyd has joined #openstack-nova | 16:42 | |
| *** unicell has joined #openstack-nova | 16:43 | |
| *** annashen has quit IRC | 16:44 | |
| *** mpavone has quit IRC | 16:44 | |
| *** jichen has quit IRC | 16:44 | |
| *** annashen has joined #openstack-nova | 16:45 | |
| *** doron_ has quit IRC | 16:45 | |
| *** unicell has quit IRC | 16:46 | |
| *** unicell has joined #openstack-nova | 16:46 | |
| pczesno | mriedem, sdague : https://review.openstack.org/171304/ is ready for your inspection | 16:46 |
| *** ujjain has quit IRC | 16:51 | |
| *** afazekas has joined #openstack-nova | 16:52 | |
| *** pkoniszewski has quit IRC | 16:53 | |
| *** tonytan4ever has joined #openstack-nova | 16:55 | |
| *** isd has quit IRC | 16:55 | |
| *** yamahata has joined #openstack-nova | 16:57 | |
| *** derekh has quit IRC | 16:57 | |
| dansmith | kashyap: http://logs.openstack.org/14/172614/13/experimental/check-tempest-dsvm-aiopcpu-full/599095f/logs/10.209.0.151-subnode/screen-n-cpu.txt.gz?level=TRACE#_2015-04-11_19_32_41_959 | 16:59 |
| mriedem | pczesno: got it | 16:59 |
| dansmith | kashyap: I think that error is coming from libvirt, but do you know for sure? | 16:59 |
| comstud | oh libvirt | 16:59 |
| mriedem | oh comstud | 16:59 |
| comstud | oh me | 16:59 |
| mriedem | what are you slumming around for? | 17:00 |
| comstud | i miss you guys | 17:00 |
| pczesno | mriedem: thanks !! | 17:00 |
| mriedem | we can fix that | 17:00 |
| *** armax has joined #openstack-nova | 17:00 | |
| comstud | i can fix it too | 17:00 |
| comstud | i can say hi more often | 17:00 |
| comstud | ;p | 17:00 |
| *** sbfox has joined #openstack-nova | 17:00 | |
| cfriesen | bauzas: thanks. and sorry, will stop it. :) | 17:01 |
| bauzas | cfriesen: no worries, we are here for helping you | 17:01 |
| *** VW_ has quit IRC | 17:02 | |
| bauzas | sdague: do you know where in infra I can ask for changing the Gerrit dashboards ? | 17:02 |
| *** rajesht has quit IRC | 17:02 | |
| *** haomaiw__ has joined #openstack-nova | 17:02 | |
| bauzas | sdague: ie. https://review.openstack.org/#/projects/openstack/nova,dashboards/important-changes:important-changes-dashboard should not return the ones having Jenkins -1 IMGO | 17:03 |
| bauzas | IMHO | 17:03 |
| *** marun has joined #openstack-nova | 17:03 | |
| bauzas | at least for the "changes needing review" | 17:03 |
| *** haomaiwang has quit IRC | 17:03 | |
| *** VW_ has joined #openstack-nova | 17:03 | |
| *** VW_ has quit IRC | 17:03 | |
| *** flaper87 has quit IRC | 17:03 | |
| *** VW_ has joined #openstack-nova | 17:04 | |
| *** Marga_ has quit IRC | 17:04 | |
| *** neelashah has quit IRC | 17:04 | |
| *** Marga_ has joined #openstack-nova | 17:05 | |
| * bauzas is just lost by the number of infra repos :D | 17:05 | |
| *** Marga_ has quit IRC | 17:05 | |
| *** andymaier has quit IRC | 17:06 | |
| *** Marga_ has joined #openstack-nova | 17:06 | |
| *** marun has quit IRC | 17:06 | |
| mriedem | dansmith: here is the instance log for that failure if it helps, but it doesn't appear helpful http://logs.openstack.org/14/172614/13/experimental/check-tempest-dsvm-aiopcpu-full/599095f/logs/10.209.0.151-subnode/libvirt/qemu/instance-00000004.txt.gz | 17:07 |
| dansmith | mriedem: yeah | 17:07 |
| mriedem | there is debug logging in the libvirt logs | 17:07 |
| *** subscope_ has quit IRC | 17:08 | |
| *** Maike has quit IRC | 17:08 | |
| *** afazekas has quit IRC | 17:09 | |
| *** annegentle has quit IRC | 17:10 | |
| *** irenab has quit IRC | 17:10 | |
| *** dave-mccowan has quit IRC | 17:11 | |
| *** sbfox has quit IRC | 17:12 | |
| *** BobBall is now known as BobBall_AWOL | 17:12 | |
| *** ozialien has joined #openstack-nova | 17:12 | |
| *** dave-mccowan has joined #openstack-nova | 17:12 | |
| mriedem | this is the error in the libvirt log | 17:13 |
| mriedem | http://logs.openstack.org/14/172614/13/experimental/check-tempest-dsvm-aiopcpu-full/599095f/logs/10.209.0.151-subnode/libvirt/libvirtd.txt.gz#_2015-04-11_19_32_41_946 | 17:13 |
| mriedem | not very helpful i guess | 17:13 |
| *** pixelb has quit IRC | 17:13 | |
| *** flaper87 has joined #openstack-nova | 17:16 | |
| *** tonytan4ever has quit IRC | 17:16 | |
| *** rushil has joined #openstack-nova | 17:16 | |
| *** sbfox has joined #openstack-nova | 17:17 | |
| *** fesp has joined #openstack-nova | 17:18 | |
| *** achanda has joined #openstack-nova | 17:18 | |
| *** ajayaa has quit IRC | 17:19 | |
| mriedem | dansmith: super old related bug https://bugs.launchpad.net/nova/+bug/984996 | 17:20 |
| openstack | Launchpad bug 984996 in OpenStack Compute (nova) "Instance directory does not exist: Unable to pre-create chardev file console.log" [Medium,Invalid] | 17:20 |
| *** fesp has quit IRC | 17:21 | |
| mriedem | in that case it was seen in reboot | 17:21 |
| dansmith | yeah | 17:21 |
| mriedem | also on ask.openstack.org but their solution was NFS | 17:21 |
| mriedem | https://ask.openstack.org/en/question/45282/issue-solvedunable-to-pre-create-chardev-file-optstackdatanovainstanceconsolelog-in-remote-compute-node/ | 17:21 |
| dansmith | I'm sure we're responsible for creating the instance directory, not libvirt, so it must be a nova issue | 17:22 |
| dansmith | yeah | 17:22 |
| *** jwcroppe has quit IRC | 17:22 | |
| mriedem | well, i re-opened the old nova bug | 17:23 |
| mriedem | and linked in the logs | 17:23 |
| *** dsanders has joined #openstack-nova | 17:23 | |
| bauzas | dansmith: so, I was just giving you a change that you would be interested IMHO https://review.openstack.org/#/c/133663/ | 17:24 |
| *** tonytan4ever has joined #openstack-nova | 17:24 | |
| *** jwcroppe has joined #openstack-nova | 17:25 | |
| *** adalbas has quit IRC | 17:25 | |
| dansmith | bauzas: I'm a couple deep on the stack at the moment | 17:25 |
| bauzas | dansmith: np, that was just FYI | 17:25 |
| bauzas | dansmith: there is no urgency | 17:25 |
| bauzas | dansmith: it was related to another discussion I had here | 17:26 |
| *** jcoufal has quit IRC | 17:26 | |
| *** zhenguo has quit IRC | 17:27 | |
| *** ozialien has quit IRC | 17:31 | |
| *** aix has quit IRC | 17:32 | |
| dansmith | mriedem: yeah, it's done in pre_live_migration() | 17:32 |
| *** baoli_ has quit IRC | 17:32 | |
| dansmith | unless we're on shared storage | 17:32 |
| *** pixelb has joined #openstack-nova | 17:33 | |
| *** Marga_ has quit IRC | 17:33 | |
| dansmith | makes me wonder if that migrate_data patch (which broke rpcapi) was further broken in its calculation of the sharedness of storage | 17:33 |
| dansmith | clarkb: this test did pass shortly after you got it fixed up the first time, right? | 17:34 |
| mriedem | dansmith: are you seeing this fail 100% of the time now? | 17:34 |
| *** afazekas has joined #openstack-nova | 17:34 | |
| dansmith | clarkb: ^ | 17:34 |
| mriedem | dansmith: because i've had green runs on the aiopcpu job for weeks (before last week) here: https://review.openstack.org/#/c/169520/ | 17:35 |
| *** irenab has joined #openstack-nova | 17:35 | |
| dansmith | yeah, which is why I was asking, because I thought this was working | 17:36 |
| clarkb | dansmith: ya the test worked until just a few days ago | 17:36 |
| mriedem | we don't index the compute node's logs so it might be hard to tell when it started failing | 17:36 |
| *** browne has joined #openstack-nova | 17:36 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 17:36 | |
| mriedem | it was at least working on 4/9 | 17:37 |
| dansmith | argh | 17:37 |
| *** harlowja_away is now known as harlowja | 17:37 | |
| dansmith | I cut my finger real bad over the weekend and the bandage makes me keep hitting keys I don't want | 17:37 |
| dansmith | like tab | 17:37 |
| *** baoli_ has joined #openstack-nova | 17:37 | |
| *** aix has joined #openstack-nova | 17:37 | |
| cburgess | dansmith: I want to lol at that, but I've been there many times before. | 17:38 |
| *** dikonoor has joined #openstack-nova | 17:38 | |
| dansmith | cburgess: yeah, it's annoying | 17:38 |
| dansmith | clarkb: this would be on master, right? | 17:39 |
| dikonoor | alex_xu : This is about https://review.openstack.org/#/c/150352/ | 17:39 |
| clarkb | dansmith: yes this is all on master | 17:39 |
| dansmith | nothing in pre_live_migration has changed in years | 17:39 |
| *** Marga_ has joined #openstack-nova | 17:40 | |
| *** neelashah has joined #openstack-nova | 17:40 | |
| *** dboik has quit IRC | 17:41 | |
| clarkb | dansmith: we should probably rerun that job against code that isn't touching the multinode stuff in d-g, since I have been seeing this in various d-g changes that could be affecting live migration themselves | 17:42 |
| dikonoor | alex_xu : the change in https://review.openstack.org/#/c/150352/ has added a check that the user who creates a flavor must be an admin (instead of loading the rule defined in the nova policy.json) | 17:42 |
| clarkb | dansmith: I didn't think they were though because they were unrelated changes | 17:42 |
| *** dboik has joined #openstack-nova | 17:42 | |
| dansmith | clarkb: is this running on every nova check now, or only experimental? | 17:42 |
| dansmith | I thought the former | 17:42 |
| clarkb | dansmith: only experimental | 17:42 |
| mriedem | dansmith: all | 17:42 |
| dansmith | lol | 17:42 |
| mriedem | https://review.openstack.org/#/c/172465/ | 17:42 |
| mriedem | check-tempest-dsvm-aiopcpu-full | 17:43 |
| mriedem | it's in check | 17:43 |
| clarkb | oh fun, ignore me then :) | 17:43 |
| mriedem | the neutron backed one is experimental | 17:43 |
| mriedem | it's experimental in devstack | 17:43 |
| dansmith | is it failing in check or only experimental? | 17:43 |
| clarkb | and it failed there so not going to be my d-g changes triggering it | 17:43 |
| clarkb | dansmith: both see http://logs.openstack.org/65/172465/2/check/check-tempest-dsvm-aiopcpu-full/6f4611e/console.html | 17:43 |
| dansmith | okay | 17:43 |
| *** tonytan_brb has joined #openstack-nova | 17:43 | |
| mriedem | remember that each project gets to define it's check and experimental queue jobs :) so it gets confusing keeping track between projects | 17:44 |
| *** Marga_ has quit IRC | 17:44 | |
| mriedem | dansmith: clarkb: this merged on 4/8 https://review.openstack.org/#/c/168408/ | 17:45 |
| *** tonytan4ever has quit IRC | 17:45 | |
| *** marun has joined #openstack-nova | 17:45 | |
| mriedem | but it passed the job on that change so must not be an issue | 17:45 |
| clarkb | mriedem: yes that was the change that made the job pass :) | 17:45 |
| clarkb | mriedem: it was not passing before that | 17:45 |
| clarkb | because ec2 | 17:45 |
| *** irenab has quit IRC | 17:46 | |
| mriedem | hmm, it was passing here http://logs.openstack.org/20/169520/1/check/check-tempest-dsvm-aiopcpu-full/b5d5510/ | 17:46 |
| mriedem | that's 4/2 | 17:46 |
| dansmith | heh, if I had viewed debug level, I'd have seen that we're indeed raising that out of deep in libvirt | 17:46 |
| clarkb | mriedem: yes ec2 broke for a few days before the 8th. My change made it work | 17:46 |
| mriedem | ah ok | 17:46 |
| mriedem | BobBall_AWOL: did the xen ci get sorted out? | 17:47 |
| dansmith | mriedem: looks like that node is the *sending* node | 17:48 |
| dansmith | makes me wonder if this error is after the migration should be done and we've deleted the directory or something? | 17:48 |
| *** aix has quit IRC | 17:49 | |
| mriedem | "VM running on src, migration failed" | 17:49 |
| mriedem | i'd have to dig into the code | 17:49 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove and deprecate unused conductor service calls https://review.openstack.org/133664 | 17:49 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Refactor some service tests and make them not require db https://review.openstack.org/133665 | 17:49 |
| *** VW_ has quit IRC | 17:49 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Convert service and servicegroup to objects https://review.openstack.org/133663 | 17:49 |
| dansmith | mriedem: where do you see that? | 17:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: DNM: testing ceph job with unskipped test_volume_boot_pattern https://review.openstack.org/172983 | 17:51 |
| mriedem | http://logs.openstack.org/14/172614/13/experimental/check-tempest-dsvm-aiopcpu-full/599095f/logs/10.209.0.151-subnode/screen-n-cpu.txt.gz#_2015-04-11_19_32_42_241 | 17:51 |
| mriedem | debug logs right after the failure | 17:51 |
| *** r-daneel has joined #openstack-nova | 17:51 | |
| dansmith | ah yep | 17:51 |
| *** aysyd has quit IRC | 17:51 | |
| *** eglynn has joined #openstack-nova | 17:52 | |
| *** ozialien has joined #openstack-nova | 17:52 | |
| *** aysyd has joined #openstack-nova | 17:53 | |
| *** VW_ has joined #openstack-nova | 17:54 | |
| *** VW__ has joined #openstack-nova | 17:55 | |
| *** dboik_ has joined #openstack-nova | 17:55 | |
| mriedem | dansmith: check the timestamp here http://logs.openstack.org/14/172614/13/experimental/check-tempest-dsvm-aiopcpu-full/599095f/logs/screen-n-cpu.txt.gz#_2015-04-11_19_32_43_458 | 17:56 |
| mriedem | that's on the other node | 17:56 |
| *** markvoelker has joined #openstack-nova | 17:56 | |
| dansmith | well, I was just looking at the line below, where we remove the instance directory | 17:57 |
| *** eglynn has quit IRC | 17:57 | |
| dansmith | clarkb: just to be clear, these do not share any storage, right? | 17:57 |
| clarkb | dansmith: correct | 17:57 |
| clarkb | dansmith: its using the libvirt block sync migration method iirc | 17:57 |
| *** eglynn has joined #openstack-nova | 17:57 | |
| *** afazekas has quit IRC | 17:57 | |
| dansmith | it is, | 17:58 |
| *** mgagne_ is now known as mgagne | 17:58 | |
| dansmith | I just want to make sure they're also not sharing any storage | 17:58 |
| clarkb | yup no nfs or ceph or gluster here | 17:58 |
| *** dboik has quit IRC | 17:58 | |
| *** VW_ has quit IRC | 17:59 | |
| *** Nikolay_St has joined #openstack-nova | 17:59 | |
| *** annegentle has joined #openstack-nova | 17:59 | |
| dansmith | mriedem: ah, delete_instance_files tries to delete an _resize directory, even if it's not there I think | 17:59 |
| *** otter768 has joined #openstack-nova | 18:00 | |
| *** afazekas has joined #openstack-nova | 18:01 | |
| *** markvoelker has quit IRC | 18:01 | |
| mriedem | pre_live_migration sure could use a hell of a lot more debug logging | 18:04 |
| *** otter768 has quit IRC | 18:05 | |
| *** dikonoor has quit IRC | 18:05 | |
| mriedem | o | 18:05 |
| Nic | I'd also take "fewer bugs" ;) | 18:05 |
| mriedem | i'm looking here console_file | 18:05 |
| mriedem | oops | 18:05 |
| mriedem | http://git.openstack.org/cgit/openstack/nova/tree/nova/virt/libvirt/driver.py#n5793 | 18:05 |
| mriedem | Nic: well, if we could get the job voting that tests live migration, that should be the case | 18:05 |
| *** afazekas has quit IRC | 18:05 | |
| mriedem | dansmith: the instances are sharing a volume in this test right? | 18:05 |
| dansmith | mriedem: are they? | 18:06 |
| mriedem | i thought you said they were last week | 18:06 |
| dansmith | if it's block migration, should be no | 18:06 |
| dansmith | unless there is a volume attached also | 18:06 |
| mriedem | checking | 18:06 |
| dansmith | that pre_live_migration stuff is what I was looking at in terms of it assuming shared storage | 18:06 |
| dansmith | but it hasn't changed in a while | 18:06 |
| mriedem | no there isn't a volume - there is a test for that but it's skipped | 18:07 |
| mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_live_migration.py#n112 | 18:07 |
| dansmith | okay | 18:08 |
| *** Marga_ has joined #openstack-nova | 18:09 | |
| *** VW__ has quit IRC | 18:10 | |
| *** dboik_ has quit IRC | 18:11 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Cleanup conductor unused methods https://review.openstack.org/168856 | 18:11 |
| dansmith | mriedem: did you say apr2 is the last day it worked? | 18:11 |
| openstackgerrit | Mathieu Gagné proposed openstack/nova: Don't ignore template argument in get_injected_network_template https://review.openstack.org/171847 | 18:12 |
| *** ijw has joined #openstack-nova | 18:12 | |
| mriedem | dansmith: no, 4/9 | 18:12 |
| mriedem | dansmith: well, that's the last time i have a passing run | 18:12 |
| *** Marga_ has quit IRC | 18:12 | |
| clarkb | mriedem: thats sounds about right (though I only have my memory to work off of) | 18:12 |
| *** Marga_ has joined #openstack-nova | 18:13 | |
| dansmith | so c380987aa8844a46b2d50e55350e89e0791d76b6 merged on apr 9 | 18:13 |
| *** moshele has quit IRC | 18:13 | |
| *** Marga_ has quit IRC | 18:13 | |
| mriedem | b/c on 4/10 we broke it with the rpc api version bump stuff | 18:13 |
| dansmith | which could affect live migration | 18:13 |
| mriedem | yeah i saw that | 18:13 |
| *** Marga_ has joined #openstack-nova | 18:13 | |
| dansmith | lemme try a straw man revert to see if it passes on that | 18:14 |
| mriedem | although we shouldn't be doing numa stuff here | 18:14 |
| mriedem | b/c of the old libvirt version on trusty | 18:14 |
| dansmith | it touches a lot of host and cpu calculation stuff | 18:14 |
| dansmith | I know | 18:14 |
| dansmith | I gotta do something else while that runs | 18:15 |
| mriedem | re-bandage your finger? | 18:15 |
| mriedem | eat? | 18:15 |
| clarkb | thanks a bunch for looking into this (also nice to see this test env being useful just before release :) ) | 18:16 |
| clarkb | hopefully I get the parallelized setup change in soon so that the tests don't take as much time | 18:16 |
| mriedem | clarkb: yes this is very nice, we just need to get it passing and voting | 18:16 |
| mriedem | there has been pretty good process with CI this release, with this, cells and ceph jobs | 18:16 |
| mriedem | s/process/progress/ | 18:16 |
| *** renatoarmani has joined #openstack-nova | 18:17 | |
| clarkb | the parallelized setup gets the job runtime to be basically the same as the single node runs | 18:17 |
| *** Nikolay_St has quit IRC | 18:17 | |
| *** rpodolyaka has quit IRC | 18:22 | |
| *** tlbr has quit IRC | 18:22 | |
| *** Sukhdev has quit IRC | 18:23 | |
| *** coolsvap|afk is now known as coolsvap | 18:26 | |
| *** aysyd has quit IRC | 18:28 | |
| *** aysyd has joined #openstack-nova | 18:28 | |
| *** lucasagomes is now known as lucas-dinner | 18:30 | |
| *** ijw_ has joined #openstack-nova | 18:32 | |
| *** annegentle has quit IRC | 18:33 | |
| *** VW_ has joined #openstack-nova | 18:33 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Fix collection of metadata for a snapshot of volume backed instance https://review.openstack.org/170243 | 18:33 |
| *** ozialien has quit IRC | 18:33 | |
| *** Mike_D_laptop has joined #openstack-nova | 18:33 | |
| *** ijw has quit IRC | 18:34 | |
| *** coolsvap is now known as coolsvap|afk | 18:36 | |
| *** tonytan_brb is now known as tonytan4ever | 18:36 | |
| *** Marga_ has quit IRC | 18:38 | |
| *** Nikolay_St has joined #openstack-nova | 18:38 | |
| *** Marga_ has joined #openstack-nova | 18:38 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Fix collection of metadata for a snapshot of volume backed instance https://review.openstack.org/170243 | 18:38 |
| *** dboik has joined #openstack-nova | 18:40 | |
| *** armax has quit IRC | 18:40 | |
| *** lpetrut has joined #openstack-nova | 18:41 | |
| *** armax has joined #openstack-nova | 18:42 | |
| *** dprince has quit IRC | 18:43 | |
| *** whenry has joined #openstack-nova | 18:47 | |
| openstackgerrit | Vilobh Meshram proposed openstack/nova: WIP: Replace service groups with the tooz groups https://review.openstack.org/172502 | 18:53 |
| *** adalbas has joined #openstack-nova | 18:54 | |
| *** tjones1 has joined #openstack-nova | 18:55 | |
| *** eglynn has quit IRC | 18:55 | |
| dansmith | sigh | 18:57 |
| openstackgerrit | Dan Smith proposed openstack/nova: DNM: Testing live migration regression https://review.openstack.org/173041 | 18:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: add debug logging to pre_live_migration https://review.openstack.org/173042 | 18:58 |
| mriedem | dansmith: could rebase on that for more logging ^ | 18:58 |
| *** tjones1 has quit IRC | 18:59 | |
| *** dprince has joined #openstack-nova | 18:59 | |
| *** rajesht has joined #openstack-nova | 18:59 | |
| dansmith | mriedem: hmm, conflicts? | 18:59 |
| dansmith | I gotta jump on a call | 19:00 |
| dansmith | feel free to, or I will later | 19:00 |
| *** dboik has quit IRC | 19:00 | |
| dansmith | nm, too many "revert-*" branches | 19:02 |
| openstackgerrit | Dan Smith proposed openstack/nova: DNM: Testing live migration regression https://review.openstack.org/173041 | 19:02 |
| *** rajesht has quit IRC | 19:04 | |
| *** patrickeast_ has joined #openstack-nova | 19:06 | |
| *** patrickeast has quit IRC | 19:06 | |
| *** patrickeast_ is now known as patrickeast | 19:06 | |
| *** ijw has joined #openstack-nova | 19:07 | |
| *** baoli_ has quit IRC | 19:09 | |
| *** ijw_ has quit IRC | 19:09 | |
| *** patrickeast has quit IRC | 19:12 | |
| *** Nikolay_St has quit IRC | 19:14 | |
| *** Nikolay_1t has joined #openstack-nova | 19:15 | |
| *** jichen has joined #openstack-nova | 19:19 | |
| *** marun has quit IRC | 19:20 | |
| *** marun has joined #openstack-nova | 19:20 | |
| *** coolsvap|afk is now known as coolsvap | 19:21 | |
| *** promulo_ has joined #openstack-nova | 19:28 | |
| mriedem | ganso_: are you around? | 19:28 |
| *** dboik has joined #openstack-nova | 19:28 | |
| *** Sukhdev has joined #openstack-nova | 19:28 | |
| *** promulo__ has quit IRC | 19:28 | |
| *** kmartin has quit IRC | 19:33 | |
| *** xyang1 has joined #openstack-nova | 19:33 | |
| *** hdd has quit IRC | 19:35 | |
| *** rushiagr is now known as rushiagr_away | 19:36 | |
| mriedem | jbernard: dansmith: mtreinish: test_volume_boot_pattern is passing with the ceph job now on master: http://logs.openstack.org/83/172983/2/check/check-tempest-dsvm-full-ceph/36e8413/console.html#_2015-04-13_19_00_06_375 | 19:39 |
| dansmith | mriedem: bitchin' | 19:40 |
| mtreinish | mriedem: awesome | 19:40 |
| *** Mike_D_laptop has quit IRC | 19:40 | |
| mtreinish | will +2 the revert, do you have a link? | 19:40 |
| * mtreinish is lazy | 19:40 | |
| mriedem | mtreinish: https://review.openstack.org/#/c/172980/ | 19:40 |
| mriedem | but we should wait for the backport to kilo | 19:40 |
| mriedem | -W until then | 19:41 |
| mtreinish | mriedem: bah, lets expose all the wholes in testing while we can | 19:41 |
| *** flashgordon has quit IRC | 19:42 | |
| mriedem | well, people would probably not be happy with kilo being broken right away https://review.openstack.org/#/c/170913/ | 19:42 |
| mriedem | after lobbying to make that voting | 19:42 |
| mriedem | the cinder guys would freak out | 19:42 |
| mtreinish | mriedem: heh, no one would notice for a bit :) | 19:43 |
| mtreinish | well ttx might | 19:43 |
| *** rushiagr_away is now known as rushiagr | 19:45 | |
| *** emagana has joined #openstack-nova | 19:46 | |
| dansmith | mriedem: looks like the revert already failed | 19:48 |
| *** dboik has quit IRC | 19:48 | |
| *** coolsvap is now known as coolsvap|afk | 19:48 | |
| mriedem | 2015-04-13 19:36:18.110 | {1} tempest.api.compute.admin.test_live_migration.LiveBlockMigrationTestJSON.test_live_block_migration [9.873548s] ... FAILED | 19:49 |
| mriedem | yup | 19:49 |
| mriedem | i'm cleaning up another live migrate race fail patch, maybe when i'm done with that we'll have new results | 19:49 |
| mriedem | my dashboard is littered with -Ws :( | 19:49 |
| bauzas | alaski: soooooo, I'm back working on cells | 19:50 |
| bauzas | alaski: do you have a few mins for discussing on the left failures for the job ? | 19:51 |
| *** sdake has joined #openstack-nova | 19:53 | |
| alaski | bauzas: sure | 19:53 |
| *** patrickeast has joined #openstack-nova | 19:54 | |
| *** kmartin has joined #openstack-nova | 19:54 | |
| *** Marga_ has quit IRC | 19:56 | |
| bauzas | alaski: so, now that we merged all our patches thanks to melwitt, it leaves 3 failures | 19:56 |
| bauzas | http://logs.openstack.org/31/169631/11/check/check-tempest-dsvm-cells/a35572e/console.html | 19:56 |
| bauzas | http://logs.openstack.org/60/163060/18/check/check-tempest-dsvm-cells/5b58baf/console.html | 19:56 |
| bauzas | (two different runs) | 19:56 |
| *** sdake_ has quit IRC | 19:56 | |
| *** achanda has quit IRC | 19:57 | |
| *** coolsvap has joined #openstack-nova | 19:57 | |
| bauzas | alaski: I found a 4th failure here http://logs.openstack.org/47/171847/2/check/check-tempest-dsvm-cells/c9c4aaa/console.html | 19:57 |
| bauzas | alaski: so I would vote for whitelisting tempest.thirdparty.boto.test_ec2_instance_run.InstanceRunTest.test_run_idempotent_instances | 19:58 |
| bauzas | and tempest.thirdparty.boto.test_ec2_instance_run.InstanceRunTest.test_compute_with_volumes | 19:58 |
| bauzas | alaski: as the EC2 API is deprecated anyway, I think we can safely assume to drop support for them | 19:58 |
| artom | In the case of SPICE proxies, does the server_proxyclient_address option really hardcode "the address to which proxy clients (like nova-spicehtml5proxy) should connect"? | 19:59 |
| *** annegentle has joined #openstack-nova | 19:59 | |
| artom | I read that and understand that by default, it essentially forces a nova-spicehtml5proxy to run on every compute node because the default value is 127.0.0.1... | 20:00 |
| *** renatoarmani has quit IRC | 20:00 | |
| alaski | bauzas: I would love to blacklist those, but let me look at what they're testing first | 20:00 |
| *** dboik has joined #openstack-nova | 20:01 | |
| bauzas | alaski: it leaves the racy tempest.api.compute.servers.test_server_actions.ServerActionsTestJSON.test_rebuild_server_in_stop_state and tempest.api.compute.servers.test_disk_config.ServerDiskConfigTestJSON.test_rebuild_server_with_manual_disk_config | 20:01 |
| *** otter768 has joined #openstack-nova | 20:01 | |
| bauzas | alaski: ok sure | 20:01 |
| bauzas | alaski: as we discussed last week, I think we need the racy Instance.save() method for fixing the last 2 failures | 20:02 |
| bauzas | we need *to solve* | 20:02 |
| *** tsekiyam_ has joined #openstack-nova | 20:03 | |
| *** sudipto has joined #openstack-nova | 20:03 | |
| *** promulo__ has joined #openstack-nova | 20:04 | |
| *** promulo_ has quit IRC | 20:05 | |
| *** tsekiyama has quit IRC | 20:05 | |
| *** otter768 has quit IRC | 20:05 | |
| *** annegentle has quit IRC | 20:10 | |
| *** lucas-dinner has quit IRC | 20:10 | |
| *** asettle is now known as asettle-gym | 20:11 | |
| alaski | bauzas: okay, I think I understand the idempotent test now | 20:12 |
| alaski | that one should work, and if it doesn't there's a race with system_metadata being set | 20:12 |
| bauzas | alaski: mmm ? | 20:12 |
| *** hdd has joined #openstack-nova | 20:13 | |
| alaski | the ec2 api waits for compute/api:create() to return and then pulls the instances and sets a key in system_metadata | 20:13 |
| *** rpodolyaka has joined #openstack-nova | 20:13 | |
| alaski | if that's getting dropped the issue is larger than just the ec2 api not working | 20:14 |
| *** tlbr has joined #openstack-nova | 20:14 | |
| alaski | bauzas: what I'm getting at is that I'm fine blacklisting these tests to get to green, with open bugs for these last tests. I just want good open bugs | 20:15 |
| bauzas | alaski: oh I see | 20:16 |
| vladikr | dansmith, Hi, could you please look at this again? : https://review.openstack.org/#/c/155006, just to make sure It's the right direction | 20:17 |
| alaski | bauzas: so the idempotent test looks like a race with setting system_metadata | 20:17 |
| *** neelashah has quit IRC | 20:17 | |
| dansmith | vladikr: yeah | 20:17 |
| vladikr | thanks | 20:17 |
| bauzas | alaski: but correct me, sys_metadata is related to the instance save method ? :) | 20:18 |
| alaski | bauzas: yeah, these might all be related to the same race | 20:18 |
| bauzas | alaski: that's what I think | 20:18 |
| bauzas | alaski: ok, just open a new bug, and I will whitelist this failure | 20:19 |
| dansmith | vladikr: to be honest, I like that less than what you had, because it moves the numa stuff into the base HostInfo class and further away from the test where it's used | 20:20 |
| dansmith | vladikr: and I'm not sure it will address any of jaypipes' concerns | 20:20 |
| dansmith | vladikr: since he was -1 on that detail, I think we should try to catch him when he's here and figure out a path forward | 20:21 |
| dansmith | vladikr: if for some reason he likes this better, then I'll be fine with it | 20:21 |
| *** Marga_ has joined #openstack-nova | 20:24 | |
| *** sbfox has quit IRC | 20:24 | |
| alaski | bauzas: https://bugs.launchpad.net/nova/+bug/1443640 | 20:24 |
| openstack | Launchpad bug 1443640 in OpenStack Compute (nova) "Cells: race condition when saving an instance" [Undecided,New] | 20:24 |
| bauzas | alaski: okay, I'll work on fixing that, I have some ideas | 20:24 |
| dansmith | vladikr: I guess since he's not around, I should put the above in the review | 20:24 |
| *** achanda has joined #openstack-nova | 20:25 | |
| vladikr | dansmith, sure, thanks! I'd prefer to have this method in the numa class as well, I just didn't find another normal way how to reuse this code in the test_driver without creating a dependency the test numa module.. | 20:26 |
| dansmith | vladikr: does this actually get more reuse though? | 20:26 |
| dansmith | vladikr: I mean, the _get_numa_topology() bit is what he was concerned about not testing, but does this actually test it more than you did before? | 20:26 |
| vladikr | dansmith, I have the following patch : https://review.openstack.org/#/c/172448/ | 20:26 |
| dansmith | I don't see it, if so | 20:26 |
| vladikr | where I reused this in the driver tests | 20:27 |
| bauzas | alaski: re: tempest.thirdparty.boto.test_ec2_instance_run.InstanceRunTest.test_compute_with_volumes I don't think we're hitting that race condition | 20:27 |
| dansmith | vladikr: okay, I hadn't looked at that | 20:27 |
| dansmith | vladikr: that will probably make him happier | 20:27 |
| alaski | bauzas: yeah, that just looks like a problem booting | 20:27 |
| *** lpetrut has left #openstack-nova | 20:27 | |
| alaski | bauzas: starting to dig | 20:27 |
| vladikr | dansmith, hope so :) | 20:27 |
| bauzas | alaski: me too | 20:27 |
| dansmith | argh | 20:28 |
| * dansmith curses his finger | 20:28 | |
| *** busterswt has quit IRC | 20:29 | |
| dansmith | vladikr: commented, how's that? | 20:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: retry to undefine network filters during _post_live_migration https://review.openstack.org/169520 | 20:30 |
| mriedem | jogo: cleaned up and now with tests ^ | 20:30 |
| artom | Well, to answer my own question, it does. | 20:30 |
| *** sbfox has joined #openstack-nova | 20:30 | |
| vladikr | dansmith, great, thanks! | 20:30 |
| artom | The IP of the SPICE host is indeed hardcoded in the config file. | 20:30 |
| dansmith | cool | 20:31 |
| artom | How does that work outside of devstack? | 20:31 |
| artom | What if you have many compute hosts? | 20:31 |
| vladikr | artom, that should run on the controller | 20:31 |
| vladikr | not on the compute nodes, afaik | 20:32 |
| artom | You need one proxy per compute. | 20:32 |
| bauzas | alaski: stupid question but where is logged the EC2 API in devstack jobs ? | 20:32 |
| artom | vladikr, ah, so that's how. | 20:32 |
| bauzas | alaski: n-api too ?same process ? | 20:32 |
| *** lpetrut has joined #openstack-nova | 20:32 | |
| *** lpetrut has quit IRC | 20:32 | |
| artom | vladikr, but wait... | 20:32 |
| alaski | bauzas: I was expecting n-api | 20:32 |
| alaski | bauzas: but I'm not finding any request-ids... | 20:32 |
| bauzas | alaski: the same, hence my question | 20:33 |
| artom | The SPICE host to which the html5proxy connects is hardcoded in the config file. | 20:33 |
| *** jgrimm is now known as zz_jgrimm | 20:33 | |
| *** sudipto has quit IRC | 20:33 | |
| artom | So you need one proxy process, each with its own config file, per SPICE host. | 20:33 |
| *** HenryG has quit IRC | 20:33 | |
| artom | (SPICE host == compute node) | 20:33 |
| artom | Is there a 1:1 ratio of controller to compute nodes? | 20:34 |
| bauzas | alaski: and to be clear, I can find the ec2 worker in the n-api log... | 20:34 |
| *** kmartin has quit IRC | 20:34 | |
| *** welldannit has quit IRC | 20:36 | |
| vladikr | artom, no you need one proxy, it should run on the controller. when you connect to it and auth, you will also pass the vm id you want to connect to and the proxy will create the tunnel between your client and the vm on the compute noe | 20:36 |
| vladikr | node | 20:36 |
| alaski | bauzas: I don't think the ec2 api logs anything for a GET | 20:36 |
| artom | vladikr, that's not how it worked in my devstack... | 20:36 |
| artom | I don't mean SPICE native :) | 20:37 |
| artom | I mean the existing mess ;) | 20:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: retry to undefine network filters during _post_live_migration https://review.openstack.org/169520 | 20:37 |
| *** READ10 has joined #openstack-nova | 20:37 | |
| *** HenryG has joined #openstack-nova | 20:37 | |
| *** baoli has joined #openstack-nova | 20:38 | |
| *** hdd has quit IRC | 20:38 | |
| *** thangp has quit IRC | 20:39 | |
| bauzas | alaski: correct... | 20:39 |
| *** tsekiyam_ has quit IRC | 20:41 | |
| *** apuimedo has joined #openstack-nova | 20:41 | |
| *** annashen has quit IRC | 20:41 | |
| *** diegows has joined #openstack-nova | 20:42 | |
| *** tsekiyama has joined #openstack-nova | 20:43 | |
| *** jichen has quit IRC | 20:45 | |
| *** annashen has joined #openstack-nova | 20:45 | |
| *** baoli_ has joined #openstack-nova | 20:45 | |
| *** unicell has quit IRC | 20:45 | |
| bauzas | alaski: mmm, I wonder if that's not related to some cell-ability we provided for the API | 20:46 |
| bauzas | alaski: that wouldn't be done on the EC2 API | 20:46 |
| *** sbfox has quit IRC | 20:46 | |
| bauzas | alaski: unfortunately, we're blind on the error | 20:46 |
| *** baoli has quit IRC | 20:47 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: DNM Log in ec2 api describe_instance https://review.openstack.org/173080 | 20:47 |
| alaski | bauzas: hopefully that helps | 20:48 |
| cfriesen | is there a non-ceilometer replacement for notification_driver=nova.openstack.common.notifier.rpc_notifier ? | 20:49 |
| alaski | bauzas: ahh, this might be it http://logs.openstack.org/31/169631/11/check/check-tempest-dsvm-cells/a35572e/logs/screen-n-cell-region.txt.gz?#_2015-04-13_09_47_52_230 | 20:51 |
| *** kmartin has joined #openstack-nova | 20:51 | |
| *** mjturek1 has quit IRC | 20:52 | |
| alaski | bauzas: must be http://logs.openstack.org/31/169631/11/check/check-tempest-dsvm-cells/a35572e/logs/screen-n-cell-child.txt.gz?#_2015-04-13_09_47_31_978 | 20:52 |
| *** pixelb has quit IRC | 20:53 | |
| *** sdake_ has joined #openstack-nova | 20:53 | |
| alaski | bauzas: we should just blacklist that test, we don't have support for security groups in cells | 20:53 |
| *** andreykurilin has quit IRC | 20:55 | |
| *** ivasev has quit IRC | 20:56 | |
| *** sdake has quit IRC | 20:57 | |
| *** n0ano has quit IRC | 20:57 | |
| bauzas | alaski: oh ok, thanks for investigating, I was unscoped :) | 20:58 |
| *** markvoelker has joined #openstack-nova | 20:59 | |
| alaski | bauzas: I eventually realized that the Tempest test class name is used in the instance name | 20:59 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make ComputeNode.service_id nullable to match db schema https://review.openstack.org/169284 | 20:59 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Don't query compute_node through service object in nova-manage https://review.openstack.org/168907 | 20:59 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Avoid querying for Service in resource tracker https://review.openstack.org/169288 | 21:01 |
| bauzas | alaski: cool to know | 21:02 |
| *** vladikr has quit IRC | 21:02 | |
| *** hdd has joined #openstack-nova | 21:03 | |
| bauzas | alaski: okay, will work on the racy problem by tomorrow, see ya | 21:03 |
| *** markvoelker has quit IRC | 21:03 | |
| alaski | bauzas: later | 21:03 |
| *** ozialien has joined #openstack-nova | 21:04 | |
| *** aysyd has quit IRC | 21:07 | |
| *** annegentle has joined #openstack-nova | 21:09 | |
| *** Marga_ has quit IRC | 21:09 | |
| *** Marga_ has joined #openstack-nova | 21:10 | |
| *** rushiagr is now known as rushiagr_away | 21:10 | |
| *** mmedvede has quit IRC | 21:11 | |
| *** aysyd has joined #openstack-nova | 21:11 | |
| *** tjones1 has joined #openstack-nova | 21:11 | |
| *** annashen has quit IRC | 21:13 | |
| *** ZZelle_ has quit IRC | 21:14 | |
| *** tjones1 has quit IRC | 21:15 | |
| *** hdd has quit IRC | 21:16 | |
| *** adrian_otto has left #openstack-nova | 21:16 | |
| *** annashen has joined #openstack-nova | 21:17 | |
| *** andreykurilin has joined #openstack-nova | 21:20 | |
| *** aix has joined #openstack-nova | 21:21 | |
| *** haomaiw__ has quit IRC | 21:23 | |
| *** gszasz has quit IRC | 21:25 | |
| *** Sukhdev has quit IRC | 21:28 | |
| *** mmedvede has joined #openstack-nova | 21:32 | |
| *** mrda-away is now known as mrda | 21:33 | |
| *** mjturek1 has joined #openstack-nova | 21:34 | |
| *** asettle-gym is now known as asettle | 21:34 | |
| *** frickler_ has joined #openstack-nova | 21:36 | |
| *** frickler has quit IRC | 21:38 | |
| *** oro has joined #openstack-nova | 21:39 | |
| *** jwcroppe has quit IRC | 21:40 | |
| *** jwcroppe has joined #openstack-nova | 21:41 | |
| *** mriedem is now known as mriedem_away | 21:42 | |
| *** apuimedo has quit IRC | 21:43 | |
| *** jwcroppe has quit IRC | 21:45 | |
| mriedem_away | dansmith: before i take off, one interesting thing from the live migrate job with debug logs | 21:45 |
| mriedem_away | https://review.openstack.org/#/c/173042/1/nova/virt/libvirt/driver.py | 21:45 |
| mriedem_away | i don't see this showing up at all: "migrate_data in pre_live_migration" | 21:46 |
| dansmith | mriedem_away: that's kinda what I was worried about | 21:46 |
| mriedem_away | which means the migrate_data dict is None and we set is_shared_block_storage=True | 21:46 |
| dansmith | yeah | 21:46 |
| dansmith | I wonder if we're rpc-pinned | 21:46 |
| * dansmith checks | 21:46 | |
| clarkb | dansmith: mriedem_away could that be related to a job configuration problem? | 21:46 |
| dansmith | yeah | 21:46 |
| mriedem_away | dansmith: kilo-compat setting then? | 21:47 |
| dansmith | we're not pinned though | 21:47 |
| *** tjones1 has joined #openstack-nova | 21:47 | |
| *** tjones1 has left #openstack-nova | 21:47 | |
| dansmith | mriedem_away: didn't we pass this test with the two rpc fixes post rc1 ? | 21:48 |
| mriedem_away | checking, that was late on friday | 21:48 |
| mriedem_away | https://review.openstack.org/#/c/172445/ | 21:48 |
| kashyap | dansmith, Sorry, I was totally afk from afternoon to now -- I saw a prompt go by as I opened my IRC client. | 21:48 |
| mriedem_away | nope | 21:48 |
| mriedem_away | http://logs.openstack.org/45/172445/2/check/check-tempest-dsvm-aiopcpu-full/7c5a35f/console.html.gz#_2015-04-10_17_15_29_004 | 21:48 |
| mriedem_away | shazbot | 21:48 |
| dansmith | kashyap: np, we've ruled out some libvirtness anyway, thanks | 21:49 |
| dansmith | mriedem_away: okay, then must be something still broken there then | 21:49 |
| *** jwcroppe has joined #openstack-nova | 21:49 | |
| mriedem_away | yeah | 21:49 |
| mriedem_away | is migrate_data getting dropped? | 21:49 |
| mriedem_away | i'm going to mark the bug for kilo anyway | 21:49 |
| kashyap | dansmith, Thanks. /me hits the hay. See ya. | 21:49 |
| *** MVenesio has joined #openstack-nova | 21:51 | |
| kashyap | dansmith, Okay, I now looked at the error, and curious, did you figure the root cause? | 21:51 |
| mriedem_away | kashyap: not yet | 21:51 |
| kashyap | Is that something w/ NFS/Permissions? | 21:51 |
| mriedem_away | that's what we're talking about ^ | 21:51 |
| *** welldannit has joined #openstack-nova | 21:51 | |
| dansmith | kashyap: we think it's nova's problem of indicating shared storage when not | 21:51 |
| * kashyap still adjusting to staring at a screen, after a grueling 200 KM road journey | 21:51 | |
| mriedem_away | dansmith: ha i see it | 21:52 |
| dansmith | mriedem_away: yeah? | 21:52 |
| mriedem_away | f yeah | 21:52 |
| mriedem_away | sec | 21:52 |
| *** Marga_ has quit IRC | 21:52 | |
| *** tjones1 has joined #openstack-nova | 21:52 | |
| *** tjones1 has left #openstack-nova | 21:52 | |
| dansmith | we shouldn't be defaulting migrate_data=None in the 4.0 API, at least | 21:52 |
| dansmith | but that's not it here | 21:52 |
| dansmith | oh f | 21:52 |
| dansmith | I see it now too :P | 21:53 |
| mriedem_away | yup | 21:53 |
| dansmith | lol | 21:53 |
| mriedem_away | pushing up the fix | 21:53 |
| dansmith | <-- stupid | 21:53 |
| dansmith | mriedem_away: cool, thanks | 21:53 |
| dansmith | mriedem_away: I can backport it so you can leave when you push | 21:53 |
| mriedem_away | clarkb: big ups for that test :) | 21:53 |
| dansmith | clarkb: indeed, we'd have shipped live migration 3x broken without it :) | 21:53 |
| mriedem_away | dansmith: ok, yeah | 21:53 |
| clarkb | jogo: helped a bunch too :) | 21:54 |
| dansmith | man, I stared at this line for several minutes and didn't see it | 21:54 |
| *** pkoniszewski has joined #openstack-nova | 21:54 | |
| dansmith | while we wait, | 21:55 |
| *** dave-mccowan has quit IRC | 21:55 | |
| dansmith | I'm wondering if "big ups" means anything past 1999 | 21:55 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Pass migrate_data to pre_live_migration https://review.openstack.org/173099 | 21:56 |
| *** Marga_ has joined #openstack-nova | 21:56 | |
| mriedem_away | hey, word to your mother | 21:56 |
| *** dave-mccowan has joined #openstack-nova | 21:56 | |
| dansmith | mriedem_away: nice job, I'll push up a backport, thanks | 21:56 |
| mriedem_away | and big ups for debug logging :) | 21:56 |
| dansmith | heh | 21:56 |
| mriedem_away | alright, i'm out, didn't run tox | 21:57 |
| *** Marga_ has quit IRC | 21:57 | |
| mriedem_away | ttyl | 21:57 |
| dansmith | later | 21:57 |
| kashyap | Nice, I arrived, smart people figure the fix out of thin air. | 21:58 |
| * kashyap hits the hay for real. | 21:58 | |
| *** ozialien has quit IRC | 21:59 | |
| *** markvoelker_ has joined #openstack-nova | 22:00 | |
| *** artom has quit IRC | 22:00 | |
| *** pkoniszewski has quit IRC | 22:02 | |
| *** otter768 has joined #openstack-nova | 22:02 | |
| *** unicell has joined #openstack-nova | 22:05 | |
| *** andreykurilin has quit IRC | 22:05 | |
| *** otter768 has quit IRC | 22:06 | |
| *** tonytan4ever has quit IRC | 22:07 | |
| *** flwang1 has joined #openstack-nova | 22:08 | |
| flwang1 | garyk: ping | 22:08 |
| *** mwagner_lap has quit IRC | 22:10 | |
| *** oro has quit IRC | 22:12 | |
| *** adalbas has quit IRC | 22:14 | |
| *** oro has joined #openstack-nova | 22:16 | |
| *** asettle has quit IRC | 22:19 | |
| *** dboik has quit IRC | 22:21 | |
| *** dboik has joined #openstack-nova | 22:22 | |
| *** jwcroppe has quit IRC | 22:27 | |
| *** annegentle has quit IRC | 22:29 | |
| *** obondarev has quit IRC | 22:29 | |
| *** MVenesio has quit IRC | 22:30 | |
| *** VW_ has quit IRC | 22:30 | |
| *** rushil has quit IRC | 22:30 | |
| *** rushil has joined #openstack-nova | 22:31 | |
| *** Hosam has quit IRC | 22:32 | |
| *** baoli_ has quit IRC | 22:32 | |
| *** IanGovett has quit IRC | 22:33 | |
| dansmith | clarkb: mriedem_away: 2015-04-13 22:27:13.667 | {0} tempest.api.compute.admin.test_live_migration.LiveBlockMigrationTestJSON.test_live_block_migration [19.312009s] ... ok | 22:37 |
| dansmith | \o/ | 22:37 |
| *** obondarev has joined #openstack-nova | 22:37 | |
| *** marun has quit IRC | 22:38 | |
| dansmith | somebody please +W this so we can have nice things: https://review.openstack.org/#/c/173099/1 | 22:39 |
| dansmith | jogo: mikal ^ | 22:39 |
| *** VW_ has joined #openstack-nova | 22:39 | |
| *** jecarey has quit IRC | 22:41 | |
| *** tjones2 has joined #openstack-nova | 22:42 | |
| *** annashen has quit IRC | 22:43 | |
| *** VW_ has quit IRC | 22:44 | |
| *** rushil has quit IRC | 22:44 | |
| *** Marga_ has joined #openstack-nova | 22:49 | |
| *** igordcard_ has joined #openstack-nova | 22:50 | |
| *** mtanino has quit IRC | 22:51 | |
| *** coolsvap has quit IRC | 22:51 | |
| *** tjones2 has left #openstack-nova | 22:53 | |
| *** dprince has quit IRC | 22:56 | |
| *** dims_ has joined #openstack-nova | 23:03 | |
| *** dims has quit IRC | 23:04 | |
| *** mtanino has joined #openstack-nova | 23:04 | |
| *** tjones1 has joined #openstack-nova | 23:07 | |
| alex_xu | dansmith: hi, just question, should we bump this version in aliases to the lastest version https://github.com/openstack/nova/blob/master/nova/scheduler/rpcapi.py#L102 ? | 23:07 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Store context in local store after spawn_n https://review.openstack.org/173112 | 23:11 |
| *** Raildo_ has joined #openstack-nova | 23:15 | |
| dansmith | alex_xu: meaning bump it to 4.2? | 23:16 |
| *** ganso_ has quit IRC | 23:17 | |
| *** jamielennox|away is now known as jamielennox | 23:22 | |
| *** dboik_ has joined #openstack-nova | 23:22 | |
| *** dboik_ has quit IRC | 23:22 | |
| *** dboik_ has joined #openstack-nova | 23:22 | |
| cfriesen | hoping someone with live migration knowledge can answer this...does https://bugs.launchpad.net/nova/+bug/1398999 (Block migrate with attached volumes copies volumes to themselves) apply to the boot-from-volume case? | 23:23 |
| openstack | Launchpad bug 1398999 in OpenStack Compute (nova) "Block migrate with attached volumes copies volumes to themselves" [Undecided,Confirmed] - Assigned to Chris St. Pierre (stpierre) | 23:23 |
| *** isd has joined #openstack-nova | 23:23 | |
| *** dboik has quit IRC | 23:25 | |
| *** Raildo_ has quit IRC | 23:26 | |
| *** Raildo_ has joined #openstack-nova | 23:27 | |
| *** Raildo__ has joined #openstack-nova | 23:27 | |
| *** dave-mccowan has quit IRC | 23:28 | |
| cfriesen | or maybe I should be asking more generally whether nova fully supports live migration of boot-from-volume instances with files on local storage on the compute node | 23:31 |
| *** tjones1 has quit IRC | 23:32 | |
| *** tjones1 has joined #openstack-nova | 23:33 | |
| *** mwagner_lap has joined #openstack-nova | 23:35 | |
| *** marun has joined #openstack-nova | 23:36 | |
| *** asettle has joined #openstack-nova | 23:40 | |
| *** r-daneel has quit IRC | 23:43 | |
| *** zzzeek has quit IRC | 23:43 | |
| alex_xu | dansmith: yes | 23:45 |
| *** marun has quit IRC | 23:47 | |
| *** markvoelker has joined #openstack-nova | 23:49 | |
| dansmith | alex_xu: yeah, we should, but it's not a huge deal, so we could just backport to kilo once it's released | 23:49 |
| alex_xu | dansmith: ok, got it, let me work out patch for it. | 23:51 |
| *** markvoelker_ has quit IRC | 23:51 | |
| dansmith | alex_xu: cool, thanks | 23:51 |
| alex_xu | dansmith: np | 23:51 |
| *** tjones1 has quit IRC | 23:52 | |
| *** markvoelker_ has joined #openstack-nova | 23:53 | |
| *** markvoelker_ has quit IRC | 23:54 | |
| *** dboik_ has quit IRC | 23:55 | |
| *** dboik has joined #openstack-nova | 23:56 | |
| *** markvoelker has quit IRC | 23:56 | |
| cfriesen | with the current code, if I want AMQP notifications what do I need to set for "notification_driver" in nova.conf? The website says nova.openstack.common.notifier.rpc_notifier but that doesn't exist anymore. | 23:57 |
| *** salv-orl_ has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!