Wednesday, 2016-11-02

*** mlavalle has quit IRC00:06
*** browne has quit IRC00:09
*** jerrygb has joined #openstack-nova00:17
*** jerrygb_ has joined #openstack-nova00:20
*** jerrygb has quit IRC00:23
*** ducttape_ has joined #openstack-nova00:28
*** lamt has joined #openstack-nova00:31
*** mtanino has quit IRC00:35
*** smatzek has joined #openstack-nova00:42
*** kiran-r has quit IRC00:43
*** smatzek_ has joined #openstack-nova00:44
*** baoli has joined #openstack-nova00:47
*** kiran-r has joined #openstack-nova00:48
*** smatzek has quit IRC00:48
*** baoli has quit IRC00:48
*** baoli has joined #openstack-nova00:50
*** baoli_ has joined #openstack-nova00:51
*** jerrygb_ has quit IRC00:52
*** baoli has quit IRC00:55
*** baoli_ has quit IRC00:56
*** tuanluong has joined #openstack-nova00:59
*** tbachman has quit IRC01:00
*** hoangcx has joined #openstack-nova01:01
*** tovin07 has joined #openstack-nova01:05
*** huanxie has joined #openstack-nova01:06
*** aignatov has quit IRC01:08
*** Zer0Byte__ has quit IRC01:10
*** ducttape_ has quit IRC01:10
*** aignatov has joined #openstack-nova01:11
*** rmcall has quit IRC01:11
*** ducttape_ has joined #openstack-nova01:16
*** mbirru has joined #openstack-nova01:17
*** ducttape_ has quit IRC01:19
*** erbadr has joined #openstack-nova01:23
*** ducttape_ has joined #openstack-nova01:23
*** Apoorva has quit IRC01:24
*** ducttape_ has quit IRC01:28
*** Daisy has joined #openstack-nova01:31
*** rmcall has joined #openstack-nova01:33
*** baoli has joined #openstack-nova01:33
*** rmcall has quit IRC01:33
*** tbachman has joined #openstack-nova01:34
*** rmcall has joined #openstack-nova01:34
*** maticue has quit IRC01:36
*** mtanino has joined #openstack-nova01:42
*** Apoorva has joined #openstack-nova01:43
*** masayukig_ has joined #openstack-nova01:45
*** stewie925 has quit IRC01:45
*** Apoorva has quit IRC01:47
*** Daisy has quit IRC01:49
*** masayukig has quit IRC01:50
*** masayukig_ is now known as masayukig01:52
*** zhangyufei has joined #openstack-nova01:54
*** smatzek_ has quit IRC01:54
*** Daisy has joined #openstack-nova01:55
*** Daisy has quit IRC01:55
*** Daisy has joined #openstack-nova01:55
*** thorst has quit IRC01:55
openstackgerritzhaolihui proposed openstack/nova: Fix failed to boot vm with pci_alias's name including space  https://review.openstack.org/39207701:55
*** thorst has joined #openstack-nova01:56
*** baoli has quit IRC01:59
*** baoli has joined #openstack-nova01:59
*** mtanino has quit IRC02:01
*** baoli has quit IRC02:04
*** thorst has quit IRC02:04
*** jerrygb has joined #openstack-nova02:05
*** jvgrant has quit IRC02:14
*** jvgrant has joined #openstack-nova02:14
*** zhangyufei has quit IRC02:15
*** tbachman has quit IRC02:18
*** ducttape_ has joined #openstack-nova02:24
*** armax has quit IRC02:26
*** ducttape_ has quit IRC02:29
*** zhangyufei has joined #openstack-nova02:29
*** tlian has quit IRC02:37
*** sree has joined #openstack-nova02:40
*** tbachman has joined #openstack-nova02:42
*** sree has quit IRC02:45
*** tbachman_ has joined #openstack-nova02:45
*** tbachman has quit IRC02:47
*** tbachman_ is now known as tbachman02:47
*** ljxiash has joined #openstack-nova02:55
*** Daisy has quit IRC02:55
*** Daisy has joined #openstack-nova02:56
*** dave-mccowan has quit IRC02:57
*** FL1SK has joined #openstack-nova03:00
*** shaohe_feng has quit IRC03:00
*** thorst has joined #openstack-nova03:02
*** rmcall has quit IRC03:03
*** ssurana has joined #openstack-nova03:04
*** Jeffrey4l has joined #openstack-nova03:07
*** thorst has quit IRC03:10
*** ducttape_ has joined #openstack-nova03:20
openstackgerritprashkre proposed openstack/nova: Separate CRUD policy for server_groups  https://review.openstack.org/39111303:32
*** mtanino has joined #openstack-nova03:33
*** Daisy has quit IRC03:38
*** ducnc has joined #openstack-nova03:39
*** markvoelker has quit IRC03:41
*** lifeless has quit IRC03:41
*** lifeless has joined #openstack-nova03:42
*** ducttape_ has quit IRC03:44
*** phuongnh has joined #openstack-nova03:44
*** Daisy has joined #openstack-nova03:44
*** lamt has quit IRC03:48
*** Daisy has quit IRC03:49
*** ljxiash has quit IRC03:57
*** ljxiash has joined #openstack-nova03:57
*** ljxiash has quit IRC04:02
*** coolsvap has joined #openstack-nova04:08
*** thorst has joined #openstack-nova04:09
*** Dinesh_Bhor has joined #openstack-nova04:12
*** thorst has quit IRC04:15
*** jerrygb has quit IRC04:19
*** jerrygb has joined #openstack-nova04:24
*** erbadr has quit IRC04:28
*** Daisy has joined #openstack-nova04:29
*** raunak has joined #openstack-nova04:40
*** markvoelker has joined #openstack-nova04:42
*** markvoelker has quit IRC04:48
*** mtanino has quit IRC04:48
*** ducttape_ has joined #openstack-nova04:54
*** hoangcx has quit IRC04:54
*** erbadr has joined #openstack-nova04:54
*** mriedem has quit IRC04:54
*** ducttape_ has quit IRC05:00
*** raunak has quit IRC05:00
*** jerrygb has quit IRC05:03
*** sree has joined #openstack-nova05:04
*** huanxie has quit IRC05:04
*** jerrygb has joined #openstack-nova05:06
*** sree has quit IRC05:09
*** ssurana has quit IRC05:11
*** thorst has joined #openstack-nova05:12
*** pc_m has quit IRC05:16
*** thorst has quit IRC05:20
*** prateek has joined #openstack-nova05:23
*** ljxiash has joined #openstack-nova05:24
*** sheel has joined #openstack-nova05:33
*** huanxie has joined #openstack-nova05:37
*** pc_m has joined #openstack-nova05:43
*** jerrygb has quit IRC05:53
*** jerrygb has joined #openstack-nova05:57
*** garyk1 has joined #openstack-nova05:58
*** garyk1 has quit IRC05:58
*** bkopilov has joined #openstack-nova06:04
*** hshiina has joined #openstack-nova06:04
*** garyk1 has joined #openstack-nova06:05
*** garyk1 has quit IRC06:07
*** garyk1 has joined #openstack-nova06:09
*** garyk1 has quit IRC06:11
*** garyk1 has joined #openstack-nova06:11
*** garyk1 has quit IRC06:11
*** erbadr has quit IRC06:19
*** shaohe_feng has joined #openstack-nova06:19
*** erbadr has joined #openstack-nova06:19
*** thorst has joined #openstack-nova06:19
*** garyk1 has joined #openstack-nova06:20
*** sridharg has joined #openstack-nova06:20
*** nkrinner_afk has quit IRC06:21
*** jerrygb has quit IRC06:22
*** nkrinner has joined #openstack-nova06:23
*** thorst has quit IRC06:25
*** jerrygb has joined #openstack-nova06:27
*** namnh has joined #openstack-nova06:27
openstackgerrithuyupeng proposed openstack/nova: Fix type error  https://review.openstack.org/39242606:31
*** yuntongjin has joined #openstack-nova06:34
*** prashkre has joined #openstack-nova06:38
*** moshele has joined #openstack-nova06:40
*** zhangyufei has quit IRC06:42
*** zhangyufei has joined #openstack-nova06:43
*** markvoelker has joined #openstack-nova06:44
*** pbandark has joined #openstack-nova06:44
*** moshele has quit IRC06:46
*** dikonoor has joined #openstack-nova06:47
*** dikonoor has quit IRC06:48
*** markvoelker has quit IRC06:50
*** rcernin has joined #openstack-nova06:50
openstackgerrithuyupeng proposed openstack/nova: Fix typo error  https://review.openstack.org/39242606:50
*** noama has quit IRC06:51
*** dikonoor has joined #openstack-nova06:53
*** noama has joined #openstack-nova06:53
*** noama_ has joined #openstack-nova06:53
*** noama has quit IRC06:54
*** noama_ has quit IRC06:54
*** noama has joined #openstack-nova06:54
*** noama_ has joined #openstack-nova06:54
*** noama_ has quit IRC06:54
*** noama has quit IRC06:55
*** trisq has joined #openstack-nova06:55
*** noama has joined #openstack-nova06:55
*** noama has quit IRC06:57
*** noama has joined #openstack-nova06:57
*** noama has quit IRC07:00
*** noama has joined #openstack-nova07:01
*** ekuris has joined #openstack-nova07:04
*** mjura has joined #openstack-nova07:04
*** jerrygb has quit IRC07:06
*** ekuris has quit IRC07:07
*** ekuris has joined #openstack-nova07:07
*** belmoreira has joined #openstack-nova07:10
*** jerrygb has joined #openstack-nova07:10
*** mjura has quit IRC07:11
*** mjura has joined #openstack-nova07:11
*** nkrinner has quit IRC07:13
*** nherciu has joined #openstack-nova07:15
*** tesseract has joined #openstack-nova07:17
*** tesseract is now known as Guest4871207:17
*** nkrinner has joined #openstack-nova07:19
*** thorst has joined #openstack-nova07:23
*** nkrinner has quit IRC07:23
*** nkrinner has joined #openstack-nova07:24
*** tobasco has quit IRC07:27
*** tobasco has joined #openstack-nova07:28
*** avolkov has joined #openstack-nova07:29
*** huanxie has quit IRC07:29
*** huanxie has joined #openstack-nova07:29
*** thorst has quit IRC07:30
*** fragatina has joined #openstack-nova07:30
*** andreas_s has joined #openstack-nova07:31
*** liverpooler has joined #openstack-nova07:36
*** lpetrut has joined #openstack-nova07:36
*** fragatina has quit IRC07:39
*** tobasco has quit IRC07:39
*** fragatina has joined #openstack-nova07:39
*** tobasco has joined #openstack-nova07:41
*** gmann_ has joined #openstack-nova07:42
*** jerrygb has quit IRC07:42
*** sree has joined #openstack-nova07:46
*** jerrygb has joined #openstack-nova07:46
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #openstack-nova07:48
*** jerrygb_ has joined #openstack-nova07:50
*** jerrygb has quit IRC07:52
*** moshele has joined #openstack-nova07:53
*** ducttape_ has joined #openstack-nova07:55
*** jerrygb has joined #openstack-nova07:56
*** edand has joined #openstack-nova07:56
*** jerrygb_ has quit IRC07:58
*** ljxiash_ has joined #openstack-nova07:59
*** zzzeek has quit IRC08:00
*** ducttape_ has quit IRC08:00
*** zzzeek has joined #openstack-nova08:01
*** ljxiash has quit IRC08:03
*** ihrachys has joined #openstack-nova08:05
*** yamahata has quit IRC08:07
*** jerrygb has quit IRC08:08
*** pcaruana has joined #openstack-nova08:09
*** markvoelker has joined #openstack-nova08:13
*** prashkre has quit IRC08:13
*** jerrygb has joined #openstack-nova08:13
*** fandi has joined #openstack-nova08:14
*** markus_z has joined #openstack-nova08:16
*** markvoelker has quit IRC08:19
*** yuntongjin has quit IRC08:21
openstackgerritHuan Xie proposed openstack/nova: xenapi: support the hotplug of a neutron port  https://review.openstack.org/36550508:24
*** andreas_s has quit IRC08:24
*** xinliang has quit IRC08:25
*** ihrachys has quit IRC08:25
*** oanson has joined #openstack-nova08:26
*** jerrygb_ has joined #openstack-nova08:27
*** ducnc has quit IRC08:27
*** thorst has joined #openstack-nova08:27
*** ducnc has joined #openstack-nova08:28
*** xek__ is now known as xek08:28
*** jerrygb has quit IRC08:29
openstackgerritMoshe Levi proposed openstack/nova: pci: remove pci device from claims and allocations when freeing it  https://review.openstack.org/36918008:29
*** prashkre has joined #openstack-nova08:33
*** thorst has quit IRC08:35
*** andreas_s has joined #openstack-nova08:37
*** mgoddard has joined #openstack-nova08:37
*** xinliang has joined #openstack-nova08:38
*** avolkov has quit IRC08:40
*** jaosorior has joined #openstack-nova08:45
*** lpetrut has quit IRC08:45
*** roeyc has joined #openstack-nova08:46
*** lucas-afk is now known as lucasagomes08:51
*** haplo37 has quit IRC08:51
*** haplo37_ is now known as haplo3708:51
*** haplo37_ has joined #openstack-nova08:54
*** sandanar has joined #openstack-nova08:54
*** GB21 has joined #openstack-nova08:55
*** noama has quit IRC08:57
*** noama has joined #openstack-nova08:58
snikitingaryk1: Hi! I want to ask you one question about Diagnostic info spec: https://review.openstack.org/#/c/357884/9/specs/ocata/approved/restore-vm-diagnostics.rst08:58
snikitinMatt said that you may know why during previous implementations we didn't implement driver_private_data field. Do you have some information about it?08:58
*** matrohon has joined #openstack-nova09:00
*** yassine has joined #openstack-nova09:04
*** yassine is now known as Guest8764209:04
pkholkinjohnthetubaguy: bauzas: hey folks! please return to the spec when you have time https://review.openstack.org/#/c/381912/, thanks!09:07
*** jerrygb_ has quit IRC09:08
*** Daisy has quit IRC09:11
*** Daisy has joined #openstack-nova09:11
*** jerrygb has joined #openstack-nova09:12
*** pilgrimstack has joined #openstack-nova09:13
*** jaosorior has quit IRC09:14
*** jaosorior has joined #openstack-nova09:14
*** GB21 has quit IRC09:15
*** ihrachys has joined #openstack-nova09:18
bauzaspkoniszewski: roger, roger.09:20
bauzasoops09:20
bauzaspkholkin: ditto.09:20
*** kiran-r has quit IRC09:23
*** derekh has joined #openstack-nova09:26
*** ducttape_ has joined #openstack-nova09:26
garyk1snikitin: hi09:26
*** dencaval has joined #openstack-nova09:27
garyk1snikitin: i think that it was there to enable each driver to provide additional information. say driver foo has feature bar. this could be very useful for diagnostics. the private fields could be used as a means to display this until other drivers support the same data09:28
*** sree has quit IRC09:29
*** sree has joined #openstack-nova09:30
*** ducttape_ has quit IRC09:30
*** brault|a_ is now known as brault|away09:30
*** sree_ has joined #openstack-nova09:31
*** sree_ is now known as Guest7651209:32
*** thorst has joined #openstack-nova09:32
dencavalhey guys, I have configured cache=none by virsh edit command for a guest vm, but the guest seems to keep using cache from host. Do you have any ideas why it is not working to disable the cache?09:33
*** sree has quit IRC09:34
*** edand has quit IRC09:34
*** hshiina has quit IRC09:35
*** markus_z has quit IRC09:35
*** coreywright has quit IRC09:36
*** Guest76512 has quit IRC09:36
*** thorst has quit IRC09:39
*** markvoelker has joined #openstack-nova09:41
openstackgerritGleb Stepanov proposed openstack/nova: Nested Quota: Set default values to subproject  https://review.openstack.org/38372809:42
openstackgerritGleb Stepanov proposed openstack/nova: Add nested quota support  https://review.openstack.org/38372909:42
openstackgerritGleb Stepanov proposed openstack/nova: Add child_hard_limits column  https://review.openstack.org/38372709:42
openstackgerritGleb Stepanov proposed openstack/nova: Add Hierarchy info  https://review.openstack.org/38372609:42
*** markvoelker has quit IRC09:46
openstackgerritGleb Stepanov proposed openstack/nova: Add _rebuild_instance helper method  https://review.openstack.org/37600409:46
*** jaosorior has quit IRC09:48
*** ijw has joined #openstack-nova09:49
*** jaosorior has joined #openstack-nova09:49
*** ijw has quit IRC09:49
*** ijw has joined #openstack-nova09:50
*** ihrachys has quit IRC09:50
*** ihrachys has joined #openstack-nova09:51
*** jerrygb has quit IRC09:54
*** jerrygb has joined #openstack-nova09:58
*** markus_z has joined #openstack-nova09:58
*** Daisy has quit IRC09:58
*** ducnc has quit IRC09:59
*** Daisy has joined #openstack-nova09:59
*** sree has joined #openstack-nova10:03
*** sree has quit IRC10:03
mdboothjohnthetubaguy: If you have a sec, could you take a look at this mechanical change danpb asked for whilst reviewing my nfs patch: https://review.openstack.org/#/c/392231/10:03
*** sree has joined #openstack-nova10:04
mdboothIt's trivial but it touches lots of files, so it's likely to attract merge conflicts10:04
snikitingaryk1: I meant that in previous V3 diagnostic info implementation this field was *not* included in diagnostic info object https://github.com/openstack/nova/blob/master/nova/virt/diagnostics.py#L9310:04
snikitinIt was in spec but it was *not* implemented. And Matt said that you may khow why10:04
openstackgerritJianghua Wang proposed openstack/nova-specs: XenAPI: add common os-xenapi libary  https://review.openstack.org/37927910:04
*** sahid has joined #openstack-nova10:05
*** roeyc has quit IRC10:05
openstackgerritArvinder Singh proposed openstack/python-novaclient: Code to remove excetion for scrub command.  https://review.openstack.org/39024310:05
*** tovin07 has quit IRC10:06
*** Daisy_ has joined #openstack-nova10:06
johnthetubaguymdbooth: oh, outch, let me hit that right now10:06
mdboothjohnthetubaguy: Thanks10:07
*** matrohon has quit IRC10:08
*** jaosorior is now known as jaosorior_lunch10:08
*** sree has quit IRC10:08
*** phuongnh has quit IRC10:08
kashyapmdbooth: I see that's pre-req for the NFS vol mount / unmount patch -- https://review.openstack.org/#/c/383859/10:09
kashyap"that" == https://review.openstack.org/#/c/392231/10:09
*** Daisy has quit IRC10:10
mdboothkashyap: Yeah, danpb didn't like that I was complicating an interface he was trying to clean up10:10
kashyapmdbooth: Yeah, noticed your summary (thanks for that) which clearly outlined the above10:11
*** matrohon has joined #openstack-nova10:12
*** zhangyufei has quit IRC10:14
*** erbadr has quit IRC10:14
*** tuanluong has quit IRC10:16
*** ishant|away has quit IRC10:17
*** danpb has joined #openstack-nova10:17
*** sambetts|afk is now known as sambetts10:18
openstackgerritjichenjc proposed openstack/python-novaclient: Add version pin for image related function  https://review.openstack.org/39252310:20
*** harbor_log is now known as potdirect10:22
*** ishant|away has joined #openstack-nova10:24
*** cdent has joined #openstack-nova10:24
*** ociuhandu has quit IRC10:29
*** guano has joined #openstack-nova10:31
*** cdelatte has quit IRC10:32
*** sree has joined #openstack-nova10:34
*** liusheng has quit IRC10:34
*** portdirect has joined #openstack-nova10:34
*** liusheng has joined #openstack-nova10:36
*** thorst has joined #openstack-nova10:37
*** potdirect has quit IRC10:37
garyk1snikitin: ok, now i recall. jay pipes did not like it. so we removed it10:39
cdentbauzas: we need to arrange a hangout to talk with dan, jay, ed and whoever else about what to do about the scheduler interface to the placement api. When do you think would be a good time. I'm available today or tomorrow, pretty much whenever, but not friday.10:39
*** pilgrimstack has quit IRC10:40
*** pilgrimstack has joined #openstack-nova10:41
*** trisq has quit IRC10:41
*** avolkov has joined #openstack-nova10:44
*** tbachman has quit IRC10:44
*** lpetrut has joined #openstack-nova10:45
*** thorst has quit IRC10:45
*** roeyc has joined #openstack-nova10:47
*** namnh has quit IRC10:48
*** portdirect has quit IRC10:48
bauzascdent: sure, I was in PTO the last 2 days, but I'm there now10:49
*** al_indigo has joined #openstack-nova10:50
cdentbauzas: any particular time work better for you?10:52
cdentI think edleafe is still away, but he may be back tomorrow10:52
cdentbut I can probably proxy his point of view (which is not quite the same as mine) if required10:52
bauzascdent: I don't have a precise agenda10:54
bauzascdent: I'm just separating my WIP change into 2 changes, one that will be the object method, and one being the REST API call10:54
cdentbauzas: if I remember rightly from friday the main goal was to resolve the details of the rest api call10:55
cdentsince jay wasn't in the room we said "let's hangout next week"10:55
bauzasyup10:55
bauzasyup10:55
bauzasthe REST API call is just the interface for the scheduler10:56
bauzasbut we also need a backend method for getting the narrowed list of RPs10:56
*** ducttape_ has joined #openstack-nova10:56
bauzascdent: and given we need to have an agreement on the REST API, I prefer to split those two so we can first merge the DB call10:57
*** rtheis has joined #openstack-nova10:57
cdentmakes sense10:57
mosheledanpb: hi can you review User-controlled SR-IOV ports allocation - https://review.openstack.org/#/c/18224210:58
*** ducttape_ has quit IRC11:01
*** gszasz has joined #openstack-nova11:01
*** guano has quit IRC11:04
*** haplo37 has quit IRC11:04
*** ociuhandu has joined #openstack-nova11:06
*** haplo37 has joined #openstack-nova11:07
johnthetubaguypkholkin: thats for those spec updates, its looking close, I just have questions about the last column in table 211:10
*** portdirect has joined #openstack-nova11:11
*** cdelatte has joined #openstack-nova11:11
*** Oku_OS is now known as Oku_OS-away11:12
*** tojuvone_ has quit IRC11:12
*** rtheis_ has joined #openstack-nova11:13
*** rtheis_ has quit IRC11:13
*** roeyc has quit IRC11:13
*** cdent has quit IRC11:13
*** rtheis_ has joined #openstack-nova11:13
*** rtheis has quit IRC11:14
*** GB21 has joined #openstack-nova11:15
*** matrohon has quit IRC11:16
*** sree has quit IRC11:16
*** sree has joined #openstack-nova11:17
*** markus_z has quit IRC11:20
*** jaosorior_lunch is now known as jaosorior11:21
*** sree has quit IRC11:21
*** sridharg has quit IRC11:22
*** sree has joined #openstack-nova11:22
*** hshiina has joined #openstack-nova11:23
*** roeyc has joined #openstack-nova11:23
*** daemontool has joined #openstack-nova11:25
*** sridharg has joined #openstack-nova11:27
*** sree has quit IRC11:27
*** Daisy_ has quit IRC11:28
*** Daisy has joined #openstack-nova11:28
*** smatzek_ has joined #openstack-nova11:29
*** Oku_OS-away is now known as Oku_OS11:30
*** sree has joined #openstack-nova11:32
openstackgerritStephen Finucane proposed openstack/nova: conf: Standardize formatting of virt  https://review.openstack.org/39256411:32
openstackgerritStephen Finucane proposed openstack/nova: conf: Detail the 'injected_network_template' opt  https://review.openstack.org/39256511:32
openstackgerritStephen Finucane proposed openstack/nova: conf: Move virt options to a group  https://review.openstack.org/39256611:32
openstackgerritStephen Finucane proposed openstack/nova: conf: Add 'preallocate_images' to 'virt' group  https://review.openstack.org/39256711:32
*** gouthamr has quit IRC11:33
*** GB21 has quit IRC11:33
*** Daisy has quit IRC11:33
sahidjohnthetubaguy: we have an important spec related to the work on realtime worklow, do you have a moment to look at it ? https://review.openstack.org/#/c/284094/7/specs/ocata/approved/libvirt-emulator-threads-policy.rst11:34
*** dave-mccowan has joined #openstack-nova11:35
*** bvanhav_ has joined #openstack-nova11:36
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Provide an object facade for a subset of RPs  https://review.openstack.org/38624211:36
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Expose a REST API method for a specific list of RPs  https://review.openstack.org/39256911:36
*** sree has quit IRC11:36
*** baoli has joined #openstack-nova11:38
openstackgerritStephen Finucane proposed openstack/nova: hardware: Split '_add_cpu_pinning_constraint'  https://review.openstack.org/36746911:40
openstackgerritStephen Finucane proposed openstack/nova: hardware: Flatten functions  https://review.openstack.org/36747011:40
openstackgerritStephen Finucane proposed openstack/nova: Standardize '_get_XXX_constraints' functions  https://review.openstack.org/38507111:40
openstackgerritStephen Finucane proposed openstack/nova: Rename '_numa_get_constraints_XXX' functions  https://review.openstack.org/38507211:40
openstackgerritStephen Finucane proposed openstack/nova: De-duplicate _numa_get_flavor_XXX_map_list  https://review.openstack.org/38507411:40
*** sree has joined #openstack-nova11:41
*** baoli_ has joined #openstack-nova11:42
*** markvoelker has joined #openstack-nova11:42
sfinucanmacsz: Could you take a look at this again? https://review.openstack.org/#/c/356611/11:43
macszsfinucan: ah, ok11:44
*** nkrinner has quit IRC11:44
*** baoli has quit IRC11:45
*** markvoelker has quit IRC11:46
*** mtanino has joined #openstack-nova11:46
*** nkrinner has joined #openstack-nova11:47
*** cdelatte has quit IRC11:47
*** thorst has joined #openstack-nova11:49
*** liusheng has quit IRC11:50
*** sree has quit IRC11:50
*** sree has joined #openstack-nova11:51
*** liusheng has joined #openstack-nova11:52
*** mtanino has quit IRC11:52
openstackgerritMaciej Szankin proposed openstack/nova: config options: Improve help for SPICE  https://review.openstack.org/31704611:53
*** nkrinner has quit IRC11:54
*** gouthamr has joined #openstack-nova11:54
*** sree has quit IRC12:00
openstackgerritsahid proposed openstack/nova: object: add new field cpu_emulator_threads_policy  https://review.openstack.org/38502212:00
openstackgerritsahid proposed openstack/nova: virt: add numa constraint for emulator threads policy  https://review.openstack.org/38502312:00
openstackgerritsahid proposed openstack/nova: libvirt: return overhead if isolate emulator threads requested  https://review.openstack.org/38536412:00
openstackgerritsahid proposed openstack/nova: virt: update overhead to take into account vCPUs  https://review.openstack.org/38502412:00
*** cdelatte has joined #openstack-nova12:00
*** cdent has joined #openstack-nova12:00
*** paul-carlton2 has joined #openstack-nova12:00
*** sree has joined #openstack-nova12:01
*** prateek has quit IRC12:01
*** krtaylor has joined #openstack-nova12:03
*** sridharg has quit IRC12:03
*** trisq has joined #openstack-nova12:03
*** moshele has quit IRC12:03
johnthetubaguysahid: sfinucan: danpb: we keep adding lots of fine grained control the on the NUMA / real time front, do we have the overall direction documented somwhere? I am just worried we are making something thats really hard for deployers to use. Feel we are passing through virt concepts, rather than abstractions. Maybe thats what they need.12:05
*** lucasagomes is now known as lucas-hungry12:05
*** tbachman has joined #openstack-nova12:06
*** ducttape_ has joined #openstack-nova12:07
*** nkrinner has joined #openstack-nova12:07
sahidjohnthetubaguy: from what i know we do not have such document, it's really related to isolate a cpu from external noise I think we want to continue the work added with realtime12:08
johnthetubaguyyeah, it totally seems useful, I didn't like how we did that the first time, but was important to make forward progress, and the old way was simpler. Its just I don't feel I have all the context around this12:10
danpbjohnthetubaguy: the general vision is that we expose ability to express usage policies, but not direct controls over host placement/allocation12:10
johnthetubaguysfinucan: did you do some admin docs at one point, maybe that would help me?12:10
danpbjohnthetubaguy: eg we have the assumption that the tenant user / cloud admin have zero knowledge of the host12:10
johnthetubaguyyeah, that makes sense, to avoid config hell12:11
sfinucanjohnthetubaguy: Sure - one sec12:11
openstackgerritDaniel Berrange proposed openstack/os-vif: host_info: add ability to filter list of supported vifs  https://review.openstack.org/39259112:11
sfinucanjohnthetubaguy: http://docs.openstack.org/admin-guide/compute-cpu-topologies.html12:11
*** liusheng has quit IRC12:13
johnthetubaguysfinucan: perfect, that helps get my head straight again12:14
*** cdent has quit IRC12:14
*** moshele has joined #openstack-nova12:14
*** sree has quit IRC12:14
*** sridharg has joined #openstack-nova12:15
*** hshiina has quit IRC12:16
johnthetubaguysfinucan: danpb: sahid: if we added hw:realtime_optimised=true/false, as an alias for multiple (changing between releases) keys, would that help for all the pinning options?12:19
*** jaypipes has joined #openstack-nova12:19
*** cdent has joined #openstack-nova12:19
*** ljxiash_ has quit IRC12:19
jaypipes*yawn* morning folks12:19
johnthetubaguysfinucan: danpb: sahid: where true is optimise (isolated) guest performance, false is optimise usage across the host12:20
*** maticue has joined #openstack-nova12:20
danpbjohnthetubaguy: i don't think that'd be flexible enough12:20
*** edmondsw has joined #openstack-nova12:21
sahidjohnthetubaguy: wel that seems complicated, users which wnat to use such workload really have to understand what they are going, like configuring the realtime_maks, the cpu_isolated, the pinning, hugepages...12:21
danpbas there's multiple definitions of what "realtime optimized" means and no single definition suits all uses12:21
sahidI don't see how we can handle such options in one12:21
*** markus_z has joined #openstack-nova12:21
danpbfor example, some guest OS require certain vCPUs are non-realtime, while other guest OS require all vCPUs to be realtime12:22
*** sridharg has quit IRC12:22
*** takedakn has joined #openstack-nova12:22
danpbso you can't hide that behind a single knob, without breaking some set of guest OS12:22
*** sridharg has joined #openstack-nova12:22
danpbhence the need for hw_realtime_mask12:22
johnthetubaguydanpb: so I am not thinking as a replacement, just a kind of "max" setting, agreed some workloads need more detail12:22
danpbwhat you describe is really just a predefined out of the box flavour when a bunjch of settings turned on12:23
danpbi could see devstack, for example, creating a "realtime optimized"  flavour as a convenience12:24
*** trisq has quit IRC12:24
danpblikewise for other deployment tools like triple-o12:24
pkholkinjohnthetubaguy: John, I've looked your comments, do you see some problems with empty?12:24
danpbbut don't think nova needs to get into the business, particularly since we only just made the decision to drop all our pre-defined flavours on the basis that they were no directly usable by anyone12:24
johnthetubaguydanpb: its about making it really easy for deployers and the 80% use case for real time (which may not be a thing)12:24
pkholkinand what list k1,k2 do you mean? there is {k1:v1} or {k1:v1, k2:v2}12:25
danpbjohnthetubaguy: sure, that's what devstack/triple-o/fuel/whatever tool can do12:25
sahidjohnthetubaguy: it's also really related to the guest image12:25
johnthetubaguypkholkin: its just that empty aggregate actually has the exclusive key set on it right?12:25
johnthetubaguydanpb: but this is in the flavor, real people have to think and make decisions about that right?12:26
danpbjohnthetubaguy: there's work being done on triple-o for exactly this, to let it deploy a realtime optimized host + default flavour to use it12:26
johnthetubaguydanpb: if we controlled what that was, as we add new best practices, users wouldn't need to change anything12:26
pkholkinjohnthetubaguy: we can have aggreagate with empty meta, and we accept only empty images/flavors in strict  case12:27
*** sree has joined #openstack-nova12:27
johnthetubaguypkholkin: right, the strict case is enabled by metadata on the aggregate though, so its not really empty any more?12:28
danpbjohnthetubaguy: i don't think nova should be in that business for exactly the reasons that we dropped all our default flavours12:28
danpbnova can document best practice recommendations, but i don't think nova should be implementing them - the deployment tools and/or cloud admin should be doing that12:29
*** trisq has joined #openstack-nova12:29
*** xyang1 has joined #openstack-nova12:30
pkholkinjohnthetubaguy:  empty on empty is ok, I don't think it is a problem12:30
*** liverpooler has quit IRC12:31
johnthetubaguydanpb: it just feels like the 80% deployer wants to say "best isolated guest performance" vs "make best use of host resources", right now we force them to go learn about NUMA and how it affects virt technologies, which seems bad for the 80% case, but agreed its wanted for people who need to get12:31
pkholkinsmth on empty aggregate - is bad12:31
johnthetubaguypkholkin: I mean empty actually means, in an aggregate with the isolation metadata set, which doesn't seem clear from the spec12:32
danpbjohnthetubaguy: like i said though, deployment tools can implement this default policies, so admins don't need to understand it12:32
*** liusheng has joined #openstack-nova12:32
*** liverpooler has joined #openstack-nova12:33
danpbthe deployment tools are in the position to then let the knowledgable admins customize it at time of deployment12:33
danpbwhich is not something that's easily done if the policy is implemented in nova code12:33
*** baoli_ has quit IRC12:34
pkholkinjohnthetubaguy: do you mean that 'strict_mode=true', so aggregate meta is not empty?12:35
johnthetubaguypkholkin: yes12:36
*** baoli has joined #openstack-nova12:36
pkholkinjohnthetubaguy: undestand :) mhhh12:36
*** gstepanov has joined #openstack-nova12:36
pkholkinhmmm)12:36
johnthetubaguypkholkin: mostly because you could also have one without that set12:36
johnthetubaguyone aggregate that is12:36
pkholkinI will add that I don't consider it12:36
johnthetubaguypkholkin: the other side of this is what does strict mean for the empty aggregate, we actually have to look at all aggregates in the system and what keys they have, I believe?12:37
johnthetubaguypkholkin: oops, actually, we don't, its just any key that has the known separator/prefix12:38
shewlessHi mdbooth and ftersin: Just wondering how this review is coming along: https://review.openstack.org/#/c/346263.12:38
pkholkinjohnthetubaguy: our approach is 'per-aggregate', so each is individual, empty aggregate is with {} or {'strict_mode:false} meta12:39
openstackgerritDaniel Berrange proposed openstack/os-vif: host_info: fix has_plugin/get_plugin methods on HostInfo  https://review.openstack.org/39261312:40
openstackgerritDaniel Berrange proposed openstack/os-vif: host_info: fix has_vif/get_vif methods on HostPluginInfo  https://review.openstack.org/39261412:40
*** ducttape_ has quit IRC12:40
*** markvoelker has joined #openstack-nova12:41
johnthetubaguydanpb: it just feels like there is an 80% case we are forcing lots of people to understand, including general API users listing flavors in the API, I get some folks need/want the extra control and detail. I should try write that up I guess, to make my case more clearly. Its possible its a make believe thing.12:42
*** jerrygb has quit IRC12:42
openstackgerritManas Mandlekar proposed openstack/nova: Don't destroy evacuated instance if host didn't change  https://review.openstack.org/39262012:43
*** ljxiash has joined #openstack-nova12:43
*** gouthamr has quit IRC12:44
openstackgerritDaniel Berrange proposed openstack/os-vif: host_info: add ability to filter list of supported vifs  https://review.openstack.org/39259112:44
*** jerrygb has joined #openstack-nova12:45
*** gouthamr has joined #openstack-nova12:45
*** jheroux has joined #openstack-nova12:47
ftersinshewless: hi. it's still waiting for reviewers. mdbooth, danpb, and mriedem are welcome12:47
*** liusheng has quit IRC12:47
*** liusheng has joined #openstack-nova12:48
danpbftersin: i want mdbooth to ack it first12:49
alex_xunova api subteam meeting is in 10 mins at #openstack-meeting-412:50
*** hshiina has joined #openstack-nova12:51
*** trisq has quit IRC12:52
*** ducttape_ has joined #openstack-nova12:58
*** tlian has joined #openstack-nova12:58
*** sheel has quit IRC13:00
*** bvanhav_ has quit IRC13:06
*** haplo37_ has quit IRC13:07
*** READ10 has joined #openstack-nova13:07
*** bvanhav_ has joined #openstack-nova13:09
*** bvanhav_ is now known as bvanhav13:09
openstackgerritStephen Finucane proposed openstack/nova: Ignore IOError when creating 'console.log'  https://review.openstack.org/39264313:09
*** haplo37_ has joined #openstack-nova13:09
openstackgerritPavel Kholkin proposed openstack/nova-specs: Strict isolation of group of hosts for image and flavor  https://review.openstack.org/38191213:09
*** prashkre has quit IRC13:10
*** ducttape_ has quit IRC13:10
*** sudipto has joined #openstack-nova13:10
*** sudipto_ has joined #openstack-nova13:10
*** prashkre has joined #openstack-nova13:10
*** mtanino has joined #openstack-nova13:11
*** Guest81678 has quit IRC13:13
*** ekhugen has quit IRC13:13
openstackgerritJay Pipes proposed openstack/nova: placement: implement ResourceClass.create()  https://review.openstack.org/38684413:14
openstackgerritJay Pipes proposed openstack/nova: placement: raise KeyError unknown resource class  https://review.openstack.org/39006213:14
openstackgerritJay Pipes proposed openstack/nova: placement: REST API for resource classes  https://review.openstack.org/39195913:14
openstackgerritJay Pipes proposed openstack/nova: placement: adds ResourceClass.get_by_name()  https://review.openstack.org/39191713:14
openstackgerritJay Pipes proposed openstack/nova: placement: implement destroy() resource class  https://review.openstack.org/39191813:14
openstackgerritJay Pipes proposed openstack/nova: placement: add ResourceClass and ResourceClassList  https://review.openstack.org/38684313:14
*** andreas_s has quit IRC13:14
openstackgerritDan Smith proposed openstack/nova: Make n-net refuse to start unless using CellsV1  https://review.openstack.org/39223613:16
*** mriedem has joined #openstack-nova13:16
*** coreywright has joined #openstack-nova13:17
*** coolsvap has quit IRC13:18
prashkremriedem: Hi, Good Morning. I have put up my first Openstack patch for review(https://review.openstack.org/391113). Could you take a look when you get a chance.Thank you !13:18
sahidjaypipes: can you have a look at this fix when you have a moment? https://review.openstack.org/#/c/356383/13:18
*** eharney_ has joined #openstack-nova13:19
*** andreas_s has joined #openstack-nova13:20
*** ralonsoh has joined #openstack-nova13:20
*** mdrabe has joined #openstack-nova13:20
*** lucas-hungry is now known as lucasagomes13:22
mriedemprashkre: done13:22
*** raunak has joined #openstack-nova13:23
prashkremriedem: Thank you!13:24
*** matrohon has joined #openstack-nova13:25
*** takashin has joined #openstack-nova13:28
*** hshiina has quit IRC13:28
*** ijw has quit IRC13:28
openstackgerritPaul Carlton proposed openstack/nova: Attach/detach root volume for shelved_offloaded  https://review.openstack.org/31744613:29
openstackgerritPaul Carlton proposed openstack/nova: Adding device_present field to bdm table  https://review.openstack.org/26805313:29
*** ssurana has joined #openstack-nova13:33
*** mriedem has left #openstack-nova13:33
*** mriedem has joined #openstack-nova13:33
*** tblakes has joined #openstack-nova13:35
*** smatzek_ has quit IRC13:35
mriedemprashkre: http://docs.openstack.org/developer/nova/code-review.html?highlight=release%20notes#release-notes13:35
mdboothftersin: I'm looking now, btw13:36
*** baoli_ has joined #openstack-nova13:36
*** baoli has quit IRC13:36
mriedemprashkre: i'd just put something in the 'fixes' or 'other' section of a release note13:36
*** baoli has joined #openstack-nova13:37
*** sandanar has quit IRC13:38
*** baoli__ has joined #openstack-nova13:38
*** ducttape_ has joined #openstack-nova13:38
*** baoli_ has quit IRC13:41
johnthetubaguyalex_xu: for that parameter validation, maybe we should add an example API with it? like a really simple one?13:41
*** baoli has quit IRC13:42
*** takedakn has quit IRC13:42
mdboothftersin: Thanks for the fantastic ignore_bdi_for_swap comment, btw.13:42
*** baoli__ has quit IRC13:43
*** roeyc has quit IRC13:43
prashkremriedem: do we need any extra access rights to edit the content in release notes. how can I put my content in it.13:43
*** rodrigods has quit IRC13:43
mriedemyou generate one for your patch13:43
*** rodrigods has joined #openstack-nova13:43
mriedemprashkre: http://docs.openstack.org/developer/reno/usage.html#creating-new-release-notes13:43
*** baoli has joined #openstack-nova13:44
mriedemprashkre: so: tox -e venv -- reno new bug-163615713:44
mriedemand then edit the file created under nova/releasenotes/notes/ that starts with bug-163615713:44
mriedemthe reno should point out there are now separate policies for server group actions13:44
mriedemand that it's backward compatible by default13:45
dansmithbauzas: are you around this week?13:47
*** burt has joined #openstack-nova13:47
*** tblakes has quit IRC13:49
Dinesh_BhorHi all, Can I have more eyes on this: https://review.openstack.org/#/c/392159/?13:49
openstackgerritMaciej Szankin proposed openstack/nova: conf: Move api options to a group  https://review.openstack.org/38449013:49
prashkremriedem: sure. will try it out and get back to you. thank you.13:50
openstackgerritStephen Finucane proposed openstack/nova: Ignore IOError when creating 'console.log'  https://review.openstack.org/39264313:50
*** roeyc has joined #openstack-nova13:50
*** trisq has joined #openstack-nova13:52
*** nherciu has quit IRC13:52
*** nherciu_ has joined #openstack-nova13:52
*** jaosorior has quit IRC13:53
*** trisq has quit IRC13:53
*** gmann_ has quit IRC13:53
ftersinmdbooth, i hope it's clear enough13:54
*** andreas_s has quit IRC13:54
mdboothftersin: Yep. Do you happen to remember where we update the bdms during resize in the existing code, btw?13:55
mdboothI've found it before, but I'm struggling right now.13:55
ftersinwe do not update them, afaik13:56
*** ducttape_ has quit IRC13:56
ftersinit is the reason of https://review.openstack.org/#/c/372721/13:56
*** andreas_s has joined #openstack-nova13:56
*** tblakes has joined #openstack-nova13:56
mdboothftersin: Well, the code which updates instance.root_gb and instance.swap_mb13:56
lbeliveaujaypipes: can you please have a look at this bug fix when you have a chance: https://review.openstack.org/#/c/363884/13:57
ftersinwhen a new flavor is being set as the instance flavor13:57
*** sree has quit IRC13:57
mdboothAh, it's called by _finish_resize13:57
*** baoli has quit IRC13:58
mdboothI had it in my head that we were updating the root bdm or something, but I'm going senile.13:58
Dinesh_BhorThanks mriedem13:58
*** smatzek has joined #openstack-nova13:59
*** baoli has joined #openstack-nova13:59
*** yamahata has joined #openstack-nova13:59
mdboothftersin: So, during creation we're using the bdm, during resize we're using the flavor directly...14:00
ftersinmdbooth, it's impossible to permanently keep in mind all this spaghetty14:00
mdboothYep, that merits a big comment :)14:00
*** sree has joined #openstack-nova14:01
*** GB21 has joined #openstack-nova14:01
*** sree has quit IRC14:02
ftersinmdbooth: to be more accurate, we've started to use bdms during creation, but forgot about resize14:02
mdboothftersin: Yup14:02
mdboothffs, gerrit >:(14:04
* mdbooth types his comment again14:04
*** markus_z has quit IRC14:04
mdboothftersin: How about rebuild after resize?14:06
*** whenry has quit IRC14:06
*** tongli has joined #openstack-nova14:06
*** sree has joined #openstack-nova14:06
mdboothOr manually specified swap which is smaller than the flavor swap size.14:07
mdbooth... I'm not sure we can even handle that.14:07
openstackgerritsahid proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448914:07
openstackgerritsahid proposed openstack/nova: Change RPC post_live_migration_at_destination from cast to call.  https://review.openstack.org/38968714:07
*** mlavalle has joined #openstack-nova14:08
mdboothftersin: Incidentally, why have you put the swap resize fix in here?14:08
mdboothEphemeral resize still doesn't work either, right?14:08
ftersinmdbooth: someone needed swap resize, filed the bug (mentioned in the review comments), and corresponding fix was merged14:09
* mdbooth has a horrible suspicion he's asking to not have something somebody else specifically asked to have added...14:09
mdboothftersin: Let me read the earlier comments first.14:10
ftersini.e. swap resize works now, it is used by someone, and we have to support it14:10
mdboothftersin: 'works'14:10
*** eharney_ has quit IRC14:11
openstackgerritsahid proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448914:11
openstackgerritsahid proposed openstack/nova: Change RPC post_live_migration_at_destination from cast to call.  https://review.openstack.org/38968714:11
mdboothIf they rebuild, they'll get a smaller swap disk though, right?14:11
ftersini mean this: https://bugs.launchpad.net/nova/+bug/124422214:11
openstackLaunchpad bug 1244222 in OpenStack Compute (nova) "nova-resize does not resize swap disk" [Low,Fix released] - Assigned to Thang Pham (thang-pham)14:11
ftersinmdbooth: probably they will14:12
*** bjolo_ has joined #openstack-nova14:12
*** danpb has quit IRC14:14
*** baoli has quit IRC14:15
*** baoli has joined #openstack-nova14:15
*** danpb has joined #openstack-nova14:15
*** sree has quit IRC14:16
*** baoli_ has joined #openstack-nova14:16
*** baoli has quit IRC14:20
*** tbachman has quit IRC14:20
*** ducttape_ has joined #openstack-nova14:22
*** fandi has quit IRC14:24
*** sree has joined #openstack-nova14:25
*** mriedem1 has joined #openstack-nova14:25
*** tbachman has joined #openstack-nova14:26
openstackgerritSergey Nikitin proposed openstack/nova-specs: Restore V3 Diagnostics info  https://review.openstack.org/35788414:26
*** ducttape_ has quit IRC14:26
*** mriedem has quit IRC14:26
jaypipessahid, lbeliveau: yes, reviewing them now.14:27
snikitinjaypipes: Hi Jay! I'm trying to restore Diagnostic info API which will return some standardised information about instances from hypervisors. https://review.openstack.org/#/c/357884/  And I heard from garyk that you was against adding of field 'driver_private_data' with some driver specific information about instance. Do you still against it?14:28
openstackgerritDan Smith proposed openstack/nova: Return build_requests instead of instances  https://review.openstack.org/37426714:28
openstackgerritDan Smith proposed openstack/nova: WIP Add schedule_and_build_instances conductor method  https://review.openstack.org/36755714:28
openstackgerritDan Smith proposed openstack/nova: WIP Add schedule_and_build_instances RPC method  https://review.openstack.org/37932014:28
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937914:28
jaypipessnikitin: is this a normal end-user API?14:28
*** daemontool has quit IRC14:31
*** daemontool has joined #openstack-nova14:32
mdboothftersin: create_image unit tests make me sad :( I'm going to have to rewrite them again. Not your problem, though.14:32
mriedem1jaypipes: no it's admin-only14:32
*** mriedem1 is now known as mriedem14:32
*** baoli_ has quit IRC14:32
*** sree has quit IRC14:33
*** akshai has joined #openstack-nova14:34
jaypipesmriedem: k, I'll look at that and give my opinion. Generally I don't care to leak private implementation details out through the public HTTP API... but we'll see.14:34
mriedemsnikitin: nothing is using that driver_private_data thing in the virt drivers today14:34
mriedemsnikitin: so if nothing is using it, and there is no obvious thing to add there, i'd opt to just remove it from the spec14:34
mriedemsnikitin: the Diagnostics object will be versioned, so if we need to add fields to it for some reason we can do that, and bump the version14:35
*** anush has quit IRC14:36
*** eharney has joined #openstack-nova14:36
openstackgerritJay Pipes proposed openstack/nova: placement: implement ResourceClass.create()  https://review.openstack.org/38684414:38
openstackgerritJay Pipes proposed openstack/nova: placement: raise KeyError unknown resource class  https://review.openstack.org/39006214:38
openstackgerritJay Pipes proposed openstack/nova: placement: REST API for resource classes  https://review.openstack.org/39195914:38
openstackgerritJay Pipes proposed openstack/nova: placement: adds ResourceClass.get_by_name()  https://review.openstack.org/39191714:38
openstackgerritJay Pipes proposed openstack/nova: placement: implement destroy() resource class  https://review.openstack.org/39191814:38
openstackgerritJay Pipes proposed openstack/nova: placement: add ResourceClass and ResourceClassList  https://review.openstack.org/38684314:38
*** sree has joined #openstack-nova14:39
*** dikonoor has quit IRC14:40
*** prashkre has quit IRC14:41
*** jaosorior has joined #openstack-nova14:41
*** victorhe has joined #openstack-nova14:42
*** dmellado is now known as dmellado|lunch14:42
openstackgerritSergey Nikitin proposed openstack/nova-specs: Restore V3 Diagnostics info  https://review.openstack.org/35788414:43
*** jed56 has joined #openstack-nova14:43
*** sree has quit IRC14:43
snikitinmriedem: ok, lets remove it14:43
*** sree has joined #openstack-nova14:44
mriedemsnikitin: more comments inline https://review.openstack.org/#/c/357884/1114:44
snikitinlooking14:44
mriedemit's mostly good, just needs to remove some inaccuracies14:44
*** paw has quit IRC14:45
*** markus_z has joined #openstack-nova14:45
*** mgoddard_ has joined #openstack-nova14:47
*** ducttape_ has joined #openstack-nova14:48
*** tbachman has quit IRC14:49
mdboothftersin: Done. I've asked for 1 mechanical test change. If you ping me later I'll be happy to +1.14:50
*** mgoddard has quit IRC14:50
*** raunak has quit IRC14:51
*** rrecio has joined #openstack-nova14:52
mriedemdiana_clarke: want to fix the typo in https://review.openstack.org/#/c/386771/ ? then i'm +214:52
mriedemvery nicely done spec14:52
*** mtanino has quit IRC14:52
*** mgoddard_ has quit IRC14:53
mriedemmdbooth: did you have plans for https://review.openstack.org/#/c/382449/ or should i hack on that a bit?14:53
*** mtanino has joined #openstack-nova14:53
*** mgoddard_ has joined #openstack-nova14:53
mriedemi think we had agreed to make some short-term hacky fixes in there and then save the longer-term event listening stuff for later14:53
*** victorhe has quit IRC14:54
mdboothmriedem: Sec, let me refresh my memory14:54
*** sai has quit IRC14:55
*** raunak has joined #openstack-nova14:56
*** raunak has left #openstack-nova14:56
mdboothmriedem: Are you happy hacking on it?14:56
*** rrecio has quit IRC14:56
mdboothI'm probably not going to get to it imminently, tbh.14:57
mriedemi've also lost some context, but i think i was looking for a short-term backoff retry loop checking the xml for "ready='yes'"14:57
mdboothkashyap: Do you want to hack on it?14:57
mriedemi'd prefer kashyap to work on it14:57
mdboothSo with that fix specifically I think we just want to add tests and get it in.14:57
* kashyap clicks14:57
kashyapFirst trying to read context14:57
mdboothWe also need to fix the real problem, though, which is the xml thing you mention, or the event handling14:58
*** victorhe has joined #openstack-nova14:58
*** daemontool has quit IRC14:58
*** sai has joined #openstack-nova14:58
mriedemtrue, we could just do this incrementally14:58
mriedemthis is an obvious fix, add tests, get'r done14:58
mriedemthen do the backoff retry which is backportable if we want14:58
mriedemthen the event listener14:58
jaypipessahid: currently reading through libvirt docs... :( this is going to get ugly.14:59
mriedembrb14:59
kashyapmriedem: If it's the same mount / unmount issue, to honestly admit I was not able to completely my wrap my head around the intricacies, and mdbooth was /is far more competent in figuring out the details.14:59
bauzasdansmith: hola, yup, since today, was just at a doctor's appointment :)14:59
bauzasyesterday was a bank holiday :)14:59
*** rrecio has joined #openstack-nova14:59
dansmithbauzas: okay14:59
kashyapOh, this is a different issue14:59
bauzasdansmith: jaypipes: just lemme know when you want to have a discussion about the REST API call for the scheduler15:00
openstackgerritArtom Lifshitz proposed openstack/nova: Wait for ports down during evacuate  https://review.openstack.org/38508615:00
mdboothkashyap: Yeah, this is the older one where you did the libvirt sleuthing.15:00
dansmithbauzas: so we have a cells meeting scheduled for today, but it would just be us I thnk15:00
kashyapmdbooth: Sorry, my eyes were warped after 2 hours of looking logs from another downstream bug15:00
*** _ducttape_ has joined #openstack-nova15:00
kashyapJust recuperating15:00
dansmithbauzas: I'm trying to wrap my head around all the code currently proposed and address some of my own feedback on them15:01
*** cdelatte has quit IRC15:01
pkoniszewskihello nova, is someone aware whether conf parser has been changed recently? I've observed some issues with libvirt live migration on trunk, basically because all strings read from nova.conf are treated as unicode15:01
dansmithbauzas: do you think we should meet or punt for today?15:01
*** mriedem1 has joined #openstack-nova15:01
bauzasdansmith: I'm fine for today's meeting, but we have an internal meeting one hour before :)15:01
mriedem1kashyap: mdbooth: ok so i'll hack on unit tests for this15:01
mriedem1and play around with the backoff retry15:01
openstackgerritDiana Clarke proposed openstack/nova-specs: Simple tenant usage pagination  https://review.openstack.org/38677115:01
dansmithbauzas: I don't, AFAICT15:02
dansmithbauzas: you're fine meeting or fine canceling?15:02
diana_clarkemriedem1: done, thanks!15:02
bauzasdansmith: fine for the meeting15:02
*** mgoddard_ has quit IRC15:02
sahidjaypipes: what do you mean by "this is going to get ugly" ? do you want me to refactor the change or something15:02
mriedem1dansmith: +2 on the simple tenant usage paging spec https://review.openstack.org/#/c/386771/15:02
*** mgoddard has joined #openstack-nova15:02
bauzasdansmith: 6pm UTC right?15:02
dansmithbauzas: I'm trying to figure out: do you have something to discuss with just me? :)15:02
kashyapmdbooth: Sure.  (Disregard me earlier response, as that applies to a different issue -- volume attach / detach race that mdbooth was investigating)15:02
dansmithbauzas: 2100UTC by my calendar15:03
mriedem1bauzas: are you talking about the cellsv2 meeting?15:03
bauzasdansmith: I thought you were asking me for the cells meeting :)15:03
dansmithI am15:03
bauzasoh okay, 2100UTC is better for me then15:03
jaypipessahid: :) no, I mean reading the libvirt documentation gives me zero insight into what the emulatorpin has to do with realtime stuff.15:03
bauzassorry for the confusion :)15:03
*** Zer0Byte__ has joined #openstack-nova15:03
dansmithbauzas: I'm not asking about scheduling15:03
dansmithbauzas: I'm asking if there is any point in having the meeting if it's just the two f us15:04
bauzascellsv2 meeting is fine by me for 2100UTC15:04
bauzasoh ok15:04
bauzas:p15:04
*** ducttape_ has quit IRC15:04
bauzasgrah, my brain is probably still in vacation...15:04
*** mriedem has quit IRC15:04
mriedem1dansmith: fwiw i probably won't be there today15:04
mriedem1have a conflict15:04
bauzasdansmith: so, good point, what could we be discussing for the cellsv2 meeting, I dunno, maybe around the last points we discussed during the meetup15:05
bauzasmaybe about following up15:05
dansmithbauzas: right my point is we both just discussed things on friday, and I'm busy getting up to speed with the things proposed15:05
bauzasbut if mriedem1 can't be there and melwitt too, then meh.15:05
dansmithso if nobody else is here, I don't see the point15:05
bauzasokay, then let's punt it15:05
sahidjaypipes: ohh i see :) - we just want to ensure that the threads owned by qemu to make the guest running, to be pinned on host CPUs which are not intended to be used for RT15:06
kashyapmriedem1 I can work on that issue, as you may have other things on your plate.15:06
openstackgerritSergey Nikitin proposed openstack/nova-specs: Restore V3 Diagnostics info  https://review.openstack.org/35788415:06
mriedem1kashyap: so you'll add tests to https://review.openstack.org/#/c/382449/ ?15:07
bauzasdansmith: does melwitt need some kind of discussion again for the quotas ?15:07
dansmithbauzas: she's on PTO this week15:07
bauzasor do we have kind of agreement here?15:07
bauzasoh okay, then let's punt that meeting15:07
snikitinmriedem1: fixed your comments15:07
kashyapmriedem1: I want to  give it a shot, and you / MattB could review15:07
mriedem1kashyap: go nuts15:07
mriedem1let my -1 be your guide15:08
kashyapHeh15:08
kashyapmriedem1: Just FYI - tomm evening / Fri, I have to attend some urgent errands, just noting if you don't see me active on that this week15:08
kashyap(Don't assume I dropped it :-))15:08
jaypipessahid: so you are saying that in order to support the realtime contract, the QEMU emulator process must run on the same physical CPU as all of the virtual CPUs are pinned to?15:08
mriedem1kashyap: ok, maybe diana_clarke would like to write the tests15:09
mriedem1i'm happy to do it, but then i can't +215:09
kashyapSure, if she has cycles right now, can go ahead15:09
*** matrohon has quit IRC15:09
danpbjaypipes: no,  that's not right15:09
jaypipesdanpb: could you explain for me please? I'm not following along this bug report unfortunately. :(15:09
danpbjaypipes: the emulator thread(s) must *never* run on a pCPU that is used by a guest vCPU marked as real-time15:09
jaypipesdanpb: ah.15:10
danpbas that guest vCPU will starve the emulator thread15:10
jaypipesunderstood. that makes more sense.15:10
sahidjaypipes: that is the "normal" contract, for the realtime contract we want the QEMU emulator process pinned only on host CPUs which are not used for RT15:10
sahidyes as danpb said better than me15:10
danpbby default Nova currently requires that at least 1 vCPU be marked non-realtime, and so we've currently placed the emulator threads on the pCPU used by the non-realtime vCPU15:10
*** paw has joined #openstack-nova15:10
*** mriedem1 is now known as mriedem15:10
jaypipesdanpb: k15:10
danpbjaypipes: some (non-Linux) guest OS require that *all* vCPUs are realtime15:11
*** lpetrut has quit IRC15:11
danpbjaypipes: thus, we need to enhance nova so that it can place emulator threads on a pCPU that is not used by *any* vCPU, and account for that extra pCPU usage by the guest15:11
*** annegentle has joined #openstack-nova15:12
danpbthis will enable a much wider variety of real time operating systems to be used on nova15:12
*** _ducttape_ has quit IRC15:12
*** mnestratov has joined #openstack-nova15:12
*** ducttape_ has joined #openstack-nova15:12
*** browne has joined #openstack-nova15:13
bauzasjaypipes: FWIW, I sliced the WIP patch for the RP API into 2 separate commits, one for the object interface and one for the needs-consensus REST API call15:13
jaypipesdanpb, sahid: I'm still confused by the cpu_realtime_mask=^0 thing. How would that value be determined by an operator (the ^0 value..)15:13
jaypipesbauzas: cool, thank you for that!15:13
danpbjaypipes: that basically tells you how many vCPUs are realtime vs non-realtime15:14
bauzasjaypipes: AFAIR, we now longer use indexes for RC classes, so I guess I should query the classname string instead of the index for https://review.openstack.org/#/c/386242/3 right?15:14
sahiddanpb: hum I think you are talking about the spec "emulator threads" where jaypipes is (so far) only reviewing the fix https://review.openstack.org/#/c/356383/15:14
danpbso you have to decide that based on what guest OS you're using15:14
danpbso most commonly you'd set that against the image meta15:14
bauzasjaypipes: that means the resource class name needs and is actually unique, right?15:14
*** garyk1 has quit IRC15:15
jaypipesdanpb, sahid: or rather... I'm wondering what the purpose of the cpu_realtime_mask thing is at all. Can't we figure out where to place the emulator thread simply by looking at the cpuset of vcpus and choosing one not in that list?15:15
*** tongli has quit IRC15:15
bauzasjaypipes: hence the proposal to have prefixes for the custom resource classes, right,15:15
danpbjaypipes: if you have say 4 vcpus, we need to say how many of those vCPUs are real-time vs non-realtime - that's what cpu_realtime_mask determines - there's no other info to make that decision on15:16
jaypipesbauzas: the resource_classes.name column is unique and indexed.15:16
*** lpetrut has joined #openstack-nova15:17
*** anush has joined #openstack-nova15:17
jaypipesdanpb: ah, ok, I got it now. sorry, I wasn't aware that you could have non-realtime vCPU and realtime vCPUs all on the same guest.15:17
bauzasjaypipes: cool then15:17
*** diegows_ has quit IRC15:18
danpbjaypipes: yeah, for linux it is in fact mandatory to have that15:18
*** tbachman has joined #openstack-nova15:18
danpbjaypipes: because its a legacy os and so many parts of userspace are not able to run real time15:18
*** mdrabe has quit IRC15:18
danpbso for linux you essentially always need at least 1 vcpu non-realtime15:18
jaypipesdanpb: k, I think I understand this space a bit better now thanks very much for your help. you too sahid! :)15:19
danpbits only the dedicated  realtime operating systems where you find all vCPUs real time15:19
sahidjaypipes: no problem, thanks to take time for reviewing it, sorry if my "explanations" confused you15:20
* danpb thinks this is a reasonable illustration of linux real-time capabilities ;-P http://3.bp.blogspot.com/-qye_di2qj1Q/U8Cenb0RJoI/AAAAAAAAAgk/vRQaZfFDLjg/s1600/Dog-rocket.png15:20
kashyapLOL15:21
jaypipesdanpb: lol :)15:21
jaypipesdanpb: looks like mriedem's Mortimer..15:21
kashyapSheesh, that's crazy15:21
*** woodster_ has joined #openstack-nova15:21
*** lpetrut has quit IRC15:23
*** pbandark has quit IRC15:26
*** mdrabe has joined #openstack-nova15:26
*** tbachman has quit IRC15:28
*** tbachman has joined #openstack-nova15:30
*** moshele has quit IRC15:32
*** haplo37 has quit IRC15:33
johnthetubaguypaul-carlton2: are you still working on this live-migration spec? https://review.openstack.org/#/c/347161/ I think raj_singh might have some folks who could help out with that15:36
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/39275315:36
*** takashin has left #openstack-nova15:37
*** harlowja_at_home has joined #openstack-nova15:38
*** mmedvede has quit IRC15:38
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements  https://review.openstack.org/37752715:38
*** roeyc has quit IRC15:39
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements  https://review.openstack.org/39282615:40
*** rama_y has joined #openstack-nova15:42
*** haplo37 has joined #openstack-nova15:42
ftersinmdbooth: thanks15:44
*** tblakes has quit IRC15:45
diana_clarkemriedem, kashyap: sure, I'll look into tests for https://review.openstack.org/#/c/382449/15:46
mriedemthanks15:46
kashyapdiana_clarke: Thanks!15:46
openstackgerritJulian proposed openstack/nova: Transform instance.snapshot notifications  https://review.openstack.org/33012815:46
openstackgerritJulian proposed openstack/nova: Remove redundant VersionedObject Fields  https://review.openstack.org/38984015:47
openstackgerritJulian proposed openstack/nova: Create schema generation for AddressBase  https://review.openstack.org/39154615:47
*** dmellado|lunch is now known as dmellado15:47
*** Zer0Byte__ has quit IRC15:48
*** mmedvede has joined #openstack-nova15:50
openstackgerritMatt Riedemann proposed openstack/nova: api-ref: distinguish between 2.10 for keypair user_id in responses  https://review.openstack.org/39284515:50
*** ekhugen_alt has joined #openstack-nova15:50
*** egarbade- has joined #openstack-nova15:51
*** krtaylor has quit IRC15:54
openstackgerritMatt Riedemann proposed openstack/nova-specs: Restore V3 Diagnostics info  https://review.openstack.org/35788415:55
mriedemsnikitin: +2 on the diagnostics spec now15:55
*** rrecio_ has joined #openstack-nova15:56
*** andymaier has joined #openstack-nova15:57
*** ekuris has quit IRC15:57
kashyapdiana_clarke: If you have functional change related context questions, feel free to ask here or on the review...15:57
*** rrecio has quit IRC15:59
*** anushkrishnamurt has joined #openstack-nova16:02
*** harlowja_at_home has quit IRC16:03
mriedemjaypipes: dansmith: i was going to write a single session recap for the 3 placement/resource providers sessions - is that ok with you or did you want to write those up individually?16:04
mriedemthey aren' tsuper fun to write so i figured you'd be ok with me doing it16:04
dansmithfine with me16:05
mriedemok16:05
*** pcaruana has quit IRC16:06
*** oanson has quit IRC16:06
*** mgoddard_ has joined #openstack-nova16:07
jaypipesmriedem: fine with me, yes16:07
bauzasmriedem: heh, I love why we're asking you to write those16:07
mriedemdoes not compute16:07
*** rcernin has quit IRC16:09
*** anush has quit IRC16:10
*** mgoddard has quit IRC16:10
openstackgerritJohn Garbutt proposed openstack/nova-specs: Live Migration of Rescued Instances  https://review.openstack.org/34716116:11
bauzasjaypipes: given I'm updating https://review.openstack.org/#/c/386242/3/nova/objects/resource_provider.py for now using the RC cache (for getting the class ID), I was thinking how it could be useful for operators troubleshooting resource usage to get the resource class IDs16:11
bauzasjaypipes: given they could be checking SQL, they need to know which class name corresponds with each RC id16:11
bauzasjaypipes: are we envisaging to expose those matchings at the REST API level ?16:12
bauzasif that's in the custom-resource-class spec, please blame me, I need to reread it16:12
paul-carlton2johnthetubaguy, raj_singh should go ahead I will not have time to work on it16:13
johnthetubaguypaul-carlton2: OK, thanks for confirming, we certainly have a head start with all your patches that are already up there16:13
raj_singhpaul-carlton2: Thx16:14
*** armax has joined #openstack-nova16:14
johnthetubaguypkoniszewski: just a quick heads up, trying to kick start the work on live-migrate of rescued things16:14
johnthetubaguy(see above)16:14
*** ssurana has quit IRC16:15
bauzasjaypipes: well, AFAICS, that's not in https://review.openstack.org/#/c/391959/5/nova/api/openstack/placement/handlers/resource_class.py16:15
openstackgerritJohn Garbutt proposed openstack/nova-specs: Live Migration of Rescued Instances  https://review.openstack.org/34716116:15
jaypipesbauzas: no plans to expose the autoincrementing ID outside of the HTTP API, no.16:17
bauzasjaypipes: do you think it could be something operators would need ?16:17
bauzasjaypipes: I'm just with my MySQL workbench tool, and I'd love to know what means "1" as a resource class index :)16:18
bauzasoops s/index/id16:18
bauzasof course, I can guess it now16:18
bauzasbut with nested resource classes, I think it would be far more difficult :)16:19
jaypipesbauzas: there's no such thing as nested resource classes :)16:20
jaypipesonly nested resource providers.16:20
bauzasoh man, right16:20
pkoniszewskijohnthetubaguy: oh, thanks, i will be happy to help there, code, review, tests, anything :)16:20
jaypipesbut anyway, I can kind of think it would be useful to add the standard resource class index/ids into the resource_classes table, but I still don't want to leak those IDs out of the API.16:20
bauzasjaypipes: right, that's the main problem I see16:21
jaypipesbauzas: if you wanted to propose a script that added the stadnard resource classes index IDs into resource_classes, I'd be cool with that.16:21
bauzasjaypipes: I just see that as a very specific REST call for operators wanting to troubleshoot16:21
johnthetubaguypkoniszewski: tests and review would be a great help once we get that moving16:21
bauzasor anything but REST16:21
jaypipesbauzas: just so you could, say, do a join against resource_classes to get the resource class name?16:21
bauzasjaypipes: I'm confused, I thought we were not persisting the non-custom classes?16:22
*** baoli has joined #openstack-nova16:22
jaypipesbauzas: we aren't.16:23
bauzasjaypipes: so, you're proposing to persist those?16:23
jaypipesbauzas: but of course, we could put them in resource_classes and eventually get rid of the entire fields.ResourceClassField16:23
jaypipesbauzas: that, though, wouldn't be backwards-compatible which is the reason for the resource class cache stuff to begin with.16:24
bauzasyeah16:24
bauzasjaypipes: the remediation could be a "dump" tool that would store the mappings into that table, but that's ugly :)16:24
bauzaseither way, I need to think more of that16:25
bauzasthe fact that we're using an internal cache for the mapping as the single source means that somehow we need to assert that what could be in the DB is just a view, not the real list16:26
*** jaosorior has quit IRC16:28
*** bjolo_ has quit IRC16:31
mriedemcdent: something we never talked about at the summit in https://etherpad.openstack.org/p/ocata-nova-summit-resource-providers-quantitative was documenting the placement api-ref16:32
mriedemcdent: did you have ideas on how to generate those docs?16:32
cdentmriedem: sweat and tears?16:32
mriedemwell i thought you said that gabbi could somehow be used for that16:32
mriedemor are we just going to do it like the api-ref we already have for the compute api?16:33
*** bvanhav has quit IRC16:33
cdentnot that I recall. you may be thinking of this thing which people asked for: https://review.openstack.org/#/c/370204/ (that's now out of date)16:33
*** zul_ has joined #openstack-nova16:33
*** mjura has quit IRC16:33
*** zul_ has quit IRC16:33
cdentbut there's nothing built into gabbi for dumping _good_ docs16:33
cdentso yeah, probably basically like api-ref16:33
mriedemok16:33
mriedemthe compute api-ref uses the api-samples for the examples16:34
mriedemwhich was basically i think how https://review.openstack.org/#/c/370204/ came up16:34
*** Guest48712 has quit IRC16:34
cdentah, okay16:34
mriedemi wonder if swagger could be used16:35
mriedemthere were reasons it wasn't used for the compute api, which i thought had to do with microversions,16:35
mriedembut sdague and oomichi would know most about that16:35
*** zul has quit IRC16:35
cdentgiven time and energy there's long termish plans for swagger<->gabbi interoperation16:35
*** zul has joined #openstack-nova16:36
cdentthere's no reason why things like the api-samples couldn't be created (in anything, including gabbi) for supporting the placemen docs16:36
*** pilgrimstack has quit IRC16:36
*** dikonoor has joined #openstack-nova16:36
cdentI believe the issues that sdague reported with swagger were that it wasn't great for apis that already exist16:36
cdentas it imposes its own sets of constraints16:37
*** mmedvede has quit IRC16:37
mriedemok, hmm, maybe we could hook something into the placement tests to just generate the api samples16:37
mriedemto dump those json files16:37
mriedemwe have something like that for the nova functional api_samples tests16:37
mriedemanyway, don't want to rathole on it right now16:38
cdentit's on my radar, and I think we can come up with a way to make it relatively straightforward16:38
*** zul has quit IRC16:39
*** zul has joined #openstack-nova16:39
*** ayoung has quit IRC16:40
*** pilgrimstack has joined #openstack-nova16:40
*** anushkrishnamurt has quit IRC16:40
*** nkrinner is now known as nkrinner_afk16:42
*** baoli has quit IRC16:42
*** baoli has joined #openstack-nova16:42
cdentedleafe: are you still in bcn or transit or are you back in the world?16:44
*** mmedvede has joined #openstack-nova16:45
*** paul-carlton2 has quit IRC16:45
mriedemjaypipes: dansmith: i don't think, besides this ironic flavors extra specs thing, that we ever got into what's going to be required to upgrade to ocata wrt resource providers - in https://etherpad.openstack.org/p/ocata-nova-summit-resource-providers-quantitative16:45
mriedemam i missing something?16:45
*** sridharg has quit IRC16:45
mriedemlike, for example, does the placement service become required in ocata?16:46
mriedemi thought that was the plan16:46
dansmithyeah, required in ocata16:46
dansmithso stand it up in newton, run it enough to get all your data in there before you roll to ocata16:46
mriedemand you break in ocata if you don't have it because the scheduler is going to rely on the placement data?16:47
dansmithyeah16:47
*** artom has quit IRC16:48
dansmithnow that you're saying this, I'm thinking we probably weren't as forceful in our release notes about what "optional" meant and what we expect you to do in newton16:48
jaypipesmriedem: yup, what dansmith is saying.16:48
mriedemyeah the release notes don't really say anything. they just say the compute tries to communicate with it via the service catalog 'placement' entry, but if it's not there, it just logs a warning and doesn't try again16:49
mriedemhttp://docs.openstack.org/releasenotes/nova/newton.html16:50
*** Apoorva has joined #openstack-nova16:51
dansmithmriedem: I wonder if we're getting to the point where a "nova-manage ready_to_upgrade?" command would be warranted16:51
dansmithwe have kindof a lot of irons in the fire at the moment that we could report16:51
*** Apoorva has quit IRC16:51
mriedemyeah maybe16:52
cdentdansmith: I bet people would like that16:52
mriedemno cell mappings, you're not ready,16:52
mriedemno placement service, you're not ready16:52
mdboothdanpb: Quick question about Host.get_connection()16:53
*** belmoreira has quit IRC16:53
*** ljxiash has quit IRC16:53
*** artom has joined #openstack-nova16:54
*** bvanhav has joined #openstack-nova16:54
mdboothIf I want to have the mount manager go up and down automatically based on the connection callback, which seems like a good idea, I need _get_connection to be reentrant16:54
mdboothwhich it currently isn't16:54
openstackgerritDrew Thorstensen (thorst) proposed openstack/nova-specs: PowerVM Initial Driver Work  https://review.openstack.org/38177216:55
*** Apoorva has joined #openstack-nova16:55
mdboothBecause _get_connection unconditionally takes _wrapped_conn_lock, so it deadlocks when it reenters16:55
mdboothNaively I could set _wrapped_conn before calling the callback, and not take the lock if it's already set16:56
mriedemjroll: jaypipes: can you help clarify L16 in here https://etherpad.openstack.org/p/ocata-nova-summit-resource-providers-quantitative - at least wrt upgrades to ocata? i see ironic has the node.resource_class field in the ironic 1.21 microversion. so when upgrading to ocata, do i need to have a flavor extra_spec with key 'node_class' that maps a flavor to a specific ironic node.resource_class value?16:57
mdboothHowever, that would mean exposing the connection before we've registered things against it16:57
mdboothSo that doesn't sound like a good idea16:57
mriedemjroll: jaypipes: or just, the ironic driver in nova when reporting resources will provide the node.resource_class and the resource tracker will use that to create a resource provider of that class?16:57
*** krtaylor has joined #openstack-nova16:58
danpbmdbooth: how about just spawning a greenthread from the callback to populate the mount manager16:59
danpbi don't think you need to populte mount manager synchronously from the callback17:00
*** tlian has quit IRC17:00
mdboothdanpb: Gimme a minute to consider that17:00
danpband indeed it is probably a bad idea to do it synchronously17:00
danpbas if populating mount manager talks to libvirt, and libvirt drops in the middle17:00
danpbthen you could re-enter mount manager setup code17:00
mdboothMountManager locks that17:01
jrollmriedem: the latter, afaik, but the notes appear to say the former (I wasn't in that session)17:01
mdbooth... which would also deadlock17:01
mdboothfun17:01
jrollmriedem: I think the extra_spec thing was to support the weird "flavor matches multiple node classes" use case17:02
*** pilgrimstack has quit IRC17:02
*** mgoddard has joined #openstack-nova17:02
*** mgoddard_ has quit IRC17:02
mriedemok i'm not privy to that weirdness17:03
*** pilgrimstack has joined #openstack-nova17:03
mriedemand https://review.openstack.org/#/c/312696/ doesn't mention it17:03
*** oomichi_ has joined #openstack-nova17:04
*** sree has quit IRC17:06
*** andreas_s has quit IRC17:06
*** tlian has joined #openstack-nova17:07
jaypipessahid: k, review of the rt patch done.17:09
jaypipessahid: if you can fix up those comments and doc issues, I can re-review fairly quickly and push it.17:10
*** sree has joined #openstack-nova17:10
jaypipesmriedem: yes, the resource tracker will be creating inventory/allocation records for the ironic nodes (inventory records will use the node.node_class as the custom resource class). we still need to figure out the "what to do with the flavor extraspec thing".17:11
openstackgerritChris Dent proposed openstack/nova: Add rudimentary CORS support to placement API  https://review.openstack.org/39289117:12
*** tbachman has quit IRC17:12
*** ftersin has quit IRC17:14
*** sree has quit IRC17:15
openstackgerritDaniel Berrange proposed openstack/os-vif: host_info: fix get_common_version method on HostVIFInfo  https://review.openstack.org/39289217:15
jrolljaypipes: can we just not do a flavor extra spec thing17:15
*** ssurana has joined #openstack-nova17:17
*** rrecio_ has quit IRC17:19
*** ihrachys has quit IRC17:20
*** tblakes has joined #openstack-nova17:20
*** harlowja has quit IRC17:21
mriedemedleafe: you and jaypipes got this all figured out right? http://lists.openstack.org/pipermail/openstack-dev/2016-October/105965.html17:22
*** sree has joined #openstack-nova17:24
openstackgerritMatt Riedemann proposed openstack/nova-specs: Resource providers: custom resource classes  https://review.openstack.org/31269617:27
openstackgerritJulian proposed openstack/nova: Remove redundant VersionedObject Fields  https://review.openstack.org/38984017:29
cdentjaypipes: on an Inventory object of the total-reserved * allocation ratio is called "capacity" what's the overarching name of the thing described by min_unit, max_unit, step_size?17:31
cdentInventory constraints?17:31
*** lucasagomes is now known as lucas-afk17:32
*** tlian has quit IRC17:33
*** bvanhav has quit IRC17:34
*** bvanhav has joined #openstack-nova17:35
*** sree has quit IRC17:37
*** paw has quit IRC17:38
*** baoli has quit IRC17:38
*** fragatina has quit IRC17:40
*** tlian has joined #openstack-nova17:40
*** tblakes has quit IRC17:41
*** tbachman has joined #openstack-nova17:43
*** harlowja has joined #openstack-nova17:44
*** tlian has quit IRC17:50
*** sree has joined #openstack-nova17:51
*** rmcall has joined #openstack-nova17:53
*** marst has joined #openstack-nova17:54
cfriesen__jaypipes: hopefully this has come up already, but for CPU/RAM (and probably disk) we don't want to allow an instance to overcommit against itself.  we do want it to be able to overcommit against other instances though.17:54
*** anush has joined #openstack-nova17:55
*** derekh has quit IRC17:55
openstackgerritFeodor Tersin proposed openstack/nova: Fix cold migration with qcow2 ephemeral disks  https://review.openstack.org/34626317:58
*** dikonoor has quit IRC17:58
*** sree has quit IRC18:00
*** tlian has joined #openstack-nova18:01
*** ralonsoh has quit IRC18:01
*** Jeffrey4l has quit IRC18:02
*** haplo37_ has quit IRC18:05
jaypipesmriedem: yes, we sorted out the nested resource providers modeling.18:06
jaypipescfriesen__: I'm not following you.. could you elaborate please?18:06
jaypipescdent: sure, that sounds like a good term.18:07
jaypipesjroll: I'm still thinking on it :)18:07
jaypipesjroll: working on the resource tracker integrations for custom resource classes right now. should have a much better idea of end game after that's done.18:07
*** lpetrut has joined #openstack-nova18:08
*** haplo37_ has joined #openstack-nova18:08
openstackgerritJay Pipes proposed openstack/nova: placement: implement ResourceClass.create()  https://review.openstack.org/38684418:08
openstackgerritJay Pipes proposed openstack/nova: placement: raise KeyError unknown resource class  https://review.openstack.org/39006218:08
openstackgerritJay Pipes proposed openstack/nova: placement: REST API for resource classes  https://review.openstack.org/39195918:08
openstackgerritJay Pipes proposed openstack/nova: placement: adds ResourceClass.get_by_name()  https://review.openstack.org/39191718:08
openstackgerritJay Pipes proposed openstack/nova: placement: implement destroy() resource class  https://review.openstack.org/39191818:09
openstackgerritJay Pipes proposed openstack/nova: placement: add ResourceClass and ResourceClassList  https://review.openstack.org/38684318:09
jaypipesmriedem: ^^ whack-a-mole fixed.18:09
mriedemthat poor mole18:09
jaypipesindeed.18:09
jaypipescdent: in response to your comment about the "why microversion the aggregates stuff if either way it will return a 404", I fall back to the argument I made yesterday that the microversion history is a way to document changes to the API so just go ahead and make a microversion for it.18:11
cfriesen__jaypipes: what cdent above said reminded me of it, so I thought I'd make sure it was still possible with the new resource providers.  Basically there needs to be enough _total_ resources for the instance to fit (without factoring in overcommit).18:12
cfriesen__jaypipes: then there needs to be enough free space (factoring in overcommit) for the instance to fit.18:12
jaypipescfriesen__: ah, yes, absolutely.18:12
jaypipescfriesen__: sorry for some reason I thought you were referring to the conversation about realtime pinning I had with sahid and danpb earlier :)18:13
jaypipeslbeliveau: ok, finally now able to get to your review!18:14
lbeliveaujaypipes: thanks a bunch !18:14
*** anush has quit IRC18:14
openstackgerritsahid proposed openstack/nova: libvirt: fix incorrect host cpus giving to emulator threads when RT  https://review.openstack.org/35638318:14
jaypipeslbeliveau: sorry for the delay!18:14
lbeliveaujaypipes: no worries, know you are busy18:15
openstackgerritGábor Antal proposed openstack/nova: Transform instance.rebuild notification  https://review.openstack.org/38492218:15
*** jvgrant_ has joined #openstack-nova18:15
sahidjaypipes: thanks for your review, i addressed some of them and reply to your question about cpu_online18:15
jaypipeslbeliveau: btw, do you know about using /g in the mysql client?18:15
sahidwill do the refactor tomorrow18:15
*** paw has joined #openstack-nova18:15
cdentjaypipes: roger on the microversionoger18:15
mnestratovhi all18:15
lbeliveaujaypipes: haven't use it, let me check it out18:16
mnestratovtrivial https://review.openstack.org/#/c/388643/ needs one more +218:16
jaypipeslbeliveau: if you use /g at the end of your SQL statement in the mysql client, it displays in columnar format instead of tabular format, making copy/paste into launchpad bugs much nicer :)18:16
jaypipeslbeliveau: /g instead of ;18:16
openstackgerritLee Yarwood proposed openstack/nova: encryptors: Replace mangled passphrases for LuksEncryptor volumes  https://review.openstack.org/38667018:16
lbeliveaujaypipes: haaa good to know, thanks !!!18:16
jaypipesnp :)18:16
*** openstackgerrit has quit IRC18:18
*** sahid has quit IRC18:18
*** openstackgerrit has joined #openstack-nova18:18
*** jvgrant has quit IRC18:18
lbeliveaujaypipes: btw I would be glad to work on resource provider if you are looking for  others to contribute, can do PCI and numa or anything (even working on unit tests would be good)18:20
cdentjaypipes, dansmith: I'm working on https://bugs.launchpad.net/nova/+bug/1623545 but just realized: https://github.com/openstack/nova/blob/74fadb4bcf272ab93a7bf8270f087adf588735d9/nova/scheduler/client/report.py#L7318:22
openstackLaunchpad bug 1623545 in OpenStack Compute (nova) "placement API allocation handling does not check min_unit, max_unit, step_size" [Medium,Confirmed] - Assigned to Ukesh (ukeshkumar)18:22
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755718:23
openstackgerritDan Smith proposed openstack/nova: WIP Add schedule_and_build_instances RPC method  https://review.openstack.org/37932018:23
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937918:23
openstackgerritDan Smith proposed openstack/nova: Add CellDatabase fixture  https://review.openstack.org/39292518:23
*** GB21 has quit IRC18:24
*** andymaier has quit IRC18:24
*** edand has joined #openstack-nova18:25
*** anush has joined #openstack-nova18:26
openstackgerritGábor Antal proposed openstack/nova: Transform instance.rebuild notification  https://review.openstack.org/38492218:27
snetidanpb, can you please check this: https://review.openstack.org/#/c/338553/. I've added response to your comment18:29
jaypipeslbeliveau: https://review.openstack.org/#/c/363884 reviewed. push up those two little fixes and I'm good +2ing.18:31
*** _ducttape_ has joined #openstack-nova18:31
lbeliveaujaypipes: thanks, looking at it ...18:32
*** rrecio has joined #openstack-nova18:34
*** edand has quit IRC18:34
*** ducttape_ has quit IRC18:35
*** jed56 has quit IRC18:37
jaypipescdent: yeah, we're hard-coding the min-unit/max-unit stuff at the moment. Idea long-term is to have that stuff configurable all via the placement API of course.18:40
cdentjaypipes: sure, but the point is that if I fix the bug, then the resource tracker will start failing18:40
cdentI have code ready to push which makes it obvious, so if you give me a sec to write the commit message it'll be clear18:40
*** sree has joined #openstack-nova18:41
jaypipescdent: yeah, max_unit for VCPU should be the total number of cores on the machine. for memory, should be the total amount of memory on the machine.18:42
*** marst has quit IRC18:42
jaypipescdent: for disk, should be the maximum volume size.18:42
jaypipess/volume/disk/18:42
jaypipescdent: for the record, my original patch that added inventory records in the resource tracker did just that: https://review.openstack.org/#/c/363583/1/nova/scheduler/client/report.py18:44
cdent;)18:44
jaypipescdent: I missed that when reviewing the patch that actually landed.18:44
*** oomichi_ has quit IRC18:44
*** baoli has joined #openstack-nova18:45
*** sree has quit IRC18:46
*** ociuhandu has quit IRC18:46
*** gszasz has quit IRC18:48
*** matrohon has joined #openstack-nova18:50
*** matrohon has quit IRC18:50
* cdent glares at gerrit18:50
*** markus_z has quit IRC18:51
*** portdirect is now known as portdirect_away18:51
*** sudipto has quit IRC18:52
*** sudipto_ has quit IRC18:52
openstackgerritChris Dent proposed openstack/nova: DNM [placement] Enforce min_unit, max_unit and step_size  https://review.openstack.org/39293318:52
cdentjaypipes ^^ . Is it worth making a bug against the resource tracker?18:53
*** _ducttape_ has quit IRC18:58
*** sree has joined #openstack-nova18:59
jaypipescdent: sure, go for it.18:59
*** ducttape_ has joined #openstack-nova18:59
*** sree_ has joined #openstack-nova19:01
*** sree_ is now known as Guest8178219:02
cdentjaypipes: https://bugs.launchpad.net/nova/+bug/163868119:03
openstackLaunchpad bug 1638681 in OpenStack Compute (nova) "resource tracker sets wrong max_unit in placement Inventory" [Undecided,Triaged]19:03
cdentthat's the end of my day, I'll check back in on that, and the enforcement code tomorrow19:03
* cdent waves19:03
*** cdent has quit IRC19:03
*** sree has quit IRC19:03
*** lamt has joined #openstack-nova19:04
*** ducttape_ has quit IRC19:04
*** andymaier has joined #openstack-nova19:05
*** andymaier has quit IRC19:06
*** Guest81782 has quit IRC19:06
*** akshai has quit IRC19:08
*** Daisy has joined #openstack-nova19:09
openstackgerritGábor Antal proposed openstack/nova: Transform instance.create notification  https://review.openstack.org/38462119:09
*** slaweq_ has joined #openstack-nova19:10
*** anush has quit IRC19:11
*** knikolla has left #openstack-nova19:12
*** moshele has joined #openstack-nova19:13
*** knikolla has joined #openstack-nova19:14
*** Daisy has quit IRC19:14
*** ducttape_ has joined #openstack-nova19:14
dansmithbauzas: still around?19:19
*** jerrygb has quit IRC19:21
*** jerrygb has joined #openstack-nova19:22
openstackgerritLudovic Beliveau proposed openstack/nova: PCI: Fix PCI with fully qualified address  https://review.openstack.org/36388419:22
lbeliveaujaypipes: ^19:23
*** marst has joined #openstack-nova19:23
*** jerrygb has quit IRC19:26
*** lamt has quit IRC19:27
*** annegentle has quit IRC19:27
*** mmedvede has quit IRC19:28
*** tblakes has joined #openstack-nova19:29
*** moshele has quit IRC19:30
*** ayoung has joined #openstack-nova19:34
*** tblakes has quit IRC19:35
*** oanson has joined #openstack-nova19:36
*** liverpooler has quit IRC19:37
jaypipeslbeliveau: done.19:38
lbeliveauthanks19:38
lbeliveaujaypipes: ping me if you need cheap labor on resource provider ;)19:38
*** READ10 has quit IRC19:40
*** bvanhav has quit IRC19:42
*** jerrygb has joined #openstack-nova19:42
jaypipeslbeliveau: :)19:43
jaypipesvladikr: ah, doh, yeah, totally didn't think of that. thanks!19:44
*** anush has joined #openstack-nova19:44
*** rcernin has joined #openstack-nova19:45
*** sfinucan has left #openstack-nova19:45
*** sfinucan has joined #openstack-nova19:45
*** sridharg has joined #openstack-nova19:46
*** dave-mcc_ has joined #openstack-nova19:47
*** dave-mccowan has quit IRC19:48
*** rtheis_ has quit IRC19:49
openstackgerritJay Pipes proposed openstack/nova: placement: REST API for resource classes  https://review.openstack.org/39195919:49
openstackgerritJay Pipes proposed openstack/nova: placement: adds ResourceClass.get_by_name()  https://review.openstack.org/39191719:49
openstackgerritJay Pipes proposed openstack/nova: placement: implement destroy() resource class  https://review.openstack.org/39191819:49
*** dtp has joined #openstack-nova19:50
*** danpb has quit IRC19:50
*** matrohon has joined #openstack-nova19:54
*** openstack has joined #openstack-nova19:59
dansmithbauzas: so, I'm massively confused by what was going on in this patch around schedule_instances and the request spec stuff: https://review.openstack.org/#/c/367557/1120:00
dansmithbauzas: I've mostly backed it all out with a little hacky bit to make it work20:00
dansmithbauzas: running pep8 now, but maybe you could take a look when I push it up?20:01
bauzasdansmith: lemme look20:01
dansmithbauzas: incoming20:03
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755720:03
openstackgerritDan Smith proposed openstack/nova: WIP Add schedule_and_build_instances RPC method  https://review.openstack.org/37932020:03
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937920:03
*** mmedvede has joined #openstack-nova20:04
bauzasdansmith: fair enough20:04
bauzasdansmith: which point in particular is confusing you?20:04
dansmithbauzas: this: https://review.openstack.org/#/c/367557/9/nova/scheduler/utils.py20:05
dansmithand,20:05
dansmithhttps://review.openstack.org/#/c/367557/9/nova/conductor/manager.py@78920:05
*** anushkrishnamurt has joined #openstack-nova20:05
bauzassec, otp :(20:05
*** anush has quit IRC20:05
dansmithso if you can look at that, and then check out by "XXX is this right?" section here: https://review.openstack.org/#/c/367557/12/nova/conductor/manager.py@85520:06
dansmiththat would be helpful20:06
dansmiths/by/my/20:06
openstackgerritLee Yarwood proposed openstack/nova: encryptors: Workaround mangled passphrases  https://review.openstack.org/38667020:07
*** anushkrishnamurt has quit IRC20:09
*** Zara has quit IRC20:11
*** knikolla has left #openstack-nova20:11
*** knikolla has joined #openstack-nova20:12
*** devananda has quit IRC20:12
*** mriedem has quit IRC20:13
*** openstackgerrit has quit IRC20:18
*** openstackgerrit has joined #openstack-nova20:18
*** ihrachys has joined #openstack-nova20:19
*** slaweq_ has quit IRC20:19
bauzasdansmith: back, sorry loooong family call :p20:20
* dansmith is impressed with the number of calls bauzas gets20:20
bauzasoh man, really ? :)20:21
cburgessYeah for some reason I would think dansmith would hate being on the phone much less for a long period of time.20:23
*** akshai has joined #openstack-nova20:24
bauzasdansmith: so, about https://review.openstack.org/#/c/367557/12/nova/conductor/manager.py@855 I'm curious why we're not rather passing the RequestSpec object directly in the new schedule_and_run_instances() method20:24
dansmithcburgess: heh20:24
dansmithbauzas: we are?20:24
bauzasdansmith: the main problem I had when writing that series is that the scheduler utils methods are still using the legacy dicts20:24
*** marst has quit IRC20:25
bauzasdansmith: so, I wrote the to_legacy() helpers for passing the dicts20:25
*** marst has joined #openstack-nova20:25
dansmithbauzas: right, there are a bunch of TODOs everywhere, which probably need addressing20:25
dansmithbauzas: but that's my point -- that's why I'm passing the legacy-ified version20:25
bauzasdansmith: yeah, that's something I needed to cleanup, and something edleafe wanted to help20:25
bauzasdansmith: about your point, you want to set the instance uuid, right?20:26
bauzasdansmith: if so, why ?20:26
*** akshai_ has joined #openstack-nova20:26
bauzasdansmith: because the instance uuid should already be set by the API, nope ?20:26
dansmithbauzas: I need to else the server groups setup thing fails20:26
cfriesen__possibly a weird question...but does anyone know of a heat template for setting up a multi-node devstack environment?20:27
dansmithbauzas: this is the patch to move instance creation out of the api.. I don't even understand what it means for a reqspec to have a uuid, and num_instances=>120:27
bauzasdansmith: mmm, I'm pretty sure that the API would provide this when calling this RPC method20:27
dansmithbauzas: but stuff below _schedule_instances() seems to need it there20:27
dansmithbauzas: where? I don't see instance uuid in the ReqSpec object20:28
bauzasdansmith: I mean, the request_spec dict passed as a param would have an uuid20:28
bauzasdansmith: in the instance props20:28
dansmithoh, heh, there it is20:28
dansmithlooked a minute ago and missed it20:28
dansmithbauzas: well, I dont' see instance_props either20:28
bauzasdansmith: sec20:29
*** smatzek has quit IRC20:29
*** akshai has quit IRC20:29
bauzasdansmith: see the comment here https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L28220:31
*** devananda has joined #openstack-nova20:31
bauzasdansmith: we're not persisting all the instance object in the ReqSpec20:31
dansmith...right20:31
*** kiran-r has joined #openstack-nova20:31
bauzasdansmith: so, when we're returning a legacy dict, it only gets what the ReqSpec object knows20:31
*** sridharg has quit IRC20:31
openstackgerritKam Nasim proposed openstack/nova: Fixes exception during a swap to no swap resize  https://review.openstack.org/30061620:32
dansmithbauzas: so this seems to work for me: http://pastebin.com/67bhMfF220:32
dansmithdoes that look right?20:32
bauzasdansmith: we could have been calling the instance object to get all the fields that the ReqSpec doesn't know, but that's not necessary given the goal of that ReqSpec object is just to persist what is needed by the scheduler20:32
*** mriedem has joined #openstack-nova20:32
dansmithbauzas: just using the uuid from the buildrequest in the test so it matches20:32
bauzasdansmith: yup, zactly20:33
dansmithokay20:33
bauzasdansmith: you need somehow to pass a right ReqSpec20:33
bauzasfor the UTs20:33
*** eharney has quit IRC20:33
dansmithI didn't see instance_uuid in there before so I didn't think that was a thing I could do20:34
bauzasdansmith: so, when you're writing tests, just create a Spec object that provides fields what are looked up20:34
dansmithbut I just missed it20:34
bauzasdansmith: okay20:34
*** annegentle has joined #openstack-nova20:34
bauzasdansmith: tbc, that RPC method called schedule_and_build_instances() would get called with request_spec and filter_properties that are in the API20:35
dansmithyeah20:35
bauzasdansmith: I just think we can kill in fire the filter_props parameter in that method20:35
*** moshele has joined #openstack-nova20:36
bauzasdansmith: given that if the request_spec parameter is supporting a RequestSpec object, those filter_props dict keys could be found in that object20:36
openstackgerritDan Smith proposed openstack/nova: Return build_requests instead of instances  https://review.openstack.org/37426720:36
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755720:36
openstackgerritDan Smith proposed openstack/nova: Add CellDatabase fixture  https://review.openstack.org/39292520:36
openstackgerritDan Smith proposed openstack/nova: WIP Add schedule_and_build_instances RPC method  https://review.openstack.org/37932020:36
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937920:36
dansmithokay well, comment away ^ :)20:36
bauzasdansmith: sec, just thinking of something20:37
bauzasdansmith: are we planning to reschedule by calling that method ?20:37
bauzasdansmith: or would we be calling the existing conductor method ?20:37
dansmiththe existing one, IIRC20:37
dansmiththe spec spec-ifies that IIRC20:37
bauzasyou mean ? https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L182620:39
bauzasdansmith: again, I need to cleanup that, my strategy being to pass down a RequestSpec object down to the compute manager, so that it can reschedule by passing that object back to the conductor20:40
dansmithyeah, it would be good to be able to avoid polluting this new call with history20:40
bauzasdansmith: but I'd appreciate a clean conductor API that would certainly accept only a RequestSpec object instead of the legacy tuple (request_spec,  filter_props)20:41
bauzasdansmith: I said that, because you could get old history passed bottom/up to the conductor from the compute in case it's a reschedule20:41
*** tblakes has joined #openstack-nova20:41
dansmithhttps://review.openstack.org/#/c/381275/4/specs/ocata/approved/cells-scheduling-interaction.rst@7320:42
*** moshele has quit IRC20:42
bauzasdansmith: if we're clear with the path that this new "api_db conductor" method will just be top/down and then calling the regular build_instances conductor method that can be called in a reschedule, I'm fine20:42
dansmithI think the spec is clear there20:42
*** matrohon has quit IRC20:43
bauzasdansmith: yeah, I agree, the spec is clear, I just wanted to stress that point at the implementation phase :)20:43
bauzashaving two conductor processes could be confusing, but I'm not20:43
*** matrohon has joined #openstack-nova20:43
*** marst_ has joined #openstack-nova20:44
*** oanson has quit IRC20:46
*** marst has quit IRC20:47
*** eharney has joined #openstack-nova20:48
*** edmondsw has quit IRC20:50
*** matrohon has quit IRC20:51
*** matrohon has joined #openstack-nova20:51
*** matrohon has quit IRC20:52
*** matrohon has joined #openstack-nova20:52
dtpis there a cells v2 meeting this week?20:52
*** matrohon has quit IRC20:53
*** matrohon has joined #openstack-nova20:53
eanderssonbtw cells! anyone with insight into cells that could take a look at https://review.openstack.org/#/c/390984/ ?20:53
bauzasdtp: nope, see http://lists.openstack.org/pipermail/openstack-dev/2016-November/106578.html20:55
dtpthanks20:55
*** lpetrut has quit IRC20:55
*** krtaylor has quit IRC20:55
*** Sukhdev has joined #openstack-nova20:57
*** akshai_ has quit IRC20:57
*** matrohon has quit IRC21:01
*** slaweq_ has joined #openstack-nova21:01
*** matrohon has joined #openstack-nova21:02
dansmithdtp: no, we canceled it (on the ML)21:03
*** akshai has joined #openstack-nova21:03
dansmithheh, yeah, late to that convo21:03
*** esberglu has quit IRC21:03
dtp:) i've joined the ML now21:04
*** esberglu has joined #openstack-nova21:04
*** sach-m has joined #openstack-nova21:05
*** sree has joined #openstack-nova21:07
*** esberglu has quit IRC21:08
*** sach-m has quit IRC21:08
*** oomichi_online has joined #openstack-nova21:09
*** sach-m has joined #openstack-nova21:09
*** sree has quit IRC21:12
*** xyang1 has quit IRC21:12
*** sach-m_ has joined #openstack-nova21:13
*** sach-m has quit IRC21:13
mriedemyou know what's terrible? 3 5 year old girls all yelling at alexa to play them songs.21:15
* mriedem head explodes21:15
*** tblakes has quit IRC21:18
*** sach-m_ has quit IRC21:19
*** sach-m has joined #openstack-nova21:20
*** lpetrut has joined #openstack-nova21:20
sach-mHi Everyone, I am sure you all watched the hybrid cloud demo @Barcelona keynote. To take the effort further, we are proposing a nova driver for AWS. Blueprints are here: https://blueprints.launchpad.net/nova/+spec/nova-aws. Thanks!21:20
*** sambetts is now known as sambetts|afk21:23
tbachmanmriedem: fight back. Blare ESPN on all possible displays.21:23
JayFmriedem: sounds like the only worst thing would be only one of them doing it, and you having to listen to some frozen song for the 900th time21:24
*** dave-mcc_ has quit IRC21:24
mriedemffs, now there is fighting and crying over which song is picked21:24
*** lpetrut has quit IRC21:24
*** nherciu_ has quit IRC21:25
mriedemi do have the ability to play some slayer and drown them out...21:25
tbachmanreintroduce them to Headbanger’s Ball21:26
tbachman(er, “introduce”)21:27
jlkmriedem: welcome to our glorious future.21:30
*** sach-m has quit IRC21:30
*** sach-m has joined #openstack-nova21:30
*** thorst has quit IRC21:31
*** maticue has quit IRC21:31
openstackgerritMatt Riedemann proposed openstack/nova: Transform instance.shutdown notifications  https://review.openstack.org/32921421:32
*** esberglu has joined #openstack-nova21:34
*** sach-m has quit IRC21:34
bauzasheh, I was just about replying to sach-m21:34
bauzashttps://twitter.com/sylvainbauza/status/79119731205173248021:35
*** sach-m has joined #openstack-nova21:36
*** esberglu has quit IRC21:38
*** fandi has joined #openstack-nova21:39
*** zzelle has joined #openstack-nova21:40
*** browne has quit IRC21:41
mriedembauzas: that omni thing is using the openstack api, it's just using a virt driver that's talking to aws21:42
mriedemit's the same idea as jacket21:42
mriedemexcept with code21:42
mriedemsame as fusion compute21:42
mriedemsame as powervc21:42
bauzasmriedem: the real problem I see with that is that's a cloud-ish driver21:43
oomichi_onlinemriedem: can you help https://review.openstack.org/#/c/387644/ ?   tempest patch (https://review.openstack.org/#/c/381362/) requires for the merger21:44
bauzasmriedem: for example, how can we know which AWS resources do we still have unless by saying quotas ?21:44
*** akshai has quit IRC21:46
*** sach-m has quit IRC21:46
*** jheroux has quit IRC21:46
*** sach-m has joined #openstack-nova21:47
*** akshai has joined #openstack-nova21:47
*** maticue has joined #openstack-nova21:48
oomichi_onlinebauzas: will nova become a proxy of aws??21:48
*** sach-m has quit IRC21:49
bauzasoomichi: if you mean that OpenStack will be the controlplane and AWS the dataplane, then yes if using that AWS drivre21:49
*** sach-m has joined #openstack-nova21:49
*** sach-m has quit IRC21:49
*** sach-m has joined #openstack-nova21:49
oomichi_onlinebauzas: oh, interesting idea21:50
bauzasoomichi_online: well, sure, but I'm not sure a nova driver should be fine for that21:50
openstackgerritErik Olof Gunnar Andersson proposed openstack/nova: Re-worked ram filter and weights to support overcommit  https://review.openstack.org/39098421:50
oomichi_onlinebauzas: heh, yeah. I have the same question for that21:51
bauzasoomichi_online: the abstraction means that the operator knows the resources their have21:51
bauzasoomichi_online: while here, this is not21:51
*** sach-m has quit IRC21:51
*** sach-m has joined #openstack-nova21:52
bauzashere, that would mean that there would only be one compute node with all the resources being the AWS quotas, I guess21:52
oomichi_onlinebauzas: not sure that "nova should be abstraction layer for all resources including AWS"21:52
oomichi_onlineinteresting anyways :)21:53
mriedemoomichi_online: done21:53
bauzasoomichi_online: if you see their repos, it's not only nova21:53
oomichi_onlinemriedem: thanks :)21:53
*** eharney has quit IRC21:53
bauzasoomichi_online: nova is just proxying calls to EC221:53
bauzasthe other AWS resources being drived by Cinder or Neutron21:53
bauzasdriven*21:54
mriedemsure i wouldn't actually use it21:54
mriedembut it's not a new concept21:54
oomichi_onlinebauzas: oh, they has some implementation already? cool21:55
bauzasmriedem: I agree, I just wonder if that concept is something we still love :)21:55
mriedemi don't love it21:56
mriedempersonally21:56
mriedemsome customer might, until they run into resource sync issues21:56
mriedema service provider would probably hate dealing with it21:56
bauzasmriedem: that's my point21:56
mriedembut red hat doesn't have to support it so meh :)21:57
bauzasshhhht21:57
*** smatzek has joined #openstack-nova21:57
bauzassome might have ideas21:57
*** tblakes has joined #openstack-nova21:58
sach-mmriedem: which resource sync issues?21:59
mriedemsach-m: you do something with your resources running in AWS via ec2 directly and things are out of sync in the nova db21:59
mriedemnova doesn't deal with out of band changes very well21:59
sach-mmriedem: We already do that for VMware and KVM in our deployments through some extensions added for openstack. We can do the same for hybrid work as well22:00
mriedemsure you can, and some do22:01
mriedemit's not fun22:01
mriedempowervc does it also22:01
*** diana_clarke has quit IRC22:01
sach-mmriedem: Isn't this a generic problem with any driver though?22:02
*** harlowja has quit IRC22:02
cfriesen__is it expected that for stable/mitaka when doing a "nova evacuate" we lose files specified with --file at boot?22:02
bauzassach-m: that's not really a problem, that's rather than nova isn't just an hypervisor layer22:04
bauzassach-m: meaning that nova is there for consuming all the resources, not just proxying calls to libvirt22:04
bauzas(or any other driver)22:04
sach-mbauzas: It does that for VMware already22:05
bauzassach-m: where we have lots of problems22:05
bauzassach-m: because for example, any operation that would be made out of Nova is hard to reconcile22:06
cfriesen__sach-m: if you go mucking around with libvirt directly behind nova's back you could get things into a bad state too.22:06
*** mriedem is now known as mriedem_away22:06
bauzassach-m: in your proposal, what if the user does some resize operation in AWS directly?22:07
bauzasalso, what would mean a "live-migration" operation from a Nova standpoint if we don't know how many computes are running behind our AWS tenant?22:08
sach-mbauzas: I do agree in general with the assessment that resource sync is going to be an issue. But with AWS, there's a chance to reconcile state in a better way -- everything comes in standard t-shirt sizes (flavors).22:08
*** mdrabe has quit IRC22:09
sach-mbauzas: I've seen weird issues in KVM/VMware where the instance sizes have no counterpart in nova flavors22:09
sach-mbauzas: My understanding is that AWS and most public cloud providers do not support live migration feature today22:10
bauzasthat really depends on your policy22:10
bauzaseither way, I was thinking when I saw the keynote that there was an upper service that would do that proxy call, and not nova22:11
*** lpetrut has joined #openstack-nova22:11
bauzassach-m: tbc, I certainly understand the usecase22:12
bauzassach-m: and I think it's very valid to propose that22:12
bauzassach-m: I'm more concerned by the implementation that leaves a huge number of open questions we know we have22:12
sach-mbauzas: understood22:13
*** victorhe has quit IRC22:15
*** matrohon has quit IRC22:18
*** matrohon has joined #openstack-nova22:19
*** zzelle has quit IRC22:19
*** burt has quit IRC22:21
*** browne has joined #openstack-nova22:23
*** sach-m has quit IRC22:27
*** ducttape_ has quit IRC22:28
*** sach-m has joined #openstack-nova22:28
*** flaper87 has quit IRC22:29
*** smatzek has quit IRC22:30
jlkAt summit, didn't Nova reject a driver that sat in front of a cloud like thing, FusionSphere?22:30
jlkthe sentiment was that the VMWare vsphere driver was a mistake, one that shouldn't be repeated with fusionsphere, which would apply to AWS too22:30
*** rrecio has quit IRC22:31
*** matrohon has quit IRC22:32
sach-mjlk: What's the goal of nova though? Just be a virtualization layer to manage KVM?22:32
*** matrohon has joined #openstack-nova22:32
jlkthat's... a difficult question.22:33
jlkObviously not, given that there is a driver for Xen22:33
jlkgiven that KVM support is behind libvirt, I'm not know how much is KVM specific, or "anything libvirt supports".22:34
sach-mbecause many other platforms -- hyper-v, vmware, and so on are going to have their own management server.22:34
sach-mEssentially, we are saying -- use nova as virtual machine manager not a cloud API22:35
jlkthat's correct22:35
sach-mI don't agree with that. OpenStack cloud API brings in a lot more capabilities like self service, key management over virtualization22:36
*** marst_ has quit IRC22:36
jlkbut the nova part is simply VM management22:36
*** huanxie has quit IRC22:36
jlkwith cooperation from Neutron for networking, glance for images, keystone for authentication22:36
sach-mWhat about quotas, access to images, flavors, etc? There's a valuable delegation & self service that nova provides which a platform like VMware does not22:37
*** larainema has quit IRC22:37
jlkAre you sure there's not a higher price VMWare offering that provides that?22:39
sach-mthat's basically the reason people put openstack on top of VMware22:39
*** larainema has joined #openstack-nova22:39
jlkIIRC people put OpenStack on top of vmware to make use of vmware as the HV, and to save money on higher $$ VMware products22:39
sach-mVMware has its own cloud offering, yes. But it is not open.22:39
jlksure, so if you want those things, buy VMWare22:40
jlkdon't try to slap openstack on top of it22:40
*** pradk has quit IRC22:41
*** matrohon has quit IRC22:42
*** matrohon has joined #openstack-nova22:42
*** mlavalle has quit IRC22:44
*** ihrachys has quit IRC22:45
*** gouthamr has quit IRC22:46
*** kiran-r has quit IRC22:49
*** ducttape_ has joined #openstack-nova22:53
*** matrohon has quit IRC22:54
bauzasjaypipes: silly question, but should we accept "name" and "uuid" as resource class names ? :p22:57
*** tblakes has quit IRC22:58
*** ducttape_ has quit IRC22:58
*** vladikr has quit IRC23:01
*** sach-m has quit IRC23:03
*** david-lyle has quit IRC23:04
*** harlowja has joined #openstack-nova23:08
*** sach-m has joined #openstack-nova23:09
*** Guest87642 has quit IRC23:12
*** thorst has joined #openstack-nova23:13
*** owalsh has quit IRC23:14
*** owalsh has joined #openstack-nova23:15
*** ducttape_ has joined #openstack-nova23:16
*** thorst has quit IRC23:18
*** mtanino has quit IRC23:20
*** annegentle has quit IRC23:22
*** sach-m has quit IRC23:30
*** david-lyle has joined #openstack-nova23:30
*** sach-m has joined #openstack-nova23:31
*** dave-mccowan has joined #openstack-nova23:31
*** rcernin has quit IRC23:31
*** lpetrut has quit IRC23:32
*** sach-m has quit IRC23:37
*** sach-m has joined #openstack-nova23:38
*** ducttape_ has quit IRC23:38
*** sach-m has quit IRC23:43
*** sach-m has joined #openstack-nova23:43
*** ducttape_ has joined #openstack-nova23:50
*** esberglu has joined #openstack-nova23:53
*** gouthamr has joined #openstack-nova23:59

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