Thursday, 2015-11-12

*** Piet has quit IRC00:02
*** doug-fish has joined #openstack-nova00:02
*** baoli has quit IRC00:03
*** doug-fish has quit IRC00:06
*** jamielennox is now known as jamielennox|away00:08
*** sacharya has joined #openstack-nova00:11
*** su_zhang has joined #openstack-nova00:11
*** tonytan4ever has quit IRC00:14
*** achanda has joined #openstack-nova00:15
*** sacharya has quit IRC00:15
*** su_zhang has quit IRC00:16
*** doug-fish has joined #openstack-nova00:17
*** xyang1 has quit IRC00:20
*** doug-fish has quit IRC00:21
*** jamielennox|away is now known as jamielennox00:21
openstackgerritStephen Finucane proposed openstack/nova: tests_fields: Remove most enum-type field tests  https://review.openstack.org/24437000:23
openstackgerritStephen Finucane proposed openstack/nova: test_fields: Remove 'test_stringify' tests  https://review.openstack.org/24437100:23
openstackgerritStephen Finucane proposed openstack/nova: test_fields: Remove all 'Enum' subclass tests  https://review.openstack.org/24437200:23
*** EinstCrazy has quit IRC00:24
*** jwcroppe has quit IRC00:26
*** takedakn has joined #openstack-nova00:27
*** IanGovett has quit IRC00:27
*** kfarr has quit IRC00:27
*** yamahata has joined #openstack-nova00:28
*** Piet has joined #openstack-nova00:29
*** sfinucan has quit IRC00:32
*** angdraug has quit IRC00:34
*** oomichi has joined #openstack-nova00:34
*** achanda has quit IRC00:35
*** takedakn has quit IRC00:40
*** suro-patz has quit IRC00:40
*** mnestratov has quit IRC00:41
*** liusheng has quit IRC00:46
*** liusheng has joined #openstack-nova00:47
*** diogogmt has quit IRC00:48
*** vilobhmm has quit IRC00:48
*** yamahata has quit IRC00:49
*** jyuso1 has joined #openstack-nova00:51
*** stackdump has quit IRC00:54
*** doug-fish has joined #openstack-nova00:56
*** doug-fish has quit IRC01:00
*** zhangjn has joined #openstack-nova01:03
*** electrocucaracha has quit IRC01:03
*** EinstCrazy has joined #openstack-nova01:03
*** yamahata has joined #openstack-nova01:06
*** ssurana has quit IRC01:09
*** kmartin_ has joined #openstack-nova01:09
*** apoorvad has quit IRC01:10
*** kmartin has quit IRC01:13
*** signed8bit_ZZZzz is now known as signed8bit01:15
*** sneti has joined #openstack-nova01:15
*** zhenguo has joined #openstack-nova01:18
*** otter768 has joined #openstack-nova01:18
*** yamahata has quit IRC01:21
*** smcginnis has quit IRC01:21
*** terryyao has joined #openstack-nova01:22
*** doug-fish has joined #openstack-nova01:22
openstackgerritWen Zhi Yu proposed openstack/nova: Clean up network resources when reschedule fails  https://review.openstack.org/24347701:26
*** doug-fish has quit IRC01:27
*** jwcroppe has joined #openstack-nova01:27
*** rfolco has joined #openstack-nova01:29
*** otter768 has quit IRC01:29
*** mriedem has joined #openstack-nova01:31
*** signed8bit is now known as signed8bit_ZZZzz01:37
*** haomaiwa_ has joined #openstack-nova01:37
*** smcginnis has joined #openstack-nova01:40
openstackgerritoctopuszhang proposed openstack/nova: update connect_volume test  https://review.openstack.org/24400201:43
*** jwcroppe has quit IRC01:44
*** mtanino has quit IRC01:45
openstackgerritoctopuszhang proposed openstack/nova: update connect_volume test  https://review.openstack.org/24400201:45
*** Yanyanhu has joined #openstack-nova01:47
*** su_zhang has joined #openstack-nova01:48
*** janiceonchn has quit IRC01:54
*** sacharya has joined #openstack-nova01:54
*** nic1 has quit IRC01:56
*** su_zhang has quit IRC01:56
*** claudiub has quit IRC01:58
*** sacharya has quit IRC01:59
*** haomaiwa_ has quit IRC02:01
*** haomaiwa_ has joined #openstack-nova02:01
*** doug-fish has joined #openstack-nova02:02
*** mc_nair has quit IRC02:03
*** terryyao has quit IRC02:04
*** terryyao has joined #openstack-nova02:05
*** doug-fish has quit IRC02:06
*** stackdump has joined #openstack-nova02:06
*** smatzek has joined #openstack-nova02:11
*** diogogmt has joined #openstack-nova02:13
*** hemna has quit IRC02:13
*** signed8bit_ZZZzz is now known as signed8bit02:13
*** ijw has quit IRC02:14
*** baoli has joined #openstack-nova02:15
*** thedodd has quit IRC02:16
openstackgerritoctopuszhang proposed openstack/nova: Use JSON format instead of json format  https://review.openstack.org/22087502:16
*** baoli has quit IRC02:17
*** baoli has joined #openstack-nova02:17
openstackgerritoctopuszhang proposed openstack/nova: Use JSON format instead of json format  https://review.openstack.org/22087502:18
*** baoli has quit IRC02:18
*** baoli has joined #openstack-nova02:19
*** seanw1 has quit IRC02:20
*** seanw1 has joined #openstack-nova02:21
*** sneti has quit IRC02:24
*** janiceonchn has joined #openstack-nova02:24
*** mdrabe has joined #openstack-nova02:26
*** doug-fish has joined #openstack-nova02:28
*** spandhe has quit IRC02:29
*** EinstCrazy has quit IRC02:29
*** venkat_p has joined #openstack-nova02:30
*** venkat_p has left #openstack-nova02:31
*** mdrabe has quit IRC02:32
*** stackdump has quit IRC02:32
*** doug-fish has quit IRC02:33
*** gcb has joined #openstack-nova02:36
*** lnxnut has joined #openstack-nova02:37
openstackgerritMatt Riedemann proposed openstack/nova: Revert "Exclude all BDM checks for cells"  https://review.openstack.org/24269802:39
openstackgerritMatt Riedemann proposed openstack/nova: WIP: cells: create block device mappings in API cell  https://review.openstack.org/24264902:39
openstackgerritMatt Riedemann proposed openstack/nova: Make BlockDeviceMapping object support uuid  https://review.openstack.org/24260302:39
openstackgerritMatt Riedemann proposed openstack/nova: Add uuid column to BlockDeviceMapping  https://review.openstack.org/24260202:39
*** janiceonchn has quit IRC02:39
*** klkumar has joined #openstack-nova02:48
*** rfolco has quit IRC02:49
*** achanda has joined #openstack-nova02:54
*** dims has quit IRC02:57
*** haomaiwa_ has quit IRC03:01
*** haomaiwa_ has joined #openstack-nova03:01
*** kmartin__ has joined #openstack-nova03:03
*** kmartin_ has quit IRC03:06
*** dims has joined #openstack-nova03:07
*** doug-fish has joined #openstack-nova03:07
openstackgerrithuan proposed openstack/nova: XenAPI:Resolve Nova/Neutron race condition  https://review.openstack.org/24112703:07
*** doug-fish has quit IRC03:11
*** su_zhang has joined #openstack-nova03:13
*** signed8bit has quit IRC03:14
*** dims has quit IRC03:14
*** dims has joined #openstack-nova03:16
*** spandhe has joined #openstack-nova03:18
*** mriedem has quit IRC03:18
*** terryyao_ has joined #openstack-nova03:21
*** terryyao has quit IRC03:24
*** dims has quit IRC03:25
*** otter768 has joined #openstack-nova03:30
*** doug-fish has joined #openstack-nova03:34
*** otter768 has quit IRC03:35
*** spandhe has quit IRC03:37
*** doug-fish has quit IRC03:38
*** jwcroppe has joined #openstack-nova03:40
*** doug-fish has joined #openstack-nova03:45
*** doug-fish has quit IRC03:49
*** stevemar_ has quit IRC03:49
*** stevemar_ has joined #openstack-nova03:50
*** jwcroppe has quit IRC03:50
*** Sree has joined #openstack-nova03:51
*** klkumar has quit IRC03:51
*** Sree has quit IRC03:51
*** Sree has joined #openstack-nova03:52
*** su_zhang has quit IRC03:52
*** stevemar_ has quit IRC03:53
*** klkumar has joined #openstack-nova03:53
*** smatzek has quit IRC03:57
*** haomaiwa_ has quit IRC04:01
*** haomaiwang has joined #openstack-nova04:01
*** suro-patz has joined #openstack-nova04:10
*** suro-patz has quit IRC04:11
*** pixelbeat has quit IRC04:12
*** Marga_ has quit IRC04:14
*** spandhe has joined #openstack-nova04:15
*** klkumar has quit IRC04:15
openstackgerritjaveme proposed openstack/nova: update task state for the confirm_resize  https://review.openstack.org/24443304:21
*** suro-patz has joined #openstack-nova04:23
*** spandhe has quit IRC04:23
*** klkumar has joined #openstack-nova04:23
*** suro-patz has quit IRC04:23
*** suro-patz has joined #openstack-nova04:24
*** garyk1 has joined #openstack-nova04:28
*** terryyao has joined #openstack-nova04:32
*** terryyao_ has quit IRC04:33
*** haomaiwang has quit IRC04:37
*** haomaiwang has joined #openstack-nova04:42
*** jwcroppe has joined #openstack-nova04:44
*** haomaiwang has quit IRC04:56
*** edleafe has quit IRC04:57
*** achanda has quit IRC05:04
*** links has joined #openstack-nova05:07
*** chinmaya has joined #openstack-nova05:12
*** haomaiwang has joined #openstack-nova05:13
*** fawadkhaliq has joined #openstack-nova05:14
chinmaya:garyk hi Gary. have a query on resize/cold migrate in vmware driver05:14
garyk1sure, all ears05:17
*** sacharya has joined #openstack-nova05:19
*** baoli has quit IRC05:20
chinmaya:garyk1 suppose i have 2 cluster compute nodes, and i trigger resize, generally i see the scheduler picks up the other cluster as compute node. but the VM resides on the same cluster,05:23
chinmayaand nova show, will show the hypervisor as the other cluster05:23
chinmaya: garyk1 are you planning on considering the dest - param05:24
garyk1chinmaya: the resize is done in place05:24
garyk1that is we do not need to move it to another cluster.05:25
garyk1chinmaya: we are planning to do that for live migrations05:25
chinmaya:garyk1 yeah, but cold migration also goes in the same flow05:25
garyk1yup. agree. that too will be addressed05:26
*** suro-patz has quit IRC05:27
chinmayagaryk1: "we do not need to move it to another cluster" im not quite sure about this..05:27
*** haomaiwang has quit IRC05:27
chinmayaif resize_one_same_host is false05:27
garyk1chinmaya: i do not understand. if one does a resize then the properties of the instance are changed.05:27
garyk1why do you need to move this to a new host?05:28
chinmaya*resize_on_same_host05:28
garyk1in the vmware case a host == cluster05:28
chinmayasuppose there 2 clusters, and i have seen a behavior, scheduler was picking up another cluster05:29
garyk1https://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/driver.py#L13105:29
chinmayayeah right, 2 clusters = 2 hosts.05:29
garyk1chinmaya: it does not have anything to do with the scheduler05:29
garyk1chinmaya: can you please explain the problem that you saw05:30
chinmayai'll . give me a sec :)05:31
*** otter768 has joined #openstack-nova05:31
*** otter768 has quit IRC05:36
chinmayagaryk1: actually i was trying these things on stable/kilo. https://github.com/openstack/nova/blob/stable/kilo/nova/compute/manager.py#L390705:36
chinmayaif allow_resize_to_same_host is set to false,05:36
chinmayaand i was observing after the resize, nova show used to show the other cluster05:37
chinmayaas the hypervisor_host_name05:37
*** haomaiwang has joined #openstack-nova05:38
chinmayahttps://github.com/openstack/nova/blob/stable/kilo/nova/compute/manager.py#L4072 , "migration.dest_compute " was the name of the other compute cluster05:39
garyk1chinmaya: let me check that and get back to you05:40
garyk1the resize it self is done in place. i see no reason why it should be moved to another cluster05:40
chinmayawhat if the new flavor needs a new compute05:41
chinmayai mean, existing compute doesnt have that much space05:41
garyk1then the compute will be at capacity05:42
garyk1can you please provide a use case?05:44
*** haomaiwang has quit IRC05:50
chinmayasure. im running two nova compute services for managing two clusters,  while performing resize (and set allow_resize_to_same_host = false) i was getting this > https://github.com/openstack/nova/blob/stable/kilo/nova/compute/resource_tracker.py#L205 dest_compute as other cluster05:51
*** jinxing has joined #openstack-nova05:54
garyk1at the moment the code supports allow_resize_to_same_host05:55
garyk1if we want to set this as false then we need to add in the cold and live migration support05:55
*** jinxing has quit IRC05:55
*** TakashiN has joined #openstack-nova05:56
*** TakashiN is now known as takashin05:56
*** takashin has left #openstack-nova05:57
*** takashin has joined #openstack-nova05:58
*** jinxing has joined #openstack-nova05:58
*** abhishekk has joined #openstack-nova06:00
chinmayagaryk1: got it. and in cold migration support, we need to clone the vm anyways right, by treating the "dest" param ?06:01
chinmayabut generally, these dest param is the host ip, which wont work in vmware case.  it needs manager changes, no ?06:02
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata  https://review.openstack.org/24272706:02
garyk1yes, we need to do the clone06:02
*** achanda has joined #openstack-nova06:05
*** Marga_ has joined #openstack-nova06:06
*** irenab has quit IRC06:07
*** emagana has joined #openstack-nova06:07
*** haomaiwa_ has joined #openstack-nova06:09
chinmaya:garyk1 one more question. Are you planning on supporting cold migration on different hosts ?06:10
chinmayais there any bug/bp ?06:10
*** doug-fish has joined #openstack-nova06:10
garyk1chinmaya: it is something that we plan to do. i guess i need to post a spec… but feel free to do that if you want06:10
*** achanda has quit IRC06:10
abhishekkjohnthetubaguy: hi you around?06:12
chinmayagaryk1: sure.. however i feel live migration is a better value add. but can only be supported from vc 6 and above ?06:13
*** jerrygb has quit IRC06:14
*** doug-fish has quit IRC06:15
*** achanda has joined #openstack-nova06:15
garyk1chinmaya: yes, that is correct06:17
garyk1we need to leverage v6 migrations across clusters06:17
*** cfriesen__ has quit IRC06:17
garyk1that was the plan.06:17
chinmayagaryk1: i see. Thanks for your time Gary.06:18
garyk1thank you. have a good one06:18
*** abhishekk has quit IRC06:18
openstackgerritjaveme proposed openstack/nova: update task state for the confirm_resize  https://review.openstack.org/24443306:21
*** haomaiwa_ has quit IRC06:21
*** markmc has quit IRC06:22
*** gongysh has joined #openstack-nova06:22
*** browne has quit IRC06:23
*** markmc has joined #openstack-nova06:24
*** haomaiwang has joined #openstack-nova06:25
*** jnclyz12483 has joined #openstack-nova06:26
*** jnclyz12483 has quit IRC06:26
*** jnclyz12483 has joined #openstack-nova06:27
*** jnclyz12483 has quit IRC06:27
*** Sree_ has joined #openstack-nova06:28
*** jnclyz12483 has joined #openstack-nova06:28
*** Sree_ is now known as Guest7054906:28
*** jnclyz12483 has quit IRC06:28
*** jnclyz12483 has joined #openstack-nova06:29
*** jnclyz12483 has quit IRC06:29
*** jnclyz12483 has joined #openstack-nova06:30
*** Sree has quit IRC06:30
*** jnclyz12483 has quit IRC06:31
*** jnclyz12483 has joined #openstack-nova06:31
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Fix wrong usage of config option use_neutron_default_nets  https://review.openstack.org/24306106:32
openstackgerritChangBo Guo(gcb) proposed openstack/nova: test:clean up wrong value in method CONF.set_override  https://review.openstack.org/24241606:32
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enhance value check for option notify_on_state_change  https://review.openstack.org/24271706:32
*** jnclyz12483 has quit IRC06:33
*** mnestratov has joined #openstack-nova06:36
*** mjura has joined #openstack-nova06:39
*** irenab has joined #openstack-nova06:41
*** sc68cal has quit IRC06:45
*** links has quit IRC06:45
*** emagana has quit IRC06:46
*** emagana has joined #openstack-nova06:52
*** suro-patz has joined #openstack-nova06:54
*** jhesketh has quit IRC06:54
*** masayukig has quit IRC06:54
*** sacharya has quit IRC06:54
*** gszasz has joined #openstack-nova06:54
*** baoli has joined #openstack-nova06:55
*** jhesketh has joined #openstack-nova06:59
*** masayukig has joined #openstack-nova07:00
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-nova07:01
*** baoli has quit IRC07:01
*** josecastroleon has joined #openstack-nova07:03
*** haomaiwang has quit IRC07:04
*** links has joined #openstack-nova07:05
*** scheuran has joined #openstack-nova07:07
*** moshele has joined #openstack-nova07:08
*** emagana has quit IRC07:09
*** jwcroppe has quit IRC07:10
*** zhangjn has quit IRC07:12
*** sirushti has quit IRC07:14
*** gridinv has quit IRC07:16
*** gibi has quit IRC07:16
eliqiaohi nova, I get No sql_connection parameter is established when I update nova to latest. anyone seen this?07:16
eliqiaois there workaround now/07:17
*** gridinv has joined #openstack-nova07:17
*** sirushti has joined #openstack-nova07:17
*** gibi has joined #openstack-nova07:17
*** nkrinner has quit IRC07:17
*** nkrinner has joined #openstack-nova07:17
eliqiaoseems related to blueprint new-oslodb-enginefacade07:18
*** adun153 has joined #openstack-nova07:18
adun153What's the purpose of nova-cert and nova-conductor?07:19
*** suro-patz has quit IRC07:19
*** achanda has quit IRC07:20
*** lpetrut has joined #openstack-nova07:20
eliqiaoadun153: http://docs.openstack.org/kilo/config-reference/content/section_conductor.html07:21
*** stevemar_ has joined #openstack-nova07:21
*** salv-orlando has joined #openstack-nova07:23
adun153Could you please give me an example of data that nova-compute need to write to, or read from the database?07:23
*** haomaiwang has joined #openstack-nova07:24
*** stevemar_ has quit IRC07:24
*** doron has joined #openstack-nova07:26
openstackgerritlyanchih proposed openstack/python-novaclient: Client repr hosts list return value will raise AttributeError: host  https://review.openstack.org/24447607:29
*** adun153 has quit IRC07:29
*** zhangjn has joined #openstack-nova07:29
*** adun153 has joined #openstack-nova07:31
*** otter768 has joined #openstack-nova07:32
*** zhangjn has quit IRC07:32
*** zhangjn has joined #openstack-nova07:33
*** armax has quit IRC07:34
*** zhangjn has quit IRC07:34
*** otter768 has quit IRC07:36
eliqiaoadun153: such as nova-compute wants to get an instance by instance name/ip it will send request to nova-conductor07:37
*** haomaiwang has quit IRC07:38
*** haomaiwang has joined #openstack-nova07:40
*** alex_klimov has joined #openstack-nova07:43
*** rdopiera has joined #openstack-nova07:43
*** alexschm has joined #openstack-nova07:44
*** haomaiwang has quit IRC07:51
*** doron is now known as doron_afk07:51
*** paul-carlton has joined #openstack-nova07:52
*** sacharya has joined #openstack-nova07:55
*** aix has joined #openstack-nova07:55
openstackgerritNikola Dipanov proposed openstack/nova: compute: split check_can_live_migrate_destination  https://review.openstack.org/24448407:56
*** haomaiwa_ has joined #openstack-nova07:56
openstackgerritNikola Dipanov proposed openstack/nova: live-migrate: Change the status Migration is created with  https://review.openstack.org/24448507:56
openstackgerritNikola Dipanov proposed openstack/nova: Don't track migrations in 'accepted' state  https://review.openstack.org/24448607:56
openstackgerritNikola Dipanov proposed openstack/nova: Move the Migration set-status-if-exists pattern to a method  https://review.openstack.org/24448707:56
openstackgerritNikola Dipanov proposed openstack/nova: live-mig: Mark migration as failed on fail to schedule  https://review.openstack.org/24448807:56
openstackgerritNikola Dipanov proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448907:56
*** sahid has joined #openstack-nova07:58
*** josecastroleon has quit IRC07:59
*** sacharya has quit IRC07:59
*** sahid has quit IRC08:00
*** haomaiwa_ has quit IRC08:01
*** haomaiwang has joined #openstack-nova08:01
*** sahid has joined #openstack-nova08:05
*** lpetrut has quit IRC08:06
*** josecastroleon has joined #openstack-nova08:08
*** takashin has quit IRC08:10
*** paul-carlton has quit IRC08:12
*** sudipto has joined #openstack-nova08:12
*** liverpooler has joined #openstack-nova08:13
openstackgerritNikola Dipanov proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448908:13
*** markmc has quit IRC08:14
*** haomaiwang has quit IRC08:14
*** haomaiwang has joined #openstack-nova08:16
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: adds os-win library  https://review.openstack.org/21210208:17
*** liverpooler has quit IRC08:17
*** liverpooler has joined #openstack-nova08:17
*** jichen has joined #openstack-nova08:19
tangchen__Nikola: Hi nikola, please take a look at this series when you are free. https://review.openstack.org/#/c/241476/08:25
*** paul-carlton has joined #openstack-nova08:26
ndipanovtangchen__, yeah have it marked already no worries08:27
tangchen__ndipanov: Thx. :)08:27
openstackgerritPawel Koniszewski proposed openstack/nova-specs: Pause VM during live migration  https://review.openstack.org/22904008:29
openstackgerritgaryk proposed openstack/nova: VMware: expand support for Opaque networks  https://review.openstack.org/16575008:30
bauzasgood morning Nova08:31
*** sc68cal has joined #openstack-nova08:31
*** haomaiwang has quit IRC08:32
*** haomaiwang has joined #openstack-nova08:33
*** hparekh has quit IRC08:34
*** rotbeard has joined #openstack-nova08:34
*** andymaier has joined #openstack-nova08:35
*** andymaier_ has joined #openstack-nova08:35
*** andymaier_ has quit IRC08:38
*** Pablo|off| has joined #openstack-nova08:40
BobBallyes bauzas, I know, and I'm working on it :)08:40
BobBall:D08:40
BobBall+ good morning08:40
bauzaseh eh08:40
*** ociuhandu has quit IRC08:41
bauzasthat's a traditional French time when going to work, y'know ;)08:41
bauzasthat, and taking 30 mins for chatting and taking coffee before actually08:41
bauzasworking08:41
BobBallI was expecting you to point out that the XenServer CI was broken last night by a Nova merge :)08:41
*** hparekh has joined #openstack-nova08:43
bauzasBobBall: oh nooooo, I'm just caffeinating now after a long pause because $day-off yesterday08:43
*** mpavone has joined #openstack-nova08:43
BobBallahhh great :)08:43
bauzasthat - and reading emails08:43
bauzasbut glad to hear you're on the problem :)08:44
BobBall:D08:45
*** salv-orlando has quit IRC08:46
*** achanda has joined #openstack-nova08:47
*** Pablo|off| is now known as pcaruana08:48
*** romainh has joined #openstack-nova08:49
*** rcernin has joined #openstack-nova08:50
*** danpb has joined #openstack-nova08:57
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: removes *Utils modules and unit tests  https://review.openstack.org/21591708:57
*** lucas-dinner is now known as lucasagomes09:00
*** haomaiwang has quit IRC09:00
*** matrohon has joined #openstack-nova09:00
*** paul-carlton has quit IRC09:01
*** ociuhandu has joined #openstack-nova09:01
*** jistr has joined #openstack-nova09:03
*** gcb has quit IRC09:04
*** gcb has joined #openstack-nova09:05
*** yassine__ has joined #openstack-nova09:07
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add serial console proxy  https://review.openstack.org/18403809:08
*** haomaiwang has joined #openstack-nova09:08
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add serial console proxy  https://review.openstack.org/18403809:08
*** markmc has joined #openstack-nova09:10
openstackgerritJianghua Wang proposed openstack/nova-specs: XenAPI: Add spec to support Virtual GPU (VGPU)  https://review.openstack.org/22935109:10
*** aix has quit IRC09:19
*** claudiub has joined #openstack-nova09:19
*** doron_afk has left #openstack-nova09:20
*** aix has joined #openstack-nova09:21
openstackgerritClaudiu Belu proposed openstack/nova: Libvirt: Fix SMBFS volume attach race condition  https://review.openstack.org/18761909:22
*** eliqiao has quit IRC09:23
*** eliqiao has joined #openstack-nova09:24
*** rk4n has joined #openstack-nova09:28
openstackgerritJianghua Wang proposed openstack/nova-specs: XenAPI: Add spec to support Virtual GPU (VGPU)  https://review.openstack.org/22935109:28
*** pixelbeat has joined #openstack-nova09:28
*** terryyao has quit IRC09:29
*** otter768 has joined #openstack-nova09:32
tangchen__hi, just a little question. If I use nova lock xxx to lock an instance, is there any interface that I can check if the server is locked ?09:33
tangchen__I tried nova show, nova list --long09:33
tangchen__there is no such info09:33
*** achanda has quit IRC09:34
*** haomaiwang has quit IRC09:34
*** ihrachys has joined #openstack-nova09:35
*** jinxing has quit IRC09:36
*** jinxing has joined #openstack-nova09:36
*** gcb has quit IRC09:37
*** otter768 has quit IRC09:37
andrearosatangchen__: nova show reports the locked field09:38
*** haomaiwa_ has joined #openstack-nova09:38
tangchen__andrearosa: No, I didn't find that field in nova show output.09:38
andrearosatangchen__: I did, tested just now on a devstack installation, maybe it is enabled by some specific extension, let me check09:39
*** garyk has quit IRC09:42
tangchen__andrearosa: Please refer to: http://paste.openstack.org/show/478638/09:43
tangchen__andrearosa: This is my output.09:43
andrearosado you have /me looking09:44
andrearosasorry I am looking at your output09:44
openstackgerritNikola Dipanov proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448909:44
tangchen__andrearosa: Thx for helping. ;)09:44
*** garyk1 has quit IRC09:45
alexschmdanpb: Hi, do you know any second core I could ask to review the fix for live migration with serial console?09:45
alexschmdapnb: https://review.openstack.org/#/c/215102/09:45
alexschmdanpb: you gave your +2 two months ago and nothing happened since that point in time09:46
*** openstackgerrit has quit IRC09:46
*** openstackgerrit has joined #openstack-nova09:47
openstackgerritjaveme proposed openstack/nova: update task state for the confirm_resize  https://review.openstack.org/24443309:49
andrearosatangchen__: this is my output for an instance owned by demo user in devstack: http://paste.openstack.org/show/478640/09:49
bauzastangchen__: which version are you running ?09:50
bauzastangchen__: IIRC, there was a bp to show it09:50
bauzassec09:50
bauzasgot it09:50
bauzastangchen__: andrearosa: https://github.com/openstack/nova/blame/master/nova/api/openstack/compute/views/servers.py#L31109:50
*** Sree has joined #openstack-nova09:50
bauzastangchen__: which means that your client has to query for an API version >= 2.909:51
*** flyingtt has joined #openstack-nova09:51
andrearosabauzas: thanks09:51
bauzastangchen__: https://github.com/openstack/nova/commit/49a572a043f27623a15af5f1b8e54c3a560b805c09:51
tangchen__bauzas, andrearosa: OK, I see.  Thank you guys very much.09:52
*** Guest70549 has quit IRC09:54
*** haomaiwa_ has quit IRC09:58
*** andymaier has quit IRC09:58
*** haomaiwa_ has joined #openstack-nova09:59
*** rk4n has quit IRC09:59
openstackgerritjaveme proposed openstack/nova: update task state for the confirm_resize  https://review.openstack.org/24443310:00
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: config options: add hacking check for help text length  https://review.openstack.org/24207810:00
*** haomaiwa_ has quit IRC10:01
*** haomaiwa_ has joined #openstack-nova10:01
*** ZZelle has joined #openstack-nova10:02
*** gszasz has quit IRC10:02
*** Yanyanhu has quit IRC10:04
johnthetubaguygibi: just approving the notification blueprints, did I get the arrows the correct way around in the dependency tree? https://blueprints.launchpad.net/nova/+spec/service-status-notification10:04
gibijohnthetubaguy: cheking..,10:07
*** gszasz has joined #openstack-nova10:07
*** stackdump has joined #openstack-nova10:07
gibijohnthetubaguy: yes, this is my view as well to start with the new notification10:07
*** adun153 has quit IRC10:08
*** derekh has joined #openstack-nova10:09
*** haomaiwa_ has quit IRC10:11
*** stackdump has quit IRC10:12
*** haomaiwang has joined #openstack-nova10:13
bauzasandrearosa: catching you because PaulMurray isn't there, I'm missing a bit of information about Bristol :)10:14
bauzasin particular the train station and how far it is from the office10:14
bauzasandrearosa: I got most of the information by http://www.hpl.hp.com/bristol/ and http://www.hpl.hp.com/downloads/HP_Labs_Bristol.pdf but I want to make sure it's correct10:15
bauzasbefore me booking flights etc.10:15
*** openstackgerrit has quit IRC10:16
*** openstackgerrit has joined #openstack-nova10:17
openstackgerritSimona Iuliana Toader proposed openstack/nova: Hyper-V: Adds host maintenance implementation  https://review.openstack.org/20415710:19
johnthetubaguygibi: cool, thanks for thecking10:20
*** sacharya has joined #openstack-nova10:21
andrearosabauzas: sorry I was in a meeting with PaulMurray. Filton Abbey wood station or/and Bristol Parkway are the two stations close to the office, they are 10-15 mins walk10:21
bauzasokay, that's what I understood then10:22
gibijohnthetubaguy: thanks for approving :)10:22
bauzasandrearosa: Google tells me it's rather 20-30 mins walk for both of them :p10:22
andrearosaand if you need taxi Parkway is your choice as it is bigger than Filton.10:22
andrearosa30 mins it too much, I think Filton is a bit shorted than Parkway10:23
andrearosas/shorted/shorter10:23
bauzasandrearosa: https://www.google.fr/maps/dir/Filton+Abbey+Wood/HP+Bristol,+Long+Down+Ave,+Stoke+Gifford,+Bristol,+South+Gloucestershire+BS34+8QZ/@51.5034813,-2.5587824,17z/data=!3m1!4b1!4m14!4m13!1m5!1m1!1s0x487191c88fb332f7:0xc0ce3149aee89f36!2m2!1d-2.5635251!2d51.5035814!1m5!1m1!1s0x487191cc572dbc15:0x8348734b13d56a2c!2m2!1d-2.5541369!2d51.500937!3e210:24
*** kaisers has quit IRC10:25
*** sacharya has quit IRC10:25
andrearosabauzas: that route suggests to take left when you leave the station but if you go right is easier, so let'say 20 mins10:28
bauzasack10:28
*** haomaiwang has quit IRC10:31
*** janiceonchn has joined #openstack-nova10:32
*** kaisers has joined #openstack-nova10:33
*** janiceonchn has quit IRC10:33
*** haomaiwang has joined #openstack-nova10:36
*** deepakcs has joined #openstack-nova10:36
*** paul-carlton has joined #openstack-nova10:36
paul-carlton1bauzas I walk from Filton Abbey Wood to the office everyday, it is a 10 to 15 minutes walk10:38
*** garyk has joined #openstack-nova10:38
bauzasokay, I don't want to run a contest, good to know :)10:38
andrearosalol10:38
bauzas:D10:38
paul-carlton1When you leave the station head for MOD office and turn right up the hill , path takes you round MOD office, less roads to cross and I think it is slightly quicker10:40
openstackgerritBalazs Gibizer proposed openstack/nova: Remove onSharedStorage from evacuate API  https://review.openstack.org/24356210:40
*** jinxing has quit IRC10:45
openstackgerritJohn Garbutt proposed openstack/nova: Rearranges to create new Compute API Guide  https://review.openstack.org/23018610:46
openstackgerritJohn Garbutt proposed openstack/nova: Rearranges to create new Compute API Guide  https://review.openstack.org/23018610:47
*** jistr_ has joined #openstack-nova10:48
*** oomichi has quit IRC10:48
*** haomaiwang has quit IRC10:48
*** ihrachys_ has joined #openstack-nova10:48
*** aix_ has joined #openstack-nova10:48
*** rcernin_ has joined #openstack-nova10:48
*** ihrachys has quit IRC10:48
*** jistr has quit IRC10:49
openstackgerritSimona Iuliana Toader proposed openstack/nova: Hyper-V: Adds host maintenance implementation  https://review.openstack.org/20415710:49
*** deepakcs has quit IRC10:49
*** rcernin has quit IRC10:49
*** rcernin has joined #openstack-nova10:50
*** ihrachys has joined #openstack-nova10:50
*** 7F1AAZM53 has joined #openstack-nova10:51
*** aix has quit IRC10:51
*** ihrachys_ has quit IRC10:52
*** jgjhgsonchn88523 has joined #openstack-nova10:53
*** aix_ has quit IRC10:53
*** rcernin_ has quit IRC10:53
*** eglynn has joined #openstack-nova10:53
*** jistr_ has quit IRC10:54
*** Sree has quit IRC10:56
*** Sree has joined #openstack-nova10:57
*** baoli has joined #openstack-nova10:58
*** 7F1AAZM53 has quit IRC11:01
*** Sree has quit IRC11:01
*** haomaiwang has joined #openstack-nova11:01
*** haomaiwang has quit IRC11:02
*** baoli has quit IRC11:03
*** garyk has quit IRC11:04
*** garyk has joined #openstack-nova11:04
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add SerialConsoleOps class  https://review.openstack.org/18404011:05
openstackgerritClaudiu Belu proposed openstack/nova: Removes support for Hyper-V Server 2008 R2  https://review.openstack.org/23764311:05
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Nova serial console access support  https://review.openstack.org/14500411:05
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: adds os-win library  https://review.openstack.org/21210211:05
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add serial console handler class  https://review.openstack.org/18403911:05
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add serial console proxy  https://review.openstack.org/18403811:05
*** aix_ has joined #openstack-nova11:05
*** jistr_ has joined #openstack-nova11:05
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825511:07
openstackgerritAdelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location  https://review.openstack.org/17972711:07
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004511:07
*** zhenguo has quit IRC11:07
*** e0ne has joined #openstack-nova11:09
*** rk4n has joined #openstack-nova11:12
*** jamielennox is now known as jamielennox|away11:12
*** haomaiwang has joined #openstack-nova11:12
openstackgerritJianghua Wang proposed openstack/nova: xenapi: OVS agent updates the wrong port when using XenServer + Neutron  https://review.openstack.org/24284611:12
*** alex_klimov has quit IRC11:13
*** dims_ has joined #openstack-nova11:14
*** baoli has joined #openstack-nova11:14
*** irenab has quit IRC11:14
*** jerrygb has joined #openstack-nova11:15
*** lpetrut has joined #openstack-nova11:16
*** romainh has left #openstack-nova11:16
openstackgerritJohn Garbutt proposed openstack/nova-specs: Blueprint to use libvirt storage pools  https://review.openstack.org/23204511:17
openstackgerritJohn Garbutt proposed openstack/nova-specs: Blueprint to use libvirt storage pools  https://review.openstack.org/23204511:17
*** gszasz has quit IRC11:18
*** baoli has quit IRC11:19
openstackgerritJohn Garbutt proposed openstack/nova-specs: Blueprint to use libvirt storage pools  https://review.openstack.org/23204511:19
*** mgoddard_ has quit IRC11:19
*** mgoddard_ has joined #openstack-nova11:19
*** jerrygb has quit IRC11:19
dims_danpb : johnthetubaguy : couple of quick policy.json fixes if you get a chance https://review.openstack.org/#/c/242902/ and https://review.openstack.org/#/c/243090/11:22
*** haomaiwang has quit IRC11:24
johnthetubaguydims_: I do wonder if we can have something in oslo.polcy to help us find these missing policy rules in the gate, like a raise exceptions mode, that would be awesome stuff11:27
dims_johnthetubaguy y, something that tracks these things and finally dump it at the end11:27
openstackgerritNikola Dipanov proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448911:28
johnthetubaguydims_: possibly, I would be fine with it raising an exception and triggering a 500 error, just for testing, like add a policy rule of "raise exception on missing rule" or something?11:28
dims_right11:28
*** haomaiwang has joined #openstack-nova11:30
*** terryyao has joined #openstack-nova11:30
*** josecastroleon has quit IRC11:30
dims_thanks danpb and johnthetubaguy - will whip up something soon11:31
johnthetubaguydims_: awesome :)11:32
dims_though johnthetubaguy - that would be in oslo.policy, which we are yet to adopt11:32
johnthetubaguydims_: yeah, gives us a reason to get some good stuff11:32
*** otter768 has joined #openstack-nova11:33
*** mwagner has quit IRC11:34
*** fawadkhaliq has quit IRC11:35
openstackgerritPetrut Lucian proposed openstack/nova: Hyper-V: removes *Utils modules and unit tests  https://review.openstack.org/21591711:37
*** otter768 has quit IRC11:38
*** smatzek has joined #openstack-nova11:39
*** lnxnut has quit IRC11:39
*** baoli has joined #openstack-nova11:41
*** haomaiwang has quit IRC11:41
*** alex_klimov has joined #openstack-nova11:42
*** baoli has quit IRC11:46
*** gcb has joined #openstack-nova11:47
johnthetubaguyandrearosa: hey, I was just looking at this spec: https://review.openstack.org/#/c/226199/10/specs/mitaka/approved/enhance-instance-operations-during-live-migration.rst,cm11:47
*** haomaiwang has joined #openstack-nova11:47
johnthetubaguyandrearosa: I am not sure I understand how much we gain, vs the effort to ensure it works11:47
andrearosajohnthetubaguy: that is one of the main concerns raised against that proposal11:48
andrearosajohnthetubaguy: the idea comes from our experience where you can have a very long time for live migration11:48
andrearosaand we have ppl asking for still managing their instances11:49
andrearosajohnthetubaguy: for example if the live migration takes11:49
andrearosa20 hours why the customer couldn't get access to the console or have a look at the console.log file for 20 hours?11:49
*** ndipanov has quit IRC11:50
andrearosajohnthetubaguy: I agree that is not one of the must to have feature for live-migration11:50
johnthetubaguyandrearosa: it feels like the cancel live migrate is more useful11:52
johnthetubaguyandrearosa: honestly, if we get live-migrate tested and working this release, I think adding that console access next release makes sense, just feels like its a cycle to early to worry about this11:53
andrearosajohnthetubaguy: it is, but why do you see one excluding the other?11:53
johnthetubaguyandrearosa: its more the testing11:53
johnthetubaguyandrearosa: I see cancel as more widely useful (if you need to snapshot, or resize, etc)11:53
andrearosajohnthetubaguy: your point is about priority? I mean you don't want to spend time on these things as there are more important feature to implement?11:55
*** gszasz has joined #openstack-nova11:55
johnthetubaguyandrearosa: its a combination of things really, priority is probably the main one11:55
johnthetubaguyandrearosa: we don't have live-migrate working correctly yet, it doesn't seem appropriate to start allowing concurrent operations during live-migrate until we have the basic things solid11:56
johnthetubaguy(and well tested)11:56
johnthetubaguydanpb: looking at the split network plane for live-migrate stuff11:56
andrearosajohnthetubaguy: I see your point and I agree, I thought to get it approved just in case we can make everything working in time, probably unlikely11:57
johnthetubaguydanpb: confused why we have both CONF.libvirt.live_migration_uri and CONF.my_live_migration_ip in this case, doesn't live_migration_uri already help, or is that the management traffic side?11:57
*** terryyao has quit IRC11:57
*** LiuNanke has joined #openstack-nova11:57
*** terryyao has joined #openstack-nova11:58
johnthetubaguyandrearosa: yeah, it feels too soon right now, lets not get distracted from all the live-migrate bug fixes, and keeping the gating tests stable, etc11:58
danpbjohnthetubaguy: live migration URI is used by the source host to initiate connections11:58
andrearosajohnthetubaguy: ok I am fine to not get it approved for this cycle11:58
openstackgerritChangBo Guo(gcb) proposed openstack/nova: test:clean up wrong value in method CONF.set_override  https://review.openstack.org/24241611:58
danpbjohnthetubaguy: it needs to be expanded to contain the IP address of the *target* host, which CONF.my_live_migration_ip provides11:59
danpbjohnthetubaguy: the target host IP is passed back to the source host in the pre_live_migration phase11:59
*** gcb_ has joined #openstack-nova11:59
danpbjohnthetubaguy: you can't hardcode the IP address in CONF.libvirt.live_migration_uri, because on any one host you obviously need to be able to migrate to arbitrary target host11:59
johnthetubaguydanpb: ah, so thats the bit I was missing, its always got the substitution variables in there12:00
johnthetubaguydanpb: thanks, makes sense now12:00
danpblive_migration_ip is basically replacing the use of compute_nodes.hostname field12:00
openstackgerritBalazs Gibizer proposed openstack/nova: Remove onSharedStorage from evacuate API  https://review.openstack.org/24356212:00
*** haomaiwang has quit IRC12:01
*** haomaiwang has joined #openstack-nova12:01
*** gcb has quit IRC12:03
openstackgerritPaul Carlton proposed openstack/nova-specs: Query and cancel ongoing live migrations  https://review.openstack.org/22882812:04
openstackgerritjichenjc proposed openstack/nova: libvirt:Fix a typo of test cases  https://review.openstack.org/24311712:04
*** rfolco has joined #openstack-nova12:04
openstackgerritPaul Carlton proposed openstack/nova-specs: Query and cancel ongoing live migrations  https://review.openstack.org/22882812:05
openstackgerritPaul Carlton proposed openstack/nova-specs: Query and cancel ongoing live migrations  https://review.openstack.org/22882812:07
*** marzif has joined #openstack-nova12:08
*** jgjhgsonchn88523 has quit IRC12:08
*** romainh has joined #openstack-nova12:10
*** haomaiwang has quit IRC12:11
*** haomaiwang has joined #openstack-nova12:11
alexschmjohnthetubaguy: can you add this review to the agenda of the meeting today? https://review.openstack.org/#/c/215102/12:12
alexschmjohnthetubaguy: I'm looking for another core to review this, it got a +2 already from danpb two months ago12:12
johnthetubaguyalexschm: its not really a "stuck" review as such, agreed we want someone else to take a look12:12
johnthetubaguyI have reviewed it, but don't know that code enough for a +212:13
alexschmjohnthetubaguy: right, so should I raise it in the open discussion?12:13
johnthetubaguyalexschm: yeah, that works12:15
alexschmjohnthetubaguy: ok thx12:15
*** IanGovett has joined #openstack-nova12:18
*** chinmaya has quit IRC12:18
*** josecastroleon has joined #openstack-nova12:19
*** jistr_ is now known as jistr12:20
openstackgerritSimona Iuliana Toader proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot  https://review.openstack.org/20958112:20
*** haomaiwang has quit IRC12:22
*** haomaiwang has joined #openstack-nova12:23
openstackgerritjichenjc proposed openstack/nova: Add DiskFormat as Enum in fields  https://review.openstack.org/22985412:25
openstackgerritjichenjc proposed openstack/nova: Make ImageMeta:disk_format use DiskFormatField  https://review.openstack.org/23146012:25
openstackgerritPaul Carlton proposed openstack/nova-specs: Migrate libvirt volumes  https://review.openstack.org/23205312:25
*** samueldmq has joined #openstack-nova12:29
openstackgerritPaul Carlton proposed openstack/nova-specs: Cancel ongoing live migrations  https://review.openstack.org/22882812:30
*** Sree__ has joined #openstack-nova12:31
openstackgerritPawel Koniszewski proposed openstack/nova-specs: Pause VM during live migration  https://review.openstack.org/22904012:37
openstackgerritSimona Iuliana Toader proposed openstack/nova: objects: added 'os_secure_boot' property to ImageMetaProps object  https://review.openstack.org/23759312:38
*** baoli has joined #openstack-nova12:39
openstackgerritPawel Koniszewski proposed openstack/nova-specs: Pause VM during live migration  https://review.openstack.org/22904012:40
johnthetubaguydansmith: sdague: I think this might break upgrade, if so, we might want to check if we have let any of these slip in already: https://review.openstack.org/#/c/243496/112:41
* johnthetubaguy goes for lunch12:41
*** haomaiwang has quit IRC12:42
sdaguejohnthetubaguy: oh, you are probably right12:42
openstackgerritjichenjc proposed openstack/nova: Change Invalid exception to a specified exception  https://review.openstack.org/23250612:43
*** baoli has quit IRC12:44
*** sacharya has joined #openstack-nova12:45
*** lbelivea has joined #openstack-nova12:45
*** sacharya has quit IRC12:49
*** zhangguoqing has joined #openstack-nova12:51
*** zhangguoqing has quit IRC12:52
*** zhangguoqing has joined #openstack-nova12:53
*** sudipto has quit IRC12:53
*** baoli has joined #openstack-nova12:53
*** arif-ali has quit IRC13:01
*** thorst has joined #openstack-nova13:02
*** LiuNanke has quit IRC13:02
*** jinxing has joined #openstack-nova13:04
openstackgerritClaudiu Belu proposed openstack/nova: Removes support for Hyper-V Server 2008 R2  https://review.openstack.org/23764313:08
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: adds os-win library  https://review.openstack.org/21210213:08
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add serial console proxy  https://review.openstack.org/18403813:08
*** haomaiwang has joined #openstack-nova13:08
*** mwagner has joined #openstack-nova13:09
*** IanGovett has quit IRC13:11
openstackgerritjichenjc proposed openstack/nova-specs: Add display-flavor-id-in-migration.rst  https://review.openstack.org/21943113:11
*** annegentle has joined #openstack-nova13:12
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: adds os-win library  https://review.openstack.org/21210213:15
*** jerrygb has joined #openstack-nova13:15
*** gcb_ has quit IRC13:17
*** annegentle has quit IRC13:17
*** jerrygb_ has joined #openstack-nova13:18
*** aysyd has joined #openstack-nova13:19
*** jerrygb has quit IRC13:22
*** IanGovett has joined #openstack-nova13:23
*** arif-ali has joined #openstack-nova13:24
*** jlanoux has joined #openstack-nova13:24
*** dims_ has quit IRC13:24
*** doug-fish has joined #openstack-nova13:26
*** dims has joined #openstack-nova13:26
*** burgerk has joined #openstack-nova13:26
*** MVenesio has joined #openstack-nova13:29
*** gcb_ has joined #openstack-nova13:30
*** terryyao has quit IRC13:30
alex_xujohnthetubaguy: hi, should we mention virtual doc sprint in today meeting? just advert it and see if any feedback about the date or anything else. but looks likes fine, no concern from the maillist13:31
johnthetubaguyalex_xu: oh, yeah, we can add that to the dates list13:31
johnthetubaguyalex_xu: good idea13:32
johnthetubaguyalex_xu: do you want to quickly add that in there for me now?13:32
alex_xujohnthetubaguy: yea, let me add it13:32
johnthetubaguyalex_xu: thank you!13:32
alex_xujohnthetubaguy: np13:32
*** edmondsw has joined #openstack-nova13:32
*** otter768 has joined #openstack-nova13:34
openstackgerritjichenjc proposed openstack/nova: Make lock policy default to admin or owner  https://review.openstack.org/24221313:37
*** diogogmt has quit IRC13:37
*** stevemar_ has joined #openstack-nova13:38
*** otter768 has quit IRC13:39
*** jerrygb_ has quit IRC13:39
*** diogogmt has joined #openstack-nova13:40
*** edleafe has joined #openstack-nova13:41
*** Piet has quit IRC13:45
*** alex_klimov has quit IRC13:47
*** alex_klimov has joined #openstack-nova13:47
*** garyk has quit IRC13:47
*** boris-42 has quit IRC13:48
*** su_zhang has joined #openstack-nova13:49
openstackgerritjichenjc proposed openstack/nova-specs: Add enable-pci-api-function.rst  https://review.openstack.org/24466613:49
*** vladikr has quit IRC13:51
*** e0ne has quit IRC13:51
*** vladikr has joined #openstack-nova13:51
*** e0ne has joined #openstack-nova13:52
*** ptm_away is now known as PaulMurray13:54
*** BobBall has quit IRC13:54
johnthetubaguysdague: there is an API doc thing, linked to a project change form annegentle to get the API concept guide going, I think its ready to merge now: https://review.openstack.org/#/c/230186/13:58
sdaguejohnthetubaguy: great, I'll take a look soon13:58
johnthetubaguysdague: thanks13:58
openstackgerritPavel Kholkin proposed openstack/nova: enginefacade: 'migration'  https://review.openstack.org/24349613:58
*** mdrabe has joined #openstack-nova13:59
johnthetubaguymikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes garyk edleafe dims moshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov kashyap aloga rgeragnov bauzas xyang tpatil med_ nic scottda nagyz belliott dguitarbite sdague jroll diana_clarke _diana_ raildo jichen13:59
johnthetubaguy gjayavelu mdorman klindgren sorrison belmoreira mgagne mrda mmmpork PaulMurray gcb meeting time14:00
pkholkinjohnthetubaguy, sdague: Hello guys! Please take a look at updated patch in migration section https://review.openstack.org/#/c/243496/2. We decided to ignore use_slave parameter and not to change object version14:00
sdaguepkholkin: there was another patch that had a similar issue, right?14:01
*** haomaiwang has quit IRC14:01
dguitarbitehello14:01
*** lykinsbd has joined #openstack-nova14:01
*** haomaiwang has joined #openstack-nova14:01
*** diogogmt has quit IRC14:01
pkholkinjohnthetubaguy, sdague: yes, we marked it as wip now and will change soon14:01
*** andreykurilin has left #openstack-nova14:01
*** andreykurilin has joined #openstack-nova14:02
pkholkinjohnthetubaguy, sdague: also, we added 'async' decorator there, it is needed for enginefacade to use slave_connection if it is14:02
*** EinstCrazy has joined #openstack-nova14:03
*** e0ne has quit IRC14:04
*** mriedem has joined #openstack-nova14:04
openstackgerritNiall Bunting proposed openstack/nova: Update GlanceClient exceptions to new versions  https://review.openstack.org/24467314:04
*** rlrossit has joined #openstack-nova14:06
*** Sree__ has quit IRC14:07
*** korzen has joined #openstack-nova14:08
edmondswtonyb, we spoke about https://bugs.launchpad.net/nova/+bug/1415087 in Tokyo. Did you push up a WIP patch and can you point me to it?14:10
openstackLaunchpad bug 1415087 in OpenStack Compute (nova) "[OSSA 2015-011] Format-guessing and file disclosure in image convert (CVE-2015-1850, CVE-2015-1851)" [High,Incomplete] - Assigned to Tony Breeds (o-tony)14:10
openstackgerritMatthew Booth proposed openstack/nova: Fix typo in test_post_select_populate  https://review.openstack.org/24417814:11
*** salv-orlando has joined #openstack-nova14:12
*** ctrath has joined #openstack-nova14:12
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add SerialConsoleOps class  https://review.openstack.org/18404014:13
openstackgerritClaudiu Belu proposed openstack/nova: Removes support for Hyper-V Server 2008 R2  https://review.openstack.org/23764314:13
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: adds os-win library  https://review.openstack.org/21210214:13
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add serial console handler class  https://review.openstack.org/18403914:13
openstackgerritClaudiu Belu proposed openstack/nova: HyperV: Add serial console proxy  https://review.openstack.org/18403814:13
*** e0ne has joined #openstack-nova14:13
*** w_verdugo has quit IRC14:15
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: removes *Utils modules and unit tests  https://review.openstack.org/21591714:15
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: config options: centralize section "serial_console"  https://review.openstack.org/24417714:17
mriedemVek: we're planning a novaclient minor release so let's hold up on approving anything right now14:19
*** EinstCrazy has quit IRC14:19
*** nelsnelson has joined #openstack-nova14:20
*** jichen has quit IRC14:20
*** marzif has quit IRC14:20
*** mc_nair has joined #openstack-nova14:21
*** rotbeard has quit IRC14:21
*** ijuwang has joined #openstack-nova14:22
*** BobBall has joined #openstack-nova14:22
*** jerrygb has joined #openstack-nova14:22
*** annegentle has joined #openstack-nova14:22
*** nelsnels_ has quit IRC14:23
*** jyuso1 has quit IRC14:23
*** cfriesen__ has joined #openstack-nova14:23
*** pm90_ has joined #openstack-nova14:24
openstackgerritBob Ball proposed openstack/nova: XenAPI: Correct hypervisor type in Horizon's admin view  https://review.openstack.org/23737414:24
mriedemsdague: so novaclient release today or tuesday?14:25
sdaguemriedem: I'm fine with today14:26
*** MVenesio has quit IRC14:26
*** w_verdugo has joined #openstack-nova14:26
*** liverpooler has quit IRC14:28
*** zhangguoqing has quit IRC14:29
*** fawadkhaliq has joined #openstack-nova14:31
*** MVenesio has joined #openstack-nova14:31
*** xyang1 has joined #openstack-nova14:34
*** smatzek has quit IRC14:34
johnthetubaguymriedem: sdague: today is cool14:34
mriedemok i posted a heads up to the -dev list14:34
openstackgerritGábor Antal proposed openstack/nova: Replaced deprecated timeutils methods  https://review.openstack.org/24117914:35
openstackgerritMarian Horban proposed openstack/nova: Double detach volume causes server fault  https://review.openstack.org/24468314:35
openstackgerritMarian Horban proposed openstack/nova: Double detach volume causes server fault  https://review.openstack.org/24468314:35
*** lpetrut has quit IRC14:39
openstackgerritAndrey Kurilin proposed openstack/nova: [Py34] Enable api.openstack.test_wsgi unit test  https://review.openstack.org/23107614:39
*** lpetrut has joined #openstack-nova14:39
*** su_zhang has quit IRC14:41
openstackgerritPawel Koniszewski proposed openstack/nova-specs: Pause VM during live migration  https://review.openstack.org/22904014:41
BobBalljohnthetubaguy: Regarding https://review.openstack.org/#/c/231103/4/nova/virt/xenapi/volume_utils.py (sorry - I forgot you'd made comments there) - which bit specifically are you meaning to use six for?14:41
mriedemwhat's the story with the dr?14:43
bauzasmriedem: I guess it's http://www.doctorwho.tv/events/doctor-who-experience/14:45
mriedemoh "doctor who"14:45
mriedemnow it makes sense14:45
*** EinstCrazy has joined #openstack-nova14:46
*** eharney has joined #openstack-nova14:51
*** jgjhgsonchn88523 has joined #openstack-nova14:51
*** ssurana has joined #openstack-nova14:51
openstackgerritAndrey Kurilin proposed openstack/nova: [Py34] api.openstack.test_faults.TestFaultWrapper  https://review.openstack.org/23147314:54
openstackgerritAndrey Kurilin proposed openstack/nova: [Py34] api.openstack.test_faults.TestFaultWrapper  https://review.openstack.org/23147314:55
*** _yonglihe has joined #openstack-nova14:57
openstackgerritMoshe Levi proposed openstack/nova: pci utils: fix the check if pci device is physical_function  https://review.openstack.org/22716014:57
*** whenry_ has quit IRC14:58
*** haomaiwang has quit IRC14:58
*** pm90_ is now known as pratikmallya14:58
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996414:59
johnthetubaguyBobBall: its str(v)15:00
*** dims has quit IRC15:01
*** EinstCrazy has quit IRC15:01
*** mjura has quit IRC15:01
*** smatzek has joined #openstack-nova15:02
mriedemsdague: we might want to slip this into novaclient 2.35.0 https://review.openstack.org/#/c/232605/15:02
mriedemsdague: left off my +W to see if you wanted to skim it15:02
*** stackdump has joined #openstack-nova15:03
sdagueyeh, it's tests, I'd say land it15:03
mriedemyeah, my thinking too, +W15:04
*** cdent has joined #openstack-nova15:04
*** korzen has quit IRC15:04
BobBalljohnthetubaguy: I think I must be missing something... str(int) works in both 2.7 and 3; why do we need to use six?15:04
*** dims has joined #openstack-nova15:07
*** diogogmt has joined #openstack-nova15:07
bauzasBobBall: because str != unicode in py2.715:07
*** diogogmt has quit IRC15:07
mdboothPaulMurray: Does your cabal of live migrators have a weekly IRC meeting, yet? Looking at https://etherpad.openstack.org/p/mitaka-live-migration15:08
*** stevemar_ has quit IRC15:08
*** diogogmt has joined #openstack-nova15:10
kashyapmdbooth: Seems like it's on Tuesdays at 1400 UTC15:10
mdboothkashyap: From which magicall well did you divine this?15:10
mriedembauzas: py315:10
kashyapmdbooth: Heh, it was asked here yesterday15:10
mdboothAh, ok15:11
*** sfinucan has joined #openstack-nova15:11
mriedemmdbooth: ML thread on it too15:11
bauzasmriedem: aaaaand  ? :)15:11
kashyapmdbooth: I just looked up my logs for that; also it was announced on the list, but well-buried.15:11
*** njirap has joined #openstack-nova15:11
mriedemnvm15:11
mdboothmriedem: D'oh so there is15:11
*** sneti has joined #openstack-nova15:12
*** lnxnut has joined #openstack-nova15:16
*** ssurana has quit IRC15:16
*** mrkz has joined #openstack-nova15:16
*** breitz has quit IRC15:18
*** signed8bit has joined #openstack-nova15:18
*** breitz has joined #openstack-nova15:18
*** EinstCrazy has joined #openstack-nova15:19
*** edtubill has joined #openstack-nova15:20
*** marzif has joined #openstack-nova15:21
*** ssurana has joined #openstack-nova15:21
*** thangp has joined #openstack-nova15:24
*** mpavone has quit IRC15:26
*** _yonglihe has quit IRC15:27
*** gcb_ has quit IRC15:27
openstackgerritPavel Kholkin proposed openstack/nova: enginefacade: 'service'  https://review.openstack.org/24117815:29
*** sneti has quit IRC15:29
*** stevemar_ has joined #openstack-nova15:29
*** sneti has joined #openstack-nova15:29
*** eglynn has quit IRC15:29
*** mtanino has joined #openstack-nova15:31
mhorban_@sdague: Hello, Could you please look at https://review.openstack.org/#/c/238983/ ? It is fix of "services no longer reliably stop in stable/liberty / master" bug.15:31
BobBallbauzas: I'm still confused... What should I use instead of str(int) from six?  I see the difference that py3 will return unicode strings, but I don't understand why that causes a problem in this case https://review.openstack.org/#/c/231103/4/nova/virt/xenapi/volume_utils.py and what I need to do to fix it?15:31
PaulMurraymdbooth, https://wiki.openstack.org/wiki/Meetings/NovaLiveMigration15:31
mdboothPaulMurray: Found it, thanks15:31
PaulMurraymdbooth, anything for the agenda, let me know15:32
mdboothPaulMurray: The other way round. I was hoping I might take something from the agenda and work on it.15:32
mdboothWe'll see.15:32
bauzasBobBall: so, I'm not johnthetubaguy so my point is that it's a style thing - not really a problem15:32
*** nkrinner has quit IRC15:32
PaulMurraymdbooth, cool - anything at all or do you have a topic in mind?15:33
bauzasBobBall: I mean, you can mix both and that will work - but like johnthetubaguy it's better to be explicit15:33
*** mriedem is now known as mriedem_away15:33
mdboothPaulMurray: Nothing in particular. I've poked a fair amount of the code over time so I mostly know what's going on, and I just thought I might pitch in wherever.15:34
BobBallok, but I still don't see which methods from six are actually helpful here...15:35
*** otter768 has joined #openstack-nova15:35
BobBallIt's probably very obvious to you but I've not written much explicitely 2/3 compatible code :D15:35
PaulMurraymdbooth, there are specs to review at the moment and of course bugs. For the features we can ask if there is anything specific in the meeting. If not I'm sure I can find something interesting to get your teeth into15:36
mdboothPaulMurray: Ok, will see. Bugs are fine, too.15:36
PaulMurraymdbooth, and thanks for the offer15:37
bauzasBobBall: six.text_type() ;)15:38
BobBallthanks.  *goes looking*15:39
*** mpavone has joined #openstack-nova15:39
johnthetubaguybauzas: I thought it broke for some reason, I could be wrong15:39
*** otter768 has quit IRC15:39
bauzasjohnthetubaguy: no, it works15:40
bauzasjohnthetubaguy: that's not stylistic, but that works15:40
johnthetubaguyBobBall: hmm, odd, OK then15:40
BobBallah - I see - so the suggestion is six.text_type(v) rather than str(v)15:40
johnthetubaguyyeah15:40
*** fawadkhaliq has quit IRC15:40
*** scheuran has quit IRC15:40
BobBallVery happy to change that! thanks for the explanation15:40
johnthetubaguyBobBall: if it still works with str() on python34, then maybe we don't need it15:41
johnthetubaguyBobBall: the best way to fix this stuff is to get all the XenAPI tests passing on python3415:41
johnthetubaguyBobBall: I know hyper-v have their whole compute driver running on python34 now, which is cool15:41
BobBallIndeed15:42
*** klkumar has quit IRC15:42
openstackgerritPavel Kholkin proposed openstack/nova: enginefacade: 'compute_node'  https://review.openstack.org/24118915:42
krotschecksdague, johnthetubaguy: Thanks for the +2's on the CORS patch :)15:42
BobBallwell I can't currently even run the py27 unit tests on my local machine now - let alone py34 :D15:43
*** baoli has quit IRC15:44
bauzasBobBall: johnthetubaguy: to make it perfectly clear, 'foo'.join(collection(str(v)) works15:44
bauzasin py34 and py2715:44
BobBallawesome bauzas - that's nice15:44
johnthetubaguybauzas: yeah, I should read up on that more, I thought str() was always bad for some reason15:44
*** burgerk has quit IRC15:45
johnthetubaguyI guess str() is unicode in py3415:45
*** baoli has joined #openstack-nova15:46
openstackgerritPavel Kholkin proposed openstack/nova: enginefacade: 'compute_node'  https://review.openstack.org/24118915:46
*** salv-orlando has quit IRC15:46
*** baoli has quit IRC15:48
bauzasjohnthetubaguy: yup15:48
*** tonytan4ever has joined #openstack-nova15:49
bauzasjohnthetubaguy: well, it's a string to be clear, but it's unicoded15:49
johnthetubaguybauzas: right15:50
*** sfinucan has quit IRC15:50
*** sfinucan has joined #openstack-nova15:50
*** Piet has joined #openstack-nova15:51
*** achanda has joined #openstack-nova15:51
dansmithjohnthetubaguy: where is the spec to remove the use_slave stuff?15:52
*** baoli has joined #openstack-nova15:52
*** aix_ has quit IRC15:52
*** claudiub has quit IRC15:53
johnthetubaguydansmith: so thats a good question, it seems to be happening as part of this: https://blueprints.launchpad.net/nova/+spec/new-oslodb-enginefacade15:54
johnthetubaguyhttp://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/oslo_db-enginefacade.html#code-changes15:54
dansmithokay15:54
*** ccarmack has joined #openstack-nova15:55
johnthetubaguydansmith: I think it now moves to using the read decorator to imply use_slave by default15:55
dansmithso we just added that to everything for a couple years for seemingly no reason? :)15:55
johnthetubaguyalthough I think they must be assuming a syncronous read slave, at that point15:55
dansmithI guess I'm not sure how we can always know if we can read from the slave15:55
mdboothdansmith: use_slave is different15:55
*** pradk_ has joined #openstack-nova15:55
mdboothIt means it's acceptable for it to be async15:55
dansmithif we're reading something that depends on synchronization with a write..15:55
mdboothi.e. a bit out of date15:55
johnthetubaguywell, I think that was the folks in utar, when you have a lagging slave15:55
*** angdraug has joined #openstack-nova15:55
*** pradk_ is now known as pradk15:56
johnthetubaguymdbooth: yeah, thats what I thought15:56
mdboothSo it's only used by periodic tasks, I think15:56
johnthetubaguymdbooth: yeah15:56
mdboothIIRC there's an async_reader decorator which is separate15:56
dansmithmdbooth: well, it was supposed to be used by lots of stuff, we just don't ever seem to go through the thought process of whether reading out of data data is okay for each case15:56
*** aix has joined #openstack-nova15:56
johnthetubaguydansmith: it did seem a little random15:56
mdboothdansmith: The new decorator should make it easier. The design point is that it's a decision the developer should make, not a user.15:57
dansmithI guess my primary concern is that we're not going to start reading from slaves more than we are now, in places where it might break things15:57
mdboothNope15:57
mdboothThe reader decorator doesn't use async15:57
dansmithmdbooth: define developer and user in that case15:57
mdboothHas to be done explicitly15:57
dansmithright, which is what use_slave was supposed to get us15:57
johnthetubaguyright, use_slave was explicit15:58
dansmithlike, anything that reads flavors mostly anywhere other than api should be able to read from a slave15:58
*** baoli has quit IRC15:58
mdboothdeveloper == person writing the python code for a periodic task, and is writing it in the explicit knowledge that the data might be a bit out of date15:58
dansmithwe sucked at ever thinking about it and making those places actually declare that it's okay15:58
johnthetubaguyI guess the idea is to use the decorator and context instead?15:58
dansmithsee, I don't actually think that periodics can always use the slave,15:58
johnthetubaguydansmith: you mean we didn't use it in enough places to be useful?15:59
dansmithbecause we need to know about task_state changes immediately to prevent periodics from racing with things15:59
mdboothAnyway, it makes it easier to use or not use.15:59
dansmithjohnthetubaguy: we didn't consider "can this run against the slave" ever.. we added it everywhere so we could start making that decision, but we never actually started marking things as "async okay"15:59
johnthetubaguydansmith: oh, I thought we did add a few of those15:59
mtaninoHi, I'd like to know the policy of feature proposal at Nova. I'd like to add an option to existing API, in this case, do I need both SPEC and BP? Are there good reference to know Nova's poroposal policy?16:00
dansmithand we can't really make that call at the db layer only16:00
mdboothdansmith: It's not for the db layer. It's for the layer above.16:00
johnthetubaguyso I thought it was the object method caller that needs to decide right?16:00
*** baoli has joined #openstack-nova16:00
dansmithmdbooth: we can't make it at the object layer16:00
*** signed8bit is now known as signed8bit_ZZZzz16:00
johnthetubaguyright, has to be outside the object later16:00
dansmithjohnthetubaguy: right, it's the caller of the object thing16:00
dansmithi.e. the code doing a lookup of flavors, for example16:01
dansmithwe can't make the call inside list_flavors()16:01
dansmiths/call/decision/ for disambiguation16:01
mdboothIt can live above that, even. If you decorate a chunk of a periodic task with async_reader, anything in there would use a slave if it was available16:01
johnthetubaguyso I think we might want to revise this spec...16:01
dansmithmdbooth: but the periodic tasks never talk to the database16:01
johnthetubaguyneed to replace calls that use use_slave to use the new thing, at a minimum16:01
dansmithmdbooth: so you can't decorate them with anything db-related and have it honored16:01
dansmithjohnthetubaguy: I'm not sure how they can16:01
*** achanda has quit IRC16:02
mdboothdansmith: The decorator stores state in the request context16:02
dansmithunless there is some other communication channel I'm missing16:02
mdboothSo it percolates16:02
dansmithah, okay16:02
dansmithso just to be clear,16:02
dansmiththe object stores the context,16:02
dansmithso you have to look up the object with the async-ified context,16:02
mdboothYup16:02
dansmithand then everything else done to that object will have that async flag16:02
mdboothAnd you can set that anywhere you have a request context16:03
dansmithunless you specifically alter the in-object context16:03
*** achanda has joined #openstack-nova16:03
dansmithand of course,16:03
mdboothActually... I wonder if this is a wart16:03
dansmithif you pass that object to another service over rpc, it will lose the async flag, which is probably desirable16:03
dansmithbut needs to be obvious I think16:03
mdboothSo the db context is stored as a thread locally type thing on the request context16:04
*** achanda has quit IRC16:04
mdboothAnd it's modified based on lexical scope16:04
mdboothIf an object is storing a request context16:04
mdbooth...16:04
*** signed8bit_ZZZzz is now known as signed8bit16:04
mdboothIts db context is going to be different depending on the lexical scope of the code calling it16:04
dansmithstoring the context is inbuilt into the object machinery, in case it's not clear16:05
dansmithit's not like only some objects do that16:05
mdboothRight16:05
*** kfarr has joined #openstack-nova16:05
*** pratikmallya has quit IRC16:06
mdboothSo, any object operation which fetches from the db will fetch using the context of the calling code.16:06
dansmithyou mean object code that calls db.* methods directly, right?16:07
mdboothDirectly or indirectly, doesn't matter16:07
mdboothIf a db operation results using its request context...16:07
openstackgerritSven Anderson proposed openstack/nova: Fix is_volume_backed_instance() for unset image_ref  https://review.openstack.org/24361616:07
johnthetubaguyI think I am starting to remember why we didn't do this using context in the first place...16:08
*** electrocucaracha has joined #openstack-nova16:08
dansmithI'm confused about how this does or does not propagate over RPC I guess16:08
mdboothIt doesn't propagate over rpc16:08
mdboothat all16:08
dansmiththat seems to be my concern then16:08
mdboothSo it needs to?16:09
dansmithbecause everything that should be deciding whether something is async-tolerant is on the calling side of RPC16:09
*** tongli has joined #openstack-nova16:09
dansmitheverything is an exaggeration, but .. most things16:09
mdboothThis is when your fetching an object remotely over rpc...16:10
mdboothThat's a good point16:10
mdboothWhich I don't think I considered16:10
mdboothyou're16:10
*** baoli has quit IRC16:10
mdboothOk, so it sounds like some use_slave will continue to exist at the rpc layer16:11
dansmithyes, which is all object/db operations for a whole set of services,16:11
bauzasmriedem_away: meh, when you're back, I'd like to discuss about your good point on https://review.openstack.org/#/c/242008/5/releasenotes/source/index.rst,cm and explain why master16:11
dansmithto be more services during an upgrade if we throw a switch16:11
mdboothRight, I realise now16:11
dansmithmdbooth: I don't particularly like having to have use_slave as a parameter for everything, mind you, but I think whatever we do has to have rpc in mind16:12
mdboothYes, I think it will have to remain for rpc calls which need it. However, it doesn't need to go further than that, because those rpc calls can then set the appropriate context immediately.16:12
mdboothAnd it will propagate without being passed explicitly.16:12
*** ihrachys has quit IRC16:13
dansmithmdbooth: that means every object method needs to take that param,16:13
dansmithbut that it tells the db layer what to do based on the context, right?16:13
* mdbooth isn't familiar with this part of rpc. Are all object methods also rpcs?16:14
* mdbooth is thinking the top-level entry into a service16:14
mdboothBut yes, it could still be a lot16:14
mdboothThis doesn't add uses of use_slave, though. It just doesn't get rid of as many as I thought.16:15
mdboothHmm, although you'll then need to explicitly pull out the async context when making an rpc call16:16
*** thedodd has joined #openstack-nova16:16
mdboothBecause the db facade definitely doesn't do that for you...16:16
*** thedodd has quit IRC16:16
*** stevemar_ has quit IRC16:17
*** davideagnello has quit IRC16:17
*** mriedem_away is now known as mriedem16:17
mriedembauzas: i'm here now16:17
mdboothdansmith: Yes, I think use_slave requires a rethink.16:17
*** davideagnello has joined #openstack-nova16:17
*** links has quit IRC16:18
*** stevemar_ has joined #openstack-nova16:18
mdboothProbably not a big one, though.16:18
dansmithmdbooth: any object method that is remotable is an rpc yes16:18
mdboothRight, we're not going to break that.16:19
*** eglynn has joined #openstack-nova16:19
mdboothdansmith: I think a conservative first implementation, which is what this is, should leave use_slave in place.16:19
bauzasmriedem: so reno looks at the branches16:19
mriedemah16:19
bauzasmriedem: not at the files16:19
mdboothWe'll then add a little decorator in sqla/api.py which pulls it out of kwargs and set @reader or @async_reader according to passed args16:20
bauzasmriedem: http://docs.openstack.org/developer/reno/sphinxext.html#sphinx-extension16:20
mdboothThat's functionally equivalent to what we have now16:20
mdboothAnd leaves scope for improvement16:20
bauzasmriedem: like, when we'll release and cut the new mitaka branch, we'll have to add a new rst file asking to look at the mitaka branch16:20
bauzasmriedem: like I did for the liberty branch16:20
mdboothDoes that make sense? Where do you want me to write that?16:21
*** mdrabe has quit IRC16:21
bauzasmriedem: in the meantime, we can ask guys to write notes that 'll be seen in that specific page16:21
*** kbyrne has joined #openstack-nova16:21
mriedembauzas: yeah, i guess we could also do :version: 13.0.016:21
mriedembut maybe that's getting ahead of ourselves16:22
bauzasmriedem: yeah that's another option16:22
mriedemwhen we start cutting stable point releases, i think we'll need to use :version:16:22
bauzasmriedem: right16:22
mriedemsince stable/liberty could be 12.0.0 or 12.1.016:22
bauzasmriedem: yup16:22
bauzasmriedem: so, glance just discovered they actually needed that kind of 'show me the current' patch for the CI16:22
*** burgerk has joined #openstack-nova16:23
bauzasmriedem: hence me providing it16:23
mriedembauzas: not following, you mean this https://review.openstack.org/#/c/242008/5/releasenotes/notes/start-using-reno-cd0063e9cdb87daa.yaml?16:23
bauzasmriedem: sec16:24
snikitinjohnthetubaguy: Hi John. I want to ask you about very old patch 'instance_tags'. https://review.openstack.org/#/c/128940/ In Liberty it was blocked because Jay didn't want to use extensions.16:25
bauzasmriedem: http://lists.openstack.org/pipermail/openstack-dev/2015-November/078940.html16:25
snikitinjohnthetubaguy:I changed my code to use plugin mechanism. In this way tags formally was an extension (it was just a child class of Extension), but it was a core extension, always turned on.16:25
*** emagana has joined #openstack-nova16:25
snikitinjohnthetubaguy: So we planed that in Mitaka we will use some new way to create new API controllers without any extensions. Do we have this new way? if not, how I can create new API controller?16:25
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996416:26
*** mpavone has quit IRC16:27
bauzasmriedem: oh I even see I should backport my 'master' change to liberty...16:27
johnthetubaguysnikitin: I wasn't sure what jaypipes was thinking there, it should be clearer in the code base now, it just needs to live here: https://github.com/openstack/nova/tree/master/nova/api/openstack/compute16:27
mriedembauzas: yeah dhellmann called it 'unreleased' https://review.openstack.org/#/c/243302/1/releasenotes/source/unreleased.rst16:27
dansmithjohnthetubaguy: so that means we put use_slave back into the patch you asked for it to be removed, yes? :)16:28
bauzasmriedem: yup, I'm going to follow that pattern16:28
johnthetubaguydansmith: yeah, sorry about that16:28
bauzasmriedem: that's not very sexy, that's basically copy/paste from other projects :/16:28
dansmithdiana_clarke: ^16:28
johnthetubaguydansmith: do you have the link for that?16:28
diana_clarkedansmith: thanks16:28
johnthetubaguyI can go find it16:28
mriedembauzas: i'm not sure it needs to be sexy16:29
diana_clarkedansmith: not that it was actrually using 'use_slave' before, but I guess someone could16:29
*** MVenesio_ has joined #openstack-nova16:29
*** MVenesio has quit IRC16:30
dansmithdiana_clarke: it's an rpc interface, so getting it right first is beneficial16:30
*** emagana has quit IRC16:30
*** emagana_ has joined #openstack-nova16:30
*** jdurgin1 has joined #openstack-nova16:31
johnthetubaguydiana_clarke: ping me when ready for review again, I was basically +2 on that apart from the use_slave thing16:32
*** garyk has joined #openstack-nova16:32
*** EinstCrazy has quit IRC16:35
pkholkinjohnthetubaguy: could you please take another look at the patch related to use_slave too) https://review.openstack.org/#/c/243496/16:35
openstackgerritSylvain Bauza proposed openstack/nova: Add "unreleased" release notes page  https://review.openstack.org/24200816:35
openstackgerritSylvain Bauza proposed openstack/nova: Add reno for release notes management  https://review.openstack.org/24200716:35
pkholkinwe changed two similar patches too16:35
*** ssurana has quit IRC16:36
*** hemnafk is now known as hemna16:36
bauzasmriedem: ^16:36
bauzasmriedem: I made the changes to look like other changes16:36
bauzasmriedem: I'm not backporting them to the liberty branhc16:36
bauzasnow*16:36
*** kmartin__ has quit IRC16:36
mriedembauzas: +216:37
bauzasmriedem: I would hold until Monday to +W them16:38
mriedemwhy?16:38
*** sacharya has joined #openstack-nova16:39
*** rdopiera has quit IRC16:39
*** fawadkhaliq has joined #openstack-nova16:40
*** mwagner has quit IRC16:41
bauzasmriedem: well, nvm the CI job is not yet merged...16:41
*** pratikmallya has joined #openstack-nova16:41
*** paul-carlton has quit IRC16:41
*** mdrabe has joined #openstack-nova16:42
*** romainh has left #openstack-nova16:42
*** r-daneel has joined #openstack-nova16:42
*** kmartin has joined #openstack-nova16:42
openstackgerritBob Ball proposed openstack/nova: XenAPI: Workaround for 6.5 iSCSI bug  https://review.openstack.org/23110316:43
*** moshele has quit IRC16:45
*** mgoddard__ has joined #openstack-nova16:45
*** pratikmallya has quit IRC16:45
*** jdurgin1 has quit IRC16:45
*** pratikmallya has joined #openstack-nova16:46
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996416:46
dansmithjohnthetubaguy: ^16:47
openstackgerritNiall Bunting proposed openstack/nova: Update GlanceClient exceptions to new versions  https://review.openstack.org/24467316:47
*** mgoddard_ has quit IRC16:48
*** alexschm has quit IRC16:48
*** aix has quit IRC16:49
*** annegentle has quit IRC16:49
*** annegentle has joined #openstack-nova16:49
*** rcernin has quit IRC16:50
*** ctrath has quit IRC16:52
*** jinxing has quit IRC16:53
bauzasmriedem: so, everything is cleaned up now, I'm going to write the CI patch you proposed by only running tempest and tests when no relnodes are touched16:53
*** rlrossit has quit IRC16:53
mriedemsweet16:54
*** ctrath has joined #openstack-nova16:55
*** rlrossit has joined #openstack-nova16:55
*** alex_klimov has quit IRC16:57
diana_clarkedansmith: I'm nervous about the hash change in test_objects.py. When I rebased that patch today, I need to update it & I don't fully understand why.16:57
diana_clarkedansmith: https://review.openstack.org/#/c/229964/14..15/nova/tests/unit/objects/test_objects.py16:57
*** mgoddard has joined #openstack-nova16:58
*** armax has joined #openstack-nova16:58
*** mgoddard__ has quit IRC16:58
*** marzif has quit IRC16:59
dansmithdiana_clarke: it's because you changed the args of the remotable, right?16:59
*** josecastroleon has quit IRC16:59
*** marzif has joined #openstack-nova17:00
diana_clarkedansmith: Oh!!! I think it's finally coming together for me. Thanks! Also, I need to revert that then.17:00
*** ihrachys has joined #openstack-nova17:00
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996417:02
*** stackdump has quit IRC17:02
*** e0ne has quit IRC17:02
mriedemjohnthetubaguy: fyi, soon we'll need a performance team CPL :) https://wiki.openstack.org/wiki/Meetings/Performance#Agenda_for_next_meeting17:04
mriedemi added conductor to that since it's come up twice now in the ops list as a performance issue17:05
*** marzif has quit IRC17:05
*** marzif has joined #openstack-nova17:06
*** pcaruana has quit IRC17:06
*** lykinsbd has quit IRC17:06
johnthetubaguymriedem: yeah, unsure how much the pymsql stuff has helped us there17:06
mriedemdid you see zombie mysqldb? https://review.openstack.org/#/c/243822/17:06
johnthetubaguymriedem: I know belliott was going to take a look at that17:06
*** jlanoux has quit IRC17:07
johnthetubaguyalaski: did we try pymysql yet, instead of the mysqldb stuff?17:07
alaskiI don't think we have17:07
*** baoli has joined #openstack-nova17:07
alaskiI haven't heard of it at least17:08
johnthetubaguyalaski: ah, so we only test pymysql in the gate now right, and its evetlet safe, but slower, but in theory it should mean the nova-cells workers and conductor get more concurrency per process than the old C lib stuff, in theory17:09
*** njirap has quit IRC17:09
dansmithjohnthetubaguy: why slower?17:09
dansmithI think it's slower at some things, but ...17:09
*** baoli_ has joined #openstack-nova17:09
dansmithjohnthetubaguy: you know we were able to halve a bunch of worker values because of that move right?17:09
johnthetubaguydansmith: yeah, I mean slower at some things, but its no longer blocking, as I understood it17:09
dansmithwe couldn't quite halve conductor, but we did reduce it17:09
johnthetubaguydansmith: right17:10
*** baoli has quit IRC17:12
*** exploreshaifali has joined #openstack-nova17:13
mriedemjohnthetubaguy: it's not only that, mysql-python doesn't support py317:14
mriedempymysql does17:14
*** gszasz has quit IRC17:15
*** rmart04 has joined #openstack-nova17:16
*** rk4n has quit IRC17:17
alaskijohnthetubaguy: our biggest issue is not really having a good testbed for comparing these things.  It would be amazing if we could get something setup and get some actual numbers17:18
*** mwagner has joined #openstack-nova17:20
mriedemwould testing it against the large ops job results help?17:20
*** ssurana has joined #openstack-nova17:20
*** marzif has quit IRC17:20
*** rmart04 has quit IRC17:21
*** rmart04 has joined #openstack-nova17:21
johnthetubaguymriedem: agreed about py34, I think that was the deciding vote for pymysql17:22
alaskimriedem: as long as not too much is faked out.  I don't remember what actually gets missed in the fake driver in that test.17:22
johnthetubaguymriedem: I think large ops is worth a try, I thought the issue was more about the unpredictable performance of the VM it lands on17:22
mriedemalaski: just the hypervisor stuff really17:23
johnthetubaguyI do wonder about getting like a single on metal box, and running some test in a loop to check for regressions, i.e. trying to remove the VM variability out of the equation. Or maybe just a 30GB standard VM would do the trick17:23
mriedemalaski: it hits all the same nova db stuff (except probably volumes)17:24
*** rcernin has joined #openstack-nova17:24
alaskimriedem: that seems like a good start then17:25
alaskiwe recently switched off the mysqldb driver and ran into issues with instance_update_and_get_original backing up in the nova cells service17:25
*** apoorvad has joined #openstack-nova17:25
dansmithmriedem: what do you mean? we're running largeops with pymysql now aren't we?17:26
alaskiand it wasn't much slower than the mysqldb version, but just enough to cause a wait when a lot came in17:26
*** rmart04_ has joined #openstack-nova17:26
*** afaranha has left #openstack-nova17:26
sdaguedanpb: do you have thoughts about why we're getting this error back from libvirt so much in the gate - "libvirtError: Error while building firewall: Some rules could not be created for interface vnet5: Failure to execute command '$EBT -t nat -A libvirt-P-vnet5  -p 0x0800 -j P-vnet5-ipv4' : 'Illegal target name 'P-vnet5-ipv4'.'."17:26
mriedemdansmith: i meant a DNM patch to test out https://review.openstack.org/#/c/243822/17:27
mriedemlike in devstack17:27
dansmithokay17:27
*** rmart04 has quit IRC17:28
*** rmart04_ is now known as rmart0417:28
*** nic has joined #openstack-nova17:30
*** suro-patz has joined #openstack-nova17:31
johnthetubaguyalaski: I suspect if we moved to pymysql it might be a different story?17:32
*** josecastroleon has joined #openstack-nova17:32
alaskijohnthetubaguy: it seems likely that we would get different performance characteristics.  Whether or not it's enough of an improvement to work for us is hard to say without some testing17:34
*** stackdump has joined #openstack-nova17:35
*** EinstCrazy has joined #openstack-nova17:35
sdagueis it just that the nwfilterDefineXML is async, and sometimes we get to the guest definition before that's taken17:36
johnthetubaguyalaski: agreed with the need to test it to find out17:36
*** otter768 has joined #openstack-nova17:36
*** paul-carlton has joined #openstack-nova17:36
danpbsdague: IIRC, "Illegal target name" means the target does not exist17:37
danpbsdague: which could be a sign that something else is racing with libvirt for firewall access17:37
dansmithjohnthetubaguy: we need an ack from you on this before we can land it: https://review.openstack.org/#/c/190399/17:38
dansmithand a release note about having to do it17:38
sdaguedanpb: right, we've got a bunch of nwfilterDefineXML calls (looks like 5 or so) to set up the nwfilter before guest start17:38
danpbsdague: seems like the version of libvirt you run there isn;t using ebtables --concurrent flag for locking17:38
johnthetubaguydansmith: taking a peak17:39
danpbso its possible newer libvirt would not have the problem17:39
dansmithjohnthetubaguy: could you also peek?17:39
sdaguedanpb: that is also true, but that's not what's failing in this case17:39
sdaguehttp://logs.openstack.org/86/230186/20/gate/gate-tempest-dsvm-postgres-full/f791773//logs/screen-n-cpu.txt.gz?level=TRACE#_2015-11-12_16_10_36_85917:40
*** MVenesio has joined #openstack-nova17:40
*** spandhe has joined #openstack-nova17:40
sdagueI am wondering if it's possible that the nwfilterDefineXML operations are all async, and thus backing up, then we request the guest gets created with a reference to the nwfilter, and devices we expect to be there are not17:41
*** MVenesio_ has quit IRC17:41
johnthetubaguydansmith: heh17:42
sdaguethis seems to be a distinct failure different from the ebtables --concurrent issue17:42
*** otter768 has quit IRC17:42
*** ccesario has quit IRC17:42
*** EinstCrazy has quit IRC17:42
sdaguemaybe just understanding who / what creates the P-vnet5-ipv4 table entry17:42
*** aix has joined #openstack-nova17:43
danpbsdague: if you look in libvirtd.log.gz you'll see17:43
danpbThis log message shows the bit where we create that table entry17:44
danpb2015-11-12 16:10:36.690+0000: 15521: debug : virCommandRunAsync:2282 : About to run /bin/sh -c 'EBT="/sbin/ebtables"17:44
*** tonytan4ever has quit IRC17:44
danpband the very next comment libvirt runs in that thread shows the failure17:44
danpb2015-11-12 16:10:36.742+0000: 15521: error : virCommandWait:2399 : internal error: Child process (/bin/sh -c 'EBT="/sbin/ebtables"17:44
danpbthis seems to rather suggest something else changing ebtables  has raced with libvirt and killed the chain17:45
dansmithnova messes with ebtables itself at times, right?17:45
danpbyeah, that would be the prime suspect17:46
sdaguewill any touch of ebtables scramble that?17:46
dansmithdanpb: are the calls we make to libvirt that touch that synchronous, or does the daemon async some of it?17:46
*** david-lyle has quit IRC17:46
danpbdansmith: iirc they're asynchronous17:47
dansmiththen that's got to be it17:47
danpblibvirt will serialize everything it does, but not serialize with external things17:47
danpbthis is why I fear lack of the --concurrent flag might be causing us the problem here17:47
*** ccesario has joined #openstack-nova17:48
sdaguedanpb: can it race with itself?17:48
danpbno, libvirt has a mutex that it serializes all its own calls on17:48
danpband the logs don't show any other libvirt threads doing iptables stuff in this time period anyway17:49
*** markmc has quit IRC17:49
*** e0ne has joined #openstack-nova17:50
openstackgerritBob Ball proposed openstack/nova: XenAPI: Cope with more Cinder backends  https://review.openstack.org/24478917:50
sdaguehttp://logs.openstack.org/86/230186/20/gate/gate-tempest-dsvm-postgres-full/f791773//logs/libvirt/libvirtd.txt.gz#_2015-11-12_16_08_30_95617:50
sdaguethat's the assync command17:50
sdaguewhich includes an rm_chain on libvirt-J-vnet517:51
sdaguethen 39ms later there is - http://logs.openstack.org/86/230186/20/gate/gate-tempest-dsvm-postgres-full/f791773//logs/libvirt/libvirtd.txt.gz#_2015-11-12_16_08_30_98517:51
sdaguewhich is where the fail is17:51
danpbsdague: libvirt-J-vnet5 is not the problem one - it was P-vnet5-ipv4  that has the fatal error17:53
*** cdent has quit IRC17:53
openstackgerritPaul Carlton proposed openstack/nova-specs: Cancel ongoing live migrations  https://review.openstack.org/22882817:53
danpbsdague: the error you point to is expected - libvirt is just cleaning up any historical cruft at that point & ignores that particular error17:54
openstackgerritBob Ball proposed openstack/nova: XenAPI: Cope with more Cinder backends  https://review.openstack.org/24478917:54
*** derekh has quit IRC17:54
*** IanGovett has quit IRC17:55
sdagueok, I guess we're running this big ebtables block here - http://logs.openstack.org/86/230186/20/gate/gate-tempest-dsvm-postgres-full/f791773//logs/libvirt/libvirtd.txt.gz#_2015-11-12_16_08_30_98517:56
sdagueand then about 150ms later get the failure - http://logs.openstack.org/86/230186/20/gate/gate-tempest-dsvm-postgres-full/f791773//logs/libvirt/libvirtd.txt.gz#_2015-11-12_16_10_36_74217:57
sdagueis that failure virCommandWait an async return, or another command?17:57
*** su_zhang has joined #openstack-nova17:58
sdaguethe only ebtables bits that nova does are the dhcp blocks in -t filter, so actual racing with libvirt which seems to only use -t nat seems weird18:01
sdagueand I actually think in the gate, we skip all the nova ebtables code paths entirely18:01
sdagueoh, nope there is one nat rule18:02
*** signed8bit is now known as signed8bit_ZZZzz18:03
*** su_zhang has quit IRC18:03
danpbsdague: presumably you meant to point to this in your first link  http://logs.openstack.org/86/230186/20/gate/gate-tempest-dsvm-postgres-full/f791773//logs/libvirt/libvirtd.txt.gz#_2015-11-12_16_10_36_69018:03
sdaguedanpb: yep, copy/paste link fail18:04
*** su_zhang has joined #openstack-nova18:04
* danpb hates the eb/iptables code in this particular version of libvirt - debugging the auto-generated shell scripts is horrible18:05
sdagueso, yeh, we don't hit the code path for -t nat in the gate on ebtables, and I'm not even clear if we hit the ebtables bits of nova in our config (I've never seen anything in ebtables outside of the nat table)18:05
danpbdo we do *anything* ebtables related in the gate18:06
danpbor pehraps nova-network/neutron do something18:06
sdagueI'm looking at the nova net code18:06
*** PaulMurray is now known as ptm_away18:07
sdaguewe definitely don't ever do this - https://github.com/openstack/nova/blob/02de17c72ac0f0824df2b3d460739a58cb6b5285/nova/network/linux_net.py#L73818:07
sdaguebecause that depends on a forced config we don't have18:07
*** claudiub has joined #openstack-nova18:08
sdagueoh, the floating ips code puts stuff in nat18:09
*** lykinsbd has joined #openstack-nova18:09
*** jichen has joined #openstack-nova18:09
sdaguethough not going near any of the interfaces listed18:10
sdaguehttps://github.com/openstack/nova/blob/02de17c72ac0f0824df2b3d460739a58cb6b5285/nova/network/linux_net.py#L832-L83618:10
*** lucasagomes is now known as lucas-afk18:10
*** paul-carlton has quit IRC18:11
*** sahid has quit IRC18:12
*** ccarmack has quit IRC18:13
*** sfinucan has quit IRC18:15
*** sfinucan has joined #openstack-nova18:15
*** ccarmack has joined #openstack-nova18:16
sdaguewell, I need to find lunch for reals. Anyway, trying to figure out if this is a preventable issue currently, because this one is pretty prevalent18:16
sdague8 fails in 24 hrs / 177 fails in 10 days18:16
*** josecastroleon has quit IRC18:16
*** ctrath has quit IRC18:16
sdaguehttps://bugs.launchpad.net/nova/+bug/150136618:17
openstackLaunchpad bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed]18:17
*** electrocucaracha has quit IRC18:17
*** vilobhmm has joined #openstack-nova18:19
*** ctrath has joined #openstack-nova18:19
*** bnemec has quit IRC18:21
*** ctrath has quit IRC18:21
*** rook- has joined #openstack-nova18:22
*** rlrossit has quit IRC18:22
*** ccarmack has quit IRC18:22
*** jistr has quit IRC18:22
*** sfinucan has quit IRC18:22
*** lykinsbd_ has joined #openstack-nova18:23
*** rk4n has joined #openstack-nova18:25
*** lykinsbd has quit IRC18:26
*** eme has joined #openstack-nova18:26
emeHi I am facing issue connecting to nova vm.18:26
*** achanda has joined #openstack-nova18:26
emeI have added icmp, tcp rule but still can't ping or ssh to it.18:26
emeCan anyone please help?18:26
*** ccarmack has joined #openstack-nova18:27
*** ctrath has joined #openstack-nova18:27
*** rlrossit has joined #openstack-nova18:27
*** rk4n has quit IRC18:27
*** tonytan4ever has joined #openstack-nova18:27
*** paul-carlton has joined #openstack-nova18:28
*** browne has joined #openstack-nova18:29
*** lykinsbd has joined #openstack-nova18:29
*** lykinsbd has quit IRC18:30
*** nic has quit IRC18:30
*** lykinsbd_ has quit IRC18:31
*** nic has joined #openstack-nova18:31
*** lykinsb__ has joined #openstack-nova18:31
*** yassine__ has quit IRC18:31
*** tdurakov has joined #openstack-nova18:32
tdurakovsdague, hi, are you around?18:32
*** su_zhang has quit IRC18:32
*** ev1lkry has joined #openstack-nova18:32
dansmithtdurakov: reportedly lunching at the moment :)18:32
ev1lkryeverything I have read states that when editing the policy.json for nova that no services need to be restarted but when doing "compute_extension:flavormanage": [ ], it still says that the polixy doesnt hallow compute_extension:flavormanage to be perfomed on my other users18:32
ev1lkryi've edited the /nova/policy.json file18:33
dansmithev1lkry: nova-api will need to be restarted at least18:33
tdurakovdansmith, oh, thank you:)18:33
*** smatzek has quit IRC18:33
ev1lkrydansmith that was it thank you18:34
*** ijuwang_ has joined #openstack-nova18:36
*** ijuwang has quit IRC18:36
*** ijuwang_ is now known as ijuwang18:37
*** smatzek has joined #openstack-nova18:37
*** yamahata has joined #openstack-nova18:38
mriedemev1lkry: are there docs that are wrong somewhere?18:39
*** fawadkhaliq has quit IRC18:39
openstackgerritClaudiu Belu proposed openstack/nova: Removes support for Hyper-V Server 2008 R2  https://review.openstack.org/23764318:40
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: adds os-win library  https://review.openstack.org/21210218:40
*** signed8bit_ZZZzz is now known as signed8bit18:41
openstackgerritMatt Riedemann proposed openstack/nova: Revert "Exclude all BDM checks for cells"  https://review.openstack.org/24269818:41
openstackgerritMatt Riedemann proposed openstack/nova: WIP: cells: create block device mappings in API cell  https://review.openstack.org/24264918:41
openstackgerritMatt Riedemann proposed openstack/nova: Make BlockDeviceMapping object support uuid  https://review.openstack.org/24260318:41
openstackgerritMatt Riedemann proposed openstack/nova: Add uuid column to BlockDeviceMapping  https://review.openstack.org/24260218:41
*** zul has quit IRC18:41
ev1lkrydansmith : I have read in some non openstack.org sites that you do not need to restart services when editing the policy.json file however the docs i've see doesnt say either for example: http://docs.openstack.org/kilo/config-reference/content/policy-json-file.html18:42
*** zul has joined #openstack-nova18:42
alaskiev1lkry: for some projects that might be true, like Glance I think, but not for Nova18:42
*** EmilienM has quit IRC18:42
*** ociuhandu has quit IRC18:42
*** sfinucan has joined #openstack-nova18:43
*** electrocucaracha has joined #openstack-nova18:44
*** su_zhang has joined #openstack-nova18:45
mriedemev1lkry: docs bug reported https://bugs.launchpad.net/openstack-manuals/+bug/151572918:45
openstackLaunchpad bug 1515729 in openstack-manuals "Appendix A. The policy.json file in OpenStack Configuration Reference - liberty" [Undecided,New]18:45
*** djchou has joined #openstack-nova18:45
*** EmilienM has joined #openstack-nova18:45
ev1lkryawesome, thanks mridem18:46
mriedemsdague: dims: when we do novaclient 3.0 we should also drop py26 support18:46
mriedemsince oslo wants to do that anyway18:46
*** e0ne has quit IRC18:47
*** su_zhang has quit IRC18:47
*** rmart04 has quit IRC18:47
*** danpb has quit IRC18:48
sfinucanmriedem: could you have a peek at this modification to the CPU thread pinning spec? It's not "trivial", but johnthetubaguy has signed off on it and the changes are minimal https://review.openstack.org/#/c/244198/3/specs/mitaka/approved/virt-driver-cpu-pinning.rst18:48
sfinucanfigure I can't get any code merged on that until the specs line up18:49
*** lpetrut has quit IRC18:49
sfinucanalso, johnthetubaguy: should I/can I rework the original spec to remove the references to CPU thread pinning? What's there now is out-of-date for a number of reasons (policies have changed names, wrong release etc.)18:50
mriedemsfinucan: who originally approved it?18:50
mriedemi'd ask them18:50
*** Marga_ has quit IRC18:50
mriedemi don't have context18:50
sfinucanI thought you did?18:50
mriedemi think i approved a re-approval from liberty18:50
*** yamahata has quit IRC18:50
sfinucanAh, that's my mistake18:50
sfinucanOK, I'll do that. Sorry for the interruption18:51
mriedemnp18:51
*** su_zhang has joined #openstack-nova18:51
*** tdurakov has quit IRC18:52
*** lykinsbd has joined #openstack-nova18:54
*** paul-carlton has quit IRC18:54
*** tdurakov has joined #openstack-nova18:55
*** mnestratov has quit IRC18:55
*** electrocucaracha has quit IRC18:56
*** tonytan_brb has joined #openstack-nova18:56
*** lykinsb__ has quit IRC18:56
*** lykinsbd has quit IRC18:56
*** Piet has quit IRC18:57
*** lykinsbd has joined #openstack-nova18:57
sfinucanalaski: Would you be able to look at this spec? https://review.openstack.org/#/c/244198/3/18:58
*** mriedem has quit IRC18:58
*** angdraug has quit IRC18:58
sfinucanIt's an extension of the original CPU pinning spec, which you reviewed way back when. Not much has changed, but I do need another +2 from someone familar with the idea https://review.openstack.org/#/c/93652/18:59
*** tonytan4ever has quit IRC18:59
alaskisfinucan: I can look at it, but I'd really like to see input from danpb since he authored the original18:59
*** tonytan4ever has joined #openstack-nova19:01
*** tonytan_brb has quit IRC19:01
sfinucanalaski: fortunately, he's already +2d it :)19:01
alaskihmm, I'm not seeing that19:03
*** Piet has joined #openstack-nova19:04
*** mriedem has joined #openstack-nova19:05
*** kmartin has quit IRC19:05
*** kmartin has joined #openstack-nova19:06
sfinucanalaski: Ah, my mistake. Wrong name19:07
sfinucandrat19:07
sfinucanI'll reach out to danpb19:07
*** ctrath has quit IRC19:08
*** Piet has quit IRC19:09
*** ctrath has joined #openstack-nova19:11
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Last sync from oslo-incubator  https://review.openstack.org/24481819:11
openstackgerritDavanum Srinivas (dims) proposed openstack/python-novaclient: Last sync from oslo-incubator  https://review.openstack.org/24482719:12
*** ArxCruz has joined #openstack-nova19:12
*** aix has quit IRC19:15
*** ArxCruz has quit IRC19:17
*** lykinsbd has quit IRC19:17
*** sfinucan has quit IRC19:18
*** sfinucan has joined #openstack-nova19:18
openstackgerritGábor Antal proposed openstack/nova: Replaced deprecated timeutils methods  https://review.openstack.org/24117919:19
*** lykinsbd has joined #openstack-nova19:19
*** achanda has quit IRC19:21
*** achanda has joined #openstack-nova19:21
*** su_zhang has quit IRC19:28
openstackgerritHans Lindgren proposed openstack/nova: Use TimeFixture from oslo_utils to override time in tests  https://review.openstack.org/23761619:32
*** korzen has joined #openstack-nova19:32
*** ccarmack has quit IRC19:33
*** bnemec has joined #openstack-nova19:33
*** pratikmallya has quit IRC19:35
*** otter768 has joined #openstack-nova19:37
*** lykinsbd has quit IRC19:38
*** ijw has joined #openstack-nova19:39
*** med_ is now known as openstackgerrit_19:39
*** angdraug has joined #openstack-nova19:40
*** openstackgerrit_ is now known as med_19:40
*** pm90_ has joined #openstack-nova19:40
*** carolynvs has joined #openstack-nova19:40
annegentlehey all, I'm looking in the nova codebase for an example of an already-microversioned API call.19:41
annegentleso far I think I'm seeing just RPC API calls that are versioned19:41
mriedemannegentle: sec19:41
annegentlemriedem: cool thanks19:41
mriedemannegentle: https://review.openstack.org/#/c/168966/19:41
mriedemthat one was pretty simple19:42
mriedemhttps://review.openstack.org/#/c/168966/8/nova/api/openstack/compute/plugins/v3/fixed_ips.py19:42
sdaguedansmith: in looking at ebtables rules that are on a gate rule not by libvirt, it's one PREROUTING rule for floating ips19:42
*** alex_klimov has joined #openstack-nova19:42
sdaguehttp://logs.openstack.org/94/244194/6/check/gate-grenade-dsvm-multinode-nv/fd8b32f/logs/worlddump-2015-11-12-141039-cinder_resources_created.txt.gz - -p IPv4 --logical-in br100 --ip-src 10.1.0.2 --ip-dst ! 10.1.0.0/20 -j redirect19:42
*** otter768 has quit IRC19:42
annegentlemriedem: dere it is. Ok carolynvs so looking at https://review.openstack.org/#/c/168966/8/doc/api_samples/versions/versions-get-resp.json that's the JSON19:42
sdaguewhich looks like we delete and reinsert every time a floating ip is added, which seems a little odd19:42
annegentlecarolynvs: for the response. Hm...19:42
carolynvsThanks annegentle: I'm still getting used to gerrit :)19:42
annegentlecarolynvs: heh yes.19:43
*** ijw has quit IRC19:43
annegentlemriedem: so to request a particular version, where do you put the header? (Or maybe I misunderstand)19:43
carolynvsI'm guessing that I need to include "X-OpenStack-Nova-API-Version: 2.4" in the header of my request19:44
dansmithsdague: I'm not familiar enough with ebtables to know what that means19:44
dansmithsdague: in iptables there would be a redirect --to $something19:44
mriedemcarolynvs: right19:44
sdagueright, I'm not hugely familiar as well, however it seems odd that it seems to be called on every floating ip add19:45
sdaguebut it's always the same rule19:45
carolynvsso I should go off of the sample json version response when figuring out what microversion should be used for new changes discussed in the release notes19:45
dansmithsdague: never a different IP?19:45
sdaguewhich means it's flapping ebtables for no particular reason19:45
dansmithsdague: yeah, that's weird indeed19:45
cburgesssdague Is it always the same floating IP?19:45
dansmithsdague: I think cburgess knew something about ebtables at one point19:45
sdaguewell, we have 2 floating ips there19:45
dansmithspeak of the devil19:45
sdagueand one rule19:45
cburgessI should have kept my mouth shut.... :P19:45
sdague--ip-dst !19:45
sdaguecburgess: no, you totally shouldn't, because the dark magics should be explained19:46
annegentlemriedem: then if I'm reading rest_api_version_history.rst right, do you get the added keypair type for 2.2 also? Or does your request for keypair types need to have X-OpenStack-Nova-API-Version: 2.2" in it?19:46
cburgessLOL19:46
cburgesssdague OK let me look at this log you posted...19:46
sdaguecburgess: well, it's a dump19:46
mriedemannegentle: it's cumulative19:46
mriedemso 2.4 includes 2.2 and 2.319:46
annegentlemriedem: ok so 2.4 would give me the rest19:47
annegentlemriedem: ok thanks!19:47
mriedemyup19:47
cburgesssdague The output of ebtables -t nat -L that I'm looking at, where does this come from? Compute node, or nova-network node?19:47
*** greghaynes_ has quit IRC19:47
sdaguecburgess: this is an all in one for the gate19:47
cburgessOK19:47
cburgessand 10.1.0.2 is the fixed IP of the instance?19:47
sdagueone sec19:48
*** mc_nair has quit IRC19:48
sdagueit is the fixed ip for one instance19:48
sdaguethere is a 10.1.0.3 instance as well19:48
sdagueyeh, that's the super confusing thing19:49
cburgessOK so... the specific test we are running here is what exactly? Launch instance, add floating, remove floating add floating? And the question is why the etables stuff?19:49
sdagueat this point in the test we've launched 2 instances, add floating ips, and pinged them19:49
*** greghaynes_ has joined #openstack-nova19:49
sdaguemostly just trying to understand19:50
sdagueis it odd that there aren't rules for both of them?19:50
*** rmart04 has joined #openstack-nova19:51
cburgessSeems that way. Let me look at what this rule is actually doing.19:51
*** carolynvs has left #openstack-nova19:51
cburgesssdague dansmith yeah this is weird... redirect in an ebtables rule triggers a MAC address re-write.19:52
dansmithoh19:52
dansmithis that to make floatings work at L2?19:52
suro-patzrlrossit: please check https://review.openstack.org/#/c/243825/, need some input from you19:52
cburgessIf I recall it basically changes the destination MAC to the interface the packets was recieved on, which would be the bridge in this case I think.19:52
cburgessdansmith sdague Let me go see where we configure this and why...19:53
tdurakovsdague, ping19:54
sdaguetdurakov: what's up?19:54
*** stackdump has quit IRC19:54
tdurakovsdague, hi, need your help with https://review.openstack.org/#/c/243013/19:54
cburgesssdague This is on master or a stable branch we are seeing this?19:55
tdurakovif everything is ok, it would be great to merge this, so i could debug config switching19:55
sdaguecburgess: this is liberty, for unrelated reasons the only place we're regularly dumping this state in the middle of a run is on grenade testing19:56
*** electrocucaracha has joined #openstack-nova19:56
sdagueso I have a ton of these dumps around in the gate to look at what "normal" looks like19:56
sdaguethis run passes19:56
sdaguetdurakov: +219:57
cburgesssdague OK so in the test we have 2 instances in the same network each with a floating IP but only one of these ebtables rules?19:57
sdagueyes19:58
cburgesssdague So yeah it looks like we should have 2 rules so thats odd.19:58
tdurakovsdague, thank you!19:58
sdaguewhich I only just noticed in staring at this19:58
cburgessSo the original bug from vish is https://bugs.launchpad.net/nova/+bug/109698519:58
openstackLaunchpad bug 1096985 in OpenStack Compute (nova) "Floating ips on separate interface + external gateway does not work" [Medium,Fix released] - Assigned to Vish Ishaya (vishvananda)19:58
sdagueit actually doesn't look like n-net did anything for the second guest19:58
*** whenry__ has joined #openstack-nova19:58
sdaguewhich is even weirder19:58
cburgessI have a general idea of what vish is doing here and why. But yeah... it really seems like we should have two of those. Basically you should have 1 rule for each fixed IP in this configuration.19:59
sdagueyeh, the code looks like it does that19:59
sdaguehowever, something else went wrong here20:00
sdagueoh... wait20:00
sdaguenevermind, I picked a terrible job to look at20:00
cburgessOK20:00
sdaguethis is actually the multinode job20:00
sdaguethe other guest is on the other node20:00
sdaguegdi20:00
cburgesssdague Also it seems like we aren't getting the full rule output there.20:01
*** doug-fish has quit IRC20:01
cburgessAs in the full rule we are adding is...20:01
sdaguecburgess: no?20:01
cburgess--logical-in %s -p ipv4 --ip-src %s  ! --ip-dst %s -j redirect --redirect-target ACCEPT20:01
sdagueif you know how I should invoke ebtables to get something better, let me know, I'll update the dump tool20:01
cburgessThe output seems to stop after -j redirect.20:01
mriedemgaryk: you have a couple of small issues to cleanup in https://review.openstack.org/#/c/165750/ and then i'm +220:01
cburgesssdague: nevermind ACCEPT is the default target so I suspect the tool just isn't showing you the default.20:02
sdaguecburgess: hmmm... I'm running: sudo ebtables -t nat -L20:02
cburgesssdague Yeah that should be fine.20:02
cburgessThats what I use.20:02
cburgesssdague So yeah if the other VM is on a different node then that makes sense why we only have a single rule.20:03
*** ihrachys has quit IRC20:03
sdagueso, actually in the all in one case I'm seeing no rules20:04
*** artom has joined #openstack-nova20:04
*** sfinucan has quit IRC20:04
*** sfinucan_ has joined #openstack-nova20:04
*** ihrachys has joined #openstack-nova20:04
*** sfinucan_ has quit IRC20:04
*** yassine__ has joined #openstack-nova20:04
*** lykinsbd has joined #openstack-nova20:04
*** iurygregory has quit IRC20:04
*** sfinucan has joined #openstack-nova20:04
sdagueok, anyway, this makes a bit more sense now20:05
cburgesssdague dansmith So yeah in general what that rule is doing is saying.. if we get a packet from a fixed IP with a floating IP and VM is talking to a destination outside the local network and the floating IP is on a different device rewrite the L2 frame so that its coming from the device with the floating IP.20:05
*** markmc has joined #openstack-nova20:05
cburgessIts kind of a weird overly specific thing.20:05
cburgessBut whatever.20:05
*** ericksonsantos has quit IRC20:05
*** doug-fish has joined #openstack-nova20:05
cburgesssdague dansmith The idea is that in a multi-host setup you could have a dedicate VLAN/NIC for floating IPs thats separate from where all your tenant traffic is.20:06
*** rmart04 has quit IRC20:06
*** korzen has quit IRC20:06
dansmithcburgess: surely this is part of some thing to avoid people being confused about floating ips on the same L2 where the NAT rules would otherwise kinda break us20:06
*** artom_ has quit IRC20:06
cburgessdansmith Its actually to guard against asymetric routing.20:07
dansmithwhere you take one path out and another path in, right?20:07
cburgessYes20:07
dansmithyeah20:07
*** IanGovett has joined #openstack-nova20:07
cburgessSo basically.. internet -> floating ip -> VM -> tenant gateway20:07
*** electrocucaracha has quit IRC20:07
cburgessThats asymetric20:07
dansmithyeah20:08
cburgessWith this rule you get a return back out the interface that hosts the floating IP.20:08
*** lpetrut has joined #openstack-nova20:08
sdagueok, so I think the code makes sense now. I'm slightly surprised that when we are running all in one their appear to be no rules created20:08
cburgesssdague Its only created in a specifc topo20:09
cburgesssdague brb20:09
sdagueoh, right20:09
mriedemdims: was this the thing zzzeek was fixing in oslo.db? http://logs.openstack.org/04/240804/3/gate/gate-osc-dsvm-functional/9a34c38/logs/screen-n-api.txt.gz?level=TRACE#_2015-11-11_22_52_34_53320:09
sdaguebecause we only run multihost on the actual multinode job20:09
sdagueso, right, I think that means on all non multinode jobs nova never runs ebtables in the gate20:10
*** mc_nair has joined #openstack-nova20:11
dansmithsdague: it's nice that we test reality now :)20:11
sdaguedansmith: does that seem like a reasonable hypothesis20:11
dansmithdansmith: yeah20:12
sdaguedansmith: sure, but it also means that all the ebtables explodes that were blamed on nova20:12
sdaguenova never runs the command20:12
cburgesssdague bac20:17
cburgesser.. back20:17
openstackgerritStephen Finucane proposed openstack/nova: Add 'hw_cpu_threads_policy' to ImageMetaProps  https://review.openstack.org/20264720:17
openstackgerritStephen Finucane proposed openstack/nova: Add 'cpu_policy' and 'cpu_threads_policy' fields  https://review.openstack.org/20264820:17
openstackgerritStephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=avoid' filtering  https://review.openstack.org/20264920:17
openstackgerritStephen Finucane proposed openstack/nova: trivial: Add some logs to 'numa_topology_filter'  https://review.openstack.org/20265020:17
openstackgerritStephen Finucane proposed openstack/nova: Add 'hw:cpu_threads_policy=require' scheduling  https://review.openstack.org/20265120:17
*** raildo is now known as raildo-afk20:17
cburgessWell nova runs ebtables for a number of different reasons.20:18
cburgesssdague So I don't think we can say it "never" runs ebtables.20:18
sdaguecburgess: in an all in one without multihost set20:19
cburgessand if you mean those failures from last year that jogo looked it I'm pretty sure it was ebtables in nova since i wrote the fix for it.. :P20:19
cburgesssdague Fair. Some jobs might not run it.20:19
sdaguecburgess: no, I mean a current set of fails20:20
*** nikhil has joined #openstack-nova20:20
sdagueBug 1501366 - libvirtError: Error while building firewall: Some rules could not be created for interface20:20
openstackbug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed] https://launchpad.net/bugs/150136620:20
*** tonytan4ever has quit IRC20:22
*** kfarr has quit IRC20:24
tdurakovsdague, could you plus 2 once again? https://review.openstack.org/#/c/243013/720:24
openstackgerritjichenjc proposed openstack/nova: Change Invalid exception to a specified exception  https://review.openstack.org/23250620:24
sdagueactually, we do a log.debug whenever we run ebtables because of oslo.process20:25
cburgesssdague I'm not convinced that my retry fix is related. Also people are conflating my fix and what libvirt 1.2.11 buys you.20:25
sdaguecburgess: right, I'm not actually saying anything about your fix20:26
sdagueI actually just want to figure out what's up with some of these fails20:26
cburgesssdague Ok actually.... I'm wrong.. the log posted in https://bugs.launchpad.net/nova/+bug/1501366 is in fact the exact error my fix addresses.20:27
openstackLaunchpad bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed]20:27
cburgessIf my fix wasn't back ported to juno then that would be the cause.20:27
bauzasmriedem: dansmith: I can still see some BDM changes20:27
cburgesssdague Wait sorry I meant https://bugs.launchpad.net/nova/+bug/150155820:28
openstackLaunchpad bug 1501558 in OpenStack Compute (nova) "nova-net: libvirtError: Error while building firewall: Some rules could not be created for interface: Unable to update the kernel" [High,Confirmed]20:28
bauzasmriedem: dansmith: since I was on PTO yesterday, is there still a discussion for that - given also the convo we had during today's meeting?20:28
sdaguecburgess: right, that's another one20:28
mriedembauzas: i was still debugging20:28
bauzasmriedem: tbh, I'm reviewing the changes for ramping up on BDMs20:29
cburgesssdague Yeah sorry to many similar bugs here. So yeah my fix is needed for 1501558 in Juno (mriedem you might care about that). But 1501366 look to be something else.20:29
*** electrocucaracha has joined #openstack-nova20:29
mriedemwe're killing juno next week20:30
dansmithin a field20:30
dansmithto gangsta rap20:30
mriedemdie mofo die mofo die20:30
sdagueby looking at the n-net logs it looks like we don't tend to run any ebtables commands during a grenade run20:30
dansmithbauzas: there wasn't much that wasn't on the review20:31
sdagueand only a few tempest tests trigger it happening during a longer run20:31
dansmithbauzas: apparently what I did was not what the others wanted, but I'm so distracted I haven't tried to figure out why20:31
bauzasmeh20:31
mriedemnow i'm listening to https://www.youtube.com/watch?v=qNa6xImZeo420:32
mriedemso i'm distracted20:32
sdaguemriedem: heh20:32
*** ijw has joined #openstack-nova20:33
*** kmartin has quit IRC20:35
openstackgerritStephen Finucane proposed openstack/nova-specs: Move thread policy features between specs  https://review.openstack.org/24485720:36
openstackgerritAnne Gentle proposed openstack/nova: docs: update the API faults concept guide  https://review.openstack.org/24306620:36
*** tonytan4ever has joined #openstack-nova20:36
*** ijw_ has joined #openstack-nova20:36
*** ijw has quit IRC20:37
*** ctrath has quit IRC20:40
*** signed8bit is now known as signed8bit_ZZZzz20:41
mriedemrlrossit: damn your keen eye https://review.openstack.org/#/c/243852/20:41
mriedemi don't want to write unit tests for that20:41
*** signed8bit_ZZZzz is now known as signed8bit20:43
*** ociuhandu has joined #openstack-nova20:44
*** ctrath has joined #openstack-nova20:44
*** pixelbeat has quit IRC20:49
openstackgerritStephen Finucane proposed openstack/nova: test_fields: Remove all 'Enum' subclass tests  https://review.openstack.org/24437220:52
openstackgerritStephen Finucane proposed openstack/nova: tests_fields: Remove most enum-type field tests  https://review.openstack.org/24437020:53
*** rlrossit has quit IRC20:53
*** eme has quit IRC20:53
*** tdurakov has quit IRC20:56
*** rlrossit has joined #openstack-nova20:58
*** gyee has joined #openstack-nova20:58
*** lpetrut has quit IRC20:58
mriedemugh, who wants to pad stats? https://review.openstack.org/#/c/232827/20:58
*** su_zhang has joined #openstack-nova20:59
*** lpetrut has joined #openstack-nova20:59
cburgessmriedem sdague I posted a comment to https://bugs.launchpad.net/nova/+bug/1501558 to try and clarify the retry loop logic vs --concurrent and libvrit 1.2.11.21:01
openstackLaunchpad bug 1501558 in OpenStack Compute (nova) "nova-net: libvirtError: Error while building firewall: Some rules could not be created for interface: Unable to update the kernel" [High,Confirmed]21:01
*** yassine__ has quit IRC21:02
*** su_zhang has quit IRC21:03
sdaguemriedem: some of those changes annoy me a little, like filesystem => file system21:04
sdagueas the no space version is fine21:04
mriedemsdague: i know,21:05
*** djchou has quit IRC21:05
mriedemsdague: i was about to say in my +2 comment that i felt bad acting like i was supporting this type of change21:05
*** kmartin has joined #openstack-nova21:07
sdaguewell, I commented21:07
*** jichen has quit IRC21:07
*** pm90__ has joined #openstack-nova21:08
*** pm90__ has quit IRC21:08
*** kmartin has quit IRC21:08
*** kmartin has joined #openstack-nova21:08
*** suro-patz has quit IRC21:10
mriedemsdague: also https://review.openstack.org/#/c/218730/21:11
mriedemi noted the commit that removed the usage in my comment21:11
mriedemwhich should have been in the commit message21:11
*** pm90_ has quit IRC21:11
*** Marga_ has joined #openstack-nova21:12
sdaguethat one I can get behind21:12
*** smatzek has quit IRC21:12
mriedemalso easy https://review.openstack.org/#/c/241976/21:16
*** tdurakov has joined #openstack-nova21:17
dansmithgot it21:18
mriedemBobBall: so xenproject ci was passing force_config_drive=always here? https://review.openstack.org/#/c/237762/21:20
mriedemor where is the failure?21:20
cburgesssdague mriedem So for https://bugs.launchpad.net/nova/+bug/1501366 it is the ebtables race that we originally fixed in https://bugs.launchpad.net/nova/+bug/1316621 but from the other side (aka libvirt is hitting the race, not nova). I posted a comment on how to fix it. Short version libvirt 1.2.11 isn't good enough.21:21
openstackLaunchpad bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed]21:21
openstackLaunchpad bug 1316621 in OpenStack Compute (nova) "ebtables calls can race with libvirt" [Medium,Fix released] - Assigned to Chet Burgess (cfb-n)21:21
sdaguecburgess: thanks much for the input there!21:22
*** Piet has joined #openstack-nova21:22
mriedemBobBall: nvm, i see it here http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/62/237762/9/15586/logs/nova.conf21:22
mriedemBobBall: ok, but that option was deprecated for a release, and the change wasn't xen specific, so we missed it, sorry21:22
bauzasmriedem: AFAIK, BobBall is on UK TZ so I guess he would reply by tomorrow :)21:23
*** achanda has quit IRC21:23
mriedemthat's fine21:25
mriedemif yo'ure on irc, you're fair game for pings21:25
mriedemalso commented in the review21:25
*** stevemar_ has quit IRC21:25
*** achanda has joined #openstack-nova21:26
*** achanda has quit IRC21:26
*** su_zhang has joined #openstack-nova21:26
mriedemsfinucan: weren't you supposed to add danpb to this? https://review.openstack.org/#/c/244370/21:26
*** cn28h has joined #openstack-nova21:26
*** achanda has joined #openstack-nova21:27
*** diogogmt has quit IRC21:27
sfinucanI added him to the bug21:27
mriedemi doubt anyone follows bug email21:27
mriedemi added him to the review21:27
sfinucanAh, OK. I didn't know that21:27
sfinucanThere's two approaches. I'll stick him on the other21:27
*** mdrabe has quit IRC21:29
*** otter768 has joined #openstack-nova21:38
*** pm90_ has joined #openstack-nova21:39
*** shardy has quit IRC21:39
*** claudiub has quit IRC21:40
*** pm90__ has joined #openstack-nova21:42
*** otter768 has quit IRC21:42
*** pm90_ has quit IRC21:44
*** thangp has quit IRC21:44
alaskidansmith: no rush, but when you have some time your thoughts on the comment on line 321 at https://review.openstack.org/#/c/211753/27 would be appreciated21:48
alaskibasically, what to do about serialized objects in the db and bumping object versions21:48
dansmithalaski: okay21:49
sdaguecburgess: so I'm going to say a thing, and you are going to hate me, and I'll regret it. But I'll say it anyway.21:50
*** jhesketh has quit IRC21:50
cburgesssdague OK....21:51
mriedemsdague thinks justin bieber is over rated21:51
mriedemthere, he said it21:51
sdagueit almost feels like we should have a thing replace ebtables with a version that is a shell script wrapper to the real one that includes a flock21:51
*** ZZelle has quit IRC21:51
cburgessmriedem No argument here21:51
sdaguethen we stop having to play this giant dance of who's got what version that supports what thing21:52
* sdague waits for the hate21:52
cburgesssdague OK so... I don't hate you, but I hate that. I don't think its reasonable for openstack to say "here replace your ebtables command and introduce all these potential issue around upgradability"21:52
sdagueyeh, fair21:52
sdaguewe could document it as a way around the problem for people that can't upgrade the world21:53
cburgesssdague So if we want to provide a script some place in tree and say "if you can't run version X of this and Y of this you could also use this script" I think thats great.21:53
*** ZZelle has joined #openstack-nova21:53
cburgessBut forcing/requiring that script... I don't see it.21:53
*** jhesketh has joined #openstack-nova21:53
sdagueyeh, that seems like a plan21:54
*** achanda has quit IRC21:54
cburgesssdague We might even be able to be slick and look at how its called in libvirt and might be able to use just path tricks to use the script.21:54
*** achanda has joined #openstack-nova21:54
*** e0ne has joined #openstack-nova21:54
sdaguehmmmm21:54
sdaguenow that's a thought21:54
*** aysyd has quit IRC21:54
cburgesssdague Still wouldn't be automatic. Would still be a documented thing for deployer to go change libvirt path to find script is they don't have the right versions.21:55
cburgessBut thats way more palatable then replacing a system binary.21:55
cburgesssdague So are you saying we want to fix nova to do the right thing with the right versions and support a fallback to retry+script with docs around using the work around?21:56
sdagueyeh21:56
cburgessI can get behind that.21:56
cburgessAs a goal for M? Or back port as well?21:56
sdaguewell, master first21:56
sdaguedecisions can be made later after that seems to work21:57
cburgesssdague bug or spec?21:57
sdaguethe fallback script will get reasonably tested because we can't get to a new enough libvirt this cycle, so we'd use that in the gate to stop randomly exploding21:57
*** spandhe has quit IRC21:58
sdaguecburgess: this is fine to be the solution to the bug at hand I think21:58
sdaguedansmith / mriedem opinions?21:58
dansmithsdague: I think you're going to hell for this21:58
*** Marga_ has quit IRC21:58
mriedemi'm leaving for parent / teacher conferences21:58
dansmithnow, let me read the scrollback21:58
cburgessLOL21:58
mriedemi'll ask them what they think21:58
sdaguedansmith: I'm not convinced your answer will be different once you read it21:58
dansmithso, you can't use path tricks from inside nova to cause libvirt to use the script,21:59
cburgessdansmith He is going to hell, but it is the least ugly choice.21:59
dansmithyou'll need it to be set for the daemon21:59
cburgessdansmith Right it would be documented.21:59
dansmithcburgess: I'm willing to sacrifice sdague's eternal soul, how about you?21:59
cburgessThere would have to be a doc that says "If you can't run with these versions you should do the following...."21:59
sdaguealso, it looks like the commands out of the daemon are not using path21:59
cburgessdansmith I'm agree with sdague so I think my soul is on the line too.21:59
*** mriedem has quit IRC22:00
sdaguethey are /sbin/ebtables directly22:00
cburgesssdague As in its just 'ebtable' or its calling with full path?22:00
dansmithcburgess: if your soul isn't on the review, you're safe :)22:00
cburgessDamn.22:00
cburgessOK.22:00
sdaguehttp://logs.openstack.org/86/230186/20/gate/gate-tempest-dsvm-postgres-full/f791773//logs/libvirt/libvirtd.txt.gz#_2015-11-12_16_00_43_48022:00
cburgesssdague Well.. thats not a guarantee.. I would check the code first to be sure.22:00
sdaguecburgess: sure, but I'm pretty sure this is generating that shell script as such22:01
dansmiththey could be generating the shell with the path they've resolved,22:01
dansmithbut it wouldn't make a lot of sense22:01
cburgessRight exactly22:01
cburgessOh find... let me load the code and check...22:01
cburgessGive me a minute.22:01
cburgessdansmith sdague Actually path trick or not doesn't matter. Thats an implementation details of the proposed solution right?22:01
sdagueyeh22:02
dansmithwell,22:02
cburgesssdague Also instead of flock shouldn't we use oslo locking for this?22:02
sdagueit looks like it's resolved at compile time22:02
dansmithit means that people will have to hack their environment22:02
sdaguecburgess: no, because this would be a stupid shell script22:02
cburgessdansmith Yes22:02
cburgesssdague Ermm...22:02
dansmithcburgess: I expect there's zero chance we'll be okay with that22:02
dansmithsince a yum update would break you22:02
cburgessdansmith I agree22:02
cburgessBut... there is no solution to this problem without the proper versions.22:03
dansmithwell,22:03
sdaguedansmith: right, that's why it would be a script in tree with instructions for manual install22:03
dansmiththere is of course22:03
sdaguenot something we'd do22:03
dansmithwe could either:22:03
cburgessSo... we propose fixing nova to do the right thing with the proper versions and also providing a work around.22:03
dansmith1. Ask libvirt to take it as a runtime argument in config22:03
sdagueand you could build a package with triggers that does the replace22:03
dansmith2. Ask libvirt to flock the ebtables binary it's using, and we do the same22:03
dansmithwell, maybe we can't flock the binary22:04
sdaguedansmith: new libvirt already does that22:04
dansmithbut some location22:04
dansmithoh22:04
sdaguethe --concurrency flag22:04
dansmithsorry I missed that22:04
sdaguebut you need 1.2.1122:04
cburgessdansmith OK but given libvirt has already fixed the problem by using --concurrent and it would still require everyone to get a new version of libvirt.. why not just use --concurrent?22:04
dansmithso this only matters for older libvirt?22:04
dansmithsorry I glazed over that22:04
sdagueand an ebtables that's new enough to support it22:04
cburgessdansmith Yeah22:04
*** Marga_ has joined #openstack-nova22:04
sdagueyeh, this is a "well, you can't upgrade, you can do this thing instead even though it's hacky"22:04
dansmithwell, then if it's a hack for older systems, then sure, a sample script and a box of chocolates22:05
dansmithyep22:05
*** jamielennox|away is now known as jamielennox22:05
*** vilobhmm has quit IRC22:05
*** exploreshaifali has quit IRC22:05
cburgessWhere old = gate22:05
*** vilobhmm has joined #openstack-nova22:05
sdaguecburgess: also, ebtables path is set in configure.ac it looks like22:05
sdagueso compile time22:05
sdaguebut variable per system22:05
*** vilobhmm has quit IRC22:06
cburgessOK so they have to replace the binary then. Oh well.22:06
cburgesssdague dansmith OK so we agree, its ugly, and hacky, and not great, but best we can do?22:08
*** MVenesio has quit IRC22:08
dansmithcburgess: actually I think we agree that people that haven't upgraded will take whatever god-awful solution we poop out, which I'm okay with :)22:08
cburgessdansmith OK22:09
cburgesssdague How fast did you want this?22:09
sdaguecburgess: if you are signing up for a thing, and I don't have to write this, as soon as you can :)22:09
cburgesssdague Well the how long determines if I can do it... but... I did loose ATC and need to land something again... :P22:10
sdaguefwiw, --concurrent is supported in ebtables as old as trusty, so I don't actually understand why we don't use it22:10
cburgessAre you sure?22:10
cburgessI thought I looked and it wasn't in trusty.22:10
sdagueos1:~> ebtables --version22:10
sdagueebtables v2.0.10-4 (December 2011)22:10
sdagueos1:~> ebtables --help | grep concur22:11
sdague--concurrent                  : use a file lock to support concurrent scripts22:11
cburgessroot@mcp1.trial6.mc:/home/cfb$ dpkg -l | grep ebtables22:11
cburgessii  ebtables                                     2.0.9.2-2ubuntu2                             Ethernet bridge frame table administration22:11
cburgessroot@mcp1.trial6.mc:/home/cfb$ ebtables --concurrent22:11
cburgessUnknown argument: '--concurrent'.22:11
cburgessI don't know where you got that ebtables from.22:11
cburgessI don't think thats the default trust one.22:11
*** Marga_ has quit IRC22:11
cburgesstrusty22:11
sdagueii  ebtables                             2.0.10.4-3ubuntu1       amd64                   Ethernet bridge frame table administration22:11
cburgessWTF22:11
sdagueyou didn't take updates :)22:11
cburgessCan you show me... 'apt-cache showpkg ebtables'?22:11
dansmithalaski: comments commentified22:12
sdaguePackage: ebtables22:12
sdagueVersions:22:12
sdague2.0.10.4-3ubuntu1 (/var/lib/apt/lists/gallifrey_ubuntu_dists_trusty_main_binary-amd64_Packages) (/var/lib/dpkg/status)22:12
sdagueI have a local mirror22:12
sdaguebut it's just trusty22:12
dansmithtrusty?22:12
dansmithmore like rusty!22:12
dansmithhey-o22:12
cburgesssdague I don't get it... I have the same... wtf...22:13
* dansmith channels mriedem while he's at PTA or whatever22:13
cburgesssdague http://packages.ubuntu.com/precise/net/ebtables22:13
cburgessSo... I'm really not surer where you are getting that from.22:14
sdaguehttp://packages.ubuntu.com/trusty/ebtables22:14
dansmiththat's precise homeboy22:14
sdagueyeh, that's precise :)22:14
cburgessdansmith LOL sorry yes...22:14
cburgesssdague dansmith When I wrote the original retry loop gate was still precise, thats why I didn't do it. I forgot gate was upgraded since then.22:14
sdagueyeh22:14
cburgessSo yeah.. we can fix it now that we have trusty.22:14
cburgessbut..22:15
alaskidansmith: thanks22:15
cburgessWe also need libvirt 1.2.11. We can't just call --concurrent and assume it will work. We need both sides to use --concurrent or we have to retry.22:15
cburgessWe should probably just have it use --concurrent and do retry to make it easier.22:15
sdaguecburgess: sure, though there is no harm if we call --concurrent22:15
sdagueyeh, that's what I was thinking22:15
*** david-lyle has joined #openstack-nova22:16
cburgesssdague OK I'll work on it. Do we want to try and detect versions or just require the newer version and explode if we don't have --concurrent?22:16
sdaguethen if you have 1.2.11, you're fine22:16
sdaguefor mitaka, I think we can probably just assume you need ebtables > has this22:17
sdaguedocument in the release notes when we add that flag that it's a requirement22:17
sdagueI assume 2.0.10 was the version that mattered?22:18
*** eharney has quit IRC22:18
cburgessYeah I think so.22:18
cburgesssdague Wait add flag? You want a flag?22:18
*** sacharya has quit IRC22:19
sdaguecburgess: by flag I meant --concurrent always being passed to ebtables22:19
sdagueno, this is not configurable22:19
*** boris-42 has joined #openstack-nova22:19
cburgesssdague Got it sorry ok...22:19
sdaguecburgess: then you can be an atc :)22:21
sdaguehonestly, that patch is probably 30 minutes22:21
cburgesssdague OK assigned https://bugs.launchpad.net/nova/+bug/1501366 to myself22:21
openstackLaunchpad bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed] - Assigned to Chet Burgess (cfb-n)22:21
*** thorst has quit IRC22:22
sdaguecburgess: thanks!22:22
cburgesssdague Yeah its the script thats a bit more involved...22:22
sdagueyeh, the script will be22:22
openstackgerritSumant Murke proposed openstack/nova-specs: Add new attribute to nova show API response  https://review.openstack.org/24488822:22
cburgesssdague but first step is get nova doing the right thing.22:22
*** thorst has joined #openstack-nova22:22
*** ArxCruz has joined #openstack-nova22:22
*** dims_ has joined #openstack-nova22:23
*** tdurakov has quit IRC22:24
sdaguecburgess: honestly, I think you can just do this - https://github.com/openstack-infra/puppet-elastic_recheck/blob/master/files/er_safe_run.sh22:25
*** dims has quit IRC22:25
sdagueflock /well/know/lock/location /sbin/ebtables.real $@22:25
cburgesssdague So I think the issue is going to be that the "well known location" is different between different OSes22:26
cburgessOn ubunut it should be /var/run/lock but I think on RH its different.22:26
cburgessSo we might need a quick lsb_release check and change path, but I get where you are doing.22:26
*** thorst has quit IRC22:27
sdaguecburgess: /var/lock is FHS22:27
sdagueubuntu links it to /var/run/lock22:27
sdaguebut /var/lock is probably the right answer22:27
sdaguehttp://www.pathname.com/fhs/pub/fhs-2.3.html#VARLOCKLOCKFILES22:27
cburgesssdague You assume everyone uses FHS but ok... we can run with that.22:31
sdagueyes, I assume that if you are running linux that doesn't comply with FHS then I hate you22:31
sdaguebecause that's one of the few things the distros all generally agreed on22:32
cburgesssdague I'm going to do it as 2 patches though. One to fix it the right way, then one to add the work around.22:32
*** Marga_ has joined #openstack-nova22:32
sdaguecburgess: that is perfectly kosher22:32
*** lnxnut has quit IRC22:32
dansmithand appreciated22:33
*** annegentle has quit IRC22:33
cburgessnod22:33
sdaguecburgess: yes, definitely thanks in advance for jumping in here22:36
*** emagana_ has quit IRC22:36
*** alex_klimov has quit IRC22:37
*** emagana has joined #openstack-nova22:37
*** lpetrut has quit IRC22:39
*** tongli has quit IRC22:41
*** pm90_ has joined #openstack-nova22:44
openstackgerritSumant Murke proposed openstack/nova-specs: Add new attribute to nova show API response  https://review.openstack.org/24488822:44
*** kaisers has quit IRC22:45
*** burgerk has quit IRC22:45
*** thorst has joined #openstack-nova22:46
*** ctrath has quit IRC22:46
*** pm90__ has quit IRC22:47
*** thorst_ has joined #openstack-nova22:48
*** rlrossit has quit IRC22:49
*** annegentle has joined #openstack-nova22:49
*** ev1lkry has quit IRC22:50
*** annegentle has quit IRC22:50
*** thorst has quit IRC22:51
*** thorst_ has quit IRC22:53
*** tonytan4ever has quit IRC22:54
*** ctrath1 has joined #openstack-nova22:55
*** edtubill has quit IRC22:55
*** kaisers has joined #openstack-nova22:56
*** stevemar_ has joined #openstack-nova22:56
*** pumaranikar has joined #openstack-nova22:57
*** IanGovett has quit IRC22:57
*** stevemar_ has quit IRC22:59
pumaranikarHi I am working on bug https://bugs.launchpad.net/nova/+bug/1250066. It says I have to update "updated_at" field when attach/detach volume to an instance23:03
openstackLaunchpad bug 1250066 in OpenStack Compute (nova) "Need change instance's property updated_at when attach/detach volume" [Low,Confirmed] - Assigned to Pushkar Umaranikar (pushkar-umaranikar)23:03
pumaranikarBut instance.save methond won't work here as updates set will be empty23:03
pumaranikarAny suggestions about How Can I approach to reslove ?23:04
*** ZZelle has quit IRC23:08
*** rk4n has joined #openstack-nova23:12
openstackgerritSumant Murke proposed openstack/nova-specs: Add new attribute to nova show API response  https://review.openstack.org/24488823:12
*** jamielennox is now known as jamielennox|away23:13
*** obondarev has quit IRC23:14
*** rk4n has quit IRC23:15
*** obondarev has joined #openstack-nova23:15
*** rk4n has joined #openstack-nova23:16
*** rk4n has quit IRC23:16
*** pixelbeat has joined #openstack-nova23:17
*** aix has joined #openstack-nova23:18
*** darrenc is now known as darrenc_afk23:18
*** gyee has quit IRC23:19
*** mc_nair has quit IRC23:20
*** gyee has joined #openstack-nova23:21
*** ihrachys has quit IRC23:21
*** jamielennox|away is now known as jamielennox23:22
*** ctrath1 has quit IRC23:23
*** kmartin has quit IRC23:23
*** sneti has quit IRC23:24
*** rk4n has joined #openstack-nova23:24
*** rk4n has quit IRC23:29
*** ihrachys has joined #openstack-nova23:30
*** ijuwang has quit IRC23:31
*** pradk has quit IRC23:33
*** pradk has joined #openstack-nova23:35
*** nikhil_k_ has joined #openstack-nova23:37
*** rk4n has joined #openstack-nova23:38
*** otter768 has joined #openstack-nova23:39
*** darrenc_afk is now known as darrenc23:39
*** nikhil_k has quit IRC23:40
*** otter768 has quit IRC23:43
*** rk4n has quit IRC23:47
*** ArxCruz has quit IRC23:47
*** ArxCruz has joined #openstack-nova23:48
*** takashin has joined #openstack-nova23:49
*** ArxCruz has quit IRC23:49
*** emagana has quit IRC23:49
*** hparekh has quit IRC23:50
*** pumaranikar has quit IRC23:50
*** rk4n has joined #openstack-nova23:50
*** hwoarang has quit IRC23:51
*** markmc has quit IRC23:52
*** hwoarang has joined #openstack-nova23:52
*** jgjhgsonchn88523 has quit IRC23:54
*** achanda has quit IRC23:55
*** markmc has joined #openstack-nova23:55
*** e0ne has quit IRC23:56
*** ihrachys has quit IRC23:57
*** hparekh has joined #openstack-nova23:57
*** pm90_ has quit IRC23:57
*** xyang1 has quit IRC23:58

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