Friday, 2017-01-06

*** baoli has quit IRC00:02
*** jwcroppe has quit IRC00:03
*** tjones has quit IRC00:11
*** yuikotakadamori has joined #openstack-nova00:12
*** stewie925 has joined #openstack-nova00:17
*** tjones has joined #openstack-nova00:21
*** tjones has quit IRC00:24
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec  https://review.openstack.org/39719300:25
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Abort Cold Migration  https://review.openstack.org/33473200:25
openstackgerritTakashi NATSUME proposed openstack/nova-specs: List/show all server migration types  https://review.openstack.org/40723700:25
*** yuikotakadamori has quit IRC00:25
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Fix formatting errors and warnings  https://review.openstack.org/40736400:25
stewie925hello has anyone been able to successfully install a multi-node (controller, compute, network) Openstack on their machine?00:26
*** hongbin has quit IRC00:29
*** chyka has quit IRC00:30
*** roeyc has quit IRC00:31
*** rama_y has quit IRC00:32
*** mriedem has joined #openstack-nova00:36
*** sdague has quit IRC00:36
*** dave-mccowan has joined #openstack-nova00:37
*** liusheng has joined #openstack-nova00:40
*** markvoelker has quit IRC00:41
mriedemgfdi http://logs.openstack.org/06/417206/2/gate/gate-nova-python35-db/61fcf07/console.html#_2017-01-05_23_10_44_73862300:42
*** mtanino has quit IRC00:43
*** thorst_ has joined #openstack-nova00:51
openstackgerritMatt Riedemann proposed openstack/nova: Fix ksa mocking in test_cinderclient_unsupported_v1  https://review.openstack.org/41720600:52
*** ijw has quit IRC00:55
*** thorst_ has quit IRC00:57
*** armax has quit IRC00:59
*** markvoelker has joined #openstack-nova00:59
*** Sukhdev has quit IRC00:59
*** jose-phillips has quit IRC01:00
*** salv-orlando has joined #openstack-nova01:01
*** phuongnh has joined #openstack-nova01:03
mriedemeasy +W https://review.openstack.org/#/c/415776/201:05
*** salv-orlando has quit IRC01:06
*** lamt has quit IRC01:06
*** fyxim has quit IRC01:13
*** nikhil has quit IRC01:13
*** macsz has quit IRC01:13
*** shaner has quit IRC01:13
*** mat128 has quit IRC01:13
*** Dinesh_Bhor has quit IRC01:13
*** larainema has quit IRC01:13
*** jroll has quit IRC01:13
*** agarner has quit IRC01:13
*** robcresswell has quit IRC01:13
*** jamespage has quit IRC01:13
*** NightKhaos has quit IRC01:13
*** Hazelesque has quit IRC01:13
*** jlvillal has quit IRC01:13
*** rnoriega has quit IRC01:13
*** flaper87 has quit IRC01:13
*** mwhahaha has quit IRC01:13
*** Prophidys has quit IRC01:13
*** raorn has quit IRC01:13
*** geekinutah has quit IRC01:13
*** DuncanT has quit IRC01:13
*** sballe_ has quit IRC01:13
*** andreaf has quit IRC01:13
*** greghaynes has quit IRC01:13
*** mirrorbox has quit IRC01:13
*** mkoderer__ has quit IRC01:13
*** Kimmo_ has quit IRC01:13
*** johnthetubaguy has quit IRC01:13
*** raorn_ has joined #openstack-nova01:13
*** rpodolyaka has quit IRC01:13
*** betherly has quit IRC01:13
*** boris-42 has quit IRC01:13
*** sc68cal has quit IRC01:13
*** ejat has quit IRC01:13
*** aloga has quit IRC01:13
*** ediardo has quit IRC01:13
*** timello has quit IRC01:13
*** dgonzalez has quit IRC01:13
*** mlakat has quit IRC01:13
*** StevenK has quit IRC01:13
*** karlamrhein has quit IRC01:13
*** zhenguo has quit IRC01:13
*** jamielennox has quit IRC01:13
*** tobasco has quit IRC01:13
*** NobodyCam has quit IRC01:13
*** mrodden has quit IRC01:13
*** lyarwood has quit IRC01:13
*** scottynomad has quit IRC01:13
*** antonym has quit IRC01:13
*** pumaranikar has quit IRC01:13
*** ariesyous has quit IRC01:13
*** tr3buchet has quit IRC01:13
*** aunnam has quit IRC01:13
*** johnthetubaguy_ has joined #openstack-nova01:14
*** mrodden_ has joined #openstack-nova01:14
*** Hazelesque has joined #openstack-nova01:14
*** Kimmo_ has joined #openstack-nova01:14
*** jlvillal has joined #openstack-nova01:14
*** aleph1 has joined #openstack-nova01:14
*** dgonzalez has joined #openstack-nova01:14
*** lyarwood_ has joined #openstack-nova01:14
*** mirrorbox has joined #openstack-nova01:14
*** greghaynes has joined #openstack-nova01:14
*** tobasco has joined #openstack-nova01:14
*** aloga has joined #openstack-nova01:14
*** Dinesh_Bhor has joined #openstack-nova01:14
*** mat128 has joined #openstack-nova01:14
*** tr3buchet has joined #openstack-nova01:14
*** mlakat has joined #openstack-nova01:14
*** NightKhaos has joined #openstack-nova01:14
*** StevenK has joined #openstack-nova01:14
*** jamespag` has joined #openstack-nova01:14
*** mrodden_ is now known as mrodden01:14
*** mirrorbox has quit IRC01:14
*** mirrorbox has joined #openstack-nova01:14
*** mkoderer___ has joined #openstack-nova01:14
*** johnthetubaguy_ is now known as johnthetubaguy01:14
*** aries has joined #openstack-nova01:14
*** jroll has joined #openstack-nova01:14
*** antonym has joined #openstack-nova01:14
*** antonym has quit IRC01:15
*** antonym has joined #openstack-nova01:15
*** shaner has joined #openstack-nova01:15
*** ejat has joined #openstack-nova01:15
*** jamespag` has quit IRC01:15
*** jamespag` has joined #openstack-nova01:15
*** little has joined #openstack-nova01:15
*** john51 has quit IRC01:15
*** Prophidys has joined #openstack-nova01:16
*** jroll is now known as Guest5451501:16
*** john51 has joined #openstack-nova01:16
*** ejat has quit IRC01:16
*** ejat has joined #openstack-nova01:16
*** scottynomad has joined #openstack-nova01:16
*** sc68cal has joined #openstack-nova01:16
*** andreaf has joined #openstack-nova01:16
*** rnoriega has joined #openstack-nova01:16
openstackgerritDan Peschman proposed openstack/nova: No API cell up-call to delete consoleauth tokens  https://review.openstack.org/41592201:17
*** serverascode has quit IRC01:17
*** emerson has quit IRC01:17
*** jamielennox has joined #openstack-nova01:18
*** robcresswell has joined #openstack-nova01:19
*** test222 has quit IRC01:20
*** Sukhdev has joined #openstack-nova01:20
*** dtp has quit IRC01:20
*** mwhahaha has joined #openstack-nova01:21
*** macsz has joined #openstack-nova01:21
*** hieulq has joined #openstack-nova01:21
*** dave-mccowan has quit IRC01:21
*** tuanluong has joined #openstack-nova01:21
*** larainema has joined #openstack-nova01:22
*** aunnam has joined #openstack-nova01:22
*** timello has joined #openstack-nova01:22
*** pumaranikar has joined #openstack-nova01:22
*** tjones has joined #openstack-nova01:23
*** ediardo has joined #openstack-nova01:24
*** dave-mccowan has joined #openstack-nova01:24
*** thorst_ has joined #openstack-nova01:25
*** boris-42 has joined #openstack-nova01:25
*** geekinutah has joined #openstack-nova01:25
*** thorst_ has quit IRC01:25
*** kevinz has joined #openstack-nova01:26
*** emerson has joined #openstack-nova01:26
*** DuncanT has joined #openstack-nova01:26
*** fyxim has joined #openstack-nova01:26
*** karlamrhein has joined #openstack-nova01:27
*** nikhil has joined #openstack-nova01:27
*** zhenguo has joined #openstack-nova01:31
*** sballe_ has joined #openstack-nova01:32
*** ducttape_ has joined #openstack-nova01:32
*** liusheng has quit IRC01:34
*** liusheng has joined #openstack-nova01:35
*** dulek has quit IRC01:37
*** test222 has joined #openstack-nova01:38
*** rama_y has joined #openstack-nova01:39
*** test222 has joined #openstack-nova01:40
*** NobodyCam has joined #openstack-nova01:40
*** sreeram_v has joined #openstack-nova01:41
*** dulek has joined #openstack-nova01:43
*** browne has quit IRC01:43
*** serverascode has joined #openstack-nova01:43
*** rama_y has quit IRC01:43
*** unicell1 has quit IRC01:43
*** unicell has joined #openstack-nova01:43
*** rpodolyaka has joined #openstack-nova01:45
*** betherly has joined #openstack-nova01:45
*** sreeram_v has quit IRC01:45
*** unicell has quit IRC01:48
*** unicell has joined #openstack-nova01:48
*** yuntongjin has joined #openstack-nova01:48
*** mtanino has joined #openstack-nova01:49
*** Kevin_Zheng has joined #openstack-nova01:50
*** Sukhdev has quit IRC01:53
*** yuntongjin has quit IRC01:56
*** huanxie has joined #openstack-nova02:00
*** salv-orlando has joined #openstack-nova02:03
*** yuntongjin has joined #openstack-nova02:03
*** mtanino has quit IRC02:04
*** stewie925 has quit IRC02:04
*** ljxiash has joined #openstack-nova02:04
openstackgerritint32bit proposed openstack/python-novaclient: Allow multiple tag add/delete from cli  https://review.openstack.org/41463502:06
*** salv-orlando has quit IRC02:07
*** takashin_ has joined #openstack-nova02:07
*** ducttape_ has quit IRC02:08
*** takashin has quit IRC02:08
openstackgerrithuangtianhua proposed openstack/python-novaclient: Make _console() public  https://review.openstack.org/41436402:12
*** liverpooler has joined #openstack-nova02:12
*** thorst_ has joined #openstack-nova02:16
*** thorst_ has quit IRC02:17
*** dave-mccowan has quit IRC02:22
*** catintheroof has joined #openstack-nova02:22
*** gongysh has joined #openstack-nova02:25
*** tbachman has quit IRC02:28
*** ducttape_ has joined #openstack-nova02:28
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Fix "Message object do not support addition".  https://review.openstack.org/41452902:35
*** Guest54515 is now known as jroll02:36
*** jroll has joined #openstack-nova02:36
*** ijw has joined #openstack-nova02:38
*** tbachman has joined #openstack-nova02:42
*** ijw has quit IRC02:43
openstackgerritDan Peschman proposed openstack/nova: No API cell up-call to delete consoleauth tokens  https://review.openstack.org/41592202:44
*** jwcroppe has joined #openstack-nova02:46
*** ducttape_ has quit IRC02:48
*** catintheroof has quit IRC02:50
*** raunak has quit IRC02:51
openstackgerritZhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail  https://review.openstack.org/40857102:59
*** ducttape_ has joined #openstack-nova03:02
*** gcb has quit IRC03:05
openstackgerritMatt Riedemann proposed openstack/python-novaclient: Add support for showing aggregate UUIDs (v2.41)  https://review.openstack.org/41726503:06
*** hongbin has joined #openstack-nova03:08
openstackgerritMatt Riedemann proposed openstack/python-novaclient: Add support for showing aggregate UUIDs (v2.41)  https://review.openstack.org/41726503:11
*** thorst_ has joined #openstack-nova03:12
*** thorst_ has quit IRC03:12
*** raunak has joined #openstack-nova03:29
*** jwcroppe has quit IRC03:30
*** jwcroppe has joined #openstack-nova03:30
*** jwcroppe has quit IRC03:34
*** dimtruck is now known as zz_dimtruck03:35
*** ljxiash has quit IRC03:35
*** zz_dimtruck is now known as dimtruck03:35
*** yamahata has quit IRC03:35
*** ljxiash has joined #openstack-nova03:35
*** yuntongjin has quit IRC03:36
*** ducttape_ has quit IRC03:37
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2)  https://review.openstack.org/40895503:38
*** ljxiash has quit IRC03:40
*** dimtruck is now known as zz_dimtruck03:44
*** tlian has quit IRC03:49
*** amotoki has joined #openstack-nova03:50
*** dikonoor has joined #openstack-nova04:03
*** psachin has joined #openstack-nova04:04
*** tbachman has quit IRC04:07
*** yamahata has joined #openstack-nova04:08
*** gongysh has quit IRC04:09
*** ssurana has joined #openstack-nova04:10
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2)  https://review.openstack.org/40896404:17
*** tbachman has joined #openstack-nova04:17
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Fix "Message object do not support addition".  https://review.openstack.org/41452904:17
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004204:17
openstackgerritTakashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host  https://review.openstack.org/41492604:17
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Microversion 2.41 - Enable cold migration with target host  https://review.openstack.org/40670704:18
*** ljxiash has joined #openstack-nova04:19
*** tiendc has joined #openstack-nova04:22
*** mriedem has quit IRC04:22
*** ljxiash has quit IRC04:24
*** jwcroppe has joined #openstack-nova04:27
*** nicolasbock has joined #openstack-nova04:35
*** links has joined #openstack-nova04:40
*** sreeram_v has joined #openstack-nova04:40
*** mdnadeem has joined #openstack-nova04:40
*** ducttape_ has joined #openstack-nova04:45
*** ljxiash has joined #openstack-nova04:46
*** gongysh has joined #openstack-nova04:52
*** Sukhdev has joined #openstack-nova04:54
*** raunak has quit IRC04:55
*** prateek has joined #openstack-nova04:56
*** unicell has quit IRC04:59
*** hongbin has quit IRC04:59
*** ducttape_ has quit IRC04:59
*** unicell1 has joined #openstack-nova05:02
*** ratailor has joined #openstack-nova05:10
*** yuntongjin has joined #openstack-nova05:11
*** thorst_ has joined #openstack-nova05:13
*** salv-orlando has joined #openstack-nova05:14
*** thorst_ has quit IRC05:18
*** salv-orlando has quit IRC05:19
*** sreeram_v has quit IRC05:21
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Add set/get_traits method to ResourceProvder object  https://review.openstack.org/37620105:24
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Add Traits API to placement service  https://review.openstack.org/37620005:24
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Caches and synchronizes the traits from os_traits  https://review.openstack.org/41600705:24
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Add resource provider traits API  https://review.openstack.org/37620205:24
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects  https://review.openstack.org/37619905:24
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Import os_traits library  https://review.openstack.org/37738105:24
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Add Traits related table to the api database  https://review.openstack.org/37619805:24
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Report cpu features to placement Traits API  https://review.openstack.org/41728805:24
*** ayogi has joined #openstack-nova05:30
*** Sukhdev has quit IRC05:34
*** sudipto has joined #openstack-nova05:35
*** sudipto_ has joined #openstack-nova05:35
*** gcb has joined #openstack-nova05:38
*** nandal has joined #openstack-nova05:44
*** unicell has joined #openstack-nova05:51
*** unicell1 has quit IRC05:54
*** mdnadeem has quit IRC05:58
*** mdnadeem has joined #openstack-nova06:00
*** takashin_ has left #openstack-nova06:00
*** yuntongjin has quit IRC06:02
openstackgerritTao Li proposed openstack/nova: When using ceph storage in nova, the libvirt driver should not cleanup the root disk except booting scenario such as evacuating instance failed. If the root disk is removed, maybe some user datas in it lost.  https://review.openstack.org/41729106:05
*** rama_y has joined #openstack-nova06:13
*** gongysh has quit IRC06:15
*** salv-orlando has joined #openstack-nova06:15
*** salv-orlando has quit IRC06:20
*** yuntongjin has joined #openstack-nova06:20
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Report cpu features to placement Traits API  https://review.openstack.org/41728806:21
openstackgerritAlex Xu proposed openstack/nova: [POC][placement] Add resource provider traits API  https://review.openstack.org/37620206:21
*** phuongnh has quit IRC06:25
*** amotoki has quit IRC06:25
*** rama_y has quit IRC06:26
*** dulek has quit IRC06:26
*** dulek has joined #openstack-nova06:31
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Fix "Message object do not support addition".  https://review.openstack.org/41452906:35
openstackgerritPooja Jadhav proposed openstack/nova: Don't translate exceptions w/ no message  https://review.openstack.org/41729506:37
*** nandal has quit IRC06:37
*** yuntongjin has quit IRC06:42
*** mdnadeem has quit IRC06:42
*** gongysh has joined #openstack-nova06:42
*** gjayavelu has joined #openstack-nova06:43
*** sridharg has joined #openstack-nova06:46
*** diga has joined #openstack-nova06:47
*** moshele has joined #openstack-nova06:52
*** gjayavelu has quit IRC06:52
*** mdnadeem has joined #openstack-nova06:54
*** rcernin has quit IRC06:55
*** ducttape_ has joined #openstack-nova07:00
*** satyar has joined #openstack-nova07:00
*** tiendc has quit IRC07:01
*** ducttape_ has quit IRC07:05
*** lpetrut has joined #openstack-nova07:06
*** moshele has quit IRC07:07
*** tesseract has joined #openstack-nova07:09
*** tjones has quit IRC07:09
*** pcaruana has joined #openstack-nova07:10
*** moshele has joined #openstack-nova07:11
*** rcernin has joined #openstack-nova07:13
*** thorst_ has joined #openstack-nova07:14
*** moshele has quit IRC07:15
*** salv-orlando has joined #openstack-nova07:16
*** tiendc has joined #openstack-nova07:17
*** thorst_ has quit IRC07:19
*** salv-orlando has quit IRC07:21
*** yuntongjin has joined #openstack-nova07:22
*** gouthamr has quit IRC07:24
*** trananhkma has joined #openstack-nova07:28
*** ijw_ has joined #openstack-nova07:33
*** voelzmo has joined #openstack-nova07:34
*** ijw_ has quit IRC07:37
*** xinliang has quit IRC07:40
*** voelzmo has quit IRC07:42
*** jwcroppe has quit IRC07:51
*** jwcroppe has joined #openstack-nova07:51
*** ljxiash has quit IRC07:52
*** ljxiash has joined #openstack-nova07:52
openstackgerritWang Qing wu proposed openstack/nova: WIP(4): Introduce base PowerVM driver (cfg drv)  https://review.openstack.org/40940407:53
*** ljxiash has quit IRC07:53
*** ljxiash has joined #openstack-nova07:53
openstackgerritRadoslav Gerganov proposed openstack/nova: Add connection pool size to vSphere settings  https://review.openstack.org/41408507:55
*** jwcroppe has quit IRC07:56
*** ljxiash has quit IRC07:58
*** sahid has joined #openstack-nova07:58
*** oomichi has quit IRC08:02
*** oomichi has joined #openstack-nova08:03
*** lpetrut has quit IRC08:05
*** slaweq has quit IRC08:06
*** voelzmo has joined #openstack-nova08:11
*** voelzmo has quit IRC08:15
*** abalutoiu has joined #openstack-nova08:16
*** voelzmo has joined #openstack-nova08:17
*** salv-orlando has joined #openstack-nova08:17
*** abalutoiu has quit IRC08:18
*** voelzmo has quit IRC08:20
*** EmilienM has quit IRC08:24
*** sahid has quit IRC08:31
*** Kevin_Zheng has quit IRC08:31
*** rpodolyaka has quit IRC08:31
*** NobodyCam has quit IRC08:31
*** zhenguo has quit IRC08:31
*** ediardo has quit IRC08:31
*** tuanluong has quit IRC08:31
*** hieulq has quit IRC08:31
*** macsz has quit IRC08:31
*** robcresswell has quit IRC08:31
*** greghaynes has quit IRC08:31
*** johnthetubaguy has quit IRC08:31
*** jamespag` has quit IRC08:31
*** markvoelker has quit IRC08:31
*** harlowja has quit IRC08:31
*** pkoniszewski has quit IRC08:31
*** bauzas has quit IRC08:31
*** afazekas has quit IRC08:31
*** haukebruno has quit IRC08:31
*** sheel has quit IRC08:31
*** satyar has quit IRC08:31
*** links has quit IRC08:31
*** karlamrhein has quit IRC08:31
*** fyxim has quit IRC08:31
*** timello has quit IRC08:31
*** jamielennox has quit IRC08:31
*** tr3buchet has quit IRC08:31
*** Dinesh_Bhor has quit IRC08:31
*** mirrorbox has quit IRC08:31
*** Hazelesque has quit IRC08:31
*** artom has quit IRC08:31
*** sapcc-bot has quit IRC08:31
*** efried has quit IRC08:31
*** tlbr has quit IRC08:31
*** tdurakov has quit IRC08:31
*** mtreinish has quit IRC08:31
*** kaisers has quit IRC08:31
*** rdo has quit IRC08:31
*** dosaboy has quit IRC08:31
*** dansmith has quit IRC08:31
*** openstackgerrit has quit IRC08:31
*** adisky_ has quit IRC08:31
*** Oku_OSz has quit IRC08:31
*** oomichi has quit IRC08:31
*** yuntongjin has quit IRC08:31
*** tbachman has quit IRC08:31
*** psachin has quit IRC08:31
*** huanxie has quit IRC08:31
*** serverascode has quit IRC08:31
*** liusheng has quit IRC08:31
*** pumaranikar has quit IRC08:31
*** larainema has quit IRC08:31
*** mwhahaha has quit IRC08:31
*** john51 has quit IRC08:31
*** jroll has quit IRC08:31
*** jlvillal has quit IRC08:31
*** raorn_ has quit IRC08:31
*** esmiurium has quit IRC08:31
*** sileht has quit IRC08:31
*** kaisers1 has quit IRC08:31
*** snikitin has quit IRC08:31
*** masayukig has quit IRC08:31
*** doffm has quit IRC08:31
*** belliott has quit IRC08:31
*** dillaman has quit IRC08:31
*** karthiks has quit IRC08:31
*** nkrinner_afk has quit IRC08:31
*** _Fdaisuke_ has quit IRC08:31
*** salv-orlando has quit IRC08:31
*** trananhkma has quit IRC08:31
*** dikonoor has quit IRC08:31
*** nikhil has quit IRC08:31
*** DuncanT has quit IRC08:31
*** geekinutah has quit IRC08:31
*** emerson has quit IRC08:31
*** Prophidys has quit IRC08:31
*** ejat has quit IRC08:31
*** NightKhaos has quit IRC08:31
*** StevenK has quit IRC08:31
*** tobasco has quit IRC08:31
*** eandersson has quit IRC08:31
*** Jeffrey4l_ has quit IRC08:31
*** bkopilov has quit IRC08:31
*** yhvh has quit IRC08:31
*** liangy has quit IRC08:31
*** aslezil has quit IRC08:31
*** raj_singh has quit IRC08:31
*** dharinic has quit IRC08:31
*** slunkad has quit IRC08:31
*** mrda has quit IRC08:31
*** ttx has quit IRC08:31
*** mariusv has quit IRC08:31
*** gibi has quit IRC08:31
*** Nakato has quit IRC08:31
*** nklenke has quit IRC08:31
*** Cristina_ has quit IRC08:31
*** TheJulia has quit IRC08:31
*** csatari has quit IRC08:31
*** vdrok has quit IRC08:31
*** gus has quit IRC08:31
*** mrhillsman has quit IRC08:31
*** mariojv has quit IRC08:31
*** lennyb has quit IRC08:31
*** Gorian has quit IRC08:31
*** fungi has quit IRC08:31
*** igordcard has quit IRC08:31
*** zz_dimtruck has quit IRC08:31
*** FL1SK has quit IRC08:31
*** test222 has quit IRC08:31
*** sballe_ has quit IRC08:31
*** boris-42 has quit IRC08:31
*** tyrola has quit IRC08:31
*** sulo has quit IRC08:31
*** hughsaunders has quit IRC08:31
*** Vek has quit IRC08:31
*** tomhambleton has quit IRC08:31
*** markmcclain has quit IRC08:31
*** BlackDex has quit IRC08:31
*** rcernin has quit IRC08:31
*** kevinz has quit IRC08:31
*** antonym has quit IRC08:31
*** jvgrant_ has quit IRC08:31
*** jgrassler has quit IRC08:31
*** federico3 has quit IRC08:31
*** aignatov has quit IRC08:31
*** rwmjones has quit IRC08:31
*** Guest66666 has quit IRC08:31
*** keekz has quit IRC08:31
*** dmellado_ has quit IRC08:31
*** inara has quit IRC08:31
*** ayogi has quit IRC08:31
*** ratailor has quit IRC08:31
*** ssurana has quit IRC08:31
*** yamahata has quit IRC08:31
*** betherly has quit IRC08:31
*** rnoriega has quit IRC08:31
*** scottynomad has quit IRC08:31
*** Kimmo_ has quit IRC08:31
*** mrodden has quit IRC08:31
*** burt has quit IRC08:31
*** htruta has quit IRC08:31
*** zhs_ has quit IRC08:31
*** tonyb has quit IRC08:31
*** coreywright has quit IRC08:31
*** liuyulong has quit IRC08:31
*** paw has quit IRC08:31
*** lifeless has quit IRC08:31
*** jamespd has quit IRC08:31
*** mikal has quit IRC08:31
*** rodrigods has quit IRC08:31
*** bnemec has quit IRC08:31
*** cfriesen has quit IRC08:31
*** zul has quit IRC08:31
*** ltomasbo has quit IRC08:31
*** gmann has quit IRC08:31
*** jschlueter has quit IRC08:31
*** sudipto has quit IRC08:31
*** prateek has quit IRC08:31
*** liverpooler has quit IRC08:31
*** aunnam has quit IRC08:31
*** mkoderer___ has quit IRC08:31
*** eharney has quit IRC08:31
*** d34dh0r53 has quit IRC08:31
*** Anticimex has quit IRC08:31
*** hwoarang has quit IRC08:31
*** zzzeek has quit IRC08:31
*** rha has quit IRC08:31
*** s1rp has quit IRC08:31
*** yarkot has quit IRC08:31
*** jschwarz has quit IRC08:31
*** frickler has quit IRC08:31
*** pabardina has quit IRC08:31
*** alex_xu has quit IRC08:31
*** ildikov has quit IRC08:31
*** pcaruana has quit IRC08:31
*** tesseract has quit IRC08:31
*** unicell has quit IRC08:31
*** nicolasbock has quit IRC08:31
*** andreaf has quit IRC08:31
*** little has quit IRC08:31
*** aries has quit IRC08:31
*** mlakat has quit IRC08:31
*** aleph1 has quit IRC08:31
*** dgonzalez has quit IRC08:31
*** shaohe_feng has quit IRC08:31
*** diana_clarke has quit IRC08:31
*** jaypipes has quit IRC08:31
*** devananda has quit IRC08:31
*** smcginnis has quit IRC08:31
*** kevinbenton has quit IRC08:31
*** jotr has quit IRC08:31
*** Madasi has quit IRC08:31
*** gabor_antal has quit IRC08:31
*** jeblair has quit IRC08:31
*** erhudy has quit IRC08:31
*** kylek3h has quit IRC08:31
*** ajo has quit IRC08:31
*** amrith has quit IRC08:31
*** kashyap has quit IRC08:31
*** clayton has quit IRC08:31
*** adreznec has quit IRC08:31
*** tiendc has quit IRC08:31
*** mdnadeem has quit IRC08:31
*** sridharg has quit IRC08:31
*** gcb has quit IRC08:31
*** sudipto_ has quit IRC08:31
*** sc68cal has quit IRC08:31
*** aloga has quit IRC08:31
*** lyarwood_ has quit IRC08:31
*** mat128 has quit IRC08:31
*** haplo37 has quit IRC08:31
*** jgriffith has quit IRC08:31
*** cburgess has quit IRC08:31
*** sean-k-mooneyAFK has quit IRC08:31
*** haplo37_ has quit IRC08:31
*** tojuvone has quit IRC08:31
*** _gryf has quit IRC08:31
*** david-lyle has quit IRC08:31
*** toure has quit IRC08:31
*** scottda has quit IRC08:31
*** acabot has quit IRC08:31
*** aweeks has quit IRC08:34
*** rmcadams has quit IRC08:34
*** yuntongjin has joined #openstack-nova08:37
*** nherciu has joined #openstack-nova08:37
*** rwmjones has joined #openstack-nova08:37
*** tobasco has joined #openstack-nova08:37
*** gibi has joined #openstack-nova08:37
*** kevinz has joined #openstack-nova08:37
*** Nakato has joined #openstack-nova08:37
*** EmilienM has joined #openstack-nova08:37
*** unicell1 has joined #openstack-nova08:37
*** mvk has joined #openstack-nova08:37
*** lpetrut has joined #openstack-nova08:37
*** oomichi has joined #openstack-nova08:37
*** sahid has joined #openstack-nova08:37
*** trananhkma has joined #openstack-nova08:37
*** tiendc has joined #openstack-nova08:37
*** rcernin has joined #openstack-nova08:37
*** pcaruana has joined #openstack-nova08:37
*** tesseract has joined #openstack-nova08:37
*** satyar has joined #openstack-nova08:37
*** mdnadeem has joined #openstack-nova08:37
*** sridharg has joined #openstack-nova08:37
*** gcb has joined #openstack-nova08:37
*** sudipto_ has joined #openstack-nova08:37
*** sudipto has joined #openstack-nova08:37
*** ayogi has joined #openstack-nova08:37
*** ratailor has joined #openstack-nova08:37
*** prateek has joined #openstack-nova08:37
*** links has joined #openstack-nova08:37
*** nicolasbock has joined #openstack-nova08:37
*** tbachman has joined #openstack-nova08:37
*** ssurana has joined #openstack-nova08:37
*** psachin has joined #openstack-nova08:37
*** liverpooler has joined #openstack-nova08:37
*** huanxie has joined #openstack-nova08:37
*** Kevin_Zheng has joined #openstack-nova08:37
*** betherly has joined #openstack-nova08:37
*** rpodolyaka has joined #openstack-nova08:37
*** serverascode has joined #openstack-nova08:37
*** NobodyCam has joined #openstack-nova08:37
*** liusheng has joined #openstack-nova08:37
*** sballe_ has joined #openstack-nova08:37
*** zhenguo has joined #openstack-nova08:37
*** nikhil has joined #openstack-nova08:37
*** karlamrhein has joined #openstack-nova08:37
*** fyxim has joined #openstack-nova08:37
*** geekinutah has joined #openstack-nova08:37
*** boris-42 has joined #openstack-nova08:37
*** ediardo has joined #openstack-nova08:37
*** pumaranikar has joined #openstack-nova08:37
*** timello has joined #openstack-nova08:37
*** aunnam has joined #openstack-nova08:37
*** larainema has joined #openstack-nova08:37
*** tuanluong has joined #openstack-nova08:37
*** hieulq has joined #openstack-nova08:37
*** macsz has joined #openstack-nova08:37
*** mwhahaha has joined #openstack-nova08:37
*** robcresswell has joined #openstack-nova08:37
*** jamielennox has joined #openstack-nova08:37
*** rnoriega has joined #openstack-nova08:37
*** andreaf has joined #openstack-nova08:37
*** sc68cal has joined #openstack-nova08:37
*** scottynomad has joined #openstack-nova08:37
*** john51 has joined #openstack-nova08:37
*** Prophidys has joined #openstack-nova08:37
*** little has joined #openstack-nova08:37
*** ejat has joined #openstack-nova08:37
*** antonym has joined #openstack-nova08:37
*** jroll has joined #openstack-nova08:37
*** aries has joined #openstack-nova08:37
*** mkoderer___ has joined #openstack-nova08:37
*** jamespag` has joined #openstack-nova08:37
*** StevenK has joined #openstack-nova08:37
*** NightKhaos has joined #openstack-nova08:37
*** mlakat has joined #openstack-nova08:37
*** tr3buchet has joined #openstack-nova08:37
*** mat128 has joined #openstack-nova08:37
*** Dinesh_Bhor has joined #openstack-nova08:37
*** aloga has joined #openstack-nova08:37
*** greghaynes has joined #openstack-nova08:37
*** mirrorbox has joined #openstack-nova08:37
*** lyarwood_ has joined #openstack-nova08:37
*** dgonzalez has joined #openstack-nova08:37
*** aleph1 has joined #openstack-nova08:37
*** jlvillal has joined #openstack-nova08:37
*** Kimmo_ has joined #openstack-nova08:37
*** Hazelesque has joined #openstack-nova08:37
*** mrodden has joined #openstack-nova08:37
*** johnthetubaguy has joined #openstack-nova08:37
*** raorn_ has joined #openstack-nova08:37
*** markvoelker has joined #openstack-nova08:37
*** eandersson has joined #openstack-nova08:37
*** eharney has joined #openstack-nova08:37
*** burt has joined #openstack-nova08:37
*** harlowja has joined #openstack-nova08:37
*** shaohe_feng has joined #openstack-nova08:37
*** diana_clarke has joined #openstack-nova08:37
*** artom has joined #openstack-nova08:37
*** jvgrant_ has joined #openstack-nova08:37
*** jaypipes has joined #openstack-nova08:37
*** d34dh0r53 has joined #openstack-nova08:37
*** sapcc-bot has joined #openstack-nova08:37
*** efried has joined #openstack-nova08:37
*** pkoniszewski has joined #openstack-nova08:37
*** Anticimex has joined #openstack-nova08:37
*** hwoarang has joined #openstack-nova08:37
*** haplo37 has joined #openstack-nova08:37
*** tlbr has joined #openstack-nova08:37
*** tdurakov has joined #openstack-nova08:37
*** sileht has joined #openstack-nova08:37
*** esmiurium has joined #openstack-nova08:37
*** mtreinish has joined #openstack-nova08:37
*** dansmith has joined #openstack-nova08:37
*** devananda has joined #openstack-nova08:37
*** kaisers has joined #openstack-nova08:37
*** bauzas has joined #openstack-nova08:37
*** smcginnis has joined #openstack-nova08:37
*** rdo has joined #openstack-nova08:37
*** dosaboy has joined #openstack-nova08:37
*** haukebruno has joined #openstack-nova08:37
*** afazekas has joined #openstack-nova08:37
*** htruta has joined #openstack-nova08:37
*** jgrassler has joined #openstack-nova08:37
*** zhs_ has joined #openstack-nova08:37
*** sheel has joined #openstack-nova08:37
*** openstackgerrit has joined #openstack-nova08:37
*** zzzeek has joined #openstack-nova08:37
*** rha has joined #openstack-nova08:37
*** adisky_ has joined #openstack-nova08:37
*** tyrola has joined #openstack-nova08:37
*** sulo has joined #openstack-nova08:37
*** kaisers1 has joined #openstack-nova08:37
*** yhvh has joined #openstack-nova08:37
*** jgriffith has joined #openstack-nova08:37
*** Oku_OSz has joined #openstack-nova08:37
*** snikitin has joined #openstack-nova08:37
*** tonyb has joined #openstack-nova08:37
*** s1rp has joined #openstack-nova08:37
*** coreywright has joined #openstack-nova08:37
*** liuyulong has joined #openstack-nova08:37
*** masayukig has joined #openstack-nova08:37
*** paw has joined #openstack-nova08:37
*** liangy has joined #openstack-nova08:37
*** aslezil has joined #openstack-nova08:37
*** raj_singh has joined #openstack-nova08:37
*** dharinic has joined #openstack-nova08:37
*** yarkot has joined #openstack-nova08:37
*** hughsaunders has joined #openstack-nova08:37
*** kevinbenton has joined #openstack-nova08:37
*** jotr has joined #openstack-nova08:37
*** lifeless has joined #openstack-nova08:37
*** Madasi has joined #openstack-nova08:37
*** cburgess has joined #openstack-nova08:37
*** slunkad has joined #openstack-nova08:37
*** mrda has joined #openstack-nova08:37
*** jamespd has joined #openstack-nova08:37
*** jschwarz has joined #openstack-nova08:37
*** mariusv has joined #openstack-nova08:37
*** rodrigods has joined #openstack-nova08:37
*** mikal has joined #openstack-nova08:37
*** Vek has joined #openstack-nova08:37
*** belliott has joined #openstack-nova08:37
*** doffm has joined #openstack-nova08:37
*** bnemec has joined #openstack-nova08:37
*** sean-k-mooneyAFK has joined #openstack-nova08:37
*** haplo37_ has joined #openstack-nova08:37
*** dillaman has joined #openstack-nova08:37
*** Guest66666 has joined #openstack-nova08:37
*** frickler has joined #openstack-nova08:37
*** cfriesen has joined #openstack-nova08:37
*** karthiks has joined #openstack-nova08:37
*** zul has joined #openstack-nova08:37
*** nklenke has joined #openstack-nova08:37
*** Cristina_ has joined #openstack-nova08:37
*** gabor_antal has joined #openstack-nova08:37
*** nkrinner_afk has joined #openstack-nova08:37
*** tojuvone has joined #openstack-nova08:37
*** _Fdaisuke_ has joined #openstack-nova08:37
*** TheJulia has joined #openstack-nova08:37
*** tomhambleton has joined #openstack-nova08:37
*** csatari has joined #openstack-nova08:37
*** vdrok has joined #openstack-nova08:37
*** pabardina has joined #openstack-nova08:37
*** fungi has joined #openstack-nova08:37
*** mariojv has joined #openstack-nova08:37
*** markmcclain has joined #openstack-nova08:37
*** gus has joined #openstack-nova08:37
*** mrhillsman has joined #openstack-nova08:37
*** _gryf has joined #openstack-nova08:37
*** lennyb has joined #openstack-nova08:37
*** zz_dimtruck has joined #openstack-nova08:37
*** BlackDex has joined #openstack-nova08:37
*** jeblair has joined #openstack-nova08:37
*** Gorian has joined #openstack-nova08:37
*** keekz has joined #openstack-nova08:37
*** david-lyle has joined #openstack-nova08:37
*** ltomasbo has joined #openstack-nova08:37
*** alex_xu has joined #openstack-nova08:37
*** toure has joined #openstack-nova08:37
*** erhudy has joined #openstack-nova08:37
*** kylek3h has joined #openstack-nova08:37
*** ajo has joined #openstack-nova08:37
*** scottda has joined #openstack-nova08:37
*** ildikov has joined #openstack-nova08:37
*** inara has joined #openstack-nova08:37
*** amrith has joined #openstack-nova08:37
*** gmann has joined #openstack-nova08:37
*** jschlueter has joined #openstack-nova08:37
*** kashyap has joined #openstack-nova08:37
*** acabot has joined #openstack-nova08:37
*** adreznec has joined #openstack-nova08:37
*** clayton has joined #openstack-nova08:37
*** aignatov has joined #openstack-nova08:38
*** yamahata has joined #openstack-nova08:38
*** bkopilov has joined #openstack-nova08:38
*** Jeffrey4l_ has joined #openstack-nova08:38
*** ttx has joined #openstack-nova08:38
*** yuntongjin has quit IRC08:38
*** yuntongjin has joined #openstack-nova08:38
*** igordcard has joined #openstack-nova08:38
*** EmilienM has quit IRC08:39
*** EmilienM has joined #openstack-nova08:39
*** dmellado has joined #openstack-nova08:39
*** betherly has quit IRC08:39
*** federico3 has joined #openstack-nova08:39
*** tdurakov has quit IRC08:40
*** TheJulia has quit IRC08:40
*** csatari has quit IRC08:40
*** vdrok has quit IRC08:40
*** ediardo has quit IRC08:40
*** ildikov has quit IRC08:40
*** ameade has quit IRC08:40
*** thorst_ has joined #openstack-nova08:40
*** mwhahaha has quit IRC08:40
*** fyxim has quit IRC08:40
*** andrewbogott has quit IRC08:40
*** cargonza has quit IRC08:40
*** karlamrhein has quit IRC08:40
*** serverascode has quit IRC08:41
*** nikhil has quit IRC08:41
*** lpetrut has quit IRC08:43
*** thorst_ has quit IRC08:45
*** trananhkma has quit IRC08:45
*** DuncanT has joined #openstack-nova08:45
*** aweeks has joined #openstack-nova08:45
*** csatari has joined #openstack-nova08:48
*** ediardo has joined #openstack-nova08:48
*** ildikov has joined #openstack-nova08:48
*** fyxim has joined #openstack-nova08:49
*** yuntongjin has quit IRC08:50
*** mwhahaha has joined #openstack-nova08:50
*** cargonza has joined #openstack-nova08:51
*** rmcadams has joined #openstack-nova08:52
*** andrewbogott has joined #openstack-nova08:52
*** nikhil has joined #openstack-nova08:56
*** ist has joined #openstack-nova08:57
*** karlamrhein has joined #openstack-nova08:57
*** TheJulia has joined #openstack-nova08:58
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:00
*** tdurakov has joined #openstack-nova09:00
*** test222 has joined #openstack-nova09:01
*** ducttape_ has joined #openstack-nova09:01
*** thorst_ has joined #openstack-nova09:01
*** serverascode has joined #openstack-nova09:02
*** zhs_ has quit IRC09:04
*** xinliang has joined #openstack-nova09:05
*** ducttape_ has quit IRC09:06
*** thorst_ has quit IRC09:06
*** armax has joined #openstack-nova09:06
*** ameade has joined #openstack-nova09:12
*** ssurana has quit IRC09:14
*** vdrok has joined #openstack-nova09:15
*** betherly has joined #openstack-nova09:15
*** salv-orlando has joined #openstack-nova09:18
*** liuyulong is now known as yulong|away09:20
*** lucas-afk is now known as lucasagomes09:22
*** salv-orlando has quit IRC09:23
*** romainh has joined #openstack-nova09:25
*** derekh has joined #openstack-nova09:26
*** mvk has quit IRC09:33
*** yassine has joined #openstack-nova09:39
*** yassine is now known as Guest7215409:40
*** cdent has joined #openstack-nova10:02
*** little has quit IRC10:03
*** slaweq_ has joined #openstack-nova10:05
*** rmart04 has joined #openstack-nova10:10
*** kevinz has quit IRC10:13
*** amotoki has joined #openstack-nova10:15
*** slaweq_ has quit IRC10:16
*** satyar has quit IRC10:17
*** slaweq_ has joined #openstack-nova10:18
*** salv-orlando has joined #openstack-nova10:19
*** mvk has joined #openstack-nova10:20
*** dikonoor has joined #openstack-nova10:21
*** FL1SK has joined #openstack-nova10:22
*** salv-orlando has quit IRC10:23
*** slaweq_ has quit IRC10:26
*** tuanluong has quit IRC10:26
*** slaweq_ has joined #openstack-nova10:27
*** yamahata has quit IRC10:28
openstackgerritGábor Antal proposed openstack/nova: Transform instance.reboot notifications  https://review.openstack.org/38295910:28
*** sudipto_ has quit IRC10:36
*** sudipto has quit IRC10:36
openstackgerritGábor Antal proposed openstack/nova: Transform instance.reboot.error notification  https://review.openstack.org/41179110:36
bauzascdent: morning10:37
cdentmorning bauzas10:37
bauzascdent: saw your comment on https://review.openstack.org/#/c/392569/22/nova/api/openstack/placement/handlers/resource_provider.py@6110:37
bauzascdent: if I'm not adding member_of, we wouldn't accept it by my change, right?10:38
bauzascdent: but we're already accepting it, right?10:38
*** jamespag` is now known as jamespage10:39
cdentwe're accepting it by accident and when we do it will result in an error, and the intention is to merge the api code that handles it asap: https://review.openstack.org/#/c/407742/10:39
* cdent locates some irc log10:40
*** purplerbot has quit IRC10:41
bauzascdent: okay, so I'll remove it from the schema then10:42
*** purplerbot has joined #openstack-nova10:42
cdentstuff near hear in the logs: http://p.anticdent.org/48rX10:42
cdentsigh: here10:43
bauzascdent: okay, what I understand is that we contracted on member_of but without having the right API10:43
bauzascdent: so, stopping to accept it is good to me10:43
bauzascdent: when you'll add the API version for it, it'll be clear that now we explicitly accept member_of by the schema you'll add10:44
cdentI think what we're doing is simply fixing a bug10:44
cdentyes10:44
bauzascdent: okay, putting a new PS10:44
*** shaner has quit IRC10:50
*** shaner has joined #openstack-nova10:51
*** raunak has joined #openstack-nova10:54
*** amotoki has quit IRC10:57
*** raunak_ has joined #openstack-nova10:58
*** raunak has quit IRC10:59
*** raunak_ is now known as raunak10:59
*** slaweq_ has quit IRC10:59
*** nherciu_ has joined #openstack-nova11:00
*** huanxie has quit IRC11:00
*** nherciu__ has joined #openstack-nova11:01
*** ducttape_ has joined #openstack-nova11:02
*** thorst_ has joined #openstack-nova11:02
*** nherciu has quit IRC11:03
bauzascdent: mmm, trying to find a good pattern for the decorator using something related to the PlacementClient now that we agreed that it's just a member of SchedulerClient and not an inherited11:04
*** nherciu_ has quit IRC11:04
bauzascdent: the disabled field that the decorator looks at means that we should review it like self.placement_client._disabled but I don't like it11:04
*** ljxiash has joined #openstack-nova11:06
*** ducttape_ has quit IRC11:06
*** thorst_ has quit IRC11:07
*** ljxiash has quit IRC11:10
*** abalutoiu has joined #openstack-nova11:10
cdentbauzas: that suggests either some functionlity needs to move from scheduler client to placement client or vice versa?11:12
bauzascdent: I think I have an idea11:13
cdentIt seems like only doing the http methods in the placement client is perhaps a bit limited11:13
*** Jeffrey4l_ has quit IRC11:14
*** Jeffrey4l_ has joined #openstack-nova11:14
*** rfolco has joined #openstack-nova11:15
openstackgerritsahid proposed openstack/nova: numa: add numa constraints for emulator threads policy  https://review.openstack.org/38502311:18
openstackgerritsahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested  https://review.openstack.org/38536411:18
openstackgerritsahid proposed openstack/nova: objects: add new field cpus_reserved in NUMACell  https://review.openstack.org/40087411:18
openstackgerritsahid proposed openstack/nova: numa: fit instance toplogy with cpus reserved onto cores  https://review.openstack.org/40087511:18
openstackgerritsahid proposed openstack/nova: virt: update overhead to take into account vCPUs  https://review.openstack.org/38502411:18
openstackgerritsahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU  https://review.openstack.org/40087811:18
openstackgerritsahid proposed openstack/nova: numa: take into account cpus reserved  https://review.openstack.org/40087611:18
openstackgerritsahid proposed openstack/nova: numa: update numa usage incording the cpu reserved  https://review.openstack.org/40087711:18
*** salv-orlando has joined #openstack-nova11:20
*** amotoki has joined #openstack-nova11:20
*** thorst_ has joined #openstack-nova11:23
*** romainh has quit IRC11:24
*** salv-orlando has quit IRC11:24
*** tiendc has quit IRC11:27
*** thorst_ has quit IRC11:27
*** gongysh has quit IRC11:29
*** maestropandy has joined #openstack-nova11:30
*** gongysh has joined #openstack-nova11:30
*** maestropandy has left #openstack-nova11:31
mdboothI have a python3 problem: I have some code which maps over a list of tuples. In python2 I do:11:33
mdboothmap(lambda (x, y): x, mylist)11:33
mdboothi.e. the output is a new list containing only the first element of each tuple11:33
mdboothIn python3 this is a syntax error11:34
mdboothI've mashed the keyboard a bit, but I can't find a solution, and I haven't yet worked out what to google11:35
* mdbooth finds that writing code which simultaneously parses in 2 different languages is a fun and useful exercise.11:36
*** gongysh has quit IRC11:36
*** prateek is now known as prateek|afk11:36
*** smatzek has joined #openstack-nova11:41
cdentmdbooth: will a list comp work for you: [a for a, b in x]11:42
mdboothYeah, list comp is the way to go, thanks cdent11:42
* mdbooth wonders if list comp returns a list or an iterator?11:42
mdboothThe original original code actually constructed an iterator, not a list11:43
cdentyou can do a generator comprehension instead if you want11:43
bauzasyeah that's the problem I'd see with a list comprehension11:46
*** ayogi has quit IRC11:46
*** amotoki has quit IRC11:50
bauzasmdbooth: so, map in py2 returns a list11:51
bauzashttps://docs.python.org/2/library/functions.html#map11:51
mdboothbauzas: original original code was itertools.imap11:51
bauzasmdbooth: while in py3, it's an iterable https://docs.python.org/3/library/functions.html#map11:51
bauzaslike most of the changes for the list stuff11:51
openstackgerritsahid proposed openstack/nova: libvirt: update the active migrations DS to support migration result  https://review.openstack.org/40926011:51
openstackgerritsahid proposed openstack/nova: libvirt: ensure live-migration success  https://review.openstack.org/40926111:51
*** amotoki has joined #openstack-nova11:52
mdboothwhich I'd already simplified to share11:52
bauzasmdbooth: oh okaty11:52
mdboothAs it's also not in py311:52
bauzasbecause it's in stdlib :)11:52
mdboothAlso, because we hate users11:52
*** hieulq_ has joined #openstack-nova11:54
* mdbooth wonders why 'define itertools.imap == map with a deprecation notice' was rejected11:54
bauzasmdbooth: six has a 'map' rename11:54
*** hieulq_ has quit IRC11:54
mdboothbauzas: Yeah, had already found that. Concentrating on the renaming and syntax is probably why I didn't think of using a comprehension instead :)11:55
*** lpetrut has joined #openstack-nova11:55
*** raunak has quit IRC11:55
*** tbachman has quit IRC11:57
bauzasmdbooth: an alternative could be using zip11:58
bauzaslemme test11:58
mdboothbauzas: comprehension is the right solution. It's actually much neater that what I'd originally written.11:59
bauzasmdbooth: but you're no longer creating an iterator, right?N12:00
mdboothThere's also a generator comprehension12:00
mdbooth(x for (x, y) in foo)12:01
bauzasagreed12:01
*** raunak has joined #openstack-nova12:03
*** dave-mccowan has joined #openstack-nova12:03
bauzascdent: given the effort it requires to properly seam all the client stuff within a proper module (given how we intrisected the use in the report client), do you agree with me leaving some kind of ugly but necessary inheritance aspect for a PlacementClient?12:03
bauzascdent: ideally, we would have different objects per resource12:03
bauzaslike we do for most of our clientcode12:04
cdentbauzas: I think if the base class is just put(), post(), etc, then I'm not sure I would bother?12:04
cdentBut if it makes sense, then sure, if it makes things more clear or clean.12:05
*** dulek has quit IRC12:05
bauzascdent: the problem is I'm beginning to include all the resource-related methods for inventories and allocations12:05
cdentI think we intentionally chose _not_ to use different objects per resource in the client as that was felt to be overkill given the types of interactions being done12:05
*** edmondsw has joined #openstack-nova12:05
bauzascdent: and that's an hard work to do so12:05
*** edmondsw has quit IRC12:06
*** edmondsw has joined #openstack-nova12:06
*** amotoki has quit IRC12:06
*** hieulq_ has joined #openstack-nova12:08
*** jamielennox is now known as jamielennox|away12:08
*** moshele has joined #openstack-nova12:10
*** hieulq_ has quit IRC12:11
*** cdelatte has joined #openstack-nova12:12
*** dulek has joined #openstack-nova12:12
*** amotoki has joined #openstack-nova12:13
*** sdague_ has joined #openstack-nova12:14
*** lucasagomes is now known as lucas-hungry12:14
*** hieulq_ has joined #openstack-nova12:14
mdboothIs anybody working on the gate snafu, btw?12:15
mdboothnova.tests.unit.test_cinder.CinderV1TestCase.test_cinderclient_unsupported_v112:16
mdboothcdent: You currently have a zuul job failing in ^^^ btw12:17
cdentmdbooth: that was talked about last night12:17
* cdent locats12:17
cdenthttps://bugs.launchpad.net/nova/+bug/165439312:18
openstackLaunchpad bug 1654393 in OpenStack Compute (nova) ""keystoneauth1.exceptions.catalog.EmptyCatalog: The service catalog is empty." in test_cinderclient_unsupported_v1" [High,In progress] - Assigned to Matt Riedemann (mriedem)12:18
*** bvanhav has joined #openstack-nova12:18
cdentmatt's got a fix pending12:18
mdboothcdent: Impact is nothing can land, right?12:18
cdentI was under the impression it was racy12:18
johnthetubaguysfinucan: I wasn't sure about this one: https://review.openstack.org/#/c/413658/2 does my comment make sense?12:19
* sfinucan looks12:20
*** salv-orlando has joined #openstack-nova12:21
sfinucanjohnthetubaguy: Yup, `null` would be the JSON variant12:21
*** hieulq_ has quit IRC12:22
*** sean-k-mooneyAFK is now known as sean-k-mooney12:23
sfinucanjohnthetubaguy: In turn, any idea if the comments on this patch correct? I've no idea, I'm afraid :/ https://review.openstack.org/#/c/416959/1/nova/conf/quota.py12:23
* sfinucan assumes sdague_ is offline12:24
*** rmart04 has quit IRC12:24
johnthetubaguysfinucan: oh, that is correct I think12:25
johnthetubaguysfinucan: all the network ones, there is a bit in the code that would confirm that, one sec12:25
*** salv-orlando has quit IRC12:25
sdague_I'm back now, what's up?12:26
sfinucanjohnthetubaguy: Yeah, I've found in rather touch to delineate between nova-net and neutron in 'nova/network' where most of these variables are used12:26
*** sdague_ is now known as sdague12:26
*** hieulq_ has joined #openstack-nova12:26
sfinucansdague: Any idea about the the validity of comments on https://review.openstack.org/#/c/416959/1/nova/conf/quota.py ?12:27
*** hieulq_ has quit IRC12:28
*** rtheis has joined #openstack-nova12:28
*** amotoki has quit IRC12:28
sdaguesfinucan: I'm pretty sure Takashi is right, those also only have an effect when nova-network is in play12:29
sfinucansdague: johnthetubaguy: Cool cool. I'll respin that to drop those so12:31
*** catintheroof has joined #openstack-nova12:32
*** catintheroof has quit IRC12:32
*** catintheroof has joined #openstack-nova12:32
*** ducttape_ has joined #openstack-nova12:33
*** amotoki has joined #openstack-nova12:35
*** mdnadeem has quit IRC12:36
*** catintheroof has quit IRC12:36
*** catintheroof has joined #openstack-nova12:37
*** ducttape_ has quit IRC12:38
*** hieulq_ has joined #openstack-nova12:39
*** hieulq_ has quit IRC12:39
*** thorst_ has joined #openstack-nova12:42
*** moshele has quit IRC12:42
*** diga has quit IRC12:53
*** gcb has quit IRC12:58
*** JoseMello has joined #openstack-nova13:00
*** tbachman has joined #openstack-nova13:01
*** amotoki has quit IRC13:03
*** maestropandy has joined #openstack-nova13:04
*** maestropandy has left #openstack-nova13:04
*** baoli has joined #openstack-nova13:05
*** amotoki has joined #openstack-nova13:05
*** baoli has quit IRC13:06
*** baoli has joined #openstack-nova13:06
*** tblakes has joined #openstack-nova13:07
*** ratailor has quit IRC13:08
*** maestropandy1 has joined #openstack-nova13:10
*** maestropandy1 has left #openstack-nova13:10
*** hongbin has joined #openstack-nova13:10
*** hongbin has quit IRC13:11
*** amotoki has quit IRC13:12
*** maestropandy2 has joined #openstack-nova13:13
*** maestropandy2 has left #openstack-nova13:13
jaypipesgood morning Novas.13:14
*** amotoki has joined #openstack-nova13:15
*** lucas-hungry is now known as lucasagomes13:15
*** rmart04 has joined #openstack-nova13:16
*** amotoki has quit IRC13:16
*** rmart04 has quit IRC13:18
*** rmart04 has joined #openstack-nova13:21
*** rmart04 has left #openstack-nova13:21
*** salv-orlando has joined #openstack-nova13:22
*** vladikr has joined #openstack-nova13:23
*** rmart04 has joined #openstack-nova13:24
*** rmart04 has quit IRC13:26
*** salv-orlando has quit IRC13:26
*** maestropandy has joined #openstack-nova13:27
*** mdrabe has joined #openstack-nova13:31
*** maestropandy has quit IRC13:32
*** kevinz has joined #openstack-nova13:33
*** nklenke has quit IRC13:34
*** prateek|afk has quit IRC13:35
*** edleafe is now known as figleaf13:36
openstackgerritStephen Finucane proposed openstack/nova: libvirt: Make 'get_domain' private  https://review.openstack.org/41737813:36
*** sudipto has joined #openstack-nova13:38
*** sudipto_ has joined #openstack-nova13:38
*** ayogi has joined #openstack-nova13:40
*** baoli has quit IRC13:41
*** baoli has joined #openstack-nova13:42
*** ducttape_ has joined #openstack-nova13:47
*** tlian has joined #openstack-nova13:49
*** kevinz has quit IRC13:51
*** jheroux has joined #openstack-nova13:51
*** emerson has joined #openstack-nova13:52
*** sfinucan has left #openstack-nova13:52
*** salv-orlando has joined #openstack-nova13:53
*** amotoki has joined #openstack-nova13:54
*** sean-k-mooney has quit IRC13:54
*** sean-k-mooney has joined #openstack-nova13:54
openstackgerritEd Leafe proposed openstack/nova: Remove the Allocation.create() method.  https://review.openstack.org/41675113:54
openstackgerritEd Leafe proposed openstack/nova: Add check for invalid allocation amounts  https://review.openstack.org/41675213:54
*** romainh has joined #openstack-nova13:54
openstackgerritStephen Finucane proposed openstack/nova: rbd: Remove unnecessary 'encode' calls  https://review.openstack.org/41235613:55
*** mtanino has joined #openstack-nova13:56
*** raunak has quit IRC13:56
*** xinliang has quit IRC13:57
*** romainh1 has joined #openstack-nova13:58
*** raunak has joined #openstack-nova13:58
*** romainh1 has quit IRC13:59
*** romainh has quit IRC13:59
*** romainh has joined #openstack-nova13:59
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate yet more nova-net options  https://review.openstack.org/41695913:59
*** salv-orlando has quit IRC14:00
*** READ10 has joined #openstack-nova14:01
*** bvanhav_ has joined #openstack-nova14:04
*** bvanhav has quit IRC14:05
openstackgerritmelissaml proposed openstack/nova: Fix a typo in documents  https://review.openstack.org/41738914:10
*** xinliang has joined #openstack-nova14:10
*** amotoki has quit IRC14:10
*** lamt has joined #openstack-nova14:13
cdentmdbooth: maybe you have some thoughts on this: Do you know to what extent eventlet gets involved when running the unit tests?14:14
mdboothcdent: I think sdague would be a better authority on this stuff, but to the best of my knowledge: not really.14:15
mdboothI understand that tox fires up several workers in independent processes, which then run tests serially.14:15
cdent(I was asking in relation to your comment on matt's fix for the gate failure)14:15
mdboothSo 'races' shouldn't really occur.14:15
*** ist has quit IRC14:16
mdboothHowever, you can get ordering artifacts if tests don't clean up after themselves properly.14:16
cdentyeah, that was my thinking too, but if eventlet does getinvolved, then game over14:16
cdents/over/confused/14:16
mdboothHowever, I'd defer entirely to sdague :)14:16
sdaguecdent: what is the eventlet concern when tests are run?14:16
cdentSo I've been assuming it doesn't (as that would be bad as it introduces ambiguity).14:17
*** slaweq_ has joined #openstack-nova14:17
mdboothIncidentally, I'm just starting to look at this myself.14:17
cdentsdague: If eventlet is present and monkey patching file io in some sense, you might get an unexpected yield to an unexpected greenthread?14:17
cdentI don't really know, I'm experiening curiosity. Expressing it to see if anyone else knows what's up.14:18
sdaguecdent: eventlet is definitely present14:18
*** jamesdenton has joined #openstack-nova14:18
sdaguebut, there are going to need to be spawns to have greenlets running, right? We use that to our advantage on the functional tests with the single process cloud14:20
cdentThat's effectively my question: If your first statement/question is true, not an issue.14:21
*** bkopilov has quit IRC14:21
sdaguethe parallel running of tests is via testr spawning dedicated processes with subsets of the tests, and running subunit serially in each14:22
*** slaweq_ has quit IRC14:22
*** gcb has joined #openstack-nova14:22
cdentyeah, that part I understand (and know too well, dealing with trying to get gabbi to behave properly in that scenario (it does))14:23
sdagueso... as long as tests clean up after themselves, there shouldn't be interference between tests. But cleaning up after themselves is sometimes not super clear, especially if globals are set14:23
*** ociuhandu has joined #openstack-nova14:24
*** tbachman has quit IRC14:25
*** baoli has quit IRC14:29
*** dikonoor has quit IRC14:30
*** ducttape_ has quit IRC14:31
*** tbachman has joined #openstack-nova14:31
*** links has quit IRC14:37
*** smatzek has quit IRC14:37
openstackgerritJohn Garbutt proposed openstack/nova: correct misleading wording  https://review.openstack.org/41365814:38
openstackgerritJohn Garbutt proposed openstack/nova: correct misleading wording  https://review.openstack.org/41365814:38
*** nklenke has joined #openstack-nova14:41
*** Dave has quit IRC14:41
*** BobBall has joined #openstack-nova14:44
*** rmart04 has joined #openstack-nova14:46
*** liusheng has quit IRC14:47
*** ayogi has quit IRC14:49
mdboothcdent: https://bugs.launchpad.net/nova/+bug/1654393/comments/214:49
openstackLaunchpad bug 1654393 in OpenStack Compute (nova) ""keystoneauth1.exceptions.catalog.EmptyCatalog: The service catalog is empty." in test_cinderclient_unsupported_v1" [High,In progress] - Assigned to Matt Riedemann (mriedem)14:49
* cdent looks14:50
cdentgo you. mdbooth++14:50
* mdbooth is starting to suspect global state14:50
johnthetubaguywas just looking at matt's patch he has up for that14:51
*** ratailor has joined #openstack-nova14:51
*** jaypipes is now known as leakypipes14:52
mdboothjohnthetubaguy: Yeah, I wasn't convinced.14:54
mdboothSo, it looks like it fails because _SESSION already exists14:55
*** ducttape_ has joined #openstack-nova14:55
johnthetubaguyyeah, someone already set it up somehow14:55
mdboothjohnthetubaguy: See my comment14:56
*** ist has joined #openstack-nova14:56
mdboothLooks like it's pretty easy to initialise14:56
johnthetubaguyso if _SESSION is set, the old mock would never get run14:56
johnthetubaguybut thats not true after his patch14:56
johnthetubaguyI think?14:56
*** sfinucan has joined #openstack-nova14:57
johnthetubaguyhang on, I am missreading this14:58
mdboothjohnthetubaguy: I'm going to submit an alt patch which does cinder.reset_globals() instead14:58
johnthetubaguymdbooth: ah, gotcha14:58
mdboothBecause that's the real issue here. It's not that mock is stamping on things, it's that something previously initialised global state.14:59
johnthetubaguyyeah, its global state already being tweaked14:59
mdboothAh... and BaseCinderTestCase also does that14:59
* mdbooth likes this better. Patch v soon.15:00
openstackgerritStephen Finucane proposed openstack/nova: console: Move proxies to 'console/proxy'  https://review.openstack.org/40819215:00
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate 'console_driver'  https://review.openstack.org/40819315:00
openstackgerritStephen Finucane proposed openstack/nova: tests: Replace use of CONF with monkey patching  https://review.openstack.org/40819115:00
*** hongbin has joined #openstack-nova15:00
*** ducttape_ has quit IRC15:01
*** mriedem has joined #openstack-nova15:01
*** ducttape_ has joined #openstack-nova15:01
*** smatzek has joined #openstack-nova15:03
cdentIs it fair to ask "Why do we have global state?"?15:04
mriedemi'm going to be sending off the user survey feedback question today so it's last call for questions15:04
*** nicolasbock has quit IRC15:04
johnthetubaguycdent: its a cache I believe15:05
openstackgerritMatthew Booth proposed openstack/nova: Reset global state in CinderV1TestCase  https://review.openstack.org/41740815:06
*** ducttape_ has quit IRC15:06
johnthetubaguycdent: still a valid question15:06
cdentmriedem: "Do you wish there was an orchestration layer?"15:06
mriedemmdbooth: i've already had a patch up for that since yesterday15:06
mdboothmriedem: Yeah, I read it earlier.15:06
mriedemmdbooth: my first attempt at it failed, but the one that you -1ed has passed jenkins several times already15:07
mdboothmriedem: See ^^^. I'm doing reset_globals(). Your patch mocks _SESSION. It has the same effect, but reset_globals() is how the other tests do it.15:07
mdboothAnd it's more explicit as to why it's being done.15:08
*** cleong has joined #openstack-nova15:08
mriedemmdbooth: the reset_globals() thing seems like a hack to begin with15:08
mriedemmdbooth: we shouldn't have to deal with a global between concurrently running tests15:08
mdboothYou need a hack like this everywhere you have global state.15:08
mriedemwhich is why i'm mocking out _SESSION altogether15:08
mdboothWe don't have concurrently running tests.15:08
mriedemsure we do, between test classes at least15:09
mdboothThat's why I was -1 on your patch, because I think it obscured the issue.15:09
*** romainh1 has joined #openstack-nova15:09
mdboothI don't think so. They run serially in multiple processes.15:09
mdboothThe issue here wasn't a race. It was that something else previously initialised the global state.15:09
mriedemmdbooth: then how is it that it was never reproduced locally?15:10
mdboothIt is.15:10
mriedemif it was dirty state?15:10
mriedemi never hit it locally15:10
mdboothSee my comment 2 on the bug.15:10
mdboothRunning those 2 tests in order reproduces the bug deterministically.15:10
*** zz_dimtruck is now known as dimtruck15:10
sdaguemriedem: test grouping and order is an awkward level of deterministic.15:11
sdagueso probably your .testrepository past results are impacting test run order for you and you just aren't getting the bad sequence15:11
*** romainh has quit IRC15:12
mriedemso why not just get rid of the need for reset_globals() altogether?15:12
mriedemif we have to rely on that it seems we've failed15:12
sdaguemtreinish: do you know if there is a way for us to turn off the order optimizer?15:12
*** satyar has joined #openstack-nova15:12
mriedemmtreinish: isn't awake yet :)15:13
*** salv-orlando has joined #openstack-nova15:13
*** marst_ has joined #openstack-nova15:13
mdboothmriedem: To do that you'd need to remove the global state, i.e. not have a global _SESSION15:13
mdboothI think perhaps we could have a central repository for resetting global state, though.15:13
*** voelzmo has joined #openstack-nova15:14
mdboothWe currently do it piecemeal all over the place.15:14
mdboothIf every time we added global state we also added it to a super-global reset_globals()?15:14
*** jaugustine_ has joined #openstack-nova15:14
*** john51 has quit IRC15:16
*** rama_y has joined #openstack-nova15:16
*** john51_ has joined #openstack-nova15:16
sdagueyeh, that would be less prone to errors. A general _reset_globals cleanup in the base test case15:16
mriedemi guess i don't care at this point, i don't see a problem with how i've fixed this, but don't really want to argue (2 sips into coffee), so i'll let others on the core team decide which should go in15:16
*** salv-orlando has quit IRC15:17
sdagueso I'm inclined at this point to take mriedem's approach, because that good practice for that test in question - https://review.openstack.org/#/c/41720615:17
sdaguebut I also think a global cleanup on test.py (especially if it came with a warning about what globals it found that it was reseting so you could figure out what test cases weren't cleaning up) would be good15:18
mdboothWFM. Is it worth fixing up the commit message to not refer to races? That's what confused me, and is why I kept looking.15:18
*** sapcc-bot has quit IRC15:19
cdentUnfortunately we have a tendency to overload "race" to mean anything that is non-deterministic in practice.15:19
mtreinishmriedem: I am awake15:19
*** sapcc-bot has joined #openstack-nova15:19
mriedem:)15:19
mtreinishsdague: you mean the scheduler's optimization based on previous times15:20
sdaguemtreinish: yeh15:20
*** nicolasbock has joined #openstack-nova15:20
mtreinishsdague: the easiest way is just to nuke times.dbm15:20
mtreinishI don't think there is a flag in testr run to disable it15:20
*** voelzmo has quit IRC15:20
sdaguemtreinish: right, but I want to turn it off in tox15:20
mriedemfwiw i deleted all of .tox and .testrepository within the last week15:20
sdaguemriedem: sure, but anything there impacts the run order15:21
mtreinishsdague: add rm .testrepository/times.dbm to the command in the tox.ini?15:21
mtreinishI can push a patch up to testr to skip it, but it'll just go in my pile of open PRs15:21
*** mdrabe_ has joined #openstack-nova15:21
mdboothsdague: It's easy enough to pull the test ordering out of the subunit output15:22
*** ociuhandu has quit IRC15:22
*** ljxiash has joined #openstack-nova15:22
mdboothWhich is what I did15:22
sdaguemdbooth: yeh, it's a little less fun to replicate that run order15:22
mdboothhttps://wiki.openstack.org/wiki/Testr#Reproducing_Failures15:22
sdagueoh, there it is15:23
*** mdrabe has quit IRC15:23
mriedemjohnthetubaguy: dansmith: can you guys get this in unless there are any issues? https://review.openstack.org/#/c/415511/ - jay's API code is ready for +2 and i got the novaclient change up for it last night so would like to move forward15:24
openstackgerritSamantha Blanco proposed openstack/nova: Add missing CLI commands in support-matrix.ini  https://review.openstack.org/41723215:24
johnthetubaguymriedem: looking now ish15:24
mriedemthanks15:25
mriedemjohnthetubaguy: all i added in the latest PS was the response example15:25
*** rmart04 has quit IRC15:25
*** raunak has quit IRC15:26
*** efried has quit IRC15:26
*** ljxiash has quit IRC15:26
*** jwcroppe has joined #openstack-nova15:26
*** prateek has joined #openstack-nova15:27
*** raunak has joined #openstack-nova15:27
*** rmart04 has joined #openstack-nova15:28
*** ducttape_ has joined #openstack-nova15:28
*** gouthamr has joined #openstack-nova15:33
openstackgerritEd Leafe proposed openstack/nova: Trivial indentation fix  https://review.openstack.org/41742215:33
*** slaweq_ has joined #openstack-nova15:33
*** romainh1 has quit IRC15:37
sdaguemriedem: +A15:38
sdaguethanks for adding that example in there15:38
mriedemsdague: thanks15:38
*** andrewbogott has quit IRC15:38
*** andrewbogott has joined #openstack-nova15:38
*** raunak has quit IRC15:39
*** annegentle has joined #openstack-nova15:39
*** slaweq_ has quit IRC15:43
*** raj_singh_ has joined #openstack-nova15:47
*** Dave has joined #openstack-nova15:49
*** sneti has joined #openstack-nova15:49
kashyapIf anyone got a moment, https://review.openstack.org/#/c/411936/ ("libvirt: avoid generating script with empty path")15:50
*** mvk has quit IRC15:50
*** psachin is now known as psachin|brb15:51
openstackgerritEd Leafe proposed openstack/nova: Remove the Allocation.create() method.  https://review.openstack.org/41675115:52
openstackgerritEd Leafe proposed openstack/nova: Add check for invalid allocation amounts  https://review.openstack.org/41675215:52
*** rcernin has quit IRC15:53
*** sneti has quit IRC15:54
*** rmart04 has quit IRC15:55
mriedemkashyap: looking, have an upgrade question coming15:55
kashyapmriedem: Sure, I assume you'll write on the review15:55
mriedemyup15:55
*** annegentle has quit IRC15:56
kashyapmdbooth: When you have time, your eyes are also appreciated (https://review.openstack.org/#/c/411936/)15:56
mriedemkashyap: posted15:57
* kashyap reads15:57
*** gcb has quit IRC15:57
*** psachin|brb is now known as psachin15:57
*** tjones has joined #openstack-nova15:58
*** cleong has quit IRC15:58
mriedemkashyap: yeah see sgordon's comment in the bug https://bugs.launchpad.net/nova/+bug/1649527/comments/115:58
openstackLaunchpad bug 1649527 in OpenStack Compute (nova) "nova creates an invalid ethernet/bridge interface definition in virsh xml" [Undecided,In progress] - Assigned to Neil Jerram (neil-jerram)15:58
*** _ducttape_ has joined #openstack-nova15:58
sahidjamielennox|away:15:59
sahidoops, sorry for that15:59
kashyapmriedem: Yes, the upgrade question is a good one.15:59
*** ist has quit IRC15:59
kashyapWill have to bring it up with upstream libvir list15:59
kashyapmriedem: Thanks for the quick look.  I'll post there once I learn more from the upstream folks.16:00
mriedemkashyap: i guess the question would be how to recreate an upgrade issue where we hit that, i.e. create a vm via nova with libvirt 1.3.1, then upgrade libvirt to 2.0.0, and then try to do something with the vm (restart? stop/start?)16:00
mriedemkashyap: if nova just blows the domain away and recreates it, then we probably don't care once we have the fix16:01
mriedemit's just the xml parse part that i worry about16:01
*** sridharg has quit IRC16:01
sgordonmriedem, it also has to be the specific networking solutions that use that specific vif path16:01
sgordonmriedem, i think neil had identified that was midonet, vrouter, and one or two others16:01
mriedemok, either way the fix looks good, just thinking we should handle '' or None in the parse_dom16:02
*** ducttape_ has quit IRC16:02
mriedemhard to say what else we can do w/o an upgrade recreate16:02
openstackgerritBalazs Gibizer proposed openstack/nova: handle uninited fields in notification payload  https://review.openstack.org/41585716:02
*** rmart04 has joined #openstack-nova16:03
* kashyap back in a few min16:03
openstackgerritBalazs Gibizer proposed openstack/nova: Transform missing delete notifications  https://review.openstack.org/41029716:04
*** Dave has quit IRC16:05
*** voelzmo has joined #openstack-nova16:06
kashyapmriedem: Yeah, the upgrade scenario you outlined, with specific VIF path sounds correct.  I haven't tested it myself, just noticed on the upstream thread.16:06
kashyapWill see if I can test it early next week w/ an upgrade setup16:06
*** unicell1 has quit IRC16:06
openstackgerritMatt Riedemann proposed openstack/python-novaclient: Fix "Message object do not support addition".  https://review.openstack.org/41452916:06
*** rmart04 has quit IRC16:08
*** sudipto_ has quit IRC16:09
*** sudipto has quit IRC16:09
*** _ducttape_ has quit IRC16:10
*** ducttape_ has joined #openstack-nova16:11
*** sneti has joined #openstack-nova16:13
*** salv-orlando has joined #openstack-nova16:14
*** ratailor has quit IRC16:15
*** ducttape_ has quit IRC16:17
*** ducttape_ has joined #openstack-nova16:17
*** sheel has quit IRC16:17
*** salv-orlando has quit IRC16:18
*** cleong has joined #openstack-nova16:20
*** bvanhav__ has joined #openstack-nova16:22
*** bvanhav__ is now known as bvanhav16:23
*** bvanhav_ has quit IRC16:23
*** jaugustine_ has quit IRC16:28
*** yamahata has joined #openstack-nova16:31
*** derekh has quit IRC16:33
*** dtp has joined #openstack-nova16:34
openstackgerritBalazs Gibizer proposed openstack/nova: handle uninited fields in notification payload  https://review.openstack.org/41585716:34
*** baoli has joined #openstack-nova16:35
mriedemBobBall: can we get a run of tempest.api.compute.servers.test_attach_interfaces.AttachInterfacesTestJSON.test_create_list_show_delete_interfaces in the xenserver ci on patch https://review.openstack.org/#/c/365505/ ?16:36
mriedemBobBall: nevermind https://review.openstack.org/#/c/416797/16:36
*** efried has joined #openstack-nova16:37
BobBall:)16:37
mriedemthat's a pretty slick way to test that stuff out16:37
BobBallI guess other CIs have a fixed exclusion list in their jenkins job configuration?16:38
mriedemnot sure how others do it16:38
BobBallAh well.  Did you also see that, now that the os-xenapi devstack plugin improvement is merged, the first os-xenapi change is passing the CI (that one has been a convoluted mess to get all the parts in the right order for every change to pass the CI in order...)16:39
*** ducttape_ has quit IRC16:40
mriedemBobBall: nope, just saw it was still failing last night16:40
BobBallYeah - the change was merged this morning, so it's green now.16:40
*** ducttape_ has joined #openstack-nova16:40
mriedemcool16:40
mriedemwill check that out later today16:40
mriedemare you good with the hotplug patch? i'm just going over the changes from my previous review.16:41
BobBalljohnthetubaguy: ^^ Your input on https://review.openstack.org/#/c/406059/ would also be appreciated16:41
*** rama_y has quit IRC16:41
BobBallThink so - let me double check.16:41
*** dtp has quit IRC16:43
melwittmriedem: do you think we could backport this to newton to make running simple_cell_setup easier? https://review.openstack.org/#/c/409970/16:44
*** dtp has joined #openstack-nova16:45
*** tuanla has joined #openstack-nova16:45
mriedemwell,16:45
mriedemmelwitt: is the TODO in there ok for newton?16:45
mriedem# TODO(melwitt): Remove this when the oslo.messaging function16:45
mriedem    # for assembling a transport url from ConfigOpts is available16:45
mriedemi don't know which change that is referring to in oslo.messaging16:46
*** tuanla has quit IRC16:46
*** tuanla has joined #openstack-nova16:46
*** kfarr has joined #openstack-nova16:46
*** satyar has quit IRC16:46
*** tesseract has quit IRC16:47
*** tuanla has quit IRC16:47
melwittmriedem: ah, right. that TODO is invalid but there were specific changes in oslo.messaging to make it use [DEFAULT]/transport_url, so need to see if those are part of newton16:48
mriedemnot only part of newton, but probably part of the minimum required version of oslo.messaging in newton16:49
mriedemto ensure that it works with the minimum package dependencies16:49
melwittright16:49
*** jose-phillips has joined #openstack-nova16:50
*** nic has joined #openstack-nova16:52
openstackgerritJulian proposed openstack/nova: Transform keypair.create notification  https://review.openstack.org/40868916:52
*** tuanla_ has joined #openstack-nova16:53
*** jaosorior has joined #openstack-nova16:55
*** cleong has quit IRC16:57
*** cleong has joined #openstack-nova16:57
*** jaugustine_ has joined #openstack-nova16:59
*** amotoki has joined #openstack-nova16:59
*** raunak has joined #openstack-nova17:02
*** yamahata has quit IRC17:02
mriedemBobBall: oh so close https://review.openstack.org/#/c/365505/17:03
*** mlavalle has joined #openstack-nova17:04
openstackgerritStephen Finucane proposed openstack/nova: Ignore IOError when creating 'console.log'  https://review.openstack.org/39264317:06
sfinucansahid, vladikr: Any chance ye could have a look at ^ when you have a chance?17:07
vladikrsfinucan, sure17:07
BobBallmriedem: I think we're OK - aren't all API calls here synchronized on the VM object?17:08
BobBallLet me find that...17:08
*** prateek has quit IRC17:08
mriedemBobBall: nope17:08
BobBallI thought they were in the XenAPI driver17:08
mriedemoh, well that might be news to me17:08
BobBallNope, it's not17:09
BobBallI was sure they were at one point :D17:09
BobBallEasy enough to add a utils.synchronized to the plug method though?17:09
mriedemyeah17:10
mriedemalthough17:10
mriedemattach interface is an rpc call17:10
mriedembut i guess if you lock on the instance then that should be ok17:10
BobBallyeah - I meant on the instance.17:11
mriedemhopefully 2 plugs for the same instance at the same time could complete in under 60 seconds17:11
leakypipesdansmith: mind taking a look at https://review.openstack.org/#/c/413068/3/oslo_versionedobjects/base.py? I have a suspicion that doing that is not intended...17:11
BobBallThe synchronization I was thinking of is already there for VBD plug / unplug17:11
BobBalloh yes, for sure.17:11
*** sahid has quit IRC17:11
BobBallCould probably just synchronize the creation (not hotplug) - once we have the record in the XenServer DB it will not announce that as an allowed device17:12
dansmithleakypipes: yep, you're right17:12
leakypipesphew.17:13
*** salv-orlando has joined #openstack-nova17:15
openstackgerritMatt Riedemann proposed openstack/nova: Fix TypeError in _update_from_compute_node race  https://review.openstack.org/41682217:15
*** romainh has joined #openstack-nova17:17
*** prateek has joined #openstack-nova17:17
*** salv-orlando has quit IRC17:19
dansmithmriedem: what do I do for this? use <int> or something? https://review.openstack.org/#/c/410471/6/nova/tests/functional/api_sample_tests/test_hypervisors.py17:20
*** jaosorior has quit IRC17:21
dansmithor %(int)s I guess17:22
*** nicolasbock has quit IRC17:23
*** abalutoiu has quit IRC17:23
*** rtheis has quit IRC17:24
dansmithhmm, nope17:24
leakypipesdansmith: couldn't you just do: \d+17:24
dansmithleakypipes: yes, but just trying to figure out how to use the integrated one he suggested17:24
leakypipesBTW, is superdan in hiding? ;)17:24
dansmithoh jeez is it friday?17:25
*** dansmith is now known as superdan17:25
leakypipessure is!17:25
leakypipes:)17:25
leakypipesthere he is!17:25
leakypipesto save the day.17:25
sean-k-mooneyleakypipes: hi jay would you have time to review sfinucan mtu patch to os-vif https://review.openstack.org/#/c/40877917:25
bauzassuperdan: leakypipes: mriedem: folks, I have an important question for you17:25
bauzasoh man, my manners17:25
*** bauzas is now known as bauwser17:25
leakypipessean-k-mooney: yup, will do. thx for the heads up.17:25
leakypipessuperdan: yeah, looks like %(int)s is correct there.17:26
bauwsersuperdan: leakypipes: mriedem: so, since I wanted to write the scheduler-side of the placement call for getting the list of RPs, I thought about the fact we wrote the methods within the client.report module17:26
superdanleakypipes: doesn't work17:26
bauwsersuperdan: leakypipes: mriedem: that's why I began to cut that from the module and rather create a new module with a PlacementClient17:27
leakypipessdague: perhaps you might know the answer to superdan's dilemma?17:27
bauwsersuperdan: leakypipes: mriedem: but given it will need some tests modifications, that change will be huge to review17:27
superdansdague: I'm trying (desperately) to run devstack locally with cellsv1 and I keep getting use_neutron=True, despite using the ENABLED_SERVICES list from the gate run17:28
superdansdague: any idea if there's something else that will trigger neutron? I need n-net for cellsv117:28
bauwsersuperdan: leakypipes: mriedem: do you guys think we should maybe just use the existing and cut the placementclient afterwards?N17:28
leakypipesbauwser: yes.17:28
bauwserokay, I'll put my existing modifications in a WIP change and -2 it17:29
bauwsertbh, I'm a bit afraid of the time flying17:29
superdansdague: hmm, maybe the interface/network configs17:29
openstackgerritDan Smith proposed openstack/nova: Fix non-parameterized service id in hypervisors sample tests  https://review.openstack.org/41047117:31
openstackgerritDan Smith proposed openstack/nova: Make the SingleCellSimple fixture a little more comprehensive  https://review.openstack.org/41047217:31
openstackgerritGábor Antal proposed openstack/nova: Transform instance.reboot notifications  https://review.openstack.org/38295917:32
*** ociuhandu has joined #openstack-nova17:34
*** sneti has quit IRC17:34
melwittmriedem: so newton requires oslo.messaging>=5.2.0 and all of the rpc backends already supported config via [DEFAULT]/transport_url at that time. so it should be okay17:37
superdanmelwitt: I have no idea why, but suddenly the host calls are failing in cellsv1 with my patch: http://logs.openstack.org/75/396775/21/check/gate-tempest-dsvm-cells-ubuntu-xenial/2d615a2/logs/screen-n-api.txt.gz?#_2017-01-06_16_01_16_39017:37
superdanmelwitt: timing out calling through cells to the compute host.. super weird since that patch only touches instance listings17:38
*** jaugustine_ has quit IRC17:38
openstackgerritJulian proposed openstack/nova: Transform instance.add_fixed_ip notification  https://review.openstack.org/33287617:38
*** lucasagomes is now known as lucas-afk17:39
leakypipessfinucan, sean-k-mooney: questions on commit message.17:39
melwittsuperdan: lettuce see17:39
superdanmelwitt: just talking out loud17:39
melwittoh, yeah. I was thinking maybe since listing does a /detail call, it's trying to pull some host info? that doesn't really make sense though17:40
sean-k-mooneyleakypipes: thanks jay ill take a look at it now. im just opening a bug for the windows issues that sfinucan change uncovered17:41
*** dd-sedanka- has joined #openstack-nova17:41
*** lpetrut has quit IRC17:41
superdanmelwitt: it's just the test hitting the /hypervisors api afaict17:42
melwittyeah, I see that now. weird17:42
leakypipessuperdan: so did you figure out the substitution thing?17:46
superdanleakypipes: no, I went with the + instead of * thing, and will change it if mriedem requires and/or tells me how17:47
superdanleakypipes: but I'm hoping he'll just let it go as is17:47
*** woodster_ has joined #openstack-nova17:47
leakypipessuperdan: but I see "%(int:service_id)s"17:47
leakypipessuperdan: should that just be "%(service_id)s"17:48
leakypipes?17:48
superdannone of those work for me17:49
*** psachin has quit IRC17:49
*** jerrygb has joined #openstack-nova17:50
openstackgerritGábor Antal proposed openstack/nova: Added instance.reboot.error to the legacy notifications  https://review.openstack.org/41181617:54
openstackgerritGábor Antal proposed openstack/nova: Transform instance.reboot.error notification  https://review.openstack.org/41179117:54
melwittsuperdan: I guess the only thing I notice is if you look at the test_results all of the tests failed with a timeout of some kind, but for some reason only the hypervisors api tests are showing any traces in the logs of a timeout17:55
*** romainh has quit IRC17:55
melwittso I wonder if it's a side effect17:55
superdanbut.. look at the change.. it's just adding stuff to get_all instances17:56
superdanseems like.. totally unrelated17:56
melwittI know. I was thinking maybe other tests that do a listing bombarded the queue so hard that it made all the other tests time out too? I dunno17:57
superdanI dunno, or if we locked up the compute node somehow17:57
melwitthm, yeah17:57
superdanthe hypervisors one is the only one I saw with any tracebacks anywhere,17:57
superdanso I was trying to follow that thread17:57
melwittthere's a guru meditation report in the n-cpu.log17:59
melwittis that usual?17:59
superdanany time we fail the run we do that I think17:59
*** browne has joined #openstack-nova18:00
superdanalso, I see debug in n-api that shows us listing cells prperly and stuff18:00
superdanFound 2 cells: cell0,cell1 _get_instances_by_filters_all_cells18:00
superdanwhich is what this thing changes18:00
superdanalthough, the two patches before this don't run the cells job18:00
superdanbecause they're test only18:00
*** unicell has joined #openstack-nova18:00
superdanso I wonder if this is just failing recently by chance? seems too unlikely, but...18:00
melwittyeah, I dunno. I wondered the same18:01
*** pcaruana has quit IRC18:01
*** sneti has joined #openstack-nova18:01
*** rama_y has joined #openstack-nova18:02
superdanI see other runs that passed it a few hours ago, so.. must not be18:04
superdanI just.. don't even18:04
*** bvanhav has quit IRC18:04
melwitt:(18:04
*** rfolco has quit IRC18:09
*** thorst_ has quit IRC18:10
leakypipesbauwser: cdent's patch adding microversion for member_of is now merged, FYI.18:10
leakypipesbauwser: I'm sure you noticed the merge conflict already, eh?18:11
*** sajeesh has joined #openstack-nova18:12
sean-k-mooneyleakypipes: by the way do you know who is in the bug triage team for os-vif? is it just danial?18:12
leakypipessean-k-mooney: dan berrange?18:12
leakypipessean-k-mooney: danpb no longer works on Nova or os-vif.18:12
leakypipessean-k-mooney: so it's just you, me, sfinucan and moshele.18:13
sean-k-mooneyleakypipes: ya i remember the ml post18:13
sfinucanleakypipes: I'm in the bug triage team? :S :)18:13
leakypipessean-k-mooney: and since sfinucan isn't a member of os-vif-core I think we should probably consider adding him and mriedem :)18:13
leakypipessfinucan: I'm "volunteering" you :)18:13
sfinucanleakypipes: Yay. Love being "volunteered" :)18:14
sean-k-mooneyleakypipes: looking at launchpad he is set as the driver and os-vif drivers is the maintaienr but im not in os-vif drivers18:14
leakypipessean-k-mooney: who are you referring to? danpb?18:14
jrollleakypipes: that's "voluntold"18:15
sean-k-mooneyleakypipes: os-vif drivers is currently dan berrange, you and russel18:15
leakypipesjroll: :)18:15
leakypipessean-k-mooney: sec, changing.18:15
sean-k-mooneyleakypipes: https://launchpad.net/~os-vif-drivers/+members18:15
*** salv-orlando has joined #openstack-nova18:15
openstackgerritGábor Antal proposed openstack/nova: Transform instance.reboot.error notification  https://review.openstack.org/41179118:16
leakypipessean-k-mooney, sfinucan, moshele: you are all now os-vif-drivers.18:17
sfinucanleakypipes: Does it come with a pay rise?18:17
sean-k-mooneyleakypipes: thanks18:18
sfinucan:)18:18
sean-k-mooneysfinucan: it come with more work18:18
leakypipesmriedem: you too. :)18:18
sean-k-mooneysfinucan: same thing right18:18
sfinucansean-k-mooney: My favorite :)18:18
leakypipessean-k-mooney: feel like submitting the request to add sfinucan and mriedem to os-vif-core18:18
leakypipes?18:18
sean-k-mooneyin gerrit or to the mailing list18:19
sean-k-mooneybut im fine with adding both18:19
*** salv-orlando has quit IRC18:20
*** mvk has joined #openstack-nova18:20
sean-k-mooneyleakypipes: in gerrit any member of a group can add others so i can add sfinucan and mriedem  to the os-vif-core if we dont need to propose them for core membership on the mailing list first18:21
leakypipessean-k-mooney: no need to propose, no. just go ahead and add.18:21
sean-k-mooneyok will do.18:21
leakypipessean-k-mooney: since danpb has stepped away indefinitely and both sfinucan and mriedem are familiar with the os-vif code.18:22
*** bvanhav has joined #openstack-nova18:22
sean-k-mooneysfinucan mriedem you are now both in os-vif-core enjoy the extra reviewing :)18:23
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Add service_token for nova-cinder interaction  https://review.openstack.org/39739918:23
sfinucansean-k-mooney, leakypipes Cheers :)18:23
* sfinucan still needs to come up with a clever casual nick18:23
leakypipessfinucan: with great power...18:24
leakypipessfinucan: so does sean-k-mooney :)18:24
sfinucan...comes great electricity bill18:24
*** sfinucan has left #openstack-nova18:24
leakypipeslol18:24
*** sfinucan has joined #openstack-nova18:24
leakypipeswelcome back.18:24
sfinucanCtrl + W kills tabs in HexChat also. Who knew?18:25
leakypipes:)18:25
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Add service_token for nova-cinder interaction  https://review.openstack.org/39739918:25
sean-k-mooneysfinucan: i opened a bug for the windows issue with os-vif https://bugs.launchpad.net/os-vif/+bug/1654616 i will try and adress it next week18:25
openstackLaunchpad bug 1654616 in os-vif "_plug_vif_windows uses linux net fuctions that do not work correctly on windows." [Undecided,New]18:25
leakypipessean-k-mooney: some MTU was set to 0? :(18:25
sean-k-mooneyleakypipes: that was the nova default until mitaka18:26
sean-k-mooneyleakypipes: and neutron default18:26
bauwserleakypipes: oh no18:26
leakypipessean-k-mooney: I'm presuming it was meant to mean "default to whatever the PMTU is of the existing segment?"18:26
bauwserleakypipes: okay, I'll rebase anyway18:26
*** claudiub has joined #openstack-nova18:26
leakypipesbauwser: yeah, I have to as well :)18:27
leakypipesbauwser: we're all competing for the API microversion mutex :)18:27
sean-k-mooneyleakypipes: no up to mitaka it was just 0 which in python ment "if mut" was false and nova and neutorn did not set one at all18:27
bauwserI should write a git hook for that :)18:27
bauwserleakypipes: ^18:27
openstackgerritStephen Finucane proposed openstack/os-vif: vif_plug_ovs: Always set MTU when plugging devices  https://review.openstack.org/40877918:27
leakypipessean-k-mooney: ah, ok, makes sense now.18:27
leakypipesbauwser: I know, right?18:28
bauwserleakypipes: something like "lemme verify the last microversion in gerrit, and if conflicting, then sed my number" :)18:28
leakypipesbauwser: :)18:28
sfinucanleakypipes: Let me know if that's ^ any clearer18:29
* sfinucan calls it in for the evening18:29
*** jaugustine_ has joined #openstack-nova18:29
sean-k-mooneyleakypipes: by the way will you be at the ptg in febuary. we should grab some time to talk about what needs to be done in os-vif if people are around18:29
bauwsersfinucan: call it for yourself :p18:30
* leakypipes has to get sean-k-mooney to stop using the term "evacuate" when he means "live migrate" :)18:30
sean-k-mooney:)18:30
leakypipessean-k-mooney: yup, totes.18:30
* bauwser goes afk for a couple of hours before coming back18:30
sean-k-mooneywell in horizon there is an evacuate button that i like to press to see what happens18:31
sean-k-mooneyi assumed that that is waht the operator would do after marking the host as down for scheuling :)18:31
*** yamahata has joined #openstack-nova18:31
*** jaugustine_ has quit IRC18:34
leakypipessfinucan: quick correction needed.18:35
*** prateek has quit IRC18:36
sean-k-mooneyleakypipes: sfinucan: the [1] not refereing which installer it is?18:36
leakypipessean-k-mooney: the big, red, shiny red button: http://img00.deviantart.net/90a4/i/2011/258/e/8/don__t_press_the_button__stimpy_by_megbeth-d49x1oz.jpg18:36
*** prateek has joined #openstack-nova18:36
leakypipessean-k-mooney: right, that's why I said I think you're missing a footnote :)18:36
sean-k-mooneyi had not reloaded yet but yes i see that comment18:37
sean-k-mooneyi was going to leave the same comment18:37
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/41597818:38
*** Jeffrey4l_ has quit IRC18:39
*** Jeffrey4l_ has joined #openstack-nova18:39
edmondswsdague can we merge https://review.openstack.org/#/c/398610/ as-is? Someone else could do the additional work you suggest in your comment, but I don't think this should have to wait for that18:40
* mriedem tries to stumble back through all of the scrollback18:41
*** jaugustine_ has joined #openstack-nova18:41
mriedemsuperdan: fyi if you didn't get the answer yet https://review.openstack.org/#/c/416401/18:42
*** sneti has quit IRC18:42
*** thorst_ has joined #openstack-nova18:42
mriedemmelwitt: superdan: server details list does have an extention in the latest microversion to pull host service status18:44
mriedemsuperdan: melwitt: http://docs.openstack.org/developer/nova/api_microversion_history.html#id1418:45
openstackgerritGábor Antal proposed openstack/nova: Transform instance.reboot.error notification  https://review.openstack.org/41179118:46
melwittmriedem: thanks. we're seeing a failure where a test that calls direct to the hypervisors api is failing though too18:46
mriedemsean-k-mooney: you know https://launchpad.net/~os-vif-drivers/+members doesn't mean core team right?18:47
melwittI wonder if the server listing causes an rpc connection to get cached to the wrong queue topic or something, because what we're seeing is compute not responding on the 'cells' topic18:47
melwittwith cells v1, compute has to listen/respond on both the 'compute' and 'cells' topics18:48
sean-k-mooneymriedem: yes i wanted to be able to do bug triage18:48
mriedemah ok18:48
mriedembtw, https://review.openstack.org/#/admin/groups/1175,members i don't need to be in the core team there :)18:48
mriedemi think i've reviewed maybe 2 os-vif patches18:48
sean-k-mooneymriedem: leakypipes "volentold" you but if you do not want to be on it or dont feel comfortable reviewing it in general i can remove you again18:49
leakypipesmriedem is an all-round amazing velvet doberman of a programmer. he is amazing. thus the desire to make him an os-vif-core.18:51
* leakypipes slinks back into cave.18:51
mriedemsean-k-mooney: removed myself :)18:51
sean-k-mooneymriedem: no worries, leakypipes looks like we need to up our flattery game.18:52
sean-k-mooneyok i have to run so ill talk to everyone later18:53
mriedemheh, i just can't handle the pressure18:53
leakypipesboo :(18:56
*** jerrygb_ has joined #openstack-nova18:57
*** amotoki has quit IRC18:58
melwittlooks like that host service status is just a servicegroup api query for up/down so shouldn't be related18:59
*** jerrygb has quit IRC19:00
*** ducttape_ has quit IRC19:00
*** ducttape_ has joined #openstack-nova19:00
cdentI'm finding myself wishing I could create nameless resource providers over the api. But it is really only for testing, not real life, so I guess that doesn't count.19:02
*** lpetrut has joined #openstack-nova19:04
mriedemleakypipes: superdan: i'm cool with https://review.openstack.org/#/c/410471/ now19:05
*** xyang1 has joined #openstack-nova19:06
mriedembauwser: leakypipes: i was at lunch and missed the discussion about the placement client stuff - was the decision to NOT pull that out and instead just re-use the scheduler report client stuff in the host manager?19:07
*** tuanla_ has quit IRC19:07
*** salv-orlando has joined #openstack-nova19:07
leakypipesmriedem, superdan: Wallaby'd.19:11
leakypipesmriedem, bauwser: well, what I was saying was to not create a placement client and instead do raw calls into the placement API in the same vein as the scheduler report client.19:12
mriedemleakypipes: superdan: got a problem in this https://review.openstack.org/#/c/410472/19:13
superdanmriedem: leakypipes thanks19:13
mriedemleakypipes: oh, ok...well....we'd have that same pattern in 3 places in nova then...19:14
mriedemscheduler report client, nova-status, and then the host manager19:14
mriedemand as seen with the 401 issue this week, if we have a bug in one we have to check all of the others19:15
leakypipesmriedem, bauwser: I'd be cool pulling the very basic safe_connect() and get/put/delete/post methods into a base class, I suppose, but nothing more than that.19:16
bauwserleakypipes: I already wrote that, lemme upload it19:16
bauwserleakypipes: I just want to not wait for that19:17
mriedemleakypipes: that's what this does https://review.openstack.org/#/c/417111/2/nova/scheduler/client/__init__.py19:17
leakypipesmriedem, bauwser: i.e. I don't want to hold up progress putting together an object-oriented placementclient-type thing.19:17
bauwserexactly19:17
mriedemyeah i agree with not holding it up19:17
bauwserthe change is becoming huge19:17
mriedembut...if it's a simple lieft19:17
mriedem*lift19:17
mriedembauwser: what is huge about it? the tests?19:17
bauwserso I'll write the scheduler modification before that change19:17
bauwsermriedem: zacvtly19:17
bauwsermriedem: and because I copy/paste a lot19:17
bauwsercreating a new module19:18
leakypipesbauwser: yeah, I don't see that patch being too big?19:18
bauwserleakypipes: mriedem: you'll see19:18
bauwserjust rebasing the API change and you'll see, folks19:18
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Objectify _validate_requested_network_ids  https://review.openstack.org/41195019:18
leakypipesok19:18
leakypipesbauwser: the only thing I'd like to see is the use of encapsulation instead of inheritance there.19:19
mriedembauwser: so the alternative is just using the existing stuff in the scheduler report client class right?19:19
leakypipesbauwser: in other words, just have the scheduler report client contain an instance of PlacementClient.19:20
bauwserleakypipes: yup19:20
mriedemif it's just a matter of using the scheduler report client right now because it's available and easy, then just do that and we can split apart and cleanup later19:20
*** voelzmo has quit IRC19:20
*** nic has quit IRC19:20
bauwserleakypipes: I wrote an encapsulation like cdent asked19:20
leakypipesk19:20
*** voelzmo has joined #openstack-nova19:21
bauwserleakypipes: I wrote an inheritance because I was lazy19:21
*** rmk has quit IRC19:21
*** raginbajin has quit IRC19:21
*** rmk has joined #openstack-nova19:21
*** rmk has quit IRC19:21
*** rmk has joined #openstack-nova19:21
*** jowisz_ has joined #openstack-nova19:21
*** bhagyashri_s has joined #openstack-nova19:22
*** bhagyashris has quit IRC19:22
*** lucas-afk has quit IRC19:22
*** jowisz has quit IRC19:22
*** lucasagomes has joined #openstack-nova19:22
bauwsermriedem: just a question : we created a stable 14.0.2 pin but we have a DB migration in it, which requires a db sync, shouldn't we have created a 14.1.0 ?19:23
jrollmriedem: I see you in my meeting agenda, are you attening or did you just want me to say words about that19:23
jrollI'm up to date on the statuses of those things19:23
*** ducttape_ has quit IRC19:23
*** ducttape_ has joined #openstack-nova19:24
*** ducttape_ has joined #openstack-nova19:24
*** voelzmo has quit IRC19:25
leakypipessuperdan: your eyeballs on https://review.openstack.org/#/c/415031/ would be appreciated. I know it's a blocker for mlavalle :)19:25
mriedembauwser: do we have rules anywhere about versions on stable branches that include a db migration?19:25
mriedembauwser: it was additive19:25
mriedembauwser: fyi our next stable/newton is going to have an online data migration bug fix in it too19:26
mriedemnot a schema migration though19:26
superdanleakypipes: I have to figure out this cellsv1 thing before I can do anything else or we're all fscked19:27
superdanso yeah, maybe but not today sorry19:27
leakypipessuperdan: not a problem.19:27
leakypipessuperdan: lemme know if I can assist on the cellsv1 issues.19:27
superdanleakypipes: got any guns?19:28
*** jaugustine_ has quit IRC19:28
mriedemhe does live in FL...19:28
mriedemthere are probably some just lying around19:29
superdanI could loan him one19:29
jrollhe could just send floridaman your way19:29
superdansdague: back around yet?19:29
bauwsermriedem: sorry, was diverted19:29
bauwsermriedem: lemme look at semver rules in the governance repo19:30
bauwsermriedem: but AFAIR, .z versions means just apt-get and then restarty19:30
bauwseror yum :)19:30
bauwsererf19:30
bauwserdnf even19:30
bauwser(shhhht, don't tell my employer :p )19:30
jrollheh19:31
mriedemheh19:31
jrollfeels like db migrations would be a .y but I'd probably ask release folks19:31
*** jaugustine_ has joined #openstack-nova19:31
mriedemeither way,19:31
mriedemthat ship has kind of sailed...19:31
mriedemas of like a month ago19:31
bauwseryeah I know19:31
bauwsermriedem: it's somehow an internal discussion, y'know19:31
mriedembauwser: i figured19:32
bauwserjust popping up here :)19:32
mriedemtell them i'm sorry,19:32
mriedembauwser: however, if they care about how upstream stable releases are versioned,19:32
mriedemthere is a way they can get involved in monitoring that...19:32
bauwsermriedem: no worries, I'll ask them to leave tar and feather at home19:32
* mriedem felt good about that dig19:32
openstackgerritDan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing  https://review.openstack.org/39677519:33
openstackgerritDan Smith proposed openstack/nova: Make the SingleCellSimple fixture a little more comprehensive  https://review.openstack.org/41047219:33
melwittsuperdan: I dunno if you saw the scrollback but I was thinking I wonder if the hosts api call is going out on the wrong topic ('compute' instead of 'cells' topic, since the rpc connects get cached via cell uuid) and that's why it doesn't get a response there. I just don't get how that would start up with your patch and not being failing as-is19:33
openstackgerritMatthew Edmonds proposed openstack/nova: update policy UT fixtures  https://review.openstack.org/39861019:33
*** READ10 has quit IRC19:34
superdanmelwitt: hmm19:35
superdanmelwitt: I just put up a test what just comments out the fanout to the cells, which will fail a bunch of things, but I want to see if it affects the host bit19:35
mriedembauwser: so where are we at with cleaning up https://review.openstack.org/#/c/392569/ ?19:36
leakypipessuperdan: I live in Florida. of course I do.19:36
melwittbecause what's failing is this 'intercell' communication, the 'cells' topic19:36
superdanmelwitt: well, it's not actually.. if you look, the cells-child is failing to call to the compute via compute topic, which is cascading up19:37
*** cfriesen has quit IRC19:37
melwittsuperdan: I thought this is showing a send to the 'cells' topic? http://logs.openstack.org/75/396775/21/check/gate-tempest-dsvm-cells-ubuntu-xenial/2d615a2/logs/screen-n-api.txt.gz?#_2017-01-06_16_06_16_67819:38
superdanmelwitt: right, but hang on19:38
melwittlemme look in the other logs again19:38
superdanmelwitt: http://logs.openstack.org/75/396775/21/check/gate-tempest-dsvm-cells-ubuntu-xenial/2d615a2/logs/screen-n-cell-child.txt.gz?level=TRACE#_2017-01-06_16_01_16_49319:38
superdanthat's what gets called, which makes a call to compute, which times out19:39
*** cdelatte has quit IRC19:42
*** sajeesh has quit IRC19:42
melwittit says FANOUT topic 'cells.intercell.response' for the same req-id. you're saying the call to compute will go on the 'compute' topic though, and that's not responding and thus doesn't end up responding on the cells topic19:43
superdanmelwitt: but it's a call to compute_rpcapi19:43
superdan File "/opt/stack/new/nova/nova/compute/rpcapi.py", line 920, in get_host_uptime19:43
*** jerrygb_ has quit IRC19:43
melwittyeah. okay. so it goes on cells topic -> compute topic then replies go backward. hmm19:44
superdanright19:44
*** marst_ has quit IRC19:44
cdentleakypipes: you might find this amusing or interesting: https://tank.peermore.com/_/8760ec93-b39a-4dd5-849f-d880572fb2cd19:46
*** salv-orlando has quit IRC19:46
cdentthat's a gabbit using a new $HISTORY feature in gabbi + some yaml anchors to play games with the placement api19:46
*** baoli has quit IRC19:46
*** voelzmo has joined #openstack-nova19:47
cdent./stack.sh && openrc admin && gabbi-run http://localhost < that.yaml19:47
cdentgonna flesh it out a bit more, but going to dinner now19:47
cdents/openrc/. openrc/19:48
* cdent waves19:48
*** cdent has quit IRC19:48
melwittsuperdan: for cells v1 do we have a host mapping set up that has the child cell transport_url in it?19:50
melwittsorry cell mapping19:50
superdanmelwitt: yeah, should, but hmm, I wonder if that points to the wrong thing19:50
superdanmelwitt: in the devstack job, do we really have any different mq exchanges or anything?19:50
* superdan looks19:50
melwittI noticed the nova.conf and nova-cells.conf have different transport urls. but I don't get how it ever worked then19:51
superdanooooh19:51
superdanbecause19:51
superdanI just changed that in a not yet merged dep patch :)19:51
superdancheck the depends-on19:51
melwittahhh19:52
superdanplease oh please tell me you spot something wrong in there :D19:53
superdanPUHLEASE19:53
superdanotherwise leakypipes will have to shoot me,19:53
superdanand I hear he has bad aim and might have to take a few shots19:53
melwittthinking ... I think we need two cells, one for the api and one for the compute, being that they have different transport urls? I dunno, still thinking19:54
*** ljxiash has joined #openstack-nova19:54
*** weshay is now known as weshay_brb19:54
superdanno, because the cellsv1 messaging isn't switched I think19:54
superdanbut we do need the cell mappings we have to point to the right place,19:55
superdanso that when something calls on compute, it does so on the right bus19:55
melwittright ... so is that get_transport_url getting the right thing, because it needs to get it out of the nova-cells.conf19:55
superdanso,19:55
superdanright, I was going to say, get_transport_url is probably the api cell url,19:55
superdannot the compute-cell url, which is what we need right?19:56
mlavalleleakypipes thanks for the help :-)19:56
melwittyeah, I'm hoping that's what's happening because otherwise I dunno19:56
melwittlooking for what get_transport_url does19:56
superdanyeah,19:56
superdanI need "get_transport_url child_cell"19:57
superdansonofa19:57
melwittyeah, need to pass the virtual_host. I really hope that fixes things19:57
superdanyeah19:58
openstackgerritSylvain Bauza proposed openstack/nova: WIP/DNM : Put all the magic in a PlacementClient  https://review.openstack.org/41711119:58
openstackgerritSylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs  https://review.openstack.org/39256919:58
openstackgerritDan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing  https://review.openstack.org/39677519:58
superdanmelwitt: we shall see ^19:58
*** ljxiash has quit IRC19:58
superdanmelwitt: thanks for spotting that19:58
*** voelzmo_ has joined #openstack-nova19:59
bauwsermriedem: leakypipes: rebased https://review.openstack.org/39256919:59
*** amotoki has joined #openstack-nova19:59
melwittfingers crossed19:59
bauwsermriedem: leakypipes: and uploaded latest version of the placementclient module - but that's not a goal for now I think19:59
*** tbachman has quit IRC20:01
*** jerrygb has joined #openstack-nova20:02
*** voelzmo has quit IRC20:02
*** jerrygb has quit IRC20:03
*** jerrygb has joined #openstack-nova20:03
*** voelzmo_ has quit IRC20:03
*** slaweq_ has joined #openstack-nova20:04
*** marst has joined #openstack-nova20:07
openstackgerritIldiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow  https://review.openstack.org/33028520:07
openstackgerritIldiko Vancsa proposed openstack/nova: Remove check_attach  https://review.openstack.org/33535820:07
mriedemildikov: hey, btw,20:07
*** moshele has joined #openstack-nova20:08
mriedemthat wip patch to use cinder v3 in nova - totally failed with the same endpoint not found that you were seeing20:08
mriedemhttp://logs.openstack.org/82/385682/3/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0a4185b/logs/screen-n-cpu.txt.gz?level=TRACE#_2017-01-05_19_30_30_35320:08
*** cdelatte has joined #openstack-nova20:08
ildikovmriedem: I'm glad to know it's not just my ghost20:08
ildikovmriedem: I fixed it locally by adding the endpoint_template to the config, but I still experience weird things20:09
ildikovmriedem: oh and tnx for the heads up on this :)20:09
openstackgerritSarafraj Singh proposed openstack/nova: Add service_token for nova-cinder interaction  https://review.openstack.org/39739920:10
openstackgerritSarafraj Singh proposed openstack/nova: Add service_token for nova-neutron interaction  https://review.openstack.org/41039420:11
*** jerrygb_ has joined #openstack-nova20:13
*** claudiub|2 has joined #openstack-nova20:13
*** jaugustine_ has quit IRC20:13
*** raj_singh_ has quit IRC20:14
*** jerrygb has quit IRC20:15
*** claudiub has quit IRC20:16
*** jerrygb has joined #openstack-nova20:17
*** jaugustine_ has joined #openstack-nova20:17
*** voelzmo has joined #openstack-nova20:19
*** jerrygb_ has quit IRC20:19
*** satyar has joined #openstack-nova20:20
*** jaugustine_ has quit IRC20:21
*** tbachman has joined #openstack-nova20:22
*** Swami has joined #openstack-nova20:22
*** voelzmo has quit IRC20:23
*** dtp_ has joined #openstack-nova20:28
*** tbachman has quit IRC20:29
*** adisky_ has quit IRC20:29
*** satyar has quit IRC20:30
*** satyar has joined #openstack-nova20:30
*** dtp has quit IRC20:30
*** cleong has quit IRC20:32
openstackgerritDiana Clarke proposed openstack/nova: Query parameter validation for usages API  https://review.openstack.org/41750020:32
*** bvanhav has quit IRC20:33
openstackgerritDiana Clarke proposed openstack/nova: Query parameter validation for usages API  https://review.openstack.org/41750020:36
melwitthm, grenade jobs failing with "/opt/stack/new/grenade/projects/70_cinder/resources.sh:destroy:199 :   openstack server remove floating ip cinder_server1 172.24.5.6 | get() takes exactly 1 argument (2 given)"20:37
*** jaugustine_ has joined #openstack-nova20:42
*** weshay_brb is now known as weshay20:45
*** salv-orlando has joined #openstack-nova20:47
openstackgerritDiana Clarke proposed openstack/nova: Query parameter validation for hypervisors API  https://review.openstack.org/41750720:48
*** baoli has joined #openstack-nova20:50
*** salv-orlando has quit IRC20:52
*** moshele has quit IRC20:53
*** dimtruck is now known as zz_dimtruck20:56
*** voelzmo has joined #openstack-nova20:56
openstackgerritClaudiu Belu proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004520:57
*** Dave has joined #openstack-nova21:00
*** voelzmo has quit IRC21:01
*** moshele has joined #openstack-nova21:07
*** dd-sedanka- has quit IRC21:08
*** dave-mccowan has quit IRC21:10
openstackgerritDiana Clarke proposed openstack/nova: Query parameter validation for hypervisors API  https://review.openstack.org/41750721:10
*** Sukhdev has joined #openstack-nova21:12
*** browne has quit IRC21:13
*** voelzmo has joined #openstack-nova21:14
*** JoseMello has quit IRC21:15
*** zz_dimtruck is now known as dimtruck21:15
*** gongysh has joined #openstack-nova21:15
*** catintheroof has quit IRC21:18
*** voelzmo has quit IRC21:18
*** browne has joined #openstack-nova21:19
*** cdelatte has quit IRC21:19
*** tbachman has joined #openstack-nova21:21
*** jaugustine_ has quit IRC21:25
*** smatzek has quit IRC21:27
*** dtp_ is now known as dtp21:29
*** dave-mccowan has joined #openstack-nova21:30
*** voelzmo has joined #openstack-nova21:32
*** Jeffrey4l_ has quit IRC21:35
*** satyar has quit IRC21:35
*** voelzmo has quit IRC21:36
openstackgerritDiana Clarke proposed openstack/nova: Query parameter validation for usages API  https://review.openstack.org/41750021:36
*** armax has quit IRC21:39
*** armax has joined #openstack-nova21:40
*** efried has quit IRC21:41
*** weshay is now known as weshay_bbiab21:42
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: fix live migration with CSVs  https://review.openstack.org/38044821:43
*** Jeffrey4l_ has joined #openstack-nova21:47
*** salv-orlando has joined #openstack-nova21:48
*** mdrabe_ has quit IRC21:48
*** voelzmo has joined #openstack-nova21:50
*** thorst_ has quit IRC21:50
*** tbachman has quit IRC21:52
*** salv-orlando has quit IRC21:52
superdanmelwitt: well, that got it back down from insane-fail to lots-of-fail21:53
superdanwhich I call major progress21:53
superdanlike, finally some sane failures: http://logs.openstack.org/75/396775/23/check/gate-tempest-dsvm-cells-ubuntu-xenial/c10b6b3/logs/screen-n-api.txt.gz?level=TRACE#_2017-01-06_20_19_05_13321:54
melwittsuperdan: heh, yeah. I saw there's a lot of UnknownCell errors in there now but hopefully that won't be too hard to track down21:54
*** voelzmo has quit IRC21:54
superdanyeah, I dunno what that means, but.. it's something to chase at least21:55
melwittdefinitely a big improvement over crazy pants21:55
superdanhuh, it means instance.cell_name is not set21:55
melwittmeaning instance wasn't assigned to a cell v1 cell somehow. or hasn't been scheduled yet21:56
superdanyeah21:56
*** smatzek has joined #openstack-nova21:58
*** sneti has joined #openstack-nova21:58
melwitthm, so when an instance record is written to a cell db, when it syncs to the top the cell_name is set based on the message routing path?21:59
melwittI don't think I ever knew how cell_name got set, but it looks like it happens in instance_update_at_top in cells/messaging22:00
superdanthat's..odd22:01
superdanI guess so the cell doesn't have to know its name or something?22:01
*** catintheroof has joined #openstack-nova22:01
melwittseems like it. I dunno22:01
*** dtp has quit IRC22:02
*** smatzek has quit IRC22:02
*** efried has joined #openstack-nova22:03
superdanman22:04
melwittsuperdan: I wonder if it's something like, it's reading the instance from the cell db so it doesn't have cell_name there. cell_name is only in the record at the top?22:04
superdanI'm just shocked that this continues to work with none of us really knowing about it22:04
superdanyeah might be22:04
melwittyeah well, we've taken care not to touch the code too much. and I broke things when I object-ified some stuff there that had to get fixed22:05
*** thorst_ has joined #openstack-nova22:05
*** smatzek has joined #openstack-nova22:05
openstackgerritDan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing  https://review.openstack.org/39677522:07
superdantheory-testing hack ^22:07
melwittI wonder if there are any other problems with reading instances from the compute cell db rather than the api cell db. I hope not22:07
*** jaugustine_ has joined #openstack-nova22:09
superdanwell,22:09
superdanit may surprise some people that don't expect connections from api to those dbs22:09
superdanbut that's kinda part of the migration path anyway22:09
melwittI'm not sure that will work, because I don't think cell_name is dirtied in the compute cell22:09
superdanoh, because that makes an rpc call back to messaging22:10
melwittyeah, the instance.save() will call up to instance_update_at_top, and that function will set cell_name only at the top22:10
melwitthttps://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L100922:11
*** thorst_ has quit IRC22:11
superdanright but you mean call up to the regional cell service, which is across rpc22:11
melwittand that's not going to sync back down22:11
superdanwait22:11
superdanoh yeah22:11
melwittyeah rpc call to the api cell22:11
*** jheroux has quit IRC22:11
melwittI think22:11
superdanI was thinking that was a direct call into there because the name was the same, but I jumped the api->messaging gap22:12
superdanso I should be able to get it from conf actually22:12
melwittwell, I mean the compute cell will never run the code that changes the cell_name I mean22:12
*** jerrygb has quit IRC22:13
melwittI dunno, I'm getting confused. anyway22:13
*** jaugustine_ has quit IRC22:13
*** gongysh_ has joined #openstack-nova22:15
openstackgerritDan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing  https://review.openstack.org/39677522:15
superdanhack @322:15
superdaner, #222:15
*** jaugustine_ has joined #openstack-nova22:17
*** gongysh has quit IRC22:17
*** weshay_bbiab is now known as weshay22:17
*** jamesdenton has quit IRC22:18
melwittsuperdan: I think that might not work because I think the instance.create() goes only to the api cell db, then later when instance.save() happens in the api cell, that's when it gets written to the compute cell db :\22:19
*** sneti has quit IRC22:20
melwittas part of the sync down22:21
superdanmelwitt: we have to create() once in the child cell22:21
superdanwe can't just save22:21
*** dave-mccowan has quit IRC22:22
melwittsuperdan: okay. I had thought when we were looking at replays of compute/api the instance create isn't one of them and so I went looking around for how the record ever ends up in the compute cell db22:23
superdanmelwitt: well, we have to call db.instance_create() at least22:24
superdanbut I can't imagine we don't go through the object, because we'd miss out on the other stuff we create there,22:24
superdanlike instance_extra and such22:24
superdanbut if you know other, please point to it22:24
smcginnisDoes Nova actual use limits? "nova limits"22:25
smcginnisAnd if so, where are these set? I've found references that limits are set in the project.conf file, but nothing stating what needs to actually be set.22:25
melwittsuperdan: yeah, double checking to see if I missed something last time22:25
smcginnisCinder has absolute limits too, but it appears to be left over from the Nova split. Trying to figure out if they are actually used/working.22:26
*** xyang1 has quit IRC22:35
mriedemsmcginnis: limits == quotas22:38
mriedemos-quota-sets API22:38
smcginnismriedem: nope22:39
mriedemthe limits API is just the read-only API for getting the quota limits22:39
smcginnismriedem: Not from what I've seen. Though maybe that's changed in Nova?22:39
smcginnismriedem: Ah, that might explain some things. :)22:39
mriedemsmcginnis: well,22:39
mriedem...22:39
mriedem...22:39
smcginnismriedem: Some really misleading stuff out there.22:39
mriedemtrying to figure out how long i want to make this22:39
mriedemquotas come from 3 different places in nova,22:39
*** efried has quit IRC22:40
mriedem1. config22:40
smcginnismriedem: It's late on Friday. Probably better just to tell me to step off.22:40
mriedem2. quota classes22:40
melwittsuperdan: this is where I  don't see how instance.create() is ever called in compute cells https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L194-L198 so ... I dunno, we'll see22:40
mriedem3. i might only be thinking of 222:40
mriedembasically if you don't have default quotas overridden in the API, then the default quotas for all projects comes from config,22:40
mriedembut if you change default quota in the REST API, then it's in the nova DB22:40
mriedemand we never read from config for that resource type again22:41
mriedemand you can tailor default quota globally or per-tenant22:41
mriedemi think that was my #422:41
mriedem*#322:41
*** figleaf is now known as edleafe22:41
mriedem1. check if there is quota in the db for this project and resource, if not,22:41
smcginnismriedem: OK, that makes sense. Maybe just thrown off by having two different names and some old or incorrect comments I read.22:41
smcginnismriedem: Thanks22:41
mriedem2. check if there is default quota in the db for this resource, if not,22:41
mriedem3, check config22:41
superdanmelwitt: okay, but.. the only thing in db_api that will create an instance row is db.instance_create(), and I don't see us calling that from anywhere other than the object, right?22:41
mriedemsmcginnis: http://lists.openstack.org/pipermail/openstack-dev/2016-December/108997.html and the linked spec might help a bit22:42
mriedemthe spec gives more background in a distilled way on how this all works today22:42
mriedemi should actually move the problem description from https://review.openstack.org/#/c/411035/2/specs/pike/approved/deprecate-os-quota-class-sets.rst into the devref22:42
*** tblakes has quit IRC22:43
mriedemsince it took me a day plus sanity checking from melwitt to sort it out22:43
smcginnisheh, how timely.22:43
melwittsuperdan: that's what I wasn't sure about, I assumed db.instance_update_and_get_original can somehow create an instance row, being that I never found a place instance.create() was called22:43
*** lamt has quit IRC22:43
superdanpretty sure not.. it should fail immediately if you do22:43
*** jaugustine_ has quit IRC22:44
superdanmelwitt: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2696-L269722:44
superdanshould fail right there22:44
melwittoh, hmm. I'm stumped then22:45
superdanah22:45
*** armax has quit IRC22:46
superdanwell,22:46
superdanthere is this weird automatic hijack of the compute rpc,22:46
superdanwhich might have had it before22:47
superdanwhere we neuter the rpc at the api level, and auto-call to the child in some cases22:47
superdanbut create isn't in there22:47
superdanbuild is though22:47
*** roeyc has joined #openstack-nova22:47
superdananyway, seems like it has to be something like that22:47
superdanif not, I have no idea how it works,22:48
superdanbut I've said that like three times today already so,..22:48
melwittyeah ... well, at least that means the hack will probably work, so yey for that at least22:49
superdanoh yeah, here22:49
*** salv-orlando has joined #openstack-nova22:49
*** kfarr has quit IRC22:49
superdanhttps://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L85-L8622:49
superdanwhich is set somewhere so that we get to hijack calls22:49
melwittO.o22:50
superdanand I think there's some split of things that are new (objecty) and old (not objecty) somewhere, but I always forget that piece22:50
superdanmaybe that's in messaging22:50
melwitton top of the whole cells_api thing? damn22:50
superdanyeah, there is a ton of indirection and magic in here22:50
superdanthis is why I'm always talking about all the replaying  :)22:50
superdanbecause there's .. a .. lot of it22:51
superdanand several ways of doing it22:51
melwittyeah, I thought all of the replaying could be seen in the ComputeCellsAPI methods and never realized there are separate bits that can interleave in there from the RPCProxy whatever22:51
*** omarrr has joined #openstack-nova22:52
superdanyeah, there are implicit paths in tere22:52
superdan*there22:52
melwitt"good times"22:53
*** salv-orlando has quit IRC22:54
*** ducttape_ has quit IRC23:00
*** thorst has joined #openstack-nova23:00
melwittsuperdan: gdi http://logs.openstack.org/75/396775/25/check/gate-tempest-dsvm-cells-ubuntu-xenial/7105ddd/logs/screen-n-api.txt.gz?level=TRACE#_2017-01-06_22_45_29_71123:01
superdangah23:01
openstackgerritDan Smith proposed openstack/nova: WTFWIP: Multicell support for instance listing  https://review.openstack.org/39677523:01
*** thorst has quit IRC23:03
*** edmondsw has quit IRC23:06
*** jamesdenton has joined #openstack-nova23:12
*** burt has quit IRC23:13
mriedemjroll: when can we get an ironicclient release for https://review.openstack.org/#/c/335964/ ?23:17
mriedemjroll: like, early next week?23:17
mriedemor is that already in the works?23:17
*** gongysh_ has quit IRC23:17
jrollmriedem: see the comments there. tl;dr nova won't use portgroups for things until attach/detach is implemented. that still has a client patch out there. so no point in releasing now23:18
jroller, not there, but on the nova patch23:18
mriedemhmm23:19
mriedemso https://review.openstack.org/#/c/206163/23:19
mriedemhas a comment from vdrok: "This patch is not needed anymore, as after this one https://review.openstack.org/364413 it will be happening on ironic side."23:19
mriedemoh which is https://review.openstack.org/#/c/364413/23:20
*** moshele has quit IRC23:20
mriedemfor attach/detach23:20
jrollright, so here's the two ways we can do this23:20
jroll1) (the way we're doing it) do attach/detach first, nova just makes an api call, ironic does the ports/portgroup/order/etc logic23:20
jroll2) implement portgroups in plug_vifs now, make that work, then rip it all out to do attach/detach23:20
mriedem2 is https://review.openstack.org/#/c/206163/ right?23:21
jrollyes, that should be abandoned23:21
mriedemjroll: so the other one for attach/detach depends on this ironic patch https://review.openstack.org/#/c/404240/23:21
mriedemwhich looks like it's not ready23:21
jrollit's pretty close23:22
mriedemand the ironicclient change for that https://review.openstack.org/#/c/364420/ is dependent on the same ironic api change,23:22
jrollwe keep merging crap that conflicts, it seems, I'll make sure we stop doing that23:22
mriedemso i guess the api change would realistically have to be merged by eow next week,23:22
mriedemto get the client release in by 1/1923:22
mriedemand then the nova change 1 week after that :)23:23
jrollyeah I think we can get the client next week even23:23
mriedemi assume the ironicites like their schedules like they like their jeans, tight23:23
jrollno we just have too much shit to do and keep crossing wires23:23
mriedemyeah i know, i just wanted to make a skinny jeans joke23:23
mriedemok i'll abandon #223:24
jrollcool, thanks23:24
jrolllooks like this is rebased on it, which isn't good https://review.openstack.org/#/c/388756/23:24
jrollbut I'll make sure that gets cleaned up23:24
mriedemok. do i get to supersede the portgroups bp by the attach/detach bp, or is there still something unique about the portgroups bp that we keep it around, like this network metadata patch?23:25
jrollportgroups depends on attach/detach23:25
mriedemin other words, does https://review.openstack.org/#/c/388756/ get rebased on https://review.openstack.org/#/c/364413/ ?23:26
jrollyep23:26
mriedemok i'll just leave the LP bits alone for now then23:26
mriedemthanks for clarifying23:26
mriedemi'm going down my list and checking it twice, obviously23:26
jrollthat passes in the metadata to tell the instance it's a bond23:26
jrollno problem23:26
jrollbtw, are you attending our meeting monday or just wanted to put that thing in the agenda as a note?23:27
mriedemjroll: just put it as a note23:28
jrollcool, I'll announce it23:29
*** smatzek has quit IRC23:29
*** jamesdenton has quit IRC23:33
mriedemjroll: what is higher priority, vif attach/detach or rescue/unrescue, because the latter is in merge conflict down the stack - i'm assuming the former is higher priority though23:34
mriedemshould maybe also push on this next week since the ironic API change is already done https://review.openstack.org/#/c/407977/23:35
*** ducttape_ has joined #openstack-nova23:36
*** jamielennox|away is now known as jamielennox23:39
jrollmriedem: the former, I don't expect rescue to land in nova this cycle23:41
jroll+123:41
mriedemok. if i remember after going through the list i'll update the ironic meeting agenda with things i think are close but need the push23:41
jrollawesome, thank you23:44
*** lpetrut has quit IRC23:44
mriedemclaudiub|2: are you planning a release request for os-win to get https://review.openstack.org/#/c/398215/ released? that's needed for https://review.openstack.org/#/c/398509/ which you need to update.23:44
*** jwcroppe has quit IRC23:45
claudiub|2mriedem: indeed i am. monday23:45
mriedemclaudiub|2: great, thanks23:45
*** jwcroppe has joined #openstack-nova23:45
claudiub|2thanks as well. :)23:45
openstackgerritYuming Ma proposed openstack/nova-specs: Nova BFV by reserving volume  https://review.openstack.org/41754123:47
*** jwcroppe has quit IRC23:50
*** salv-orlando has joined #openstack-nova23:50
openstackgerritYuming Ma proposed openstack/nova-specs: Nova BFV by reserving volume  https://review.openstack.org/41754123:50
*** salv-orlando has quit IRC23:55

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