Thursday, 2015-07-16

*** dannywilson has quit IRC00:01
*** ijw has quit IRC00:02
*** ijw has joined #openstack-nova00:03
gmann_mtreinish: in later case Devstack have to run Juno jobs runs for v2 only00:03
*** Sukhdev has quit IRC00:03
*** mutoulbj has quit IRC00:03
*** mtanino has quit IRC00:13
*** tsekiyama has quit IRC00:15
mtreinishgmann_: well I think changing the default in tempest is a bit risky because of existing deployments00:17
mtreinisheven if they're kilo or newer there is no guarantee they're running with 2.1 and things used to just work for them00:17
mtreinishgmann_: what I'd really like to see is making 2.1 the compute endpoint type by default in the catalog00:17
mtreinishand make the 2.0 endpoint the special case one00:18
gmann_mtreinish: yes, Tempest one might be risky,00:19
gmann_mtreinish: I will update the patches (in current proposed patch it plays with api-paste.ini for making v21 default) for making v21 default through catalog side.00:21
gmann_and see how it goes, i pending for long time and i remember you have mentioned many time :)00:21
gmann_and i forgot to follow up on those, Sorry for that. I will do those by today or max by tomorrow00:22
mtreinishno worries, it's not like it's super critical. It'll just be good to get it sorted so it's a little less confusing for people00:23
*** salv-orlando has quit IRC00:24
gmann_mtreinish: yea00:25
*** salv-orlando has joined #openstack-nova00:26
*** achanda_ has quit IRC00:26
*** ijw_ has joined #openstack-nova00:31
*** annashen has joined #openstack-nova00:32
*** vilobhmm2 has quit IRC00:32
*** annashen has quit IRC00:33
*** salv-orlando has quit IRC00:34
*** ijw has quit IRC00:34
*** mdrabe has joined #openstack-nova00:35
*** artom has quit IRC00:36
*** ujjain has quit IRC00:36
*** ujjain has joined #openstack-nova00:37
*** ujjain has quit IRC00:37
*** ujjain has joined #openstack-nova00:37
*** andreykurilin__ has quit IRC00:37
n0anomriedem_away, ping me when you're back on-line00:41
*** pixelbeat_ has joined #openstack-nova00:42
*** yonglihe has joined #openstack-nova00:43
*** newbie_2 has quit IRC00:49
*** angdraug has joined #openstack-nova00:49
oomichialex_xu_: hi, morning00:50
alex_xu_oomichi: hi, morning00:50
*** ijw_ has quit IRC00:50
*** ijw has joined #openstack-nova00:51
oomichialex_xu_: a little question about api-wg: do you know current status of approved but not-merged patches?00:51
*** signed8bit_ZZZzz is now known as signed8bit00:51
oomichialex_xu_: example is https://review.openstack.org/#/c/185180/00:51
*** davideagnello has quit IRC00:52
oomichialex_xu_: the final deadline of these patches are over already, but we cannot merge them yet00:52
alex_xu_oomichi: I confuse on that also, but I think it means already merge00:52
*** baoli has joined #openstack-nova00:52
*** newbie_2 has joined #openstack-nova00:53
oomichialex_xu_: yeah, I also have the same feeling. do we have an irc channel for api-wg?00:53
alex_xu_oomichi: it's worth ask in api-wg channel00:53
*** angdraug has quit IRC00:53
alex_xu_oomichi: #openstack-api00:53
*** davideagnello has joined #openstack-nova00:54
oomichialex_xu_: thanks, I will try it :)00:54
alex_xu_oomichi: np00:54
*** newbie_2 has quit IRC01:00
*** Longgeek has joined #openstack-nova01:00
*** signed8bit is now known as signed8bit_ZZZzz01:02
*** mutoulbj has joined #openstack-nova01:03
*** ssurana has quit IRC01:03
anishmriedem_away: dansmith failed pep8 before, can I get the +2 again https://review.openstack.org/#/c/193414/01:04
*** ssurana has joined #openstack-nova01:04
*** ssurana has quit IRC01:08
*** mutoulbj has quit IRC01:09
*** casusbelli has joined #openstack-nova01:12
*** liusheng has joined #openstack-nova01:12
*** casusbel_ has quit IRC01:14
*** mdrabe has quit IRC01:14
*** openstack has joined #openstack-nova01:24
*** yingjun has joined #openstack-nova01:25
*** annegentle has quit IRC01:26
*** yamahata has quit IRC01:26
*** asettle has joined #openstack-nova01:27
*** raildo has quit IRC01:30
*** ijw has quit IRC01:32
*** mestery_ is now known as mestery01:34
*** patrickeast has quit IRC01:35
*** ijw has joined #openstack-nova01:36
*** baoli_ has joined #openstack-nova01:36
*** annegentle has joined #openstack-nova01:37
*** jdandrea has quit IRC01:38
*** tpatil has quit IRC01:38
*** baoli has quit IRC01:38
*** signed8bit_ZZZzz is now known as signed8bit01:41
*** annegentle has quit IRC01:42
*** kaisers1 has quit IRC01:44
*** ijw has quit IRC01:49
*** park_ has joined #openstack-nova01:50
*** rdo has quit IRC01:51
*** signed8bit is now known as signed8bit_ZZZzz01:52
*** IanGovett has joined #openstack-nova01:54
*** signed8bit_ZZZzz is now known as signed8bit01:56
*** MentalRay__ has joined #openstack-nova01:57
*** dims_ has quit IRC01:58
*** yingjun has quit IRC01:58
*** zul has joined #openstack-nova02:00
*** IanGovett has quit IRC02:01
*** doude has quit IRC02:04
*** dave-mccowan has joined #openstack-nova02:05
*** signed8bit has quit IRC02:06
*** doude has joined #openstack-nova02:07
*** doude has quit IRC02:07
*** doude has joined #openstack-nova02:07
*** annegentle has joined #openstack-nova02:10
*** shuquan has joined #openstack-nova02:13
*** annegentle has quit IRC02:13
*** dims__ has joined #openstack-nova02:18
*** markvoelker has quit IRC02:22
*** baoli_ has quit IRC02:23
*** ijw has joined #openstack-nova02:24
*** wwriverrat has quit IRC02:25
*** unicell has quit IRC02:26
*** vilobhmm has joined #openstack-nova02:26
*** yingjun has joined #openstack-nova02:26
*** yingjun has quit IRC02:27
*** yingjun has joined #openstack-nova02:27
*** vilobhmm1 has joined #openstack-nova02:28
*** jdurgin has quit IRC02:28
*** yingjun_ has joined #openstack-nova02:29
*** yingjun has quit IRC02:29
*** Longgeek has joined #openstack-nova02:30
*** dims__ has quit IRC02:30
*** vilobhmm has quit IRC02:30
*** bkopilov has quit IRC02:31
*** jyuso1 has quit IRC02:32
openstackgerritJeffrey Zhang proposed openstack/nova: Support all_tenants search_opts for neutron  https://review.openstack.org/18765702:34
*** yamahata has joined #openstack-nova02:34
*** baoli has joined #openstack-nova02:35
*** Longgeek has quit IRC02:35
*** yamahata has quit IRC02:36
*** yamahata has joined #openstack-nova02:37
*** diogogmt has joined #openstack-nova02:43
*** sorrison_laptop has joined #openstack-nova02:45
*** baoli has quit IRC02:47
*** baoli has joined #openstack-nova02:49
*** yamahata has quit IRC02:50
*** hakimo_ has joined #openstack-nova02:53
*** gryf_ has quit IRC02:54
*** hakimo has quit IRC02:54
*** gryf has joined #openstack-nova02:55
*** ijw has quit IRC02:56
*** dave-mccowan has quit IRC02:56
*** shuquan has quit IRC03:00
*** yingjun_ has quit IRC03:03
*** andrewbogott_afk has quit IRC03:08
*** otter768 has quit IRC03:08
openstackgerritChangBo Guo(gcb) proposed openstack/nova: [WIP] Use explicit greenthread.sleep instead of time.sleep  https://review.openstack.org/20236503:09
*** casusbelli has quit IRC03:11
*** casusbelli has joined #openstack-nova03:11
*** bradjones has quit IRC03:14
*** tpeoples has joined #openstack-nova03:14
*** casusbelli has quit IRC03:18
*** casusbelli has joined #openstack-nova03:19
*** bradjones has joined #openstack-nova03:19
*** bradjones has quit IRC03:19
*** bradjones has joined #openstack-nova03:19
*** park_heijlong has quit IRC03:23
*** annashen has joined #openstack-nova03:23
*** annashen has quit IRC03:28
*** pixelbeat_ has quit IRC03:28
*** xyang1 has quit IRC03:30
*** jyuso1 has joined #openstack-nova03:32
*** baoli has quit IRC03:33
*** markvoelker has joined #openstack-nova03:33
*** dannywilson has joined #openstack-nova03:34
*** markvoelker has quit IRC03:34
*** markvoelker has joined #openstack-nova03:34
*** deepthi has joined #openstack-nova03:35
openstackgerritxing-yang proposed openstack/nova-specs: EMC ScaleIO Data Client Shim Libvirt Volume Driver  https://review.openstack.org/18194103:35
*** wwriverrat has joined #openstack-nova03:36
*** markvoelker_ has joined #openstack-nova03:37
*** markvoelker has quit IRC03:39
*** y_sawai has joined #openstack-nova03:40
*** gryf has quit IRC03:41
*** Nikolay_St has quit IRC03:41
*** unicell has joined #openstack-nova03:43
*** briancurtin has quit IRC03:45
*** scottda has quit IRC03:45
*** wanghao has quit IRC03:45
*** serverascode has quit IRC03:45
*** zhiyan has quit IRC03:45
*** angular_mike_ has quit IRC03:45
*** rex_lee has quit IRC03:46
*** y_sawai has quit IRC03:46
*** h00327910_ has quit IRC03:46
*** y_sawai has joined #openstack-nova03:46
*** erlon has quit IRC03:46
*** ramishra has quit IRC03:46
*** boris-42 has quit IRC03:46
*** y_sawai has quit IRC03:47
*** aweeks has quit IRC03:47
openstackgerritxing-yang proposed openstack/nova: Add os-brick based LibvirtVolumeDriver for ScaleIO  https://review.openstack.org/19445403:47
*** coolsvap|away is now known as coolsvap03:47
openstackgerritRen Qiaowei proposed openstack/nova: Remove unnecessary executable permission  https://review.openstack.org/20237603:48
*** unicell has quit IRC03:48
*** unicell1 has joined #openstack-nova03:48
*** wanghao has joined #openstack-nova03:48
*** scottda has joined #openstack-nova03:48
*** kaisers1 has joined #openstack-nova03:49
*** garyk has quit IRC03:49
*** dougwig has quit IRC03:50
*** StevenK has quit IRC03:51
*** freyes has quit IRC03:52
*** dannywilson has quit IRC03:52
*** dannywilson has joined #openstack-nova03:52
openstackgerritxing-yang proposed openstack/nova-specs: EMC ScaleIO Data Client Shim Libvirt Volume Driver  https://review.openstack.org/18194103:53
*** zhiyan has joined #openstack-nova03:54
*** StevenK has joined #openstack-nova03:55
*** rajesht has joined #openstack-nova03:55
*** briancurtin has joined #openstack-nova03:56
*** serverascode has joined #openstack-nova03:58
*** bkopilov has joined #openstack-nova03:59
*** rex_lee has joined #openstack-nova04:02
*** achanda has joined #openstack-nova04:02
*** shuquan has joined #openstack-nova04:03
openstackgerritwanghao proposed openstack/nova: Image/block device mapping combination is not valid  https://review.openstack.org/17198404:03
*** aweeks has joined #openstack-nova04:04
*** oomichi has quit IRC04:05
*** lyanchih has quit IRC04:06
*** boris-42 has joined #openstack-nova04:08
*** erlon has joined #openstack-nova04:08
*** annashen has joined #openstack-nova04:08
*** ijw has joined #openstack-nova04:10
*** h00327910_ has joined #openstack-nova04:11
*** hemna has joined #openstack-nova04:11
*** kiran-r has joined #openstack-nova04:15
*** shuquan has quit IRC04:15
*** belmoreira has joined #openstack-nova04:15
*** jwcroppe has joined #openstack-nova04:16
*** Longgeek has joined #openstack-nova04:17
*** tonytan4ever has quit IRC04:18
*** angular_mike_ has joined #openstack-nova04:22
*** Longgeek has quit IRC04:25
*** Longgeek has joined #openstack-nova04:25
*** ssurana has joined #openstack-nova04:27
*** belmoreira has quit IRC04:28
*** annashen has quit IRC04:28
*** dougwig has joined #openstack-nova04:29
*** ramishra has joined #openstack-nova04:29
openstackgerritZhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE  https://review.openstack.org/20237904:29
*** tonytan4ever has joined #openstack-nova04:29
*** hemna has quit IRC04:30
*** shyu has quit IRC04:32
*** shadyabhi has quit IRC04:33
openstackgerritZhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE  https://review.openstack.org/20237904:34
openstackgerritShiina, Hironori proposed openstack/nova: Add an API to inject an NMI to an instance  https://review.openstack.org/20238004:37
*** zzzeek has joined #openstack-nova04:37
*** freyes has joined #openstack-nova04:40
*** irenab has quit IRC04:40
*** shyama has joined #openstack-nova04:41
*** MentalRay__ has quit IRC04:41
*** Nikolay_St has joined #openstack-nova04:42
*** kiran-r has quit IRC04:47
*** y_sawai has joined #openstack-nova04:47
openstackgerritRen Qiaowei proposed openstack/nova: Remove unnecessary executable permission  https://review.openstack.org/20237604:49
*** markvoelker has joined #openstack-nova04:49
*** garyk has joined #openstack-nova04:51
*** zzzeek has quit IRC04:52
*** doude has quit IRC04:52
*** y_sawai has quit IRC04:52
*** markvoelker_ has quit IRC04:53
*** ssurana has quit IRC04:53
*** Jeffrey4l_ has quit IRC04:57
*** Mike_D_laptop has joined #openstack-nova04:59
*** tonytan4ever has quit IRC05:00
*** SourabhP has joined #openstack-nova05:02
*** Mike_D_laptop has quit IRC05:04
*** otter768 has joined #openstack-nova05:09
*** kiran-r has joined #openstack-nova05:09
*** ssurana has joined #openstack-nova05:10
*** Jeffrey4l_ has joined #openstack-nova05:10
openstackgerritFeodor Tersin proposed openstack/nova: Fix FlavorDiskTooSmall boot fault for volume-backed instances  https://review.openstack.org/19656905:11
*** veena has joined #openstack-nova05:12
*** otter768 has quit IRC05:14
*** deepakcs has joined #openstack-nova05:15
deepakcsHello, can any cores pls bless https://review.openstack.org/#/c/194538/4 (It already has a lot of +1s)05:16
*** _gryf has joined #openstack-nova05:20
openstackgerritChangBo Guo(gcb) proposed openstack/nova: [WIP] Use explicit greenthread.sleep instead of time.sleep  https://review.openstack.org/20236505:30
*** _kiran_ has joined #openstack-nova05:33
*** asettle has quit IRC05:34
*** kiranr has joined #openstack-nova05:35
*** kiranr has quit IRC05:35
*** lyanchih has joined #openstack-nova05:36
_gryfjohnthetubaguy, hi05:37
*** kiran-r has quit IRC05:37
*** _kiran_ has quit IRC05:39
*** sudipto has joined #openstack-nova05:39
*** SourabhP has quit IRC05:42
*** shuquan has joined #openstack-nova05:42
openstackgerritAlex Xu proposed openstack/nova: Remove db layer hard-code permission checks for instance_get_all_by_host_and_not_type  https://review.openstack.org/20238905:42
*** RuiChen has quit IRC05:42
*** vishwana_ has joined #openstack-nova05:43
*** RuiChen has joined #openstack-nova05:43
*** moshele has joined #openstack-nova05:43
*** irenab has joined #openstack-nova05:44
deepakcsjohnthetubaguy: ^^05:44
*** vishwanathj has quit IRC05:47
*** y_sawai has joined #openstack-nova05:48
*** y_sawai_ has joined #openstack-nova05:48
*** ankit has joined #openstack-nova05:49
*** shuquan has quit IRC05:49
*** Murali has joined #openstack-nova05:50
rajeshtmikal, alaski: you around ??05:50
openstackgerritGhanshyam Mann proposed openstack/nova: Show 'locked_by' information in server Show & List Detail  https://review.openstack.org/20239105:50
*** itzikb_ has joined #openstack-nova05:52
*** y_sawai has quit IRC05:52
*** shuquan has joined #openstack-nova05:53
*** wanghua has joined #openstack-nova05:53
*** dboik has joined #openstack-nova05:57
*** shuquan has quit IRC05:57
*** shuquan has joined #openstack-nova05:58
*** dboik has quit IRC06:02
*** jmccrory has quit IRC06:02
*** nkrinner_afk is now known as nkrinner06:07
*** shuquan has quit IRC06:08
*** jmccrory has joined #openstack-nova06:09
*** shuquan has joined #openstack-nova06:10
*** Sukhdev has joined #openstack-nova06:11
*** scheuran has joined #openstack-nova06:11
*** cfriesen has quit IRC06:12
*** Nikolay_St has quit IRC06:12
*** annashen has joined #openstack-nova06:12
*** y_sawai_ has quit IRC06:13
*** moshele has quit IRC06:13
*** ildikov has quit IRC06:18
*** mpaolino has joined #openstack-nova06:19
*** mpaolino has quit IRC06:25
*** belmoreira has joined #openstack-nova06:25
*** dannywilson has quit IRC06:26
*** asettle has joined #openstack-nova06:26
openstackgerritAbhishek Kekane proposed openstack/nova: libvirt: Kill rsync/scp processes before deleting instance  https://review.openstack.org/19298606:28
*** vilobhmm1 has quit IRC06:31
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/19721306:31
*** jichen has joined #openstack-nova06:37
*** annashen has quit IRC06:37
*** jichen has quit IRC06:38
*** shyu has joined #openstack-nova06:39
*** jichen has joined #openstack-nova06:39
*** abhishekk has joined #openstack-nova06:39
*** shuquan has quit IRC06:43
*** sahid has joined #openstack-nova06:43
*** asettle has quit IRC06:45
*** Sukhdev has quit IRC06:45
*** oro_ has joined #openstack-nova06:47
*** oro has joined #openstack-nova06:47
*** shuquan has joined #openstack-nova06:47
*** abhishekk has quit IRC06:49
*** jdurgin has joined #openstack-nova06:50
*** jdurgin has quit IRC06:50
*** Longgeek has quit IRC06:51
openstackgerritSean McCully proposed openstack/nova: Creates disk.fallocate_test  https://review.openstack.org/18824906:52
*** Longgeek has joined #openstack-nova06:52
*** Kevin_Zheng has quit IRC06:55
*** Jhelf has joined #openstack-nova06:58
*** alex_klimov has joined #openstack-nova06:59
*** moshele has joined #openstack-nova07:03
*** ildikov has joined #openstack-nova07:04
*** markus_z has joined #openstack-nova07:05
*** bkopilov has quit IRC07:08
*** otter768 has joined #openstack-nova07:10
*** rdopiera has joined #openstack-nova07:10
*** andrearosa has joined #openstack-nova07:11
*** otter768 has quit IRC07:15
*** Jeffrey4l_ is now known as Jeffrey4l07:17
*** romainh has joined #openstack-nova07:20
*** lyanchih has quit IRC07:21
*** Nikolay_St has joined #openstack-nova07:21
*** bkopilov has joined #openstack-nova07:21
*** Jeffrey4l has quit IRC07:22
*** lyanchih has joined #openstack-nova07:22
*** foexle has joined #openstack-nova07:27
*** lan has quit IRC07:33
*** lpetrut has joined #openstack-nova07:34
*** Jeffrey4l has joined #openstack-nova07:35
*** sorrison_laptop has quit IRC07:37
*** Maike has joined #openstack-nova07:38
*** jlanoux has joined #openstack-nova07:41
*** pixelbeat_ has joined #openstack-nova07:42
*** achanda has quit IRC07:46
*** eglynn-officeafk has quit IRC07:50
*** johnma has quit IRC07:52
*** oro_ has quit IRC07:55
*** oro has quit IRC07:55
openstackgerritheijlong proposed openstack/nova: Add uuid into SecurityGroup object  https://review.openstack.org/20197007:56
*** lucasagomes has joined #openstack-nova08:00
*** pixelbeat_ has quit IRC08:01
*** markvoelker has quit IRC08:04
*** armax has quit IRC08:06
*** ssurana has quit IRC08:06
openstackgerritIWAMOTO Toshihiro proposed openstack/nova: Omit RescheduledException in instance_fault.message  https://review.openstack.org/16575108:07
deepakcsHello, can any cores pls bless https://review.openstack.org/#/c/194538/4 (It already has a lot of +1s)08:08
*** park_heijlong has joined #openstack-nova08:09
*** markvoelker has joined #openstack-nova08:11
openstackgerritGhanshyam Mann proposed openstack/nova: Add top,bottom & change version tests for microversion 2.4  https://review.openstack.org/20243108:14
openstackgerritSean McCully proposed openstack/nova: Creates disk.fallocate_test  https://review.openstack.org/18824908:15
*** y_sawai has joined #openstack-nova08:15
*** markvoelker has quit IRC08:16
*** y_sawai has quit IRC08:16
*** jistr has joined #openstack-nova08:16
*** markvoelker has joined #openstack-nova08:17
*** ssurana has joined #openstack-nova08:18
*** shyama has quit IRC08:20
*** markvoelker has quit IRC08:25
openstackgerritVictor Stinner proposed openstack/nova: Fix Python 3 issues in nova.utils and nova.tests  https://review.openstack.org/19798208:25
*** derekh has joined #openstack-nova08:27
*** shuquan has quit IRC08:28
*** shuquan has joined #openstack-nova08:28
*** haypo has joined #openstack-nova08:28
*** redbeard has joined #openstack-nova08:29
*** markvoelker has joined #openstack-nova08:29
haypojohnthetubaguy, sdague: hi. i had to update my python3 fix https://review.openstack.org/#/c/197982/ because of another fix related to py3 (written by sdague to fix a random failure of py34)08:29
openstackgerritlyanchih proposed openstack/nova: neutron - race conditions in security-group additions  https://review.openstack.org/20243608:31
*** boris-42 has quit IRC08:32
*** markvoelker has quit IRC08:33
*** oro_ has joined #openstack-nova08:35
*** oro has joined #openstack-nova08:35
openstackgerritAndreas Scheuring proposed openstack/nova: Add macvtap as libvirt virtual interface (vif) type to Nova's libvirt driver  https://review.openstack.org/18228308:35
*** aarefiev has joined #openstack-nova08:35
*** jecarey has quit IRC08:36
*** alex_klimov has quit IRC08:38
*** yassine has joined #openstack-nova08:40
*** oro_ has quit IRC08:41
*** exploreshaifali has joined #openstack-nova08:41
*** kmARC has joined #openstack-nova08:42
*** markvoelker has joined #openstack-nova08:44
*** rdo has joined #openstack-nova08:45
*** exploreshaifali has quit IRC08:46
*** salv-orlando has joined #openstack-nova08:46
*** rajesht has quit IRC08:47
*** markvoelker has quit IRC08:48
*** aix has joined #openstack-nova08:48
*** shyu has quit IRC08:48
johnthetubaguydeepakcs: do you have all the code up for review for that spec now?08:50
johnthetubaguydeepakcs: I am meaning this one: https://blueprints.launchpad.net/nova/+spec/volume-snapshot-improvements08:50
*** belmoreira has quit IRC08:51
deepakcsjohnthetubaguy: yes @ https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/volume-snapshot-improvements,n,z08:51
deepakcsjohnthetubaguy: working on review comments for the "needed by" patch08:51
*** ssurana has quit IRC08:52
openstackgerritDmitry Guryanov proposed openstack/nova: libvirt: do relative block rebase only with non-null base  https://review.openstack.org/20244208:52
johnthetubaguydeepakcs: can you please mark it as NeedsCodeReview, that should get it more attention I hope08:52
deepakcsjohnthetubaguy: maybe i missed something, how to mark it as NeedsCodeReview ?08:53
johnthetubaguydeepakcs: where it says started, you should be able to move it to NeedsCodeReview08:53
johnthetubaguythere should be a little pencil next to it08:53
deepakcsjohnthetubaguy: ah in the lp link, i was looking at the patch link :)08:54
deepakcsjohnthetubaguy: done, my bad, didn't realise i had to do that in lp08:55
*** shuquan has quit IRC08:55
johnthetubaguydeepakcs: no worries, it just helps tell people when you are done, otherwise its hard to tell08:55
deepakcsjohnthetubaguy: agree, I had my patches in WIP for some time, before i moved them to 'ready for review', but forgot abt the LP part08:56
*** park_ has quit IRC08:56
johnthetubaguydeepakcs: easy done, no worries, all fixed now08:57
*** gszasz has joined #openstack-nova08:57
deepakcsjohnthetubaguy: :)08:58
*** markvoelker has joined #openstack-nova08:58
*** Jeffrey4l has quit IRC08:58
*** foexle has quit IRC09:00
*** foexle has joined #openstack-nova09:01
nagyz_i know not the best channel but does any one of you have a link to recent keystoneclient middleware documentation? I'm trying to understand what's needed and what's not needed in the authtoken configuration as it seems to be a mess and differ project by project09:01
*** jichen has quit IRC09:01
*** doude has joined #openstack-nova09:03
*** doude has quit IRC09:03
*** doude has joined #openstack-nova09:03
*** markvoelker has quit IRC09:03
*** shuquan has joined #openstack-nova09:05
*** belmoreira has joined #openstack-nova09:05
*** yonglihe has quit IRC09:06
*** Jhelf has quit IRC09:06
*** alex_klimov has joined #openstack-nova09:06
openstackgerritGhanshyam Mann proposed openstack/nova: Show 'locked_by' information in server Show & List Detail  https://review.openstack.org/20239109:07
*** lyanchih has quit IRC09:08
*** lyanchih has joined #openstack-nova09:09
*** otter768 has joined #openstack-nova09:11
*** prashantvarma has joined #openstack-nova09:11
openstackgerritGhanshyam Mann proposed openstack/nova: Show 'locked_by' information in server Show & List Detail  https://review.openstack.org/20239109:12
*** tpeoples has quit IRC09:12
*** bkopilov has quit IRC09:14
*** ndipanov has joined #openstack-nova09:14
*** otter768 has quit IRC09:16
openstackgerritlyanchih proposed openstack/nova: NUMATopologyFilter raise exception and not continue filter next node  https://review.openstack.org/20063009:16
*** Jeffrey4l has joined #openstack-nova09:17
*** pixelbeat_ has joined #openstack-nova09:19
openstackgerritDmitry Guryanov proposed openstack/nova: libvirt: do relative block rebase only with non-null base  https://review.openstack.org/20244209:22
openstackgerritWenPing Zhang proposed openstack/nova: blueprint support-yfs-storage-protocol  https://review.openstack.org/20245909:25
alex_xu_andreykurilin: one more explain for 'get_current', you can try request to '/2.1' or '/2' instead of '/', you can get current endpoint's version info, then you needn't find out which version info belong to your current endpoint with '/'09:27
*** alex_xu_ is now known as alex_xu09:27
*** bkopilov has joined #openstack-nova09:28
andreykurilinalex_xu: hi! ok. I'll try it now09:29
alex_xuandreykurilin: cool, thanks!09:29
andreykurilinalex_xu: btw, I don't see your mark for https://review.openstack.org/#/c/167408/ :)09:29
alex_xuandreykurilin: yea, I want to see more how thing works more in later patch before give my point for that one09:30
andreykurilinalex_xu: you can check it with https://review.openstack.org/#/c/136458/ , since other patches add only support of "latest"09:31
*** rubasov has joined #openstack-nova09:31
alex_xuandreykurilin: yea, that patch need update?09:32
alex_xuandreykurilin: I didn't saw the detail yet, just saw the unittest failed09:33
andreykurilinalex_xu: only recheck is needed due too issue with mock09:33
alex_xuandreykurilin: ok, cool, will try that patch later09:33
andreykurilinwhich was fixed in master09:33
andreykurilinalex_xu: nice09:33
*** prashantvarma has quit IRC09:34
*** shyama has joined #openstack-nova09:35
openstackgerritJohn Garbutt proposed openstack/nova: Adding support for InfiniBand SR-IOV vif type  https://review.openstack.org/18705209:37
*** annashen has joined #openstack-nova09:38
*** salv-orlando has quit IRC09:40
*** annashen has quit IRC09:42
*** sudipto has quit IRC09:42
*** sudipto has joined #openstack-nova09:48
*** kjelly has quit IRC09:48
*** kjelly has joined #openstack-nova09:48
*** kjelly has quit IRC09:48
*** deepthi has quit IRC09:50
johnthetubaguysahid: I am just looking at https://blueprints.launchpad.net/nova/+spec/convert-consoles-to-objects as there is no code up for review yet, I think that needs to be delayed until M, let me know if I am missing something09:50
sdaguehaypo: +A09:50
sahidjohnthetubaguy: the spec has been approved 10days ago so yes.09:52
johnthetubaguysahid: yeah, I think we need to keep aiming to get specs approved before the summit, its kinda always been the intention, but we have never really managed it09:54
johnthetubaguythe burn out at the end of the release doesn't help, but hopefully the backlog being open should help this09:54
*** ociuhandu has joined #openstack-nova09:55
openstackgerritJeffrey Zhang proposed openstack/nova: Add total_vms field to ComputeNode  https://review.openstack.org/20247209:57
*** Jeffrey4l has quit IRC09:57
openstackgerritChangBo Guo(gcb) proposed openstack/nova: [WIP] Use explicit greenthread.sleep instead of time.sleep  https://review.openstack.org/20236509:58
hayposdague: cool, thanks09:58
hayposdague: did you see my comments on dims patch for reseeding OpenSSL RNG after fork? i proposed to use /dev/urandom instead09:58
*** aix has quit IRC09:58
hayposdague: it's now recommended by most security guys09:59
sdaguefor which thing?09:59
sdaguereally, security people are recommending urandom for seeding openssl?09:59
sdaguethat seems terrible09:59
hayposdague: no. they recommand to only use /dev/urandom, not use OpenSSL for random10:00
andreykurilinalex_xu: oh... HTTPClient is not friendly-written:( there is no ability to send get request to "url/v21", it can send only to "url/v21/<token>" :( so we need to list all versions when HTTPClient is used and rewrite it in future10:00
sdaguewhere is the entropy being consumed?10:00
*** shuquan has quit IRC10:01
sdaguebecause if the answer is for keypairs, then I think the answer should be a definitive no10:01
andreykurilinalex_xu: OR we can make simple "request.get", since we don't need authentication stuff10:01
hayposdague: the patch, https://review.openstack.org/#/c/191581/10:01
sdaguebecause otherwise you get this - https://www.schneier.com/blog/archives/2014/01/security_risks_9.html10:02
hayposdague: it's crypto/utils.py in oslo incubator. i don't know how it is used10:02
hayposdague: i understood that /dev/urandom is good enough to generate keys10:02
sdagueright, that's what scares the crap out of me10:02
sdaguehaypo: please provide a reference for that fact10:02
*** claudiub has joined #openstack-nova10:03
*** zhenguo has quit IRC10:03
hayposdague: i gave links: https://review.openstack.org/#/c/191581/2/openstack/common/crypto/utils.py & https://review.openstack.org/#/c/191581/5/openstack/common/crypto/utils.py10:04
sudiptojohnthetubaguy, is the rule to not have features in which are half way through implementation - even though it might be non-priority? :( It sounds a bit unfair for people who have got their patches half way through the review :(10:05
*** hshiina has joined #openstack-nova10:05
johnthetubaguysudipto: so the idea is to optimise for completed blueprints by 30th July10:05
sdaguehaypo: so that assumes that this is on systems that aren't fresh created and have accumulated enough entropy in their buffers10:05
johnthetubaguysudipto: I am starting with blueprints with no code up for review10:05
sdaguewhich, seems like a poor assumption in cloud10:06
sdagueif these are freshly deployed compute nodes10:06
johnthetubaguysdague: yeah I think thats why we have nova-cert10:06
sdagueso the problem is, we have no idea where this function is going to be used10:06
sudiptojohnthetubaguy, ok - so if a spec has half the code in now, and let's say half the code done in the next couple of days (before July 30th) - will it still be considered?10:07
sdaguejohnthetubaguy: sure, but I'm concerned about hiding this fact behind oslo layers10:07
johnthetubaguysdague: yeah, true, its easy to use by accident in the wrong place10:07
johnthetubaguysudipto: maybe, we should talk about your specific case, technically, the rule is, get all your code up before today, we had 104 approved blueprints this morning, most of the low priority ones do have code up for review already, so we want to give them preference10:09
sudiptojohnthetubaguy, ok.10:09
*** afazekas has joined #openstack-nova10:10
sdagueright, having been ansilarily involved in a firedrill where all the certs on a range of embed systems were brute forcable, I'd like us to not put that bug into our system10:10
johnthetubaguysudipto: to set expectations, I can't see us merging more that 50% of the blueprints that are already in NeedsCodeReview before 30th, we just don't have enough folks reviewing code right now :(10:11
sdaguehaypo: this was the incident - http://www.bishopfox.com/blog/2012/03/ssl-key-generation-weaknesses/10:11
sudiptojohnthetubaguy, i have 2 code drops scheduled. My first code has been up for review since July 7th, and finally - we have it in some shape to be merged. But when  I heard about the deadline, i am not sure if i should go  ahead with the second code drop...10:11
johnthetubaguysudipto: what is the second code drop?10:12
*** ociuhandu has quit IRC10:12
sdagueand the root cause, was these systems did not have sufficient entropy.10:12
johnthetubaguysudipto: I guess I am trying to work out why its not up for review already10:13
*** liusheng has quit IRC10:13
*** liusheng has joined #openstack-nova10:13
sudiptojohnthetubaguy, ok, as you might know the recent compute monitor framework went for a change - it's using the nova objects now - and for my blueprint, i needed to introduce a nova object that would support reporting of bw on NUMA nodes. So this was code drop 1. So we reached a consensus on the object 2 days back. Then the second code drop was meant to be for the reporting bit (the metrics - that would be actually fetched via PCP)...10:14
hayposdague: can you please add a comment giving your two links? i don't have the bandwidth yet to digest these two interesting articles10:14
sudiptojohnthetubaguy, I am wondering, if it's a good idea to have the code drop 2 now ....10:14
sdaguehaypo: sure10:15
johnthetubaguysudipto: if you haven't started that code yet, possibly not worth it10:15
sdaguehaypo: honestly, this is a case where I think getting the OpenStack Security team involved in this review would be good. They exist for a reason. If I'm being overly paranoid and they say so, I'm fine with it.10:15
sudiptojohnthetubaguy, honestly, i have the code ready - however, I am yet to finish on the UTs...10:15
sdagueBut I also know it's a common mistake, and rng for crypto is hard10:16
johnthetubaguysudipto: OK, understood, I mean you can try, just upload what you have, and mark it as WIP, thats a middle ground, but honestly, its getting very close to the wire at this point :(10:16
sudiptojohnthetubaguy, ok that's very reasonable. Just marred by the situation i guess :(10:17
*** garyk has quit IRC10:18
_gryfjohnthetubaguy, hi10:18
_gryfjohnthetubaguy, i've changed the status of the BP to NeedsCodeReview this morning (since the implementation is done and half of the patches are already accepted)10:19
johnthetubaguysudipto: yeah, this freeze is all about getting the things we agreed as a priority done (like objects), and getting as much value for our users by getting completed blueprints merged, but yeah, this is not a happy time in the released10:19
_gryfjohnthetubaguy, do it is enough to make it still i l release instead of m?10:20
johnthetubaguy_gryf: sounds good, yeah NeedsCodeReview = all the code us submitted for review10:20
_gryfjohnthetubaguy, cool, since it would be scary to wait entire cycle to make it merge :)10:21
johnthetubaguy_gryf: its just a case of getting the code merged before the 30th July10:21
johnthetubaguythe big message here is, the more reviews everyone does of each others code, the more likely we can get more blueprints merged10:22
_gryfjohnthetubaguy, ack. Who can I ask for review changes which touches the topics of service groups and REST API?10:22
_gryfjohnthetubaguy, (i mean the core team)10:23
_gryfjohnthetubaguy, since one of my pacth already have a bunch of +110:23
*** dims_ has joined #openstack-nova10:24
johnthetubaguy_gryf: its a waiting game at this point, but for the REST API, there is the API subteam with some great reviewers who can help10:25
_gryfjohnthetubaguy, ok, thanx for the info10:26
*** _gryf has quit IRC10:26
*** pixelbeat_ has quit IRC10:27
*** aix has joined #openstack-nova10:28
*** _gryf has joined #openstack-nova10:30
openstackgerritSudipta Biswas proposed openstack/nova: Allow NUMA based reporting for Monitors  https://review.openstack.org/19897210:32
*** afazekas has quit IRC10:35
*** Jeffrey4l has joined #openstack-nova10:36
*** dims_ has quit IRC10:39
*** salv-orlando has joined #openstack-nova10:40
*** lyanchih has quit IRC10:43
*** exploreshaifali has joined #openstack-nova10:43
openstackgerritWenPing Zhang proposed openstack/nova: blueprint support-yfs-storage-protocol  https://review.openstack.org/20245910:45
*** shyama has quit IRC10:46
*** shyama_ has joined #openstack-nova10:46
*** shyama_ is now known as shyama10:46
*** sudipto has quit IRC10:46
*** afazekas has joined #openstack-nova10:48
*** garyk has joined #openstack-nova10:54
*** IanGovett has joined #openstack-nova10:55
*** coolsvap has quit IRC10:58
*** coolsvap has joined #openstack-nova10:58
johnthetubaguypark_heijlong: just looking at the state of this blueprint, I don't think all the code is up for review yet? https://blueprints.launchpad.net/nova/+spec/add-id-to-security-groups-for-server-show10:59
ndipanovso found this little gremlin https://bugs.launchpad.net/nova/+bug/147525410:59
openstackLaunchpad bug 1475254 in OpenStack Compute (nova) "NovaObjectSerializer cannot handle backporting a nested object" [High,Confirmed]10:59
ndipanovmorale of the story - humans should not raise exceptions in recursive functions11:01
*** sudipto has joined #openstack-nova11:02
*** mwagner_lap has quit IRC11:05
openstackgerritPaul Murray proposed openstack/nova: Remove conductor api from resource tracker  https://review.openstack.org/20250111:08
*** otter768 has joined #openstack-nova11:12
openstackgerritDmitry Guryanov proposed openstack/nova: libvirt: do relative block rebase only with non-null base  https://review.openstack.org/20244211:12
* alex_xu sending text to park_heijlong, notify him...11:12
*** belmoreira has quit IRC11:15
openstackgerritPaul Murray proposed openstack/nova: Remove conductor api from resource tracker  https://review.openstack.org/20250111:15
*** xgsa has quit IRC11:16
*** xgsa has joined #openstack-nova11:16
openstackgerritsahid proposed openstack/nova: hardware: take into account pages size when computing memory  https://review.openstack.org/20250411:16
sahidndipanov: can you take a look at this, it's still work in progress https://review.openstack.org/20250411:16
*** otter768 has quit IRC11:17
sahidit's about how we compute memory usage when host have kind of pages11:17
*** IanGovett has quit IRC11:17
openstackgerritGhanshyam Mann proposed openstack/nova: Show 'locked_by' information in server Show & List Detail  https://review.openstack.org/20239111:21
*** xgsa has quit IRC11:23
*** xgsa has joined #openstack-nova11:24
*** pixelbeat_ has joined #openstack-nova11:24
*** lyanchih has joined #openstack-nova11:24
*** lucasagomes is now known as lucas-hungry11:25
*** veena has quit IRC11:26
openstackgerritSergey Vilgelm proposed openstack/nova: Switch to the oslo_utils.fileutils  https://review.openstack.org/19957511:35
*** GLaupre has quit IRC11:37
bauzasjohnthetubaguy: edleafe: I see the email for the scheduler, I'll try to reply for it today11:40
*** belmoreira has joined #openstack-nova11:40
bauzasjohnthetubaguy: edleafe: tl;dr: anyone can do anything out of tree, and then coming back for discussing11:40
bauzasjohnthetubaguy: edleafe: but we need to be very clear on modifying the current in-tree schedulers11:41
johnthetubaguybauzas: yep, but I appreciate people like edleafe being up ideas for discussion, thats really healthy11:41
bauzasjohnthetubaguy: sorry, I pinged edleafe now even if I know he's AFK due to his TZ11:41
*** jichen has joined #openstack-nova11:43
*** aysyd has joined #openstack-nova11:45
*** thorst has joined #openstack-nova11:47
garykis there any chance that one or more cores can please look at https://review.openstack.org/#/c/188789/ - there are tons of patches waiting for this one to get in. this is starting the metadata object support. please i implore you11:54
*** Longgeek has quit IRC11:54
*** IanGovett has joined #openstack-nova11:58
*** dprince has joined #openstack-nova12:02
*** ptm_away is now known as PaulMurray12:03
PaulMurrayndipanov, have you seen https://bugs.launchpad.net/nova/+bug/127567512:04
openstackLaunchpad bug 1275675 in OpenStack Compute (nova) "Version change in ObjectField does not work with back-levelling" [Wishlist,Confirmed] - Assigned to Paul Murray (pmurray)12:04
*** bkopilov has quit IRC12:04
PaulMurrayndipanov, and https://review.openstack.org/#/c/78605/12:04
* PaulMurray didn't assign that to himself??12:05
*** Longgeek has joined #openstack-nova12:05
* PaulMurray must have clicked on something12:05
openstackgerritJordan Pittier proposed openstack/nova: libvirt:when deleting snapshot use qemu-img to blockRebase if domain is stopped  https://review.openstack.org/19273612:07
openstackgerritJordan Pittier proposed openstack/nova: libvirt:when deleting snapshot use qemu-img to blockRebase if domain is stopped  https://review.openstack.org/19273612:08
*** lyanchih has quit IRC12:08
*** vladikr has joined #openstack-nova12:11
* kashyap notes randomly: if you're using `gertty` and like to view the Gate job logs on CLI, invoke it this way: BROWSER=lynx gertty (thanks fungi).12:12
*** deepakcs has quit IRC12:13
*** dims_ has joined #openstack-nova12:14
*** tellesnobrega_ has joined #openstack-nova12:16
*** tellesnobrega_ has quit IRC12:16
dims_sdague: good morning, i ran devstack with nova + oslo.service here (https://review.openstack.org/#/c/202075/) - grenade jobs seem to be doing ok.12:17
*** nkrinner is now known as nkrinner_afk12:18
*** jcoufal has joined #openstack-nova12:19
ndipanovPaulMurray, no12:20
ndipanovbut your fix is not correct I think12:20
ndipanovnow that I look at it12:20
ndipanovadding tests now12:20
ndipanovwill post a fix soon12:21
ndipanovbut I think it's the same thing12:21
*** redbeard has quit IRC12:22
ndipanovPaulMurray, it's the same bug12:22
ndipanovPaulMurray, how come you just abandoned the fix12:24
ndipanovit's a pretty bad bug12:24
sdaguedims_: so, actually it's gate-devstack-dsvm-updown that we care about12:26
sdaguethe grenade jobs won't see it until the next release, because kilo is still the old code12:26
sdaguebut, I see we have a bunch of runs there12:27
sdagueso I'm good with it12:27
*** doude has quit IRC12:28
*** READ10 has joined #openstack-nova12:29
*** markvoelker has joined #openstack-nova12:29
dims_sdague: ack. thanks!12:29
*** doude has joined #openstack-nova12:29
*** dims_ has quit IRC12:30
*** shuquan has joined #openstack-nova12:30
*** salv-orlando has quit IRC12:31
*** dims__ has joined #openstack-nova12:32
johnthetubaguysdague: do you know where we document what "dsvm-updown" does? I have lost track a bit12:33
sdaguejohnthetubaguy: it's not all that documented, it's a devstack functional job12:33
johnthetubaguysdague: it was the "updown" bit, is that turning the services off an on again?12:34
*** markvoelker has quit IRC12:34
sdagueit's turning them off again12:34
sdagueso bring everything up, and shut it all down12:35
*** shuquan has quit IRC12:35
sdagueto make sure that shutting down works12:35
johnthetubaguycool, that makes sense12:35
johnthetubaguyits not updownup12:35
PaulMurrayndipanov, I abandoned the fix because I got into one of those "is this what we really want to do" discussions and it was down as a wish...........12:35
sdagueso that there isn't a big run around at release when we try to flip grenade to a new stable, and then we find out kill on services is broken12:36
sdaguewhich... was the kilo situation12:36
ndipanovPaulMurray, bs - it's a horrible bug12:36
PaulMurrayndipanov, also the line at the time was this shouldn't happen so its more like a runtime sanity check12:36
ndipanovalso bs12:36
johnthetubaguysdague: +1 its awesome to add that test in12:36
PaulMurrayndipanov, but I do agree with you12:36
ndipanovit is happening to me now12:36
ndipanovwe can't land a fix because of this12:36
ndipanovI will mark my bug as duplicate and post a fix12:36
ndipanova fix is very simple actually12:36
ndipanovthe fix that is12:37
PaulMurrayndipanov, cool - I'll take a look. Actually I am about to go on holiday for a couple of weeks - so I might not be around for long12:37
PaulMurrayend of week that is12:38
*** redbeard has joined #openstack-nova12:38
*** bfic has joined #openstack-nova12:38
*** sudipto has quit IRC12:39
*** dims__ has quit IRC12:39
*** IanGovett has quit IRC12:39
*** IanGovett1 has joined #openstack-nova12:39
*** park_ has joined #openstack-nova12:40
ndipanovyeah I'll post this in like 20 min12:40
*** park__ has joined #openstack-nova12:40
ndipanovjust to get the tests working PaulMurray12:40
*** lpetrut has quit IRC12:41
*** park_ has quit IRC12:41
*** park__ has quit IRC12:41
*** park_ has joined #openstack-nova12:41
*** park_ has quit IRC12:42
*** park_heijlong_ has joined #openstack-nova12:42
*** markvoelker has joined #openstack-nova12:43
*** IanGovett has joined #openstack-nova12:44
*** shuquan has joined #openstack-nova12:44
*** krtaylor has quit IRC12:44
*** IanGovett1 has quit IRC12:45
park_heijlong_johnthetubaguy: hi, for the bp https://blueprints.launchpad.net/nova/+spec/add-id-to-security-groups-for-server-show, there is only 1 patch left, I could finish it this night12:46
park_heijlong_johnthetubaguy: could I catch the L release freeze..?12:47
johnthetubaguypark_heijlong_: OK, if you can get that up, drop me an email, and I will see if we can sneak that in. The big problem, is thats its really close to the wire to get enough reviews to get it merged by 30th July12:47
*** kylek3h has joined #openstack-nova12:48
*** markvoelker has quit IRC12:48
*** cpschult has joined #openstack-nova12:48
park_heijlong_johnthetubaguy: yep, make sense, I will try best to... thank you in advance12:49
*** lucas-hungry is now known as lucasagomes12:49
*** IanGovett1 has joined #openstack-nova12:49
johnthetubaguysdague: I think we spoke before about documenting these ideas, would love a quick review of some of this, if you get chance: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule#How_can_I_get_my_code_merged_faster.3F12:49
*** jcoufal has quit IRC12:50
*** IanGovett has quit IRC12:50
*** krtaylor has joined #openstack-nova12:51
*** mjturek1 has joined #openstack-nova12:52
*** salv-orlando has joined #openstack-nova12:53
*** vladikr has quit IRC12:54
*** lpetrut has joined #openstack-nova12:54
markus_zWhat's your strategy to deprecate (private) methods which get deleted in a following patch set?12:54
markus_zI couldn't found a decorator for that.12:54
*** IanGovett has joined #openstack-nova12:54
markus_zs/found/find12:55
*** jcoufal has joined #openstack-nova12:55
*** diogogmt has quit IRC12:55
*** bfic has quit IRC12:55
*** diogogmt has joined #openstack-nova12:56
*** IanGovett1 has quit IRC12:56
edleafetonyb: still around? You just missed me earlier12:57
*** markvoelker has joined #openstack-nova12:57
*** thorst has quit IRC12:58
johnthetubaguyedleafe: saw your mail about the scheduler, sounds interesting, just wondered if you saw my attempted write up of the ideas I raised at the summit: https://review.openstack.org/#/c/191914/12:59
*** jcoufal has quit IRC12:59
edleafejohnthetubaguy: sure, I've read it, and it covers what I remember being discussed at the summit13:00
*** zhenguo has joined #openstack-nova13:01
*** markvoelker has quit IRC13:02
*** mwagner_lap has joined #openstack-nova13:02
johnthetubaguyedleafe: cool, I am just being cheaky and seeing if I could get someone to try the kafka idea (its basically a rehash of the no-db-scheduler blueprint)13:02
edleafejohnthetubaguy: I didn't add my ideas because we hadn't talked about them13:02
johnthetubaguyedleafe: another backlog spec is welcome, but trying them out is probably more useful13:03
edleafejohnthetubaguy: one thing I know is that the community can argue forever on an idea13:03
johnthetubaguyedleafe: certainly, I see the bottleneck as the data sync, so its the right bit to focus on, for max benefit13:03
edleafejohnthetubaguy: I wanted to try to present some numbers first13:03
edleafejohnthetubaguy: it's also key to separation, oddly enough13:04
edleafejohnthetubaguy: if this is successful, there is no reason that it wouldn't be available to cinder, neutron, etc13:04
*** bfic has joined #openstack-nova13:04
*** thorst has joined #openstack-nova13:04
johnthetubaguyedleafe: so we will want a single block of state for that I think, but thats a different discussion13:05
edleafejohnthetubaguy: yes13:05
bauzasedleafe: I'm trying to provide you a reply by today13:05
*** markvoelker has joined #openstack-nova13:05
edleafejohnthetubaguy: what I hope to get out of the midcycle are some goals to shoot for13:05
bauzasedleafe: to be clear, I like to see some out-of-tree drivers for that13:05
johnthetubaguyedleafe: I am trying to sketch out the direction of where we are thinking in devref, so we can get more folks involved: https://review.openstack.org/#/c/192260/13:06
bauzasedleafe: but I wonder how to do that correctly in-tree13:06
*** mdrabe has joined #openstack-nova13:06
edleafebauzas: this is simply a POC experiment13:06
bauzasedleafe: if so, why do you need some help/view ?13:06
*** markvoelker_ has joined #openstack-nova13:06
edleafebauzas: it is a waste of time to debate how to integrate it until we know that it's worth the effort13:06
bauzasedleafe: well, anyone can POC a new driver13:07
edleafebauzas: I need to know what sort of results would be definitive13:07
johnthetubaguyedleafe: yeah, thats a good goal, hopefully this is a starting point: https://review.openstack.org/#/c/192260/13:07
edleafebauzas: it's not a driver13:07
bauzasedleafe: so, johnthetubaguy's backlog spec is there for that13:07
edleafebauzas: it's a different approach to sharing data13:07
bauzasedleafe: I know Cassandra , but I would like to understand why13:08
johnthetubaguyedleafe: so you can plug that as a new driver, I think this pattern would work: https://github.com/openstack/nova/blob/master/nova/scheduler/caching_scheduler.py13:08
bauzasedleafe: because here, we don't need a technology, rather a design disussion13:08
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Add support of latest version in CLI  https://review.openstack.org/18428113:08
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Add "get_current" method to VersionManager  https://review.openstack.org/20229613:08
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 1  https://review.openstack.org/15256913:08
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Set "latest" as default compute api version  https://review.openstack.org/18465813:08
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 2  https://review.openstack.org/16740813:08
edleafebauzas: because I have seen it perform in situations that had the same issues as we are having13:08
*** itzikb_ has quit IRC13:09
edleafebauzas: the technology enables the design13:09
bauzasedleafe: please don't13:09
bauzasedleafe: see that http://sbauza.github.io/2015/04/20/#/4 ?13:09
edleafebauzas: we have been trying to re-invent much of that technology using other things13:09
johnthetubaguybauzas: its just one implementation of a different approach, giving it a whirl in a prototype is fine I think13:09
bauzasedleafe: anyone can work on a new driver and see how it works13:09
*** rlrossit has joined #openstack-nova13:09
*** markvoelker has quit IRC13:10
*** yingjun has joined #openstack-nova13:10
bauzasedleafe: you only have to implement a very limited interface for your own scheduler13:10
bauzasedleafe: and that can be called thanks to an opt13:10
edleafebauzas: you don't seem to understand that this isn't simply "another driver"13:11
bauzasedleafe: so I'm really +1 on giving you room for that13:11
openstackgerritgaryk proposed openstack/nova: Neutron: configure requested amount of fixed_ips  https://review.openstack.org/16386413:11
garykmriedem_away: you have a sec ti talk about https://github.com/openstack/nova/commit/1ba72081715869fe6d0ffba3436fb09ef9499335?13:11
edleafebauzas: another driver would have to consume the same data in the same way, and emit it in the same way13:11
garykthe change prevents us from deleting an instance …13:11
bauzasedleafe: you're talking about a backend, right?13:12
*** vladikr has joined #openstack-nova13:12
bauzasedleafe: incorrect13:12
bauzasedleafe: another driver can provide its own way to consume resources13:12
edleafebauzas: all I'm asking is "What results would impress you so much that it would be worth considering this type of change?"13:12
bauzasedleafe: be sure that lots of alternatives exist out-of-tree13:12
edleafebauzas: this would require changing how resources are reported, too13:12
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Adds support for x509 certificates as keypairs  https://review.openstack.org/13645813:13
*** otter768 has joined #openstack-nova13:13
edleafebauzas: don't fixate on a particular solution13:13
bauzasedleafe: my take on that is to say - anyone can do anything he wants13:13
*** coolsvap is now known as coolsvap|away13:13
edleafebauzas: tell me what I could show you that would be significant13:13
bauzasedleafe: but if that requires in-tree implementation, take it in small steps13:13
edleafebauzas: other experiments could use the same metrics13:13
edleafebauzas: this might meet those expectations, but others could, too13:13
bauzasedleafe: right, what we need to consider is the migration path from the current situation to the one you wabnt13:14
edleafebauzas: you're getting ahead of things again13:14
bauzasedleafe: that's exactly why we're working on fixing stuff and reducing debt13:14
edleafediscussing what small steps we should take should come *after* we know it's going to be worth it, not before13:14
*** bnemec has joined #openstack-nova13:14
bauzasedleafe: so, play with the POC, use it even downstream13:14
johnthetubaguyedleafe: for me, look at doing 1000 builds in parallel with a setup where you have space for many 1100 builds, and do that as quickly as possible, with minimal retries, at least thats the kind of thing I am worried about13:15
edleafebauzas: you seem to want to iron out all the details before we even know that this will be an improvement13:15
edleafejohnthetubaguy: thank you!13:15
edleafejohnthetubaguy: finally someone understands what I asked! :)13:15
bauzasokay, I appreciate your tone :)13:16
* bauzas looking back to code13:16
johnthetubaguyedleafe: ah, so I miss-read your email then, I didn't get thats what you wanted, but yeah, thats the use case that sucks right now :)13:16
*** dims_ has joined #openstack-nova13:16
mnaseri've had this patch for around a week now with 2 +1s.. any cores free to have a look over?  it's a pretty straight forward patch.. https://review.openstack.org/#/c/199516/13:16
johnthetubaguyedleafe: parallelism in the current schedule means more bad descisions13:16
edleafejohnthetubaguy: I know that I can experiment out-of-tree without anyone's approval13:17
edleafejohnthetubaguy: I only want to know what sort of results would make the community want to consider possibly taking it in-tree13:17
*** cpschult has quit IRC13:17
*** otter768 has quit IRC13:17
johnthetubaguyedleafe: yeah, I understand that now13:17
johnthetubaguyedleafe: the two reviews should help describe that: https://review.openstack.org/200001 https://review.openstack.org/#/c/191914 if they don't then need more work13:18
johnthetubaguyedleafe: to be clear I am fairly sure they need a lot more work, but I would love some ideas on how to get what you need from them13:19
edleafejohnthetubaguy: thanks. I'll be sure to absorb them before the midcycle13:19
*** jichen_ has joined #openstack-nova13:19
edleafejohnthetubaguy: I like playing with ideas without focusing on why they *won't* work13:20
johnthetubaguyedleafe: the first one is the more important one, getting a devref to describe the scheduler work13:20
openstackgerritNikola Dipanov proposed openstack/nova: Fix the incorrect PciDeviceList version number  https://review.openstack.org/20128013:20
openstackgerritNikola Dipanov proposed openstack/nova: objects: fix deserialization with nested backleveling  https://review.openstack.org/20255413:20
ndipanovdansmith, mriedem_away ^^13:20
*** jichen has quit IRC13:20
*** jichen_ is now known as jichen13:20
johnthetubaguyedleafe: its a really useful approach for performance work13:20
ndipanovwe should really land the first patch asap - I will propose a kilo backport too13:21
ndipanovPaulMurray, ^^13:21
nihiliferbauzas: do you have some time for review today?13:21
*** ivasev has joined #openstack-nova13:21
johnthetubaguyedleafe: the stuff I tried and prototyped last year (that generated the CachingScheduler) is what has lead me in this current direction13:21
edleafejohnthetubaguy: ok, I'll dig into it13:21
johnthetubaguyedleafe: fully recommend the crazy protoype, it was extremely educational!13:22
nihiliferbauzas: just want to know whether the approach in https://review.openstack.org/#/c/195783/ is now good for you13:22
edleafejohnthetubaguy: :)13:22
andreykurilinsdague, johnthetubaguy, alex_xu: hi! I fixed all patches in novaclient based on our yesterday's discussion.13:22
andreykurilin*the day before yesterday13:22
bauzasedleafe: were you at the session about scaling out the scheduler ? https://etherpad.openstack.org/p/YVR-nova-scalling-out-scheduler-for-cells13:23
edleafebauzas: of course13:23
*** zzzeek has joined #openstack-nova13:24
*** ctrath has joined #openstack-nova13:24
bauzasedleafe: okay, so I feel that something is unclear to me13:24
bauzasedleafe: by reading again your idea, you're thinking about changing how we report those metrics so we can scale13:24
haypojohnthetubaguy: sorry, but i'm not sure that i understood the status "NeedsCoreReview" for https://blueprints.launchpad.net/nova/+spec/nova-python3 -- can I continue to submit python3 patches next weeks? or should I wait until november?13:24
johnthetubaguyhaypo: you should wait, really, at least after the end of today, I think you should wait13:25
bauzasedleafe: but that sounds very close to what we discussed during that session13:25
bauzasedleafe: and where the outcome was "we need to define first our thoughts"13:25
edleafebauzas: well, yes, it certainly sparked my thinking13:26
edleafebauzas: when I first created cells ("zones" back then) I had modeled them using Cassandra13:26
johnthetubaguyhaypo: the basic idea is that we need to get what we have up for review merged by 30th June, after that point, we try to just focus on the priority stuff13:26
edleafebauzas: that got shot down by the "MySQL Everywhere" sentiment13:27
openstackgerritDmitry Guryanov proposed openstack/nova: libvirt: do relative block rebase only with non-null base  https://review.openstack.org/20244213:27
bauzasedleafe: I really want to stop talking about implementations :)13:27
edleafebauzas: all the problems of the original zones design came from trying to force one design into another implementation13:27
bauzasedleafe: what we need is a way to provide resources to a distributed system13:27
*** gyee has joined #openstack-nova13:27
*** edmondsw has joined #openstack-nova13:28
edleafebauzas: hmmmm... what could possibly do that? :)13:28
edleafebauzas: let's please stop arguing about one approach over another, until we have some results13:28
edleafebauzas: let's make choices based on data, not feelings or biases13:29
edleafebauzas: if I run this experiment and the numbers don't impress, I'll be the first person to say "don't do this"13:29
edleafebauzas: but I think it's a bad idea for long-term development to shoot down experiments before they are run13:30
*** ankit has quit IRC13:30
*** yingjun has quit IRC13:31
*** yingjun has joined #openstack-nova13:31
bauzasedleafe: again, I really want that experiment13:32
bauzasedleafe: I even proposed one https://etherpad.openstack.org/p/liberty-nova-scalable-scheduler13:32
*** jlearman has joined #openstack-nova13:33
bauzasedleafe: so that's pretty cool to see you're volunteer for working on that13:33
edleafebauzas: did you ever get a chance to try it?13:33
bauzasedleafe: and your proposal is very close to what johnthetubaguy proposed with Kafka13:33
*** jcoufal has joined #openstack-nova13:33
edleafebauzas: so if we can get an experimental setup in place, we could eventually try out several approaches and see which is strongest13:34
edleafebauzas: but that's still getting ahead of ourselves13:34
bauzasedleafe: my proposal was focusing on having a distributed lock for consuming resources13:34
bauzasedleafe: yours and johnthetubaguy are more about how we get those resources13:35
*** jaypipes has joined #openstack-nova13:35
bauzasedleafe: that's where I think it doesn't really need something to be done to be tested13:36
*** yingjun has quit IRC13:36
edleafebauzas: yes: how they are reported, compared, and claimed13:36
johnthetubaguybauzas: I did put both in my spec, thats one of the issues I need to fix in that spec actually13:36
*** thangp has joined #openstack-nova13:36
bauzasedleafe: what I'm missing with your proposal is how it interacts with the existing design13:37
*** bkopilov has joined #openstack-nova13:37
bauzasedleafe: I understood you want more than a drvier13:37
*** lyanchih has joined #openstack-nova13:37
edleafebauzas: not sure I understand13:37
bauzasedleafe: you can't really do a big-bang, right?13:37
edleafebauzas: meaning what?13:38
bauzasedleafe: you have the existing architecture, the scheduler client reporting stats by writing a DB row, the scheduler reading that once and consuming that, and you have the RT claims13:38
bauzasedleafe: you're planning to touch the 3 above things13:39
bauzasedleafe: IIUC13:39
edleafebauzas: yes, it's all open to change in the experiment13:39
bauzasedleafe: but what I'm missing is how you guarantee that we keep a continuous deployment13:40
bauzaswell, not 'CD', but rather a continuous delivery13:40
johnthetubaguybauzas: no of that really matters in this context right?13:40
edleafebauzas: ah, so you're getting back to the in-tree discussion13:40
*** tjones1 has joined #openstack-nova13:40
edleafebauzas: we can talk about that, but only after it's clear that we want to go down that path13:41
bauzasedleafe: well, isn't that the goal ?13:41
ndipanovdansmith, what up with -2s?13:41
ndipanovI can fix the patch13:41
*** tjones1 has left #openstack-nova13:41
ndipanovif we don't fix that we can't fix the version skip13:41
edleafebauzas: from my proposal:13:41
edleafe"I would ask that we acknowledge for now that:13:41
dansmithndipanov: the patch proposes to fundamentally change the behavior, yes?13:41
edleafea) it will be disruptive and painful to switch something like this at13:41
edleafethis point in Nova's development13:41
edleafeb) it would have to provide *significant* improvement to make such a13:41
ndipanovdansmith, pls move to -113:42
edleafechange worthwhile13:42
ndipanovdansmith, let me think if this will fix our case13:42
edleafebauzas: I know that any such change is going to be difficult13:42
edleafebauzas: I don't know that it will be worthwhile13:42
ndipanovI think it would13:42
*** rajesht has joined #openstack-nova13:42
edleafebauzas: the experiment is simply to answer the second point13:42
*** ferest has joined #openstack-nova13:44
bauzasedleafe: okay, then you wonder what is considered "significant", then ?13:44
bauzasin other words, you need to define success KPIs ?13:44
ndipanovdansmith, well it will not work in this case13:44
edleafebauzas: exactly13:44
bauzassorry, success criterias13:44
ndipanovit would cause an infinet loop I think13:44
ndipanovbackport instance <- get the same thing <- call for backport <- get the same thing ...13:45
bauzasedleafe: well, we don't have so many figures upstream, and I feel the best people to reply to that are operatoes13:45
bauzasedleafe: you lost me by providing a technical description to a simple question : what do you consider an an improvement for the scheduler ?13:45
bauzasedleafe: and like I said, the ones who know the best about that are ops13:46
ndipanovdansmith, we could fix this with a two step approach13:46
dansmithndipanov: not if you make the change in the one that fixes the version, or if you drop the relationship first13:46
*** jdandrea has joined #openstack-nova13:46
ndipanovnot sure I follow13:46
* bauzas is just melting on his chair13:47
*** ferest has quit IRC13:47
*** unicell has joined #openstack-nova13:47
*** jlearman has quit IRC13:47
bauzasedleafe: I remember some good chat I had with a RAX operations guy at the summit13:48
*** rajesht has quit IRC13:48
bauzasedleafe: this guy was having good insights for helping how to improve the scheduler13:48
*** unicell1 has quit IRC13:48
bauzassomeone from alaski's team IIRC13:48
*** exploreshaifali has quit IRC13:48
edleafebauzas: I'd certainly like to help the RAX use case13:48
ndipanovdansmith, we could merge a patch wehre we never sends 1.2 to kilo nodes13:48
ndipanovthen fix it in kilo13:48
ndipanovthen fix it in master again to send it to kilo nodes13:49
bauzasedleafe: what I'm trying to explain is that if you need help for that, fire an email to openstack-operators@ and people would be glad to help you13:49
edleafebauzas: I might also need some of their cloud to test across global deployments13:49
ndipanovbut that serializer code is still busted13:49
*** irenab has quit IRC13:49
bauzasedleafe: just mention that you're planning to work on an experiment to identify what could be the benefits but you need to weigh the benefits13:49
bauzasedleafe: not sure the ML thread on -dev@ is that clear too - I'll try to reply it asap13:50
*** bfic has quit IRC13:50
edleafebauzas: sure. These are the sorts of things I want to come away from the midcycle with, so that I can start to create a plan13:50
bauzasedleafe: not sure the midcycle can help more that that13:51
bauzasthan13:51
alaskibauzas: not sure who you would have talked to, it could have been any of the ops folks as they're all pretty opinionated about the scheduler :)13:51
*** jlearman has joined #openstack-nova13:51
edleafealaski: ya think? :)13:51
*** jcoufal has quit IRC13:52
alaskiit's hard not to focus on the pain :)13:52
*** wanghua has quit IRC13:52
alaskiedleafe: johnthetubaguy already outlined one use case, but more generally our interest is in reducing races when we're near capacity13:53
edleafealaski: good to know13:53
edleafealaski: I'll probably need your help designing a test that would show that13:53
alaskiin the current scheduler we have too much contention over a subset of the remaining capacity.  our weighers don't do a great job of spreading requests13:53
dansmithndipanov: if you drop the relationship from master first, so that it will not send 1.2, then write a patch to fix the serializer to ask for the proper top version, add pcidevicelist 1.2, then re-add the relationship you should be good right?13:54
edleafealaski: "your" meaning RAX folks13:54
alaskiedleafe: yes13:54
johnthetubaguyedleafe: alaski: yeah +1, the above scenario I was talking about focuses on that13:54
ndipanovdrop the relationship completely ?13:54
ndipanovor just make sure to never send 1.2 to kilo nodes13:54
dansmithndipanov: it's not getting us anything right now, right?13:54
alaskiedleafe: bursts of builds are another class of problem, but it sort of boils down to the same thing.  too much contention on a subset of available capacity13:55
*** dboik has joined #openstack-nova13:55
ndipanovwhat do you mean by drop the relationship?13:55
*** dboik has quit IRC13:55
ndipanovexactly13:55
*** dboik has joined #openstack-nova13:55
ndipanovdansmith, ^13:55
*** mriedem_away is now known as mriedem13:56
dansmithndipanov: remove it13:56
*** artom has joined #openstack-nova13:56
*** yingjun has joined #openstack-nova13:56
ndipanovlike delete the field from the instance object?13:57
*** signed8bit has joined #openstack-nova13:57
dansmithfrom pcidevicelist13:58
bauzasalaski: that's one of the reasons I was working on a distributed lock, because that's mostly where the conditions at the limits are hit13:58
*** jichen has quit IRC13:58
* ndipanov confused13:58
* bauzas is probably writing Franglish13:58
*** cfriesen has joined #openstack-nova13:58
*** jichen has joined #openstack-nova13:58
*** shuquan has quit IRC13:59
*** redbeard has quit IRC13:59
* edleafe is going to get more coffee14:00
openstackgerritShiina, Hironori proposed openstack/nova: Add a dirver API to inject an NMI  https://review.openstack.org/20238014:00
*** dboik_ has joined #openstack-nova14:00
*** yamahata has joined #openstack-nova14:01
*** thangp has quit IRC14:02
ndipanovdansmith, when you have some time pls explain what you meant in a bit more words... :)14:02
*** Kennan has quit IRC14:02
*** alex_klimov has quit IRC14:03
*** Kennan has joined #openstack-nova14:03
*** dboik has quit IRC14:04
alaskibauzas: yes, a distributed lock would help.  But I think that gets to edleafes point that some of this work is rebuilding a system like cassandra which brings that stuff in "for free"14:06
bauzasalaski: hence johnthetubaguy's backlog spec :p14:06
bauzasalaski: https://review.openstack.org/#/c/191914/14:06
*** thangp has joined #openstack-nova14:07
johnthetubaguyCassandra could be a cool alternative, for the reasons alaski mentions, its worth a quick punt14:07
*** cpschult has joined #openstack-nova14:08
bauzasheh, are we sure Greek-ish stuff are the best ?14:08
* bauzas jk14:08
dansmithndipanov: first, we need to fix the serializer so it asks for the proper top-level version and backport that, yes? until that's done, the rest is moot14:09
ndipanovnope14:09
ndipanovif we do that14:09
ndipanovthen we would get an infinite loop14:09
ndipanovsince kilo serializer causes this14:09
ndipanovso if we fix it and backport it14:10
ndipanovthen fix the version on master14:10
alaskijohnthetubaguy: bauzas to be clear I have no affinity towards Cassandra.  Kafka or any other system would be just as interesting to try14:10
ndipanovthat fix will cause kilo serializer to keep sending back the instance when it fails on PciDeviceList14:10
ndipanovand keep getting the same thing back14:10
dansmithndipanov: you mean because kilo doesn't know about the new pcidevice or pcidevicelist, right?14:11
alaskibauzas: what I do really like is the consideration of how computes are reporting resources and how it might change, as right now it's heavily biased towards a filtering scheduler implementation, as it should given that it's what we have14:11
*** wxy has joined #openstack-nova14:11
johnthetubaguyalaski: FWIW, same here, I kinda like the idea of falling back to mysql, too, assuming thats simple, which it might well not be...14:11
*** jecarey has joined #openstack-nova14:11
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Specify current directory during processutils.execute using new cwd param  https://review.openstack.org/19160414:12
ndipanovyes dansmith14:12
alaskijohnthetubaguy: what I really want to see would be a mysql implementation and a cassandra and a kafka and a ... implementataion that could all work very differently.  which means the important bit is figuring out how to report resources14:13
alaskijohnthetubaguy: and not have a mysql fallback for a kafka implementation, just accept that they're different things14:13
*** mutoulbj has joined #openstack-nova14:13
*** wxy_ has joined #openstack-nova14:14
*** redbeard has joined #openstack-nova14:14
johnthetubaguyalaski: yeah true, "fallback" might best being "use the old system" well current system, I guess14:14
*** alex_klimov has joined #openstack-nova14:14
*** wxy_ has left #openstack-nova14:15
*** sudipto has joined #openstack-nova14:15
dansmithndipanov: so if you fix the version in master, but don't add the relationship from instance to the new pcidevicelist, and then backport that change to kilo, then you can fix the relationship in master so we start sending the thing and kilo will accept it, yes?14:15
* bauzas regrets his non-presence at the midcycle, given that convo14:15
*** claudiub has quit IRC14:15
ndipanovdansmith, yes that's what was my plan B14:15
*** alex_klimov has quit IRC14:15
bauzasalaski: johnthetubaguy: well, what I want to avoid is pulling some hard dependency which would make us a bit unsatisfied14:15
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py  https://review.openstack.org/18796814:15
bauzasalaski: johnthetubaguy: take the SG drivers as an example14:16
bauzasalaski: johnthetubaguy: so any improvement using a different and cool backend is +1 to me, but that shouldn't be the reason for the change14:16
*** mdrabe has quit IRC14:16
*** ijuwang has joined #openstack-nova14:16
bauzasalaski: johnthetubaguy: we should conceptually fix our problems and have different SLAs based on backend features14:17
bauzasexactly like a virt driver14:17
*** wxy has quit IRC14:17
bauzasall of that being MHO of course14:17
johnthetubaguybauzas: I think we agree, but sometimes you just need to try out ideas14:18
alaskiyeah, that's basically my thought as well14:18
edleafejohnthetubaguy: +114:18
openstackgerritjichenjc proposed openstack/nova: Cleanup HTTPRequest for security_groups test  https://review.openstack.org/18263014:19
openstackgerritjichenjc proposed openstack/nova: Change group controller of V2 test cases  https://review.openstack.org/18261314:19
openstackgerritjichenjc proposed openstack/nova: Add schema check for security_groups  https://review.openstack.org/18261414:19
bauzasjohnthetubaguy: well, I do agree but while I consider it as valid, I also consider that sometimes we need to fix our interfaces first :)14:19
*** fifieldt_ has quit IRC14:19
bauzasjohnthetubaguy: tbh, I really like POCs and I wish I could do one - but I also need to reduce our debt :)14:19
bauzasjust becase the current model is pretty broken14:20
*** xyang2 has joined #openstack-nova14:20
alaskibauzas: before bringing it in tree the interfaces would need to be fixed, but that's a ways off.  and a PoC could help us understand what the interfaces should look like14:20
johnthetubaguyalaski: +114:20
ndipanovdansmith, but we should fix the deserialization regardless...14:21
*** unicell has quit IRC14:21
bauzasalaski: I do admit that :)14:21
dansmithndipanov: agreed, might as well get that in the pipeline14:22
bauzasI feel sorry to be seen as conservative, while I really consider PoCs as good14:22
ndipanovdansmith, but as we've seen just asking for a blanket top level version may not work and can bite us again14:22
ndipanovso maybe we need something a bit more robust14:22
ndipanovwithout causing potentially N RPC calls14:22
ndipanovmaybe walk the tree, see what's unsupported - make a single request14:23
dansmithndipanov: sure, so do you want to work on that in o.vo and port it over to nova before we fix the bug, or do you want to make it ask the proper toplevel right now so we can proceed?14:23
*** markvoelker_ has quit IRC14:24
ndipanovfirst I will fix the versions14:25
ndipanovso even a simple version fix would need to go o.vo first, right?14:26
*** mriedem has quit IRC14:27
jlearmanHi, newb here, just getting myself set up to help out (I'll probably start by upgrading Nova UT testcases from mox to mock).  I'm planning to use virtualbox with ubuntu on my Windows laptop, but haven't started down that path yet.  Any reason that won't work, or is there a better suggestion for Windows?14:27
*** alex_klimov has joined #openstack-nova14:27
*** mriedem has joined #openstack-nova14:27
*** jichen has quit IRC14:29
*** annegentle has joined #openstack-nova14:30
*** jichen has joined #openstack-nova14:30
dansmithndipanov: you mean asking for the proper toplevel, right?14:30
dansmithndipanov: if you want to do that in nova, I'll mirror it on the o.vo side if it looks clean and we can do it in parallel14:30
ndipanovdansmith, since it's not on the critical path for this version thing14:31
ndipanovI can do it in o.vo14:32
*** rfolco has quit IRC14:32
*** sudipto has quit IRC14:32
dansmithit's not? maybe because you're getting an instance fault anyway?14:32
*** yingjun has quit IRC14:32
ndipanovso like I said14:32
ndipanovwell I think it's better to fix the versions first even if with 2 patches14:33
ndipanovand then properly fix the serializer14:33
ndipanov(walk the tree, backport all that are too new in one go)14:34
dansmithoh, you mean that14:34
ndipanovas opposed to just backport the latest top level14:34
ndipanovbecause as is demonstrated by this issue14:35
dansmithI want to fix the current serializer asking for the proper toplevel ASAP, because that's what we're currently depending on14:35
ndipanovit still leaves a few edge cases open14:35
ndipanovok but that's still broken for the situation we are currently in14:35
openstackgerritMatthew Booth proposed openstack/nova: Implement compare-and-swap for instance update  https://review.openstack.org/20259314:35
dansmithso I'll fix the toplevel issue14:35
ndipanovand we still need a 2 step fix for the versions14:35
dansmithyeah, I understand, but the toplevel version thing affects everyone14:36
alex_xuandreykurilin: hi, when you begin vacation?, may I update your patch when your vacation?14:36
ndipanovok14:36
ndipanovbut it's still not the proper fix - that's all I'm saying14:36
dansmithit's not going to fix you for this, I get that14:36
openstackgerritShiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI  https://review.openstack.org/20259514:37
ndipanovbut yes less wrong than blatantly wrong is better for sure14:37
*** mdrabe has joined #openstack-nova14:37
openstackgerritMike Dorman proposed openstack/nova: Add console allowed origins setting  https://review.openstack.org/20167714:37
*** tonytan4ever has joined #openstack-nova14:38
mdboothjaypipes mriedem johnthetubaguy: This version adds a retry when it detects the race instead of raising a different exception: https://review.openstack.org/#/c/202593/14:38
jaypipesmdbooth: yep, it's in the queu.14:38
jaypipesqueue...14:38
mdboothI thought you americans called them lines14:38
dansmithmdbooth: not for this :)14:39
* mdbooth is confused :)14:39
dansmithheh14:39
johnthetubaguymdbooth: ah, cool, thanks for pushing on that14:40
*** lpetrut has quit IRC14:41
ndipanovso dansmith I also saw that we keep asking for that backport in one case for block device14:42
ndipanovah but that can happen when Block device mapping is sent over RPC14:43
ndipanovcool cool14:43
ndipanovnevermind14:43
*** jichen_ has joined #openstack-nova14:45
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtISCSIVolumeDriver into it's own module  https://review.openstack.org/20230114:46
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module  https://review.openstack.org/19731114:46
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module  https://review.openstack.org/19734114:46
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module  https://review.openstack.org/19685914:46
*** jichen has quit IRC14:48
*** jichen_ is now known as jichen14:48
openstackgerritShiina, Hironori proposed openstack/nova: Add a cells RPC API to inject an NMI  https://review.openstack.org/20260514:49
*** READ10 has quit IRC14:49
*** rfolco has joined #openstack-nova14:51
*** markvoelker has joined #openstack-nova14:53
openstackgerritjichenjc proposed openstack/nova: Check flavor type before add tenant access  https://review.openstack.org/12433814:53
*** eglynn has joined #openstack-nova14:55
*** jichen has quit IRC14:58
*** Longgeek has quit IRC14:58
*** Jeffrey4l has quit IRC14:58
openstackgerritNikola Dipanov proposed openstack/nova: Fix the incorrect PciDeviceList version number  https://review.openstack.org/20128014:58
openstackgerritNikola Dipanov proposed openstack/nova: objects: send PciDeviceList 1.2 to all code that can handle it  https://review.openstack.org/20261614:58
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261714:59
*** redbeard has quit IRC15:00
*** Longgeek has joined #openstack-nova15:00
*** ociuhandu has joined #openstack-nova15:00
edleafejlearman: Yeah, that sounds like a good setup15:01
*** READ10 has joined #openstack-nova15:01
andreykurilinalex_xu: from Monday:) but I'll available in the evening first 3 days. feel free to change them, if I'll be unavailable:)15:02
openstackgerritSudipta Biswas proposed openstack/nova: Allow NUMA based reporting for Monitors  https://review.openstack.org/19897215:02
openstackgerritSudipta Biswas proposed openstack/nova: Base enablement for memory bandwidth monitor  https://review.openstack.org/20262315:02
*** karimb has joined #openstack-nova15:02
andreykurilinalex_xu: *from Monday and till 31 July15:03
jaypipesdansmith: super easy one: https://review.openstack.org/#/c/202501/15:03
dansmithjaypipes: will look in a bit15:03
*** Murali has quit IRC15:03
jrolljaypipes: hey fyi I revved the ironic multiple compute spec15:04
jrollgoing to try to get initial code before next week15:04
*** mriedem is now known as mriedem_away15:04
jaypipesjroll: coolio. thank you sir!15:04
*** sudipto has joined #openstack-nova15:04
jrollnp15:04
*** alex_klimov has quit IRC15:06
*** coolsvap|away is now known as coolsvap15:06
*** Longgeek has quit IRC15:07
*** tpeoples has joined #openstack-nova15:08
*** mnestratov has quit IRC15:09
*** lyanchih has quit IRC15:09
*** sudipto_ has joined #openstack-nova15:09
*** amotoki has joined #openstack-nova15:10
*** vladikr has quit IRC15:10
*** Jeffrey4l has joined #openstack-nova15:11
*** unicell has joined #openstack-nova15:12
*** sudipto has quit IRC15:12
*** belmoreira has quit IRC15:13
*** otter768 has joined #openstack-nova15:14
*** redbeard has joined #openstack-nova15:15
*** mutoulbj has quit IRC15:16
*** hshiina has left #openstack-nova15:16
*** dannywilson has joined #openstack-nova15:16
*** mnestratov has joined #openstack-nova15:17
openstackgerritAndrea Rosa proposed openstack/nova: Manage an empty machine-id file  https://review.openstack.org/20264015:18
*** dguryanov|2 has joined #openstack-nova15:18
*** hemna has joined #openstack-nova15:18
*** otter768 has quit IRC15:18
*** armax has joined #openstack-nova15:18
*** afazekas has quit IRC15:20
*** hemna has quit IRC15:20
dansmithPaulMurray: around?15:20
dansmithjaypipes: on that conductor api thing, I don't get why we need to add the object mocks if we're just removing unused conductor mocks15:24
dansmithjaypipes: it's not like the object calls get turned into conductor calls and we can catch them with either type of mock...15:25
jaypipesdansmith: hmm.15:25
dansmithI guess I'll comment on the review and see if he pops up15:26
jaypipesdansmith: so, for the test_no_op_init_compute_node() one, PaulMurray's code is correcting a bad unit test. The create_mock and service_mock objects are being tested on lines 819-820 to ensure that they are not called. Previously, the test was passing because the mocks were the completely wrong thing (the conductor calls), but now, the tests are passing when referring to the correct thing (ComputeNode.create() and Servi15:29
jaypipesce.get_by_compute_host)15:29
*** rdopiera has quit IRC15:29
*** tsekiyama has joined #openstack-nova15:30
openstackgerritMatthew Booth proposed openstack/nova: Implement compare-and-swap for instance update  https://review.openstack.org/20259315:30
jaypipesdansmith: for the test_compute_node_loaded() test, same thing. line 855 is testing that the ComputeNode.create() method is NOT called, which is the correct assertion. the test was previously passing because the wrong thing was being mocked (and of course, the assertion that the mock was not called was passing as a false positive)15:30
openstackgerritStephen Finucane proposed openstack/nova: Add 'hw_cpu_threads_policy' image metadata prop  https://review.openstack.org/20264715:31
openstackgerritStephen Finucane proposed openstack/nova: Add 'cpu_policy' and 'cpu_threads_policy' fields  https://review.openstack.org/20264815:31
openstackgerritStephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=avoid' filtering  https://review.openstack.org/20264915:31
openstackgerritStephen Finucane proposed openstack/nova: trivial: Add some logs to 'numa_topology_filter'  https://review.openstack.org/20265015:31
openstackgerritStephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=require' scheduling  https://review.openstack.org/20265115:31
openstackgerritStephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=separate' scheduling  https://review.openstack.org/20265215:31
openstackgerritStephen Finucane proposed openstack/nova: Make use of 'InstanceNUMACell.cpu_policy' field  https://review.openstack.org/20265315:31
openstackgerritStephen Finucane proposed openstack/nova: trivial: Make use of 'CPUAllocation' enum values  https://review.openstack.org/20265415:31
openstackgerritStephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=isolate' scheduling  https://review.openstack.org/20265515:31
*** thangp has quit IRC15:31
dansmithjaypipes: my point is, the objects don't make the conductor calls15:32
dansmithjaypipes: so if the code is using the objects, it should have been failing before without those mocks15:32
dansmithjaypipes: but the patch doesn't change the code, just the test15:32
jaypipesdansmith: the patch corrects the tests that were previously referring to conductor calls and providing false positive assertions about not being called.15:33
openstackgerritsahid proposed openstack/nova: hardware: take into account pages size when computing memory  https://review.openstack.org/20250415:34
*** vladikr has joined #openstack-nova15:34
*** gyee has quit IRC15:34
jaypipesdansmith: and the patch simply removes the conducto_api attribute from the resource tracker.15:34
dansmithjaypipes: it doesn't. all it changes are what is mocked out, but the new mocks and the old mocks aren't in the same code path15:34
dansmithjaypipes: let me pull this down and play with it and maybe I can make my point15:34
jaypipesok15:34
*** thangp has joined #openstack-nova15:34
*** ociuhandu has quit IRC15:35
andreykurilinsdague: Fix for stable/kilo branch of novaclient is ready: https://review.openstack.org/20265615:36
sdagueandreykurilin: thanks15:36
*** scheuran has quit IRC15:36
*** Mike_D_laptop has joined #openstack-nova15:37
dansmithjaypipes: I assume it's because we assert that a bunch of things are not called. So we can mock them out for no reason and they're still ... not called :)15:37
*** andreykurilin__ has joined #openstack-nova15:38
*** gyee has joined #openstack-nova15:39
*** andreykurilin has quit IRC15:39
*** MentalRay__ has joined #openstack-nova15:39
*** ig0r_ has quit IRC15:40
sdaguedansmith: what happens to the symbolic compute levels (i.e. kilo) if we backport an rpc change to stable kilo?15:40
sdaguedo those get bumped?15:40
*** rgerganov_ has joined #openstack-nova15:41
garykmriedem_away:  you around?15:41
*** sudipto_ has quit IRC15:41
garykwe have a serious problem with you have a sec ti talk about https://github.com/openstack/nova/commit/1ba72081715869fe6d0ffba3436fb09ef9499335?15:41
garyknot a serious problem with you :)15:41
garykjust a serious problem with the change that you made. https://github.com/openstack/nova/commit/1ba72081715869fe6d0ffba3436fb09ef9499335?15:42
*** dprince has quit IRC15:42
garykproblem is that when an instance gets into error state it can never be deleted…15:42
dansmithsdague: well, in general we don't backport rpc stuff unless it's absolutely critical15:42
dansmithsdague: if we have to, we do it as a .z version.. updating the symbol probably depends on what the change is..15:43
*** ivasev has quit IRC15:43
sdagueok, I was just thinking of our upgrade testing, which uses the symbol to pin15:43
dansmithsdague: can I ask what the change is before I commit? :)15:44
sdagueit came up in doing grenade multinode - https://review.openstack.org/#/c/199091/15:44
sdagueI realized that pinning everyone means they might not be using the latest rpc version on stable15:45
sdaguewhich is the same issue on the pcpu jobs15:45
dansmithsdague: so is this hypothetical?15:45
sdaguealso, man, did I wish this was in the db instead of in the config15:45
sdagueyes, it's a hypothetical question15:45
*** vishwanathj has joined #openstack-nova15:45
dansmithokay, so first off: https://review.openstack.org/#/c/201733/15:46
dansmithsecond, if we backport a thing, we do it as a .z and it has to be like fully compatible with the .0 so the pin doesn't matter as much.. we'd be backporting a fix, not a feature, so that should be doable15:46
dansmithmoving the pin doesn't really matter because we discard the .z for compatibility checking15:47
sdaguedansmith: ok, so https://review.openstack.org/#/c/201733/ kind of excites me. You imagining an admin call to set that in the cluster?15:47
*** vishwana_ has quit IRC15:47
*** rgerganov_ has quit IRC15:48
dansmithsdague: no, it'll be determined by code levels, but the services will be able to decide what to do based on the quorum15:48
dansmithsdague: so, the new rpc version starts being used when you upgrade the last service15:48
*** lpetrut has joined #openstack-nova15:49
dansmithsdague: in order to make that work for non-rpc things, we need to register api services, which we don't currently do15:49
*** markus_z has quit IRC15:49
dansmithbut for rpc pinning purposes it should be enough as-is I think15:49
dansmithwe need to land that in lemming so we can use it in mitasktska-whatever-it-is15:49
sdaguedansmith: it will remove the need for the config file bit though, right?15:50
dansmithyeah15:50
sdagueI'm ok with automatic quorum15:50
sdaguethough what happens with stale compute_node entries?15:50
dansmithit also means that if we start up and we're older than the oldest version in the db, we can punt on startup15:50
sdagueright, yes, all sounds very reasonable. ++15:50
dansmithsdague: yeah, those'll need to be removed15:50
sdaguewell, I wonder if we provide an admin API call to just force it, which would flag all the non upgraded compute nodes as down15:51
dansmithnot computenode, but service15:51
sdaguesure, service15:51
dansmithwell, that doesn't really work15:52
*** annegentle has quit IRC15:52
dansmithbecause if the thing is still up and old, and you just decide it's too old, you need to fence it or shut it down or whatever15:52
dansmithotherwise it'll still consume messages from shared queues and such15:52
sdagueit would under the circumstances that it is actually down and not coming back15:52
*** annegentle has joined #openstack-nova15:52
dansmithyes15:52
sdaguebut has stale records15:52
sdaguewhich I would imagine would happen a reasonable amount15:53
dansmithbut we can also just make it not start up when it comes back if it's older than the minimum version15:53
sdagueright, but the service entries don't auto delete right?15:53
dansmithor rather, older than the oldest still-running node15:53
sdagueif the node died15:53
sdagueand it's not coming back15:53
dansmithno, but we have a notion of liveness15:53
sdagueok15:53
sdagueanyway, right, there might be an edge here, but that's all just details and bugs15:53
dansmithbut yes, we still need to not require operators to manually clean up service records for things that are never coming back for sure15:54
sdague++ on approach15:54
dansmithit's all edge cases I think, but yeah :)15:54
sdagueyeh, but they mostly can be worked through in sane ways15:54
dansmithI think so15:54
openstackgerritheijlong proposed openstack/nova: Add microversion to allow uuid in server show response data  https://review.openstack.org/20266515:54
*** paul-carlton has joined #openstack-nova15:56
park_heijlong_johnthetubaguy: I finished the last patch for my bp, appreciated for your comments, make you update, thank you15:57
*** ddaskal has joined #openstack-nova15:58
*** ctrath has quit IRC15:59
openstackgerritMatthew Booth proposed openstack/nova: Thoughts on atomic host/node transfer during evacuate  https://review.openstack.org/20266715:59
*** jogo has quit IRC15:59
mdboothdansmith: Update host and node atomically during evacuate: https://review.openstack.org/#/c/202667/116:00
mdboothIt's a WIP, very untidy, unit test failures, etc16:00
mdboothInteresting bits are in compute/api.py and compute/manager.py16:01
*** angdraug has joined #openstack-nova16:01
mdboothOnly a couple of lines16:01
dansmithokay16:02
*** ctrath has joined #openstack-nova16:02
*** jlanoux has quit IRC16:02
mdboothdansmith: That's literally all I'm going to do with it for the moment :)16:02
mdboothI have no current plans to develop that, it's just an idea, with code16:02
*** dboik_ has quit IRC16:03
dansmithseems reasonable16:04
*** coolsvap is now known as coolsvap|away16:05
dansmithmight be easier to make objects track the previous version and just let you say "only do this if $attr is what we previously knew it to be"16:05
dansmithi.e. save(consistent=['vm_state', 'task-state', 'host', 'node'])16:05
*** lpetrut has quit IRC16:05
dansmithwell, maybe s/easier/cleaner/16:06
mdboothYeah16:06
mdboothAlthough expected can take a list16:06
PaulMurrayhi dansmith - you had a question about the mocks16:06
dansmithPaulMurray: yeah, I commented16:06
mdboothSo task_state: ['REBUILDING', 'SPAWNING']16:06
*** park_heijlong_ has quit IRC16:06
dansmithmdbooth: true, but it's a little different than host and node and other general purpose things16:07
mdboothYup16:07
tdurakovbauzas, Hi Sylvain, are you around?16:07
*** mutoulbj has joined #openstack-nova16:07
*** sahid has quit IRC16:07
*** ijw has quit IRC16:07
bauzastdurakov: not really :/16:08
bauzaspretty busy16:08
*** newbie_2 has joined #openstack-nova16:09
tdurakovbauzas, short question, how scheduler refactoring is going? what about this one https://bugs.launchpad.net/nova/+bug/1214943 ?16:09
openstackLaunchpad bug 1214943 in OpenStack Compute (nova) "Live migration should use the same memory over subscription logic as instance boot" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza)16:09
mdboothdansmith: anyway, I know you were robustifying it, thought that could be a useful tool16:09
mdboothLet me know if you think it's worth spending time on16:09
*** nic has joined #openstack-nova16:10
*** tjones1 has joined #openstack-nova16:12
*** sudipto has joined #openstack-nova16:12
*** oro has quit IRC16:13
*** whenry has joined #openstack-nova16:13
*** mriedem_away is now known as mriedem16:13
*** kmARC has quit IRC16:13
mriedemgaryk: that would have been a problem before that revert too16:13
garykmriedem: ah, ok. i think that the issue is that the task state is not being reverted. i am currenly debugging16:14
mriedemgaryk: if you have a stack trace you could see if whatever method is being hit in the compute manager, if it gets to the compute manager, has the @reverts_task_state decorator16:14
mriedemhowever, if it's only the compute_api.delete that fails and puts it in error state, then yeah that could be a problem16:14
garykmriedem: i am checking that the reverts task works now16:15
mriedemgaryk: fwiw, ankit re-proposed that change https://review.openstack.org/#/c/194063/16:16
*** jogo has joined #openstack-nova16:16
mriedembut this ^ supposedly has the race issue in mind16:16
bauzaswarning, paste bomb16:16
bauzasor rather, gerrit bomb16:17
openstackgerritSylvain Bauza proposed openstack/nova: Modify Scheduler RPC API select_dests()  https://review.openstack.org/19920516:17
openstackgerritSylvain Bauza proposed openstack/nova: Prepare filters for using RequestSpec object  https://review.openstack.org/19791216:17
openstackgerritSylvain Bauza proposed openstack/nova: Use RequestSpec in the ChanceScheduler  https://review.openstack.org/19905116:17
openstackgerritSylvain Bauza proposed openstack/nova: Add RequestSpec methods for primitiving into dicts  https://review.openstack.org/18719716:17
openstackgerritSylvain Bauza proposed openstack/nova: Use RequestSpec object in HostManager  https://review.openstack.org/19125116:17
openstackgerritSylvain Bauza proposed openstack/nova: Use the same pci_requests field for all filters and HostManager  https://review.openstack.org/19125016:17
openstackgerritSylvain Bauza proposed openstack/nova: Remove unnecessary method in FilterScheduler  https://review.openstack.org/18848516:17
openstackgerritSylvain Bauza proposed openstack/nova: Consider that all scheduler calls are IO Ops  https://review.openstack.org/18848616:17
openstackgerritSylvain Bauza proposed openstack/nova: Create RequestSpec object  https://review.openstack.org/14552816:17
openstackgerritSylvain Bauza proposed openstack/nova: Use RequestSpec object in the FilterScheduler  https://review.openstack.org/18848716:17
openstackgerritSylvain Bauza proposed openstack/nova: Add two fields to ImageMetaProps object  https://review.openstack.org/19791116:17
openstackgerritSylvain Bauza proposed openstack/nova: Modify Affinity filter for RequestSpec  https://review.openstack.org/20267316:17
openstackgerritSylvain Bauza proposed openstack/nova: Modify Aggregate filters for RequestSpec  https://review.openstack.org/20267416:17
openstackgerritSylvain Bauza proposed openstack/nova: Modify default filters for RequestSpec  https://review.openstack.org/20267516:17
openstackgerritSylvain Bauza proposed openstack/nova: Modify metric-related filters for RequestSpec  https://review.openstack.org/20267616:17
openstackgerritSylvain Bauza proposed openstack/nova: Modify NUMA, PCI and num_instances filters for RequestSpec  https://review.openstack.org/20267716:17
openstackgerritSylvain Bauza proposed openstack/nova: Modify left filters for RequestSpec  https://review.openstack.org/20267816:17
*** MentalRay__ has quit IRC16:17
garykmriedem: ok, i'll check that16:17
* bauzas competing for the Longest Ever Patch Series16:17
garykmriedem: nah, it still has that dodgy code: if original_task_state in (task_states.DELETING,16:18
garyk                                           task_states.SOFT_DELETING):16:18
*** thorst has quit IRC16:18
*** rgerganov_ has joined #openstack-nova16:18
*** sambol has joined #openstack-nova16:19
*** ZZelle has quit IRC16:20
*** chuckcarmack has quit IRC16:20
*** ZZelle has joined #openstack-nova16:20
*** dboik has joined #openstack-nova16:20
*** chuckcarmack has joined #openstack-nova16:20
*** zsais has joined #openstack-nova16:20
*** sudipto has quit IRC16:21
jaypipesdansmith: right, that's what I was trying (badly) to explain above...16:21
ericksonsantoswhooaa16:22
ericksonsantos(about those changes ^)16:22
dansmithjaypipes: you still think the patch is correct?16:22
jaypipesdansmith: yes.16:23
chuckcarmackdims_: Would it be ok I if worked on https://bugs.launchpad.net/nova/+bug/1434866 as a newbie?  Its low hanging fruit.16:24
openstackLaunchpad bug 1434866 in OpenStack Compute (nova) "nova-cells and nova-idmapshift missing in documentation" [Low,Triaged]16:24
dims_chuckcarmack: +1 go for it :)16:24
chuckcarmackdims_:  thanks, I'll give it a try16:24
dansmithjaypipes: PaulMurray: the patch is making up for the fact that the patch to convert to objects didn't adjust the tests, right?16:24
garykmriedem: the revert task state is called - problem is that this does not actually set the task state to None… man this is so broken16:25
jaypipesdansmith: correct.16:25
dims_chuckcarmack: it's not assigned to anyone so it's fair game, i had a review earlier, feel free to use anything from it which you feel useful16:25
dansmithjaypipes: so that's a change that has nothing to do with "remove conductor api from resource_tracker"16:25
dansmithjaypipes: it's confusing because it seems like we're replacing conductor with objects, but we're not16:25
chuckcarmackdims_:  yes, you have a good start on it16:25
dansmithjaypipes: so IMHO, there should be a separate patch that says "in commit $foo we forgot to change the tests, this fixes that" and then let the conductor removal stand on its own16:26
jaypipesdansmith: well, the actual code change does indeed remove the conductor_api attribute frmo the resource tracker.16:26
*** edmondsw has quit IRC16:26
jaypipesdansmith: sure, that would be fine with me as well.16:26
dansmithjaypipes: right, but it has nothing to do with the object mock changes16:26
jaypipessure, understood.16:26
PaulMurraydansmith, yes - only caught it when I removed the conductor api from resource tracker because they were using it to set up the mocks16:26
dansmithyeah16:26
*** redbeard has quit IRC16:27
PaulMurraydansmith, the tests work without the mocks - the mocks are there to check it is doing (or not doing) the calls it should (or shouldn't)16:27
dansmithif you want to keep them as one patch, please call it out very specifically in the commit message, but I'd still prefer two16:27
dansmithPaulMurray: yeah, I know16:27
PaulMurraysorry - I should read back16:27
dansmiththe problem is that it's confusing because it seems like you're removing the conductor usage and replacing it with objects usage, and you're not16:27
dansmithand since the test is testing for things that never happen, I think it'd be better to be super explicit about it16:28
*** lpetrut has joined #openstack-nova16:28
*** achanda has joined #openstack-nova16:28
dansmithbecause later when that test is failing, and this is the last patch that touched it.. we won't remember that the previously patch actually made the change and this one just silently cleaned up the tests16:28
PaulMurraywell, actually the tests test for something that should never happen. If the code is wrong they might catch it16:28
*** achanda has quit IRC16:29
dansmithright, isn't that what I said? :)16:29
*** dims_ has quit IRC16:30
PaulMurrayThis patch was intended to remove conductor_api from resource tracker. In doing that I found those tests broke because they were wrong - so I corrected them16:31
*** armax has quit IRC16:31
PaulMurrayDo you want me to change the tests too?16:32
*** dims_ has joined #openstack-nova16:32
*** dims_ has quit IRC16:32
dansmithPaulMurray: I think you should fix the tests in patch 1, and remove conductor in patch 216:32
PaulMurray:)16:32
PaulMurraymy patches always become 10 patches, even the easy ones16:33
*** rlrossit has left #openstack-nova16:33
dansmithPaulMurray: and jaypipes and I will +2 them super quick16:33
dansmithPaulMurray: well, here's the other option16:33
dansmithI can just shut up and you can get someone else to +2 :)16:33
*** tpeoples has quit IRC16:33
*** mwagner_lap has quit IRC16:34
PaulMurrayoh all right then, let me see what I can do......16:34
*** tonytan4ever has quit IRC16:35
dansmithPaulMurray: just don't worry about it16:35
*** gyee has quit IRC16:35
dansmithI'll do it myself16:35
dansmithit'll take less time than arguing16:35
dansmithI already have the thing pulled down anywa16:35
*** achanda has joined #openstack-nova16:35
*** vilobhmm has joined #openstack-nova16:36
*** mutoulbj has quit IRC16:37
*** redbeard has joined #openstack-nova16:37
*** lpetrut has quit IRC16:38
edleafedansmith: hey, if I argue with you, will you do my work, too? :)16:40
*** jogo has quit IRC16:40
*** hemnafk is now known as hemna16:41
*** jogo has joined #openstack-nova16:41
PaulMurrayedleafe, you have to get to a special place with dansmith - It took me 2 years of arguing :)16:43
ericksonsantoslol16:43
edleafePaulMurray: thanks for the tip! :)16:43
openstackgerritDan Smith proposed openstack/nova: Remove conductor api from resource tracker  https://review.openstack.org/20250116:44
openstackgerritDan Smith proposed openstack/nova: Fix test_tracker object mocks  https://review.openstack.org/20269016:44
*** rwsu has quit IRC16:44
PaulMurraydansmith, thanks a lot - I have to rush to get my car from servicing before they lock me out16:44
*** nic has quit IRC16:44
dansmithPaulMurray: np16:44
*** annashen has joined #openstack-nova16:45
dansmithedleafe: at least you know that my motive is just to get it right and not to be nitpicky for the sake of it because it costs me nothing16:45
edleafedansmith: of course - just having a little fun with ya16:46
*** lucasagomes is now known as lucas-dinner16:47
tdurakovmriedem, hi16:49
bauzasjaypipes: FYI https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/request-spec-object,n,z becomes very fat, it would be awesome if you could review some of them :)16:49
tdurakovmriedem, could you take a look at https://review.openstack.org/#/c/184495/ ansd subsequent one?16:50
*** y_sawai has joined #openstack-nova16:52
*** thorst has joined #openstack-nova16:52
jaypipesbauzas: will do this evening.16:52
*** mutoulbj has joined #openstack-nova16:52
openstackgerritNikola Dipanov proposed openstack/nova: Allow for migration object to be passed to _move_claim  https://review.openstack.org/19967916:53
openstackgerritNikola Dipanov proposed openstack/nova: rt: generalize claim code to be useful for other move actions  https://review.openstack.org/19713816:53
openstackgerritNikola Dipanov proposed openstack/nova: rt: move filtering of mygration by type lower in the call stack  https://review.openstack.org/19967816:53
openstackgerritNikola Dipanov proposed openstack/nova: compute: Make rebuild_instance RPC take an optional migration  https://review.openstack.org/20048516:53
openstackgerritNikola Dipanov proposed openstack/nova: Define migration states and transitions and use when changing status  https://review.openstack.org/19766916:53
openstackgerritNikola Dipanov proposed openstack/nova: Introduce a simple state machine module  https://review.openstack.org/19766816:53
*** tpeoples has joined #openstack-nova16:54
ndipanovmriedem, https://review.openstack.org/#/c/201280/16:54
ndipanovstill waiting for the check though :(16:54
garykmriedem: problem is that there is an exception in the compute node - for example unable to delete disks. the task state is updated correctly. subsequent deletes are not sent to the compute node. still trying to understand why. problem is restarting services hs it working16:55
ndipanovbut would be good to land this and the kilo backport soon if possible16:55
sdaguejohnthetubaguy: if you are still awake - https://review.openstack.org/#/c/152703 was the provider networks in the config drive thing that infra wanted last cycle, but missed the deadline. Curious on your thinking16:55
*** y_sawai has quit IRC16:55
johnthetubaguysdague: ah, yeah16:56
*** romainh has left #openstack-nova16:56
johnthetubaguysdague: we could raise the priority in the hope it gets more attention?16:57
sdaguejohnthetubaguy: sure, though, honestly, it's a single review, and you were one of the ones that were interested in that particular case16:58
*** rgerganov_ has quit IRC16:58
sdagueI was poking you as an interested reviewer, not as ptl :)16:58
*** oro has joined #openstack-nova17:00
*** aix has quit IRC17:00
*** derekh has quit IRC17:00
*** kmARC has joined #openstack-nova17:00
*** jistr has quit IRC17:00
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20070817:04
*** ildikov has quit IRC17:05
*** ssurana has joined #openstack-nova17:09
*** jcoufal has joined #openstack-nova17:09
openstackgerritDmitry Guryanov proposed openstack/nova: libvirt: do relative block rebase only with non-null base  https://review.openstack.org/20244217:10
openstackgerritMing Yang proposed openstack/nova: Case-insensitive filters  https://review.openstack.org/20172517:13
openstackgerritMing Yang proposed openstack/nova: Refactor get_filtered_hosts in filter host manager  https://review.openstack.org/19287817:13
*** krtaylor has quit IRC17:13
*** armax has joined #openstack-nova17:14
*** otter768 has joined #openstack-nova17:14
openstackgerritDan Smith proposed openstack/nova: Fix test_tracker object mocks  https://review.openstack.org/20269017:16
openstackgerritDan Smith proposed openstack/nova: Remove conductor api from resource tracker  https://review.openstack.org/20250117:17
*** Maike has quit IRC17:17
*** haypo has left #openstack-nova17:18
*** IanGovett1 has joined #openstack-nova17:18
*** slong_ has quit IRC17:19
*** otter768 has quit IRC17:19
*** signed8bit has quit IRC17:19
dansmithjaypipes: just fixed commit message wrapping and added co-authored for PaulMurray: https://review.openstack.org/#/c/202690/2'17:19
*** SourabhP has joined #openstack-nova17:19
*** slong_ has joined #openstack-nova17:19
*** IanGovett has quit IRC17:19
*** signed8bit has joined #openstack-nova17:20
jaypipesdansmith: pushed.17:20
*** xyang2 has quit IRC17:20
*** xyang2 has joined #openstack-nova17:21
*** achanda has quit IRC17:25
*** mtanino has joined #openstack-nova17:25
bauzasjaypipes: ack, thanks17:25
*** rfolco has quit IRC17:26
*** y_sawai has joined #openstack-nova17:27
*** vilobhmm has quit IRC17:28
*** vilobhmm has joined #openstack-nova17:28
*** karimb has quit IRC17:30
*** yamahata has quit IRC17:31
openstackgerritDmitry Guryanov proposed openstack/nova: libvirt: resize ploop disks with prl_disk_tool  https://review.openstack.org/18239017:32
*** dims_ has joined #openstack-nova17:32
*** annegentle has quit IRC17:33
*** armax has quit IRC17:34
*** exploreshaifali has joined #openstack-nova17:34
*** achanda has joined #openstack-nova17:35
openstackgerritVilobh Meshram proposed openstack/nova: WIP : Servicegroup foundational refactoring for Control Plane  https://review.openstack.org/20271417:36
johnthetubaguysdague: ah, sorry, yeah, was attempting to multitask and failed17:36
*** casusbelli has quit IRC17:36
openstackgerritMoshe Levi proposed openstack/nova: Adding support for InfiniBand SR-IOV vif type  https://review.openstack.org/18705217:38
*** dims_ has quit IRC17:39
*** achanda has quit IRC17:40
*** annashen has quit IRC17:40
*** davideagnello has quit IRC17:40
vilobhmmjohnthetubaguy : ping17:41
*** annashen has joined #openstack-nova17:41
*** lucas-dinner has quit IRC17:43
*** zhenguo has quit IRC17:43
*** achanda has joined #openstack-nova17:46
*** krtaylor has joined #openstack-nova17:46
johnthetubaguyvilobhmm: hi17:47
*** annashen has quit IRC17:47
*** jcoufal has quit IRC17:48
*** edmondsw has joined #openstack-nova17:49
*** annashen has joined #openstack-nova17:49
*** igordcard has joined #openstack-nova17:50
*** edmondsw has quit IRC17:51
*** edmondsw has joined #openstack-nova17:51
openstackgerritMatt Riedemann proposed openstack/nova: Deprecate the osapi_v3.enabled option  https://review.openstack.org/20272417:52
*** moshele has quit IRC17:52
*** dims_ has joined #openstack-nova17:52
*** dims_ has quit IRC17:53
vilobhmmjohnthetubaguy : regarding the code freeze for L release ; will WIP suffice to the creteria for a feature to be accepted for L release….because as per your update code needs to be up by 16th and needs to be merged by july 30th…i have one feature where i am missing unit tests (which I will try but might not be able to complete by 16th) and hence proposed them as WIP…so thought will check with you17:53
*** lucas-dinner has joined #openstack-nova17:56
*** gyee has joined #openstack-nova17:57
*** dims_ has joined #openstack-nova17:57
*** paul-carlton has quit IRC17:58
dguryanov|2Hello!17:59
*** y_sawai has quit IRC17:59
*** tjones1 has quit IRC18:00
dguryanov|2Could someone, please, look at https://review.openstack.org/#/c/202442/ . It a fix for a bug, introduced few days ago. Several cinder CI's are blocked by it.18:00
dims_mriedem: getting my pm? :)18:00
*** amotoki has quit IRC18:00
*** rfolco has joined #openstack-nova18:01
dansmithdims_: earlier he told me he was busy18:02
dims_dansmith: thanks!18:02
*** Kennan has quit IRC18:02
dansmithbut he might've just been avoiding me18:02
dims_haha18:02
dansmith(for obvious reasons)18:02
*** dboik has quit IRC18:02
*** redbeard has quit IRC18:03
*** ssurana1 has joined #openstack-nova18:03
*** Kennan has joined #openstack-nova18:03
*** Sukhdev has joined #openstack-nova18:04
*** rfolco has quit IRC18:05
*** ssurana has quit IRC18:06
*** Marga_ has quit IRC18:06
*** davideagnello has joined #openstack-nova18:06
*** Marga_ has joined #openstack-nova18:07
*** pixelbeat_ has quit IRC18:07
*** patrickeast has joined #openstack-nova18:07
*** rfolco has joined #openstack-nova18:09
*** _gryf has quit IRC18:10
*** xek has quit IRC18:10
*** isq has quit IRC18:10
mgagneI'm having an issue with Nova kilo (2015.1.0) (with cells) where the instance_extra.flavor field in the api cell is not properly updated after a resize. (the field in compute cell is properly updated) Is there any known bugs and/or fix for it?18:10
*** doude has quit IRC18:11
*** xek has joined #openstack-nova18:12
*** _gryf has joined #openstack-nova18:12
*** isq has joined #openstack-nova18:13
*** tonytan4ever has joined #openstack-nova18:14
*** MentalRay__ has joined #openstack-nova18:15
*** signed8bit is now known as signed8bit_ZZZzz18:16
*** achanda has quit IRC18:16
ndipanovdansmith, https://review.openstack.org/20273218:19
*** baoli has joined #openstack-nova18:19
dansmithndipanov: I did it this morning18:19
dansmithhttps://review.openstack.org/#/c/202641/18:20
dansmithsorry, I thought it was clear I would do it18:20
*** y_sawai has joined #openstack-nova18:21
*** ildikov has joined #openstack-nova18:22
*** Nikolay_St has quit IRC18:22
ndipanovdansmith, fwiw mine is cleaner :P18:22
*** y_sawai has quit IRC18:22
*** mtanino has quit IRC18:22
ndipanovdansmith, can you +2 the kilo backport of that version fix if it pasess check today18:23
ndipanovI'd need to drop off soon18:23
*** dboik has joined #openstack-nova18:23
dansmithdid the master patch succeed?18:23
ndipanovwell the check is still running seems like it's pretty backed up :(18:23
ndipanovit's running now18:24
*** achanda has joined #openstack-nova18:26
*** agireud has joined #openstack-nova18:29
*** MentalRay__ has quit IRC18:30
*** promulo_ has joined #openstack-nova18:30
*** erlon has quit IRC18:30
*** jcoufal has joined #openstack-nova18:32
*** jcoufal has quit IRC18:32
*** yassine has quit IRC18:34
*** mutoulbj has quit IRC18:35
*** zsais has quit IRC18:35
openstackgerritDan Smith proposed openstack/nova: Add version counter to Service object  https://review.openstack.org/20173318:37
*** achanda has quit IRC18:38
johnthetubaguyvilobhmm: so the deadline was today really, and it has to be complete and ready for review not WIP by the end of today, if that happens do drop me an email18:38
openstackgerritNikola Dipanov proposed openstack/nova: objects: Don't import CellMapping from the objects module  https://review.openstack.org/20273718:38
*** jwang has joined #openstack-nova18:38
johnthetubaguyvilobhmm: sorry I have to sign off now, just got dragged into a meeting when you pinged me18:38
ndipanovdansmith, a light objects one ^18:39
* ndipanov &18:39
vilobhmmjohnthetubaguy : np..will try and let you know18:39
*** xyang2 has quit IRC18:40
vilobhmmjaypipes : ping18:41
vilobhmmdansmith : ping18:41
melwittmgagne: I don't think that's a known bug cc alaski18:42
*** otter768 has joined #openstack-nova18:42
mgagnemelwitt: trying to backport the patch atm18:42
mgagnemelwitt: and see if it fixes anything18:43
alaskimelwitt: it was never reported before, because it was fixed by instance_udpate_at_top going to objects18:43
melwittalaski: ah, okay18:44
*** gszasz has quit IRC18:46
*** signed8bit_ZZZzz is now known as signed8bit18:47
alaskimelwitt: I was just chatting in #openstack-operators with mgagne so I already had context :)18:47
*** mtanino has joined #openstack-nova18:47
*** mutoulbj has joined #openstack-nova18:48
dansmithvilobhmm: ack18:50
melwittalaski: nice! I was wondering, did I miss something? heh18:50
*** dims_ has quit IRC18:51
*** dims_ has joined #openstack-nova18:52
*** otter768 has quit IRC18:52
*** mutoulbj has quit IRC18:53
vilobhmmdansmith : today saw a  bug  in resource tracker code and thought will check with you if you have some time18:54
dansmithvilobhmm: I'm not the best to ask about RT stuff, but you can try18:55
vilobhmmhttps://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L771 if reserved_host_disk_mb is never set and if used the default value ; "self.compute_node.local_gb_used" will always return 018:55
vilobhmmwhich is bad ! IMHO18:55
*** setuid has quit IRC18:55
vilobhmmand can lead to -ve values in disk space calculations18:56
*** shyama has quit IRC18:56
vilobhmmits been the same since many releases now18:56
vilobhmmdansmith : ^^18:56
*** dims_ has quit IRC18:56
vilobhmmthe fix is trivial and if needed can submit a patch for it…18:57
mriedemvilobhmm: just report a bug and push up a change for review18:57
mriedemif you think there is an issue18:58
dansmithvilobhmm: I think that's well-understood actually18:58
dansmithvilobhmm: I think jaypipes would be the one to ask18:58
vilobhmmmriedem : agree…since i am new to resource tracker thought will check dansmith or jaypipes …18:59
mriedemlocal_gb_used would be 0 so you're not reserving any disk gb for the host18:59
vilobhmmmriedem : will do18:59
vilobhmmmriedem : so does it mean that the code behaves as expected19:00
mriedemit appears so19:00
vilobhmmif so then when we used local_gb_used  to calculate free_space it report inconsistent values19:00
*** annashen has quit IRC19:00
vilobhmmsometimes even -ve values19:01
*** jecarey_ has joined #openstack-nova19:01
vilobhmmfor free_space19:01
mriedemvilobhmm: maybe read up on the original change for reasons https://review.openstack.org/#/c/13182/19:01
*** jecarey has quit IRC19:02
vilobhmm mriedem : alrite…lemme check19:02
*** setuid has joined #openstack-nova19:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtISCSIVolumeDriver into it's own module  https://review.openstack.org/20230119:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module  https://review.openstack.org/19731119:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move volume/remotefs/quobyte modules under volume subdir  https://review.openstack.org/19322319:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module  https://review.openstack.org/19734119:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module  https://review.openstack.org/19685919:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move the LibvirtScalityVolumeDriver into it's own module  https://review.openstack.org/19324919:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move lvm/dmcrypt/rbd_utils modules under storage subdir  https://review.openstack.org/19319519:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtAOEVolumeDriver into it's own module  https://review.openstack.org/19627619:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move the LibvirtQuobyteVolumeDriver into the quobyte module  https://review.openstack.org/19323619:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtFibreChannelVolumeDriver into it's own module  https://review.openstack.org/19327819:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtGlusterfsVolumeDriver into it's own module  https://review.openstack.org/19338019:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move fibre channel specific utilities into fc module  https://review.openstack.org/19331719:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move the LibvirtGPFSVolumeDriver into it's own module  https://review.openstack.org/19324019:03
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtSMBFSVolumeDriver into it's own module  https://review.openstack.org/19638019:03
*** cpschult has quit IRC19:03
*** READ10 has quit IRC19:04
*** SourabhP has quit IRC19:06
*** tjones1 has joined #openstack-nova19:08
nagyz_can anybody enlighten me about keystone's authtoken's auth_uri vs identity_uri? I know this is not #openstack-keystone but maybe the folks here know :-) and I'm not getting an answer there.19:08
nagyz_I see that in the nova docs we're setting auth_uri AND auth_url19:09
nagyz_but not identity_uri19:09
mriedemjamielennox: ^19:09
dansmithkeystone midcycle is going right now19:09
dansmithhence the lack of people on irc I bet19:09
nagyz_ah19:10
mriedemso that's where bknudson has been all week...19:10
mriedemi knew he didn't take vacations...19:10
nagyz_it was exactly jamielennox's code that tripped me over in heat actually which I spent like 3 hours tracking down yesterday19:10
nagyz_I've got back the (not-so-user-friendly) exception of NoneType not having a replace function from like 100 layers deep.. without the original exception's origin. :)19:10
* nagyz_ loves python19:11
ndipanovhttps://review.openstack.org/#/c/201280/ mriedem pls19:11
ndipanovand the kilo backport19:11
*** belmoreira has joined #openstack-nova19:18
*** lpetrut has joined #openstack-nova19:20
*** agireud has quit IRC19:20
*** annegentle has joined #openstack-nova19:21
*** cpschult has joined #openstack-nova19:22
*** agireud has joined #openstack-nova19:22
*** baoli has quit IRC19:23
*** agireud has quit IRC19:26
*** baoli has joined #openstack-nova19:28
*** Nikolay_St has joined #openstack-nova19:29
*** agireud has joined #openstack-nova19:29
*** annashen has joined #openstack-nova19:33
mgagneI would like a second opinion on this (yet to merge) review because I think we are clearly ending in the wrong direction: https://review.openstack.org/#/c/152703/519:36
sambolalex_xu: I'm starting on the next part of the V3 code restructure, moving the V2.1 code to the correct directories. cc/ edleafe zsais19:37
tonybmgagne: It's +W and running through the check queue now, so I think it's too late for a second opinion.19:39
mgagnetonyb: I think we need to reconsider it and revert it if necessary19:40
tonybmgagne: Sure, a revert is possible19:40
mgagnethe change got approved less than 2 hours after a supportive comment although serious concerns were raised previously. I didn't have time to raised my own concerns due to time constraint.19:42
jlearmanRunning tox for the first time, stuff is missing such as pg_config and xslt-config, which I installed from libpq-dev and libxslt1.dev.  Now, "pyconfig.h" is missing.  Did I miss an important step when installing Ubuntu 14.04?19:48
sdagueit's not too late, the point is right now this is a legit problem for infra cloud coming online19:48
sdagueto give everyone additional resources to run tests with19:48
sdagueand it doesn't hurt anyone else19:49
*** harlowja has joined #openstack-nova19:49
*** harlowja_ has quit IRC19:49
*** tpatil has joined #openstack-nova19:51
dansmithit does affect our guests19:52
dansmithand it's kindof undoable from that point of view19:52
dansmithTBH, I didn't vote on this because it makes me uneasy as well19:52
*** jecarey_ has quit IRC19:52
*** harlowja has quit IRC19:52
*** jecarey has joined #openstack-nova19:52
dansmith(I also didn't -1 it either)19:52
*** harlowja has joined #openstack-nova19:53
*** jecarey_ has joined #openstack-nova19:53
sdaguewell, if we want to hold till next week to talk it through, that's fine. I guess I didn't see the concern in exposing provider networks if that's the way you configured your network19:53
jrollsdague: mgagne makes a valid point there, I think19:54
sdaguejenkins rando failed it anyway19:54
lifelessjlearman: sadly yes, thats commonly tricky. You might find https://rbtcollins.wordpress.com/2015/07/12/bootstrapping-developer-environments-for-openstack/ helpful19:54
mgagnejroll: we already implemented all the blueprint locally a couple of months ago. I know the baremetal use case is very valid. but we have to think more about it then just exposing everything and hope cloud-init will know what to do with it.19:55
jrollmgagne: yeah, I think I agree with you19:55
mgagnejroll: we made the exposition of vlan info conditional based on the compute driver19:55
jrollmy colleague wrote the patch, I primarily work on ironic as well, I'll be around next week if we want to talk about it19:56
sdagueok, but this is coming off the provider networks only from neutron, which means you are only going to get this information if you are on provider networks right?19:56
dansmithyeah, that pretty much sucks19:56
dansmithconditional networkinfo based on the compute driver19:56
sdaguewhich means you were given a port19:56
mgagnesdague: we use provider networks in our virtual cloud19:56
dansmitheven if it is "ironic is special"19:56
mgagnesdague: and virtual instance aren't using vlan interfaces. it is transparent19:57
*** jecarey has quit IRC19:57
jrollso, like I said, I'll be at the sprint next week. if y'all want to set aside an hour to hash this out I'm very +1 on that19:57
mgagnedansmith: it really depends on how you wish to expose the network to the baremetal: access or trunk mode19:57
jrollthere's valid points here.19:57
jlearmanThanks, lifeless!19:58
dansmithmgagne: yeah, but that doesn't need to be virt-driver-specific19:58
sdagueso, it would probably be worth getting mordred and jeblair into the conversation as well, because this is basically their needed feature19:58
mgagnedansmith: virt driver specific is ONE possible implementation which I happen to use and fit my need19:59
jrollit might also depend on the neutron backend19:59
dansmithmgagne: okay, sure19:59
mgagnedansmith: the case is virtual instance shouldn't be exposed to this information because they don't need it19:59
dansmithhaving a -1 from a neutron core is kindof bad form for us to merge, IMHO19:59
mgagnedansmith: and will only cause confusion19:59
*** Mike_D_laptop has quit IRC20:02
*** achanda has joined #openstack-nova20:03
*** Sukhdev has quit IRC20:04
*** exploreshaifali has quit IRC20:04
sdagueok, I'll hold. mgagne can you get this out on the mailing list for discussion, because all the parties involved are not going to be in any one location20:05
openstackgerritDan Smith proposed openstack/nova: Add version counter to Service object  https://review.openstack.org/20173320:07
*** achanda has quit IRC20:09
*** xyang1 has joined #openstack-nova20:09
*** cpschult has quit IRC20:10
openstackgerritGiridhar Jayavelu proposed openstack/nova: Derive resource quota from Flavor's extra_spec when attaching interface to an instance  https://review.openstack.org/19617120:10
openstackgerritGiridhar Jayavelu proposed openstack/nova: Create a new dictionary for type_data in VMwareAPIVMTestCase class  https://review.openstack.org/20276820:10
*** belmoreira has quit IRC20:11
*** salv-orlando has quit IRC20:11
*** agireud has quit IRC20:13
melwittdansmith: you need to create your blueprint on that one :)20:14
dansmithmelwitt: what?20:14
*** Sukhdev has joined #openstack-nova20:15
dansmithoh20:15
dansmithservice-version-nuymber?20:15
*** agireud has joined #openstack-nova20:17
dansmithmelwitt: done.. I'll let johnthetubaguy do the approval bits so it doesn't look like I'm approving my own stuff :)20:17
openstackgerritAnthony Lee proposed openstack/nova: Fix live-migrations usage of the wrong connector information  https://review.openstack.org/20277020:18
*** achanda has joined #openstack-nova20:19
*** cpschult has joined #openstack-nova20:23
*** ZZelle_ has joined #openstack-nova20:23
mordredsdague: what did I do?20:25
*** igordcard has quit IRC20:25
sdaguemordred: there was some discussion of the patch to get the provider network info into config drive20:25
sdagueand concerns20:26
mordredya- there was a discussion with jroll and devananda a while ago in #openstack-ironic too20:26
sdagueok20:26
sdaguehow about we get this out to the mailing list20:26
sdaguebecause there are a lot of pockets of people20:26
mordredso - I agree that exposing vlan info to a guest that does not need vlan info is silly20:26
jrollsdague: +120:26
mordredbut that's the nice thing about the format of the metadata20:26
sdagueand I don't think they are talking to each other20:26
sdaguebut, I'm running away for beer o'clock, because I just got grenade multinode d-g actually working, doing the right thing, and passing all the tests20:27
mordredugh. :( That means I'll have to read a mailing list thread - but ok20:27
mordredI honestly have zero ideas why this is problematic - but maybe the mailing list thread will illuminate them for me20:28
sdagueright, so someone start that thread who is vested in this.20:28
sdagueok, beer20:28
mordredI'll send an email20:28
mordredshoudl I tag it with nova, neutron and ironic20:28
mordred?20:28
*** annegentle has quit IRC20:28
dansmithcome on20:29
dansmithyou can do better than that20:29
dansmithand your tags better have parameters20:29
clarkb[tags]20:29
jroll[all] [vlan] [tags] [neutron] [nova] [ironic] [magnum]20:29
jlearmanTrying to run openstack for the first time, getting "ERROR: openstack No user with a name or ID of '4268ea30f4154909844eaa6a788dc376' exists."  Any clues?20:29
*** sambol has quit IRC20:30
dansmith[nova:network=neutron:relevance=50%:priority=12][neutron:driver=foo:priority=27][ironic:isweird=True]20:30
jrolllol20:30
mordredoh god20:30
jrollnice.20:30
*** jasondotstar has joined #openstack-nova20:30
*** SourabhP has joined #openstack-nova20:30
*** agireud has quit IRC20:31
jlearmanthe emotes really improve it20:31
*** agireud has joined #openstack-nova20:32
mordredmgagne: oh! I think I see from your comment where we may be missing each other20:32
mordredmgagne: and it's possible that I'm looking at the consumption not the implementation side20:32
mordredmgagne: I do not believe that vlan information should be included in the instance metadata if the hypervisor has already decapsulated the traffic for you20:33
mordredI agree - that woudl be crazypants20:33
mgagnemordred: yep20:33
mgagnemordred: I already implemented a similar patch into our installation and I do understand the baremetal use case because we did address it (in our way) in our implementation.20:33
*** doude has joined #openstack-nova20:33
*** doude has quit IRC20:33
*** doude has joined #openstack-nova20:33
mgagnemordred: addressed in the way you describe20:33
mordredmgagne: great - so is it just the implementation of this patch you have issue with?20:34
*** doude has quit IRC20:34
mordredand not the concept of being able to express vlan information in the metadata if it's needed?20:34
mgagnemordred: well yea because it breaks the world20:34
mordredawesome!20:34
*** doude has joined #openstack-nova20:34
*** doude has quit IRC20:34
*** doude has joined #openstack-nova20:34
mgagnemordred: vlan is part of the spec20:34
mordredvlan should be an optional part of the spec - and it shoudl be there if and only if the compute target needs to know taht vlan info20:34
mgagneyep20:35
mordredok. if that is not what's happening here, then I agree with you20:35
openstackgerritAnthony Lee proposed openstack/nova: Fix live-migrations usage of the wrong connector information  https://review.openstack.org/20277020:35
mordredmgagne: I want glean or cloud-init to be able to look at the metadata and say "oh, look, it says I need a vlan, I should configure it"20:35
mgagnemordred: sure20:36
mordredand have that always work and always be true20:36
mgagnemordred: have fun: https://github.com/jayofdoom/cloud-init-debian-pkg/blob/master/debian/patches/rackspace-onmetal-configdrive.patch20:36
mgagnemordred: if I install a similar cloud-init in a virtual image, everything falls apart20:36
mordredmgagne: oh - I don't use cloud-init :)20:36
*** krtaylor has quit IRC20:36
*** dims_ has joined #openstack-nova20:37
mordredmgagne: I wrote a minimal replacement that only does network and ssh keys20:37
mgagnemordred: well, we have to considerate other sane people too =)20:37
mordredmgagne: looking at the spec, I expect it will take me approx 20 minutes to add the vlan support to it once we agree on an input format20:37
mordredmgagne: :)20:37
*** alex_klimov has joined #openstack-nova20:37
mordredmgagne: you shoudl start using glean!!!20:37
jrollmgagne: sane people don't use cloud-init :P20:37
mgagnemordred: I provide solutions for common people =)20:38
lifelessI really wish I understood the hate-on that folk have for cloud-init20:38
lifelessI realise its become crufty20:38
jrollsource: I use cloud-init20:38
mordredlifeless: too complex20:38
mgagneworks for me20:38
lifelessbut it still solves the core use cases20:38
mordredlifeless: does completely unnecessary things20:38
mordredlifeless: we're having to work aroudn like 8 things that cloud-init breaks in infra right now20:38
mgagneonly problem I have is that it's hard as hell to get stuff merged upstream20:38
lifelessmordred: which can all be turned off20:38
lifelessmordred: (AIUI)20:39
mordredlifeless: OR - I could just not have a thing installed on my system which consistently breaks it in the first place20:39
* mgagne opened a can of worms20:39
mordredrather than installing a complex piece of software which I think include complex configs for disabling much of20:39
*** thangp has quit IRC20:39
*** xyang1 has quit IRC20:39
lifelessmgagne: indeed20:39
mordreds/think/then can/20:39
*** xyang1 has joined #openstack-nova20:39
mgagneok, let me reword that: please consider insane people who didn't discover the light yet =)20:40
mordredlifeless: I do not need a remote shell execution environment that is in addition to ssh. I have ssh. ssh is consistent and debuggable.20:40
mordredmgagne: hahaha. nice :)20:40
mgagnemordred: so the way I addressed it is by making the ironic driver includes extra data in the instance so I can determine if network info should include vlan info or not.20:41
*** dave-mccowan has joined #openstack-nova20:41
mordredmgagne: how did you get the extra data into the instance?20:41
mordredmgagne: (if there is an eas(y|ier) way to solve this, I'm in)20:41
mgagnemordred: by patching ironic driver and updating instance_system_metadata20:41
mordredmgagne: sorry - I asked a vague question20:42
mordredmgagne: how does that cause that information to be communicated into the inside of the instance?20:42
dansmithmgagne: :(20:42
mgagnemordred: a big if/else20:42
mordredmgagne: where?20:42
mgagnemordred: in network info implementation20:42
mgagnemordred: we have our own because upstream wasn't ready yet20:42
mordredok - so you're still using teh network_info.json as the communication vehicle20:42
mgagnemordred: sure20:42
mordredAWESOME20:42
mordredthat's literally all I care about20:43
mgagnemordred: you just have to provide the right info20:43
mordredif it takes 2 more years to figure out the right way to get the information into network_info.json that's fine with me20:43
mordredI just want to know that the network_info.json interface is a thing20:43
mordredthat I can use20:43
mordredinside of the instance20:43
*** oro has quit IRC20:43
mordredfrom config-drive20:43
mgagnemordred: well I already waited 2 years :-/20:43
jrollmgagne: I'd love to see your patch20:44
openstackgerritMatt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277720:44
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277820:44
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277920:44
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert GlusterFS driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278020:44
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert GPFS volume driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278120:44
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert Quotbyte driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278220:44
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278320:44
*** kmARC has quit IRC20:44
mgagnejroll: nothing special tbh. if baremetal: else:20:44
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278320:44
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278220:44
*** salv-orlando has joined #openstack-nova20:45
*** tonytan4ever has quit IRC20:48
*** sambol has joined #openstack-nova20:48
tonybc20:49
tonybstupid macos and no focus follows mouse!20:50
*** tpatil has quit IRC20:51
mriedemnova meeting in 8 minutes20:52
*** tpatil has joined #openstack-nova20:52
*** tonytan4ever has joined #openstack-nova20:52
tonybHey I can do that one ;P  How very novel20:52
*** otter768 has joined #openstack-nova20:53
*** tpatil has left #openstack-nova20:54
*** otter768 has quit IRC20:58
*** ociuhandu has joined #openstack-nova21:02
*** gyee has quit IRC21:02
*** armax has joined #openstack-nova21:03
*** ijw has joined #openstack-nova21:06
*** signed8bit is now known as signed8bit_ZZZzz21:11
*** signed8bit_ZZZzz is now known as signed8bit21:12
*** dane-fichter has joined #openstack-nova21:17
*** doude has quit IRC21:20
*** dane-fichter has quit IRC21:22
*** chuckcarmack has left #openstack-nova21:29
jrolljohnthetubaguy: I went looking for a blueprint and noticed this should have been approved as it was landed in kilo https://blueprints.launchpad.net/nova/+spec/use-configdrive-with-ironic21:29
johnthetubaguyjroll: ah, oops, good heads up21:30
johnthetubaguyjroll: I marked that as implemented in kilo, I think21:31
openstackgerritJim Rollenhagen proposed openstack/nova-specs: Ironic: Multiple compute host support  https://review.openstack.org/19445321:31
jrolljohnthetubaguy: cool, just noticed it was pending approval so thought I'd point it out21:31
johnthetubaguyjroll: we unapprove when it gets bumped from a release21:32
jrolljohnthetubaguy: right, it didn't get bumped, it fully landed in kilo.21:32
johnthetubaguyjroll: I guess it got merged anyways21:32
johnthetubaguyjroll: it probably missed feature freeze, but merged afterwards, or something21:33
jrollgotcha21:33
johnthetubaguymriedem: I think I got those all approved now21:35
*** krtaylor has joined #openstack-nova21:36
*** mriedem has quit IRC21:38
gabriel-bezerrajohnthetubaguy: that code has not been queued into Zuul yet. Is it normal?21:42
*** boris-42 has joined #openstack-nova21:42
gabriel-bezerrahttps://review.openstack.org/19067521:42
gabriel-bezerradansmith: ^21:43
*** edmondsw has quit IRC21:45
*** bnemec has quit IRC21:46
*** alex_klimov has quit IRC21:47
*** angdraug has quit IRC21:48
*** agireud has quit IRC21:49
*** mutoulbj has joined #openstack-nova21:51
gabriel-bezerrajohnthetubaguy: I've just talked to -infra people. Can you remove the -2 from the patch, please? Zuul won't enqueue while the -2 be there: https://review.openstack.org/19067521:55
*** mutoulbj has quit IRC21:56
*** annashen has quit IRC21:57
*** lucas-dinner has quit IRC21:59
*** vilobhmm has quit IRC21:59
*** vilobhmm has joined #openstack-nova22:00
*** jecarey_ has quit IRC22:01
*** dboik has quit IRC22:02
*** davideagnello has quit IRC22:04
*** cpschult has quit IRC22:05
*** burt has quit IRC22:05
*** ociuhandu has quit IRC22:06
*** ijw_ has joined #openstack-nova22:06
openstackgerritJim Rollenhagen proposed openstack/nova: Fix deleted markers with changed-since  https://review.openstack.org/20281222:08
*** ijw has quit IRC22:09
*** achanda has quit IRC22:10
*** ddaskal has quit IRC22:10
*** mdrabe has quit IRC22:11
*** lucas-dinner has joined #openstack-nova22:11
*** Sukhdev has quit IRC22:14
*** jasondotstar has quit IRC22:15
*** Sukhdev has joined #openstack-nova22:16
*** pixelbeat_ has joined #openstack-nova22:17
*** signed8bit is now known as signed8bit_ZZZzz22:19
andreykurilin__johnthetubaguy: hi22:19
andreykurilin__Can you look at https://review.openstack.org/#/c/202656/ ? This a fix for stable/kilo branch of novaclient, which is broken now.22:19
andreykurilin__johnthetubaguy: ^22:20
*** thorst has quit IRC22:21
tonybandreykurilin__: I think you've missed johnthetubaguy it's 11pm for him.22:22
andreykurilin__tonyb: oh...got it. thanks:(22:23
edleafetonyb: saw that you pinged me yesterday. Wassup?22:24
tonybedleafe: You're in San Antonio right?22:24
edleafetonyb: yep22:25
edleafetonyb: you headed to mother ship before the midcycle?22:25
tonybedleafe: wanna grab lunch or a beer?22:25
tonybedleafe: I'm presently in the mother ship22:25
edleafetonyb: can't tonight - daughter's birthday22:26
edleafetonyb: how long are you around?22:26
tonybedleafe: okay np22:26
tonybI fly to MN on Monday.22:26
edleafetonyb: me too22:26
*** vipul is now known as vipoul22:27
*** vipoul is now known as vipul22:27
edleafetonyb: you got plans for tomorrow?22:27
*** achanda has joined #openstack-nova22:27
tonybedleafe: Just work, but if you're free I can ditch that ;P22:27
edleafetonyb: heh22:28
edleafetonyb: ibm is a little stricter :)22:29
edleafetonyb: how about later in the day? Where are you staying?22:29
tonybAloft near the airport22:29
tonybedleafe: but I'll probably pick up a car tomorrow at some stage.22:29
* tonyb shoudl book that ;P22:30
edleafetonyb: wanna meet at Lion & Rose next door?22:30
tonybedleafe: Sure.  That's pretty easy and I'm told that do whiskey samplers ;P22:31
*** sambol has quit IRC22:31
*** mriedem has joined #openstack-nova22:32
*** xyang1 has quit IRC22:33
*** lpetrut has quit IRC22:37
*** IanGovett1 has quit IRC22:38
*** mmedvede has quit IRC22:39
*** achanda has quit IRC22:41
*** ijw_ has quit IRC22:44
*** lpetrut has joined #openstack-nova22:45
*** signed8bit_ZZZzz is now known as signed8bit22:45
*** foexle has quit IRC22:46
*** lucas-dinner has quit IRC22:46
*** mmedvede has joined #openstack-nova22:47
*** ctrath has quit IRC22:50
*** otter768 has joined #openstack-nova22:54
*** thorst has joined #openstack-nova22:54
*** signed8bit is now known as signed8bit_ZZZzz22:55
*** thorst has quit IRC22:56
*** vilobhmm has quit IRC22:57
*** vilobhmm has joined #openstack-nova22:57
openstackgerritDan Smith proposed openstack/nova: Fix the incorrect PciDeviceList version number  https://review.openstack.org/20128022:58
*** otter768 has quit IRC22:58
*** pixelbeat_ has quit IRC22:59
*** ijuwang has quit IRC23:01
*** hogepodge has quit IRC23:02
*** kylek3h has quit IRC23:05
*** tpeoples has quit IRC23:05
*** zzzeek has quit IRC23:08
*** achanda has joined #openstack-nova23:09
*** salv-orlando has quit IRC23:16
kfox1111can you tag flavors with metadata that isn't used by the schedular?23:18
kfox1111(something magnum can look at?)23:18
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20070823:19
*** zigo has quit IRC23:21
*** zigo has joined #openstack-nova23:22
*** jasondotstar has joined #openstack-nova23:25
*** hemna is now known as hemnafk23:25
*** openstackstatus has joined #openstack-nova23:34
*** ChanServ sets mode: +v openstackstatus23:34
*** lpetrut has quit IRC23:34
mriedemkfox1111: i'm not aware of validation on extra_specs having specific keys23:36
kfox1111looking at the docs, it looks like the schedular uses al the values to filter, and if I throw some magnum something key in there, it may cause the vm to fail to launch?23:37
*** ijw has joined #openstack-nova23:38
*** aysyd has quit IRC23:39
*** erlon has joined #openstack-nova23:39
kfox1111can a filter remove stuff from extra_specs before it goes to the next filter?23:42
*** wanghua has joined #openstack-nova23:46
melwittkfox1111: I don't think so. I think if a filter doesn't care about a key, it will ignore it. but I don't know for certain23:49
*** jaypipes has quit IRC23:50
kfox1111I'm thinking I've tagged some flavors with a piece of metadata, and then put the same key/value in a host aggregate to pin flavors to aggregates.23:50
kfox1111I think that use case might break if arbitrary extra metadata's added to extra specs.23:50
kfox1111nova/filter.py line 89, looks like it does not allow a class to remove filter_properties?23:52
kfox1111except.... it looks like its passing in a hash.. that should be by reference. so changes to it should propigate...23:54
*** barra204 has joined #openstack-nova23:54
melwittyes, that is a thing you can do to pin flavors to host aggregates23:55
*** mriedem1 has joined #openstack-nova23:55
*** mriedem has quit IRC23:55
kfox1111so it would break without a filter of some kind to remove the extra_spec before it hit the host aggrigate filter.23:56
kfox1111its looking like it would be possible to write such a filter... not sure how nova would feal about such a thing though. thoughts?23:56
mriedem1kfox1111: out of tree filters aren't a new thing23:57
kfox1111yeah, but the api kind of is for filtering out hosts that dont match the extra spec info. its a bit of a strange use to have the filter just leave the list untouched and tweak the extra spec data itself.23:58
melwittkfox1111: why do you think you need to remove an arbitrary extra_spec? I don't see what you mean in nova/filters.py23:59
kfox1111ok. example.23:59

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