*** clenimar_ has quit IRC | 00:06 | |
*** zenoway has joined #openstack-nova | 00:12 | |
*** pacoblox has quit IRC | 00:15 | |
*** zenoway has quit IRC | 00:16 | |
*** amotoki has quit IRC | 00:17 | |
*** mrkz has joined #openstack-nova | 00:21 | |
*** mgoddard has joined #openstack-nova | 00:22 | |
*** hoangcx has joined #openstack-nova | 00:22 | |
*** jamesdenton has quit IRC | 00:24 | |
*** mgoddard has quit IRC | 00:27 | |
*** zul has joined #openstack-nova | 00:28 | |
*** ssurana has quit IRC | 00:29 | |
*** Sukhdev has quit IRC | 00:30 | |
*** jamesdenton has joined #openstack-nova | 00:30 | |
*** jamesdenton has quit IRC | 00:33 | |
*** jamesden_ has joined #openstack-nova | 00:33 | |
*** jamesdenton has joined #openstack-nova | 00:34 | |
*** jamesden_ has quit IRC | 00:38 | |
*** sacharya_ has quit IRC | 00:38 | |
*** thorst_ has joined #openstack-nova | 00:49 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests https://review.openstack.org/300262 | 00:50 |
---|---|---|
*** diogogmt has joined #openstack-nova | 00:52 | |
*** thorst_ has quit IRC | 00:54 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 unit tests https://review.openstack.org/278810 | 00:54 |
*** thorst_ has joined #openstack-nova | 00:55 | |
openstackgerrit | Hieu LE proposed openstack/nova: Config options: improve libvirt help text (1) https://review.openstack.org/316539 | 00:57 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 5) https://review.openstack.org/329979 | 00:58 |
*** sdake has joined #openstack-nova | 00:58 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2) https://review.openstack.org/329978 | 00:58 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 4) https://review.openstack.org/329227 | 00:58 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 3) https://review.openstack.org/328851 | 00:58 |
*** thorst_ has quit IRC | 00:59 | |
*** zenoway has joined #openstack-nova | 00:59 | |
openstackgerrit | Hieu LE proposed openstack/nova: Config options: improve libvirt help text (2) https://review.openstack.org/316550 | 00:59 |
*** Apoorva has quit IRC | 01:01 | |
*** jamesdenton has quit IRC | 01:03 | |
*** zenoway has quit IRC | 01:04 | |
*** jamesdenton has joined #openstack-nova | 01:04 | |
*** diogogmt has quit IRC | 01:06 | |
*** artom has quit IRC | 01:13 | |
*** cheneydc has joined #openstack-nova | 01:13 | |
*** sdake has quit IRC | 01:14 | |
*** sdake has joined #openstack-nova | 01:16 | |
*** artom has joined #openstack-nova | 01:18 | |
*** EinstCrazy has joined #openstack-nova | 01:21 | |
*** sdake_ has joined #openstack-nova | 01:22 | |
*** Jeffrey4l has quit IRC | 01:24 | |
*** sdake has quit IRC | 01:24 | |
*** ducncvn has quit IRC | 01:25 | |
*** baoli has joined #openstack-nova | 01:28 | |
*** jamesdenton has quit IRC | 01:29 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 01:29 |
*** Daisy has joined #openstack-nova | 01:33 | |
*** Apoorva has joined #openstack-nova | 01:33 | |
*** rama_y has quit IRC | 01:38 | |
*** Jeffrey4l has joined #openstack-nova | 01:38 | |
*** jamesdenton has joined #openstack-nova | 01:44 | |
*** mtanino has quit IRC | 01:45 | |
*** claudiub has quit IRC | 01:47 | |
*** Daisy has quit IRC | 01:47 | |
*** Daisy has joined #openstack-nova | 01:48 | |
*** hoangcx has quit IRC | 01:51 | |
*** hoangcx has joined #openstack-nova | 01:51 | |
openstackgerrit | Anh Tran proposed openstack/nova: Enables Py34 tests for unit.api.openstack.compute.test_server_actions https://review.openstack.org/329435 | 01:51 |
*** Daisy has quit IRC | 01:52 | |
*** Sukhdev has joined #openstack-nova | 01:57 | |
*** sacharya has joined #openstack-nova | 01:59 | |
*** Apoorva has quit IRC | 02:05 | |
Qiming | anyone help point me to a link of nova mid-cycle meetup, if any? | 02:06 |
Qiming | thanks | 02:06 |
*** Daisy has joined #openstack-nova | 02:06 | |
*** jamesdenton has quit IRC | 02:07 | |
tonyb | Qiming: https://wiki.openstack.org/wiki/Sprints#Future_sprints_for_Newton | 02:09 |
*** sacharya_ has joined #openstack-nova | 02:09 | |
Qiming | tonyb, thank you! | 02:09 |
tonyb | Qiming: np | 02:10 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Include exception in _try_deallocate_network error log https://review.openstack.org/330850 | 02:10 |
*** sacharya has quit IRC | 02:13 | |
*** sdake has joined #openstack-nova | 02:16 | |
*** sdake_ has quit IRC | 02:19 | |
*** pwnall has quit IRC | 02:19 | |
*** mtanino has joined #openstack-nova | 02:22 | |
*** houming has joined #openstack-nova | 02:24 | |
*** ivase has quit IRC | 02:27 | |
*** ivase has joined #openstack-nova | 02:28 | |
*** prateek has joined #openstack-nova | 02:32 | |
*** jogo has quit IRC | 02:32 | |
*** rrecio has joined #openstack-nova | 02:34 | |
*** thorst_ has joined #openstack-nova | 02:38 | |
*** thorst_ has quit IRC | 02:41 | |
*** thorst_ has joined #openstack-nova | 02:41 | |
*** thorst_ has quit IRC | 02:45 | |
*** jogo has joined #openstack-nova | 02:46 | |
*** tbachman has quit IRC | 02:46 | |
*** mrkz has quit IRC | 02:48 | |
*** mdrabe has joined #openstack-nova | 02:49 | |
*** efried has left #openstack-nova | 02:49 | |
*** tbachman has joined #openstack-nova | 02:51 | |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: DO NOT MERGE - attach configdrive after instance XML test https://review.openstack.org/330134 | 02:52 |
*** rrecio has quit IRC | 02:53 | |
*** adisky has joined #openstack-nova | 02:53 | |
*** rama_y has joined #openstack-nova | 02:53 | |
*** jamesdenton has joined #openstack-nova | 02:54 | |
*** d_code has quit IRC | 02:56 | |
*** jamesdenton has quit IRC | 02:57 | |
*** gjayavelu has joined #openstack-nova | 02:59 | |
*** d_code has joined #openstack-nova | 02:59 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support https://review.openstack.org/318953 | 03:02 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance https://review.openstack.org/316275 | 03:02 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add REST API support for get me a network https://review.openstack.org/316398 | 03:02 |
*** jamielennox is now known as jamielennox|away | 03:03 | |
*** gyee has quit IRC | 03:07 | |
*** amotoki has joined #openstack-nova | 03:07 | |
*** amotoki has quit IRC | 03:09 | |
*** jamielennox|away is now known as jamielennox | 03:19 | |
*** Kevinpang has joined #openstack-nova | 03:24 | |
*** baoli has quit IRC | 03:25 | |
*** jogo has quit IRC | 03:26 | |
*** d_code has quit IRC | 03:27 | |
*** d_code has joined #openstack-nova | 03:29 | |
*** d_code has quit IRC | 03:29 | |
openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: OVS agent updates the wrong port with Neutron https://review.openstack.org/242846 | 03:30 |
*** jogo has joined #openstack-nova | 03:30 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys https://review.openstack.org/329697 | 03:37 |
*** zerda2 has joined #openstack-nova | 03:47 | |
*** markvoelker has quit IRC | 03:47 | |
*** mdrabe has quit IRC | 03:48 | |
*** mriedem has quit IRC | 03:51 | |
*** Daisy has quit IRC | 03:55 | |
*** Daisy has joined #openstack-nova | 03:55 | |
*** Daisy has quit IRC | 03:58 | |
*** Daisy has joined #openstack-nova | 03:58 | |
*** amotoki has joined #openstack-nova | 04:00 | |
openstackgerrit | huan proposed openstack/nova: XenAPI: Support neutron security group https://review.openstack.org/251271 | 04:02 |
*** lpetrut has joined #openstack-nova | 04:03 | |
*** ivase has quit IRC | 04:07 | |
*** ivase has joined #openstack-nova | 04:08 | |
*** Daisy has quit IRC | 04:11 | |
*** Daisy has joined #openstack-nova | 04:11 | |
*** prateek has quit IRC | 04:13 | |
*** Daisy has quit IRC | 04:16 | |
*** amotoki has quit IRC | 04:18 | |
*** links has joined #openstack-nova | 04:20 | |
*** prateek has joined #openstack-nova | 04:24 | |
openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: OVS agent updates the wrong port with Neutron https://review.openstack.org/242846 | 04:24 |
*** yamahata has joined #openstack-nova | 04:27 | |
*** lpetrut has quit IRC | 04:28 | |
*** rha_ has joined #openstack-nova | 04:29 | |
*** ociuhandu has quit IRC | 04:29 | |
*** rha has quit IRC | 04:29 | |
*** cheneydc has quit IRC | 04:35 | |
*** ociuhandu has joined #openstack-nova | 04:42 | |
*** amotoki has joined #openstack-nova | 04:45 | |
*** Kevinpang has quit IRC | 04:45 | |
*** ijw_ has joined #openstack-nova | 04:46 | |
*** ijw has quit IRC | 04:47 | |
*** markvoelker has joined #openstack-nova | 04:47 | |
*** sdake_ has joined #openstack-nova | 04:50 | |
*** markvoelker has quit IRC | 04:52 | |
*** sdake has quit IRC | 04:53 | |
*** Daisy has joined #openstack-nova | 04:53 | |
*** ijw_ has quit IRC | 04:59 | |
*** ijw has joined #openstack-nova | 04:59 | |
*** Daisy has quit IRC | 05:02 | |
*** Daisy has joined #openstack-nova | 05:02 | |
*** ratailor has joined #openstack-nova | 05:05 | |
*** sacharya_ has quit IRC | 05:06 | |
*** Daisy has quit IRC | 05:07 | |
*** Daisy has joined #openstack-nova | 05:07 | |
*** salv-orl_ has joined #openstack-nova | 05:09 | |
*** Daisy has quit IRC | 05:10 | |
*** Daisy has joined #openstack-nova | 05:10 | |
*** salv-orlando has quit IRC | 05:12 | |
*** zenoway has joined #openstack-nova | 05:13 | |
*** lpetrut has joined #openstack-nova | 05:14 | |
*** ecelik has joined #openstack-nova | 05:15 | |
*** pc_m has quit IRC | 05:16 | |
*** zenoway has quit IRC | 05:18 | |
*** browne has joined #openstack-nova | 05:25 | |
*** links has quit IRC | 05:27 | |
*** sdake has joined #openstack-nova | 05:27 | |
*** links has joined #openstack-nova | 05:28 | |
*** chandankumar has joined #openstack-nova | 05:30 | |
*** sdake_ has quit IRC | 05:31 | |
*** Daisy has quit IRC | 05:32 | |
*** Daisy has joined #openstack-nova | 05:32 | |
*** cfarquhar has quit IRC | 05:35 | |
*** Daisy has quit IRC | 05:37 | |
*** sdake_ has joined #openstack-nova | 05:38 | |
*** sdake has quit IRC | 05:41 | |
*** Daisy has joined #openstack-nova | 05:42 | |
*** sdake_ has quit IRC | 05:50 | |
*** amotoki has quit IRC | 05:56 | |
*** amotoki has joined #openstack-nova | 06:02 | |
*** abalutoiu has joined #openstack-nova | 06:02 | |
*** Daisy has quit IRC | 06:04 | |
*** Daisy has joined #openstack-nova | 06:04 | |
*** belmoreira has joined #openstack-nova | 06:06 | |
*** browne has quit IRC | 06:07 | |
*** rama_y has quit IRC | 06:08 | |
*** sridharg has joined #openstack-nova | 06:08 | |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (7) https://review.openstack.org/330898 | 06:09 |
*** Daisy has quit IRC | 06:09 | |
*** nkrinner has joined #openstack-nova | 06:10 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 06:13 |
*** Daisy has joined #openstack-nova | 06:15 | |
*** pc_m has joined #openstack-nova | 06:15 | |
*** Sukhdev has quit IRC | 06:17 | |
openstackgerrit | Li Yingjun proposed openstack/nova: Implement hypervisor notifications https://review.openstack.org/315312 | 06:20 |
*** rcernin has joined #openstack-nova | 06:23 | |
*** gjayavelu has quit IRC | 06:23 | |
*** mtanino has quit IRC | 06:24 | |
*** david-lyle has quit IRC | 06:26 | |
*** andreas_s has joined #openstack-nova | 06:28 | |
*** wanghao has quit IRC | 06:28 | |
*** pcaruana has joined #openstack-nova | 06:29 | |
*** paul-carlton2 has joined #openstack-nova | 06:30 | |
*** lpetrut has quit IRC | 06:33 | |
*** zenoway has joined #openstack-nova | 06:36 | |
openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new configuration option to turn auto converge on/off https://review.openstack.org/266001 | 06:37 |
*** sahid has joined #openstack-nova | 06:37 | |
*** ssurana has joined #openstack-nova | 06:46 | |
*** abalutoiu has quit IRC | 06:47 | |
*** mpavone has joined #openstack-nova | 06:49 | |
*** sheel has joined #openstack-nova | 06:50 | |
*** Daisy_ has joined #openstack-nova | 06:50 | |
*** ssurana has quit IRC | 06:50 | |
*** Daisy has quit IRC | 06:52 | |
*** Daisy_ has quit IRC | 06:57 | |
*** Daisy has joined #openstack-nova | 06:58 | |
*** fragatina has quit IRC | 06:59 | |
gibi_ | xenogear: hi, I saw on the wiki that you indicated you will start delete_ip transformation. Yesterday I made a note on the wiki not to start new transformation until the current ones are close to merge to avoid reivew overload. | 07:01 |
*** gjayavelu has joined #openstack-nova | 07:01 | |
*** jichen has joined #openstack-nova | 07:01 | |
gibi_ | xenogear: with the current amount of open transformation reviews and I feel we will not be able to merge all of them in the next two weeks, before the freeze | 07:01 |
gibi_ | xenogear: so I just wanted to let you know that it is pretty possible that your work will not fit into the current cycle | 07:02 |
*** ihrachys has joined #openstack-nova | 07:03 | |
*** amotoki has quit IRC | 07:03 | |
*** paul-carlton2 has quit IRC | 07:05 | |
*** cfriesen_ has quit IRC | 07:06 | |
*** ivase has quit IRC | 07:07 | |
*** syjulian has quit IRC | 07:08 | |
*** ivase has joined #openstack-nova | 07:08 | |
*** ihrachys has quit IRC | 07:08 | |
*** salv-orl_ has quit IRC | 07:09 | |
*** salv-orlando has joined #openstack-nova | 07:09 | |
*** tesseract has joined #openstack-nova | 07:10 | |
*** romainh has joined #openstack-nova | 07:12 | |
*** hogepodge has quit IRC | 07:12 | |
*** bauzas is now known as bauwser | 07:13 | |
bauwser | p!spy | 07:13 |
*** bauwser is now known as bauzas | 07:14 | |
bauzas | p!spy | 07:14 |
*** matrohon has joined #openstack-nova | 07:15 | |
*** zenoway has quit IRC | 07:15 | |
*** jlanoux has joined #openstack-nova | 07:20 | |
*** abalutoiu has joined #openstack-nova | 07:20 | |
openstackgerrit | Andreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 1 before functional change https://review.openstack.org/323761 | 07:22 |
openstackgerrit | Andreas Scheuring proposed openstack/nova: libvirt: virtlogd: use "log" element in char devices https://review.openstack.org/323760 | 07:22 |
openstackgerrit | Andreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 3 before functional change https://review.openstack.org/323763 | 07:22 |
openstackgerrit | Andreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 2 before functional change https://review.openstack.org/323762 | 07:22 |
openstackgerrit | Andreas Scheuring proposed openstack/nova: libvirt: virtlogd: use the logd sub-element for char devices https://review.openstack.org/323765 | 07:22 |
openstackgerrit | Andreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 4 before functional change https://review.openstack.org/323764 | 07:22 |
openstackgerrit | Andreas Scheuring proposed openstack/nova: libvirt: virtlogd: introduce domain xml element "log" https://review.openstack.org/323759 | 07:22 |
*** bauzas is now known as bauwser | 07:24 | |
*** ssurana has joined #openstack-nova | 07:30 | |
openstackgerrit | liuxiuli proposed openstack/nova: And reserved_huge_pages to memory_mb_used https://review.openstack.org/330928 | 07:32 |
*** dmk0202 has joined #openstack-nova | 07:32 | |
*** cgalan has joined #openstack-nova | 07:32 | |
*** ijw has quit IRC | 07:33 | |
*** dmk0202 has quit IRC | 07:33 | |
*** ssurana has quit IRC | 07:35 | |
*** salv-orlando has quit IRC | 07:35 | |
*** gcb has joined #openstack-nova | 07:35 | |
*** claudiub has joined #openstack-nova | 07:37 | |
*** EinstCra_ has joined #openstack-nova | 07:38 | |
*** EinstCrazy has quit IRC | 07:40 | |
*** EinstCra_ has quit IRC | 07:43 | |
*** EinstCrazy has joined #openstack-nova | 07:43 | |
bauwser | good morning Nova | 07:43 |
*** Kevin-Zhao has joined #openstack-nova | 07:43 | |
*** zhangjn has quit IRC | 07:45 | |
*** rcernin has quit IRC | 07:45 | |
*** Daisy has quit IRC | 07:46 | |
*** Daisy has joined #openstack-nova | 07:46 | |
*** zhangjn has joined #openstack-nova | 07:50 | |
*** Daisy has quit IRC | 07:51 | |
*** Daisy has joined #openstack-nova | 07:51 | |
*** lpetrut has joined #openstack-nova | 07:57 | |
*** efoley has joined #openstack-nova | 07:58 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:00 | |
*** pacoblox has joined #openstack-nova | 08:00 | |
*** gjayavelu has quit IRC | 08:02 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Use is_valid_ipv4 from oslo.utils https://review.openstack.org/326922 | 08:05 |
*** diga has joined #openstack-nova | 08:05 | |
*** gjayavelu has joined #openstack-nova | 08:06 | |
*** yamahata has quit IRC | 08:07 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
*** rcernin has joined #openstack-nova | 08:10 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
openstackgerrit | LisaLi proposed openstack/nova: Attach/detach encryptors during swap_volume https://review.openstack.org/252809 | 08:15 |
*** pacoblox has quit IRC | 08:19 | |
*** tojuvone_ has joined #openstack-nova | 08:19 | |
*** mgoddard has joined #openstack-nova | 08:20 | |
*** huyang_ has joined #openstack-nova | 08:20 | |
openstackgerrit | YaoZheng proposed openstack/nova: live migration check source failed caused bdm.device_path lost https://review.openstack.org/330944 | 08:20 |
*** ameade has quit IRC | 08:21 | |
*** BobBall_AWOL is now known as BobBall | 08:21 | |
*** jlvillal has quit IRC | 08:22 | |
*** jlvillal has joined #openstack-nova | 08:22 | |
*** ameade has joined #openstack-nova | 08:22 | |
*** tyrola has joined #openstack-nova | 08:22 | |
*** huyang has quit IRC | 08:23 | |
*** tojuvone has quit IRC | 08:23 | |
*** huyang_ is now known as huyang | 08:23 | |
*** paul-carlton2 has joined #openstack-nova | 08:23 | |
*** tyrola has quit IRC | 08:23 | |
*** Daisy has quit IRC | 08:26 | |
*** Daisy has joined #openstack-nova | 08:26 | |
*** dmk0202 has joined #openstack-nova | 08:27 | |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (8) https://review.openstack.org/330948 | 08:27 |
openstackgerrit | Stefan Amann proposed openstack/nova: Support FICON-attached ECKD volumes on System z https://review.openstack.org/320832 | 08:27 |
*** daemontool has joined #openstack-nova | 08:28 | |
*** Daisy_ has joined #openstack-nova | 08:29 | |
*** Daisy has quit IRC | 08:33 | |
*** mkrcmari__ has joined #openstack-nova | 08:36 | |
*** thomasem_ has quit IRC | 08:36 | |
*** rmstar has joined #openstack-nova | 08:36 | |
*** andreas_s_ has joined #openstack-nova | 08:36 | |
*** kaisers has quit IRC | 08:37 | |
*** Jeffrey4l has quit IRC | 08:37 | |
*** abalutoiu_ has joined #openstack-nova | 08:37 | |
*** jlvillal has quit IRC | 08:37 | |
*** tojuvone__ has joined #openstack-nova | 08:37 | |
*** liverpoo1er has joined #openstack-nova | 08:38 | |
*** kaisers has joined #openstack-nova | 08:38 | |
*** bswartz has quit IRC | 08:39 | |
*** cgalan has quit IRC | 08:39 | |
*** rodrigods has quit IRC | 08:40 | |
*** xwizard___ has joined #openstack-nova | 08:40 | |
*** pumaranikar has quit IRC | 08:40 | |
*** rodrigods has joined #openstack-nova | 08:41 | |
*** harlowja_ has quit IRC | 08:41 | |
*** pkoniszewski has quit IRC | 08:41 | |
*** jlvillal has joined #openstack-nova | 08:42 | |
*** Daisy_ has quit IRC | 08:42 | |
*** daemontool_ has joined #openstack-nova | 08:42 | |
*** clduser has quit IRC | 08:42 | |
*** anteaya has quit IRC | 08:42 | |
*** Daisy has joined #openstack-nova | 08:42 | |
*** ishant|away has quit IRC | 08:42 | |
*** mvk_ has quit IRC | 08:42 | |
*** andreas_s has quit IRC | 08:42 | |
*** rmstar_ has quit IRC | 08:42 | |
*** xwizard has quit IRC | 08:42 | |
*** ekarlso has quit IRC | 08:42 | |
*** abalutoiu has quit IRC | 08:42 | |
*** ishant|away has joined #openstack-nova | 08:42 | |
*** purplerbot has quit IRC | 08:42 | |
*** thomasem has joined #openstack-nova | 08:42 | |
*** lxsli0 has joined #openstack-nova | 08:42 | |
*** pkoniszewski has joined #openstack-nova | 08:43 | |
*** nicolasbock has quit IRC | 08:43 | |
*** igordcard has quit IRC | 08:43 | |
*** nicolasbock_ has joined #openstack-nova | 08:43 | |
*** mdavidson has quit IRC | 08:43 | |
*** liverpooler has quit IRC | 08:43 | |
*** tlbr has quit IRC | 08:43 | |
*** tojuvone_ has quit IRC | 08:43 | |
*** zeroDivisible has quit IRC | 08:43 | |
*** kaisers1 has quit IRC | 08:43 | |
*** aignatov has quit IRC | 08:43 | |
*** lxsli has quit IRC | 08:43 | |
*** xek has quit IRC | 08:43 | |
*** clduser has joined #openstack-nova | 08:43 | |
andreas_s_ | Hi, we have a nova patchset, that we can't get out of a merge conflict. Locally rebasing works fine but after pushing it, it suddenly is in merge conflict again | 08:43 |
andreas_s_ | anybody an idea? | 08:43 |
*** zeroDivisible has joined #openstack-nova | 08:43 | |
*** yhvh has quit IRC | 08:43 | |
andreas_s_ | https://review.openstack.org/320832 | 08:43 |
*** xek has joined #openstack-nova | 08:43 | |
*** aignatov has joined #openstack-nova | 08:43 | |
claudiub | you sure that you fetched master? | 08:43 |
*** pumaranikar has joined #openstack-nova | 08:43 | |
andreas_s_ | claudiub, yep, we even did a fresh nova clone | 08:44 |
*** mtreinish has quit IRC | 08:44 | |
*** daemontool has quit IRC | 08:44 | |
*** tlbr has joined #openstack-nova | 08:44 | |
*** yhvh has joined #openstack-nova | 08:44 | |
*** mdavidson has joined #openstack-nova | 08:44 | |
*** ihrachys has joined #openstack-nova | 08:44 | |
claudiub | andreas_s_: I think it's because the Depends-On patch is in a merge conflict. | 08:45 |
*** kaisers1 has joined #openstack-nova | 08:45 | |
*** anteaya has joined #openstack-nova | 08:45 | |
andreas_s_ | claudiub, ah ok, interessting. But that's a little bit confusing, as the depends on is for os-brick.. | 08:46 |
*** mtreinish has joined #openstack-nova | 08:46 | |
claudiub | andreas_s_: This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. it refers to the Depends-On patch. | 08:46 |
andreas_s_ | anyhow, we'll try to rebase that first | 08:46 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 08:46 |
mdbooth | Anybody else unable to log in to gerrit this morning? | 08:46 |
andreas_s_ | claudiub, got it! Thanks you! | 08:46 |
mdbooth | I get an OpenID error: Provider is not supported, or was incorrectly entered. | 08:47 |
*** tpeoples has quit IRC | 08:47 | |
*** Daisy_ has joined #openstack-nova | 08:47 | |
mdbooth | Ah, nm. After reproducing 3 times it seems to have fixed itself :) | 08:47 |
*** macsz has quit IRC | 08:47 | |
claudiub | andreas_s_: when you add a Depends-On line in the commit message, Jenkins and other CIs will also include that patch (no matter what project) in its tests. if it couldn't be cherry-picked, it fails. | 08:47 |
*** yassine__ has joined #openstack-nova | 08:48 | |
claudiub | mdbooth: 3 has always been a magical number. :) | 08:48 |
*** Jeffrey4l has joined #openstack-nova | 08:48 | |
* mdbooth hires a shaman for the rest of his bugs | 08:48 | |
*** macsz has joined #openstack-nova | 08:49 | |
*** tpeoples has joined #openstack-nova | 08:49 | |
*** igordcard has joined #openstack-nova | 08:49 | |
kashyap | mdbooth: I'm about to login | 08:50 |
kashyap | Ah, you said "it fixed itself" | 08:50 |
*** Daisy has quit IRC | 08:50 | |
mdbooth | kashyap: Don't forget to cast the appropriate bones first | 08:51 |
andreas_s_ | claudiub, thanks for this explanation - wasn't ware of that | 08:51 |
*** ekarlso has joined #openstack-nova | 08:51 | |
kashyap | Heh | 08:51 |
claudiub | andreas_s_: no problem. :) | 08:51 |
BobBall | johnthetubaguy: ping | 08:55 |
*** EinstCrazy has quit IRC | 08:55 | |
*** Daisy has joined #openstack-nova | 08:58 | |
*** Daisy_ has quit IRC | 08:58 | |
*** EinstCrazy has joined #openstack-nova | 08:59 | |
*** Daisy_ has joined #openstack-nova | 09:04 | |
*** hoangcx has quit IRC | 09:05 | |
*** hoangcx_ has joined #openstack-nova | 09:05 | |
*** Daisy_ has quit IRC | 09:06 | |
*** Daisy has quit IRC | 09:06 | |
*** EinstCra_ has joined #openstack-nova | 09:06 | |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (9) https://review.openstack.org/330964 | 09:08 |
*** EinstCrazy has quit IRC | 09:09 | |
*** mpavone has quit IRC | 09:11 | |
*** sambetts|afk is now known as sambetts | 09:12 | |
*** mpavone has joined #openstack-nova | 09:12 | |
bauwser | do we success Jenkins passing away ? :p | 09:14 |
*** DaiDV has joined #openstack-nova | 09:16 | |
*** gjayavelu has quit IRC | 09:16 | |
*** openstackgerrit has quit IRC | 09:18 | |
*** openstackgerrit has joined #openstack-nova | 09:18 | |
*** zenoway has joined #openstack-nova | 09:19 | |
*** ssurana has joined #openstack-nova | 09:20 | |
BobBall | I'm sure there will be a number of parties :) | 09:22 |
*** cdent has joined #openstack-nova | 09:23 | |
*** ssurana has quit IRC | 09:24 | |
*** purplerbot has joined #openstack-nova | 09:25 | |
*** alex_xu has quit IRC | 09:26 | |
*** mfedosin has joined #openstack-nova | 09:26 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/nova: Change ignored exception in Ironic virt driver https://review.openstack.org/330984 | 09:27 |
*** alex_xu has joined #openstack-nova | 09:28 | |
*** lxsli0 is now known as lxsli | 09:32 | |
openstackgerrit | melanie witt proposed openstack/nova: WIP Make InstanceGroup object favor the API database https://review.openstack.org/330995 | 09:34 |
openstackgerrit | liuxiuli proposed openstack/nova: And reserved_huge_pages to memory_mb_used https://review.openstack.org/330928 | 09:34 |
openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Skip driver detach calls for non local instances https://review.openstack.org/318266 | 09:39 |
*** mgoddard_ has joined #openstack-nova | 09:42 | |
*** mgoddard has quit IRC | 09:42 | |
johnthetubaguy | BobBall: sorry, missed your ping, whats up? | 09:45 |
*** purplerbot has quit IRC | 09:45 | |
BobBall | No worries. Was just checking if you were around today. See PM | 09:45 |
*** purplerbot has joined #openstack-nova | 09:47 | |
* cdent pats purplerbot | 09:47 | |
*** mfedosin has quit IRC | 09:52 | |
*** PaulMurray has joined #openstack-nova | 09:55 | |
bauwser | do we know if we default limit/marker for all our API resources or only for the servers ? | 10:02 |
bauwser | ie. nova list etc. | 10:02 |
bauwser | johnthetubaguy: ^ do you know that ? | 10:03 |
bauwser | ie. I know we paginate nova list for sure, but I was wondering for paginating other API resources | 10:03 |
*** openstackgerrit has quit IRC | 10:03 | |
johnthetubaguy | hmm, it might just be servers | 10:03 |
johnthetubaguy | so we don't do it for images | 10:03 |
johnthetubaguy | because thats a proxy, and the server we talk to doesn't do it | 10:03 |
bauwser | ok | 10:03 |
*** openstackgerrit has joined #openstack-nova | 10:03 | |
bauwser | I think os-hosts is not paginating too | 10:03 |
johnthetubaguy | but I am not sure about the others, it would seem reasonable to limit those | 10:03 |
bauwser | tbc, we don't have a default behaviour accepting pagination for any API resource action ? | 10:04 |
bauwser | if so, it's per resource/action I see | 10:04 |
johnthetubaguy | right, pagination is added per resource | 10:04 |
bauwser | even per action I guess | 10:04 |
bauwser | like a GET /servers/id doesn't make sense to be paginated | 10:05 |
johnthetubaguy | it largely has to, because to get any benefit, we need to send that detail down to the DB call | 10:05 |
*** eglynn has joined #openstack-nova | 10:05 | |
johnthetubaguy | right | 10:05 |
*** rcernin has quit IRC | 10:05 | |
*** mfedosin has joined #openstack-nova | 10:05 | |
bauwser | okay, I guess I should sneak peak at the API ref documentation and see what resources are having limit/marker :) | 10:06 |
bauwser | johnthetubaguy: thanks for clarifying that :) | 10:06 |
johnthetubaguy | yeah, not sure that detail is complete yet... | 10:08 |
openstackgerrit | huan proposed openstack/nova: XenAPI: Support neutron security group https://review.openstack.org/251271 | 10:11 |
*** tbachman has quit IRC | 10:14 | |
*** hoangcx_ has quit IRC | 10:22 | |
*** ociuhandu has quit IRC | 10:23 | |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (10) https://review.openstack.org/331013 | 10:25 |
*** rmart04 has joined #openstack-nova | 10:30 | |
*** tbachman has joined #openstack-nova | 10:35 | |
*** smatzek has joined #openstack-nova | 10:37 | |
*** gjayavelu has joined #openstack-nova | 10:38 | |
*** ivase has quit IRC | 10:41 | |
*** ociuhandu has joined #openstack-nova | 10:43 | |
*** gjayavelu has quit IRC | 10:43 | |
*** huyang has quit IRC | 10:44 | |
*** huyang has joined #openstack-nova | 10:45 | |
*** tbachman_ has joined #openstack-nova | 10:46 | |
*** permalac has joined #openstack-nova | 10:47 | |
*** sree has joined #openstack-nova | 10:48 | |
*** tbachman has quit IRC | 10:50 | |
*** tbachman_ is now known as tbachman | 10:50 | |
*** rtheis has joined #openstack-nova | 10:52 | |
*** hogepodge has joined #openstack-nova | 10:54 | |
openstackgerrit | sahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop https://review.openstack.org/174854 | 10:54 |
cdent | bauwser, johnthetubaguy: you might "enjoy" the email I've just sent to os-dev.... | 10:55 |
*** tbachman_ has joined #openstack-nova | 10:55 | |
*** eglynn has quit IRC | 10:55 | |
*** tbachman has quit IRC | 10:57 | |
*** tbachman_ is now known as tbachman | 10:57 | |
openstackgerrit | Lucian Petrut proposed openstack/nova: Py3: fix serial console output https://review.openstack.org/302182 | 10:59 |
*** sree has quit IRC | 11:01 | |
openstackgerrit | Lucian Petrut proposed openstack/nova: Py3: fix serial console output https://review.openstack.org/302182 | 11:02 |
openstackgerrit | Claudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests https://review.openstack.org/300262 | 11:03 |
openstackgerrit | John Garbutt proposed openstack/nova: Create _validate_requested_port_ids https://review.openstack.org/329600 | 11:04 |
openstackgerrit | Claudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests https://review.openstack.org/278834 | 11:04 |
*** andreas_s_ has quit IRC | 11:04 | |
openstackgerrit | John Garbutt proposed openstack/nova: Extract _validate_requested_network_ids https://review.openstack.org/329601 | 11:05 |
claudiub | johnthetubaguy: hi. if you have a little bit of free time, can you take a look at a python 3 compatibility patch? it has been reviewed a few times. Thanks! https://review.openstack.org/#/c/278810/18 | 11:08 |
*** romainh has quit IRC | 11:09 | |
*** diga has quit IRC | 11:11 | |
*** sree has joined #openstack-nova | 11:13 | |
*** ssurana has joined #openstack-nova | 11:20 | |
BobBall | dansmith: Your comments have been addressed on https://review.openstack.org/#/c/242846/. This bugfix is a pre-requisite for one of our BPs, so we'd appreciate a re-review when you have time. | 11:22 |
*** thorst_ has joined #openstack-nova | 11:23 | |
*** hoangcx has joined #openstack-nova | 11:25 | |
*** ssurana has quit IRC | 11:25 | |
bauwser | cdent: lemme tell you a story | 11:33 |
*** gcb has quit IRC | 11:33 | |
* cdent sits comfortably | 11:33 | |
bauwser | cdent: once upon a time, a very small project tried to define a REST API using first Flask, then Pecan | 11:33 |
openstackgerrit | Paul Murray proposed openstack/nova: Add periodic task to clean expired console tokens https://review.openstack.org/325381 | 11:33 |
openstackgerrit | Paul Murray proposed openstack/nova: Add console auth token records to the database https://review.openstack.org/325414 | 11:33 |
openstackgerrit | Paul Murray proposed openstack/nova: Add console auth object https://review.openstack.org/320063 | 11:33 |
openstackgerrit | Paul Murray proposed openstack/nova: Add instance uuid to console URLs https://review.openstack.org/325382 | 11:33 |
openstackgerrit | Paul Murray proposed openstack/nova: Add console auth tokens db api methods https://review.openstack.org/319964 | 11:34 |
openstackgerrit | Paul Murray proposed openstack/nova: Add console auth tokens table and model https://review.openstack.org/319327 | 11:34 |
bauwser | cdent: both were interesting to use, but at the end, the project failed | 11:34 |
*** hoangcx has quit IRC | 11:34 | |
bauwser | cdent: the outcome of that story ? that's not really a huge crisp to me, I just care about people writing code they can understand :) | 11:34 |
cdent | me too, which is why I've gone the way I have. I think flask and pecan are _far_ harder to understand than what I've written thus far | 11:35 |
*** aysyd has joined #openstack-nova | 11:35 | |
bauwser | pecan was cool to use (at least routing was far more understandable) but given we use the standard-ish WSGI stuff in Nova for a while, I tend to say "meh" | 11:36 |
bauwser | the real terrible thing we struggle was with input validation, using He-Who-Cannot-Be-Named aka. WSME | 11:37 |
cdent | huh. I think the routing in pecan is completely opaque and that object-dispatch is a very bad model to use with APIs that claim to RESTful | 11:37 |
cdent | never speak of WSME! :) | 11:37 |
bauwser | (in front of a mirror) WSME, WSME, WSME ! | 11:38 |
* bauwser quit | 11:38 | |
bauwser | :p | 11:38 |
cdent | with regard to "standard-ish WSGI stuff in Nova" <- do people really think that the way Nova does WSGI is anything close to normal? I find the nova wsgi code to be completely alien, way way too complex, and impossible to trace | 11:38 |
*** lucasagomes is now known as lucas-hungry | 11:39 | |
bauwser | cdent: so, enough TGIF, just to say I'm not that expert to voice here, just saying that I care about people being able to review and code what we're proposing, that's it, and me tending to prefer things close to what we have in the current Nova tree | 11:39 |
* cdent steps to the left, to the right, to the front, to the back and turns slowly around | 11:39 | |
bauwser | https://m.popkey.co/cedcc1/eLw7m.gif | 11:40 |
cdent | I think the "the way nova does it" is only good for people who have been with nova for quite a while, and there really aren't that many of those people. There are far more potential contributors who have never seen nova. | 11:40 |
cdent | ha! | 11:40 |
*** sree__ has joined #openstack-nova | 11:41 | |
cdent | for that latter group their reaction to nova is much like JohnT there | 11:41 |
bauwser | heh | 11:42 |
*** irenab has joined #openstack-nova | 11:42 | |
*** sree has quit IRC | 11:42 | |
openstackgerrit | sahid proposed openstack/nova: network: handle unauthorized exception from neutron https://review.openstack.org/312014 | 11:43 |
bauwser | honestly, I don't think the ramp-up on that API code is so hard | 11:43 |
bauwser | either way, people coming to Nova know there is an history | 11:44 |
*** hoangcx has joined #openstack-nova | 11:44 | |
bauwser | of course, copy/pasting is never a good idea if you don't understand what you're doing in between | 11:44 |
bauwser | but, here, I just don't want to redo the wheel, that's it :-) | 11:45 |
bauwser | in particular since one of my roles is to look at what people write and set my mind on the project I'm looking to | 11:46 |
bauwser | in particular wrt the patterns we use and the things we agreed in the past | 11:46 |
bauwser | having separate codepaths is like darwinian evolution | 11:46 |
*** irenab has quit IRC | 11:47 | |
*** sree has joined #openstack-nova | 11:47 | |
cdent | bauwser: keep in mind that all the api code that is being written is not nova. it something that is intentionally separate from nova, so should look and be different, yeah? | 11:47 |
*** rcernin has joined #openstack-nova | 11:47 | |
bauwser | cdent: sure, it's like if Nova was about to write a new version, right? | 11:48 |
cdent | Or perhaps another way to think of it is this: If we have started it as a standalone project in the first place, would there be any debate? | 11:48 |
bauwser | cdent: there would, for sure | 11:48 |
cdent | I mean about the being similar to nova's code | 11:48 |
*** Jeffrey4l has quit IRC | 11:48 | |
cdent | sure there should be debate on how to do the api | 11:48 |
*** Jeffrey4l has joined #openstack-nova | 11:48 | |
cdent | but the question would be around framework vs framework-less | 11:49 |
bauwser | but we're still discussing about the placement API being part of nova, do we ? :) | 11:49 |
cdent | I think for most people that's a done deal. | 11:49 |
*** gcb has joined #openstack-nova | 11:49 | |
cdent | It is inevitable that it has to separate so that it can provide a good service to mlutiple projectg clients, not just nova. | 11:49 |
*** houming has quit IRC | 11:49 | |
cdent | And so that it can evolve independently of the history of nova | 11:50 |
bauwser | I still haven't seen consensus on that being a separate *service* :) | 11:50 |
bauwser | but either way, it's not the discussion we're having originally | 11:50 |
*** sree__ has quit IRC | 11:51 | |
cdent | I think you should talk to jay, dan, sean, johnthetubaguy, matt r about the plans with the placement api. Or maybe I should. :) | 11:51 |
cdent | I think it is a key part of this discussion because it drives a lot of the debate around how to do the api. Nova's API structure was declared inappropriate for this new API. That was decided. | 11:52 |
cdent | What was not decided was what the new way should be | 11:52 |
bauwser | I agree :) | 11:53 |
bauwser | tbc, I agree with the point that the Nova API structure is inappropriate | 11:53 |
*** EinstCra_ has quit IRC | 11:53 | |
bauwser | what I think we haven't concluded is that this new API should be part of a new separate *project* | 11:54 |
*** eharney has joined #openstack-nova | 11:55 | |
*** eglynn has joined #openstack-nova | 11:56 | |
openstackgerrit | John Garbutt proposed openstack/nova: Extract _validate_requested_network_ids https://review.openstack.org/329601 | 11:56 |
*** EinstCrazy has joined #openstack-nova | 11:57 | |
cdent | bauwser: I thought that was concluded, but let's assume for the time being it hasn't. Given that, what, to you, are the reasons why it should not become its own project? | 11:57 |
*** Kevin-Zhao has quit IRC | 11:58 | |
*** Kevin-Zhao has joined #openstack-nova | 11:58 | |
bauwser | it's Friday, are you sure we should have this discussion now and here ? | 11:58 |
cdent | I dunno, I'm curious, and starting next week I'll be travelling so won't have much opportunity to have it from the comfort of my comfy chair. | 11:59 |
*** EinstCra_ has joined #openstack-nova | 12:00 | |
cdent | It's a position you've been holding for quite some time that to me is in opposition to what nearly everyone else is saying, yet I've not really heard a coherent statement of your reasons why. | 12:00 |
cdent | I simply want to understand so that I know what's going on. | 12:00 |
openstackgerrit | John Garbutt proposed openstack/nova: Extract _validate_requested_network_ids https://review.openstack.org/329601 | 12:01 |
*** EinstCrazy has quit IRC | 12:02 | |
*** cdelatte has quit IRC | 12:03 | |
*** EinstCrazy has joined #openstack-nova | 12:03 | |
bauwser | cdent: I think http://lists.openstack.org/pipermail/openstack-dev/2015-March/060179.html still applies here | 12:03 |
*** dave-mccowan has joined #openstack-nova | 12:04 | |
openstackgerrit | bailin.zhang proposed openstack/nova: Delete reduplicate code in test_compute_mgr.py https://review.openstack.org/331108 | 12:04 |
bauwser | we're just discussing about splitting the nova-scheduler, not creating a cross-project scheduler | 12:04 |
*** markvoelker has joined #openstack-nova | 12:04 | |
bauwser | the fact that the other projects could consume that API for providing their own resources doesn't necessarly mean that it would be a cross-project scheduler (ie. giving volumes to Cinder) | 12:04 |
bauwser | s/ie./eg. | 12:05 |
*** dmk0202 has quit IRC | 12:05 | |
bauwser | in other words, people deploying a Compute service would necessarly install a separate package, I'm happy with that | 12:06 |
*** haukebruno has quit IRC | 12:06 | |
*** EinstCra_ has quit IRC | 12:06 | |
*** pradk has joined #openstack-nova | 12:06 | |
bauwser | but that wouldn't mean that this package would be usable by other projects | 12:06 |
bauwser | tbc, service projects | 12:06 |
*** kfarr has joined #openstack-nova | 12:07 | |
*** pradk has quit IRC | 12:07 | |
cdent | okay, but what does have to do with whether lifting and shifting the code is a good idea or not? | 12:07 |
tdurakov | hi folks | 12:07 |
tdurakov | don't you know is danpb is on vacation or not? | 12:07 |
*** mtanino has joined #openstack-nova | 12:07 | |
*** sree has quit IRC | 12:08 | |
cdent | tdurakov: he was around yesterday or the day before but that's all I know | 12:08 |
*** rfolco has joined #openstack-nova | 12:08 | |
tdurakov | cdent: same for me | 12:08 |
jroll | cdent: I think it should use pecan/wsme | 12:08 |
tdurakov | kashyap: hi, are you around? | 12:08 |
* jroll runs away | 12:08 | |
* cdent gives jroll a cookie | 12:08 | |
bauwser | jroll: trolls are accepted on Fridays | 12:09 |
jroll | \o/ | 12:09 |
* jroll comes back and eats cookies | 12:09 | |
openstackgerrit | Zhihai Song proposed openstack/nova: Transform instance.power_on notifications https://review.openstack.org/329746 | 12:09 |
jroll | cdent: so my worry with "don't use a framework" is that I suspect over time it will turn into a framework of its own | 12:11 |
cdent | jroll: I don't know what you're eating but I only gave you _one_ cookie, sir! | 12:11 |
*** EinstCra_ has joined #openstack-nova | 12:11 | |
cdent | jroll: did you have a chance to look at the code? | 12:11 |
jroll | cdent: I'm skimming now | 12:11 |
cdent | it's a very valid concern | 12:11 |
*** EinstCra_ has quit IRC | 12:11 | |
jroll | cdent: if there's one thing I've learned from openstack, it's that eventually all openstack code becomes abstracted beyond all recognition (for better or worse) | 12:12 |
cdent | and yeah, it is kind of inevitable, but I balance that against the inspectability bit, so to my eye it's really a question whether we can lay in "transparency" as a design constraint | 12:13 |
*** pradk has joined #openstack-nova | 12:13 | |
cdent | and you're right, enforcing that constraint is apparently impossible in openstack | 12:13 |
*** EinstCra_ has joined #openstack-nova | 12:13 | |
cdent | so maybe I'm just using this as exercise to tilt at all the broken windmills in openstack :( | 12:13 |
jroll | I mean, I fully support your efforts in theory | 12:14 |
jroll | but idk if it's the best route long term | 12:14 |
cdent | "abstracted beyond all recognition" is, to me, the worst possible flaw in code | 12:14 |
cdent | jroll: thus the email, I wanted to get people engaged now before it is too late | 12:14 |
jroll | yep, I will have a think on it | 12:14 |
*** EinstCrazy has quit IRC | 12:14 | |
cdent | if we wanted "abstracted beyond all recognition" why aren't we using java? | 12:14 |
jroll | I also don't think flask is "too" magical, having used flask, django, pecan | 12:15 |
jroll | but I'll save that bit for the list :) | 12:15 |
cdent | they are all too magical to me :) | 12:15 |
jroll | cdent: were you in the room for the DLM discussion? | 12:15 |
bauwser | oh please | 12:15 |
cdent | in tokyo? yes | 12:15 |
jroll | in vancouver I thnk | 12:15 |
jroll | er yeah | 12:15 |
jroll | "OH NO JVM" | 12:15 |
cdent | or maybe it was vancouver | 12:15 |
cdent | but yeah, I was there | 12:15 |
* cdent doesn't mind jvm, but does mind java | 12:16 | |
jroll | I think the main reason is that nothing starts with a goal of looking like java around here | 12:16 |
jroll | it just inevitably does | 12:16 |
cdent | my first reaction when I came into the openstack universe was "these are not python developers, these are java developers doing pythong". Which was unfair and untrue but it was the reaction I had. | 12:16 |
cdent | jroll: preventing that is part of why my little api uses mostly functions, not classes | 12:17 |
cdent | it's only uses classes to give paste something to hang on to | 12:17 |
cdent | i'd avoid paste too if I could... | 12:17 |
jroll | ++ | 12:18 |
* jroll has succeeded with not having paste in ironic yet | 12:18 | |
cdent | at the moment the only reason it is in there was to "make it easy to get started" but it could go away | 12:18 |
* cdent leaves himself a comment | 12:18 | |
*** zhangjn has quit IRC | 12:19 | |
*** cdelatte has joined #openstack-nova | 12:19 | |
openstackgerrit | bailin.zhang proposed openstack/nova: Unplug vifs when VM create failed https://review.openstack.org/331113 | 12:20 |
cdent | bauwser: what were you "oh please"ing? | 12:20 |
openstackgerrit | Matthew Booth proposed openstack/nova: Never copy a swap disk during cold migration https://review.openstack.org/331115 | 12:21 |
openstackgerrit | Matthew Booth proposed openstack/nova: Document the expectations of all callers of libvirt's _create_image https://review.openstack.org/331116 | 12:21 |
openstackgerrit | Matthew Booth proposed openstack/nova: Move root disk snapshot into finish_migration https://review.openstack.org/331117 | 12:21 |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Don't re-resize disks in finish_migration() https://review.openstack.org/331118 | 12:21 |
bauwser | cdent: I was also there, and my intent behind "oh please" was not implicitely ask to *not* reopen this kind of discussion :) | 12:21 |
*** EinstCra_ has quit IRC | 12:21 | |
jroll | hehe | 12:21 |
*** chenfei has quit IRC | 12:22 | |
bauwser | was to* | 12:22 |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Create Session instance if possible https://review.openstack.org/304035 | 12:22 |
*** zhangjn has joined #openstack-nova | 12:23 | |
*** cdelatte has quit IRC | 12:25 | |
*** chenfei has joined #openstack-nova | 12:25 | |
cdent | jroll, bauwser: I look forward to extensive, informative and engaging emails from you both in response to that message I posted. ;) | 12:25 |
* cdent has lunch | 12:25 | |
*** jamesdenton has joined #openstack-nova | 12:25 | |
jroll | cdent: damn, now I have commitments | 12:26 |
*** ivase has joined #openstack-nova | 12:26 | |
bauwser | cdent: sure, I'll try to figure out the best meme I can give you by email | 12:26 |
*** pwnall has joined #openstack-nova | 12:26 | |
*** cdelatte has joined #openstack-nova | 12:30 | |
*** haukebruno has joined #openstack-nova | 12:31 | |
*** edmondsw has joined #openstack-nova | 12:34 | |
*** baoli has joined #openstack-nova | 12:34 | |
*** baoli_ has joined #openstack-nova | 12:36 | |
*** tbachman_ has joined #openstack-nova | 12:36 | |
*** ljxiash has joined #openstack-nova | 12:36 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transfrom wrap_exception notification to versioned format https://review.openstack.org/310700 | 12:36 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts https://review.openstack.org/330513 | 12:36 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Enhance notification sample test base https://review.openstack.org/321610 | 12:36 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Handle multiple samples per versioned notification https://review.openstack.org/321609 | 12:36 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.update notification https://review.openstack.org/325923 | 12:36 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.delete notifications https://review.openstack.org/313654 | 12:37 |
*** hoangcx has quit IRC | 12:38 | |
*** hoangcx has joined #openstack-nova | 12:39 | |
*** cdelatte has quit IRC | 12:39 | |
openstackgerrit | Julian proposed openstack/nova: Transform instance.snapshot notifications https://review.openstack.org/330128 | 12:39 |
*** tbachman has quit IRC | 12:40 | |
*** tbachman_ is now known as tbachman | 12:40 | |
*** alaski is now known as lascii | 12:40 | |
*** baoli has quit IRC | 12:40 | |
*** cdelatte has joined #openstack-nova | 12:41 | |
*** baoli_ has quit IRC | 12:41 | |
openstackgerrit | Julian proposed openstack/nova: Transform instance.snapshot notifications https://review.openstack.org/330128 | 12:42 |
*** bswartz has joined #openstack-nova | 12:43 | |
*** hoangcx has quit IRC | 12:43 | |
*** chandankumar has quit IRC | 12:44 | |
*** kylek3h has joined #openstack-nova | 12:45 | |
*** kylek3h has quit IRC | 12:45 | |
*** kylek3h has joined #openstack-nova | 12:45 | |
kashyap | tdurakov: Hey, was away for some lunch | 12:47 |
tdurakov | kashyap: could you please take a look https://review.openstack.org/#/c/320897/4 | 12:48 |
* kashyap clicks | 12:48 | |
tdurakov | kashyap: especially interesting this thread https://www.redhat.com/archives/libvir-list/2016-May/msg01937.html | 12:48 |
kashyap | tdurakov: Give me a few, finishing a write-up on a QEMU related bug. | 12:48 |
tdurakov | kk | 12:48 |
*** pradk has quit IRC | 12:50 | |
*** lucas-hungry is now known as lucasagomes | 12:50 | |
*** ssurana has joined #openstack-nova | 12:53 | |
*** edleafe is now known as figleaf | 12:53 | |
*** ssurana has quit IRC | 12:57 | |
openstackgerrit | Zhihai Song proposed openstack/nova: Transform instance.power_on notifications https://review.openstack.org/329746 | 12:59 |
kashyap | tdurakov: Have seen that patch in passing before. Have to read the full context. (/me be back in a bit.) | 13:02 |
*** gcb has quit IRC | 13:02 | |
*** cdelatte has quit IRC | 13:03 | |
*** zhs_ss has quit IRC | 13:05 | |
*** MikeG451_ has joined #openstack-nova | 13:07 | |
*** tbachman_ has joined #openstack-nova | 13:08 | |
*** mdrabe has joined #openstack-nova | 13:09 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: Added support for new block device format in vmops https://review.openstack.org/246299 | 13:10 |
*** tbachman has quit IRC | 13:10 | |
*** tbachman_ is now known as tbachman | 13:10 | |
*** zerda2 has quit IRC | 13:10 | |
*** xyang1 has joined #openstack-nova | 13:10 | |
johnthetubaguy | cdent: at a first glance, the API code looks good just using webob directly, +1 killing paste.ini | 13:10 |
openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref: os-volumes.inc https://review.openstack.org/320048 | 13:12 |
cdent | johnthetubaguy: thanks, yeah, one of the good outcomes of the discussion above was remembering paste could go | 13:12 |
bauwser | johnthetubaguy: cdent: tbc, given http://docs.openstack.org/developer/nova/policies.html#public-contractual-apis I'm fine with removing paste.ini for the placement API | 13:14 |
*** nkrinner has quit IRC | 13:14 | |
bauwser | (cdent: just explaining why I'm okay with that :p) | 13:14 |
cdent | johnthetubaguy: this puts you on the hook to write on that thread too, in an "extensive, informative and engaging" fashion... :D | 13:15 |
*** tblakeslee has joined #openstack-nova | 13:16 | |
openstackgerrit | Julian proposed openstack/nova: WIP Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2) https://review.openstack.org/325435 | 13:17 |
figleaf | cdent: Before I reply on the list, can you summarize what the objection to Selector is? Is it simiply adding another dependency, or is it more than that? | 13:17 |
*** nkrinner has joined #openstack-nova | 13:19 | |
cdent | figleaf: on the requirements review <https://review.openstack.org/#/c/329386/> tony has some objections to adding "another WSGI provider". His objections are not super strong, but that there were any suggests that there will be more in the community and I thought it was worthwhile to get the conversation started now instead of near the end of the cylce. | 13:19 |
cdent | Since selector isn't a "WSGI provider" I think the concerns are really more about the strategies I mention in the email | 13:20 |
figleaf | cdent: ah, thanks. So is there a way to achieve your approach with an existing (in Nova) package? | 13:20 |
figleaf | Or is selector simply the cleanest? | 13:20 |
openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref, os-services.inc https://review.openstack.org/321123 | 13:21 |
*** pacoblox has joined #openstack-nova | 13:21 | |
cdent | the Routes package, already used by Nova, does a similar thing but in a way that is a) harder to follow/trace/understand b) broken when it comes to correct HTTP | 13:22 |
*** akshai has joined #openstack-nova | 13:23 | |
*** nkrinner has quit IRC | 13:24 | |
*** pacoblox has quit IRC | 13:26 | |
*** takedakn has joined #openstack-nova | 13:26 | |
figleaf | cdent: broken in the sense that a) it doesn't do what you would expect it to or b) isn't strictly following HTTP conventions? | 13:27 |
cdent | for example: it is incapable of informing its client code of when a 405 is the correct response and thus causes a 404 instead | 13:28 |
cdent | (that is, when it can't find a route, it always returns None, with no indicator that there is a route, but not on at the desired method) | 13:29 |
cdent | s/on at/one at/ | 13:29 |
*** mrkz has joined #openstack-nova | 13:29 | |
*** mriedem has joined #openstack-nova | 13:31 | |
andreykurilin | mriedem: hi! | 13:32 |
figleaf | cdent: ok, thanks for the info | 13:32 |
cdent | you're welcome, thanks for asking | 13:32 |
mriedem | andreykurilin: hi | 13:33 |
mriedem | really pounced on me | 13:33 |
andreykurilin | :) | 13:34 |
andreykurilin | could you ping someone from keystone team to look at https://review.openstack.org/#/c/304035/ ? I tried to find someone several times, but my attempts were not successful :( | 13:34 |
*** efried has joined #openstack-nova | 13:35 | |
*** mtanino has quit IRC | 13:35 | |
mriedem | andreykurilin: sure | 13:37 |
andreykurilin | mriedem: thanks | 13:37 |
andreykurilin | btw, your review are preferable too:) | 13:38 |
mriedem | andreykurilin: bah | 13:38 |
mriedem | are any of these jobs running with keystone v3? | 13:38 |
mriedem | looks like this one is http://logs.openstack.org/35/304035/9/check/gate-tempest-dsvm-neutron-src-python-novaclient/47a0c38/logs/etc/openstack/clouds.yaml.txt.gz | 13:39 |
mriedem | and functional http://logs.openstack.org/35/304035/9/check/gate-novaclient-dsvm-functional/aa98073/logs/etc/openstack/clouds.yaml.txt.gz | 13:39 |
andreykurilin | mriedem: yes | 13:39 |
andreykurilin | and we have two functional tests for keystone v2 and v3 | 13:40 |
andreykurilin | novaclient.tests.functional.test_auth.TestAuthentication | 13:40 |
andreykurilin | mriedem: no, we do not have them:( | 13:40 |
*** smatzek has quit IRC | 13:41 | |
mriedem | i must have been thinking of this https://review.openstack.org/#/c/313588/ | 13:41 |
*** eantyshev has left #openstack-nova | 13:41 | |
*** dansmith is now known as superdan | 13:41 | |
*** eantyshev has joined #openstack-nova | 13:41 | |
andreykurilin | mriedem: it will break one functional test | 13:42 |
*** cdelatte has joined #openstack-nova | 13:42 | |
eantyshev | Hi, everyone! wanted to ask for reviews in https://review.openstack.org/190843 | 13:43 |
mriedem | andreykurilin: hmm, ok, that v3-only job would be non-voting to start, so wouldn't break us i guess | 13:43 |
andreykurilin | mriedem: sounds good | 13:43 |
*** pacoblox has joined #openstack-nova | 13:44 | |
*** pacoblox has quit IRC | 13:44 | |
*** mgoddard__ has joined #openstack-nova | 13:46 | |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Fix the clean up of resize-revert for volume backed instances https://review.openstack.org/329030 | 13:46 |
*** links has quit IRC | 13:46 | |
mriedem | hemna: have you seen this before? http://openstack-3rd-party-storage-ci-logs.virtuozzo.com/43/190843/26/check/dsvm-tempest-kvm/3d395b4/logs/screen-n-cpu.txt.gz | 13:47 |
mriedem | look for privsep log: privsep: request[78077616]: (3, 'os_brick.privileged.rootwrap.execute_root', ('systool', '-c', 'fc_host', '-v'), {}) | 13:47 |
mriedem | "privsep log: privsep: request[78077616]: (3, 'os_brick.privileged.rootwrap.execute_root', ('systool', '-c', 'fc_host', '-v'), {})" | 13:47 |
mriedem | eantyshev: we shouldn't be sharing rootwrap filters between nova and os-brick anymore https://review.openstack.org/#/c/190843/26/etc/nova/rootwrap.d/compute.filters | 13:48 |
mriedem | that should be handled with privsep | 13:48 |
*** mgoddard_ has quit IRC | 13:49 | |
*** cdelatte has quit IRC | 13:49 | |
*** fragatina has joined #openstack-nova | 13:49 | |
wasmum | anyone seen anything like this when rebuilding a failed neutron node in an ha environment? http://pastebin.com/80i1w6tB | 13:50 |
wasmum | looks like the switch db is not populating? | 13:50 |
*** ctrath has joined #openstack-nova | 13:50 | |
*** ametts has joined #openstack-nova | 13:50 | |
*** fragatin_ has joined #openstack-nova | 13:50 | |
*** sudipto has joined #openstack-nova | 13:51 | |
*** ecelik has quit IRC | 13:51 | |
*** rodrigods has quit IRC | 13:52 | |
*** rodrigods has joined #openstack-nova | 13:52 | |
eantyshev | mriedem: OK, I will search the Nova code for usage samples of privsep. Could you point me to the similar places? | 13:52 |
wasmum | in the vswitch db log i get | 13:52 |
wasmum | 2016-06-17T13:44:32.142Z|00009|jsonrpc|WARN|unix: send error: Broken pipe | 13:53 |
wasmum | 2016-06-17T13:44:32.146Z|00010|reconnect|WARN|unix: connection dropped (Broken pipe) | 13:53 |
mriedem | wasmum: see channel topic please | 13:53 |
mriedem | eantyshev: try getting with hemna on that | 13:53 |
mriedem | or gus | 13:53 |
*** fragatina has quit IRC | 13:54 | |
wasmum | mriedem: poop, sorry client shrunk room names, thought this was neutron | 13:54 |
*** armax has joined #openstack-nova | 13:54 | |
*** ctrath has quit IRC | 13:54 | |
eantyshev | mriedem: Thanks | 13:55 |
*** mmandel has joined #openstack-nova | 13:55 | |
*** pradk has joined #openstack-nova | 13:56 | |
gus | mriedem: those systool -c fc_host "failures" are normal with os-brick afaik. | 13:57 |
*** ctrath has joined #openstack-nova | 13:57 | |
gus | mriedem: it runs a bunch of fibrechannel commands, which fail if you don't have fibrechannel drivers, and then os-brick ignores/adapts-to the error exception. | 13:58 |
*** mriedem has quit IRC | 13:58 | |
gus | I never looked too closely at the actual call site. | 13:58 |
*** mriedem has joined #openstack-nova | 13:59 | |
*** burgerk has joined #openstack-nova | 13:59 | |
*** smatzek has joined #openstack-nova | 14:02 | |
*** mfedosin has quit IRC | 14:06 | |
*** cdelatte has joined #openstack-nova | 14:06 | |
*** gjayavelu has joined #openstack-nova | 14:07 | |
mriedem | eantyshev: the vz storage ci must be using oslo.privsep 1.4.0 | 14:08 |
mriedem | because i see https://bugs.launchpad.net/os-brick/+bug/1593743 | 14:08 |
openstack | Launchpad bug 1593743 in os-brick "(privsep) u'systool -c fc_host -v' failed. Not Retrying." [Undecided,New] | 14:08 |
mriedem | os-brick 1.4.0 is blacklisted in global-requirements right now | 14:08 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Rename _cleanup_resize to confirm_migration https://review.openstack.org/331158 | 14:10 |
eantyshev | mriedem: we don't amend global-requirements, oslo.privsep is of 1.8.0 in the latest run: http://openstack-3rd-party-storage-ci-logs.virtuozzo.com/59/323759/5/check/dsvm-tempest-kvm/4ea0cbe/logs/pip2-freeze.txt.gz | 14:10 |
*** sdake has joined #openstack-nova | 14:12 | |
eantyshev | mriedem: though, I see the error in n-cpu.log | 14:12 |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Update setup.cfg for memory bandwidth monitor https://review.openstack.org/329785 | 14:13 |
*** syjulian has joined #openstack-nova | 14:13 | |
*** cdelatte has quit IRC | 14:14 | |
mriedem | sudipto: did you address jay's -1 here? https://review.openstack.org/#/c/203163/ | 14:14 |
mriedem | eantyshev: you are using os-brick from master though right? | 14:15 |
mriedem | -e git+https://git.openstack.org/openstack/os-brick@0fbaad573060a56c5f9998b0182bcda8c3e7fe3e#egg=os_brick | 14:15 |
sudipto | mriedem, yeah i did. | 14:15 |
*** Jeffrey4l has quit IRC | 14:15 | |
mriedem | sudipto: please reply in the review to explain | 14:15 |
sudipto | mriedem, sure will. | 14:15 |
mriedem | eantyshev: the vz driver is in os-brick now right? if so, the vz storage ci shouldn't be running with os-brick from trunk | 14:16 |
mriedem | it should use the version from g-r | 14:16 |
*** zul has quit IRC | 14:16 | |
*** zul has joined #openstack-nova | 14:16 | |
mriedem | eantyshev: yeah vz has been in os-brick for awhile | 14:17 |
mriedem | https://github.com/openstack/os-brick/commit/ff44cb5457b0e33ffa9c9b4f63634e4b65f66ef4 | 14:17 |
eantyshev | mriedem: Oh. indeed. We kept "LIBS_FROM_GIT=os-brick" in localrc since patches were on review | 14:17 |
mriedem | eantyshev: that should be rmeoved | 14:18 |
mriedem | *removed | 14:18 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/331174 | 14:19 |
eantyshev | mriedem: will remove this, thanks again for pointing out! | 14:19 |
*** hemanthm|afk is now known as hemanthm | 14:20 | |
mnestratov | hi all | 14:20 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/331184 | 14:20 |
*** zul_ has joined #openstack-nova | 14:21 | |
*** pacoblox has joined #openstack-nova | 14:21 | |
*** mtanino has joined #openstack-nova | 14:21 | |
mriedem | mnestratov: hi | 14:21 |
*** mfedosin has joined #openstack-nova | 14:21 | |
openstackgerrit | Luis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available https://review.openstack.org/329077 | 14:22 |
mnestratov | mriedem: just to let you know about our Virtuzoo Compute CI status - we enabled it and reports again but we didn't remove patching yet | 14:22 |
mriedem | mnestratov: why not? | 14:22 |
*** zul has quit IRC | 14:23 | |
mnestratov | this is temporarily because we encountered a problem and we are solving it now | 14:23 |
mriedem | mnestratov: ok...i noticed last night that the resize patch was going to merge with the rescue patch, i would advise against that. if the resize patch depends on the changes in the rescue patch, they should be stacked up as dependencies. | 14:24 |
mnestratov | mriedem: while we were patching some other change broke our upstream code that we didn't notice - trying to fix it now | 14:24 |
mriedem | mnestratov: stacking those changes would make them easier to review | 14:24 |
mriedem | *and*, | 14:24 |
mriedem | for the rescue change we should enable running the rescue tests in tempest | 14:24 |
mnestratov | mriedem: ok | 14:24 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: update guest time after suspend https://review.openstack.org/316116 | 14:25 |
openstackgerrit | John Garbutt proposed openstack/nova: Make _validate_requested_network_ids return a dict https://review.openstack.org/329849 | 14:26 |
*** Jeffrey4l has joined #openstack-nova | 14:28 | |
mnestratov | mriedem: actually I thought that it makes sense to merge them because they are pretty close, but I see your point now - we will make them dependent | 14:31 |
*** cdelatte has joined #openstack-nova | 14:31 | |
*** david-lyle has joined #openstack-nova | 14:32 | |
mriedem | mnestratov: it's easier to review and test them separately | 14:33 |
mriedem | with 2 weeks left for non-priority blueprints you want to make the reviews as painless as possible :) | 14:33 |
mnestratov | mriedem: absolutely :) | 14:34 |
*** pacoblox has quit IRC | 14:34 | |
claudiub | hm, this wiki is a bit wrong. https://wiki.openstack.org/wiki/Pci-api-support there is no " | 14:34 |
gibi_ | mriedem: thanks for checking the https://review.openstack.org/#/c/310700/ yesterday. I think I answered / fixed your concerns. | 14:34 |
claudiub | or other pci related commands. | 14:35 |
mriedem | claudiub: i'm sure that's probably really old, probably at least juno | 14:35 |
mriedem | Date of page creation02:38, 20 November 2013 | 14:36 |
*** zz_dimtruck is now known as dimtruck | 14:36 | |
bauwser | woah, just saw Gerrit having the "included-in" dropdown list which gives us the list of branches and tags having the related change | 14:36 |
claudiub | mriedem: seems so. But I am wondering why the command was never added to python-novaclient. the os-pci nova API endpoint exists. | 14:37 |
*** cdelatte has quit IRC | 14:37 | |
claudiub | all I see is an abandoned patch on python-novaclient related to adding pci support. | 14:37 |
mriedem | bauwser: welcome to convenience | 14:38 |
mriedem | claudiub: no idea on the CLI | 14:38 |
*** sridharg has quit IRC | 14:38 | |
mriedem | it was pre-microversions | 14:38 |
bauwser | mriedem: well, my git repo will <3 that | 14:38 |
bauwser | because I was grepping the SHA1 :p | 14:38 |
bauwser | oops, the change-id rather | 14:39 |
*** pacoblox has joined #openstack-nova | 14:39 | |
*** cfriesen_ has joined #openstack-nova | 14:39 | |
*** sacharya has joined #openstack-nova | 14:40 | |
openstackgerrit | Luis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available https://review.openstack.org/329077 | 14:41 |
claudiub | mriedem: yeah, I think it was just a few months before microversions. https://review.openstack.org/#/c/75324/7 it seems it was adding the pci-list and pci-show commands | 14:41 |
claudiub | imo, I think it would still be worthwhile to add it. | 14:42 |
openstackgerrit | jichenjc proposed openstack/nova: WIP: remove scheduler_hints https://review.openstack.org/331194 | 14:42 |
*** ssurana has joined #openstack-nova | 14:42 | |
claudiub | although, I don't know, do we need a blueprint for something like this? | 14:42 |
mriedem | claudiub: if it's not using microversions, then i'd say do it in openstackclient | 14:42 |
mriedem | s/using/dependent on/ | 14:42 |
claudiub | aham. | 14:43 |
claudiub | artom: hello. got a minute? :) | 14:44 |
openstackgerrit | Luis Tomas proposed openstack/nova: Add automatic switching to postcopy mode when migration is not progressing https://review.openstack.org/328499 | 14:44 |
*** ihrachys has quit IRC | 14:45 | |
*** baoli has joined #openstack-nova | 14:45 | |
*** ljxiash has quit IRC | 14:47 | |
*** ssurana has quit IRC | 14:47 | |
*** sacharya_ has joined #openstack-nova | 14:47 | |
openstackgerrit | jichenjc proposed openstack/nova: WIP: remove scheduler_hints https://review.openstack.org/331194 | 14:48 |
openstackgerrit | jichenjc proposed openstack/nova: WIP: remove extension of scheduler hints totally https://review.openstack.org/331202 | 14:48 |
*** ratailor has quit IRC | 14:48 | |
*** prateek has quit IRC | 14:48 | |
artom | claudiub, hey :) Not right now, I took a PTO day to get my daughter to the doctor - this afternoon is you're still around, otherwise Monday | 14:48 |
artom | Or write to the mailing list and I'll reply async :) | 14:48 |
*** cdelatte has joined #openstack-nova | 14:48 | |
*** mpavone has quit IRC | 14:49 | |
claudiub | artom: Oh, I see. :) Well, I'll be around. When you come back, drop a ping pls. :) | 14:49 |
*** mpavone has joined #openstack-nova | 14:49 | |
*** sacharya has quit IRC | 14:50 | |
*** rrecio has joined #openstack-nova | 14:51 | |
*** mpavone has quit IRC | 14:51 | |
*** rrecio_ has joined #openstack-nova | 14:51 | |
*** mpavone has joined #openstack-nova | 14:52 | |
*** mpavone has quit IRC | 14:52 | |
*** krtaylor has joined #openstack-nova | 14:53 | |
openstackgerrit | Nicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/331204 | 14:53 |
*** pcaruana has quit IRC | 14:54 | |
*** rrecio has quit IRC | 14:55 | |
*** hoangcx has joined #openstack-nova | 14:55 | |
PaulMurray | what's a good example to copy for cells stuff ? | 14:55 |
*** hoangcx has quit IRC | 14:55 | |
*** hoangcx has joined #openstack-nova | 14:55 | |
*** jmarianu has quit IRC | 14:55 | |
*** ctrath has quit IRC | 14:57 | |
bauwser | PaulMurray: what do you want to do ? | 14:57 |
*** mgoddard has joined #openstack-nova | 14:58 | |
*** mgoddard__ has quit IRC | 14:58 | |
*** stewie925 has joined #openstack-nova | 15:00 | |
*** ctrath has joined #openstack-nova | 15:00 | |
PaulMurray | I'm putting console connection authroizations in the child database (this is to get rid of consoleauth) | 15:01 |
PaulMurray | so at api level I have an instance id and a console token | 15:01 |
PaulMurray | I need to get the console info from the child database | 15:01 |
PaulMurray | so I guess the process is - use isntance uuid to get child cell db | 15:02 |
PaulMurray | get info from child db | 15:02 |
bauwser | well, it's not an upcall, just a top-down | 15:02 |
PaulMurray | that's right | 15:02 |
*** diogogmt has joined #openstack-nova | 15:02 | |
doffm_ | bauwser: PaulMurray: That sounds ok, I'd still like to take a look at the use case. Is this to make an existing API work? | 15:02 |
PaulMurray | it will actually be from the console proxies | 15:03 |
bauwser | PaulMurray: the question I'm wondering is if we should automate this kind of call by looking up the InstanceMapping directly in a contextmanager | 15:03 |
*** doffm_ is now known as doffm | 15:03 | |
bauwser | but that's something maybe I missed from the last cells v2 meetings | 15:03 |
bauwser | ie. how we should call the child DBs explicitely or implicitely | 15:04 |
*** diogogmt has quit IRC | 15:04 | |
doffm | I guess explicitly if we are in the API cell. | 15:04 |
PaulMurray | are there any examples that do this already | 15:04 |
PaulMurray | or are they all in old v1 world code | 15:05 |
*** tesseract has quit IRC | 15:05 | |
sahid | mriedem: any chance you look at this https://review.openstack.org/#/c/174854/ i'm scary to see it blocked for the next release one more time | 15:06 |
bauwser | PaulMurray: nothing I know yet for cells v2 | 15:06 |
doffm | PaulMurray: Yeah, nothing I know of. | 15:06 |
bauwser | PaulMurray: AFAIR, we'll be using a contextmanager for selecting how to call the child cell | 15:06 |
bauwser | there is a review from melwitt IIRC | 15:06 |
doffm | PaulMurray: What spec are you working on? | 15:07 |
*** rcernin has quit IRC | 15:07 | |
PaulMurray | https://review.openstack.org/#/c/301158/ | 15:07 |
*** Oku_OS is now known as Oku_OS-away | 15:07 | |
PaulMurray | convert-consoles-to-objects | 15:08 |
*** rama_y has joined #openstack-nova | 15:08 | |
*** pacoblox has quit IRC | 15:08 | |
mfedosin | mriedem: hi! May I ask you a question? I noticed that nova.image code looks not so good... So, what if I refactor it a little bit, fix typos and add missing docstrings and parameters descriptions? | 15:09 |
*** pacoblox has joined #openstack-nova | 15:09 | |
*** amotoki has joined #openstack-nova | 15:09 | |
PaulMurray | melwitt, do you have any idea about above ? | 15:09 |
lascii | PaulMurray: is this a db or RPC call? | 15:10 |
mriedem | mfedosin: non-functional code comment / docstring etc changes are always welcome | 15:10 |
*** Oku_OS-away is now known as Oku_OS | 15:10 | |
doffm | PaulMurray: Ahhh, ok. There are some examples of choosing a cell and doing a database switch. | 15:10 |
PaulMurray | ahh, no wonder I couldn't find alaski..... I should have guessed | 15:10 |
melwitt | PaulMurray: what bauwser mentioned is this where we could do lookups for compute rpc calls automatically https://review.openstack.org/#/c/326906/1/nova/compute/rpcapi.py@398 | 15:10 |
mriedem | sahid: is that the last change in that bp series? | 15:10 |
sahid | mriedem: yes it is | 15:11 |
PaulMurray | lascii, it will be a lookup | 15:11 |
*** tbachman has quit IRC | 15:11 | |
PaulMurray | lascii, I will also have to do an rpc call to compute node after, but that is an existing call | 15:11 |
PaulMurray | so not worried about that right now | 15:11 |
mriedem | sahid: ok, yeah, i've randomly tried to remember to review that one since i know the first part is merged | 15:11 |
mfedosin | mriedem: great! | 15:11 |
PaulMurray | melwitt, thanks | 15:12 |
lascii | PaulMurray: okay. RPC calls are being converted in https://review.openstack.org/#/c/322181/ | 15:12 |
sahid | mriedem: cool thanks | 15:12 |
mriedem | and i want to get to https://review.openstack.org/#/c/312014/ but have been sidetracked | 15:13 |
sahid | mriedem: yes this one too, i was less scary to be blocked since it's a bug | 15:14 |
lascii | PaulMurray: for db calls I'm not aware of any helper methods yet. You need to use the instance or host info to pull a CellMapping and then use the context.target_cell context manager around the db call | 15:14 |
openstackgerrit | Nicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/331204 | 15:15 |
*** browne has joined #openstack-nova | 15:15 | |
*** mmandel has quit IRC | 15:19 | |
*** sdake has quit IRC | 15:20 | |
*** tbachman has joined #openstack-nova | 15:20 | |
bauwser | lascii: melwitt: PaulMurray: yup, sorry, I was unclear, I was referring to the RPC calls | 15:23 |
*** cdelatte has quit IRC | 15:24 | |
*** mdrabe has quit IRC | 15:24 | |
openstackgerrit | Anusha Unnam proposed openstack/nova: [WIP]Improve help text for compute_opts https://review.openstack.org/329514 | 15:25 |
*** afazekas|dentist is now known as afazekas | 15:25 | |
*** pacoblox has quit IRC | 15:25 | |
*** Kevin-Zhao has quit IRC | 15:26 | |
openstackgerrit | Anusha Unnam proposed openstack/nova: Transform instance.resize notifications https://review.openstack.org/329255 | 15:27 |
*** mgoddard_ has joined #openstack-nova | 15:27 | |
openstackgerrit | Andrew Laski proposed openstack/nova: Add CellMappingList object https://review.openstack.org/329216 | 15:28 |
*** cdelatte has joined #openstack-nova | 15:28 | |
*** rmart04 has quit IRC | 15:29 | |
*** mgoddard has quit IRC | 15:31 | |
*** ijw has joined #openstack-nova | 15:34 | |
*** mriedem has quit IRC | 15:35 | |
*** sree has joined #openstack-nova | 15:36 | |
*** jdillaman has quit IRC | 15:37 | |
openstackgerrit | Julian proposed openstack/nova: WIP Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2) https://review.openstack.org/325435 | 15:37 |
*** mmandel has joined #openstack-nova | 15:37 | |
*** mriedem has joined #openstack-nova | 15:38 | |
*** yamahata has joined #openstack-nova | 15:38 | |
*** belmoreira has quit IRC | 15:40 | |
*** pacoblox has joined #openstack-nova | 15:40 | |
*** tbachman has quit IRC | 15:41 | |
openstackgerrit | Anusha Unnam proposed openstack/nova: Transform instance.resize notifications https://review.openstack.org/329255 | 15:41 |
*** eglynn has quit IRC | 15:41 | |
*** Kevin-Zhao has joined #openstack-nova | 15:44 | |
*** flwang has quit IRC | 15:44 | |
openstackgerrit | Ronald Bradford proposed openstack/nova: api-ref: parameter verification os-fixed-ips https://review.openstack.org/326120 | 15:44 |
*** paul-carlton2 has quit IRC | 15:47 | |
*** flwang has joined #openstack-nova | 15:48 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Log the network when neutron won't apply security groups https://review.openstack.org/331236 | 15:51 |
*** mdrabe has joined #openstack-nova | 15:51 | |
*** sdake has joined #openstack-nova | 15:51 | |
*** matrohon has quit IRC | 15:54 | |
*** sree has quit IRC | 15:55 | |
*** Kevin-Zhao has quit IRC | 15:55 | |
*** Kevin-Zhao has joined #openstack-nova | 15:55 | |
auggy | good morning! | 15:56 |
*** mgoddard_ has quit IRC | 15:59 | |
*** tonytan4ever has joined #openstack-nova | 15:59 | |
*** mgoddard has joined #openstack-nova | 15:59 | |
*** sree has joined #openstack-nova | 15:59 | |
*** sree has quit IRC | 15:59 | |
*** adisky has quit IRC | 16:00 | |
openstackgerrit | Ronald Bradford proposed openstack/nova: api-ref: parameter verification os-agents https://review.openstack.org/326116 | 16:00 |
stewie925 | good morning auggy! | 16:01 |
*** permalac has quit IRC | 16:03 | |
hemna | mriedem, hey | 16:05 |
hemna | mriedem, the systool -c fc_host errors are normal to happen when someone doesn't have a Fibre Channel HBA on their system. | 16:06 |
*** jlanoux has quit IRC | 16:06 | |
*** sdake has quit IRC | 16:07 | |
mriedem | hemna: it logs a trace and warning each time it hits that | 16:07 |
mriedem | there should be a way to suppress that if it's not an error | 16:07 |
mriedem | hemna: and from what i can tell these aren't FC drivers | 16:08 |
mriedem | so not sure why it would be checking that anyway | 16:08 |
*** sridharg has joined #openstack-nova | 16:08 | |
*** tbachman has joined #openstack-nova | 16:08 | |
*** dmk0202 has joined #openstack-nova | 16:09 | |
*** mgoddard has quit IRC | 16:09 | |
*** hoangcx has quit IRC | 16:11 | |
hemna | every time a connector dict is built we have to call each connector object to collect it's info from the system | 16:11 |
hemna | we don't know if we are doing an iSCSI attach or FC attach | 16:11 |
hemna | or RBD, etc. | 16:11 |
*** mdavidson has quit IRC | 16:11 | |
*** jogo has quit IRC | 16:12 | |
hemna | so, I can see if I can redirect stderr to /dev/null to suppress those I guess | 16:12 |
*** tbachman_ has joined #openstack-nova | 16:12 | |
mriedem | hemna: is this because of your connector refactor change? | 16:12 |
hemna | not really | 16:13 |
hemna | the FC connector was always called | 16:13 |
*** pacoblox has quit IRC | 16:13 | |
hemna | since FC was added into libvirt way back | 16:13 |
hemna | privsep is just logging everything now | 16:13 |
openstackgerrit | John Garbutt proposed openstack/nova: Reorder allocate_for_instance preamble https://review.openstack.org/329850 | 16:13 |
hemna | we used to just capture stderr w/ putils and ignore it | 16:13 |
*** cdent has quit IRC | 16:14 | |
mriedem | hmm, privsep doesn't take a flag or something to tell it to be quiet? | 16:14 |
*** tbachman has quit IRC | 16:14 | |
*** tbachman_ is now known as tbachman | 16:14 | |
hemna | I'll look into it | 16:14 |
hemna | I'm not sure, but if it doesn't it should | 16:14 |
hemna | I don't think it should be automatically logging command stderr | 16:14 |
hemna | it should leave it to who calls privsep exec | 16:15 |
*** gjayavelu has quit IRC | 16:15 | |
mriedem | ok, i opened a bug against os-brick, maybe that just gets oslo.privsep added to it | 16:16 |
mriedem | low priority | 16:16 |
hemna | kew | 16:16 |
*** pacoblox has joined #openstack-nova | 16:16 | |
hemna | mriedem, https://github.com/openstack/oslo.privsep/blob/master/oslo_privsep/daemon.py#L285 | 16:17 |
hemna | that's the issue | 16:17 |
hemna | stderr=_fd_logger() | 16:17 |
hemna | it always logs all errors | 16:17 |
hemna | it should capture those and return them to the caller IMHO | 16:18 |
mriedem | ah | 16:18 |
*** sridharg has quit IRC | 16:18 | |
hemna | is gus still up? | 16:19 |
hemna | should I abandon my os-brick patch for the grenade issue ? | 16:19 |
openstackgerrit | John Garbutt proposed openstack/nova: WIP Extract _update_ports_for_instance https://review.openstack.org/329851 | 16:20 |
openstackgerrit | John Garbutt proposed openstack/nova: WIP: Extract port create from allocate_for_instance https://review.openstack.org/328966 | 16:20 |
mriedem | hemna: i think so | 16:20 |
mriedem | i posted comments why in there | 16:20 |
*** edand has joined #openstack-nova | 16:20 | |
hemna | https://review.openstack.org/#/c/329766/ | 16:21 |
hemna | that one ? | 16:21 |
mriedem | no | 16:23 |
mriedem | hemna: this one https://review.openstack.org/#/c/329586/ | 16:23 |
hemna | oh yah, sorry. | 16:23 |
hemna | I'm looking for the privsep patch that fixes the same issue | 16:23 |
openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips https://review.openstack.org/329704 | 16:24 |
*** takedakn has quit IRC | 16:25 | |
mriedem | hemna: that's this https://review.openstack.org/#/c/329766/ | 16:25 |
mriedem | hemna: used by nova here https://review.openstack.org/#/c/329769/ | 16:25 |
hemna | ah ok, so he's setting it to rootwrap from the get go. | 16:26 |
hemna | ok I'll move my os-brick patch to WIP until this stuff lands | 16:26 |
hemna | and then I'll abandon it then. | 16:26 |
mriedem | i'd like to see that pass gate-grenade-dsvm-multinode first, | 16:27 |
mriedem | so i'm going to see if a devstack or devstack-gate change with both as dependencies will make that happen | 16:27 |
mriedem | the nova jobs don't run with the oslo.privsep from source | 16:27 |
mriedem | so the dep doesn't work | 16:27 |
hemna | ok I WIP'd mine | 16:28 |
hemna | just to prevent it from landing for the time being. | 16:28 |
*** Oku_OS is now known as Oku_OS-away | 16:28 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support https://review.openstack.org/318953 | 16:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance https://review.openstack.org/316275 | 16:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add REST API support for get me a network https://review.openstack.org/316398 | 16:29 |
*** Oku_OS-away is now known as Oku_OS | 16:29 | |
*** mriedem is now known as mriedem_lunch | 16:30 | |
hemna | mriedem_lunch, thanks for the help | 16:31 |
auggy | i have a bug related question... i see a lot of cases where someone reports a bug and then assigns it to themselves, sometimes they start work on it, sometimes they don't | 16:33 |
-openstackstatus- NOTICE: zuul was restarted for a software upgrade; events between 16:08 and 16:30 were missed, please recheck any changes uploaded during that time | 16:33 | |
auggy | generally we've been telling them to set the status to "In Progress" and push a patch within 2 weeks | 16:33 |
auggy | but, shouldn't it go through a proper Confirmed/Triage process? | 16:33 |
auggy | or is that necessary? | 16:33 |
auggy | because if they just start work on it, it could be something really major and people miss it because it bypasses the usual filters... or maybe it's not a direction we want to go in, so they'd be wasting their work... | 16:34 |
*** tbachman_ has joined #openstack-nova | 16:35 | |
*** Oku_OS is now known as Oku_OS-away | 16:36 | |
*** Apoorva has joined #openstack-nova | 16:36 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config Options: Improve help text for debugger https://review.openstack.org/311822 | 16:36 |
*** Apoorva has quit IRC | 16:37 | |
*** zenoway has quit IRC | 16:37 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: "nova list-extensions" not showing summary for all https://review.openstack.org/286130 | 16:37 |
*** tbachman has quit IRC | 16:37 | |
*** tbachman_ is now known as tbachman | 16:37 | |
*** smatzek has quit IRC | 16:37 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Improve help text for consoleauth config options https://review.openstack.org/319378 | 16:41 |
*** pushkaru has joined #openstack-nova | 16:47 | |
openstackgerrit | Bob Ball proposed openstack/nova: WIP: XenAPI: Perform disk operations in dom0 https://review.openstack.org/331254 | 16:47 |
*** ijw has quit IRC | 16:49 | |
*** lucasagomes is now known as lucas-afk | 16:51 | |
*** daemontool_ has quit IRC | 16:52 | |
*** harlowja has joined #openstack-nova | 16:53 | |
*** abalutoiu_ has quit IRC | 16:56 | |
*** timello has quit IRC | 16:58 | |
*** pacoblox has quit IRC | 17:02 | |
*** Apoorva has joined #openstack-nova | 17:02 | |
* PaulMurray says goodbye for the weekend | 17:04 | |
*** PaulMurray has quit IRC | 17:05 | |
*** diga has joined #openstack-nova | 17:05 | |
*** sheel has quit IRC | 17:05 | |
*** DaiDV has quit IRC | 17:06 | |
*** irenab has joined #openstack-nova | 17:06 | |
*** sudipto has quit IRC | 17:07 | |
*** pacoblox has joined #openstack-nova | 17:07 | |
*** tonytan4ever has quit IRC | 17:07 | |
*** xyang1 has quit IRC | 17:08 | |
*** xyang1 has joined #openstack-nova | 17:08 | |
*** tblakeslee has quit IRC | 17:09 | |
*** Kevin-Zhao has quit IRC | 17:10 | |
*** irenab has quit IRC | 17:11 | |
*** mvk_ has joined #openstack-nova | 17:11 | |
*** amotoki has quit IRC | 17:13 | |
*** timello has joined #openstack-nova | 17:13 | |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (10) https://review.openstack.org/331013 | 17:13 |
*** mkrcmari__ has quit IRC | 17:15 | |
*** smatzek has joined #openstack-nova | 17:19 | |
*** browne has quit IRC | 17:19 | |
artom | claudiub, hey, I have some time now, if you're up for it :) | 17:20 |
openstackgerrit | John Garbutt proposed openstack/nova: WIP Extract _update_ports_for_instance https://review.openstack.org/329851 | 17:24 |
openstackgerrit | John Garbutt proposed openstack/nova: WIP: Extract port create from allocate_for_instance https://review.openstack.org/328966 | 17:24 |
claudiub | artom: yeay. :) I have a couple of questions about the pci device tagging. There are a couple of things I didn't quite understand. For example, nova boot --nic net-id=12345,tag=nfvfunc1 ... so this basically means that a physical NIC will be attached to the VM, right? How do you know which one to attach, given the tag? | 17:25 |
artom | claudiub, not a physical nic, no, it's the same vnic as would be attached without the tag. | 17:26 |
*** ctrath has quit IRC | 17:27 | |
artom | The tag doesn't identify what vnic to attach, it identifies vnic metadata. | 17:27 |
artom | So in your example, in the livbirt case, whatever device element exists in the domain XML the represent the vnic attached to network 12345 will have some of its attributes exposed in the metadata with that tag | 17:28 |
artom | *that represents | 17:29 |
claudiub | aham. so this basically a way tell the guest which nic is which. | 17:30 |
artom | Right on. | 17:31 |
claudiub | ah, I see. | 17:31 |
*** diga has quit IRC | 17:31 | |
*** ijw has joined #openstack-nova | 17:32 | |
artom | claudiub, yeah, basically if you think DHCP is the devil but you still want applications inside the VM to know which VNIC they should provision with what IP, you would use tags. | 17:34 |
*** irenab has joined #openstack-nova | 17:36 | |
*** edand has quit IRC | 17:36 | |
claudiub | artom: aham.. as far NICs go, what is the role of the virt drivers? I mean, you can probably generate the necessary metadata outside the drivers as well. | 17:37 |
claudiub | artom: as far as the 'type', 'bus', 'mac', and 'tags' go, at least. the address looks like a pci device address. | 17:38 |
artom | claudiub, there's some new metadata that's being exposed to go along with the tag, for example what bus the VNIC is on and what's its address on that bus. Only the virt drivers have this information, as it concerns the virtual hardware. | 17:38 |
artom | claudiub, exactly, bus and address are only known to the virt driver. | 17:39 |
*** cfriesen__ has joined #openstack-nova | 17:39 | |
*** cfriesen_ has quit IRC | 17:39 | |
claudiub | artom: I see. | 17:40 |
*** sdake has joined #openstack-nova | 17:40 | |
*** irenab has quit IRC | 17:41 | |
*** fragatina has joined #openstack-nova | 17:43 | |
*** cfriesen has joined #openstack-nova | 17:43 | |
*** lpetrut has quit IRC | 17:43 | |
claudiub | artom: so, moving on to block device mappings / disks. For the bus type, from the hyper-v perspective, I can easily do that. Not really sure about the address though. | 17:43 |
*** fragatin_ has quit IRC | 17:43 | |
*** cfriesen__ has quit IRC | 17:44 | |
*** sambetts is now known as sambetts|afk | 17:44 | |
claudiub | but from what I see in the libvirt implementation, it isn't set either. | 17:45 |
*** jed56 has quit IRC | 17:45 | |
artom | claudiub, well, we try to find its address | 17:46 |
artom | https://review.openstack.org/#/c/264016/44/nova/virt/libvirt/driver.py 7449 | 17:46 |
artom | But it can be None if it's not feasible | 17:46 |
claudiub | aah, I see. I didn't notice it, as the method is called _prepare_device_bus. :) | 17:47 |
claudiub | ok. I think I got it now. | 17:48 |
claudiub | hopefully. :) | 17:48 |
artom | claudiub, yeah, maybe _prepare_device_bus isn't the clearest indication of what the method is actually doing | 17:49 |
claudiub | I'll probably send a patch for this next week for hyper-v. Will add you as a reviewer. :) | 17:49 |
artom | claudiub, cool, I'll get to put my hyperextensive knowledge of hyperV to the test ;) | 17:49 |
*** kfarr has quit IRC | 17:50 | |
claudiub | artom: weeell, there is 0 or almost 0 hyper-v specific code in nova, so python knowledge will help you the most. :) | 17:50 |
artom | claudiub, sweet, I'll at least be able to fake it then :D | 17:52 |
claudiub | artom: fake it 'til you make it, it is a good policy in life. :D | 17:52 |
*** yamahata has quit IRC | 17:53 | |
*** ctrath has joined #openstack-nova | 17:57 | |
*** tonytan4ever has joined #openstack-nova | 17:57 | |
*** abalutoiu has joined #openstack-nova | 17:57 | |
*** mriedem_lunch is now known as mriedem | 17:59 | |
mriedem | auggy: i wouldn't worry about it too much - they risk wasting their time pushing a fix for what might not be a bug, or they push the wrong fix, and then they find out in review | 17:59 |
mriedem | auggy: if they really think it's important they can ask about it here or in the dev list | 17:59 |
*** fragatina has quit IRC | 18:00 | |
*** fragatina has joined #openstack-nova | 18:01 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: [WIP]Improve help text for cloudpipe https://review.openstack.org/331285 | 18:01 |
*** gjayavelu has joined #openstack-nova | 18:04 | |
*** pacoblox has quit IRC | 18:04 | |
*** harlowja has quit IRC | 18:04 | |
*** tblakeslee has joined #openstack-nova | 18:06 | |
*** browne has joined #openstack-nova | 18:08 | |
*** stewie925 has quit IRC | 18:09 | |
*** irenab has joined #openstack-nova | 18:09 | |
*** stewie925 has joined #openstack-nova | 18:11 | |
*** yassine__ has quit IRC | 18:11 | |
mriedem | sahid: on your use usb tablet thing, what happens if i migrate from one host to another that isn't configured the same? | 18:11 |
*** tbachman has quit IRC | 18:12 | |
*** thorst_ has quit IRC | 18:13 | |
*** irenab has quit IRC | 18:13 | |
*** thorst_ has joined #openstack-nova | 18:14 | |
mriedem | i guess if the image meta properties are telling the libvirt driver what to do then we should be ok | 18:15 |
mriedem | and if not using image meta props, we'd still have the same issue today with the use_usb_tablet object if that's different between hosts | 18:16 |
mriedem | s/object/config/ | 18:16 |
mriedem | seems we shouldn't even have the config option.... | 18:16 |
*** tbachman has joined #openstack-nova | 18:16 | |
*** thorst_ has quit IRC | 18:18 | |
*** thorst_ has joined #openstack-nova | 18:19 | |
*** ssurana has joined #openstack-nova | 18:23 | |
mriedem | superdan: my good friend! https://review.openstack.org/#/c/174854/34/nova/objects/image_meta.py | 18:23 |
mriedem | ImageMetaProps is currently at 1.13 | 18:23 |
*** ctrath has quit IRC | 18:24 | |
mriedem | ImageMeta used to bump versions when ImageMetaProps would, but no longer does | 18:24 |
superdan | right | 18:24 |
mriedem | so that's on purpose? | 18:24 |
superdan | what's on purpose? not bumping it in this patch? | 18:24 |
mriedem | well it looks like we stopped bumping ImageMeta version when ImageMetaProps would change awhile back | 18:24 |
superdan | right, because manifests | 18:25 |
mriedem | i just wasn't sure if that was a conscious decision | 18:25 |
mriedem | ok | 18:25 |
superdan | t'was | 18:25 |
*** mkrcmari__ has joined #openstack-nova | 18:25 | |
*** yamahata has joined #openstack-nova | 18:25 | |
*** ssurana has quit IRC | 18:26 | |
*** ssurana has joined #openstack-nova | 18:27 | |
*** ctrath has joined #openstack-nova | 18:27 | |
*** fragatina has quit IRC | 18:27 | |
*** mvk_ has quit IRC | 18:28 | |
*** mrkz has quit IRC | 18:29 | |
*** irenab has joined #openstack-nova | 18:29 | |
*** Sukhdev has joined #openstack-nova | 18:32 | |
*** irenab has quit IRC | 18:34 | |
*** aghadge has joined #openstack-nova | 18:34 | |
*** ijw has quit IRC | 18:34 | |
*** pushkaru has quit IRC | 18:36 | |
openstackgerrit | melanie witt proposed openstack/nova: Enable sqlite FK enforcement in Database fixture https://review.openstack.org/331305 | 18:39 |
mriedem | melwitt: ooo i've been wanting that made generic for a long time | 18:39 |
openstackgerrit | Chris Friesen proposed openstack/nova: Allow logging detailed info only on sched failure https://review.openstack.org/331306 | 18:40 |
melwitt | yay :) | 18:40 |
*** simondodsley has joined #openstack-nova | 18:40 | |
cfriesen | hi folks...I'd appreciate some comments on this ^. Note, it's only an initial draft to get comments on the overall approach. Unit tests would be added of course. | 18:40 |
*** jdandrea has joined #openstack-nova | 18:41 | |
*** moshele has joined #openstack-nova | 18:45 | |
*** stewie925 has quit IRC | 18:48 | |
jdandrea | Can anyone help shed light on this? If we are using PciPassthroughFilter and NUMATopologyFilter, and we don't specify a NUMA node, will Nova pick a common NUMA node for allocating related VFs and VMs, or is it indeterminate/random? *looking through code* | 18:48 |
openstackgerrit | Ed Leafe proposed openstack/nova: Correct the network config option help text https://review.openstack.org/318726 | 18:50 |
openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (3) https://review.openstack.org/281901 | 18:50 |
openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (4) https://review.openstack.org/282039 | 18:50 |
openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (2) https://review.openstack.org/280438 | 18:50 |
*** locust has joined #openstack-nova | 18:50 | |
locust | hello, as I see I can crete new instances of specified type like mini nano | 18:51 |
locust | will it be posseble to create my own types? | 18:52 |
locust | like create arm emulated mashine | 18:52 |
locust | or change amount of rum belongig to specified mashine? | 18:53 |
*** aysyd has quit IRC | 18:58 | |
*** locust_ has joined #openstack-nova | 18:59 | |
locust_ | so | 18:59 |
locust_ | where there ansver? | 18:59 |
locust_ | sorry was disk | 18:59 |
figleaf | locust_: those are admin-level operations | 19:00 |
*** locust has quit IRC | 19:00 | |
figleaf | locust_: if it's your cloud, then yes, you can do those things | 19:00 |
locust_ | hm | 19:00 |
locust_ | Iam considering openstack like esxi alternative | 19:01 |
*** mrkz has joined #openstack-nova | 19:01 | |
*** moshele has quit IRC | 19:02 | |
locust_ | figleaf: but it is not from horisont like... | 19:02 |
*** aysyd has joined #openstack-nova | 19:03 | |
*** fragatina has joined #openstack-nova | 19:04 | |
figleaf | locust_: the best place to ask questions like that would be in the #openstack channel, not here | 19:05 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 19:06 |
*** efoley has quit IRC | 19:10 | |
*** shakamunyi has joined #openstack-nova | 19:13 | |
*** fragatina has quit IRC | 19:14 | |
*** fragatina has joined #openstack-nova | 19:15 | |
openstackgerrit | Nicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/203751 | 19:18 |
*** zenoway has joined #openstack-nova | 19:20 | |
*** MikeG451_ has quit IRC | 19:20 | |
*** mfedosin has quit IRC | 19:22 | |
*** stewie925 has joined #openstack-nova | 19:24 | |
*** zenoway has quit IRC | 19:25 | |
stewie925 | hello all | 19:25 |
stewie925 | was checking on api ref to work on, found "os-flavor-extra-specs.inc" listed as TO-DO (parameter, example, body) but it looks it has been completed per https://review.openstack.org/#/c/310706/ | 19:26 |
openstackgerrit | Andrew Laski proposed openstack/nova: Cells: Handle delete with BuildRequest https://review.openstack.org/325985 | 19:27 |
mriedem | stewie925: method verification was complete | 19:28 |
mriedem | see the .. needs: part of https://review.openstack.org/#/c/310706/1/api-ref/source/os-flavor-extra-specs.inc | 19:28 |
stewie925 | yes, mriedem - I see that too, but it looks like those other verifications have been done | 19:28 |
stewie925 | well, at least the parameter verificiation was done, but it wasnt stricken out from the source | 19:29 |
*** pacoblox has joined #openstack-nova | 19:30 | |
mriedem | stewie925: i think it was just method verification on that one | 19:30 |
*** pushkaru has joined #openstack-nova | 19:31 | |
stewie925 | mriedem: I see.... | 19:31 |
*** harlowja has joined #openstack-nova | 19:32 | |
mriedem | superdan: lascii: so i've got a riddle for you | 19:32 |
mriedem | since sdague is on a walkabout | 19:32 |
*** thumpba has joined #openstack-nova | 19:32 | |
mriedem | https://review.openstack.org/#/c/326940/ has duplicated the API samples for os-hypervisors simply because the v2.31 microversion in that change has subclassed the API samples for os-hypervisors, | 19:33 |
mriedem | but the only thing that changes in that change is adding paging support for listing hypervisors, | 19:33 |
mriedem | so it seems pointless to test GET on os-hypervisors with 2.31, | 19:33 |
mriedem | unless that's just purely for regression testing | 19:33 |
*** Jeffrey4l has quit IRC | 19:34 | |
*** ociuhandu has quit IRC | 19:34 | |
mriedem | note that the API samples tests don't even mean anything since those only verify the request/response body, and this is handling limit/marker in the query | 19:34 |
superdan | yeah | 19:34 |
*** dimtruck is now known as zz_dimtruck | 19:34 | |
mriedem | the REST API unit test actually tests the limit/marker part | 19:35 |
*** ijw has joined #openstack-nova | 19:35 | |
mriedem | otherwise we're just duplicating and running a bunch of tests that work the same as beofre | 19:35 |
mriedem | *before | 19:35 |
mriedem | i have a feeling this might just be 'how it's done' | 19:36 |
superdan | so what are you asking? | 19:37 |
superdan | if we should have an exclusion list? | 19:37 |
mriedem | well, for api samples tests, should we only test listing hypervisors (what actually changed) | 19:37 |
mriedem | even though that api sample test doesn't test the change really since the body isn't changing | 19:38 |
mriedem | same for the unit tests since those are also just subclassed and pass a different microversion | 19:38 |
*** ssurana has quit IRC | 19:38 | |
mriedem | i'm just wary of subclassing tests because of all of the unnecessary duplication | 19:39 |
mriedem | maybe this is a ML question? | 19:39 |
*** tonytan4ever has quit IRC | 19:39 | |
*** ijw has quit IRC | 19:40 | |
superdan | I'm not sure I'm following what you're saying | 19:40 |
superdan | but we have an exclusion list of db migrations that we don't require tests for, | 19:40 |
superdan | like the placeholder ones | 19:40 |
*** cdelatte has quit IRC | 19:40 | |
mriedem | i think we're talking about different things | 19:41 |
mriedem | i'll just post to the dev list and sean can enlighten me when he's back | 19:42 |
mriedem | there are other problems in this change that have to be addressed until then anyway | 19:42 |
*** jichen has quit IRC | 19:45 | |
melwitt | zzzeek: hi, I was wondering if you could suggest a workaround for this problem https://bugs.launchpad.net/sqlalchemy-migrate/+bug/1552351 I'm trying to enable fk constraint enforcement in sqlite for unit/functional tests | 19:46 |
openstack | Launchpad bug 1552351 in sqlalchemy-migrate "sqlite alter table messes foreign keys" [Undecided,New] | 19:46 |
*** mriedem has quit IRC | 19:46 | |
zzzeek | melwitt: during a migration or during tests? | 19:47 |
*** Jeffrey4l has joined #openstack-nova | 19:47 | |
melwitt | zzzeek: during tests, but we have some tests that do migrations, it got tripped up there http://logs.openstack.org/05/331305/1/check/gate-nova-python27-db/e6300b6/console.html#_2016-06-17_19_06_48_843531 | 19:47 |
*** thumpba has quit IRC | 19:48 | |
melwitt | I wanted to check with you if there's any other option other than turning them off for migration tests | 19:48 |
*** samueldmq has quit IRC | 19:49 | |
zzzeek | melwitt: not likely. however is this a side effect of sqlalchemy-migrate's copy-and-replace scheme for SQLite tables ? | 19:49 |
*** Apoorva has quit IRC | 19:49 | |
zzzeek | i dont understand what "will also rename foreign key contraints from *other* tables to migration_tmp" means in the bz | 19:50 |
zzzeek | that's a rename of a table | 19:50 |
zzzeek | melwitt: for alembic, we require that pragma FKs is turned off for SQLite migrations. so i'd stick w/ that | 19:51 |
melwitt | zzzeek: yeah, I don't really understand it, just that it sounded like the same issue in that this test is doing a table alter and it was fine before I enabled fk | 19:51 |
melwitt | zzzeek: okay, thank you! | 19:52 |
zzzeek | melwitt: SQLite's developers are actively opposed to the idea of schema migrations and all of our tools are just badly trying to work aroudn it. ideally nova and no other openstack project would use migrations for sqlite during tests, just create_all | 19:52 |
*** lpetrut has joined #openstack-nova | 19:52 | |
*** mriedem has joined #openstack-nova | 19:53 | |
stewie925 | hi all - am wondering how we can determine which Nova command invokes a Nova API rest method? | 19:53 |
melwitt | zzzeek: okay, good to know | 19:53 |
mriedem | i've officially decided to quit and take up turnip farming | 19:53 |
mriedem | stewie925: nova --debug ... | 19:53 |
mtreinish | mriedem: why turnips? Do they grow well in rochester? | 19:54 |
*** ijw has joined #openstack-nova | 19:55 | |
*** markvoelker has quit IRC | 19:55 | |
*** raildo is now known as raildo-afk | 19:57 | |
stewie925 | thank you mriedem | 19:58 |
mriedem | mtreinish: because i imagine turnip farming has been perfected over thousands of years | 19:58 |
mriedem | so i don't have to ask wtf? | 19:59 |
mriedem | it's just, this is how you do turnips | 19:59 |
*** rama_y has quit IRC | 19:59 | |
cfriesen | so I just noticed something interesting....we do not allow a single instance to overcommit CPU or RAM against itself, but we do allow it to overcommit disk. Does that make sense? | 20:00 |
doffm | I'm sure I've seen a bug for this. | 20:06 |
doffm | *goes searches for a bug. | 20:06 |
*** tbachman has quit IRC | 20:07 | |
melwitt | cfriesen: have you seen this https://review.openstack.org/#/c/242251/ | 20:07 |
*** thumpba has joined #openstack-nova | 20:07 | |
*** rama_y has joined #openstack-nova | 20:07 | |
*** tbachman has joined #openstack-nova | 20:08 | |
*** zz_dimtruck is now known as dimtruck | 20:08 | |
*** markvoelker has joined #openstack-nova | 20:10 | |
*** rook-lappy has quit IRC | 20:14 | |
*** rook-lappy has joined #openstack-nova | 20:14 | |
cfriesen | melwitt: no, but isn't that the whole point of overcommit, that we allow committing more than the total available free space? Following the example of the RAM filter we would only disallow it if the instance asked for more than the total disk space | 20:17 |
*** akshai has quit IRC | 20:18 | |
*** Apoorva has joined #openstack-nova | 20:18 | |
cfriesen | melwitt: but it's definitely related to the issue...will take a closer look | 20:18 |
openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the linuxnet options (4) https://review.openstack.org/275423 | 20:18 |
mriedem | cfriesen: i'm assuming the disk overcommit gets a pass b/c of the failure to accurately report shared storage? | 20:18 |
melwitt | cfriesen: yeah, it took me awhile to understand the patch and how it works with overcommit | 20:18 |
mriedem | cfriesen: jaypipes would probably know about it | 20:19 |
*** thumpba has quit IRC | 20:19 | |
*** thumpba has joined #openstack-nova | 20:20 | |
cfriesen | mriedem: if we did a simple check analogous to the RAMFilter, it'd just be "don't let a single instance take more than the total storage space on this hypervisor, regardless of overcommit" | 20:20 |
cfriesen | The patch melwitt pointed at is looking at host_state.free_disk_physical_mb, not the total available storage | 20:20 |
cfriesen | so it's more strict | 20:20 |
melwitt | cfriesen: what do you mean by total available storage, total available on the hypervisor? | 20:21 |
cfriesen | melwitt. yes, total storage available for instance root disk, ephemeral, swap, etc. | 20:22 |
*** ihrachys has joined #openstack-nova | 20:22 | |
*** irenab has joined #openstack-nova | 20:22 | |
melwitt | I thought that's what the patch is doing, adding a new attribute to track that because before, it was taking the min() between that and a calculated storage value that subtracts overcommit | 20:22 |
cfriesen | melwitt: I just pointed out what appears to be a mismatch between comment and code in that patch | 20:23 |
*** thorst_ has quit IRC | 20:23 | |
cfriesen | melwitt: it looks like he's using compute.disk_available_least, which I think is how much space would be available if all the instances used all the space they were allocated. Which I think sort of defeats the purpose of the overcommit ratio, no? | 20:24 |
doffm | cfriesen: Seems to. | 20:24 |
melwitt | andrewbogott: are you around? | 20:24 |
andrewbogott | melwitt: I am! What's up? | 20:25 |
doffm | cfriesen: This seems wrong. Free disk mb < request. | 20:25 |
melwitt | okay, I thought disk_available_least was the actual space used, which could be more than what the flavors disk total is with COW | 20:26 |
doffm | Blocks all overcommit. | 20:26 |
* andrewbogott reads backscroll | 20:26 | |
*** thumpba_ has joined #openstack-nova | 20:26 | |
melwitt | actual space available, I mean | 20:26 |
*** irenab has quit IRC | 20:27 | |
andrewbogott | It's been a few months since I thought about that patch… my intent was certainly to check the physical available space | 20:28 |
melwitt | the names of existing variables are really confusing, IMO | 20:29 |
cfriesen | I think you could just compare against the existing "total_usable_disk_gb" | 20:29 |
*** thumpba has quit IRC | 20:29 | |
doffm | Which is what the ram filter does. | 20:29 |
openstackgerrit | Ed Leafe proposed openstack/nova: WIP - Accept multiple hosts for live migration https://review.openstack.org/327809 | 20:29 |
melwitt | total usable includes used though | 20:29 |
cfriesen | melwitt: yes | 20:29 |
doffm | They are totally confusing. I mean free available totally used free. | 20:30 |
cfriesen | melwitt: it does for the RAM FIlter too | 20:30 |
cfriesen | melwitt: the intent of the check is to prevent an instance from overcomitting against itself. once you start looking at resources consumed by other instances then you're looking at overcommit relative to other instances, which is a different issue | 20:30 |
melwitt | okay ... I guess I was thinking if usable includes used then it will be as though the entire hypervisor is free, which wouldn't solve the problem of scheduling to a full host | 20:31 |
cfriesen | so there should be several checks: 1) is the instance bigger than the host can physically provide, if so then disallow it. 2) does the instance fit once factoring in overcommit ratios and free space | 20:31 |
doffm | So if you have 16gb of ram total you cant put a 32 gb instance on it even with a 2x overcommit. | 20:31 |
doffm | But you can put 3 8gb instances. | 20:31 |
*** ssurana has joined #openstack-nova | 20:32 | |
cfriesen | doffm: exactly | 20:32 |
*** aghadge has quit IRC | 20:34 | |
*** mvk_ has joined #openstack-nova | 20:34 | |
melwitt | cfriesen, doffm: so this value you're saying is instances using all the space? I didn't see how https://github.com/openstack/nova/blob/master/nova/virt/libvirt/utils.py#L410 | 20:37 |
*** ssurana has quit IRC | 20:37 | |
*** mkrcmari__ has quit IRC | 20:38 | |
*** figleaf is now known as edleafe | 20:38 | |
*** ihrachys has quit IRC | 20:38 | |
melwitt | it is subtracting overcommitted for available_least https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5199 | 20:39 |
melwitt | sigh ... need to go back through all of this | 20:40 |
andrewbogott | Older versions of my patch just used compute.free_disk_gb | 20:40 |
andrewbogott | Is it possible that this confusion is just due to a sloppy merge on my part? | 20:41 |
andrewbogott | https://review.openstack.org/#/c/242251/5/nova/scheduler/host_manager.py | 20:41 |
andrewbogott | Or is compute.free_disk_gb /also/ not the value I want? | 20:41 |
*** edleafe has quit IRC | 20:41 | |
cfriesen | andrewbogott: to be analogous to the RAMFilter and CPUfilter you should check whether the instance is trying to get more than the _total_ resource, not the free amount | 20:41 |
melwitt | andrewbogott: no, I think jaypipes asked you to change it to available_least IIRC | 20:41 |
*** edleafe has joined #openstack-nova | 20:42 | |
cfriesen | andrewbogott: I think it should be: if host_state.total_usable_disk_gb < requested_disk: | 20:42 |
cfriesen | any checks against the "free" space are not about overcommit against itself, which is what the comment talks about | 20:43 |
melwitt | cfriesen: I think that's like you said, the first step. he's trying to solve a host that began as a candidate when all the space was free but now isn't because of taken up space | 20:43 |
andrewbogott | melwitt: here's a bit of history: I originally wrote a patch that let me specify minimum % free to prevent the scheduler. Basically "You can overprovision but if you hit xx% full then for god's sake stop" | 20:43 |
cfriesen | melwitt: right, that's a separate issue from overcommit against itself. | 20:43 |
*** pushkaru has quit IRC | 20:44 | |
andrewbogott | Someone (I don't remember who) pointed out that the other filters (e.g. RAM) already did that but rather than allowing a user-specified ratio they effectively capped at "one more full instance" | 20:44 |
*** tblakeslee has quit IRC | 20:44 | |
*** inc0 has joined #openstack-nova | 20:44 | |
andrewbogott | Which fixed my particular concern, more or less, so I tried to mirror the RAM filter code | 20:44 |
andrewbogott | From my point of view, it's just drawing an arbitary line: "This host is pretty much full so stop scheduling, even if the commit ratio gets you good numbers" | 20:45 |
cfriesen | andrewbogott: that % full check *should* be the disk overcommit ratio, as long as those calculations are correct. (which for a long time they were not) | 20:45 |
andrewbogott | I don't know if that clarifies | 20:45 |
*** tblakeslee has joined #openstack-nova | 20:45 | |
doffm | andrewbogott: Yeah, that is not really what the checks in the core and ram filters are doing. | 20:45 |
doffm | If you look later in the disk filter function the 'if not usable_disk_mb >= requested_disk:' check happens. | 20:46 |
melwitt | okay, so none of the filters do a check for literal free resources? | 20:46 |
cfriesen | melwitt: that's later...separate from the "overcommit against itself" check | 20:46 |
mriedem | shite | 20:46 |
*** fragatina has quit IRC | 20:47 | |
mriedem | the massive config option centralization thing has it's foibals | 20:47 |
andrewbogott | melwitt: my experience was that they did not. So that a host could have a 100% full physical disk, but if the overcommit ratio said it wasn't committed for 140% it would go ahead and schedule | 20:47 |
melwitt | cfriesen: so how does it schedule to a full host if it ever checks the literally free space? | 20:47 |
*** MikeG451_ has joined #openstack-nova | 20:47 | |
mriedem | we probably should have had a job that listed deprecated options before and after a change to make sure the diff didn't change | 20:47 |
mriedem | https://review.openstack.org/#/c/309239/6/nova/conf/libvirt.py@60 | 20:47 |
lascii | mriedem: I was out for your earlier riddle re: sample tests, but I responded on the ML | 20:47 |
auggy | mriedem: mtreinish: if you started with beets and switched to turnips, you could say would have to turnip the beets | 20:48 |
*** thorst_ has joined #openstack-nova | 20:49 | |
mtreinish | auggy: haha | 20:49 |
*** jamesdenton has quit IRC | 20:49 | |
cfriesen | melwitt: I think there are still holes in the disk filter checks (we had some patches against kilo, haven't checked yet if they're needed on mitaka). But the "don't overcommit against yourself" should be checking the total disk space, not the free disk space. Later on we should check if there's room for this instance after factoring in used space and overcommit and such. | 20:50 |
openstackgerrit | Nicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd https://review.openstack.org/331350 | 20:50 |
mriedem | auggy: as in, turn up the beats? | 20:51 |
mriedem | auggy: that is not something i would ever say | 20:51 |
andrewbogott | To clarify, we should check if there's room for this instance WITHOUT factoring in overcommit | 20:51 |
auggy | but you *could* | 20:51 |
cfriesen | andrewbobott: for the "don't overcommit against yourself" test, yes. | 20:51 |
mriedem | auggy: never | 20:51 |
auggy | or if your beats were particularly large, you could say you have fat beets | 20:51 |
mriedem | auggy: i might say, turn up the jams | 20:52 |
auggy | ooh beat jam | 20:52 |
auggy | beet jam rather | 20:52 |
auggy | i wonder if turnip jam is a thing | 20:52 |
cfriesen | andrewbogott: (sorry, can't spell). Later on we'd probably *also* want to check against the available space, factoring in overcommit, space stored by qcow backing store files, etc. | 20:52 |
*** fragatina has joined #openstack-nova | 20:53 | |
andrewbogott | cfriesen: I'm sorry, I'm not following. | 20:53 |
andrewbogott | Can you give me an example? | 20:53 |
andrewbogott | Or, actually — here's a different question | 20:54 |
andrewbogott | Suppose a host's drive is literally, physically full | 20:54 |
andrewbogott | but the overcommit ration still says there's space | 20:54 |
andrewbogott | Where is the code that prevents us from scheduling on that box? | 20:54 |
doffm | andrewbogott: There isn't any. | 20:54 |
doffm | You schedule to it. | 20:54 |
andrewbogott | ok! | 20:54 |
*** mkrcmari__ has joined #openstack-nova | 20:54 | |
andrewbogott | And, do we agree that that's a bad thing? | 20:54 |
doffm | That is what your overcommit tells you to do. | 20:54 |
andrewbogott | So that's a "not agree"? | 20:55 |
doffm | No I agree. | 20:55 |
doffm | Disk overcommit is different to RAM and vCPU. | 20:55 |
doffm | Go over limits there and there isn't really a problem. | 20:56 |
doffm | Go over actual disk limits and the build will fail. | 20:56 |
melwitt | right | 20:56 |
andrewbogott | ok, great, happy to hear that :) | 20:56 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Re-deprecate use_usb_tablet config option https://review.openstack.org/331353 | 20:56 |
doffm | So there should be THREE checks in this file. | 20:57 |
andrewbogott | So… (I'm going back to the beginning here, sorry) | 20:57 |
melwitt | disk should have extra stuff in it bc of that | 20:57 |
andrewbogott | What should we consider 'too full for more scheduling'? | 20:57 |
*** fragatina has quit IRC | 20:57 | |
mriedem | lascii: 20 seconds per test or just the test class itself | 20:57 |
*** mvk_ has quit IRC | 20:57 | |
andrewbogott | One option is: we need at least (requested size / overcommit size) | 20:58 |
lascii | mriedem: the test class. 8 tests | 20:58 |
andrewbogott | another option is we need at least (requested size) | 20:58 |
melwitt | but what I'm thinking is, if we need to check actually available space in the end, how is the "don't overcommit against yourself" total usable space check useful? | 20:58 |
mriedem | lascii: ok | 20:58 |
doffm | Not the instance disk size. Imagine you have an instance that has a requested disk of 400gb. | 20:58 |
mriedem | not as bad then | 20:58 |
andrewbogott | another is we need at least (user-specified %) | 20:58 |
doffm | And you only have 200gb actually free. | 20:58 |
doffm | You should probably still schedule there. If you have a big overcommit. | 20:58 |
mriedem | lascii: superdan: also, we should get this in, super easy https://review.openstack.org/331353 | 20:58 |
mriedem | so we don't lose the removal window | 20:58 |
andrewbogott | ok… so you're voting for (requested size / overcommit size) | 20:58 |
doffm | I guess i am. | 20:59 |
doffm | But I haven't thought about it too much. :) Maybe just a configurable limit. | 20:59 |
andrewbogott | (For my purposes, that sounds like living on the edge, basically setting up a host to immediately crumble as soon as any instances grow. But your argument is nonetheless logically sound :) ) | 20:59 |
superdan | mriedem: why was it undeprecated? | 20:59 |
andrewbogott | doffm: Hah! A configurable limit was what I request in the original bug | 20:59 |
doffm | Oh. :) | 20:59 |
*** smatzek has quit IRC | 21:00 | |
andrewbogott | lemme see if I can dig up that patch | 21:00 |
*** ivase has quit IRC | 21:01 | |
*** mvk has joined #openstack-nova | 21:01 | |
*** ivase has joined #openstack-nova | 21:01 | |
lascii | superdan: it seems it was accidentally undeprecated when the option moved to a central location | 21:01 |
*** mkrcmari__ has quit IRC | 21:02 | |
*** ssurana has joined #openstack-nova | 21:02 | |
superdan | lascii: ah | 21:02 |
*** davidlenwell has quit IRC | 21:02 | |
andrewbogott | doffm: so, check out patchset 4, and comments there: https://review.openstack.org/#/c/242251/4 | 21:03 |
*** ssurana has quit IRC | 21:04 | |
openstackgerrit | melanie witt proposed openstack/nova: WIP Make InstanceGroup object favor the API database https://review.openstack.org/330995 | 21:04 |
mriedem | superdan: goof up | 21:05 |
superdan | aye | 21:05 |
doffm | andrewbogott: Ok, will review. | 21:05 |
openstackgerrit | Diana Clarke proposed openstack/nova: Image: add create_from_image & create_from_func https://review.openstack.org/318948 | 21:06 |
*** ivase has quit IRC | 21:06 | |
andrewbogott | doffm: of course, I may still be checking the wrong value in that patch, since… a dozen variables with indistiguishable names, etc. | 21:06 |
*** burgerk has quit IRC | 21:06 | |
*** zenoway has joined #openstack-nova | 21:07 | |
*** ivase has joined #openstack-nova | 21:07 | |
cfriesen | andrewbogott: So I see a number of checks: 1) is the instance asking for more space than physically exists. (this is the "don't overcommit against yourself" check) | 21:08 |
andrewbogott | cfriesen: by 'exists' you mean, total disk size, not free space, correct? | 21:09 |
cfriesen | andrewbogott: yes. 2) is there enough space to run the instance given the current image size in the free space available, assuming the instance doesn't grow | 21:09 |
cfriesen | and 3) once we factor in the disk overcommit ratio, is there enough space for this instance and all the other instances to fit in the disk | 21:10 |
melwitt | cfriesen: the second one is tricky though because you could probably still run the instance even if you don't have free space > image size in a lot of cases | 21:10 |
andrewbogott | cfriesen: how would 2) pass but 3) fail? | 21:10 |
andrewbogott | Would it be with an overcommit ration of <1.0? | 21:10 |
cfriesen | melwitt: if you don't already have the backing file on that hypervisor then you're going to have to download it, and it's going to consume space equal to the image size | 21:11 |
*** ivase has quit IRC | 21:11 | |
cfriesen | andrewbogott: if you already have the backing file downloaded, then the incremental cost of an instance is basically nothing until it starts writing to its disk | 21:11 |
melwitt | cfriesen: okay, I thought that meant the requested disk size. you mean only the image size | 21:11 |
doffm | Jeez, working out what the 'least' value means in the libvirt code is not fun. | 21:11 |
cfriesen | andrewbogott: assuming qcow2 here | 21:12 |
andrewbogott | cfriesen: ok, I see what you're saying. So 2) should really check the image size against available disk space | 21:12 |
*** daemontool has joined #openstack-nova | 21:12 | |
melwitt | doffm: tell me about it! | 21:12 |
andrewbogott | (this, presuming we have any idea what available disk space is) | 21:12 |
*** thorst_ has quit IRC | 21:12 | |
*** thumpba_ has quit IRC | 21:12 | |
andrewbogott | or does it need to be smarter, determine w/not the image is already present? | 21:13 |
cfriesen | andrewbogott: yes, and ideally for qcow2 we should check whether we've already got the backing file cached or not | 21:13 |
*** davidlenwell has joined #openstack-nova | 21:13 | |
*** ametts has quit IRC | 21:13 | |
*** pacoblox has quit IRC | 21:13 | |
*** ivase has joined #openstack-nova | 21:13 | |
andrewbogott | cfriesen: ok, I follow what you're saying now | 21:13 |
*** pacoblox has joined #openstack-nova | 21:13 | |
cfriesen | the theoretical max size consumed by N qcow instances is sum(unique backing files) + sum(instance flavor disks) | 21:14 |
andrewbogott | cfriesen: your scheme doesn't allow for slack of any kind, but maybe that's an unrelated issue? | 21:14 |
cfriesen | andrewbogott: thats where the ratio comes in. you want more slack, lower the ratio | 21:14 |
andrewbogott | so... | 21:14 |
cfriesen | or maybe you need a separate "slack" config option | 21:14 |
cfriesen | :) | 21:14 |
cfriesen | oh, and just to make things interesting, for some reason we have a backing store file for swap disks. | 21:15 |
*** stewie925 has quit IRC | 21:16 | |
andrewbogott | So, here's my gut reaction: Improving our scheduling math so that we hit 100% capacity on the nose is not a useful goal. | 21:16 |
doffm | Ok, so the least value is saying "How much space would we have free if all servers suddenly took up all their allocated disk space". | 21:16 |
andrewbogott | 100% capacity + overprovision > 1.0 = immediate disaster | 21:16 |
cfriesen | doffm: that's my understanding | 21:16 |
doffm | Which means cfriesen is probably right and the patch is currently blocking overcommit. | 21:16 |
andrewbogott | cfriesen: So, you've outlined a useful way to hit 100% on the mark, but that's not a thing we want to do | 21:16 |
andrewbogott | (as I understand it, if overcommit == 1.0 then the scheduler already works properly, hits 100% on the mark, and everyone is happy) | 21:17 |
*** ivase has quit IRC | 21:17 | |
cfriesen | andrewbogott: so add in a "slack" config option. We currently run with no overcommit, and don't ever want to run out of disk. We had to patch kilo to properly handle things, haven't checked Mitaka yet. | 21:18 |
andrewbogott | cfriesen: So, I propose that I revive the early version of my patch (explicit slack) | 21:19 |
andrewbogott | https://review.openstack.org/#/c/242251/4 | 21:19 |
andrewbogott | and create a different bug about scheduling precision | 21:20 |
cfriesen | andrewbogott: I think your current patch has some usefulness too, just change it to look at the total disk space instead of free. | 21:20 |
doffm | cfriesen: +1 | 21:20 |
andrewbogott | cfriesen: ok — at which point it will be a first approximation of goal 2) ? | 21:20 |
*** ssurana has joined #openstack-nova | 21:21 | |
doffm | andrewbogott: Yeah, the issue is really the use of the 'least' rather than 'really_free' | 21:21 |
cfriesen | no, goal 1 | 21:21 |
andrewbogott | Ah! Yes, ok, I remember 1 | 21:21 |
openstackgerrit | Rawlin Peters proposed openstack/os-vif: Ensure the OVS bridge exists when plugging https://review.openstack.org/330818 | 21:21 |
andrewbogott | so I will be accidentally fixing a problem that never occurred to me, I can live with that :) | 21:21 |
andrewbogott | stay tuned... | 21:21 |
cfriesen | :) | 21:21 |
*** markvoelker has quit IRC | 21:21 | |
*** rtheis has quit IRC | 21:22 | |
cfriesen | andrewbogott: that will make it be exactly analogous to the "don't overcommit against yourself" checks in the cpu/ram filters | 21:22 |
* andrewbogott nods | 21:22 | |
*** stewie925 has joined #openstack-nova | 21:22 | |
*** ijw has quit IRC | 21:23 | |
*** ijw has joined #openstack-nova | 21:23 | |
*** irenab has joined #openstack-nova | 21:24 | |
doffm | Yay! | 21:24 |
*** ssurana has quit IRC | 21:25 | |
*** ssurana has joined #openstack-nova | 21:25 | |
*** dave-mccowan has quit IRC | 21:26 | |
*** tbachman_ has joined #openstack-nova | 21:26 | |
*** lascii is now known as alaski | 21:28 | |
*** irenab has quit IRC | 21:28 | |
*** tbachman has quit IRC | 21:29 | |
*** tbachman_ is now known as tbachman | 21:29 | |
openstackgerrit | Rawlin Peters proposed openstack/os-vif: Ensure the OVS bridge exists when plugging https://review.openstack.org/330818 | 21:30 |
*** rfolco has quit IRC | 21:32 | |
*** dmk0202 has quit IRC | 21:33 | |
*** dmk0202 has joined #openstack-nova | 21:34 | |
*** thorst has joined #openstack-nova | 21:36 | |
*** mdrabe has quit IRC | 21:36 | |
*** erlon has quit IRC | 21:36 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: Rename libvirt conf entry iscsi_use_multipath https://review.openstack.org/331357 | 21:38 |
*** pacoblox has quit IRC | 21:39 | |
*** thorst has quit IRC | 21:39 | |
*** thorst has joined #openstack-nova | 21:40 | |
mriedem | what's up with this? https://github.com/openstack/nova/blob/master/setup.cfg#L41 | 21:40 |
mriedem | i guess vestigial? https://review.openstack.org/#/c/240159/3/setup.cfg | 21:41 |
mriedem | didn't we already drop all of the ERT stuff? | 21:42 |
superdan | probably vestigal | 21:43 |
superdan | like your tailbone | 21:43 |
*** sahid has quit IRC | 21:44 | |
mriedem | i still use mine | 21:45 |
*** pacoblox has joined #openstack-nova | 21:45 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Remove the nova.compute.resources entrypoint https://review.openstack.org/331358 | 21:46 |
*** tblakeslee has quit IRC | 21:47 | |
*** xyang1 has quit IRC | 21:48 | |
*** thorst has quit IRC | 21:49 | |
*** dmk0202 has quit IRC | 21:50 | |
*** pacoblox has quit IRC | 21:51 | |
*** daemontool has quit IRC | 21:53 | |
*** pwnall has quit IRC | 21:56 | |
*** pacoblox has joined #openstack-nova | 21:57 | |
*** ctrath has quit IRC | 21:58 | |
*** mriedem has quit IRC | 21:59 | |
*** pacoblox has quit IRC | 22:01 | |
*** pacoblox has joined #openstack-nova | 22:02 | |
*** kylek3h has quit IRC | 22:02 | |
*** lpetrut has quit IRC | 22:04 | |
*** pacoblox has quit IRC | 22:05 | |
openstackgerrit | Andrew Bogott proposed openstack/nova: Prevent scheduling of instances that can't possibly fit https://review.openstack.org/242251 | 22:05 |
*** pacoblox has joined #openstack-nova | 22:06 | |
andrewbogott | cfriesen, doffm, melwitt: I have to run, but ^ should implement the against-yourself scheduling issue. I had to tweak a couple of out-of-band tests and I'm not entirely sure I didn't render them incorrect. | 22:06 |
* andrewbogott runs | 22:06 | |
*** ijw has quit IRC | 22:07 | |
*** mmandel has quit IRC | 22:08 | |
*** pushkaru has joined #openstack-nova | 22:10 | |
openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for compute_opts https://review.openstack.org/329514 | 22:11 |
*** baoli has quit IRC | 22:13 | |
*** dimtruck is now known as zz_dimtruck | 22:13 | |
*** pradk has quit IRC | 22:13 | |
*** burt has quit IRC | 22:13 | |
*** ijw has joined #openstack-nova | 22:13 | |
*** fragatina has joined #openstack-nova | 22:14 | |
*** fragatin_ has joined #openstack-nova | 22:15 | |
*** efried has quit IRC | 22:17 | |
*** zz_dimtruck is now known as dimtruck | 22:18 | |
*** fragatina has quit IRC | 22:19 | |
*** dimtruck is now known as zz_dimtruck | 22:20 | |
*** rahuls has quit IRC | 22:24 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 22:32 |
*** abalutoiu has quit IRC | 22:37 | |
*** sdake has quit IRC | 22:38 | |
*** mvk_ has joined #openstack-nova | 22:39 | |
*** sdake has joined #openstack-nova | 22:41 | |
*** mvk has quit IRC | 22:42 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Transform instance.suspend notifications https://review.openstack.org/329089 | 22:44 |
*** thorst has joined #openstack-nova | 22:45 | |
*** aysyd has quit IRC | 22:45 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys https://review.openstack.org/329697 | 22:45 |
*** MikeG451_ has quit IRC | 22:45 | |
*** syjulian has quit IRC | 22:48 | |
*** pacoblox has quit IRC | 22:48 | |
*** pacoblox has joined #openstack-nova | 22:48 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Transform instance.suspend notifications https://review.openstack.org/329089 | 22:50 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 22:51 |
*** pacoblox has quit IRC | 22:51 | |
*** thorst has quit IRC | 22:53 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Transform instance.suspend notifications https://review.openstack.org/329089 | 22:53 |
*** zenoway has quit IRC | 22:55 | |
*** zenoway has joined #openstack-nova | 22:56 | |
*** pacoblox has joined #openstack-nova | 22:56 | |
*** edmondsw has quit IRC | 22:59 | |
*** mrkz has quit IRC | 23:00 | |
*** zenoway has quit IRC | 23:01 | |
*** rfolco has joined #openstack-nova | 23:05 | |
*** ctrath has joined #openstack-nova | 23:06 | |
*** rama_y has quit IRC | 23:07 | |
*** rama_y has joined #openstack-nova | 23:08 | |
*** zenoway has joined #openstack-nova | 23:08 | |
*** ssurana1 has joined #openstack-nova | 23:12 | |
*** sacharya_ has quit IRC | 23:13 | |
*** rrecio_ has quit IRC | 23:13 | |
*** ssurana has quit IRC | 23:15 | |
*** markvoelker has joined #openstack-nova | 23:16 | |
*** omnipresent has joined #openstack-nova | 23:17 | |
*** stewie925 has quit IRC | 23:17 | |
*** zenoway has quit IRC | 23:20 | |
*** fragatin_ has quit IRC | 23:21 | |
*** dave-mccowan has joined #openstack-nova | 23:22 | |
*** rfolco has quit IRC | 23:24 | |
*** ssurana1 has quit IRC | 23:25 | |
*** fragatina has joined #openstack-nova | 23:25 | |
*** dave-mccowan has quit IRC | 23:26 | |
*** fragatina has quit IRC | 23:26 | |
*** fragatina has joined #openstack-nova | 23:26 | |
*** pushkaru has quit IRC | 23:37 | |
*** claudiub has quit IRC | 23:38 | |
*** yamahata has quit IRC | 23:45 | |
*** eharney has quit IRC | 23:46 | |
*** pushkaru has joined #openstack-nova | 23:47 | |
*** zz_dimtruck is now known as dimtruck | 23:50 | |
*** thorst has joined #openstack-nova | 23:51 | |
*** rrecio has joined #openstack-nova | 23:53 | |
*** rrecio_ has joined #openstack-nova | 23:55 | |
*** fragatina has quit IRC | 23:55 | |
*** Sukhdev has quit IRC | 23:55 | |
*** rrecio has quit IRC | 23:58 | |
*** thorst has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!