Monday, 2017-12-04

*** itlinux has joined #openstack-nova00:03
openstackgerritPeter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver.  https://review.openstack.org/14073300:08
*** thorst has joined #openstack-nova00:16
*** yangyapeng has quit IRC00:16
*** yangyapeng has joined #openstack-nova00:16
*** sree has joined #openstack-nova00:17
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref  https://review.openstack.org/52300700:19
*** yangyapeng has quit IRC00:21
*** thorst has quit IRC00:21
*** Oku_OS-away has joined #openstack-nova00:22
*** sree has quit IRC00:22
*** yassine has joined #openstack-nova00:29
*** yassine is now known as Guest6895000:30
*** david-lyle has joined #openstack-nova00:30
*** david-lyle has quit IRC00:31
*** david-lyle has joined #openstack-nova00:31
*** cheneydc has joined #openstack-nova00:33
*** salv-orlando has joined #openstack-nova00:44
*** jmlowe has joined #openstack-nova00:47
*** salv-orlando has quit IRC00:49
*** thorst has joined #openstack-nova00:52
*** huanxie has quit IRC00:54
*** thorst has quit IRC00:57
*** yangyapeng has joined #openstack-nova00:58
*** slaweq has joined #openstack-nova00:59
*** jichen has joined #openstack-nova00:59
*** psachin has joined #openstack-nova01:00
*** yangyapeng has quit IRC01:03
*** slaweq has quit IRC01:03
*** phuongnh has joined #openstack-nova01:06
*** yangyapeng has joined #openstack-nova01:08
*** trungnv has joined #openstack-nova01:17
*** edmondsw has joined #openstack-nova01:19
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations  https://review.openstack.org/51727301:21
*** edmondsw has quit IRC01:24
*** tetsuro has joined #openstack-nova01:25
*** annp has quit IRC01:27
*** thorst has joined #openstack-nova01:27
*** yangyapeng has quit IRC01:27
*** yangyapeng has joined #openstack-nova01:28
*** yamahata has quit IRC01:28
*** thorst has quit IRC01:28
*** thorst has joined #openstack-nova01:29
*** thorst has quit IRC01:30
*** claudiub has quit IRC01:35
*** jmlowe has quit IRC01:44
*** salv-orlando has joined #openstack-nova01:45
*** yamamoto has joined #openstack-nova01:45
*** salv-orlando has quit IRC01:49
*** ankit_ has joined #openstack-nova01:49
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.56 - Enable cold migration with target host  https://review.openstack.org/40670701:49
*** Yingxin has joined #openstack-nova01:52
openstackgerritZhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance()  https://review.openstack.org/52404701:53
*** gmann_afk is now known as gmann01:54
*** thorst has joined #openstack-nova02:01
*** thorst has quit IRC02:06
*** lei-zh has joined #openstack-nova02:06
*** sree has joined #openstack-nova02:08
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Update Instance action's updated_at when action event updated.  https://review.openstack.org/50747302:08
*** sree_ has joined #openstack-nova02:10
*** thorst has joined #openstack-nova02:10
*** sree_ is now known as Guest2515902:10
*** Guest25159 has quit IRC02:12
*** sree has quit IRC02:14
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for shelve_offload instances  https://review.openstack.org/52365302:14
*** thorst has quit IRC02:14
*** phuongnh has quit IRC02:15
*** phuongnh has joined #openstack-nova02:16
*** gbarros has quit IRC02:23
*** Apoorva has quit IRC02:23
*** TuanLA has joined #openstack-nova02:27
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances  https://review.openstack.org/52367602:34
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632602:38
*** salv-orlando has joined #openstack-nova02:45
*** salv-orlando has quit IRC02:50
*** sridharg has joined #openstack-nova02:51
*** edmondsw has joined #openstack-nova03:07
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Genericify the instance_list stuff  https://review.openstack.org/52425303:09
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Split instance_list into instance and multi_cell  https://review.openstack.org/52430203:09
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support.  https://review.openstack.org/51490403:09
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Implement query param schema for migration index  https://review.openstack.org/51864403:09
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations  https://review.openstack.org/51727303:09
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040603:09
*** edmondsw has quit IRC03:12
*** cheneydc has quit IRC03:14
*** thorst has joined #openstack-nova03:17
*** sree has joined #openstack-nova03:20
*** thorst has quit IRC03:22
openstackgerritguanzuoyu proposed openstack/nova master: Use actual availability zone rather than original value  https://review.openstack.org/52503403:23
openstackgerritZhenyu Zheng proposed openstack/nova master: Mention API behavior change when over quota limit  https://review.openstack.org/51563103:29
*** itlinux has quit IRC03:32
*** edmondsw has joined #openstack-nova03:32
*** edmondsw has quit IRC03:36
*** slaweq has joined #openstack-nova03:46
*** salv-orlando has joined #openstack-nova03:46
*** tetsuro has quit IRC03:49
*** slaweq has quit IRC03:50
*** salv-orlando has quit IRC03:51
*** yamamoto has quit IRC03:53
*** yamamoto has joined #openstack-nova03:55
*** thorst has joined #openstack-nova03:57
*** thorst has quit IRC04:02
*** annp has joined #openstack-nova04:10
*** links has joined #openstack-nova04:13
*** jmlowe has joined #openstack-nova04:17
*** yamahata has joined #openstack-nova04:21
*** sree has quit IRC04:23
*** udesale has joined #openstack-nova04:25
*** udesale has quit IRC04:25
*** sree has joined #openstack-nova04:25
*** udesale has joined #openstack-nova04:25
*** adisky_ has joined #openstack-nova04:31
*** thorst has joined #openstack-nova04:37
*** itlinux has joined #openstack-nova04:40
*** thorst has quit IRC04:41
*** lei-zh has quit IRC04:43
*** salv-orlando has joined #openstack-nova04:47
*** yamamoto_ has joined #openstack-nova04:47
*** yamamoto has quit IRC04:51
*** salv-orlando has quit IRC04:52
openstackgerritjichenjc proposed openstack/nova master: remove glance usage inside compute  https://review.openstack.org/51139204:53
openstackgerritjichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url  https://review.openstack.org/51139704:53
*** edmondsw has joined #openstack-nova04:58
*** lei-zh has joined #openstack-nova05:01
*** jmlowe has quit IRC05:02
openstackgerritjichenjc proposed openstack/nova master: remove reserve_quota_delta  https://review.openstack.org/52146905:02
openstackgerritjichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback  https://review.openstack.org/52147005:02
*** edmondsw has quit IRC05:03
*** janki has joined #openstack-nova05:05
openstackgerritjichenjc proposed openstack/nova master: Remove 'nova-manage host' and 'nova-manage agent'  https://review.openstack.org/52183705:07
openstackgerritjichenjc proposed openstack/nova master: Remove 'nova-manage shell' command  https://review.openstack.org/52183505:07
openstackgerritjichenjc proposed openstack/nova master: Remove 'nova-manage account' and 'nova-manage project'  https://review.openstack.org/52183305:07
openstackgerritjichenjc proposed openstack/nova master: Remove 'nova-manage logs' command  https://review.openstack.org/52213305:07
*** thorst has joined #openstack-nova05:10
*** Alex_Staf has joined #openstack-nova05:11
*** gcb has joined #openstack-nova05:11
*** thorst has quit IRC05:15
*** jaosorior has joined #openstack-nova05:16
*** Alex_Staf has quit IRC05:20
*** fragatina has quit IRC05:21
*** fragatina has joined #openstack-nova05:21
*** yikun has quit IRC05:26
*** itlinux has quit IRC05:28
*** anthonyper has quit IRC05:29
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: validate scsi controller model  https://review.openstack.org/52505505:29
*** fragatina has quit IRC05:30
*** zhurong has joined #openstack-nova05:31
*** andymccr has quit IRC05:32
*** andymccr has joined #openstack-nova05:33
*** anthonyper has joined #openstack-nova05:33
*** thorst has joined #openstack-nova05:37
*** anthonyper has quit IRC05:38
*** thorst has quit IRC05:42
*** anthonyper has joined #openstack-nova05:42
*** thorst has joined #openstack-nova05:45
*** ratailor has joined #openstack-nova05:46
*** salv-orlando has joined #openstack-nova05:48
*** thorst has quit IRC05:50
*** moshele has joined #openstack-nova05:51
*** hshiina has joined #openstack-nova05:51
*** salv-orlando has quit IRC05:53
*** clarkb has quit IRC06:02
*** gongysh has joined #openstack-nova06:15
*** thorst has joined #openstack-nova06:24
*** Tom-Tom has joined #openstack-nova06:26
*** thorst has quit IRC06:29
*** yamahata has quit IRC06:31
*** xinliang has quit IRC06:32
*** clarkb has joined #openstack-nova06:33
*** suresh12 has joined #openstack-nova06:36
*** salv-orlando has joined #openstack-nova06:40
*** slaweq has joined #openstack-nova06:41
*** xinliang has joined #openstack-nova06:44
openstackgerritHironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648706:44
*** slaweq has quit IRC06:45
*** lajoskatona has joined #openstack-nova06:46
*** edmondsw has joined #openstack-nova06:46
*** edmondsw has quit IRC06:51
*** thorst has joined #openstack-nova06:57
*** zhurong has quit IRC06:57
*** trungnv has quit IRC06:59
*** pooja_jadhav has joined #openstack-nova07:00
*** thorst has quit IRC07:01
*** edand_ has joined #openstack-nova07:04
*** suresh12 has quit IRC07:08
*** ralonsoh has joined #openstack-nova07:13
*** dayou has quit IRC07:14
*** zhurong has joined #openstack-nova07:20
*** fragatina has joined #openstack-nova07:27
*** yikun has joined #openstack-nova07:28
*** sahid has joined #openstack-nova07:30
*** thorst has joined #openstack-nova07:36
*** thorst has quit IRC07:41
*** karthiks has joined #openstack-nova07:44
*** damien_r has quit IRC07:46
*** rcernin has quit IRC07:49
*** dayou has joined #openstack-nova08:02
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2'  https://review.openstack.org/48438608:03
*** damien_r has joined #openstack-nova08:10
*** slaweq has joined #openstack-nova08:11
*** tesseract has joined #openstack-nova08:13
*** damien_r has quit IRC08:13
*** damien_r has joined #openstack-nova08:13
*** AlexeyAbashkin has joined #openstack-nova08:16
*** Tom-Tom has quit IRC08:17
*** Tom-Tom has joined #openstack-nova08:17
openstackgerritPeter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver.  https://review.openstack.org/14073308:18
*** Tom-Tom has quit IRC08:19
*** Tom-Tom has joined #openstack-nova08:19
*** zhurong has quit IRC08:20
*** andreas_s has joined #openstack-nova08:21
*** pcaruana has joined #openstack-nova08:23
*** sshwarts has joined #openstack-nova08:30
*** salv-orlando has quit IRC08:32
*** salv-orlando has joined #openstack-nova08:32
*** sree_ has joined #openstack-nova08:32
*** sree_ is now known as Guest2537208:33
openstackgerritBalazs Gibizer proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs  https://review.openstack.org/51960108:34
*** sree has quit IRC08:35
*** edmondsw has joined #openstack-nova08:35
*** rcernin has joined #openstack-nova08:35
*** salv-orlando has quit IRC08:36
*** claudiub has joined #openstack-nova08:39
*** edmondsw has quit IRC08:40
*** ragiman has joined #openstack-nova08:40
*** yamamoto_ has quit IRC08:47
*** jpena|off is now known as jpena08:48
*** salv-orlando has joined #openstack-nova08:50
*** claudiub has quit IRC08:57
*** hshiina has quit IRC08:58
*** yamamoto has joined #openstack-nova08:58
*** zhurong has joined #openstack-nova09:02
*** claudiub has joined #openstack-nova09:04
*** mvk has joined #openstack-nova09:05
bauzasmorning folks09:05
*** gongysh has quit IRC09:07
*** trungnv has joined #openstack-nova09:09
*** Guest13268 is now known as zigo09:09
*** aloga has quit IRC09:21
*** aloga has joined #openstack-nova09:22
*** Guest25372 has quit IRC09:24
*** sree has joined #openstack-nova09:24
*** gongysh has joined #openstack-nova09:25
*** sree has quit IRC09:29
*** jaianshu has joined #openstack-nova09:30
*** lucas-afk is now known as lucasagomes09:31
gibletbauzas: good morning09:31
*** rodolof has joined #openstack-nova09:32
*** alexchadin has joined #openstack-nova09:32
*** tetsuro has joined #openstack-nova09:35
*** jangutter has joined #openstack-nova09:38
openstackgerritTheodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields  https://review.openstack.org/52511009:38
*** derekh has joined #openstack-nova09:38
*** sree has joined #openstack-nova09:40
*** lei-zh has quit IRC09:41
*** claudiub has quit IRC09:41
*** moshele has quit IRC09:42
*** slaweq has quit IRC09:49
*** slaweq has joined #openstack-nova09:50
*** slaweq has quit IRC09:50
openstackgerritSilvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts  https://review.openstack.org/52224509:52
*** gszasz has joined #openstack-nova09:56
*** slaweq has joined #openstack-nova09:56
*** alexchadin has quit IRC10:05
*** sree has quit IRC10:13
*** alexchadin has joined #openstack-nova10:13
*** sree has joined #openstack-nova10:13
*** ragiman_ has joined #openstack-nova10:15
*** jaianshu has quit IRC10:16
*** jaianshu has joined #openstack-nova10:16
*** ragiman has quit IRC10:16
*** sree has quit IRC10:18
*** sree has joined #openstack-nova10:19
*** sree has quit IRC10:23
*** jaianshu has quit IRC10:23
*** giblet is now known as gibi10:25
*** zhurong has quit IRC10:26
*** Yingxin has quit IRC10:27
*** jaianshu has joined #openstack-nova10:28
*** jichen has quit IRC10:28
*** Yingxin has joined #openstack-nova10:30
*** yamamoto has quit IRC10:31
*** annp has quit IRC10:33
*** phuongnh has quit IRC10:37
*** mvk has quit IRC10:40
*** gszasz has quit IRC10:42
*** edand_ has quit IRC10:42
*** andreas_s has quit IRC10:42
*** andreas_s has joined #openstack-nova10:43
*** slaweq_ has joined #openstack-nova10:43
*** rodolof has quit IRC10:45
*** sambetts|afk is now known as sambetts10:47
*** slaweq_ has quit IRC10:48
*** sridharg has quit IRC10:49
*** gmann is now known as gmann_afk10:52
*** ragiman_ has quit IRC10:54
*** andreas_s has quit IRC10:56
*** ralonsoh has quit IRC10:58
*** Tom-Tom has quit IRC10:59
*** Tom-Tom has joined #openstack-nova11:00
*** jaianshu_ has joined #openstack-nova11:02
*** ralonsoh has joined #openstack-nova11:03
*** Tom-Tom has quit IRC11:04
*** andreas_s has joined #openstack-nova11:06
*** jaianshu has quit IRC11:06
*** ragiman_ has joined #openstack-nova11:06
*** edand_ has joined #openstack-nova11:07
*** mvk has joined #openstack-nova11:09
mdboothAnybody got a minute to look at this bugfix: https://review.openstack.org/#/c/524681/11:10
mdboothHonesty in advertising: it's only a couple of lines, but probably requires extra care due to its place bouncing around in the middle of the live migration workflow.11:11
mdboothStill, seems like an obvious fix, and its making users grumpy11:11
*** gongysh has quit IRC11:14
*** dayou has quit IRC11:15
*** yamamoto has joined #openstack-nova11:16
*** alexchadin has quit IRC11:19
*** andreas_s has quit IRC11:19
*** alexchadin has joined #openstack-nova11:20
*** andreas_s has joined #openstack-nova11:20
*** andreas_s has quit IRC11:24
*** brault has joined #openstack-nova11:25
gibimdbooth: looks good to me11:26
*** Tom-Tom has joined #openstack-nova11:26
*** alexchadin has quit IRC11:30
*** alexchadin has joined #openstack-nova11:30
*** dtantsur|afk is now known as dtantsur11:30
*** Tom-Tom has quit IRC11:30
kaisersfinucannot: Hi! (hope i've the right id :) ) Regarding the exception in https://review.openstack.org/#/c/522245 , i'm happy to do this differently. I just need a way to differentiate one of the exceptions in validate volume so if you have suggestions, pls fire away11:31
*** finucannot is now known as stephenfin11:31
stephenfinkaisers: This is for the purposes of the test?11:32
kaisersno, for connect_volume()11:32
kaisersThat has a special treatment for the stale mount point situation. It's where i want to detect and handle that special case.11:33
*** gszasz has joined #openstack-nova11:33
stephenfinAh, I see your comments now11:33
kaisersstephenfin: (i thought this would be faster via irc than a comments discussion... )11:33
stephenfinkaisers: Hmm, OK then if it's serving a purpose a separate exception makes sense11:35
stephenfin*purpose other than making testing slightly easier11:35
stephenfinand it should be in nova.exception. There are a lot of virt-specific exceptions in there alright11:35
stephenfinAt least I can't spot many exceptions kept outside of there11:36
kaisersstephenfin: ok, i see that point11:36
kaisersi'll move the exception and rethink it's naming and send it with the next update11:36
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref  https://review.openstack.org/52291811:37
*** moshele has joined #openstack-nova11:40
*** sean-k-mooney2 has joined #openstack-nova11:43
sean-k-mooney2o/11:44
*** andreas_s has joined #openstack-nova11:44
mdboothgibi: Thanks!11:45
*** lpetrut has joined #openstack-nova11:46
*** andreas_s has quit IRC11:46
*** andreas_s has joined #openstack-nova11:46
*** cdent has joined #openstack-nova11:51
*** ratailor has quit IRC11:52
openstackgerritTheodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields  https://review.openstack.org/52511011:54
*** smatzek has joined #openstack-nova11:56
*** jpena is now known as jpena|lunch11:57
*** gszasz has quit IRC11:57
openstackgerritChris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers  https://review.openstack.org/52163911:58
openstackgerritChris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests  https://review.openstack.org/52164011:58
*** gszasz has joined #openstack-nova11:58
*** TuanLA has quit IRC12:01
*** yangyapeng has quit IRC12:02
*** Tom-Tom has joined #openstack-nova12:06
*** yangyape_ has joined #openstack-nova12:06
*** lennyb__ has quit IRC12:07
*** gcb has quit IRC12:10
*** lucasagomes is now known as lucas-hungry12:10
*** yangyape_ has quit IRC12:11
*** edmondsw has joined #openstack-nova12:11
*** edmondsw has quit IRC12:16
*** lpetrut has quit IRC12:17
*** BryanS68 has joined #openstack-nova12:24
*** lpetrut has joined #openstack-nova12:25
*** kuzko has quit IRC12:29
*** kuzko has joined #openstack-nova12:31
*** tetsuro has quit IRC12:31
*** rcernin has quit IRC12:31
*** kuzko has quit IRC12:35
*** yangyapeng has joined #openstack-nova12:36
*** derekh has quit IRC12:36
*** kuzko has joined #openstack-nova12:43
*** alexchadin has quit IRC12:44
efriedHey kids!12:45
*** mhenkel has quit IRC12:47
*** sridharg has joined #openstack-nova12:47
cdenthey uncle eric12:48
*** links has quit IRC12:54
*** mhenkel has joined #openstack-nova12:55
sean-k-mooney2:) hey efried you seem to be in a good mood. good weekend?13:02
efriedYeah, very eventful.  My buddy got his BJJ black belt, which is a pretty big deal.13:02
*** dtantsur is now known as dtantsur|lunch13:03
sean-k-mooney2oh cool. does that mean he can now teach.13:03
sean-k-mooney2or does he need second dan for that13:04
*** gszasz has quit IRC13:04
efriedHe's been teaching for a while.13:04
*** jpena|lunch is now known as jpena13:04
efriedI guess it doesn't really mean anything except to people in the BJJ community, or near enough to it to recognize the significance.13:05
efriedBasically, it means he's good.  Really good.13:05
sean-k-mooney2hehe yes blackbelts in any martial art tend to be :) one of my co workers recently started BJJ13:06
*** alexchadin has joined #openstack-nova13:07
*** salv-orlando has quit IRC13:07
*** salv-orlando has joined #openstack-nova13:08
*** lucas-hungry is now known as lucasagomes13:08
*** jaypipes has joined #openstack-nova13:08
efriedBJJ is one of the harder ones, generally speaking.  Most times it takes around a decade to get there (though my buddy did it in a little over 7 years, which is fairly quick - he's got talent).13:09
*** ttsiouts_ has joined #openstack-nova13:11
sean-k-mooney2jaypipes: o/ its faily early still in your neck of the woods right?13:11
jaypipessean-k-mooney2: quarter after eight13:11
*** salv-orlando has quit IRC13:12
sean-k-mooney2jaypipes: ya i taught you were -5 from me.13:12
*** ttsiouts_ has quit IRC13:14
sean-k-mooney2so general question. in pike there was no way to request a spcecific resouce class to be allocated as part of a vm boot request correct e.g. in placement13:14
efriedsean-k-mooney2 I believe that is correct, except for the Ironic case.13:18
efried...which isn't really a resource class.13:19
sean-k-mooney2right13:19
jaypipessean-k-mooney2: no, you could/can specify an override in the flavor's extra specs "resources" key.13:20
sean-k-mooney2ok thats what i taught. am for neted resouce provided sriov is going to be the first example resouce correct13:20
sean-k-mooney2jaypipes: oh really?13:20
jaypipessean-k-mooney2: no. Ironic has used resources in the flavor extra specs already13:20
jaypipessean-k-mooney2: and vCPU, memory and disk can also be overridden from the flavor.vcpus/memory_mb/disk_gb values13:21
*** slaweq_ has joined #openstack-nova13:21
efriedsean-k-mooney2 I think the first real world example may be VGPUs in the xen driver.13:22
sean-k-mooney2jaypipes: and if i was to do that it wont break any of the internal accounting? e.g. if flavor says 4 vcpus and i override the resouce request to 2 or 8 bad thing wont happen? or this is a gun and i should be careful where i point it13:23
*** edmondsw has joined #openstack-nova13:23
jaypipessean-k-mooney2: accounting will be done with the override.13:23
sean-k-mooney2ok cool so if i were to do this always claim >= what is used by normal flavor properties.13:25
*** slaweq_ has quit IRC13:26
efriedsean-k-mooney2 For the Standard Three (VCPU, MEMORY_MB, and DISK_GB) in pike, it's in fact the exact same code path by the time it gets to actually claiming anything.  The override/conversion gets done really early.13:26
sean-k-mooney2efried: yes but i would assume the xml generation would use the VCPU not the override in resouces13:27
efriedsean-k-mooney2 Maybe I'm not understanding what you mean by "override".  Where are you overriding?13:27
jaypipesefried: the extra specs resources key/value pair are overrides from what is in, say, flavor.vcpus13:27
efriedRight, sean-k-mooney2 in ^ this case, you're safe.13:28
efriedFor a sec I was scared you were, like, doing something in the virt driver that was different from what came down in the flavor.13:28
sean-k-mooney2efried: well i was just trying to make sure you couldnt have flavor.vpus=4 and resouces.vcpus=2 and end up spawning a vm with 4 cpus but only claim 213:30
jaypipessean-k-mooney2: no, you can't do that13:31
efriedsean-k-mooney2 oh, it's an interesting point.  I imagine it's up to the virt driver what they key off of to decide how much resource to allocate.  Let me check whether we actually overwrite the flavor.vcpus value for the copy we send down to virt...13:31
*** links has joined #openstack-nova13:31
jaypipessean-k-mooney2: but you can have flavor.vcpus = 2 and resources:VCPU=2 and the instance will get 2.13:31
jaypipesdoh, I meant resources:VCPU=4 and get 4.13:31
sean-k-mooney2jaypipes: ok well since the flavor is admin only i guess that is ok. if this was available via image metadata that could be bad13:33
jaypipessean-k-mooney2: ack13:33
*** janki has quit IRC13:33
*** yamahata has joined #openstack-nova13:34
*** oanson has quit IRC13:35
sean-k-mooney2ok well that good to know it will likely be helpful at some point.13:37
*** oanson has joined #openstack-nova13:37
sean-k-mooney2have ye heard of vhost-vfio13:38
*** dtantsur|lunch is now known as dtantsur13:38
*** rabel has joined #openstack-nova13:40
sean-k-mooney2it is a form of vhost offload like vhost-user which leaverages the kenel vfio-pci driver to provide sriov like performance that can also support livemigration/ovs hardware offload.13:41
jaypipessean-k-mooney2: no, haven't heard of it until just now. sounds like yet another avenue for hard-to-debug issues in live migration :)13:43
*** edmondsw has quit IRC13:43
*** jaianshu_ has quit IRC13:43
*** hongbin has joined #openstack-nova13:43
sean-k-mooney2jaypipes: hehe proably an im likely going to be one of the first to enjoy that experience13:43
jaypipessean-k-mooney2: you always are :)13:44
jaypipesstrace and gdb ftw13:44
sean-k-mooney2this is the semi internal poc that im being pull upstream work to work on. it leaverages mdevs instead of vfs to represent the virtual interfaces which is fun...13:45
jaypipessean-k-mooney2: what advantage does Nvidia's mdev framework have over the SR-IOV VF interface?13:46
sean-k-mooney2jaypipes: live migrtation and the ability to create them at runtime are 2 of the advantages13:47
jaypipessean-k-mooney2: why can't VFs be created at runtime?13:47
*** jamesdenton has joined #openstack-nova13:48
sean-k-mooney2jaypipes: the linux kernel does not have an interface to do so without removeing and reallocating all vfs on the pf13:48
sean-k-mooney2our nics and technically dpdk can13:48
jaypipessean-k-mooney2: interesting. why didn't nvidia enhance the existing SR-IOV drivers/VF interface instead of inventing a whole new interface?13:49
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721513:49
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480713:49
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569313:49
sean-k-mooney2jaypipes: well sriov is a standard by the pcisig and when you create a vf you need to create a new pci endpoint on the host pcie bus for that vf which is no trivial in hardware at runtime13:50
jaypipesalex_xu: great catch on https://review.openstack.org/#/c/384807. I've updated and added a new func test to catch CannotDeleteParentResourceProvider13:50
*** itlinux has joined #openstack-nova13:50
jaypipessean-k-mooney2: and what's different about mdev in that respect?13:51
*** Tom-Tom_ has joined #openstack-nova13:51
*** abalutoiu has joined #openstack-nova13:52
sean-k-mooney2they are attached to a new bus /sys/bus/mdev and not the pci bus. you create them by echoing a uuid to a file under a partent device then acress them via the uuid after tehy are created13:52
jaypipessean-k-mooney2: ack. but why couldn't the PCI spec be changed to allow a similar functionality? I'm just curious13:53
*** ttsiouts_ has joined #openstack-nova13:53
*** Tom-Tom has quit IRC13:54
sean-k-mooney2jaypipes: im guessing backwards compatiblity but im not sure. one of the other advanatages of vfio-mdevs ist that it does not need hardware support. it can all be done in the driver13:54
jaypipessean-k-mooney2: ack13:54
*** ttsiouts has quit IRC13:55
* efried brags about PowerVM again, where we've had the ability to do live-migrateable SR-IOV VF with dynamic allocation since ocata13:55
*** edleafe- is now known as edleafe13:55
sean-k-mooney2so if you had a nic without sriov with say 4 tx/rx pairs and you impmented mdev support you could create 4 mdevs one for each queue pair and use it like 4 nics. sriov would require hardware support to do the same13:55
jaypipessean-k-mooney2: or just use PowerVM, which solves all problems for everyone.13:57
efriedCorrection: Since Newton.13:57
jaypipesheh13:57
* jaypipes turns off snark-o-meter13:57
sean-k-mooney2jaypipes: :) certily solve my problem of exposing intel ad vaule in hardware13:58
*** hongbin has quit IRC13:58
*** ttsiouts_ is now known as ttsiouts13:58
*** thorst has joined #openstack-nova13:59
*** hongbin has joined #openstack-nova13:59
openstackgerritLajos Katona proposed openstack/nova master: Extend ServerMovingTests with custom resources  https://review.openstack.org/49739913:59
*** ttsiouts is now known as ttsiouts_13:59
*** yamahata has quit IRC13:59
sean-k-mooney2so any way i may have to backport whatever i come upwith to pike hence my earlir question though im trying to avoid that give howmuch is changeing in this space13:59
edleafeScheduler subteam meeting starting now in #openstack-meeting-alt14:00
sean-k-mooney2oh im late for a meeting too...14:00
efriedsean-k-mooney2 It would be a tough backport, because yeah, queens is going to have huge deltas in this area.14:01
*** vladikr has joined #openstack-nova14:01
alex_xujaypipes: there is another comment at https://review.openstack.org/#/c/377215/6614:02
*** slaweq_ has joined #openstack-nova14:02
*** liverpooler has joined #openstack-nova14:02
sean-k-mooney2efried: yep and doing it in a way that will be upgradable to queens will be very hard which is why im trying to push that out of scope but first i need to understand if its technically possible or not14:03
*** takashin_ has joined #openstack-nova14:04
efriedsean-k-mooney2 It might be doable for custom resource classes.  What did you have in mind?14:04
*** takashin has quit IRC14:06
*** slaweq_ has quit IRC14:06
sean-k-mooney2custom resouce class create by neutron ovs agent extention, + os-vif plugin to create mdevs during plug + new vnic type to request a vhost-vfio port like we do with sriov14:06
sean-k-mooney2+ the smallest patch to nova possibel to cover xml generation changes in libvirt driver and doing the resouce claim in placement.14:07
*** lyan has joined #openstack-nova14:08
*** yamamoto_ has joined #openstack-nova14:08
*** salv-orlando has joined #openstack-nova14:08
*** cleong has joined #openstack-nova14:09
*** hongbin has quit IRC14:11
*** yamamoto has quit IRC14:11
*** mlavalle has joined #openstack-nova14:11
*** pchavva has joined #openstack-nova14:11
*** karthiks has quit IRC14:12
*** salv-orlando has quit IRC14:13
gibisean-k-mooney2: be aware that in pike custom resource classes only works if it is reported by the nova-compute in the RP that represents the compute node. All the other option needs either shared RP or nested RP14:13
jaypipesalex_xu: responded.14:13
*** felipemonteiro has joined #openstack-nova14:15
alex_xujaypipes: yea, the uuid and name aren't in opposition, but sorry, I didn't get what your mean about the 'member_of'14:16
*** felipemonteiro_ has joined #openstack-nova14:16
*** yamamoto_ has quit IRC14:20
openstackgerritSilvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts  https://review.openstack.org/52224514:20
*** felipemonteiro has quit IRC14:20
*** alexchadin has quit IRC14:22
openstackgerritSilvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts  https://review.openstack.org/52224514:23
*** gbarros has joined #openstack-nova14:24
jaypipesalex_xu: that was responding to bauzas' comment about using member_of instead of tree. member_of is for aggregates, not for provider trees.14:27
*** gcb has joined #openstack-nova14:27
alex_xujaypipes: ah, got it14:28
*** yamamoto has joined #openstack-nova14:29
*** tssurya has quit IRC14:29
bauzasjaypipes: oh my bad, you're right14:31
*** smatzek has quit IRC14:31
*** takashin_ has left #openstack-nova14:31
*** takashin has joined #openstack-nova14:31
*** smatzek has joined #openstack-nova14:31
bauzasjaypipes: tbc, I'm not opiniated by which key we should use, just the point that we need to make sure it works with a call having all of them :)14:31
openstackgerritMatthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping  https://review.openstack.org/24260214:32
openstackgerritMatthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid  https://review.openstack.org/24260314:32
jaypipesbauzas: ack14:32
bauzas"tree" is maybe a bit difficult to understand, but I'm not an English folk :)14:32
*** yamamoto has quit IRC14:35
*** smatzek has quit IRC14:36
*** rabel has quit IRC14:40
*** rabel has joined #openstack-nova14:40
*** cdent has quit IRC14:42
*** gszasz has joined #openstack-nova14:42
sean-k-mooney2gibi: ah yes that is a good point. i would prefer to use nested resouce providers as logically the mdev resouce are a nested resouce of a parent device. in my case a pcie device but technically mdevs could be childern of device on buses other the pci bus.14:45
*** amodi has joined #openstack-nova14:46
gibisean-k-mooney2: I agree that the best would be to have nest RPs for it14:46
*** tssurya has joined #openstack-nova14:46
*** thorst has quit IRC14:46
efriedjaypipes bauzas alex_xu Oh, I wasn't following the conversation in here; I left a (verbose, as usual) comment.14:46
jaypipesefried: on what?14:47
*** BryanS68 has quit IRC14:47
*** BryanS68 has joined #openstack-nova14:47
jaypipesefried: in any case, I'm currently adding func tests that add member_of and uuid filters to a tree filter.14:48
*** thorst has joined #openstack-nova14:48
*** yamamoto has joined #openstack-nova14:52
*** peter-hamilton has joined #openstack-nova14:52
*** gouthamr has joined #openstack-nova14:52
*** psachin has quit IRC14:53
*** salv-orlando has joined #openstack-nova14:57
*** esberglu has joined #openstack-nova14:58
*** tidwellr has joined #openstack-nova14:59
*** smatzek has joined #openstack-nova14:59
*** edand_ has quit IRC15:00
alex_xuefried: I don't the tree and resources will work as your expected. The 'GET /resource_providers' just a simple filtering, it is different with the filtering of 'GET /allocation_candidates'15:01
efriedjaypipes alex_xu Sorry, I was talking about https://review.openstack.org/#/c/377215/15:02
efriedI'm coming at it from the perspective of how I think the REST API should work, not looking at the code.15:02
efriedThe func tests jaypipes is suggesting should bear out what I said in my comment; and if not, it's a bug that we can fix, right?15:03
*** thorst has quit IRC15:03
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete  https://review.openstack.org/52523115:04
*** yamamoto has quit IRC15:04
*** burt has quit IRC15:06
*** Tom-Tom_ has quit IRC15:06
jianghuaw_bauzas, the vGPU relative calls have been there since the minimal version of XS supported by nova. So there is no need to check XenServer's version.15:07
*** takashin has left #openstack-nova15:07
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721515:07
stephenfindansmith: Question on this patch for you https://review.openstack.org/#/c/514309/15:07
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480715:07
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569315:07
jaypipesbauzas, alex_xu, efried: k, func tests added for tree filter + uuid/member_of15:07
dansmithstephenfin: where?15:08
bauzasjianghuaw_: gtk15:08
stephenfindansmith: The 'tempest-dsvm-neutron-src-oslo.versionedobjects' job15:08
stephenfindansmith: I migrated that because it seemed we were the only caller15:08
bauzasjianghuaw_: kaboom then15:09
stephenfinBut I see now that there are many 'tempest-dsvm-neutron-src-{library}' jobs and '{library}' is usually wildcarded, so that should probably live in oslo.versionedobjects15:09
jianghuaw_bauzas, thanks:-)15:09
dansmithstephenfin: sorry, I blame the early hour for my confusion but.. what are you asking? if other projects have o.vo jobs?15:10
* bauzas going back to write the libvirt create MDEV15:10
*** links has quit IRC15:10
stephenfindansmith: Yes. Just a sanity check15:11
dansmithstephenfin: tbh, that's not a thing I know and I would expect it to be more of a core oslo thing, policy for running library jobs against projects that use them and such15:12
dansmithstephenfin: I think nova, neutron, ironic at least use o.vo15:12
jianghuaw_bauzas, I guess this one would be much easier as you ever +2/+w on it:-) https://review.openstack.org/#/c/51621715:12
bauzasjianghuaw_: next one :)15:13
jianghuaw_cool:-)15:13
efriedjaypipes It would be nice to have a func test for resources + tree -- I think that was a main concern of alex_xu's.  But we can do it in a followup too.15:13
stephenfindansmith: That's fair. I picked on you as you were the one that enabled the test for nova (commit ac96f95 in openstack-infra/project-config). No good deed :)15:13
*** edmondsw has joined #openstack-nova15:13
stephenfinI'll go bug AJaeger15:13
dansmithstephenfin: probably at the behest (and handholding) of some infra deity :)15:13
*** ijw has quit IRC15:14
*** awaugama has joined #openstack-nova15:14
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data  https://review.openstack.org/40751415:17
*** edmondsw has quit IRC15:18
*** AlexeyAbashkin has quit IRC15:20
efriedjaypipes I was about to rebase "make _Provider private" -- did you have any feedback on uuid_set() (https://review.openstack.org/#/c/520243/) so I can hit it at the same time?15:20
*** thorst has joined #openstack-nova15:24
*** yamamoto has joined #openstack-nova15:25
*** yamamoto has quit IRC15:25
*** itlinux has quit IRC15:26
*** esberglu_ has joined #openstack-nova15:27
*** gbarros has quit IRC15:27
*** thorst has quit IRC15:29
*** esberglu has quit IRC15:29
*** armax has joined #openstack-nova15:30
*** moshele has quit IRC15:33
*** gbarros has joined #openstack-nova15:35
openstackgerritEric Fried proposed openstack/nova master: Make _Provider really private  https://review.openstack.org/52393215:37
openstackgerritEric Fried proposed openstack/nova master: ProviderTree.uuid_set()  https://review.openstack.org/52024315:37
efriedjaypipes ^15:37
mdboothdansmith: You mentioned last week that you didn't think I could add a unique constraint to the uuid column in the first instance. You also say the same explicitly in a comment, so I assume there's a good reason. Do you recall what it is?15:40
dansmithmdbooth: you can if it's nullable I think, but I think I thought you were asserting that you were going to create the column and make it non-nullable/unique right away and that's what I was trying to say was not okay15:41
mdboothdansmith: Yeah, I was going for unique/nullable.15:41
mdboothAny reason I shouldn't add the unique constraint to your patch?15:42
* mdbooth likes constraints15:42
dansmithif you can add it to the schema without code changes or moving data, then it's cool to add it yeah15:42
mdboothWill do, thanks15:43
tssuryamriedem: ping15:45
*** hongbin has joined #openstack-nova15:48
*** andreas_s has quit IRC15:50
*** andreas_s has joined #openstack-nova15:51
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524215:51
mdboothdansmith: Second question: update by uuid. I'm assuming that when you were adding this first time round you anticipated needing this. I don't think I need it. I could add it for completeness, or omit it until there's a caller. Personal preference would be the latter.15:51
*** tidwellr has quit IRC15:52
*** slaweq is now known as slaweq_15:53
*** tidwellr has joined #openstack-nova15:53
dansmithmdbooth: I probably had it for tests or something, but sure, no need to have it if there is no user15:53
*** gcb has quit IRC15:53
*** esberglu_ has quit IRC15:56
*** esberglu has joined #openstack-nova15:56
*** ttsiouts_ has quit IRC15:57
*** tssurya has quit IRC15:57
*** david-lyle has quit IRC15:58
*** david-lyle has joined #openstack-nova15:59
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates  https://review.openstack.org/50039615:59
*** edand_ has joined #openstack-nova15:59
*** slaweq_ has quit IRC16:00
*** yamamoto has joined #openstack-nova16:00
*** andreas_s has quit IRC16:00
*** andreas_s has joined #openstack-nova16:01
*** sdague has joined #openstack-nova16:01
*** slaweq_ has joined #openstack-nova16:03
mdboothdansmith: Thanks16:07
*** sshwarts has quit IRC16:07
*** slaweq_ has quit IRC16:08
*** mriedem has joined #openstack-nova16:09
*** coreywright has quit IRC16:10
ildikovmriedem: hi, I updated the patch with the fixes to your comments16:13
ildikovmriedem: I didn't have the stomach as of yet at least to do the split...16:13
*** amodi has quit IRC16:13
mriedemildikov: ok16:13
*** _pewp_ has quit IRC16:14
*** ttsiouts has joined #openstack-nova16:14
*** _pewp_ has joined #openstack-nova16:14
ildikovmriedem: if you have time plz continue to review and I will fix it in either one or two patches16:14
mriedemi hope to get to it today, but have to work on some other stuff this morning16:15
*** tssurya has joined #openstack-nova16:16
ildikovappreciated if you can get to it today16:16
*** lajoskatona has left #openstack-nova16:19
*** ttsiouts_ has joined #openstack-nova16:20
*** cdent has joined #openstack-nova16:21
*** yamamoto has quit IRC16:22
*** coreywright has joined #openstack-nova16:23
*** slaweq_ has joined #openstack-nova16:25
*** edmondsw has joined #openstack-nova16:26
*** thorst has joined #openstack-nova16:28
*** itlinux has joined #openstack-nova16:31
*** ttsiouts_ has quit IRC16:32
*** artom_ has quit IRC16:32
*** artom has joined #openstack-nova16:32
tssuryamriedem: I am stuck on this - https://bugs.launchpad.net/nova/+bug/1729806/comments/1 , because both those columns are of type "Text()" and SQL doesn't allow text type columns to have a unique constraint; any pointers on how you usually do this in nova ?16:33
openstackLaunchpad bug 1729806 in OpenStack Compute (nova) "nova-manage update_cell does not check if the same combination of transport_url and database_connection already exists" [Low,In progress] - Assigned to Surya Seetharaman (tssurya)16:33
*** armax has quit IRC16:33
*** thorst has quit IRC16:33
mriedemildikov: i looked at the diff, comments in https://review.openstack.org/#/c/330285/16:33
*** armax has joined #openstack-nova16:34
openstackgerritEric Fried proposed openstack/nova master: Make _Provider really private  https://review.openstack.org/52393216:34
dansmithtssurya: you could check before create, which isn't as good as a constraint, but it'd be an easy thing to do16:36
openstackgerritSilvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts  https://review.openstack.org/52224516:36
ildikovmriedem: tnx!16:36
*** ragiman_ has quit IRC16:36
*** medberry_ has joined #openstack-nova16:36
mriedemildikov: and another one just now16:37
mriedemildikov: the shelved offloaded case probably needs some thought16:37
mriedemcould also see what johnthetubaguy thinks16:37
ildikovmriedem: ok, I'll wait a bit longer before touching it :)16:37
mriedemfor all intents and purposes, a reserved volume is going to be the same as an in-use volume to nova16:37
ildikovmriedem: sigh, I will look into the comments shortly16:37
johnthetubaguyI was a touch worried about the change in shelved offloaded16:38
mriedemthere is an api behavior change with attaching a volume to a shelved offloaded instance with the new flow,16:38
mriedemwhere the volume is left 'reserved' rather than 'in-use'16:38
johnthetubaguyyeah, that seemed nasty16:38
mriedemi could see client side app tooling failing / timing out on that16:38
mriedemlike tempest even16:38
mriedemlots of client code waits for an attachment to go to in-use before it considers it done16:38
johnthetubaguyyes, good point16:39
tssuryadansmith: Yes, that is what I am doing in my review, I was just trying to add the constraint part too16:39
dansmithtssurya: sounds like we might not be able to do the constraint part.. since urls can't be large, I'm not sure converting it to something other than text is a good idea16:39
*** medberry_ has quit IRC16:39
johnthetubaguymriedem: I did raise that in a previous version, but forgot about it shortly after, +1 on it being bad16:40
dansmithtssurya: we actually have some long urls ourselves, since we encode all the HA hosts in the transport url, and I think we do the same for our galera endpoints16:40
*** medberry_ has joined #openstack-nova16:40
*** slaweq_ has quit IRC16:40
efriedcdent dansmith Did you see my "themed CI results" last week?  I made one monochrome for cdent (http://184.172.12.213/manual/htmlify_logs/logs/n-cpu.txt.gz?theme=monochrome) and one solarized for both of you (http://184.172.12.213/manual/htmlify_logs/logs/n-cpu.txt.gz?theme=solarized)16:40
tssuryadansmith: oh okay, yea then I will not pursue it, this came up https://review.openstack.org/#/c/523817/ in parallel to my patchset, so I thought I would investigate this16:40
dansmithtssurya: cool16:41
tssuryadansmith: thanks then, I will remove the constraint part then16:41
mriedemedleafe: i'm going to make a concerted push on the alternate hosts series today, do you plan on rebasing https://review.openstack.org/#/q/topic:bp/return-alternate-hosts+status:open to resolve the merge conflicts at the top?16:41
*** AJaeger_ has joined #openstack-nova16:41
*** chyka has joined #openstack-nova16:41
*** trozet has quit IRC16:41
dansmithefried: I did not, but cool.. that still ends up with ascii in the logs if I curl download it though right?16:41
efrieddansmith I've actually got solutions that go both ways.16:42
AJaeger_sdague: could you review https://review.openstack.org/#/c/524353/ ,please? This updates jobs for os-api-ref and gets rid of tools/tox_install.sh16:42
jaypipesefried: you are currently rebasing the series that ends in 520246, yes?16:42
dansmithefried: ....16:42
edleafemriedem: I was waiting until there was renewed interest in the series. I can start rebasing now.16:42
dansmithefried: well, from the purely browser-based view, solarized looks good to my eyes :)16:42
mriedemedleafe: yup, it's my #1 priority today16:42
efrieddansmith I think what I've got on that server right now preserves the codes, because that's the way I prefer it, and that's my server :)16:43
mriedemedleafe: i plan on having that series +2ed by thursday16:43
*** gyee has joined #openstack-nova16:43
edleafemriedem: on it16:43
efrieddansmith Okay, duly noted.  The way I've got it set up, it's fairly easy to add themes, and (in case you didn't notice) there are new links at the top allowing you to flip between themes.16:43
*** jpena is now known as jpena|brb16:43
dansmithefried: yeah, curl gives me ansified files16:43
cdentefried: I did see it, solarized was also easy on my eyes. If there’s a way to curl and get non-ansified files that would be a cherry on top16:44
cdent(sort of as another theme, I guess)16:44
ildikovmriedem: johnthetubaguy: got busy, but will check back on it and we can chat about it16:44
dansmiththe theming happens in the browser I imagine16:44
dansmithso doesn't affect curl16:44
*** mvk has quit IRC16:44
* cdent misses CGI16:44
efrieddansmith cdent I have solutions that *always* or *never* strip the ANSI colors; what I'd like to get to eventually is a way for you to curl download with a querystring that allows you to choose.16:44
dansmithI'm not a hard -2 on the ansi logs, but I'd prefer it if I can get it16:45
*** maciejjozefczyk has quit IRC16:45
dansmithefried: that'd be cool16:45
*** danpawlik_ has quit IRC16:45
dansmithefried: just put me a curl link on the top somewhere so I can copy that, kinda like the download links in gerrit16:45
*** itlinux has quit IRC16:46
efrieddansmith The theming is actually an HTML filter that sits on the apache server and kicks in when the user agent is a browser (or something).  Then there's a separate filter that you get when it's a download, that doesn't do the HTML conversion at all.16:46
efrieddansmith ++ good call16:46
dansmithefried: ah, okay I dunno how that works, but cool16:46
efrieddansmith I usually right-click Save-As the file from the directory view, but a copyable link would be neat.16:46
*** danpawlik_ has joined #openstack-nova16:46
efrieddansmith In case you're interested, the theme-ification patch: https://review.openstack.org/#/c/524744/16:46
dansmithefried: yeah, copyable would be nice, because I wget them to my dev box.. right-click-save doesn't do what I want because it'd be on my local machine instead16:47
*** maciejjozefczyk has joined #openstack-nova16:47
efrieddansmith I gotcha.16:47
dansmithbut yeah, aside from my complaints, looks awesome16:48
dansmithsometimes being able to just switch the theme based on what you're looking for can be nice, and/or what you've been staring at for hours16:48
*** itlinux has joined #openstack-nova16:50
*** yamamoto has joined #openstack-nova16:51
*** medberry_ has quit IRC16:51
*** edand_ has quit IRC16:53
*** sridharg has quit IRC16:55
mriedemjaypipes: easy refactor https://review.openstack.org/#/c/523492/16:56
* artom feels out of the loop16:56
mriedemin the alt hosts series16:56
artomWhat happened to nova-specs?16:56
mriedemartom: ?16:56
artomhttps://review.openstack.org/#/c/490824/ merged October 3rd16:56
*** esberglu has quit IRC16:56
artomLatest commit in https://github.com/openstack/nova-specs/commits/master/specs is september 2116:57
mriedemefried: ^ didn't you ask infra about this a week or two ago?16:57
*** lpetrut_ has joined #openstack-nova16:57
*** emccormick has joined #openstack-nova16:57
mriedemhttp://git.openstack.org/cgit/openstack/nova-specs/log/16:57
mriedemthe github mirror repo is definitely stale16:57
artomYeah16:57
*** esberglu has joined #openstack-nova16:58
* artom will use git.openstack.org16:58
artomAnd maybe bring it up in #infra?16:58
efriedmriedem After spec freeze I noticed that they weren't getting published to specs.o.o -- sounds like that's a different issue.16:58
*** amodi has joined #openstack-nova16:58
efried...which was resolved, at least at the time.16:58
clarkbmriedem: artom efried the github mirror being stale is a known issue, the repo is corrupt according to github and won't accept pushes16:58
clarkbif we take a gerrit downtime we can surgery the repo but we haven't had opportunity for that yet16:59
artomclarkb, aha, thanks!16:59
efriedartom FWIW, that spec is published: https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/libvirt-qemu-native-luks.html16:59
mriedemoh yes16:59
clarkbits been a low priority because we do host working mirrors elsewhere and github is just barely a thing we support16:59
cfriesenhas anyone ever considered an option to "tox -e pep8" that would run it only on the files in the top N commits?16:59
artomefried, doh, of course!16:59
efriedcfriesen -- -HEAD16:59
mriedemcfriesen: tox -e fast816:59
artomclarkb, yep, fair enough, thanks for, err, being aware?16:59
* efried was not aware of fast8 -- what's that about?17:00
mriedemfast8 is different17:00
mriedemfast8 is pep8 on just the files changed17:00
artommriedem, I thought we're only at Fast and Furious 6 or 717:00
clarkbwith the holidays coming up, things tend to slow down and we can probably more easily sneak in an extended gerrit downtime17:00
efriedAh, so same as -- -HEAD17:00
mriedemyes17:00
clarkbI'll add it to tomorrows meeting agenda17:00
*** lpetrut has quit IRC17:00
mriedembecause typing 'tox -e pep8 -- -HEAD' was too tedious17:00
openstackgerritEd Leafe proposed openstack/nova master: Add Selection objects  https://review.openstack.org/49923917:00
openstackgerritEd Leafe proposed openstack/nova master: Refactor the code to check for sufficient hosts  https://review.openstack.org/52024217:00
openstackgerritEd Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver  https://review.openstack.org/49585417:00
cfriesenmriedem: efried: sweet...guess I should check the ini file first. :)17:00
openstackgerritEd Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object  https://review.openstack.org/52349217:00
openstackgerritEd Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts  https://review.openstack.org/51015917:00
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670717:00
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135717:00
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135817:00
edleafemriedem: ^^ there ya go17:00
mriedemthanks17:01
efriedcfriesen The other thing I do when I'm working on a series is: source .tox/pep8/bin/activate to get into the pep8 virtualenv, and leave that window open.  Then from within that window I'll do: git diff --name-only HEAD~1 | grep '\.py$' | xargs flake817:01
*** kmalloc has joined #openstack-nova17:01
efriedcfriesen HEAD~{whatever-number-here} to go back that many patches.17:01
*** yamamoto has quit IRC17:01
*** slaweq_ has joined #openstack-nova17:01
*** thorst has joined #openstack-nova17:02
emccormickHey, if anyone has a sec for a possibly dumb question: Is there a reason that it's required to have the instance type in the domain xml match the instance type in nova.conf?17:02
emccormickI ask because I'm working on migrating backends from local to rbd, and while running instances continue to function after the change to nova.conf, and can also be migrated, they cannot be rebooted.17:03
* efried also leaves venv windows open to shortcut test runs for pyXX, functional, docs, etc.17:03
emccormickjust wondering if that's something that could be changed or if it would break things horribly17:03
*** lpetrut_ has quit IRC17:03
efriedcfriesen The only caveat to the permanent-venv deal is if you need to rebuild the venv for whatever reason; sometimes running via tox will figure that bit out for you without needing to use -r; but sitting in the venv you get nuffink.17:04
*** imacdonn has joined #openstack-nova17:04
edleafeefried: I use aliases to do roughly the same thing17:05
cfriesenefried: that's a useful trick and a good caveat.17:05
efriededleafe Aliases for what, running flake8 in the venv?17:05
openstackgerritChris Friesen proposed openstack/nova master: unreserve volumes after scheduling has failed  https://review.openstack.org/52525317:06
edleafeefried: like this:17:06
edleafealias ptt='source .tox/py27/bin/activate; python -m testtools.run'17:06
cfriesen^ we ran into this while testing Pike, looks like it's still an issue in master17:06
edleafeI have different aliases for functional, py3, etc.17:06
efriededleafe Ah, gotcha.17:06
*** slaweq_ has quit IRC17:06
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project  https://review.openstack.org/51430917:08
openstackgerritStephen Finucane proposed openstack/nova master: Resolve review comments from I41b03a347  https://review.openstack.org/52525517:08
cfriesenThe issue that the patch above ^^^ fixes is that if you do boot-from-vol and then fail scheduling, currently the volume remains reserved and is stuck in the "attaching" state.17:08
*** slaweq_ has joined #openstack-nova17:08
*** felipemonteiro_ has quit IRC17:09
efriedmriedem The _ensure_resource_provider error handling cherry-pick for pike is looking green now (https://review.openstack.org/#/c/524618/) -- will proceed to backport to ocata unless some reason not to?17:09
mriedemgo ahead17:11
openstackgerritSurya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url  https://review.openstack.org/51855617:12
*** slaweq_ has quit IRC17:14
*** andreas_s has quit IRC17:14
*** andreas_s has joined #openstack-nova17:14
*** slaweq_ has joined #openstack-nova17:15
efriedurgh, merge conflict (which I guess isn't surprising).  Will hit it after "lunch".17:16
*** udesale has quit IRC17:17
*** slaweq_ has quit IRC17:20
*** chyka has quit IRC17:20
*** tesseract has quit IRC17:20
*** chyka has joined #openstack-nova17:21
*** andreas_s has quit IRC17:23
*** jpena|brb is now known as jpena17:25
*** slaweq_ has joined #openstack-nova17:26
*** dmsimard has joined #openstack-nova17:26
dmsimardkashyap: o/ mriedem said you could help with a bug we're trying to nail down. Have you gotten familiar with https://bugs.launchpad.net/nova/+bug/1735823 ?17:26
openstackLaunchpad bug 1735823 in OpenStack Compute (nova) "Nova can hang when creating a VM with disk injection" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)17:26
dmsimardtl;dr, we thought we ran libguestfs with force_tcg for disk injection but we didn't, so in some environments (such as OVH nodepool cloud regions) with broken nested virtualization, this failed horribly17:27
dmsimardWe believe the fix is here: https://review.openstack.org/#/c/524748/ -- however, when testing it here: https://review.openstack.org/#/c/524750/, we're seeing increased failures.. and outside of OVH too.17:28
dmsimardBefore the fix (at OVH) http://logs.openstack.org/46/523646/1/check/legacy-tempest-dsvm-neutron-full-centos-7/5bf092c/job-output.txt#_2017-11-29_03_02_38_031436 -- After the fix (at RAX): http://logs.openstack.org/50/524750/1/check/legacy-tempest-dsvm-neutron-full-centos-7/a7f051e/job-output.txt.gz#_2017-12-04_15_25_13_21131717:29
dmsimardI did a recheck in https://review.openstack.org/#/c/524750/ to make sure it wasn't a flap and it does look like it breaks things.17:29
*** mvk has joined #openstack-nova17:30
*** slaweq_ has quit IRC17:31
stephenfinefried: Interesting little way to speed up tox - 'touch nova.egg-ingo'17:32
stephenfinI read the tox source a while back to figure out why, but it skips the "check if any dependencies are out-of-date" step, which half of what takes so long17:32
cdentmriedem: I’m thinking that https://bugs.launchpad.net/nova/+bug/1734491 may have back port potential?17:32
openstackLaunchpad bug 1734491 in OpenStack Compute (nova) "placement keystonemiddleware_authtoken ignores OS_PLACEMENT_CONFIG_DIR" [Undecided,In progress] - Assigned to Chris Dent (cdent)17:32
stephenfinProbably a bug in tox but I haven't figured out why yet17:32
jaypipesefried: let's go with get_provider_uuids() then.17:36
*** andreas_s has joined #openstack-nova17:37
*** ralonsoh has quit IRC17:38
ericyoungAny idea when the patch for ScaleIO ephemeral/image volume support will bubble to the top of the review list? https://review.openstack.org/#/c/407440/17:38
cdentjaypipes: the point on the jsonschema comment was: webob is passing a string to jsonschema, one that looks like an int, but is not an int17:40
cdentjust int will _not_ work17:41
*** thorst_ has joined #openstack-nova17:41
jaypipescdent: if that's the case, then we shouldn't have "integer" in there at all, since we'll never be supplying actual numbers.17:41
*** slaweq_ has joined #openstack-nova17:42
cdentunless we change webob out from under it (this has been discussed further up the review stack)17:42
cdentI can make it just string if you like, but the way it is now is to be as flexible yet accurate as possible17:42
cdentit is also what the nova api does17:43
cdentso I copied it17:43
*** thorst has quit IRC17:44
jaypipescdent: meh...17:44
cdentjaypipes: much meh made merry many months, mostly17:44
*** andreas_s has quit IRC17:46
*** andreas_s has joined #openstack-nova17:46
*** slaweq_ has quit IRC17:47
*** felipemonteiro has joined #openstack-nova17:49
*** lucasagomes is now known as lucas-afk17:49
*** armax has quit IRC17:49
*** tidwellr has quit IRC17:50
*** felipemonteiro_ has joined #openstack-nova17:50
*** tidwellr has joined #openstack-nova17:50
*** thorst_ has quit IRC17:51
*** felipemonteiro has quit IRC17:54
*** armax has joined #openstack-nova17:54
*** tidwellr has quit IRC17:55
*** Apoorva has joined #openstack-nova17:55
*** moshele has joined #openstack-nova17:57
*** trozet has joined #openstack-nova17:58
*** andreas_s has quit IRC18:00
*** rabel has quit IRC18:00
*** thorst has joined #openstack-nova18:02
*** moshele has quit IRC18:03
*** slaweq_ has joined #openstack-nova18:04
*** thorst has quit IRC18:07
*** dtantsur is now known as dtantsur|afk18:08
*** slaweq_ has quit IRC18:09
mriedemedleafe: ok i've made it through the code parts of https://review.openstack.org/#/c/510159/18:11
mriedemedleafe: some things that need to be cleaned up in there, but we could arguably do those in a follow up,18:11
mriedemi will go through test changes after lunch18:11
*** slaweq_ has joined #openstack-nova18:12
edleafemriedem: a follow-up is fine.18:15
*** thorst has joined #openstack-nova18:15
*** suresh12 has joined #openstack-nova18:16
*** slaweq_ has quit IRC18:17
*** mriedem has quit IRC18:19
*** jpena is now known as jpena|away18:20
*** thorst has quit IRC18:20
*** suresh12 has quit IRC18:21
*** salv-orlando has quit IRC18:23
*** salv-orlando has joined #openstack-nova18:24
*** thorst has joined #openstack-nova18:24
openstackgerritMerged openstack/nova master: XenAPI: get vGPU stats from hypervisor  https://review.openstack.org/51296518:26
*** trozet has quit IRC18:28
*** salv-orlando has quit IRC18:28
*** suresh12 has joined #openstack-nova18:30
*** sambetts is now known as sambetts|afk18:30
*** Apoorva has quit IRC18:33
*** Apoorva has joined #openstack-nova18:34
*** thorst has quit IRC18:38
*** Sanatics has joined #openstack-nova18:38
*** salv-orlando has joined #openstack-nova18:39
*** moshele has joined #openstack-nova18:40
*** Sanatics has quit IRC18:40
*** thorst has joined #openstack-nova18:41
*** moshele has quit IRC18:42
*** thorst has quit IRC18:43
*** tidwellr has joined #openstack-nova18:44
*** rwellum has quit IRC18:44
*** rwellum has joined #openstack-nova18:44
*** jpena|away is now known as jpena|off18:48
*** slaweq_ has joined #openstack-nova18:52
*** fragatina has quit IRC18:53
*** armax has quit IRC18:55
*** slaweq_ has quit IRC18:57
*** jmlowe has joined #openstack-nova19:00
*** erlon has joined #openstack-nova19:01
*** corey_ has joined #openstack-nova19:01
*** cleong has quit IRC19:03
*** sdague has quit IRC19:03
*** adisky_ has quit IRC19:10
*** andreas_s has joined #openstack-nova19:17
*** gouthamr has quit IRC19:20
*** auggy has quit IRC19:22
*** ttsiouts_ has joined #openstack-nova19:22
*** ttsiouts_ has quit IRC19:22
*** gszasz has quit IRC19:26
* edleafe runs off to grab lunch19:27
*** armax has joined #openstack-nova19:29
*** liusheng has quit IRC19:29
*** liusheng has joined #openstack-nova19:29
*** nore__ has joined #openstack-nova19:30
*** slaweq_ has joined #openstack-nova19:32
*** nore_rabel has quit IRC19:32
*** andreas_s has quit IRC19:35
*** fragatina has joined #openstack-nova19:37
*** slaweq_ has quit IRC19:37
*** gouthamr has joined #openstack-nova19:37
*** mriedem has joined #openstack-nova19:39
*** nore__ has quit IRC19:40
*** nore__ has joined #openstack-nova19:40
*** thorst has joined #openstack-nova19:42
efriedstephenfin That's a cool tip, thanks.  I doubt it would be considered a "bug" since you're effectively hacking a metadata marker tox uses internally for exactly that purpose.19:43
*** tssurya_ has joined #openstack-nova19:43
*** chyka_ has joined #openstack-nova19:44
*** tidwellr has quit IRC19:44
*** tidwellr_ has joined #openstack-nova19:44
*** gouthamr has quit IRC19:45
*** andreas_s has joined #openstack-nova19:45
*** dave-mccowan has joined #openstack-nova19:45
*** _pewp_ has quit IRC19:45
*** penick has joined #openstack-nova19:45
*** _pewp_ has joined #openstack-nova19:46
*** chyka has quit IRC19:47
*** artom has quit IRC19:47
*** artom has joined #openstack-nova19:48
*** chyka_ has quit IRC19:49
*** thorst has quit IRC19:51
*** trozet has joined #openstack-nova19:51
*** chyka has joined #openstack-nova19:54
*** chyka has quit IRC19:59
*** suresh12 has quit IRC19:59
mriedemedleafe: ok, found a few more missing tests in https://review.openstack.org/#/c/510159/19:59
*** Apoorva has quit IRC19:59
mriedemedleafe: should probably just add those in there rather than a follow up19:59
mriedemthe comment updates could all be done in a follow up20:00
*** itlinux has quit IRC20:01
*** slaweq_ has joined #openstack-nova20:02
*** andreas_s has quit IRC20:03
*** thorst has joined #openstack-nova20:04
*** slaweq__ has joined #openstack-nova20:05
*** slaweq_ has quit IRC20:07
mriedemmoving onto the next change in the series20:07
*** priteau has joined #openstack-nova20:08
*** slaweq__ has quit IRC20:09
*** Apoorva has joined #openstack-nova20:11
*** sdague has joined #openstack-nova20:12
efriededleafe jaypipes What about uuid_iter() ?  (And make it an iterable)20:12
*** slaweq__ has joined #openstack-nova20:13
efriedAh, I think I remember why I didn't do that originally - don't want the caller holding (and possibly abandoning) the lock.20:13
*** suresh12 has joined #openstack-nova20:14
openstackgerritEric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids()  https://review.openstack.org/52024320:20
efriededleafe jaypipes ^20:20
edleafemriedem: if I have to change that patch, might as well do 'em all there.20:28
jaypipesefried: +2 from me. nice.20:30
mriedemedleafe: well, if you're going to update all of those comments,20:30
efriedjaypipes Thanks20:30
jaypipesefried: though you didn't update the make _provider really private patch.20:30
mriedemyou should then do that in the change earlier in the series that actually makes the scheduler drivers return Selection objects back up to the SchedulerManager20:30
efriedjaypipes Uh, was I supposed to?20:30
jaypipesefried: nah, just figured you would... rebase the entire thing to get em up to the same rev20:31
*** moshele has joined #openstack-nova20:31
efriedjaypipes Hum, that's weird; I wouldn't have thought I needed to do that.20:31
jaypipesefried: since gerrit shows the get_provider_uuids() based on an old rev of the dep patch20:31
efriedyeah - why?20:31
efriedI rebased both of 'em earlier.20:31
mriedemedleafe: specifically https://review.openstack.org/#/c/495854/ made the comments invalid20:31
mriedemhttps://review.openstack.org/#/c/510159/ just needs more tests20:32
edleafemriedem: ok20:32
efriedjaypipes ohh, I know what happened.20:32
mriedemedleafe: comments in https://review.openstack.org/#/c/516707/ too - i think we need to not touch anything in the compute manager/claims/RT code there in that patch20:32
efriedI manually edited two characters in the commit message of the bottom patch.20:32
openstackgerritEric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids()  https://review.openstack.org/52024320:33
efriedjaypipes ^ simple rebase from gerrit, kept your +2.  Thanks for pointing it out.20:33
*** slaweq_ has joined #openstack-nova20:33
*** awaugama has quit IRC20:35
jaypipesefried: coolio.20:36
*** gouthamr has joined #openstack-nova20:36
*** slaweq_ has quit IRC20:39
*** slaweq_ has joined #openstack-nova20:40
openstackgerritDan Smith proposed openstack/nova master: Fix cellsv1 build request deletion if multiple api databases are used  https://review.openstack.org/52318720:40
openstackgerritDan Smith proposed openstack/nova master: Fix cellsv1 messaging test  https://review.openstack.org/52530420:40
efrieddansmith Got some review bandwidth for these two? https://review.openstack.org/#/c/523932/  https://review.openstack.org/#/c/520243/  They're pretty simple.20:41
dansmithefried: no, and also, you're doing it wrong20:41
efriedWell, that figures.  What am I doing wrong?20:42
dansmithefried: acting like I have review bandwidth will never work.. playing to my guilt usually will20:42
dansmithwatch and learn:20:42
dansmithdansmith: you haven't reviewed any of my patches in a while and I'm starting to feel like you're a bad person. prove me wrong and review these easy ones I have selected for you...20:42
efriedokay, let me give that a try:20:43
* dansmith waits20:43
mriedemedleafe: some comments that can be updated in https://review.openstack.org/#/c/511357/ when you update the series20:43
*** salv-orlando has quit IRC20:43
efrieddansmith: reviewing these simple patches will not only further the cause of Very Important blueprints, it will make you more handsome and popular.20:43
*** salv-orlando has joined #openstack-nova20:43
dansmithefried: weak, but passable20:43
efriedDangit, yeah, there's almost no guilt in there at all.20:43
dansmithbut it plays to other things that work for me, so..20:44
mriedempassive aggressive works too20:44
efriedLike this: dansmith I think it's great how much time you have for cells stuff.20:44
mriedemno20:45
efrieddamn20:45
*** salv-orlando has quit IRC20:45
efriedI really suck at this.20:45
dansmithyeah, that just makes me mad :)20:45
*** salv-orlando has joined #openstack-nova20:45
mriedemheh, yeah :)20:45
*** slaweq_ has quit IRC20:45
dansmithlol20:45
mriedemyou need to be a true midwesterner to master passive aggressive20:45
efriedMy in-laws are from Wisconsin.  That's as close as I can come.20:45
mriedempsh, wisconsin20:46
efriedIf I had my mother in law asking, it would be like, "dansmith, it's okay if you don't want to review my patches; I'll be dead soon and then it won't matter".20:46
mriedemcheese heads20:46
mriedemand serial killers20:46
mriedemefried: that's more like it20:46
dansmithyeah, getting there20:47
* efried takes notes20:47
*** clutchbeyers has joined #openstack-nova20:50
*** jaosorior has quit IRC20:51
*** penick has quit IRC20:52
*** suresh12 has quit IRC20:52
*** penick has joined #openstack-nova20:52
*** suresh12 has joined #openstack-nova20:53
*** artom has quit IRC20:53
*** artom has joined #openstack-nova20:54
cfriesenis it a bug that "nova quota-show" has "injected_file_path_bytes" but "nova limits" doesn't have an equivalent entry?20:54
mriedemcfriesen: meh20:54
*** pchavva has quit IRC20:55
mriedemisn't that maxPersonalitySize?20:55
cfriesenSo I see "Personality" and "Personality Size"  in limits (in Newton) but for quotas I see injected_files/injected_file_content_bytes/injected_file_path_bytes20:56
*** thorst has quit IRC20:56
clutchbeyerslooking to achieve some type of multi tenancy exclusive isolation on host aggregates, I noticed there were some blueprints that were abandoned. Is anyone aware of any work being done or other mechanism to exclude tenants to an aggregate or global filter that wont provision a tenant to an aggregate that doesnt have such filter meta set?20:57
*** awolf has joined #openstack-nova20:57
melwittcfriesen: did it use to show it in 'nova limits' but doesn't anymore? if so, that's a bug20:58
melwittoh, newton. then not a bug. quota rewriting didn't happen until pike20:59
melwittor rather, not a regression. one could argue whether it's a bug in general21:00
cfriesenmelwitt: okay, will be upgrading soon.21:00
cfriesenwhy do we have both APIs anyway?21:00
openstackgerritEric Fried proposed openstack/nova stable/ocata: Proper error handling by _ensure_resource_provider  https://review.openstack.org/52530921:00
cfriesenqutoa and limits I mean21:01
efriedmriedem ^ There's that cherry-pick.  That was a pretty fun merge.21:01
melwittthat, I don't know21:01
cfriesenokay, thanks21:01
mriedemcfriesen: i'd bet it was likely due to v2.0 api extension stuff21:02
*** chyka has joined #openstack-nova21:02
*** thorst has joined #openstack-nova21:02
*** chyka has quit IRC21:02
mriedemclutchbeyers: https://review.openstack.org/#/c/381912/21:04
mriedemclutchbeyers: no one is working on that21:05
*** chyka has joined #openstack-nova21:05
mriedemmelwitt looked at something related recently, saying maybe it was a regression that we needed this anyway21:05
*** itlinux has joined #openstack-nova21:05
melwittI think that was the lack of counting keypairs for the used limits API. it always returns 021:05
*** itlinux has quit IRC21:06
* melwitt should actually read the link first21:06
*** thorst has quit IRC21:07
melwittthis is the redhat bug where it's demonstrated to be a regression in mitaka https://bugzilla.redhat.com/show_bug.cgi?id=143559721:09
openstackbugzilla.redhat.com bug 1435597 in openstack-nova "AggegateImagePropertiesIsolatefiler not working" [High,New] - Assigned to sbauza21:09
*** slaweq_ has joined #openstack-nova21:10
melwittgiven that, I was a bit lost about how it got turned into a blueprint-required feature21:11
melwittI want to dig into it more to find out what actually happened back then21:11
*** suresh12 has quit IRC21:13
melwitti.e. find what changed that made it stop working and whether it was intentional21:14
*** suresh12 has joined #openstack-nova21:15
*** slaweq_ has quit IRC21:15
melwittI had done some looking while at the summit but didn't find anything yet21:15
*** awaugama has joined #openstack-nova21:16
*** thorst has joined #openstack-nova21:17
*** nicolasbock has joined #openstack-nova21:19
*** suresh12 has quit IRC21:19
*** liverpooler has quit IRC21:19
*** tssurya_ has quit IRC21:20
*** thorst has quit IRC21:20
*** thorst has joined #openstack-nova21:21
*** AJaeger_ has left #openstack-nova21:21
*** smatzek has quit IRC21:23
mriedemhuh, how does this ever work?21:25
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L659521:25
mriedemah oh nvm21:25
*** yamahata has joined #openstack-nova21:26
*** corey_ has quit IRC21:30
*** yamahata has quit IRC21:32
*** tidwellr has joined #openstack-nova21:34
*** tidwellr_ has quit IRC21:34
*** penick has quit IRC21:36
*** thorst has quit IRC21:36
*** penick has joined #openstack-nova21:37
*** pcaruana has quit IRC21:39
mikalmriedem: I commented on that bug, but mostly to say "you people confuse me"21:39
mikalIn other news, Amazon shopping just launched in my country21:40
*** moshele has quit IRC21:40
mriedemmikal: the perf one?21:40
mikalYep, https://bugs.launchpad.net/nova/+bug/173566321:46
openstackLaunchpad bug 1735663 in OpenStack Compute (nova) "privsep.daemon takes 9 seconds during taking volume attaching operation" [Undecided,New]21:46
*** slaweq_ has joined #openstack-nova21:46
openstackgerritMatt Riedemann proposed openstack/nova master: Add regression test for rebuilding a volume-backed server  https://review.openstack.org/52120021:47
openstackgerritMatt Riedemann proposed openstack/nova master: Get original image_id from volume for volume-backed instance rebuild  https://review.openstack.org/52139121:47
openstackgerritMatt Riedemann proposed openstack/nova master: Fail fast if changing image on a volume-backed server rebuild  https://review.openstack.org/52066021:47
mriedemdansmith: think i addressed your comments in ^21:48
dansmithslright21:48
*** thorst has joined #openstack-nova21:50
*** slaweq_ has quit IRC21:51
*** thorst has quit IRC21:54
*** mriedem has quit IRC21:56
*** nore__ has quit IRC21:56
*** clutchbeyers has quit IRC21:56
*** slaweq__ has quit IRC21:58
*** Apoorva_ has joined #openstack-nova22:02
*** takashin has joined #openstack-nova22:02
*** tidwellr has quit IRC22:04
*** gouthamr has quit IRC22:04
*** Apoorva has quit IRC22:05
*** rcernin has joined #openstack-nova22:07
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add aggregate link note in API ref  https://review.openstack.org/52150222:07
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API  https://review.openstack.org/52409422:07
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref  https://review.openstack.org/52154122:08
openstackgerritTakashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells  https://review.openstack.org/52140022:08
openstackgerritTakashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell'  https://review.openstack.org/51377122:08
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API  https://review.openstack.org/52450622:09
*** thorst has joined #openstack-nova22:10
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Fix foreign key constraint error  https://review.openstack.org/51688022:10
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID  https://review.openstack.org/52360622:11
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix missing marker functions  https://review.openstack.org/51457922:11
*** peter-hamilton has quit IRC22:11
openstackgerritHongbin Lu proposed openstack/nova master: [WIP] Handle RPC timeout on get_console_output  https://review.openstack.org/52533522:12
*** namnh has joined #openstack-nova22:14
*** thorst has quit IRC22:14
*** salv-orlando has quit IRC22:15
*** salv-orlando has joined #openstack-nova22:15
*** slaweq_ has joined #openstack-nova22:19
*** salv-orlando has quit IRC22:20
*** awolf has quit IRC22:20
*** namnh has quit IRC22:21
efriedjaypipes Been staring at this for a bit, can't figure out where 1.10 is coming from: http://logs.openstack.org/93/385693/83/check/legacy-grenade-dsvm-neutron/b513223/logs/screen-n-cpu.txt.gz#_Dec_04_15_47_11_98873022:21
*** slaweq_ has quit IRC22:24
*** awaugama has quit IRC22:25
*** salv-orlando has joined #openstack-nova22:27
*** BryanS68 has quit IRC22:30
gryfefried, looks like you run pike version of placement against latest source22:31
efriedgryf Wow, that would be... bad.  How can you tell?22:31
gryf1.10 was the max microversion in pike22:32
*** Apoorva_ has quit IRC22:33
efriedcdent If you're still around -- how does legacy-grenade-dsvm-neutron spin up the placement API?  Is there some chance we could be using the pike version in latest master?22:33
*** armax has quit IRC22:33
efriedcdent And wouldn't lots of other stuff have broken by now if that were the case?22:33
*** Apoorva has joined #openstack-nova22:33
cdentefried: I assume you’re talking about the thing that’s claiming 1.10 is the max version?22:33
efriedcdent yeah22:34
cdentwas just reading that in my email22:34
cdentit could be that a new compute is talking to an older placement, which I think is something we _may_ do intentionally in some tests22:35
efriedoy vay, that could be... complicated to fix.22:35
cdentand for the most part we’ve tried to make the report client be smart about that22:36
* cdent looks at the code in the changes22:36
*** armax has joined #openstack-nova22:36
cdentefried: hmmm. yeah, that code doesn’t have a fall back, but I’m not sure we want that or not. jaypipes, mriedem or dansmith will be able to say more on whether we need to be able to work with new compute/old placement22:37
efriedcdent Okay, I gotcha.22:38
cdentIf we don’t need to support that, the other chance is simply that the new placement hasn’t manage to get restarted yet22:38
dansmithold compute needs to work with new placement22:38
cdentdansmith: yes, but vice versa?22:38
dansmithnot vice versa22:39
efriedcdent "hasn't managed to get restarted yet" - so actually a recheck may clear it?22:39
cdentefried: probably, but it may be that the process mgt in grenade is insufficiently robust and could be robustified22:40
efriedBTW, this was just exposed because of https://review.openstack.org/#/c/524263/ -- I think otherwise we would have blown right by this error and not noticed.  So it may be a long-standing latent bug.22:40
cdentyup, I dig22:41
*** lyan has quit IRC22:41
cdentefried: if you haven’t got time to think about grenade right now (or ever), I’m happy to go digging instead, but wouldn’t be until tomorrow22:41
efriedWell, I guess I'll try a recheck, but it seems like something that ought to be fixed *somewhere*22:42
efriedcdent I've never heard of grenade.22:42
efriedAnd it's jaypipes' patch anyway :P22:42
cdentcode ownership is, like, _so_ 1990, but yeah, my curiosity is piqued22:43
cdentgrenade is the sort of framework used to manage upgrade testing22:43
cdenta devstack in the previous version is started, verified a bit22:43
cdentthen new code is installed, but old configruations kept22:44
*** Yingxin has quit IRC22:44
*** Yingxin has joined #openstack-nova22:48
openstackgerritEd Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver  https://review.openstack.org/49585422:50
openstackgerritEd Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object  https://review.openstack.org/52349222:50
openstackgerritEd Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts  https://review.openstack.org/51015922:50
edleafemriedem: ^^ addressed your comments22:50
edleafemriedem: will start on https://review.openstack.org/#/c/516707/ tomorrow22:51
*** namnh has joined #openstack-nova22:51
openstackgerritEric Fried proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721522:52
openstackgerritEric Fried proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480722:52
openstackgerritEric Fried proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569322:52
efriedjaypipes ^ this is just a rebase on top of _Provider-private so I can start rebasing the other stuff in that series (e.g. _set_inventory doing nested stuff)22:52
*** felipemonteiro_ has quit IRC22:53
*** thorst has joined #openstack-nova22:53
*** priteau has quit IRC22:54
*** namnh has quit IRC22:56
*** slaweq_ has joined #openstack-nova22:57
*** thorst has quit IRC22:58
jaypipesefried: roger22:59
*** suresh12 has joined #openstack-nova23:00
*** slaweq_ has quit IRC23:02
*** namnh has joined #openstack-nova23:03
*** namnh has quit IRC23:08
*** gouthamr has joined #openstack-nova23:08
*** esberglu has quit IRC23:18
*** namnh has joined #openstack-nova23:19
*** esberglu has joined #openstack-nova23:19
*** mlavalle has quit IRC23:21
*** namnh has quit IRC23:23
*** esberglu has quit IRC23:23
openstackgerritEric Fried proposed openstack/nova master: Scheduler set_inventory_for_provider does nested  https://review.openstack.org/52064323:24
efriedjaypipes Like that ^23:24
efriedjaypipes That _validate_provider thing really comes into its own now.23:24
efriedin the tests23:24
*** gmann_afk is now known as gmann23:27
efriedjaypipes cdent et al: FYI I'm out tomorrow; working W, Th, and first half of Fr; then (officially) out til Jan.  I'll probably be able to check in periodically to shepherd simple rebases/reviews, but not a lot of big stuff.23:28
*** namnh has joined #openstack-nova23:28
cdentefried: roger that. I’m fairly around throughout most of december, happy to mangle your stuff as required during that time23:29
efriedcdent I really hope to get things in a state where that would be remotely possible before I bugger off.  Right now most of my patches are kind of a mess.23:29
efriedcdent And thank you for the offer.23:30
*** esberglu has joined #openstack-nova23:31
*** thorst has joined #openstack-nova23:33
*** namnh has quit IRC23:33
*** thorst has quit IRC23:33
*** slaweq_ has joined #openstack-nova23:35
*** penick has quit IRC23:37
*** armax has quit IRC23:38
*** yamamoto has joined #openstack-nova23:38
*** smatzek has joined #openstack-nova23:38
*** esberglu has quit IRC23:39
*** slaweq_ has quit IRC23:40
*** slaweq_ has joined #openstack-nova23:41
*** efried is now known as efried_cya_wed23:41
*** namnh has joined #openstack-nova23:42
*** smatzek has quit IRC23:43
*** thorst has joined #openstack-nova23:45
*** slaweq_ has quit IRC23:46
*** namnh has quit IRC23:47
*** thorst has quit IRC23:49
*** thorst has joined #openstack-nova23:51
openstackgerritHongbin Lu proposed openstack/nova master: Add support for listing hosts in cellv2  https://review.openstack.org/52475523:52
*** trinaths has joined #openstack-nova23:52
*** armax has joined #openstack-nova23:53
*** jmlowe has quit IRC23:55
*** thorst has quit IRC23:55
*** gouthamr has quit IRC23:56
*** gouthamr has joined #openstack-nova23:58
*** namnh has joined #openstack-nova23:58
*** hongbin has quit IRC23:59
*** ankit_ has quit IRC23:59
*** felipemonteiro has joined #openstack-nova23:59
*** danice has joined #openstack-nova23:59

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