Thursday, 2016-06-16

*** openstack has joined #openstack-nova05:58
*** zenoway has joined #openstack-nova05:59
*** diga has quit IRC06:00
*** gzhai2 has quit IRC06:02
*** alex_xu has quit IRC06:03
*** zenoway has quit IRC06:04
*** gzhai2 has joined #openstack-nova06:04
*** alex_xu has joined #openstack-nova06:06
*** gjayavelu has joined #openstack-nova06:07
*** ljxiash has joined #openstack-nova06:09
*** gjayavelu has quit IRC06:11
*** lpetrut has quit IRC06:13
*** ljxiash has quit IRC06:14
*** andreas_s has joined #openstack-nova06:16
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907706:18
*** diga has joined #openstack-nova06:22
*** tbachman has quit IRC06:22
*** zenoway has joined #openstack-nova06:23
*** lpetrut has joined #openstack-nova06:24
*** diga_ has quit IRC06:24
*** rcernin has joined #openstack-nova06:24
*** tbachman has joined #openstack-nova06:25
*** diga has quit IRC06:26
*** zenoway has quit IRC06:28
*** EinstCrazy has quit IRC06:28
*** EinstCrazy has joined #openstack-nova06:29
*** EinstCrazy has quit IRC06:30
*** EinstCrazy has joined #openstack-nova06:30
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907706:30
*** chandankumar has joined #openstack-nova06:31
*** chandankumar has quit IRC06:32
*** chandankumar has joined #openstack-nova06:33
*** adisky has joined #openstack-nova06:33
*** Daisy_ has joined #openstack-nova06:36
openstackgerritLuis Tomas proposed openstack/nova: Add automatic switching to postcopy mode when migration is not progressing  https://review.openstack.org/32849906:37
*** rama_y has joined #openstack-nova06:38
*** diga has joined #openstack-nova06:39
*** Daisy has quit IRC06:39
*** sree_ has joined #openstack-nova06:40
*** lpetrut has quit IRC06:40
*** EinstCrazy has quit IRC06:41
*** sree_ is now known as Guest2139606:41
*** andreykurilin has quit IRC06:42
*** sree has quit IRC06:43
*** moshele has joined #openstack-nova06:43
openstackgerritPawel Koniszewski proposed openstack/nova: libvirt: Live migration fails when config_drive_format=iso9660  https://review.openstack.org/23465906:44
*** thorst_ has joined #openstack-nova06:44
*** EinstCrazy has joined #openstack-nova06:45
*** ratailor has quit IRC06:46
*** Daisy has joined #openstack-nova06:49
*** salv-orlando has quit IRC06:49
*** salv-orlando has joined #openstack-nova06:50
*** Kevin_Zheng has joined #openstack-nova06:51
*** Sukhdev has quit IRC06:51
*** Daisy_ has quit IRC06:51
*** hoangcx has quit IRC06:51
*** afazekas is now known as afazekas|dentist06:52
*** thorst_ has quit IRC06:52
*** hoangcx has joined #openstack-nova06:54
*** salv-orl_ has joined #openstack-nova06:55
*** salv-orlando has quit IRC06:57
*** zenoway has joined #openstack-nova06:58
*** Daisy_ has joined #openstack-nova06:59
*** ratailor has joined #openstack-nova07:00
*** Daisy has quit IRC07:00
*** zenoway has quit IRC07:00
openstackgerritAndrey Volkov proposed openstack/nova: Apply cell context in compute API  https://review.openstack.org/32218107:01
*** zenoway has joined #openstack-nova07:01
*** markvoelker_ has quit IRC07:01
*** markvoelker has joined #openstack-nova07:01
*** Daisy_ has quit IRC07:04
*** mpavone has joined #openstack-nova07:08
*** ratailor has quit IRC07:08
*** tesseract has joined #openstack-nova07:09
*** jichen has joined #openstack-nova07:12
*** jamielennox is now known as jamielennox|away07:12
*** ratailor has joined #openstack-nova07:13
*** sahid has joined #openstack-nova07:13
*** jed56 has joined #openstack-nova07:16
*** bharathk has quit IRC07:16
*** bharathk has joined #openstack-nova07:17
*** pcaruana has joined #openstack-nova07:17
*** permalac has quit IRC07:19
*** matrohon has joined #openstack-nova07:20
*** roeyc has joined #openstack-nova07:22
*** rama_y has quit IRC07:22
*** paul-carlton2 has joined #openstack-nova07:24
*** jlanoux has joined #openstack-nova07:36
*** Guest21396 has quit IRC07:37
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907707:39
*** jlanoux_ has joined #openstack-nova07:41
*** jlanoux has quit IRC07:41
*** acabot has joined #openstack-nova07:42
*** kashyap` is now known as kashyap07:46
*** diga has quit IRC07:46
*** baater has joined #openstack-nova07:47
baaterhttp://paste.openstack.org/show/516488/     can anyone help me to solve this issue ?07:48
*** salv-orl_ has quit IRC07:49
*** thorst_ has joined #openstack-nova07:49
*** salv-orlando has joined #openstack-nova07:49
*** edand has joined #openstack-nova07:49
*** danpb has joined #openstack-nova07:51
*** abalutoiu has joined #openstack-nova07:52
*** sree has joined #openstack-nova07:52
*** hoangcx_ has joined #openstack-nova07:54
*** hoangcx has quit IRC07:55
*** sfinucan has joined #openstack-nova07:55
*** thorst_ has quit IRC07:56
*** zzzeek has quit IRC08:00
bauzasgood morning Nova08:00
*** zzzeek has joined #openstack-nova08:00
baaterhttp://paste.openstack.org/show/516488/ can anyone help me to solve this issue ?08:00
baateranyone ?08:04
*** hoangcx_ is now known as hoangcx08:04
*** jaosorior has quit IRC08:07
*** lpetrut has joined #openstack-nova08:07
openstackgerritPaul Carlton proposed openstack/python-novaclient: [microversion] Bump to 2.31  https://review.openstack.org/31602608:07
*** jaosorior has joined #openstack-nova08:08
*** permalac has joined #openstack-nova08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c  https://review.openstack.org/30334508:10
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c  https://review.openstack.org/30334508:10
*** lpetrut1 has joined #openstack-nova08:12
*** lpetrut has quit IRC08:12
*** lpetrut1 is now known as lpetrut08:12
baaterhttp://paste.openstack.org/show/516488/ can anyone help me to solve this issue ?08:17
*** Daisy has joined #openstack-nova08:17
*** Daisy_ has joined #openstack-nova08:18
*** Daisy has quit IRC08:18
openstackgerritPawel Koniszewski proposed openstack/nova: Remove block_migration argument from live migration operations  https://review.openstack.org/29531608:22
openstackgerritPawel Koniszewski proposed openstack/nova: Remove block_migration from LM post operations  https://review.openstack.org/29531508:22
openstackgerritPawel Koniszewski proposed openstack/nova: Remove block_migration from LM rollback  https://review.openstack.org/29282608:22
*** lucas-afk is now known as lucasagomes08:28
*** BobBall_AWOL is now known as BobBall08:30
openstackgerritMatthew Booth proposed openstack/nova: Remove unused context argument to _default_block_device_names()  https://review.openstack.org/32992808:32
openstackgerritMatthew Booth proposed openstack/nova: Rename compute manager _check_dev_name to _add_missing_dev_names  https://review.openstack.org/32993008:32
openstackgerritMatthew Booth proposed openstack/nova: rename libvirt has_default_ephemeral  https://review.openstack.org/32936608:32
openstackgerritMatthew Booth proposed openstack/nova: Remove virt.block_device._NoLegacy exception  https://review.openstack.org/32992708:32
openstackgerritStefan Amann proposed openstack/nova: Support FICON-attached ECKD volumes on System z  https://review.openstack.org/32083208:33
*** Daisy_ has quit IRC08:34
*** Daisy has joined #openstack-nova08:34
*** Daisy has quit IRC08:36
*** Daisy has joined #openstack-nova08:36
*** shirly_ has joined #openstack-nova08:39
*** shirly_ has left #openstack-nova08:40
*** sree has quit IRC08:42
*** jamie_h has joined #openstack-nova08:43
*** yassine__ has joined #openstack-nova08:45
openstackgerritAnh Tran proposed openstack/nova: Enables Py34 tests for unit.api.openstack.compute.test_server_actions  https://review.openstack.org/32943508:46
*** zenoway has quit IRC08:46
*** derekh has joined #openstack-nova08:47
*** zenoway has joined #openstack-nova08:47
*** sree has joined #openstack-nova08:48
*** sree has quit IRC08:49
*** sree has joined #openstack-nova08:49
baaterhttp://paste.openstack.org/show/516488/ can anyone help me to solve this issue ?08:50
*** romainh has joined #openstack-nova08:50
*** zenoway has quit IRC08:52
*** Daisy has quit IRC08:53
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.update notification  https://review.openstack.org/32592308:53
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.delete notifications  https://review.openstack.org/31365408:53
mdbooth_baater: Deployment issues are best asked in #openstack08:53
*** zenoway has joined #openstack-nova08:53
*** Daisy has joined #openstack-nova08:53
*** mdbooth_ is now known as mdbooth08:53
*** thorst_ has joined #openstack-nova08:54
*** baater has quit IRC08:54
*** salv-orlando has quit IRC08:55
*** salv-orlando has joined #openstack-nova08:56
*** Daisy_ has joined #openstack-nova08:56
openstackgerritsahid proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448908:57
openstackgerritsahid proposed openstack/nova: libvirt: pass a reference of libvirt driver to migration  https://review.openstack.org/32899708:57
openstackgerritsahid proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML  https://review.openstack.org/28674408:57
*** Daisy has quit IRC08:58
*** Daisy has joined #openstack-nova08:58
*** artom has quit IRC08:58
*** Daisy has quit IRC08:58
*** Daisy has joined #openstack-nova08:58
*** artom has joined #openstack-nova09:00
*** artom has quit IRC09:00
*** artom has joined #openstack-nova09:01
*** Daisy_ has quit IRC09:01
*** amotoki has quit IRC09:01
*** thorst_ has quit IRC09:01
*** andreykurilin has joined #openstack-nova09:02
*** dmk0202 has joined #openstack-nova09:02
lyarwoodmdbooth: morning, just a reminder if you have time https://review.openstack.org/#/c/329030/ could use a review09:05
mdboothlyarwood: Yup, gimme a few minutes.09:05
lyarwoodmdbooth: np, thanks again09:05
*** PaulMurray has joined #openstack-nova09:09
*** Kevin_Zheng has quit IRC09:11
*** sree has quit IRC09:16
*** ljxiash has joined #openstack-nova09:22
*** mnestratov has joined #openstack-nova09:24
*** ihrachys has joined #openstack-nova09:26
*** paul-carlton2 has quit IRC09:27
danpbhmm, is it valid to change the order of method parameters in  the nova/compute/rpcapi.py file09:27
danpbeg https://review.openstack.org/#/c/327713/5/nova/compute/rpcapi.py09:27
*** mkoderer__ has quit IRC09:28
danpbi would think that is a back-compatibility problem for the RPC api wrt other services still running old versions with the other ordering09:28
*** paul-carlton2 has joined #openstack-nova09:31
*** Daisy has quit IRC09:32
openstackgerritZhenyu Zheng proposed openstack/nova: [WIP] Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040609:32
*** cdent has joined #openstack-nova09:34
openstackgerritChris Dent proposed openstack/nova: Add Allocation and AllocationList objects  https://review.openstack.org/28244209:34
openstackgerritChris Dent proposed openstack/nova: Add delete_, update_ and add_ inventory to ResourceProvider  https://review.openstack.org/32827609:34
openstackgerritChris Dent proposed openstack/nova: Add support for resource inventories to placement API  https://review.openstack.org/32915209:34
openstackgerritChris Dent proposed openstack/nova: Add a ResourceProviderList object  https://review.openstack.org/32830709:34
openstackgerritChris Dent proposed openstack/nova: Add set_inventory() method on ResourceProvider  https://review.openstack.org/32644009:34
openstackgerritChris Dent proposed openstack/nova: Add initial framing of placement API  https://review.openstack.org/32914909:34
openstackgerritChris Dent proposed openstack/nova: WIP Add a Usage and UsageList object  https://review.openstack.org/32957509:34
openstackgerritChris Dent proposed openstack/nova: Add support for resource_providers urls  https://review.openstack.org/32915109:34
openstackgerritChris Dent proposed openstack/nova: Add support for usages in the placement API  https://review.openstack.org/32964309:34
openstackgerritChris Dent proposed openstack/nova: Add microversion support for the placement API  https://review.openstack.org/32915009:34
openstackgerritChris Dent proposed openstack/nova: WIP experiment with the remainder of the placement API  https://review.openstack.org/29310409:34
*** ociuhandu has quit IRC09:39
*** srushti has quit IRC09:43
*** Kevin_Zheng has joined #openstack-nova09:43
*** prateek has quit IRC09:44
*** prateek has joined #openstack-nova09:44
*** zenoway has quit IRC09:44
*** zenoway has joined #openstack-nova09:46
openstackgerritBob Ball proposed openstack/nova: XenAPI: Integers returned from XAPI are actually strings  https://review.openstack.org/33005609:46
*** zenoway has quit IRC09:50
*** roeyc has quit IRC09:53
mdboothlyarwood: Done https://review.openstack.org/#/c/329030/09:56
*** thorst_ has joined #openstack-nova09:59
*** cheneydc has quit IRC10:01
*** diga has joined #openstack-nova10:02
lyarwoodmdbooth: thanks10:02
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (5)  https://review.openstack.org/33042910:04
*** pacoblox has joined #openstack-nova10:05
*** thorst_ has quit IRC10:06
*** mgoddard_ has joined #openstack-nova10:08
*** pacoblox has quit IRC10:08
*** salv-orlando has quit IRC10:09
*** sambetts|afk is now known as sambetts10:09
*** mgoddard has quit IRC10:11
openstackgerritRajesh Tailor proposed openstack/nova: Fix race for compute node create record, on compute-start  https://review.openstack.org/30441510:12
*** rwmjones_ is now known as rwmjones10:13
*** jamielennox|away is now known as jamielennox10:15
*** chandankumar has quit IRC10:19
openstackgerritsahid proposed openstack/nova: libvirt: pass libvirt reference methods to migration  https://review.openstack.org/32899710:21
openstackgerritsahid proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML  https://review.openstack.org/28674410:21
*** ociuhandu has joined #openstack-nova10:25
*** mvk_ has quit IRC10:26
*** sree has joined #openstack-nova10:29
*** tbachman has quit IRC10:30
*** roeyc has joined #openstack-nova10:34
*** roeyc has quit IRC10:36
*** hoangcx has quit IRC10:36
*** smatzek has joined #openstack-nova10:36
*** winston-d_ has quit IRC10:37
*** sdake has joined #openstack-nova10:40
openstackgerritbailin.zhang proposed openstack/nova: Unplug vifs when VM create failed  https://review.openstack.org/30621910:41
openstackgerritbailin.zhang proposed openstack/nova: Delete reduplicate code in test_compute_mgr.py  https://review.openstack.org/32373310:41
*** sdake_ has joined #openstack-nova10:42
*** ljxiash has quit IRC10:42
*** ljxiash has joined #openstack-nova10:43
*** roeyc has joined #openstack-nova10:45
*** sdake has quit IRC10:45
*** ljxiash has quit IRC10:47
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (6)  https://review.openstack.org/33046010:52
*** mvk_ has joined #openstack-nova10:53
*** luis5tb has quit IRC10:54
openstackgerritAnh Tran proposed openstack/nova: Enables Py34 tests for unit.api.openstack.compute.test_server_actions  https://review.openstack.org/32943510:55
artomdanpb, thanks for all your reviews. I'm just going to push out a series that (hopefully) fixes some tests, we haven't addressed your feedback yet10:56
openstackgerritArtom Lifshitz proposed openstack/nova: libvirt: attach configdrive after instance XML  https://review.openstack.org/30333510:57
openstackgerritArtom Lifshitz proposed openstack/nova: objects: adding an update method to virtual_interface  https://review.openstack.org/31061810:57
openstackgerritArtom Lifshitz proposed openstack/nova: objects: Add devices_metadata to instance object  https://review.openstack.org/32792010:57
openstackgerritArtom Lifshitz proposed openstack/nova: db: add a device_metadata column to instance_extra  https://review.openstack.org/31888310:57
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging metadata API support  https://review.openstack.org/30451110:57
openstackgerritArtom Lifshitz proposed openstack/nova: objects: new InstanceDevicesMetadata object  https://review.openstack.org/31888410:57
openstackgerritArtom Lifshitz proposed openstack/nova: Device tagging API support  https://review.openstack.org/30451010:57
*** cdelatte has joined #openstack-nova10:57
openstackgerritArtom Lifshitz proposed openstack/nova: Compute manager device tagging support  https://review.openstack.org/26401710:57
openstackgerritArtom Lifshitz proposed openstack/nova: Libvirt driver implementation of device tagging  https://review.openstack.org/26401610:57
*** tyrola has joined #openstack-nova10:58
*** tbachman has joined #openstack-nova10:59
*** Kevin-Zhao has joined #openstack-nova11:00
Kevin-ZhaoHi11:00
danpbartom: fwiw, overall i think the series looks pretty good now - the config drive issue is the only big problem i see11:02
danpband hopefully that's easy enough to address11:02
*** thorst_ has joined #openstack-nova11:03
danpbartom: i'm away for a week as of 4pm today btw...11:04
pkoniszewskidanpb: hi, given that you will be away for a next week and that non priority freeze is near, can i ask you for your opinion on what we should do if, e.g., post copy is permitted but is not available due to old version of libvirt/QEMU? should we raise exception or just log a warning and move forward?11:06
danpbpkoniszewski: simply don't use post-copy - no need for a warning either11:06
danpbpkoniszewski: that's why we called the config parameter  "permit_post_copy" and not "use_post_copy" - it'll be perfectly reasonable if post-copy is not available - at most you could do LOG.info, but if you did that you'd want to make sure we only emit it once, not every time we run a migration11:07
*** salv-orlando has joined #openstack-nova11:07
*** moshele has quit IRC11:07
pkoniszewskidanpb: thanks, that's what i wanted to see in implementation (+ warning), but if you say that warning is not needed, well, it's just a permission...11:08
*** moshele has joined #openstack-nova11:08
*** tyrola has quit IRC11:09
pkoniszewskidanpb: yeah, right now it will be emitted once, during init_host11:09
danpbthat's fine then11:09
pkoniszewskidanpb: ok, thanks11:10
*** salv-orlando has quit IRC11:10
*** sdake_ has quit IRC11:10
*** salv-orlando has joined #openstack-nova11:11
*** thorst_ has quit IRC11:11
*** gjayavelu has joined #openstack-nova11:12
*** rtheis has joined #openstack-nova11:13
*** irenab has joined #openstack-nova11:13
*** irenab has quit IRC11:15
*** gjayavelu has quit IRC11:16
*** chandankumar has joined #openstack-nova11:20
*** kfarr has joined #openstack-nova11:21
*** hoangcx has joined #openstack-nova11:24
openstackgerritZhihai Song proposed openstack/nova: Transform instance.power_on notifications  https://review.openstack.org/32974611:24
artomdanpb, thanks for the encouragement - your +2s won't remain if the patches are updated, right?11:31
*** lucasagomes is now known as lucas-hungry11:31
*** jed56 has quit IRC11:34
*** kaisers1 has joined #openstack-nova11:34
*** jed56 has joined #openstack-nova11:34
openstackgerritZhihai Song proposed openstack/nova: Transform instance.power_on notifications  https://review.openstack.org/32974611:34
*** luis5tb has joined #openstack-nova11:36
*** hoangcx has quit IRC11:39
*** hoangcx has joined #openstack-nova11:40
*** ljxiash has joined #openstack-nova11:42
*** kaisers1 has quit IRC11:44
*** moshele has quit IRC11:45
*** salv-orlando has quit IRC11:45
*** kaisers1 has joined #openstack-nova11:45
tpatzigalaski_: johnthetubaguy: do you share the proposal to split the patch: https://review.openstack.org/#/c/280301/ ?11:46
*** salv-orlando has joined #openstack-nova11:46
*** gjayavelu has joined #openstack-nova11:46
*** ljxiash has quit IRC11:47
tpatzigalaski_: johnthetubaguy: functional tests will then only work if all patches are in place11:47
*** andreykurilin has quit IRC11:47
*** RuiChen has quit IRC11:48
*** andreykurilin has joined #openstack-nova11:48
mdboothMy magnus opus: http://lists.openstack.org/pipermail/openstack-dev/2016-June/097529.html11:49
mdboothmagnum11:49
mdboothHmm, something in my email stack uses proportional fonts.11:49
mdboothNo matter.11:49
*** kolesovdv has joined #openstack-nova11:52
*** RuiChen has joined #openstack-nova11:53
*** thorst_ has joined #openstack-nova11:53
*** ducttape_ has quit IRC11:54
*** ducttape_ has joined #openstack-nova11:54
*** claudiub has joined #openstack-nova11:55
*** thorst__ has joined #openstack-nova11:55
eliqiaohi danpb, could you help to take a look at libvirt driver patch: https://review.openstack.org/#/q/topic:bp/support-perf-event , thanks in advanced.11:56
openstackgerritZhenyu Zheng proposed openstack/nova: [WIP] Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040611:57
openstackgerritjichenjc proposed openstack/nova: Not using mox stubs in nova.tests.unit.volume  https://review.openstack.org/30659711:57
Kevin-ZhaoHi All ,  I have submit a bug here about scsi controller index problem : https://bugs.launchpad.net/nova/+bug/1591827 And here is the patch to fix it https://review.openstack.org/#/c/329403/11:58
openstackLaunchpad bug 1591827 in OpenStack Compute (nova) "Scsi device can't be recognized in Guest in aarch64" [Undecided,In progress] - Assigned to Kevin Zhao (kevin-zhao)11:58
*** thorst_ has quit IRC11:58
Kevin-ZhaoCould you give some comments ? Thanks:-)11:58
*** Oku_OS is now known as Oku_OS-away11:58
*** hoangcx has quit IRC11:59
*** hoangcx has joined #openstack-nova11:59
*** Oku_OS-away is now known as Oku_OS12:00
*** bharathk has quit IRC12:00
openstackgerritecelik proposed openstack/nova: Improve the help text for cells options (8)  https://review.openstack.org/31661412:05
*** sdake has joined #openstack-nova12:05
*** diga has quit IRC12:05
*** zenoway has joined #openstack-nova12:06
*** ducttape_ has quit IRC12:07
*** zerda2 has quit IRC12:08
*** hoangcx has quit IRC12:10
*** claudiub has quit IRC12:12
*** hoangcx has joined #openstack-nova12:13
*** smatzek has quit IRC12:13
openstackgerritChris Dent proposed openstack/nova: Add Allocation and AllocationList objects  https://review.openstack.org/28244212:13
openstackgerritChris Dent proposed openstack/nova: Add delete_, update_ and add_ inventory to ResourceProvider  https://review.openstack.org/32827612:13
openstackgerritChris Dent proposed openstack/nova: Add support for resource inventories to placement API  https://review.openstack.org/32915212:13
openstackgerritChris Dent proposed openstack/nova: Add a ResourceProviderList object  https://review.openstack.org/32830712:13
openstackgerritChris Dent proposed openstack/nova: Add set_inventory() method on ResourceProvider  https://review.openstack.org/32644012:13
openstackgerritChris Dent proposed openstack/nova: Add initial framing of placement API  https://review.openstack.org/32914912:13
openstackgerritChris Dent proposed openstack/nova: WIP Add a Usage and UsageList object  https://review.openstack.org/32957512:13
openstackgerritChris Dent proposed openstack/nova: Add support for resource_providers urls  https://review.openstack.org/32915112:13
openstackgerritChris Dent proposed openstack/nova: Add support for usages in the placement API  https://review.openstack.org/32964312:13
openstackgerritChris Dent proposed openstack/nova: Add microversion support for the placement API  https://review.openstack.org/32915012:13
openstackgerritChris Dent proposed openstack/nova: WIP experiment with the remainder of the placement API  https://review.openstack.org/29310412:13
*** ratailor has quit IRC12:14
openstackgerritTimofey Durakov proposed openstack/nova: live_migration_cleanup method introduced  https://review.openstack.org/30261112:15
openstackgerritTimofey Durakov proposed openstack/nova: Cleanup for post/rollback of live-migrations  https://review.openstack.org/28065312:15
openstackgerritTimofey Durakov proposed openstack/nova: New field for LibvirtLiveMigrateData  https://review.openstack.org/28674512:15
openstackgerritTimofey Durakov proposed openstack/nova: move disk_info calc to check methods  https://review.openstack.org/29116012:15
*** slagle_ is now known as slagle12:16
openstackgerritTimofey Durakov proposed openstack/nova: implement async rpc for live-migration  https://review.openstack.org/29116112:16
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907712:17
*** claudiub has joined #openstack-nova12:19
*** hoangcx has quit IRC12:19
*** lucas-hungry is now known as lucasagomes12:22
*** hoangcx has joined #openstack-nova12:23
*** rcernin has quit IRC12:24
*** gjayavelu has quit IRC12:27
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 1 before functional change  https://review.openstack.org/32376112:27
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: use "log" element in char devices  https://review.openstack.org/32376012:27
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 3 before functional change  https://review.openstack.org/32376312:27
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 2 before functional change  https://review.openstack.org/32376212:27
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: use the logd sub-element for char devices  https://review.openstack.org/32376512:27
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: refactoring 4 before functional change  https://review.openstack.org/32376412:27
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: virtlogd: introduce domain xml element "log"  https://review.openstack.org/32375912:27
*** prateek has quit IRC12:27
johnthetubaguytpatzig: it is very common to have a patch that adds the compute api.py and DB changes, then a second patch that adds the API12:28
johnthetubaguytpatzig: this does look too big for a single patch, really12:28
*** kfarr has quit IRC12:28
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Fix the clean up of a resize for volume backed instances  https://review.openstack.org/32903012:29
tpatzigjohnthetubaguy: ok, then i split it up in api and DB part. just wanted to make sure beforehand12:30
johnthetubaguytpatzig: totally worth checking, no problem12:30
*** pradk has joined #openstack-nova12:31
tpatzigjohnthetubaguy: but the functional tests will then require both patches12:31
*** tbachman has quit IRC12:32
*** rook-lappy has quit IRC12:33
*** smatzek has joined #openstack-nova12:34
*** vladikr has joined #openstack-nova12:35
lyarwoodmdbooth: https://review.openstack.org/329030 updated again, I've gone back to self._is_booted_from_volume just to make this a saner backport for now, I'll post a follow up series cleaning things up more once this lands.12:35
*** rook-lappy has joined #openstack-nova12:36
*** ducttape_ has joined #openstack-nova12:36
*** moshele has joined #openstack-nova12:37
*** ducttape_ has quit IRC12:38
*** rcernin has joined #openstack-nova12:39
Kevin-ZhaoHi All ,  I have submit a bug here about scsi controller index problem : https://bugs.launchpad.net/nova/+bug/1591827 And here is the patch to fix it https://review.openstack.org/#/c/329403/    Pls help to review :-)12:40
openstackLaunchpad bug 1591827 in OpenStack Compute (nova) "Scsi device can't be recognized in Guest in aarch64" [Undecided,In progress] - Assigned to Kevin Zhao (kevin-zhao)12:40
*** zul_ has joined #openstack-nova12:41
*** tbachman has joined #openstack-nova12:41
*** kylek3h has joined #openstack-nova12:41
*** jamesdenton has joined #openstack-nova12:42
*** zul_ has quit IRC12:42
*** zul_ has joined #openstack-nova12:42
*** zul_ has quit IRC12:43
*** zul_ has joined #openstack-nova12:44
*** eharney has quit IRC12:44
*** edmondsw has joined #openstack-nova12:45
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.update notification  https://review.openstack.org/32592312:45
openstackgerritBalazs Gibizer proposed openstack/nova: Pre-add instance actions to avoid merge conflicts  https://review.openstack.org/33051312:45
*** rodrigods has quit IRC12:46
*** rodrigods has joined #openstack-nova12:46
*** kylek3h has quit IRC12:47
*** baoli has joined #openstack-nova12:47
*** zul_ has quit IRC12:49
*** annegentle has joined #openstack-nova12:49
*** zul has quit IRC12:49
*** zul has joined #openstack-nova12:50
*** roeyc has quit IRC12:50
*** tbachman_ has joined #openstack-nova12:51
openstackgerritLudovic Beliveau proposed openstack/nova: Trivial: Remove unused argument from update_usage_from_migration()  https://review.openstack.org/32923312:51
*** baoli_ has joined #openstack-nova12:53
*** gcb has quit IRC12:53
*** rcernin has quit IRC12:54
*** tbachman has quit IRC12:54
*** tbachman_ is now known as tbachman12:54
*** baoli has quit IRC12:55
*** ljxiash has joined #openstack-nova12:55
*** aysyd has joined #openstack-nova12:58
*** thorst__ is now known as thorst_12:58
tdurakovdanpb: hi, are you around?12:58
tdurakovread your comment, so got a question, if nova will use all available live-migration tweak in future by default, what is the point exposing this over config?12:59
tdurakovpkoniszewski, hi, maybe you could answer^12:59
*** roeyc has joined #openstack-nova13:00
*** rlrossit has joined #openstack-nova13:00
danpbtdurakov: the config gives the option to forceably disable the method13:01
*** tbachman has quit IRC13:01
danpbtdurakov: eg if a particular deployment is ultra-paranoid they might not wish to use post-copy13:01
tdurakovdanpb: ok, sounds valid13:02
danpbthis is all discussed & dealt with in the spec already13:02
tdurakovdanpb: looks like I missed this part13:02
mdboothlyarwood: kk, will look again in a bit13:03
openstackgerritlinbing proposed openstack/nova: VMware: Fix bug of TypeError when getting reference of VCenter cluster is None.  https://review.openstack.org/25767413:03
tdurakovdanpb: haven't found this in spec, anyway thank you for explicitly sharing this details.13:06
*** rcernin has joined #openstack-nova13:07
danpbtdurakov: most of it was probably in review comments13:08
*** mdrabe has joined #openstack-nova13:10
*** pacoblox has joined #openstack-nova13:11
*** tbachman has joined #openstack-nova13:13
*** hemanthm_ is now known as hemanthm13:14
openstackgerritChris Dent proposed openstack/nova: Add Allocation and AllocationList objects  https://review.openstack.org/28244213:15
openstackgerritChris Dent proposed openstack/nova: Add delete_, update_ and add_ inventory to ResourceProvider  https://review.openstack.org/32827613:15
openstackgerritChris Dent proposed openstack/nova: Add support for resource inventories to placement API  https://review.openstack.org/32915213:15
*** pacoblox has quit IRC13:15
openstackgerritChris Dent proposed openstack/nova: Add a ResourceProviderList object  https://review.openstack.org/32830713:15
openstackgerritChris Dent proposed openstack/nova: Add set_inventory() method on ResourceProvider  https://review.openstack.org/32644013:15
openstackgerritChris Dent proposed openstack/nova: Add initial framing of placement API  https://review.openstack.org/32914913:15
openstackgerritChris Dent proposed openstack/nova: WIP Add a Usage and UsageList object  https://review.openstack.org/32957513:15
openstackgerritChris Dent proposed openstack/nova: Add support for resource_providers urls  https://review.openstack.org/32915113:15
openstackgerritChris Dent proposed openstack/nova: Add support for usages in the placement API  https://review.openstack.org/32964313:15
openstackgerritChris Dent proposed openstack/nova: Add microversion support for the placement API  https://review.openstack.org/32915013:15
openstackgerritChris Dent proposed openstack/nova: WIP experiment with the remainder of the placement API  https://review.openstack.org/29310413:15
*** eharney has joined #openstack-nova13:17
*** pwnall has joined #openstack-nova13:19
*** mriedem has joined #openstack-nova13:19
*** ivase has joined #openstack-nova13:20
*** EinstCrazy has quit IRC13:20
*** salv-orlando has quit IRC13:21
openstackgerritJulian proposed openstack/nova: Transform instance.snapshot notifications  https://review.openstack.org/33012813:21
*** salv-orlando has joined #openstack-nova13:21
*** EinstCrazy has joined #openstack-nova13:21
openstackgerritJulian proposed openstack/nova: Transform instance.snapshot notifications  https://review.openstack.org/33012813:22
*** bharath has joined #openstack-nova13:22
*** avolkov has joined #openstack-nova13:23
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907713:24
*** sree has quit IRC13:24
*** tbachman_ has joined #openstack-nova13:25
*** sree has joined #openstack-nova13:25
*** EinstCrazy has quit IRC13:26
*** akshai has joined #openstack-nova13:26
*** rfolco has joined #openstack-nova13:26
mriedemwow, last week we had 86 approved blueprints, this week we have 9913:26
mriedema lot more specs approved than i thought13:26
mriedemso i guess we're averaging about ~100 approved blueprints per release now13:27
*** akshai_ has joined #openstack-nova13:27
*** raildo-a` is now known as raildo13:27
rlrossitmriedem: how many get completed per release on average?13:27
*** tbachman has quit IRC13:28
*** tbachman_ is now known as tbachman13:28
dimsmriedem : claudiub : Question on hyperv cluster driver - is that on the table to be added to Nova? atuvenie was implying that work is in progress yesterday13:28
mriedemrlrossit: 60 implemented in mitaka, or partially implemented13:29
mriedem61 in liberty13:29
*** diogogmt has quit IRC13:29
*** sree has quit IRC13:30
dimsmriedem : claudiub : ah looks like there's a blueprint approved for Newton by johnthetubaguy - https://blueprints.launchpad.net/nova/+spec/hyper-v-cluster13:30
rlrossitso that's not *too* far off to approve 100 of them... granted that could get a little closer to 60, like 75 or so13:30
gibi_mriedem: hi! It is a friendly reminder that I answered your comment in https://review.openstack.org/#/c/310700/ Transfrom wrap_exception notification to versioned format13:31
*** akshai has quit IRC13:31
*** diogogmt has joined #openstack-nova13:31
*** rlrossit has quit IRC13:32
mriedemdims: was there a question?13:32
mriedemdims: i'm interested in your recent interest in hyper-v13:32
dimsmriedem : are we ok with the concept now? i remember we had a lot of pushback on the vmware driver13:32
*** tbachman_ has joined #openstack-nova13:33
dimsmriedem : i sat on a 1 hour demo yesterday :)13:33
dimsthat's all :)13:33
mriedemat cloudbase?13:33
dimscloudbase folks were are a mirantis event13:33
mriedemah13:33
mriedemi haven't read the spec or code on the hyper-v cluster bp13:34
mriedemgibi_: looking13:34
dimsack. thanks mriedem13:34
dansmithour mission statement says no more hiding of large systems behind a single driver13:34
openstackgerritEyal Posener proposed openstack/nova: Fix nova-compute service start order  https://review.openstack.org/33055613:34
dimsdansmith : right13:34
openstackgerritLudovic Beliveau proposed openstack/nova: Fix revert on migration with SR-IOV  https://review.openstack.org/32617413:35
openstackgerritLudovic Beliveau proposed openstack/nova: Allocate PCI devices on migration  https://review.openstack.org/32898313:35
openstackgerritLudovic Beliveau proposed openstack/nova: Update binding:profile for SR-IOV ports  https://review.openstack.org/24257313:35
*** tbachman has quit IRC13:35
*** tbachman_ is now known as tbachman13:35
mriedemso how did the hyper-v cluster spec get approved in mitaka?13:35
claudiubdims: indeed, it still is13:35
*** mtanino has joined #openstack-nova13:35
*** hoangcx has quit IRC13:35
dimshey claudiub : the demo was great btw. the reference to cluster driver was during Q&A13:36
dansmithmriedem: what?13:36
claudiubwell, that's the first news I hear from that. I'm glad. :)13:36
dimsdansmith : https://blueprints.launchpad.net/nova/+spec/hyper-v-cluster13:37
dimsclaudiub : was there a spec too?13:37
mriedemhttps://review.openstack.org/#/c/279043/13:37
mriedempreviously approved in liberty and mitaka13:37
claudiubyah, that's the spec13:37
mriedemhttp://specs.openstack.org/openstack/nova-specs/specs/newton/approved/hyper-v-cluster.html13:37
claudiubdims: so, we are doing things a bit differently from vmware. vmware was doing a single-node cluster. meaning that one single host represented the cluster. with this, came a lot of scheduling / resource allocation issues.13:38
claudiubwhile we register every node in the cluster as a nova-compute node.13:38
dimsclaudiub : ah. i see13:39
mdboothdansmith: Found a yak to shave for persistent instance storage while writing http://lists.openstack.org/pipermail/openstack-dev/2016-June/097529.html : the driver has no reference to the root disk's BDM object (unless it's a volume, in which case we don't care anyway).13:39
dimsclaudiub : Option A or B? "Hyper-V Cluster controlled by a single nova-compute service" or "nova-compute service on each Hyper-V Cluster Node."13:39
dimswhich one did you pick for impl?13:40
mriedemThis blueprint will address Option B, as its value far outweighs Option A.13:40
claudiubyep13:40
dansmithhow does option b differ from normal then?13:40
dimsclaudiub mriedem : perfect. thanks!13:40
dimsdansmith : option B is similar to what we have in vmware driver i believe where we need one nova-compute per cluster13:41
*** rmart04 has joined #openstack-nova13:41
gibi_mriedem: thanks. I will try to improve the doc generation as a follow up13:42
dansmithso how does that address the problem of trying to schedule an instance that wants 4 cpus to a cluster with two systems that have only two to give?13:42
claudiubdansmith: option B says that it is going to register all cluster nodes in nova. so you'll have 2 compute nodes with 2 cpus each, as per your example.13:43
dimsclaudiub : ^^ that one's yours :)13:43
*** hoangcx has joined #openstack-nova13:43
dansmithclaudiub: meaning using a single compute host with multiple compute node entries, right?13:43
*** ducttape_ has joined #openstack-nova13:43
claudiubdansmith: plus, IMO, even in a vmware case, I believe jaypipes recent efforts with the nova inventory can solve that issue.13:44
dansmithi.e. multiple resource trackers, one for each compute node13:44
claudiubdansmith: no. one compute node with just itself.13:44
dansmithclaudiub: so how does that differ from just a bunch of regular hypervisor nodes?13:45
mriedemclaudiub: so how is that different from how you deploy hyper-v today?13:45
mriedem:)13:45
*** sameo has joined #openstack-nova13:45
openstackgerritTakashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/test_metadata.py  https://review.openstack.org/32437313:46
*** ecelik has quit IRC13:46
claudiubit differs with the fact that some operations like migration / live migration are a lot faster in a cluster. plus, the instances are marked as highly-available, even if one of the compute nodes goes down.13:46
claudiubor more13:46
dansmithyeah that last one is a problem.. I don't understand how nova can handle that,13:47
openstackgerritMaciej Szankin proposed openstack/nova: WIP Transform instance.reboot notifications  https://review.openstack.org/33022413:47
dansmithunless you do something nefarious from down in the driver13:47
mriedemi can already see the chart with a long rectangle at the bottom spanning across multiple boxes on top of it...13:47
*** pacoblox has joined #openstack-nova13:47
*** mmandel has joined #openstack-nova13:47
*** zenoway has quit IRC13:49
claudiubdansmith: well, when an instance failovers, the new host's nova-compute will pick it up and give the conductor a heads-up.13:49
openstackgerritLaszlo Hegedus proposed openstack/nova: Replace root volume during rebuild  https://review.openstack.org/30507913:49
dansmithclaudiub: right, that is what seems problematic to me13:49
mriedemand then the periodic power state check task will stop the vm13:49
openstackgerritTakashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/test_metadata.py  https://review.openstack.org/32437313:49
claudiubmriedem: why would it stop the vm?13:50
dansmithclaudiub: anyway, having read through the spec I'm not as opposed to this as what I thought was the goal (option A) but I still think that many issues (like this hot-move thing) are likely to be uncovered13:50
mriedemclaudiub: vmware ran into an issue with HAing behind nova, the nova db has the instance in a given state but the hypervisor says it's something else, and it stops the vm13:50
*** tbachman_ has joined #openstack-nova13:50
mriedembecause it's happening outside of nova-api13:50
dansmithwish we could re-review that spec, cut out option A and discuss more detail on option B13:50
dansmithmriedem: ironic hits the same issue13:51
*** MikeG451_ has joined #openstack-nova13:51
claudiubmriedem: honestly, I haven't seen this issue, since I was working on it. typically, a failover can happen very fast.13:51
*** ducttape_ has quit IRC13:52
dansmith"strategy: always win the race"13:52
dansmithgot it :)13:52
claudiubdansmith: can you detail some of those concerns of yours? :)13:53
*** tbachman has quit IRC13:53
*** tbachman_ is now known as tbachman13:53
*** zenoway has joined #openstack-nova13:53
*** ctrath has joined #openstack-nova13:53
*** mrkz has joined #openstack-nova13:54
mriedemclaudiub: the issue i was talking about has some history here https://review.openstack.org/#/c/218975/13:54
dansmithclaudiub: mriedem's state thing is one, you also just can't re-host an instance, you'll have to do all the math to unallocate it from the source, allocate resources on the destination, rebind the ports in neutron, etc13:54
dansmithdoing all of that from inside your driver is not cool13:54
mriedemat the time alaski_ and i had some ideas of how to make that better, and i was going to write  something up but it got lost and i'd have to refigure it out13:55
mriedemnova meeting in #openstack-meeting in 4 minutes13:56
dansmithmdbooth: lets resync after the meeting, sorry13:56
mdboothdansmith: Not urgent. Probably punt it to next week, tbh.13:56
mriedemmdbooth: thanks for that writeup btw, i've started it but will have to finish it later13:58
claudiubdansmith: yeah, you're right about the nova resource claims, that's something that escaped my thoughts. will look into that. as for rebinding the ports in neutron: it already works. the ports are already configured properly, so the VMs networks are already fine. as for the neutron ports, neutron-hyperv-agent will see new ports on the host and will then bind them.13:58
mriedemwould be good to capture some of that in the docs at some point maybe - we have a docs page on BDMs but probably not at this low-level detail - the docs we have are more about the API view13:58
claudiubso, no new changes to how neutron-hyperv-agent  and how it works.13:58
*** kfarr has joined #openstack-nova13:58
dansmithmdbooth: okay13:59
*** gagehugo has joined #openstack-nova13:59
claudiubdansmith: mriedem: as for the state thingy. it sounds clear to me: if the VMs state in DB says it's turned-off, the VM should be turned off, what exactly is the issue?13:59
paul-carlton2mriedem, looking at live migration of rescued instances, seems to me that there is an existing bug, if you specify a rescue image that has kernel and ramdisk it ignores it and uses the instance's kernel and ram disk!14:00
*** jaugustine has joined #openstack-nova14:00
mriedemadrian_otto akuriata alevine alexpilotti aloga andreykurilin anteaya artom auggy14:00
mriedembauzas belliott belmoreira bobball cburgess claudiub danpb dguitarbite _diana_14:00
mriedemdiana_clarke dims duncant edleafe efried flip214 funzo garyk gcb gjayavelu14:00
mriedemirina_pov jaypipes jcookekhugen jgrimm jichen jlvillal jroll kashyap klindgren14:00
mriedemkrtaylor lbeliveau lxsli macsz markus_z mdorman med_ mikal mjturek mnestratov14:00
*** jaugustine has quit IRC14:00
mriedemmoshele mrda nagyz ndipanov neiljerram nic Nisha PaulMurray raildo rgeragnov14:00
mriedemsc68cal scottda sdague sileht sorrison swamireddy thomasem thorst tjones tonyb14:00
mriedemtpatil tpatzig xyang rdopiera raj_singh woodster sahid rbradfor junjie gsilvis14:00
mriedemmtanino14:00
mriedemnova meeting heads up14:00
*** rama_y has joined #openstack-nova14:01
*** diogogmt has quit IRC14:01
*** jaugustine has joined #openstack-nova14:01
* auggy opens a bleary uncaffeinated eye14:01
paul-carlton2should I ignore this for now and raise a bug to fix later?14:01
*** rlrossit_ has joined #openstack-nova14:01
kashyapmriedem: Which IRC channel is this?14:01
*** _diana_ is now known as diana_clarke14:01
kashyap#openstack-meeting I suppose.  /me always forgets14:01
*** woodster_ has joined #openstack-nova14:01
bauzasyup14:01
*** cleong has joined #openstack-nova14:01
mriedempaul-carlton2: seems the existing bug should be fixed? idk14:02
mriedemmeeting14:02
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907714:02
paul-carlton2catch you after meeting mriedem14:02
*** sridharg has quit IRC14:03
*** stewie925 has joined #openstack-nova14:04
*** sheel has quit IRC14:05
*** roeyc has quit IRC14:06
*** annegentle has quit IRC14:07
*** sacharya has joined #openstack-nova14:08
*** zenoway has quit IRC14:08
*** zenoway has joined #openstack-nova14:08
bzurawskiEmilienM: Hey, are you around ?14:10
EmilienMbzurawski: what's up14:11
bzurawskiwhen you have a little time, can you look at this: https://review.openstack.org/#/c/329991/ i've done what you said in comment14:12
bzurawskiEmilienM: and btw. have you notice problems with gate-jobs for puppet-nova, or it only me ?14:12
openstackgerritLuis Tomas proposed openstack/nova: Add automatic switching to postcopy mode when migration is not progressing  https://review.openstack.org/32849914:13
openstackgerritLuis Tomas proposed openstack/nova: Extend live-migration-force-complete to use postcopy if available  https://review.openstack.org/32907714:13
*** smatzek has quit IRC14:14
EmilienMbzurawski: if you join #puppet-openstack, you'll notice our CI is having issues14:15
EmilienMwe're working on it14:15
EmilienMbzurawski: yeah I'll look your patch, thanks!14:15
*** jaugustine has quit IRC14:16
*** ri0 has joined #openstack-nova14:18
*** sacharya_ has joined #openstack-nova14:18
*** zz_dimtruck is now known as dimtruck14:18
bzurawskiEmilienM: oki, i will join :)14:19
bzurawskiThanks14:19
*** daemontool has joined #openstack-nova14:19
openstackgerritSilvan Kaiser proposed openstack/nova: Switch Nova Quobyte volume driver to rely on external client mount.  https://review.openstack.org/26475214:21
*** sacharya has quit IRC14:21
*** kaisers_ has joined #openstack-nova14:21
*** bharath has quit IRC14:21
*** burgerk has joined #openstack-nova14:21
*** Kevin-Zhao has quit IRC14:22
*** lucas___ has joined #openstack-nova14:23
*** zenoway has quit IRC14:26
*** tblakeslee has joined #openstack-nova14:26
*** ctrath has quit IRC14:26
openstackgerritMaciej Szankin proposed openstack/nova: WIP Transform instance.reboot notifications  https://review.openstack.org/33022414:26
*** zenoway has joined #openstack-nova14:26
*** jistr is now known as jistr|mtg14:28
*** ctrath has joined #openstack-nova14:28
*** ri0 has quit IRC14:29
*** amotoki has joined #openstack-nova14:29
*** sheel has joined #openstack-nova14:30
*** zenoway has quit IRC14:31
bzurawskimdbooth: hey are you around ?14:31
mdboothbzurawski: Yep.14:32
*** ssurana has joined #openstack-nova14:32
bzurawskii have a question about my review: https://review.openstack.org/#/c/314928/14:32
*** sacharya_ has quit IRC14:32
*** pacoblox has quit IRC14:32
*** sacharya has joined #openstack-nova14:33
*** pacoblox has joined #openstack-nova14:33
mdboothbzurawski: Just reading melwitt's comment14:33
mdboothbzurawski: How can I help?14:34
bzurawskibecuase melwitt said something about unit tests, so i just want to say, am i right thinking that in this test where i faked cleanup_volume14:34
bzurawskimdbooth: i should faked  _destroy_volume from RBDDriver, and check with which arguments it was called ?14:35
*** artom has quit IRC14:35
mdboothmelwitt has considerably more experience in this area, you should take her advice over mine14:35
*** rama_y has quit IRC14:35
bzurawskimdbooth: oki, I'm gonna ask her :)14:35
bzurawskiand one more thing, this lamdas14:35
bzurawskilambdas*14:35
mdboothBut I think she's described it fairly clearly in the comment?14:36
*** danpb has quit IRC14:36
bzurawskimdbooth: yep, i've read it one more time, and now I'm get it :)14:37
bzurawskisorry for bothering ;(14:37
mdboothnp14:37
openstackgerritJulian proposed openstack/nova: WIP Transform instance.snapshot notifications  https://review.openstack.org/33012814:37
*** jaugustine has joined #openstack-nova14:37
openstackgerritEyal Posener proposed openstack/nova: Fix nova-compute service start order  https://review.openstack.org/33055614:39
*** artom has joined #openstack-nova14:40
*** efried has joined #openstack-nova14:41
gibi_mriedem, rlrossit_: I will be around for the next hour or so if we want to talk about the wrap_exception, after that I have to be somewhere else14:41
*** pcaruana has quit IRC14:41
rlrossit_gibi_: I'll probably just fill him in this afternoon when I'm sitting across a table from him14:42
gibi_rlrossit_: ahh that is too easy :)14:42
gibi_rlrossit_: thanks for doing it14:43
rlrossit_no prob14:43
*** Kevin-Zhao has joined #openstack-nova14:44
*** kaisers_ has quit IRC14:44
*** ssurana has quit IRC14:45
*** kmartin has joined #openstack-nova14:45
*** ssurana has joined #openstack-nova14:46
*** kaisers_ has joined #openstack-nova14:48
*** tyrola has joined #openstack-nova14:49
*** jaosorior has quit IRC14:49
*** mmandel has quit IRC14:50
*** mriedem1 has joined #openstack-nova14:50
*** mriedem has quit IRC14:51
*** kylek3h has joined #openstack-nova14:52
*** READ10 has joined #openstack-nova14:52
*** pacoblox has quit IRC14:52
*** jlanoux_ has quit IRC14:53
*** efried has quit IRC14:53
*** sdake has quit IRC14:54
*** efried has joined #openstack-nova14:54
*** diogogmt has joined #openstack-nova14:55
*** tbachman has quit IRC14:56
*** salv-orlando has quit IRC14:56
*** amotoki_ has joined #openstack-nova14:57
*** salv-orlando has joined #openstack-nova14:57
*** mriedem1 is now known as mriedem14:57
*** EinstCrazy has joined #openstack-nova14:58
*** Kevin-Zhao has quit IRC14:59
*** chandankumar has quit IRC14:59
*** sdake has joined #openstack-nova14:59
*** salv-orlando has quit IRC15:00
*** amotoki has quit IRC15:00
*** salv-orlando has joined #openstack-nova15:00
*** david-lyle_ is now known as david-lyle15:00
*** tblakeslee has quit IRC15:01
*** pushkaru has joined #openstack-nova15:03
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for compute db_opts  https://review.openstack.org/32671815:03
*** tblakeslee has joined #openstack-nova15:04
paul-carlton2mriedem, your meeting done?15:05
*** rrecio has joined #openstack-nova15:05
mriedempaul-carlton2: in meetings for another hour15:06
paul-carlton2ok15:06
*** amotoki_ has quit IRC15:06
*** kaisers_ has quit IRC15:09
*** kaisers_ has joined #openstack-nova15:10
*** adisky has quit IRC15:10
*** rrecio_ has joined #openstack-nova15:12
*** rrecio has quit IRC15:12
*** mmandel has joined #openstack-nova15:12
*** pacoblox has joined #openstack-nova15:14
*** diga has joined #openstack-nova15:16
*** jistr|mtg is now known as jistr15:19
kashyapmdbooth: Nice write-up (on -devel list)!  I had a probably small (and silly?) question (wrote to the list).15:22
mriedempaul-carlton2: is your question just whether or not using the instance kernel/ramdisk over the image kernel/ramdisk is a bug?15:23
mriedemand is this in the compute manager or the api or the libvirt driver?15:23
openstackgerritWaldemar Znoinski proposed openstack/nova: dummy change, please ignore  https://review.openstack.org/33064015:25
*** Dinesh_ has joined #openstack-nova15:25
*** ihrachys has quit IRC15:25
*** ihrachys_ has joined #openstack-nova15:25
*** jlanoux has joined #openstack-nova15:25
*** Dinesh_ has quit IRC15:26
*** pacoblox has quit IRC15:26
paul-carlton2mriedem, nop, when you rescue and specify an image to rescue from it sets the image ref to boot from when rescued correctly but just uses the kernel and ramdisk of the instance's original boot image15:26
*** Swami has joined #openstack-nova15:27
paul-carlton2this leads to a failure to boot when you are rescuing an instance with an image that has no kernel and ramdisk using a rescue image that has them15:27
openstackgerritMikhail Feoktistov proposed openstack/nova: libvirt: support user password settings in virtuozzo  https://review.openstack.org/33064315:27
*** sacharya_ has joined #openstack-nova15:28
mriedempaul-carlton2: ok, sounds like a bug then15:28
paul-carlton2mriedem, obviously a bug, I was thinking I'd raise a bug and fix it but don't want my live migration of rescused inastances to be dependant on it15:28
paul-carlton2so will fix if afterwards15:29
*** Swami has quit IRC15:29
*** andreas_s has quit IRC15:29
mriedempaul-carlton2: it doesn't sound like a blocker for your bp so it sounds reasonable to fix it later15:29
paul-carlton2I have a an other question, if you have a mo mriedem15:29
*** sacharya has quit IRC15:30
*** sacharya has joined #openstack-nova15:30
cfriesen_Is it possible for nova-scheduler to have multiple _schedule() operations going on in parallel in a single scheduler process?15:30
paul-carlton2if the instance is rescued using a different image then live migration will fail for libvirt (probably xenAPI too) because the destination side of the migration will not have the image underlay15:31
*** pacoblox has joined #openstack-nova15:31
paul-carlton2mriedem, however the reescue operation just uses the rescue image ref to boot the instance form the new image but does not save it anywhere so live migration has no way to find out what it is and download hte image base15:32
*** edand has quit IRC15:32
mdboothkashyap: Thanks for reading :)15:33
*** sacharya_ has quit IRC15:33
paul-carlton2my thinking is to have rescue store the rescue image id in instance system metadata and retrieve it from there in live migration (if present)15:33
*** openstackgerrit has quit IRC15:34
paul-carlton2mriedem, doe this sound reasonable to you?15:34
*** openstackgerrit has joined #openstack-nova15:34
kashyapmdbooth: Your references and "Reader beaware"es made them easy :-)15:34
*** pacoblox has quit IRC15:35
openstackgerritKaren Bradshaw proposed openstack/nova: api-ref, os-availability-zone.inc  https://review.openstack.org/32184715:35
*** tbachman has joined #openstack-nova15:35
kashyapmdbooth: Speaking of which, your comment on volume keys: "volumes are keyed by device name, eg 'vda', 'vdb'" reminded me of this "fun" bug that I discovered earlier this week15:35
*** pacoblox has joined #openstack-nova15:35
kashyapmdbooth: Where when you hot plug a disk to libvirt, you can give it a partition name ('vdb1', instead of 'vdb')15:36
*** artom has quit IRC15:36
kashyapAnd, libvirt will generate a same QEMU disk alias "drive-virtio-disk1" for both 'vdb' and 'vdb1'.  Reproducer here: https://bugzilla.redhat.com/show_bug.cgi?id=134626515:36
openstackbugzilla.redhat.com bug 1346265 in libvirt "Disallow disk hot-plugging with invalid target labels (e.g. disk partition number for non-paravirt disks)" [Unspecified,Assigned] - Assigned to pkrempa15:36
mdboothkashyap: What does that even mean?15:37
kashyap:-)15:37
mdboothCan qemu generate partition tables on the fly?15:37
*** jaugustine has quit IRC15:37
kashyapmdbooth: Look at the reproducer, it's much easier to see there.15:37
*** artom has joined #openstack-nova15:38
*** eharney_ has joined #openstack-nova15:38
mdboothkashyap: Ah... user-supplied data?15:38
kashyapYeah15:38
kashyapTarget devices15:38
openstackgerritVladik Romanovsky proposed openstack/nova: DO NOT MERGE - attach configdrive after instance XML test  https://review.openstack.org/33013415:38
kashyapI called them "lablels".15:38
mdboothkashyap: Fun task for you: put a black hat on and go looking for exploits15:38
kashyap"Target" here (for a disk): <target dev='vdb' bus='virtio'/>15:38
*** rmart04 has quit IRC15:38
*** eharney has quit IRC15:39
mdboothWhat can I do if I can pass user-supplied data right down to qemu like that?15:39
kashyapmdbooth: Yesterday I crashed QEMU with a simple block-device add!15:39
kashyapEasy reproducer:15:39
kashyap{'execute':'blockdev-add','arguments':{'options':{'driver':'raw'}}}15:39
kashyapComplete detail here: https://lists.gnu.org/archive/html/qemu-devel/2016-06/msg04163.html~15:39
kashyapAnd, Eric Blake has already fixed it while I was asleep!15:40
cfriesen_kashyap: is it possible to hit this with nova commands?15:40
kashyapmdbooth: No, as near as I know.  Let me think15:40
kashyapErr, the above is for cfriesen_ ^15:40
*** Kevin-Zhao has joined #openstack-nova15:40
kashyapmdbooth: Libvirt will convert that target device 'vdb' into something that QEMU can understand.  (Your comment sounds like QEMU is using it as-is)15:41
mdboothkashyap: Can you specify a device name to the bdm when booting a disk?15:41
mdbooths/disk/instance/15:41
*** diga has quit IRC15:41
mdboothI think that's now bdmv1 worked, no?15:41
openstackgerritKaren Bradshaw proposed openstack/nova: api-ref, os-availability-zone.inc  https://review.openstack.org/32184715:42
kashyapmdbooth: In past life, I was part of a white-hat vulnerability assessment team (spent only a year there) before moving to Red Hat :_)15:42
mdboothPerhaps it's already doing good validation, I'm just speculating btw15:42
kashyapmdbooth: Hmm, device name and BDM...15:42
*** pacoblox has quit IRC15:42
*** pacoblox has joined #openstack-nova15:43
kashyapcfriesen_: It's not possible, unless a user can trigger libvirt's `qemu-monitor-command` through which one can trigger arbitrary QEMU monitor commands (one the above), _and_ have access to libvirt socket.15:43
mdboothkashyap: I was confused by above, btw, thought that was a nova bug :) However, you've got me wondering about device_name anyway.15:44
*** eharney_ is now known as eharney15:44
kashyapmdbooth: I do not know the answer to your question without testing.  I think you'd deduce it from the code faster than me, given your essay.15:44
*** sacharya has quit IRC15:44
kashyapmdbooth: Yeah, I should've phrased it better.  It's only libvirt target devices.  Not yet at Nova level.15:44
*** sacharya has joined #openstack-nova15:45
*** pacoblox has quit IRC15:46
Kevin-ZhaoCould I ask a question about nova volume-attach?15:46
Kevin-Zhaomdbooth: when running volume-attach, I find the device is "virtio"15:47
Kevin-Zhaomdbooth: But if I want to add a "SCSI" disk ,how can I do?15:48
mdboothKevin-Zhao: I'm not the best person to ask about that, I'm afraid. Couldn't answer you off the top of my head without digging out the code.15:48
*** mdrabe has quit IRC15:49
Kevin-Zhao mdbooth: Ok,thanks all the same15:50
Kevin-ZhaoWhen run volume-attach, will attach the device:  <driver name="qemu" type="raw" cache="none" io="native"/>15:51
Kevin-Zhao  <source dev="/dev/disk/by-path/ip-10.164.2.160:3260-iscsi-iqn.2010-10.org.openstack:volume-f467c851-da96-4a07-87e0-96c8f9d9eb54-lun-1"/>15:51
Kevin-Zhao  <target bus="virtio" dev="vdb"/>15:51
Kevin-Zhao  <serial>f467c851-da96-4a07-87e0-96c8f9d9eb54</serial>15:51
Kevin-Zhao</disk>15:51
Kevin-ZhaoBut I want a  <target bus="scsi" dev="sdb"/>    instead...Because of on Aarch64, only support scsi hot-plug15:52
kashyapKevin-Zhao: mdbooth: I think these image meta properties should gives a SCSI bug (incidentally I'm trying to test it):15:52
*** romainh1 has joined #openstack-nova15:53
kashyap'hw_scsi_model=virtio-scsi'  -- which exposes the SCSI controller15:53
kashyapAnd: 'hw_disk_bus=scsi' -- which exposes the type of disk controller15:53
*** EinstCrazy has quit IRC15:53
kashyapKevin-Zhao: ^ The above should get you what you need.15:53
mriedemBobBall: anthonyper: have you seen this https://bugs.launchpad.net/nova/+bug/1591840 ?15:53
openstackLaunchpad bug 1591840 in OpenStack Compute (nova) "tempest test of test_aggregates.py is broken for XenServer" [Undecided,New]15:53
*** pacoblox has joined #openstack-nova15:54
BobBallmriedem: already fixed15:54
*** rahuls has joined #openstack-nova15:54
BobBallmriedem: but the bug was filed against nova not tempest so it didn't get updated - 1 sec.15:54
mriedempaul-carlton2: i'm not familiar with the details of what's failing there, but system_metadata is an option to hack around this15:55
*** romainh has quit IRC15:55
mriedemBobBall: cool, thanks15:55
BobBallmriedem: https://review.openstack.org/#/c/328836/ was the fix, FYI.  Adding comments + closing bug (if I can)15:55
*** ctrath has quit IRC15:55
kashyapKevin-Zhao: So, setting these properties on a Glance image should get you what you need: `glance image-update --property hw_disk_bus=scsi --property hw_scsi_model=virtio-scsi fedora24-openstack-sda`15:55
*** belmoreira has joined #openstack-nova15:56
*** romainh has joined #openstack-nova15:56
*** sacharya_ has joined #openstack-nova15:57
*** romainh1 has quit IRC15:58
Kevin-Zhaokashyap: Yeah, I have add the --property hw_scsi_model=virtio-scsi ,but not add "hw_disk_bus=scsi"15:58
*** akshai has joined #openstack-nova15:58
*** Apoorva has joined #openstack-nova15:58
*** mpavone has quit IRC15:59
Kevin-Zhaokashyap: I will update the image and test again, the default bus will change to scsi with "hw_disk_bus=scsi",right?15:59
kashyapYes, it should definitely.16:00
*** sacharya has quit IRC16:00
*** smatzek has joined #openstack-nova16:00
Kevin-Zhaokashyap: Good~16:00
*** tesseract has quit IRC16:00
*** lucas___ has quit IRC16:00
*** mriedem1 has joined #openstack-nova16:00
*** kaisers_ has quit IRC16:01
*** akshai_ has quit IRC16:01
openstackgerritMaciej Szankin proposed openstack/nova: Consider deprecating isolated_images  https://review.openstack.org/27135416:01
*** lucas___ has joined #openstack-nova16:01
*** ctrath has joined #openstack-nova16:01
*** mriedem has quit IRC16:02
cfriesen_mriedem: do you know if it's possible for nova-scheduler to have multiple _schedule() operations going on at once?  Or does it pull an RPC request off the queue, handle it, send a reponse back, and pull the next request?16:02
cfriesen_I'm not totally up on the guts of how eventlet and the RPC code interact.16:02
mriedem1edmondsw: are you going to put up a patch for https://bugs.launchpad.net/nova/+bug/1593295 ?16:03
openstackLaunchpad bug 1593295 in OpenStack Compute (nova) "incorrect nova api documentation for revert resize" [Undecided,New]16:03
*** mriedem1 is now known as mriedem16:03
mriedemedmondsw: i'm not sure what you mean by "since I hope you can revert a resize that never actually made it to  status RESIZED, in which case there should be more to it than that."16:06
*** lucasagomes is now known as lucas-afk16:06
*** abalutoiu has quit IRC16:06
*** lucas___ has quit IRC16:06
*** BjoernT has joined #openstack-nova16:06
mriedemedmondsw: i think what it's saying is if the vm_status stays 'RESIZED' after you try to revert, i.e. it doesn't go to active or shutdown, then then the revert failed16:06
*** tyrola has quit IRC16:06
edmondswmriedem, if it sits long enough for me to get more familiar with resize... I was just starting to look at how that works, so I'm not sure I'm familiar enough to fix it today16:07
*** sacharya has joined #openstack-nova16:07
*** Kevin-Zhao has quit IRC16:07
mriedemthere is obviously a typo in "You can only confirm the resized server where the status is VERIFY_RESIZE and the vm_status is RESIZED." - that should be s/confirm/revert/16:08
*** kolesovdv has quit IRC16:08
edmondswmriedem: if it was in the RESIZED state before you try to revert, sure... but what if it was in another state? Is RESIZED the only state that's valid precondition for revert?16:08
mriedemedmondsw: yes https://github.com/openstack/nova/blob/master/nova/compute/api.py#L244916:09
mriedemyou can't revert the resize on a pending resize or something, that would be bonker city16:09
edmondswok then sure, maybe it's just as simple as s/confirm/revert/... if so, I can try to throw up a patch for that shortly16:09
*** ansiwen_ is now known as ansiwen16:10
mriedemi'll run a quick test and make sure the status values in there are all correct16:10
*** sacharya_ has quit IRC16:10
*** akshai_ has joined #openstack-nova16:10
edmondswmriedem, tx16:10
*** mdrabe has joined #openstack-nova16:11
*** mgoddard_ has quit IRC16:11
*** akshai has quit IRC16:13
mriedem"and a migration_status of reverted" is also busted16:13
mriedemshould be task_state of 'resize_reverting'16:14
*** burt has joined #openstack-nova16:14
*** permalac has quit IRC16:15
mriedemedmondsw: i'll push a change quick16:15
edmondswmriedem, I just did16:16
mriedemlink?16:16
edmondswwell, it's being slow...16:16
openstackgerritMatthew Edmonds proposed openstack/nova: fix typo in revert resize api docs  https://review.openstack.org/33067216:17
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 1)  https://review.openstack.org/32885016:17
*** paul-carlton has joined #openstack-nova16:17
edmondswI missed your second change16:17
edmondswmriedem I can add that real quick16:17
*** paul-carlton has quit IRC16:17
*** mgoddard has joined #openstack-nova16:18
edmondswmriedem, "A successfully reverting resize operation shows a status of ACTIVE or SHUTOFF and a task_state of resize_reverting."16:20
edmondswshould I add something about what it will look like AFTER it's completed?16:20
*** sdake has quit IRC16:20
edmondswmust be some way to know when it's done... "On completion, it will show a migration_status of reverted" ?16:21
*** paul-carlton2 has quit IRC16:21
mriedemno, on completion it's ACTIVE or SHUTOFF16:21
*** paul-carlton2 has joined #openstack-nova16:21
edmondswwhich the current text is shows while it's reverting...16:22
*** tblakeslee has quit IRC16:22
mriedemyeah, that's wrong16:22
mriedemcomments inline16:22
*** sfinucan has quit IRC16:23
mriedemedmondsw: http://paste.openstack.org/show/516690/16:23
*** tblakeslee has joined #openstack-nova16:23
mriedemyou could add an 'Asynchronous Postconditions' section16:23
paul-carlton2mriedem, Thanks, I'll code it thus and let others review, I'll need to update the spec too, this was an oversight on my part16:24
mriedemsaying if it's successful the status is ACTIVE or SHUTOFF, depending on the server status before the resize was started16:24
mriedemedmondsw: you're well on your way to becoming an api-ref cleanup hero! http://burndown.dague.org/16:27
*** moshele has quit IRC16:27
*** sdake has joined #openstack-nova16:28
*** sahid has quit IRC16:29
openstackgerritMatthew Edmonds proposed openstack/nova: fix errors in revert resize api docs  https://review.openstack.org/33067216:30
*** matrohon has quit IRC16:33
*** lucas___ has joined #openstack-nova16:33
*** bharath has joined #openstack-nova16:34
*** david-lyle has quit IRC16:35
edmondswmriedem sdague, that link is a wonderful thing to see! awesome!16:36
edmondswmriedem, I had already made my changes mentioning ACTIVE or SHUTOFF before I saw your comment... should I leave them or split out into a separate section as you suggested?16:37
*** tblakeslee has quit IRC16:37
*** paul-carlton2 has quit IRC16:38
*** lucas___ has quit IRC16:38
*** kfarr has quit IRC16:38
mriedemedmondsw: i'd probably put them in a postcondition section16:38
*** yamahata has joined #openstack-nova16:39
mriedemthere are a couple of other server actions that have asynchronous postcondition sections16:39
mriedemwell, sec, let me read this first16:39
*** jlanoux has quit IRC16:39
*** lucas___ has joined #openstack-nova16:41
openstackgerritMatt Riedemann proposed openstack/nova: fix errors in revert resize api docs  https://review.openstack.org/33067216:41
edmondswmriedem, could just move that whole paragraph to an asynchronous postconditions section16:41
mriedemedmondsw: just leave it, this looks fine16:41
mriedemi updated one thing16:42
mriedem+216:42
mriedemthanks16:42
edmondswty16:42
*** dmk0202 has quit IRC16:42
*** lucas___ has quit IRC16:44
*** lucas___ has joined #openstack-nova16:44
*** erlon has joined #openstack-nova16:44
edmondswmriedem, actually, I had a comment on your change16:44
*** efoley has joined #openstack-nova16:44
* auggy dusts herself off after cleaning up some stale bugs16:44
mriedemedmondsw: replied16:45
mriedemedmondsw: it's ambiguous16:45
mriedemso i assumed OS-EXT-STS:vm_state16:45
*** diga has joined #openstack-nova16:45
mriedemedmondsw: i'd be fine with VERIFY_RESIZE also16:46
mriedemmaybe that's better, feel free to update - i'll +2 again16:46
edmondswupdating16:48
*** romainh has quit IRC16:48
openstackgerritMatthew Edmonds proposed openstack/nova: fix errors in revert resize api docs  https://review.openstack.org/33067216:48
*** david-lyle has joined #openstack-nova16:51
*** prateek has joined #openstack-nova16:55
openstackgerritMatthew Edmonds proposed openstack/nova: fix errors in revert resize api docs  https://review.openstack.org/33067216:56
edmondswcreated the asynch post section as well16:56
*** abalutoiu has joined #openstack-nova16:58
*** derekh has quit IRC16:59
*** pacoblox has quit IRC17:00
*** belmoreira has quit IRC17:00
openstackgerritMatthew Booth proposed openstack/nova: Introduce ImageCacheLocalDir  https://review.openstack.org/27966917:02
mdboothdiana_clarke: ^^^ I've updated ImageCacheLocalDir following review from you, dansmith and mriedem. Don't think you need to rebase just yet as I haven't written any functional tests yet, but fyi.17:03
dansmithcool17:04
*** pacoblox has joined #openstack-nova17:04
dansmithmdbooth: the reason I didn't first mention the functional tests on the review is I'm not sure whether we should hold that up for them or not.. history has shown that code to be often-broken and we test it very litt,17:05
dansmithso reviewing it makes me want some more serious tests for it for sure,17:05
mdboothdansmith: Well now the code's all in one place it's at least testable :)17:05
dansmithbut if it doesn't make sense to bolt those to that, I don't want to stand in the way17:05
dansmithyeah, that's good :)17:06
mdboothdansmith: I'm a fan of functional tests for this sort of thing. Infra issues aside, It won't be a chore.17:06
*** mvk_ has quit IRC17:06
*** akshai_ has quit IRC17:06
dansmithmdbooth: cool17:06
*** chandankumar has joined #openstack-nova17:07
mdboothdansmith: What extent would you want to go to with locking tests?17:07
dansmithmdbooth: you're mocking them all out in the unit tests right?17:08
*** daemontool has quit IRC17:08
mdboothdansmith: Possibly related, I wrote this: https://review.openstack.org/#/c/325496/17:08
mdboothdansmith: In fact, is that something like the functional tests you want?17:08
*** salv-orl_ has joined #openstack-nova17:08
dansmithmdbooth: yeah maybe.. I was also thinking something that mocks out as little of the file manipulation as possible17:11
dansmithi.e. point it at a tempdir, give it a couple images to fetch, cultivate, expire, etc17:11
mdboothI'd prefer not to extend into the 'consumer' side of ImageCacheManager right now17:12
* dansmith isn't sure what that means17:12
*** salv-orlando has quit IRC17:12
*** bharath has quit IRC17:12
mdboothSo there's ImageCacheLocalDir, which creates things in the image cache17:12
*** tbachman has quit IRC17:12
mdboothAnd ImageCacheManager, which deletes them randomly17:12
mdbooth^H^H^H^H^H^H^H manages them17:13
*** bharath has joined #openstack-nova17:13
mdboothI've only addressed the part of the code which puts things into the image cache in the first place17:13
*** Daisy has joined #openstack-nova17:13
*** diga has quit IRC17:13
dansmithah, okay17:13
dansmithfair enough17:13
mdboothWe obviously want that to be compatible with ImageCacheManager, but testing the whole of ImageCacheManager would be enormous scope creep17:14
mdboothIncidentally, I called it ImageCacheLocalDir, because I think there should be multiple implementation of the image cache17:15
mdboothAnd this one uses a local directory17:15
*** diga has joined #openstack-nova17:15
dansmithwell, regardless, I still hope for better testing of the whole deal someday17:15
mdboothFor sure17:15
dansmithwe do a terrible job of testing non-request paths like our periodics and this17:15
*** BobBall is now known as BobBall_AWOL17:16
pc_mHi Nova folks! I'm trying to backport a change from master to Liberty in my local repo (review 292499). I was able to get the changes needed (some things were already in master and I needed to add them too). However, I'm stuck on one thing, which I'm hoping someone can advise on?17:17
mriedempc_m: you're digging your own grave with that reserved host / numa backport thing my friend17:17
openstackgerritMatthew Booth proposed openstack/nova: Add concurrency tests for ImageCacheLocalDir  https://review.openstack.org/32549617:17
pc_mmriedem: Is there any alternative?17:18
*** Daisy has quit IRC17:18
mriedemdepends on your situation but probably not17:18
pc_mmriedem: I think I have it functioning OK (the database is updated correctly).17:18
pc_mThe issue is with versioning, which I don't understand the mech.17:18
pc_mThe patch increases version to 1.1. for NUMAPagesTopology object.17:19
openstackgerritMaciej Szankin proposed openstack/nova: Remove redundant DEPRECATED tag from help messages  https://review.openstack.org/32911717:19
pc_mWHen I try to boot a VM, I get a remote error saying incompatble version 1.1 for that object.17:19
mriedempc_m: there is a compat method in the object that you probably need to update17:20
pc_mmriedem: There is a obj_make_compatible() that checks the version.17:20
*** sambetts is now known as sambetts|afk17:21
*** rcernin has quit IRC17:21
pc_mit removes the new field, if < 1.117:21
pc_mI want to use the new 1.1 version though, and am not sure what I need to change to do that. Is this specified on the nova client?17:22
pc_m(the version)17:22
*** eharney has quit IRC17:23
*** bauzas has quit IRC17:23
*** browne has joined #openstack-nova17:23
*** bauzas has joined #openstack-nova17:23
*** cdent has quit IRC17:24
openstackgerritAugustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys  https://review.openstack.org/32969717:24
*** ljxiash has quit IRC17:24
*** ihrachys_ has quit IRC17:24
openstackgerritMaciej Szankin proposed openstack/nova: config options: improve help for noVNC  https://review.openstack.org/31945317:26
pc_mmriedem: The two places (that I could find) that create a NUMAPagesTopology object, use the new reserved field.17:26
*** sudipto has quit IRC17:26
openstackgerritAugustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys  https://review.openstack.org/32969717:26
*** lpetrut has quit IRC17:27
*** boltR_ has quit IRC17:27
*** moshele has joined #openstack-nova17:28
*** akshai has joined #openstack-nova17:28
*** eharney has joined #openstack-nova17:28
*** paul-carlton2 has joined #openstack-nova17:31
*** xyang1 has joined #openstack-nova17:33
*** paul-carlton2 has quit IRC17:35
*** tbachman has joined #openstack-nova17:38
openstackgerritMaciej Szankin proposed openstack/nova: config options: Improve help for base  https://review.openstack.org/32455917:38
*** mvk_ has joined #openstack-nova17:39
openstackgerritPaul Murray proposed openstack/nova: Add periodic task to clean expired console tokens  https://review.openstack.org/32538117:40
openstackgerritPaul Murray proposed openstack/nova: Add console auth token records to the database  https://review.openstack.org/32541417:40
openstackgerritPaul Murray proposed openstack/nova: Add console auth object  https://review.openstack.org/32006317:40
openstackgerritPaul Murray proposed openstack/nova: Add instance uuid to console URLs  https://review.openstack.org/32538217:40
openstackgerritPaul Murray proposed openstack/nova: Add console auth tokens db api methods  https://review.openstack.org/31996417:41
openstackgerritPaul Murray proposed openstack/nova: Add console auth tokens table and model  https://review.openstack.org/31932717:41
openstackgerritPushkar Umaranikar proposed openstack/nova: [WIP] Improve help text for compute manager options  https://review.openstack.org/32963917:41
*** rlrossit_ has quit IRC17:42
*** stewie925 has quit IRC17:45
mriedeminventory.resource_class_id is just an index into the ResourceClass field enum right?17:46
dansmithfor now yeah17:46
*** eharney has quit IRC17:46
openstackgerritdane-fichter proposed openstack/nova-specs: API support for image signing  https://review.openstack.org/32338917:47
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 5)  https://review.openstack.org/32997917:48
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2)  https://review.openstack.org/32997817:48
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 4)  https://review.openstack.org/32922717:48
openstackgerritClaudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 1)  https://review.openstack.org/33003317:48
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 6)  https://review.openstack.org/32922817:48
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 3)  https://review.openstack.org/32885117:48
claudiubyeees, one of the review numbers is a palindrome.. :D17:49
*** diga has quit IRC17:50
diana_clarkemdbooth: Thanks for pointing to your updated comment about get_func_output_path in that review. I was just about to do that.17:51
mriedemdansmith: why doesn't something like https://review.openstack.org/#/c/326440/22/nova/objects/resource_provider.py@66 happen in the Inventory object?17:53
mriedemnvm, i guess it's just faster to do this directly17:53
dansmithmriedem: it used to and they decided they wanted it to be a thing on the RP17:53
mriedemthan load the list of inventory objects and delete them one by one17:53
* PaulMurray england 2 : 1 wales - life was nearly not worth living for a while there17:54
dansmiththat too I suppose17:54
*** stewie925 has joined #openstack-nova17:55
*** chandankumar has quit IRC17:55
mriedemif isinstance(res_class, six.string_types):17:56
mriedem            res_class = fields.ResourceClass.index(res_class)17:56
mriedemis it just me or is that busted?17:56
mriedemoh nvm17:56
mriedemguh this is messing me up17:58
mriedemhttps://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L24117:58
*** mtanino has quit IRC17:59
mriedemand https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L18318:00
*** baoli_ has quit IRC18:00
*** artom has quit IRC18:00
*** dimtruck is now known as zz_dimtruck18:01
*** artom has joined #openstack-nova18:01
*** artom has quit IRC18:01
*** artom has joined #openstack-nova18:02
mriedemhow does passing a ResourceClassField into fields.ResourceClass.index() work when ResourceClass.index is checking the index on it's backing ALL tuple which has strings in it18:02
mriedembecaues enum fields are strings18:02
mriedemand it gets coerced18:02
mriedemok18:02
*** Sukhdev has joined #openstack-nova18:03
*** rlrossit has joined #openstack-nova18:04
*** kmartin has quit IRC18:04
*** moshele has quit IRC18:05
*** rcernin has joined #openstack-nova18:07
*** tblakeslee has joined #openstack-nova18:08
*** akshai has quit IRC18:08
*** akshai has joined #openstack-nova18:11
*** mtanino has joined #openstack-nova18:11
*** jaugustine has joined #openstack-nova18:13
*** PaulMurray has quit IRC18:13
*** yassine__ has quit IRC18:13
*** eharney has joined #openstack-nova18:14
*** baoli has joined #openstack-nova18:16
mriedemdansmith: so i was going to -1 this but check my comments first https://review.openstack.org/#/c/326440/18:17
dansmithmriedem: reading18:18
*** bharath has quit IRC18:19
mriedemok brb18:19
dansmithmriedem: good catch on the docstring.. I don't think putting the IDs in there is helpful, but if you want them then that's fine I guess18:20
* dansmith hates how most of our exceptions require reading before they can be used18:21
*** lpetrut has joined #openstack-nova18:22
openstackgerritMaciej Szankin proposed openstack/nova: WIP Transform instance.reboot notifications  https://review.openstack.org/33022418:23
*** mriedem1 has joined #openstack-nova18:23
*** mkoderer__ has joined #openstack-nova18:24
pc_mdansmith: If a versioned object adds a field and the version is increase, what other changes are needed to prevent incompatible version error?18:25
* pc_m trying to apply change from master to Liberty in my local repo.18:25
*** mriedem has quit IRC18:26
dansmithpc_m: only dropping the version down18:26
pc_m?18:26
pc_mThe patch I'm applying adds a field to NUMAPagesTopology and increases the version to 1.118:27
pc_mIn UTs, I get a failure saying objects have changed dependencies and that I need to bump up parent versions?18:27
*** kmartin has joined #openstack-nova18:27
*** ctrath has quit IRC18:28
dansmithpc_m: right because you're trying to backport across a major change where we dropped dependency mappings, which means there is no legit dep mapping for you to backport18:28
dansmithso you're just going to have to break something or cheat to make it happy18:28
openstackgerritjichenjc proposed openstack/nova: Update stable API doc to indicate code removal  https://review.openstack.org/32706618:29
*** mriedem1 is now known as mriedem18:29
*** rama_y has joined #openstack-nova18:30
mriedempc_m: dansmith: i think it's related to the fact that NUMATopologyWhatever that has the reserved field is also a list in NUMACell or some other object, which is the parent, and the obj_relationships set there wasn't updated18:30
mriedemi could be wrong18:30
mriedembut that's the parent/child thing i think18:30
mriedembut yeah, this is all not really something you want to do, it's backporting RPC API version changes basically18:30
pc_mmriedem: Yes, NUMAPagesTopology is contained inside of a NUMACell.18:31
pc_mmriedem: dansmith: Looking at NUMACell, it has 1.2 and a comment that 1.2 is for the NUMAPagesTopology field.18:32
*** csd_ has quit IRC18:32
dansmithmriedem: it's because it was added after we dropped relationships18:32
pc_mSo a bit confused.18:32
*** ctrath has joined #openstack-nova18:32
*** sacharya has quit IRC18:32
*** sacharya_ has joined #openstack-nova18:32
dansmithmriedem: replied to your comment on that inventory one18:33
*** sacharya has joined #openstack-nova18:33
dansmithmriedem: the user is a computer, hence why I don't think it's critical18:33
pc_mdansmith: I did see an obj_relationships tuple in the Liberty code and not on master18:33
*** baoli has quit IRC18:34
mriedemdansmith: yeah computers log things at times too18:34
mriedemanyway18:34
pc_mdansmith: Any suggestions on how I can make this work?18:34
dansmithpc_m: yes18:34
dansmithpc_m: upgrade18:34
pc_mdansmith: I can't :(18:35
dansmithpc_m: then, no.18:35
pc_mis the obj_relationships specifying the version of the parent and of the child?18:35
openstackgerritPrateek Arora proposed openstack/nova: Delete traces of in-progress snapshot on VM being deleted  https://review.openstack.org/29451318:35
mriedemdansmith: +W this and i'll push a change to fix the docstring https://review.openstack.org/#/c/326440/18:35
dansmithmriedem: I dropped my +2.. totally happy to fix the docstring now18:36
pc_mI see in Liberty it has 1.2 and 1.0 in the tuple (1.2 is the NUMACell version, and 1.0 was the NUMAPagesTopology version, that I updated to 1.1). Not sure I understand all this...18:36
mriedemdansmith: i didn't want to mess up the series18:36
dansmithmriedem: and I'm totes happy for the full gettysburg address to be in the exception message :)18:36
*** baoli has joined #openstack-nova18:37
mriedem"Dear resource provider user, it appears you are trying to update inventory. While we appreciate your request, some dickweed got in before you, so please retry. Thank you and have a blessed day!"18:37
*** sacharya_ has quit IRC18:37
dansmithLOL18:37
* dansmith almost spit out sandwich18:38
*** d_code has joined #openstack-nova18:39
mriedemok i'll inline fix the docstring, it'll goof the series but jay and chris aren't around to rebase it18:39
mriedemi figure they'd rather rebase than waste a day o nthat18:39
mriedemoh i see +W now18:39
*** pushkaru has quit IRC18:40
d_codeHI, Iā€™m trying to do a PCI passthrough. Iā€™ve got two different devices that I want to pass through to a given flavor. Is it currently possible (in mitaka) to have a flavor pick device A or B?18:40
d_codeit seems that pci_alias can only associate a single device currently18:40
d_codeand that a flavor can specify multiple pci_aliases, but only in an ā€œANDā€ specification18:41
*** zz_dimtruck is now known as dimtruck18:41
openstackgerritAugustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys  https://review.openstack.org/32969718:42
*** prateek has quit IRC18:42
*** mgoddard has quit IRC18:42
*** tonytan4ever has joined #openstack-nova18:43
*** dmk0202 has joined #openstack-nova18:44
dansmithd_code: see topic?18:44
d_codedansmith: my apologiesā€¦when Iā€™ve gone to #openstack in the past theyā€™ve referred me here18:45
*** dmk0202 has quit IRC18:45
openstackgerritMaciej Szankin proposed openstack/nova: Transform instance.shutdown notifications  https://review.openstack.org/32921418:45
*** akshai_ has joined #openstack-nova18:45
d_codeitā€™s hard to delineate between whatā€™s currently in dev and whats in mainline18:45
d_codeopenstack just moves too fast18:46
d_codebutā€¦  Iā€™ll head over there18:46
*** akshai has quit IRC18:47
openstackgerritMatt Riedemann proposed openstack/nova: Fix docstring nits from ResourceProvider.set_inventory() review  https://review.openstack.org/33074518:49
mriedemdansmith: ^18:49
*** akshai_ has quit IRC18:50
*** kmartin has quit IRC18:50
dansmithmriedem: shall I nit out on your preposition fix?18:51
*** rlrossit has quit IRC18:52
mriedemgo for it18:53
dansmithI chickened out18:53
*** salv-orl_ has quit IRC18:54
*** salv-orlando has joined #openstack-nova18:55
*** baoli has quit IRC18:56
mriedemctrath: http://docs.openstack.org/developer/nova/notifications.html18:57
openstackgerritClaudiu Belu proposed openstack/nova: numa: Fixes NUMA topology related unit tests  https://review.openstack.org/27883518:57
openstackgerritMaciej Szankin proposed openstack/nova: Transform instance.shutdown notifications  https://review.openstack.org/32921418:57
openstackgerritClaudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests  https://review.openstack.org/27883418:57
mriedemctrath: this is older also https://wiki.openstack.org/wiki/SystemUsageData18:57
openstackgerritClaudiu Belu proposed openstack/nova: Fixes python 3 unit tests  https://review.openstack.org/27881018:57
mriedemctrath: https://wiki.openstack.org/wiki/NotificationEventExamples - and ceilometer and designate listen for nova notifications18:58
mriedemctrath: https://github.com/openstack/designate/tree/master/designate/notification_handler18:58
openstackgerritAndrew Bogott proposed openstack/nova: Prevent scheduling of instances on full hosts.  https://review.openstack.org/24225118:59
mriedemctrath: and rackspace uses this for monitoring https://github.com/openstack/stacktach18:59
openstackgerritClaudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests  https://review.openstack.org/30026219:00
*** ociuhandu has quit IRC19:02
*** ijw has joined #openstack-nova19:03
*** ijw has quit IRC19:06
*** ijw has joined #openstack-nova19:06
*** lpetrut has quit IRC19:13
*** sacharya has quit IRC19:13
*** sacharya has joined #openstack-nova19:13
*** d_code has quit IRC19:21
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: Transform instance.shutdown notifications  https://review.openstack.org/32921419:21
openstackgerritAugustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys  https://review.openstack.org/32969719:23
openstackgerritAnusha Unnam proposed openstack/nova: Transform instance.resize notifications  https://review.openstack.org/32925519:27
*** harshad has joined #openstack-nova19:28
zigoAfter installing nova Newton b1, I get an empty flavor-list.19:29
zigoHas something I'm not aware of changed?19:30
zigoLike, no flavors by default?19:30
*** Jeffrey4l has quit IRC19:30
*** harshad has quit IRC19:31
zigodansmith: ^19:31
zigobauzas: ^19:31
dansmithzigo: yep, posts on the operators and dev mailing lists about it19:31
*** Jeffrey4l has joined #openstack-nova19:31
zigodansmith: Ok, cheers. Do you remember what the topic was?19:32
*** ctrath has quit IRC19:32
dansmithzigo: [openstack-dev] [nova] FYI: Removing default flavors from nova19:32
zigoFound it! :)19:32
dansmithtwo months ago to the day :)19:33
*** ctrath has joined #openstack-nova19:33
zigodansmith: I'm only finishing Newton b1 and testing it today! :)19:33
*** d_code has joined #openstack-nova19:33
openstackgerritJoao Targino proposed openstack/python-novaclient: Update README to comply with Identity v3  https://review.openstack.org/33075819:33
zigodansmith: I only do beta releases, not trunk yet.19:33
dansmith*gasp*19:34
zigodansmith: In your post, you're saying that the flavor have "moved". But moved to where?19:34
zigoIn nova-api ?19:34
dansmithzigo: the default flavors have not moved (well, except to /dev/null)19:35
dansmithzigo: the data structure moved to the api db yes19:35
zigodansmith: Ok, so I guess, I could potentially add this to the install of the nova-api package then.19:35
dansmithyup19:36
*** ociuhandu has joined #openstack-nova19:37
*** READ10 has quit IRC19:37
zigodansmith: Thanks a lot, doing the work now.19:38
dansmithnp19:38
*** dimtruck is now known as zz_dimtruck19:39
openstackgerritPushkar Umaranikar proposed openstack/nova: Improve help text for compute manager options  https://review.openstack.org/32963919:39
*** dmk0202 has joined #openstack-nova19:40
*** d_code has quit IRC19:43
*** mfedosin has quit IRC19:43
*** dmk0202 has quit IRC19:43
*** d_code has joined #openstack-nova19:44
openstackgerritAndy Hsiang proposed openstack/nova: Transform instance.power_off notifications  https://review.openstack.org/33076119:44
*** lpetrut has joined #openstack-nova19:46
openstackgerritPushkar Umaranikar proposed openstack/nova: Improve help text for compute manager options  https://review.openstack.org/32963919:47
*** tblakeslee has quit IRC19:47
zigodansmith: oh, the IDs aren't 1,2,3,4 ... :/19:48
*** stewie925 has quit IRC19:48
zigoThis breaks tempest! :)19:48
* zigo gives up on adding default flavors then.19:48
dansmithzigo: what ids? flavor ids are strings, always have been, if that's what you mean19:49
zigodansmith: Yeah, but now we get a UUID, not just 1, 2, 3 ...19:49
zigoSo I can't put a default value in tempest that would just work given defaults flavors.19:49
zigoThat's what I mean.19:50
zigoNot a big deal, it just needs a bit workout.19:50
dansmithnot sure what you mean by uuid.. nothing has changed around flavor ids with the move, IIRC19:50
dansmithyou can create debian-small, debian-large or something19:51
*** rlrossit has joined #openstack-nova19:51
*** fragatin_ has quit IRC19:51
zigo:)19:52
zigosure19:52
zigoAs I wrote, not a big deal, really.19:53
*** parveen has joined #openstack-nova19:58
parveenhello19:58
*** ssurana has quit IRC19:58
parveeni have configure mitaka on xenserver19:59
*** stewie925 has joined #openstack-nova19:59
parveenbut i have fail to launch the instance on that19:59
parveencan any one help ................?19:59
*** jichen has quit IRC19:59
parveengetting error NovaException: Found no network for bridge brq1e73868c-7e20:00
parveenon noav log20:00
parveenon nova log20:00
*** salv-orlando has quit IRC20:00
*** salv-orlando has joined #openstack-nova20:01
dansmithparveen: see topic20:01
parveenwhich topic..?20:02
*** Apoorva has quit IRC20:02
dansmithparveen: channel topic: This channel is for Nova development. For support of Nova deployments, please use #openstack.20:02
*** ssurana has joined #openstack-nova20:03
*** lucas___ has quit IRC20:04
*** ssurana has quit IRC20:04
*** sheel has quit IRC20:05
*** matrohon has joined #openstack-nova20:06
*** ssurana has joined #openstack-nova20:07
*** zz_dimtruck is now known as dimtruck20:07
*** baoli has joined #openstack-nova20:07
openstackgerritClaudiu Belu proposed openstack/nova: policy: Add defaults in code (part 1)  https://review.openstack.org/32885020:08
*** fragatina has joined #openstack-nova20:10
*** fragatina has quit IRC20:10
*** fragatina has joined #openstack-nova20:11
*** rlrossit has quit IRC20:13
*** akshai has joined #openstack-nova20:13
*** rlrossit has joined #openstack-nova20:14
*** tblakeslee has joined #openstack-nova20:14
*** ayoung has quit IRC20:18
*** rlrossit has quit IRC20:18
*** parveen has left #openstack-nova20:19
*** openstackstatus has joined #openstack-nova20:19
*** ChanServ sets mode: +v openstackstatus20:19
*** tonytan4ever has quit IRC20:19
*** lucas-afk has quit IRC20:21
*** zul has quit IRC20:21
*** zul has joined #openstack-nova20:22
*** Apoorva has joined #openstack-nova20:22
*** dmk0202 has joined #openstack-nova20:23
*** gyee has joined #openstack-nova20:24
openstackgerritClif Houck proposed openstack/nova: Add Image Remote Authentication Verification  https://review.openstack.org/33077320:26
*** rlrossit has joined #openstack-nova20:27
*** ihrachys has joined #openstack-nova20:27
*** lucasagomes has joined #openstack-nova20:28
*** atarakt_ has joined #openstack-nova20:32
*** lpetrut has quit IRC20:38
atarakt_Hello, I'd like to finish this feature https://review.openstack.org/#/c/203751 based on this blueprint https://blueprints.launchpad.net/nova/+spec/purge-deleted-instances-cmd20:38
atarakt_do I have to do something on the blueprint or only push the rebased/fixed code in the review ?20:39
mriedemrlrossit: https://github.com/openstack/nova/blob/master/nova/notifications/base.py#L6820:40
rlrossitmriedem: https://github.com/openstack/nova/blob/c867b3b85d29a7756d67d304346f2998850e6c04/nova/notifications/objects/base.py#L13520:42
rlrossitit looks like the dot in the publisher_id is going away in that anyways20:42
mriedemrlrossit: yeah that object is created from here https://review.openstack.org/#/c/310700/19/nova/exception_wrapper.py20:43
mriedemwhich starts from https://github.com/openstack/nova/blob/master/nova/notifications/base.py#L68 i think20:44
mriedemthis versioned notifications stuff scares me20:45
*** hemanthm is now known as hemanthm|afk20:46
*** gagehugo has quit IRC20:47
Prophidys_atarakt_, like talked with mriedem and dansmith I think it's just a re-submit asis of the blueprint20:48
mriedematarakt_: Prophidys_: the spec should probably be re-proposed20:49
mriedematarakt_: Prophidys_: https://specs.openstack.org/openstack/nova-specs/readme.html#previously-approved-specifications20:50
atarakt_mriedem: thanks =)20:50
*** lpetrut has joined #openstack-nova20:51
*** ihrachys has quit IRC20:52
*** zul has quit IRC20:52
*** jaugustine has quit IRC20:52
*** baoli has quit IRC20:53
*** efoley has quit IRC20:53
*** ihrachys has joined #openstack-nova20:53
*** ssurana has quit IRC20:53
*** sacharya has quit IRC20:54
*** sacharya has joined #openstack-nova20:55
*** sacharya_ has joined #openstack-nova20:56
*** ssurana has joined #openstack-nova20:56
*** ssurana has quit IRC20:56
*** jamie_h has quit IRC20:57
*** rtheis has quit IRC20:57
*** ssurana has joined #openstack-nova20:58
*** sacharya has quit IRC20:59
openstackgerritVladik Romanovsky proposed openstack/nova: DO NOT MERGE - attach configdrive after instance XML test  https://review.openstack.org/33013421:01
*** baoli has joined #openstack-nova21:02
tjonesi have 3 easy peasy patches for review that remove mox - wilting on the vineā€¦  https://review.openstack.org/#/c/265476/ https://review.openstack.org/#/c/270303/ https://review.openstack.org/#/c/271069/1021:02
*** rfolco has quit IRC21:03
*** mmandel has quit IRC21:05
*** aysyd has quit IRC21:06
*** sacharya has joined #openstack-nova21:10
*** sachary__ has joined #openstack-nova21:11
*** thorst_ has quit IRC21:11
*** thorst_ has joined #openstack-nova21:11
*** sacharya_ has quit IRC21:13
openstackgerritNicolas Hicher proposed openstack/nova-specs: Spec: add nova-manage db purge-deleted-instances  https://review.openstack.org/33079121:14
*** smatzek has quit IRC21:14
*** sacharya has quit IRC21:14
*** krtaylor has quit IRC21:15
*** thorst_ has quit IRC21:16
*** eharney has quit IRC21:19
*** ihrachys_ has joined #openstack-nova21:19
*** sachary__ has quit IRC21:20
*** sacharya has joined #openstack-nova21:20
*** baoli has quit IRC21:21
*** sacharya_ has joined #openstack-nova21:21
*** baoli has joined #openstack-nova21:21
openstackgerritNicolas Hicher proposed openstack/nova: Purge soft-deleted instances cmd  https://review.openstack.org/20375121:22
*** ihrachys has quit IRC21:22
*** rrecio_ has quit IRC21:23
*** thorst_ has joined #openstack-nova21:23
*** mmandel has joined #openstack-nova21:23
*** ctrath has quit IRC21:24
*** mdrabe has quit IRC21:25
*** sacharya has quit IRC21:25
*** thorst_ has quit IRC21:28
*** rlrossit has quit IRC21:32
*** tblakeslee has quit IRC21:32
*** diogogmt has quit IRC21:34
*** tblakeslee has joined #openstack-nova21:34
*** diogogmt has joined #openstack-nova21:36
*** rcernin has quit IRC21:36
*** dimtruck is now known as zz_dimtruck21:37
*** zz_dimtruck is now known as dimtruck21:37
*** dimtruck is now known as zz_dimtruck21:37
*** matrohon has quit IRC21:38
mriedemhow does this logic make sense21:38
mriedemhttps://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L176821:38
*** dmk0202 has quit IRC21:38
mriedemif admin_state_up is False then consider the vif active21:38
*** cleong has quit IRC21:39
*** lpetrut has quit IRC21:39
*** ssurana has quit IRC21:43
*** thorst_ has joined #openstack-nova21:44
*** thorst__ has joined #openstack-nova21:46
*** burgerk has quit IRC21:46
openstackgerritSujitha proposed openstack/nova: Improve help text for upgrade_levels options  https://review.openstack.org/32915421:46
*** ssurana has joined #openstack-nova21:47
*** woodster_ has quit IRC21:48
*** thorst_ has quit IRC21:49
*** thorst__ has quit IRC21:50
*** mriedem has quit IRC21:51
*** yamahata has quit IRC21:51
*** mriedem has joined #openstack-nova21:52
*** mriedem has quit IRC21:52
*** zz_dimtruck is now known as dimtruck21:53
*** dimtruck is now known as zz_dimtruck21:57
*** sameo has quit IRC21:57
*** sameo has joined #openstack-nova21:57
*** amotoki has joined #openstack-nova22:03
*** dmk0202 has joined #openstack-nova22:04
*** browne has quit IRC22:04
*** BjoernT has quit IRC22:04
*** xyang1 has quit IRC22:04
*** ayoung has joined #openstack-nova22:07
*** akshai has quit IRC22:11
*** baoli has quit IRC22:11
*** baoli has joined #openstack-nova22:12
*** kylek3h has quit IRC22:16
*** atarakt_ has quit IRC22:17
*** clenimar_ has joined #openstack-nova22:21
openstackgerritVladik Romanovsky proposed openstack/nova: DO NOT MERGE - attach configdrive after instance XML test  https://review.openstack.org/33013422:23
*** ssurana has quit IRC22:25
*** ssurana has joined #openstack-nova22:26
*** baoli has quit IRC22:27
*** baoli has joined #openstack-nova22:27
*** salv-orlando has quit IRC22:29
openstackgerritAndy Hsiang proposed openstack/nova: Transform instance.power_off notifications  https://review.openstack.org/33076122:29
*** stewie925 has quit IRC22:29
*** salv-orlando has joined #openstack-nova22:30
openstackgerritAndrew Bogott proposed openstack/nova: Prevent scheduling of instances on full hosts.  https://review.openstack.org/24225122:31
*** mriedem has joined #openstack-nova22:32
*** darrenc is now known as darren_afk22:34
*** salv-orlando has quit IRC22:34
*** ssurana has quit IRC22:35
*** salv-orlando has joined #openstack-nova22:35
*** jmarianu has joined #openstack-nova22:35
*** clenimar_ has quit IRC22:37
*** jamesdenton has quit IRC22:38
*** abalutoiu has quit IRC22:39
*** dmk0202 has quit IRC22:39
*** ssurana has joined #openstack-nova22:39
*** baoli has quit IRC22:40
openstackgerritMatt Riedemann proposed openstack/nova: Skip network allocation if 'none' is requested  https://review.openstack.org/30963822:42
openstackgerritMatt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support  https://review.openstack.org/31895322:42
openstackgerritMatt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance  https://review.openstack.org/31627522:42
openstackgerritMatt Riedemann proposed openstack/nova: Add REST API support for get me a network  https://review.openstack.org/31639822:42
openstackgerritSujitha proposed openstack/nova: Improve help text for glance options  https://review.openstack.org/32400722:42
*** mmandel has quit IRC22:50
*** darren_afk is now known as darrenc22:55
*** browne has joined #openstack-nova22:57
*** ivase has quit IRC22:57
*** ivase has joined #openstack-nova22:59
*** mrkz has quit IRC22:59
*** MikeG451_ has quit IRC22:59
*** clenimar_ has joined #openstack-nova23:01
*** alaski_ is now known as alaski23:02
*** zenoway has joined #openstack-nova23:04
*** ayoung has quit IRC23:07
*** pradk has quit IRC23:07
*** edmondsw has quit IRC23:07
*** salv-orl_ has joined #openstack-nova23:08
*** zenoway has quit IRC23:09
*** diogogmt has quit IRC23:10
*** browne has quit IRC23:10
*** salv-orlando has quit IRC23:11
openstackgerritRawlin Peters proposed openstack/os-vif: Ensure the OVS bridge exists when plugging  https://review.openstack.org/33081823:12
*** claudiub|2 has joined #openstack-nova23:16
*** claudiub has quit IRC23:20
*** ihrachys_ has quit IRC23:20
*** ihrachys has joined #openstack-nova23:21
*** claudiub has joined #openstack-nova23:23
*** claudiub|2 has quit IRC23:27
*** ivase has quit IRC23:27
*** ivase has joined #openstack-nova23:27
*** hoangcx has quit IRC23:37
*** jamesdenton has joined #openstack-nova23:39
*** sshen_ has quit IRC23:43
*** ayoung has joined #openstack-nova23:44
openstackgerritSujitha proposed openstack/nova: Remove mox from tests/unit/virt/test_imagecache.py  https://review.openstack.org/30482123:44
*** sameo has quit IRC23:44
*** salv-orl_ has quit IRC23:47
*** salv-orlando has joined #openstack-nova23:48
*** sshen has joined #openstack-nova23:48
*** ducncvn has joined #openstack-nova23:51
*** ducnc has quit IRC23:51
*** ihrachys has quit IRC23:51
*** tblakeslee has quit IRC23:52
*** sdake has quit IRC23:57
*** ssurana has quit IRC23:58
*** ssurana has joined #openstack-nova23:58

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