Thursday, 2015-04-16

*** markvoelker_ has quit IRC00:02
*** Kevin_Zheng has joined #openstack-nova00:08
*** warewang has joined #openstack-nova00:08
*** oro has quit IRC00:09
*** pixelb has quit IRC00:09
*** yamahata has quit IRC00:10
*** achanda_ has joined #openstack-nova00:11
*** achanda_ has quit IRC00:16
*** achanda has quit IRC00:16
*** READ10 has quit IRC00:19
*** annashen has joined #openstack-nova00:21
*** signed8bit_ZZZzz has quit IRC00:25
*** ozialien has joined #openstack-nova00:35
*** r-daneel has quit IRC00:37
*** dims_ has quit IRC00:40
*** mdbooth has quit IRC00:46
*** asettle has joined #openstack-nova00:47
*** yingjun has joined #openstack-nova00:50
*** annashen has quit IRC00:52
*** mdbooth has joined #openstack-nova00:52
*** a5t3r1x has quit IRC00:55
*** promulo_ has joined #openstack-nova00:57
*** aswadr has joined #openstack-nova00:58
*** jwcroppe has quit IRC01:02
*** promulo__ has quit IRC01:02
*** marun_ has joined #openstack-nova01:06
*** ozialien has quit IRC01:06
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server diagnostics & ext events tests for v2 & v2.1  https://review.openstack.org/17417601:07
*** markvoelker has joined #openstack-nova01:07
*** marun has quit IRC01:08
*** tjones1 has joined #openstack-nova01:10
*** browne has quit IRC01:12
*** READ10 has joined #openstack-nova01:12
*** Kevin_Zheng has quit IRC01:13
*** warewang has quit IRC01:14
*** park_heijlong has joined #openstack-nova01:15
*** thomasem has quit IRC01:15
*** Kevin_Zheng has joined #openstack-nova01:15
*** Administrator_ has joined #openstack-nova01:15
*** shaohe_feng1 has quit IRC01:16
*** baoli has joined #openstack-nova01:17
*** mriedem has joined #openstack-nova01:18
*** baoli has quit IRC01:22
*** jogo has quit IRC01:24
*** tjones1 has left #openstack-nova01:25
*** armax has quit IRC01:28
*** sbfox has quit IRC01:28
*** markvoelker has quit IRC01:30
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server groups tests between v2 and v2.1  https://review.openstack.org/17418101:30
*** tjones1 has joined #openstack-nova01:36
*** mtanino has quit IRC01:37
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server groups tests between v2 and v2.1  https://review.openstack.org/17418101:39
*** dims has joined #openstack-nova01:40
*** erkules_ has joined #openstack-nova01:44
*** yingjun has quit IRC01:47
*** dims has quit IRC01:47
*** erkules has quit IRC01:47
*** yingjun has joined #openstack-nova01:47
*** annashen has joined #openstack-nova01:53
*** kaisers1 has joined #openstack-nova01:56
*** kaisers has quit IRC01:59
*** rfolco has joined #openstack-nova02:02
*** jogo has joined #openstack-nova02:05
*** harlowja is now known as harlowja_away02:05
*** popw has joined #openstack-nova02:07
*** strictlyb has quit IRC02:08
*** ujjain has quit IRC02:08
*** shadyabhi has quit IRC02:09
*** shadyabhi has joined #openstack-nova02:09
*** browne has joined #openstack-nova02:13
*** sdake_ has joined #openstack-nova02:16
*** unicell has quit IRC02:16
*** annashen has quit IRC02:17
*** patrickeast has quit IRC02:18
*** bkopilov has quit IRC02:18
*** sdake__ has joined #openstack-nova02:19
*** sdake has quit IRC02:19
*** sdake_ has quit IRC02:22
*** baoli has joined #openstack-nova02:27
*** MVenesio has joined #openstack-nova02:27
openstackgerritEric Brown proposed openstack/nova: Use correct LOG translation indicator for errors  https://review.openstack.org/17400402:30
*** asettle has quit IRC02:32
*** liusheng has joined #openstack-nova02:33
*** VW_ has joined #openstack-nova02:36
*** VW_ has quit IRC02:43
*** dims has joined #openstack-nova02:48
tjones1gilliard: mikal asked me to complete this patch that chris did and you commented on.  I think i have addressed your concerns.  https://review.openstack.org/#/c/127427/02:49
*** strictlyb has joined #openstack-nova02:50
*** shadyabhi has quit IRC02:51
*** yingjun has quit IRC02:53
*** dims has quit IRC02:53
*** Longgeek has joined #openstack-nova02:57
*** yingjun has joined #openstack-nova02:58
*** IanGovett has quit IRC03:06
*** jwcroppe has joined #openstack-nova03:07
*** marun_ has quit IRC03:07
*** MVenesio has quit IRC03:08
*** MVenesio has joined #openstack-nova03:09
*** VW_ has joined #openstack-nova03:12
*** shadyabhi has joined #openstack-nova03:14
*** armax has joined #openstack-nova03:15
*** tjones1 has quit IRC03:16
*** annashen has joined #openstack-nova03:17
*** tjones1 has joined #openstack-nova03:19
*** asettle has joined #openstack-nova03:21
*** tjones1 has quit IRC03:22
*** baoli has quit IRC03:26
*** VW_ has quit IRC03:30
*** marun_ has joined #openstack-nova03:31
*** achanda has joined #openstack-nova03:33
sdake__anyone seen a problem where nova doesn't show networks when creating 30 vms at once03:36
sdake__http://ur1.ca/k6nvi -> http://paste.fedoraproject.org/211516/1549491403:36
*** haomaiwang has quit IRC03:37
*** bkopilov has joined #openstack-nova03:40
*** annashen has quit IRC03:43
*** marun_ has quit IRC03:44
*** shi has quit IRC03:46
*** shi has joined #openstack-nova03:46
*** chaochin has quit IRC03:47
*** Marga_ has joined #openstack-nova03:49
*** Marga_ has quit IRC03:49
*** Marga_ has joined #openstack-nova03:50
*** MVenesio has quit IRC03:50
*** MVenesio has joined #openstack-nova03:50
*** MVenesio has quit IRC03:54
*** strictlyb has quit IRC03:56
*** mriedem has quit IRC03:57
*** garyk has quit IRC03:58
*** yingjun has quit IRC03:59
*** mriedem has joined #openstack-nova03:59
*** arnaud___ has joined #openstack-nova04:02
*** isd has joined #openstack-nova04:03
*** strictlyb has joined #openstack-nova04:03
*** mriedem has quit IRC04:04
*** Longgeek_ has joined #openstack-nova04:05
*** Longgeek has quit IRC04:06
*** oro has joined #openstack-nova04:06
*** READ10 has quit IRC04:09
*** rushiagr_away is now known as rushiagr04:10
*** dsanders has quit IRC04:16
openstackgerritEric Brown proposed openstack/nova: Use correct LOG translation indicator for errors  https://review.openstack.org/17400404:22
*** ankit_ag has joined #openstack-nova04:25
*** neelashah has joined #openstack-nova04:25
*** neelashah has quit IRC04:25
*** ankit8188 has joined #openstack-nova04:27
*** ankit_ag has quit IRC04:27
*** garyk has joined #openstack-nova04:28
*** achanda has quit IRC04:30
*** yingjun has joined #openstack-nova04:30
*** Nikolay_St has quit IRC04:33
*** dsanders has joined #openstack-nova04:34
*** fawadkhaliq has joined #openstack-nova04:38
*** vilobhmm1 has joined #openstack-nova04:39
*** Nikolay_St has joined #openstack-nova04:40
*** frickler_ is now known as frickler04:43
*** annashen has joined #openstack-nova04:43
*** vivekd has joined #openstack-nova04:48
*** arnaud___ has quit IRC04:52
*** Kevin_Zheng has quit IRC04:56
*** Kevin_Zheng has joined #openstack-nova04:56
*** otter768 has quit IRC04:59
*** achanda has joined #openstack-nova05:00
*** stevemar has joined #openstack-nova05:02
*** rushiagr is now known as rushiagr_away05:02
*** vilobhmm1 has quit IRC05:02
*** vilobhmm1 has joined #openstack-nova05:05
*** _gryf|afk is now known as _gryf05:07
*** Marga_ has quit IRC05:07
*** telemonster has quit IRC05:12
*** annashen has quit IRC05:13
*** telemonster has joined #openstack-nova05:13
*** Kevin_Zheng has quit IRC05:16
*** Administrator_ has quit IRC05:17
*** deepthi has joined #openstack-nova05:23
*** vilobhmm1 has quit IRC05:29
*** dave-mcc_ has quit IRC05:31
*** ajayaa has joined #openstack-nova05:33
*** stevemar has quit IRC05:40
*** dims has joined #openstack-nova05:45
*** vivekd has quit IRC05:46
openstackgerritGhanshyam Mann proposed openstack/nova: Merge server groups tests between v2 and v2.1  https://review.openstack.org/17418105:46
*** Nikolay_St has quit IRC05:47
*** doron_afk has joined #openstack-nova05:48
*** hdd has joined #openstack-nova05:50
*** dims has quit IRC05:50
*** rushiagr_away is now known as rushiagr05:51
*** vivekd has joined #openstack-nova05:51
*** rajesht has joined #openstack-nova05:56
*** xyang1 has quit IRC06:05
*** vivekd has quit IRC06:08
*** annashen has joined #openstack-nova06:11
*** vivekd has joined #openstack-nova06:12
openstackgerritheijlong proposed openstack/nova: Merge used-limits functional tests of v2 and v2.1  https://review.openstack.org/17421506:12
*** oro has quit IRC06:13
*** ajayaa has quit IRC06:16
*** lpetrut has joined #openstack-nova06:16
*** bnemec has quit IRC06:18
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/17194306:18
*** jcoufal has joined #openstack-nova06:18
*** browne has quit IRC06:18
*** dsanders has quit IRC06:20
*** armax_ has joined #openstack-nova06:22
*** armax has quit IRC06:23
*** armax_ is now known as armax06:23
*** apuimedo has joined #openstack-nova06:24
*** isd has quit IRC06:24
*** TobiasE has joined #openstack-nova06:26
*** vivekd has quit IRC06:28
*** jcoufal has quit IRC06:30
*** sahid has joined #openstack-nova06:34
*** cfriesen has quit IRC06:35
*** dmn has joined #openstack-nova06:36
*** moshele has joined #openstack-nova06:37
*** ajayaa has joined #openstack-nova06:38
*** armax has quit IRC06:38
*** asettle has quit IRC06:44
*** annashen has quit IRC06:44
*** fawadkhaliq has quit IRC06:48
*** neelabh has joined #openstack-nova06:49
openstackgerritBelmiro Moreira proposed openstack/nova: No disable reason defined for new services  https://review.openstack.org/13664506:50
*** nkrinner has joined #openstack-nova06:50
*** fawadkhaliq has joined #openstack-nova06:52
*** fawadk has joined #openstack-nova06:53
*** fawadk has quit IRC06:57
*** fawadkhaliq has quit IRC06:57
*** yingjun has quit IRC06:58
*** Mike_D_laptop has joined #openstack-nova06:59
*** otter768 has joined #openstack-nova07:00
*** irenab has joined #openstack-nova07:00
*** dsanders has joined #openstack-nova07:00
*** bnemec has joined #openstack-nova07:00
*** yingjun has joined #openstack-nova07:00
*** gszasz has joined #openstack-nova07:01
*** achanda has quit IRC07:02
*** matrohon has joined #openstack-nova07:02
*** dansmith has quit IRC07:04
*** Administrator_ has joined #openstack-nova07:04
*** achanda has joined #openstack-nova07:04
*** Kevin_Zheng has joined #openstack-nova07:04
*** otter768 has quit IRC07:04
*** markus_z has joined #openstack-nova07:05
*** kashyap has quit IRC07:06
openstackgerritSudipta Biswas proposed openstack/nova: Allow discrete blocks of online pCPUs for pinning  https://review.openstack.org/17318707:06
*** abhishekk has joined #openstack-nova07:07
*** dansmith has joined #openstack-nova07:08
*** dansmith is now known as Guest6767207:08
openstackgerritheijlong proposed openstack/nova: Merge create-backup functional tests of v2 and v2.1  https://review.openstack.org/17422607:10
*** mpavone has joined #openstack-nova07:13
openstackgerritheijlong proposed openstack/nova: Merge create-backup functional tests of v2 and v2.1  https://review.openstack.org/17422607:16
*** liusheng has quit IRC07:18
*** romainh has joined #openstack-nova07:19
*** park_heijlong has quit IRC07:21
*** kashyap has joined #openstack-nova07:21
*** oro has joined #openstack-nova07:23
*** tojuvone has joined #openstack-nova07:24
*** chaochin has joined #openstack-nova07:24
*** achanda has quit IRC07:25
*** jichen has joined #openstack-nova07:25
*** dsanders has quit IRC07:26
*** foexle has joined #openstack-nova07:26
*** kashyap` has joined #openstack-nova07:29
*** dmn has quit IRC07:31
*** neelabh has quit IRC07:31
*** park_heijlong has joined #openstack-nova07:35
openstackgerritheijlong proposed openstack/nova: Merge suspend-server functional tests of v2 and v2.1  https://review.openstack.org/17423007:35
*** belmoreira has joined #openstack-nova07:37
*** jlanoux has joined #openstack-nova07:39
*** jistr has joined #openstack-nova07:43
*** annashen has joined #openstack-nova07:43
openstackgerritsahid proposed openstack/nova-specs: process: add new section History  https://review.openstack.org/17205807:44
openstackgerritsahid proposed openstack/nova: objects: remove field ListOfEnumField  https://review.openstack.org/17128707:45
openstackgerritGhanshyam Mann proposed openstack/nova: Merge V2 and V2.1 hypervisor functional tests  https://review.openstack.org/16708207:55
*** eglynn has joined #openstack-nova07:56
*** fawadkhaliq has joined #openstack-nova07:58
*** doude has quit IRC07:59
*** fawadkhaliq has quit IRC08:03
*** doron_afk has quit IRC08:03
*** vivekd has joined #openstack-nova08:03
*** doron_afk has joined #openstack-nova08:04
*** andreykurilin has joined #openstack-nova08:05
*** Nikolay_St has joined #openstack-nova08:06
*** doude has joined #openstack-nova08:06
openstackgerritheijlong proposed openstack/nova: Merge servers-ips functional tests of v2 and v2.1  https://review.openstack.org/17423708:08
*** ociuhandu has joined #openstack-nova08:08
*** lucasagomes has joined #openstack-nova08:14
*** yingjun has quit IRC08:14
openstackgerritHans Lindgren proposed openstack/nova: Parse objects related datetimes using timeutils.parse_isotime() in db api  https://review.openstack.org/17424208:14
openstackgerritHans Lindgren proposed openstack/nova: Make objects serialize_args() handle datetimes in positional args  https://review.openstack.org/17424308:14
openstackgerritHans Lindgren proposed openstack/nova: Add a TaskLog object  https://review.openstack.org/17424408:14
openstackgerritHans Lindgren proposed openstack/nova: Make instance usage audit use the brand new TaskLog object  https://review.openstack.org/17424508:14
openstackgerritHans Lindgren proposed openstack/nova: Remove unused compute utils methods  https://review.openstack.org/17424608:14
openstackgerritHans Lindgren proposed openstack/nova: Remove and deprecate conductor task_log methods  https://review.openstack.org/17424708:14
*** annashen has quit IRC08:16
*** yingjun has joined #openstack-nova08:19
*** derekh has joined #openstack-nova08:19
*** vivekd_ has joined #openstack-nova08:21
*** vivekd has quit IRC08:22
*** vivekd_ is now known as vivekd08:22
*** Nikolay_St has quit IRC08:23
*** fawadkhaliq has joined #openstack-nova08:24
*** andreykurilin has quit IRC08:26
*** doron_afk is now known as doron08:29
*** zz_johnthetubagu is now known as johnthetubaguy08:29
*** derekh is now known as derekh_afk08:30
*** c0m0 has joined #openstack-nova08:31
*** doron is now known as doron_afk08:32
*** obondarev_ has quit IRC08:33
*** ndipanov has joined #openstack-nova08:33
*** Nikolay_St has joined #openstack-nova08:34
sahidndipanov: hello, can i have you reviews on some tiny small it's pain for me to maintain them and i guess after your reveiw that will help to make it moving08:35
ndipanovsahid, sure08:36
ndipanovsorry was busy with those pci bugs this week did almost no reviews other than that08:36
*** sdake__ has quit IRC08:36
*** park_heijlong has quit IRC08:36
*** obondarev has joined #openstack-nova08:37
sahidndipanov: np at all08:37
sahidjust if you can add them in your queue :)08:37
sahidhttps://review.openstack.org/#/c/164177/, https://review.openstack.org/#/c/171502/, https://review.openstack.org/#/c/164168/08:37
*** sdake has joined #openstack-nova08:39
*** Mike_D_laptop has quit IRC08:40
*** mestery has joined #openstack-nova08:40
*** obondarev has quit IRC08:43
*** mestery_ has quit IRC08:43
*** Murali has joined #openstack-nova08:47
*** garyk has quit IRC08:51
*** garyk has joined #openstack-nova08:52
*** Nikolay_St has quit IRC08:53
*** jyuso1 has joined #openstack-nova08:53
*** Nikolay_St has joined #openstack-nova08:53
rajeshtdims: you around ?08:55
*** pkholkin has quit IRC08:56
openstackgerritDeepak C Shetty proposed openstack/nova: WIP: Nova part of volume snapshot improvements  https://review.openstack.org/17281309:01
*** otter768 has joined #openstack-nova09:01
openstackgerritDeepak C Shetty proposed openstack/nova: libvirt: Use 'relative' flag for online snapshot's commit/rebase operations  https://review.openstack.org/16880509:01
openstackgerritHans Lindgren proposed openstack/nova: Make objects serialize_args() handle datetimes in positional args  https://review.openstack.org/17424309:03
openstackgerritHans Lindgren proposed openstack/nova: Remove and deprecate conductor task_log methods  https://review.openstack.org/17424709:03
openstackgerritHans Lindgren proposed openstack/nova: Remove unused compute utils methods  https://review.openstack.org/17424609:03
openstackgerritHans Lindgren proposed openstack/nova: Make instance usage audit use the brand new TaskLog object  https://review.openstack.org/17424509:03
openstackgerritHans Lindgren proposed openstack/nova: Add a TaskLog object  https://review.openstack.org/17424409:03
*** otter768 has quit IRC09:05
*** Nikolay_St has quit IRC09:08
*** Nikolay_St has joined #openstack-nova09:09
*** pask81 has joined #openstack-nova09:09
*** jaypipes has quit IRC09:10
*** pkholkin has joined #openstack-nova09:12
openstackgerritQin Zhao proposed openstack/nova: Use host AZ instead of instance AZ in scheduler request_spec  https://review.openstack.org/17426909:14
*** ujjain has joined #openstack-nova09:14
*** annashen has joined #openstack-nova09:15
*** pask81 has quit IRC09:16
*** ujjain has quit IRC09:19
*** ujjain has joined #openstack-nova09:19
*** neelabh has joined #openstack-nova09:19
*** dims has joined #openstack-nova09:19
*** pask81 has joined #openstack-nova09:19
*** obondarev has joined #openstack-nova09:22
rajeshtdims: you there ?09:22
*** jang has joined #openstack-nova09:22
*** jang has quit IRC09:23
*** dmn has joined #openstack-nova09:24
*** dims has quit IRC09:24
*** lpetrut has quit IRC09:25
ndipanovpczesno, ping09:29
*** yingjun has quit IRC09:34
*** irenab has quit IRC09:36
*** aswadr has quit IRC09:36
*** vivekd has quit IRC09:40
openstackgerritSudipta Biswas proposed openstack/nova: Allow discrete blocks of online pCPUs for pinning  https://review.openstack.org/17318709:43
*** aix has joined #openstack-nova09:44
*** Maike has joined #openstack-nova09:45
*** ishant has joined #openstack-nova09:45
*** sudipto has joined #openstack-nova09:46
sudiptosahid, wanted to talk throught your comments on my patch at https://review.openstack.org/173187. right time?09:47
*** afazekas has joined #openstack-nova09:47
*** annashen has quit IRC09:49
*** obondarev has quit IRC09:50
*** vivekd has joined #openstack-nova09:50
*** dmn has quit IRC09:52
*** neelabh has quit IRC09:53
*** Marga_ has joined #openstack-nova09:55
*** obondarev has joined #openstack-nova09:59
*** doron_afk has quit IRC10:00
*** lpetrut has joined #openstack-nova10:04
*** IanGovett has joined #openstack-nova10:06
*** dims has joined #openstack-nova10:07
*** irenab has joined #openstack-nova10:07
*** vivekd has quit IRC10:09
*** aloga_ is now known as aloga10:14
*** irenab has quit IRC10:15
pczesnondipanov: pong10:15
pczesnondipanov: i saw the comments about adding tests, i'll do that today10:16
*** cbader has quit IRC10:18
*** asselin has quit IRC10:19
*** pask81 has quit IRC10:23
openstackgerritAnkit Agrawal proposed openstack/nova: Detach volume after deleting the instance  https://review.openstack.org/14573810:24
openstackgerritRajesh Tailor proposed openstack/nova: Fix kwargs['migration'] KeyError in @errors_out_migration decorator  https://review.openstack.org/17428810:25
*** pask81 has joined #openstack-nova10:27
ndipanovpczesno, that would be awesome thanks10:27
ndipanovshout if you need help as consume_from_instance tests will need proper reworking (they are really basic now)10:28
ndipanovpczesno, ^^10:28
pczesnondipanov: ok, thanks10:28
ndipanovpczesno, I think we still have a chance to make it for kilo which will make everyone's life easier10:29
pczesnondipanov: agree, that priority 1 for me today, i'll let you know once i have something10:30
ndipanovgood to know!10:36
ndipanovthanks for the help pczesno10:36
*** doron_afk has joined #openstack-nova10:39
*** pask81 has quit IRC10:39
*** popw has quit IRC10:40
*** irenab has joined #openstack-nova10:42
*** kmartin_ has quit IRC10:46
*** vivekd has joined #openstack-nova10:46
*** fawadkhaliq has quit IRC10:46
*** annashen has joined #openstack-nova10:47
rajeshtdims: thanks :)10:48
*** irenab has quit IRC10:48
dimsrajesht: my pleasure.10:50
*** irenab has joined #openstack-nova10:52
*** beagles has quit IRC10:53
*** derekh_afk is now known as derekh10:54
*** xgsa has quit IRC10:54
*** irenab has quit IRC10:54
*** irenab has joined #openstack-nova10:55
*** Marga_ has quit IRC10:59
*** otter768 has joined #openstack-nova11:02
ndipanovjohnthetubaguy, ping11:03
johnthetubaguyndipanov: hi11:03
ndipanovhttps://bugs.launchpad.net/nova/+bug/1383465 - I think we can actually call this fix released in RC111:03
openstackLaunchpad bug 1383465 in OpenStack Compute (nova) "[pci-passthrough] nova-compute fails to start" [High,In progress] - Assigned to Yongli He (yongli-he)11:03
johnthetubaguyndipanov: yeah, I did wonder about that one, then raise another bug for the follow up fixes?11:03
ndipanovjohnthetubaguy, it's already there11:04
ndipanovlet me find it11:04
ndipanovhttps://bugs.launchpad.net/nova/+bug/141576811:04
openstackLaunchpad bug 1415768 in OpenStack Compute (nova) juno "the pci deivce assigned to instance is inconsistent with DB record when restarting nova-compute" [High,In progress] - Assigned to Nikola Đipanov (ndipanov)11:04
ndipanovgah no it's not11:04
ndipanovsigh11:04
ndipanovyeah - raise another one11:04
ndipanovwill talk to the author about it - since it's not kilo material anymore imho11:04
ndipanovjohnthetubaguy, fwiw - I proposed it to stable juno too and am talking to the stable team about it (seeing that it is esentially a refactoring) https://review.openstack.org/#/c/173225/11:05
*** MVenesio has joined #openstack-nova11:05
johnthetubaguyndipanov: OK, I mean we are open for liberty now right11:06
*** otter768 has quit IRC11:06
ndipanovI think so11:06
ndipanovthe worst part of that bug should be fixed now (compute service failing) so no pressure on the other thing11:07
*** deepthi has quit IRC11:07
*** erlon has joined #openstack-nova11:08
johnthetubaguyndipanov: we got the worst bit into RC1 already though, I thought?11:09
johnthetubaguyor did that not make it?11:09
*** MVenesio has quit IRC11:10
ndipanovjohnthetubaguy, yes we did11:11
johnthetubaguyoh, wait, i see, we need a clean bug to push to stable I guess11:11
johnthetubaguyif you could sort the state of that out, that would be awesome11:12
ndipanovwell not sure I have the karma to do it11:12
ndipanovimho we should move it to fix released11:12
ndipanovand add the also affects bit11:12
ndipanovwhich  I think is done by stable release team11:12
*** xgsa has joined #openstack-nova11:12
johnthetubaguyndipanov: OK, if you can raise the extra bug for the left over bits, and update that commit to point to the new bug, ping me, and I think I can do the rest11:14
ndipanovjohnthetubaguy, sorry that this is such a pain - but it's a pretty bad bug for people who care about pci + it was actually quite tricky to figure out what's going11:14
ndipanovjohnthetubaguy, will do!11:14
johnthetubaguyndipanov: no problem at all, I meant to try sort this one out, certainly looks nasty for pci folks11:14
*** amotoki_ has joined #openstack-nova11:15
*** aix has quit IRC11:17
*** claudiub has joined #openstack-nova11:20
openstackgerritVincent Hou proposed openstack/nova: Correct the order of the parameters in swap_volume  https://review.openstack.org/17430711:22
*** annashen has quit IRC11:23
*** tojuvone has quit IRC11:30
openstackgerritVincent Hou proposed openstack/nova: Correct the order of the parameters in swap_volume  https://review.openstack.org/17430711:34
claudiubjohnthetubaguy: hello11:34
johnthetubaguyclaudiub: hi, afraid I am about to run to lunch, but can I help?11:35
*** vivekd has quit IRC11:35
claudiubjohnthetubeguy: I see. Well, from what I know, master is unfrozen for new BPs, right? If so, can you remove the -2s from some commits?11:35
claudiubjohnthetubaguy: The commits are:  https://review.openstack.org/#/c/138255/  https://review.openstack.org/#/c/127159/  https://review.openstack.org/#/c/145004/11:37
claudiubjohnthetubaguy: enjoy your meal!11:37
johnthetubaguyclaudiub: you need an approved blueprint for liberty, which can mean resubmitting your spec11:39
johnthetubaguyclaudiub: once the blueprint is approved for liberty, we can then remove the -211:39
* johnthetubaguy runs away for lunch11:39
claudiubjohnthetubaguy: I see. Thanks!11:41
*** lucasagomes is now known as lucas-hungry11:45
*** aix has joined #openstack-nova11:45
*** Kevin_Zheng has quit IRC11:45
*** Administrator_ has quit IRC11:45
*** MVenesio has joined #openstack-nova11:46
*** vivekd has joined #openstack-nova11:46
*** fawadkhaliq has joined #openstack-nova11:47
*** jamielennox is now known as jamielennox|away11:47
garykis there any chance of begging for reviews in nova or are we all working on bugs?11:49
openstackgerritsahid proposed openstack/nova: libvirt: introduces a new Guest object  https://review.openstack.org/17431311:51
openstackgerritsahid proposed openstack/nova: libvirt: update driver to use introduced plugin Base  https://review.openstack.org/17431411:51
openstackgerritsahid proposed openstack/nova: libvirt: update driver to use introduced plugin CpuAndNUma  https://review.openstack.org/17431511:51
openstackgerritsahid proposed openstack/nova: libvirt: update driver to use introduced plugin Membacking  https://review.openstack.org/17431611:51
openstackgerritAlex Xu proposed openstack/nova: Add rpc client side version control for conductor task API  https://review.openstack.org/17415211:51
openstackgerritAlex Xu proposed openstack/nova: Update rpc version aliases for kilo  https://review.openstack.org/17414811:51
*** fawadkhaliq has quit IRC11:52
*** shi has quit IRC11:53
*** ujjain has quit IRC11:53
*** shi has joined #openstack-nova11:54
*** ujjain- has joined #openstack-nova11:54
*** ujjain- has quit IRC11:55
*** ujjain- has joined #openstack-nova11:55
*** ishant has quit IRC11:56
openstackgerritAlex Xu proposed openstack/nova: Remove instance group member when delete instance  https://review.openstack.org/17366412:01
*** ujjain- is now known as ujjain12:01
*** pixelb has joined #openstack-nova12:02
*** Marga_ has joined #openstack-nova12:06
pczesnondipanov: hi, i got more sad things12:07
ndipanovpczesno, no surprise there ...12:08
ndipanovwhat up12:08
pczesnondipanov: the pci_requests that are passed in request_spec are always empty12:08
ndipanovhmmm12:08
pczesnondipanov: they are fine in filter_properties though12:08
ndipanovhow soe?12:08
ndipanovhow so12:09
*** vladikr has joined #openstack-nova12:09
pczesnothis is how they look in filter_properties:12:10
pczesnou'pci_requests': InstancePCIRequests(instance_uuid=<?>,requests=[InstancePCIRequest])12:10
pczesnothis is request spec:12:10
pczesnou'pci_requests': {u'instance_uuid': u'2ea62db5-bb16-4d39-a919-a4e2b54d7a0e', u'requests': []}12:10
*** atuvenie has joined #openstack-nova12:10
*** atuvenie_ has joined #openstack-nova12:10
pczesnosomething bad happens in obj_to_primitive?12:10
ndipanovpczesno, fekin obj_to_primitive!12:11
*** atuvenie_ has quit IRC12:11
ndipanovhmmm12:11
pczesnothe impact of this is that you can boot one vm with pci just fine but when you boot multile it could fail12:12
ndipanovpczesno, would be good to knwo the root cause of theis12:12
ndipanovthis*12:12
pczesnook, i'll thake a look12:13
*** tojuvone has joined #openstack-nova12:14
pczesnondipanov: one question though12:14
*** dims has quit IRC12:14
ndipanovshoot12:14
pczesnondipanov: in build_request_spec12:14
*** vivekd has quit IRC12:14
pczesnohttps://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L5312:15
ndipanovyeah?12:15
pczesnothe instance argument can be either Instance or Flavor, do you know on what it depened?12:15
ndipanovno - that's not right12:16
ndipanovinstance is always instance12:16
ndipanovinstance_type is the flavor12:16
*** rasoto_ has joined #openstack-nova12:16
ndipanovright?12:16
pczesnooh, right12:16
pczesnogot it12:16
ndipanovthat's the old name for flavor12:16
ndipanovthat we didn't quite get rid of12:16
ndipanovf12:16
*** dims__ has joined #openstack-nova12:17
*** zhenguo has quit IRC12:17
rasoto_hello all, I am looking for information on how to enable multipath within openstack, can someone point me in the right direction?12:17
pczesnoso it must be somewhere in compat_instance that pci_requests got broken12:17
*** beagles has joined #openstack-nova12:17
*** dprince has joined #openstack-nova12:20
*** vivekd has joined #openstack-nova12:22
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004512:23
ndipanovpczesno, that was my guess too12:23
*** annashen has joined #openstack-nova12:25
openstackgerritYongli He proposed openstack/nova: resource tracker style pci resource management  https://review.openstack.org/13132112:25
*** MVenesio has quit IRC12:25
*** bkopilov has quit IRC12:27
*** heyongli_ has joined #openstack-nova12:28
*** ivasev has joined #openstack-nova12:29
*** cbader has joined #openstack-nova12:30
pczesnondipanov: i think i got it12:30
pczesnohttps://github.com/openstack/nova/blob/master/nova/objects/instance_pci_requests.py#L5512:31
pczesnoInstancePCIRequests is not of type ObjectListBase12:31
pczesnoso the requests list will not be converted to primitives12:32
*** heyongli_ has quit IRC12:32
*** raildo has joined #openstack-nova12:32
*** vivekd has quit IRC12:36
ndipanovpczesno, right - that's what I was chatting about with bauzas yesterday12:39
ndipanovbut it hits the final else in https://github.com/openstack/nova/blob/master/nova/objects/base.py#L91212:39
ndipanovpczesno, or actually first it hits https://github.com/openstack/nova/blob/master/nova/objects/base.py#L90112:40
ndipanovconverts it to dict with instance_uuid12:40
ndipanovand then when it hits the requests field12:40
ndipanovwhich is a list12:40
ndipanovit recursively goes to line 91212:40
ndipanovso should get [<InstancePciRequest >]12:41
pczesnondipanov: ok, yes, but in request spec we get []12:41
ndipanovpczesno, that's the million dollar question  - why!12:41
ndipanovor rather12:41
ndipanov4$ question12:41
ndipanov:)12:42
openstackgerritTimofey Durakov proposed openstack/nova: fixed tests in test_iptables_network to work with random PYTHONHASHSEED  https://review.openstack.org/16335212:42
openstackgerritTimofey Durakov proposed openstack/nova: refactored tests to pass in test_inject to pass with random PYTHONHASHSEED  https://review.openstack.org/16335412:42
openstackgerritTimofey Durakov proposed openstack/nova: fix for vmware test_driver_api to pass with random PYTHONHASHSEED  https://review.openstack.org/16335712:42
openstackgerritTimofey Durakov proposed openstack/nova: fixed tests in test_matchers to pass with random PYTHONHASHSEED  https://review.openstack.org/16335812:42
openstackgerritTimofey Durakov proposed openstack/nova: fixed tests in test_instance to pass with random PYTHONHASHSEED  https://review.openstack.org/16113512:42
openstackgerritTimofey Durakov proposed openstack/nova: refactored test in vmware test_read_write_util to pass with random PYTHONHASHSEED  https://review.openstack.org/16335912:42
openstackgerritTimofey Durakov proposed openstack/nova: Refactored tests in test_neutron_driver to pass with random PYTHONHASHSEED  https://review.openstack.org/16336112:42
openstackgerritTimofey Durakov proposed openstack/nova: refactored tests in test_objects to pass with random PYTHONHASHSEED  https://review.openstack.org/16131312:42
openstackgerritTimofey Durakov proposed openstack/nova: Removed 'PYTHONHASHSEED=0' from tox.ini  https://review.openstack.org/16387712:42
openstackgerritTimofey Durakov proposed openstack/nova: Fixed tests in test_glance to pass with random PYTHONHASHSEED  https://review.openstack.org/16336412:42
openstackgerritTimofey Durakov proposed openstack/nova: Fixed test in test_tracker to work with random PYTHONHASHSEED  https://review.openstack.org/17435612:42
openstackgerritgaryk proposed openstack/nova: Resource tracker: unable to restart nova compute  https://review.openstack.org/17379412:42
pczesnondipanov: ok, i'll look further12:42
*** baoli has joined #openstack-nova12:44
*** vivekd has joined #openstack-nova12:45
*** aysyd has joined #openstack-nova12:46
*** asettle has joined #openstack-nova12:46
*** abhishekk has quit IRC12:46
*** lucas-hungry is now known as lucasagomes12:49
*** jichen has quit IRC12:51
*** Marga_ has quit IRC12:51
openstackgerritSudipta Biswas proposed openstack/nova: Allow discrete blocks of online pCPUs for pinning  https://review.openstack.org/17318712:54
*** dboik has joined #openstack-nova12:55
*** mriedem has joined #openstack-nova12:56
*** deepthi has joined #openstack-nova12:57
*** annashen has quit IRC12:57
*** VW_ has joined #openstack-nova12:57
*** Marga_ has joined #openstack-nova12:58
*** eharney has quit IRC13:00
*** pkoniszewski has joined #openstack-nova13:00
*** baoli has quit IRC13:01
*** mpaolino has joined #openstack-nova13:01
*** baoli has joined #openstack-nova13:02
*** otter768 has joined #openstack-nova13:03
*** artom has joined #openstack-nova13:03
*** annegentle has joined #openstack-nova13:04
*** Nikolay_St has quit IRC13:04
*** otter768 has quit IRC13:07
*** deepthi has quit IRC13:08
*** Nikolay_St has joined #openstack-nova13:09
*** jecarey has joined #openstack-nova13:09
*** fawadkhaliq has joined #openstack-nova13:10
*** neelashah has joined #openstack-nova13:10
*** abhishekk has joined #openstack-nova13:10
*** jecarey_ has joined #openstack-nova13:12
kaisers1Hello Nova! Question regarding nova configuration: In devstack, how can i configure which system user Nova is using when creating snapshots (via qemu)? Seems by default root user is used?13:15
*** Marga_ has quit IRC13:15
*** jecarey has quit IRC13:15
rasoto_hello all, does anyone has experience enabling iscsi multipath in openstack?13:16
*** ferest has joined #openstack-nova13:18
sdagueoh, hey, nova-compute doesn't start if there was a boot from volume server out there - http://logs.openstack.org/91/173791/11/check/check-grenade-dsvm/fc725f5/logs/new/screen-n-cpu.txt.gz ?13:19
*** Longgeek_ has quit IRC13:20
*** ferest has quit IRC13:22
*** jecarey__ has joined #openstack-nova13:23
garykkaisers1: maybe kashyap knows13:24
ndipanovsdague, that's the same bug (different symptoms) that we keep hitting13:24
ndipanovsdague, TL; DR _get_instance_disk_info is the second most evil bit of code in nova13:24
kaisers1garyk: thanks for the hint!13:25
sdaguendipanov: ok, so it's clearly not solved then13:25
ndipanovnope13:25
ndipanovand it's not easy to solve13:25
sdaguewell, on the upside, I have a test for it13:25
bauzasndipanov: the first most evil bit of code being ? (sorry for digressing :D )13:25
ndipanovbauzas, objects.base.obj_to_primitive13:26
*** sdake_ has joined #openstack-nova13:26
bauzaslol I should have bet13:26
sdaguendipanov: so is someone working on that, or do we tell people they have to shut down their boot from volume servers manually before upgrade?13:26
ndipanovsdague, well if you can reprodce it 100% of the time then I'd say we have a problem13:27
ndipanovbut I don't think anyone is working on it tbh13:27
sdaguewell, it's blocking the grenade refactor13:27
*** jecarey_ has quit IRC13:27
sdagueI can run recheck and see if it's intermitent fail13:27
sdaguebut it did fail all 3 configs13:28
sdagueit fails in n-net and neutron cases13:29
*** jamie_h has joined #openstack-nova13:29
*** rasoto_ has left #openstack-nova13:29
sdagueit works when we don't upgrade n-cpu13:29
sdague(that part fails in a different place)13:29
*** sdake has quit IRC13:29
*** markvoelker has joined #openstack-nova13:30
sdaguethis is the patch in question - https://review.openstack.org/#/c/173791/13:30
*** rasoto_ has joined #openstack-nova13:30
*** tjones1 has joined #openstack-nova13:31
ndipanovsdague, OK I will try to look into it as soon as my hands are untied from the stuff I am on now13:31
ndipanovthere is for sure a dirty fix that we can apply13:31
*** tjones1 has left #openstack-nova13:31
*** xgsa has quit IRC13:32
ndipanovto make it not block you until this is resolved13:32
sdaguendipanov: yeh, so I can just not test this case13:35
sdaguebut it's 100% reproducable13:35
sdaguewhich means, everyone with boot from volume systems is probably boned13:36
ndipanovsdague, yep13:36
sdagueI just did a local reproduce13:36
Guest67672is this a regression/13:36
*** Guest67672 is now known as dansmith13:36
ndipanovwell13:36
sdaguedansmith: I don't know, we've never tested this before13:37
ndipanovit might be13:37
dansmithI'm asking ndipanov13:37
*** dansmith is now known as Guest1445313:37
ndipanovthat function should be skipping volumes13:37
ndipanovbut it does not and then stacktraces13:37
ndipanov(the main problem is that it parses libvirt xml for this info instead of asking nova)13:37
sdagueso, we have previous tests which attached volumes to servers13:37
sdaguethose pass13:37
sdaguebut non of them were boot from volume13:38
ndipanovsdague, aha13:38
sdaguewhich I put into this new test, because it seemed like a thing you'd care about13:38
sdagueI've spent 2 days fighting neutron here, so I didn't really understand this fail was in here because I was failing earlier with neutron config issues13:39
*** xgsa has joined #openstack-nova13:39
Guest14453so is this because conversion to the newer bdm format isn't complete?13:39
Guest14453that seemed to stall out in some odd places13:40
Guest14453dammit13:40
*** signed8bit has joined #openstack-nova13:40
ndipanovwell it's not really related13:40
ndipanovI think13:40
Guest14453well, you were blaming object primitive conversion, so I figured it was13:40
*** mwagner_lap has quit IRC13:40
*** irenab has quit IRC13:40
*** tjones1 has joined #openstack-nova13:41
ndipanovah no no - I said that's the most evil bit of code in nova, not that it was related13:41
ndipanov:)13:41
* Guest14453 rolls eyes13:41
Guest14453well anyway, not being able to restart nova-compute under any circumstance is more evil than anything, IMHO13:42
ndipanovsdague, is it possible to get instance xml from the reporudcer?13:42
Guest14453major operator frustration there13:42
*** rushil has joined #openstack-nova13:42
ndipanovsdague, even if you can pastebin it from your local reproduce scenario?13:42
*** Guest14453 is now known as dansmith_13:43
sdaguendipanov: sure, I've got a vm in that state, let's start a bug.13:43
sdagueThe reproduce is also reasonably simple13:43
sdagueif you have a vm to spare13:43
*** burt has joined #openstack-nova13:43
ankit8188alex_xu: hi13:43
*** irenab has joined #openstack-nova13:45
sdaguelet me file the bug, and get details there13:45
ndipanovsdague, ok13:45
bauzasjohnthetubaguy: alaski: we discussed about that briefly but here is an attempt to draw that out https://blueprints.launchpad.net/nova/+spec/shared-state-scheduler13:45
bauzasjohnthetubaguy: alaski: I made a few nodes in an etherpad https://etherpad.openstack.org/p/liberty-nova-scalable-scheduler13:46
*** ganso_ has joined #openstack-nova13:46
johnthetubaguybauzas: were you going to put that into spec form as well?13:46
bauzasjohnthetubaguy: for sure, it's just the first step13:46
bauzasjohnthetubaguy: I'm planning to POC it for L13:47
bauzasLiberty13:47
*** annegentle has quit IRC13:48
johnthetubaguybauzas: OK, interesting, we can have it as an experimental extra driver, if that helps move it forward13:48
alaskibauzas: thanks.  in meetings most of the morning but will take a good look after13:49
*** IanGovett has quit IRC13:49
*** tjones1 has quit IRC13:49
*** IanGovett has joined #openstack-nova13:51
sdaguendipanov: https://bugs.launchpad.net/nova/+bug/144502113:51
openstackLaunchpad bug 1445021 in OpenStack Compute (nova) "nova-compute does not start after upgrade from juno->kilo if there are boot from volume servers running" [Critical,New]13:51
sdaguendipanov: ok, what other info did you want?13:52
*** Sukhdev has joined #openstack-nova13:52
*** fawadkhaliq has quit IRC13:52
bauzasjohnthetubaguy: yeah that's the idea - deliver a new driver13:52
*** xgsa has quit IRC13:53
ndipanovsdague, libvirt.xml files from instances13:54
johnthetubaguyclaudiub: turns out those blueprints didn't need a spec, you just needed to target them for liberty, so I got that all unblocked for you now13:54
ndipanovsdague, it's in the logs nvm13:55
ndipanovhmmm or is it13:55
dansmith_it should be13:55
*** signed8bit has quit IRC13:56
*** jgrimm is now known as zz_jgrimm13:56
*** annashen has joined #openstack-nova13:56
*** zzzeek has joined #openstack-nova13:56
alex_xuankit8188: hi13:57
*** dave-mccowan has joined #openstack-nova13:58
ankit8188alex_xu: this is regarding https://review.openstack.org/#/c/171523/13:58
*** dansmith_ is now known as dansmith13:58
sdaguendipanov: somewhere in http://logs.openstack.org/91/173791/11/check/check-grenade-dsvm/fc725f5/logs/libvirt/qemu/13:58
lpetrutjohnthetubaguy: Hi, thanks for reapproving the Hyper-V retated blueprints!13:58
claudiubjohnthetubeguy: I see, thanks! :) I missed one more blueprint, which was approved for Kilo: https://blueprints.launchpad.net/nova/+spec/hyper-v-ovs-vif13:58
sdagueoh, we only have the qemu command lines13:58
ankit8188alex_xu: if we set soft_deleted=True when status=soft-delete, then we can get soft delted instances by status filter13:59
ndipanovsdague, that's command lines only13:59
ndipanovyeah13:59
dansmithsdague: should be in nova logs13:59
dansmithcompute13:59
sdaguedansmith: ah, yeh, on the old side13:59
ndipanovdansmith, if debug was on13:59
sdaguedebug is always on in the gate13:59
*** jecarey__ has quit IRC13:59
johnthetubaguylpetrut: claudiub: cool, I can approve those two, they just needed retargeting to liberty, so they got reviewed that way13:59
ankit8188but Eli Qiao had already proposed this change https://review.openstack.org/#/c/118641/113:59
ankit8188which is in abandoned state right now13:59
*** jecarey__ has joined #openstack-nova13:59
sdaguendipanov: http://logs.openstack.org/91/173791/11/check/check-grenade-dsvm/fc725f5/logs/old/screen-n-cpu.txt.gz#_2015-04-16_11_33_46_83414:00
ankit8188alex_xu: so in my opinion we can handle this issue separately by submitiing new patch set on https://review.openstack.org/#/c/118641/114:00
*** xgsa has joined #openstack-nova14:01
*** zhenguo has joined #openstack-nova14:01
johnthetubaguyabhishekk mikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes gilliard garyk edleafe dims moshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov its nova meeting time in #openstack-meeting14:02
ankit8188alex_xu: And your comment #1 can be addressed in my current patch https://review.openstack.org/#/c/171523/14:02
*** xyang1 has joined #openstack-nova14:02
ankit8188alex_xu: Please let me know your opinion14:03
ndipanovsdague, ofund it14:03
*** fawadkhaliq has joined #openstack-nova14:04
alex_xuankit8188: separated patch sounds ok fo rme14:04
*** kaisers1 has quit IRC14:04
sdake_quick bug report question (is this a bug?) I launch 30 vms with ports and floating isp14:05
sdake_http://paste.fedoraproject.org/211516/15494914/14:05
ankit8188alex_xu: There is already a patch submiited to fix your comment #2 https://review.openstack.org/#/c/118641/114:05
*** ferest has joined #openstack-nova14:05
sdake_some of the networks fail to attach14:05
alex_xuankit8188: but without Eli's patch, how do you fix in nova client?14:05
sdake_yet, within the vm, dhcp is retrieved and metadata is accessed properly14:06
alex_xuankit8188: Then need another patch after Eli's patch merged?14:06
ankit8188alex_xu: This patch is to fix delete api issue14:06
sdake_so its not like they totally fail to attach, just dispaly14:06
sdake_anyone seen anythinglike this?14:06
sdake_(neutron)14:06
*** Sukhdev has quit IRC14:06
ankit8188alex_xu: so it does not send status=soft_deleted from nova client14:06
ankit8188alex_xu: please refer nova client patch https://review.openstack.org/#/c/171524/14:07
ankit8188alex_xu: submitted against the same bug https://launchpad.net/bugs/143684914:08
openstackLaunchpad bug 1436849 in OpenStack Compute (nova) "Inconsistent delete API in case of soft-delete" [Low,In progress] - Assigned to Ankit Agrawal (ankitagrawal)14:08
*** bkopilov has joined #openstack-nova14:08
*** thomasem has joined #openstack-nova14:08
*** xgsa has quit IRC14:08
alex_xuankit8188: ok, will try to catch up it14:08
ankit8188alex_xu: ok, thank you !14:09
alex_xuankit8188: np14:09
sdake_dansmith could I borrow your vast knowledge of nova quickly to tell me if this is a known issue or not? ^14:09
*** ayoung has quit IRC14:09
*** ferest has quit IRC14:10
dansmithsdague: the nova meeting is going on right now, ask me in 50 minutes14:10
*** shaohe_feng has joined #openstack-nova14:10
*** haomaiwang has joined #openstack-nova14:11
*** HenryG_ is now known as HenryG14:11
*** dboik has quit IRC14:12
*** cfriesen has joined #openstack-nova14:13
*** dboik has joined #openstack-nova14:13
sdake_dansmith ack - this is sdake btw, :)  i'll ask then thanks14:14
dansmithsdague: sorry, tab fail14:14
dansmithdammit14:14
sdake_its ok :)14:14
dansmithsdake_: you need a new nick14:14
sdake_sdague neds a new nick :)14:14
sdake_I was here first14:14
sdaguesdake_: I'm not so sure about that :)14:15
openstackgerritDan Smith proposed openstack/nova: Fix test_swap_volume_api_usage()  https://review.openstack.org/17407314:15
openstackgerritDan Smith proposed openstack/nova: Fix test_attach_interface_failure()  https://review.openstack.org/17408014:15
openstackgerritDan Smith proposed openstack/nova: Fix test_set_admin_password_bad_state()  https://review.openstack.org/17408114:15
sdake_debateable sague :)14:15
*** haomaiwang has quit IRC14:15
sdake_anyway we are both not changing nicks so we have to live with the tab comletion :)14:15
*** mwagner_lap has joined #openstack-nova14:16
*** xgsa has joined #openstack-nova14:16
*** markvoelker_ has joined #openstack-nova14:16
*** dulek has quit IRC14:17
*** dulek has joined #openstack-nova14:18
*** annegentle has joined #openstack-nova14:18
*** markvoelker has quit IRC14:19
*** doron_afk is now known as doron14:20
*** thomasem has quit IRC14:21
*** thomasem has joined #openstack-nova14:21
*** emagana has joined #openstack-nova14:21
*** tonytan4ever has joined #openstack-nova14:23
*** markvoelker has joined #openstack-nova14:23
*** ajayaa has quit IRC14:24
openstackgerritLan Qi Song proposed openstack/nova: Vmware:Fix how VolumeOps is getting host information  https://review.openstack.org/13518214:24
*** ferest has joined #openstack-nova14:25
*** markvoelker_ has quit IRC14:26
*** mtanino has joined #openstack-nova14:27
*** thangp has joined #openstack-nova14:27
*** yingjun has joined #openstack-nova14:28
*** hdd has quit IRC14:28
*** mpaolino has quit IRC14:29
*** sdake has joined #openstack-nova14:29
*** ankit8188 has quit IRC14:29
*** andreykurilin1 has quit IRC14:30
*** andreykurilin has joined #openstack-nova14:30
*** eharney has joined #openstack-nova14:30
*** annashen has quit IRC14:32
*** doron is now known as doron_afk14:32
*** sdake_ has quit IRC14:33
*** markvoelker has quit IRC14:34
*** Hosam has joined #openstack-nova14:37
*** jaypipes has joined #openstack-nova14:39
rajeshtdansmith, alaski, jogo, mriedem: could you please review these patches.14:41
dansmithsdake: I don't know the answer to your question14:41
rajeshtdansmith, alaski, jogo, mriedem: https://review.openstack.org/#/c/122347/14:41
rajeshtdansmith, alaski, jogo, mriedem: https://review.openstack.org/#/c/135202/14:41
*** ajayaa has joined #openstack-nova14:41
dansmithsdake: assuming this is with some driver other than libvirt?14:41
sdakedansmith thanks for getting back to me14:41
andreykurilinFalks, can someone look at  http://lists.openstack.org/pipermail/openstack-dev/2015-April/061183.html and give some answers?14:41
rajeshtdansmith, alaski, jogo, mriedem: https://review.openstack.org/#/c/170118/14:42
sdakedansmith using neutron + libvirt + nova in containers14:42
rajeshtdansmith, alaski, jogo, mriedem: I missed to raise it in meeting.14:42
sdakeI found vif_plugging_timeout = 0 going to try that14:42
dansmithsdake: meaning it's set to zero?14:42
sdakeatm its set to 30014:42
dansmithsdake: that would potentially affect actual connection14:43
sdakei am setting it to zero14:43
sdakeask.openstack.org says thats a workaround to this problem14:43
dansmithsdake: don't set it to zero, else libvirt doesn't wait for neutron to finish anything14:43
*** garyk has quit IRC14:43
dansmithsdake: lol14:43
sdake;-)14:43
sdakethere is also ntohgin in teh logs indicating failure14:43
*** gszasz has quit IRC14:43
sdakei am using juno RDO14:44
dansmithsdake: setting that to zero means that nova just boots VMs without waiting for neutron to finish doing the vif plugging. So if that seems like it would help, then...sure :)14:44
sdakeand it is not easy to change to a newer version14:44
*** oro has quit IRC14:44
rodrigodshi! noob question: what happens if a user tries to create an instance in a project that nova doesn't have any information on its database? (but the project exists in keystone)14:44
sdaguesdake: so based on what I heard in the philly ops meeting, if you bulk throw that much at neutron, it's probably going to loose stuff.14:44
*** abhishekk has quit IRC14:44
sdakeis this a rate limiting patch in neutron ?14:45
sdakeor is it just busted?14:45
sdagueno, it's just races14:45
sdagueand deadlocks14:45
sdakesoftware should handle overload gracefully14:45
sdakeoh I see, bugs :)14:45
sdaguesure, but I'd say the event not getting back to nova probably is event not getting sent from neutron14:45
sdakeI wonder if this ones fixed14:46
dansmithsdake: sdague if the events weren't coming back, then there would be errors14:46
sdakesdague my debug shows neutron-server never gets the event from nova actually14:46
dansmithI think it's more likely something before that, like talking to neutron way before we even get to booting the vms14:46
dansmithsdake: right14:46
dansmithsdake: which is why making that change shouldn't do anything14:46
dansmithexcept maybe make things worse14:46
dansmithas nova will be less tolerant of neutron14:47
sdakedansmith the vms that don't have the vif plugging sent get their dhcp address and can retreive metadata from the metadata server14:47
*** IanGovett1 has joined #openstack-nova14:47
dansmiththis *could* be related to the info_cache update issue, but it could also be something else, but it's all before the vif plugging event I think14:47
jaypipesOK, I'm not opening any more chromium tabs until I finish reading/doing reviews on all my open ones. .. :(14:47
sdagueok, so back to the boot from volume bug...14:47
dansmithsdake: right, which is why the vif plugging event isn't the problem14:47
ndipanovsdague, I have a patch just writting the test14:48
ndipanovit's really just make sure we don't get these obvious fails14:48
dansmithsdake: if you disable that, the VMs will be allowed to boot earlier and will likely fail DHCP before neutron has hooked them up14:48
ndipanovdon't expect too much14:48
sdakedansmith i see14:48
*** hemnafk is now known as hemna14:48
sdakeneutron is definately setting up the dhcp/metadata14:49
sdakewhat happens next, the vif interface event is sent?14:49
*** IanGovett has quit IRC14:49
sdakebtw, its not a quota problem it happens with random numbers of vms14:49
*** nikhil_k is now known as flaper87-214:49
*** flaper87-2 is now known as nikhil_k14:50
bauzasjaypipes: morning14:50
dansmithsdake: right, we're saying we think that it's load before the event part14:50
jaypipesbauzas: marooning.14:50
dansmithsdake: i.e. when you do that boot request, nova makes a huge flurry of requests to neutron to prepare for networking all the instances14:50
bauzasjaypipes: thought you would be interested in reading https://etherpad.openstack.org/p/liberty-nova-summit-ideas14:50
*** smccully has joined #openstack-nova14:50
*** subscope has quit IRC14:50
bauzasjaypipes: nah, mornooning14:50
sdakeI am a competent developer, I can help debug and get er fixed but I need a pointer or two ;)14:50
dansmithsdake: then when the booting happens, the event system makes sure that happens at the proper rate14:50
bauzasjaypipes: I don't like Adam Levine14:50
*** atuvenie has quit IRC14:51
*** zz_jgrimm is now known as jgrimm14:51
bauzasjaypipes: https://etherpad.openstack.org/p/liberty-nova-summit-ideas L58 tbp14:51
sdakewhich event system dansmith?14:51
bauzasjaypipes: and https://blueprints.launchpad.net/nova/+spec/shared-state-scheduler also14:51
jaypipesbauzas: I need to close out some tabs before opening new ones... :)14:51
dansmithsdake: the nova-waits-to-boot-a-vm-until-neutron-sends-the-vif_plugged-event14:51
sdakeI am running on local node with 16 core xeon, I think I could set it unlimited14:51
dansmithsdake: the vif_plugging_timeout thing you were asking about14:51
bauzasjaypipes: lolol14:52
sdakedansmith the vms are all booting14:52
bauzasjaypipes: so just tease at the URL14:52
sdakeand all getting dhcp addresses14:52
dansmithsdake: yes, I get that :)14:52
sdakecool14:52
sdakejust wanted to verify14:52
sdakeso at that point, nova does what, sends a vif pugged ievent?14:53
pczesnondipanov: https://bugs.launchpad.net/nova/+bug/144504014:53
openstackLaunchpad bug 1445040 in OpenStack Compute (nova) "InstancePCIRequests.obj_from_db fails to get requests from db" [Undecided,New] - Assigned to Przemyslaw Czesnowicz (pczesno)14:53
ndipanovpczesno, awesome!!!14:54
sdakenova boots vm, tells neutron (get me some address), neutron tells nova the address, nova boots, nova waits for vm to boot, nova sends vif plugged event to neutron14:54
sdakeis this the flow14:54
ndipanovI mean that you found it14:54
sdake?14:54
dansmithsdake: no, it's something like:14:55
dansmithsdake: nova asks neutron to setup ports, nova starts the VM paused, nova waits for neutron to send vif_plugged, nova unpauses the VM14:56
*** nkrinner has quit IRC14:56
dansmithsdake: however, what I'm trying to say is I think that you problem is *way* before the "vm starts paused" part14:56
dansmithsdake: are you pre-creating the ports in neutron or making nova do it?14:56
sdaguendipanov: so, this might be a test env issue, but it's still probably good if we don't prevent nova-compute from starting14:56
sdakehmm heat template..14:57
sdakehttp://ur1.ca/k6rgi -> http://paste.fedoraproject.org/211924/1429196214:57
jaypipesrpodolyaka: heya, why do some lines in http://paste.openstack.org/show/166892/ begin with things like <180> or <182>?14:57
sdakei think the first, but I am not sure what it is actually doing14:57
jaypipesrpodolyaka: I'm looking into your fix for https://bugs.launchpad.net/nova/+bug/144076214:57
openstackLaunchpad bug 1440762 in OpenStack Compute (nova) "Rebuild an instance with attached volume fails" [High,In progress] - Assigned to Roman Podoliaka (rpodolyaka)14:57
jaypipesrpodolyaka: and curious why some log messages look different than others.14:57
*** Marga_ has joined #openstack-nova14:59
*** baoli has quit IRC15:01
*** baoli has joined #openstack-nova15:01
*** dboik_ has joined #openstack-nova15:02
*** yingjun has quit IRC15:02
*** otter768 has joined #openstack-nova15:03
*** annegentle has quit IRC15:04
*** markvoelker has joined #openstack-nova15:04
sdakedansmith would the port creation outside of nova cause problems?15:05
*** dboik has quit IRC15:05
*** marun_ has joined #openstack-nova15:05
dansmithsdake: port creation outside of nova yields higher success rates15:05
sdakeyou said the events are rate limited dansmith?15:06
*** zhenguo has quit IRC15:06
ndipanovsdague, why do you think it's a test env issue15:06
jaypipesdansmith, ndipanov: https://review.openstack.org/#/c/170907/ fixes a pretty important bug and is an easy win. just FYI.15:07
dansmithsdake: no, waiting for the events means that we boot instances at the rate at which neutron can wire then15:07
dansmith*them15:07
sdaguendipanov: well, it's partially one, maybe15:07
sdaguethe stop_cinder call is also stopping tgt15:08
*** Maike has quit IRC15:08
zzzeekheya, can someone explain to me why nova migrations regarding indexes have to assume that the index they are dealing with might not exist on a downgrade, and additionally why I am actually observing this within the CI servers, even though I cannot reproduce this issue locally ?15:08
sdaguehowever... until we attempt to restart nova-compute, the guest is still up15:08
sdagueand I can still read files out of the volume15:08
sdagueso... idk15:08
*** otter768 has quit IRC15:08
jaypipeszzzeek: why would a migration downgrade need to assume the index would not exist?15:09
*** ajayaa has quit IRC15:09
jaypipeszzzeek: unless of course the migration in question was *removing* an index...15:09
zzzeekjaypipes: take a look at 248_add_expire_reservations_index.py as well as 277_add_fixed_ip_updated_index.py15:09
zzzeekjaypipes: yes removing it15:09
zzzeekjaypipes: as well as my own build that is failing https://review.openstack.org/#/c/154243/515:09
sdakedansmith why do you think the problem si before the vm starts paused event if all the vms are wired?15:09
jaypipesdamn it. new tab in chromium. suckered by zzzeek ;)15:10
*** armax has joined #openstack-nova15:10
dansmithsdake: because that's when we record the actual information from neutron, which is displayed in the column you're seeing as blank15:10
zzzeekjaypipes: seems like i have to add a “well if the index doesnt exist then dont drop it!” but i have no idea what is making that happen15:10
zzzeekjaypipes: altuhoguh I guess it has to do with my taking over a placeholder?15:10
ndipanovsdague, right but that bug will happen in the following scenario - for whatever reason volumes are down15:10
sdakedansmith thanks for the bone i'll debug from that15:10
zzzeekjaypipes: oh.    script runs the old version of the code up to rev 290 which is all placeholders15:11
ndipanovand the periodic task kicks in to gather resources15:11
sdakehappen to have a nova line 3?15:11
sdakeline 3?15:11
sdakeline #?15:11
*** vivekd has quit IRC15:11
zzzeekjaypipes: geez.   how TF do we use these silly placeholder things?     I have a new migration.  i need to backport to kilo and juno.  where do i put it?15:11
*** markvoelker has quit IRC15:11
ndipanovthat is becuase _get_isntance_disk_info guesses wrong that something is a local disk when it's a volume15:11
dansmithzzzeek: first empty placeholder15:11
zzzeekdansmith: OK but my build is failng b.c. i think it runs the migration in its placeholder form and then compares15:11
sdakesounds like neutron is sending the wrong event information to nova-compute or whatever records that info15:11
zzzeekdansmith: https://review.openstack.org/#/c/154243/515:12
sdaguezzzeek: in master you put it *after* the place holders15:12
sdaguethe place holders are for stable15:12
zzzeeksdague: OK so in master, I put it at 291, in backport, I put it at 281?15:12
dansmithright, he said backport to kilo15:12
sdaguedansmith: this review is master15:12
dansmithhadn't looked yet15:12
dansmithzzzeek: yeah15:12
*** rushiagr is now known as rushiagr_away15:12
*** rushil has quit IRC15:12
zzzeekdansmith: but then what happens when someone upgrafes from kilo to liberty and it hits 291, which has already happened in their 281 ?15:13
dansmithzzzeek: has to be idempotent15:13
zzzeekdansmith: ah!  hence the “all our migrations need to quqery the DB to see if we’ve happened yet"15:13
zzzeekdansmith: wow, what a crapshow !15:13
*** atuvenie has joined #openstack-nova15:13
dansmithyes15:13
dansmithzzzeek: thanks!15:13
dansmithzzzeek: got better ideas?15:13
zzzeekdansmith: yes its called alembic15:13
*** lpetrut has left #openstack-nova15:14
zzzeekdansmith: however the online schema migration idea might work also15:14
dansmithdoes that actually help this case?15:14
jaypipeszzzeek: still looking, bear with me.15:14
dansmithI thought that's why jerdfeldt is doing his thing..15:14
zzzeekdansmith: yes because you can string together migrations into dependency trees15:14
sdaguezzzeek: so, indexes are a special case15:15
*** rushil has joined #openstack-nova15:15
zzzeekdansmith: yes it is but he leapfrogged my new alembic features that allow branching and merging15:15
sdaguebecause sites might have debugged a performance issue and applied local indexes15:15
openstackgerritSudipta Biswas proposed openstack/nova: Allow discrete blocks of online pCPUs for pinning  https://review.openstack.org/17318715:15
dansmithsdague: +115:15
sdagueand we don't want to explode if they did that15:15
zzzeeksdague: does that mean they removed nova builtin indexes?  this is a better reason15:15
*** rushil has quit IRC15:16
zzzeeksdague: though ideally you’d ahve migration directives that do this without hardcoding in each migration file as I see here15:16
openstackgerritSudipta Biswas proposed openstack/nova: Allow discrete blocks of online pCPUs for pinning  https://review.openstack.org/17318715:16
sdaguezzzeek: downgrade is all going away anyway15:16
sdakedansmith would you happen to have a github link to where the event is delivered to be recorded by the db?15:16
zzzeeksdague: that too :)15:16
sdake(in nova-neutron integration)15:16
dansmithsdake: it's not an event for that info, it's where nova makes blocking calls to neutron for that stuff, and it's spread all over15:17
sdaguezzzeek: yes, ideally, you would. And I think we did onc.15:17
dansmithsdake: neutron does send us a network-changed event at times, but it just prompts us to re-poll15:17
*** jcoufal has joined #openstack-nova15:17
jaypipessdague: still doesn't explain (IMO) why we're seeing "Can't DROP 'virtual_interfaces_uuid_idx'; check that column/key exists")" in this particular case, though, right?15:18
jaypipessdague: I mean, the very same migration does add that index...15:18
*** kmartin has joined #openstack-nova15:18
sdaguejaypipes: I have no idea what turbo hipster tries to do here15:19
zzzeekjaypipes: I think this script installs nova *before* my code updates15:19
zzzeekjaypipes: so it runs 281->290 placeholder migrations15:19
zzzeekjaypipes: so it thinks it ran 281, but it was actualyl blank15:19
zzzeekjaypipes: that is my theory anyway15:19
dansmiththe placeholders are in master,15:19
dansmithnot kilo15:19
dansmithso they shouldn't have been applied yet15:19
dansmiththat's the point15:19
sdaguedansmith: the patch is master15:20
dansmithI know, he's talking about having applied the placeholders already before we lay down the new commit,r giht?15:20
ndipanovsdague, so ...15:20
ndipanovafter looking into this a bit more15:20
sdakedansmith is it nova-compute that does this work or some other process?15:20
jaypipeszzzeek: no, that's definitely not the case (the placeholders being applied before your code)15:21
sdakethe blocking neutron calls15:21
dansmithsdake: nova-api, nova-conductor probably, and nova-compute some too.. like I said, it's all over15:21
*** browne has joined #openstack-nova15:21
ndipanova nice fix will take a bit more time...15:21
ndipanovbut it's actually not too hard15:21
dansmithsdague: what jaypipes said :)15:21
ndipanov(I think lol)15:21
ndipanovI need to cotext switch for a bit15:22
ndipanovbut will post it later today15:22
sdaguendipanov: ok, well a fix would be nice, though I might end up working around it15:22
jaypipeszzzeek: though, let me double check... going through the logs therte.15:22
ndipanovsdague, how?15:22
ndipanovby not bouncing iscsi target daemon?15:22
sdagueyeh15:22
zzzeekso sdague , just triple chekcing - in master I totally call this 291, dont touch placeholders, and kilo it will be 281 again not touching placeholder.  meaning we never *remove* placeholder files in a commit, right?15:22
sdaguezzzeek: yep15:23
dansmithzzzeek: you replace the placeholder in kilo15:23
ndipanovsdague, fair enough but it really is not a huge deal15:23
zzzeeksdague: OK I read the email taht’s referred to in the placeholder and it didnt really make it clear what I was doing, thanks!15:23
sdakedansmith interesting, I would not have expected nova-api and nova-conductor to pause and unpause vms based upon neutron events :)15:23
*** jecarey_ has joined #openstack-nova15:23
dansmithzzzeek: meaning, you call the placeholder the same number15:23
ndipanovso if there are consequences to that that will make the test less robust/realistic15:23
zzzeekdansmith: so kilo *has* the placeholder, OK15:23
*** shaohe_feng has quit IRC15:24
sdakedansmith seems like compute should do that in one go15:24
dansmithzzzeek: so it has something in that slot, yeah15:24
ndipanovI'd say gimme some time - it would be awesome to confirm that this was the issue15:24
dansmithsdake: they don't15:24
*** alexpilotti has joined #openstack-nova15:24
sdaguendipanov: yeh, so I'll keep a way to test for this15:24
dansmithsdake: but nova-conductor and nova-api talk to neutron ahead of time to setup and gather the networking info15:24
*** jecarey__ has quit IRC15:25
sdakedansmith and they store in the db?15:25
dansmithsdake: yes15:25
sdakeinteresting15:25
*** jecarey has joined #openstack-nova15:25
dansmithsdake: and it gets passed over rpc15:25
sdakeyup get that part, so the nova-api or nova-conductor is storing junk data15:25
*** ZZelle has joined #openstack-nova15:25
sdakeyet passing good data over rpc15:26
openstackgerritMichael Bayer proposed openstack/nova: Add an index to virtual_interfaces.uuid  https://review.openstack.org/15424315:26
jaypipeszzzeek: hmm, honestly, I'm not sure why turbo-hipster is showing this index as not existing, but precedence apparently is to check for the existence of the index :( https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/248_add_expire_reservations_index.py15:26
dansmithsdake: that info gets refreshed from neutron at times, so you can't say that definitively15:26
zzzeekjaypipes: yup that was the original thingy :)15:26
dansmithsdake: entirely possible that it is good, then empty, then good again15:26
sdakeright - I have seen that  refresh in practice15:26
jaypipeszzzeek: and I see you just pushed a similar patch :)15:26
dansmithsdake: and we see races like that too often15:26
zzzeekSo i can see why online schema migrations happened.   if this is the current process :)15:26
zzzeekjaypipes: yes15:26
*** atuvenie has quit IRC15:27
jaypipeszzzeek: +2 from me.15:27
zzzeekwith alembic branches, we could have individual roots for “icehouse”, “juno”, “kilo”, etc. and establish dependencies between them15:28
sdakedansmith so another possibility is the refresh is busted15:28
dansmithzzzeek: fwiw, we almost never backport migrations15:28
zzzeekso if you had a migration that was for juno, you could add it to the “juno” root in master, and backport all the way through without change15:28
sdakei find it hard to believe a db store followed by rpc message send could cause problems15:28
zzzeekdansmith: OK weill this is an RH issue they want for A4.  should I backport only in the RH build ?15:28
dansmithzzzeek: these placeholders are here for emergencies15:28
*** atuvenie has joined #openstack-nova15:29
sdakedansmith i'll debug and get back with a patch if I find the problem :)15:29
*** jecarey_ has quit IRC15:29
*** annashen has joined #openstack-nova15:29
dansmithzzzeek: that's an option.. TBH, I'm not sure the backport to stable is legit upstream or not.. depends on the impact I guess15:29
dansmithzzzeek: https://wiki.openstack.org/wiki/StableBranch#Appropriate_Fixes15:30
zzzeekdansmith: Can I CC you on the BZ and you tell me where to put it?   or am i being too lazy15:30
jaypipeszzzeek: is that A4 letter size? :P15:30
dansmithzzzeek: see the "absolutely no db schema changes" bit :)15:30
zzzeekjaypipes: i’m leaking over into employer-speak :)15:30
jaypipesheh, no worries :)15:30
dansmithzzzeek: you should run it past one of the stable dudes like adam_g15:30
dansmithzzzeek: if they're okay with it, then might as well put it into stable first15:31
*** russellb has quit IRC15:32
*** ozialien has joined #openstack-nova15:32
dansmithsdague: I think I'm going to need to make grenade run the nova manage command to do the flavor porting15:32
dansmithsdague: we'll see when I get this dummy migration up15:32
sdaguedansmith: ok15:32
*** jodah has joined #openstack-nova15:32
sdaguewe haven't branched stable/kilo yet, because I'm trying to get this refactor finished15:33
sdagueso it might need to wait a little15:33
dansmithokay15:33
zzzeekdansmith: when you say “stable” you mean stable/kilo right15:33
*** tsekiyama has joined #openstack-nova15:34
dansmithzzzeek: stable/* yes15:34
*** Nikolay_St has quit IRC15:35
zzzeekdansmith: OK well ill wait for master here to be merged then I’ll propose kilo/juno backports and link adam_g on those gerrits, sound like a plan ?15:35
*** russellb has joined #openstack-nova15:35
*** romainh has left #openstack-nova15:36
dansmithzzzeek: yes15:36
sdakedansmith nova-api creates the vif-plugged event15:36
*** dave-mccowan has quit IRC15:37
sdakedansmith does that happen prior to unpause?15:37
dansmithsdake: it creates it in response to neutron sending it.. it's the thing that neutron sends us to unpause15:38
*** ozialien has left #openstack-nova15:38
*** Marga_ has quit IRC15:39
*** dsanders has joined #openstack-nova15:39
*** angdraug has joined #openstack-nova15:40
*** Nikolay_St has joined #openstack-nova15:40
*** sudipto has quit IRC15:41
*** jcoufal has quit IRC15:43
*** TobiasE has quit IRC15:44
openstackgerritVladik Romanovsky proposed openstack/nova: tests: use numa xml automatic generation in libvirt tests  https://review.openstack.org/17244815:44
openstackgerritVladik Romanovsky proposed openstack/nova: tests: introduce a NUMAServersTest class  https://review.openstack.org/15500615:44
*** hdd has joined #openstack-nova15:45
*** READ10 has joined #openstack-nova15:45
vladikrjaypipes, Hi, any chance you could look at  https://review.openstack.org/172448 again, dansmith comment is at the bottom15:46
*** annegentle has joined #openstack-nova15:46
dansmithjaypipes: hopefully that is still in your tab stack from when I asked, so not a new tab :P15:46
jaypipesdansmith: actually, it is :)15:47
jaypipesso, yes, vladikr, I will shortly. :)15:47
vladikrjaypipes, thanks :)15:47
*** Murali has quit IRC15:47
*** tojuvone has quit IRC15:48
*** busterswt has joined #openstack-nova15:49
*** dsanders has quit IRC15:50
*** Nikolay_St has quit IRC15:50
openstackgerritQin Zhao proposed openstack/nova: Use host AZ instead of instance AZ in scheduler request_spec  https://review.openstack.org/17426915:52
*** lpetrut has joined #openstack-nova15:52
jaypipessdague: hey, do you have handy a link to some page that explains why HTTP 501 Not Implemented should *not* be used by a REST API server to indicate which parts of the API are implemented by the server?15:54
*** tonytan4ever has quit IRC15:54
*** dave-mccowan has joined #openstack-nova15:55
sdaguejaypipes: http://www.w3.org/Protocols/rfc2616/rfc2616.txt - section 5.1.1 defines what METHOD means in the HTTP context15:56
jaypipessdague: ty sir.15:56
sdagueyeh, actually, I think that section explains my whole position15:56
jaypipessdague: you recommend instead a 404 or 400 yes?15:56
sdaguejaypipes: yes15:56
sdague40x15:56
*** mpavone has quit IRC15:56
sdagueI'm not sure I have an opinion beyond that15:57
sdaguebut 501 is kind of wrong15:57
*** russellb has quit IRC15:57
*** garyk has joined #openstack-nova15:59
*** jlanoux has quit IRC16:00
*** russellb has joined #openstack-nova16:01
*** Sukhdev has joined #openstack-nova16:02
*** ferest has quit IRC16:03
*** doron_afk has quit IRC16:03
*** ferest has joined #openstack-nova16:03
*** emagana has quit IRC16:05
rodrigodshi! can someone clarify a doubt? what happens if a user tries to create an instance in a project that nova doesn't have any information on its database? (but the project exists in keystone)16:05
jogorajesht: the patches are too late for Kilo at this point (although maybe we can get them in for RC2)16:05
*** pask81 has joined #openstack-nova16:05
*** Nikolay_St has joined #openstack-nova16:06
*** annashen has quit IRC16:06
*** Marga_ has joined #openstack-nova16:07
*** markvoelker has joined #openstack-nova16:08
*** ferest has quit IRC16:08
*** jistr has quit IRC16:08
*** ociuhandu has quit IRC16:08
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004516:09
*** afazekas has quit IRC16:11
*** irenab has quit IRC16:12
*** emagana has joined #openstack-nova16:13
*** markvoelker has quit IRC16:13
*** hdd has quit IRC16:14
*** ociuhandu has joined #openstack-nova16:14
*** marun_ is now known as marun16:14
*** rushiagr_away is now known as rushiagr16:15
*** rasoto_ has quit IRC16:15
*** annegentle has quit IRC16:15
*** Marga_ has quit IRC16:16
openstackgerritPrzemyslaw Czesnowicz proposed openstack/nova: Use list of requests in InstancePCIRequests.obj_from_db.  https://review.openstack.org/17444816:17
*** asselin has joined #openstack-nova16:18
jaypipesfinally below 500 unread emails..16:18
*** baoli has quit IRC16:20
pczesnodansmith: do You have a minute?16:20
*** pkoniszewski has quit IRC16:21
*** cbader has quit IRC16:22
*** baoli has joined #openstack-nova16:24
*** baoli has quit IRC16:24
ndipanovpczesno, can I maybe help?16:24
pczesnondipanov: great16:24
pczesnondipanov: so i fixed the issue we talked about earlier and it uncovered another issue16:25
ndipanovlooool16:25
pczesnohttps://github.com/openstack/nova/blob/d1fb8d0fbdd6cb95c43b02f754409f1c728e8cd0/nova/scheduler/host_manager.py#L27116:25
ndipanovyes?16:25
pczesnondipanov: the code here (and code in pci/stats.py) expects that pci_requests is an object16:26
ndipanovpczesno, yes16:26
pczesnobut it's not it's a dict inside the instance dict16:26
ndipanovnope16:26
ndipanovit's a list of objects inside a dict inside an instance dict16:26
*** Nikolay_St has quit IRC16:27
ndipanovbecause of the buggyness of obj_to_primitive16:27
*** baoli has joined #openstack-nova16:27
pczesnondipanov: it's a list of dicts inside a dict inside an instance dict16:27
ndipanovpczesno, nope16:27
*** Nikolay_St has joined #openstack-nova16:27
ndipanovwell let me check16:27
pczesnobut until i fixed the other issue the list was always empty16:27
ndipanovbut16:27
pczesno u'pci_requests': {u'instance_uuid': u'0b30a4e6-a56c-4a67-b891-f70f384e430e', u'requests': [{u'count': 1, u'request_id': u'ae67f481-344d-44f6-bd42-44a206bbb369', u'alias_name': None, u'spec': [{u'physical_network': u'default'}], u'is_new': False}]}16:28
pczesnothat's from instance dict16:28
*** krtaylor has quit IRC16:28
*** annashen has joined #openstack-nova16:28
ndipanovpczesno, that does not seem right16:30
*** mriedem has quit IRC16:30
pczesnondipanov: well, for sure it doesn't work right now16:31
ndipanovpczesno, can you post your fix16:31
pczesnohttps://review.openstack.org/17444816:31
pczesnondipanov: i don't understand why pci_requests wouldn't be a dict16:32
*** Marga_ has joined #openstack-nova16:32
ndipanovbecause obj_to_primitive called in build_request_spec16:32
*** annegentle has joined #openstack-nova16:32
*** mriedem has joined #openstack-nova16:32
*** pask81 has quit IRC16:33
ndipanovif you have a field that is a list of objects on an object that is NOT a subclass of ObjectListBase16:33
ndipanovobj_to_primitive will leave it as a list of objects16:33
pczesnondipanov: yeah, i know but it seems to be working anyway and converts it to dict16:33
ndipanovpczesno, well let me try to figure out why16:34
*** dulek has quit IRC16:34
pczesnondipanov: ok, thanks16:34
openstackgerritJoe Gordon proposed openstack/nova: Replace BareMetalDriver with IronicDriver in option help string  https://review.openstack.org/16370016:35
*** c0m0 has quit IRC16:35
*** Nikolay_St has quit IRC16:36
*** jecarey has quit IRC16:37
openstackgerritJoe Gordon proposed openstack/nova-specs: process: add new section History  https://review.openstack.org/17205816:37
ndipanovpczesno, I know why16:38
*** markus_z has quit IRC16:38
ndipanovjsonutils.to_primitive16:39
*** derekh has quit IRC16:39
ndipanovbut that's been there for a very long time16:39
ndipanovnot sure how was this ever working16:39
*** EmilienM is now known as EmilienM|afk16:39
pczesnoit was working because the requests in pci_requests where always empty list until i did the fix16:40
pczesnondipanov: where do you see the  jsonutils.to_primitive16:42
pczesno?16:42
sdakedansmith are notifications guaranteed delivery?16:43
sdakeoslo.notifications that is16:43
dansmithsdake: no idea16:43
*** belmoreira has quit IRC16:43
*** claudiub has quit IRC16:43
*** patrickeast has joined #openstack-nova16:44
ndipanovpczesno, last line of build_request_spec16:45
*** andreykurilin__ has joined #openstack-nova16:45
*** atuvenie has quit IRC16:47
*** matrohon has quit IRC16:48
*** amotoki_ has quit IRC16:50
dansmithmriedem: in your nullinstanceuuidscan test,16:51
dansmithmriedem: you do a downgrade() in setUp().. what does that dowgrade to?16:52
*** rajesht has quit IRC16:52
pczesnondipanov: i've no clue what the proper solution is, should it be fixed in consume_from_instance, in pci/stats, or should actually a list of objects be passed in request_spec16:53
*** krtaylor has joined #openstack-nova16:53
ndipanovpczesno, well definitely fixed in consume from instance16:54
*** annashen has quit IRC16:54
*** russellb has quit IRC16:55
dansmithmriedem: oh, I see16:56
*** emagana has quit IRC16:56
pczesnondipanov: ok , thanks again16:57
*** cfriesen has quit IRC16:58
*** harlowja_away is now known as harlowja16:58
*** andreykurilin__ has quit IRC16:58
*** mtanino has quit IRC16:59
*** russellb has joined #openstack-nova17:00
*** IanGovett1 has quit IRC17:00
*** annashen has joined #openstack-nova17:01
*** Marga__ has joined #openstack-nova17:02
*** Marga_ has quit IRC17:02
*** IanGovett has joined #openstack-nova17:02
*** Sukhdev has quit IRC17:02
*** unicell has joined #openstack-nova17:03
*** dsanders has joined #openstack-nova17:03
jaypipesooh I can finally see favicons in my tabs.17:04
*** otter768 has joined #openstack-nova17:04
*** MVenesio has joined #openstack-nova17:04
*** sahid has quit IRC17:05
*** emagana has joined #openstack-nova17:06
*** apuimedo has quit IRC17:07
*** sbfox has joined #openstack-nova17:07
*** BobBall is now known as BobBall_AWOL17:08
*** otter768 has quit IRC17:09
*** annegentle has quit IRC17:09
*** cfriesen has joined #openstack-nova17:11
*** ajayaa has joined #openstack-nova17:12
*** markvoelker has joined #openstack-nova17:12
*** annegentle has joined #openstack-nova17:14
*** lucasagomes is now known as lucas-dinner17:15
*** dboik_ has quit IRC17:18
*** achanda has joined #openstack-nova17:18
*** dboik has joined #openstack-nova17:19
*** IanGovett1 has joined #openstack-nova17:19
*** foexle has quit IRC17:20
*** IanGovett has quit IRC17:21
*** neelashah has quit IRC17:23
*** IanGovett has joined #openstack-nova17:23
*** IanGovett1 has quit IRC17:24
*** Maike has joined #openstack-nova17:25
*** tonytan4ever has joined #openstack-nova17:25
mriedemdansmith: got it? was at lunch.17:27
dansmithyeah17:27
*** Marga__ has quit IRC17:30
*** bkopilov has quit IRC17:31
*** angdraug has quit IRC17:31
*** jecarey has joined #openstack-nova17:32
*** bkopilov has joined #openstack-nova17:32
*** Nic has joined #openstack-nova17:33
openstackgerritDan Smith proposed openstack/nova: Wedge DB migrations if flavor migrations are not complete  https://review.openstack.org/17448017:35
dansmithwedge-o-matic ^17:35
*** bkopilov has quit IRC17:37
*** neelashah has joined #openstack-nova17:37
*** dboik_ has joined #openstack-nova17:37
*** Marga_ has joined #openstack-nova17:38
*** browne has quit IRC17:39
*** bkopilov has joined #openstack-nova17:40
*** dboik has quit IRC17:41
*** markvoelker has quit IRC17:43
*** EmilienM|afk is now known as EmilienM17:43
*** ociuhandu has quit IRC17:45
*** jecarey has quit IRC17:45
*** yamahata has joined #openstack-nova17:45
*** bkopilov has quit IRC17:45
*** bkopilov has joined #openstack-nova17:46
*** alexpilotti has quit IRC17:48
*** dave-mccowan has quit IRC17:50
*** vipul has quit IRC17:50
*** jodah has quit IRC17:50
dansmithdoh17:52
*** igordcard_ has joined #openstack-nova17:52
openstackgerritDan Smith proposed openstack/nova: Wedge DB migrations if flavor migrations are not complete  https://review.openstack.org/17448017:52
*** fawadkhaliq has quit IRC17:53
*** bkopilov has quit IRC17:54
*** EmilienM is now known as EmilienM|afk17:55
*** isd has joined #openstack-nova17:57
*** aix has quit IRC17:57
*** fawadkhaliq has joined #openstack-nova17:57
*** sdake_ has joined #openstack-nova17:59
*** vipul has joined #openstack-nova17:59
*** jodah has joined #openstack-nova18:00
*** jaypipes has quit IRC18:01
*** jecarey has joined #openstack-nova18:01
*** jaypipes has joined #openstack-nova18:01
*** Marga_ has quit IRC18:02
*** sdake has quit IRC18:02
*** Maike has quit IRC18:03
*** ferest has joined #openstack-nova18:03
*** ijw has joined #openstack-nova18:06
*** Marga_ has joined #openstack-nova18:06
*** jichen has joined #openstack-nova18:08
*** ferest has quit IRC18:08
*** markmcclain has quit IRC18:09
openstackgerritMatt Riedemann proposed openstack/nova: Fix order of args from compute v4 proxy change_instance_metadata  https://review.openstack.org/17452418:11
mriedemdansmith: dims__: ^18:11
mriedemalaski: ^18:11
*** fawadkhaliq has quit IRC18:11
*** johnthetubaguy is now known as zz_johnthetubagu18:11
* dansmith hangs his head in shame whilst reviewing18:12
openstackgerritMatt Riedemann proposed openstack/nova: Fix order of args from compute v4 proxy change_instance_metadata  https://review.openstack.org/17452418:12
mriedemupdated the commit message typo18:12
*** dulek has joined #openstack-nova18:12
alaskiso much craziness in arg ordering18:12
*** moshele has quit IRC18:12
dansmithI looked over that file once since all these issues and didn't catch this18:12
dansmiththe alternative if we continue doing it this way would be to just dispatch out of the proxy with everything by keyword18:13
dansmithand then it wouldn't matter18:13
dansmithinstance=instance, diff=diff, etc18:13
dansmithor18:13
openstackgerritDoug Hellmann proposed openstack/python-novaclient: Uncap library requirements for liberty  https://review.openstack.org/17454118:13
dansmithOR18:13
dansmithI could suck less.18:13
*** sdake has joined #openstack-nova18:13
alaski+1 on keywords18:15
dansmithso,18:15
dansmithmriedem could just change his patch to be *that*18:16
*** andreykurilin__ has joined #openstack-nova18:17
mriedemyou want me to use kwargs now?18:17
*** sdake_ has quit IRC18:17
mriedembut but i've already changed topic branches18:17
mriedemnvm18:18
dansmithjust saying, it's an option18:18
*** baoli has quit IRC18:18
*** achanda has quit IRC18:20
*** Maike has joined #openstack-nova18:22
*** browne has joined #openstack-nova18:23
*** Sukhdev has joined #openstack-nova18:24
*** wwwBUKOLAYcom has joined #openstack-nova18:26
*** neelashah has quit IRC18:27
*** vilobhmm1 has joined #openstack-nova18:27
*** achanda has joined #openstack-nova18:28
*** vilobhmm11 has joined #openstack-nova18:29
*** vilobhmm1 has quit IRC18:32
*** ArxCruz has joined #openstack-nova18:33
*** neelashah has joined #openstack-nova18:33
*** vladik has joined #openstack-nova18:33
*** EmilienM|afk is now known as EmilienM18:33
*** rdo has quit IRC18:34
*** dprince has quit IRC18:35
*** fawadkhaliq has joined #openstack-nova18:36
*** rdo has joined #openstack-nova18:36
*** dboik_ has quit IRC18:37
*** dboik has joined #openstack-nova18:39
openstackgerritPrzemyslaw Czesnowicz proposed openstack/nova: Fix handling of pci_requests in consume_from_instance.  https://review.openstack.org/17339418:41
openstackgerritPrzemyslaw Czesnowicz proposed openstack/nova: Use list of requests in InstancePCIRequests.obj_from_db.  https://review.openstack.org/17444818:41
*** yamahata has quit IRC18:42
*** markvoelker has joined #openstack-nova18:42
*** ArxCruz has quit IRC18:44
*** ArxCruz has joined #openstack-nova18:45
*** rajesht has joined #openstack-nova18:48
*** rdo has quit IRC18:49
*** Marga_ has quit IRC18:49
*** Marga_ has joined #openstack-nova18:49
*** Maike has quit IRC18:50
*** rdo has joined #openstack-nova18:51
*** achanda has quit IRC18:52
*** rajesht has quit IRC18:52
*** Marga_ has quit IRC18:54
*** Marga_ has joined #openstack-nova18:54
dansmithhmm is t-h broken?18:54
*** doude has quit IRC18:54
*** doude_ has joined #openstack-nova18:54
*** sudipto has joined #openstack-nova18:55
*** ArxCruz has quit IRC18:58
*** Hosam has quit IRC18:58
vilobhmm11want to know at this point https://github.com/openstack/nova/blob/master/nova/service.py#L208 is the webserver started ? so that in case the join() throws some exception I can return something like "raise exc.HTTPServiceUnavailable(explanation=msg)"18:59
*** ozialien has joined #openstack-nova18:59
dansmithvilobhmm11: you shouldn't return that error there ever19:00
dansmithvilobhmm11: because that class is used by more things that *aren't* http servers than those that are19:00
vilobhmm11dansmith : ok what is the preferred way…19:01
dansmithvilobhmm11: raise a generic error19:01
*** thomasem has quit IRC19:01
*** ijw has quit IRC19:01
vilobhmm11ok19:01
vilobhmm11so what i was thinking might be wrong please see here https://gist.github.com/vilobhmm/2255592994b14f3b7cfd19:01
*** tonytan4ever has quit IRC19:01
vilobhmm11will raise generic error than raising specific ones19:02
*** tonytan4ever has joined #openstack-nova19:02
vilobhmm11thanks dansmith19:02
dansmithHTTPBadRequest is not a generic error19:02
dansmithand raising it from service is not okay, IMHO19:03
dansmithalso expect that catching tooz-related things in nova/service.py is not right either, but I'd have to see the whole patch19:03
*** cbader has joined #openstack-nova19:04
vilobhmm11dansmith : in this patch on line #48 https://review.openstack.org/#/c/172502/2/nova/servicegroup/drivers/tz.py19:04
*** thomasem has joined #openstack-nova19:05
vilobhmm11i would modify it as https://gist.github.com/vilobhmm/2255592994b14f3b7cfd#file-gistfile1-txt-L2819:05
*** otter768 has joined #openstack-nova19:05
vilobhmm11to raise "raise exception.ToozBackendConnectionError()"19:05
dansmiththe servicegroup stuff needs to be generic,19:06
vilobhmm11if this exception is caught in nova/service.py https://gist.github.com/vilobhmm/2255592994b14f3b7cfd#file-gistfile1-txt-L619:06
dansmithwhich means it shouldn't be raising tooz-specific exceptions, and thus nova/service.py shouldn't have to catch them either19:06
vilobhmm11raise "raise exc.HTTPServiceUnavailable(explanation=msg)" https://gist.github.com/vilobhmm/2255592994b14f3b7cfd#file-gistfile1-txt-L819:06
*** fawadkhaliq has quit IRC19:06
*** sbfox has quit IRC19:07
*** jamie_h has quit IRC19:08
vilobhmm11dansmith : do you mean to say we should handle tooz specific exceptions at servicegroup layer itself ?19:08
vilobhmm11but then how will the end client be notified of the error ?19:08
*** rushiagr is now known as rushiagr_away19:08
dansmithvilobhmm11: I mean you should handle them in the tooz servicegroup driver and raise a generic exception to be caught in nova/service19:09
dansmithi.e.19:09
dansmithexcept ToozBrokeSomething: raise exception.ServiceGroupSomethingBroke()19:09
vilobhmm11ok19:10
*** otter768 has quit IRC19:10
vilobhmm11I get that but when you except exception.ServiceGroupSomethingBroke: at nova/service.py what error you raise to the client ? that was the whole question19:11
vilobhmm11dansmith : ^^19:11
sudiptobauzas, are you very strict about condensing the exception block?19:11
vilobhmm11i was raising raise exc.HTTPServiceUnavailable which you said is improper at this place19:12
dansmithvilobhmm11: correct19:12
*** achanda has joined #openstack-nova19:12
dansmithvilobhmm11: raising anything in that method isn't going to go back to any "client" anyway, right?19:12
vilobhmm11true19:13
vilobhmm11not completely sure though as i am still figuring out how things are constructed and destructed in this code path19:13
dansmiththat method is called when you start the service from the command line19:14
vilobhmm11ok19:14
dansmithletting something raise out of there such that we get a stack trace on the command line is also not reasonable19:14
dansmithnow you see why raising HTTP*Error is very wrong :)19:14
vilobhmm11yes19:14
*** markvoelker has quit IRC19:14
*** hdd has joined #openstack-nova19:15
vilobhmm11so you suggest just logging something and not raising anything is that correct to interpret ?19:15
dansmithI dunno what the right behavior there is.. it depends on the semantics of the error19:16
dansmithif the error is failure to join the tooz cluster (or whatever), should it continue at all? or print an error and exit?19:16
dansmithretry?19:16
dansmithideally something other than "shit the bed" or "act like everything is okay when it's not" :)19:16
vilobhmm11ok19:17
dansmithvilobhmm11: jaypipes would be a good person to sanity check whatever behavior you choose, although he is a BFD and thus very busy these days19:18
vilobhmm11BFD?19:18
vilobhmm11sure thing19:18
dansmithhttp://www.urbandictionary.com/define.php?term=BFD19:18
vilobhmm11:P19:19
vilobhmm11thanks dansmith for the pointers as always very helpful19:19
dansmithnp19:20
*** mpavone has joined #openstack-nova19:21
*** pixelb has quit IRC19:27
*** annashen has quit IRC19:32
sudiptodansmith, can i ask for a review here: https://review.openstack.org/#/c/173187/ .. whenever you have sometime.19:33
*** Marga_ has quit IRC19:35
*** annegentle has quit IRC19:36
dansmithsudipto: I'd like to see sahid and bauzas leave their +1 on that before I do19:37
sudiptodansmith, Sure...19:37
bauzassudipto: let me review it19:37
sudiptobauzas, thanks!19:37
dansmithleft a -1 for something anyway19:38
dansmithyou should limit the scope of what you have inside a try..except to only that that can raise the exception you're guarding against, IMHO19:38
bauzassudipto: I said the same as dansmith on the previous PS19:39
bauzassudipto: you need to do the exception closed to where it's raising19:40
sudiptobauzas, dansmith yeah19:40
sudiptobauzas, I was just speculating it. Let me get it done.19:40
bauzasokay19:40
bauzass/closed/closer - maybe that's why you misunderstood me - harsh French people here19:41
*** lucas-dinner has quit IRC19:41
sudiptobauzas, No no :) I just actually wrote the code by moving the exception block closer and then pinged you here on IRC.19:42
*** ajayaa has quit IRC19:43
bauzassudipto: eh, I was just running my lawn mower today19:43
bauzassudipto: so I was just afk :)19:43
bauzassudipto: I'm on the CET timezone (UTC+2), so please understand that I'm sometimes not in front of my computer :)19:44
sudiptobauzas,haha, yeah that's perfectly fine...I am on IST - UTC - 5.5 hours lol19:45
sudiptobauzas, It's 1:15 AM right now19:45
bauzasUTC+5.5 then19:45
sudiptobauzas, darn yeah -5.5 from GMT?19:45
*** andreykurilin__ has quit IRC19:45
sudiptobauzas, sorry for my bad!19:45
sudiptothey mean the same :d19:46
bauzasUTC = GMT without daylight19:46
openstackgerritDan Smith proposed openstack/nova: Wedge DB migrations if flavor migrations are not complete  https://review.openstack.org/17448019:46
sudiptobauzas, certainly i should sleep :D19:47
bauzas:)19:47
-openstackstatus- NOTICE: gerrit has been restarted to clear a problem with its event stream. any gerrit changes updated or approved between 19:14 and 19:46 utc will need to be rechecked or have their approval reapplied for zuul to pick them up19:47
sudiptobauzas, coming to this though, if i move the exception block to the get_online_cpus call only - the code would have an if check like this: if online_pcpus: if available_ids <= online_pcpus is that ok?19:47
cfriesenbauzas: pretty sure GMT == UTC19:48
bauzascfriesen: yeah but that can be confusing because GMT is a timezone19:50
bauzascfriesen: and some countries are running daylight19:50
bauzascfriesen: while UTC is just a standart time19:50
bauzasso the countries can be GMT on winter and GMT+1 on summer, but sometimes it leaves people thinking it's only GMT19:51
*** yamahata has joined #openstack-nova19:52
openstackgerritSudipta Biswas proposed openstack/nova: Allow discrete blocks of online pCPUs for pinning  https://review.openstack.org/17318719:52
sudiptobauzas, did a patch set with your comments incorporated. can you take a look pls?19:53
*** bkopilov has joined #openstack-nova19:53
bauzassudipto: go to bed, I'll make some comments19:54
*** lucas-dinner has joined #openstack-nova19:54
sudiptobauzas, yeah i guess that's a better idea.19:54
sudiptobauzas, thanks though for the review. Really appreciate it.19:54
*** sudipto has quit IRC19:56
jaypipesvilobhmm11: I can review your patch. do you have a link? (won't be today, though... likely tomorrow)19:57
*** dboik has quit IRC19:58
*** dboik has joined #openstack-nova19:58
openstackgerritDan Smith proposed openstack/nova: Wedge DB migrations if flavor migrations are not complete  https://review.openstack.org/17448019:58
*** kashyap has quit IRC19:59
*** irenab has joined #openstack-nova20:00
*** matrohon has joined #openstack-nova20:00
*** annegentle has joined #openstack-nova20:01
*** baoli has joined #openstack-nova20:02
vilobhmm11jaypipes : thanks https://review.openstack.org/#/c/172502/220:03
vilobhmm11its still WIP20:03
jaypipesk, no worries20:03
vilobhmm11i have a clear idea on how to move service group to tooz just that had soem feedback on error handling by harlowja so wanted to be sure what format nova would be happy to accept20:04
*** baoli has quit IRC20:04
vilobhmm11so needed some pointers on that and as you might have seen was discussing the same with dansmith earlier during the day20:05
*** baoli has joined #openstack-nova20:05
jaypipesyes, I saw that. dansmith is absolutely correct.20:05
*** Marga_ has joined #openstack-nova20:06
sdakedanms you weren't kidding when you said instance ips are all over the code base20:07
*** irenab has quit IRC20:07
dansmithsdake: yep20:07
dansmithafk for a but20:07
dansmithbit20:07
* dansmith &20:07
vilobhmm11ok thanks jaypipes for having a look.20:07
*** hanzhang has joined #openstack-nova20:08
*** annegentle has quit IRC20:10
*** Marga_ has quit IRC20:11
*** annegentle has joined #openstack-nova20:11
*** shi has quit IRC20:11
*** dboik has quit IRC20:11
*** markvoelker has joined #openstack-nova20:12
*** dboik has joined #openstack-nova20:12
*** tonytan4ever has quit IRC20:13
*** claudiub has joined #openstack-nova20:14
openstackgerritAndrew Boik proposed openstack/nova: Associate floating IPs with first v4 fixed IP if none specified  https://review.openstack.org/16870120:15
*** andreykurilin__ has joined #openstack-nova20:16
mriedemdims__: alaski: dansmith: damn, kwargs it's going to have to be http://logs.openstack.org/24/174524/2/check/gate-nova-tox-functional/78cdca1/console.html#_2015-04-16_19_38_03_08920:19
*** jwcroppe has quit IRC20:20
*** atuvenie has joined #openstack-nova20:20
dims__mriedem: ouch20:20
mriedemeasy peasy20:20
mriedemfixing now20:20
alaskiI like how it says to report the bug in the HTTP exception20:21
*** artom has quit IRC20:22
*** annashen has joined #openstack-nova20:23
dims__mriedem: what are you going to do?20:23
*** jecarey has quit IRC20:23
mriedemdims__: kwargs20:23
*** Sukhdev has quit IRC20:24
*** emagana has quit IRC20:24
dims__mriedem: add parameter= for every call in every method?20:24
*** dprince has joined #openstack-nova20:24
mriedemdims__: no, change_instance_metadata20:24
mriedemi'm running functional tests this time20:24
dims__just this line? self.manager.change_instance_metadata(ctxt, instance, diff)20:24
*** jecarey has joined #openstack-nova20:25
*** MVenesio has quit IRC20:25
*** jecarey_ has joined #openstack-nova20:26
*** ZZelle_ has joined #openstack-nova20:29
*** jecarey has quit IRC20:30
*** tonytan4ever has joined #openstack-nova20:30
*** mpavone has quit IRC20:31
mriedemyeah20:34
*** isd has quit IRC20:34
*** achanda has quit IRC20:35
browneVek: can you remove the -1 on https://review.openstack.org/#/c/157931/ now that Liberty is open.  thanks20:37
*** ozialien has quit IRC20:38
*** achanda has joined #openstack-nova20:40
jaypipesdansmith, mriedem: https://review.openstack.org/#/c/174448/ <- good bug fix.20:40
mriedemwho is this jaypipes guys?20:40
mriedem*guy20:40
jaypipesmriedem: back from vacation.20:40
*** mpavone has joined #openstack-nova20:41
dims__reminds, me...next week i am off mriedem20:43
openstackgerritBo Quan proposed openstack/nova: Correct the help text for the compute option.  https://review.openstack.org/16780520:43
*** markvoelker has quit IRC20:44
mriedemcrazy mirantis guys with their vacations20:44
dansmithjaypipes: that needs tests, IMHO, but I +Wd anyway20:45
jaypipesdansmith: so... I'm actually reconsidering whether https://review.openstack.org/#/c/174448/2 is a good idea...20:45
dansmithjaypipes: why?20:45
jaypipesdansmith: because every other obj_from_db() method assumes that the parameter supplied to it is a row from the DB.20:45
dansmithjaypipes: well, this is extra though20:46
jaypipeshmm, yeah, I uess.20:46
dansmithjaypipes: which is a bag of unrelated things20:46
jaypipesOK, yes...20:46
*** emagana has joined #openstack-nova20:46
*** annegentle has quit IRC20:46
dansmithso it doesn't really make much sense to pass the row I think20:46
dansmithjaypipes: unfortunate that we didn't have any test coverage to catch such a broken thing20:46
dansmithjaypipes: but I'm assuming you hit this because you were reviewing vladikr's tests ... RIGHT? :)20:47
jaypipesyes.20:48
jaypipesexactly.20:48
dansmithI was sure that was the case, just checking :D20:48
*** neelashah has quit IRC20:49
openstackgerritMatt Riedemann proposed openstack/nova: Use kwargs from compute v4 proxy change_instance_metadata  https://review.openstack.org/17452420:52
*** thangp has quit IRC20:52
mriedemdansmith: alaski: dims__: ^20:52
mriedemnova functional passed locally20:52
*** dprince has quit IRC20:54
*** mpavone has quit IRC20:54
openstackgerritMatt Riedemann proposed openstack/nova: Delay STOPPED lifecycle event for all domains, not just Xen  https://review.openstack.org/17277520:54
mriedemsdague: i'm not sure why that was in merge conflict, because it wasn't locally ^20:54
mriedemi rebased anyway20:54
mriedemnvm, i guess it was trivial20:54
bauzasjaypipes: if you have time, I would like to discuss with you about the next specs for the scheduler in L20:55
bauzasjaypipes: now that you rebooted20:55
jaypipesbauzas: how about tomorrow early morning?20:56
bauzasmy time ? sure !20:56
jaypipesbauzas: :)20:56
bauzas9am CEST20:56
bauzasnah, kidding20:56
jaypipesbauzas: I'll be up and going around 8am EST.20:56
bauzasprobably caffeine is not an option20:56
bauzasjaypipes: but yeah, we can sync up20:57
jaypipesthx. still trying to get through what's on my plate right now...20:57
*** cburgess_ has quit IRC20:58
*** isd has joined #openstack-nova20:58
jaypipesdown to 350 emails unread and 30 tabs in chromium open.20:58
bauzasjaypipes: again, try cloning yourself20:58
*** cburgess has joined #openstack-nova20:58
*** vladik has quit IRC20:58
*** vladik has joined #openstack-nova20:59
jaypipesbauzas: I don't like children. :)20:59
bauzasjaypipes: eh, who said children ?20:59
*** vladik has quit IRC20:59
bauzasjaypipes: http://i.imgur.com/TMvhs.jpg21:00
jaypipesbauzas: <321:00
bauzasjaypipes: that said, see mine https://twitter.com/sylvainbauza/status/58802901475907993621:00
jaypipesbauzas: :)21:00
bauzas7mo, in the next year, should be able to go in high school21:01
*** flwang1 has joined #openstack-nova21:01
*** patrickeast_ has joined #openstack-nova21:02
*** annashen has quit IRC21:02
*** patrickeast has quit IRC21:02
*** patrickeast_ is now known as patrickeast21:02
bauzasalaski: did you wrote the cells bugs ?21:03
*** andreykurilin__ has quit IRC21:04
bauzasalaski: just to know if we can just write the changes for getting the cells job happy ?21:05
*** dulek has quit IRC21:05
*** rfolco has quit IRC21:06
*** baoli has quit IRC21:06
*** otter768 has joined #openstack-nova21:06
*** baoli has joined #openstack-nova21:07
*** Marga_ has joined #openstack-nova21:07
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004521:08
*** ivasev has quit IRC21:08
*** apuimedo has joined #openstack-nova21:10
*** otter768 has quit IRC21:11
*** gyee_ has joined #openstack-nova21:11
*** harlowja is now known as harlowja_away21:11
*** Marga_ has quit IRC21:12
*** emagana has quit IRC21:12
*** mwagner_lap has quit IRC21:13
*** sbfox has joined #openstack-nova21:13
*** EmilienM is now known as EmilienM|afk21:13
*** atuvenie has quit IRC21:13
*** tonytan4ever has quit IRC21:13
openstackgerritHans Lindgren proposed openstack/nova: Don't query compute_node through service object in nova-manage  https://review.openstack.org/16890721:14
openstackgerritHans Lindgren proposed openstack/nova: Make ComputeNode.service_id nullable to match db schema  https://review.openstack.org/16928421:14
ZZelle_Hi everyone21:14
openstackgerritHans Lindgren proposed openstack/nova: Avoid querying for Service in resource tracker  https://review.openstack.org/16928821:15
ZZelle_IS there a document describing the content of notifications published by nova?21:15
*** annegentle has joined #openstack-nova21:16
*** isd has quit IRC21:16
*** Marga_ has joined #openstack-nova21:18
*** ijw has joined #openstack-nova21:22
*** dims__ has quit IRC21:23
*** Marga_ has quit IRC21:23
*** Marga_ has joined #openstack-nova21:24
*** cbader has quit IRC21:25
*** eharney has quit IRC21:30
*** aysyd has quit IRC21:30
*** matrohon has quit IRC21:30
*** lpetrut has quit IRC21:32
*** tsekiyam_ has joined #openstack-nova21:33
flwang1mriedem: ping21:33
*** Marga_ has quit IRC21:34
flwang1mriedem: could you pls help take a look at https://review.openstack.org/164494 ?21:34
mriedemZZelle_: https://wiki.openstack.org/wiki/SystemUsageData - might be a bit outdated21:34
*** ijw has quit IRC21:34
flwang1mriedem: i'm not really sure if the rpc version i'm using is correct21:34
mriedemflwang1: probably not going to happen today, it's in my inbox though21:34
*** Marga_ has joined #openstack-nova21:35
flwang1mriedem: cool, pls leave your comments, thank you21:35
ZZelle_mriedem, thx21:35
*** jecarey_ has quit IRC21:35
*** yamahata has quit IRC21:36
*** tsekiyama has quit IRC21:36
*** yamahata has joined #openstack-nova21:37
*** jecarey has joined #openstack-nova21:37
*** tsekiyam_ has quit IRC21:37
*** lucas-dinner has quit IRC21:38
*** emagana has joined #openstack-nova21:38
*** jamielennox|away is now known as jamielennox21:39
*** markvoelker has joined #openstack-nova21:42
*** annegentle has quit IRC21:44
*** annegentle has joined #openstack-nova21:45
*** sbfox has quit IRC21:46
*** jichen has quit IRC21:47
*** annegentle has quit IRC21:47
*** annegentle has joined #openstack-nova21:48
*** apuimedo has quit IRC21:48
*** n0ano has quit IRC21:50
*** hdd has quit IRC21:52
*** EmilienM|afk is now known as EmilienM21:55
*** yamahata has quit IRC22:02
*** otter768 has joined #openstack-nova22:03
*** burt has quit IRC22:06
*** dims__ has joined #openstack-nova22:08
*** annegentle has quit IRC22:09
*** annegentle has joined #openstack-nova22:10
*** busterswt has quit IRC22:11
*** BadCub is now known as BadCub_Hiding22:13
*** harlowja_away is now known as harlowja22:13
*** markvoelker has quit IRC22:13
*** baoli has quit IRC22:13
*** baoli has joined #openstack-nova22:14
*** annegentle has quit IRC22:15
*** Raildo_ has joined #openstack-nova22:16
*** Raildo__ has joined #openstack-nova22:16
*** Raildo__ has quit IRC22:17
*** yamahata has joined #openstack-nova22:21
jogojaypipes: https://review.openstack.org/#/c/173316/2 would like your thoughts on that one22:23
jaypipesjogo: you got it.22:23
*** dboik_ has joined #openstack-nova22:25
jogojaypipes: if you think the differences are fine etc. I will fast track the approval22:25
jaypipesjogo: most of them are, yes, but just double-checking it now.22:26
*** hanzhang has quit IRC22:26
*** hanzhang has joined #openstack-nova22:27
*** pixelb has joined #openstack-nova22:28
*** smccully has quit IRC22:28
*** sdake_ has joined #openstack-nova22:28
*** dboik has quit IRC22:29
jaypipesjogo: k, done.22:29
jaypipes+1 from me.22:29
*** otter768 has quit IRC22:29
*** Marga_ has quit IRC22:29
*** dboik_ has quit IRC22:30
*** Marga_ has joined #openstack-nova22:30
*** Marga_ has quit IRC22:30
jogojaypipes: thanks approving22:31
*** krtaylor has quit IRC22:31
jaypipescheers22:31
jaypipesok, out for now...22:31
*** Marga_ has joined #openstack-nova22:31
jaypipesciao22:31
*** jaypipes has quit IRC22:31
*** sdake has quit IRC22:31
*** annegentle has joined #openstack-nova22:31
*** n0ano has joined #openstack-nova22:32
*** VW_ has quit IRC22:32
*** Marga_ has quit IRC22:32
*** Marga_ has joined #openstack-nova22:32
*** annegentle has quit IRC22:33
*** sdake_ has quit IRC22:33
*** annegentle has joined #openstack-nova22:33
*** ferest has joined #openstack-nova22:34
*** otter768 has joined #openstack-nova22:35
*** annegentle has quit IRC22:37
*** Nic has quit IRC22:37
*** ferest has quit IRC22:39
*** EmilienM is now known as EmilienM|afk22:40
*** mriedem has quit IRC22:42
*** markvoelker has joined #openstack-nova22:48
*** markvoelker_ has joined #openstack-nova22:49
*** dims__ has quit IRC22:50
*** markvoel_ has joined #openstack-nova22:50
*** jecarey has quit IRC22:51
*** markvoe__ has joined #openstack-nova22:51
*** markvoelker has quit IRC22:53
*** gyee_ has quit IRC22:53
*** markvoelker_ has quit IRC22:54
*** dboik_ has joined #openstack-nova22:54
*** Marga_ has quit IRC22:54
*** Marga_ has joined #openstack-nova22:55
*** markvoel_ has quit IRC22:55
*** otter768 has quit IRC22:55
*** ganso_ has quit IRC22:57
*** otter768 has joined #openstack-nova22:58
*** jgrimm is now known as zz_jgrimm23:04
*** Raildo_ has quit IRC23:04
*** Marga_ has quit IRC23:05
*** baoli has quit IRC23:05
*** zzzeek has quit IRC23:07
*** gyee_ has joined #openstack-nova23:09
*** EmilienM|afk is now known as EmilienM23:10
*** rdo has quit IRC23:20
*** rdo has joined #openstack-nova23:22
*** igordcard_ has quit IRC23:27
*** sdake has joined #openstack-nova23:34
*** Sukhdev has joined #openstack-nova23:34
*** Marga_ has joined #openstack-nova23:36
*** pixelb has quit IRC23:36
*** hemna is now known as hemnafk23:39
*** Marga_ has quit IRC23:40
*** otter768 has quit IRC23:42
jogoanyone use shelving here?23:43
jogohaving issues getting it to work with nova-network and a floating IP23:43
*** browne has quit IRC23:46
*** otter768 has joined #openstack-nova23:48
cburgessNot I said the fly.23:50
*** emagana has quit IRC23:50
alaskibauzas: I did not get to the cells bugs.  got overwhelmed with meetings today23:54
alaskibauzas: I'm out tomorrow morning but I will get to them after that23:54
*** pixelb has joined #openstack-nova23:55
*** sdake_ has joined #openstack-nova23:55
*** Kevin_Zheng has joined #openstack-nova23:57
*** rfolco has joined #openstack-nova23:58
*** markvoe__ has quit IRC23:58
*** sdake has quit IRC23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!