Friday, 2016-06-17

*** clenimar_ has quit IRC00:06
*** zenoway has joined #openstack-nova00:12
*** pacoblox has quit IRC00:15
*** zenoway has quit IRC00:16
*** amotoki has quit IRC00:17
*** mrkz has joined #openstack-nova00:21
*** mgoddard has joined #openstack-nova00:22
*** hoangcx has joined #openstack-nova00:22
*** jamesdenton has quit IRC00:24
*** mgoddard has quit IRC00:27
*** zul has joined #openstack-nova00:28
*** ssurana has quit IRC00:29
*** Sukhdev has quit IRC00:30
*** jamesdenton has joined #openstack-nova00:30
*** jamesdenton has quit IRC00:33
*** jamesden_ has joined #openstack-nova00:33
*** jamesdenton has joined #openstack-nova00:34
*** jamesden_ has quit IRC00:38
*** sacharya_ has quit IRC00:38
*** thorst_ has joined #openstack-nova00:49
openstackgerritClaudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests  https://review.openstack.org/30026200:50
*** diogogmt has joined #openstack-nova00:52
*** thorst_ has quit IRC00:54
openstackgerritClaudiu Belu proposed openstack/nova: Fixes python 3 unit tests  https://review.openstack.org/27881000:54
*** thorst_ has joined #openstack-nova00:55
openstackgerritHieu LE proposed openstack/nova: Config options: improve libvirt help text (1)  https://review.openstack.org/31653900:57
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 5)  https://review.openstack.org/32997900:58
*** sdake has joined #openstack-nova00:58
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2)  https://review.openstack.org/32997800:58
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 4)  https://review.openstack.org/32922700:58
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 3)  https://review.openstack.org/32885100:58
*** thorst_ has quit IRC00:59
*** zenoway has joined #openstack-nova00:59
openstackgerritHieu LE proposed openstack/nova: Config options: improve libvirt help text (2)  https://review.openstack.org/31655000:59
*** Apoorva has quit IRC01:01
*** jamesdenton has quit IRC01:03
*** zenoway has quit IRC01:04
*** jamesdenton has joined #openstack-nova01:04
*** diogogmt has quit IRC01:06
*** artom has quit IRC01:13
*** cheneydc has joined #openstack-nova01:13
*** sdake has quit IRC01:14
*** sdake has joined #openstack-nova01:16
*** artom has joined #openstack-nova01:18
*** EinstCrazy has joined #openstack-nova01:21
*** sdake_ has joined #openstack-nova01:22
*** Jeffrey4l has quit IRC01:24
*** sdake has quit IRC01:24
*** ducncvn has quit IRC01:25
*** baoli has joined #openstack-nova01:28
*** jamesdenton has quit IRC01:29
openstackgerritZhenyu Zheng proposed openstack/nova: [WIP] Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040601:29
*** Daisy has joined #openstack-nova01:33
*** Apoorva has joined #openstack-nova01:33
*** rama_y has quit IRC01:38
*** Jeffrey4l has joined #openstack-nova01:38
*** jamesdenton has joined #openstack-nova01:44
*** mtanino has quit IRC01:45
*** claudiub has quit IRC01:47
*** Daisy has quit IRC01:47
*** Daisy has joined #openstack-nova01:48
*** hoangcx has quit IRC01:51
*** hoangcx has joined #openstack-nova01:51
openstackgerritAnh Tran proposed openstack/nova: Enables Py34 tests for unit.api.openstack.compute.test_server_actions  https://review.openstack.org/32943501:51
*** Daisy has quit IRC01:52
*** Sukhdev has joined #openstack-nova01:57
*** sacharya has joined #openstack-nova01:59
*** Apoorva has quit IRC02:05
Qiminganyone help point me to a link of nova mid-cycle meetup, if any?02:06
Qimingthanks02:06
*** Daisy has joined #openstack-nova02:06
*** jamesdenton has quit IRC02:07
tonybQiming: https://wiki.openstack.org/wiki/Sprints#Future_sprints_for_Newton02:09
*** sacharya_ has joined #openstack-nova02:09
Qimingtonyb, thank you!02:09
tonybQiming: np02:10
openstackgerritMatt Riedemann proposed openstack/nova: Include exception in _try_deallocate_network error log  https://review.openstack.org/33085002:10
*** sacharya has quit IRC02:13
*** sdake has joined #openstack-nova02:16
*** sdake_ has quit IRC02:19
*** pwnall has quit IRC02:19
*** mtanino has joined #openstack-nova02:22
*** houming has joined #openstack-nova02:24
*** ivase has quit IRC02:27
*** ivase has joined #openstack-nova02:28
*** prateek has joined #openstack-nova02:32
*** jogo has quit IRC02:32
*** rrecio has joined #openstack-nova02:34
*** thorst_ has joined #openstack-nova02:38
*** thorst_ has quit IRC02:41
*** thorst_ has joined #openstack-nova02:41
*** thorst_ has quit IRC02:45
*** jogo has joined #openstack-nova02:46
*** tbachman has quit IRC02:46
*** mrkz has quit IRC02:48
*** mdrabe has joined #openstack-nova02:49
*** efried has left #openstack-nova02:49
*** tbachman has joined #openstack-nova02:51
openstackgerritVladik Romanovsky proposed openstack/nova: DO NOT MERGE - attach configdrive after instance XML test  https://review.openstack.org/33013402:52
*** rrecio has quit IRC02:53
*** adisky has joined #openstack-nova02:53
*** rama_y has joined #openstack-nova02:53
*** jamesdenton has joined #openstack-nova02:54
*** d_code has quit IRC02:56
*** jamesdenton has quit IRC02:57
*** gjayavelu has joined #openstack-nova02:59
*** d_code has joined #openstack-nova02:59
openstackgerritMatt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support  https://review.openstack.org/31895303:02
openstackgerritMatt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance  https://review.openstack.org/31627503:02
openstackgerritMatt Riedemann proposed openstack/nova: Add REST API support for get me a network  https://review.openstack.org/31639803:02
*** jamielennox is now known as jamielennox|away03:03
*** gyee has quit IRC03:07
*** amotoki has joined #openstack-nova03:07
*** amotoki has quit IRC03:09
*** jamielennox|away is now known as jamielennox03:19
*** Kevinpang has joined #openstack-nova03:24
*** baoli has quit IRC03:25
*** jogo has quit IRC03:26
*** d_code has quit IRC03:27
*** d_code has joined #openstack-nova03:29
*** d_code has quit IRC03:29
openstackgerritJianghua Wang proposed openstack/nova: XenAPI: OVS agent updates the wrong port with Neutron  https://review.openstack.org/24284603:30
*** jogo has joined #openstack-nova03:30
openstackgerritAugustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys  https://review.openstack.org/32969703:37
*** zerda2 has joined #openstack-nova03:47
*** markvoelker has quit IRC03:47
*** mdrabe has quit IRC03:48
*** mriedem has quit IRC03:51
*** Daisy has quit IRC03:55
*** Daisy has joined #openstack-nova03:55
*** Daisy has quit IRC03:58
*** Daisy has joined #openstack-nova03:58
*** amotoki has joined #openstack-nova04:00
openstackgerrithuan proposed openstack/nova: XenAPI: Support neutron security group  https://review.openstack.org/25127104:02
*** lpetrut has joined #openstack-nova04:03
*** ivase has quit IRC04:07
*** ivase has joined #openstack-nova04:08
*** Daisy has quit IRC04:11
*** Daisy has joined #openstack-nova04:11
*** prateek has quit IRC04:13
*** Daisy has quit IRC04:16
*** amotoki has quit IRC04:18
*** links has joined #openstack-nova04:20
*** prateek has joined #openstack-nova04:24
openstackgerritJianghua Wang proposed openstack/nova: XenAPI: OVS agent updates the wrong port with Neutron  https://review.openstack.org/24284604:24
*** yamahata has joined #openstack-nova04:27
*** lpetrut has quit IRC04:28
*** rha_ has joined #openstack-nova04:29
*** ociuhandu has quit IRC04:29
*** rha has quit IRC04:29
*** cheneydc has quit IRC04:35
*** ociuhandu has joined #openstack-nova04:42
*** amotoki has joined #openstack-nova04:45
*** Kevinpang has quit IRC04:45
*** ijw_ has joined #openstack-nova04:46
*** ijw has quit IRC04:47
*** markvoelker has joined #openstack-nova04:47
*** sdake_ has joined #openstack-nova04:50
*** markvoelker has quit IRC04:52
*** sdake has quit IRC04:53
*** Daisy has joined #openstack-nova04:53
*** ijw_ has quit IRC04:59
*** ijw has joined #openstack-nova04:59
*** Daisy has quit IRC05:02
*** Daisy has joined #openstack-nova05:02
*** ratailor has joined #openstack-nova05:05
*** sacharya_ has quit IRC05:06
*** Daisy has quit IRC05:07
*** Daisy has joined #openstack-nova05:07
*** salv-orl_ has joined #openstack-nova05:09
*** Daisy has quit IRC05:10
*** Daisy has joined #openstack-nova05:10
*** salv-orlando has quit IRC05:12
*** zenoway has joined #openstack-nova05:13
*** lpetrut has joined #openstack-nova05:14
*** ecelik has joined #openstack-nova05:15
*** pc_m has quit IRC05:16
*** zenoway has quit IRC05:18
*** browne has joined #openstack-nova05:25
*** links has quit IRC05:27
*** sdake has joined #openstack-nova05:27
*** links has joined #openstack-nova05:28
*** chandankumar has joined #openstack-nova05:30
*** sdake_ has quit IRC05:31
*** Daisy has quit IRC05:32
*** Daisy has joined #openstack-nova05:32
*** cfarquhar has quit IRC05:35
*** Daisy has quit IRC05:37
*** sdake_ has joined #openstack-nova05:38
*** sdake has quit IRC05:41
*** Daisy has joined #openstack-nova05:42
*** sdake_ has quit IRC05:50
*** amotoki has quit IRC05:56
*** amotoki has joined #openstack-nova06:02
*** abalutoiu has joined #openstack-nova06:02
*** Daisy has quit IRC06:04
*** Daisy has joined #openstack-nova06:04
*** belmoreira has joined #openstack-nova06:06
*** browne has quit IRC06:07
*** rama_y has quit IRC06:08
*** sridharg has joined #openstack-nova06:08
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (7)  https://review.openstack.org/33089806:09
*** Daisy has quit IRC06:09
*** nkrinner has joined #openstack-nova06:10
openstackgerritZhenyu Zheng proposed openstack/nova: [WIP] Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040606:13
*** Daisy has joined #openstack-nova06:15
*** pc_m has joined #openstack-nova06:15
*** Sukhdev has quit IRC06:17
openstackgerritLi Yingjun proposed openstack/nova: Implement hypervisor notifications  https://review.openstack.org/31531206:20
*** rcernin has joined #openstack-nova06:23
*** gjayavelu has quit IRC06:23
*** mtanino has quit IRC06:24
*** david-lyle has quit IRC06:26
*** andreas_s has joined #openstack-nova06:28
*** wanghao has quit IRC06:28
*** pcaruana has joined #openstack-nova06:29
*** paul-carlton2 has joined #openstack-nova06:30
*** lpetrut has quit IRC06:33
*** zenoway has joined #openstack-nova06:36
openstackgerritPawel Koniszewski proposed openstack/nova: Add new configuration option to turn auto converge on/off  https://review.openstack.org/26600106:37
*** sahid has joined #openstack-nova06:37
*** ssurana has joined #openstack-nova06:46
*** abalutoiu has quit IRC06:47
*** mpavone has joined #openstack-nova06:49
*** sheel has joined #openstack-nova06:50
*** Daisy_ has joined #openstack-nova06:50
*** ssurana has quit IRC06:50
*** Daisy has quit IRC06:52
*** Daisy_ has quit IRC06:57
*** Daisy has joined #openstack-nova06:58
*** fragatina has quit IRC06: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-nova07:01
*** jichen has joined #openstack-nova07: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 freeze07: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 cycle07:02
*** ihrachys has joined #openstack-nova07:03
*** amotoki has quit IRC07:03
*** paul-carlton2 has quit IRC07:05
*** cfriesen_ has quit IRC07:06
*** ivase has quit IRC07:07
*** syjulian has quit IRC07:08
*** ivase has joined #openstack-nova07:08
*** ihrachys has quit IRC07:08
*** salv-orl_ has quit IRC07:09
*** salv-orlando has joined #openstack-nova07:09
*** tesseract has joined #openstack-nova07:10
*** romainh has joined #openstack-nova07:12
*** hogepodge has quit IRC07:12
*** bauzas is now known as bauwser07:13
bauwserp!spy07:13
*** bauwser is now known as bauzas07:14
bauzasp!spy07:14
*** matrohon has joined #openstack-nova07:15
*** zenoway has quit IRC07:15
*** jlanoux has joined #openstack-nova07:20
*** abalutoiu has joined #openstack-nova07:20
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 1 before functional change  https://review.openstack.org/32376107:22
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: use "log" element in char devices  https://review.openstack.org/32376007:22
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 3 before functional change  https://review.openstack.org/32376307:22
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 2 before functional change  https://review.openstack.org/32376207:22
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: use the logd sub-element for char devices  https://review.openstack.org/32376507:22
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 4 before functional change  https://review.openstack.org/32376407:22
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: introduce domain xml element "log"  https://review.openstack.org/32375907:22
*** bauzas is now known as bauwser07:24
*** ssurana has joined #openstack-nova07:30
openstackgerritliuxiuli proposed openstack/nova: And reserved_huge_pages to memory_mb_used  https://review.openstack.org/33092807:32
*** dmk0202 has joined #openstack-nova07:32
*** cgalan has joined #openstack-nova07:32
*** ijw has quit IRC07:33
*** dmk0202 has quit IRC07:33
*** ssurana has quit IRC07:35
*** salv-orlando has quit IRC07:35
*** gcb has joined #openstack-nova07:35
*** claudiub has joined #openstack-nova07:37
*** EinstCra_ has joined #openstack-nova07:38
*** EinstCrazy has quit IRC07:40
*** EinstCra_ has quit IRC07:43
*** EinstCrazy has joined #openstack-nova07:43
bauwsergood morning Nova07:43
*** Kevin-Zhao has joined #openstack-nova07:43
*** zhangjn has quit IRC07:45
*** rcernin has quit IRC07:45
*** Daisy has quit IRC07:46
*** Daisy has joined #openstack-nova07:46
*** zhangjn has joined #openstack-nova07:50
*** Daisy has quit IRC07:51
*** Daisy has joined #openstack-nova07:51
*** lpetrut has joined #openstack-nova07:57
*** efoley has joined #openstack-nova07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** pacoblox has joined #openstack-nova08:00
*** gjayavelu has quit IRC08:02
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Use is_valid_ipv4 from oslo.utils  https://review.openstack.org/32692208:05
*** diga has joined #openstack-nova08:05
*** gjayavelu has joined #openstack-nova08:06
*** yamahata has quit IRC08:07
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c  https://review.openstack.org/30334508:10
*** rcernin has joined #openstack-nova08:10
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c  https://review.openstack.org/30334508:10
openstackgerritLisaLi proposed openstack/nova: Attach/detach encryptors during swap_volume  https://review.openstack.org/25280908:15
*** pacoblox has quit IRC08:19
*** tojuvone_ has joined #openstack-nova08:19
*** mgoddard has joined #openstack-nova08:20
*** huyang_ has joined #openstack-nova08:20
openstackgerritYaoZheng proposed openstack/nova: live migration check source failed caused bdm.device_path lost  https://review.openstack.org/33094408:20
*** ameade has quit IRC08:21
*** BobBall_AWOL is now known as BobBall08:21
*** jlvillal has quit IRC08:22
*** jlvillal has joined #openstack-nova08:22
*** ameade has joined #openstack-nova08:22
*** tyrola has joined #openstack-nova08:22
*** huyang has quit IRC08:23
*** tojuvone has quit IRC08:23
*** huyang_ is now known as huyang08:23
*** paul-carlton2 has joined #openstack-nova08:23
*** tyrola has quit IRC08:23
*** Daisy has quit IRC08:26
*** Daisy has joined #openstack-nova08:26
*** dmk0202 has joined #openstack-nova08:27
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (8)  https://review.openstack.org/33094808:27
openstackgerritStefan Amann proposed openstack/nova: Support FICON-attached ECKD volumes on System z  https://review.openstack.org/32083208:27
*** daemontool has joined #openstack-nova08:28
*** Daisy_ has joined #openstack-nova08:29
*** Daisy has quit IRC08:33
*** mkrcmari__ has joined #openstack-nova08:36
*** thomasem_ has quit IRC08:36
*** rmstar has joined #openstack-nova08:36
*** andreas_s_ has joined #openstack-nova08:36
*** kaisers has quit IRC08:37
*** Jeffrey4l has quit IRC08:37
*** abalutoiu_ has joined #openstack-nova08:37
*** jlvillal has quit IRC08:37
*** tojuvone__ has joined #openstack-nova08:37
*** liverpoo1er has joined #openstack-nova08:38
*** kaisers has joined #openstack-nova08:38
*** bswartz has quit IRC08:39
*** cgalan has quit IRC08:39
*** rodrigods has quit IRC08:40
*** xwizard___ has joined #openstack-nova08:40
*** pumaranikar has quit IRC08:40
*** rodrigods has joined #openstack-nova08:41
*** harlowja_ has quit IRC08:41
*** pkoniszewski has quit IRC08:41
*** jlvillal has joined #openstack-nova08:42
*** Daisy_ has quit IRC08:42
*** daemontool_ has joined #openstack-nova08:42
*** clduser has quit IRC08:42
*** anteaya has quit IRC08:42
*** Daisy has joined #openstack-nova08:42
*** ishant|away has quit IRC08:42
*** mvk_ has quit IRC08:42
*** andreas_s has quit IRC08:42
*** rmstar_ has quit IRC08:42
*** xwizard has quit IRC08:42
*** ekarlso has quit IRC08:42
*** abalutoiu has quit IRC08:42
*** ishant|away has joined #openstack-nova08:42
*** purplerbot has quit IRC08:42
*** thomasem has joined #openstack-nova08:42
*** lxsli0 has joined #openstack-nova08:42
*** pkoniszewski has joined #openstack-nova08:43
*** nicolasbock has quit IRC08:43
*** igordcard has quit IRC08:43
*** nicolasbock_ has joined #openstack-nova08:43
*** mdavidson has quit IRC08:43
*** liverpooler has quit IRC08:43
*** tlbr has quit IRC08:43
*** tojuvone_ has quit IRC08:43
*** zeroDivisible has quit IRC08:43
*** kaisers1 has quit IRC08:43
*** aignatov has quit IRC08:43
*** lxsli has quit IRC08:43
*** xek has quit IRC08:43
*** clduser has joined #openstack-nova08: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 again08:43
andreas_s_anybody an idea?08:43
*** zeroDivisible has joined #openstack-nova08:43
*** yhvh has quit IRC08:43
andreas_s_https://review.openstack.org/32083208:43
*** xek has joined #openstack-nova08:43
*** aignatov has joined #openstack-nova08:43
claudiubyou sure that you fetched master?08:43
*** pumaranikar has joined #openstack-nova08:43
andreas_s_claudiub, yep, we even did a fresh nova clone08:44
*** mtreinish has quit IRC08:44
*** daemontool has quit IRC08:44
*** tlbr has joined #openstack-nova08:44
*** yhvh has joined #openstack-nova08:44
*** mdavidson has joined #openstack-nova08:44
*** ihrachys has joined #openstack-nova08:44
claudiubandreas_s_: I think it's because the Depends-On patch is in a merge conflict.08:45
*** kaisers1 has joined #openstack-nova08:45
*** anteaya has joined #openstack-nova08: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-nova08:46
claudiubandreas_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 first08:46
openstackgerritZhenyu Zheng proposed openstack/nova: [WIP] Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040608:46
mdboothAnybody else unable to log in to gerrit this morning?08:46
andreas_s_claudiub, got it! Thanks you!08:46
mdboothI get an OpenID error: Provider is not supported, or was incorrectly entered.08:47
*** tpeoples has quit IRC08:47
*** Daisy_ has joined #openstack-nova08:47
mdboothAh, nm. After reproducing 3 times it seems to have fixed itself :)08:47
*** macsz has quit IRC08:47
claudiubandreas_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-nova08:48
claudiubmdbooth: 3 has always been a magical number. :)08:48
*** Jeffrey4l has joined #openstack-nova08:48
* mdbooth hires a shaman for the rest of his bugs08:48
*** macsz has joined #openstack-nova08:49
*** tpeoples has joined #openstack-nova08:49
*** igordcard has joined #openstack-nova08:49
kashyapmdbooth: I'm about to login08:50
kashyapAh, you said "it fixed itself"08:50
*** Daisy has quit IRC08:50
mdboothkashyap: Don't forget to cast the appropriate bones first08:51
andreas_s_claudiub, thanks for this explanation - wasn't ware of that08:51
*** ekarlso has joined #openstack-nova08:51
kashyapHeh08:51
claudiubandreas_s_: no problem. :)08:51
BobBalljohnthetubaguy: ping08:55
*** EinstCrazy has quit IRC08:55
*** Daisy has joined #openstack-nova08:58
*** Daisy_ has quit IRC08:58
*** EinstCrazy has joined #openstack-nova08:59
*** Daisy_ has joined #openstack-nova09:04
*** hoangcx has quit IRC09:05
*** hoangcx_ has joined #openstack-nova09:05
*** Daisy_ has quit IRC09:06
*** Daisy has quit IRC09:06
*** EinstCra_ has joined #openstack-nova09:06
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (9)  https://review.openstack.org/33096409:08
*** EinstCrazy has quit IRC09:09
*** mpavone has quit IRC09:11
*** sambetts|afk is now known as sambetts09:12
*** mpavone has joined #openstack-nova09:12
bauwserdo we success Jenkins passing away ? :p09:14
*** DaiDV has joined #openstack-nova09:16
*** gjayavelu has quit IRC09:16
*** openstackgerrit has quit IRC09:18
*** openstackgerrit has joined #openstack-nova09:18
*** zenoway has joined #openstack-nova09:19
*** ssurana has joined #openstack-nova09:20
BobBallI'm sure there will be a number of parties :)09:22
*** cdent has joined #openstack-nova09:23
*** ssurana has quit IRC09:24
*** purplerbot has joined #openstack-nova09:25
*** alex_xu has quit IRC09:26
*** mfedosin has joined #openstack-nova09:26
openstackgerritYuriy Zveryanskyy proposed openstack/nova: Change ignored exception in Ironic virt driver  https://review.openstack.org/33098409:27
*** alex_xu has joined #openstack-nova09:28
*** lxsli0 is now known as lxsli09:32
openstackgerritmelanie witt proposed openstack/nova: WIP Make InstanceGroup object favor the API database  https://review.openstack.org/33099509:34
openstackgerritliuxiuli proposed openstack/nova: And reserved_huge_pages to memory_mb_used  https://review.openstack.org/33092809:34
openstackgerritLee Yarwood proposed openstack/nova: compute: Skip driver detach calls for non local instances  https://review.openstack.org/31826609:39
*** mgoddard_ has joined #openstack-nova09:42
*** mgoddard has quit IRC09:42
johnthetubaguyBobBall: sorry, missed your ping, whats up?09:45
*** purplerbot has quit IRC09:45
BobBallNo worries.  Was just checking if you were around today.  See PM09:45
*** purplerbot has joined #openstack-nova09:47
* cdent pats purplerbot 09:47
*** mfedosin has quit IRC09:52
*** PaulMurray has joined #openstack-nova09:55
bauwserdo we know if we default limit/marker for all our API resources or only for the servers ?10:02
bauwserie. nova list etc.10:02
bauwserjohnthetubaguy: ^ do you know that ?10:03
bauwserie. I know we paginate nova list for sure, but I was wondering for paginating other API resources10:03
*** openstackgerrit has quit IRC10:03
johnthetubaguyhmm, it might just be servers10:03
johnthetubaguyso we don't do it for images10:03
johnthetubaguybecause thats a proxy, and the server we talk to doesn't do it10:03
bauwserok10:03
*** openstackgerrit has joined #openstack-nova10:03
bauwserI think os-hosts is not paginating too10:03
johnthetubaguybut I am not sure about the others, it would seem reasonable to limit those10:03
bauwsertbc, we don't have a default behaviour accepting pagination for any API resource action ?10:04
bauwserif so, it's per resource/action I see10:04
johnthetubaguyright, pagination is added per resource10:04
bauwsereven per action I guess10:04
bauwserlike a GET /servers/id doesn't make sense to be paginated10:05
johnthetubaguyit largely has to, because to get any benefit, we need to send that detail down to the DB call10:05
*** eglynn has joined #openstack-nova10:05
johnthetubaguyright10:05
*** rcernin has quit IRC10:05
*** mfedosin has joined #openstack-nova10:05
bauwserokay, I guess I should sneak peak at the API ref documentation and see what resources are having limit/marker :)10:06
bauwserjohnthetubaguy: thanks for clarifying that :)10:06
johnthetubaguyyeah, not sure that detail is complete yet...10:08
openstackgerrithuan proposed openstack/nova: XenAPI: Support neutron security group  https://review.openstack.org/25127110:11
*** tbachman has quit IRC10:14
*** hoangcx_ has quit IRC10:22
*** ociuhandu has quit IRC10:23
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (10)  https://review.openstack.org/33101310:25
*** rmart04 has joined #openstack-nova10:30
*** tbachman has joined #openstack-nova10:35
*** smatzek has joined #openstack-nova10:37
*** gjayavelu has joined #openstack-nova10:38
*** ivase has quit IRC10:41
*** ociuhandu has joined #openstack-nova10:43
*** gjayavelu has quit IRC10:43
*** huyang has quit IRC10:44
*** huyang has joined #openstack-nova10:45
*** tbachman_ has joined #openstack-nova10:46
*** permalac has joined #openstack-nova10:47
*** sree has joined #openstack-nova10:48
*** tbachman has quit IRC10:50
*** tbachman_ is now known as tbachman10:50
*** rtheis has joined #openstack-nova10:52
*** hogepodge has joined #openstack-nova10:54
openstackgerritsahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop  https://review.openstack.org/17485410:54
cdentbauwser, johnthetubaguy: you might "enjoy" the email I've just sent to os-dev....10:55
*** tbachman_ has joined #openstack-nova10:55
*** eglynn has quit IRC10:55
*** tbachman has quit IRC10:57
*** tbachman_ is now known as tbachman10:57
openstackgerritLucian Petrut proposed openstack/nova: Py3: fix serial console output  https://review.openstack.org/30218210:59
*** sree has quit IRC11:01
openstackgerritLucian Petrut proposed openstack/nova: Py3: fix serial console output  https://review.openstack.org/30218211:02
openstackgerritClaudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests  https://review.openstack.org/30026211:03
openstackgerritJohn Garbutt proposed openstack/nova: Create _validate_requested_port_ids  https://review.openstack.org/32960011:04
openstackgerritClaudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests  https://review.openstack.org/27883411:04
*** andreas_s_ has quit IRC11:04
openstackgerritJohn Garbutt proposed openstack/nova: Extract _validate_requested_network_ids  https://review.openstack.org/32960111:05
claudiubjohnthetubaguy: 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/1811:08
*** romainh has quit IRC11:09
*** diga has quit IRC11:11
*** sree has joined #openstack-nova11:13
*** ssurana has joined #openstack-nova11:20
BobBalldansmith: 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-nova11:23
*** hoangcx has joined #openstack-nova11:25
*** ssurana has quit IRC11:25
bauwsercdent: lemme tell you a story11:33
*** gcb has quit IRC11:33
* cdent sits comfortably11:33
bauwsercdent: once upon a time, a very small project tried to define a REST API using first Flask, then Pecan11:33
openstackgerritPaul Murray proposed openstack/nova: Add periodic task to clean expired console tokens  https://review.openstack.org/32538111:33
openstackgerritPaul Murray proposed openstack/nova: Add console auth token records to the database  https://review.openstack.org/32541411:33
openstackgerritPaul Murray proposed openstack/nova: Add console auth object  https://review.openstack.org/32006311:33
openstackgerritPaul Murray proposed openstack/nova: Add instance uuid to console URLs  https://review.openstack.org/32538211:33
openstackgerritPaul Murray proposed openstack/nova: Add console auth tokens db api methods  https://review.openstack.org/31996411:34
openstackgerritPaul Murray proposed openstack/nova: Add console auth tokens table and model  https://review.openstack.org/31932711:34
bauwsercdent: both were interesting to use, but at the end, the project failed11:34
*** hoangcx has quit IRC11:34
bauwsercdent: 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
cdentme 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 far11:35
*** aysyd has joined #openstack-nova11:35
bauwserpecan 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
bauwserthe real terrible thing we struggle was with input validation, using He-Who-Cannot-Be-Named aka. WSME11:37
cdenthuh. 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 RESTful11:37
cdentnever speak of WSME! :)11:37
bauwser(in front of a mirror) WSME, WSME, WSME !11:38
* bauwser quit11:38
bauwser:p11:38
cdentwith 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 trace11:38
*** lucasagomes is now known as lucas-hungry11:39
bauwsercdent: 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 tree11:39
* cdent steps to the left, to the right, to the front, to the back and turns slowly around11:39
bauwserhttps://m.popkey.co/cedcc1/eLw7m.gif11:40
cdentI 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
cdentha!11:40
*** sree__ has joined #openstack-nova11:41
cdentfor that latter group their reaction to nova is much like JohnT there11:41
bauwserheh11:42
*** irenab has joined #openstack-nova11:42
*** sree has quit IRC11:42
openstackgerritsahid proposed openstack/nova: network: handle unauthorized exception from neutron  https://review.openstack.org/31201411:43
bauwserhonestly, I don't think the ramp-up on that API code is so hard11:43
bauwsereither way, people coming to Nova know there is an history11:44
*** hoangcx has joined #openstack-nova11:44
bauwserof course, copy/pasting is never a good idea if you don't understand what you're doing in between11:44
bauwserbut, here, I just don't want to redo the wheel, that's it :-)11:45
bauwserin particular since one of my roles is to look at what people write and set my mind on the project I'm looking to11:46
bauwserin particular wrt the patterns we use and the things we agreed in the past11:46
bauwserhaving separate codepaths is like darwinian evolution11:46
*** irenab has quit IRC11:47
*** sree has joined #openstack-nova11:47
cdentbauwser: 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-nova11:47
bauwsercdent: sure, it's like if Nova was about to write a new version, right?11:48
cdentOr 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
bauwsercdent: there would, for sure11:48
cdentI mean about the being similar to nova's code11:48
*** Jeffrey4l has quit IRC11:48
cdentsure there should be debate on how to do the api11:48
*** Jeffrey4l has joined #openstack-nova11:48
cdentbut the question would be around framework vs framework-less11:49
bauwserbut we're still discussing about the placement API being part of nova, do we ? :)11:49
cdentI think for most people that's a done deal.11:49
*** gcb has joined #openstack-nova11:49
cdentIt 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 IRC11:49
cdentAnd so that it can evolve independently of the history of nova11:50
bauwserI still haven't seen consensus on that being a separate *service* :)11:50
bauwserbut either way, it's not the discussion we're having originally11:50
*** sree__ has quit IRC11:51
cdentI think you should talk to jay, dan, sean, johnthetubaguy, matt r about the plans with the placement api. Or maybe I should. :)11:51
cdentI 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
cdentWhat was not decided was what the new way should be11:52
bauwserI agree :)11:53
bauwsertbc, I agree with the point that the Nova API structure is inappropriate11:53
*** EinstCra_ has quit IRC11:53
bauwserwhat 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-nova11:55
*** eglynn has joined #openstack-nova11:56
openstackgerritJohn Garbutt proposed openstack/nova: Extract _validate_requested_network_ids  https://review.openstack.org/32960111:56
*** EinstCrazy has joined #openstack-nova11:57
cdentbauwser: 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 IRC11:58
*** Kevin-Zhao has joined #openstack-nova11:58
bauwserit's Friday, are you sure we should have this discussion now and here ?11:58
cdentI 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-nova12:00
cdentIt'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
cdentI simply want to understand so that I know what's going on.12:00
openstackgerritJohn Garbutt proposed openstack/nova: Extract _validate_requested_network_ids  https://review.openstack.org/32960112:01
*** EinstCrazy has quit IRC12:02
*** cdelatte has quit IRC12:03
*** EinstCrazy has joined #openstack-nova12:03
bauwsercdent: I think http://lists.openstack.org/pipermail/openstack-dev/2015-March/060179.html still applies here12:03
*** dave-mccowan has joined #openstack-nova12:04
openstackgerritbailin.zhang proposed openstack/nova: Delete reduplicate code in test_compute_mgr.py  https://review.openstack.org/33110812:04
bauwserwe're just discussing about splitting the nova-scheduler, not creating a cross-project scheduler12:04
*** markvoelker has joined #openstack-nova12:04
bauwserthe 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
bauwsers/ie./eg.12:05
*** dmk0202 has quit IRC12:05
bauwserin other words, people deploying a Compute service would necessarly install a separate package, I'm happy with that12:06
*** haukebruno has quit IRC12:06
*** EinstCra_ has quit IRC12:06
*** pradk has joined #openstack-nova12:06
bauwserbut that wouldn't mean that this package would be usable by other projects12:06
bauwsertbc, service projects12:06
*** kfarr has joined #openstack-nova12:07
*** pradk has quit IRC12:07
cdentokay, but what does have to do with whether lifting and shifting the code is a good idea or not?12:07
tdurakovhi folks12:07
tdurakovdon't you know is danpb is on vacation or not?12:07
*** mtanino has joined #openstack-nova12:07
*** sree has quit IRC12:08
cdenttdurakov: he was around yesterday or the day before but that's all I know12:08
*** rfolco has joined #openstack-nova12:08
tdurakovcdent: same for me12:08
jrollcdent: I think it should use pecan/wsme12:08
tdurakovkashyap: hi, are you around?12:08
* jroll runs away12:08
* cdent gives jroll a cookie12:08
bauwserjroll: trolls are accepted on Fridays12:09
jroll\o/12:09
* jroll comes back and eats cookies12:09
openstackgerritZhihai Song proposed openstack/nova: Transform instance.power_on notifications  https://review.openstack.org/32974612:09
jrollcdent: so my worry with "don't use a framework" is that I suspect over time it will turn into a framework of its own12:11
cdentjroll: I don't know what you're eating but I only gave you _one_ cookie, sir!12:11
*** EinstCra_ has joined #openstack-nova12:11
cdentjroll: did you have a chance to look at the code?12:11
jrollcdent: I'm skimming now12:11
cdentit's a very valid concern12:11
*** EinstCra_ has quit IRC12:11
jrollcdent: 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
cdentand 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 constraint12:13
*** pradk has joined #openstack-nova12:13
cdentand you're right, enforcing that constraint is apparently impossible in openstack12:13
*** EinstCra_ has joined #openstack-nova12:13
cdentso maybe I'm just using this as exercise to tilt at all the broken windmills in openstack :(12:13
jrollI mean, I fully support your efforts in theory12:14
jrollbut idk if it's the best route long term12:14
cdent"abstracted beyond all recognition" is, to me, the worst possible flaw in code12:14
cdentjroll: thus the email, I wanted to get people engaged now before it is too late12:14
jrollyep, I will have a think on it12:14
*** EinstCrazy has quit IRC12:14
cdentif we wanted "abstracted beyond all recognition" why aren't we using java?12:14
jrollI also don't think flask is "too" magical, having used flask, django, pecan12:15
jrollbut I'll save that bit for the list :)12:15
cdentthey are all too magical to me :)12:15
jrollcdent: were you in the room for the DLM discussion?12:15
bauwseroh please12:15
cdentin tokyo? yes12:15
jrollin vancouver I thnk12:15
jroller yeah12:15
jroll"OH NO JVM"12:15
cdentor maybe it  was vancouver12:15
cdentbut yeah, I was there12:15
* cdent doesn't mind jvm, but does mind java12:16
jrollI think the main reason is that nothing starts with a goal of looking like java around here12:16
jrollit just inevitably does12:16
cdentmy 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
cdentjroll: preventing that is part of why my little api uses mostly functions, not classes12:17
cdentit's only uses classes to give paste something to hang on to12:17
cdenti'd avoid paste too if I could...12:17
jroll++12:18
* jroll has succeeded with not having paste in ironic yet12:18
cdentat the moment the only reason it is in there was to "make it easy to get started" but it could go away12:18
* cdent leaves himself a comment12:18
*** zhangjn has quit IRC12:19
*** cdelatte has joined #openstack-nova12:19
openstackgerritbailin.zhang proposed openstack/nova: Unplug vifs when VM create failed  https://review.openstack.org/33111312:20
cdentbauwser: what were you "oh please"ing?12:20
openstackgerritMatthew Booth proposed openstack/nova: Never copy a swap disk during cold migration  https://review.openstack.org/33111512:21
openstackgerritMatthew Booth proposed openstack/nova: Document the expectations of all callers of libvirt's _create_image  https://review.openstack.org/33111612:21
openstackgerritMatthew Booth proposed openstack/nova: Move root disk snapshot into finish_migration  https://review.openstack.org/33111712:21
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Don't re-resize disks in finish_migration()  https://review.openstack.org/33111812:21
bauwsercdent: 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 IRC12:21
jrollhehe12:21
*** chenfei has quit IRC12:22
bauwserwas to*12:22
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Create Session instance if possible  https://review.openstack.org/30403512:22
*** zhangjn has joined #openstack-nova12:23
*** cdelatte has quit IRC12:25
*** chenfei has joined #openstack-nova12:25
cdentjroll, bauwser: I look forward to extensive, informative and engaging emails from you both in response to that message I posted. ;)12:25
* cdent has lunch12:25
*** jamesdenton has joined #openstack-nova12:25
jrollcdent: damn, now I have commitments12:26
*** ivase has joined #openstack-nova12:26
bauwsercdent: sure, I'll try to figure out the best meme I can give you by email12:26
*** pwnall has joined #openstack-nova12:26
*** cdelatte has joined #openstack-nova12:30
*** haukebruno has joined #openstack-nova12:31
*** edmondsw has joined #openstack-nova12:34
*** baoli has joined #openstack-nova12:34
*** baoli_ has joined #openstack-nova12:36
*** tbachman_ has joined #openstack-nova12:36
*** ljxiash has joined #openstack-nova12:36
openstackgerritBalazs Gibizer proposed openstack/nova: Transfrom wrap_exception notification to versioned format  https://review.openstack.org/31070012:36
openstackgerritBalazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts  https://review.openstack.org/33051312:36
openstackgerritBalazs Gibizer proposed openstack/nova: Enhance notification sample test base  https://review.openstack.org/32161012:36
openstackgerritBalazs Gibizer proposed openstack/nova: Handle multiple samples per versioned notification  https://review.openstack.org/32160912:36
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.update notification  https://review.openstack.org/32592312:36
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.delete notifications  https://review.openstack.org/31365412:37
*** hoangcx has quit IRC12:38
*** hoangcx has joined #openstack-nova12:39
*** cdelatte has quit IRC12:39
openstackgerritJulian proposed openstack/nova: Transform instance.snapshot notifications  https://review.openstack.org/33012812:39
*** tbachman has quit IRC12:40
*** tbachman_ is now known as tbachman12:40
*** alaski is now known as lascii12:40
*** baoli has quit IRC12:40
*** cdelatte has joined #openstack-nova12:41
*** baoli_ has quit IRC12:41
openstackgerritJulian proposed openstack/nova: Transform instance.snapshot notifications  https://review.openstack.org/33012812:42
*** bswartz has joined #openstack-nova12:43
*** hoangcx has quit IRC12:43
*** chandankumar has quit IRC12:44
*** kylek3h has joined #openstack-nova12:45
*** kylek3h has quit IRC12:45
*** kylek3h has joined #openstack-nova12:45
kashyaptdurakov: Hey, was away for some lunch12:47
tdurakovkashyap: could you please take a look https://review.openstack.org/#/c/320897/412:48
* kashyap clicks12:48
tdurakovkashyap: especially interesting this thread https://www.redhat.com/archives/libvir-list/2016-May/msg01937.html12:48
kashyaptdurakov: Give me a few, finishing a write-up on a QEMU related bug.12:48
tdurakovkk12:48
*** pradk has quit IRC12:50
*** lucas-hungry is now known as lucasagomes12:50
*** ssurana has joined #openstack-nova12:53
*** edleafe is now known as figleaf12:53
*** ssurana has quit IRC12:57
openstackgerritZhihai Song proposed openstack/nova: Transform instance.power_on notifications  https://review.openstack.org/32974612:59
kashyaptdurakov: Have seen that patch in passing before.  Have to read the full context.  (/me be back in a bit.)13:02
*** gcb has quit IRC13:02
*** cdelatte has quit IRC13:03
*** zhs_ss has quit IRC13:05
*** MikeG451_ has joined #openstack-nova13:07
*** tbachman_ has joined #openstack-nova13:08
*** mdrabe has joined #openstack-nova13:09
openstackgerritClaudiu Belu proposed openstack/nova: Added support for new block device format in vmops  https://review.openstack.org/24629913:10
*** tbachman has quit IRC13:10
*** tbachman_ is now known as tbachman13:10
*** zerda2 has quit IRC13:10
*** xyang1 has joined #openstack-nova13:10
johnthetubaguycdent: at a first glance, the API code looks good just using webob directly, +1 killing paste.ini13:10
openstackgerritKaren Bradshaw proposed openstack/nova: api-ref: os-volumes.inc  https://review.openstack.org/32004813:12
cdentjohnthetubaguy: thanks, yeah, one of the good outcomes of the discussion above was remembering paste could go13:12
bauwserjohnthetubaguy: cdent: tbc, given http://docs.openstack.org/developer/nova/policies.html#public-contractual-apis I'm fine with removing paste.ini for the placement API13:14
*** nkrinner has quit IRC13:14
bauwser(cdent: just explaining why I'm okay with that :p)13:14
cdentjohnthetubaguy: this puts you on the hook to write on that thread too, in an "extensive, informative and engaging" fashion... :D13:15
*** tblakeslee has joined #openstack-nova13:16
openstackgerritJulian proposed openstack/nova: WIP Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2)  https://review.openstack.org/32543513:17
figleafcdent: 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-nova13:19
cdentfigleaf: 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
cdentSince selector isn't a "WSGI provider" I think the concerns are really more about the strategies I mention in the email13:20
figleafcdent: ah, thanks. So is there a way to achieve your approach with an existing (in Nova) package?13:20
figleafOr is selector simply the cleanest?13:20
openstackgerritKaren Bradshaw proposed openstack/nova: api-ref, os-services.inc  https://review.openstack.org/32112313:21
*** pacoblox has joined #openstack-nova13:21
cdentthe 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 HTTP13:22
*** akshai has joined #openstack-nova13:23
*** nkrinner has quit IRC13:24
*** pacoblox has quit IRC13:26
*** takedakn has joined #openstack-nova13:26
figleafcdent: 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
cdentfor example: it is incapable of informing its client code of when a 405 is the correct response and thus causes a 404 instead13: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
cdents/on at/one at/13:29
*** mrkz has joined #openstack-nova13:29
*** mriedem has joined #openstack-nova13:31
andreykurilinmriedem: hi!13:32
figleafcdent: ok, thanks for the info13:32
cdentyou're welcome, thanks for asking13:32
mriedemandreykurilin: hi13:33
mriedemreally pounced on me13:33
andreykurilin:)13:34
andreykurilincould 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-nova13:35
*** mtanino has quit IRC13:35
mriedemandreykurilin: sure13:37
andreykurilinmriedem: thanks13:37
andreykurilinbtw, your review are preferable too:)13:38
mriedemandreykurilin: bah13:38
mriedemare any of these jobs running with keystone v3?13:38
mriedemlooks 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.gz13:39
mriedemand functional http://logs.openstack.org/35/304035/9/check/gate-novaclient-dsvm-functional/aa98073/logs/etc/openstack/clouds.yaml.txt.gz13:39
andreykurilinmriedem: yes13:39
andreykurilinand we have two functional tests for keystone v2 and v313:40
andreykurilinnovaclient.tests.functional.test_auth.TestAuthentication13:40
andreykurilinmriedem: no, we do not have them:(13:40
*** smatzek has quit IRC13:41
mriedemi must have been thinking of this https://review.openstack.org/#/c/313588/13:41
*** eantyshev has left #openstack-nova13:41
*** dansmith is now known as superdan13:41
*** eantyshev has joined #openstack-nova13:41
andreykurilinmriedem: it will break one functional test13:42
*** cdelatte has joined #openstack-nova13:42
eantyshevHi, everyone! wanted to ask for reviews in https://review.openstack.org/19084313:43
mriedemandreykurilin: hmm, ok, that v3-only job would be non-voting to start, so wouldn't break us i guess13:43
andreykurilinmriedem: sounds good13:43
*** pacoblox has joined #openstack-nova13:44
*** pacoblox has quit IRC13:44
*** mgoddard__ has joined #openstack-nova13:46
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Fix the clean up of resize-revert for volume backed instances  https://review.openstack.org/32903013:46
*** links has quit IRC13:46
mriedemhemna: 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.gz13:47
mriedemlook 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
mriedemeantyshev: 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.filters13:48
mriedemthat should be handled with privsep13:48
*** mgoddard_ has quit IRC13:49
*** cdelatte has quit IRC13:49
*** fragatina has joined #openstack-nova13:49
wasmumanyone seen anything like this when rebuilding a failed neutron node in an ha environment?  http://pastebin.com/80i1w6tB13:50
wasmumlooks like the switch db is not populating?13:50
*** ctrath has joined #openstack-nova13:50
*** ametts has joined #openstack-nova13:50
*** fragatin_ has joined #openstack-nova13:50
*** sudipto has joined #openstack-nova13:51
*** ecelik has quit IRC13:51
*** rodrigods has quit IRC13:52
*** rodrigods has joined #openstack-nova13:52
eantyshevmriedem: OK, I will search the Nova code for usage samples of privsep. Could you point me to the similar places?13:52
wasmumin the vswitch db log i get13:52
wasmum2016-06-17T13:44:32.142Z|00009|jsonrpc|WARN|unix: send error: Broken pipe13:53
wasmum2016-06-17T13:44:32.146Z|00010|reconnect|WARN|unix: connection dropped (Broken pipe)13:53
mriedemwasmum: see channel topic please13:53
mriedemeantyshev: try getting with hemna on that13:53
mriedemor gus13:53
*** fragatina has quit IRC13:54
wasmummriedem: poop, sorry client shrunk room names, thought this was neutron13:54
*** armax has joined #openstack-nova13:54
*** ctrath has quit IRC13:54
eantyshevmriedem: Thanks13:55
*** mmandel has joined #openstack-nova13:55
*** pradk has joined #openstack-nova13:56
gusmriedem: those systool -c fc_host "failures" are normal with os-brick afaik.13:57
*** ctrath has joined #openstack-nova13:57
gusmriedem: 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 IRC13:58
gusI never looked too closely at the actual call site.13:58
*** mriedem has joined #openstack-nova13:59
*** burgerk has joined #openstack-nova13:59
*** smatzek has joined #openstack-nova14:02
*** mfedosin has quit IRC14:06
*** cdelatte has joined #openstack-nova14:06
*** gjayavelu has joined #openstack-nova14:07
mriedemeantyshev: the vz storage ci must be using oslo.privsep 1.4.014:08
mriedembecause i see https://bugs.launchpad.net/os-brick/+bug/159374314:08
openstackLaunchpad bug 1593743 in os-brick "(privsep) u'systool -c fc_host -v' failed. Not Retrying." [Undecided,New]14:08
mriedemos-brick 1.4.0 is blacklisted in global-requirements right now14:08
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Rename _cleanup_resize to confirm_migration  https://review.openstack.org/33115814:10
eantyshevmriedem: 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.gz14:10
*** sdake has joined #openstack-nova14:12
eantyshevmriedem: though, I see the error in n-cpu.log14:12
openstackgerritSudipta Biswas proposed openstack/nova: Update setup.cfg for memory bandwidth monitor  https://review.openstack.org/32978514:13
*** syjulian has joined #openstack-nova14:13
*** cdelatte has quit IRC14:14
mriedemsudipto: did you address jay's -1 here? https://review.openstack.org/#/c/203163/14:14
mriedemeantyshev: you are using os-brick from master though right?14:15
mriedem-e git+https://git.openstack.org/openstack/os-brick@0fbaad573060a56c5f9998b0182bcda8c3e7fe3e#egg=os_brick14:15
sudiptomriedem, yeah i did.14:15
*** Jeffrey4l has quit IRC14:15
mriedemsudipto: please reply in the review to explain14:15
sudiptomriedem, sure will.14:15
mriedemeantyshev: the vz driver is in os-brick now right? if so, the vz storage ci shouldn't be running with os-brick from trunk14:16
mriedemit should use the version from g-r14:16
*** zul has quit IRC14:16
*** zul has joined #openstack-nova14:16
mriedemeantyshev: yeah vz has been in os-brick for awhile14:17
mriedemhttps://github.com/openstack/os-brick/commit/ff44cb5457b0e33ffa9c9b4f63634e4b65f66ef414:17
eantyshevmriedem: Oh. indeed. We kept "LIBS_FROM_GIT=os-brick" in localrc since patches were on review14:17
mriedemeantyshev: that should be rmeoved14:18
mriedem*removed14:18
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/33117414:19
eantyshevmriedem: will remove this, thanks again for pointing out!14:19
*** hemanthm|afk is now known as hemanthm14:20
mnestratovhi all14:20
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements  https://review.openstack.org/33118414:20
*** zul_ has joined #openstack-nova14:21
*** pacoblox has joined #openstack-nova14:21
*** mtanino has joined #openstack-nova14:21
mriedemmnestratov: hi14:21
*** mfedosin has joined #openstack-nova14:21
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907714:22
mnestratovmriedem: just to let you know about our Virtuzoo Compute CI status - we enabled it and reports again but we didn't remove patching yet14:22
mriedemmnestratov: why not?14:22
*** zul has quit IRC14:23
mnestratovthis is temporarily because we encountered a problem and we are solving it now14:23
mriedemmnestratov: 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
mnestratovmriedem: while we were patching some other change broke our upstream code that we didn't notice - trying to fix it now14:24
mriedemmnestratov: stacking those changes would make them easier to review14:24
mriedem*and*,14:24
mriedemfor the rescue change we should enable running the rescue tests in tempest14:24
mnestratovmriedem: ok14:24
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: update guest time after suspend  https://review.openstack.org/31611614:25
openstackgerritJohn Garbutt proposed openstack/nova: Make _validate_requested_network_ids return a dict  https://review.openstack.org/32984914:26
*** Jeffrey4l has joined #openstack-nova14:28
mnestratovmriedem: 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 dependent14:31
*** cdelatte has joined #openstack-nova14:31
*** david-lyle has joined #openstack-nova14:32
mriedemmnestratov: it's easier to review and test them separately14:33
mriedemwith 2 weeks left for non-priority blueprints you want to make the reviews as painless as possible :)14:33
mnestratovmriedem: absolutely :)14:34
*** pacoblox has quit IRC14:34
claudiubhm, 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
claudiubor other pci related commands.14:35
mriedemclaudiub: i'm sure that's probably really old, probably at least juno14:35
mriedemDate of page creation02:38, 20 November 201314:36
*** zz_dimtruck is now known as dimtruck14:36
bauwserwoah, just saw Gerrit having the "included-in" dropdown list which gives us the list of branches and tags having the related change14:36
claudiubmriedem:  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 IRC14:37
claudiuball I see is an abandoned patch on python-novaclient related to adding pci support.14:37
mriedembauwser: welcome to convenience14:38
mriedemclaudiub: no idea on the CLI14:38
*** sridharg has quit IRC14:38
mriedemit was pre-microversions14:38
bauwsermriedem: well, my git repo will <3 that14:38
bauwserbecause I was grepping the SHA1 :p14:38
bauwseroops, the change-id rather14:39
*** pacoblox has joined #openstack-nova14:39
*** cfriesen_ has joined #openstack-nova14:39
*** sacharya has joined #openstack-nova14:40
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907714:41
claudiubmriedem: 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 commands14:41
claudiubimo, I think it would still be worthwhile to add it.14:42
openstackgerritjichenjc proposed openstack/nova: WIP: remove scheduler_hints  https://review.openstack.org/33119414:42
*** ssurana has joined #openstack-nova14:42
claudiubalthough, I don't know, do we need a blueprint for something like this?14:42
mriedemclaudiub: if it's not using microversions, then i'd say do it in openstackclient14:42
mriedems/using/dependent on/14:42
claudiubaham.14:43
claudiubartom: hello. got a minute? :)14:44
openstackgerritLuis Tomas proposed openstack/nova: Add automatic switching to postcopy mode when migration is not progressing  https://review.openstack.org/32849914:44
*** ihrachys has quit IRC14:45
*** baoli has joined #openstack-nova14:45
*** ljxiash has quit IRC14:47
*** ssurana has quit IRC14:47
*** sacharya_ has joined #openstack-nova14:47
openstackgerritjichenjc proposed openstack/nova: WIP: remove scheduler_hints  https://review.openstack.org/33119414:48
openstackgerritjichenjc proposed openstack/nova: WIP: remove extension of scheduler hints totally  https://review.openstack.org/33120214:48
*** ratailor has quit IRC14:48
*** prateek has quit IRC14:48
artomclaudiub, hey :) Not right now, I took a PTO day to get my daughter to the doctor - this afternoon is you're still around, otherwise Monday14:48
artomOr write to the mailing list and I'll reply async :)14:48
*** cdelatte has joined #openstack-nova14:48
*** mpavone has quit IRC14:49
claudiubartom: Oh, I see. :) Well, I'll be around. When you come back, drop a ping pls. :)14:49
*** mpavone has joined #openstack-nova14:49
*** sacharya has quit IRC14:50
*** rrecio has joined #openstack-nova14:51
*** mpavone has quit IRC14:51
*** rrecio_ has joined #openstack-nova14:51
*** mpavone has joined #openstack-nova14:52
*** mpavone has quit IRC14:52
*** krtaylor has joined #openstack-nova14:53
openstackgerritNicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd  https://review.openstack.org/33120414:53
*** pcaruana has quit IRC14:54
*** rrecio has quit IRC14:55
*** hoangcx has joined #openstack-nova14:55
PaulMurraywhat's a good example to copy for cells stuff ?14:55
*** hoangcx has quit IRC14:55
*** hoangcx has joined #openstack-nova14:55
*** jmarianu has quit IRC14:55
*** ctrath has quit IRC14:57
bauwserPaulMurray: what do you want to do ?14:57
*** mgoddard has joined #openstack-nova14:58
*** mgoddard__ has quit IRC14:58
*** stewie925 has joined #openstack-nova15:00
*** ctrath has joined #openstack-nova15:00
PaulMurrayI'm putting console connection authroizations in the child database (this is to get rid of consoleauth)15:01
PaulMurrayso at api level I have an instance id and a console token15:01
PaulMurrayI need to get the console info from the child database15:01
PaulMurrayso I guess the process is - use isntance uuid to get child cell db15:02
PaulMurrayget info from child db15:02
bauwserwell, it's not an upcall, just a top-down15:02
PaulMurraythat's right15:02
*** diogogmt has joined #openstack-nova15: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
PaulMurrayit will actually be from the console proxies15:03
bauwserPaulMurray: the question I'm wondering is if we should automate this kind of call by looking up the InstanceMapping directly in a contextmanager15:03
*** doffm_ is now known as doffm15:03
bauwserbut that's something maybe I missed from the last cells v2 meetings15:03
bauwserie. how we should call the child DBs explicitely or implicitely15:04
*** diogogmt has quit IRC15:04
doffmI guess explicitly if we are in the API cell.15:04
PaulMurrayare there any examples that do this already15:04
PaulMurrayor are they all in old v1 world code15:05
*** tesseract has quit IRC15:05
sahidmriedem: 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 time15:06
bauwserPaulMurray: nothing I know yet for cells v215:06
doffmPaulMurray: Yeah, nothing I know of.15:06
bauwserPaulMurray: AFAIR, we'll be using a contextmanager for selecting how to call the child cell15:06
bauwserthere is a review from melwitt IIRC15:06
doffmPaulMurray: What spec are you working on?15:07
*** rcernin has quit IRC15:07
PaulMurrayhttps://review.openstack.org/#/c/301158/15:07
*** Oku_OS is now known as Oku_OS-away15:07
PaulMurrayconvert-consoles-to-objects15:08
*** rama_y has joined #openstack-nova15:08
*** pacoblox has quit IRC15:08
mfedosinmriedem: 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-nova15:09
*** amotoki has joined #openstack-nova15:09
PaulMurraymelwitt, do you have any idea about above ?15:09
lasciiPaulMurray: is this a db or RPC call?15:10
mriedemmfedosin: non-functional code comment / docstring etc changes are always welcome15:10
*** Oku_OS-away is now known as Oku_OS15:10
doffmPaulMurray: Ahhh, ok. There are some examples of choosing a cell and doing a database switch.15:10
PaulMurrayahh, no wonder I couldn't find alaski..... I should have guessed15:10
melwittPaulMurray: 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@39815:10
mriedemsahid: is that the last change in that bp series?15:10
sahidmriedem: yes it is15:11
PaulMurraylascii, it will be a lookup15:11
*** tbachman has quit IRC15:11
PaulMurraylascii, I will also have to do an rpc call to compute node after, but that is an existing call15:11
PaulMurrayso not worried about that right now15:11
mriedemsahid: ok, yeah, i've randomly tried to remember to review that one since i know the first part is merged15:11
mfedosinmriedem: great!15:11
PaulMurraymelwitt, thanks15:12
lasciiPaulMurray: okay. RPC calls are being converted in https://review.openstack.org/#/c/322181/15:12
sahidmriedem: cool thanks15:12
mriedemand i want to get to https://review.openstack.org/#/c/312014/ but have been sidetracked15:13
sahidmriedem: yes this one too, i was less scary to be blocked since it's a bug15:14
lasciiPaulMurray: 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 call15:14
openstackgerritNicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd  https://review.openstack.org/33120415:15
*** browne has joined #openstack-nova15:15
*** mmandel has quit IRC15:19
*** sdake has quit IRC15:20
*** tbachman has joined #openstack-nova15:20
bauwserlascii: melwitt: PaulMurray: yup, sorry, I was unclear, I was referring to the RPC calls15:23
*** cdelatte has quit IRC15:24
*** mdrabe has quit IRC15:24
openstackgerritAnusha Unnam proposed openstack/nova: [WIP]Improve help text for compute_opts  https://review.openstack.org/32951415:25
*** afazekas|dentist is now known as afazekas15:25
*** pacoblox has quit IRC15:25
*** Kevin-Zhao has quit IRC15:26
openstackgerritAnusha Unnam proposed openstack/nova: Transform instance.resize notifications  https://review.openstack.org/32925515:27
*** mgoddard_ has joined #openstack-nova15:27
openstackgerritAndrew Laski proposed openstack/nova: Add CellMappingList object  https://review.openstack.org/32921615:28
*** cdelatte has joined #openstack-nova15:28
*** rmart04 has quit IRC15:29
*** mgoddard has quit IRC15:31
*** ijw has joined #openstack-nova15:34
*** mriedem has quit IRC15:35
*** sree has joined #openstack-nova15:36
*** jdillaman has quit IRC15:37
openstackgerritJulian proposed openstack/nova: WIP Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2)  https://review.openstack.org/32543515:37
*** mmandel has joined #openstack-nova15:37
*** mriedem has joined #openstack-nova15:38
*** yamahata has joined #openstack-nova15:38
*** belmoreira has quit IRC15:40
*** pacoblox has joined #openstack-nova15:40
*** tbachman has quit IRC15:41
openstackgerritAnusha Unnam proposed openstack/nova: Transform instance.resize notifications  https://review.openstack.org/32925515:41
*** eglynn has quit IRC15:41
*** Kevin-Zhao has joined #openstack-nova15:44
*** flwang has quit IRC15:44
openstackgerritRonald Bradford proposed openstack/nova: api-ref: parameter verification os-fixed-ips  https://review.openstack.org/32612015:44
*** paul-carlton2 has quit IRC15:47
*** flwang has joined #openstack-nova15:48
openstackgerritMatt Riedemann proposed openstack/nova: Log the network when neutron won't apply security groups  https://review.openstack.org/33123615:51
*** mdrabe has joined #openstack-nova15:51
*** sdake has joined #openstack-nova15:51
*** matrohon has quit IRC15:54
*** sree has quit IRC15:55
*** Kevin-Zhao has quit IRC15:55
*** Kevin-Zhao has joined #openstack-nova15:55
auggygood morning!15:56
*** mgoddard_ has quit IRC15:59
*** tonytan4ever has joined #openstack-nova15:59
*** mgoddard has joined #openstack-nova15:59
*** sree has joined #openstack-nova15:59
*** sree has quit IRC15:59
*** adisky has quit IRC16:00
openstackgerritRonald Bradford proposed openstack/nova: api-ref: parameter verification os-agents  https://review.openstack.org/32611616:00
stewie925good morning auggy!16:01
*** permalac has quit IRC16:03
hemnamriedem, hey16:05
hemnamriedem, 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 IRC16:06
*** sdake has quit IRC16:07
mriedemhemna: it logs a trace and warning each time it hits that16:07
mriedemthere should be a way to suppress that if it's not an error16:07
mriedemhemna:  and from what i can tell these aren't FC drivers16:08
mriedemso not sure why it would be checking that anyway16:08
*** sridharg has joined #openstack-nova16:08
*** tbachman has joined #openstack-nova16:08
*** dmk0202 has joined #openstack-nova16:09
*** mgoddard has quit IRC16:09
*** hoangcx has quit IRC16:11
hemnaevery time a connector dict is built we have to call each connector object to collect it's info from the system16:11
hemnawe don't know if we are doing an iSCSI attach or FC attach16:11
hemnaor RBD, etc.16:11
*** mdavidson has quit IRC16:11
*** jogo has quit IRC16:12
hemnaso, I can see if I can redirect stderr to /dev/null to suppress those I guess16:12
*** tbachman_ has joined #openstack-nova16:12
mriedemhemna: is this because of your connector refactor change?16:12
hemnanot really16:13
hemnathe FC connector was always called16:13
*** pacoblox has quit IRC16:13
hemnasince FC was added into libvirt way back16:13
hemnaprivsep is just logging everything now16:13
openstackgerritJohn Garbutt proposed openstack/nova: Reorder allocate_for_instance preamble  https://review.openstack.org/32985016:13
hemnawe used to just capture stderr w/ putils and ignore it16:13
*** cdent has quit IRC16:14
mriedemhmm, privsep doesn't take a flag or something to tell it to be quiet?16:14
*** tbachman has quit IRC16:14
*** tbachman_ is now known as tbachman16:14
hemnaI'll look into it16:14
hemnaI'm not sure, but if it doesn't it should16:14
hemnaI don't think it should be automatically logging command stderr16:14
hemnait should leave it to who calls privsep exec16:15
*** gjayavelu has quit IRC16:15
mriedemok, i opened a bug against os-brick, maybe that just gets oslo.privsep added to it16:16
mriedemlow priority16:16
hemnakew16:16
*** pacoblox has joined #openstack-nova16:16
hemnamriedem, https://github.com/openstack/oslo.privsep/blob/master/oslo_privsep/daemon.py#L28516:17
hemnathat's the issue16:17
hemnastderr=_fd_logger()16:17
hemnait always logs all errors16:17
hemnait should capture those and return them to the caller IMHO16:18
mriedemah16:18
*** sridharg has quit IRC16:18
hemnais gus still up?16:19
hemnashould I abandon my os-brick patch for the grenade issue ?16:19
openstackgerritJohn Garbutt proposed openstack/nova: WIP Extract _update_ports_for_instance  https://review.openstack.org/32985116:20
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Extract port create from allocate_for_instance  https://review.openstack.org/32896616:20
mriedemhemna: i think so16:20
mriedemi posted comments why in there16:20
*** edand has joined #openstack-nova16:20
hemnahttps://review.openstack.org/#/c/329766/16:21
hemnathat one ?16:21
mriedemno16:23
mriedemhemna: this one https://review.openstack.org/#/c/329586/16:23
hemnaoh yah, sorry.16:23
hemnaI'm looking for the privsep patch that fixes the same issue16:23
openstackgerritSTEW TY proposed openstack/nova: Complete verification for os-floating-ips  https://review.openstack.org/32970416:24
*** takedakn has quit IRC16:25
mriedemhemna: that's this https://review.openstack.org/#/c/329766/16:25
mriedemhemna: used by nova here https://review.openstack.org/#/c/329769/16:25
hemnaah ok, so he's setting it to rootwrap from the get go.16:26
hemnaok I'll move my os-brick patch to WIP until this stuff lands16:26
hemnaand then I'll abandon it then.16:26
mriedemi'd like to see that pass gate-grenade-dsvm-multinode first,16:27
mriedemso i'm going to see if a devstack or devstack-gate change with both as dependencies will make that happen16:27
mriedemthe nova jobs don't run with the oslo.privsep from source16:27
mriedemso the dep doesn't work16:27
hemnaok I WIP'd mine16:28
hemnajust to prevent it from landing for the time being.16:28
*** Oku_OS is now known as Oku_OS-away16:28
openstackgerritMatt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support  https://review.openstack.org/31895316:29
openstackgerritMatt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance  https://review.openstack.org/31627516:29
openstackgerritMatt Riedemann proposed openstack/nova: Add REST API support for get me a network  https://review.openstack.org/31639816:29
*** Oku_OS-away is now known as Oku_OS16:29
*** mriedem is now known as mriedem_lunch16:30
hemnamriedem_lunch, thanks for the help16:31
auggyi 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't16: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 time16:33
auggygenerally we've been telling them to set the status to "In Progress" and push a patch within 2 weeks16:33
auggybut, shouldn't it go through a proper Confirmed/Triage process?16:33
auggyor is that necessary?16:33
auggybecause 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-nova16:35
*** Oku_OS is now known as Oku_OS-away16:36
*** Apoorva has joined #openstack-nova16:36
openstackgerritPushkar Umaranikar proposed openstack/nova: Config Options: Improve help text for debugger  https://review.openstack.org/31182216:36
*** Apoorva has quit IRC16:37
*** zenoway has quit IRC16:37
openstackgerritPushkar Umaranikar proposed openstack/nova: "nova list-extensions" not showing summary for all  https://review.openstack.org/28613016:37
*** tbachman has quit IRC16:37
*** tbachman_ is now known as tbachman16:37
*** smatzek has quit IRC16:37
openstackgerritPushkar Umaranikar proposed openstack/nova: Improve help text for consoleauth config options  https://review.openstack.org/31937816:41
*** pushkaru has joined #openstack-nova16:47
openstackgerritBob Ball proposed openstack/nova: WIP: XenAPI: Perform disk operations in dom0  https://review.openstack.org/33125416:47
*** ijw has quit IRC16:49
*** lucasagomes is now known as lucas-afk16:51
*** daemontool_ has quit IRC16:52
*** harlowja has joined #openstack-nova16:53
*** abalutoiu_ has quit IRC16:56
*** timello has quit IRC16:58
*** pacoblox has quit IRC17:02
*** Apoorva has joined #openstack-nova17:02
* PaulMurray says goodbye for the weekend17:04
*** PaulMurray has quit IRC17:05
*** diga has joined #openstack-nova17:05
*** sheel has quit IRC17:05
*** DaiDV has quit IRC17:06
*** irenab has joined #openstack-nova17:06
*** sudipto has quit IRC17:07
*** pacoblox has joined #openstack-nova17:07
*** tonytan4ever has quit IRC17:07
*** xyang1 has quit IRC17:08
*** xyang1 has joined #openstack-nova17:08
*** tblakeslee has quit IRC17:09
*** Kevin-Zhao has quit IRC17:10
*** irenab has quit IRC17:11
*** mvk_ has joined #openstack-nova17:11
*** amotoki has quit IRC17:13
*** timello has joined #openstack-nova17:13
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (10)  https://review.openstack.org/33101317:13
*** mkrcmari__ has quit IRC17:15
*** smatzek has joined #openstack-nova17:19
*** browne has quit IRC17:19
artomclaudiub, hey, I have some time now, if you're up for it :)17:20
openstackgerritJohn Garbutt proposed openstack/nova: WIP Extract _update_ports_for_instance  https://review.openstack.org/32985117:24
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Extract port create from allocate_for_instance  https://review.openstack.org/32896617:24
claudiubartom: 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
artomclaudiub, not a physical nic, no, it's the same vnic as would be attached without the tag.17:26
*** ctrath has quit IRC17:27
artomThe tag doesn't identify what vnic to attach, it identifies vnic metadata.17:27
artomSo 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 tag17:28
artom*that represents17:29
claudiubaham. so this basically a way tell the guest which nic is which.17:30
artomRight on.17:31
claudiubah, I see.17:31
*** diga has quit IRC17:31
*** ijw has joined #openstack-nova17:32
artomclaudiub, 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-nova17:36
*** edand has quit IRC17:36
claudiubartom: 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
claudiubartom: as far as the 'type', 'bus', 'mac', and 'tags' go, at least. the address looks like a pci device address.17:38
artomclaudiub, 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
artomclaudiub, exactly, bus and address are only known to the virt driver.17:39
*** cfriesen__ has joined #openstack-nova17:39
*** cfriesen_ has quit IRC17:39
claudiubartom: I see.17:40
*** sdake has joined #openstack-nova17:40
*** irenab has quit IRC17:41
*** fragatina has joined #openstack-nova17:43
*** cfriesen has joined #openstack-nova17:43
*** lpetrut has quit IRC17:43
claudiubartom: 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 IRC17:43
*** cfriesen__ has quit IRC17:44
*** sambetts is now known as sambetts|afk17:44
claudiubbut from what I see in the libvirt implementation, it isn't set either.17:45
*** jed56 has quit IRC17:45
artomclaudiub, well, we try to find its address17:46
artomhttps://review.openstack.org/#/c/264016/44/nova/virt/libvirt/driver.py 744917:46
artomBut it can be None if it's not feasible17:46
claudiubaah, I see. I didn't notice it, as the method is called _prepare_device_bus. :)17:47
claudiubok. I think I got it now.17:48
claudiubhopefully. :)17:48
artomclaudiub, yeah, maybe _prepare_device_bus isn't the clearest indication of what the method is actually doing17:49
claudiubI'll probably send a patch for this next week for hyper-v. Will add you as a reviewer. :)17:49
artomclaudiub, cool, I'll get to put my hyperextensive knowledge of hyperV to the test ;)17:49
*** kfarr has quit IRC17:50
claudiubartom: weeell, there is 0 or almost 0 hyper-v specific code in nova, so python knowledge will help you the most. :)17:50
artomclaudiub, sweet, I'll at least be able to fake it then :D17:52
claudiubartom: fake it 'til you make it, it is a good policy in life. :D17:52
*** yamahata has quit IRC17:53
*** ctrath has joined #openstack-nova17:57
*** tonytan4ever has joined #openstack-nova17:57
*** abalutoiu has joined #openstack-nova17:57
*** mriedem_lunch is now known as mriedem17:59
mriedemauggy: 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 review17:59
mriedemauggy: if they really think it's important they can ask about it here or in the dev list17:59
*** fragatina has quit IRC18:00
*** fragatina has joined #openstack-nova18:01
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: [WIP]Improve help text for cloudpipe  https://review.openstack.org/33128518:01
*** gjayavelu has joined #openstack-nova18:04
*** pacoblox has quit IRC18:04
*** harlowja has quit IRC18:04
*** tblakeslee has joined #openstack-nova18:06
*** browne has joined #openstack-nova18:08
*** stewie925 has quit IRC18:09
*** irenab has joined #openstack-nova18:09
*** stewie925 has joined #openstack-nova18:11
*** yassine__ has quit IRC18:11
mriedemsahid: 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 IRC18:12
*** thorst_ has quit IRC18:13
*** irenab has quit IRC18:13
*** thorst_ has joined #openstack-nova18:14
mriedemi guess if the image meta properties are telling the libvirt driver what to do then we should be ok18:15
mriedemand 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 hosts18:16
mriedems/object/config/18:16
mriedemseems we shouldn't even have the config option....18:16
*** tbachman has joined #openstack-nova18:16
*** thorst_ has quit IRC18:18
*** thorst_ has joined #openstack-nova18:19
*** ssurana has joined #openstack-nova18:23
mriedemsuperdan: my good friend! https://review.openstack.org/#/c/174854/34/nova/objects/image_meta.py18:23
mriedemImageMetaProps is currently at 1.1318:23
*** ctrath has quit IRC18:24
mriedemImageMeta used to bump versions when ImageMetaProps would, but no longer does18:24
superdanright18:24
mriedemso that's on purpose?18:24
superdanwhat's on purpose? not bumping it in this patch?18:24
mriedemwell it looks like we stopped bumping ImageMeta version when ImageMetaProps would change awhile back18:24
superdanright, because manifests18:25
mriedemi just wasn't sure if that was a conscious decision18:25
mriedemok18:25
superdant'was18:25
*** mkrcmari__ has joined #openstack-nova18:25
*** yamahata has joined #openstack-nova18:25
*** ssurana has quit IRC18:26
*** ssurana has joined #openstack-nova18:27
*** ctrath has joined #openstack-nova18:27
*** fragatina has quit IRC18:27
*** mvk_ has quit IRC18:28
*** mrkz has quit IRC18:29
*** irenab has joined #openstack-nova18:29
*** Sukhdev has joined #openstack-nova18:32
*** irenab has quit IRC18:34
*** aghadge has joined #openstack-nova18:34
*** ijw has quit IRC18:34
*** pushkaru has quit IRC18:36
openstackgerritmelanie witt proposed openstack/nova: Enable sqlite FK enforcement in Database fixture  https://review.openstack.org/33130518:39
mriedemmelwitt: ooo i've been wanting that made generic for a long time18:39
openstackgerritChris Friesen proposed openstack/nova: Allow logging detailed info only on sched failure  https://review.openstack.org/33130618:40
melwittyay :)18:40
*** simondodsley has joined #openstack-nova18:40
cfriesenhi 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-nova18:41
*** moshele has joined #openstack-nova18:45
*** stewie925 has quit IRC18:48
jdandreaCan 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
openstackgerritEd Leafe proposed openstack/nova: Correct the network config option help text  https://review.openstack.org/31872618:50
openstackgerritEd Leafe proposed openstack/nova: Improve the help text for the network options (3)  https://review.openstack.org/28190118:50
openstackgerritEd Leafe proposed openstack/nova: Improve the help text for the network options (4)  https://review.openstack.org/28203918:50
openstackgerritEd Leafe proposed openstack/nova: Improve the help text for the network options (2)  https://review.openstack.org/28043818:50
*** locust has joined #openstack-nova18:50
locusthello, as I see I can crete new instances of specified type like mini nano18:51
locustwill it be posseble to create my own types?18:52
locustlike create arm emulated mashine18:52
locustor change amount of rum belongig to specified mashine?18:53
*** aysyd has quit IRC18:58
*** locust_ has joined #openstack-nova18:59
locust_so18:59
locust_where there ansver?18:59
locust_sorry was disk18:59
figleaflocust_: those are admin-level operations19:00
*** locust has quit IRC19:00
figleaflocust_: if it's your cloud, then yes, you can do those things19:00
locust_hm19:00
locust_Iam considering openstack like esxi alternative19:01
*** mrkz has joined #openstack-nova19:01
*** moshele has quit IRC19:02
locust_figleaf: but it is not from horisont like...19:02
*** aysyd has joined #openstack-nova19:03
*** fragatina has joined #openstack-nova19:04
figleaflocust_: the best place to ask questions like that would be in the #openstack channel, not here19:05
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451119:06
*** efoley has quit IRC19:10
*** shakamunyi has joined #openstack-nova19:13
*** fragatina has quit IRC19:14
*** fragatina has joined #openstack-nova19:15
openstackgerritNicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd  https://review.openstack.org/20375119:18
*** zenoway has joined #openstack-nova19:20
*** MikeG451_ has quit IRC19:20
*** mfedosin has quit IRC19:22
*** stewie925 has joined #openstack-nova19:24
*** zenoway has quit IRC19:25
stewie925hello all19:25
stewie925was 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
openstackgerritAndrew Laski proposed openstack/nova: Cells: Handle delete with BuildRequest  https://review.openstack.org/32598519:27
mriedemstewie925: method verification was complete19:28
mriedemsee the .. needs: part of https://review.openstack.org/#/c/310706/1/api-ref/source/os-flavor-extra-specs.inc19:28
stewie925yes, mriedem - I see that too, but it looks like those other verifications have been done19:28
stewie925well, at least the parameter verificiation was done, but it wasnt stricken out from the source19:29
*** pacoblox has joined #openstack-nova19:30
mriedemstewie925: i think it was just method verification on that one19:30
*** pushkaru has joined #openstack-nova19:31
stewie925mriedem: I see....19:31
*** harlowja has joined #openstack-nova19:32
mriedemsuperdan: lascii: so i've got a riddle for you19:32
mriedemsince sdague is on a walkabout19:32
*** thumpba has joined #openstack-nova19:32
mriedemhttps://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
mriedembut the only thing that changes in that change is adding paging support for listing hypervisors,19:33
mriedemso it seems pointless to test GET on os-hypervisors with 2.31,19:33
mriedemunless that's just purely for regression testing19:33
*** Jeffrey4l has quit IRC19:34
*** ociuhandu has quit IRC19:34
mriedemnote 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 query19:34
superdanyeah19:34
*** dimtruck is now known as zz_dimtruck19:34
mriedemthe REST API unit test actually tests the limit/marker part19:35
*** ijw has joined #openstack-nova19:35
mriedemotherwise we're just duplicating and running a bunch of tests that work the same as beofre19:35
mriedem*before19:35
mriedemi have a feeling this might just be 'how it's done'19:36
superdanso what are you asking?19:37
superdanif we should have an exclusion list?19:37
mriedemwell, for api samples tests, should we only test listing hypervisors (what actually changed)19:37
mriedemeven though that api sample test doesn't test the change really since the body isn't changing19:38
mriedemsame for the unit tests since those are also just subclassed and pass a different microversion19:38
*** ssurana has quit IRC19:38
mriedemi'm just wary of subclassing tests because of all of the unnecessary duplication19:39
mriedemmaybe this is a ML question?19:39
*** tonytan4ever has quit IRC19:39
*** ijw has quit IRC19:40
superdanI'm not sure I'm following what you're saying19:40
superdanbut we have an exclusion list of db migrations that we don't require tests for,19:40
superdanlike the placeholder ones19:40
*** cdelatte has quit IRC19:40
mriedemi think we're talking about different things19:41
mriedemi'll just post to the dev list and sean can enlighten me when he's back19:42
mriedemthere are other problems in this change that have to be addressed until then anyway19:42
*** jichen has quit IRC19:45
melwittzzzeek: 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 tests19:46
openstackLaunchpad bug 1552351 in sqlalchemy-migrate "sqlite alter table messes foreign keys" [Undecided,New]19:46
*** mriedem has quit IRC19:46
zzzeekmelwitt: during a migration or during tests?19:47
*** Jeffrey4l has joined #openstack-nova19:47
melwittzzzeek: 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_84353119:47
*** thumpba has quit IRC19:48
melwittI wanted to check with you if there's any other option other than turning them off for migration tests19:48
*** samueldmq has quit IRC19:49
zzzeekmelwitt: not likely.  however is this a side effect of sqlalchemy-migrate's copy-and-replace scheme for SQLite tables ?19:49
*** Apoorva has quit IRC19:49
zzzeeki dont understand what "will also rename foreign key contraints from *other* tables to migration_tmp" means in the bz19:50
zzzeekthat's a rename of a table19:50
zzzeekmelwitt: for alembic, we require that pragma FKs is turned off for SQLite migrations.  so i'd stick w/ that19:51
melwittzzzeek: 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 fk19:51
melwittzzzeek: okay, thank you!19:52
zzzeekmelwitt: 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_all19:52
*** lpetrut has joined #openstack-nova19:52
*** mriedem has joined #openstack-nova19:53
stewie925hi all - am wondering how we can determine which Nova command invokes a Nova API rest method?19:53
melwittzzzeek: okay, good to know19:53
mriedemi've officially decided to quit and take up turnip farming19:53
mriedemstewie925: nova --debug ...19:53
mtreinishmriedem: why turnips? Do they grow well in rochester?19:54
*** ijw has joined #openstack-nova19:55
*** markvoelker has quit IRC19:55
*** raildo is now known as raildo-afk19:57
stewie925thank you mriedem19:58
mriedemmtreinish: because i imagine turnip farming has been perfected over thousands of years19:58
mriedemso i don't have to ask wtf?19:59
mriedemit's just, this is how you do turnips19:59
*** rama_y has quit IRC19:59
cfriesenso 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
doffmI'm sure I've seen a bug for this.20:06
doffm*goes searches for a bug.20:06
*** tbachman has quit IRC20:07
melwittcfriesen: have you seen this https://review.openstack.org/#/c/242251/20:07
*** thumpba has joined #openstack-nova20:07
*** rama_y has joined #openstack-nova20:07
*** tbachman has joined #openstack-nova20:08
*** zz_dimtruck is now known as dimtruck20:08
*** markvoelker has joined #openstack-nova20:10
*** rook-lappy has quit IRC20:14
*** rook-lappy has joined #openstack-nova20:14
cfriesenmelwitt: 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 space20:17
*** akshai has quit IRC20:18
*** Apoorva has joined #openstack-nova20:18
cfriesenmelwitt: but it's definitely related to the issue...will take a closer look20:18
openstackgerritEd Leafe proposed openstack/nova: Improve the help text for the linuxnet options (4)  https://review.openstack.org/27542320:18
mriedemcfriesen: i'm assuming the disk overcommit gets a pass b/c of the failure to accurately report shared storage?20:18
melwittcfriesen: yeah, it took me awhile to understand the patch and how it works with overcommit20:18
mriedemcfriesen: jaypipes would probably know about it20:19
*** thumpba has quit IRC20:19
*** thumpba has joined #openstack-nova20:20
cfriesenmriedem: 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
cfriesenThe patch melwitt pointed at is looking at host_state.free_disk_physical_mb, not the total available storage20:20
cfriesenso it's more strict20:20
melwittcfriesen: what do you mean by total available storage, total available on the hypervisor?20:21
cfriesenmelwitt. yes, total storage available for instance root disk, ephemeral, swap, etc.20:22
*** ihrachys has joined #openstack-nova20:22
*** irenab has joined #openstack-nova20:22
melwittI 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 overcommit20:22
cfriesenmelwitt: I just pointed out what appears to be a mismatch between comment and code in that patch20:23
*** thorst_ has quit IRC20:23
cfriesenmelwitt: 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
doffmcfriesen: Seems to.20:24
melwittandrewbogott: are you around?20:24
andrewbogottmelwitt: I am!  What's up?20:25
doffmcfriesen: This seems wrong. Free disk mb < request.20:25
melwittokay, I thought disk_available_least was the actual space used, which could be more than what the flavors disk total is with COW20:26
doffmBlocks all overcommit.20:26
* andrewbogott reads backscroll20:26
*** thumpba_ has joined #openstack-nova20:26
melwittactual space available, I mean20:26
*** irenab has quit IRC20:27
andrewbogottIt's been a few months since I thought about that patch… my intent was certainly to check the physical available space20:28
melwittthe names of existing variables are really confusing, IMO20:29
cfriesenI think you could just compare against  the existing "total_usable_disk_gb"20:29
*** thumpba has quit IRC20:29
doffmWhich is what the ram filter does.20:29
openstackgerritEd Leafe proposed openstack/nova: WIP - Accept multiple hosts for live migration  https://review.openstack.org/32780920:29
melwitttotal usable includes used though20:29
cfriesenmelwitt: yes20:29
doffmThey are totally confusing. I mean free available totally used free.20:30
cfriesenmelwitt: it does for the RAM FIlter too20:30
cfriesenmelwitt: 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 issue20:30
melwittokay ... 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 host20:31
cfriesenso 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 space20:31
doffmSo if you have 16gb of ram total you cant put a 32 gb instance on it even with a 2x overcommit.20:31
doffmBut you can put 3 8gb instances.20:31
*** ssurana has joined #openstack-nova20:32
cfriesendoffm: exactly20:32
*** aghadge has quit IRC20:34
*** mvk_ has joined #openstack-nova20:34
melwittcfriesen, 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#L41020:37
*** ssurana has quit IRC20:37
*** mkrcmari__ has quit IRC20:38
*** figleaf is now known as edleafe20:38
*** ihrachys has quit IRC20:38
melwittit is subtracting overcommitted for available_least https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L519920:39
melwittsigh ... need to go back through all of this20:40
andrewbogottOlder versions of my patch just used compute.free_disk_gb20:40
andrewbogottIs it possible that this confusion is just due to a sloppy merge on my part?20:41
andrewbogotthttps://review.openstack.org/#/c/242251/5/nova/scheduler/host_manager.py20:41
andrewbogottOr is compute.free_disk_gb /also/ not the value I want?20:41
*** edleafe has quit IRC20:41
cfriesenandrewbogott: 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 amount20:41
melwittandrewbogott: no, I think jaypipes asked you to change it to available_least IIRC20:41
*** edleafe has joined #openstack-nova20:42
cfriesenandrewbogott: I think it should be:   if host_state.total_usable_disk_gb < requested_disk:20:42
cfriesenany checks against the "free" space are not about overcommit against itself, which is what the comment talks about20:43
melwittcfriesen: 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 space20:43
andrewbogottmelwitt: 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
cfriesenmelwitt: right, that's a separate issue from overcommit against itself.20:43
*** pushkaru has quit IRC20:44
andrewbogottSomeone (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 IRC20:44
*** inc0 has joined #openstack-nova20:44
andrewbogottWhich fixed my particular concern, more or less, so I tried to mirror the RAM filter code20:44
andrewbogottFrom 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
cfriesenandrewbogott: 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
andrewbogottI don't know if that clarifies20:45
*** tblakeslee has joined #openstack-nova20:45
doffmandrewbogott: Yeah, that is not really what the checks in the core and ram filters are doing.20:45
doffmIf you look later in the disk filter function the 'if not usable_disk_mb >= requested_disk:' check happens.20:46
melwittokay, so none of the filters do a check for literal free resources?20:46
cfriesenmelwitt: that's later...separate from the "overcommit against itself" check20:46
mriedemshite20:46
*** fragatina has quit IRC20:47
mriedemthe massive config option centralization thing has it's foibals20:47
andrewbogottmelwitt: 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 schedule20:47
melwittcfriesen: so how does it schedule to a full host if it ever checks the literally free space?20:47
*** MikeG451_ has joined #openstack-nova20:47
mriedemwe probably should have had a job that listed deprecated options before and after a change to make sure the diff didn't change20:47
mriedemhttps://review.openstack.org/#/c/309239/6/nova/conf/libvirt.py@6020:47
lasciimriedem: I was out for your earlier riddle re: sample tests, but I responded on the ML20:47
auggymriedem: mtreinish: if you started with beets and switched to turnips, you could say would have to turnip the beets20:48
*** thorst_ has joined #openstack-nova20:49
mtreinishauggy: haha20:49
*** jamesdenton has quit IRC20:49
cfriesenmelwitt: 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
openstackgerritNicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd  https://review.openstack.org/33135020:50
mriedemauggy: as in, turn up the beats?20:51
mriedemauggy: that is not something i would ever say20:51
andrewbogottTo clarify, we should check if there's room for this instance WITHOUT factoring in overcommit20:51
auggybut you *could*20:51
cfriesenandrewbobott: for the "don't overcommit against yourself" test, yes.20:51
mriedemauggy: never20:51
auggyor if your beats were particularly large, you could say you have fat beets20:51
mriedemauggy: i might say, turn up the jams20:52
auggyooh beat jam20:52
auggybeet jam rather20:52
auggyi wonder if turnip jam is a thing20:52
cfriesenandrewbogott: (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-nova20:53
andrewbogottcfriesen: I'm sorry, I'm not following.20:53
andrewbogottCan you give me an example?20:53
andrewbogottOr, actually — here's a different question20:54
andrewbogottSuppose a host's drive is literally, physically full20:54
andrewbogottbut the overcommit ration still says there's space20:54
andrewbogottWhere is the code that prevents us from scheduling on that box?20:54
doffmandrewbogott: There isn't any.20:54
doffmYou schedule to it.20:54
andrewbogottok!20:54
*** mkrcmari__ has joined #openstack-nova20:54
andrewbogottAnd, do we agree that that's a bad thing?20:54
doffmThat is what your overcommit tells you to do.20:54
andrewbogottSo that's a "not agree"?20:55
doffmNo I agree.20:55
doffmDisk overcommit is different to RAM and vCPU.20:55
doffmGo over limits there and there isn't really a problem.20:56
doffmGo over actual disk limits and the build will fail.20:56
melwittright20:56
andrewbogottok, great, happy to hear that :)20:56
openstackgerritMatt Riedemann proposed openstack/nova: Re-deprecate use_usb_tablet config option  https://review.openstack.org/33135320:56
doffmSo there should be THREE checks in this file.20:57
andrewbogottSo… (I'm going back to the beginning here, sorry)20:57
melwittdisk should have extra stuff in it bc of that20:57
andrewbogottWhat should we consider 'too full for more scheduling'?20:57
*** fragatina has quit IRC20:57
mriedemlascii: 20 seconds per test or just the test class itself20:57
*** mvk_ has quit IRC20:57
andrewbogottOne option is:  we need at least (requested size / overcommit size)20:58
lasciimriedem: the test class. 8 tests20:58
andrewbogottanother option is we need at least (requested size)20:58
melwittbut 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
mriedemlascii: ok20:58
doffmNot the instance disk size. Imagine you have an instance that has a requested disk of 400gb.20:58
mriedemnot as bad then20:58
andrewbogottanother is we need at least (user-specified %)20:58
doffmAnd you only have 200gb actually free.20:58
doffmYou should probably still schedule there. If you have a big overcommit.20:58
mriedemlascii: superdan: also, we should get this in, super easy https://review.openstack.org/33135320:58
mriedemso we don't lose the removal window20:58
andrewbogottok… so you're voting for (requested size / overcommit size)20:58
doffmI guess i am.20:59
doffmBut 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
superdanmriedem: why was it undeprecated?20:59
andrewbogottdoffm: Hah!  A configurable limit was what I request in the original bug20:59
doffmOh. :)20:59
*** smatzek has quit IRC21:00
andrewbogottlemme see if I can dig up that patch21:00
*** ivase has quit IRC21:01
*** mvk has joined #openstack-nova21:01
*** ivase has joined #openstack-nova21:01
lasciisuperdan: it seems it was accidentally undeprecated when the option moved to a central location21:01
*** mkrcmari__ has quit IRC21:02
*** ssurana has joined #openstack-nova21:02
superdanlascii: ah21:02
*** davidlenwell has quit IRC21:02
andrewbogottdoffm: so, check out patchset 4, and comments there: https://review.openstack.org/#/c/242251/421:03
*** ssurana has quit IRC21:04
openstackgerritmelanie witt proposed openstack/nova: WIP Make InstanceGroup object favor the API database  https://review.openstack.org/33099521:04
mriedemsuperdan: goof up21:05
superdanaye21:05
doffmandrewbogott: Ok, will review.21:05
openstackgerritDiana Clarke proposed openstack/nova: Image: add create_from_image & create_from_func  https://review.openstack.org/31894821:06
*** ivase has quit IRC21:06
andrewbogottdoffm: 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 IRC21:06
*** zenoway has joined #openstack-nova21:07
*** ivase has joined #openstack-nova21:07
cfriesenandrewbogott: 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
andrewbogottcfriesen: by 'exists' you mean, total disk size, not free space, correct?21:09
cfriesenandrewbogott: 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 grow21:09
cfriesenand 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 disk21:10
melwittcfriesen: 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 cases21:10
andrewbogottcfriesen: how would 2) pass but 3) fail?21:10
andrewbogottWould it be with an overcommit ration of <1.0?21:10
cfriesenmelwitt: 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 size21:11
*** ivase has quit IRC21:11
cfriesenandrewbogott: if you already have the backing file downloaded, then the incremental cost of an instance is basically nothing until it starts writing to its disk21:11
melwittcfriesen: okay, I thought that meant the requested disk size. you mean only the image size21:11
doffmJeez, working out what the 'least' value means in the libvirt code is not fun.21:11
cfriesenandrewbogott:  assuming qcow2 here21:12
andrewbogottcfriesen: ok, I see what you're saying.  So 2) should really check the image size against available disk space21:12
*** daemontool has joined #openstack-nova21:12
melwittdoffm: tell me about it!21:12
andrewbogott(this, presuming we have any idea what available disk space is)21:12
*** thorst_ has quit IRC21:12
*** thumpba_ has quit IRC21:12
andrewbogottor does it need to be smarter, determine w/not the image is already present?21:13
cfriesenandrewbogott: yes, and ideally for qcow2 we should check whether we've already got the backing file cached or not21:13
*** davidlenwell has joined #openstack-nova21:13
*** ametts has quit IRC21:13
*** pacoblox has quit IRC21:13
*** ivase has joined #openstack-nova21:13
andrewbogottcfriesen: ok, I follow what you're saying now21:13
*** pacoblox has joined #openstack-nova21:13
cfriesenthe theoretical max size consumed by N qcow instances is sum(unique backing files) + sum(instance flavor disks)21:14
andrewbogottcfriesen: your scheme doesn't allow for slack of any kind, but maybe that's an unrelated issue?21:14
cfriesenandrewbogott: thats where the ratio comes in.  you want more slack, lower the ratio21:14
andrewbogottso...21:14
cfriesenor maybe you need a separate "slack" config option21:14
cfriesen:)21:14
cfriesenoh, and just to make things interesting, for some reason we have a backing store file for swap disks.21:15
*** stewie925 has quit IRC21:16
andrewbogottSo, 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
doffmOk, 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
andrewbogott100% capacity + overprovision > 1.0 = immediate disaster21:16
cfriesendoffm: that's my understanding21:16
doffmWhich means cfriesen is probably right and the patch is currently blocking overcommit.21:16
andrewbogottcfriesen: So, you've outlined a useful way to hit 100% on the mark, but that's not a thing we want to do21: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 IRC21:17
cfriesenandrewbogott: 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
andrewbogottcfriesen: So, I propose that I revive the early version of my patch (explicit slack)21:19
andrewbogotthttps://review.openstack.org/#/c/242251/421:19
andrewbogottand create a different bug about scheduling precision21:20
cfriesenandrewbogott: I think your current patch has some usefulness too, just change it to look at the total disk space instead of free.21:20
doffmcfriesen: +121:20
andrewbogottcfriesen: ok — at which point it will be a first approximation of goal 2) ?21:20
*** ssurana has joined #openstack-nova21:21
doffmandrewbogott: Yeah, the issue is really the use of the 'least' rather than 'really_free'21:21
cfriesenno, goal 121:21
andrewbogottAh!  Yes, ok, I remember 121:21
openstackgerritRawlin Peters proposed openstack/os-vif: Ensure the OVS bridge exists when plugging  https://review.openstack.org/33081821:21
andrewbogottso I will be accidentally fixing a problem that never occurred to me, I can live with that :)21:21
andrewbogottstay tuned...21:21
cfriesen:)21:21
*** markvoelker has quit IRC21:21
*** rtheis has quit IRC21:22
cfriesenandrewbogott: that will make it be exactly analogous to the "don't overcommit against yourself" checks in the cpu/ram filters21:22
* andrewbogott nods21:22
*** stewie925 has joined #openstack-nova21:22
*** ijw has quit IRC21:23
*** ijw has joined #openstack-nova21:23
*** irenab has joined #openstack-nova21:24
doffmYay!21:24
*** ssurana has quit IRC21:25
*** ssurana has joined #openstack-nova21:25
*** dave-mccowan has quit IRC21:26
*** tbachman_ has joined #openstack-nova21:26
*** lascii is now known as alaski21:28
*** irenab has quit IRC21:28
*** tbachman has quit IRC21:29
*** tbachman_ is now known as tbachman21:29
openstackgerritRawlin Peters proposed openstack/os-vif: Ensure the OVS bridge exists when plugging  https://review.openstack.org/33081821:30
*** rfolco has quit IRC21:32
*** dmk0202 has quit IRC21:33
*** dmk0202 has joined #openstack-nova21:34
*** thorst has joined #openstack-nova21:36
*** mdrabe has quit IRC21:36
*** erlon has quit IRC21:36
openstackgerritWalter A. Boring IV (hemna) proposed openstack/nova: Rename libvirt conf entry iscsi_use_multipath  https://review.openstack.org/33135721:38
*** pacoblox has quit IRC21:39
*** thorst has quit IRC21:39
*** thorst has joined #openstack-nova21:40
mriedemwhat's up with this? https://github.com/openstack/nova/blob/master/setup.cfg#L4121:40
mriedemi guess vestigial? https://review.openstack.org/#/c/240159/3/setup.cfg21:41
mriedemdidn't we already drop all of the ERT stuff?21:42
superdanprobably vestigal21:43
superdanlike your tailbone21:43
*** sahid has quit IRC21:44
mriedemi still use mine21:45
*** pacoblox has joined #openstack-nova21:45
openstackgerritMatt Riedemann proposed openstack/nova: Remove the nova.compute.resources entrypoint  https://review.openstack.org/33135821:46
*** tblakeslee has quit IRC21:47
*** xyang1 has quit IRC21:48
*** thorst has quit IRC21:49
*** dmk0202 has quit IRC21:50
*** pacoblox has quit IRC21:51
*** daemontool has quit IRC21:53
*** pwnall has quit IRC21:56
*** pacoblox has joined #openstack-nova21:57
*** ctrath has quit IRC21:58
*** mriedem has quit IRC21:59
*** pacoblox has quit IRC22:01
*** pacoblox has joined #openstack-nova22:02
*** kylek3h has quit IRC22:02
*** lpetrut has quit IRC22:04
*** pacoblox has quit IRC22:05
openstackgerritAndrew Bogott proposed openstack/nova: Prevent scheduling of instances that can't possibly fit  https://review.openstack.org/24225122:05
*** pacoblox has joined #openstack-nova22:06
andrewbogottcfriesen, 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 runs22:06
*** ijw has quit IRC22:07
*** mmandel has quit IRC22:08
*** pushkaru has joined #openstack-nova22:10
openstackgerritAnusha Unnam proposed openstack/nova: Improve help text for compute_opts  https://review.openstack.org/32951422:11
*** baoli has quit IRC22:13
*** dimtruck is now known as zz_dimtruck22:13
*** pradk has quit IRC22:13
*** burt has quit IRC22:13
*** ijw has joined #openstack-nova22:13
*** fragatina has joined #openstack-nova22:14
*** fragatin_ has joined #openstack-nova22:15
*** efried has quit IRC22:17
*** zz_dimtruck is now known as dimtruck22:18
*** fragatina has quit IRC22:19
*** dimtruck is now known as zz_dimtruck22:20
*** rahuls has quit IRC22:24
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451122:32
*** abalutoiu has quit IRC22:37
*** sdake has quit IRC22:38
*** mvk_ has joined #openstack-nova22:39
*** sdake has joined #openstack-nova22:41
*** mvk has quit IRC22:42
openstackgerritPushkar Umaranikar proposed openstack/nova: Transform instance.suspend notifications  https://review.openstack.org/32908922:44
*** thorst has joined #openstack-nova22:45
*** aysyd has quit IRC22:45
openstackgerritAugustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys  https://review.openstack.org/32969722:45
*** MikeG451_ has quit IRC22:45
*** syjulian has quit IRC22:48
*** pacoblox has quit IRC22:48
*** pacoblox has joined #openstack-nova22:48
openstackgerritPushkar Umaranikar proposed openstack/nova: Transform instance.suspend notifications  https://review.openstack.org/32908922:50
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451122:51
*** pacoblox has quit IRC22:51
*** thorst has quit IRC22:53
openstackgerritPushkar Umaranikar proposed openstack/nova: Transform instance.suspend notifications  https://review.openstack.org/32908922:53
*** zenoway has quit IRC22:55
*** zenoway has joined #openstack-nova22:56
*** pacoblox has joined #openstack-nova22:56
*** edmondsw has quit IRC22:59
*** mrkz has quit IRC23:00
*** zenoway has quit IRC23:01
*** rfolco has joined #openstack-nova23:05
*** ctrath has joined #openstack-nova23:06
*** rama_y has quit IRC23:07
*** rama_y has joined #openstack-nova23:08
*** zenoway has joined #openstack-nova23:08
*** ssurana1 has joined #openstack-nova23:12
*** sacharya_ has quit IRC23:13
*** rrecio_ has quit IRC23:13
*** ssurana has quit IRC23:15
*** markvoelker has joined #openstack-nova23:16
*** omnipresent has joined #openstack-nova23:17
*** stewie925 has quit IRC23:17
*** zenoway has quit IRC23:20
*** fragatin_ has quit IRC23:21
*** dave-mccowan has joined #openstack-nova23:22
*** rfolco has quit IRC23:24
*** ssurana1 has quit IRC23:25
*** fragatina has joined #openstack-nova23:25
*** dave-mccowan has quit IRC23:26
*** fragatina has quit IRC23:26
*** fragatina has joined #openstack-nova23:26
*** pushkaru has quit IRC23:37
*** claudiub has quit IRC23:38
*** yamahata has quit IRC23:45
*** eharney has quit IRC23:46
*** pushkaru has joined #openstack-nova23:47
*** zz_dimtruck is now known as dimtruck23:50
*** thorst has joined #openstack-nova23:51
*** rrecio has joined #openstack-nova23:53
*** rrecio_ has joined #openstack-nova23:55
*** fragatina has quit IRC23:55
*** Sukhdev has quit IRC23:55
*** rrecio has quit IRC23:58
*** thorst has quit IRC23:59

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