Monday, 2016-09-12

*** tonytan4ever has joined #openstack-nova00:02
*** ociuhandu has joined #openstack-nova00:04
*** thorst has joined #openstack-nova00:06
*** ducttape_ has joined #openstack-nova00:08
*** thorst has quit IRC00:13
*** ducttape_ has quit IRC00:19
*** ducttape_ has joined #openstack-nova00:22
*** TravT has joined #openstack-nova00:22
*** lixiaoy1 has joined #openstack-nova00:24
*** takashin has joined #openstack-nova00:28
*** TravT has quit IRC00:29
*** TravT has joined #openstack-nova00:30
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix wrong parameters in os-volumes.inc  https://review.openstack.org/34711200:30
openstackgerritTakashi NATSUME proposed openstack/nova: Fix unnecessary string interpolation  https://review.openstack.org/35757700:30
openstackgerritTakashi NATSUME proposed openstack/nova: Disassociate floating ip when fixed ip is removed  https://review.openstack.org/35029900:30
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix 'id' (attachment_id) parameters  https://review.openstack.org/34986300:31
openstackgerritTakashi NATSUME proposed openstack/nova: Fix MonitorMetric obj_make_compatible  https://review.openstack.org/36179200:31
openstackgerritTakashi NATSUME proposed openstack/nova: Fix an error in archiving 'migrations' table  https://review.openstack.org/32368400:31
openstackgerritTakashi NATSUME proposed openstack/nova: Add a hacking rule for string interpolation at logging  https://review.openstack.org/33484400:31
openstackgerritTakashi NATSUME proposed openstack/nova: Set 'serial' to new volume ID in swap volumes  https://review.openstack.org/25713500:31
openstackgerritTakashi NATSUME proposed openstack/nova: Treat 'body_map' as a mapping of HTTP request bodies  https://review.openstack.org/36345700:31
*** TravT has quit IRC00:31
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host  https://review.openstack.org/35710500:31
*** TravT has joined #openstack-nova00:32
*** ducttape_ has quit IRC00:32
*** takashin_ has joined #openstack-nova00:34
*** TravT_ has joined #openstack-nova00:34
openstackgerritTakashi NATSUME proposed openstack/nova: Remove mox in test_block_device_mapping_v1.py  https://review.openstack.org/30874600:35
openstackgerritTakashi NATSUME proposed openstack/nova: Remove mox in test_block_device_mapping_v1.py  https://review.openstack.org/30874600:35
*** TravT has quit IRC00:35
*** kaisers1 has joined #openstack-nova00:35
*** bswartz has joined #openstack-nova00:36
openstackgerritTakashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/cells  https://review.openstack.org/32328700:36
openstackgerritTakashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/cells  https://review.openstack.org/32328700:36
*** takashin has quit IRC00:37
*** kaisers has quit IRC00:38
openstackgerritTakashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/virt/disk  https://review.openstack.org/32277800:39
*** fragatina has joined #openstack-nova00:39
openstackgerritTakashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/virt/disk  https://review.openstack.org/32277800:39
*** hoangcx has joined #openstack-nova00:39
*** unicell has joined #openstack-nova00:46
*** aabashkin_ has joined #openstack-nova00:47
*** GB21 has joined #openstack-nova00:47
*** AlexeyAbashkin has quit IRC00:48
*** gongysh has quit IRC00:50
*** unicell has quit IRC00:52
*** cheneydc has joined #openstack-nova01:01
*** baoli has joined #openstack-nova01:01
*** mingyu has quit IRC01:01
*** baoli has quit IRC01:06
*** gongysh has joined #openstack-nova01:10
*** thorst has joined #openstack-nova01:11
*** thorst has quit IRC01:19
*** cheneydc has quit IRC01:20
*** ducttape_ has joined #openstack-nova01:33
*** hoangcx has quit IRC01:35
*** hoangcx has joined #openstack-nova01:36
*** mriedem has quit IRC01:36
*** ducttape_ has quit IRC01:38
*** hoangcx2 has joined #openstack-nova01:38
*** hoangcx has quit IRC01:40
*** wangqun has joined #openstack-nova01:41
*** hoangcx has joined #openstack-nova01:42
*** hoangcx2 has quit IRC01:45
*** hoangcx2 has joined #openstack-nova01:49
*** thorst has joined #openstack-nova01:49
*** hoangcx has quit IRC01:51
*** hieulq has joined #openstack-nova01:52
*** vishwanathj has joined #openstack-nova01:56
*** EinstCrazy has joined #openstack-nova01:57
*** vishwanathj has quit IRC01:57
*** jamielennox is now known as jamielennox|away02:00
*** artom has quit IRC02:03
*** jamielennox|away is now known as jamielennox02:04
*** artom has joined #openstack-nova02:04
*** artom has quit IRC02:05
*** artom has joined #openstack-nova02:05
*** wangqun_ has joined #openstack-nova02:09
*** akshai_ has quit IRC02:09
*** wangqun has quit IRC02:09
*** lixiaoy1 has quit IRC02:14
*** lixiaoy1 has joined #openstack-nova02:14
*** yamahata has joined #openstack-nova02:16
*** hoangcx has joined #openstack-nova02:17
*** hoangcx2 has quit IRC02:18
*** lifeless_ has quit IRC02:19
*** artom has quit IRC02:21
openstackgerritTakashi NATSUME proposed openstack/nova: Add a hacking rule for string interpolation at logging  https://review.openstack.org/33484402:33
*** lifeless has joined #openstack-nova02:34
*** ducttape_ has joined #openstack-nova02:34
*** kevinz has joined #openstack-nova02:38
*** ducttape_ has quit IRC02:39
*** hoangcx has quit IRC02:42
*** hoangcx has joined #openstack-nova02:42
*** thorst has quit IRC02:46
*** thorst has joined #openstack-nova02:47
*** Jeffrey4l has quit IRC02:52
*** RuiChen has joined #openstack-nova02:52
*** thorst has quit IRC02:56
*** oanson has joined #openstack-nova02:57
*** Jeffrey4l has joined #openstack-nova02:58
*** hoangcx2 has joined #openstack-nova03:00
*** hoangcx has quit IRC03:00
*** hoangcx has joined #openstack-nova03:03
*** hoangcx2 has quit IRC03:04
*** oanson has quit IRC03:05
openstackgerritRui Chen proposed openstack/python-novaclient: Fix Resource.__eq__ mismatch semantics of object equal  https://review.openstack.org/25102003:11
*** oanson has joined #openstack-nova03:18
*** ducttape_ has joined #openstack-nova03:22
*** ducttape_ has quit IRC03:22
*** ociuhandu has quit IRC03:25
*** baoli has joined #openstack-nova03:26
*** thorst has joined #openstack-nova03:28
*** RuiChen has quit IRC03:29
*** RuiChen has joined #openstack-nova03:31
*** baoli has quit IRC03:34
*** baoli has joined #openstack-nova03:34
*** baoli has quit IRC03:35
*** lixiaoy1 has quit IRC03:35
*** baoli has joined #openstack-nova03:36
*** RuiChen_ has joined #openstack-nova03:37
*** Dinesh_Bhor has joined #openstack-nova03:39
*** baoli has quit IRC03:40
*** gongysh has quit IRC03:42
*** lixiaoy1 has joined #openstack-nova03:42
openstackgerritwanghuagong proposed openstack/nova: Correct chinese translations error  https://review.openstack.org/36853603:45
*** tuanla_ has joined #openstack-nova03:46
*** tuanla_ has quit IRC03:47
*** tuanla_ has joined #openstack-nova03:48
*** thorst has quit IRC03:52
*** unicell has joined #openstack-nova03:57
*** unicell has quit IRC03:57
*** hshiina has joined #openstack-nova03:57
*** bhagyashri has joined #openstack-nova04:02
*** unicell has joined #openstack-nova04:02
*** hshiina has quit IRC04:03
*** amotoki has joined #openstack-nova04:05
*** yamahata has quit IRC04:08
*** yamahata has joined #openstack-nova04:08
*** bhagyashri has quit IRC04:16
*** gouthamr has joined #openstack-nova04:19
*** jdurgin has joined #openstack-nova04:20
*** ducttape_ has joined #openstack-nova04:23
*** oanson has quit IRC04:26
*** baoli has joined #openstack-nova04:27
*** ducttape_ has quit IRC04:28
*** Sukhdev has joined #openstack-nova04:29
*** lixiaoy1 has quit IRC04:32
*** baoli has quit IRC04:34
*** thorst has joined #openstack-nova04:34
*** liuyulong_ is now known as liuyulong04:34
*** kaisers has joined #openstack-nova04:39
*** thorst has quit IRC04:39
*** pbandark has joined #openstack-nova04:41
*** thorst has joined #openstack-nova04:46
*** thorst has quit IRC04:50
*** adisky has joined #openstack-nova04:51
*** salv-orlando has joined #openstack-nova04:52
*** panbalag has quit IRC04:59
*** Jeffrey4l has quit IRC05:00
*** thorst has joined #openstack-nova05:00
*** Jeffrey4l has joined #openstack-nova05:00
*** jaosorior has joined #openstack-nova05:02
*** lpetrut has joined #openstack-nova05:02
*** GB21 has quit IRC05:04
*** gcb has joined #openstack-nova05:07
*** unicell has quit IRC05:07
*** unicell1 has joined #openstack-nova05:07
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Fix SafeConfigParser DeprecationWarning in Python 3.2  https://review.openstack.org/36841705:08
*** unicell has joined #openstack-nova05:11
*** unicell1 has quit IRC05:12
*** lixiaoy1 has joined #openstack-nova05:14
*** prateek has joined #openstack-nova05:14
*** unicell1 has joined #openstack-nova05:17
*** unicell has quit IRC05:17
*** GB21 has joined #openstack-nova05:18
*** claudiub has joined #openstack-nova05:21
*** ducttape_ has joined #openstack-nova05:24
*** wangqun_ has quit IRC05:27
*** ducttape_ has quit IRC05:29
*** rcernin has joined #openstack-nova05:33
*** paul-carlton2 has joined #openstack-nova05:43
openstackgerritTakashi NATSUME proposed openstack/nova: Add functional tests for cold migration with target  https://review.openstack.org/36855705:49
*** lpetrut has quit IRC05:50
openstackgerritedan david proposed openstack/nova: WIP: Assign mac address to vf netdevice when using macvtap port  https://review.openstack.org/36412105:53
*** houming has joined #openstack-nova05:57
*** gongysh has joined #openstack-nova05:59
*** thorst has quit IRC06:01
*** sudipto has joined #openstack-nova06:01
*** sudipto_ has joined #openstack-nova06:01
*** thorst has joined #openstack-nova06:02
*** lpetrut has joined #openstack-nova06:03
*** gongysh has quit IRC06:05
*** thorst has quit IRC06:05
*** thorst has joined #openstack-nova06:05
*** oanson has joined #openstack-nova06:08
*** RuiChen_ has quit IRC06:11
prateekI am trying to live-migrate instances in devstack and I am getting an error http://paste.openstack.org/show/572105/ , can anyone help me ?06:13
*** diga has joined #openstack-nova06:16
*** haplo37 has quit IRC06:19
*** nherciu has joined #openstack-nova06:19
*** mjura has joined #openstack-nova06:21
*** ljxiash has joined #openstack-nova06:21
*** coolsvap has joined #openstack-nova06:21
*** haplo37 has joined #openstack-nova06:22
openstackgerritGhanshyam Mann proposed openstack/nova: Use tempest tox with regex first  https://review.openstack.org/36856606:24
*** ducttape_ has joined #openstack-nova06:25
openstackgerrithuan proposed openstack/nova: xenapi: support the hotplug of a neutron port  https://review.openstack.org/36550506:26
*** mjura has quit IRC06:26
*** lpetrut has quit IRC06:27
*** hoangcx2 has joined #openstack-nova06:27
*** mjura has joined #openstack-nova06:27
*** hoangcx has quit IRC06:27
*** andreas_s has joined #openstack-nova06:28
openstackgerritRikimaru Honjo proposed openstack/nova: Nova shelve creates duplicated images in cells  https://review.openstack.org/32700106:29
*** ducttape_ has quit IRC06:29
*** lpetrut has joined #openstack-nova06:31
*** sridharg has joined #openstack-nova06:37
openstackgerritRikimaru Honjo proposed openstack/nova: Add checks for preventing unexpected detaching volume  https://review.openstack.org/35461706:39
*** ircuser-1 has quit IRC06:41
*** nkrinner_afk is now known as nkrinner06:45
*** sahid has joined #openstack-nova06:46
*** pcaruana has joined #openstack-nova06:49
*** thorst has quit IRC06:49
*** thorst has joined #openstack-nova06:50
*** thorst has quit IRC06:55
*** davidlenwell has quit IRC06:55
*** tesseract- has joined #openstack-nova07:00
*** gouthamr has quit IRC07:01
*** jichen has joined #openstack-nova07:02
*** gouthamr has joined #openstack-nova07:02
*** thorst has joined #openstack-nova07:02
*** gouthamr has quit IRC07:03
*** Sukhdev has quit IRC07:04
*** lpetrut has quit IRC07:07
*** thorst has quit IRC07:07
*** davidlenwell has joined #openstack-nova07:08
*** acabot has joined #openstack-nova07:08
*** markus_z has joined #openstack-nova07:09
*** jsheeren has joined #openstack-nova07:11
*** matrohon has joined #openstack-nova07:13
*** takashin_ has left #openstack-nova07:17
*** mgoddard has joined #openstack-nova07:20
*** hoangcx2 has quit IRC07:25
*** jlanoux has joined #openstack-nova07:25
*** ducttape_ has joined #openstack-nova07:26
*** hoangcx has joined #openstack-nova07:27
*** pilgrimstack has joined #openstack-nova07:27
*** liuyulong has quit IRC07:29
*** mnestratov has joined #openstack-nova07:30
*** ducttape_ has quit IRC07:31
*** liuyulong has joined #openstack-nova07:31
*** moshele has joined #openstack-nova07:37
*** hoangcx2 has joined #openstack-nova07:38
markus_zCould a 2nd core please look at this regression bug fix? https://review.openstack.org/#/c/335132/707:39
*** hoangcx has quit IRC07:39
*** Oku_OS-away is now known as Oku_OS07:40
*** kevinz has quit IRC07:41
prateekpkoniszewski, can you please tell me the reason why you abandoned this fix, i am planning to work on it so would need some context07:42
*** kevinz has joined #openstack-nova07:42
*** thorst has joined #openstack-nova07:49
*** jlanoux has quit IRC07:49
*** jlanoux has joined #openstack-nova07:49
*** ttx has quit IRC07:53
*** thorst has quit IRC07:53
pkoniszewskiprateek: which fix?07:54
prateekpkoniszewski, https://review.openstack.org/#/c/361174/07:54
*** thorst has joined #openstack-nova07:55
*** ttx has joined #openstack-nova07:56
*** ttx has quit IRC07:59
*** ttx has joined #openstack-nova07:59
*** thorst has quit IRC08:00
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** thorst has joined #openstack-nova08:02
pkoniszewskiprateek: it wasn't me who abandoned this fix08:02
*** roeyc has joined #openstack-nova08:02
*** thorst has quit IRC08:03
*** andymaier has joined #openstack-nova08:03
*** thorst has joined #openstack-nova08:03
pkoniszewskiprateek: alaski left good comment, it needs to be addressed and that's it, i think08:03
*** andymaier has quit IRC08:04
*** baoli has joined #openstack-nova08:05
*** danpb has joined #openstack-nova08:06
*** hoangcx2 has quit IRC08:06
*** hoangcx2 has joined #openstack-nova08:06
*** hoangcx has joined #openstack-nova08:10
*** hoangcx2 has quit IRC08:10
*** baoli has quit IRC08:10
BobBallmgagne: I suggest you email wg-openstack@lists.xenproject.org if there are issues with the XenProject CI08:11
*** lucas-afk is now known as lucasagomes08:17
*** Cristina_ has joined #openstack-nova08:19
*** Cristina_ has quit IRC08:20
*** aabashkin_ is now known as AlexeyAbashkin08:20
*** Cristina_ has joined #openstack-nova08:20
*** CristinaPauna has quit IRC08:22
*** AlexeyAbashkin has quit IRC08:22
*** palexster has joined #openstack-nova08:23
*** ducttape_ has joined #openstack-nova08:27
prateekpkoniszewski, ok, thanks08:28
*** thorst has quit IRC08:28
prateekpkoniszewski, i have a 1 + 2 devstack setup but while live migrating i get an error http://paste.openstack.org/show/572105/, can you let me know how do you get a setup ready for live migration ?08:29
*** salv-orl_ has joined #openstack-nova08:31
*** ducttape_ has quit IRC08:31
*** thorst has joined #openstack-nova08:32
*** mgoddard has quit IRC08:32
*** salv-orlando has quit IRC08:34
*** zenoway has joined #openstack-nova08:34
*** lpetrut has joined #openstack-nova08:42
*** yamahata has quit IRC08:43
*** efoley has joined #openstack-nova08:43
*** mgoddard has joined #openstack-nova08:47
*** jaosorior is now known as jaosorior_brb08:51
*** diga has quit IRC08:55
*** diga has joined #openstack-nova08:56
*** lixiaoy1 has quit IRC08:56
openstackgerritMaciej Szankin proposed openstack/nova: Fix opt indentation for scheduler.py  https://review.openstack.org/33973609:02
*** openstackgerrit has quit IRC09:04
*** openstackgerrit has joined #openstack-nova09:04
*** hoonetorg has joined #openstack-nova09:05
*** derekh has joined #openstack-nova09:05
openstackgerritStephen Finucane proposed openstack/nova: conf: Make list->dict conversion more specific  https://review.openstack.org/35933809:06
openstackgerritStephen Finucane proposed openstack/nova: objects: Move 'arch' to 'fields.Architecture'  https://review.openstack.org/35606709:06
openstackgerritStephen Finucane proposed openstack/nova: objects: Move 'vm_mode' to 'fields.VMMode'  https://review.openstack.org/35655309:06
openstackgerritStephen Finucane proposed openstack/nova: objects: Move 'hv_type' to 'fields.HVType'  https://review.openstack.org/35642409:06
openstackgerritStephen Finucane proposed openstack/nova: objects: Move 'cpumodel' to 'fields.CPU*'  https://review.openstack.org/35639809:06
*** abalutoiu has joined #openstack-nova09:07
*** mingyu has joined #openstack-nova09:07
*** hoangcx has quit IRC09:08
*** hoangcx has joined #openstack-nova09:09
openstackgerritStephen Finucane proposed openstack/nova: Add deprecated_since parameter  https://review.openstack.org/35363109:09
openstackgerritStephen Finucane proposed openstack/nova: hardware: Split out 'numa' and 'compat' modules  https://review.openstack.org/36746709:12
*** andymaier has joined #openstack-nova09:12
*** andymaier has quit IRC09:13
*** zz_dimtruck is now known as dimtruck09:13
openstackgerritLucian Petrut proposed openstack/nova: HyperV: ensure config drives are copied as well during resizes  https://review.openstack.org/36482909:17
*** bauwser is now known as bauzas09:17
*** ducttape_ has joined #openstack-nova09:27
*** houming has quit IRC09:31
*** ducttape_ has quit IRC09:32
*** jdurgin has quit IRC09:37
*** dimtruck is now known as zz_dimtruck09:37
*** ljxiash has quit IRC09:38
*** mfedosin has joined #openstack-nova09:43
*** mingyu_ has joined #openstack-nova09:44
*** mingyu has quit IRC09:46
openstackgerritQiming Teng proposed openstack/nova: [api-ref] fix availability_zone for server create  https://review.openstack.org/36868909:49
*** fishcried has joined #openstack-nova09:51
*** baoli has joined #openstack-nova09:54
*** karthiks has joined #openstack-nova09:57
*** hoangcx has quit IRC09:58
*** gszasz has joined #openstack-nova09:58
*** baoli has quit IRC09:58
*** mingyu has joined #openstack-nova09:59
openstackgerritAndrey Volkov proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755709:59
openstackgerritAndrey Volkov proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937909:59
johnthetubaguypkoniszewski: just looking at your comment on this one https://review.openstack.org/#/c/358599/10:01
johnthetubaguypkoniszewski: did you find Mitaka -> Newton working OK?10:02
*** mingyu_ has quit IRC10:02
*** mingyu has quit IRC10:02
*** mingyu has joined #openstack-nova10:02
*** thorst has quit IRC10:02
pkoniszewskijohnthetubaguy: yeah, but i just found out that there is one more pitfall10:04
pkoniszewskijohnthetubaguy: it introduces another regression for mitaka -> newton, because newton will populate the data during check_can_live_migrate_destination, but the actuall check will never happen10:05
pkoniszewskijohnthetubaguy: it mitaka it is part of pre_live_migration which won't be executed (mitaka is source side, pre_live_migration is executed on destination)10:05
pkoniszewskijohnthetubaguy: in newton the check is in check_can_live_migrate_source, but Mitaka is an old release that does not support such check in this step10:06
pkoniszewskijohnthetubaguy: so actually we are broken in both cases10:06
*** jaosorior_brb is now known as jaosorior10:06
johnthetubaguyoh, right, we can't go back in time to add the data for the new check10:06
*** mingyu has quit IRC10:07
johnthetubaguyif we had to chose only one that worked, I would choose mitaka->newton FWIW10:07
johnthetubaguywe totally need live-migrate adding into grenade, now we have at least one setup stable, although not 100% sure if that would hit this10:08
*** thorst_ has joined #openstack-nova10:11
*** thorst_ has quit IRC10:15
*** fragatin_ has joined #openstack-nova10:15
*** thorst has joined #openstack-nova10:17
pkoniszewskijohnthetubaguy: with current configuration we wouldn't hit this issue, we have default graphics configuration so that, e.g., vnc listen address is set to localhost10:18
pkoniszewskiwonder if we can change it10:18
*** fragatina has quit IRC10:19
pkoniszewskii'm trying to run LM job with grenade, perhaphs I need to focus a bit more on this https://review.openstack.org/#/c/364809/ and add some tests that live migrate VMs back and forth10:20
*** kaisers has quit IRC10:20
johnthetubaguypkoniszewski: agreed with all that, I thought we already had patches up that modified the existing job to add the back and forth bit in, may have been abandonded.10:21
pkoniszewskijohnthetubaguy: I think we don't need to choose anything, we can still move the check back to pre_live_migration as this fix is not critical for running live migrations and then move it once again in ocata10:21
pkoniszewskii'm not aware of any patches, talked with tdurakov and he wasn't aware of any too10:21
*** thorst has quit IRC10:21
*** markvoelker has joined #openstack-nova10:22
johnthetubaguypkoniszewski: I think dansmith and mriedem had something for when we added the objectification stuff in10:22
*** thorst has joined #openstack-nova10:23
johnthetubaguypkoniszewski: yeah, basically revert the patch you reference does seem the best way forward10:23
sfinucandanpb: are these able to be abandoned now? they're clogging up my numa/libvirt review dash :) https://review.openstack.org/#/q/topic:mig-debug10:23
johnthetubaguypkoniszewski: and leave the early populate in so we can do that change next time10:23
sfinucan...amazing as the commit messages are :)10:23
*** jsheeren has quit IRC10:24
*** jvgrant_ has quit IRC10:26
pkoniszewskijohnthetubaguy: agree, i think it's the best we can do to solve the issue and prepare for moving the code in Ocata10:26
*** markvoelker has quit IRC10:26
*** jvgrant has joined #openstack-nova10:26
*** thorst has quit IRC10:27
kashyapsfinucan: This was the key commit message -- https://review.openstack.org/#/c/335549/ , and the nasty bug it was trying to debug isn't still fixed yet - https://bugs.launchpad.net/nova/+bug/152489810:32
openstackLaunchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed]10:32
*** kaisers has joined #openstack-nova10:33
kashyapsfinucan: Not implying they shouldn't be abandoned (despite the great commit art).  :-)10:33
*** namnh has joined #openstack-nova10:33
danpbsfinucan: sure10:33
namnhHi everyone. Now I would like to run command for expand DB schema in Nova. Do you know the command to do this10:35
namnh?10:35
*** shangxdy has joined #openstack-nova10:37
*** shangxdy has quit IRC10:39
namnhI found the "nova-manage db sync" command. But IMO. this command will run three options including: expand, migration, contract. Do you know how to separate three one?10:39
*** GB21 has quit IRC10:40
rpodolyakanamnh: IIUC, separate steps were removed in https://review.openstack.org/#/c/239922/10:41
rpodolyakanamnh: so you apply the changes to the DB schema by the means of "nova-manage db sync"10:42
johnthetubaguypkoniszewski: I might try work on that patch now, if that helps?10:42
*** EinstCrazy has quit IRC10:42
rpodolyakanamnh: and you have a separate step to force online data migrations - https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L90610:42
rpodolyakanamnh: which otherwise would be performed by nova-conductor when the corresponding rows are retrieved from the DB10:43
johnthetubaguynamnh: +1 rpodolyaka, basically we only do operations in "db sync" that can happen while the previous release is still running, we delay the contract till the following release, basically10:44
*** shangxdy has joined #openstack-nova10:45
johnthetubaguynamnh: I am looking at getting some better in tree docs around that, need to check with raj_singh about who is going to be working on that10:45
*** code-R has joined #openstack-nova10:45
*** mingyu has joined #openstack-nova10:46
*** rodrigods has quit IRC10:47
*** code-R_ has joined #openstack-nova10:47
*** rodrigods has joined #openstack-nova10:47
namnhrpodolyaka, thanks for your reply. It means that when I want to "expand to DB, I just need to run the "nova-manager db sync". So when I upgrade nova-conductor then what should I run the command?10:47
rpodolyakanamnh: you run db sync first, then you upgrade nova-conductor (and restart) - that's it10:48
rpodolyakanamnh: then when you upgrade to the *next* OS release you *may need* to force online data migrations10:49
rpodolyakaso that all rows are migrated properly10:49
johnthetubaguyhmm, you should force the online data migrations after you complete your upgrade, ideally10:49
*** tbachman has quit IRC10:49
openstackgerritliuxiuli proposed openstack/nova-specs: Clearing memory will be needed after deleting vm  https://review.openstack.org/34315010:49
johnthetubaguywell, I mean after all the nodes have been upgraded and restarted10:50
rpodolyakasure10:50
*** code-R has quit IRC10:50
rpodolyakajust not all rows at once :) not sure what limit we default to10:50
*** jsheeren has joined #openstack-nova10:51
johnthetubaguywe default to all, but if you have a decent sized cloud, thats not what you want10:51
namnhrpodolyaka, johnthetubaguy: Let me send you the link: http://docs.openstack.org/developer/nova/upgrade.html#process10:51
johnthetubaguyyeah, thats the doc I was saying I want to get updated10:51
johnthetubaguygoing to work with raj_singh to get that sorted sometime this week, with a bit of luck10:52
*** diogogmt_ has joined #openstack-nova10:52
*** cdent has joined #openstack-nova10:52
*** gongysh has joined #openstack-nova10:52
openstackgerritJohn Garbutt proposed openstack/nova: Revert "libvirt: move graphic/serial consoles check to pre_live_migration"  https://review.openstack.org/36873210:52
namnhrpodolyaka, johnthetubaguy: Could you please tell me at step 2 and step 10. What command should I run?10:53
*** smatzek has joined #openstack-nova10:53
*** claudiub has quit IRC10:53
johnthetubaguynamnh: that whole process is wrong10:53
*** diogogmt has quit IRC10:53
*** diogogmt_ is now known as diogogmt10:53
johnthetubaguynamnh: see the note, its more what we want to happen eventually10:54
johnthetubaguynamnh: step 10 no longer exists10:54
*** baoli has joined #openstack-nova10:54
*** gongysh has quit IRC10:56
namnhjohnthetubaguy, oh. maybe I am wrong but i think the steps are right. Currently, I have three controllers node. That mean I need to expand DB schema first then I will upgrade nova-conductor on each controller. After I finished to upgrade on three node, then I need to contract DB.10:58
*** baoli has quit IRC10:59
openstackgerritDao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console()  https://review.openstack.org/32815710:59
johnthetubaguynamnh: the steps are wrong, we don't contract the DB10:59
namnhjohnthetubaguy, But: we can't upgrade nova-conductor on three controller node at the same time. I have to upgrade sequently. So I think we need to use "expand" first then upgrade nova-conductor on three node. Finally we will migrade DB then use "contract"11:02
namnhIf we run "contract" fist. we will be confict code.11:03
*** hoonetorg has quit IRC11:03
*** tuanla_ has quit IRC11:03
namnhjohnthetubaguy, I mean that. How do you think? Regrading to your option, now we did not have the commnd to use "contract"11:04
namnh?11:04
johnthetubaguynamnh: so the current tested process is something like: "nova db sync; nova api_db sync", turn off all controllers, turn on new controller code, update nova-compute nodes, "nova db online_data_migrations"11:04
johnthetubaguynamnh: we don't have a contract comand11:04
openstackgerritPrateek Arora proposed openstack/nova: Delete traces of in-progress snapshot on VM being deleted  https://review.openstack.org/29451311:05
*** kaisers has quit IRC11:06
johnthetubaguynamnh: well, we did have for one release, but it was untested and marked as experimental for that release, then removed in the following release, you only need to run "db sync" to the expand (it does contracts from previous releases at the same time, if needed)11:06
*** rtheis has joined #openstack-nova11:06
openstackgerritliuxiuli proposed openstack/nova-specs: Clearing memory will be needed after deleting vm  https://review.openstack.org/34315011:07
*** claudiub has joined #openstack-nova11:09
*** mingyu has quit IRC11:09
*** cdelatte has joined #openstack-nova11:10
*** GB21 has joined #openstack-nova11:10
namnhjohnthetubaguy, well.. as your process. It run on one controller or multi controller?11:11
namnhcurrelty, I am three controller active-active. I am trying to upgrade Nova without downtime.11:11
johnthetubaguynamnh: you have to turn off all three controllers, then turn on the new code on all three controllers11:11
johnthetubaguynamnh: right, we don't support that today11:11
johnthetubaguynamnh: the plan in the future is to upgrade the nova-api nodes last, using a loadbalancer to drain the connections between the old and new api nodes, but thats still a work in progress11:12
*** liusheng has joined #openstack-nova11:18
*** daemontool has joined #openstack-nova11:19
namnhjohnthetubaguy, thanks Mr.Garbutt. Surely. I will think this again.11:20
*** code-R_ has quit IRC11:20
namnhjohnthetubaguy, Now I have to go out my office. it's so late. But tomorrow. could I discuss this problem with you again?11:21
*** bvanhav has joined #openstack-nova11:22
johnthetubaguynamnh: yes, I should be around most UK hours11:24
*** code-R has joined #openstack-nova11:25
*** lucasagomes is now known as lucas-hungry11:25
namnhjohnthetubaguy, see you. thanks for your time. :)11:27
*** namnh has quit IRC11:27
*** thorst has joined #openstack-nova11:29
*** ducttape_ has joined #openstack-nova11:29
*** baoli has joined #openstack-nova11:29
*** thorst has quit IRC11:32
*** baoli has quit IRC11:33
*** ducttape_ has quit IRC11:33
*** sdague has joined #openstack-nova11:34
*** dixiaoli has joined #openstack-nova11:35
dixiaolihi11:35
openstackgerritjichenjc proposed openstack/nova: Refactored query user_id code for quota-sets  https://review.openstack.org/28016511:38
*** thorst has joined #openstack-nova11:38
*** baoli has joined #openstack-nova11:39
dixiaoliDid anyone meet the following error when attaching volumes when using ceph backend for cinder ?11:39
dixiaoli libvirtError: internal error: unable to execute QEMU command 'device_add': Property 'virtio-blk-device.drive' can't find value 'drive-virtio-disk1'11:40
dixiaoliAnd I am using libvirt 1.3.1-1ubuntu10 and qemu 1:2.5+dfsg-5ubuntu10.411:40
dixiaoliThanks a lot ~~11:40
dixiaoliThe detailed nova-compute.log is here:  http://paste.openstack.org/show/572189/11:41
*** thorst has quit IRC11:43
*** code-R_ has joined #openstack-nova11:50
*** baoli_ has joined #openstack-nova11:50
*** baoli has quit IRC11:53
*** code-R has quit IRC11:53
*** daemontool has quit IRC11:56
*** thorst has joined #openstack-nova11:56
*** lbeliveau has joined #openstack-nova11:56
*** eharney has quit IRC11:57
*** thorst has quit IRC12:00
*** Jeffrey4l_ has joined #openstack-nova12:01
*** Jeffrey4l_ has quit IRC12:01
*** Jeffrey4l_ has joined #openstack-nova12:01
*** markvoelker has joined #openstack-nova12:02
*** lamt has quit IRC12:03
*** Jeffrey4l_ has quit IRC12:05
*** daemontool has joined #openstack-nova12:05
*** jaypipes has joined #openstack-nova12:07
*** hshiina has joined #openstack-nova12:08
*** panbalag has joined #openstack-nova12:09
*** mingyu has joined #openstack-nova12:10
openstackgerritMaciej Szankin proposed openstack/nova: Deprecate all topic conf options  https://review.openstack.org/35160812:11
kashyapdixiaoli: Seems like your issue is already fixed here -- https://bugs.launchpad.net/ubuntu/+source/qemu/+bug/149589512:12
openstackLaunchpad bug 1495895 in qemu (Ubuntu) "Unable to attach rados block device to instances" [High,Fix released]12:12
openstackgerritStephen Finucane proposed openstack/nova: hardware: Rework docstrings  https://review.openstack.org/36746812:13
openstackgerritStephen Finucane proposed openstack/nova: hardware: Standardize '_get_XXX_constraints' functions  https://review.openstack.org/36746912:13
openstackgerritStephen Finucane proposed openstack/nova: hardware: Group functions by broader function  https://review.openstack.org/36814412:13
openstackgerritStephen Finucane proposed openstack/nova: hardware: Flatten functions  https://review.openstack.org/36747012:13
openstackgerritStephen Finucane proposed openstack/nova: hardware: Split out 'numa' and 'compat' modules  https://review.openstack.org/36746712:13
*** panbalag has quit IRC12:14
*** mingyu has quit IRC12:14
openstackgerritMaciej Szankin proposed openstack/nova: Improve consistency in serial_console  https://review.openstack.org/34890212:16
*** pradk has joined #openstack-nova12:16
*** diga has quit IRC12:17
*** panbalag has joined #openstack-nova12:17
*** ducttape_ has joined #openstack-nova12:19
*** thorst has joined #openstack-nova12:20
openstackgerritMaciej Szankin proposed openstack/nova: Config options: improve libvirt help text (2)  https://review.openstack.org/31655012:21
openstackgerritMaciej Szankin proposed openstack/nova: Improve consistency in libvirt  https://review.openstack.org/35367912:21
*** thorst has quit IRC12:21
*** pbandark has quit IRC12:21
*** panbalag has quit IRC12:22
*** coolsvap is now known as _coolsvap_12:22
*** fragatin_ has quit IRC12:23
openstackgerritStephen Finucane proposed openstack/nova: conf: Add 'deprecated_reason' to osapi opts  https://review.openstack.org/35706112:23
dixiaolikashyap, thanks for your information, this bug fixed in release 2.3, but I am using version 2.5 and still meeting it. :(12:25
kashyapdixiaoli: Hmm, sorry, haven't dug into it deeply (but I do see another unfixed bug -- https://bugs.launchpad.net/nova/+bug/1502028).  You might get a better response on the operators list or general OpenStack list.  As this channel concerns mostly with Nova development12:26
openstackLaunchpad bug 1502028 in OpenStack Compute (nova) "cannot attach a volume when using multiple ceph backends" [Low,Confirmed] - Assigned to Kevin Zhao (kevin-zhao)12:26
*** panbalag has joined #openstack-nova12:26
*** Cristina_ is now known as CristinaPauna12:27
dixiaolikashyap, I tried to stop the instance, and then attach the volume, it can be attached successfully. But when I tried to start the instance , I met the following error: libvirtError: Failed to connect socket to '/var/run/libvirt/libvirt-sock': Connection refused12:27
*** daemontool has quit IRC12:27
*** thorst has joined #openstack-nova12:27
*** daemontool has joined #openstack-nova12:28
dixiaolikashyap, ok, thanks12:28
openstackgerritMaciej Szankin proposed openstack/nova: Improve consistency in libvirt  https://review.openstack.org/35367912:28
*** lucas-hungry is now known as lucasagomes12:29
openstackgerritMaciej Szankin proposed openstack/nova: Improve help text for glance options  https://review.openstack.org/32400712:29
*** tbachman has joined #openstack-nova12:29
*** roeyc has quit IRC12:31
pkoniszewskialaski: hi, i think that the change you just +Wed should wait a bit, probably we will want to move this check (of serial consoles and graphic addresses) back to live_migration_operation12:32
*** thorst has quit IRC12:32
pkoniszewskialaski: i mean this one https://review.openstack.org/#/c/335132/12:32
pkoniszewskialaski: here is revert https://review.openstack.org/#/c/368732/112:32
*** prateek has quit IRC12:32
*** garyk has joined #openstack-nova12:33
pkoniszewskialaski: and here is why https://review.openstack.org/#/c/35859912:33
johnthetubaguyyeah, I am just starting to rebase that12:33
alaskireading12:34
*** thorst has joined #openstack-nova12:36
openstackgerritdane-fichter proposed openstack/nova: Use cursive for signature verification  https://review.openstack.org/35123212:38
*** garyk has quit IRC12:39
openstackgerritMaciej Szankin proposed openstack/nova: Consider deprecating isolated_images  https://review.openstack.org/27135412:39
*** mmedvede has quit IRC12:42
openstackgerritAndrew Laski proposed openstack/nova: libvirt: fix live-migration with serial console check  https://review.openstack.org/33513212:42
*** artom has joined #openstack-nova12:43
*** ducttape_ has quit IRC12:44
alaskipkoniszewski: johnthetubaguy okay, I kicked that patch out of the queue12:44
pkoniszewskialaski: thanks! it will make it easier to fix backward compatibility regression12:45
alaskithe issue with N->M seems difficult to address without either reverting for now, or backporting something to M12:45
pkoniszewskiso i think that we need two releases to really move the check to check_can_live_migrate_source, doesnt look like we have any other option12:46
alaskiwell, we could backport the changes so that the check and data population are handled properly in M. But I don't think it's appropriate to do so12:47
*** pradk has quit IRC12:47
*** pbandark has joined #openstack-nova12:49
*** vladikr has joined #openstack-nova12:52
*** artom has quit IRC12:53
*** cleong has joined #openstack-nova12:54
*** jaosorior has quit IRC12:56
*** shangxdy has quit IRC12:56
*** jaosorior has joined #openstack-nova12:57
*** mingyu has joined #openstack-nova12:58
*** artom has joined #openstack-nova12:58
*** Jeffrey4l has quit IRC12:59
*** READ10 has joined #openstack-nova12:59
*** Jeffrey4l has joined #openstack-nova12:59
openstackgerritMaciej Szankin proposed openstack/nova: Remove redundant check from code  https://review.openstack.org/35012213:01
*** hshiina has quit IRC13:01
*** karthiks has quit IRC13:04
*** matrohon has quit IRC13:04
openstackgerritMaciej Szankin proposed openstack/nova: Add related options to floating ip config options  https://review.openstack.org/35556113:04
*** oanson has quit IRC13:05
*** mdrabe has joined #openstack-nova13:07
*** rlrossit has joined #openstack-nova13:08
*** mingyu has quit IRC13:08
*** mfedosin has quit IRC13:11
*** sridharg has quit IRC13:12
*** pwnall has joined #openstack-nova13:13
*** pwnall is now known as Guest4130613:13
*** mingyu has joined #openstack-nova13:14
*** eharney has joined #openstack-nova13:15
alex_xujaypipes: sorry, I mean PUT_INVENTORY_RECORD_SCHEMA['properties'].pop("resource_provider_generation")13:16
*** lamt has joined #openstack-nova13:16
*** zz_dimtruck is now known as dimtruck13:17
*** zenoway has quit IRC13:17
*** zenoway has joined #openstack-nova13:18
*** Guest41306 has quit IRC13:18
*** matrohon has joined #openstack-nova13:18
*** vipuls has quit IRC13:18
*** mmedvede has joined #openstack-nova13:19
*** openstackgerrit has quit IRC13:19
*** mmedvede has quit IRC13:19
*** openstackgerrit has joined #openstack-nova13:19
*** mmedvede has joined #openstack-nova13:20
*** vipul has joined #openstack-nova13:20
openstackgerritTuan Luong-Anh proposed openstack/nova: Using assertIsNotNone() instead of assertIsNot(None,)  https://review.openstack.org/36181513:22
*** hferenc has quit IRC13:23
*** kaisers has joined #openstack-nova13:25
*** ducttape_ has joined #openstack-nova13:26
*** zenoway has quit IRC13:27
*** akshai has joined #openstack-nova13:27
*** moshele has quit IRC13:28
*** annegentle has joined #openstack-nova13:29
edleafealex_xu: jaypipes is only removing the 'required' nature of the generation. It can optionally be present13:30
*** EinstCrazy has joined #openstack-nova13:31
alex_xuedleafe: why we have generation at toplevel, then also have generation for each inventory?13:31
*** smatzek has quit IRC13:32
*** artom has quit IRC13:32
edleafealex_xu: not sure what you mean. What toplevel?13:33
*** artom has joined #openstack-nova13:34
*** gabor_antal_ has joined #openstack-nova13:34
*** gabor_antal has quit IRC13:34
alex_xuedleafe: https://review.openstack.org/#/c/368035/4/nova/api/openstack/placement/handlers/inventory.py@69 we have resource_provider_generation at there. Then we also have resource_provider_generation at https://review.openstack.org/#/c/368035/4/nova/api/openstack/placement/handlers/inventory.py@29 for each inventory13:35
*** hoangcx has joined #openstack-nova13:35
*** hshiina has joined #openstack-nova13:35
*** takedakn has joined #openstack-nova13:36
*** annegentle has quit IRC13:36
*** zenoway has joined #openstack-nova13:36
*** lamt has quit IRC13:36
*** annegentle has joined #openstack-nova13:36
*** tbachman has quit IRC13:37
*** mfedosin has joined #openstack-nova13:37
*** vishwanathj has joined #openstack-nova13:39
edleafealex_xu: ah, I see what you mean. Let me double-check against the GET patterns13:39
*** tblakes has joined #openstack-nova13:40
*** vishwanathj has quit IRC13:40
*** ducttape_ has quit IRC13:40
openstackgerritDan Smith proposed openstack/nova: More conservative inventory updates  https://review.openstack.org/36507313:40
*** vishwanathj has joined #openstack-nova13:41
*** rfolco has joined #openstack-nova13:41
*** lamt has joined #openstack-nova13:43
*** rlrossit has quit IRC13:45
*** ociuhandu has joined #openstack-nova13:46
*** mingyu has quit IRC13:49
*** hoangcx has quit IRC13:51
*** hoangcx has joined #openstack-nova13:52
*** mriedem has joined #openstack-nova13:53
*** ametts has joined #openstack-nova13:54
*** pradk has joined #openstack-nova13:54
*** smatzek has joined #openstack-nova13:55
*** woodster_ has joined #openstack-nova13:55
*** smatzek_ has joined #openstack-nova13:56
*** jaypipes has quit IRC13:56
*** marst has quit IRC13:56
*** tonytan_brb has joined #openstack-nova13:57
*** hoangcx2 has joined #openstack-nova13:57
edleafeScheduler subteam meeting in 2 minutes in #openstack-meeting-alt13:58
mriedemdanpb: for the warning in this bug https://bugs.launchpad.net/nova/+bug/1622270 - is that something in the host we can check for to ignore the warning?13:58
openstackLaunchpad bug 1622270 in OpenStack Compute (nova) "libvirt: excessive warning logs like "couldn't obtain the vcpu count from domain id: 6bcf0dcb-722f-44a3-a0aa-3fa42dd1075a, exception: Requested operation is not valid: cpu affinity is not supported"" [Medium,Confirmed]13:58
*** hoangcx has quit IRC13:59
danpbsounds like nova's trying to get/set cpu affinity for a QEMU TCG guest, when it should onyl do that for KVM guests13:59
danpbthat's a pretty clear bug in nova to be doing that13:59
danpb(at least until QEMU supports multi-threaded TCG which is still a way off)13:59
dansmithcdent: after this: https://review.openstack.org/#/c/366789/14:00
*** tonytan4ever has quit IRC14:00
*** smatzek has quit IRC14:00
mriedemdanpb: yeah it's this https://github.com/openstack/nova/blob/e92d7537d67c7eb0e49415b429d56be762e02679/nova/virt/libvirt/driver.py#L492214:00
*** anush__ has joined #openstack-nova14:00
cdentdansmith: hello, yes?14:00
*** BjoernT has joined #openstack-nova14:00
dansmithcdent: we need to modify my graceful allocations patch, yes? has anyone started on that that you know of?14:00
*** artom has quit IRC14:01
danpbmriedem: hmm, that's dubious for many reasons14:01
mriedemdanpb: so we should skip that if not CONF.libvirt.virt_type == 'kvm'?14:01
cdentdansmith: I think I already did it on saturday if I'm understanding you: https://review.openstack.org/#/c/366872/14:01
danpbmriedem: i wonder what the caller is actually expecting there14:01
danpbmriedem: as currently that code is counting TCG guests as having 0 vcpus !14:02
cdentdansmith: but I think it needs some checking over by you because I'm not certain I entirely understood your unit tests14:02
dansmithcdent: oh that's what you emailed me about.. I read "fixed the merge conflict" and marked as read14:02
bauzasedleafe: no scheduler meeting ?14:02
dansmithcdent: okay14:02
danpbmriedem: if the caller wants vCPU count, then we should be returning vCPU count regardless of tcg vs kvm14:02
danpbmriedem: if the caller actually wants the number of host pCPUs used by the guest14:02
mriedemdanpb: it's the RT https://github.com/openstack/nova/blob/e92d7537d67c7eb0e49415b429d56be762e02679/nova/virt/libvirt/driver.py#L527214:02
danpbmriedem: then the except: block needs to add 114:02
edleafebauzas: waiting for the previous meeting to end14:02
*** tongli has joined #openstack-nova14:03
danpband god knows what the lxc code block is returning a fixed value of 114:03
*** paul-carlton2 has quit IRC14:03
danpbmriedem: ok next question is what is desired semantics of the 'vcpus_used' attribute for RT14:03
danpbi'm thinking the RT is really after the number of pCPUs used by the guest14:04
*** prateek has joined #openstack-nova14:07
mriedemdanpb: not sure if the RT is using this or if it's something like the os-hypervisors API14:07
*** kevinz has quit IRC14:07
mriedemmorning meetings just started so i'm distracted now14:07
danpbanyway, regardless, the currently logic is broken14:07
danpband we should not be spamming the logs14:08
*** efoley has quit IRC14:08
*** efoley has joined #openstack-nova14:08
*** marst has joined #openstack-nova14:09
*** marst has quit IRC14:09
*** rwmjones has quit IRC14:09
*** marst has joined #openstack-nova14:09
*** maticue has joined #openstack-nova14:09
openstackgerritPrateek Arora proposed openstack/nova: Delete traces of in-progress snapshot on VM being deleted  https://review.openstack.org/29451314:10
mriedemdanpb: ok yeah that's what my bug was for initially, the log spamming14:10
mriedemi just wasn't sure if it was safe to downgrade that or if we had to make a conditional check14:10
openstackgerritTakashi NATSUME proposed openstack/nova: Fix MonitorMetric obj_make_compatible  https://review.openstack.org/36179214:10
*** mdrabe has quit IRC14:10
*** pwnall has joined #openstack-nova14:11
*** pwnall is now known as Guest9703014:11
*** ducttape_ has joined #openstack-nova14:12
*** sc68cal_ is now known as sc68cal14:14
*** dimtruck is now known as zz_dimtruck14:14
*** mlavalle has joined #openstack-nova14:16
*** mdrabe has joined #openstack-nova14:16
*** burt has quit IRC14:17
*** ftersin has joined #openstack-nova14:18
*** burt has joined #openstack-nova14:20
*** zenoway has quit IRC14:20
*** bhagyashri has joined #openstack-nova14:20
*** zenoway has joined #openstack-nova14:21
*** pbandark has quit IRC14:21
*** nherciu has quit IRC14:23
bhagyashrisdague: Hi, could you please review https://review.openstack.org/#/c/345191/ ?14:23
*** zenoway has quit IRC14:24
*** zenoway has joined #openstack-nova14:25
*** bhagyashri has quit IRC14:25
*** kaisers has quit IRC14:25
*** igordcard has quit IRC14:26
*** mingyu has joined #openstack-nova14:26
*** igordcard has joined #openstack-nova14:26
*** BjoernT has quit IRC14:26
*** anush___ has joined #openstack-nova14:28
*** anush__ has quit IRC14:28
*** moshele has joined #openstack-nova14:29
*** salv-orlando has joined #openstack-nova14:30
openstackgerritJohn Garbutt proposed openstack/nova: Revert "libvirt: move graphic/serial consoles check to pre_live_migration"  https://review.openstack.org/36873214:32
openstackgerritJohn Garbutt proposed openstack/nova: Fill destination check data with VNC/SPICE listen addresses  https://review.openstack.org/35859914:32
*** salv-orl_ has quit IRC14:33
johnthetubaguypkoniszewski: does that stack of patch look like what you were expecting now? https://review.openstack.org/#/c/35859914:34
*** akshai has quit IRC14:37
*** shangxdy has joined #openstack-nova14:37
*** mvk has quit IRC14:37
*** akshai has joined #openstack-nova14:38
*** hoonetorg has joined #openstack-nova14:38
*** annegentle has quit IRC14:39
*** tblakes has quit IRC14:40
*** zenoway has quit IRC14:40
*** zenoway has joined #openstack-nova14:41
*** kevinz has joined #openstack-nova14:41
mriedemcdent: any idea why jay +2ed his own change here? https://review.openstack.org/#/c/368035/14:42
*** burgerk has joined #openstack-nova14:42
openstackgerritianeta hutchinson proposed openstack/nova: Updates URL and removes trailing characters  https://review.openstack.org/36818514:42
*** rlrossit has joined #openstack-nova14:42
dansmithmriedem: it's also completely broken.. see my -1 just now14:43
*** beekneemech is now known as bnemec14:43
cdentmriedem: not entirely sure, but on patchset2 there was some weirdness with some old code being brought back in, which I fixed and I also fixed the bug that was present in ps1 and improved a test. So I guess he was saying "i agree with what chris did"14:43
*** artom has joined #openstack-nova14:44
*** BjoernT has joined #openstack-nova14:44
*** prateek_ has joined #openstack-nova14:44
cdentdansmith: I think it isn't based on anything that would make it do stuff14:44
cdentnot sure14:45
*** tonytan_brb is now known as tonytan4ever14:45
dansmithcdent: eh? the report client is now PUTting invalid data because he changed what he accepts on the server side14:45
*** zenoway has quit IRC14:45
*** zz_dimtruck is now known as dimtruck14:45
cdentdansmith: that's what I mean: _all_ it does is change the representation not anything that uses that representation14:45
dansmithyeah14:46
cdentso in terms of tests, it works. we need some kind of functional test for the rt side, probalby sooner than later14:46
*** pbandark has joined #openstack-nova14:46
*** mdrabe has quit IRC14:46
cdentdansmith: I would guess the immediate thing to do is base your changes on it?14:46
*** mdrabe has joined #openstack-nova14:46
dansmithcdent: I have to fix it first, then make my changes work on top, which I'm doing now14:47
dansmithcdent: yeah, we really need functional tests for this14:47
*** prateek has quit IRC14:47
*** burgerk_ has joined #openstack-nova14:49
cdenteveryone is all confused by the last minute changes we've been rushing (see jay's comment and my response at the end of) https://review.openstack.org/#/c/365073/14:50
*** TravT_ has quit IRC14:51
*** burgerk has quit IRC14:52
*** annegentle has joined #openstack-nova14:53
cdentdansmith: if you can point me at some existing functional tests that exercise the resource tracker I can see if I can figure something out?14:54
dansmithcdent: we don't have functional tests to directly hit the RT of course,14:56
dansmithbut all we really need is functional tests for the actual placement stuff14:56
*** yamahata has joined #openstack-nova14:56
dansmithi.e. use the scheduler report client to actually talk to a real placement endpoint and makes sure they speak the same language14:56
*** _ducttape_ has joined #openstack-nova14:56
* cdent thinks on these and other things14:57
*** ducttape_ has quit IRC15:00
*** rrecio has joined #openstack-nova15:00
*** mgoddard_ has joined #openstack-nova15:00
*** lucasagomes is now known as lucas-afk15:02
*** mjura has quit IRC15:03
*** jgriffith has quit IRC15:03
*** mgoddard has quit IRC15:04
*** jsheeren has quit IRC15:05
*** mingyu has quit IRC15:05
*** mingyu has joined #openstack-nova15:06
*** TravT has joined #openstack-nova15:06
*** daemontool_ has joined #openstack-nova15:07
*** lamt has quit IRC15:07
*** gouthamr has joined #openstack-nova15:09
*** daemontool has quit IRC15:10
*** rwmjones has joined #openstack-nova15:11
*** zenoway has joined #openstack-nova15:11
*** hshiina has quit IRC15:11
*** zul has joined #openstack-nova15:12
*** armax has joined #openstack-nova15:12
*** kevinz has quit IRC15:14
openstackgerritDan Smith proposed openstack/nova: More conservative inventory updates  https://review.openstack.org/36507315:14
openstackgerritDan Smith proposed openstack/nova: [placement] make PUT inventory consistent with GET  https://review.openstack.org/36803515:14
*** pbandark has quit IRC15:15
dansmithcdent: updated while on a call, will let it run devstack while I finish and see if it looks good ^15:15
*** kevinz has joined #openstack-nova15:15
cdent✔15:15
*** unicell has joined #openstack-nova15:16
*** moshele has quit IRC15:17
*** tblakes has joined #openstack-nova15:17
*** hemnafk has quit IRC15:18
mlavallecdent: thanks for the heads up..... I'll keep my head up watching the evolution of inventories and allocations ;-)15:19
*** unicell1 has quit IRC15:19
cdentmlavalle: it should settle out soon15:19
*** nkrinner is now known as nkrinner_afk15:20
*** flaper87 has quit IRC15:21
raj_singhjohnthetubaguy: Can you please review https://review.openstack.org/#/c/332861?15:23
*** lamt has joined #openstack-nova15:23
cfriesenAPI microversion question....microversion 2.25 requires all nodes to be running Mitaka.  Are there any other microversions for which the same is true?15:24
sdaguecfriesen: I don't think so15:24
*** moshele has joined #openstack-nova15:24
*** matrohon has quit IRC15:25
*** harlowja_at_home has joined #openstack-nova15:27
*** jheroux has joined #openstack-nova15:27
*** hemna has joined #openstack-nova15:29
*** harlowja_still_a has joined #openstack-nova15:29
*** harlowja_still_a has quit IRC15:30
*** abalutoiu has quit IRC15:31
*** adisky has quit IRC15:31
openstackgerritDan Smith proposed openstack/nova: More conservative inventory updates  https://review.openstack.org/36507315:32
*** jichen has quit IRC15:32
cfriesensdague: thanks15:33
*** jichen has joined #openstack-nova15:33
*** liangy has joined #openstack-nova15:33
*** EinstCrazy has quit IRC15:33
*** anush___ has quit IRC15:34
sdaguemostly that's an indication that the api change also requires something of the rpc api15:34
*** ircuser-1 has joined #openstack-nova15:35
*** Jeffrey4l has quit IRC15:35
*** andreas_s has quit IRC15:35
*** Jeffrey4l has joined #openstack-nova15:36
*** kaisers has joined #openstack-nova15:40
mriedemdanpb: on https://review.openstack.org/#/c/367909/ are you agreeing to make pointer_model=usbtablet by default to match the default of use_usb_tablet=True?15:43
mriedemsahid: ^15:43
danpbmriedem: yes15:43
mriedemdanpb: ok, i agree15:43
*** code-R_ has quit IRC15:43
mriedemsahid: are you able to make that change yet today?15:43
danpbmriedem: but also switch use_usb_tablet to None by default15:43
mriedemdanpb: it's a boolean so you can't15:43
danpbwell makeit False then - whatever we have todo so that it does not override pointer_model out of the box15:44
sahidthat change has to be backported, right ? how can we change the behavior of that option ?15:44
mriedemwe wouldn't backport it15:45
openstackgerritPrateek Arora proposed openstack/nova: Delete traces of in-progress snapshot on VM being deleted  https://review.openstack.org/29451315:45
mriedemif pointer_model=usbtablet and use_usb_tablet=True by default, then i think we're fine15:45
mriedemif you don't want usb, you'd set pointer_model=None and use_usb_tablet=False right?15:46
sahidhum that seems weird to use that depreacted option in same time of the new option, no ?15:47
mriedemi'd prefer to change the default on the new 'unreleased' option rather than the old existing but deprecated option15:47
mriedemunreleased for distros, not CD people15:48
danpbmriedem: having to switch 2 options to disable USB is crazy15:48
mriedemit's crazy for <1 week15:48
danpbmriedem: which is why i suggested making use_usb_table=None or False, by default, so it doesn't override pointer_model15:48
*** nherciu has joined #openstack-nova15:48
danpbmriedem: for how ever many years people deploy on Newton for15:48
mriedemthen why change pointer_model=usbtablet by default?15:48
danpbmriedem: which is way longer than 1 week15:49
*** Sukhdev has joined #openstack-nova15:49
mriedemi really just want the options to be consistent for the transition15:49
mriedemuntil we drop use_usb_tablet15:49
*** zenoway has quit IRC15:49
mriedemif people don't want usb, they are already having to set use_usb_tablet=False15:49
mriedembecause the current default is True15:49
*** moshele has quit IRC15:50
*** corvus is now known as jeblair15:50
*** efried has quit IRC15:51
openstackgerritLucian Petrut proposed openstack/nova: HyperV: ensure config drives are copied as well during resizes  https://review.openstack.org/36482915:52
*** hoangcx2 has quit IRC15:52
*** rcernin has quit IRC15:53
*** kevinz has quit IRC15:55
*** hshiina has joined #openstack-nova15:55
*** shaohe_feng has quit IRC15:55
*** pcaruana has quit IRC15:56
*** browne has joined #openstack-nova15:56
*** shaohe_feng has joined #openstack-nova15:56
*** diogogmt has quit IRC15:57
*** prateek_ has quit IRC15:58
*** markus_z has quit IRC15:59
*** jlanoux has quit IRC16:01
*** tongli has quit IRC16:01
*** Apoorva has joined #openstack-nova16:02
*** zul has quit IRC16:04
*** lucas-afk is now known as lucasagomes16:04
*** lpetrut has quit IRC16:04
*** thumpba has joined #openstack-nova16:06
*** tblakes has quit IRC16:07
*** daemontool_ has quit IRC16:09
bauzasdansmith: or others: any idea why when I'm calling out an obj_reset_changes() on a BuildRequest object, it doesn't clear out my instance and bdm nested object fields ?16:09
bauzasdansmith: is obj_reset_changes() only working with non nested objects ?16:09
*** daemontool has joined #openstack-nova16:10
*** tblakes has joined #openstack-nova16:10
*** browne has quit IRC16:11
*** hemanthm|away is now known as hemanthm16:11
bauzasdansmith: http://paste.openstack.org/show/572372/ as a proof16:11
* bauzas digs into o.vo code16:12
bauzasoh f***16:13
bauzasnevermind16:13
* bauzas should repeat on the blacboard 100 times "I should always read the docstrings before asking"16:13
bauzashttps://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L664 FWIW16:14
*** tonytan_brb has joined #openstack-nova16:14
sahidmriedem: i'm not going to update the change by today, feel free to put comment of your thinking so tomorrow i will do it16:14
bauzasalaski: there is a bug btw. on the build_req code, where we only call obj_reset_changes() without that attribute, which means we literally leave the instance nested field being seen as dirty16:15
*** sahid has quit IRC16:15
mriedemsahid: ok16:15
*** tonytan4ever has quit IRC16:16
bauzasalaski: compare https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L664 and https://github.com/openstack/nova/blob/master/nova/objects/build_request.py#L13916:16
alaskibauzas: gotcha. I think that's from before we nested an instance and bdm object so they're not accounted for16:17
alaskibauzas: is it causing as issue, or just something you noticed?16:18
bauzasalaski: I'll fix that in the patch I'm working on16:18
alaskibauzas: okay16:18
bauzasalaski: not really yet, because we're not updating it16:18
bauzasalaski: but now I'm working on a save() method, we should be calling it16:18
alaskiahh, so your patch would expose it16:18
bauzasyeah16:18
bauzasI'll write a few lines in the commit msg to explain16:19
*** diogogmt has joined #openstack-nova16:19
alaskibauzas: so honestly, I'd prefer to have that change split out in a separate patch. Would it help if I pushed something up for that?16:20
bauzasalaski: fair enough, that's just a nit16:20
bauzasalaski: I'll base my patch on top of yours16:20
alaskiokay, I'll get something up soon16:20
*** derekh has quit IRC16:21
bauzasalaski: not sure we even need an unittest for covering that specific oslo thing16:21
bauzass/thing/behaviour16:21
*** mfedosin has quit IRC16:21
alaskiyeah. I may write one just to be thorough, but I agree16:21
bauzasalaski: or we should just amend the existing create unittest for verifying that we clean up the fields16:21
mriedemdanpb: if you're still around, https://review.openstack.org/#/c/367571/ is holding up an rc-potential fix16:21
bauzasalaski: lemme know if you have time for, or I can just write it now16:21
mriedemdanpb: i'm not sure why a unit test is needed for the code refactor16:21
bauzasalaski: that's straight loaded in my mind16:22
bauzas:p16:22
alaskibauzas: if you have time for it go ahead. I was offering in case you were still working on the main patch16:22
*** sudipto has quit IRC16:22
*** sudipto_ has quit IRC16:22
bauzasalaski: I'm nearly done with that patch, I should propose the final version by the next hours16:22
*** thorst has quit IRC16:23
alaskialright, let me put this one together quickly then16:23
bauzasalaski: okay16:24
*** jgriffith_ has joined #openstack-nova16:24
bauzasalaski: IMHO, you just need an assert on the test_build_req.test_create unittest that would verify that obj_get_changes() returns nothing, that's it16:24
bauzaslitterally a +2/0 patch16:25
*** abalutoiu has joined #openstack-nova16:25
alaskiyeah16:25
alaskihopefully the tests are good enough to not have collateral damage here16:26
*** nic has joined #openstack-nova16:26
bauzasalaski: yup16:26
bauzasI actually only need to write the functional test for the object modification and I'm done, but I'll need to go dinner, so it's why it's a matter of hours, not minutes16:27
bauzasalaski: ^16:27
alaskisounds good16:27
openstackgerritMathieu Gagné proposed openstack/nova: Extended description for sync_power_state_pool_size option  https://review.openstack.org/36895116:27
*** shangxdy has quit IRC16:27
*** gszasz has quit IRC16:28
*** jgriffith_ has quit IRC16:29
*** anush__ has joined #openstack-nova16:29
*** rlrossit has quit IRC16:29
*** rlrossit has joined #openstack-nova16:30
*** sdake_ has joined #openstack-nova16:30
*** krtaylor has quit IRC16:32
*** kaisers has quit IRC16:33
*** rlrossit has quit IRC16:34
*** ssurana has joined #openstack-nova16:34
*** anush__ has quit IRC16:34
*** rlrossit has joined #openstack-nova16:35
*** rlrossit has quit IRC16:35
*** rlrossit has joined #openstack-nova16:35
*** harlowja_at_home has quit IRC16:35
*** rama_y has joined #openstack-nova16:35
*** jgriffith has joined #openstack-nova16:35
*** thorst_ has joined #openstack-nova16:37
*** takedakn has quit IRC16:37
openstackgerritSylvain Bauza proposed openstack/nova: Update BuildRequest if instance currently being scheduled  https://review.openstack.org/36574016:38
bauzasalaski: ^16:38
bauzasalaski: /me heads to dinner16:39
*** annegentle has quit IRC16:40
*** tbachman has joined #openstack-nova16:41
openstackgerritAndrew Laski proposed openstack/nova: Use recursive obj_reset_changes in BuildRequest  https://review.openstack.org/36896216:41
*** annegentle has joined #openstack-nova16:42
openstackgerritAndrew Laski proposed openstack/nova: Update BuildRequest if instance currently being scheduled  https://review.openstack.org/36574016:42
alaskibauzas: rebased for you16:43
openstackgerritStephen Finucane proposed openstack/nova: hardware: Split into modules  https://review.openstack.org/36746716:44
openstackgerritStephen Finucane proposed openstack/nova: tests: Split 'test_hardware' into modules  https://review.openstack.org/36896716:44
openstackgerritStephen Finucane proposed openstack/nova: Update references for split up hardware module  https://review.openstack.org/36896816:44
*** ametts has quit IRC16:45
dansmithcdent: FYI https://review.openstack.org/#/c/368035/516:45
cdentdansmith: awesome16:46
cdentI've just managed to run a _very_ basic functional test16:46
cdentbut miles to go yet16:46
*** diogogmt has quit IRC16:47
dansmithcdent: I'm not getting the current provider generation back from GET on inventories right now16:47
dansmithso I need to look at that16:47
* cdent is inspired to continue working on this functional test16:48
cdenttoo many unknowns and changes in flight16:48
cdent:(16:48
*** unicell has quit IRC16:48
mgagneI think I found the (deeper) root cause of the BadStatusLine exception thrown by Ironic API in nova-compute. I happens that default socket backlog is 5: https://docs.python.org/2/library/socket.html#socket.socket.listen16:54
mgagneI raised it to 128 and now, I have 0 errors16:54
*** kfarr has joined #openstack-nova16:54
*** tesseract- has quit IRC16:56
*** akshai has quit IRC16:57
melwittmgagne: ack16:57
mgagneI think running api in mod_wsgi would mitigate the problem16:58
mgagneor throw more resources to problem =)16:58
*** akshai has joined #openstack-nova16:58
*** jaosorior has quit IRC16:59
mgagnehttp://paste.openstack.org/show/572397/17:00
*** cher75 has joined #openstack-nova17:01
*** fragatina has joined #openstack-nova17:01
cdentdansmith: I think the issue you're trying to deal with is fixed in https://review.openstack.org/#/c/365633/17:03
cdentmaybe?17:03
*** mriedem1 has joined #openstack-nova17:04
dansmithcdent: hmm, I dunno, I'm not sure why it is currently not working, looking at the handler code17:04
dansmithbut I'll try it17:04
cdentdansmith: I think the handler was just wrong at some point17:05
dansmithI really need that and jay's change underneath mine17:05
*** krtaylor has joined #openstack-nova17:06
*** mriedem has quit IRC17:06
cdentdansmith: do you want to do that, or shall I? I want to put my functional test (not quite ready, but no point testing the wrong stuff) somewhere in that stack17:06
openstackgerritMathieu Gagné proposed openstack/nova: Extended description for sync_power_state_pool_size option  https://review.openstack.org/36895117:06
*** sdake has joined #openstack-nova17:06
dansmithcdent: well, I will I guess. not sure why that one is not merged orin the gate already17:07
cdentbecause it has dependencies17:07
cdentwhich was probably just wrong17:07
cdentin it I fix both the handler code and your code, but it will be wrong with regard to your code because your code has now changed17:08
cdentI did it before we decided that jay's change was going to happen17:08
cdentdansmith: I gotta run to dinner, but will be back after to continue stabbing away17:08
dansmithokay17:09
dansmithah I see17:09
dansmithI need to swap the order I guess17:09
*** sdake_ has quit IRC17:09
dansmiththat's going to be icky.. I should probably just fix the thing I need in my patch17:09
*** code-R has joined #openstack-nova17:09
*** code-R_ has joined #openstack-nova17:11
*** tbachman has quit IRC17:13
*** akshai has quit IRC17:14
*** thorst_ has quit IRC17:14
*** code-R has quit IRC17:15
*** browne has joined #openstack-nova17:15
*** Swami has joined #openstack-nova17:15
*** hshiina has quit IRC17:17
openstackgerritLudovic Beliveau proposed openstack/nova-specs: Improve usability of booting guest with SR-IOV  https://review.openstack.org/36228717:18
*** harlowja has joined #openstack-nova17:19
*** diogogmt has joined #openstack-nova17:21
*** cher75 has quit IRC17:23
*** unicell has joined #openstack-nova17:23
*** cher75 has joined #openstack-nova17:28
*** danpb has quit IRC17:28
*** tonytan_brb has quit IRC17:29
*** tblakes has quit IRC17:31
*** garyk has joined #openstack-nova17:33
*** TravT has quit IRC17:37
*** sudipto has joined #openstack-nova17:38
*** sudipto_ has joined #openstack-nova17:38
*** tbachman has joined #openstack-nova17:40
*** fragatina has quit IRC17:42
*** SamYaple has joined #openstack-nova17:42
SamYapleis there a way to enforce a vm naming convention?17:43
*** nic has quit IRC17:43
SamYapleadditionally, is there a way to prevent duplicately named VMs?17:43
*** harlowja has quit IRC17:43
*** Sukhdev has quit IRC17:44
*** harlowja has joined #openstack-nova17:45
*** thorst_ has joined #openstack-nova17:47
mriedem1display name is not unique https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/models.py#L29717:47
*** mriedem1 is now known as mriedem17:47
mriedemthere isn't a template for display_name17:48
*** thorst_ has quit IRC17:49
SamYaplemriedem: thanks. im aware it is not unique, i was asking if the funcationlity existed to enrforce uniqueness17:49
mriedemno17:49
*** thorst_ has joined #openstack-nova17:49
SamYaplethanks, i thought not17:49
SamYaplethe problem at hand is with dns. when using an external dns provider with neutron there currently isn't a way to fix or even detect two conflicting dns names17:50
SamYaplewhen the desire would be to stop that from occuring17:50
*** zul has joined #openstack-nova17:50
SamYapleenforcing a display_name convention would also allow enforcing valid DNS names if that was desired17:51
SamYapleI don't think this is doable with neutron alone.17:51
mriedemthe display_name and hostname gets a bit funky here17:52
mriedemlike in https://github.com/openstack/nova/blob/master/nova/compute/api.py#L55317:52
mriedem^ is only if you're booting multiple instances in a single request i guess17:52
*** efoley has quit IRC17:53
mriedemthere is more in https://github.com/openstack/nova/blob/master/nova/compute/api.py#L135117:53
mriedemit seems odd that it checks if display_name is None because the schema requires a server name17:54
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L5717:54
SamYaplehmmm i wonder if some check is checking hostname rather than display_name somewhere and this display_name is None check was put into place rather than fixing that somewhere else17:55
SamYapleand now its just dead code17:55
mriedemor the v2.0 api didn't require a name17:55
SamYaplethat sounds more likely17:55
*** dguitarbite has joined #openstack-nova17:56
*** eharney_ has joined #openstack-nova17:56
mriedemnope v2 required a name too https://github.com/openstack/nova/blob/kilo-eol/nova/api/openstack/compute/servers.py#L49817:56
*** tblakes has joined #openstack-nova17:56
*** eharney has quit IRC17:57
*** eharney_ is now known as eharney17:57
*** thorst_ has quit IRC17:58
*** whenry has joined #openstack-nova17:58
*** abalutoiu has quit IRC17:59
*** kaisers has joined #openstack-nova18:00
*** garyk has quit IRC18:01
*** kaisers has quit IRC18:01
*** kaisers_ has joined #openstack-nova18:01
SamYaplemriedem: so it looks like we could squeeze something in the create api and do the display name validation there fairly painlessly18:01
SamYapleobviously this won't be retroactive, but it would enforce things moving forward18:02
mriedemSamYaple: as in checking for duplicates?18:03
SamYaplemriedem: checking for dups and enfocing a convention18:04
*** akshai has joined #openstack-nova18:04
SamYaple(to match dns names for example)18:04
mriedemif you wanted to enforce uniqueness then the db schema would be the place to do that, unique per user_id/display_name,18:04
mriedemhowever, that doesn't solve your global DNS issue18:05
mriedembecause user A and user B can both have a VM of name X18:05
SamYapleyea. if you did enforce that it would leak VM names18:05
*** mbirru has quit IRC18:06
SamYaplebut that being said you could list all the A records anyway from the DNS server itself which would hold that same info18:06
*** mbirru has joined #openstack-nova18:07
SamYapleim not sure we want to enforce that at the DB schema level mriedem . if this were a config option True/False, then the DB seems like the wrong place for that check, right?18:07
mriedemcorrect18:07
SamYaplebut a simple uniqueness check on create via the api, where the api rejects it, that seems reasonable18:08
mriedemit would be an api change and require a spec18:08
mriedemyou'd have to also handle that with renames18:09
mriedemand rebuild18:09
openstackgerritChris Dent proposed openstack/nova: [placement] WIP functional tests of report client  https://review.openstack.org/36899818:10
*** GB21 has quit IRC18:10
SamYaplei don't think it would require an API change is my point, well I suppose it does to a degree. Config option set, handle create call differently18:10
cdentdansmith: that ^ might be a start at a simple "it doesn't explode" functional test. it is currently based on master but should continue to work when the inventory changes enter the picture (asuming the method on the report client stay the same)18:11
*** anush__ has joined #openstack-nova18:11
SamYaplemriedem: renames and rebuilds go through the create api?18:11
cdentmriedem: you have a look at that too please18:11
mriedemSamYaple: no they don't18:11
mriedemSamYaple: it's also an interop issue, so yeah it's api change hidden behind config18:12
mriedemso requires a spec18:12
mriedemfor discussion18:12
SamYaplecool. thats the information I needed. Thanks for your time18:12
mriedemyw18:12
*** fragatina has joined #openstack-nova18:13
*** tonytan4ever has joined #openstack-nova18:14
*** hemanthm is now known as hemanthm|lunch18:14
* cdent gets some blackberries18:15
*** lucasagomes is now known as lucas-dinner18:16
*** rloo has joined #openstack-nova18:17
*** sudipto has quit IRC18:18
*** sudipto_ has quit IRC18:18
*** dguitarbite has quit IRC18:19
*** woodster_ has quit IRC18:19
*** tonytan_brb has joined #openstack-nova18:20
*** tonytan4ever has quit IRC18:22
*** jheroux has quit IRC18:22
*** burt has quit IRC18:26
*** burt has joined #openstack-nova18:26
*** fragatina has quit IRC18:27
*** fragatina has joined #openstack-nova18:27
*** TravT has joined #openstack-nova18:37
*** jichen has quit IRC18:39
*** TravT has quit IRC18:42
mriedemdansmith: do you see any issue with getting this in https://review.openstack.org/#/c/367571/ ? it's holding up an rc change, i'm not sure why a unit test would be required for a refactor18:43
dansmithmriedem: seems okay to me if it's helpful18:44
sdaguemriedem: +A18:44
mriedemthanks18:44
sdaguedansmith: yeh, there is a fix on top of it that's needed18:44
dansmiththen yeah18:44
sdaguewhich this breaks free18:44
mriedemthe poin was to pull it out to make testing the next change easier18:44
mriedem*point18:44
sdagueyep, this seems sensible18:44
*** code-R_ has quit IRC18:45
*** lpetrut has joined #openstack-nova18:49
*** lpetrut has quit IRC18:53
mriedemhmm, in talking to myself, and using this list https://specs.openstack.org/openstack/nova-specs/priorities/newton-priorities.html18:53
mriedemour top 3-4 features for newton, were listed originally as api/cellsv2/scheduler,18:54
mriedemi'd almost think they'd be glance v2, get me a network, and api policy defaults in code18:54
mriedemfrom a user pov18:54
mriedemalthough the others are bigger efforts18:54
*** woodster_ has joined #openstack-nova18:55
*** hemanthm|lunch is now known as hemanthm18:56
*** thorst_ has joined #openstack-nova18:56
*** lpetrut has joined #openstack-nova18:58
*** artom has quit IRC18:58
*** mgoddard_ has quit IRC18:59
*** efried has joined #openstack-nova18:59
*** zul has quit IRC19:00
mriedemi love marketing19:01
cdentand marketing loves you mriedem19:02
*** thorst_ has quit IRC19:03
*** pcaruana has joined #openstack-nova19:04
*** lpetrut has quit IRC19:05
*** xyang1 has joined #openstack-nova19:05
*** TravT has joined #openstack-nova19:06
*** tongli has joined #openstack-nova19:06
*** nherciu has quit IRC19:11
*** Sukhdev has joined #openstack-nova19:14
mriedemsdague: maybe you can help me out here, for newton in https://specs.openstack.org/openstack/nova-specs/specs/newton/approved/api-no-more-extensions.html#proposed-change19:14
mriedemhow much of that got done? i know some of the folding started by not all by a long shot19:14
mriedem"In Newton we will eliminate the ability to stevedore load extensions that modify existing resources (servers / flavors), and start re-folding these extensions into the core servers / flavors code."19:15
mriedemdid we kill the ability to hook in extensions?19:15
*** raj_singh has quit IRC19:17
*** READ10 has quit IRC19:18
*** kfarr has quit IRC19:19
*** moshele has joined #openstack-nova19:24
*** mriedem has quit IRC19:24
*** matrohon has joined #openstack-nova19:26
*** bvanhav has quit IRC19:32
*** mriedem has joined #openstack-nova19:33
dansmithyes19:34
dansmithoops19:34
*** liverpooler has quit IRC19:36
*** kfarr has joined #openstack-nova19:39
*** lamt has quit IRC19:47
*** tblakes has quit IRC19:51
*** raj_singh has joined #openstack-nova19:55
*** zul has joined #openstack-nova19:55
*** raj_singh has left #openstack-nova19:55
*** raj_singh has joined #openstack-nova19:55
openstackgerritMatt Riedemann proposed openstack/nova: Add note about display_name in _populate_instance_names  https://review.openstack.org/36902919:57
*** _ducttape_ has quit IRC19:57
*** raj_singh has quit IRC19:58
*** lpetrut has joined #openstack-nova19:59
openstackgerritMatt Riedemann proposed openstack/nova: Fix reno for removal of nova-manage service command  https://review.openstack.org/36903019:59
*** daemontool has quit IRC19:59
*** thorst_ has joined #openstack-nova20:00
*** ducttape_ has joined #openstack-nova20:00
*** tongli has quit IRC20:01
*** _coolsvap_ has quit IRC20:02
openstackgerritDan Smith proposed openstack/nova: [placement] Correct serialization of inventory collections  https://review.openstack.org/36563320:02
openstackgerritDan Smith proposed openstack/nova: More conservative inventory updates  https://review.openstack.org/36507320:02
dansmithcdent: that reordering of the patches was really nasty20:03
dansmithcdent: I had a hacked up version working locally, but I need to run off for a bit20:03
dansmithcdent: if you get a chance to sanity check those that would be great20:03
cdent:(20:03
dansmithI'm sure there's some rebase noise in there20:03
cdentthe two just commited, sure will look in a sec20:03
dansmithwhen I get back I will deploy it locally and try it for reals, but I figured I'd get a jenkins run started too20:03
dansmithcdent: thanks20:03
cdentdansmith: did you see the functional test I pushed a while back?20:04
cdentit can probalby be useful here20:04
dansmithcdent: no, but I will look forward to that when I get back :)20:04
*** MVenesio has joined #openstack-nova20:04
*** cleong has quit IRC20:05
*** tblakes has joined #openstack-nova20:07
*** tongli has joined #openstack-nova20:07
*** thorst_ has quit IRC20:08
*** tongli has quit IRC20:12
*** tongli has joined #openstack-nova20:13
*** mfedosin has joined #openstack-nova20:14
openstackgerritChris Dent proposed openstack/nova: [placement] Mark HTTP error responses for translation  https://review.openstack.org/36903520:17
*** vishwanathj has quit IRC20:20
*** vishwana_ has joined #openstack-nova20:20
*** sdake_ has joined #openstack-nova20:25
*** thumpba has quit IRC20:27
*** sdake has quit IRC20:28
cdentdansmith: I need to be done for the night. I left a comment on https://review.openstack.org/#/c/365633/ reporting that is works against the functional test (with a simple change to confirm more stuf), so I think it is at least close if not perfect. Leave me a message somewhere about what I should do in the morning if anything special. good night20:28
*** cdent has quit IRC20:28
*** rmcall_ has joined #openstack-nova20:28
*** thumpba has joined #openstack-nova20:29
*** smatzek_ has quit IRC20:29
*** rmcall has quit IRC20:29
*** rmcall_ is now known as rmcall20:29
*** salv-orl_ has joined #openstack-nova20:31
*** lpetrut has quit IRC20:31
*** Sukhdev has quit IRC20:31
*** Sukhdev has joined #openstack-nova20:32
openstackgerritSylvain Bauza proposed openstack/nova: Update BuildRequest if instance currently being scheduled  https://review.openstack.org/36574020:33
*** matrohon has quit IRC20:33
*** salv-orlando has quit IRC20:34
*** eharney has quit IRC20:35
*** yamahata has quit IRC20:39
*** tongli has quit IRC20:39
*** MVenesio has quit IRC20:43
mriedemdansmith: what do you want me to look at for placement stuff?20:45
*** pcaruana has quit IRC20:46
*** mdrabe has quit IRC20:46
*** mdrabe has joined #openstack-nova20:47
mriedempkoniszewski: i'm confused about the plan in https://review.openstack.org/#/c/358599/ - are live migrations with consoles broken when going from newton->mitaka?20:47
mriedemthere seems to be some flip-flopping in there20:47
dansmithmriedem: https://review.openstack.org/#/c/368035/20:49
*** lpetrut has joined #openstack-nova20:49
*** Sukhdev has quit IRC20:50
mriedemok, i'll run that on top of 368998 first, not sure if that's sane yet or not but seems that's what cdent was doing20:50
*** Sukhdev has joined #openstack-nova20:51
*** tblakes has quit IRC20:54
*** rfolco has quit IRC20:55
*** akshai has quit IRC20:56
*** MikeG451 has quit IRC20:59
*** mmedvede has quit IRC21:00
*** MikeG451 has joined #openstack-nova21:00
*** moshele has quit IRC21:01
*** sdague has quit IRC21:02
*** rtheis_ has joined #openstack-nova21:03
*** rtheis_ has quit IRC21:03
*** thorst_ has joined #openstack-nova21:05
*** rtheis has quit IRC21:05
dansmithmriedem: oh yeah, sweet, I shall run that on the current set I'm working on to21:07
dansmith*too21:07
mriedemalso running the grenade job21:07
mriedembut that will take awhile21:07
*** mvk has joined #openstack-nova21:07
mriedemand reading his spec amendmant for this21:07
*** tbachman has quit IRC21:08
*** tbachman has joined #openstack-nova21:11
*** kaisers_ has quit IRC21:12
*** thorst_ has quit IRC21:12
*** kaisers has joined #openstack-nova21:14
*** prateek has joined #openstack-nova21:15
*** rlrossit has quit IRC21:16
mriedemgah failed already http://logs.openstack.org/35/368035/5/experimental/gate-grenade-dsvm-neutron-nova-next-ubuntu-xenial-nv/2873ddb/21:17
dansmithmriedem: but failed old side right?21:18
*** kaisers has quit IRC21:18
mriedemgah i guess21:19
mriedemhmm21:19
*** dimtruck is now known as zz_dimtruck21:19
mriedemgrenade + xenial is not fun21:19
mriedemb/c < newton is trusty, >= newton is xenial21:19
mriedemgrenade is both21:19
openstackgerritdane-fichter proposed openstack/nova: Use cursive for signature verification  https://review.openstack.org/35123221:21
*** BjoernT has quit IRC21:21
mriedemi'm not sure where this is coming from http://logs.openstack.org/35/368035/5/experimental/gate-grenade-dsvm-neutron-nova-next-ubuntu-xenial-nv/2873ddb/logs/grenade.sh.txt.gz#_2016-09-12_20_57_05_70521:22
clarkbmitaka to newton is going to remain trusty. newton to ocata will be xenial21:23
mriedemshazbot21:23
clarkbwe haven't really rolled out newton to ocata because we need all the branches to be cut which happens in a few weeks21:23
mriedemclarkb: so i need ubuntu-trusty here right? https://review.openstack.org/#/c/368087/1/jenkins/jobs/projects.yaml@739121:24
*** tblakes has joined #openstack-nova21:24
clarkbmriedem: are you intending on testing mitaka to newton there?21:24
mriedemyeah21:24
clarkbmriedem: if so then yes you should use trusty. The reason for that is the mitaka side of things which run first don't necessarily run on xenial21:24
mriedemyup21:25
mriedemfixing21:25
*** penick has joined #openstack-nova21:27
mriedemdansmith: looking at https://review.openstack.org/#/c/366888/ is POST changing?21:34
mriedemfor inventories21:34
mriedemor is that a known wart and we live with it as long as PUT and GET are consistent?21:35
dansmithmriedem: put and post are the same handler I *think*21:35
dansmithlemme look21:35
dansmithah, not21:36
dansmithpost is a little different21:36
dansmithmriedem: post only creates one inventory so it doesn't need the same change I think21:37
*** mmedvede has joined #openstack-nova21:37
mriedemok21:37
*** sdake_ has quit IRC21:38
*** sdake has joined #openstack-nova21:39
*** burgerk_ has quit IRC21:40
dansmithmriedem: so the patches above that one are important, but the middle one breaks inventory briefly before the next one fixes it,21:41
dansmithwhich is because of the ordering chosen and a huge rebase conflict21:42
dansmithI'm going to try to fix it with that functional test to avoid briefly breaking it21:42
*** prateek has quit IRC21:43
*** zz_dimtruck is now known as dimtruck21:43
openstackgerritKen'ichi Ohmichi proposed openstack/nova: Fix server group name on api-ref  https://review.openstack.org/36905421:43
*** mfedosin has quit IRC21:44
*** john51 has quit IRC21:45
*** john51 has joined #openstack-nova21:45
cfriesenIn virt/libvirt/driver.py we have LOCAL_ADDRS = ('0.0.0.0', '127.0.0.1', '::', '::1').    I think we might want to consider adding '::0'.  It's not the usual way to represent it, but it is a valid IPv6 address.21:46
*** sdake_ has joined #openstack-nova21:47
*** raunak has joined #openstack-nova21:48
openstackgerritKen'ichi Ohmichi proposed openstack/nova: Fix the name of DNS entry on api-ref  https://review.openstack.org/36905521:48
*** sdake has quit IRC21:49
openstackgerritDan Smith proposed openstack/nova: [placement] Correct serialization of inventory collections  https://review.openstack.org/36563321:49
openstackgerritDan Smith proposed openstack/nova: More conservative inventory updates  https://review.openstack.org/36507321:49
*** thumpba has quit IRC21:49
*** sdake has joined #openstack-nova21:50
*** rfolco has joined #openstack-nova21:52
mriedemdansmith: is it just me or do we not have a test to POST an inventory for an RP in https://review.openstack.org/#/c/368035/5/nova/tests/functional/api/openstack/placement/gabbits/allocations.yaml ?21:52
*** sdake_ has quit IRC21:52
*** maticue has quit IRC21:54
*** annegentle has quit IRC21:54
mriedemis gabbi test the wrong nomenclature? should i use gabbit?21:55
openstackgerritKen'ichi Ohmichi proposed openstack/nova: Fix server name on api-ref  https://review.openstack.org/36905821:55
openstackgerritKen'ichi Ohmichi proposed openstack/nova: Rename 'name' to 'secgroup_name' in api-ref  https://review.openstack.org/36905921:55
*** amotoki has quit IRC21:58
edleafemriedem: not sure, but I thought 'gabbit' was the yaml file21:59
*** Guest97030 has quit IRC22:00
mriedemdoesn't really matter to me, cdent will yell at me if i'm messing it up :)22:00
mriedemok, heading home. might check in later, terrible football games tonight so might need something more fun, like code reviews...22:02
*** pwnall has joined #openstack-nova22:02
clarkbmriedem: I just want to watch the steelers go for 2 after every touchdown22:02
*** pwnall is now known as Guest8807522:02
mriedemheh22:03
mriedemwell, it stands to be a rip roarin' 12-6 win for the steelers22:03
mriedemlike all steelers games22:03
*** mriedem has quit IRC22:03
*** ducttape_ has quit IRC22:04
*** marst has quit IRC22:08
*** Guest88075 has quit IRC22:08
*** zigo has quit IRC22:08
*** thorst_ has joined #openstack-nova22:10
*** zigo has joined #openstack-nova22:11
*** zigo is now known as Guest4138822:11
*** Guest41388 is now known as zigo_22:17
*** mdrabe has quit IRC22:18
*** thorst_ has quit IRC22:18
openstackgerritKurtis Cobb proposed openstack/nova: Doc fix for Nova API Guide, added missing word.  https://review.openstack.org/36906122:19
*** thumpba has joined #openstack-nova22:19
*** ociuhandu has quit IRC22:20
*** rfolco has quit IRC22:21
*** efried has quit IRC22:22
*** thumpba has quit IRC22:24
*** Swami has quit IRC22:38
*** mlavalle has quit IRC22:41
*** krtaylor has quit IRC22:45
*** sdake has quit IRC22:48
*** openstackgerrit has quit IRC22:48
*** openstackgerrit has joined #openstack-nova22:49
*** yamahata has joined #openstack-nova22:49
*** rmcall has quit IRC22:50
*** rmcall has joined #openstack-nova22:50
oomichimriedem: SamYaple: can we use osapi_compute_unique_server_name_scope?   https://github.com/openstack/nova/blob/master/nova/conf/compute.py#L62822:50
*** salv-orlando has joined #openstack-nova22:53
*** mriedem has joined #openstack-nova22:53
*** salv-orl_ has quit IRC22:54
*** sdague has joined #openstack-nova22:56
*** browne has quit IRC22:56
*** annegentle has joined #openstack-nova22:59
*** dimtruck is now known as zz_dimtruck23:00
*** annegentle has quit IRC23:00
*** kaisers has joined #openstack-nova23:03
*** lpetrut has quit IRC23:03
*** rama_y has quit IRC23:03
*** penick has quit IRC23:03
*** sdague has quit IRC23:04
*** pradk has quit IRC23:04
mriedemdansmith: i'm not sure what i was looking at in https://review.openstack.org/#/c/368035/ but i'm +W now23:04
dansmithmriedem: just saw, thanks23:05
*** hemna is now known as hemnafk23:05
*** kfarr has quit IRC23:05
*** tblakes has quit IRC23:06
*** kaisers has quit IRC23:07
*** zz_dimtruck is now known as dimtruck23:11
*** sdague has joined #openstack-nova23:15
*** thorst_ has joined #openstack-nova23:15
*** TravT has quit IRC23:19
*** xyang1 has quit IRC23:20
*** thorst_ has quit IRC23:23
*** anush__ has quit IRC23:23
*** raunak has quit IRC23:26
*** rfolco has joined #openstack-nova23:26
oomichisdague: johnthetubaguy: alex_xu: gmann: do we still have power to fix api-ref?  https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/api-ref-in-rst23:28
mriedemtjones: is the vmware nsx ci known to be busted?23:28
*** zul has quit IRC23:28
mriedemtjones: b/c it failed every check on https://review.openstack.org/#/c/281134/23:28
oomichistill there are wrong references, and I guess they are many because I found 4 wrong references from only one viewpoint(related to 'name')23:29
mriedemoomichi: we can still fix api-ref stuff until rc123:30
mriedemoomichi: i'd probably only focus on anything simple at this point23:30
mriedemRC1 is thursday23:30
oomichimriedem: Thanks for reminding, I misunderstood the schedule(RC1 would be 16th)23:32
*** mriedem has quit IRC23:34
*** Jeffrey4l has quit IRC23:43
*** Jeffrey4l has joined #openstack-nova23:43
*** annegentle has joined #openstack-nova23:43
*** rrecio has quit IRC23:46
*** annegentle has quit IRC23:48
*** akshai has joined #openstack-nova23:50
sdagueapi-ref should be fine all along23:51
sdaguewe build from master23:51
sdagueat some point the stable branch just drifts away23:51
*** raunak has joined #openstack-nova23:52
*** akshai_ has joined #openstack-nova23:53
*** takashin has joined #openstack-nova23:56
*** akshai has quit IRC23:56
openstackgerritJoshua Harlow proposed openstack/nova: Add log to show when instances are getting healed (and which ones)  https://review.openstack.org/36908623:58
gmannyea there are many open one also we should close23:59

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