Wednesday, 2015-10-07

*** jdurgin1 has joined #openstack-nova00:02
*** amrith is now known as _amrith_00:04
*** takedakn has quit IRC00:04
*** takedakn has joined #openstack-nova00:05
*** diogogmt has quit IRC00:05
*** andrewbogott has joined #openstack-nova00:12
*** dims_ has quit IRC00:13
*** dims_ has joined #openstack-nova00:14
*** tsekiyam_ has joined #openstack-nova00:15
*** Daisy has joined #openstack-nova00:17
*** tsekiyama has quit IRC00:18
*** darrenc_afk is now known as darrenc00:18
*** dims_ has quit IRC00:19
dansmithFYI to anyone around, we really need to get this trivial patch in:00:19
dansmithhttps://review.openstack.org/#/c/231686/00:19
*** tsekiyam_ has quit IRC00:19
*** manishg has joined #openstack-nova00:19
*** Daisy has quit IRC00:19
*** mtanino has quit IRC00:19
dansmithit's blocking an ovo patch landing to fix an upgrade issue, which prevents us from landing any new object versions without breaking things00:19
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/23123800:19
*** Daisy has joined #openstack-nova00:20
*** Daisy has quit IRC00:20
*** kgalanov has joined #openstack-nova00:20
*** Daisy has joined #openstack-nova00:20
*** tpeoples has quit IRC00:23
*** dims_ has joined #openstack-nova00:25
*** Daisy has quit IRC00:25
*** takedakn1 has joined #openstack-nova00:25
*** su_zhang has quit IRC00:26
*** takedakn has quit IRC00:26
*** takedakn1 is now known as takedakn00:26
*** dims_ has quit IRC00:26
*** dims_ has joined #openstack-nova00:26
*** artom has quit IRC00:28
*** zhangjn has quit IRC00:29
*** zhangjn has joined #openstack-nova00:32
*** takedakn has quit IRC00:32
*** manishg has quit IRC00:36
*** zhangjn has quit IRC00:37
*** Daisy has joined #openstack-nova00:37
*** IanGovett has quit IRC00:39
*** claudiub has quit IRC00:40
*** tpeoples has joined #openstack-nova00:42
*** thumpba has joined #openstack-nova00:44
*** su_zhang has joined #openstack-nova00:47
*** manishg has joined #openstack-nova00:50
*** angdraug has quit IRC00:51
*** IanGovett has joined #openstack-nova00:53
*** IanGovett has quit IRC00:54
*** ssurana has quit IRC00:54
*** Daisy has quit IRC00:55
*** browne has quit IRC00:57
*** gyee has quit IRC00:58
*** thumpba has quit IRC01:01
*** mudassirlatif has quit IRC01:04
*** baoli has joined #openstack-nova01:05
*** thumpba_ has joined #openstack-nova01:06
*** su_zhang has quit IRC01:07
*** vilobhmm111 has quit IRC01:08
*** stevemar_ has quit IRC01:11
*** jhesketh has quit IRC01:15
*** apoorvad has quit IRC01:16
*** jhesketh has joined #openstack-nova01:16
*** emagana has quit IRC01:19
*** apoorvad has joined #openstack-nova01:25
*** markvoelker has quit IRC01:27
*** diogogmt has joined #openstack-nova01:34
*** armax has joined #openstack-nova01:37
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/23123801:44
*** tjones has joined #openstack-nova01:45
*** apuimedo has quit IRC01:50
*** apuimedo has joined #openstack-nova01:51
*** achanda has joined #openstack-nova01:52
*** unicell has quit IRC01:55
*** fawadkhaliq has joined #openstack-nova01:58
*** fawadkhaliq has quit IRC01:59
*** thumpba_ has quit IRC01:59
*** zhangjn has joined #openstack-nova02:00
*** stevemar_ has joined #openstack-nova02:00
*** zhangjn has quit IRC02:05
*** zhangjn has joined #openstack-nova02:07
*** apoorvad has quit IRC02:08
*** manishg has quit IRC02:08
*** tjones has quit IRC02:14
*** kumar has joined #openstack-nova02:15
*** VikasC has quit IRC02:16
*** haomaiwang has joined #openstack-nova02:17
*** thumpba has joined #openstack-nova02:17
*** yamahata has joined #openstack-nova02:17
*** baoli has quit IRC02:18
*** changbl has joined #openstack-nova02:19
*** penick has quit IRC02:22
*** manishg has joined #openstack-nova02:24
*** thumpba has quit IRC02:28
*** haomaiwang has quit IRC02:29
*** haomaiwang has joined #openstack-nova02:30
*** thumpba_ has joined #openstack-nova02:31
*** browne has joined #openstack-nova02:31
*** tpeoples has quit IRC02:31
*** manishg has quit IRC02:33
*** julien_ZTE has quit IRC02:33
openstackgerritDiana Clarke proposed openstack/python-novaclient: Fix incorrect help for 'nova flavor-create'  https://review.openstack.org/23180302:34
*** thumpba_ has quit IRC02:34
*** haomaiwang has quit IRC02:35
*** thumpba_ has joined #openstack-nova02:37
*** haomaiwa_ has joined #openstack-nova02:39
*** mdrabe has joined #openstack-nova02:42
*** yamahata has quit IRC02:42
*** baoli has joined #openstack-nova02:45
*** tpeoples has joined #openstack-nova02:46
*** darrenc is now known as darrenc_afk02:53
*** zzzeek has quit IRC02:55
*** pixelbeat has quit IRC02:55
*** deepthi has joined #openstack-nova02:57
*** tjones has joined #openstack-nova02:58
*** baoli has quit IRC02:59
*** haomaiwa_ has quit IRC03:00
*** baoli has joined #openstack-nova03:01
*** haomaiwang has joined #openstack-nova03:03
*** achanda has quit IRC03:09
*** burgerk has joined #openstack-nova03:15
*** dave-mccowan has quit IRC03:16
*** Marga_ has quit IRC03:18
*** markvoelker has joined #openstack-nova03:19
*** Marga_ has joined #openstack-nova03:19
*** links has joined #openstack-nova03:21
*** markvoelker_ has joined #openstack-nova03:23
*** Marga_ has quit IRC03:23
*** markvoelker has quit IRC03:25
*** tjones has quit IRC03:28
*** markvoelker has joined #openstack-nova03:33
*** markvoelker has quit IRC03:33
*** markvoelker_ has quit IRC03:33
*** baoli has quit IRC03:34
*** vilobhmm11 has joined #openstack-nova03:38
*** darrenc_afk is now known as darrenc03:48
*** Sree has joined #openstack-nova03:50
*** Sree has quit IRC03:51
*** Sree has joined #openstack-nova03:52
*** Sree has quit IRC03:54
*** Sree has joined #openstack-nova03:56
*** Vimal_ has joined #openstack-nova03:57
*** Marga_ has joined #openstack-nova03:58
*** Marga_ has quit IRC03:59
*** Marga_ has joined #openstack-nova03:59
*** Marga_ has quit IRC04:01
*** Marga_ has joined #openstack-nova04:02
* Vimal_ slaps anish around a bit with a large fishbot04:03
*** VikasC has joined #openstack-nova04:03
*** Marga_ has quit IRC04:06
*** Marga_ has joined #openstack-nova04:06
*** fawadkhaliq has joined #openstack-nova04:10
*** fawadk has joined #openstack-nova04:11
*** wverdugo500 has quit IRC04:12
*** hemna has joined #openstack-nova04:12
*** fawadkhaliq has quit IRC04:15
*** burgerk has quit IRC04:16
*** su_zhang has joined #openstack-nova04:17
*** fawadkhaliq has joined #openstack-nova04:18
*** fawadk has quit IRC04:19
*** achanda has joined #openstack-nova04:20
*** achanda has quit IRC04:20
*** kumar has quit IRC04:25
*** mdrabe has quit IRC04:35
*** thumpba_ has quit IRC04:39
*** csd has joined #openstack-nova04:39
openstackgerritMichael Still proposed openstack/nova: Switch to oslo.cache lib  https://review.openstack.org/20304904:39
*** markvoelker has joined #openstack-nova04:40
*** csd has quit IRC04:40
*** Sree has quit IRC04:41
*** fawadk has joined #openstack-nova04:42
*** Sree has joined #openstack-nova04:42
*** csd has joined #openstack-nova04:44
*** thumpba has joined #openstack-nova04:44
*** markvoelker has quit IRC04:45
*** fawadkhaliq has quit IRC04:45
*** deepthi has quit IRC04:46
*** thumpba has quit IRC04:48
*** markvoelker has joined #openstack-nova04:50
*** agireud has joined #openstack-nova04:51
*** dims_ has quit IRC04:52
*** rushiagr_away is now known as rushiagr04:52
*** zhangjn has quit IRC04:53
*** markvoelker has quit IRC04:55
*** Vimal_ has quit IRC04:59
*** markvoelker has joined #openstack-nova05:02
*** markvoelker has quit IRC05:03
*** markvoelker has joined #openstack-nova05:13
*** emagana has joined #openstack-nova05:16
*** hemna has quit IRC05:18
*** achanda has joined #openstack-nova05:19
*** markvoelker has quit IRC05:21
*** yassine has joined #openstack-nova05:23
*** haomaiwang has quit IRC05:23
*** hemna has joined #openstack-nova05:24
*** alex_klimov has joined #openstack-nova05:25
*** rushiagr is now known as rushiagr_away05:27
*** csd has quit IRC05:28
*** e0ne has joined #openstack-nova05:28
*** markvoelker has joined #openstack-nova05:28
*** e0ne has quit IRC05:30
*** jamielennox is now known as jamielennox|away05:30
*** stevemar_ has quit IRC05:31
*** haomaiwang has joined #openstack-nova05:33
*** e0ne has joined #openstack-nova05:34
*** zhangjn has joined #openstack-nova05:36
*** markvoelker has quit IRC05:36
*** e0ne has quit IRC05:37
*** doron_afk has joined #openstack-nova05:39
*** csd has joined #openstack-nova05:40
*** markvoelker has joined #openstack-nova05:41
*** e0ne has joined #openstack-nova05:42
*** markvoelker_ has joined #openstack-nova05:42
*** markvoelker_ has quit IRC05:43
*** markvoelker has quit IRC05:45
*** su_zhang has quit IRC05:46
*** e0ne has quit IRC05:49
*** unicell has joined #openstack-nova05:50
*** moshele has joined #openstack-nova05:51
*** e0ne has joined #openstack-nova05:53
*** e0ne has quit IRC05:54
*** _amrith_ is now known as amrith05:55
*** markvoelker has joined #openstack-nova05:58
*** e0ne has joined #openstack-nova05:58
*** csd has quit IRC05:59
*** markvoelker_ has joined #openstack-nova06:00
*** markvoelker has quit IRC06:00
*** haomaiwang has quit IRC06:01
*** kaisers has quit IRC06:01
*** haomaiwa_ has joined #openstack-nova06:01
*** e0ne has quit IRC06:02
*** amrith is now known as _amrith_06:04
*** markvoelker_ has quit IRC06:05
*** vilobhmm11 has quit IRC06:05
*** nkrinner has joined #openstack-nova06:06
*** hemna has quit IRC06:06
*** jdurgin1 has quit IRC06:09
*** asettle has quit IRC06:09
*** alex_klimov has quit IRC06:13
*** achanda has quit IRC06:13
*** nikhil has quit IRC06:15
*** nikhil has joined #openstack-nova06:16
*** irenab has joined #openstack-nova06:16
*** markvoelker has joined #openstack-nova06:19
*** achanda has joined #openstack-nova06:19
*** unicell1 has joined #openstack-nova06:20
*** unicell has quit IRC06:20
*** e0ne has joined #openstack-nova06:21
*** jamielennox|away is now known as jamielennox06:21
*** nikhil_k has joined #openstack-nova06:25
*** markvoelker has quit IRC06:25
*** kaisers has joined #openstack-nova06:25
*** mtreinish has quit IRC06:26
*** e0ne has quit IRC06:26
openstackgerritRaga lahari proposed openstack/nova: Support for new --status parameter to "nova-manage vm list" command  https://review.openstack.org/23144706:26
*** nikhil has quit IRC06:28
*** cfriesen has quit IRC06:29
*** sahid has joined #openstack-nova06:29
*** mtreinish has joined #openstack-nova06:31
*** e0ne has joined #openstack-nova06:31
*** emagana has quit IRC06:33
*** rushiagr_away is now known as rushiagr06:33
*** e0ne has quit IRC06:35
*** alexschm has joined #openstack-nova06:37
*** achanda has quit IRC06:39
*** e0ne has joined #openstack-nova06:39
*** vlaza has joined #openstack-nova06:40
*** achanda has joined #openstack-nova06:42
*** jcoufal has joined #openstack-nova06:43
*** rdopiera has joined #openstack-nova06:46
*** scheuran has joined #openstack-nova06:47
*** markvoelker has joined #openstack-nova06:47
*** fawadk has quit IRC06:48
*** pm90_ has joined #openstack-nova06:49
*** jcoufal has quit IRC06:49
*** fawadkhaliq has joined #openstack-nova06:50
*** fawadkhaliq has quit IRC06:51
*** jcoufal has joined #openstack-nova06:52
openstackgerritVineet Menon proposed openstack/nova-specs: Add flavor tables to API database  https://review.openstack.org/23185806:52
*** markvoelker has quit IRC06:52
*** rotbeard has joined #openstack-nova06:53
*** deepthi has joined #openstack-nova06:54
*** e0ne has quit IRC06:55
*** matrohon has joined #openstack-nova06:56
*** Sree has quit IRC07:00
*** haomaiwa_ has quit IRC07:01
*** romainh has joined #openstack-nova07:01
*** haomaiwang has joined #openstack-nova07:01
*** markvoelker has joined #openstack-nova07:02
*** romainh has left #openstack-nova07:03
*** Daisy has joined #openstack-nova07:05
*** Daisy has quit IRC07:06
*** markvoelker has quit IRC07:07
*** Daisy has joined #openstack-nova07:07
*** Daisy has quit IRC07:09
*** Daisy has joined #openstack-nova07:09
*** fawadkhaliq has joined #openstack-nova07:11
*** ildikov has quit IRC07:14
*** atuvenie has joined #openstack-nova07:16
*** markvoelker has joined #openstack-nova07:16
*** Daisy has quit IRC07:16
*** armax has quit IRC07:17
*** Daisy has joined #openstack-nova07:17
*** pm90_ has quit IRC07:20
*** markvoelker has quit IRC07:21
*** Daisy has quit IRC07:22
*** vivekd has joined #openstack-nova07:22
*** pbrooko has joined #openstack-nova07:28
openstackgerritAndrey Kurilin proposed openstack/nova: [Py34] api.openstack.test_faults.TestFaultWrapper  https://review.openstack.org/23147307:29
*** tpeoples has quit IRC07:29
*** aloga_ has joined #openstack-nova07:30
*** markvoelker has joined #openstack-nova07:31
openstackgerritAndrey Kurilin proposed openstack/nova: [Py34] api.openstack.compute.legacy_v2.test_servers.Base64ValidationTest  https://review.openstack.org/23147407:31
*** pbrooko has quit IRC07:32
openstackgerritAndrey Kurilin proposed openstack/nova: [Py34] api.openstack.compute.legacy_v2.test_servers.Base64ValidationTest  https://review.openstack.org/23147407:34
*** marzif has joined #openstack-nova07:34
*** markvoelker has quit IRC07:36
*** achanda has quit IRC07:39
openstackgerritRaga lahari proposed openstack/nova: Support for new --status parameter to "nova-manage vm list" command  https://review.openstack.org/23144707:40
bauzasgood morning Nova07:41
*** Sree has joined #openstack-nova07:44
*** doron_afk has quit IRC07:44
*** jlanoux has joined #openstack-nova07:45
*** jwcroppe_ has joined #openstack-nova07:45
*** ociuhandu has quit IRC07:45
*** markvoelker has joined #openstack-nova07:46
*** doron_afk has joined #openstack-nova07:46
*** rdopiera has quit IRC07:46
*** rdopiera has joined #openstack-nova07:48
*** jwcroppe has quit IRC07:48
*** tteggel has quit IRC07:48
*** salv-orlando has quit IRC07:48
*** MarcF has joined #openstack-nova07:49
*** salv-orlando has joined #openstack-nova07:49
*** markvoelker has quit IRC07:50
*** tteggel has joined #openstack-nova07:51
*** tteggel has left #openstack-nova07:53
*** lpetrut has joined #openstack-nova07:53
*** jwcroppe_ has quit IRC07:57
*** jwcroppe has joined #openstack-nova07:59
*** markvoelker has joined #openstack-nova08:00
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-nova08:01
*** marzif has quit IRC08:02
*** markvoelker has quit IRC08:05
*** e0ne has joined #openstack-nova08:10
*** MarcF has quit IRC08:12
*** pbrooko has joined #openstack-nova08:12
*** browne has quit IRC08:14
*** markvoelker has joined #openstack-nova08:15
*** claudiub has joined #openstack-nova08:15
*** doron_afk has quit IRC08:16
*** markus_z has joined #openstack-nova08:19
*** alex_klimov has joined #openstack-nova08:19
*** markvoelker has quit IRC08:19
*** fawadkhaliq has quit IRC08:21
*** fawadkhaliq has joined #openstack-nova08:21
pilgrimstack1bauzas: Do you think Nova is in CEST timezone? :p08:24
bauzaspilgrimstack1: I'm gentle :p08:24
openstackgerritTakashi NATSUME proposed openstack/nova: Add soft deleting instance actions and events  https://review.openstack.org/23188708:24
*** Sree has quit IRC08:25
*** Sree has joined #openstack-nova08:27
*** lucas-dinner has joined #openstack-nova08:27
*** lucasagomes has joined #openstack-nova08:27
*** doron_afk has joined #openstack-nova08:28
*** markvoelker has joined #openstack-nova08:29
*** e0ne has quit IRC08:30
*** e0ne has joined #openstack-nova08:32
*** markvoelker has quit IRC08:34
*** jistr has joined #openstack-nova08:36
*** markvoelker has joined #openstack-nova08:36
*** atuvenie has quit IRC08:37
*** danpb has joined #openstack-nova08:37
*** Daisy has joined #openstack-nova08:37
*** links has quit IRC08:38
*** achanda has joined #openstack-nova08:39
*** markvoelker has quit IRC08:41
*** Daisy has quit IRC08:42
*** Daisy has joined #openstack-nova08:43
*** markvoelker has joined #openstack-nova08:44
*** achanda has quit IRC08:46
*** Murali has joined #openstack-nova08:46
*** Daisy has quit IRC08:47
*** fawadkhaliq has quit IRC08:48
*** fawadkhaliq has joined #openstack-nova08:48
*** Sree has quit IRC08:50
*** gszasz has joined #openstack-nova08:50
claudiubjohnthetubaguy: hi. the hyper-v uefi secure boot spec, the one that you previously had a +2 on, needs your +2 again. One minor change added: https://review.openstack.org/#/c/190997/ Thanks!08:50
*** markvoelker has quit IRC08:51
*** rubasov has joined #openstack-nova08:51
*** markvoelker has joined #openstack-nova08:51
*** Daisy has joined #openstack-nova08:52
johnthetubaguyclaudiub: sweet, thanks for letting me know08:53
*** subscope has joined #openstack-nova08:53
*** aix has joined #openstack-nova08:53
*** Daisy has quit IRC08:55
*** Sree has joined #openstack-nova08:55
*** Sree has quit IRC08:55
*** markvoelker has quit IRC08:56
*** moshele has quit IRC08:58
*** nikhil_k has quit IRC09:00
*** nikhil_k has joined #openstack-nova09:00
*** haomaiwang has quit IRC09:01
*** Sree has joined #openstack-nova09:01
*** 77CAAYMV4 has joined #openstack-nova09:01
*** markvoelker has joined #openstack-nova09:01
*** rushiagr is now known as rushiagr_away09:06
*** markvoelker has quit IRC09:06
*** deepthi has quit IRC09:08
*** aloga_ has quit IRC09:10
*** ildikov has joined #openstack-nova09:11
*** tbh has joined #openstack-nova09:14
*** tbh has quit IRC09:15
*** tbh has joined #openstack-nova09:15
*** markvoelker has joined #openstack-nova09:16
openstackgerritClaudiu Belu proposed openstack/nova-specs: Moves the Hyper-V NUMA instance spec to Mitaka  https://review.openstack.org/23141709:18
*** deepthi has joined #openstack-nova09:20
*** zhangjn has quit IRC09:20
*** markvoelker has quit IRC09:20
*** andrearo1 is now known as andrearosa09:21
*** lucas-dinner has quit IRC09:23
*** links has joined #openstack-nova09:25
*** markvoelker has joined #openstack-nova09:27
*** deepthi has quit IRC09:29
*** markvoelker has quit IRC09:31
*** tbh has quit IRC09:34
*** moshele has joined #openstack-nova09:34
*** atuvenie has joined #openstack-nova09:35
*** nikhil_k_ has joined #openstack-nova09:36
*** markvoelker has joined #openstack-nova09:37
jwcroppejohnthetubaguy: if you get a moment, would you mind reviewing a fix for a metric bug?  https://review.openstack.org/#/c/219153/  Jay's +2'd ... just need one more set of eyes.09:38
*** tbh has joined #openstack-nova09:39
*** romainh has joined #openstack-nova09:40
*** nikhil_k has quit IRC09:40
bauzasjwcroppe: still in Rennes or night owl ?09:41
jwcroppebauzas: night owl... who just realized he needs to go to bed!  :)09:42
bauzas:)09:42
*** deepthi has joined #openstack-nova09:42
jwcroppebauzas: lost track of time, again.09:42
*** markvoelker has quit IRC09:42
*** VikasC has quit IRC09:42
jwcroppebauzas: talk to you when I get up ... in an hour.  ;-)09:42
openstackgerritBalazs Gibizer proposed openstack/nova-specs: Remove shared storage flag in evacuate API  https://review.openstack.org/18303309:43
openstackgerritBalazs Gibizer proposed openstack/nova-specs: Add soft policies for server-group feature  https://review.openstack.org/21250409:43
*** achanda has joined #openstack-nova09:43
*** achanda has quit IRC09:46
*** achanda has joined #openstack-nova09:47
*** VikasC has joined #openstack-nova09:47
*** subscope has quit IRC09:49
*** pbrooko has quit IRC09:50
*** achanda has quit IRC09:52
*** markvoelker has joined #openstack-nova09:52
*** kaisers has quit IRC09:57
*** markvoelker has quit IRC09:57
johnthetubaguyclaudiub: I see a WIP -1 on this: https://review.openstack.org/#/c/231418/2 is there something you need to change on that?09:58
*** tbh has quit IRC09:59
*** kaisers has joined #openstack-nova09:59
*** 77CAAYMV4 has quit IRC10:01
claudiubjohnthetubaguy: needs a small update in the "Data model impact", adding a new field for ImageMetaProps10:01
*** haomaiwa_ has joined #openstack-nova10:01
claudiubjohnthetubaguy: will update it today.10:03
johnthetubaguyclaudiub: ah, gotcha, cool10:03
johnthetubaguyclaudiub: honestly, I keep forget that should have object changes listed, I am used to that just being DB schema changes10:04
johnthetubaguynot totally sure why we require that, thinking about it...10:04
openstackgerritRoman Bogorodskiy proposed openstack/nova-specs: User-controlled SR-IOV ports allocation  https://review.openstack.org/18224210:04
*** e0ne has quit IRC10:06
*** romainh has left #openstack-nova10:06
*** markvoelker has joined #openstack-nova10:06
*** kaisers has quit IRC10:06
claudiubjohnthetubaguy: well, updating an object will increment its version, making it incompatible with older versions of the objects. Other than that, dunno. Thanks for re-approving the specs. :)10:07
johnthetubaguyclaudiub: no problem, it just feels like those details are best covered in code reviews really10:08
*** zzelle has joined #openstack-nova10:08
*** zzelle is now known as Guest3136710:09
*** e0ne has joined #openstack-nova10:09
*** kaisers has joined #openstack-nova10:11
*** markvoelker has quit IRC10:11
*** Guest31367 has quit IRC10:13
*** zzelle_ has joined #openstack-nova10:15
*** zzelle_ is now known as ZZll10:16
*** ZZll is now known as ZZelle_10:16
*** yuntongjin has joined #openstack-nova10:18
*** Sree has quit IRC10:20
*** Sree has joined #openstack-nova10:21
*** markvoelker has joined #openstack-nova10:21
openstackgerritDaniel Berrange proposed openstack/nova: libvirt: update to min required version to 0.10.2  https://review.openstack.org/23191710:22
openstackgerritBalazs Gibizer proposed openstack/nova-specs: Remove service management from nova-manage  https://review.openstack.org/23191810:23
*** Sree has quit IRC10:25
*** markvoelker has quit IRC10:26
openstackgerritRaga lahari proposed openstack/nova: Support for new --status parameter to "nova-manage vm list" command  https://review.openstack.org/23144710:26
*** subscope has joined #openstack-nova10:26
*** Daisy has joined #openstack-nova10:27
*** pbrooko has joined #openstack-nova10:27
*** Daisy has quit IRC10:31
*** markvoelker has joined #openstack-nova10:35
*** mwagner_ has quit IRC10:38
*** markvoelker has quit IRC10:40
*** mdavidson has joined #openstack-nova10:40
*** Daisy has joined #openstack-nova10:41
*** kaisers has quit IRC10:42
*** pbrooko has quit IRC10:44
*** IanGovett has joined #openstack-nova10:44
*** markvoelker has joined #openstack-nova10:45
*** kaisers has joined #openstack-nova10:45
*** Daisy has quit IRC10:46
openstackgerritDaniel Berrange proposed openstack/nova: libvirt: update to min required version to 0.10.2  https://review.openstack.org/23191710:48
openstackgerritlyanchih proposed openstack/nova: libvirt: serial console ports count upper limit needs to be checked  https://review.openstack.org/20737310:49
*** salv-orlando has quit IRC10:49
*** romainh has joined #openstack-nova10:51
*** zhangjn has joined #openstack-nova10:52
*** dave-mccowan has joined #openstack-nova10:54
*** markvoelker has quit IRC10:54
*** pilgrimstack1 has quit IRC10:55
openstackgerritAlexis Lee proposed openstack/nova: Make add_metrics_to_list side-effect free  https://review.openstack.org/23091810:57
*** pixelbeat has joined #openstack-nova10:58
*** smatzek has joined #openstack-nova10:59
*** markvoelker has joined #openstack-nova11:00
*** obondarev has quit IRC11:00
*** haomaiwa_ has quit IRC11:01
*** haomaiwang has joined #openstack-nova11:01
*** njirap has joined #openstack-nova11:01
*** pilgrimstack has joined #openstack-nova11:02
*** garyk has joined #openstack-nova11:04
*** gzhai has quit IRC11:04
*** markvoelker has quit IRC11:05
*** gzhai has joined #openstack-nova11:05
openstackgerritsonu proposed openstack/nova: Replaces xrange() with range() to provide py 2/3 compatibility  https://review.openstack.org/20530711:07
*** markvoelker has joined #openstack-nova11:14
*** DRoBeR has joined #openstack-nova11:15
*** zhangjn has quit IRC11:15
johnthetubaguyclaudiub: I meant to ask, you know that python34 fix that we merged, could you get someone to follow up with expanding the list of unit tests that pass with py34 after that change?11:16
*** pbrooko has joined #openstack-nova11:17
*** pbrooko has quit IRC11:17
*** markvoelker has quit IRC11:19
*** jamielennox is now known as jamielennox|away11:19
*** claudiub|2 has joined #openstack-nova11:28
*** aix has quit IRC11:29
*** claudiub has quit IRC11:29
*** markvoelker has joined #openstack-nova11:29
*** takedakn has joined #openstack-nova11:30
*** pbrooko has joined #openstack-nova11:30
*** ndipanov has joined #openstack-nova11:31
openstackgerritJohn Garbutt proposed openstack/nova: HyperV: Add IOUtils class for asynchronous IO operations  https://review.openstack.org/18403511:33
*** markvoelker has quit IRC11:34
*** lucasagomes is now known as lucas-hungry11:36
*** subscope has quit IRC11:37
*** baoli has joined #openstack-nova11:38
openstackgerritAlexey I. Froloff proposed openstack/nova: Fix more inconsistency between Nova-Net and Neutron  https://review.openstack.org/22627411:39
*** haomaiwang has quit IRC11:39
*** romainh has left #openstack-nova11:39
*** baoli_ has joined #openstack-nova11:39
*** claudiub has joined #openstack-nova11:40
*** ildikov has quit IRC11:41
*** yuntongjin has quit IRC11:42
*** claudiub|2 has quit IRC11:43
*** baoli has quit IRC11:43
*** _amrith_ is now known as amrith11:43
*** markvoelker has joined #openstack-nova11:44
*** jcoufal has quit IRC11:45
*** takedakn1 has joined #openstack-nova11:46
*** takedakn has quit IRC11:46
*** takedakn1 is now known as takedakn11:46
*** Daisy has joined #openstack-nova11:48
*** markvoelker has quit IRC11:49
*** markvoelker has joined #openstack-nova11:50
*** Daisy has quit IRC11:51
*** thorst1 has joined #openstack-nova11:51
*** sambetts has quit IRC11:52
*** pbrooko has quit IRC11:52
*** sambetts has joined #openstack-nova11:54
*** aysyd has joined #openstack-nova11:55
*** pbrooko has joined #openstack-nova11:56
*** markvoelker has quit IRC11:56
*** Sree has joined #openstack-nova11:56
*** jcoufal has joined #openstack-nova11:57
claudiubjohnthetubaguy: i have no idea on how to expand the list of unit tests for py34. there seems to be a tests-py3.txt file that should be changed, but dunno where it is. It isn't in nova or project-config.11:59
*** ociuhandu has joined #openstack-nova11:59
johnthetubaguyclaudiub: its here: https://github.com/openstack/nova/blob/master/tests-py3.txt12:00
claudiubjohnthetubaguy: damn, I'm blind. I'll send a commit soon.12:02
*** rfolco has joined #openstack-nova12:02
*** subscope has joined #openstack-nova12:05
*** Marga_ has quit IRC12:05
*** Marga_ has joined #openstack-nova12:05
*** markvoelker has joined #openstack-nova12:07
*** david-ly_ has joined #openstack-nova12:07
openstackgerritFlavio Percoco proposed openstack/nova-specs: (follow up) Nova Image v1 -> v2  https://review.openstack.org/23135712:07
openstackgerritjichenjc proposed openstack/nova: Remove soft_deleted filter in image cache manage  https://review.openstack.org/21711812:08
*** jichen has joined #openstack-nova12:09
*** david-lyle has quit IRC12:09
*** david-ly_ is now known as david-lyle12:09
openstackgerritClaudiu Belu proposed openstack/nova: Expands python34 unit tests list  https://review.openstack.org/23194812:10
*** mwagner_ has joined #openstack-nova12:11
*** markvoelker has quit IRC12:12
*** thumpba has joined #openstack-nova12:14
*** fawadkhaliq has quit IRC12:15
*** Daisy has joined #openstack-nova12:15
*** rfolco has quit IRC12:16
*** rfolco has joined #openstack-nova12:17
*** obondarev has joined #openstack-nova12:17
*** burgerk has joined #openstack-nova12:20
*** markvoelker has joined #openstack-nova12:21
*** thumpba has quit IRC12:22
*** su_zhang has joined #openstack-nova12:23
*** russellb has quit IRC12:24
openstackgerritAlexis Lee proposed openstack/nova: Make add_metrics_to_list side-effect free  https://review.openstack.org/23091812:24
johnthetubaguyclaudiub: no worries, that keeps moving, was in tox.ini12:25
*** ccesario_ has joined #openstack-nova12:26
*** markvoelker has quit IRC12:26
BobBalldanpb: Regarding your comment on https://review.openstack.org/#/c/229351/2/specs/mitaka/approved/xenapi-add-support-for-vgpu.rst - not sure I understand... With using a key in extra_specs we can filter the hosts to find ones that have available vGPUs and use the same extra_specs definition to add the vGPU in Novas code.  Why does this approach require the admin to create 2 sets of flavors?12:26
bauzasoh johnthetubaguy was really quick with https://review.openstack.org/#/c/231542/ :)12:26
*** vivekd has quit IRC12:27
*** Daisy has quit IRC12:27
bauzasI was about to rebase for removing the prio section :)12:27
*** Daisy has joined #openstack-nova12:27
johnthetubaguybauzas: ah, yeah, we should probably do that in bulk, I just started ignoring it for the fast approves12:27
danpbBobBall: if we take  m1.small and set   vgpu=foo,  then m1.small will always result in creation of a vGPU, so if you want vms without vGPU, then you'd need to create  m1.small and m1.small-with-vgpu12:27
BobBallAh, I see what you mean, yes.12:28
danpbwith my suggestion, we'd only ever need m1.small in the common case - no need for separate m1.small-with-vgpu12:28
bauzasjohnthetubaguy: yup, I was about removing that section for my spec, I could just also change the alrady approved ones12:28
danpbof course an admin might want both, if they want to restrirt access to and/or charge extra for vGPU support, but its nice not to force use of separate flavours12:28
*** zhangjn has joined #openstack-nova12:28
johnthetubaguydanpb: most features work like that right now though, that have resource implications? I think the flavor rework jaypipes is thinking about was to fix that issue more generally12:29
danpbjohnthetubaguy: its not entirely clear-cut - some do, some dont12:30
BobBallWhich ones don't?  I'd just like to have a look at how they are implemented12:30
*** zhangjn has quit IRC12:31
johnthetubaguydanpb: thats true I guess, I wish that stuff that involved consuming finite resources were only available by flavors really12:31
danpbor at least, much of the recent stuff  we've done has allow image meta to override what flavour has done12:31
*** markvoelker has joined #openstack-nova12:31
danpbeg with NUMA, the image meta can specify the number of NUMA nodes12:31
danpbprovided it is *not* specified against the flavour12:31
*** Daisy has quit IRC12:32
danpbthe admin can set it against the flavour to prevent tenants choosing their own numa settings in image meta12:32
bauzasjohnthetubaguy: meh, totally missed https://github.com/openstack/nova-specs/commit/99bf65f023fe0e48ce49fbfa48c17024181c253112:32
danpbbut to prevent an explosion of number of flavours private-cloud would typically just let user choose whatever they want against image12:32
danpbguess we could do a similar thing for vGPU - allow a choice of enabling it on the flavour, or on the image meta, with flavour taking priority if set12:33
*** Daisy has joined #openstack-nova12:33
BobBallI'm not sure how the NUMA nodes work - are the number of numa nodes a finite resource?12:34
*** njirap has quit IRC12:34
johnthetubaguyI need to finish off that patch I started to get agreement on how we use all these leavers12:34
johnthetubaguyits got so crazily messy now12:34
johnthetubaguyI get there are use cases for both though12:35
*** Daisy has quit IRC12:36
*** markvoelker has quit IRC12:36
danpbBobBall: yeah, to some degree they are12:36
johnthetubaguybauzas: its one where I think we need to see the code really12:36
*** njirap has joined #openstack-nova12:36
bauzasjohnthetubaguy: agreed, I'm a bit torn about that one12:36
*** doron_afk has quit IRC12:36
*** Daisy has joined #openstack-nova12:36
bauzasjohnthetubaguy: because it means that it could be updating the DB12:36
danpbBobBall: eg if you request a guest has 8 NUMA nodes, then you can only run it on hosts which have 8 physical NUMA nodes, so that restricts the pool of acceptable hosts for the schedular12:36
bauzasjohnthetubaguy: or if not, just having like an in-memory lock12:37
*** doron_afk has joined #openstack-nova12:37
bauzaswhich means it would still have some scale-out problems12:37
johnthetubaguybauzas: I assume its a regular oslo lock, we have banned any more DB locks12:37
*** edmondsw has joined #openstack-nova12:37
bauzasjohnthetubaguy: agreed, but this spec means that if you have 2 processes, then you have 2 possible claims at the same time12:37
mdboothjohnthetubaguy: we've banned db locks?12:37
BobBalldanpb: I see; so usage is tracked by get_available_resource too, right?12:38
mdboothjohnthetubaguy: Got a link, I missed that.12:38
danpbBobBall: yep12:38
bauzasjohnthetubaguy: that looks a possible first step, but then we should think about how to sync 2 claims12:38
BobBallokies12:38
bauzasjohnthetubaguy: given we say that Claims are the Facade for sync'ing12:38
johnthetubaguymdbooth: kinda of yeah, we are saying prefer compare and swap style (or whatever the proper name is)12:38
bauzasjohnthetubaguy: but all of that needs a bit of discussion, I feel12:39
mdboothjohnthetubaguy: Ok, so not a hard ban. Just an expressed preference.12:39
*** zhangjn has joined #openstack-nova12:39
mdboothSorry to butt in :)12:39
*** zhangjn has quit IRC12:39
johnthetubaguymdbooth: yeah, not totally sure if we got that written down, now you mention it12:39
johnthetubaguybauzas: sure, with the code in front of us12:39
*** zhangjn has joined #openstack-nova12:40
*** Daisy has quit IRC12:40
bauzasjohnthetubaguy: I'd even try to see some convo during the summit, provided we have some time for that12:40
bauzasjohnthetubaguy: because we have lots of things trying to see how we could have sched claims12:40
*** lucas-hungry is now known as lucasagomes12:40
bauzasat least jaypipes was thniking about it, too12:40
bauzasand I'm not sure he's aware of that one, too12:40
johnthetubaguybauzas: I feel we need to see the code before we can debate it12:41
*** aswadr has joined #openstack-nova12:41
bauzasjohnthetubaguy: well, unfortunately the BP is a bit vague about what could be the implementation12:41
bauzasso yeah seeing code is good12:41
bauzasbecause I'm +1 on the move12:42
bauzasbut I'm really worried by the implementation details12:42
*** Daisy has joined #openstack-nova12:42
BobBalldanpb: How/where do we verify that vpug_enabled can only be set/used when vgpu_permitted is also set?  Is that logic in the compute driver (i.e. verify presence of both flags when actually allocating the vgpu?)12:42
danpbyeah, you'd usually put that logic  in the virt driver12:43
*** garyk has quit IRC12:43
alaskibauzas: melwitt I'm not really around today so I'll leave the cells meeting details to you.  melwitts plan to run it in case people show up with topics is great.12:43
*** garyk has joined #openstack-nova12:43
bauzasalaski: yup, I saw melwitt's point yesterday night my time12:43
*** Daisy has quit IRC12:43
bauzasalaski: we could just discuss about what we want for Mitaka12:43
*** aix has joined #openstack-nova12:43
BobBallCan do - was just wondering whether there was some additional validation when setting image props or pre-schedule validation somewhere12:43
bauzasalaski: and leave you yelling against us12:44
bauzas:p12:44
snikitinsdague: Hi Sean. I want to ask you a question about oslo.cache patch https://review.openstack.org/#/c/203049/30/nova/utils.py . You wrote that we can't deprecate it until N. Why? I thought deprecation cycle is consist of deprecating something in current release, and removing it in the next release. Am I wrong?12:44
*** dims_ has joined #openstack-nova12:44
*** zhangjn has quit IRC12:44
alaskibauzas: heh, sounds good.  I'm out for the next few days but then my plan is to push some specs up12:44
*** zhangjn has joined #openstack-nova12:45
*** markvoelker has joined #openstack-nova12:46
*** zhangjn has quit IRC12:46
*** Marga_ has quit IRC12:46
lxslimelwitt: would you look kindly on a --json flag to python-novaclient?12:47
*** Marga_ has joined #openstack-nova12:47
*** zhangjn has joined #openstack-nova12:48
sdaguesnikitin: but your patch deprecates without any of our deprecation triggers12:48
sdaguethat's my concern, people are expecting certain deprecation signals12:49
kashyapbauzas: Hi, I only noticed now your brief ping in the backchat, I didn't read all the scrollback...12:49
*** markvoelker_ has joined #openstack-nova12:49
openstackgerritSylvain Bauza proposed openstack/nova-specs: Remove Priorities section for approved specs  https://review.openstack.org/23196412:49
johnthetubaguybauzas: about the spec being vague, it defines the problem space, and the direction, the stuff thats best discussed with code in front of you is left to later, as it should be12:49
*** VikasC has quit IRC12:50
openstackgerritSudipta Biswas proposed openstack/nova: add helper method to obtain image client version  https://review.openstack.org/22215012:50
bauzasjohnthetubaguy: well, I'm a bit opiniated about having specs that give more of a direction, but that's MHO :)12:50
*** raildo-afk is now known as raildo12:51
johnthetubaguybauzas: its a balance, if its a small thing, and the code would answer more questions, do it in the code, its way less overhead, if you want to avoid re-work, feel free to agree more details up front, but its a lot more work12:52
johnthetubaguylxsli: what would --json do?12:52
johnthetubaguylxsli: i am curious12:52
*** xyang1 has joined #openstack-nova12:52
*** markvoelker has quit IRC12:53
johnthetubaguylxsli: bauzas: would be awesome if you can tidy up the whiteboard on this blueprint for me please :) https://blueprints.launchpad.net/nova/+spec/check-destination-on-migrations12:53
johnthetubaguyjust so its clear what needs review, and what has merged, etc12:53
*** markvoelker_ has quit IRC12:54
lxslijohnthetubaguy: will do12:54
bauzasjohnthetubaguy: sure it's a trade-off :)12:54
danpbbauzas: lol, several approved Mitaka specs are refering to *liberty* project priorities12:54
lxslijohnthetubaguy: --json would print JSON output instead of the formatted table12:54
johnthetubaguylxsli: awesome, thank you12:54
lxslijohnthetubaguy: like https://review.openstack.org/#/c/57834 but implemented better :)12:54
danpbmakes me even more sure that removing this section was the right thing todo12:54
johnthetubaguylxsli: ah, so more "script friendly", gotcha12:55
*** whenry has joined #openstack-nova12:55
johnthetubaguylxsli: I was trying to compare that with --debug in my head somehow12:55
*** burgerk has quit IRC12:55
lxslijohnthetubaguy: what's your +1/-1 on the idea?12:55
*** markvoelker has joined #openstack-nova12:56
*** eharney has joined #openstack-nova12:56
*** Sree has quit IRC12:56
*** sahid has quit IRC12:56
*** su_zhang has quit IRC12:56
*** Sree has joined #openstack-nova12:57
*** sahid has joined #openstack-nova12:58
openstackgerritJohn Garbutt proposed openstack/nova: Adds Hyper-V Cluster implementation  https://review.openstack.org/19903712:58
bauzaslxsli: johnthetubaguy: saw zigo's thread ?13:01
*** deepthi has quit IRC13:01
johnthetubaguybauzas: I guess not, which one?13:01
bauzassec13:01
*** Sree has quit IRC13:01
*** jaypipes has joined #openstack-nova13:01
*** rlrossit has joined #openstack-nova13:01
bauzasjohnthetubaguy: lxsli: http://lists.openstack.org/pipermail/openstack-dev/2015-October/076112.html13:02
bauzasbarely related but quite the same usecase13:02
*** links has quit IRC13:02
lxslibauzas: good user story indeed13:02
bauzaslxsli: I'm a bit concerned by the CLI contract tho13:03
johnthetubaguybauzas: yeah, thats a good point... lxsli there are plans to kill python-novaclient so maybe we just work on the openstack cli instead?13:03
lxslibauzas: I'm not sure whether the openstack client supports JSON output, I think it does? So the question is whether we bother adding it to the old one13:03
johnthetubaguybauzas: with always using the latest version, its a bit dodgy13:03
bauzasyeah that of johnthetubaguy13:03
bauzasjohnthetubaguy: yup, my thoughts was that the novaclient API was good, but that CLI wasn't exacly contracting the same way13:04
*** pbrooko has quit IRC13:04
bauzasso, giving a chance for openstackclient to call novaclient API sounds cool to me13:04
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports  https://review.openstack.org/21691713:04
bauzasand adding json support to openstackclient sounds better to me13:04
bauzasbecause if not, then just use your own tool13:05
bauzashaving a command-line interface returning serialized JSON sounds weird, why not directly having novaclient ?13:05
bauzaslxsli: any spec I could put my dirty hands on ?13:06
*** jaypipes has quit IRC13:06
lxsliIt might be the easiest way from Ruby/bash13:06
lxslibauzas: no not yet just floating the idea13:06
danpbbauzas: openstackclient supports CSV output mode13:06
bauzasdanpb: right, see my above link13:06
danpbbauzas: but it would actually make alot of sense to have a JSON mode, as CSV is incredibly poorly spec13:07
bauzasdanpb: I'm just explaining that it should probably be done by openstackclient instead of the Nova CLI13:07
danpbwhen it comes to dealing with escaping & quoting, so many CSV parsers & generators have slightly different semantics13:07
*** mc_nair has joined #openstack-nova13:07
bauzasdanpb: agreed with you13:07
danpbJSON by comparison is pretty clearly defined and has good interoperability13:07
bauzassure13:07
johnthetubaguy+1 all that13:08
lxsliOK I thought the openstackclient already did JSON but if it doesn't I'm happy to look at that first13:08
bauzasI'm not advocating for not having JSON support, just that it should be done in openstackclient IMHO13:08
danpbyeah,13:08
danpbdoesn'pt make much sense to it in novaclient if openstackclient is the future direction13:08
lxsliAre we actively not adding features to novaclient then? IE would having it in both be better than just in openstackclient?13:09
bauzaslxsli: to be clear, there are 2 things13:09
lxslis/better/worse/13:09
bauzas#1 novaclient is the python API13:09
bauzas#2 the nova CLI is the shell script wrapping the pythonclient13:10
lxsliYes that's clear13:10
bauzaslxsli: AFAIK, the plan is to support novaclient but leave openstackclient call it directly13:11
openstackgerritAlexey I. Froloff proposed openstack/nova: Fix more inconsistency between Nova-Net and Neutron  https://review.openstack.org/22627413:11
lxsliI expect novaclient itself returns data structures and it's nova CLI which formats into a table13:11
bauzasthat's what I thought was the consensus in YVR or Paris, I don't remember correctly13:11
bauzaslxsli: yup13:11
lxsliso this is a "nova CLI"/"openstackclient" feature13:11
bauzasnot sure what the slash means, but yes13:12
lxslihappy to add it to the latter first, just wondering if there's active opposition to also adding it to "nova CLI"13:12
lxslislash meant 'or'13:12
bauzaswell, I leave that (adding something to Nova CLI) to other folks like melwitt and Vek_ :)13:12
*** signed8bit has joined #openstack-nova13:13
bauzasI'm just saying that openstackclient seems the first fit for it :)13:13
lxsliOK, agreed on that13:13
*** signed8bit has quit IRC13:13
*** doron_afk has quit IRC13:13
*** doron_afk has joined #openstack-nova13:14
lxslibauzas: where do specs for python-openstackclient go, do you know offhand please?13:15
*** fawadkhaliq has joined #openstack-nova13:15
*** mdrabe has joined #openstack-nova13:16
*** su_zhang has joined #openstack-nova13:16
*** david-lyle has quit IRC13:16
*** mriedem has joined #openstack-nova13:17
edleafelxsli: there's also this: https://wiki.openstack.org/wiki/SDK-Development/PythonOpenStackSDK13:17
edleafelxsli: it's supposed to replace the python module part of novaclient, but it's been dragging without a lot of momentum lately13:17
bauzaslxsli: I suspect it's old-style https://blueprints.launchpad.net/python-openstackclient13:18
mriedemedleafe: and we were told at the yvr summit not to use the sdk in the server projects13:18
*** jichen has quit IRC13:18
lxslibauzas: thanks13:18
edleafemriedem: we should use novaclient instead?13:19
bauzasmriedem: yeah that13:19
mriedemedleafe: i guess? if you're heat or ceilometer.13:19
bauzasmriedem: I remember that discussion, forgot if it was in yvr or paris13:19
mriedemor write your own service layer that talks to the rest api like tempest does13:19
bauzas+113:19
briancurtinYVR, and i didn’t understand it at all, but someone said it and everyone immediately agreed13:19
bauzasthe API bindings are there13:19
lxsliI remember us agreeing to consume Neutron's API raw at YVR13:20
mriedemi don't understand it either13:20
bauzasto be used by other projects13:20
mriedemwhat good is an sdk if the server projects can't use it13:20
*** fawadkhaliq has quit IRC13:20
bauzasI thought the whole idea was to have one single CLI for the user, but as many as necessary bindings for the services13:20
mriedemlxsli: yeah i had a todo to go off and look at that as an alternative to beagles' spec about wrapping the neutronclient stuff in nova13:21
edleafemriedem: there was an inefficiency with keystone that was a problem for OSC, but I don't remember any other specific criticisms13:21
jrollmriedem: https://xkcd.com/927/ obviously13:21
bauzasoh that13:21
bauzasI know that xkcd number by heart13:21
lxslidtroyer: ping13:21
jrolllol13:21
johnthetubaguyjroll: I thought of you when reading a spec yesterday about resource pool tracking13:22
jrolloh?13:22
johnthetubaguyjroll: its about tracking groups of resources, like shared storage thats shared across multiple nova-computes13:23
*** annegentle has joined #openstack-nova13:23
jroll:o13:23
* jroll finds this spec13:23
*** dustins has joined #openstack-nova13:23
johnthetubaguyjroll: I might have the link handy, one sec13:23
bauzasjroll: http://www.commitstrip.com/en/2015/09/21/me-my-code-and-i/ also appies13:23
jrollI think we talked about this a bit in minnesota yeah?13:23
bauzasapplies13:23
johnthetubaguyjroll: yeah, https://review.openstack.org/#/c/225546/13:23
*** signed8bit has joined #openstack-nova13:24
jrollbauzas: heh, I like13:24
jrolljohnthetubaguy: thanks13:24
*** rdopiera is now known as rdopiera_afk13:24
bauzasjohnthetubaguy: could you put the -W button to that spec ? it could be confusing13:25
*** ctrath has joined #openstack-nova13:25
johnthetubaguybauzas: there is the big WIP bit a the top13:25
* bauzas *facepalm*13:25
*** kmartin has quit IRC13:25
bauzasthe biggest it is, the less I can see it13:26
bauzasthe lesser*13:26
johnthetubaguybauzas: heh13:26
lxslithe bigger it is, the less I can see it :)13:26
bauzasrah13:28
bauzasplus c'est gros, moins je le vois13:28
bauzasnah13:28
*** ijuwang has joined #openstack-nova13:28
lxsliEnglish: there are no ruels13:29
jrolljohnthetubaguy: interesting read, I'm not sure I want to depend on it but it could be useful13:31
mriedemdanpb: what do you think about moving https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix into the nova devref?13:31
danpbmriedem: sure, we could do that13:31
mriedemi like the change policy thing you added13:32
johnthetubaguyjroll: yeah, it might be more about the longer term thing13:32
jrollnod13:32
*** david-lyle has joined #openstack-nova13:32
DRoBeR<openstackgerrit> Flavio Percoco proposed openstack/nova-specs: (follow up) Nova Image v1 -> v2 <---- I'm actually having some issues trying to get "nova image-list" due to nova using image V1 instead of V2. :/ Is there any way I can set nova to get that through API V2, guys?13:32
danpbmriedem: we should probably incorporate some of the background rationale/info from that email of mine you quoted too13:32
mriedemthe ML from may?13:33
mriedemwhen we bumped to 0.10.2?13:33
*** smatzek has quit IRC13:33
*** tpeoples has joined #openstack-nova13:33
danpbmriedem: yeah13:33
pkholkinsdague: Hello Sean. We can't mark the option for example like 'deprecated_for_removal' because we will get conflict with the same option and tests will start fail13:35
sdaguepkholkin: right, I understand13:35
pkholkindo you have some suggestions?13:35
sdaguehowever, that seems like a big problem13:35
*** diogogmt has quit IRC13:35
sdagueno, I don't, that's why I raised it as an issue, and also poked on -oslo about it13:35
sdaguebecause this seems problematic13:35
pkholkinalso we can't even change 'help' message13:37
*** signed8bit has quit IRC13:38
*** LukeHinds has joined #openstack-nova13:38
pkholkinsdague: we don't have a good idea how to make it more clear for users13:38
openstackgerritAndrea Rosa proposed openstack/nova-specs: Allow more instance operations during live migration  https://review.openstack.org/22619913:39
mriedemhuh, seems we should have this in tree https://review.openstack.org/#/c/229013/13:39
*** thumpba has joined #openstack-nova13:39
pkholkinsdague: maybe we should add warning here? https://review.openstack.org/#/c/203049/30/nova/servicegroup/drivers/mc.py13:39
*** diogogmt has joined #openstack-nova13:39
pkholkinat the same place where the note13:40
sdaguepkholkin: possibly, we need something13:40
*** Marga_ has quit IRC13:41
*** signed8bit has joined #openstack-nova13:41
pkholkinmriedem: Good day Matt! Could you participate in our discussion about oslo.cache too?) https://review.openstack.org/#/c/203049/3013:42
pkholkinwe are speaking about good deprecating of the option there13:42
*** njirap has quit IRC13:43
*** perci has joined #openstack-nova13:43
pkholkinsdague, mriedem: we can add warning message in mc.py near the note, will it be ok?13:44
*** vladikr has joined #openstack-nova13:44
openstackgerritDan Smith proposed openstack/nova: Remove obj_relationships from objects  https://review.openstack.org/23127113:46
openstackgerritDan Smith proposed openstack/nova: Clean up tests for dropping obj_relationships  https://review.openstack.org/23127013:46
openstackgerritDan Smith proposed openstack/nova: Fix InstanceV1 backports to use context  https://review.openstack.org/23164913:46
openstackgerritDan Smith proposed openstack/nova: DNM: Canary test the backporting from a liberty compute  https://review.openstack.org/23148613:46
openstackgerritDan Smith proposed openstack/nova: Fix up Service object for manifest-based backports  https://review.openstack.org/23126913:46
openstackgerritDan Smith proposed openstack/nova: DNM: Drop Instance v1.x support  https://review.openstack.org/22069013:46
openstackgerritDan Smith proposed openstack/nova: Drop unused obj_to_primitive() override  https://review.openstack.org/23168613:46
*** links has joined #openstack-nova13:46
*** annegentle has quit IRC13:46
sdaguepkholkin: lets see what that looks like13:47
dansmiththis last one ^^^ is blocking an o.vo bug fix from landing, which we need to get released so we don't break if we have to bump an object version in a specific way13:47
mriedempkholkin: in a minute13:48
*** Marga_ has joined #openstack-nova13:48
mriedemis it odd that archive_deleted_rows doesn't attempt to sort the tables based on foreign key constraints? https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L596713:48
mriedemit just gets all of the tables from the nova db models module https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L604313:49
*** signed8bit has quit IRC13:50
bauzashey nova-drivers, some nit patch for removing the prio section would be loving a +2/+W https://review.openstack.org/#/c/231964/1 :)13:50
*** mudassirlatif has joined #openstack-nova13:51
*** zhangjn has quit IRC13:52
*** zhangjn has joined #openstack-nova13:52
*** zhangjn has quit IRC13:52
*** zzzeek has joined #openstack-nova13:53
*** zhangjn has joined #openstack-nova13:53
*** zhangjn has quit IRC13:53
*** zhangjn has joined #openstack-nova13:54
openstackgerritPavel Kholkin proposed openstack/nova: Switch to oslo.cache lib  https://review.openstack.org/20304913:54
*** EinstCrazy has joined #openstack-nova13:55
pkholkinsdague, mriedem: patch was updated13:55
*** irenab has quit IRC13:58
openstackgerritAndrea Rosa proposed openstack/nova-specs: Allow more instance operations during live migration  https://review.openstack.org/22619913:58
*** takedakn has quit IRC13:59
*** signed8bit has joined #openstack-nova14:00
*** Piet has joined #openstack-nova14:01
*** breitz has quit IRC14:02
*** breitz has joined #openstack-nova14:02
johnthetubaguymriedem: is the archive thing, the shadow table thing?14:03
mriedemyeah14:03
mriedemthere is an ops patch that basically does all of that https://review.openstack.org/#/c/229013/14:03
mriedemshowed up in the ops ML14:03
*** manishg has joined #openstack-nova14:04
seanw1In nova/compute/manager.py -> _prep_resize() a check is made against instance.host == self.host and if the allow_resize_to_same_host is false an exception is thrown.  The prep_resize executes on the current host of the instance, and I cannot find where the instance.host is updated to the migrating host before the call to prep_resize.  So without the allow_resize_to_same_host set to true, an error is always reported, but the resize complete14:04
seanw1is this a bug or am I missing something?14:04
*** markvoelker_ has joined #openstack-nova14:04
mriedemjwcroppe: ^ i'm sure you can field that one14:04
johnthetubaguymriedem: oops, so thats the first I heard of that repo, interesting14:05
mriedemi have 20 deleted instances in my devstack and ran "nova-manage db archive_deleted_rows --max_rows 10" and it didn't fail but didn't delete anything either14:05
mriedemso it's definitely busted14:05
*** smatzek has joined #openstack-nova14:05
*** burgerk has joined #openstack-nova14:05
johnthetubaguymriedem: eek, we should totally add a functional test for that thing14:06
mriedemyeah...14:06
*** Sree has joined #openstack-nova14:06
mriedemi think there is a semi functional unit test for it14:06
*** diogogmt has quit IRC14:06
mriedemtest_archive_deleted_rows14:07
*** emagana has joined #openstack-nova14:07
johnthetubaguyhmm14:07
*** burgerk_ has joined #openstack-nova14:07
mriedemmysql> select * from nova.shadow_instances;14:07
mriedemEmpty set (0.01 sec)14:07
*** markvoelker has quit IRC14:08
*** thumpba has quit IRC14:08
*** diogogmt has joined #openstack-nova14:08
dansmithjwcroppe: http://xkcd.com/1587/14:08
*** emagana has quit IRC14:09
*** burgerk has quit IRC14:10
*** emagana has joined #openstack-nova14:10
jwcroppedansmith: that's spot on14:11
*** xdccMule[60EX] has joined #openstack-nova14:11
xdccMule[60EX]Ciao a tutti :D14:11
*** alaski is now known as alaski_out14:12
xdccMule[60EX]Hello to all14:12
xdccMule[60EX]anyone here?14:12
*** whenry has quit IRC14:12
*** markvoelker_ has quit IRC14:12
edleafexdccMule[60EX]: you can just ask your question.14:13
*** diogogmt has quit IRC14:13
xdccMule[60EX]Hello, i'm a student of an Italian university14:13
xdccMule[60EX]Prof tell me about OpenStack (little overview) and he want from me14:14
*** thumpba has joined #openstack-nova14:14
xdccMule[60EX]know what's Nova and information about this14:14
*** perci is now known as njirap14:15
xdccMule[60EX]now...i'm finding document in docs.openstack14:15
johnthetubaguydansmith: I meant to catch you about those upgrade bugs, do we want to hold RC2 and get those into RC2?14:15
xdccMule[60EX]but when i see irc chat i think is more simply ask here a document14:15
*** Marga_ has quit IRC14:16
dansmithjwcroppe: I don't eat fish, but it's close yeah14:16
dansmithjohnthetubaguy: you mean the one I mentioned above?14:16
*** Marga_ has joined #openstack-nova14:16
johnthetubaguydansmith: yes, late yesterday I think14:16
edleafexdccMule[60EX]: this IRC channel is for people developing OpenStack Nova. You would probably be better off asking questions in either #openstack or #openstack-10114:17
xdccMule[60EX]ok thanks14:17
edleafexdccMule[60EX]: there are lots of people in those channels who would be very helpful14:17
*** artom has joined #openstack-nova14:18
dansmithjohnthetubaguy: hmm, hang on14:18
dansmithjohnthetubaguy: so actually this one would be really good if we can get it in: https://review.openstack.org/#/c/231649/14:18
dansmithI need to file a bug for it first though14:18
dansmithjohnthetubaguy: that one actually could affect kilo->liberty14:18
johnthetubaguydansmith: thats the one I was thinking yes14:19
*** manishg has quit IRC14:19
dansmithjohnthetubaguy: okay, let me file a bug and jam in in there and maybe we can land those quick14:20
johnthetubaguydansmith: +1 thank you!14:20
*** zhangjn has quit IRC14:20
*** cfriesen has joined #openstack-nova14:20
dansmithjohnthetubaguy: should I target for rc2 or just set the tag?14:20
johnthetubaguydansmith: feel free to target that one right away14:20
*** artom_ has joined #openstack-nova14:21
mriedemis it odd that we don't actually required admin context when calling db.api.archive_deleted_rows?14:21
mriedemwe pass one in but we don't check it14:21
johnthetubaguymriedem: yeah, the API work removed the whole admin context needed in the DB layer thing14:22
johnthetubaguymriedem: but no one has gone back to remove all the elevated bits14:22
mriedemok, was going to say, the context used here is only passed in from the nova-manage command14:22
mriedemand it's completely unused now14:22
johnthetubaguyah, yuck, some tidying up to do around that, its tempting to put that on the low hanging fruit list to get it fixed up14:23
mriedemit sort of seems odd that it was removed from db apis that are only accessed through nova-manage, but i guess we assume only admins have access to nova-manage to start with14:23
johnthetubaguyyeah, its basically file access controls on nova.conf I think14:23
*** artom has quit IRC14:23
raildojohnthetubaguy: hi :) thanks for you comments in the nested quota spec, do you have some time to talk about it?14:24
*** manishg has joined #openstack-nova14:24
openstackgerritDan Smith proposed openstack/nova: Remove obj_relationships from objects  https://review.openstack.org/23127114:24
openstackgerritDan Smith proposed openstack/nova: Clean up tests for dropping obj_relationships  https://review.openstack.org/23127014:24
openstackgerritDan Smith proposed openstack/nova: Fix InstanceV1 backports to use context  https://review.openstack.org/23164914:24
openstackgerritDan Smith proposed openstack/nova: DNM: Canary test the backporting from a liberty compute  https://review.openstack.org/23148614:24
openstackgerritDan Smith proposed openstack/nova: Fix up Service object for manifest-based backports  https://review.openstack.org/23126914:24
openstackgerritDan Smith proposed openstack/nova: DNM: Drop Instance v1.x support  https://review.openstack.org/22069014:24
johnthetubaguyraildo: yes14:24
*** manishg has quit IRC14:24
*** tsekiyama has joined #openstack-nova14:25
johnthetubaguydansmith: sweet, +W, I guess we should line up the backport14:26
dansmithjohnthetubaguy: yep, just a sec14:27
*** xdccMule[60EX]^8 has joined #openstack-nova14:27
*** xdccMule[60EX]^8 has left #openstack-nova14:27
johnthetubaguydansmith: no rush, needs to merge in master first anyways, I am getting ahead of myself14:27
raildojohnthetubaguy: great, so about the "dedicated" resoucers, it's something complicated, beacuse I agree with your comment, it's something that I don't want and I had discussed in the last summit with the cinder folks...but they decided to implement on this way14:28
raildojohnthetubaguy: so if we want to maintain the same behaviout for cinder, this will be a tricky point.14:29
raildobehaviour*14:30
dansmithjohnthetubaguy: when we're ready: https://review.openstack.org/23203314:30
*** artom_ is now known as artom14:30
*** xdccMule[60EX] has quit IRC14:31
raildojohnthetubaguy: maybe we can have both features as a config option? something that we can enabled...14:31
openstackgerritJesse J. Cook proposed openstack/nova: xenapi_device_id integer, expected string  https://review.openstack.org/23170314:31
*** ZZelle_ has quit IRC14:32
*** markvoelker has joined #openstack-nova14:32
johnthetubaguydansmith: ack, thank you!14:33
*** pm90_ has joined #openstack-nova14:33
johnthetubaguyraildo: so I don't like a config that changes API behaviour, its terrible for discovery14:34
johnthetubaguyraildo: but I am OK supporting both options in the API, if that makes sense?14:34
johnthetubaguyraildo: the key thing, is you can do the same successful API calls with both cinder and nova, I am not questioning that bit14:35
johnthetubaguyraildo: I just think nested quotas should feel the same as regular quotas, at least by default14:35
raildojohnthetubaguy: ++ totally agree with you14:36
*** fawadkhaliq has joined #openstack-nova14:37
*** pm90__ has joined #openstack-nova14:38
raildojohnthetubaguy: I'm trying implement in a way that willhave a smaller impact on the API, like to update a quota for a subproject, You just have to pass a target-project like happend on nova in others API calls14:38
*** nikhil_k_ is now known as nikhil14:38
*** EinstCrazy has quit IRC14:39
raildobut to handle with both situations we have to be more intrusive, adding something like "?overcommit"14:40
raildoin the API call14:40
*** csd has joined #openstack-nova14:40
*** pm90_ has quit IRC14:40
bauzasdansmith: so I'm reviewing https://review.openstack.org/#/c/231269/5/nova/objects/service.py,cm and I'm a bit unclear14:41
johnthetubaguyraildo: true, well it would be overcommit by default, as thats what the original API call does right?14:41
*** amotoki has joined #openstack-nova14:42
mriedemjohnthetubaguy: here is why db archive didn't work for me, it was lost in the debug log output from devstack: 2015-10-07 14:18:39.265 WARNING nova.db.sqlalchemy.api [req-149506be-fbdd-452c-bd35-76f4ba269bc3 None None] IntegrityError detected when archiving table instances14:42
bauzasdansmith: now, ovoo.obj_make_compatible_from_manifest() is called when primitiving the object, right?14:42
mriedemso yeah, totes busted14:42
dansmithbauzas: yes14:42
johnthetubaguymriedem: nice14:42
sdaguemriedem: yeh, that was reported as an issue back in phili14:42
*** mtanino has joined #openstack-nova14:43
bauzasdansmith: which itself sets the manifest and calls the previous obj_make_compatible()14:43
raildojohnthetubaguy: sounds good for me. or we can use a ?dedicated as a option and overcommit by default14:43
*** nikhil is now known as nikhil_k_14:43
*** agireud has quit IRC14:44
*** nikhil_k_ is now known as nikhil__14:44
dansmithbauzas: ....yeah...14:44
mriedemsdague: right, https://bugs.launchpad.net/nova/+bug/118352314:44
mriedemsuper old14:44
openstackLaunchpad bug 1183523 in OpenStack Compute (nova) "db-archiving fails to clear some deleted rows from instances table" [Medium,Confirmed]14:44
*** nikhil__ is now known as nikhil14:45
mriedemsdague: which has led to operators writing their own archive scripts https://review.openstack.org/#/c/229013/14:45
bauzasdansmith: at the end of the stack, it will call _do_subobject_backport() which will call the object obj_make_obj_compatible14:45
sdagueyeh14:45
mriedemwhich blows14:45
bauzasdansmith: that, I understand14:45
mriedemi'm pretty sure simply revert sorting the tables using sqla metadata, like in migration 267, we can fix the Fkey issue14:45
mriedem*reverse14:45
bauzasdansmith: now, my question is then in https://review.openstack.org/#/c/231269/5/nova/objects/service.py,cm14:46
mriedemalso, the way max_rows is used, seems like a terrible idea14:46
bauzasdansmith: shouldn't we change L129 to call super().obj_make_compatible_from_manifest() to set the manifest for the nested object?14:46
bauzasdansmith: sorry if I took time, I needed to make sure I was understanding the stack14:47
*** moshele has quit IRC14:47
*** lpetrut has quit IRC14:47
*** unicell has joined #openstack-nova14:47
*** claudiub has quit IRC14:48
openstackgerritPaul Carlton proposed openstack/nova-specs: Blueprint to use libvirt storage pools  https://review.openstack.org/23204514:48
*** Murali_ has joined #openstack-nova14:48
*** unicell1 has quit IRC14:49
*** Murali has quit IRC14:49
*** HenryG has quit IRC14:49
*** Murali_ is now known as Murali14:50
*** hogepodge has quit IRC14:50
*** dansmith has quit IRC14:50
*** hogepodge has joined #openstack-nova14:51
*** dansmith has joined #openstack-nova14:51
*** claudiub has joined #openstack-nova14:51
*** stevemar_ has joined #openstack-nova14:51
*** vlaza has quit IRC14:51
*** tpeoples has quit IRC14:52
johnthetubaguyraildo: yeah, something like that, well dedicated=true, although thats tricky, because you need all sub teants to be dedicated to get the performance gain you wanted before14:52
*** HenryG has joined #openstack-nova14:52
*** ssurana has joined #openstack-nova14:52
*** browne has joined #openstack-nova14:53
*** Daisy has joined #openstack-nova14:53
*** tjones has joined #openstack-nova14:53
raildojohnthetubaguy: you think they may be at different drivers?14:54
*** alexschm has quit IRC14:54
* raildo trying to do a brainstorm here :P14:54
*** matrohon has quit IRC14:54
*** rubasov has quit IRC14:54
*** alaski_out has quit IRC14:55
openstackgerritPaul Carlton proposed openstack/nova-specs: Blueprint to use libvirt storage pools  https://review.openstack.org/23204514:56
*** Sree has quit IRC14:57
dansmithbauzas: good catch!14:57
*** Sree has joined #openstack-nova14:57
*** alejandrito has joined #openstack-nova14:57
*** alaski has joined #openstack-nova14:58
*** kmartin has joined #openstack-nova14:58
*** diogogmt has joined #openstack-nova14:59
openstackgerritJesse J. Cook proposed openstack/nova: xenapi_device_id integer, expected string  https://review.openstack.org/23170315:00
bauzasdansmith: so, IIUC, now if a novaobject is not yet providing obj_make_obj_compatible it calls the ovoo base method, which itself calls the object obj_make_compatible right?15:00
*** tpeoples has joined #openstack-nova15:00
bauzasdansmith: so you only need to override obj_make_obj_compatible only when you want to provide the version manifest elsewhere, right?15:00
dansmithbauzas: I could have not changed this method to the new one and used the special attribute,15:01
bauzasdansmith: that's my thought15:01
bauzasdansmith: but since it's a private attribute, I'm fine15:01
dansmithbut I feel like if you need the manifest, it's better to use the new method and get it passed to you15:01
bauzasright15:01
bauzasdansmith: honestly, I prefer your way instead of just trying to call self._version_manifest15:01
dansmithyes, I just need to fix my super call of course!15:02
*** Sree has quit IRC15:02
bauzassure, but like I said, your pattern is better than just getting self._version_manifest15:03
*** diogogmt has quit IRC15:03
bauzasI should leave a comment to that once you land PS615:03
*** diogogmt_ has joined #openstack-nova15:03
bauzasbecause some could argue that you just need to call that attr directly15:03
bauzasdansmith: ^15:03
dansmithbauzas: some would be wrong in that case :)15:04
bauzasheh15:04
*** thangp has joined #openstack-nova15:04
*** agireud has joined #openstack-nova15:05
*** csd has quit IRC15:05
*** electrocucaracha has joined #openstack-nova15:06
mriedemdanpb: you must not have seen this https://review.openstack.org/#/c/229577/15:07
*** andrewbogott has quit IRC15:07
*** andrewbogott has joined #openstack-nova15:07
openstackgerritPaul Carlton proposed openstack/nova-specs: Blueprint to migrate libvirt volumes  https://review.openstack.org/23205315:08
danpbmriedem: hmm, no15:08
danpbmriedem: but i think mine is better, as we should not remove the NEXT_MIN_LIBVIRT_VERSION infrastructure15:09
danpbwe need that on an ongoing basis15:09
mriedemyeah yours is more comprehensive15:09
*** tjones has quit IRC15:09
*** garyk has quit IRC15:09
openstackgerritDan Smith proposed openstack/nova: Remove obj_relationships from objects  https://review.openstack.org/23127115:10
openstackgerritDan Smith proposed openstack/nova: Clean up tests for dropping obj_relationships  https://review.openstack.org/23127015:10
openstackgerritDan Smith proposed openstack/nova: DNM: Canary test the backporting from a liberty compute  https://review.openstack.org/23148615:10
openstackgerritDan Smith proposed openstack/nova: Fix up Service object for manifest-based backports  https://review.openstack.org/23126915:10
openstackgerritDan Smith proposed openstack/nova: DNM: Drop Instance v1.x support  https://review.openstack.org/22069015:10
*** manishg has joined #openstack-nova15:10
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Add testresources used by oslo.db fixture  https://review.openstack.org/23205515:12
*** hemnafk is now known as hemna15:12
*** Murali has quit IRC15:12
*** njirap has quit IRC15:13
*** jistr is now known as jistr|mtg15:14
*** armax has joined #openstack-nova15:14
*** tsekiyama has quit IRC15:16
*** Daisy has quit IRC15:17
*** jistr|mtg is now known as jistr15:17
*** claudiub has quit IRC15:17
*** Daisy has joined #openstack-nova15:17
*** subscope has quit IRC15:18
*** Daisy has quit IRC15:19
*** Daisy has joined #openstack-nova15:19
*** njirap has joined #openstack-nova15:20
mriedemdanpb: you don't have anything up for review to update the hypervisor support matrix about required min libvirt version for admin password support do you? if not, i'll do that.15:20
mriedemi kind of wish that wasn't landed15:20
openstackgerritMatt Riedemann proposed openstack/nova: Remove unused context parameter from db.archive_deleted_rows* methods  https://review.openstack.org/23205715:21
*** claudiub has joined #openstack-nova15:21
*** mdrabe has quit IRC15:22
danpbmriedem: nope, not don't anything15:23
danpbargh15:23
danpbnot done anything15:23
openstackgerritPaul Carlton proposed openstack/nova-specs: Blueprint to use libvirt storage pools  https://review.openstack.org/23204515:25
*** EinstCrazy has joined #openstack-nova15:25
*** subscope has joined #openstack-nova15:28
*** annegentle has joined #openstack-nova15:29
*** electrocucaracha has quit IRC15:30
*** electrocucaracha has joined #openstack-nova15:31
dansmithbauzas: https://review.openstack.org/#/c/231269/6/nova/objects/service.py,cm15:31
bauzasdansmith: yup, just commenting a few nits and double-checking the test15:31
*** jdurgin1 has joined #openstack-nova15:31
dansmithcool15:32
jwcroppemriedem: any chance we can pull https://bugs.launchpad.net/nova/+bug/1490837 into L?15:32
openstackLaunchpad bug 1490837 in OpenStack Compute (nova) "Sporadic incoherent metrics when driver.get_host_cpu_stats takes longer than 1 second to execute" [Medium,Fix committed] - Assigned to Joe Cropper (jwcroppe)15:32
mriedemumm15:33
mriedemsdague: ^ what do you think about that for stable?15:33
*** dims_ has quit IRC15:34
*** EinstCrazy has quit IRC15:34
*** nkrinner has quit IRC15:36
*** moshele has joined #openstack-nova15:37
*** mudassirlatif has quit IRC15:37
*** mpaolino has joined #openstack-nova15:38
*** mdrabe has joined #openstack-nova15:39
openstackgerritAlexey I. Froloff proposed openstack/nova: Fix more inconsistency between Nova-Net and Neutron  https://review.openstack.org/22627415:40
*** kaisers has quit IRC15:40
*** njirap has quit IRC15:40
lxslijohnthetubaguy: if I wanted to adopt an old bp, should I just make a new one?15:40
sdaguemriedem: what's the impact if we don't bring it back?15:41
mriedemjwcroppe: ^15:41
dansmithmetrics will be skewed a little15:41
dansmithwhen we're under high load15:41
*** david-lyle has quit IRC15:41
dansmithI think15:41
dansmithnot the end of the world, IMHO, just means a scheduler filter might make a less than optimal decision, AFAIK15:42
dansmithwe could wait for a bug where someone says it's really a problem15:42
*** david-lyle has joined #openstack-nova15:42
jwcropperight, metrics are messed up when systems under load15:42
bauzasdansmith: just one comment puzzling me in https://review.openstack.org/#/c/231269/6/nova/tests/unit/objects/test_service.py,cm15:42
johnthetubaguylxsli: ideally no, just re-use the old one15:42
*** links has quit IRC15:42
mriedemwell, there was a bug for the metrics thing https://bugs.launchpad.net/nova/+bug/149083715:43
openstackLaunchpad bug 1490837 in OpenStack Compute (nova) "Sporadic incoherent metrics when driver.get_host_cpu_stats takes longer than 1 second to execute" [Medium,Fix committed] - Assigned to Joe Cropper (jwcroppe)15:43
johnthetubaguylxsli: I should be able to assign it to you, I think15:43
bauzasdansmith: since you call obj_tree_get_versions with ComputeNode being a nested object, you should already have the key set unless I'm wrong?15:43
mriedemi just wasn't on the master review so wasn't sure how much is in there that is unsafe for backporting to stable15:43
mriedemsince it's not a trivial change15:43
bauzasdansmith: or do you want to force a specific version ?15:43
dansmithmriedem: right, that's his bug15:43
dansmithmriedem: I meant a bug from someone running liberty where they say "this is important because ..."15:44
*** kaisers has joined #openstack-nova15:44
*** dims_ has joined #openstack-nova15:44
mriedemsomeone other than jwcroppe :)15:45
jwcroppelol15:45
bauzasmriedem: sdague: I feel it's not really mandatory for liberty15:45
*** marcusvrn_ has quit IRC15:45
bauzasspeaking of bug 149083715:45
openstackbug 1490837 in OpenStack Compute (nova) "Sporadic incoherent metrics when driver.get_host_cpu_stats takes longer than 1 second to execute" [Medium,Fix committed] https://launchpad.net/bugs/1490837 - Assigned to Joe Cropper (jwcroppe)15:45
dansmithmriedem: no offense to jwcroppe, you get my point, right? :)15:45
mriedemunless you have a scheduler filter or metrics plugin out of tree that is completely fubar w/o the fix15:45
mriedemright?15:45
bauzasmriedem: there is only one filter15:45
bauzasmriedem: and one weigher15:45
bauzasmriedem: consuming those stats15:45
mriedemyou can't drop anything in there?15:45
jwcroppedansmith: I thought our tentacle-less bond counted for something, man!15:45
bauzasmriedem: the filter being there just for checking that we have the metrics15:46
dansmithjwcroppe: that sounds.. creepy :)15:46
jwcroppe+115:46
bauzasmriedem: most of the work is done by weighers15:46
mriedemare those pluggable?15:46
jwcroppeanyway, if mriedem feels it's too risky for L, that's fine - thought I'd ask15:46
bauzaswhich means that if the weigher is a bit buggy, then it's not really bad15:46
lxslijohnthetubaguy: great! can you assign https://blueprints.launchpad.net/nova/+spec/migrate-libvirt-volumes to paul-carlton2 please?15:46
bauzasjust that it could be not exacly the right compute15:46
jwcroppemriedem: I agree life won't end because it's not in L, so I'm cool with 'dat15:46
mriedemdis and dat?15:47
bauzasso, backporting to Liberty sounds a bit picky15:47
johnthetubaguylxsli: done, it will need a spec, etc, clearly, but that should be assigned now15:47
lxslijohnthetubaguy: yep he's working on those, thank you15:48
johnthetubaguylxsli: :)15:48
dansmithbauzas: replied, thanks for paying attention to the details15:48
*** Marga_ has quit IRC15:48
*** andreykurilin__ has joined #openstack-nova15:49
bauzasdansmith: I see, so you're overriding the key to an older version15:49
dansmithbauzas: yeah15:49
bauzasthat was my first thought hence the +115:49
andreykurilin__sdague: hi! can you look at small change to project-config? https://review.openstack.org/#/c/231877/15:49
bauzasbut I needed double-checking15:49
dansmithbauzas: in other tests, we had to do majorly confusing antics to get a single process to think it needed to backport something.. the manifest makes it super easy15:49
*** doron_afk has quit IRC15:50
bauzasokay, I see the relationship ('1.9', '1.10'), gotcha15:50
bauzasthat's a pretty old version but I'm fine15:50
* bauzas should read the explicit tuples before asking silly questions15:50
*** READ10 has joined #openstack-nova15:51
*** artom has quit IRC15:52
*** artom has joined #openstack-nova15:52
*** READ10 is now known as READ10|lunch15:53
openstackgerritMatt Riedemann proposed openstack/nova: Use DBReferenceError in archive_deleted_rows_for_table  https://review.openstack.org/23206615:53
*** jdurgin1 has quit IRC15:54
*** david-ly_ has joined #openstack-nova15:55
*** david-lyle has quit IRC15:55
*** david-ly_ has quit IRC15:56
*** david-lyle has joined #openstack-nova15:56
*** diana_clarke has quit IRC15:58
*** flwang has quit IRC15:58
*** dims__ has joined #openstack-nova15:59
*** subscope has quit IRC15:59
*** fawadkhaliq has quit IRC16:00
*** fawadk has joined #openstack-nova16:00
*** electrocucaracha has quit IRC16:02
*** manishg has quit IRC16:02
*** dims_ has quit IRC16:03
*** pm90__ has quit IRC16:04
*** tsekiyama has joined #openstack-nova16:04
*** jistr has quit IRC16:05
*** artom has quit IRC16:07
*** artom has joined #openstack-nova16:08
*** jlanoux has quit IRC16:08
*** diana_clarke has joined #openstack-nova16:10
*** scheuran has quit IRC16:10
dims__dansmith: mriedem: one of you please bless this https://review.openstack.org/#/c/232055/ will help prevent nova breakage with next oslo.db release16:11
*** lpetrut has joined #openstack-nova16:12
dansmithyep16:12
*** annegentle has quit IRC16:12
*** annegentle has joined #openstack-nova16:13
mriedemdims__: why doesn't nova just require oslo.db[fixtures]?16:13
*** marcusvrn_ has joined #openstack-nova16:13
dims__mriedem: y that would work too16:13
*** Marga_ has joined #openstack-nova16:14
mriedem-116:14
mriedemanyway, cinder meeting16:14
*** su_zhang has quit IRC16:14
dansmithmriedem: dims__: isn't this just a pre-gr-sync?16:14
dims__mriedem: i am afraid to do that as i don't know what tools will break16:14
dansmithdims__: mriedem just wanted to get me a disagreement stat.. he loves that16:15
dims__mriedem: as i don't see that syntax yet16:15
dims__lol :)16:15
mriedemi believe dhellmann said that the lib would need a major version bump for the extras stuff16:15
mriedemand you'd have to opt into using those, so would require nova to use a new min version of oslo.db to use oslo.db[fixtures]16:15
mriedemif that was released in a minor version, then well16:16
mriedemshrug?16:16
mriedemanyway, this is precisely why we started adding the extras stuff to the oslo libs16:16
dims__mriedem: i have not released anything yet. just trying things with master of all oslo branches and filing defensive reviews16:16
mriedemok, we should probably circle back with dhellmann at some point16:16
dims__and lifeless...16:17
dims__worries about the proposal bots/edit constraints etc.16:17
*** mudassirlatif has joined #openstack-nova16:17
dims__s/worries/worried/16:17
*** nic has joined #openstack-nova16:17
*** unicell has quit IRC16:19
mriedem"This should trigger a major version bump for the library, when it lands." was his comment in https://review.openstack.org/#/c/217347/16:19
*** dims__ is now known as dims16:19
dimsmriedem: y, not worried about version bump16:20
*** amotoki has quit IRC16:20
dimsmriedem: do i have to add an entry in g-r for this? will edit-constraints support the square brackets syntax16:20
dimsetc...16:20
mriedemdims: i'm not totally sure about that16:21
clarkbyou dont put that in global requirements or constraints16:21
mriedemi guess i'd see if the job fails on the nova change that tries to use it16:21
dimsmriedem: ack good point16:22
clarkbglobal reqs has individual reqs not aggregates16:22
mriedemso: oslo.db[fixtures]>=3.0?16:22
dimsclarkb: just test-requirements i guess16:22
clarkbso you want to require optional extras?16:22
clarkbwhy not unoptional them then?16:23
*** zzzeek has quit IRC16:23
dansmith"unoptional them then" -> awesome16:23
dimsmriedem: i am also using the "-e " syntax to try latest master https://github.com/dims/nova/blob/master/requirements.txt#L1316:23
*** agireud has quit IRC16:24
dimsguess, i can do a oslo.db major version bump, make a release and we have 2 ways to fix nova after that16:24
clarkbso I think you dont have to worry about constraints at all as it looks at individual items only. global reqs may need to grok it if you list that as a dep somehwere16:24
clarkbI think it would just work in global reqs16:24
openstackgerritBalazs Gibizer proposed openstack/nova-specs: Versioned notification API  https://review.openstack.org/22475516:25
dimsclarkb: ack16:25
*** agireud has joined #openstack-nova16:27
*** apoorvad has joined #openstack-nova16:31
*** pilgrimstack has left #openstack-nova16:31
*** electrocucaracha has joined #openstack-nova16:34
openstackgerritMatt Riedemann proposed openstack/nova: Log DBReferenceError in archive_deleted_rows_for_table  https://review.openstack.org/23209716:39
*** signed8bit has quit IRC16:40
*** ildikov has joined #openstack-nova16:43
*** thangp has quit IRC16:44
*** pm90_ has joined #openstack-nova16:44
*** belmoreira has joined #openstack-nova16:45
openstackgerritClaudiu Belu proposed openstack/nova: Expands python34 unit tests list  https://review.openstack.org/23194816:49
*** tonytan4ever has joined #openstack-nova16:49
*** pm90__ has joined #openstack-nova16:49
*** Murali has joined #openstack-nova16:51
jlkmordred: hey, is there a way to turn off caching in shade? It's making things extremely difficult to walk through a call with pdb16:51
*** achanda has joined #openstack-nova16:51
*** pm90_ has quit IRC16:52
*** danpb has quit IRC16:52
*** ZZelle has joined #openstack-nova16:53
openstackgerritMatt Riedemann proposed openstack/nova: Make archive_deleted_rows_for_table private  https://review.openstack.org/23210416:55
*** alaski is now known as alaski_out16:55
*** Marga_ has quit IRC16:55
*** exploreshaifali has joined #openstack-nova16:56
*** garyk has joined #openstack-nova16:57
openstackgerritMike Fedosin proposed openstack/nova: Bringing Glance v2 images to a uniform format  https://review.openstack.org/23169616:57
openstackgerritClaudiu Belu proposed openstack/nova: Replaces contextlib.nested with test.nested  https://review.openstack.org/23210816:58
*** Murali has quit IRC16:58
*** nic has quit IRC16:58
mordredjlk: it should default to off16:59
melwittmriedem: do your patches mean you tried out archive_deleted_rows recently?16:59
mordredjlk: although, hrm the decorators will make the stacks weird ... lemme try something17:00
*** manishg has joined #openstack-nova17:00
jlkyeah I had to put my pdb trace further into the code to bypass the decorator rabbit hole17:00
mordredjlk: boarding plane ... will have you a patch in an hour17:00
*** moshele has quit IRC17:01
*** rk4n has joined #openstack-nova17:01
jlkno rush, no worries.17:02
*** penick has joined #openstack-nova17:02
jlkI think keystoneclient bump fucked us though17:02
*** unicell has joined #openstack-nova17:02
*** claudiub has quit IRC17:03
*** mpaolino has quit IRC17:03
*** yamahata has joined #openstack-nova17:04
*** rk4n has quit IRC17:05
*** nic has joined #openstack-nova17:06
*** rk4n has joined #openstack-nova17:06
*** ctrath has quit IRC17:06
openstackgerritMike Fedosin proposed openstack/nova: Bringing Glance v2 images to a uniform format  https://review.openstack.org/23169617:07
mriedemmelwitt: this morning yeah17:08
mriedemmelwitt: it's busted as ever17:08
mriedemmelwitt: posted to the dev ML, and doing some cleanup before i actually dug into the reverse table sort17:08
melwittmriedem: bummer :(17:08
*** krtaylor has quit IRC17:09
openstackgerritMike Fedosin proposed openstack/nova: use helper method to remove version hardcoding  https://review.openstack.org/22857817:10
*** achanda has quit IRC17:11
*** su_zhang has joined #openstack-nova17:11
mriedemi kind of like that the novaclient functional tests rely on a devstack so we can boot a real instance17:12
*** ccarmack1 has quit IRC17:12
mriedemi'm kind of wishing the nova functional tests required a devstack now, or if we had a nova integration or dsvm tests or something17:12
*** READ10|lunch is now known as READ1017:12
mriedembecause i think the only good way to probably test db archive is with devstack, and we don't have that in the nova tree, tempest wouldn't take it and it doesn't fit in novaclient17:13
mriedemcould maybe do it in nova functional tests if we monkey with sqlite17:13
*** tsekiyama has quit IRC17:13
*** achanda has joined #openstack-nova17:13
*** ctrath has joined #openstack-nova17:14
*** sahid has quit IRC17:14
*** mnestratov has quit IRC17:18
*** ccarmack has joined #openstack-nova17:19
mriedemdansmith: johnthetubaguy: if i want to make a change to nova-manage db archive_deleted_rows to add a --verbose option and dump the tables that were touched along with how many rows were deleted before exit, does that require a specless bp?17:19
dansmithmriedem: it'd be like a single patch right?17:20
mriedemyeah17:20
mriedemjust stash results in a dict and pretty print them on exit17:20
dansmithI'd say no, but I think he might want it for recording the feature for release notes, etc17:20
mriedemok17:20
*** subscope has joined #openstack-nova17:21
openstackgerritNikola Dipanov proposed openstack/nova-specs: spec document for sriov-physical-function-passthrough blueprint  https://review.openstack.org/21247217:22
*** krtaylor has joined #openstack-nova17:23
*** lucasagomes is now known as lucas-dinner17:23
*** ijw has joined #openstack-nova17:24
mriedemdansmith: https://blueprints.launchpad.net/nova/+spec/print-table-archived-rows17:24
*** changbl has quit IRC17:25
mriedemthanks17:26
*** mudassirlatif has quit IRC17:28
*** achanda has quit IRC17:28
*** jcoufal has quit IRC17:28
*** jcoufal has joined #openstack-nova17:29
*** e0ne has quit IRC17:30
mriedemheh, nova-manage vm list shows all instances, deleted or not17:31
*** Piet has quit IRC17:31
markus_zmriedem: IIRC there was a but about the nova-manage issue, sec...17:32
*** Marga_ has joined #openstack-nova17:33
markus_zmriedem: https://bugs.launchpad.net/nova/+bug/149484117:33
openstackLaunchpad bug 1494841 in OpenStack Compute (nova) "nova-manage vm list active" [Low,In progress] - Assigned to Raga lahari (raga-lahari)17:33
* markus_z has to leave, bye17:34
*** markus_z has quit IRC17:34
openstackgerritMatt Riedemann proposed openstack/nova: Make archive_deleted_rows_for_table private  https://review.openstack.org/23210417:34
*** DRoBeR has quit IRC17:34
mordredjlk: oh? I take it you're trying ot debug to figure out why it broke? (I certainly did not intend to break things - but we did move to keystoneauth1 for getting Sessions so the auth plugins might have a slight difference in behavior that is affecting you but which we did not account for)17:34
johnthetubaguymriedem: dansmith: I don't mind either way for that little one, its nice to advertise the feature though, and I would approve a blueprint if you add it.17:36
mriedemdan got it17:36
*** lpetrut has quit IRC17:37
jlkmordred: yeah. shade glance image list is passing along a "insecure" kwarg, that when it eventually hits keystone client code, the client code throws an exception on unexpected kwarg "insecure"17:37
mordredhrm17:38
jlkbut I'm in a call now and can't debug at the moment17:38
mordredjlk: I'm going to loop in Shrews in a different channel - what version of shade are you on? latest?17:39
jlkmordred: not sure17:39
mordredkk17:39
*** signed8bit has joined #openstack-nova17:39
jlkon ab1c566cb8afbf477492180e2cc8257817a06893 it seems17:39
jlkwhich, looks "oldish"17:39
mordredwe're not seeing that in infra which uses shade for images17:39
mordredoh. yes. very17:40
*** electrocucaracha has quit IRC17:40
*** vilobhmm11 has joined #openstack-nova17:40
mordredif you got updated client libs and old shade I think that's untested - but makes me think we should maybe think about how to express that17:40
jlkyeah I had my automation pinned for various reasons. Trying master17:41
*** zul has quit IRC17:42
*** vilobhmm111 has joined #openstack-nova17:42
*** zul has joined #openstack-nova17:42
*** yamahata has quit IRC17:42
*** vishwanathj has joined #openstack-nova17:43
*** LukeHinds has quit IRC17:45
*** vilobhmm11 has quit IRC17:45
*** rlrossit has quit IRC17:46
*** electrocucaracha has joined #openstack-nova17:47
*** jichen has joined #openstack-nova17:48
*** electrocucaracha has quit IRC17:51
*** dims has quit IRC17:52
*** dims has joined #openstack-nova17:53
*** mriedem has quit IRC17:54
*** raildo is now known as raildo-afk17:59
*** achanda has joined #openstack-nova18:00
*** rlrossit has joined #openstack-nova18:00
*** dims_ has joined #openstack-nova18:00
*** e0ne has joined #openstack-nova18:03
*** pkoniszewski has quit IRC18:03
*** dims has quit IRC18:04
*** raildo-afk is now known as raildo18:04
*** xek has quit IRC18:07
*** _gryf has quit IRC18:07
*** rfolco has quit IRC18:07
*** xek has joined #openstack-nova18:08
*** dims_ has quit IRC18:08
*** artom has quit IRC18:10
*** pkoniszewski has joined #openstack-nova18:10
*** tsekiyama has joined #openstack-nova18:11
*** mriedem has joined #openstack-nova18:11
*** rfolco has joined #openstack-nova18:12
*** e0ne has quit IRC18:12
*** ccesario_ has quit IRC18:14
*** su_zhang has quit IRC18:14
*** READ10 has quit IRC18:15
*** belmoreira has quit IRC18:15
*** pixelbeat has quit IRC18:16
*** Daisy has quit IRC18:17
*** Daisy has joined #openstack-nova18:18
*** e0ne has joined #openstack-nova18:18
*** jcoufal has quit IRC18:20
*** pkoniszewski has quit IRC18:20
*** isq_ has quit IRC18:21
*** _gryf has joined #openstack-nova18:22
*** Daisy has quit IRC18:22
openstackgerritDan Smith proposed openstack/nova: Add test of claim context manager abort  https://review.openstack.org/21359118:24
*** pkoniszewski has joined #openstack-nova18:25
dansmithbauzas: ^18:25
*** aix has quit IRC18:26
*** emagana has quit IRC18:30
bauzasdansmith: +1(d18:30
bauzasuh18:30
bauzasFWIW, +1'd even18:30
dansmithcool18:30
*** achanda has quit IRC18:30
*** irenab has joined #openstack-nova18:32
*** su_zhang has joined #openstack-nova18:34
*** isq_ has joined #openstack-nova18:36
*** ociuhandu has quit IRC18:38
*** garyk1 has joined #openstack-nova18:38
*** garyk has quit IRC18:38
openstackgerritGábor Antal proposed openstack/nova: Added appender methods to remove clones from config.py  https://review.openstack.org/23214518:39
*** dims has joined #openstack-nova18:40
*** lalitd has joined #openstack-nova18:40
*** yassine has quit IRC18:41
*** emagana has joined #openstack-nova18:41
lalitdbauzas: hi18:41
openstackgerritMike Fedosin proposed openstack/nova: Bringing Glance v2 images to a uniform format  https://review.openstack.org/23169618:42
*** tsekiyam_ has joined #openstack-nova18:43
*** tsekiyam_ has quit IRC18:43
*** Piet has joined #openstack-nova18:44
*** tsekiyama has quit IRC18:46
*** mtanino has quit IRC18:46
bauzaslalitd: hi18:46
bauzaslalitd: so, could you please point me out the change requiring attention ?18:47
*** electrocucaracha has joined #openstack-nova18:47
lalitdbauzas: ya, actually the grenade and tempest tests are failing in the spec https://review.openstack.org/#/c/213041/18:48
*** tsekiyama has joined #openstack-nova18:48
* bauzas looking18:48
bauzaslalitd: okay, so you have a problem with an upgrade from liberty18:49
bauzassec18:49
lalitdin earlier patch set same failures were there , we had discussion  on that , it was due to configuration missing for api_db in grenade18:49
bauzasmriedem: dansmith: do you know if grenade and n-cpu are updated to use liberty as the original branch ?18:49
dansmithyes18:50
lalitdbauzas: ^18:50
bauzasdansmith: okay, I'll look when the change landed18:50
dansmithbauzas: but I don't think we've landed the patch to actually get the api_db db_sync run yet18:50
bauzaslalitd: lemme see when Jenkins ran18:50
dansmithbecause we haven't used it18:50
dansmithbauzas: https://review.openstack.org/#/c/190399/18:50
lalitdbauzas: cool18:50
bauzaslalitd: see dansmith's change18:51
*** achanda has joined #openstack-nova18:51
*** pixelbeat has joined #openstack-nova18:51
bauzaslalitd: I haven't yet checked the jobs, but until 190399 is merged, that should be normal that it would fail18:52
*** Marga_ has quit IRC18:54
lalitdbauzas: oks, 190399 is on high priority :)18:54
*** ntpttr has joined #openstack-nova18:56
lalitdbauzas: so probably when it ll get merge? any idea18:57
*** Marga_ has joined #openstack-nova18:57
dansmithI will update it for liberty18:57
dansmithif we need it, we can merge it18:57
dansmithbut we need to probably discuss that flavor thing first18:57
*** pixelbeat has quit IRC18:58
bauzaslalitd: yup confirmed http://logs.openstack.org/41/213041/18/check/gate-grenade-dsvm-partial-ncpu/c604874/logs/new/screen-n-api.txt.gz#_2015-09-29_12_56_36_41418:59
*** ctrath has quit IRC18:59
*** ctrath has joined #openstack-nova19:00
*** electrocucaracha has quit IRC19:00
bauzasdansmith: we had a convo in the last cells meeting about the next things, and until alaski agrees or not, we're thinking to work on data online migrations to the api db for the global tables19:01
*** flwang has joined #openstack-nova19:01
lalitdbauzas: thanks , nice job.. so it is connection missing in configuration19:01
*** Marga_ has quit IRC19:02
bauzasthat should be the liberty focus for cells - that and some sched bits (persisting the spec at least, plus if possible running a conductor)19:02
*** achanda has quit IRC19:02
bauzasthat still needs alaski's comments19:02
*** su_zhang has quit IRC19:02
*** ntpttr has quit IRC19:02
*** rdopiera_afk is now known as rdopiera19:02
dansmithbauzas: I feel like the migration from the lower DB to the API one could be done mostly offline and ahead of time19:03
bauzasdansmith: sounds good, we were mostly discussing about the process, ie. how many specs we should be providing19:04
dansmithbauzas: like, an initial sync before restarting anything19:04
dansmithwe should use service_versions to denote this change and stop looking at the non-API database after the minimum has lifted19:04
bauzasdansmith: that would ease the migration, I was thinking about the general pattern of expand/live/contract19:04
bauzasmmm interesting19:05
bauzasmelwitt: ^19:05
dansmithyeah, the reason we want to be careful here is just because it requires hitting two databases I think19:05
*** ntpttr has joined #openstack-nova19:05
dansmithbut we can control whether we look in both places based on the service version minimum19:05
dansmithI think19:06
*** ociuhandu has joined #openstack-nova19:06
* bauzas thinks about that19:06
dansmithdefinitely good discussion material for tokyo.. too bad alaski won't be there :?19:06
dansmither -> :/19:06
*** rdopiera has quit IRC19:07
dansmithI wonder if there is any reason we could not require the initial sync,19:08
dansmithhmm, well, no19:08
dansmithrequires some thinking for sure19:09
bauzasyup19:09
dansmithwe can't do the migration in a transaction, which makes it harder I think19:09
bauzasI mean, we have 2 facades here19:09
bauzasthe object facade, but we also added some db facade19:09
*** matrohon has joined #openstack-nova19:09
dansmithI don't think I get what you just said19:10
bauzassorry19:10
bauzasthinking out loud19:10
*** dims has quit IRC19:10
melwittyeah, it hasn't been clear to me how to move forward with the api database in general. I've been trying to help review the flavor patches and realized I don't know what the plan is19:10
bauzasso the general idea is to move what's api related to the api db, and leave the instance related bits to the existing db19:11
bauzaswhich means that we should care about migrating some objects from the legacy db (now called cell db) to the api db19:12
bauzasand stop having to sync between 2 DBs like we did with cells v1 and all the lovely races we have19:12
mriedemmelwitt: have you ever noticed that novaclient.utils.print_dict is basically the same as novaclient.common.cliutils.print_dict?19:13
bauzasso IIUC, the patch is caring about migrating the data19:13
*** dims has joined #openstack-nova19:13
dansmithbauzas: yeah, we get that19:13
dansmithI'm just thinking about the mechanics19:13
melwittyeah, same19:13
melwittmriedem: I have not19:14
* alaski_out pops in briefly19:14
bauzasalaski_out: go away !19:14
alaski_outI'm still at home, nothing exciting going on yet19:14
bauzashah19:14
bauzasso the mechanics seem to be described in http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/flavor-cell-api.html19:15
*** fawadk has quit IRC19:16
dansmithwell,19:16
alaski_outthe idea we discussed for flavors was to do an online migration as things were accessed, and then include a nova-manage command for final migration19:16
dansmithright I get that19:16
alaski_outmuch the same as the system_metadata -> instance_extra flavor thing19:16
dansmithI was kinda hoping we could avoid having to roll all API and conductor workers19:16
dansmiththis time19:16
dansmithbut I think we'll have to, because we can't have old api workers writing to the old location while we're trying to migrate19:16
*** tonytan4ever has quit IRC19:17
dansmithif we roll api and conductor together, then the old location is read-only and all updates go to the API DB19:17
*** tsekiyama has quit IRC19:17
dansmiththat's relatively trivial, so not a problem19:17
alaski_outwell, we can have old writing to the old place as long as updated workers read from the old place first19:17
dansmithI was just hoping to be able to push the envelope again19:17
bauzasthat's debatable19:17
dansmithalaski_out: yeah, that's just less desirable I think19:17
*** tsekiyama has joined #openstack-nova19:17
alaski_outit is extra lookups19:18
bauzashence the lookup in the spec19:18
dansmithalaski_out: we can't null out the old locations either,19:18
bauzashah, jinxed by alaski_out19:18
dansmithwhich means we have to always read from both places19:18
bauzasyup19:18
bauzasat least the old one19:18
mriedemdims: here is a fun little one https://bugs.launchpad.net/oslo-incubator/+bug/150384119:18
openstackLaunchpad bug 1503841 in python-novaclient "novaclient.utils.print_dict should be replaced with novaclient.common.cliutils.print_dict" [Undecided,New]19:18
alaski_outyes, until some cutoff time.  and a full upgrade would have to happen before that19:18
bauzasyup19:19
dansmithalaski_out: we don't have service records for the api nodes, so we don't really have any indication19:19
dansmithalaski_out: seems like just rolling api and conductor together (like you have to today anyway) is the easiest19:19
alaski_outyeah, that would definitely be easier and avoid a lot of pitfalls19:19
dansmithactually, I guess we have to read from both places all the time?19:19
dansmithhmm19:20
dansmithif we need to list all flavors and some are not migrated,19:20
dansmiththen we have to list from both19:20
alaski_outyes, that's what's been implemented19:20
dansmithhrm19:20
alaski_outit's not ideal19:20
dansmithso we could then have some nova-manage command that does the background sync,19:20
dansmithand then once it's synced, it could delete the flavors from the old db,19:21
dansmithand have our objects stop looking at the old location if there are zero flavors19:21
dansmithand cache that on the first lookup or something19:21
dansmithso that once you're migrated, your overhead is pretty low19:21
alaski_outI think the spec covered most of that, but didn't address when to stop looking at the old db19:22
dansmiththen you could use that same tool as pre-sync to seed the api database and get there quicker19:22
dansmithokay19:22
dansmithI haven't really read it19:22
alaski_outbut that last bit would be a great addition19:22
dansmithoh also I have a super idea19:22
dansmithalaski_out: gtfo of here19:22
dansmithany +2s?19:22
alaski_outlol19:23
alaski_outmy time is limited now, just trying to make the most of it19:23
dansmithalaski_out: stop it19:23
melwittalaski_out: (sorry) for the simple boot with one cell, this is supposed to happen before we deal with finishing the api db?19:23
*** tonytan4ever has joined #openstack-nova19:24
*** annegentle has quit IRC19:24
melwittI got confused thinking that the single cell would be one api db with one cell db19:24
alaski_outmelwitt: right.  we can continue to query for things in the cell db for now, since there's just one19:24
*** annegentle has joined #openstack-nova19:25
*** nelsnels_ has joined #openstack-nova19:25
alaski_outwe can't get to multiple cells until we transition everything to the api db, but we can transition boot and other things along the way19:25
*** Marga_ has joined #openstack-nova19:25
*** vishwana_ has joined #openstack-nova19:25
bauzasright19:25
melwittalaski_out: okay. I had misunderstood that. and for the other global api db tables, do we expect the specs and migrations to be like flavors?19:25
*** nelsnelson has quit IRC19:25
dansmithalaski_out: http://i1.kym-cdn.com/photos/images/facebook/000/588/827/f67.png19:25
*** ijw_ has joined #openstack-nova19:25
*** manishg_ has joined #openstack-nova19:25
*** e0ne has quit IRC19:25
alaski_outdansmith: two seconds and then I swear I'm gone19:26
dansmithTIME19:26
alaski_outfigurative seconds19:26
dansmithalaski_out: I have her number, I'll call her and get you in all kinds of trouble19:27
dansmithdon't make me19:27
*** manishg has quit IRC19:27
alaski_outmelwitt: if they can be like flavors, yes.  with the addition that dansmith suggested.  but we'll have to consider them case by case to ensure that method will work19:27
*** vishwanathj has quit IRC19:27
*** ijw has quit IRC19:27
*** dims has quit IRC19:28
alaski_outmelwitt: I'm going to volunteer dansmith to field any further questions for a few days :)19:28
alaski_outdansmith: alright, I'm out.  no call necessary :p19:28
dansmithyeah, I'm happy to help figure out migrations19:28
* alaski_out goes poof19:28
*** dims has joined #openstack-nova19:28
bauzashehe19:28
dansmithnext time I'm going to pull out my chanops and ban his ass19:29
melwittalaski_out: thanks19:29
*** Marga_ has quit IRC19:29
*** Vimal has joined #openstack-nova19:30
*** Piet has quit IRC19:30
dimsmriedem: there's not going to be any oslo-incubator stuff. trying to nuke it :)19:30
mriedemdims: where is cliutils going then?19:31
*** fawadkhaliq has joined #openstack-nova19:31
mriedemb/c i'm pushing a change for it right now :)19:31
mriedem2 changes actually19:32
*** changbl has joined #openstack-nova19:32
dimsmriedem: it's ok to push till we make an official decision.19:32
*** su_zhang has joined #openstack-nova19:33
mriedemkewl19:33
mriedemwhich brings up another thing i have to post to the ML here at some point19:33
mriedemmask_password should move back into oslo-incubator :)19:33
mriedembecause of backports, https://review.openstack.org/#/c/220622/19:33
*** gszasz has quit IRC19:34
dimsmriedem: will wait for the email :)19:34
*** electrocucaracha has joined #openstack-nova19:36
*** pixelbeat has joined #openstack-nova19:37
*** su_zhang has quit IRC19:37
mriedemdone19:38
*** lalitd has quit IRC19:38
*** ntpttr has left #openstack-nova19:39
*** tsekiyama has quit IRC19:40
vilobhmm111alaski_out : ping19:42
*** e0ne has joined #openstack-nova19:43
*** otter768_ has joined #openstack-nova19:44
*** otter768 has quit IRC19:44
*** sambetts has quit IRC19:45
*** sdague_ has joined #openstack-nova19:47
*** inara` has quit IRC19:47
*** afazekas has quit IRC19:47
*** jamiec has quit IRC19:47
*** sc68cal has quit IRC19:47
*** IanGovett has quit IRC19:47
*** sdague has quit IRC19:47
*** dhellmann has quit IRC19:47
*** salv-orlando has joined #openstack-nova19:47
*** dhellmann has joined #openstack-nova19:47
*** apoorvad has quit IRC19:47
*** sdague_ is now known as sdague19:47
*** sc68cal has joined #openstack-nova19:48
*** sambetts has joined #openstack-nova19:48
*** inara has joined #openstack-nova19:48
*** jamiec has joined #openstack-nova19:48
*** IanGovett has joined #openstack-nova19:48
vilobhmm111dansmith : ping19:49
dansmithvilobhmm111: in the middle of something right now, sorry19:49
vilobhmm111ok later then…thanks for the review on https://review.openstack.org/#/c/222423/  wanted to discuss something regarding the same…will talk later when you are free19:49
vilobhmm111dansmith : ^^19:50
*** apoorvad has joined #openstack-nova19:51
*** markvoelker has quit IRC19:51
mriedemccarmack: regular delete has this state check https://github.com/openstack/nova/blob/master/nova/compute/api.py#L191919:52
mriedemccarmack: force delete doesn't have those https://github.com/openstack/nova/blob/master/nova/compute/api.py#L195919:52
ccarmackmriedem: thanks19:53
ccarmackI'm trying to review https://review.openstack.org/#/c/163798/19:53
*** tsekiyama has joined #openstack-nova19:54
*** afazekas has joined #openstack-nova19:55
*** markvoelker has joined #openstack-nova19:55
*** yassine has joined #openstack-nova19:55
*** pixelbeat has quit IRC19:57
*** garyk1 has quit IRC19:59
*** penick has quit IRC19:59
*** thumpba has quit IRC19:59
*** penick has joined #openstack-nova19:59
*** penick has quit IRC19:59
*** ctrath has quit IRC20:02
mriedemyeah i gave up on that, was hoping alaski_out would get it straightened out20:04
mriedemdid mrs alaski go into labor yet?20:05
mriedemi missed something20:05
*** e0ne has quit IRC20:05
*** ctrath has joined #openstack-nova20:06
*** tsekiyama has quit IRC20:09
*** electrocucaracha has quit IRC20:10
*** electrocucaracha has joined #openstack-nova20:10
*** pm90__ has quit IRC20:11
*** tsekiyama has joined #openstack-nova20:12
*** emagana has quit IRC20:13
*** emagana has joined #openstack-nova20:15
*** jichen has quit IRC20:19
*** rfolco has quit IRC20:20
*** BobBall has quit IRC20:20
doffmccarmack: https://etherpad.openstack.org/p/mitaka-nova-spec-review-tracking20:21
*** smatzek has quit IRC20:21
*** BobBall has joined #openstack-nova20:22
*** BobBall is now known as Guest455920:22
*** rotbeard has quit IRC20:23
*** Marga_ has joined #openstack-nova20:26
*** seanw1 has quit IRC20:30
*** seanw1 has joined #openstack-nova20:31
*** Marga_ has quit IRC20:31
*** rfolco has joined #openstack-nova20:33
*** achanda has joined #openstack-nova20:34
*** achanda has quit IRC20:34
*** matrohon has quit IRC20:35
*** mwagner_ has quit IRC20:36
*** rfolco has quit IRC20:37
*** rfolco has joined #openstack-nova20:38
*** exploreshaifali has quit IRC20:39
*** achanda has joined #openstack-nova20:41
*** andreykurilin__ has quit IRC20:42
*** salv-orlando has quit IRC20:42
*** tsekiyama has quit IRC20:43
*** rk4n has quit IRC20:44
*** tsekiyama has joined #openstack-nova20:45
*** Marga_ has joined #openstack-nova20:49
*** moshele has joined #openstack-nova20:50
*** su_zhang has joined #openstack-nova20:52
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Enable options for oslo.reports  https://review.openstack.org/21691720:53
*** tsekiyama has quit IRC20:55
sdagueok, so can anyone who's familiar with the scheduler tell me why this reduced to 0 hosts - http://logs.openstack.org/60/231960/3/check/gate-tempest-dsvm-postgres-full/c319701/logs/screen-n-sch.txt.gz#_2015-10-07_19_45_54_27020:58
sdaguebecause I'm staring a lot at this20:58
*** salv-orlando has joined #openstack-nova20:59
*** tsekiyama has joined #openstack-nova20:59
sdaguehmmm libvirt explosion - http://logs.openstack.org/60/231960/3/check/gate-tempest-dsvm-postgres-full/c319701/logs/screen-n-cpu.txt.gz#_2015-10-07_19_45_59_59721:00
sdaguemriedem: is that an issue you've been tracking21:00
sdagueI guess we matched a host, landed there, libvirt exploded, and the retry resolved to 0 (for unknown to me reason)21:01
mriedemsdague: totes21:01
sdaguemriedem: ok, I also thought our scheduler logs had gotten better :(21:02
*** thumpba has joined #openstack-nova21:02
mriedemthis is the bug fwiw https://bugs.launchpad.net/nova/+bug/150136621:02
openstackLaunchpad bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed]21:02
mriedemthe scheduler logs should just say which filters were processed and what their results were (i think)21:02
sdagueok, why does the retry fail so hard in those cases?21:02
mriedemedleafe: did that part21:02
mriedemi don't understand the question on the retry21:03
mriedemwe only have one host right?21:03
sdaguecorrect21:03
sdagueretry isn't ever allowed to circle back?21:03
mriedemyou'd have to ask one of the scheduler trolls21:04
sdagueok21:04
mriedemhttp://logs.openstack.org/60/231960/3/check/gate-tempest-dsvm-postgres-full/c319701/logs/screen-n-sch.txt.gz#_2015-10-07_19_46_00_48721:05
mriedemhttps://github.com/openstack/nova/blob/master/nova/scheduler/filters/retry_filter.py#L4121:05
*** bswartz has joined #openstack-nova21:05
mriedemi guess you'd have to extend that for a retry limit on a failed host if you wanted that21:05
*** pm90_ has joined #openstack-nova21:05
mriedemotherwise i think the retry filter is to retry on a different host up to x times, (3 by default?) assuming you have x hosts21:06
sdagueor edge case, that if retry filter returns nothing, maybe don't blacklist hsots21:06
*** annasort has quit IRC21:06
*** achanda has quit IRC21:07
*** penick has joined #openstack-nova21:07
*** mtanino has joined #openstack-nova21:07
sdaguebecause on a full cloud it gets a bit pathologic given that it assumes you failed for capacity, but you might have failed for other reasons, like in this case21:07
*** ccesario_ has joined #openstack-nova21:07
*** harlowja has quit IRC21:08
*** mc_nair has quit IRC21:08
sdagueok, well I at least have the recheck bug then. Got to run off to lug meeting, but maybe I'll poke tomorrow.21:08
cfriesensdague: mriedem: retry normally doesn't retry the one that failed, I think21:08
sdaguecfriesen: yeh, that's what it looks like21:09
*** thorst1 has quit IRC21:09
sdaguehowever, it does seem if it goes to 0 nodes, maybe it's worth retrying the whole set21:09
*** tsekiyama has quit IRC21:09
cfriesensdague: mriedem: is there any interest in better scheduler logs (maybe optionally) ?   Currently especially some of the funky stuff with PCI/numa/pinning/hugepages it can be difficult to figure out what failed.21:09
mriedemi'm not interested in better logging no21:10
mriedemor anything useful in general21:10
*** Guest62625 is now known as mfisch21:10
*** john5223 is now known as zz_john522321:10
*** mfisch is now known as Guest4916721:10
*** zz_john5223 is now known as john522321:10
sdaguecfriesen: almost everything is hard to figure out on fail in the scheduler :)21:10
sdaguehonestly, I think we kind of need to unroll the way filters are processed to make it better21:11
cfriesensdague: mriedem: we've got a local patch that passes a "failure reason" parameter into each filter that can get filled in if the filter fails21:11
sdaguecfriesen: that seems useful, I'd say propose it21:11
*** tsekiyama has joined #openstack-nova21:11
mriedemcfriesen: you can always propose a thing and be rejected21:11
mriedembut if you don't propose it, you never know21:12
cfriesentrue enough. :)21:12
mriedemcfriesen: you can get all sassy like these two http://www.imdb.com/title/tt1041829/21:13
*** angdraug has joined #openstack-nova21:13
*** pm90__ has joined #openstack-nova21:15
*** achanda has joined #openstack-nova21:15
*** pm90_ has quit IRC21:17
bauzascatching the convo now :)21:18
bauzaswell, the RetryFilter generates some confusion IMHO21:19
*** rlrossit has quit IRC21:19
jlkHas anybody thrown up alarms about stable/kilo issues and glance server interaction?21:20
cfriesenmriedem: I prefer blackmail.  Now if only I had some dirt on anyone....21:20
mriedemhttp://docs.openstack.org/developer/nova/filter_scheduler.html?highlight=retryfilter - we could update the docs there21:20
mriedem"filters hosts that have been attempted for scheduling. Only passes hosts that have not been previously attempted."21:20
bauzasthe problem is when your computes are wedged, you think it's a scheduler issue21:21
mriedemjlk: nope21:21
mriedemjlk: well, need some context21:21
bauzasso people are complaining against something which is not really its fault21:21
mriedemoh the poor scheduler21:21
mriedem:P21:21
jlkmriedem: Installs over the last day or so we're getting errors when Nova tries to get info from glance, it's getting a traceback on a key error,21:22
mriedembauzas: i hope your kids aren't jealous of your true love for the scheduler21:22
bauzasI have a French cartoon in mind21:22
bauzascalimero21:22
bauzasbut that doesn't help21:22
jlk2015-10-07 21:16:32.411 27912 TRACE nova.api.openstack   File "/opt/bbc/openstack-11.0-master/nova/local/lib/python2.7/site-packages/glanceclient/openstack/common/apiclient/base.py", line 494, in __getattr__21:22
jlk2015-10-07 21:16:32.411 27912 TRACE nova.api.openstack     raise AttributeError(k)21:22
jlk2015-10-07 21:16:32.411 27912 TRACE nova.api.openstack AttributeError: id21:22
*** claudiub has joined #openstack-nova21:22
*** john5223 is now known as zz_john522321:22
mriedembauzas: i watched a youtube clip, makes sense i guess21:23
bauzasmriedem: my first daughter is born the same week than RAX and Nasa announced OpenStack, so she already knows why I'm overcommitted to the project :p21:23
*** yassine has quit IRC21:23
*** fawadkhaliq has quit IRC21:23
mriedemjlk: i feel like that's a solved issue in newer glanceclient21:23
mriedemyou'd have to check bug reports though21:23
jlkmriedem: it is, but stable/kilo pins to an older glanceclient21:23
mriedemjlk: you don't have to use that older glanceclient just b/c g-r has that21:23
jlkpython-glanceclient<0.18.0,>=0.15.021:23
bauzasanyway, I agree that we could somehow document better how to troubleshoot a scheduler failure21:24
mriedemg-r usually has things capped on stable not for functoinal reasons, but for gate job issues21:24
jlkmriedem: sure, just overriding that is somewhat... difficult.21:24
mriedemjlk: are you pip installing?21:24
jlkI /can/ make it work by bumping it21:24
jlkor rather by editing the requirements file in my checkout before pip installing it21:24
* bauzas thinking of a blogpost, quite classic ;)21:24
jlk(because bumping it after the install breaks all kinds of things)21:24
mriedemjlk: alternatively, if we know the fix in glanceclient, we can backport that to stable and do a patch version release21:24
*** zz_john5223 is now known as john522321:24
jlkit's certainly fixed in glance 1.1.021:25
mriedemjlk: did you find the bug for the fix?21:25
jlker glanceclient21:25
*** burt has quit IRC21:25
mriedemi found this https://bugs.launchpad.net/python-glanceclient/+bug/147677021:26
openstackLaunchpad bug 1476770 in python-glanceclient "_translate_from_glance fails with "AttributeError: id" in grenade" [Undecided,New]21:26
jlkhttps://bugs.launchpad.net/glance/+bug/1476770 is the bug I think if I just found it right21:26
mriedemjinx21:26
jlklolz21:26
mriedemoh i remember this21:26
jlkwhat I'm trying to understand is why this just started hitting us last night, and not like the day before21:26
*** rlrossit has joined #openstack-nova21:26
*** dustins has quit IRC21:27
*** xgsa has quit IRC21:27
mriedemdid you update some dependencies?21:27
*** xgsa has joined #openstack-nova21:27
mriedemthe problem in the bug was capped requests in oslo.vmware on stable which was borking glanceclient somehow21:27
jlkwell, this is CI, so every run i sa new install21:27
mriedemi don't remember the full details21:27
*** willsama has joined #openstack-nova21:27
jlkif I modify the requirements.txt before pip installing nova intoa  venv, it seems to work21:28
jlkthat'd mean forking nova for us for now21:28
mriedemwell, i'd first read through that bug report for how we fixed it in the gate21:28
*** Guest49167 is now known as mfisch21:29
mriedemjlk: https://review.openstack.org/#/c/204193/21:29
*** mfisch has quit IRC21:29
*** mfisch has joined #openstack-nova21:29
doffmnova doesn't do any sort of device passthrough for lxc guests right?21:29
mriedemjlk: this was the root cause for the gate which uses ubuntu 14.04 https://bugs.launchpad.net/python-glanceclient/+bug/1476770/comments/621:30
openstackLaunchpad bug 1476770 in python-glanceclient "_translate_from_glance fails with "AttributeError: id" in grenade" [Undecided,New]21:30
jlkmriedem: yeah, I'm struggling to figure out what changed on our end to start tripping on this, because supposedly it's all been fixed for months21:31
mriedemjlk: what version of oslo.vmware do you have?21:31
jlkwould you believe, none?21:32
*** ccarmack has quit IRC21:32
*** jamielennox|away is now known as jamielennox21:32
mriedemi don't believe that no21:32
jlk /opt/bbc/openstack-11.0-master/nova/bin/pip freeze |grep vmware21:32
jlkshow snone21:33
jlkwhat pulls in oslo.vmware ?21:33
mriedemwell, nova requires it in test-requirements b/c it's optional21:34
mriedemhttps://github.com/openstack/nova/blob/stable/kilo/test-requirements.txt#L2621:34
jlkyeah, we don't install test requirements21:34
jlkso our urllib3 isn't capped21:34
mriedemi thought glance or glancestore required it though for the vmware backend21:34
mriedemor cinder21:34
jlkalthough we don't have urllib321:34
jlkglance might, but we just install glanceclient21:34
jlkglance venv has oslo.vmware==0.11.221:35
*** ccarmack has joined #openstack-nova21:35
jlkand urllib3==1.10.421:35
mriedemcinder requires oslo.vmware in kilo https://github.com/openstack/cinder/blob/stable/kilo/requirements.txt#L4721:35
*** moshele has quit IRC21:35
mriedem0.11.2 is the version you want21:35
jlkyeah, each of our services get their own venv, so the requirements of one don't equate to the requirements of the other21:35
mriedemwhich caps urllib321:35
mriedemooo like containers!21:36
mriedemfancy21:36
jlkLooks like on the system we have ii  python-urllib3                      1.7.1-1ubuntu3                        al21:36
jlkthere is no pip installed urllib3 in nova's venv21:36
*** signed8bit has quit IRC21:37
mriedemrequests 2.8.0 was released yesterday21:37
mriedemrequests 2.8.0 2015-10-0621:37
mriedemin case that affects you at all21:37
jlkI bet that's it21:38
*** manishg_ has quit IRC21:38
mriedemwhich is uncapped in kilo https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L14321:38
jlkyeah 2.8.021:38
jlkso...21:38
jlklemme downgrade21:38
jlkoh right21:39
jlkbecause pip requests bundles urllib321:39
jlkand it's including 1.1221:39
jlkthis is it21:39
mriedemcool21:39
mriedembtw, pro tip, i found that by running this https://github.com/openstack/requirements/blob/stable/kilo/tools/what-broke.py21:39
mriedemfrom stable/kilo requirements, python tools/what-broke.py21:40
jlkoh that looks useful21:40
mriedemyeah, sdague wrote that since this is a common thing that breaks us upstream21:40
mriedemwell, before upper-constraints in liberty21:40
*** pixelbeat has joined #openstack-nova21:40
mriedemi'm surprised we haven't seen that blow up in the stable/kilo jobs upstream21:41
jlkyeah, I wonder if your requests is capped there somehow21:41
jlkusing the system package rather than pip installed21:41
mriedemmaybe21:41
jlkbut that seems to fix me21:41
mriedemurllib3 is definitely capped21:41
mriedemhttps://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L16921:41
jlkah21:41
mriedemso it's probably a thing waiting tobreak21:42
jlkmaybe21:42
jlkwait a sec21:43
jlkhow does g-r interact with the requirements file in the project itelf?21:43
jlknova requirements.txt doesn't mention urllib, but g-r does?21:43
*** tsekiyama has quit IRC21:43
mriedemnova's requirements.txt is synced from g-r21:43
mriedemvia bot21:43
jlkso this must have just changed?21:43
mriedemrequests is a transitive dependency through glanceclient21:43
jlk(pinning urllib)21:43
mriedemno, that was awhile back21:43
mriedemjuly https://review.openstack.org/#/c/204193/21:44
jlkthen why wasn't that synced to the stable/kilo requirements for nova?21:44
*** rlrossit has left #openstack-nova21:44
mriedemnova doesn't require urllib321:44
mriedemoslo.vmware does21:44
mriedemagain, transitive dep21:44
jlkoh wait,21:44
jlkso the requirements of the projects bubble up into g-r21:44
jlknot the other way around?21:44
mriedemyes21:44
mriedemeverything is centralized in g-r21:45
mriedemso we're not all using different versions of dependencies and cause conflicts21:45
mriedemwhen g-r is changed, that is synced out to the projects that use the changed dep21:45
mriedemvia bot21:45
jlkgotcha21:45
jlkso it's a "if you use this, use this version"21:45
mriedemright, like this https://review.openstack.org/#/c/201191/21:45
jlkand your tests install all the services in one pile, yeah?21:46
jlkeach service isn't isolated?21:46
mriedemright21:46
jlkthat's why you don't see it21:46
mriedemdevstack for the tempest jobs, otherwise tox for the unit test jobs21:46
jlkwith combined requirements, you're safe. But with nova requirements on its own, boom.21:46
mriedemi prefer to stay safe and warm21:47
*** tsekiyama has joined #openstack-nova21:47
*** Vimal has quit IRC21:48
*** manishg has joined #openstack-nova21:48
openstackgerritSylvain Bauza proposed openstack/nova: Add more documentation for RetryFilter  https://review.openstack.org/23224121:49
bauzasmriedem: sdague: ^21:49
jlkwould a patch to pin requests in nova be frowned upon?21:49
*** subscope has quit IRC21:51
mriedemjlk: in kilo?21:52
jlkat least in kilo, not sure if other versions are impacted21:52
mriedemyou'd have to first pin requests in g-r stable/kilo21:52
jlkoh I could just do it there, sure21:53
mriedemit's capped in juno so not a problem there21:53
mriedemwell,21:53
mriedemit's 2 parts21:53
jlkno I can't21:53
jlkI have to do it there first, then add it in nova21:53
mriedemright21:53
mriedemcap in g-r stable/juno <2.8.021:53
mriedemthen a nova change that adds requests (same as what's in the g-r change) with a Depends-On tag from the nova change to the g-r change21:53
*** harlowja has joined #openstack-nova21:53
*** rk4n has joined #openstack-nova21:54
*** rfolco_ has joined #openstack-nova21:54
mriedemwith an explanation in the nova change as to why nova needs to require requests even though nova doesn't use it directly21:54
jlkalright, I might need some help on that one when the time comes21:54
mriedemheh, if you have the explanation i can push the changes21:54
jlkwell, maybe it would be better to put the pin on glanceclient21:54
mriedemwant to just dump it in that bug report?21:54
jlksince glanceclient is the one having trouble21:54
mriedemglanceclient is pinned in g-r stable/kilo https://github.com/openstack/requirements/blob/stable/kilo/global-requirements.txt#L12521:54
mriedems/pinned/capped/21:54
jlkwhat I meant was that glanceclient is the thing that is using requests21:55
mriedemif you cap requests in g-r stable/kilo and glanceclient uses requests, it will sync to glanceclient on stalbe/kilo and we can release that as a patch version21:55
jlkso old glanceclient pulls in new requests and tries to use it, boom21:55
*** john5223 is now known as zz_john522321:55
mriedemyeah https://github.com/openstack/python-glanceclient/blob/stable/kilo/requirements.txt#L1021:55
mriedemjlk: so how about update that bug report with details21:55
mriedemand we can begin the capping game21:55
mriedemtonyb: ^21:55
jlkyeah I was just opening that tab again.21:56
* tonyb reads scroolback21:56
*** mwagner_ has joined #openstack-nova21:56
*** rfolco has quit IRC21:56
*** rfolco__ has joined #openstack-nova21:56
*** ccarmack has quit IRC21:58
mriedemjlk: reminded me that i had this https://review.openstack.org/#/c/213310/21:59
*** rfolco_ has quit IRC22:00
jlkneat22:00
jlkhttps://bugs.launchpad.net/glance/+bug/1476770 updated22:01
openstackLaunchpad bug 1476770 in python-glanceclient "_translate_from_glance fails with "AttributeError: id" in grenade" [Undecided,New]22:01
tonybjlk: please drop me the bug number so I can look/help with the capping game.22:01
tonybGah stupid scrollback.  Thanks.22:01
mriedemlet me update that reqs change on master quick22:01
tonybmriedem: cool.22:03
*** edmondsw has quit IRC22:03
*** penick has quit IRC22:03
tonybmriedem: I need a coffee anyway.  Just got back from a ride22:03
*** ociuhandu has quit IRC22:04
jlkThanks for the help, gotta run for a bit.22:04
*** tsekiyama has quit IRC22:04
*** sambetts has quit IRC22:05
*** sambetts has joined #openstack-nova22:07
*** tsekiyama has joined #openstack-nova22:07
*** su_zhang_ has joined #openstack-nova22:07
*** annegentle has quit IRC22:08
*** penick has joined #openstack-nova22:09
*** su_zhang has quit IRC22:11
openstackgerritAnne Gentle proposed openstack/nova: Rearranges to create new Compute API Guide  https://review.openstack.org/23018622:12
*** manishg has quit IRC22:12
*** tpeoples has quit IRC22:13
*** mdrabe has quit IRC22:16
*** ctrath has quit IRC22:16
mriedemjlk: https://review.openstack.org/#/c/232250/22:16
mriedemtonyb: ^22:16
mriedemhmm22:19
mriedemnova.tests.unit.db.test_db_api.ArchiveTestCase is completely racy22:20
mriedemb/c of multiple workers and a single db22:20
mriedemmtreinish: tempest has a fixture for making a test case run in serial right?22:20
*** alex_klimov has quit IRC22:20
*** burgerk_ has quit IRC22:20
*** tonytan4ever has quit IRC22:21
mtreinishmriedem: well we tell testr to parallelize at the class level22:23
mriedemyeah, i want that22:23
mtreinishand then there is an external lock fixture we use to lock against running tests at the same time22:23
mtreinishmriedem: it's an all or nothing deal to do the class thing22:24
mtreinishit's in the testr.conf22:24
mtreinishmriedem: http://git.openstack.org/cgit/openstack/tempest/tree/.testr.conf#n922:24
mtreinishgetting that all working was most of my havana cycle22:25
mriedemhttps://github.com/openstack/tempest/blob/master/tempest/common/tempest_fixtures.py#L1922:25
mriedemright?22:25
mtreinishyeah that's something we added to oslo_concurrency22:25
mtreinishthat does the same thing as the syncronized decorator (or whatever the lcok one is) with external=True22:25
mtreinishit requires config to work22:25
mtreinishmriedem: what exactly are you looking for?22:26
*** aix has joined #openstack-nova22:26
mriedemlet me push it up quick22:26
openstackgerritMatt Riedemann proposed openstack/nova: WIP: Print number of rows archived per table in db archive_deleted_rows  https://review.openstack.org/23225222:27
mtreinishmriedem: wait don't the nova db_api tests use an in memory sqlite db? what is racing?22:27
mriedemso what i had here: https://review.openstack.org/#/c/232252/1/nova/tests/unit/db/test_db_api.py22:27
mriedemi was originally doing self.assertDictEqual22:27
mriedemb/c i expected a very specific result22:27
mriedembut b/c there are other tests running i was getting stuff back in results from other tests hitting archive at the same time22:28
mriedemhonestly i'm not sure how this test class actually works today since it's very global on thedb22:28
mriedemor is it a db per test case run?22:28
mtreinishmriedem: I was pretty sure you had a fixture in place that setup a new in memory sqlite db for each test (it runs migrations fresh) so the tests are properly isolate22:29
mtreinishd22:29
mtreinishmriedem: yeah that's how I thought they worked22:29
mriedemso we do https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L19522:29
mriedemwell wtf22:29
*** dims has quit IRC22:29
mriedemcould be some dumb metadata stuff we're doing in the db api22:30
mtreinishmriedem: if you want the total insanity of a shared DB for api testing, you end up with essentially this: https://review.openstack.org/#/c/213481/22:30
mtreinishwhich is not a direction you want to go22:30
mriedemheh, no thanks22:30
*** markvoelker has quit IRC22:31
* mtreinish reminds himself to rework that to not use the shared db now that sqlite works22:33
*** markvoelker has joined #openstack-nova22:33
mtreinishmriedem: do you have an example of the failure you were hitting?22:34
mriedemsec22:34
mriedemit had some entries in the dict that were some crazy random string22:34
mriedemlike instance_info_cache:[862 char]22:35
*** Daisy has joined #openstack-nova22:36
*** stevemar_ has quit IRC22:36
*** stevemar_ has joined #openstack-nova22:37
mtreinishhmm, that sounds weird22:37
mriedemmtreinish: with just this: self.assertDictEqual(dict(instance_id_mappings=2), results)22:38
mriedemi get http://paste.openstack.org/show/475668/22:38
*** sdague has quit IRC22:38
mriedemassertDictEqual is probably not what i want to be using either22:38
*** annasort has joined #openstack-nova22:39
*** sdague has joined #openstack-nova22:39
*** rk4n has quit IRC22:40
mtreinishmriedem: yeah, I normally don't like the more involved asserts and just extract the values to check manually and do an equals22:40
mtreinishfwiw, I have a hard time following what that error output is trying to tell me22:40
*** Daisy has quit IRC22:40
*** stevemar_ has quit IRC22:41
*** rk4n has joined #openstack-nova22:41
*** manishg has joined #openstack-nova22:42
mriedemnonsense22:43
mriedemit's nonsense22:43
mriedemwe have a model object comparator assertion thing in the test_db_api module, i'll use that22:44
mriedemcompares 2 dicts based on length and key/values22:44
*** ZZelle has quit IRC22:45
mriedemassertDictEqual isn't even testtools, it's std lib https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertDictEqual22:45
*** Daisy has joined #openstack-nova22:46
mriedemheh, the test result is still completely reandom22:46
mriedem*random22:46
mriedemoh i know why22:47
*** su_zhang has joined #openstack-nova22:49
*** Daisy has quit IRC22:50
*** su_zhang_ has quit IRC22:52
*** david-lyle has quit IRC22:54
lifelessmriedem: forgot to update the incubator?22:54
mriedemlifeless: ?22:54
mriedemfor my test problems above?22:54
lifelessmriedem: guessing at why its still random order :)22:54
lifelessmriedem: but perhaps you've context switched onto something totally different :)22:54
mriedemno, the code was stuffing keys with 0 values in the dict and i only expected things with a >0 value22:54
mriedemso the test actually found a problem \o/22:55
*** david-lyle has joined #openstack-nova22:55
openstackgerritMatt Riedemann proposed openstack/nova: Print number of rows archived per table in db archive_deleted_rows  https://review.openstack.org/23225222:55
mriedemnow my 1 day blueprint is complete22:55
mriedempending the o-i syncs22:55
*** pm90__ has quit IRC22:56
*** angdraug has quit IRC22:57
*** pm90_ has joined #openstack-nova22:59
*** stevemar_ has joined #openstack-nova22:59
*** davidlenwell has quit IRC23:01
*** davidlenwell has joined #openstack-nova23:02
*** markvoelker has quit IRC23:04
*** alejandrito has quit IRC23:04
*** xyang1 has quit IRC23:08
*** markvoelker has joined #openstack-nova23:08
*** pixelbeat has quit IRC23:08
*** ijuwang has quit IRC23:09
*** harlowja has quit IRC23:10
*** david-lyle has quit IRC23:10
*** harlowja has joined #openstack-nova23:10
*** willsama has quit IRC23:10
*** david-lyle has joined #openstack-nova23:10
*** amrith is now known as _amrith_23:11
*** _amrith_ is now known as amrith23:12
*** david-lyle has quit IRC23:14
*** diogogmt_ has quit IRC23:16
*** david-lyle has joined #openstack-nova23:17
*** rk4n has quit IRC23:18
*** david-ly_ has joined #openstack-nova23:19
*** david-lyle has quit IRC23:21
*** atuvenie_ has joined #openstack-nova23:22
*** manishg has quit IRC23:22
*** dims has joined #openstack-nova23:22
*** atuvenie has quit IRC23:25
*** hemna is now known as hemnafk23:26
*** manishg has joined #openstack-nova23:27
*** thorst has joined #openstack-nova23:30
*** david-ly_ is now known as david-lyle23:31
*** mriedem has quit IRC23:31
*** mriedem has joined #openstack-nova23:32
*** tsekiyama has quit IRC23:33
*** penick has quit IRC23:34
*** lucas-dinner has quit IRC23:35
*** nic has quit IRC23:36
*** aysyd has quit IRC23:36
openstackgerritClaudiu Belu proposed openstack/nova: Replaces contextlib.nested with test.nested  https://review.openstack.org/23210823:37
*** penick has joined #openstack-nova23:37
*** thorst has quit IRC23:38
*** asettle has joined #openstack-nova23:38
*** mriedem has quit IRC23:39
*** dims has quit IRC23:40
*** tjones has joined #openstack-nova23:42
*** markvoelker has quit IRC23:42
*** tjones has quit IRC23:43
*** markvoelker has joined #openstack-nova23:43
*** tjones has joined #openstack-nova23:44
*** amrith is now known as _amrith_23:46
*** achanda has quit IRC23:47
*** stevemar_ has quit IRC23:47
*** penick has quit IRC23:49
*** achanda has joined #openstack-nova23:49
*** tjones has left #openstack-nova23:50
*** tsekiyama has joined #openstack-nova23:50
*** Marga_ has quit IRC23:50
*** aswadr has quit IRC23:52
*** markvoelker has quit IRC23:53
*** achanda has quit IRC23:53
*** dims has joined #openstack-nova23:54
*** achanda has joined #openstack-nova23:55
*** pixelbeat has joined #openstack-nova23:57

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