Monday, 2016-03-28

*** haomaiwang has quit IRC00:01
*** haomaiwang has joined #openstack-nova00:01
*** zenoway has quit IRC00:02
*** akshai has quit IRC00:02
*** achanda has quit IRC00:03
*** cfarquhar has joined #openstack-nova00:07
*** cfarquhar has quit IRC00:07
*** cfarquhar has joined #openstack-nova00:07
*** vishwanathj has joined #openstack-nova00:27
*** vishwanathj has quit IRC00:27
*** allen_gao has quit IRC00:31
*** baoli has joined #openstack-nova00:31
*** zenoway has joined #openstack-nova00:32
*** baoli has quit IRC00:34
*** allen_gao has joined #openstack-nova00:37
*** zenoway has quit IRC00:37
*** liusheng has quit IRC00:42
*** liusheng has joined #openstack-nova00:42
*** allen_gao has quit IRC00:42
*** qeelee has joined #openstack-nova00:42
*** zz_dimtruck is now known as dimtruck00:43
*** jwcroppe has joined #openstack-nova00:44
*** jwcroppe has quit IRC00:44
*** jwcroppe has joined #openstack-nova00:44
*** jwcroppe has quit IRC00:44
*** jwcroppe has joined #openstack-nova00:45
*** jwcroppe has quit IRC00:45
*** hoangcx has joined #openstack-nova00:47
*** allen_gao has joined #openstack-nova00:49
*** thorst has joined #openstack-nova00:53
*** allen_gao has quit IRC00:54
*** yuywz has joined #openstack-nova00:58
*** yuywz has joined #openstack-nova00:59
*** yuywz has quit IRC01:00
*** haomaiwang has quit IRC01:01
*** thorst has quit IRC01:01
*** allen_gao has joined #openstack-nova01:01
*** yuywz has joined #openstack-nova01:01
*** haomaiwang has joined #openstack-nova01:01
*** achanda has joined #openstack-nova01:01
openstackgerritTakashi NATSUME proposed openstack/nova: Pass BDM in swap_volume method of LibvirtDriver  https://review.openstack.org/25713501:05
openstackgerritTakashi NATSUME proposed openstack/nova: Add validations for volume_size and destination_type  https://review.openstack.org/25878801:05
*** jwcroppe has joined #openstack-nova01:06
openstackgerritTakashi NATSUME proposed openstack/nova: Replace assert statements in iptables and virt tests  https://review.openstack.org/28299201:06
*** jwcroppe has quit IRC01:06
*** jwcroppe has joined #openstack-nova01:06
*** jwcroppe has quit IRC01:06
*** ducttape_ has joined #openstack-nova01:07
openstackgerritMatt Riedemann proposed openstack/nova-specs: Re-propose ironic-networks-support for Newton  https://review.openstack.org/29776701:09
*** qeelee has quit IRC01:10
*** penick has quit IRC01:13
*** ducttape_ has quit IRC01:16
*** mriedem has quit IRC01:19
openstackgerritDongcan Ye proposed openstack/nova: VMware: Make http connection pool configurable  https://review.openstack.org/26534601:20
*** baoli has joined #openstack-nova01:24
*** sabari has quit IRC01:24
*** sc68cal has quit IRC01:27
*** qeelee has joined #openstack-nova01:28
*** EinstCrazy has joined #openstack-nova01:29
openstackgerritZhenyu Zheng proposed openstack/nova: config options: centralize section: "crypto"  https://review.openstack.org/27391601:31
*** Sukhdev has joined #openstack-nova01:38
*** sabari- has joined #openstack-nova01:38
*** achanda has quit IRC01:39
*** ircuser-1 has joined #openstack-nova01:42
*** mylu has quit IRC01:42
*** mylu has joined #openstack-nova01:43
*** haomaiwang has quit IRC01:46
*** chuck_ has joined #openstack-nova01:50
*** chuck_ has quit IRC01:50
*** cfarquhar has quit IRC01:52
*** ZZelle has quit IRC01:53
*** ljxiash has joined #openstack-nova01:55
*** jwcroppe has joined #openstack-nova01:56
*** thorst has joined #openstack-nova01:58
*** liunian_ has joined #openstack-nova02:02
*** Daisy has joined #openstack-nova02:03
*** thorst has quit IRC02:06
*** baoli has quit IRC02:06
*** allen_gao has quit IRC02:07
openstackgerrithuan proposed openstack/nova: XenAPI: Support neutron security group  https://review.openstack.org/25127102:09
*** ljxiash has quit IRC02:10
*** EinstCrazy has quit IRC02:10
*** ljxiash has joined #openstack-nova02:10
*** EinstCrazy has joined #openstack-nova02:11
*** baoli has joined #openstack-nova02:13
*** jamielennox is now known as jamielennox|away02:13
*** ljxiash has quit IRC02:14
*** allen_gao has joined #openstack-nova02:17
*** ljxiash has joined #openstack-nova02:26
*** achanda has joined #openstack-nova02:26
*** baoli has quit IRC02:32
*** ivase has quit IRC02:33
*** gcb has joined #openstack-nova02:33
*** ivase has joined #openstack-nova02:33
*** EinstCrazy has left #openstack-nova02:34
*** gmann has quit IRC02:39
*** haomaiwang has joined #openstack-nova02:41
*** baoli has joined #openstack-nova02:41
*** Sukhdev has quit IRC02:42
*** gmann has joined #openstack-nova02:44
*** baoli has quit IRC02:44
*** houming has joined #openstack-nova02:45
*** vishwanathj has joined #openstack-nova02:48
openstackgerritAllen Gao proposed openstack/nova: config options: centralize section "workarounds"  https://review.openstack.org/29808302:48
*** Sukhdev has joined #openstack-nova02:50
*** Sukhdev has quit IRC02:51
*** ivase_ has joined #openstack-nova02:51
*** baoli has joined #openstack-nova02:52
*** ivase has quit IRC02:52
*** ivase_ has quit IRC02:53
*** ivase has joined #openstack-nova02:53
*** ljxiash has quit IRC02:53
*** baoli has quit IRC02:54
*** cfarquhar has joined #openstack-nova02:55
*** cfarquhar has quit IRC02:55
*** cfarquhar has joined #openstack-nova02:55
*** ljxiash has joined #openstack-nova02:56
*** baoli has joined #openstack-nova02:59
*** dimtruck is now known as zz_dimtruck03:00
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-nova03:01
*** baoli has quit IRC03:02
*** thorst has joined #openstack-nova03:03
*** thorst_ has joined #openstack-nova03:06
*** thorst has quit IRC03:09
*** RuiChen has quit IRC03:09
*** qeelee has quit IRC03:11
*** RuiChen has joined #openstack-nova03:11
*** thorst_ has quit IRC03:11
*** baoli has joined #openstack-nova03:11
*** zenoway has joined #openstack-nova03:12
*** Daisy has quit IRC03:13
*** Daisy has joined #openstack-nova03:14
*** zenoway has quit IRC03:17
*** Daisy has quit IRC03:18
*** Kevin_Zheng has quit IRC03:22
*** EinstCra_ has joined #openstack-nova03:22
*** zz_dimtruck is now known as dimtruck03:25
*** houming has quit IRC03:27
*** dimtruck is now known as zz_dimtruck03:28
*** houming has joined #openstack-nova03:28
*** zz_dimtruck is now known as dimtruck03:28
*** links has joined #openstack-nova03:33
*** dimtruck is now known as zz_dimtruck03:38
*** achanda has quit IRC03:44
*** ivase has quit IRC03:48
*** ivase has joined #openstack-nova03:49
*** rmevans has joined #openstack-nova03:49
openstackgerritAllen Gao proposed openstack/nova: config options: centralize section "workarounds"  https://review.openstack.org/29808303:52
*** ivase_ has joined #openstack-nova03:54
*** hoangcx has quit IRC03:54
*** hoangcx has joined #openstack-nova03:55
*** ivase has quit IRC03:55
openstackgerritAllen Gao proposed openstack/nova: config options: centralize section "workarounds"  https://review.openstack.org/29808303:58
*** baoli has quit IRC03:58
*** allen_gao has quit IRC03:58
*** zz_dimtruck is now known as dimtruck03:59
*** Daisy has joined #openstack-nova04:00
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-nova04:01
*** Jeffrey4l has joined #openstack-nova04:02
*** rmevans has quit IRC04:04
*** Daisy has quit IRC04:04
*** salv-orlando has joined #openstack-nova04:06
*** allen_gao has joined #openstack-nova04:06
*** ivase has joined #openstack-nova04:06
*** salv-orlando has quit IRC04:06
*** salv-orlando has joined #openstack-nova04:07
*** ivase_ has quit IRC04:08
*** salv-orl_ has quit IRC04:09
*** thorst has joined #openstack-nova04:10
*** salv-orlando has quit IRC04:12
*** ivase_ has joined #openstack-nova04:12
*** ivase has quit IRC04:13
*** mdrabe has joined #openstack-nova04:15
*** thorst has quit IRC04:16
*** ivase_ has quit IRC04:17
*** takashin has joined #openstack-nova04:18
*** ivase has joined #openstack-nova04:18
openstackgerritSatoru Moriya proposed openstack/nova: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538504:20
openstackgerritPushkar Umaranikar proposed openstack/nova: [DO Not Merge] Test patch to check migration logs  https://review.openstack.org/29809904:23
*** ivase has quit IRC04:24
*** ivase has joined #openstack-nova04:25
*** cdelatte has joined #openstack-nova04:26
*** delattec has joined #openstack-nova04:28
*** ivase has quit IRC04:30
*** cdelatte has quit IRC04:31
*** ivase has joined #openstack-nova04:31
*** vishwanathj is now known as vishwanathj_zzz04:33
*** ivase_ has joined #openstack-nova04:35
*** ivase has quit IRC04:35
*** mdrabe has quit IRC04:40
*** ivase has joined #openstack-nova04:41
*** ivase_ has quit IRC04:41
*** Sukhdev has joined #openstack-nova04:43
*** Daisy has joined #openstack-nova04:43
*** Daisy has quit IRC04:44
*** Daisy has joined #openstack-nova04:45
*** Daisy has quit IRC04:46
*** Daisy has joined #openstack-nova04:47
*** ivase_ has joined #openstack-nova04:47
*** ivase has quit IRC04:47
*** ivase has joined #openstack-nova04:52
*** ivase_ has quit IRC04:53
*** ljxiash has quit IRC04:55
*** ljxiash has joined #openstack-nova04:56
*** ivase_ has joined #openstack-nova04:58
*** ekuris_ has joined #openstack-nova04:58
*** ivase has quit IRC04:59
*** yuywz has quit IRC04:59
*** yuywz has joined #openstack-nova05:00
*** ljxiash has quit IRC05:00
*** haomaiwang has quit IRC05:01
*** haomaiwang has joined #openstack-nova05:01
*** sudipto has joined #openstack-nova05:05
*** ivase_ has quit IRC05:05
*** ivase has joined #openstack-nova05:06
*** Goneri has quit IRC05:06
*** ivase_ has joined #openstack-nova05:10
*** ivase has quit IRC05:10
*** zenoway has joined #openstack-nova05:11
*** Daisy has quit IRC05:12
*** thorst has joined #openstack-nova05:12
*** Daisy has joined #openstack-nova05:13
*** zenoway has quit IRC05:15
*** ivase_ has quit IRC05:16
*** ivase has joined #openstack-nova05:17
*** tbachman has quit IRC05:19
*** tbachman has joined #openstack-nova05:20
*** ivase_ has joined #openstack-nova05:21
*** thorst has quit IRC05:21
*** ivase has quit IRC05:22
*** dimtruck is now known as zz_dimtruck05:26
*** ivase has joined #openstack-nova05:27
*** ivase_ has quit IRC05:27
*** ljxiash has joined #openstack-nova05:30
*** ljxiash has quit IRC05:31
*** achanda has joined #openstack-nova05:31
*** delatte has joined #openstack-nova05:32
*** ivase_ has joined #openstack-nova05:33
*** ivase has quit IRC05:33
*** ljxiash has joined #openstack-nova05:34
*** ljxiash has quit IRC05:35
*** ljxiash has joined #openstack-nova05:35
*** esracelik has joined #openstack-nova05:35
*** delattec has quit IRC05:36
*** ishant has joined #openstack-nova05:37
*** ivase_ has quit IRC05:39
*** ivase has joined #openstack-nova05:39
*** ivase_ has joined #openstack-nova05:44
*** ivase has quit IRC05:45
*** ivase_ has quit IRC05:51
*** allen_gao has quit IRC05:54
*** sridharg has joined #openstack-nova05:57
*** ivase has joined #openstack-nova05:58
*** moshele has joined #openstack-nova06:00
*** haomaiwang has quit IRC06:01
*** haomaiwang has joined #openstack-nova06:01
*** ivase_ has joined #openstack-nova06:01
*** ivase has quit IRC06:02
*** allen_gao has joined #openstack-nova06:04
*** garyk has joined #openstack-nova06:07
*** ivase has joined #openstack-nova06:07
*** ivase_ has quit IRC06:08
*** baoli has joined #openstack-nova06:10
*** lpetrut has joined #openstack-nova06:11
*** harlowja_at_home has quit IRC06:12
*** ivase has quit IRC06:13
*** baoli has quit IRC06:15
*** rk4n has joined #openstack-nova06:17
*** allen_gao has quit IRC06:17
*** zz_dimtruck is now known as dimtruck06:18
*** thorst has joined #openstack-nova06:19
*** achinm has joined #openstack-nova06:20
openstackgerritSatoru Moriya proposed openstack/nova: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538506:21
*** ivase has joined #openstack-nova06:21
*** salv-orlando has joined #openstack-nova06:21
*** harlowja_at_home has joined #openstack-nova06:23
*** rk4n has quit IRC06:24
*** Daisy_ has joined #openstack-nova06:25
*** ivase_ has joined #openstack-nova06:25
*** thorst has quit IRC06:26
*** ivase has quit IRC06:26
*** mylu has quit IRC06:27
*** Daisy has quit IRC06:28
*** Daisy_ has quit IRC06:28
*** Daisy has joined #openstack-nova06:28
*** ivase_ has quit IRC06:31
*** dedery has joined #openstack-nova06:31
*** Marga_ has quit IRC06:32
*** salv-orlando has quit IRC06:34
*** dedery has quit IRC06:37
*** liverpooler has joined #openstack-nova06:40
*** yuywz has quit IRC06:42
*** yuywz has joined #openstack-nova06:42
*** cristicalin has joined #openstack-nova06:45
*** pcaruana has joined #openstack-nova06:49
*** vishwanathj_zzz has quit IRC06:52
*** gongysh has joined #openstack-nova06:53
*** sheel has joined #openstack-nova06:54
*** allen_gao has joined #openstack-nova07:01
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-nova07:01
*** Daisy_ has joined #openstack-nova07:02
*** Daisy has quit IRC07:02
openstackgerritAbhishek Talwar proposed openstack/python-novaclient: flavor postional argument in nova boot help  https://review.openstack.org/29568107:04
*** Daisy has joined #openstack-nova07:07
*** ivase has joined #openstack-nova07:07
*** Daisy_ has quit IRC07:07
*** ivase_ has joined #openstack-nova07:11
*** ivase has quit IRC07:11
openstackgerritPreeti proposed openstack/python-novaclient: Prints "id" in place of "agent_id" for nova-agent commands.  https://review.openstack.org/29275407:12
*** lpetrut has quit IRC07:13
*** e0ne has joined #openstack-nova07:16
*** ivase_ has quit IRC07:17
*** e0ne has quit IRC07:18
*** thorst has joined #openstack-nova07:24
*** thorst has quit IRC07:31
*** dedery has joined #openstack-nova07:34
*** allen_gao has quit IRC07:35
*** ivase has joined #openstack-nova07:36
*** zenoway has joined #openstack-nova07:36
*** dedery has quit IRC07:38
*** femnad has joined #openstack-nova07:39
*** phuongnh has joined #openstack-nova07:39
*** ivase_ has joined #openstack-nova07:40
*** lpetrut has joined #openstack-nova07:40
*** ivase has quit IRC07:40
*** ivase_ has quit IRC07:46
*** allen_gao has joined #openstack-nova07:47
*** cristicalin has quit IRC07:47
*** ivase has joined #openstack-nova07:53
*** ivase_ has joined #openstack-nova07:57
*** ivase has quit IRC07:58
*** haomaiwang has quit IRC08:01
*** haomaiwa_ has joined #openstack-nova08:01
*** sabari- is now known as sabari08:01
*** salv-orlando has joined #openstack-nova08:01
*** ivase has joined #openstack-nova08:03
*** ivase_ has quit IRC08:04
*** salv-orlando has quit IRC08:07
*** lpetrut has quit IRC08:10
*** ivase has quit IRC08:10
*** ivase has joined #openstack-nova08:10
*** ivase_ has joined #openstack-nova08:14
*** e0ne has joined #openstack-nova08:14
*** rk4n has joined #openstack-nova08:15
*** ivase has quit IRC08:15
openstackgerritTakashi NATSUME proposed openstack/nova: Remove a redundant 'that'  https://review.openstack.org/29814408:16
*** rk4n has quit IRC08:18
*** rk4n has joined #openstack-nova08:18
*** ivase_ has quit IRC08:21
*** libu has joined #openstack-nova08:29
*** takashin has quit IRC08:29
*** thorst has joined #openstack-nova08:30
*** takashin has joined #openstack-nova08:30
*** ndipanov has joined #openstack-nova08:30
*** ndipanov has quit IRC08:30
*** salv-orlando has joined #openstack-nova08:33
*** sudipto has quit IRC08:33
*** sudipto has joined #openstack-nova08:33
*** dedery has joined #openstack-nova08:35
*** thorst has quit IRC08:36
*** ociuhandu has quit IRC08:36
*** qeelee has joined #openstack-nova08:36
*** salv-orlando has quit IRC08:37
*** dedery has quit IRC08:39
*** qeelee has quit IRC08:42
*** armax has joined #openstack-nova08:44
*** armax has quit IRC08:49
*** ivase has joined #openstack-nova08:51
openstackgerritgaryk proposed openstack/nova: Virt: add context to attach and detach interface  https://review.openstack.org/20936208:54
*** ivase_ has joined #openstack-nova08:55
*** ivase has quit IRC08:55
*** dedery has joined #openstack-nova08:58
openstackgerritgaryk proposed openstack/nova: Compute: remove the 'method' variable from InstanceInvalidState  https://review.openstack.org/21862908:58
*** Sukhdev has quit IRC08:59
openstackgerritHans Lindgren proposed openstack/nova: Reduce number of db calls during image cache manager periodic task  https://review.openstack.org/29802308:59
*** haomaiwa_ has quit IRC09:01
*** haomaiwang has joined #openstack-nova09:01
openstackgerritgaryk proposed openstack/nova: VMware: ensure volume is accessible by instance  https://review.openstack.org/21863909:01
*** ivase_ has quit IRC09:01
openstackgerritHans Lindgren proposed openstack/nova: Replace flavors.get_all_flavors_sorted_list() with object call  https://review.openstack.org/26880109:02
openstackgerritHans Lindgren proposed openstack/nova: Refactor and objectify flavor fakes used in api tests  https://review.openstack.org/26880009:02
openstackgerritAbhishek Talwar proposed openstack/python-novaclient: flavor postional argument in nova boot help  https://review.openstack.org/29568109:02
*** achanda has quit IRC09:04
*** thorst has joined #openstack-nova09:08
*** ivase has joined #openstack-nova09:08
*** vilobhmm11 has quit IRC09:09
*** ociuhandu has joined #openstack-nova09:10
*** achanda has joined #openstack-nova09:11
*** ndipanov has joined #openstack-nova09:12
*** ivase_ has joined #openstack-nova09:12
*** ivase has quit IRC09:13
*** thorst has quit IRC09:15
*** rqw has joined #openstack-nova09:18
*** ivase_ has quit IRC09:19
*** ljxiash has quit IRC09:22
*** ivase has joined #openstack-nova09:25
dederyHi cores, can you please take a look at https://review.openstack.org/#/c/287515 (Fix failed snapshot on VMs with SRIOV ports) ? Thanks!09:26
*** agireud has quit IRC09:27
*** achanda has quit IRC09:28
*** agireud has joined #openstack-nova09:28
*** ivase_ has joined #openstack-nova09:29
*** ivase has quit IRC09:30
*** liusheng has quit IRC09:30
*** lpetrut has joined #openstack-nova09:31
*** liusheng has joined #openstack-nova09:31
*** thorst has joined #openstack-nova09:32
*** phuongnh has quit IRC09:34
*** hoangcx has quit IRC09:34
*** ivase_ has quit IRC09:35
*** rqw has quit IRC09:36
*** salv-orlando has joined #openstack-nova09:36
*** rqw has joined #openstack-nova09:36
*** thorst has quit IRC09:37
*** ivase has joined #openstack-nova09:37
*** yamahata has quit IRC09:37
*** rqw has quit IRC09:39
*** hoangcx has joined #openstack-nova09:39
openstackgerrithuangtianhua proposed openstack/python-novaclient: Using glance 'image-list'/'image-show' in boot help message  https://review.openstack.org/29817609:41
*** ivase_ has joined #openstack-nova09:41
*** ivase has quit IRC09:42
openstackgerritjaveme proposed openstack/nova: LiveMigration should check whether the dest host has enough vCPUs  https://review.openstack.org/29817709:42
*** pcaruana has quit IRC09:42
*** al_indigo has quit IRC09:43
*** Daisy has quit IRC09:44
*** Daisy has joined #openstack-nova09:44
*** liunian_ has quit IRC09:46
*** salv-orlando has quit IRC09:46
openstackgerritMoshe Levi proposed openstack/nova: Fix detach SR-IOV when using LibvirtConfigGuestHostdevPCI  https://review.openstack.org/29630509:46
*** Daisy has quit IRC09:46
*** Daisy has joined #openstack-nova09:47
*** ivase_ has quit IRC09:47
*** Daisy has quit IRC09:48
openstackgerritJinquan Ni proposed openstack/nova: Modify PCI without numa node info treatment  https://review.openstack.org/29817909:48
openstackgerritMadhuri Kumari proposed openstack/nova: Fix API Error on hypervisor-uptime API  https://review.openstack.org/28986809:49
openstackgerritjaveme proposed openstack/nova: LiveMigration should check whether the dest host has enough vCPUs  https://review.openstack.org/29817709:50
*** pcaruana has joined #openstack-nova09:55
*** al_indigo has joined #openstack-nova09:56
*** liunian_ has joined #openstack-nova09:57
*** al_indigo has quit IRC09:58
*** ivase has joined #openstack-nova10:00
*** haomaiwang has quit IRC10:01
*** 77CAAJ17L has joined #openstack-nova10:01
achinm:garyk Hi gary, are you familiar with this patch : https://review.openstack.org/#/c/281134 , which improves the image transfer ?10:02
achinmi'm unable to find @rgerganov)10:02
achinmhere10:02
*** ivase_ has joined #openstack-nova10:04
*** ivase has quit IRC10:04
*** rook has joined #openstack-nova10:06
*** allen_gao has quit IRC10:10
*** ivase_ has quit IRC10:10
*** rook has quit IRC10:10
openstackgerritPreeti proposed openstack/python-novaclient: Prints "id" in place of "agent_id" for nova-agent commands.  https://review.openstack.org/29275410:11
*** allen_gao has joined #openstack-nova10:15
*** ivase has joined #openstack-nova10:17
*** aloga_ is now known as aloga10:18
*** 77CAAJ17L has quit IRC10:19
openstackgerritAlex Xu proposed openstack/nova-specs: Expose guest CPU instruction set extensions  https://review.openstack.org/29818810:19
*** haomaiwa_ has joined #openstack-nova10:21
*** ivase_ has joined #openstack-nova10:21
*** ivase has quit IRC10:22
*** EinstCra_ has quit IRC10:25
*** sdague has joined #openstack-nova10:27
openstackgerritJiajun Liu proposed openstack/nova: config options: centralize cinder options  https://review.openstack.org/29819110:28
*** ivase_ has quit IRC10:28
*** achanda has joined #openstack-nova10:28
*** achanda has quit IRC10:33
garykachinm: yes.10:35
*** thorst has joined #openstack-nova10:36
garykachinm: it also resolves quite a few bugs10:36
*** achanda has joined #openstack-nova10:36
*** Kevin_Zheng has joined #openstack-nova10:39
*** thorst has quit IRC10:42
*** hoangcx has quit IRC10:43
*** takashin has left #openstack-nova10:57
*** haomaiwa_ has quit IRC11:01
*** haomaiwang has joined #openstack-nova11:01
*** rodrigods has quit IRC11:03
*** rodrigods has joined #openstack-nova11:03
*** achinm has quit IRC11:07
*** pcaruana has quit IRC11:09
*** salv-orlando has joined #openstack-nova11:11
*** coreywright has quit IRC11:11
*** achanda has quit IRC11:11
*** shaohe_feng has joined #openstack-nova11:11
sdaguemorning folks11:12
*** salv-orlando has quit IRC11:13
*** liunian_ has quit IRC11:14
*** qeelee has joined #openstack-nova11:15
*** achinm has joined #openstack-nova11:17
*** pcaruana has joined #openstack-nova11:23
achinmgaryk: i was kinda debugging snapshot issue in my environment.. I was seeing snapshot ova getting uploaded to glance within 1 and half minute..11:25
garykachinm: can you try with that patch11:25
achinmgaryk: the size of the  ova image is about 1.2 Gb and capacity is 60 GB.11:25
achinmand the image which was getting exported to glance was around 400 MB11:26
*** smatzek has joined #openstack-nova11:26
*** thorst has joined #openstack-nova11:26
garykachinm: i suggest that you talk with rado then11:26
achinmgaryk: yes, i tried with that patch.  actually backported it to liberty11:26
*** doug-fish has joined #openstack-nova11:27
garykachinm: maybe in the next decade it will be approved in nova11:27
achinmgaryk : haha11:27
achinmgaryk: rgerganov  doesnt come here often, i believe11:28
achinmill just drop a mail to him , then ?11:28
garyki will see if he is around, i think that its easter monday11:28
garykyeah, maybe a mail or tomorrow he will be on line11:28
achinmoh.. i see. Sure11:29
achinmthanks, ill chat with him11:29
*** coreywright has joined #openstack-nova11:30
garykachinm: i found him, he will log in11:30
achinmgaryk : ok, cool!11:30
*** rgerganov has joined #openstack-nova11:31
achinm:rgerganov Hi Radoslav, i have some queries about this patch : https://review.openstack.org/#/c/281134/411:32
rgerganovachinm, hi, what do you want to know11:33
*** libu has quit IRC11:35
achinm was kinda debugging snapshot issue in one of my environments, it was timing out after 5 minutes. but the image was getting exported to glance... i called this update_progress while reading the chunks from VC11:35
achinmhttps://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/io_util.py#L177 , somethign like self.input.update_progress()11:36
achinmSo this helped, and i saw your patch later.11:37
achinmwithout your patch, it was the image size which was getting exported to glance  was about 370 MB odds for a instance booted with an ova image of size 1 GB and capacity 60GB11:38
achinmBTW im trying out in stable/liberty... I backported your patch to liberty..11:39
achinmBut the snapshot is taking about 10 minutes, and the image size is which got exported is 1 GB.11:40
achinmWhy is this difference  ?11:40
*** liusheng has quit IRC11:41
*** liusheng has joined #openstack-nova11:42
rgerganovachinm, did you manage to boot from the 370MB instances?11:43
rgerganovachinm, maybe something went wrong with the export hence the smaller size11:43
achinmrgerganov : i'll giev it a shot too.  I was thinking it used to export the differential disk, like how KVM does..11:44
rgerganovno, it exports the whole thing11:46
achinmso lets say if i have an instance with an ova of 1 GB, having capacity 60 GB, with flavor size 80 Gb. so it should compress 80 gb vmdk into x sized ova and upload it , right ?11:48
*** flaper87 has quit IRC11:48
rgerganovachinm, yes except the snapshot is not OVA but streamOptimized disk11:50
*** EinstCrazy has joined #openstack-nova11:50
*** flaper87 has joined #openstack-nova11:51
achinmrgerganov: oh..ok.. one more question about progress11:52
achinmrgerganov: with the patch,  when an ova image gets copied to datastore, the expectation is to see the progress in vshphere  tasks ?11:54
achinm*[im talking boot from ova scenario]11:55
*** rqw has joined #openstack-nova11:56
rgerganovachinm, yeah, the progress should appear in vCenter11:56
*** baoli has joined #openstack-nova11:57
achinmso, we are using size of the image  https://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/images.py#L41611:58
achinmwhat if the user sets size property in glance11:58
achinmwont it flaw the progress % shown ?11:58
achinmbeacuse in oslo_vmware : self._vmdk_size is used while calculating the progress11:59
rgerganovachinm, yeah, the reported progress could be incorrect but it doesn't matter12:00
rgerganovachinm, we are adding some introspection in glance that puts the real size of the image in 'virtual_size' property12:00
*** haomaiwang has quit IRC12:01
rgerganovachinm, and we use this in Nova12:01
rgerganovachinm, but that's in our VMware product12:01
*** haomaiwang has joined #openstack-nova12:01
achinmhow is it different from "size" property?12:01
rgerganov'size' is the compressed size, 'virtual_size' is the size of the virtual disk12:02
achinmwhich is the real size , i believe12:02
*** houming has quit IRC12:07
*** yuywz has quit IRC12:08
*** rgerganov has quit IRC12:08
achinmrgerganov : oops12:09
*** ducttape_ has joined #openstack-nova12:10
*** gcb has quit IRC12:12
*** jaypipes has joined #openstack-nova12:12
*** achanda has joined #openstack-nova12:12
*** ccesario has joined #openstack-nova12:13
*** aysyd has joined #openstack-nova12:13
*** achanda has quit IRC12:17
*** ivase has joined #openstack-nova12:22
*** rqw has quit IRC12:23
*** rqw has joined #openstack-nova12:24
*** ducttape_ has quit IRC12:24
*** ZZelle has joined #openstack-nova12:24
*** ivase_ has joined #openstack-nova12:24
*** gcb has joined #openstack-nova12:25
*** erlon has joined #openstack-nova12:27
*** ivase has quit IRC12:27
*** ivase has joined #openstack-nova12:28
*** rtheis has joined #openstack-nova12:28
*** ivase_ has quit IRC12:28
*** Daisy has joined #openstack-nova12:29
*** ivase has quit IRC12:30
*** edmondsw has joined #openstack-nova12:30
*** ivase has joined #openstack-nova12:30
*** mdrabe has joined #openstack-nova12:30
*** Daisy has quit IRC12:33
*** dedery has quit IRC12:36
*** pcaruana has quit IRC12:39
openstackgerritZhenyu Zheng proposed openstack/nova: [POC] Allow user defined stop  https://review.openstack.org/29822112:39
*** gcb has quit IRC12:40
edleafeSo is Gerrit still on holiday?12:47
sdagueedleafe: seems fine to me12:48
edleafesdague: ok thanks. Nothing but timeouts here.12:48
*** salv-orlando has joined #openstack-nova12:50
openstackgerritjaveme proposed openstack/nova: Missing to update task state for confirm_resize  https://review.openstack.org/24443312:51
*** rook has joined #openstack-nova12:51
*** pcaruana has joined #openstack-nova12:52
*** salv-orlando has quit IRC12:53
*** roeyc has joined #openstack-nova12:54
*** rlrossit has joined #openstack-nova12:58
*** klkumar has joined #openstack-nova12:58
*** xenogear has joined #openstack-nova13:00
*** haomaiwang has quit IRC13:01
*** haomaiwang has joined #openstack-nova13:01
*** dimtruck is now known as zz_dimtruck13:01
*** efried has joined #openstack-nova13:04
openstackgerritGleb Stepanov proposed openstack/nova: [WIP] Add console log offset  https://review.openstack.org/29247313:05
*** ccarmack has joined #openstack-nova13:09
*** efried has quit IRC13:10
*** mtanino has quit IRC13:13
*** achanda has joined #openstack-nova13:15
*** xenogear has quit IRC13:18
*** achanda has quit IRC13:19
*** mtanino has joined #openstack-nova13:21
*** ayoung has joined #openstack-nova13:22
*** vishwanathj_zzz has joined #openstack-nova13:22
*** zz_dimtruck is now known as dimtruck13:24
*** mtanino has quit IRC13:27
*** rfolco_ has joined #openstack-nova13:27
*** qeelee has quit IRC13:28
*** ishant has quit IRC13:28
*** smatzek has quit IRC13:30
*** mriedem has joined #openstack-nova13:35
*** links has quit IRC13:36
*** efried has joined #openstack-nova13:38
*** ctrath has joined #openstack-nova13:44
*** akshai has joined #openstack-nova13:45
mriedemVek: are you ok with this? https://review.openstack.org/#/c/297772/13:46
*** armax has joined #openstack-nova13:48
*** openstackgerrit has quit IRC13:48
*** openstackgerrit has joined #openstack-nova13:48
*** toure_ is now known as toure13:49
*** rqw has quit IRC13:50
*** dimtruck is now known as zz_dimtruck13:50
*** armax has quit IRC13:52
*** EinstCrazy has quit IRC13:52
*** ducttape_ has joined #openstack-nova13:52
openstackgerritJeremy Stanley proposed openstack/nova: List system dependencies for running common tests  https://review.openstack.org/20067213:54
*** eharney has joined #openstack-nova13:56
edleafeNova Scheduler subteam meeting in 2 minutes in #openstack-meeting-alt13:58
edleafe...if anyone's around, that is13:58
*** andymaier has joined #openstack-nova13:58
*** ametts has joined #openstack-nova13:59
*** rdo has quit IRC14:00
*** haomaiwang has quit IRC14:01
*** haomaiwang has joined #openstack-nova14:01
*** rdo has joined #openstack-nova14:02
*** knikolla has joined #openstack-nova14:04
*** andymaier has quit IRC14:04
*** smatzek has joined #openstack-nova14:05
*** haomaiwang has quit IRC14:06
*** syjulian has joined #openstack-nova14:07
sdaguedefinitely a slow day with all of europe off14:13
*** smatzek has quit IRC14:15
openstackgerritDan Smith proposed openstack/nova: Further hack up the n.t.unit.db.fakes module of horribleness  https://review.openstack.org/29610614:15
openstackgerritDan Smith proposed openstack/nova: Block flavor creation until main database is empty  https://review.openstack.org/29531014:15
openstackgerritDan Smith proposed openstack/nova: Add flavor migration routine  https://review.openstack.org/29477114:15
openstackgerritDan Smith proposed openstack/nova: Make Flavor create() and destroy() work against API DB  https://review.openstack.org/29475814:15
sarafrajGood morning!14:16
dansmithman, sure is nice to see that stack'o'patches so short14:16
mriedemdansmith: are you working today? or still off?14:17
dansmithmriedem: nope, it was just friday.. I'm here14:17
mriedemccarmack: i have something fun to do if you're interested14:18
*** doug-fis_ has joined #openstack-nova14:19
*** smatzek has joined #openstack-nova14:20
*** knikolla has quit IRC14:20
*** prazumovsky has joined #openstack-nova14:21
*** knikolla has joined #openstack-nova14:21
*** doug-fish has quit IRC14:21
prazumovskyHi all! Can you help me with nova client and flavor behaviour?14:21
sarafrajHello Nova folks, I just want to do my quick introduction to this channel. My name is Raj (Sarafraj Singh) and I joined Intel recently. I am part of OSIC and will be working with Rackspace (johnthetubaguy) on Nova.  Before joining Intel I worked for different storage companies delivering SDS solutions. Looking forward to work with you all!14:24
dansmithalaski: I just replied to your comment about that migration14:24
dansmithalaski: it's a little stickier than it looks on the wrapper14:24
doffmsarafraj: Welcome.14:26
*** vishwana_ has joined #openstack-nova14:26
*** vishwana_ has quit IRC14:26
sarafrajdoffm: thx14:27
*** zz_dimtruck is now known as dimtruck14:27
*** vishwanathj_zzz has quit IRC14:29
*** akshai has quit IRC14:29
*** vishwanathj has joined #openstack-nova14:29
*** cleong has joined #openstack-nova14:29
*** akshai has joined #openstack-nova14:29
openstackgerritGleb Stepanov proposed openstack/nova: [WIP] Add console log offset  https://review.openstack.org/29247314:31
*** ljxiash has joined #openstack-nova14:31
mriedemprazumovsky: just ask and if anyone can answer they will14:32
*** shangxdy has joined #openstack-nova14:36
mriedemYingxin: alex_xu: we have https://review.openstack.org/#/c/292362/ and https://review.openstack.org/#/c/154339/14:37
prazumovskyOK, look. I use novaclient for creating, getting flavor. First step: I create flavor - create complete. Now I delete this flavor with CLI - delete complete. Now I get flavor with client - it returns object with all attributes. Check flavor list - no such flavor. Do flavor-show CLI command - returns table with flavor attrbiutes. Call flavor.get_keys() - returns NotFound. I expect that after deleting flavor it deletes and returns NotFound in c14:37
prazumovskyase of calling.14:37
mriedemYingxin: alex_xu: the latter has been around for over a year, but the former looks more ready to go?14:37
mriedemYingxin: alex_xu: if they are duplicates, can we abandon one of them?14:38
*** pwnall has joined #openstack-nova14:38
*** markvoelker has joined #openstack-nova14:38
*** pushkaru has joined #openstack-nova14:39
mriedemprazumovsky: GET on a specific flavor id will return soft-deleted flavors14:39
Yingxinmriedem: alex_xu: please abandon https://review.openstack.org/#/c/154339/14:39
Yingxinsorry forgot that14:39
alaskidansmith: looking14:39
mriedemprazumovsky: so it's working as designed. the flavor isn't really gone until you purge / archive your nova database14:40
alex_xuYingxin: mriedem, yea, will do that14:40
prazumovskymriedem: got it. And how I can check that flavor is soft-deleted?14:40
mriedemalex_xu: thanks14:40
alex_xumriedem: np14:40
prazumovskyflavor has no status:)14:40
mriedemprazumovsky: it's deleted column in the db will be != 014:40
prazumovskyand via novaclient there's no way to check that flavor is already soft-deleted?14:41
mriedemprazumovsky: it won't be in nova flavor-list14:41
prazumovskyOK, thanks14:41
*** haomaiwa_ has joined #openstack-nova14:41
mriedemprazumovsky: also, fyi on this spec https://review.openstack.org/#/c/265282/14:42
alaskidansmith: I need to do the due diligence on the effects of deleting flavors for cellsv1. But I thought at this point they were mostly passed down, or stored in instance_extra, and not looked up past the api14:42
*** mylu has joined #openstack-nova14:42
dansmithalaski: meaning that deleting them from the cell db shouldn't matter?14:42
alaskidansmith: assuming no edge cases where a lookup is needed, yes14:43
*** nelsnelson has joined #openstack-nova14:43
prazumovskymriedem, thanks, will read14:43
dansmithalaski: yeah, so I really thought that we weren't clean on that yet.. isn't one of the things passed between the cells operators a script to keep flavors in sync across all the cells?14:43
dansmithI thought that was still a thing since we did all our flavor cleanup, but maybe not14:44
alaskidansmith: I don't know if it's still a thing after the migration, I need to check on that14:44
dansmithokay14:44
*** mylu has quit IRC14:44
*** mylu has joined #openstack-nova14:44
*** mnestratov|2 has joined #openstack-nova14:46
ccarmackmriedem: what was the thing you wanted me to do?14:47
*** mnestratov has quit IRC14:48
*** Daisy has joined #openstack-nova14:49
*** burgerk has joined #openstack-nova14:53
mriedemccarmack: https://bugs.launchpad.net/nova/+bug/156232314:54
openstackLaunchpad bug 1562323 in OpenStack Compute (nova) mitaka "test_server_basic_ops rename in tempest is breaking cells job" [Critical,In progress] - Assigned to Matt Riedemann (mriedem)14:54
*** Daisy has quit IRC14:54
mriedemccarmack: i worked around that to get unblocked https://review.openstack.org/#/c/298065/14:54
mriedemccarmack: but what we should be doing is using the uuid from the tempest tests for our regex file14:54
mriedemccarmack: for example the full test_id would be something like: "test_attach_detach_volume_to_instance[compute,id-fff42874-7db5-4487-a8e1-ddda5fb5288d,smoke,stress]"14:54
mriedemso you'd want to change the regex you're using to exclude id-fff42874-7db5-4487-a8e1-ddda5fb5288d instead of test_attach_detach_volume_to_instance (assuming that was 1 of the skipped tests)14:55
ccarmackmriedem: ok, so change to use the test id for the skipped tests14:55
*** mylu has quit IRC14:56
*** mylu has joined #openstack-nova14:56
mriedemccarmack: yup14:57
mriedemccarmack: i'd probably start with just test_server_basic_ops and push a test patch to make sure that works before doing the rest14:57
alaskidansmith: there are a few potential issues, not really cellsv1 related, but they're all backwards compatibility RPC code. look up the flavor if what came over the wire was not a Flavor object.14:57
alaskidansmith: for cellsv1 there is a flavor lookup for the resize code14:58
*** fawadkhaliq has joined #openstack-nova14:58
dansmithah14:58
alaskidansmith: however, if we don't touch the flavors in the cells, that should still work14:58
*** mylu has quit IRC15:00
*** haomaiwa_ has quit IRC15:01
*** haomaiwa_ has joined #openstack-nova15:01
dansmithalaski: seemed like not deleting them was the safest15:03
dansmithalaski: and create at the top level will still work per normal15:03
*** mylu has joined #openstack-nova15:03
alaskidansmith: and that just leaves an additional cleanup step for v1 users later to clear out the parent cell db?15:05
dansmithalaski: no the migration will clear the api cell's main db, right?15:06
dansmithit's just the other cell dbs that have to get cleaned up I tink15:06
alaskiyeah. I see now, the duplication error won't delete but the regular migration path will15:06
dansmithright15:07
dansmiththey do still need to clean up the cell dbs, but not as critically15:07
*** edmondsw has quit IRC15:07
alaskiyeah. though for now they really need to have the flavors in the cells for resize15:08
alaskiso that works out15:08
dansmithreally need, or really only need?15:08
alaskithe latter15:09
*** ekuris_ has quit IRC15:09
alaskior backwards RPC compatibility. But I'm really really hoping that nobody is going to migrate to cells while running older services15:10
dansmithalaski: okay so just to be clear, in order to satisfy that older RPC call the flavors still have to be look-up-able in the cell, but15:10
dansmithif they're running all new stuff, there is no reason for them to remain?15:10
alaskilet me just double check resize to make sure I have the flow right15:11
dansmiththe only reason I don't want the migrate function to delete on error is because I think it's more likely that some non-cells user will run afoul of that and we'll delete something they actually wanted15:11
*** shangxdy has quit IRC15:12
*** prazumovsky has quit IRC15:12
*** Marga_ has joined #openstack-nova15:12
*** Marga_ has quit IRC15:13
anteayamriedem: are you omitted from the -meeting channel on purpose?15:13
*** Marga_ has joined #openstack-nova15:13
*** ssurana has joined #openstack-nova15:14
*** akshai has quit IRC15:15
*** akshai has joined #openstack-nova15:15
alaskidansmith: I think we can ignore the RPC back compat stuff here, that affects everyone cellsv1 or not. For cellsv1 users if there are no flavors in the cell db resize will fail its lookup on flavor_id.15:16
alaskiso not deleting on error makes sense, and has the side effect of keeping cells resize working15:17
dansmithokay cool15:17
*** achanda has joined #openstack-nova15:17
dansmithyou wanna comment on there for the record?15:17
alaskiyep15:17
*** woodster_ has joined #openstack-nova15:19
alaski503 :(15:20
alaskioh, restarting15:20
*** coreywright has quit IRC15:21
-openstackstatus- NOTICE: Gerrit is restarting on review.openstack.org in an attempt to address an issue reading an object from the ec2-api repository15:21
moshelemriedem: hi15:22
*** achanda has quit IRC15:22
dansmithalaski: that black screen when you try to save a comment is.. like the most depressing thing ever15:24
*** dmellado is now known as dmellado|off15:24
mosheleanteaya: lennyb update me with the  details of your conversion regarding Mellanox CI I will try to talk to mriedem about this15:26
openstackgerritSean Dague proposed openstack/nova: WIP: post processing doc tree to build REST methods  https://review.openstack.org/29548615:26
*** jdurgin1 has joined #openstack-nova15:26
*** achinm has quit IRC15:26
anteayamoshele: rather than repeat myself, it is all here: http://eavesdrop.openstack.org/meetings/third_party/2016/third_party.2016-03-28-15.00.log.txt15:26
anteaya15:06:22 <anteaya> #topic nova ci's missed a regression15:27
anteayamoshele: and thanks for taking this up15:27
*** armax has joined #openstack-nova15:27
anteayamoshele: please post to the mailing list after you have composed a reply15:27
mosheleanteaya: np :)15:27
anteayathank you15:27
mosheleanteaya: sure15:28
*** fawadkhaliq has quit IRC15:28
openstackgerritSergey Belous proposed openstack/nova: libvirt: exercise vif driver 'plug' method in tests  https://review.openstack.org/27360215:31
openstackgerritSergey Belous proposed openstack/nova: libvirt: pass a real instance object into vif plug/unplug methods  https://review.openstack.org/27360315:31
*** salv-orlando has joined #openstack-nova15:32
openstackgerritSergey Belous proposed openstack/nova: libvirt: fix ivs test to use the ivs vif object  https://review.openstack.org/27360415:32
*** coreywright has joined #openstack-nova15:34
mriedemanteaya: was i supposed to be in a meeting?15:35
mriedemanteaya: i was physically at another meeting15:35
*** mdrabe has quit IRC15:36
*** liverpooler has quit IRC15:36
anteayano no no you weren't15:36
anteayaI was just going to tab complete you is all15:36
anteayayet I couldn't as you weren't in -meeting15:36
anteayaand I just wondered if that was conscious choice or oversight15:37
dansmithanyone else seeing a fail of nova.tests.unit.image.test_glance.TestDownloadNoDirectUri.test_download_direct_exception_fallback on py34 only15:37
dansmith?15:37
dansmithI see it locally, with a rebuilt tox15:38
openstackgerritDan Smith proposed openstack/nova: Block flavor creation until main database is empty  https://review.openstack.org/29531015:38
mriedemanteaya: conscious15:39
anteayamriedem: thank you15:39
*** mylu has quit IRC15:40
anteayamriedem: I had just been talking with lennyb about the nfv ci mailing list post, encouraging him and co-workers to talk to you and compose a reply to the mailing list15:40
mriedemanteaya: ah ok15:40
anteayathanks15:40
*** salv-orlando has quit IRC15:41
moshelemriedem: hi, I am lennyb manager and I wanted to talk to regarding Mellanox CI15:41
mriedemmoshele: ok15:42
*** roeyc has quit IRC15:42
moshelemriedem: so currently we have one one Job in nova that test SR-IOV for Ethernet15:42
*** garyk has quit IRC15:43
*** roeyc has joined #openstack-nova15:43
*** initio has joined #openstack-nova15:43
*** mtanino has joined #openstack-nova15:43
moshelemriedem: and the part of the break was in SR-IOV for infiniband which we only run in neutron15:43
moshelemriedem: It regarding this https://review.openstack.org/#/c/296305/15:44
openstackgerritSergey Belous proposed openstack/nova: WIP: os-vif: convert libvirt driver to use os-vif library  https://review.openstack.org/26967215:44
mriedemmoshele: it did run on the change that regressd that though http://144.76.193.39/ci-artifacts/262341/7/Nova-ML2-Sriov/15:45
moshelemriedem: so basicly our nova for SR-IOV which is ethernet only CI should pass this https://review.openstack.org/#/c/262341/15:45
*** jlk has left #openstack-nova15:45
*** eharney has quit IRC15:45
moshelemriedem: yes so regarding Mellanox CI stability15:46
mriedemmoshele: so are you saying that even if that job wouldn't have failed in the scheduler http://144.76.193.39/ci-artifacts/262341/7/Nova-ML2-Sriov/logs/n-sch.log.gz it wouldn't have caught the regression anyway?15:46
moshelemriedem: because only ib_hostdev is using LibvirtConfigGuestHostdevPCI15:47
*** mylu has joined #openstack-nova15:47
mriedemok15:48
*** apoorvad has joined #openstack-nova15:48
moshelemriedem: hw_veb is LibvirtConfigGuestInterface15:48
*** mylu has quit IRC15:49
*** mylu has joined #openstack-nova15:49
moshelemriedem: and we focusing in nova only  for testing SR-IOV Ethernet because  hardware resource problem15:49
moshelemriedem: now regarding to the Mellanox CI stability in general15:50
*** ihrachys has joined #openstack-nova15:50
mriedemmoshele: so you're trying to get some coverage of both types, but in so doing you only test ethernet in nova and infiniband in neutron15:50
mriedemright?15:50
*** roeyc has quit IRC15:50
*** sridharg has quit IRC15:51
moshelemriedem: yes15:51
mriedemok, i'm with you so far :)15:51
*** mylu has quit IRC15:53
moshelemriedem: the big problem with Mellanox CI is that we run on physical servers and we don't reimage them for  every commit15:53
moshelemriedem: so some time a commit can break the CI if we don't clean it well after it15:54
mriedemsure15:54
dansmithmoshele: that kindof defeats the purpose of running pre-commit15:54
mosheledansmith: we basically removing devstack and all the openstack repository and then deploying again15:56
*** ArxCruz has joined #openstack-nova15:58
dansmithmoshele: yeah, it just provides the opportunity for residue to color the next run15:58
mosheledansmith: so we can have problems with pip installing olded/newer packages which can break thinks, because it is not clean env15:58
dansmithmoshele: have you thought/tried doing something like a loose container or a chroot that you can build once and deploy from a tarball each time or something?15:58
dansmithyeah15:58
mosheles/thisks/thinks15:59
*** akshai has quit IRC16:00
mriedemalaski: do you think we'll want back-to-back slots for cells v2 at the design summit?16:00
mriedemjaypipes: ^ same question for scheduler, i'm assuming 'yes' for scheduler16:00
*** haomaiwa_ has quit IRC16:01
*** akshai has joined #openstack-nova16:01
*** eharney has joined #openstack-nova16:01
*** haomaiwang has joined #openstack-nova16:01
jaypipesmriedem: yes16:01
mriedemok16:01
jaypipesmriedem: I think that's a good idea. it's worked at the last summits.16:02
*** mylu has joined #openstack-nova16:02
mriedemfyi, L251 rough design summit schedule https://etherpad.openstack.org/p/newton-nova-summit-ideas16:02
*** ljxiash has quit IRC16:02
openstackgerritHemanth Makkapati proposed openstack/nova: config options: centralize xenserver options  https://review.openstack.org/29783816:02
doffmmriedem: I'm hoping we will have alot of specs to review, but they might be less contentious than the scheduler proposals.16:03
*** mylu has quit IRC16:03
moshelemriedem: we would like to move SR-IOV to virtualized  env by using  project called smix which can simulate mellanox NIC in qemu and that can improve Mellanox CI , but we are not there yet16:04
mriedemmoshele: ok16:05
openstackgerritJim Rollenhagen proposed openstack/nova: DO NOT MERGE: patch for testing ironic-tempest-full  https://review.openstack.org/29787816:05
*** akshai has quit IRC16:05
mriedemmoshele: so do you know what change caused http://144.76.193.39/ci-artifacts/262341/7/Nova-ML2-Sriov/logs/n-sch.log.gz ?16:05
mriedemmoshele: intel NFV CI also hit that ^16:05
*** akshai has joined #openstack-nova16:06
jaypipesmriedem: have you responded to my proposal to have hardware for NFV/PCI at a shared lab?16:06
mriedemjaypipes: nope16:06
mriedemjaypipes: i saw infra's replies16:07
jaypipesmriedem: do you think it's something worth pursuing further? or should I drop the idea entirely?16:07
mriedemjaypipes: i think it would be ideal, but not sure how easy it's going to be to get owners/funders16:07
*** mylu has joined #openstack-nova16:08
jaypipesmriedem: I can make it happen for Intel folks I think.16:08
jaypipesif moshele can see about mellanox, that would be good.16:09
mriedemin general i think it would be in the best interest of the various NFV CI vendors to pool resources rather than each doing it on their own16:09
moshelemriedem: this can be related to old oslo_utils maybe the version that was install was incompatible with the nova code16:10
mriedemmy immediate concern was understanding where things brokedown in communication that the CI(s) were busted but we were still landing patches that impacted them16:10
*** mdrabe has joined #openstack-nova16:11
moshelemriedem, jaypipes: I will check with my mangers regarding to the shared lab16:11
mriedemmoshele: can you get the specific details about that in a reply to that mailing list thread?16:11
mriedemmoshele: because if there was some incompatible version of oslo.utils, we shoud know about that16:11
*** BassT has joined #openstack-nova16:11
*** dimtruck is now known as zz_dimtruck16:13
*** Marga_ has quit IRC16:13
moshelemriedem: the log is very old 2016-02-16 and I don't know what was the oslo version in that run16:15
*** yamahata has joined #openstack-nova16:15
*** nic has joined #openstack-nova16:15
*** e0ne has quit IRC16:16
moshelemriedem: I will ask lennyb to add log that save the python version for each run16:16
mriedemmoshele: i do16:16
mriedemDownloading oslo.utils-3.5.0-py2.py3-none-any.whl (86kB)16:16
mriedemyeah, a pip freeze output log would be nice16:16
nikhilmriedem: hi, you got a few mins to sync today?16:18
mriedemnikhil: sure16:18
*** zenoway has quit IRC16:19
nikhilI wanted to chat on 1) upgrades 2) compat layer contact 3) cross nova-glance session at the summit16:19
nikhilmriedem: are we testing Nova with N-1 and N upgrade with Glance?16:19
*** efried has quit IRC16:20
nikhilas there's still the API proxy and the images layer (may be on both fronts)16:20
dansmithalaski: so I just realized: http://logs.openstack.org/10/295310/16/check/gate-tempest-dsvm-full/311a4cb/logs/screen-n-api.txt.gz?level=TRACE16:20
dansmithalaski: we won't be able to even run tempest until we run the migration because the flavors never get moved to API16:20
nikhilmriedem: or it may be that you want someone to work on that front?16:20
dansmithsdague: ^ if we do that in devstack it'll get in the way of us selectively doing it in grenade, but it also means we won't be able to run the new side of grenade anyway16:21
mriedemnikhil: sdague can keep me honest here, but nova has 2 upgrade jobs, one is a full upgrade from n-1 to n, and another is a partial upgrade of everything n-1 to everything except n-cpu in n16:21
*** efried has joined #openstack-nova16:21
mriedemnikhil: glance is upgrade from n-1 to n in both cases16:21
dansmithmriedem: correct16:22
mriedemnikhil: i had penciled in a glance cross-project design summit session for 2:20-3 on thursday at the summit https://etherpad.openstack.org/p/newton-nova-summit-ideas16:23
mriedemL27716:23
nikhilmriedem: thanks for the upgrade info. any chance you could point me to that job?16:24
mriedemnikhil: gate-grenade-dsvm is full upgrade16:24
nikhilmriedem: cool, thanks!16:24
mriedemgate-grenade-dsvm-multinode is partial upgrade16:24
mriedemwhere n-cpu remains at n-116:25
*** fawadkhaliq has joined #openstack-nova16:25
mriedemnikhil: what were you thinking? keeping glance n-1 or something in an upgrade job?16:25
nikhilmriedem: no, I think this looks good.16:27
nikhilmriedem: I am just trying to make sure that we've got Glance covered in all cases.16:27
dansmithnikhil: glance is already running grenade on its patches right?16:28
nikhildansmith: yeah, it is16:28
dansmithnikhil: so the only extra thing we're testing between nova and glance would be the older n-cpu service with newer glance, in the multinode job16:29
dansmithbut not old glance with new nova, or various other permutations16:29
dansmithnikhil: is there some case you were specifically concerned about? or did you just want to make sure you could say "yes we're testing upgrades" to someone? :P16:29
nikhildansmith: ah, yes. and both tbh16:30
*** Sukhdev has joined #openstack-nova16:30
nikhildansmith: so, I think we need to test old glance with new nova and new glance with old nova !16:30
nikhilat least the two cases16:30
*** femnad has quit IRC16:30
nikhilbut for now, this is just me trying to be a bit proactive16:30
nikhilbasically, when we've the new API proposed16:31
nikhiljust trying to make sure that older deployments don't blow up16:31
mriedemnikhil: i think N nova + N-1 glance would be easy to do in the gate-grenade-dsvm-multinode job16:31
nikhilsomeone asked me how is Glance testing upgrades and I realized that we are not holistic16:31
*** hieulq has joined #openstack-nova16:31
mriedemnikhil: which new API?16:32
nikhilthe new import call16:32
mriedemnikhil: you know we're not using v2 yet right? :)16:32
*** mylu has quit IRC16:32
nikhilmriedem: oh yeah, v2 with the new create call :|16:32
dansmithmriedem: maybe we should hold back glance on nova patches and nova on glance patches in the multinode job?16:32
*** browne has quit IRC16:32
mriedemdansmith: that's what i was thinking16:32
cfriesen__I've been out of the loop for a bit...has any progress been made regarding the various competing scheduler proposals?  Or are we having a session at the summit to discuss?16:33
dansmiththat will mean we lose coverage for new glance and new nova in that job, but we may be okay with the regular grenade job for that then16:33
*** armax has quit IRC16:33
nikhildansmith:  mriedem: so, the other things that ties up with this is that the Nova v1->v2  change is almost ready and we're wanting it to merge in the early part of cycle.16:33
mriedemcfriesen__: n-sch is no longer in tree16:34
mriedemsurprise16:34
nikhilthen we are still working on the 1K spec proposal for import, that may merge later in the cycle to be part of defcore. There will be back compat but nice to check it using the upgrades.16:34
*** zz_dimtruck is now known as dimtruck16:35
cfriesen__mriedem: nothing moves that fast around here.16:35
*** mylu has joined #openstack-nova16:35
mriedemnikhil: so the glance v2 spec has to be re-proposed for newton16:36
mriedemthat's step 116:36
mriedemnikhil: mfedosin said he'd work on the re-proposal for the spec16:37
mriedemnikhil: as part of that we should mark what's completed in mitaka so we have a clear idea of what still needs to be done16:37
mriedemnikhil: and i thought as sdague was helping with some of this, he mentioned that there are some testing gaps (besides upgrade), but i don't remember specifics, i thought it had something to do with the xenapi plugin stuff16:38
nikhilyeah, xenapi has testing gaps16:38
nikhil:(16:38
nikhilbut I think Mike covered it as  a part of his latest ps (will reconfirm)16:39
mriedemi think that might have been part of sdague's concern but would need him to verify16:42
*** earlephilhower has quit IRC16:42
*** david-lyle_ has joined #openstack-nova16:44
*** david-lyle has quit IRC16:44
nikhilmriedem: thanks. another thing was that johnthetubaguy has been giving periodic suggestions on the nova.image compat layer effort. I wanted to sync with both of and understand whom should I stay in touch with regarding the same and what's needs to be proposed and where?16:44
*** klkumar has quit IRC16:45
sdaguenikhil: is there a current game plan up anywhere ?16:45
*** claudiub|2 has quit IRC16:45
nikhilsdague: for the compat layer?16:45
sdaguenikhil: for the whole cut over to v216:46
*** ArxCruz has quit IRC16:46
nikhilsdague: the part related to Nova is what Mike covered last cycle. No updates on that as of yet.16:46
mriedemnikhil: i'm not sure what the 'nova.image compat layer effort' is16:46
sdaguenikhil: where is that written down?16:46
nikhilsorry, I'd use nicks: Mike == mfedosin16:46
mriedemnikhil: is 'nova.image compat layer effort' part of the v2 integration? or something new?16:47
sdaguebecause everything was really scatter shot in those patches16:47
nikhilsdague: ah, I didn't know. There's nothing new up yet. Let me take that down as an action item.16:47
*** ihrachys has quit IRC16:47
mriedemwell https://review.openstack.org/#/q/topic:bp/use-glance-v2-api,n,z16:48
*** david-lyle has joined #openstack-nova16:48
*** armax has joined #openstack-nova16:48
mriedemhttps://review.openstack.org/#/q/topic:bp/use-glance-v2-api+status:open16:48
nikhilsdague: I've mostly consulted on what the issues were and didn't know the practicing process flaws.16:48
sdaguenikhil: it wasn't so much a process flaw as a communication flaw16:48
*** david-lyle_ has quit IRC16:49
nikhilmriedem: on the compat convo, this was supposed to be the next step from the v2 integration. v1->v2 port is mostly, get rid of v1 and start using v2. compat layer is getting rid of nova.image and putting it in a separate repo..16:49
sdaguebecause you get patches like this - https://review.openstack.org/#/c/274203 - which seem to just change narrow things, and don't explain the big picture of where that's going16:49
mriedemnikhil: i don't think we even want to talk about a compat layer right now16:49
sdaguenikhil: the last time this was discussed the compat layer was glanceclient16:50
mriedemnikhil: given this has been ongoing since kilo, we really need just a narrow focus on what needs to happen for glance v2 integration, and what order16:50
mriedemany new glance compat layer stuff is a total distraction right now16:50
alaskimriedem: yes to the two cells sessions16:50
mriedemalaski: done16:51
alaskigreat16:51
nikhilmriedem: ok, I will note that. it has been communicated otherwise (or at least I haven't seen completely explained)16:51
alaskidansmith: so that sucks(flavor.create failure), first of all.16:52
sdaguenikhil: how about we start with a blank page here, because there seems to be a ton of assumptions about what people think is agreed or not. Start writing an etherpad with - here is the bare minimum we need to do16:52
nikhilsdague: there's a lot of split between the community on getting it in g-client. Lots of -1s on that approach and there's no rationale I am aware of this approach. I too wanted to catch on why people ended up with g-client.16:52
sdaguecode links to where the problematic parts of nova are16:52
nikhilsdague: Blank page it is! Perfect.16:53
*** unicell has quit IRC16:53
alaskidansmith: a couple thoughts: can devstack run the migration? does Tempest rely on the default flavors? Should we just remove the default flavors from the db migration, add documentation on how to add them if deployers need them, and have devstack create default flavors?16:53
dansmithalaski: what if we tried to do the full flavor migration on the first flavor create attempt? that should be an admin-only and non-critical-path operation16:53
sdaguenikhil: right, and explain that one. Because from my perspective glanceclient is *the library provided by the glance team to interact with glance*16:53
dansmithalaski: tempest does, yeah, but it depends on being able to create flavors to test that, so if we block it, it fails16:53
alaskidansmith: I had that thought too, and thought you would tell me I was crazy16:53
nikhilmriedem: sdague: dansmith: https://etherpad.openstack.org/p/nova-glance-newton16:53
dansmithalaski: well, I dunno16:53
dansmithalaski: it's something we could document as "if you don't want to run that migration on the first create, then run the online migrations first"16:54
dansmithalaski: we could also add a workarounds flag to disable that behavior I guess16:54
sdaguenikhil: ok, great.16:54
*** boden has joined #openstack-nova16:54
alaskidansmith: I'm somewhat on board with that though I would like to avoid the workaround flag. I think the proper thing might be to drop default flavor creation from Nova and have devstack populate them16:55
dansmithalaski: we can and should, but I also kinda don't want to block finishing this on that, given we're halfway through16:55
alaskidansmith: yeah, same here. I can get behind the migrate on create behavior, with a reno or something on it16:56
dansmithalaski: so if sdague is cool doing it in devstack before we start, then that's fine with me, and then we can work on getting the flavors out of the main migration and making devstack create them16:56
*** Swami has joined #openstack-nova16:57
dansmithalaski: well, let's see what sdague thinks16:57
sdaguedansmith: what's the thing we need to be cool about?16:57
dansmithsdague: so the problem is, we need to avoid creating flavors in the api db until the main db flavors have been migrated16:57
dansmithsdague: and since those are created in the first migration,16:57
*** syjulian has quit IRC16:57
dansmithdevstack jobs run tempest with flavors in the old location and fail the tempest flavor create tests16:58
dansmithsdague: so I think the cleanest option is to, at least for now, make devstack run the online migrations before we start up nova16:58
sdaguewhat is this going to look like for a completely blank install for people?16:58
dansmithsdague: we can do other things that would impact real deployments16:58
dansmithsdague: people would have to run that before they can create more than the default flavors16:59
sdaguemy concern with "just work around it in devstack", is what the out of the box for every other package solution is16:59
*** jdurgin1 has quit IRC16:59
dansmithsdague: it's the same for other people, because they flavors are created in the original migration16:59
sdaguecould we put an inline db migration for the 5 canned flavors into the normal flow?16:59
dansmiths/other/real/16:59
dansmithno17:00
dansmithit needs to talk to two databases, which is not a cool thing to do in a db migration for one of them,17:00
*** boden has quit IRC17:00
dansmithand it's not zero time, even if the above wasn't a total blocker (which it is)17:00
*** haomaiwang has quit IRC17:01
*** haomaiwang has joined #openstack-nova17:01
dansmithsdague: the other option is for us to just trigger the migration on the first create after startup, which is fine, but it's heavier and it's really more of a hack to our process than what most people will want I think17:01
alaskiso I know it's not cool to rewrite db history, but what about removing the addition of the default flavors in migration 216? and start expecting deployers to create those flavors?17:01
dansmithalaski: we need to do that,17:01
dansmithbut we also need to build more things in order to get us back to having flavors again17:02
*** salv-orlando has joined #openstack-nova17:02
dansmithi.e. more devstack changes to do the flavor creates and such17:02
alaskiyeah17:02
*** Marga_ has joined #openstack-nova17:02
dansmithand I just don't want to pause in the middle of this flavor process to go do that flavor process and complicate things a lot17:02
dansmithrunning the online migrations as part of a new setup is kindof the same as running them as part of a migration17:03
dansmithwe could have other things in the future that behave like this17:03
sdagueit feel like we're just exporting complexity to ops. Things used to largely just work, now, there is lots of other things that need to be done. :(17:03
dansmithsdague: it used to be all done offline though17:03
sdaguedansmith: sure17:04
dansmithsdague: it's also just one command that runs all the pending migrations,17:04
dansmithit's not like this is one thing now and another thing later for a different but similar case17:04
*** syjulian has joined #openstack-nova17:04
sdaguethat thing you said about handingly it on first request, how about doing that?17:05
sdagueand if you want to preoptimize you can run the tool up front17:05
dansmithwe can, it's just that almost no real deployer will want that17:05
dansmithyep, that's what I said above17:05
dansmithso, that's fine if you want, I just think it's a hack on our process and not very realistic :)17:05
*** ociuhandu_ has joined #openstack-nova17:05
*** ociuhandu has quit IRC17:05
*** ociuhandu_ is now known as ociuhandu17:05
sdaguedansmith: that's fine, but we still want people doing department level installs to work17:06
dansmithit's what people running single-node clouds on their dev boxes will want, but not really anyone else17:06
*** yamahata has quit IRC17:07
dansmithalaski: any more thoughts?17:07
*** yamahata has joined #openstack-nova17:07
*** ssurana has quit IRC17:07
jaypipesedleafe: see latest comment on https://review.openstack.org/#/c/286931/. Your change apparently doesn't work..17:08
alaskidoing it on flavor.create is a hack which I would like to avoid. the issue is that we put data in the db in our migrations and it needs to be migrated. asking users to do a data migration after deploy, until we fix this, seems reasonable17:08
dansmithalaski: also, on the workaround thing, I was going to say default to doing the migration, and just let them disable that auto behavior if they wanted .. i.e. if they're deploying to a cloud where sub-admins would trigger the migration when they might not want them to17:09
jaypipesedleafe: left comment on PS4.17:09
*** boden has joined #openstack-nova17:10
*** ssurana has joined #openstack-nova17:10
alaskidansmith: yeah, if we go that route I think migrate by default would be the right thing17:10
dansmithalaski: basically I feel like if we do the migrate-on-create hack, I'd hate to force other deployers to do the migration offline ahead of time to avoid taking the hit on a create from a sub-admin not paying attention to the upgrade process17:10
edleafejaypipes: ok, will look into that later. No idea where the None is coming from.17:11
jaypipesedleafe: I'm fixing it now.17:11
*** browne has joined #openstack-nova17:11
alaskidansmith: that seems a reasonable compromise to me. and once we clean up the 216 migration it can be removed.17:13
dansmithalaski: okay17:13
dansmithsdague: just to be clear, we're going to have to add flavor creation to devstack when we do that17:14
sdaguedansmith: could we put that logic back into nova-manage? nova-manage create-default-flavors17:15
*** moshele has quit IRC17:15
dansmithsdague: I don't really think that nova-manage should encode default flavor info any more than the first migration should17:15
dansmithsdague: but even if we did, it's still yet another step devstack needs to do17:16
*** salv-orlando has quit IRC17:16
sdaguedansmith: sure, but it's at least something which ends up mildly consistent. I get that sticking the flavor data in the default migration is ... odd17:16
*** mylu has quit IRC17:16
sdaguebut use just saying "oh, yeh, flavors, you need some before anything works"17:17
sdagueseems fine for large scale infrastructure17:17
sdaguebut another odd hoop to jump through for the smaller folks17:17
dansmithyeah, I dunno if I agree with that17:17
*** mylu has joined #openstack-nova17:18
dansmithnova-manage doesn't have flavor commands anymore, right?17:18
dansmithit's all done via the API now I think17:18
* dansmith looks17:18
sdagueit probably doesn't17:18
dansmithyeah17:18
alaskiI thought there was an idea from a while ago of just documenting some reasonable default flavors for deploys like that17:18
dansmithit doesn't17:18
sdaguealaski: I don't know17:18
sdagueI just know that for the last 4 years, you install nova, and you have 5 flavors17:18
dansmithalaski: right, just "run these flavor-create commands if you don't know yet" right?17:19
sdaguesaying that's not going to be the case is a big change17:19
alaskidansmith: exactly17:19
*** unicell has joined #openstack-nova17:19
dansmithalaski: yeah, and having devstack run those commands for you seems right to me17:19
*** mylu has quit IRC17:19
*** unicell has quit IRC17:19
*** unicell1 has joined #openstack-nova17:19
alaskisdague: we already expect users to bring images/volumes and other resources. some amount of setup seems reasonable17:19
dansmithyeah17:19
sdagueevery additional command we make people do manually does impact initial usability17:19
dansmithimages are as fundamental to me17:19
dansmithsdague: right, but that's what devstack is right? a demonstration of all the steps that is packaged so you can test easily and then set it up for real?17:20
dansmithI mean,17:20
dansmithanyone doing this beyond devstack is pretty much going to use one of the deploy tools, which is going to do this for you17:20
dansmiththey might have a template "flavors.yml" or something17:20
openstackgerritJay Pipes proposed openstack/nova: Make compute_node_statistics() use new schema  https://review.openstack.org/28693117:20
dansmithbut if you don't, you get the defaults17:20
alaskiyeah17:20
sdaguedansmith: and if it's a thing that nova doesn't have a command for, it will be different in every deploy tool17:21
sdagueand if that's a thing that is fine with everyone... I guess that's what it is17:21
dansmithsdague: the actual flavors? yeah...17:21
dansmiththe flavors we have now are copies of default amazon flavors from 2010 right? :)17:21
sdaguepart of what is nice with these things being nova, is there are same know defaults17:21
alaskisdague: of the clouds infra interacts with how many use the default flavors?17:21
sdaguealaski: I'm not talking about public clouds17:22
dansmithsdague: which means people encode flavor names into tests because they think every cloud has those right?17:22
*** achanda has joined #openstack-nova17:22
dansmithI feel like we just had the opposite argument about people encodingthings they believe to be always present17:22
sdaguedansmith: where do they do that?17:22
dansmithwith the flavor url17:22
*** edmondsw has joined #openstack-nova17:23
dansmithwhere does who do what?17:23
sdagueok, so I'm clearly in the minority here. It feels like we're adding a bunch more manual to people. If that's the project direction, so be it, we'll put it into devstack17:24
*** mylu has joined #openstack-nova17:24
*** david-lyle has quit IRC17:24
alaskianother aspect of this is that flavors should be matched against the hardware in use or you waste space on packing, so precanned defaults seem problematic in that regards as well17:24
dansmithnone of the default flavors work for my test box, for example17:25
dansmiththe smallest has no disk and the next one in line has too much disk for me to start more than one test instance17:25
dansmithso I always have to create another one17:25
*** david-lyle has joined #openstack-nova17:25
dansmithincidentally, it's called "foo" because I'm very creative17:25
* mriedem is asking about this in -operators17:26
*** achanda has quit IRC17:27
mriedemjlk is picking up what you're putting down at least17:27
sdagueyeh, I guess I'd feel better about this if we had other parts of the flavor interface tweaked at the same time.17:28
syjulianHi everyone, I'm a developer at AT&T and new to OpenStack. Our team created a blueprint regarding Nova Notification. If anyone has time, we would greatly appreciate any feedback or opinion. https://blueprints.launchpad.net/nova/+spec/nova-notification-heat-stack-info :)17:29
dansmithsdague: alaski: so for this migrate-on-create hack,17:29
dansmithI have to come up with a limit on how many I'm willing to migrate spontaneously,17:29
sdaguewould be nice to have GET /flavors/?min_mem=1G&min_disk=20G&min_cpu=217:29
dansmithand fail if that's not done17:29
dansmiththe problem there is,17:29
dansmithit will appear to auto-migrate for some people and not others17:29
dansmithand will leave it half-migrated for the people it doesn't work for17:30
dansmithwhich seems somewhat uncool17:30
dansmithsdague: alaski either of you have an opinion on how people will like that/17:30
alaskiyeah, can't really do it half way17:30
dansmithwell, you can, but it doesn't help,17:31
dansmithwe'd still have to fail the request17:31
dansmithwe're just hacking around tempest and devstack at that point, which seems uncool17:31
alaskiright. s/can't/shouldn't/17:31
sdagueok, so, sorry, I'm not sure if I have the whole context17:31
sdaguecan you walk me through exactly what the failure is here when tempest tries to run?17:31
*** penick has joined #openstack-nova17:31
dansmithsdague: tempest goes to test flavor creation17:32
sdaguemaybe if I get the whole picture in my head, I'll be less daft17:32
dansmithsdague: which can't complete until flavors are migrated from main to api17:32
dansmithsdague: because we need the integer ids to match up17:32
dansmithsdague: so the test fails because the migration isn't done17:32
dansmithsdague: which means we can't run the tempest flavor test until we've done that migration17:32
dansmithsdague: and you said you would rather me put the migrate-on-create hack in there to avoid just running the command in devstack during setup17:32
sdaguewhy are we depending on integer ids lining up?17:33
alaskisdague: they're exposed in the api17:33
sdaguealaski: ... are they actually?17:33
sdagueI thought it was the thing people assume is an int, but isn't17:33
dansmithsdague: but I have to put a limit value in there, which can be extremely large, but will likely timeout on large deploys, or a small value which will not get the whole deal for small ones17:33
*** Goneri has joined #openstack-nova17:34
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/flavors.py#L3717:34
dansmithso it will appear to auto-migrate for some people, but not others17:34
mriedemisn't the integer17:34
alaskiI don't know if it's visible, sorry. but you can reference it when booting, I'm fairly certain17:34
sdaguealaski: I'm not sure that you can17:34
dansmithI think alaski is right17:35
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L97417:35
nikhilsdague: I'm almost done with the etherpad on this new __init__ . Just one quick question: do you have any specific concerns on the testing gaps or this is a research item?17:36
dansmithalaski: sdague mriedem: this is why flavors on the cells stuff have to be sync'd up so tightly right?17:36
mriedemdansmith: i didn't know why the id had to be the same honestly17:36
alaskidansmith: right, lookups were happening on id at some point17:37
mriedemflavorid should be our entry point since that's what we list17:37
sdaguethe only cross reference is this - https://github.com/openstack/nova/blob/b5e65b4356f022d999482bfcc35bbf5c8cc42f8d/nova/db/sqlalchemy/models.py#L27917:37
sdaguewhich may be used by some objects17:37
sdaguebut end users never see that17:37
* mriedem fires up devstack to test this out17:37
dansmithsdague: resizes and instances maintain internal links to flavors via id too17:38
sdaguedansmith: ok17:38
dansmithsdague: which means if we let the space overlap we end up breaking some things17:38
dansmithand I think cells has some cleanup routines when it replays some operations that look up via id17:38
dansmithalaski: was that one of them this morning? the rpc compat thing?17:38
*** roeyc has joined #openstack-nova17:39
alaskithere's this objects.Flavor.get_by_id(context, flavor['id'])17:39
dansmithwe also expose the flavor low-level ids in the migrations api17:40
sdaguedansmith: ok, so I retract my earlier comment, if this is just part of normal upgrade you have to do, that's fine we can put it in devstack right after the db-sync commands17:40
*** busterswt has joined #openstack-nova17:40
*** tbachman_ has joined #openstack-nova17:40
dansmithsdague: okay17:40
dansmithsdague: so if we do the thing you want, which is "nova manage sync-all-my-shit-i-will-wait" then we could do this migrate command in there for people17:41
*** hieulq has quit IRC17:41
mriedemgibi: you might want to look at this re: notifications https://blueprints.launchpad.net/nova/+spec/nova-notification-heat-stack-info17:41
sdaguedansmith: sure17:41
*** tbachman has quit IRC17:42
*** tbachman_ is now known as tbachman17:42
sdagueit would be nice to stop having 2 different flavorid fields that confuse everyone :)17:42
sdagueand one is used half the places, and the other is used the other half the places17:42
dansmithsdague: no dude, it's flavor.id and flavor_id.. not confusing at all, come on!17:42
mriedemyeah, was just going to say that, ha17:42
mriedemflavor_id != flavorid17:42
mriedemduh!17:42
dansmithheh17:42
dansmithdid I mention I hate the flavor code yet this week?17:42
mriedemdansmith: pays the bills17:43
dansmithsho' nuf17:43
mriedemif you're feeling down, just crank this up https://www.youtube.com/watch?v=Lrle0x_DHBM17:43
*** doug-fis_ has quit IRC17:43
*** krtaylor has quit IRC17:44
*** Marga_ has quit IRC17:44
*** nelsnels_ has joined #openstack-nova17:46
*** Marga_ has joined #openstack-nova17:46
*** achanda has joined #openstack-nova17:46
*** tongli has joined #openstack-nova17:47
*** nelsnelson has quit IRC17:48
openstackgerritJeremy Stanley proposed openstack/nova: List system dependencies for running common tests  https://review.openstack.org/20067217:48
*** akshai has quit IRC17:51
doffmccarmack: About https://review.openstack.org/#/c/297977/4. What is going to happen for the other quota drivers?17:51
*** akshai has joined #openstack-nova17:52
mriedemgross, someone heated up what smells like tuna in the office17:52
*** roeyc has quit IRC17:52
doffmMmmmmm hot tuna.17:52
mriedemwas it you?17:52
jrollreason number 245768 not to work in an office17:53
dansmithsdague: depends-on won't work between nova and devstack, right/17:54
ccarmackdoffm: are there other quota drivers out of tree?17:54
jrolldansmith: it should17:54
dansmithjroll: oh, cool17:54
mriedemyes it does17:54
ccarmackI had a similiar question for the db interface, doffm17:54
jrolldansmith: with clients/libs it doesn't work because they're installed from pypi17:55
mriedemor project-config17:55
dansmithokay17:55
jrollya17:55
dansmithsweet17:55
ccarmackmriedem: If I add an API to quota.py QuotaEngine, does that mean all the out tree quota drivers have to support it?17:55
mriedemccarmack: out of tree quota driveres?17:55
*** krtaylor has joined #openstack-nova17:56
mriedemccarmack: but no17:56
jrollmriedem: with zuulv3 we get project-config dependency-ability17:56
jroll\o/17:56
mriedemccarmack: we have a virt api that's unversioned17:56
*** Sukhdev has quit IRC17:57
ccarmackmriedem: it seems like both the QuotaEngine and the DB layer API when meant to be pluggable17:57
*** e0ne has joined #openstack-nova17:57
ccarmack*were17:57
mriedemccarmack: not once sdague is done burning all of those bridges :)17:58
ccarmackmriedem: burning bridges?  explain please17:58
mriedemccarmack: deprecating plug points17:58
mriedeme.g. compute manager, network api, etc17:58
*** penick has quit IRC17:59
mriedemseveral things like that were deprecated in mitaka17:59
ccarmackmriedem: so we won't have any compatibility requirements except for the public nova api?18:00
*** baoli has quit IRC18:00
*** syjulian has quit IRC18:00
mriedemccarmack: we don't have any compat requirements for unversioned apis18:00
mriedemwe try not to break things18:00
openstackgerritGleb Stepanov proposed openstack/nova: [WIP] Add console log offset  https://review.openstack.org/29247318:00
*** haomaiwang has quit IRC18:01
doffmccarmack: It shouldn't break any other quota drivers.18:01
mriedemlike before notifications were versioned18:01
*** baoli has joined #openstack-nova18:01
mriedemwe wouldn't drop things or rename keys in the notification payload18:01
*** haomaiwang has joined #openstack-nova18:01
dansmithevery time bashate finds something in my change I lose a little self esteem18:02
*** penick has joined #openstack-nova18:03
ccarmackdoffm, mriedem: ok, doffm raised the question about my additions to the quota and db api for https://review.openstack.org/#/c/297977/418:03
mriedemyikes18:03
mriedemccarmack: does that change need to be 500+ LOC?18:03
openstackgerritGleb Stepanov proposed openstack/nova: [WIP] Add console log offset  https://review.openstack.org/29247318:04
ccarmackmriedem: I think so, a lot of that are tests18:04
*** Jeffrey4l has quit IRC18:04
*** doug-fish has joined #openstack-nova18:04
*** moshele has joined #openstack-nova18:05
ccarmack200+ are tests, 100+ is the sqlalchemy/api.py refactor18:05
*** doug-fis_ has joined #openstack-nova18:05
doffmccarmack: I don't think your change would break other quota drivers, so I wasn't really worried. Just wanted to know what the position was.18:05
doffmThe quota driver is still pluggable.18:05
ccarmackdoffm: ok18:06
*** hemanthm is now known as hemanthm|afk18:06
openstackgerritDan Smith proposed openstack/nova: Block flavor creation until main database is empty  https://review.openstack.org/29531018:06
mriedemthere are only 2 in tree quota drivers, db (default) and noop18:06
mriedemthe db quota driver uses the nova.db.api18:06
mriedemnoop doesn't do jack18:06
mriedemthere are no changes to the quota drivers 'interface' here so i don't see a problem with that18:07
ccarmackmriedem: only additions18:07
mriedemthe db api refactor scares me a bit, but just requires close review18:07
*** unicell1 has quit IRC18:07
mriedemtechnically this could be 2 changes,18:07
*** unicell has joined #openstack-nova18:07
*** ericksonsantos has quit IRC18:07
mriedem1. refactor db api to pull out common code18:07
mriedem2. add the cli18:07
doffmThe db api changes are mostly a function split of existing code.18:08
mriedemccarmack: like, you don't have any tests for your new cli18:08
mriedemoh god nevermind i missed this in the quota driver https://review.openstack.org/#/c/297977/4/nova/quota.py18:09
ccarmackmriedem: I have tests for the quotas API called by the CLI, but you are right.  Of course, the manage CLI has no existing tests I could find18:09
ccarmackwell, it had one...18:09
mriedemccarmack: nova.tests.test_nova_manage18:09
doffmThe manage cli has tests.18:09
*** doug-fish has quit IRC18:09
*** lpetrut has quit IRC18:10
ccarmackIs that the test with only 1 test?18:10
mriedemnope18:10
doffmccarmack: There are two.18:10
mriedemccarmack: why even change the quota driver?18:10
mriedemnova-manage has direct access to the db api18:10
mriedemthe nova-manage command doesn't care about the noop quota driver18:11
doffmccarmack: The one with one test is nova.tests.cmd.test_manage. Thats a mistake, it should probably be merged with nova.tests.test_nova_manage18:11
doffmLeast I think its a mistake.18:11
ccarmackmriedem:  other commands call quota18:11
mriedemdoffm: not really18:11
ccarmackand sometimes they call db directly18:12
mriedemccarmack: like ProjectCommands i guess18:12
ccarmackmriedem: also, the db doesn't know which quotas are syncable18:12
*** david-lyle_ has joined #openstack-nova18:12
mriedemccarmack: you mean the PER_PROJECT_QUOTAS?18:12
ccarmackits a mess18:13
openstackgerritGleb Stepanov proposed openstack/nova: [WIP] Add console log offset  https://review.openstack.org/29247318:13
mriedemya...18:13
ccarmackquotas has those "resources" with syncable attribute… while db has the QUOTA_SYNC_FUNCS18:13
*** david-lyle has quit IRC18:14
ccarmackbut QUOTA_SYNC_FUNCS is not a list of syncable quotas, its is a lookup from resource name to sync func18:14
mriedemalright18:15
mriedemccarmack: but can you split this into 3 changes?18:15
mriedem1. db api refactor, 2. quotas driver, 3. nova-manage cli18:15
ccarmackmriedem: why?18:15
mriedemccarmack: because i don't want to review 500 LOC18:16
*** pcaruana has quit IRC18:16
dansmithccarmack: yeah18:16
ccarmackmriedem: its a spec implementation18:16
dansmithccarmack: please18:16
ccarmack500 is not alot18:16
mriedemccarmack: oh boy18:16
ccarmackok18:16
mriedemccarmack: it is18:16
mriedempatches should be split into logical chunks when possible18:16
ccarmackI disagree that 500 is a lot for a new feature, especially if 200 is test18:17
ccarmackbut anyway, ok18:17
mriedembut you don't have tests for the cli18:17
ccarmackyup18:17
mriedemwhich someone might not notice because they are glazing over from the other 500 LOC18:17
ccarmackI'll add those18:17
mriedemwhen you split it up, we can make sure everything is cool in each change18:17
mriedemsee my masterful series starting here for get-me-a-network https://review.openstack.org/#/c/297870/18:17
*** salv-orlando has joined #openstack-nova18:18
ccarmackmriedem: ok, I'll split it into 318:18
mriedemccarmack: plan on me razzing you about this in person for at least a week18:18
ccarmackmriedem: we are collaborators, not razzers18:19
mriedemccarmack: ok, so doffm and i will gang up on you then :P18:19
ccarmackyou are a tough ptl, mriedem18:20
ccarmackby the book18:20
*** ArxCruz has joined #openstack-nova18:20
ccarmacktight ship18:20
ccarmackdotted i's and t's18:20
mriedemdon't forget lower-case j's18:20
ccarmackwait, we don't dot t's18:21
mriedemthey might in germany, idk18:22
*** edmondsw has quit IRC18:23
openstackgerritAndrew Laski proposed openstack/nova: Populate instance_mappings during boot  https://review.openstack.org/26392518:24
*** ArxCruz has quit IRC18:24
*** salv-orlando has quit IRC18:25
*** pwnall has quit IRC18:28
mriedemjaypipes: are you on a plane still?"18:29
jaypipesmriedem: leaving now for the plane.. :(18:29
jaypipesmriedem: hit me on email?18:29
mriedemsure18:30
*** initio has quit IRC18:30
mriedemjust reply to https://review.openstack.org/#/c/297871/1/nova/objects/network_request.py when you get a chance18:30
*** pcaruana has joined #openstack-nova18:30
jaypipeswill do. ciao.18:30
*** jaypipes has quit IRC18:31
*** sudipto has quit IRC18:32
*** baoli has quit IRC18:33
*** baoli has joined #openstack-nova18:35
*** pushkaru has quit IRC18:40
*** baoli has quit IRC18:40
*** pushkaru has joined #openstack-nova18:40
*** knikolla_ has joined #openstack-nova18:41
*** baoli has joined #openstack-nova18:41
*** doug-fis_ has quit IRC18:43
*** doug-fish has joined #openstack-nova18:44
*** baoli has quit IRC18:46
*** baoli has joined #openstack-nova18:48
mriedemColumn('consumer_id', String(36), nullable=False),18:49
mriedemconsumer_id VARCHAR(64) NOT NULL,18:49
mriedemi guess the data model doesn't match the spec for the allocations table huh18:49
mriedemwe must have decided to not bake in future requirements until those are actually needed18:50
*** e0ne has quit IRC18:50
*** achanda has quit IRC18:51
mriedemhuh, we have a bug18:52
mriedemhttps://github.com/openstack/nova/blob/9bc935d7d059c5ad9ff563b373691c3134e8f3ac/nova/db/sqlalchemy/models.py#L1503-L150518:52
mriedemallocations_resource_provider_class_used_idx has the used column in it, like the spec18:52
mriedemthe db migration doesn't https://github.com/openstack/nova/blob/9bc935d7d059c5ad9ff563b373691c3134e8f3ac/nova/db/sqlalchemy/migrate_repo/versions/314_add_resource_provider_tables.py#L7118:53
*** achanda has joined #openstack-nova18:54
*** fawadkhaliq has quit IRC18:55
*** hemanthm|afk is now known as hemanthm18:55
*** fawadkhaliq has joined #openstack-nova18:55
mriedemhttps://bugs.launchpad.net/nova/+bug/156300618:56
openstackLaunchpad bug 1563006 in OpenStack Compute (nova) "db: 'allocations' table migration and model have conflicting indexes" [Undecided,Confirmed]18:56
*** penick has quit IRC18:57
mriedemalaski: what's the best way to fix an index in a db migration? add the new index and then drop the old one?18:57
dansmithmriedem: we're holding off on this: https://review.openstack.org/#/c/282442/ until after summit, right/18:58
mriedemdansmith: nope18:58
*** zenoway has joined #openstack-nova18:58
mriedemwhich is why i'm looking at this18:58
dansmithmriedem: okay, I thought we were18:58
mriedemdansmith: that was the last thing left in the resource-providers blueprint18:58
*** baoli has quit IRC18:58
*** penick has joined #openstack-nova18:59
*** baoli has joined #openstack-nova18:59
dansmithmriedem: okay, but it doesn't make any sense without the inventory stuff merged18:59
mriedemdansmith: from reading the spec, which is maybe dangerous, the resource-providers blueprint just creates the 3 tables and the objects for them,19:00
*** baoli has quit IRC19:00
mriedemusing them to populate the tables comes later19:00
mriedemwith resource-providers-allocations19:00
dansmithI don't know why we have a spec for "add things we don't use"19:00
dansmithheh, okay whatever19:00
alaskimriedem: I'm not aware of any other way to do it19:00
*** haomaiwang has quit IRC19:01
mriedemdansmith: idk, i assumed this was just building blocks19:01
*** edmondsw has joined #openstack-nova19:01
mriedemit's just a patch to add objects, i think it should be able to get merged this week19:01
*** haomaiwang has joined #openstack-nova19:01
*** moshele has quit IRC19:02
*** vilobhmm11 has joined #openstack-nova19:02
dansmithmriedem: I mean that stack of things19:02
*** moshele has joined #openstack-nova19:03
mriedemthe next thing in that stack is https://review.openstack.org/#/c/281945/ which is marked for generic-resource-pools, which we said we wouldn't do before the summit19:03
dansmithmriedem: right, so, I really don't think we should land those objects until we have code that even shows they're right19:04
dansmithmriedem: I literally just carved them out of an earlier patch of someone else's to avoid landing them until we had any idea if they were right19:04
dansmithmriedem: do you know of any code that uses them at all?19:05
mriedemi don't,19:05
mriedemthe inventory stuff is supposed to right?19:05
dansmithno19:05
mriedemwhatever is in resource-providers-allocations19:05
dansmithyeah, I know we have lots of specs up, but I haven't seen code to use these allocation objects yet :)19:05
mriedemdansmith: feel free to WIP that one until something is up and using it19:06
dansmithinventory is about compute nodes, allocations are about instances19:06
mriedemand i'll gladly move on with my time :)19:06
dansmithand I've seen no work on that19:06
dansmithokay19:06
cfriesen__dansmith: doesn't this use them?  https://review.openstack.org/#/c/271823/319:06
mriedemcfriesen__: tha's a spec19:06
mriedemnot code19:06
cfriesen__okay, fair enough19:06
*** kfarr has joined #openstack-nova19:06
dansmithlol19:07
mriedemanyway, i'll push a patch to fix this index anyway since it's clearly wrong19:07
mriedemand let jay look at it when he lands19:07
*** earlephilhower has joined #openstack-nova19:10
*** smatzek_ has joined #openstack-nova19:10
nikhilmriedem: thanks for your question on the ethepad https://etherpad.openstack.org/p/nova-glance-newton line 35. Feel free to glance it over, whenever :)19:13
openstackgerritDan Smith proposed openstack/nova: Migrate compute node resource information to Inventory objects  https://review.openstack.org/27931319:13
openstackgerritDan Smith proposed openstack/nova: Add Service.get_minimum_version_multi() for multiple binaries  https://review.openstack.org/27931219:13
openstackgerritDan Smith proposed openstack/nova: Make compute_node_statistics() use new schema  https://review.openstack.org/28693119:13
*** pushkaru has quit IRC19:14
*** smatzek has quit IRC19:14
*** BassT has quit IRC19:17
*** ZZelle has quit IRC19:18
*** e0ne has joined #openstack-nova19:19
*** e0ne has quit IRC19:21
*** angdraug has joined #openstack-nova19:22
*** pushkaru has joined #openstack-nova19:22
*** doug-fis_ has joined #openstack-nova19:22
*** penick has quit IRC19:23
*** doug-fish has quit IRC19:25
*** e0ne has joined #openstack-nova19:26
*** Sukhdev has joined #openstack-nova19:27
*** doug-fis_ has quit IRC19:27
*** doug-fish has joined #openstack-nova19:28
*** moshele has quit IRC19:28
*** e0ne has quit IRC19:28
*** smatzek_ is now known as smatzek19:29
*** syjulian has joined #openstack-nova19:33
*** syjulian has quit IRC19:34
*** mylu has quit IRC19:36
*** mylu has joined #openstack-nova19:38
*** fawadkhaliq has quit IRC19:38
*** fawadkhaliq has joined #openstack-nova19:38
*** moshele has joined #openstack-nova19:39
*** zenoway has quit IRC19:40
*** pwnall has joined #openstack-nova19:40
*** zenoway has joined #openstack-nova19:42
*** salv-orlando has joined #openstack-nova19:42
*** moshele has quit IRC19:42
dansmithalaski: woot, the block migration patch passed normal the tempest/devstack job with the pre-migration19:44
alaskidansmith: awesome19:45
dansmithgotta fix up a functional thing and then I think we're about ready to get out the nails for this coffin19:46
*** lpetrut has joined #openstack-nova19:47
*** Sukhdev has quit IRC19:47
*** baoli has joined #openstack-nova19:47
*** smatzek has quit IRC19:48
*** baoli has quit IRC19:48
*** baoli has joined #openstack-nova19:49
*** omnipresent has joined #openstack-nova19:50
*** yamahata has quit IRC19:50
*** ZZelle has joined #openstack-nova19:51
*** salv-orlando has quit IRC19:52
openstackgerritTracy Jones proposed openstack/nova: config options: centralize section "guestfs"  https://review.openstack.org/24896319:52
*** omnipresent has quit IRC19:53
*** zenoway has quit IRC19:53
*** tlbr has quit IRC19:53
*** zenoway has joined #openstack-nova19:54
nikhilmriedem: sdague: dansmith: Thank you for your input today!19:54
*** ArxCruz has joined #openstack-nova19:56
*** sheel has quit IRC19:57
*** knikolla_ has quit IRC19:57
*** edtubill has joined #openstack-nova19:58
openstackgerritDan Smith proposed openstack/nova: Block flavor creation until main database is empty  https://review.openstack.org/29531019:58
*** ArxCruz has quit IRC20:00
*** haomaiwang has quit IRC20:01
*** haomaiwang has joined #openstack-nova20:01
mriedemnikhil: sure, np, thanks for focusing on this20:03
*** Sukhdev has joined #openstack-nova20:04
*** pwnall has quit IRC20:08
*** penick has joined #openstack-nova20:08
*** omnipresent has joined #openstack-nova20:09
*** baoli has quit IRC20:10
*** baoli has joined #openstack-nova20:10
*** baoli has quit IRC20:11
*** baoli has joined #openstack-nova20:11
*** zenoway_ has joined #openstack-nova20:12
*** zenoway has quit IRC20:12
*** pwnall has joined #openstack-nova20:13
openstackgerritJeremy Stanley proposed openstack/nova: List system dependencies for running common tests  https://review.openstack.org/20067220:13
*** yamahata has joined #openstack-nova20:13
*** omnipresent has quit IRC20:14
*** matrohon has joined #openstack-nova20:16
openstackgerritMatt Riedemann proposed openstack/nova: Fix allocations resource provider class used index  https://review.openstack.org/29842320:17
*** zenoway has joined #openstack-nova20:18
*** syjulian has joined #openstack-nova20:19
*** zenoway_ has quit IRC20:20
openstackgerritMark Doffman proposed openstack/nova: Add manage command for cell0.  https://review.openstack.org/27526620:21
openstackgerritMark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule  https://review.openstack.org/26782720:21
openstackgerritMark Doffman proposed openstack/nova: Manage db sync command for cell0  https://review.openstack.org/27488520:21
*** Sukhdev has quit IRC20:21
*** baoli has quit IRC20:22
*** ZZelle has quit IRC20:23
*** fawadkhaliq has quit IRC20:24
*** fawadkhaliq has joined #openstack-nova20:25
*** zenoway has quit IRC20:25
*** rmevans has joined #openstack-nova20:27
*** zenoway has joined #openstack-nova20:28
*** Sukhdev has joined #openstack-nova20:28
*** rmevans has quit IRC20:35
*** Goneri has quit IRC20:40
*** Sukhdev has quit IRC20:42
*** matrohon has quit IRC20:43
mriedemalaski: went through https://review.openstack.org/#/c/263925/ - some questions inline20:47
alaskimriedem: most of that is a result of adding in the previous change which made cell_id an internal detail of the object and added cell_mapping and me not updating to reflect that20:50
alaskiI'll fix that all up20:50
mriedemalaski: ok, i figured20:50
alaskithe mock call things I'm not sure what to do with20:50
mriedemalaski: i'm testing those out locally20:50
alaskithose are the calls I see, though I agree it's really weird20:51
mriedemlike, i'm commenting out inst_mapping.destroy() in the host mapping not found case to see if the tests handle that20:51
*** mylu has quit IRC20:51
*** achanda has quit IRC20:52
*** baoli has joined #openstack-nova20:52
*** eil397 has joined #openstack-nova20:55
alaskibecause that method is called in a loop (get_by_instance_uuid) I couldn't get at the returned mocks directly. maybe I could try using side_effect and pass mocks in20:55
mriedemyou can use side_effect to pass the iterator of results20:56
*** syjulian has quit IRC20:56
mriedemwhich would be mocks...or a single mock that you assert destroy() is called on twice20:57
*** efoley has joined #openstack-nova20:57
mriedemyeah, something is up, because this passes http://paste.openstack.org/show/492161/20:57
*** eil397 has left #openstack-nova20:58
alaskihmm. I'll fix up the other stuff and then mess with that20:59
*** fawadkhaliq has quit IRC20:59
mriedemok, i'll see if i can clean up the test after the stable team meeting21:00
mriedemwon't push it up, just a paste if i get it working21:00
*** fawadkhaliq has joined #openstack-nova21:00
*** baoli has quit IRC21:00
*** haomaiwang has quit IRC21:01
*** penick has quit IRC21:01
*** haomaiwang has joined #openstack-nova21:01
*** ericksonsantos has joined #openstack-nova21:02
*** thorst has quit IRC21:03
*** achanda has joined #openstack-nova21:04
*** Sukhdev has joined #openstack-nova21:04
*** Sukhdev has quit IRC21:05
*** Marga_ has quit IRC21:06
*** Marga_ has joined #openstack-nova21:06
*** baoli has joined #openstack-nova21:07
*** Sukhdev has joined #openstack-nova21:08
*** thorst has joined #openstack-nova21:08
mriedemalaski: like this http://paste.openstack.org/show/492164/21:08
alaskimriedem: thanks21:09
*** thorst has quit IRC21:12
*** aysyd has quit IRC21:13
*** rtheis has quit IRC21:15
*** eharney has quit IRC21:17
*** tlbr has joined #openstack-nova21:18
openstackgerritAndrew Laski proposed openstack/nova: Populate instance_mappings during boot  https://review.openstack.org/26392521:19
*** ociuhandu has quit IRC21:21
*** prometheanfire has quit IRC21:21
*** prometheanfire has joined #openstack-nova21:22
*** raildo is now known as raildo-afk21:23
openstackgerritChuck Carmack proposed openstack/nova: WIP: Change the nova tempest blacklist to use to idempotent ids  https://review.openstack.org/29843721:24
*** thorst has joined #openstack-nova21:26
*** salv-orlando has joined #openstack-nova21:27
openstackgerritRyan Rossiter proposed openstack/nova: Use ovo in test_obj_make_compatible()  https://review.openstack.org/27229921:28
*** thorst has quit IRC21:30
*** edtubill has quit IRC21:34
*** salv-orlando has quit IRC21:35
*** ociuhandu has joined #openstack-nova21:35
*** doug-fish has quit IRC21:36
*** Sukhdev has quit IRC21:36
*** achanda has quit IRC21:37
*** Sukhdev has joined #openstack-nova21:37
*** salv-orlando has joined #openstack-nova21:38
*** salv-orlando has quit IRC21:42
mriedemalaski: soo close https://review.openstack.org/#/c/263925/21:42
mriedemalaski: can you touch up that last test in conductor too for the mocks? then i'm +221:42
alaskidangit21:43
alaskiyeah21:43
*** salv-orlando has joined #openstack-nova21:45
*** cfarquhar has quit IRC21:45
*** cfarquhar has joined #openstack-nova21:48
*** cfarquhar has quit IRC21:48
*** cfarquhar has joined #openstack-nova21:48
*** ivase has quit IRC21:48
*** ctrath has quit IRC21:49
*** ivase has joined #openstack-nova21:53
*** kfarr has quit IRC21:53
*** yamahata has quit IRC21:53
*** yamahata has joined #openstack-nova21:54
*** mriedem has quit IRC21:55
*** doug-fish has joined #openstack-nova21:57
*** thorst has joined #openstack-nova21:58
*** burgerk_ has joined #openstack-nova21:58
*** ivase has quit IRC21:59
*** haomaiwang has quit IRC22:01
*** haomaiwa_ has joined #openstack-nova22:01
*** rlrossit has quit IRC22:01
openstackgerritRyan Rossiter proposed openstack/nova: Remove DictCompat from ComputeNode  https://review.openstack.org/26552122:01
*** burgerk has quit IRC22:02
dansmithalaski: http://logs.openstack.org/10/295310/19/check/gate-tempest-dsvm-postgres-full/35ab35d/logs/devstacklog.txt.gz#_2016-03-28_20_30_38_05822:02
dansmithtempest is creating flavors22:02
dansmithwell, I guess that's devstack lib/tempest actually22:02
*** Sukhdev has quit IRC22:03
dansmithalaski: heh, because the default flavors aren't useful for us either :D22:03
alaski:)22:03
dansmithI dunno why that fails on that postgres job and not any others22:04
*** achanda has joined #openstack-nova22:04
alaskithat flavor isn't shown in flavor-list22:05
alaskicould there be a race somehow?22:05
dansmithbecause it's creating it and fails after the list22:05
alaskiyeah, but the failure says it exists even though it's not listed22:06
dansmithright22:06
dansmithso the code I copied from db_api assumes that it's a flavor name conflict if it's not an id conflict22:08
alaskiahh22:08
*** burgerk_ has quit IRC22:08
dansmithbut no projects or extra specs at that point, so I'm not sure what else it couldbe22:09
dansmithoh22:09
dansmithbut this is postgres22:09
dansmithso maybe it's interpreting the UC differently somehow?22:09
*** cleong has quit IRC22:10
alaskicould be I guess. It's only on name and flavorid22:11
dansmithI guess I need a canary on top to dump out the exception and see22:11
*** apoorvad has quit IRC22:11
alaskiflavorid doesn't seem to be specified, so maybe something in the way that's generated?22:11
alaskior is that the 4222:12
melwittI think 42 is the flavorid22:12
alaskiokay22:12
dansmithyeah the 3222:12
dansmith4322:12
* dansmith headdesks22:12
dansmith*4222:12
alaskiI almost spit out my water there22:13
melwittlol22:13
dansmithheh22:13
openstackgerritDan Smith proposed openstack/nova: DNM: Canary to sniff out what postgres' problem is  https://review.openstack.org/29845222:13
*** busterswt has quit IRC22:14
*** lpetrut has quit IRC22:14
*** dimtruck is now known as zz_dimtruck22:15
*** mdrabe has quit IRC22:16
*** Marga_ has quit IRC22:19
*** penick has joined #openstack-nova22:19
*** ducttape_ has quit IRC22:19
*** Marga_ has joined #openstack-nova22:19
*** boden has quit IRC22:20
*** prometheanfire has left #openstack-nova22:20
*** thorst has quit IRC22:21
*** pwnall has quit IRC22:21
*** thorst has joined #openstack-nova22:24
*** Sukhdev has joined #openstack-nova22:25
*** pushkaru has quit IRC22:25
*** zenoway has quit IRC22:28
*** markvoelker has quit IRC22:29
*** tbachman has quit IRC22:30
*** tbachman has joined #openstack-nova22:30
*** zenoway has joined #openstack-nova22:31
*** harlowja has joined #openstack-nova22:32
*** harlowja has quit IRC22:32
*** thorst has quit IRC22:32
openstackgerritAndrew Laski proposed openstack/nova: Populate instance_mappings during boot  https://review.openstack.org/26392522:33
openstackgerritAndrew Laski proposed openstack/nova: Add additional attributes to BuildRequest  https://review.openstack.org/29845522:33
*** harlowja has joined #openstack-nova22:33
*** irenab_ has joined #openstack-nova22:35
*** zenoway has quit IRC22:36
*** doug-fish has quit IRC22:37
*** irenab has quit IRC22:38
*** irenab_ is now known as irenab22:38
*** ccarmack has quit IRC22:40
*** fawadkhaliq has quit IRC22:48
*** fawadkhaliq has joined #openstack-nova22:49
*** sdague has quit IRC22:53
*** pushkaru has joined #openstack-nova22:54
*** harlowja has quit IRC22:54
*** harlowja has joined #openstack-nova22:54
*** tongli has quit IRC22:56
*** david-lyle_ is now known as david-lyle22:57
*** akshai has quit IRC23:00
*** tbachman_ has joined #openstack-nova23:00
*** tbachman has quit IRC23:00
*** tbachman_ is now known as tbachman23:00
*** haomaiwa_ has quit IRC23:01
*** pushkaru has quit IRC23:01
*** haomaiwa_ has joined #openstack-nova23:01
*** efoley_ has joined #openstack-nova23:04
*** efoley has quit IRC23:05
*** mriedem has joined #openstack-nova23:08
*** apoorvad has joined #openstack-nova23:11
*** apoorvad has quit IRC23:13
*** apoorvad has joined #openstack-nova23:13
*** penick has quit IRC23:15
*** ducttape_ has joined #openstack-nova23:17
*** thorst has joined #openstack-nova23:20
*** salv-orlando has quit IRC23:21
*** ametts has quit IRC23:22
*** busterswt has joined #openstack-nova23:25
*** harlowja has quit IRC23:25
*** harlowja has joined #openstack-nova23:25
*** vilobhmm11 has quit IRC23:26
*** jwcroppe_ has joined #openstack-nova23:27
*** rk4n has quit IRC23:28
*** busterswt has quit IRC23:29
*** jwcroppe has quit IRC23:30
*** shakamunyi has joined #openstack-nova23:30
*** mylu has joined #openstack-nova23:31
*** barra204 has quit IRC23:32
*** zenoway has joined #openstack-nova23:34
*** zenoway has quit IRC23:39
*** achanda has quit IRC23:41
*** ducttape_ has quit IRC23:43
*** prometheanfire has joined #openstack-nova23:44
*** knikolla has quit IRC23:44
*** zz_dimtruck is now known as dimtruck23:44
*** ducttape_ has joined #openstack-nova23:46
*** diogogmt has joined #openstack-nova23:48
*** nelsnelson has joined #openstack-nova23:50
*** fawadkhaliq has quit IRC23:50
*** fawadkhaliq has joined #openstack-nova23:50
*** mylu has quit IRC23:51
openstackgerritMarcellin Fom Tchassem proposed openstack/nova: Cleanup the exception LiveMigrationWithOldNovaNotSafe  https://review.openstack.org/29847023:51
*** nelsnels_ has quit IRC23:51
*** davideagnello has quit IRC23:51
*** fawadkhaliq has quit IRC23:52
*** fawadkhaliq has joined #openstack-nova23:52
*** fawadkhaliq has quit IRC23:59
*** fawadkhaliq has joined #openstack-nova23:59
*** tbachman has quit IRC23:59

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