| *** Sukhdev has joined #openstack-nova | 00:13 | |
| *** thorst has joined #openstack-nova | 00:17 | |
| *** kaisers1 has joined #openstack-nova | 00:19 | |
| *** kaisers4 has joined #openstack-nova | 00:19 | |
| *** ducttape_ has quit IRC | 00:19 | |
| *** kaisers3 has quit IRC | 00:20 | |
| *** kaisers2 has quit IRC | 00:20 | |
| *** sdake has quit IRC | 00:22 | |
| *** efried1 has joined #openstack-nova | 00:23 | |
| *** efried has quit IRC | 00:24 | |
| *** thorst has quit IRC | 00:25 | |
| *** Sukhdev has quit IRC | 00:26 | |
| *** Sukhdev has joined #openstack-nova | 00:28 | |
| *** sdake has joined #openstack-nova | 00:32 | |
| *** dimtruck is now known as zz_dimtruck | 00:33 | |
| *** hoangcx has joined #openstack-nova | 00:33 | |
| *** masayukig_ is now known as masayukig | 00:33 | |
| *** dave-mccowan has joined #openstack-nova | 00:34 | |
| *** ijw has joined #openstack-nova | 00:35 | |
| *** ijw has quit IRC | 00:46 | |
| *** claudiub has quit IRC | 00:47 | |
| *** ducttape_ has joined #openstack-nova | 00:56 | |
| *** cheneydc has joined #openstack-nova | 00:58 | |
| *** ijw has joined #openstack-nova | 01:01 | |
| *** cheneydc has quit IRC | 01:02 | |
| *** efried has joined #openstack-nova | 01:06 | |
| *** efried1 has quit IRC | 01:07 | |
| *** ducttape_ has quit IRC | 01:08 | |
| *** gongysh has joined #openstack-nova | 01:11 | |
| *** kaisers_ has joined #openstack-nova | 01:11 | |
| mriedem | tjones: https://review.openstack.org/#/c/253666/ looks good but need to see vmware ci pass on it first | 01:12 |
|---|---|---|
| *** kaisers has quit IRC | 01:15 | |
| *** thorst has joined #openstack-nova | 01:19 | |
| *** EinstCrazy has joined #openstack-nova | 01:20 | |
| *** ijw has quit IRC | 01:21 | |
| *** EinstCra_ has joined #openstack-nova | 01:22 | |
| *** baoli_ has quit IRC | 01:22 | |
| *** thorst has quit IRC | 01:22 | |
| *** baoli has joined #openstack-nova | 01:23 | |
| *** thorst has joined #openstack-nova | 01:24 | |
| *** EinstCrazy has quit IRC | 01:25 | |
| *** baoli has quit IRC | 01:28 | |
| *** amotoki has joined #openstack-nova | 01:30 | |
| *** thorst has quit IRC | 01:31 | |
| *** Sukhdev has quit IRC | 01:33 | |
| *** ijw has joined #openstack-nova | 01:34 | |
| *** ijw has quit IRC | 01:40 | |
| *** thorst has joined #openstack-nova | 01:47 | |
| *** thorst has quit IRC | 01:48 | |
| *** ijw has joined #openstack-nova | 01:48 | |
| *** mtanino has joined #openstack-nova | 01:59 | |
| *** lambert has joined #openstack-nova | 02:03 | |
| *** ijw has quit IRC | 02:05 | |
| *** takedakn has joined #openstack-nova | 02:07 | |
| *** ducttape_ has joined #openstack-nova | 02:08 | |
| *** ducttape_ has quit IRC | 02:13 | |
| *** takedakn has quit IRC | 02:14 | |
| *** ducttape_ has joined #openstack-nova | 02:21 | |
| *** tiendc has joined #openstack-nova | 02:23 | |
| openstackgerrit | John Hua proposed openstack/nova: DNM: XenAPI: device tagging https://review.openstack.org/376078 | 02:26 |
| *** baoli has joined #openstack-nova | 02:29 | |
| *** stevemar has quit IRC | 02:30 | |
| *** stevemar has joined #openstack-nova | 02:31 | |
| *** lamt has joined #openstack-nova | 02:33 | |
| *** baoli has quit IRC | 02:33 | |
| *** sdake has quit IRC | 02:35 | |
| *** tuanla has joined #openstack-nova | 02:36 | |
| *** tbachman has quit IRC | 02:36 | |
| openstackgerrit | Yang Shengming proposed openstack/nova: [WIP]prevent resize down with ceph backend in libvirt driver https://review.openstack.org/325708 | 02:40 |
| *** amotoki has quit IRC | 02:41 | |
| *** ducttape_ has quit IRC | 02:44 | |
| openstackgerrit | Yang Shengming proposed openstack/nova: [WIP]prevent resize down with ceph backend in libvirt driver https://review.openstack.org/325708 | 02:45 |
| *** thorst has joined #openstack-nova | 02:48 | |
| *** GB21 has quit IRC | 02:56 | |
| *** thorst has quit IRC | 02:57 | |
| *** mriedem has quit IRC | 02:57 | |
| *** david-lyle has quit IRC | 03:04 | |
| *** raunak has joined #openstack-nova | 03:06 | |
| *** ijw has joined #openstack-nova | 03:06 | |
| *** ijw has quit IRC | 03:11 | |
| *** coreywright has quit IRC | 03:16 | |
| *** coreywright has joined #openstack-nova | 03:26 | |
| *** tbachman has joined #openstack-nova | 03:29 | |
| *** amotoki has joined #openstack-nova | 03:30 | |
| *** dave-mccowan has quit IRC | 03:35 | |
| *** brad[]` has quit IRC | 03:38 | |
| *** Sukhdev has joined #openstack-nova | 03:38 | |
| *** mtanino has quit IRC | 03:41 | |
| *** ducttape_ has joined #openstack-nova | 03:44 | |
| *** amotoki has quit IRC | 03:47 | |
| *** markvoelker has joined #openstack-nova | 03:48 | |
| *** ducttape_ has quit IRC | 03:49 | |
| *** markvoelker has quit IRC | 03:53 | |
| *** thorst has joined #openstack-nova | 03:55 | |
| *** gongysh has quit IRC | 03:57 | |
| *** thorst has quit IRC | 04:02 | |
| *** claudiub has joined #openstack-nova | 04:02 | |
| *** ijw has joined #openstack-nova | 04:07 | |
| *** gouthamr has quit IRC | 04:08 | |
| *** takashi has joined #openstack-nova | 04:09 | |
| *** rmcall has joined #openstack-nova | 04:10 | |
| *** rmcall has quit IRC | 04:12 | |
| *** rmcall has joined #openstack-nova | 04:13 | |
| *** ijw has quit IRC | 04:13 | |
| *** baoli has joined #openstack-nova | 04:17 | |
| *** tonytan_brb has joined #openstack-nova | 04:17 | |
| *** tonytan4ever has quit IRC | 04:19 | |
| *** baoli has quit IRC | 04:21 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/376125 | 04:24 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/376140 | 04:27 |
| *** prateek has joined #openstack-nova | 04:30 | |
| *** moshele has joined #openstack-nova | 04:36 | |
| *** raunak has quit IRC | 04:36 | |
| *** sdake has joined #openstack-nova | 04:41 | |
| *** ducttape_ has joined #openstack-nova | 04:45 | |
| *** sdake has quit IRC | 04:45 | |
| *** markvoelker has joined #openstack-nova | 04:49 | |
| *** ducttape_ has quit IRC | 04:50 | |
| *** ratailor has joined #openstack-nova | 04:51 | |
| *** markvoelker has quit IRC | 04:53 | |
| *** GB21 has joined #openstack-nova | 04:53 | |
| *** raunak has joined #openstack-nova | 04:55 | |
| *** coolsvap has joined #openstack-nova | 04:57 | |
| *** StevenK_ is now known as StevenK | 04:58 | |
| *** thorst has joined #openstack-nova | 04:59 | |
| *** thorst has quit IRC | 05:07 | |
| *** ecelik has joined #openstack-nova | 05:08 | |
| *** ijw has joined #openstack-nova | 05:09 | |
| *** paul-carlton2 has joined #openstack-nova | 05:10 | |
| *** shz has joined #openstack-nova | 05:11 | |
| *** TravT has joined #openstack-nova | 05:12 | |
| *** john51_ has joined #openstack-nova | 05:12 | |
| *** rhagarty__ has joined #openstack-nova | 05:13 | |
| *** fmccrthy_ has joined #openstack-nova | 05:13 | |
| *** NobodyCam_ has joined #openstack-nova | 05:13 | |
| *** betherly_ has joined #openstack-nova | 05:13 | |
| *** tomhambleton_ has joined #openstack-nova | 05:13 | |
| *** crinkle_ has joined #openstack-nova | 05:13 | |
| *** auggy_ has joined #openstack-nova | 05:14 | |
| *** liangy has joined #openstack-nova | 05:14 | |
| *** cfriesen has joined #openstack-nova | 05:14 | |
| *** tiendc has quit IRC | 05:15 | |
| *** DuncanT_ has joined #openstack-nova | 05:15 | |
| *** serverascode_ has joined #openstack-nova | 05:15 | |
| *** hwoarang_ has joined #openstack-nova | 05:15 | |
| *** Yingxin_ has joined #openstack-nova | 05:15 | |
| *** ijw has quit IRC | 05:15 | |
| *** hemanthm has joined #openstack-nova | 05:16 | |
| *** raorn_ has joined #openstack-nova | 05:16 | |
| *** Guest66676 has joined #openstack-nova | 05:16 | |
| *** igordcar1 has joined #openstack-nova | 05:16 | |
| *** doffm_ has joined #openstack-nova | 05:16 | |
| *** Hazelesque_ has joined #openstack-nova | 05:16 | |
| *** rosmaita_ has joined #openstack-nova | 05:16 | |
| *** yuntongjin has joined #openstack-nova | 05:16 | |
| *** dosaboy_ has joined #openstack-nova | 05:16 | |
| *** BobBall_ has joined #openstack-nova | 05:16 | |
| *** John341 has joined #openstack-nova | 05:16 | |
| *** luis5tb has joined #openstack-nova | 05:19 | |
| *** dgonzalez_ has joined #openstack-nova | 05:19 | |
| *** palexster1 has joined #openstack-nova | 05:19 | |
| *** egarbade- has joined #openstack-nova | 05:19 | |
| *** yhvh- has joined #openstack-nova | 05:19 | |
| *** jlk` has joined #openstack-nova | 05:19 | |
| *** another_larsks has joined #openstack-nova | 05:19 | |
| *** tanee_ has joined #openstack-nova | 05:19 | |
| *** swamireddy_ has joined #openstack-nova | 05:19 | |
| *** jlvillal_ has joined #openstack-nova | 05:19 | |
| *** mrodden_ has joined #openstack-nova | 05:19 | |
| *** alaski_ has joined #openstack-nova | 05:19 | |
| *** jklare_ has joined #openstack-nova | 05:20 | |
| *** niedbalski_ has joined #openstack-nova | 05:20 | |
| *** s1061123_ has joined #openstack-nova | 05:20 | |
| *** clayton_ has joined #openstack-nova | 05:20 | |
| *** cybojanek has quit IRC | 05:20 | |
| *** egarbade has quit IRC | 05:20 | |
| *** sileht has quit IRC | 05:20 | |
| *** doffm has quit IRC | 05:20 | |
| *** chenfei_ has quit IRC | 05:20 | |
| *** hwoarang has quit IRC | 05:21 | |
| *** john51 has quit IRC | 05:21 | |
| *** betherly has quit IRC | 05:21 | |
| *** manjeets has quit IRC | 05:21 | |
| *** igordcard has quit IRC | 05:21 | |
| *** serverascode has quit IRC | 05:21 | |
| *** raj_singh has quit IRC | 05:21 | |
| *** tanee has quit IRC | 05:21 | |
| *** mrhillsman has quit IRC | 05:21 | |
| *** dmellado has quit IRC | 05:21 | |
| *** niedbalski has quit IRC | 05:21 | |
| *** NobodyCam has quit IRC | 05:21 | |
| *** s1061123 has quit IRC | 05:21 | |
| *** ayoung has quit IRC | 05:21 | |
| *** jlvillal has quit IRC | 05:21 | |
| *** jlk has quit IRC | 05:21 | |
| *** ltomasbo has quit IRC | 05:21 | |
| *** clayton has quit IRC | 05:21 | |
| *** liangy_ has quit IRC | 05:21 | |
| *** mrodden has quit IRC | 05:21 | |
| *** Yingxin has quit IRC | 05:21 | |
| *** dosaboy has quit IRC | 05:21 | |
| *** cfriesen__ has quit IRC | 05:21 | |
| *** tomhambleton has quit IRC | 05:21 | |
| *** Hazelesque has quit IRC | 05:21 | |
| *** rhagarty_ has quit IRC | 05:21 | |
| *** shihanzhang has quit IRC | 05:21 | |
| *** hemanthm_ has quit IRC | 05:21 | |
| *** John341_ has quit IRC | 05:21 | |
| *** TravT_ has quit IRC | 05:21 | |
| *** auggy has quit IRC | 05:21 | |
| *** mnaser has quit IRC | 05:21 | |
| *** aunnam has quit IRC | 05:21 | |
| *** fmccrthy has quit IRC | 05:21 | |
| *** raorn has quit IRC | 05:21 | |
| *** rosmaita has quit IRC | 05:21 | |
| *** DuncanT has quit IRC | 05:21 | |
| *** larsks has quit IRC | 05:21 | |
| *** swamireddy has quit IRC | 05:21 | |
| *** kragniz has quit IRC | 05:21 | |
| *** jklare has quit IRC | 05:21 | |
| *** hogepodge has quit IRC | 05:21 | |
| *** trams has quit IRC | 05:21 | |
| *** palexster has quit IRC | 05:21 | |
| *** devananda|bbl has quit IRC | 05:21 | |
| *** crinkle has quit IRC | 05:21 | |
| *** lyarwood has quit IRC | 05:21 | |
| *** dgonzalez has quit IRC | 05:21 | |
| *** alaski has quit IRC | 05:21 | |
| *** snikitin has quit IRC | 05:21 | |
| *** rha has quit IRC | 05:21 | |
| *** Guest66666 has quit IRC | 05:21 | |
| *** jamiec has quit IRC | 05:21 | |
| *** yhvh has quit IRC | 05:21 | |
| *** SpamapS has quit IRC | 05:21 | |
| *** BobBall has quit IRC | 05:21 | |
| *** logan- has quit IRC | 05:21 | |
| *** mrodden_ is now known as mrodden | 05:21 | |
| *** Yingxin_ is now known as Yingxin | 05:21 | |
| *** dgonzalez_ is now known as dgonzalez | 05:21 | |
| *** lyarwood has joined #openstack-nova | 05:21 | |
| *** tonytan_brb has quit IRC | 05:21 | |
| *** manjeets has joined #openstack-nova | 05:21 | |
| *** clayton_ is now known as clayton | 05:21 | |
| *** rha_ has joined #openstack-nova | 05:21 | |
| *** chenfei has joined #openstack-nova | 05:21 | |
| *** logan- has joined #openstack-nova | 05:21 | |
| *** tiendc has joined #openstack-nova | 05:21 | |
| *** cybojanek has joined #openstack-nova | 05:21 | |
| *** tomhambleton_ is now known as tomhambleton | 05:22 | |
| *** dmellado has joined #openstack-nova | 05:22 | |
| *** NobodyCam_ is now known as NobodyCam | 05:22 | |
| *** raj_singh has joined #openstack-nova | 05:22 | |
| *** john51_ has quit IRC | 05:22 | |
| *** betherly_ is now known as betherly | 05:23 | |
| *** john51 has joined #openstack-nova | 05:23 | |
| *** jlvillal_ is now known as jlvillal | 05:23 | |
| *** fmccrthy_ is now known as fmccrthy | 05:23 | |
| *** jlvillal is now known as Guest97128 | 05:23 | |
| *** auggy_ is now known as auggy | 05:25 | |
| *** psachin has joined #openstack-nova | 05:26 | |
| *** ayoung has joined #openstack-nova | 05:27 | |
| *** mrhillsman has joined #openstack-nova | 05:27 | |
| *** aunnam has joined #openstack-nova | 05:28 | |
| *** devananda|bbl has joined #openstack-nova | 05:28 | |
| *** SpamapS has joined #openstack-nova | 05:28 | |
| *** snikitin has joined #openstack-nova | 05:28 | |
| *** sileht has joined #openstack-nova | 05:29 | |
| *** kragniz has joined #openstack-nova | 05:29 | |
| *** rcernin has joined #openstack-nova | 05:30 | |
| *** serverascode_ is now known as serverascode | 05:31 | |
| *** mnaser has joined #openstack-nova | 05:32 | |
| *** DuncanT_ is now known as DuncanT | 05:36 | |
| *** gongysh has joined #openstack-nova | 05:38 | |
| *** raunak has quit IRC | 05:39 | |
| *** yonglihe has joined #openstack-nova | 05:39 | |
| *** ljxiash has joined #openstack-nova | 05:41 | |
| *** ekuris has joined #openstack-nova | 05:46 | |
| *** sudipto_ has joined #openstack-nova | 05:47 | |
| *** sudswas__ has joined #openstack-nova | 05:47 | |
| *** ihrachys has joined #openstack-nova | 05:49 | |
| *** unicell1 has quit IRC | 05:49 | |
| *** sandanar has joined #openstack-nova | 05:49 | |
| *** amotoki has joined #openstack-nova | 05:50 | |
| *** unicell has joined #openstack-nova | 05:50 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Use method is_valid_cidr from oslo.utils https://review.openstack.org/376187 | 05:55 |
| *** haplo37_ has quit IRC | 05:56 | |
| *** paul-carlton2 has quit IRC | 05:57 | |
| *** haplo37_ has joined #openstack-nova | 05:58 | |
| *** ihrachys has quit IRC | 05:58 | |
| *** shihanzhang has joined #openstack-nova | 06:00 | |
| *** shz has quit IRC | 06:02 | |
| *** thorst has joined #openstack-nova | 06:05 | |
| *** nherciu has joined #openstack-nova | 06:06 | |
| *** mjura has joined #openstack-nova | 06:07 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:09 | |
| *** ijw has joined #openstack-nova | 06:12 | |
| *** thorst has quit IRC | 06:12 | |
| *** paul-carlton2 has joined #openstack-nova | 06:13 | |
| *** Oku_OS-away is now known as Oku_OS | 06:14 | |
| *** mjura has quit IRC | 06:14 | |
| *** ijw has quit IRC | 06:18 | |
| *** nkrinner_afk is now known as nkrinner | 06:18 | |
| *** yuntongjin has quit IRC | 06:21 | |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add Traits related table to the api database https://review.openstack.org/376198 | 06:21 |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199 | 06:21 |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add Traits API to placement service https://review.openstack.org/376200 | 06:21 |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add set/get_traits method to ResourceProvder object https://review.openstack.org/376201 | 06:21 |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add resource provider traits API https://review.openstack.org/376202 | 06:21 |
| *** mjura has joined #openstack-nova | 06:27 | |
| *** kevin has joined #openstack-nova | 06:28 | |
| *** kevin is now known as Guest69647 | 06:29 | |
| *** Guest69647 is now known as kevinz | 06:29 | |
| *** andreas_s has joined #openstack-nova | 06:29 | |
| *** ihrachys has joined #openstack-nova | 06:30 | |
| *** Sukhdev has quit IRC | 06:35 | |
| *** liverpooler has joined #openstack-nova | 06:37 | |
| *** yuntongjin has joined #openstack-nova | 06:40 | |
| prateek | alex_xu, https://review.openstack.org/#/c/356406/ , can i have your opinion on this ? | 06:43 |
| *** adisky has joined #openstack-nova | 06:43 | |
| *** syjulian has quit IRC | 06:44 | |
| openstackgerrit | Eric Brown proposed openstack/nova: Update man pages with latest version and dates https://review.openstack.org/325441 | 06:45 |
| *** pcaruana has joined #openstack-nova | 06:45 | |
| alex_xu | prateek: add it to my review list, will try to reach that later | 06:46 |
| prateek | alex_xu, ok, thanks | 06:46 |
| alex_xu | prateek: np | 06:47 |
| *** ttx has quit IRC | 06:48 | |
| *** ttx has joined #openstack-nova | 06:48 | |
| openstackgerrit | zhongshengping proposed openstack/nova: Use fnmatch from oslo.utils https://review.openstack.org/376221 | 06:50 |
| *** psachin has quit IRC | 06:51 | |
| *** syjulian has joined #openstack-nova | 06:56 | |
| *** psachin has joined #openstack-nova | 07:00 | |
| *** Kevin_Zheng has joined #openstack-nova | 07:03 | |
| *** CristinaPauna has joined #openstack-nova | 07:04 | |
| openstackgerrit | veena proposed openstack/nova: Adding hugepage and NUMA support check for aarch64 https://review.openstack.org/372304 | 07:07 |
| *** thorst has joined #openstack-nova | 07:10 | |
| *** acabot has joined #openstack-nova | 07:13 | |
| *** ijw has joined #openstack-nova | 07:14 | |
| *** gcb has joined #openstack-nova | 07:15 | |
| *** rha_ is now known as rha | 07:16 | |
| *** rha has joined #openstack-nova | 07:16 | |
| *** migi has joined #openstack-nova | 07:16 | |
| *** thorst has quit IRC | 07:17 | |
| *** ijw has quit IRC | 07:19 | |
| *** matrohon has joined #openstack-nova | 07:25 | |
| *** jaosorior has joined #openstack-nova | 07:29 | |
| *** jlanoux has joined #openstack-nova | 07:31 | |
| *** tiendc_ has joined #openstack-nova | 07:32 | |
| *** jklare_ has quit IRC | 07:32 | |
| *** jklare has joined #openstack-nova | 07:33 | |
| *** tiendc has quit IRC | 07:33 | |
| *** claudiub has quit IRC | 07:39 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Use method check_string_length from oslo.utils https://review.openstack.org/376251 | 07:40 |
| *** oanson has joined #openstack-nova | 07:40 | |
| *** Yingxin has quit IRC | 07:42 | |
| *** Yingxin has joined #openstack-nova | 07:43 | |
| *** pilgrimstack has joined #openstack-nova | 07:47 | |
| *** ljxiash has quit IRC | 07:51 | |
| *** liusheng has joined #openstack-nova | 07:53 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:01 | |
| *** palexster1 is now known as palexster | 08:01 | |
| *** Dinesh_Bhor has quit IRC | 08:03 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:04 | |
| *** paul-carlton2 has quit IRC | 08:05 | |
| *** yassine has joined #openstack-nova | 08:05 | |
| *** yassine is now known as Guest52852 | 08:06 | |
| *** sayalilunkad has joined #openstack-nova | 08:06 | |
| *** phuongnh has joined #openstack-nova | 08:07 | |
| openstackgerrit | John Hua proposed openstack/nova: WIP: XenAPI: device tagging https://review.openstack.org/333781 | 08:07 |
| openstackgerrit | John Hua proposed openstack/nova: DNM: XenAPI: device tagging https://review.openstack.org/376078 | 08:07 |
| *** ijw has joined #openstack-nova | 08:10 | |
| *** ljxiash_ has joined #openstack-nova | 08:12 | |
| *** trisq has joined #openstack-nova | 08:13 | |
| *** sayalilunkad has quit IRC | 08:14 | |
| *** sayalilunkad has joined #openstack-nova | 08:15 | |
| *** thorst has joined #openstack-nova | 08:15 | |
| *** ljxiash has joined #openstack-nova | 08:16 | |
| *** alexchadin has joined #openstack-nova | 08:17 | |
| *** ijw has quit IRC | 08:18 | |
| *** ljxiash__ has joined #openstack-nova | 08:20 | |
| *** ljxiash__ has quit IRC | 08:20 | |
| *** ljxiash_ has quit IRC | 08:20 | |
| *** tonytan4ever has joined #openstack-nova | 08:22 | |
| *** thorst has quit IRC | 08:22 | |
| *** ljxiash has quit IRC | 08:23 | |
| openstackgerrit | Nguyen Phuong An proposed openstack/nova: Remove last sentence https://review.openstack.org/365426 | 08:23 |
| *** ijw has joined #openstack-nova | 08:23 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: CellsV2 - Move quota tables to the API db https://review.openstack.org/372839 | 08:25 |
| *** tonytan4ever has quit IRC | 08:27 | |
| *** danpb has joined #openstack-nova | 08:30 | |
| *** TonyXu has quit IRC | 08:32 | |
| *** TonyXu has joined #openstack-nova | 08:33 | |
| *** rmart04 has joined #openstack-nova | 08:36 | |
| *** ijw has quit IRC | 08:37 | |
| *** ecelik has left #openstack-nova | 08:37 | |
| *** mamandle has joined #openstack-nova | 08:38 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Clean up instance on target node if evacuate fails https://review.openstack.org/375623 | 08:40 |
| veena | Hi bauzas, I would like to discuss about https://bugs.launchpad.net/nova/+bug/1623871 | 08:42 |
| openstack | Launchpad bug 1623871 in OpenStack Compute (nova) "Nova hugepage support does not include aarch64" [Wishlist,In progress] - Assigned to Veena (mveenasl) | 08:42 |
| *** derekh has joined #openstack-nova | 08:43 | |
| BobBall_ | johnthetubaguy: Note that mriedem has marked https://bugs.launchpad.net/nova/+bug/1616938 as high importance and we were discussing it as RC potential on Saturday. Fix is at https://review.openstack.org/#/c/375875/ and replaces the fix you were reviewing with Jianghua | 08:45 |
| openstack | Launchpad bug 1616938 in OpenStack Compute (nova) "XenAPI: failed to create image from volume backed instance with glance v2" [High,In progress] - Assigned to Matt Riedemann (mriedem) | 08:45 |
| johnthetubaguy | BobBall_: hmm, OK | 08:47 |
| *** paul-carlton2 has joined #openstack-nova | 08:49 | |
| *** ljxiash has joined #openstack-nova | 08:50 | |
| openstackgerrit | John Hua proposed openstack/nova: XenAPI: device tagging https://review.openstack.org/333781 | 08:51 |
| openstackgerrit | John Hua proposed openstack/nova: WIP: XenAPI: device tagging joint test https://review.openstack.org/376078 | 08:51 |
| *** tanee_ is now known as tanee | 08:51 | |
| *** baoli has joined #openstack-nova | 08:54 | |
| bauzas | veena: hello | 08:54 |
| bauzas | veena: sure, what'd you like to know ? | 08:54 |
| bauzas | veena: mmm, the real problem is that I discussed about the above with danpb and we agreed that it rather looks like a feature | 08:55 |
| veena | bauzas, Okay. | 08:56 |
| johnthetubaguy | BobBall_: did you discuss the caching at all? | 08:56 |
| veena | bauzas, I have tested it in my env with the fix and it is working fine. The details in pastebin are for testing the libvirt version. But I don't know how to provide the complete info of the test env | 08:57 |
| bauzas | veena: well, danpb provided his concerns in https://review.openstack.org/#/c/372304/2/nova/virt/libvirt/driver.py which I agree | 08:58 |
| openstackgerrit | Andrey Volkov proposed openstack/nova: Return build_requests instead of instances https://review.openstack.org/374267 | 08:58 |
| openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 08:58 |
| openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Make schedule_and_build_instances cell aware https://review.openstack.org/372645 | 08:58 |
| *** baoli has quit IRC | 08:59 | |
| *** edand has joined #openstack-nova | 09:01 | |
| BobBall_ | johnthetubaguy: no | 09:02 |
| veena | bauzas, okay, I'll try to address his comments and also see about how to go ahead if it is considered as a feature. Thanks | 09:03 |
| *** code-R has joined #openstack-nova | 09:04 | |
| *** dingboopt_ has joined #openstack-nova | 09:04 | |
| bauzas | veena: just tbh, that's because we want to make sure that if we accept the above, it means that Nova will be supporting that | 09:04 |
| bauzas | veena: we really want to make sure it works and it supports all the NUMA placement options | 09:05 |
| *** gszasz has joined #openstack-nova | 09:05 | |
| johnthetubaguy | bauzas: what about just adding a reno note about the fact its a feature? | 09:05 |
| bauzas | veena: that's why I think a blueprint is better, in order to make sure we all know it's adding stuff | 09:05 |
| bauzas | johnthetubaguy: that was one of my comments | 09:05 |
| johnthetubaguy | bauzas: ah, cool, I just wondering if we should do that instead of spec-less BPs | 09:06 |
| bauzas | johnthetubaguy: but I still think we shouldn't add more things we support without having at least a specless BP | 09:06 |
| johnthetubaguy | well, what does the BP give us, is my question I guess? | 09:06 |
| bauzas | johnthetubaguy: we basically agreed in the past that wishlist bugs are things we don't accept | 09:06 |
| johnthetubaguy | I know its what we would normally do, but I am just thinking out loud | 09:06 |
| johnthetubaguy | so stuff that needs discussion outside the code review, for sure | 09:07 |
| bauzas | johnthetubaguy: well, if that's a blueprint asking to be specless, that means we have people able to comment on the proposal during the nova meeting and follow-up later | 09:07 |
| *** code-R_ has joined #openstack-nova | 09:07 | |
| bauzas | johnthetubaguy: it creates more exposure to that feature, which I think is important given the fact that we're about to sign-off about us support aarch64 numa placement without having a 3rd-party CI correctly testing that :) | 09:08 |
| *** alexchadin has quit IRC | 09:08 | |
| johnthetubaguy | bauzas: no 3rd party CI is a good reason to push it for a discussion | 09:08 |
| *** alexchadin has joined #openstack-nova | 09:08 | |
| bauzas | johnthetubaguy: I'll restate my concerns in the bug report saying that I'd be by far less concerned whether it's a bugfix or a blueprint if we would have our tempest tests running on dedicated hardware | 09:09 |
| bauzas | ideally, the Intel 3rd-party CI tests I mean | 09:09 |
| bauzas | (I mean the Intel NFV CI) | 09:10 |
| *** Vijayendra__ has joined #openstack-nova | 09:10 | |
| *** code-R has quit IRC | 09:10 | |
| *** haplo37 has quit IRC | 09:10 | |
| *** haplo37 has joined #openstack-nova | 09:11 | |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add set/get_traits method to ResourceProvder object https://review.openstack.org/376201 | 09:12 |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add Traits API to placement service https://review.openstack.org/376200 | 09:12 |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add resource provider traits API https://review.openstack.org/376202 | 09:12 |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Add Trait and TraitList objects https://review.openstack.org/376199 | 09:12 |
| *** Guest94479 is now known as beagles | 09:13 | |
| *** sahid has joined #openstack-nova | 09:18 | |
| *** thorst has joined #openstack-nova | 09:20 | |
| *** jistr has quit IRC | 09:20 | |
| veena | bauzas, johnthetubaguy is there any specific test that needs to be passed for arch specific changes? should there be a CI running for aarch64 specific tests? | 09:23 |
| bauzas | veena: that's exactly the questions I'd like to see those discussed if you ask for a specless BP | 09:24 |
| bauzas | ie. how can we, as a team, make sure that we're shipping things we know that work | 09:24 |
| *** jlanoux has quit IRC | 09:25 | |
| bauzas | veena: at least adressing danpb's comments in the review pointing that your pastebien lacks of coverage is one necessary step | 09:25 |
| bauzas | veena: you should make sure that you're testing at least locally everything related to NUMA placement, incl. hugepages | 09:26 |
| *** thorst has quit IRC | 09:27 | |
| bauzas | veena: once you do that, then you could come up and say "okay, I tested and that works", and then we'll discuss how we can be sure that things you tested locally will always work, ie. making sure that we have some test coverage against any change that prevent us to merge regressions against your specific architecture | 09:27 |
| johnthetubaguy | right, think test anything you think could break because you are using aarch64, at a minimum | 09:27 |
| bauzas | veena: tl; dr: there are 2 concerns : #1 we need to make sure that aarch64 is fully supporting all NUMA options, and #2 is that aarch64 support of NUMA placement will never be broken because of something merged | 09:28 |
| bauzas | #1 can be achievable by you manually testing all NUMA options | 09:28 |
| bauzas | #2 is a bit more concerning me | 09:28 |
| Vijayendra__ | bauzas, Sure we can do that and in addition to that I am wondering if there are any tests already available which has considered this feature in x86 | 09:29 |
| danpb | bauzas: #1 is what i'm concerned about wrt merging the patch - just to be sure we've got min dependancies right | 09:29 |
| danpb | #2 requires 3rd party CI - but we didn't require that when we merge ppc64 support for this afaik, so requiring it for aarch64 would be unfair double-standard | 09:29 |
| bauzas | Vijayendra__: most of the NUMA placement possibilities are covered by a 3rd party CI called Intel NFV CI | 09:29 |
| bauzas | danpb: that's fair to say that, I agree | 09:30 |
| bauzas | danpb: I just want to point out that we could break things in the future if we don't have that | 09:30 |
| bauzas | danpb: and us need, as a team, to accept that risk | 09:30 |
| veena | bauzas, I see that Intel NFV CI has passed and there is +1 | 09:30 |
| bauzas | veena: but they didn't tested on aarch64 :) | 09:31 |
| Vijayendra__ | ok understood. Let us come back with proper test results and CI | 09:31 |
| bauzas | Vijayendra__: CI is not really an option now | 09:31 |
| bauzas | Vijayendra__: again, my point is that I'd like to have you testing if everything works first, and then come back to us so that we can agree with that mergign | 09:32 |
| bauzas | but IMHO that's just not something we can hide using a bug report | 09:32 |
| *** jlanoux has joined #openstack-nova | 09:32 | |
| bauzas | because it's not a bug, it's a lack of a different architecture support, which is different | 09:32 |
| bauzas | but I don't want to be pedantic, I just want to be pragmatic and say "please do the tests and come back to us" | 09:33 |
| bauzas | veena: ^ | 09:33 |
| danpb | imho immediate #1 priority is demonstrating that you've done real world tests | 09:36 |
| danpb | and providing information sufficient to prove that | 09:36 |
| *** andreykurilin_ has left #openstack-nova | 09:36 | |
| danpb | in this doc: http://docs.openstack.org/developer/nova/testing/libvirt-numa.html | 09:36 |
| danpb | we illustrate how to collect such data from nova DB tables to prove NUMA is being used correctly | 09:37 |
| danpb | eg, compute_nodes table to prove use of a host that has numa and huge pages enabled | 09:37 |
| danpb | instance_extra table to prove placement of guests with numa topology + huge pages in accordance with compute node capabilities | 09:38 |
| *** flaper87 has joined #openstack-nova | 09:39 | |
| *** flaper87 has quit IRC | 09:39 | |
| *** flaper87 has joined #openstack-nova | 09:39 | |
| *** alexchadin has quit IRC | 09:40 | |
| *** gongysh has quit IRC | 09:42 | |
| *** ijw has joined #openstack-nova | 09:43 | |
| *** zenoway has joined #openstack-nova | 09:47 | |
| *** zenoway has quit IRC | 09:48 | |
| *** zenoway has joined #openstack-nova | 09:48 | |
| *** ijw has quit IRC | 09:49 | |
| *** ducttape_ has joined #openstack-nova | 09:49 | |
| *** ljxiash has quit IRC | 09:49 | |
| *** ljxiash has joined #openstack-nova | 09:50 | |
| *** liuyulong has joined #openstack-nova | 09:53 | |
| *** ducttape_ has quit IRC | 09:54 | |
| *** daemontool has joined #openstack-nova | 09:54 | |
| *** ljxiash has quit IRC | 09:55 | |
| *** haplo37_ has quit IRC | 09:55 | |
| *** roeyc has joined #openstack-nova | 09:56 | |
| *** haplo37_ has joined #openstack-nova | 09:57 | |
| veena | bauzas, danpb, Thanks for the link, we will do the necessary testing and provide all the info. | 10:03 |
| *** pjm6 has joined #openstack-nova | 10:04 | |
| pjm6 | Hi there | 10:04 |
| pjm6 | anyone here use KVM with shared storage? | 10:04 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix uncaugth exception during soft reboot https://review.openstack.org/376350 | 10:04 |
| *** tiendc_ has quit IRC | 10:04 | |
| *** sdake has joined #openstack-nova | 10:06 | |
| *** yuntongjin has quit IRC | 10:07 | |
| *** mamandle has quit IRC | 10:09 | |
| *** claudiub has joined #openstack-nova | 10:11 | |
| *** sdague has joined #openstack-nova | 10:14 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Capture stdout in for test_wsgi:test_debug https://review.openstack.org/375397 | 10:14 |
| openstackgerrit | Sean Dague proposed openstack/nova: Clean up stdout/stderr leakage in cmd testing https://review.openstack.org/375414 | 10:14 |
| *** EinstCra_ has quit IRC | 10:16 | |
| *** brian_price has quit IRC | 10:16 | |
| *** mamandle has joined #openstack-nova | 10:17 | |
| *** aarefiev_ is now known as aarefiev | 10:17 | |
| *** code-R_ has quit IRC | 10:18 | |
| Vijayendra__ | bauzas, Sure. Thanks for the inputs | 10:19 |
| *** p01nt3r75 has joined #openstack-nova | 10:22 | |
| mamandle | @bauzas, need a review of - https://review.openstack.org/#/c/362794/3 | 10:22 |
| mamandle | can someone please review it? | 10:22 |
| *** sahid has quit IRC | 10:23 | |
| *** takashi has quit IRC | 10:24 | |
| *** efried has quit IRC | 10:24 | |
| *** thorst has joined #openstack-nova | 10:26 | |
| *** raorn_ is now known as raorn | 10:31 | |
| *** code-R has joined #openstack-nova | 10:32 | |
| *** jlanoux has quit IRC | 10:32 | |
| *** thorst has quit IRC | 10:33 | |
| *** nherciu_ has joined #openstack-nova | 10:35 | |
| *** tbachman has quit IRC | 10:36 | |
| *** nherciu has quit IRC | 10:38 | |
| *** baoli has joined #openstack-nova | 10:43 | |
| *** rodrigods has quit IRC | 10:43 | |
| *** rodrigods has joined #openstack-nova | 10:43 | |
| *** ijw has joined #openstack-nova | 10:45 | |
| *** dikonoor has joined #openstack-nova | 10:45 | |
| *** smatzek has joined #openstack-nova | 10:47 | |
| *** baoli has quit IRC | 10:48 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests https://review.openstack.org/278834 | 10:50 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: Do not check if folder already exists in vCenter https://review.openstack.org/376387 | 10:50 |
| *** ducttape_ has joined #openstack-nova | 10:50 | |
| *** tuanla has quit IRC | 10:50 | |
| *** ijw has quit IRC | 10:51 | |
| *** hoangcx has quit IRC | 10:54 | |
| *** code-R has quit IRC | 10:54 | |
| *** code-R has joined #openstack-nova | 10:54 | |
| *** ducttape_ has quit IRC | 10:55 | |
| *** haplo37_ has quit IRC | 10:56 | |
| *** dave-mccowan has joined #openstack-nova | 10:57 | |
| *** circ-user-P9gJZ has joined #openstack-nova | 10:58 | |
| *** haplo37_ has joined #openstack-nova | 10:59 | |
| *** jlanoux has joined #openstack-nova | 11:00 | |
| *** paul-carlton has joined #openstack-nova | 11:02 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds vNUMA implementation https://review.openstack.org/282407 | 11:03 |
| *** paul-carlton2 has quit IRC | 11:03 | |
| *** rtheis has joined #openstack-nova | 11:03 | |
| *** cdelatte has joined #openstack-nova | 11:04 | |
| *** sdake_ has joined #openstack-nova | 11:04 | |
| *** sdake has quit IRC | 11:04 | |
| *** alexchadin has joined #openstack-nova | 11:04 | |
| *** phuongnh has quit IRC | 11:06 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: Fix API doc for os-console-auth-tokens https://review.openstack.org/340969 | 11:09 |
| *** sulo_ is now known as sulo | 11:09 | |
| *** cdent has joined #openstack-nova | 11:09 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Capture stdout in for test_wsgi:test_debug https://review.openstack.org/375397 | 11:13 |
| openstackgerrit | Sean Dague proposed openstack/nova: Clean up stdout/stderr leakage in cmd testing https://review.openstack.org/375414 | 11:13 |
| *** mkoderer has joined #openstack-nova | 11:13 | |
| *** purplerbot has joined #openstack-nova | 11:17 | |
| *** gcb has quit IRC | 11:17 | |
| *** lambert has quit IRC | 11:18 | |
| *** lpetrut has joined #openstack-nova | 11:22 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova-specs: VMware: add support for live migration https://review.openstack.org/299207 | 11:22 |
| *** rmart04 has quit IRC | 11:23 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: numa: Fixes NUMA topology related unit tests https://review.openstack.org/278835 | 11:23 |
| openstackgerrit | Simona Iuliana Toader proposed openstack/nova-specs: Adds Hyper-V Shielded VMs spec https://review.openstack.org/274709 | 11:24 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: Fix API doc for os-console-auth-tokens https://review.openstack.org/340969 | 11:25 |
| *** bvanhav has joined #openstack-nova | 11:26 | |
| *** roeyc has quit IRC | 11:26 | |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] ensure that allow headers are native strings https://review.openstack.org/374800 | 11:33 |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] Allow both /placement and /placement/ to work https://review.openstack.org/374870 | 11:35 |
| openstackgerrit | Evgeny Antyshev proposed openstack/nova: Don't expect block job to exist after rebase https://review.openstack.org/375652 | 11:35 |
| *** sudswas__ has quit IRC | 11:38 | |
| *** sudipto_ has quit IRC | 11:38 | |
| *** gopenshaw has quit IRC | 11:41 | |
| *** cdent has quit IRC | 11:42 | |
| *** tanee has quit IRC | 11:42 | |
| *** sdake_ has quit IRC | 11:43 | |
| *** cdent has joined #openstack-nova | 11:46 | |
| *** gopenshaw has joined #openstack-nova | 11:47 | |
| *** tanee has joined #openstack-nova | 11:47 | |
| *** ijw has joined #openstack-nova | 11:47 | |
| *** gongysh has joined #openstack-nova | 11:47 | |
| *** alexchadin has quit IRC | 11:49 | |
| *** thorst has joined #openstack-nova | 11:49 | |
| *** ducttape_ has joined #openstack-nova | 11:51 | |
| prateek | sdague, did you get some time to write a spec for https://review.openstack.org/#/c/327706/ ? | 11:51 |
| sdague | prateek: I did not yet, and johnthetubaguy was looking at some other issues around security groups | 11:52 |
| prateek | sdague, ok | 11:52 |
| *** tonytan4ever has joined #openstack-nova | 11:54 | |
| *** ijw has quit IRC | 11:54 | |
| *** alaski_ is now known as alaski | 11:54 | |
| *** ducttape_ has quit IRC | 11:55 | |
| *** tonytan4ever has quit IRC | 11:58 | |
| *** ftersin has quit IRC | 11:59 | |
| *** jaypipes has joined #openstack-nova | 11:59 | |
| *** ftersin has joined #openstack-nova | 12:00 | |
| *** abramley has quit IRC | 12:00 | |
| *** haplo37 has quit IRC | 12:01 | |
| *** fnordahl has quit IRC | 12:01 | |
| *** fnordahl has joined #openstack-nova | 12:02 | |
| *** abramley has joined #openstack-nova | 12:06 | |
| *** GB21 has quit IRC | 12:06 | |
| *** sudswas__ has joined #openstack-nova | 12:06 | |
| *** sudipto_ has joined #openstack-nova | 12:06 | |
| *** ratailor has quit IRC | 12:07 | |
| *** dosaboy has joined #openstack-nova | 12:07 | |
| *** dosaboy has quit IRC | 12:09 | |
| *** haplo37 has joined #openstack-nova | 12:09 | |
| *** rosmaita_ is now known as rosmaita | 12:12 | |
| *** ducttape_ has joined #openstack-nova | 12:13 | |
| *** dosaboy_ has quit IRC | 12:14 | |
| *** kfarr has joined #openstack-nova | 12:15 | |
| *** PaulMurray has joined #openstack-nova | 12:18 | |
| *** GB21 has joined #openstack-nova | 12:19 | |
| *** markvoelker has joined #openstack-nova | 12:20 | |
| *** code-R has quit IRC | 12:21 | |
| *** dosaboy has joined #openstack-nova | 12:21 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/376125 | 12:27 |
| *** lamt has quit IRC | 12:27 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Renamed parameters name in config.py https://review.openstack.org/374262 | 12:28 |
| *** gouthamr has joined #openstack-nova | 12:29 | |
| *** mdrabe has joined #openstack-nova | 12:30 | |
| *** alee_ has quit IRC | 12:31 | |
| *** amotoki has quit IRC | 12:32 | |
| *** rfolco has joined #openstack-nova | 12:33 | |
| *** ducttape_ has quit IRC | 12:37 | |
| *** daemontool has quit IRC | 12:38 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: support user password settings in virtuozzo https://review.openstack.org/330643 | 12:38 |
| *** efried has joined #openstack-nova | 12:40 | |
| claudiub | johnthetubaguy: hi. I have a specless blueprint that can be approved. https://blueprints.launchpad.net/nova/+spec/hyper-v-block-device-mapping-support-ocata . Basically, this is the continuation of a blueprint that got partially completed in newton: https://blueprints.launchpad.net/nova/+spec/hyper-v-block-device-mapping-support . I only have 2 small patches left on it. Can you take a look pls? : | 12:41 |
| *** ljxiash has joined #openstack-nova | 12:41 | |
| *** markvoelker has quit IRC | 12:41 | |
| johnthetubaguy | claudiub: I will try take a peak at that in a bit | 12:42 |
| claudiub | k, thanks! | 12:42 |
| *** alexchadin has joined #openstack-nova | 12:42 | |
| *** edmondsw has joined #openstack-nova | 12:43 | |
| johnthetubaguy | claudiub: actually mriedem has the -2, on those, I would ping him on those ones | 12:43 |
| claudiub | cool, will do | 12:43 |
| *** daemontool has joined #openstack-nova | 12:44 | |
| *** mamandle has quit IRC | 12:44 | |
| *** ijw has joined #openstack-nova | 12:51 | |
| *** READ10 has joined #openstack-nova | 12:52 | |
| *** READ10 has quit IRC | 12:52 | |
| *** READ10 has joined #openstack-nova | 12:52 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Improving help text for xenapi_vmops_opts https://review.openstack.org/347085 | 12:53 |
| *** efried has left #openstack-nova | 12:54 | |
| *** efried has joined #openstack-nova | 12:55 | |
| *** ijw has quit IRC | 12:56 | |
| *** david-lyle has joined #openstack-nova | 12:56 | |
| *** markvoelker has joined #openstack-nova | 12:57 | |
| *** ducttape_ has joined #openstack-nova | 12:57 | |
| *** vishwanathj has joined #openstack-nova | 12:57 | |
| *** rlrossit has joined #openstack-nova | 12:57 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova-specs: Restore V3 Diagnostics info https://review.openstack.org/357884 | 12:57 |
| *** sdake has joined #openstack-nova | 12:57 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Recreate config drive on hard reboot https://review.openstack.org/376452 | 12:58 |
| *** ducttape_ has quit IRC | 12:58 | |
| *** xyang1 has joined #openstack-nova | 12:59 | |
| *** sudswas__ has quit IRC | 12:59 | |
| *** sudipto_ has quit IRC | 13:00 | |
| *** prateek has quit IRC | 13:00 | |
| *** andrewbogott has quit IRC | 13:01 | |
| *** andrewbogott has joined #openstack-nova | 13:01 | |
| *** daemontool has quit IRC | 13:01 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Recreate config drive on hard reboot https://review.openstack.org/376452 | 13:01 |
| *** umbSublime_ has quit IRC | 13:01 | |
| *** umbSublime_ has joined #openstack-nova | 13:01 | |
| *** takedakn has joined #openstack-nova | 13:02 | |
| *** xyang1 has quit IRC | 13:02 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: config options: Improve help for SPICE https://review.openstack.org/317046 | 13:02 |
| *** sdake_ has joined #openstack-nova | 13:03 | |
| *** daemontool has joined #openstack-nova | 13:03 | |
| *** zenoway has quit IRC | 13:04 | |
| *** trisq has quit IRC | 13:05 | |
| *** thorst is now known as thorst_afk | 13:05 | |
| *** I has joined #openstack-nova | 13:06 | |
| *** I is now known as Guest3090 | 13:06 | |
| *** sdake has quit IRC | 13:06 | |
| *** bhagyashri has joined #openstack-nova | 13:06 | |
| *** tbachman has joined #openstack-nova | 13:08 | |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] add a placement_aggregates table to api_db https://review.openstack.org/362863 | 13:08 |
| *** baoli has joined #openstack-nova | 13:09 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add {get_,set_}aggregates to objects.ResourceProvider https://review.openstack.org/355263 | 13:09 |
| *** ekuris has quit IRC | 13:11 | |
| *** gongysh has quit IRC | 13:12 | |
| *** xyang1 has joined #openstack-nova | 13:13 | |
| *** gongysh has joined #openstack-nova | 13:14 | |
| *** GB21 has quit IRC | 13:14 | |
| *** cleong has joined #openstack-nova | 13:15 | |
| *** code-R has joined #openstack-nova | 13:16 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add {get_,set_}aggregates to objects.ResourceProvider https://review.openstack.org/355263 | 13:16 |
| openstackgerrit | Chris Dent proposed openstack/nova: Implement get and set aggregates in the placement API https://review.openstack.org/357993 | 13:17 |
| *** vladikr has joined #openstack-nova | 13:18 | |
| *** code-R_ has joined #openstack-nova | 13:18 | |
| *** jaosorior has quit IRC | 13:19 | |
| *** jaosorior has joined #openstack-nova | 13:19 | |
| *** zenoway has joined #openstack-nova | 13:20 | |
| *** thorst_afk is now known as thorst | 13:20 | |
| *** claudiub has quit IRC | 13:21 | |
| *** code-R has quit IRC | 13:21 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Move notification_format and delete rpc.py https://review.openstack.org/349600 | 13:24 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: use proper context in libvirt driver unit test https://review.openstack.org/376473 | 13:24 |
| openstackgerrit | Chris Dent proposed openstack/nova: Implement get and set aggregates in the placement API https://review.openstack.org/357993 | 13:24 |
| *** amotoki has joined #openstack-nova | 13:26 | |
| *** lamt has joined #openstack-nova | 13:26 | |
| *** ekuris has joined #openstack-nova | 13:26 | |
| *** GB21 has joined #openstack-nova | 13:27 | |
| *** eharney has joined #openstack-nova | 13:27 | |
| *** mriedem has joined #openstack-nova | 13:28 | |
| *** ducttape_ has joined #openstack-nova | 13:29 | |
| *** ducttape_ has quit IRC | 13:30 | |
| *** smatzek has quit IRC | 13:31 | |
| *** tblakes has joined #openstack-nova | 13:31 | |
| mdbooth_ | snikitin: alex_xu is suggesting diagnostics is admin only. Is it really? I'd have thought this is information users would want. | 13:32 |
| *** ekuris has quit IRC | 13:32 | |
| alex_xu | mdbooth_: the current diagnostic API is admin-only | 13:34 |
| *** hshiina has joined #openstack-nova | 13:34 | |
| *** mdbooth_ is now known as mdbooth | 13:34 | |
| alex_xu | mdbooth: https://github.com/openstack/nova/blob/master/nova/policies/server_diagnostics.py#L28 | 13:34 |
| mdbooth | alex_xu: Is there any reason it needs to be? | 13:34 |
| alex_xu | mdbooth: so...that is why I think we should clear the use-case of this API | 13:35 |
| mdbooth | This sounds like something we could usefully expose to users. If that's somewhere we wanted to go we probably wouldn't want to expose disk paths. | 13:35 |
| mdbooth | alex_xu: +1 | 13:35 |
| * mdbooth assumed it was already exposed to users. | 13:35 | |
| sfinucan | Any have the URL for the most recent versions of docs? (i.e. post-Mitaka) | 13:36 |
| *** liverpooler has quit IRC | 13:36 | |
| *** Guest97128 is now known as jlvillal | 13:36 | |
| cdent | mriedem: if you're willing and able, could you release your -2 on https://review.openstack.org/#/c/362766/ | 13:37 |
| danpb | mdbooth: largely a historical accident i think, plus fact that the original API was totally ill-specified | 13:37 |
| alex_xu | mdbooth: the statistics of disk and nic of the guest shouldn't get from nova I think. I guess the celiometer is right place? | 13:37 |
| danpb | mdbooth: so not really nice to expose to end users | 13:37 |
| mdbooth | danpb: What's it for, then? | 13:37 |
| mdbooth | alex_xu: Where does ceilometer get it from? | 13:38 |
| mdbooth | Surely nova... | 13:38 |
| danpb | mdbooth: originally i think it was an adhoc hack for admins to find some perf data about mis-behaving vms | 13:38 |
| mriedem | sfinucan: which docs? | 13:38 |
| sfinucan | The most recent of these http://docs.openstack.org/mitaka/config-reference/compute/nova-conf.html | 13:38 |
| alex_xu | mdbooth: yes, from the virt driver, and put in the celiometer. I just think the Nova API isn't right API for user poll for those statistics | 13:38 |
| mriedem | mdbooth: ceilometer gets some info from libvirt directly on the ceilometer compute agent | 13:39 |
| mriedem | basically redundant with nova-compute getting them from libvirt | 13:39 |
| cdent | mdbooth: there's a lot of stuff that ceilometer gets from the virtdrivers directly because nova doesn't publish it itself. this is why there is a ceilomter compute agent | 13:39 |
| sfinucan | mriedem: I'm chasing the 'policy generator sphinx extension' to see if it's doable on my end... | 13:39 |
| cdent | (which runs on the compute nodes) | 13:39 |
| sfinucan | (but I don't know where the config one is currently being used :/) | 13:39 |
| mriedem | sfinucan: nova.conf.sample is generated in the nova devref | 13:39 |
| mriedem | which has the latest | 13:39 |
| cdent | mdbooth: there was discussion about making all the stuff that that agent gets available as notifications published by the nova-compute but it hasn't had time or energy slices | 13:40 |
| mriedem | sfinucan: http://docs.openstack.org/developer/nova/sample_config.html | 13:40 |
| mdbooth | So, in that case my suggestion of device tags is completely irrelevant | 13:40 |
| *** annegentle has joined #openstack-nova | 13:40 | |
| mdbooth | Device tags are for the user. If the info isn't being provided to the user, it has no purpose. | 13:40 |
| sfinucan | mriedem: Oh, I've that done (I think) | 13:40 |
| alex_xu | mdbooth: yea | 13:40 |
| *** sandanar has quit IRC | 13:40 | |
| mdbooth | alex_xu: Thanks. | 13:41 |
| mriedem | cdent: ok. i'm on the fence on whether or not we should have a specless blueprint for the separate placement db | 13:41 |
| alex_xu | mdbooth: np | 13:41 |
| *** alee_ has joined #openstack-nova | 13:42 | |
| mriedem | cdent: dropped the -2, but have a -1 for a specless bp | 13:43 |
| mriedem | since it's a feature add for deployments | 13:43 |
| cdent | mriedem: kewl | 13:43 |
| mriedem | and we'll want to turn it on in the placement job probably | 13:43 |
| cdent | thanks | 13:43 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: use proper context in libvirt driver unit test https://review.openstack.org/376473 | 13:43 |
| *** cnf has joined #openstack-nova | 13:44 | |
| cnf | hi | 13:44 |
| *** GB21 has quit IRC | 13:44 | |
| snikitin | alex_xu, mdbooth: So we will make that diagnostic info for admins only? | 13:44 |
| cnf | I am getting "IncompatibleObjectVersion: Version 1.3 of MigrationList is not supported" in my nova logs | 13:44 |
| *** jlk` is now known as jlk | 13:44 | |
| cnf | any hints on how to solve this? | 13:44 |
| *** rcarrillocruz has joined #openstack-nova | 13:44 | |
| *** jlk has quit IRC | 13:44 | |
| *** jlk has joined #openstack-nova | 13:44 | |
| mdbooth | snikitin: Sounds like that's the use case, no? | 13:44 |
| *** akshai has joined #openstack-nova | 13:44 | |
| *** alexchadin has quit IRC | 13:45 | |
| *** code-R_ has quit IRC | 13:45 | |
| *** Yingxin has quit IRC | 13:46 | |
| *** code-R has joined #openstack-nova | 13:46 | |
| *** sahid has joined #openstack-nova | 13:46 | |
| *** annegentle has quit IRC | 13:46 | |
| *** Yingxin has joined #openstack-nova | 13:47 | |
| *** mamandle has joined #openstack-nova | 13:47 | |
| snikitin | mdbooth: I'm ok with it. | 13:47 |
| *** brad[] has joined #openstack-nova | 13:47 | |
| *** sdake has joined #openstack-nova | 13:47 | |
| *** woodster_ has joined #openstack-nova | 13:48 | |
| mdbooth | snikitin: I think alex_xu makes an excellent point about being explicit about the use case. Sounds like I had completely misunderstood it. If it's genuinely a tool for admins to investigate errant instances, that's a completely different set of required data. | 13:48 |
| *** sdake_ has quit IRC | 13:50 | |
| *** sdake_ has joined #openstack-nova | 13:51 | |
| snikitin | mdbooth: As I understand from previous spec the initial purpose was just to standardise to response information of servers diagnostics, which was just a set of different data from each hypervisor. | 13:52 |
| *** ijw has joined #openstack-nova | 13:52 | |
| *** rlrossit has quit IRC | 13:52 | |
| mdbooth | snikitin: Sure, but that doesn't describe what the info is used for. | 13:52 |
| snikitin | this is the main use case | 13:53 |
| *** zenoway has quit IRC | 13:53 | |
| mdbooth | It's important to understand what the data's for. I didn't, which means my suggestions made no sense :) | 13:53 |
| *** burt has joined #openstack-nova | 13:53 | |
| *** zenoway has joined #openstack-nova | 13:54 | |
| *** sdake has quit IRC | 13:54 | |
| *** rlrossit_ has joined #openstack-nova | 13:54 | |
| *** tonytan4ever has joined #openstack-nova | 13:55 | |
| edleafe | Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt | 13:55 |
| mriedem | mdbooth: it's just debug data for admins | 13:56 |
| *** edand has quit IRC | 13:56 | |
| mriedem | predated ceilometer | 13:56 |
| *** smatzek has joined #openstack-nova | 13:56 | |
| mdbooth | mriedem: Yeah, that's what alex_xu is saying, makes sense. | 13:56 |
| *** mriedem is now known as mriedem_meeting | 13:56 | |
| *** rmart04 has joined #openstack-nova | 13:57 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: DNM https://review.openstack.org/375106 | 13:57 |
| *** mlavalle has joined #openstack-nova | 13:57 | |
| *** zenoway has quit IRC | 13:57 | |
| *** zenoway has joined #openstack-nova | 13:58 | |
| *** ijw has quit IRC | 13:58 | |
| *** amotoki_ has joined #openstack-nova | 13:58 | |
| *** tonytan4ever has quit IRC | 13:59 | |
| *** tonytan4ever has joined #openstack-nova | 13:59 | |
| edleafe | Scheduler meeting? Anyone? | 14:00 |
| *** GB21 has joined #openstack-nova | 14:01 | |
| *** ducttape_ has joined #openstack-nova | 14:01 | |
| *** amotoki has quit IRC | 14:01 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Add _rebuild_instance helper method https://review.openstack.org/376004 | 14:01 |
| *** cnf has left #openstack-nova | 14:03 | |
| *** prateek has joined #openstack-nova | 14:05 | |
| *** amotoki_ has quit IRC | 14:05 | |
| *** GB21 has quit IRC | 14:06 | |
| *** rlrossit_ has quit IRC | 14:06 | |
| *** haplo37 has quit IRC | 14:09 | |
| *** trisq has joined #openstack-nova | 14:10 | |
| *** takedakn has quit IRC | 14:10 | |
| *** haplo37 has joined #openstack-nova | 14:13 | |
| *** pilgrimstack has quit IRC | 14:13 | |
| *** pilgrimstack has joined #openstack-nova | 14:14 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Log parsing tool and post_test_hook script https://review.openstack.org/376531 | 14:14 |
| *** moshele has quit IRC | 14:17 | |
| *** artom has joined #openstack-nova | 14:19 | |
| artom | parkch3f963513 | 14:19 |
| BobBall_ | mriedem_meeting: FYI, depends-on works for our nova-network CI. Our Neutron CI is currently running in Rackspace cloud and has no access to the internal zuul server, so downloads the one change being tested from review.openstack.org (which is our 'old' way of testing things added before depends-on was introduced). We'll work on fixing that over the next few weeks so the neutron CI can also use d | 14:20 |
| BobBall_ | epends on | 14:20 |
| *** burgerk has joined #openstack-nova | 14:20 | |
| *** luis5tb is now known as ltomasbo | 14:20 | |
| *** cnf has joined #openstack-nova | 14:23 | |
| cnf | can anyone tell me why I get "Cannot update service status on host "compute02" since it is not registered." ? | 14:23 |
| *** ltomasbo is now known as luis5tb | 14:24 | |
| *** luis5tb is now known as ltomasbo | 14:25 | |
| *** beekneemech is now known as bnemec | 14:25 | |
| *** amotoki has joined #openstack-nova | 14:25 | |
| mdbooth | danpb: Grr http://libvirt.org/git/?p=libvirt-python.git;a=commit;h=cba2a84bc11026437bcc8ab0e3721973bb7bdff8 | 14:28 |
| mdbooth | ^^^ changed the semantics of BlockJobInfo from returning {} to returning None >:( | 14:28 |
| *** tanvirt has joined #openstack-nova | 14:31 | |
| *** annegentle has joined #openstack-nova | 14:31 | |
| *** adisky has quit IRC | 14:32 | |
| danpb | mdbooth: under what scenario ? | 14:32 |
| *** brian_price has joined #openstack-nova | 14:32 | |
| mdbooth | danpb: When the underlying libvirt call fails. | 14:33 |
| *** mtanino has joined #openstack-nova | 14:33 | |
| *** akshai has quit IRC | 14:33 | |
| danpb | hmm, if the underlying libvirt call fails i would have expected an exception | 14:33 |
| mdbooth | danpb: Don't change it again :) | 14:33 |
| mdbooth | Well, you didn't change it last time, but you know what I mean. | 14:34 |
| *** akshai has joined #openstack-nova | 14:34 | |
| *** moshele has joined #openstack-nova | 14:34 | |
| *** rlrossit has joined #openstack-nova | 14:34 | |
| mdbooth | danpb: Sorry, when the return value is zero. | 14:34 |
| * mdbooth is looking at 2 commits. | 14:35 | |
| *** anush_ has joined #openstack-nova | 14:35 | |
| *** akshai has quit IRC | 14:35 | |
| mdbooth | danpb: Hmm, maybe not. | 14:35 |
| danpb | hmm, that commit only changed when ret < 0 | 14:35 |
| danpb | the ret == 0 path returns {} before/after iiuc | 14:36 |
| mdbooth | danpb: Yep, you're right. | 14:36 |
| mdbooth | So: -1 -> None, 0 -> {}, >0 -> Populated | 14:37 |
| *** moshele has quit IRC | 14:39 | |
| *** dikonoor has quit IRC | 14:39 | |
| *** code-R has quit IRC | 14:40 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: doc: Integrate oslo_policy.sphinxpolicygen https://review.openstack.org/376547 | 14:42 |
| openstackgerrit | Hironori Shiina proposed openstack/nova: ironic: Add crash trigger dump support to ironic driver. https://review.openstack.org/376548 | 14:44 |
| mdbooth | danpb: Could you please look at https://review.openstack.org/#/c/365756/ for me? | 14:44 |
| *** dosaboy has quit IRC | 14:44 | |
| *** dosaboy has joined #openstack-nova | 14:46 | |
| *** dosaboy has quit IRC | 14:46 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hacking: Reword in the imperative https://review.openstack.org/363624 | 14:48 |
| *** nkrinner is now known as nkrinner_afk | 14:49 | |
| *** slaweq_ has joined #openstack-nova | 14:49 | |
| *** rlrossit has quit IRC | 14:49 | |
| *** akshai has joined #openstack-nova | 14:49 | |
| *** bnemec has quit IRC | 14:49 | |
| *** rrecio has joined #openstack-nova | 14:51 | |
| *** mriedem_meeting is now known as mriedem | 14:51 | |
| *** marst has joined #openstack-nova | 14:52 | |
| *** armax has joined #openstack-nova | 14:54 | |
| *** ijw has joined #openstack-nova | 14:55 | |
| *** rlrossit has joined #openstack-nova | 14:55 | |
| *** rrecio_ has joined #openstack-nova | 14:56 | |
| *** tonytan_brb has joined #openstack-nova | 14:56 | |
| *** ArxCruz has joined #openstack-nova | 14:56 | |
| *** daemontool_ has joined #openstack-nova | 14:57 | |
| *** hogepodge has joined #openstack-nova | 14:58 | |
| *** mamandle has quit IRC | 14:58 | |
| *** bnemec has joined #openstack-nova | 14:58 | |
| *** bdeschenes has joined #openstack-nova | 14:58 | |
| *** rrecio has quit IRC | 14:58 | |
| *** tonytan4ever has quit IRC | 14:59 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova-specs: Restore V3 Diagnostics info https://review.openstack.org/357884 | 14:59 |
| *** daemontool has quit IRC | 14:59 | |
| *** sdake_ has quit IRC | 14:59 | |
| *** akshai has quit IRC | 15:00 | |
| mriedem | BobBall_: yeah i saw that, thanks | 15:00 |
| *** _ducttape_ has joined #openstack-nova | 15:00 | |
| *** ijw has quit IRC | 15:00 | |
| mriedem | johnthetubaguy: did you have questions about https://review.openstack.org/#/c/375875/ ? | 15:01 |
| *** siva_krish has joined #openstack-nova | 15:01 | |
| *** gouthamr has quit IRC | 15:01 | |
| *** artom has quit IRC | 15:01 | |
| *** gouthamr has joined #openstack-nova | 15:01 | |
| *** akshai has joined #openstack-nova | 15:02 | |
| *** artom has joined #openstack-nova | 15:02 | |
| *** annegentle has quit IRC | 15:02 | |
| *** annegentle has joined #openstack-nova | 15:04 | |
| *** ducttape_ has quit IRC | 15:04 | |
| *** mjura has quit IRC | 15:04 | |
| *** jlvillal has quit IRC | 15:07 | |
| *** mvk has quit IRC | 15:07 | |
| *** jlvillal has joined #openstack-nova | 15:08 | |
| *** zz_dimtruck is now known as dimtruck | 15:08 | |
| *** psachin has quit IRC | 15:08 | |
| *** dimtruck is now known as zz_dimtruck | 15:09 | |
| *** TravT_ has joined #openstack-nova | 15:10 | |
| mriedem | jaypipes: sdague: dansmith: can i get a review on https://review.openstack.org/#/c/375875/ since we're going to want that in stable/newton for rc2, which is this week | 15:10 |
| burgerk | mriedem: Are you aware of any changes around the context being set on the thread in the spawn flow ? In a build from 0831 I was able to retrieve the context, but in a build from 0915 the context is bad or not set on the thread | 15:11 |
| *** gcb has joined #openstack-nova | 15:11 | |
| mriedem | burgerk: i'm not, alaski ^ sound familiar? | 15:11 |
| dansmith | mriedem: okay | 15:11 |
| *** sneti has joined #openstack-nova | 15:13 | |
| *** TravT has quit IRC | 15:13 | |
| *** claudiub has joined #openstack-nova | 15:14 | |
| *** TravT_ has quit IRC | 15:15 | |
| *** hemna_ has joined #openstack-nova | 15:15 | |
| *** hemna_ has quit IRC | 15:15 | |
| *** sdake has joined #openstack-nova | 15:15 | |
| *** rlrossit_ has joined #openstack-nova | 15:15 | |
| *** artom has quit IRC | 15:15 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add a post_test_hook.sh to nova and make it archive rows https://review.openstack.org/376567 | 15:16 |
| *** circ-user-P9gJZ has quit IRC | 15:17 | |
| *** kfarr has quit IRC | 15:17 | |
| sdague | mriedem: where is this called from? | 15:17 |
| mriedem | sdague: volume-backed snapshot | 15:18 |
| mriedem | sdague: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2685 | 15:18 |
| mriedem | sdague: this is what causes us to get into that disk_format block https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2622-L2627 | 15:19 |
| *** dosaboy has joined #openstack-nova | 15:19 | |
| *** rlrossit has quit IRC | 15:19 | |
| *** sandanar has joined #openstack-nova | 15:19 | |
| *** Guest3090 has quit IRC | 15:19 | |
| *** slaweq_ has quit IRC | 15:21 | |
| *** artom has joined #openstack-nova | 15:21 | |
| sdague | so... this is going to ask glance for the generic schema for disks? which we assume someone setup correctly. And that multi hypervisor environments have some reasonable answer to? | 15:22 |
| mriedem | yes | 15:22 |
| mriedem | there is a default schema | 15:22 |
| mriedem | but for xen they have to adjust it since xen doesn't support qcow2 | 15:22 |
| sdague | ok, what would the answer look like in a mixed libvirt / hyperv env? | 15:23 |
| mriedem | sdague: i guess you use a least common denominator, like raw | 15:23 |
| mriedem | sdague: this is what it looks like if it fails | 15:23 |
| mriedem | http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/81/375881/1/check/dsvm-tempest-nova-network/80d2212/logs/screen-n-api.txt.gz | 15:23 |
| *** prateek has quit IRC | 15:23 | |
| mriedem | TypeError: Unable to set 'disk_format' to 'qcow2'. Reason: 'qcow2' is not one of [None, u'ami', u'ari', u'aki', u'vhd', u'raw', u'iso'] | 15:23 |
| mriedem | ^ is the list of available disk formats for xen | 15:24 |
| *** shz has joined #openstack-nova | 15:24 | |
| sdague | I guess I'm surprised we don't just use instance.hypervisor_type | 15:24 |
| mriedem | because that's bleeding virt layer things in to the api, for one thing | 15:24 |
| mriedem | and it's a guess which could be wrong if your image schema is custom | 15:24 |
| jaypipes | mriedem: done | 15:26 |
| *** shihanzhang has quit IRC | 15:27 | |
| *** openstackstatus has joined #openstack-nova | 15:27 | |
| *** ChanServ sets mode: +v openstackstatus | 15:27 | |
| *** toure has quit IRC | 15:27 | |
| *** toure has joined #openstack-nova | 15:28 | |
| mriedem | jaypipes: thanks | 15:29 |
| *** TravT has joined #openstack-nova | 15:29 | |
| *** burt has quit IRC | 15:31 | |
| sdague | mriedem: ok, comments left which might be useful dealing with issues later | 15:32 |
| sdague | +2, but if someone else wants to look, I left the +A off for now | 15:32 |
| alaski | burgerk: mriedem I'm not familiar with any recent changes. I fixed an issue like that in 2015 https://review.openstack.org/#/c/171299/8 | 15:32 |
| alaski | setting the context on the thread local store happens behind the scenes somewhere, maybe on context init? I'd have to refresh myself | 15:33 |
| *** paul-carlton has quit IRC | 15:34 | |
| *** sneti has quit IRC | 15:34 | |
| *** harlowja_at_home has joined #openstack-nova | 15:36 | |
| *** nkrinner_afk is now known as nkrinner | 15:36 | |
| *** mkoderer has quit IRC | 15:39 | |
| -openstackstatus- NOTICE: Earlier job failures for "zuul-cloner: error: too few arguments" should now be solved, and can safely be rechecked | 15:42 | |
| *** hshiina has quit IRC | 15:44 | |
| *** akshai has quit IRC | 15:44 | |
| *** rlrossit_ has quit IRC | 15:44 | |
| *** rlrossit has joined #openstack-nova | 15:47 | |
| -openstackstatus- NOTICE: Launchpad SSO is offline, preventing login to https://review.openstack.org/, https://wiki.openstack.org/ and many other sites; no ETA has been provided by the LP admin team | 15:47 | |
| *** ChanServ changes topic to "Launchpad SSO is offline, preventing login to https://review.openstack.org/, https://wiki.openstack.org/ and many other sites; no ETA has been provided by the LP admin team" | 15:47 | |
| *** unicell1 has joined #openstack-nova | 15:48 | |
| *** unicell has quit IRC | 15:49 | |
| *** rcernin has quit IRC | 15:52 | |
| *** nherciu has joined #openstack-nova | 15:53 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Fix database poison warnings, part 19 https://review.openstack.org/376607 | 15:53 |
| diana_clarke | ^ Sadly, 42 more database poison warning slipped in since I last fixed them all. | 15:54 |
| *** annegentle has quit IRC | 15:55 | |
| *** browne has joined #openstack-nova | 15:55 | |
| diana_clarke | (so there are 5 more patches with database warning fixes up for review, and at least 3 more to come) | 15:55 |
| claudiub | mriedem: hi. the blueprint for https://review.openstack.org/#/c/282407/ has been reapproved. can you drop your -2? | 15:55 |
| *** nherciu_ has quit IRC | 15:55 | |
| claudiub | although... it seems launchpad is not working.. :/ | 15:55 |
| *** dosaboy has quit IRC | 15:56 | |
| *** dosaboy has joined #openstack-nova | 15:56 | |
| mriedem | yeah LP is down | 15:56 |
| *** ijw has joined #openstack-nova | 15:57 | |
| mriedem | seems back | 15:57 |
| *** akshai has joined #openstack-nova | 15:57 | |
| *** annegentle has joined #openstack-nova | 15:57 | |
| claudiub | ty :) | 15:57 |
| *** Sukhdev has joined #openstack-nova | 15:57 | |
| bauzas | see topic :) | 15:58 |
| *** sandanar_ has joined #openstack-nova | 15:58 | |
| claudiub | oh yeah. :) | 15:58 |
| bauzas | aaaaaand yay, it works | 15:58 |
| claudiub | fun times | 15:58 |
| bauzas | dansmith: FWIW, I had hard time this week-end figuring out a way to know whether the RequestSpec migration is all done for an operator | 16:00 |
| *** sandanar has quit IRC | 16:00 | |
| bauzas | dansmith: given we don't delete (yet) the Spec object, it's not possible to check that unless to verify each Spec | 16:01 |
| bauzas | O(n) | 16:01 |
| bauzas | dansmith: so, I think the only way to see that is whether we have a marker | 16:01 |
| alaski | bauzas: it's not made any easier by being in two dbs | 16:01 |
| *** hshiina has joined #openstack-nova | 16:02 | |
| bauzas | dansmith: if we have a marker, then the migration is not done, and if we don't have a marker, we just need to verify the first instance to see if there is Spec record for it | 16:02 |
| dansmith | bauzas: yeah, although I kinda dislike that approach | 16:02 |
| *** p01nt3r75 has quit IRC | 16:02 | |
| *** ijw has quit IRC | 16:02 | |
| dansmith | bauzas: maybe we just leave it in there for a while longer before we add it to the blocker anyway | 16:02 |
| bauzas | dansmith: fine by me then | 16:03 |
| bauzas | dansmith: I'm writing a patch anyway for deleting the old Specs | 16:03 |
| dansmith | okay | 16:03 |
| *** lucasagomes is now known as lucas-hungry | 16:03 | |
| alaski | can we just rely on ordering here? If the migration after that one is done then the req_spec is migrated. Although I guess due to the limit command that won't work, so nvm | 16:03 |
| bauzas | dansmith: alaski: FWIW https://review.openstack.org/#/c/376382/ | 16:03 |
| *** rmcadams has quit IRC | 16:03 | |
| dansmith | alaski: seems too risky to assume too much | 16:04 |
| *** mriedem is now known as mriedem_away | 16:04 | |
| bauzas | alaski: tbh, I think that once we have a db migration for deleting all the instances, we could just block the next release by checking between the number of instances and the number of spec objecrts | 16:05 |
| dansmith | alaski: if you still have the branch handy for this: https://review.openstack.org/#/c/376537/ | 16:05 |
| dansmith | alaski: can you add this: Depends-On: I16b2e00eede6af455cb74ca4e6ca951d56fdbcbc | 16:05 |
| bauzas | alaski: that's a bit racy tho given we have 2 DBs | 16:05 |
| alaski | dansmith: yeah. since users can specify a max count per run it would be possible to finish subsequent migrations without fully migrating req-specs | 16:05 |
| *** rmcadams has joined #openstack-nova | 16:05 | |
| dansmith | alaski: I would, but.. gerrit | 16:05 |
| bauzas | gerrit is back for the sake of god | 16:05 |
| alaski | dansmith: yep, one sec | 16:05 |
| dansmith | bauzas: no SSO | 16:05 |
| dansmith | ooh, there it is | 16:06 |
| bauzas | dansmith: WFM since 5 mins | 16:06 |
| dansmith | didn't work for me a minute ago | 16:06 |
| bauzas | dansmith: heh, you're probably too far from the Launchpad site, then :p | 16:06 |
| dansmith | alaski: actually, I wonder if I'll conflict with whatever you added | 16:06 |
| dansmith | oh, no | 16:07 |
| *** andreas_s has quit IRC | 16:08 | |
| alaski | gah, my dependency is messed up from splitting that out | 16:08 |
| *** I has joined #openstack-nova | 16:08 | |
| *** I has quit IRC | 16:08 | |
| dansmith | alaski: I'm confused.. where is that tools/post_test_hook.sh added? | 16:08 |
| *** mriedem_away has quit IRC | 16:08 | |
| *** mriedem has joined #openstack-nova | 16:08 | |
| alaski | forgot to git-add :( | 16:08 |
| alaski | will be up in a sec | 16:08 |
| dansmith | alaski: heh, okay | 16:08 |
| *** mriedem is now known as mriedem_away | 16:08 | |
| *** sneti has joined #openstack-nova | 16:09 | |
| *** sudipto has joined #openstack-nova | 16:09 | |
| *** sudipto_ has joined #openstack-nova | 16:09 | |
| *** Swami has joined #openstack-nova | 16:09 | |
| dansmith | alaski: so I actually want to merge mine.. are you going to un-DNM that one at some point? | 16:09 |
| *** ljxiash has quit IRC | 16:09 | |
| *** Sukhdev has quit IRC | 16:09 | |
| *** pilgrimstack has quit IRC | 16:09 | |
| dansmith | alaski: and if it's going to be a while, maybe I could go underneath your DNM one? | 16:09 |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Newton_Release_Schedule" | 16:09 | |
| -openstackstatus- NOTICE: Launchpad SSO logins are confirmed working correctly again | 16:10 | |
| alaski | dansmith: I'm not depending on the DNM one, I just need to abandon that | 16:10 |
| *** _ducttape_ has quit IRC | 16:10 | |
| dansmith | alaski: you were.. I'm confused | 16:10 |
| *** ducttape_ has joined #openstack-nova | 16:11 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Log parsing tool and post_test_hook script https://review.openstack.org/376531 | 16:12 |
| alaski | okay, that's what I was intending | 16:12 |
| alaski | dansmith: If I had that setup it was a mistake. But I'm also totally fine with having the project-config change depend on your patch and then having my Nova change depend on yours | 16:13 |
| *** david-lyle has quit IRC | 16:13 | |
| *** david-lyle has joined #openstack-nova | 16:14 | |
| *** anush_ has quit IRC | 16:14 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 16:14 |
| *** nic has joined #openstack-nova | 16:15 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add a post-test-hook to run the archive command https://review.openstack.org/376567 | 16:15 |
| dansmith | alaski: put mine atop yours ^ | 16:15 |
| *** raunak has joined #openstack-nova | 16:16 | |
| alaski | dansmith: cool | 16:16 |
| *** daemontool_ has quit IRC | 16:17 | |
| *** diablo_rojo has joined #openstack-nova | 16:17 | |
| *** dikonoor has joined #openstack-nova | 16:17 | |
| alaski | bauzas: I'll take a look at the req_spec patch in a bit, gotta talk to a lawyer about wills here shortly. for determing req_spec migration status I think we can wait until the cellsv2 migrations have run and then match instance_mappings with req_specs | 16:18 |
| mdbooth | danpb: Appreciate you got interrupted, but are you able to validate https://review.openstack.org/#/c/365756/ for me? | 16:18 |
| bauzas | alaski: no worries, take your time, my own afternoon was fully blown by a 1.5-hour awaitening for a 5-min doctor's appointment | 16:18 |
| alaski | bauzas: I hate that. better leave a bad yelp review :) | 16:19 |
| bauzas | alaski: heh, I wish I could | 16:20 |
| bauzas | alaski: but that's forbidden by law | 16:20 |
| dansmith | lol | 16:20 |
| bauzas | if you have some doctor that suck, or some teacher that sucks, then keep your mouth shut and don't stop paying him or he will sue you | 16:20 |
| *** artom has quit IRC | 16:21 | |
| *** sneti has quit IRC | 16:23 | |
| bauzas | oh my bad, I'm surprised, I could write a review actually | 16:24 |
| bauzas | nothing prevents me to do so | 16:24 |
| bauzas | \o/ | 16:24 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Clean up instance on target node if evacuate fails https://review.openstack.org/375623 | 16:24 |
| bauzas | that's just, I guess, people are not used to do so | 16:24 |
| *** zenoway has quit IRC | 16:24 | |
| *** amotoki has quit IRC | 16:24 | |
| * bauzas goes visiting Yelp :p | 16:24 | |
| *** zenoway has joined #openstack-nova | 16:25 | |
| sdague | alaski: I put a comment on that log analyzer patch, it seems a little odd to me to put that in the nova tree | 16:25 |
| *** lamt has quit IRC | 16:25 | |
| *** sahid has quit IRC | 16:26 | |
| *** lamt has joined #openstack-nova | 16:27 | |
| *** armax has quit IRC | 16:28 | |
| *** matrohon has quit IRC | 16:28 | |
| *** zenoway has quit IRC | 16:29 | |
| *** pcaruana has quit IRC | 16:29 | |
| *** sneti has joined #openstack-nova | 16:29 | |
| *** rmart04 has quit IRC | 16:30 | |
| *** jheroux has joined #openstack-nova | 16:31 | |
| *** tbachman_ has joined #openstack-nova | 16:33 | |
| *** jlanoux has quit IRC | 16:33 | |
| *** sneti has quit IRC | 16:33 | |
| *** tbachman has quit IRC | 16:33 | |
| *** tbachman_ is now known as tbachman | 16:33 | |
| *** krtaylor has joined #openstack-nova | 16:34 | |
| *** nkrinner is now known as nkrinner_afk | 16:39 | |
| *** harlowja_at_home has quit IRC | 16:39 | |
| *** unicell1 has quit IRC | 16:40 | |
| *** tbachman_ has joined #openstack-nova | 16:41 | |
| *** tbachman has quit IRC | 16:41 | |
| *** tbachman_ is now known as tbachman | 16:41 | |
| *** Apoorva has joined #openstack-nova | 16:41 | |
| *** ijw has joined #openstack-nova | 16:43 | |
| *** mriedem_away has quit IRC | 16:44 | |
| *** rama_y has joined #openstack-nova | 16:46 | |
| *** gcb has quit IRC | 16:47 | |
| *** ijw has quit IRC | 16:48 | |
| *** danpb has quit IRC | 16:52 | |
| *** akshai has quit IRC | 16:52 | |
| *** akshai has joined #openstack-nova | 16:53 | |
| *** derekh has quit IRC | 16:53 | |
| *** lamt has quit IRC | 16:54 | |
| *** erlon has joined #openstack-nova | 16:56 | |
| *** lucas-hungry is now known as lucasagomes | 16:57 | |
| *** akshai has quit IRC | 16:58 | |
| *** anush_ has joined #openstack-nova | 16:59 | |
| jaypipes | dansmith: when incrementing the version of an object, do we still need to increment the version of the corresponding list object, or is that no longer necessary? | 16:59 |
| *** ijw has joined #openstack-nova | 16:59 | |
| *** akshai has joined #openstack-nova | 17:00 | |
| *** xavvior has joined #openstack-nova | 17:01 | |
| dansmith | jaypipes: nossir | 17:02 |
| *** artom has joined #openstack-nova | 17:03 | |
| jaypipes | dansmith: that's what I thought. thx :) | 17:03 |
| *** tbachman has quit IRC | 17:05 | |
| *** paul-carlton2 has joined #openstack-nova | 17:05 | |
| *** ijw has quit IRC | 17:05 | |
| *** burt has joined #openstack-nova | 17:05 | |
| *** sneti has joined #openstack-nova | 17:06 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Move notification_format and delete rpc.py https://review.openstack.org/349600 | 17:07 |
| *** jaosorior has quit IRC | 17:07 | |
| *** tbachman has joined #openstack-nova | 17:07 | |
| *** lpetrut has quit IRC | 17:08 | |
| *** gszasz has quit IRC | 17:09 | |
| *** devananda|bbl is now known as devananda | 17:11 | |
| kashyap | dansmith: When you're about, I'm debugging a "weird" issue where a Cinder volume being listed *twice* (under "os-extended-volumes:volumes_attached") when you do a `nova show $INSTANCE_ID`. | 17:12 |
| kashyap | dansmith: So, I tried to reproduce it, after talking to a Cinder guy, this way: | 17:12 |
| kashyap | W/ Git-master - https://kashyapc.fedorapeople.org/virt/openstack/test-reset-cinder-volume-state.txt | 17:12 |
| kashyap | Note: I wrote the test description properly. Not a "dump of output" :P | 17:13 |
| dansmith | kashyap: right, so reset-state eliminates the double-attach check we have | 17:13 |
| dansmith | kashyap: so I would say "that is working as designed" | 17:13 |
| *** xavvior has quit IRC | 17:13 | |
| *** paul-carlton2 has quit IRC | 17:13 | |
| dansmith | or "you removed the brakes and the car crashed. congrats." | 17:13 |
| *** armax has joined #openstack-nova | 17:13 | |
| kashyap | dansmith: Yeah, yeah, let me finish what I have here :-) | 17:13 |
| kashyap | dansmith: First, you're right there | 17:13 |
| *** paul-carlton2 has joined #openstack-nova | 17:13 | |
| *** Guest68174 is now known as med_ | 17:14 | |
| *** med_ has quit IRC | 17:14 | |
| *** med_ has joined #openstack-nova | 17:14 | |
| kashyap | dansmith: So, in that above test, Nova only would a new ID if the connection_info has changed | 17:14 |
| *** tbachman_ has joined #openstack-nova | 17:14 | |
| kashyap | Like, migrating to a _new_ Cinder backend (e.g. NFS -> LVM, or vice-versa) | 17:14 |
| *** tbachman has quit IRC | 17:14 | |
| *** tbachman_ is now known as tbachman | 17:14 | |
| dansmith | kashyap: hmm? | 17:15 |
| kashyap | dansmith: Let me try to reprhase | 17:15 |
| *** unicell has joined #openstack-nova | 17:16 | |
| kashyap | I mean: Nova would add a new (?) *volume* ID, if the flow was something like: attach volume, change status using `reset-state`, migrate, then, re-attach | 17:16 |
| kashyap | This is speculation. I should come back (post-dinner) after doing a proper test. | 17:16 |
| dansmith | I still don't understand | 17:17 |
| dansmith | nova doesn't ever add volume_ids | 17:17 |
| dansmith | or do you mean another attachment? | 17:17 |
| *** PaulMurray has quit IRC | 17:17 | |
| *** ducttape_ has quit IRC | 17:18 | |
| kashyap | dansmith: Yes, a new entry to the list of attached volumes | 17:18 |
| kashyap | A duplicate ID | 17:18 |
| dansmith | kashyap: you're saying it only lets you do that if you migrate before attempting an attach? | 17:18 |
| *** ArxCruz has quit IRC | 17:18 | |
| *** ArxCruz has joined #openstack-nova | 17:19 | |
| kashyap | dansmith: Yep, possibly, I mean | 17:19 |
| kashyap | (Because I haven't tested) | 17:19 |
| *** ArxCruz has quit IRC | 17:19 | |
| dansmith | kashyap: I wouldn't think the migrate would matter | 17:19 |
| *** ArxCruz has joined #openstack-nova | 17:19 | |
| dansmith | kashyap: but either way, we can't really do any more sanity checking here I don't think, because with multi-attach, you will be able to attach one volume to two insances, | 17:19 |
| *** ArxCruz has quit IRC | 17:20 | |
| dansmith | so we could potentially check that you're not re-attaching to the same instance, but the whole premise is invalid anyway, so I don't think it's worth the extra db traffic | 17:20 |
| *** ArxCruz has joined #openstack-nova | 17:20 | |
| kashyap | dansmith: I see. Okay, this is not super critical. | 17:20 |
| *** ArxCruz has quit IRC | 17:21 | |
| *** ArxCruz has joined #openstack-nova | 17:21 | |
| *** ArxCruz has quit IRC | 17:21 | |
| *** ArxCruz has joined #openstack-nova | 17:22 | |
| *** ArxCruz has quit IRC | 17:22 | |
| *** ArxCruz has joined #openstack-nova | 17:23 | |
| *** ArxCruz has quit IRC | 17:23 | |
| *** ArxCruz has joined #openstack-nova | 17:23 | |
| *** paul-carlton2 has quit IRC | 17:24 | |
| kashyap | hemanthm: Are you confusing this bug with a different bug? https://bugs.launchpad.net/ossa/+bug/1449062 | 17:24 |
| openstack | Launchpad bug 1449062 in Glance "qemu-img calls need to be restricted by ulimit (CVE-2015-5162)" [High,In progress] - Assigned to Hemanth Makkapati (hemanth-makkapati) | 17:24 |
| *** ArxCruz has quit IRC | 17:24 | |
| *** ArxCruz has joined #openstack-nova | 17:24 | |
| *** ArxCruz has quit IRC | 17:25 | |
| kashyap | hemanthm: You're talking about format disclosure bug (which has a *different* CVE and was discussed before), the above fixes a *different* issue of CPU / and memory limits. | 17:25 |
| *** ArxCruz has joined #openstack-nova | 17:25 | |
| kashyap | fungi: ^ (Re: your question on the above) | 17:25 |
| *** ArxCruz has quit IRC | 17:26 | |
| fungi | thanks | 17:26 |
| *** ArxCruz has joined #openstack-nova | 17:26 | |
| hemanthm | kashyap: looking | 17:26 |
| *** ArxCruz has quit IRC | 17:26 | |
| *** rama_y has quit IRC | 17:26 | |
| *** ArxCruz has joined #openstack-nova | 17:27 | |
| hemanthm | kashyap: can you link me to the other bug | 17:27 |
| kashyap | hemanthm: If you indeed mixed it up with https://bugs.launchpad.net/nova/+bug/1415087, then please cofirm on the above ulimit bug | 17:27 |
| openstack | Launchpad bug 1415087 in OpenStack Security Advisory "[OSSA 2015-011] Format-guessing and file disclosure in image convert (CVE-2015-1850, CVE-2015-1851)" [High,Fix released] - Assigned to Tristan Cacqueray (tristan-cacqueray) | 17:27 |
| kashyap | The above is the other bug. | 17:27 |
| *** ArxCruz has quit IRC | 17:27 | |
| *** ArxCruz has joined #openstack-nova | 17:27 | |
| *** rmart04 has joined #openstack-nova | 17:28 | |
| hemanthm | oh well, I didn't know such a bug existed | 17:28 |
| *** ArxCruz has quit IRC | 17:28 | |
| *** ArxCruz has joined #openstack-nova | 17:28 | |
| kashyap | hemanthm: So, you did mix up? Just trying to confirm | 17:28 |
| *** ArxCruz has quit IRC | 17:29 | |
| *** jdillaman has quit IRC | 17:29 | |
| *** ArxCruz has joined #openstack-nova | 17:29 | |
| *** ArxCruz has quit IRC | 17:30 | |
| *** harlowja has joined #openstack-nova | 17:30 | |
| kashyap | Okay, I'll clarify there that you mixed up. (Sigh, that bug already has 75 comments, last thing it needs is more comments that are not relevant there.) | 17:30 |
| *** ArxCruz has joined #openstack-nova | 17:30 | |
| *** ArxCruz has quit IRC | 17:30 | |
| *** ArxCruz has joined #openstack-nova | 17:31 | |
| *** ArxCruz has quit IRC | 17:31 | |
| *** ArxCruz has joined #openstack-nova | 17:31 | |
| *** dikonoor has quit IRC | 17:32 | |
| *** bvanhav has quit IRC | 17:32 | |
| *** ArxCruz has quit IRC | 17:32 | |
| *** ArxCruz has joined #openstack-nova | 17:32 | |
| *** ArxCruz has quit IRC | 17:33 | |
| *** ArxCruz has joined #openstack-nova | 17:33 | |
| *** ArxCruz has quit IRC | 17:34 | |
| hemanthm | kashyap: I don't know if I mixed up | 17:34 |
| *** ArxCruz has joined #openstack-nova | 17:34 | |
| hemanthm | there seems a bit of an overlap there | 17:34 |
| hemanthm | between the two bugs I mean | 17:34 |
| *** ArxCruz has quit IRC | 17:34 | |
| kashyap | hemanthm: I don't see an overlap. They're two distinct issues: | 17:35 |
| *** ArxCruz has joined #openstack-nova | 17:35 | |
| kashyap | hemanthm: (a) CVE-2015-1851 (LP: 1415087) talks about image format guessing | 17:35 |
| *** ArxCruz has quit IRC | 17:35 | |
| openstack | Launchpad bug 1415087 in OpenStack Security Advisory "[OSSA 2015-011] Format-guessing and file disclosure in image convert (CVE-2015-1850, CVE-2015-1851)" [High,Fix released] https://launchpad.net/bugs/1415087 - Assigned to Tristan Cacqueray (tristan-cacqueray) | 17:35 |
| kashyap | hemanthm: While (b) CVE-2015-5162 (1449062) talks about setting CPU / memory usage limits during `qemu-img` calls | 17:36 |
| kashyap | Where is the overlap? | 17:36 |
| kashyap | Anyhow, I gotta run, back later. | 17:36 |
| hemanthm | kashyap: yeah, `qemu-img info` calls to be precise | 17:36 |
| hemanthm | kashyap: alright, let me know when you are back | 17:37 |
| kashyap | hemanthm: Yes, the tool is same, but the code-paths, and functional issues are different. That is what I was trying to say. | 17:37 |
| *** stackempty has joined #openstack-nova | 17:37 | |
| *** bvanhav_ has joined #openstack-nova | 17:37 | |
| stackempty | hey .. anyone here ? | 17:38 |
| artom | sfinucan, what's your patch for https://bugs.launchpad.net/nova/+bug/1303714 ? | 17:39 |
| openstack | Launchpad bug 1303714 in OpenStack Compute (nova) "force_config_drive=True don't go to db" [Wishlist,In progress] - Assigned to Stephen Finucane (stephenfinucane) | 17:39 |
| artom | stackempty, we're all figments of your imagination | 17:39 |
| *** hshiina has quit IRC | 17:39 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: allocations for generic PCI devices https://review.openstack.org/375168 | 17:39 |
| *** rama_y has joined #openstack-nova | 17:40 | |
| stackempty | artom: lol .. nice | 17:41 |
| stackempty | so i'm wondering .. if i update a flavor's cpu limits on the metadata .. does it apply to running instances .. ?? | 17:42 |
| *** salv-orlando has joined #openstack-nova | 17:43 | |
| *** ducttape_ has joined #openstack-nova | 17:43 | |
| *** mbirru has joined #openstack-nova | 17:44 | |
| artom | stackempty, you mean like, 'live'? Instance will see their number of CPUs reduced or increased while it's running? No. | 17:44 |
| *** tonytan_brb is now known as tonytan4ever | 17:45 | |
| artom | stackempty, also, as it says in the topic, the #openstack channel is where operators hang out, you'll likely get quicker feedback there. | 17:45 |
| *** dosaboy_ has joined #openstack-nova | 17:45 | |
| *** akshai has quit IRC | 17:45 | |
| *** dosaboy_ has quit IRC | 17:46 | |
| *** dosaboy has quit IRC | 17:46 | |
| *** pilgrimstack has joined #openstack-nova | 17:47 | |
| *** dosaboy has joined #openstack-nova | 17:47 | |
| stackempty | `oh ok .. thanks artom | 17:47 |
| artom | stackempty, :) | 17:48 |
| stackempty | one more thing .. after a reboot of the instance .. the flavor limits would be applied yeah ? | 17:48 |
| *** roeyc has joined #openstack-nova | 17:48 | |
| *** trisq has quit IRC | 17:50 | |
| *** mriedem has joined #openstack-nova | 17:50 | |
| jaypipes | sdague: you ever see these when running the functional tests? http://paste.openstack.org/show/582984/ | 17:51 |
| *** irenab has quit IRC | 17:51 | |
| jaypipes | sdague: they won't result in an error or anything... just pollute the output. | 17:51 |
| mriedem | jaypipes: yeah i've seen those | 17:52 |
| sdague | jaypipes: is there a related async_network stack trace somewhere? | 17:52 |
| sdague | that was the last bit of polution I was tracking, but it's extra weird | 17:52 |
| sdague | oh, yeh, _allocate_network_async | 17:52 |
| *** irenab has joined #openstack-nova | 17:52 | |
| sdague | this is the one I'm not quite sure what's going on | 17:53 |
| *** stackempty has left #openstack-nova | 17:53 | |
| jaypipes | sdague: gotcha. I might dig a little into it if I have some free time later today. | 17:54 |
| *** cdent has quit IRC | 17:56 | |
| *** haplo37 has quit IRC | 17:57 | |
| *** rmart04 has quit IRC | 17:59 | |
| *** rmart04 has joined #openstack-nova | 18:00 | |
| *** mat128 is now known as mat128|mtg | 18:01 | |
| mriedem | lbeliveau: do you know if https://bugs.launchpad.net/nova/+bug/1625220 was a regression in newton? | 18:01 |
| openstack | Launchpad bug 1625220 in OpenStack Compute (nova) "get_function_by_ifname does not return true for physical function" [Undecided,Fix released] - Assigned to edan david (edand) | 18:01 |
| *** anush_ has quit IRC | 18:03 | |
| mriedem | nope | 18:03 |
| *** haplo37 has joined #openstack-nova | 18:07 | |
| *** anush_ has joined #openstack-nova | 18:08 | |
| *** roeyc has quit IRC | 18:08 | |
| fungi | kashyap: hemanthm: thanks for working that out. so sounds like nova's all set, glance and cinder should have enough info to do their patches/backports | 18:09 |
| *** rmart04 has quit IRC | 18:09 | |
| jaypipes | sdague, zzzeek: hi guys, any chance you can have a look at this and tell me if there's something obvious I'm missing? I'm trying to get a transaction context to the DB within a Nova functional test case and figured I could do just self.context.session.connection() but I guess I am wrong :) http://paste.openstack.org/show/582985/ | 18:11 |
| *** ijw has joined #openstack-nova | 18:11 | |
| *** ihrachys has quit IRC | 18:12 | |
| *** ijw has quit IRC | 18:14 | |
| sdague | jaypipes: well, the nova context has no path to the db context, right? | 18:14 |
| *** ijw has joined #openstack-nova | 18:14 | |
| jaypipes | sdague: I'm wondering if the functional test case base has some magic method/attribute that I could use to grab a DB context... | 18:15 |
| *** ijw has quit IRC | 18:15 | |
| *** browne has quit IRC | 18:15 | |
| dansmith | sdague: so I have a thing that is behind alaski's log analysis patch.. if you think that's likely to never go in, I'll switch their order, but wasn't sure.. maybe you're okay putting it in to prove it out and then split? | 18:15 |
| dansmith | you didn't -1 hence my asking | 18:15 |
| *** jdillaman has joined #openstack-nova | 18:17 | |
| *** ijw has joined #openstack-nova | 18:17 | |
| sdague | dansmith: well, it seems like a really odd place to put it. But it also seems like it was the result of some conversation that I missed, so I was trying to understand context | 18:18 |
| *** markvoelker_ has joined #openstack-nova | 18:18 | |
| sdague | because, if we're talking about collecting things things in a gate run, I think per project json tools is probably the wrong approach | 18:18 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Fix database poison warnings, part 20 https://review.openstack.org/377001 | 18:18 |
| dansmith | sdague: long term yeah, but like many things, showing that it's useful in the project that wants it does't seem wrong to me | 18:19 |
| *** markvoelker has quit IRC | 18:19 | |
| dansmith | sdague: seems like doing the long term thing could end up with us not having data for quite a while, and more importantly, me not merging the other patch I want until we settle it :) | 18:20 |
| sdague | dansmith: but, how are you going to show usefulness without other things collecting and comparing it on the infra / qa side? | 18:20 |
| dansmith | sdague: I guess I thought alaski's goal wasn't really to plot it initially, but have some way of comparing one run you suspect was bad to another one you suspect was good | 18:20 |
| dansmith | like a "I see lots of red here.. hoo boy, yeah, 500 more error logs in this one" | 18:21 |
| *** markvoelker has joined #openstack-nova | 18:21 | |
| dansmith | ...sort of thing | 18:21 |
| sdague | dansmith: it could be, but we've got a fleet of things asking questions like this on the QA side (stackviz, er, openstack health) | 18:21 |
| sdague | it seems really odd if no one had a conversation over there before just doing this thing | 18:21 |
| *** ijw has quit IRC | 18:21 | |
| dansmith | *shrug* | 18:22 |
| dansmith | I guess the answer to my question is that I should not be behind this patch | 18:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add a post-test-hook to run the archive command https://review.openstack.org/376567 | 18:23 |
| *** markvoelker_ has quit IRC | 18:24 | |
| dansmith | alaski: when you get back, I propose we drop the depends-on your patch from the config one until you have the required discussion | 18:24 |
| *** ijw has joined #openstack-nova | 18:24 | |
| dansmith | in fact, I'll just do it.. dropping it doesn't help test it or anything | 18:24 |
| jaypipes | dansmith, sdague: do you know of any functional or unit tests in Nova where we manually grab a connection to the database to test somehting? | 18:25 |
| sdague | jaypipes: yeh, one sec | 18:25 |
| sdague | jaypipes: https://github.com/openstack/nova/blob/6383894a33013df8688948ea5825b70a48e88ae3/nova/tests/functional/wsgi/test_flavor_manage.py#L201-L212 | 18:26 |
| sdague | it goes through the db layer though | 18:26 |
| *** rlrossit has quit IRC | 18:29 | |
| *** Guest52852 has quit IRC | 18:29 | |
| hemanthm | fungi: we are close, should merge the master fix in a few mins | 18:30 |
| *** thumpba has joined #openstack-nova | 18:30 | |
| *** rlrossit has joined #openstack-nova | 18:30 | |
| fungi | thanks | 18:31 |
| *** sandanar_ has quit IRC | 18:31 | |
| jaypipes | sdague: yeah, I'm thinking I might just need to initialize a context manually using one of the enginefacade decorators.. | 18:32 |
| *** sandanar has joined #openstack-nova | 18:32 | |
| *** akshai has joined #openstack-nova | 18:34 | |
| *** sneti has quit IRC | 18:34 | |
| jaypipes | sdague: ah, nm, I can use the DatabaseFixture ... | 18:34 |
| *** akshai has quit IRC | 18:37 | |
| *** akshai has joined #openstack-nova | 18:38 | |
| *** akshai has quit IRC | 18:43 | |
| *** pilgrimstack has quit IRC | 18:44 | |
| *** liusheng has quit IRC | 18:45 | |
| *** liusheng has joined #openstack-nova | 18:45 | |
| *** browne has joined #openstack-nova | 18:45 | |
| *** diablo_rojo has quit IRC | 18:52 | |
| *** anush_ has quit IRC | 18:52 | |
| *** anush__ has joined #openstack-nova | 18:52 | |
| *** cdent has joined #openstack-nova | 18:53 | |
| *** moshele has joined #openstack-nova | 18:53 | |
| *** sneti has joined #openstack-nova | 18:55 | |
| mriedem | oh shelved_offloaded https://bugs.launchpad.net/nova/+bug/1627694 | 18:57 |
| openstack | Launchpad bug 1627694 in OpenStack Compute (nova) newton "unshelving an instance doesn't rollback volumes connections on failure" [Undecided,Confirmed] | 18:57 |
| mriedem | you scamp | 18:57 |
| *** sudipto has quit IRC | 18:57 | |
| *** sudipto_ has quit IRC | 18:57 | |
| jaypipes | mriedem: no argument from me on that :) | 19:01 |
| *** moshele has quit IRC | 19:02 | |
| *** raunak has quit IRC | 19:05 | |
| *** sandanar has quit IRC | 19:07 | |
| *** ijw_ has joined #openstack-nova | 19:07 | |
| *** artom has quit IRC | 19:09 | |
| *** ijw has quit IRC | 19:09 | |
| openstackgerrit | melanie witt proposed openstack/nova: Add quota related tables to the api database. https://review.openstack.org/331810 | 19:09 |
| *** gongysh has quit IRC | 19:11 | |
| *** gongysh has joined #openstack-nova | 19:11 | |
| *** gongysh has quit IRC | 19:11 | |
| mriedem | alaski: bauzas: any idea if unshelve a shelved_offloaded instance will attempt to reschedule if the build on the first host picked fails? | 19:13 |
| mriedem | oh it looks like it will retry | 19:14 |
| *** artom has joined #openstack-nova | 19:15 | |
| *** siva_krish has quit IRC | 19:18 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Fix database poison warnings, part 21 https://review.openstack.org/377051 | 19:21 |
| *** matrohon has joined #openstack-nova | 19:21 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Improving help text for xenapi_vmops_opts https://review.openstack.org/347085 | 19:22 |
| diana_clarke | one more set of database poison warnings to go... until more slip in | 19:23 |
| *** haplo37 has quit IRC | 19:23 | |
| artom | diana_clarke, I have no context whatsoever for those poison warnings, but is there a way to add a hacking rule for those? | 19:24 |
| mriedem | diana_clarke: wouldn't another way to tackle that problem be to change test classes over to extending from NoDBTestCase? | 19:24 |
| artom | So that more *don't* slip in? | 19:24 |
| artom | Or a unit test? | 19:25 |
| dansmith | mriedem: yes, we should be making things no-db tests wherever possible | 19:25 |
| diana_clarke | artom: once they are all gone (which they were at fix #14), we can start raising exceptions rather than printing warnings. | 19:25 |
| dansmith | mriedem: these are tests that claim to be but are touching the database anyway | 19:25 |
| diana_clarke | (and then the original test authors can fix them) | 19:25 |
| mriedem | how do they claim to not use the db? | 19:26 |
| mriedem | but do, and don't blow up? | 19:26 |
| *** haplo37 has joined #openstack-nova | 19:26 | |
| mriedem | ok so i see like class AggregateMigrationTestCase(test.NoDBTestCase): | 19:26 |
| mriedem | USES_DB_SELF = True | 19:26 |
| mriedem | so it claims to not b/c of NoDBTestCase | 19:26 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Update rolling upgrade steps from upgrades documentation https://review.openstack.org/373549 | 19:26 |
| *** jamielennox|away has quit IRC | 19:26 | |
| dansmith | mriedem: the problem is that they touch things that tickle internal oslo.db state, but don't actually do a thing that hits the db, which means a later test comes along and we find out that the internal state has already been initialized but no schema was ever built | 19:27 |
| mriedem | and USES_DB_SELF means, i'm a NoDBTestCase, but i setup the database fixture *myself* | 19:28 |
| mriedem | not inherited from the base test class | 19:28 |
| dansmith | yeah | 19:29 |
| *** _ducttape_ has joined #openstack-nova | 19:29 | |
| *** jamielennox|away has joined #openstack-nova | 19:29 | |
| *** moshele has joined #openstack-nova | 19:29 | |
| *** jamielennox|away is now known as jamielennox | 19:30 | |
| mriedem | ok, i'm not sure why AggregateMigrationTestCase doesn't just extend from test.TestCase then and use the base stuff | 19:30 |
| mriedem | since the only difference i see is AggregateMigrationTestCase does'nt get this: self.useFixture(nova_fixtures.DefaultFlavorsFixture()) | 19:30 |
| dansmith | I dunno about that one specifically, but some of the tests need to do something specific to the fixture | 19:30 |
| *** bvanhav_ has quit IRC | 19:31 | |
| dansmith | I don't fully recall, but I remember there were a few weird ones, especially functional ones that needed to control the fixtures themselves | 19:31 |
| mriedem | the functional ones i'm looking at do it b/c they only want the api db | 19:31 |
| mriedem | and not the cell db | 19:31 |
| dansmith | ah | 19:31 |
| *** ducttape_ has quit IRC | 19:32 | |
| *** hemanthm is now known as hemanthm|bbl | 19:32 | |
| burgerk | alaski: The question I asked earlier about context being bad, I tracked it down to the constructor in ClientRouter in nova/rpc.py self.run_periodic_tasks(nova.context.RequestContext()) | 19:32 |
| burgerk | that is creating a new, bad context on the thread. But I don't know why the behavior has changed | 19:32 |
| mriedem | burgerk: https://github.com/openstack/nova/commits/master/nova/context.py | 19:33 |
| *** cmartin has joined #openstack-nova | 19:33 | |
| *** sdake has quit IRC | 19:33 | |
| cmartin | If anybody is free I would appreciate some code reviews on my bugfix https://review.openstack.org/#/c/370861/ | 19:33 |
| mriedem | burgerk: maybe this change https://github.com/openstack/nova/commit/4df0869e00f4316ebb41ed9cdf4d6a433227f4d0 | 19:34 |
| *** READ10 has quit IRC | 19:34 | |
| mriedem | burgerk: probably want to talk to melwitt about that one | 19:35 |
| melwitt | I need to read the scrollback | 19:35 |
| *** diablo_rojo has joined #openstack-nova | 19:35 | |
| *** moshele has quit IRC | 19:35 | |
| mriedem | melwitt: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2016-09-26.log.html#t2016-09-26T15:11:12 | 19:36 |
| melwitt | mriedem: thanks much | 19:36 |
| *** irenab has quit IRC | 19:36 | |
| *** rlrossit has quit IRC | 19:36 | |
| *** irenab has joined #openstack-nova | 19:37 | |
| melwitt | originally I had used an admin context there like the other periodic tasks do but was asked if it was really needed, and my periodic task didn't need a context | 19:38 |
| melwitt | but your question is implying that the context can go elsewhere than just my periodic task ... looking | 19:38 |
| raj_singh | mriedem: I want to understand workflow of gate-grenade-dsvm-multinode. Can you please confirm if my understanding is right | 19:39 |
| raj_singh | 2 nodes setup, primary node has devstack with all nova services and secondary node has just nova compute (along with non-nova services). This is n-1 version. Tempest smoke is ran on this setup | 19:40 |
| raj_singh | Then grenade update devstack on primary to version n. So all new services on primary but secondary node is still running old compute. Smoke test is ran again on mix of old and new compute. | 19:40 |
| clarkb | raj_singh: yes thats basically it (not sure if tempest smoke is the test selection but some set of tempest tests are run yes) | 19:40 |
| raj_singh | clarkb: Thankyou | 19:41 |
| *** TonyXu has quit IRC | 19:41 | |
| raj_singh | Now if I want to achieve zero downtime will following setup suffice? | 19:41 |
| raj_singh | - 2 nodes setup. primary node has devstack with all nova services and secondary node has nova compute and api | 19:41 |
| raj_singh | Then grenade update devstack on primary. Other node still running gold compute and api service. Smoke test run on mix of old and new API | 19:41 |
| raj_singh | s/gold/old | 19:41 |
| raj_singh | sdague: ^ | 19:42 |
| burgerk | melwitt nova.context.RequestContext() is reseting the context and setting a bad context in threading.local | 19:42 |
| *** salv-orl_ has joined #openstack-nova | 19:42 | |
| burgerk | passing in nova.context.RequestContext(overwrite=False) stops it from doing that, but unsure of the consquences | 19:42 |
| melwitt | burgerk: oh. okay, I was not aware the context passed there could affect other tasks :( | 19:44 |
| *** haplo37_ has quit IRC | 19:45 | |
| *** crinkle_ is now known as crinkle | 19:45 | |
| *** salv-orlando has quit IRC | 19:45 | |
| *** anush__ has quit IRC | 19:45 | |
| *** zz_dimtruck is now known as dimtruck | 19:46 | |
| *** sdake has joined #openstack-nova | 19:46 | |
| *** anush_ has joined #openstack-nova | 19:47 | |
| *** haplo37_ has joined #openstack-nova | 19:48 | |
| *** ihrachys has joined #openstack-nova | 19:50 | |
| melwitt | burgerk: thanks for finding that. I'll get this fixed. my guess is it should just be an admin context instead | 19:50 |
| mriedem | burgerk: want to report a bug? | 19:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Cleanup some redundant USES_DB_SELF usage https://review.openstack.org/377062 | 19:50 |
| mriedem | dansmith: see if ^ makes any sense | 19:50 |
| burgerk | yes I will write up a bug | 19:51 |
| melwitt | thanks burgerk | 19:52 |
| dansmith | mriedem: sure | 19:52 |
| burgerk | np | 19:52 |
| alaski | mriedem: I don't think unshelve will reschedule on a failure. at least I don't recall adding that | 19:53 |
| alaski | dansmith: yeah, dropping the depends-on is good. I was more looking to spark a convo which is why I offered to stack my patch on yours earlier | 19:54 |
| dansmith | alaski: ack | 19:54 |
| mriedem | alaski: ah yeah i see | 19:55 |
| alaski | sdague: so as to why I put this in the Nova tree, while there isn't much in the current script that's Nova specific I was thinking we could add things that are. | 19:55 |
| mriedem | the reschedule is completely driven from the compute manager | 19:55 |
| mriedem | i thought it was handled somewhere in conductor | 19:55 |
| *** anush_ has quit IRC | 19:55 | |
| alaski | sdague: like info about scheduling failures/attempts or other things that don't apply to other projects | 19:55 |
| *** anush_ has joined #openstack-nova | 19:55 | |
| mriedem | guess that makes sense since conductor casts to the compute, compute does it's thing an either aborts or casts back to conductor to reschedule | 19:55 |
| alaski | sdague: I don't care where the code lives though. I just would like to be able to introspect project specific items like that | 19:56 |
| alaski | mriedem: for boot yeah. I didn't see anything like that for unshelve | 19:56 |
| mriedem | diana_clarke: left a comment on https://review.openstack.org/#/c/375891/ - might be a later todo/cleanup to add a global stub on some of the instance fault / action / reporting stuff | 19:58 |
| *** raunak has joined #openstack-nova | 19:59 | |
| *** _ducttape_ has quit IRC | 20:00 | |
| *** ducttape_ has joined #openstack-nova | 20:00 | |
| *** tblakes has quit IRC | 20:02 | |
| diana_clarke | mriedem: Indeed, some of these could definitely use a more holistic approach. Mostly, I'm just focused on making them go away at the moment. Thanks for the reviews! | 20:04 |
| mriedem | diana_clarke: can we use the AllServicesCurrent fixture here instead? https://review.openstack.org/#/c/375894/1/nova/tests/unit/api/openstack/compute/test_shelve.py | 20:05 |
| mriedem | in setUp | 20:05 |
| *** yjiang5 has joined #openstack-nova | 20:05 | |
| *** nherciu has quit IRC | 20:06 | |
| alaski | sdague: also +1 to what dansmith said above, it's helpful to just start collecting something somewhere to see where we can derive value and then expand. For now I just wanted to be able to answer the question of "does this zuul run have more warning messages than others I've looked at today". I couldn't see anything that quantifies that at this point | 20:07 |
| burgerk | melwitt: https://bugs.launchpad.net/nova/+bug/1627838 | 20:08 |
| openstack | Launchpad bug 1627838 in OpenStack Compute (nova) "Context is reset to by ClientRouter in nova/rpc.py" [Undecided,New] | 20:08 |
| melwitt | burgerk: thanks! | 20:09 |
| melwitt | patch arriving shortly | 20:09 |
| burgerk | put a couple possible solutions in there, if you want to take a look | 20:09 |
| diana_clarke | mriedem: oh! I didn't know about that fixture. I'll see if that works instead. I just found some other code that was patching and setting the min version to 1, so I copied it. | 20:10 |
| melwitt | burgerk: yeah, I think you're right that overwrite=False is what's needed. all of the other places we have the same call in the code are using get_admin_context() which happens to set overwrite=False inside it, so that's how the others work too | 20:11 |
| diana_clarke | mriedem: Actually, IIRC it goes down a more complicated path for more recent versions, which will require even more mocking.. | 20:11 |
| mriedem | diana_clarke: where are those tests actually even hitting the service version stuff? | 20:11 |
| *** nherciu has joined #openstack-nova | 20:14 | |
| *** jaypipes has quit IRC | 20:14 | |
| *** haplo37 has quit IRC | 20:15 | |
| diana_clarke | mriedem: A lot of them are hitting new code in _get_instance: https://review.openstack.org/#/c/263927/67/nova/compute/api.py | 20:16 |
| mriedem | ah yeah | 20:18 |
| mriedem | diana_clarke: so https://review.openstack.org/#/c/375894/1/nova/tests/unit/api/openstack/compute/test_shelve.py is really a REST API test | 20:18 |
| mriedem | and should probably mock out anything that hits the actual db | 20:18 |
| mriedem | like...nova.objects.instance.Instance.get_by_uuid is too low level | 20:18 |
| mriedem | in other words, maybe that should be nova.compute.api.API.get instead | 20:19 |
| *** mat128|mtg is now known as mat128 | 20:19 | |
| mriedem | the REST API tests should mock the compute API interfaces | 20:19 |
| diana_clarke | mriedem: you can see the callers, if you change the warning to an exception in fixtures.py (like so: https://review.openstack.org/#/c/349082/2/nova/tests/fixtures.py) | 20:19 |
| mriedem | yeah so this is just a bad test | 20:19 |
| mriedem | we should fix the mock | 20:19 |
| *** lucasagomes is now known as lucas-afk | 20:21 | |
| *** cdent has quit IRC | 20:21 | |
| *** akshai has joined #openstack-nova | 20:21 | |
| mriedem | diana_clarke: ok, left a comment inline https://review.openstack.org/#/c/375894/1/nova/tests/unit/api/openstack/compute/test_shelve.py | 20:21 |
| *** akshai has quit IRC | 20:22 | |
| *** morgabra_ is now known as morgabra | 20:22 | |
| diana_clarke | mriedem: okay thanks! I'm off to grab the kids, but I'll circle back to those later or tomorrow. thanks agian. | 20:22 |
| mriedem | np | 20:22 |
| *** slaweq_ has joined #openstack-nova | 20:22 | |
| *** anush_ has quit IRC | 20:22 | |
| *** Kimmo_ has quit IRC | 20:24 | |
| *** ijw_ has quit IRC | 20:24 | |
| *** haplo37 has joined #openstack-nova | 20:24 | |
| *** ijw has joined #openstack-nova | 20:25 | |
| *** rama_y has quit IRC | 20:25 | |
| *** smatzek has quit IRC | 20:25 | |
| *** gouthamr has quit IRC | 20:26 | |
| *** tblakes has joined #openstack-nova | 20:26 | |
| *** rama_y has joined #openstack-nova | 20:26 | |
| *** ijw has quit IRC | 20:26 | |
| sdague | alaski: well, you can query that off of ER today (anything that's > DEBUG level is indexed there) | 20:27 |
| sdague | alaski: part of the reason for asking if the conversation was had is that we've got a bunch of things doing nearby stuff, and it's probably worth at least talking it through first before starting to build up a whole other toolchain | 20:28 |
| *** rlrossit has joined #openstack-nova | 20:30 | |
| *** rlrossit has quit IRC | 20:31 | |
| *** code-R has joined #openstack-nova | 20:32 | |
| *** tasker has joined #openstack-nova | 20:34 | |
| *** dimtruck is now known as zz_dimtruck | 20:34 | |
| *** kfarr has joined #openstack-nova | 20:34 | |
| *** ihrachys has quit IRC | 20:35 | |
| *** Jeffrey4l has quit IRC | 20:35 | |
| *** code-R_ has joined #openstack-nova | 20:35 | |
| *** Jeffrey4l has joined #openstack-nova | 20:36 | |
| *** code-R has quit IRC | 20:38 | |
| *** ducttape_ has quit IRC | 20:38 | |
| *** krtaylor has quit IRC | 20:39 | |
| tasker | hello. I've discovered an issue with flavors in Nova. Whenever a flavor is modified, either changing a value or (dis)allowing access, the UUID flavor_id of the flavor changes. looking at the Nova database tables (`instance_types` and `instance_type_projects`) it looks like the flavor is deleted and recreated instead of a new entry being added (`instance_type_projects`) or updated (`instance_types`). Is this working as intended? | 20:41 |
| *** ducttape_ has joined #openstack-nova | 20:42 | |
| tasker | I can't find in the Nova code where it would be deleting the flavor and recreating it. | 20:42 |
| *** code-R_ has quit IRC | 20:43 | |
| *** yjiang5 has left #openstack-nova | 20:43 | |
| *** zz_dimtruck is now known as dimtruck | 20:44 | |
| mriedem | tasker: via the REST API or horizon? | 20:46 |
| tasker | either. | 20:47 |
| mriedem | tasker: horizon 'updates' a flavor by deleting it and creating a new one | 20:47 |
| mriedem | if you don't provide an id when creating a flavor, a uuid is generated | 20:47 |
| tasker | I've used python-novaclient for my other tests, same results. | 20:47 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/flavors.py#L74 | 20:47 |
| mriedem | what nova cli do you use to update a flavor? | 20:47 |
| tasker | the actual python bits. nova.flavors.* | 20:48 |
| mriedem | yeah but there is no 'update' for a flavor | 20:49 |
| mriedem | you can modify via extra specs or tenant access | 20:49 |
| tasker | but you can allow access via the python functions. | 20:49 |
| openstackgerrit | melanie witt proposed openstack/nova: Stop overwriting thread local context in ClientRouter https://review.openstack.org/377093 | 20:49 |
| mriedem | yeah https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/flavor_access.py#L43 | 20:50 |
| sdague | tasker: if you have the reproduce code for it, we can look into it. But the access change shouldn't impact things in that way | 20:51 |
| mriedem | melwitt: thanks i'll track that for newton rc2 | 20:51 |
| sdague | and there is no actual update for flavor data | 20:51 |
| sdague | flavors are meant to be idempotent | 20:51 |
| *** baoli has quit IRC | 20:52 | |
| tasker | sdague where can I upload, provide my experiments for your consideration? | 20:52 |
| sdague | tasker: I'd file a bug https://bugs.launchpad.net/nova - and attach the code there | 20:52 |
| mriedem | tasker: a bug report is a start | 20:52 |
| mriedem | also be sure to point out what version of novaclient and nova you're testing with | 20:52 |
| tasker | at bugs.launchpad.net/openstack? | 20:53 |
| tasker | mriedem sure will. | 20:53 |
| melwitt | mriedem: thanks for setting the info on the bug. looks like the linking is indeed busted, even for patch set 1 | 20:53 |
| mriedem | https://bugs.launchpad.net/nova | 20:53 |
| mriedem | tasker: ^ | 20:53 |
| tasker | roger. | 20:53 |
| alaski | sdague: so I don't think I can get what I'm looking for from ER then, as I specifically want to look at debug logs as well. But no conversation has been had yet, I just had an idea and wanted to see what it could grow into. | 20:53 |
| alaski | sdague: if you think we need a final picture before doing any work that's fine, I can start a convo at some point | 20:53 |
| *** dimtruck is now known as zz_dimtruck | 20:54 | |
| *** mvk has joined #openstack-nova | 20:54 | |
| sdague | alaski: ok, sure, which is cool. It's probably worth sketching out a little further, because if there is an expectation this grows into something that's collected on all the gate runs having an idea of how that data is all viewed in the end is kind of important | 20:54 |
| dansmith | alaski: ER also lags the actual run by quite a bit, AFAIK | 20:55 |
| sdague | otherwise, you end up with - oh... but did you check this 9th data source between runs | 20:55 |
| sdague | dansmith: it's usually within an hour | 20:55 |
| sdague | typically within 15 minutes | 20:55 |
| dansmith | sdague: yeah, but not with the actual jenkins results | 20:55 |
| dansmith | and sometimes it goes way off the rails right? | 20:55 |
| dansmith | just FWIW, I hadn't known anything about this until I went looking for the post hook today, and my first thought was that this would be a super useful easy first step | 20:56 |
| mriedem | so on a shelve we delete the instance from the hypervisor but we keep track of the network and bdms right? | 20:56 |
| dansmith | so just MHO. | 20:56 |
| *** diablo_rojo has quit IRC | 20:57 | |
| alaski | sdague: I have no expectation about how this is viewed or collected. My primary goal was to look at a single run and see if it varied from other runs. Having a log file makes direct comparison easy | 20:57 |
| dansmith | alaski: the way it was written I figured it would mostly be useful for eye-diffs of two runs | 20:58 |
| dansmith | alaski: like "oh, this one has an order of magnitude more errors than this other one" | 20:58 |
| alaski | exactly | 20:58 |
| sdague | alaski: ok, but how do you find the "good run". | 20:58 |
| dansmith | alaski: yeah, so +1 from me :) | 20:58 |
| sdague | how do you know that this run is the one that went crazy, and not that it went crazy 2 days ago and no one noticed? | 20:58 |
| dansmith | sdague: if you just care about the difference between two runs it doesn't matter.. this doesn't have to turn into a graph | 20:59 |
| alaski | right. maybe it's that a patch added a warning log and you just compare to some other patch that didn't | 21:00 |
| mriedem | so when you unshelve an instance, if that fails, we don't actually want to deallocate the network or delete the bdms, which is what https://bugs.launchpad.net/nova/+bug/1627694 is saying - it's actually working as designed. if unshelve attempt #1 fails and we delete the network and volumes, then you try to unshelve again, you might spawn the instance but the network and volumes are gone. | 21:00 |
| openstack | Launchpad bug 1627694 in OpenStack Compute (nova) "unshelving an instance doesn't rollback volumes connections on failure" [Low,Invalid] | 21:00 |
| alaski | I get that graphing would be super useful, I was just starting with something easy to see what is worth doing | 21:01 |
| sdague | well, having been down this path a few times before, I'm going to guess that it will turn into it is | 21:01 |
| *** cleong has quit IRC | 21:01 | |
| alaski | I actually have a script running locally which parses the gerrit event stream for jenkins comments, pulls down compute logs, parses them, spits out the stats. It's useful, but I want to avoid downloading 18M of logs for each run | 21:01 |
| sdague | alaski: sure, and my suggestion is starting it a little more generic from the get go | 21:02 |
| *** diablo_rojo has joined #openstack-nova | 21:02 | |
| dansmith | I don't see what this hurts | 21:02 |
| dansmith | I do see what it helps | 21:03 |
| *** cmartin has quit IRC | 21:03 | |
| alaski | sdague: doing this more generically would mean what exactly? doing this in the QA project? | 21:04 |
| sdague | probably, and at least talking through where it's going. Because it could also end up in something like os-loganalyze, so you could keep the code that parses the logs out of code that's in the run | 21:05 |
| sdague | so you can ask a new question of old data | 21:05 |
| sdague | which is often pretty useful | 21:05 |
| dansmith | this seems far too noisy to really make much use of long-term data to me | 21:05 |
| dansmith | maybe notifications would be useful long-term | 21:06 |
| * dansmith runs off for a bit | 21:06 | |
| sdague | warnings and errors would be long term useful, because you could actually use it as a burn down to get rid of gorpy ones | 21:06 |
| sdague | and realize when a bunch of new ones got injected | 21:07 |
| sdague | which gets lost all the time | 21:07 |
| melwitt | is this like a per test run log sanity checker? I've fallen into that pit in the past where a patch introduced some error logs but passed all the tests | 21:07 |
| mriedem | yes i would like alerts on new changes dumping tons of trace or warnings in the logs | 21:07 |
| *** stewie925_ has joined #openstack-nova | 21:07 | |
| mriedem | like: https://bugs.launchpad.net/nova/+bug/1622270 | 21:08 |
| openstack | Launchpad bug 1622270 in OpenStack Compute (nova) "libvirt: excessive warning logs like "couldn't obtain the vcpu count from domain id: 6bcf0dcb-722f-44a3-a0aa-3fa42dd1075a, exception: Requested operation is not valid: cpu affinity is not supported"" [Medium,Confirmed] | 21:08 |
| alaski | well, what I proposed is just simple stats like http://logs.openstack.org/06/375106/8/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/e103612/logs/n-stats.json | 21:08 |
| *** stewie925 is now known as Guest63324 | 21:08 | |
| *** stewie925_ is now known as stewie925 | 21:08 | |
| tasker | mriedem, sdague: ok. re-did my tests and i was mixing up my results from horizon and the python-novaclient. it is as you said; the novaclient is _not_ messing with the flavor's id. this is my desired behaviour. | 21:08 |
| melwitt | alaski: that's really cool | 21:09 |
| mriedem | tasker: yeah, horizon will 'update' a flavor by deleting the old one and creating a new one | 21:09 |
| *** stew925 has joined #openstack-nova | 21:09 | |
| tasker | so, can i blame horizonand call it a day? | 21:09 |
| mriedem | yes | 21:10 |
| mriedem | it's a known thing | 21:10 |
| tasker | ok. thanks for the help! | 21:10 |
| *** haplo37 has quit IRC | 21:11 | |
| *** eharney has quit IRC | 21:12 | |
| alaski | sdague: I see where you're coming from, and I don't disagree really. But I think my long term plans are different than what is shown with my simple proposal. I would like to get very project specific and start aggregating logs into flows, to say things like instance boots took 20s from API to running state. | 21:13 |
| alaski | while simple counts like I have now are also very useful | 21:14 |
| *** zz_dimtruck is now known as dimtruck | 21:14 | |
| *** tasker has quit IRC | 21:14 | |
| *** catintheroof has joined #openstack-nova | 21:15 | |
| alaski | however, the simple answer is I don't know what's going to be useful so I wanted to see this before having a conversation. I don't think I can predict what the right thing to build is | 21:15 |
| sdague | alaski: sure, which is why I mostly asked to socialize and think through these things. Because it would be pretty interesting to see how much of that we could just pull out of ES already, which lets us use existing data collection. | 21:16 |
| *** mriedem has quit IRC | 21:17 | |
| sdague | also if your workflow is pull logs, process, the thing that all those logs are getting pulled through could do that processing | 21:17 |
| alaski | but it doesn't have debug logs, which are pretty important here | 21:18 |
| sdague | alaski: ES doesn't, sure | 21:18 |
| alaski | I'm not sure what you're referring to then | 21:19 |
| sdague | http://logs.openstack.org/51/376451/1/check/gate-tempest-dsvm-full-ubuntu-xenial/7aa81e4/logs/screen-n-cpu.txt.gz | 21:20 |
| sdague | the reason that you get colorizing and linking there is because we have active code on the server | 21:20 |
| sdague | that is processing logs on the fly | 21:20 |
| *** nherciu_ has joined #openstack-nova | 21:20 | |
| *** haplo37 has joined #openstack-nova | 21:20 | |
| sdague | there is no reason it couldn't do this data collection on http://logs.openstack.org/51/376451/1/check/gate-tempest-dsvm-full-ubuntu-xenial/7aa81e4/logs/stats dynamically | 21:20 |
| alaski | sure, that gets the single log case but doesn't handle looking at all nova logs at once | 21:21 |
| sdague | alaski: there is nothing that prevents it from doing that | 21:21 |
| sdague | it's just how we wrote the handler | 21:21 |
| *** nherciu has quit IRC | 21:22 | |
| sdague | anyway, got to go handle dinner things here. Have a good night folks. | 21:23 |
| alaski | I'll take a look. I still prefer to start small and vet it before getting cross project | 21:23 |
| *** dimtruck is now known as zz_dimtruck | 21:24 | |
| *** akshai has joined #openstack-nova | 21:24 | |
| *** krtaylor has joined #openstack-nova | 21:24 | |
| *** jheroux has quit IRC | 21:25 | |
| *** matrohon has quit IRC | 21:26 | |
| *** ayoung has quit IRC | 21:26 | |
| *** ijw has joined #openstack-nova | 21:27 | |
| *** sdague has quit IRC | 21:27 | |
| *** siva_krish has joined #openstack-nova | 21:27 | |
| *** burgerk_ has joined #openstack-nova | 21:27 | |
| *** matrohon has joined #openstack-nova | 21:27 | |
| *** akshai has quit IRC | 21:29 | |
| *** Apoorva_ has joined #openstack-nova | 21:29 | |
| *** thorst has quit IRC | 21:30 | |
| *** salv-orl_ has quit IRC | 21:30 | |
| *** nherciu_ has quit IRC | 21:30 | |
| *** akshai has joined #openstack-nova | 21:30 | |
| *** ijw has quit IRC | 21:31 | |
| *** akshai has quit IRC | 21:31 | |
| *** ijw has joined #openstack-nova | 21:31 | |
| *** burgerk has quit IRC | 21:31 | |
| *** rtheis has quit IRC | 21:32 | |
| *** salv-orlando has joined #openstack-nova | 21:32 | |
| *** akshai has joined #openstack-nova | 21:32 | |
| *** akshai has quit IRC | 21:32 | |
| *** claudiub has quit IRC | 21:33 | |
| *** Apoorva has quit IRC | 21:33 | |
| *** sdake has quit IRC | 21:33 | |
| *** lamt has joined #openstack-nova | 21:35 | |
| *** thorst has joined #openstack-nova | 21:36 | |
| *** annegentle has quit IRC | 21:37 | |
| *** Kimmo_ has joined #openstack-nova | 21:37 | |
| *** thorst has quit IRC | 21:41 | |
| *** burgerk_ has quit IRC | 21:41 | |
| *** mriedem has joined #openstack-nova | 21:43 | |
| *** bdeschenes has quit IRC | 21:48 | |
| *** thorst has joined #openstack-nova | 21:49 | |
| *** gouthamr has joined #openstack-nova | 21:50 | |
| *** sdague has joined #openstack-nova | 21:51 | |
| *** vishwanathj has quit IRC | 21:51 | |
| *** alee_ has quit IRC | 21:52 | |
| *** siva_krish has quit IRC | 21:53 | |
| *** thorst has quit IRC | 21:54 | |
| mriedem | browne: tjones: still looking for a vmware nsx ci run on this https://review.openstack.org/#/c/253666/ | 21:54 |
| mriedem | looks like we're on about a 24 hour lag on vmware ci results? | 21:55 |
| mriedem | if at all, ps13 was the last one that got results | 21:55 |
| browne | yep, we're trying to figure out what is causing issues. i'm also not seeing logs properly posted | 21:56 |
| *** burt has quit IRC | 21:56 | |
| *** TravT has quit IRC | 21:57 | |
| *** Swami has quit IRC | 21:57 | |
| *** marst has quit IRC | 21:59 | |
| *** marst has joined #openstack-nova | 21:59 | |
| *** TravT has joined #openstack-nova | 22:00 | |
| mriedem | dansmith: i don't think any of the jobs that ran on https://review.openstack.org/#/c/376567/ would have tested it | 22:00 |
| tjones | it's waaaaayyyyy backed up. we are trying to add resources | 22:01 |
| *** marst has quit IRC | 22:01 | |
| *** annegentle has joined #openstack-nova | 22:01 | |
| *** diablo_rojo has quit IRC | 22:02 | |
| *** rama_y has quit IRC | 22:04 | |
| mriedem | dansmith: i'm going to put up a d-g DNM patch to test that | 22:04 |
| mriedem | tjones: ok | 22:04 |
| mriedem | tjones: are you guys at least filtering out changes that wouldn't need to run for vmware? | 22:05 |
| mriedem | like unit test changes, docs, etc? | 22:05 |
| *** annegent_ has joined #openstack-nova | 22:05 | |
| mriedem | tjones: like this https://github.com/openstack-infra/project-config/blob/af90f545caa94d8aa6b2e24bc8b5572a6f4f1288/zuul/layout.yaml#L1355 | 22:05 |
| mriedem | tjones: ^ means skip the dsvm job run if all files in the change match those regex patterns | 22:06 |
| mriedem | since changes to those files won't impact runtime | 22:06 |
| *** annegentle has quit IRC | 22:06 | |
| *** TravT has quit IRC | 22:06 | |
| tjones | we could | 22:07 |
| tjones | let me check | 22:08 |
| *** salv-orlando has quit IRC | 22:08 | |
| *** TravT has joined #openstack-nova | 22:09 | |
| *** xyang1 has quit IRC | 22:11 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Removing duplicates from columns_to_join list https://review.openstack.org/342299 | 22:12 |
| *** kfarr has quit IRC | 22:12 | |
| *** efried has quit IRC | 22:14 | |
| *** rama_y has joined #openstack-nova | 22:14 | |
| *** ayoung has joined #openstack-nova | 22:14 | |
| *** bdeschenes has joined #openstack-nova | 22:14 | |
| tjones | mriedem: argh - now it's not just the queue backed up it is also networking issues. our guy is working on it | 22:15 |
| *** tanvirt has quit IRC | 22:15 | |
| *** ijw has quit IRC | 22:16 | |
| *** artom has quit IRC | 22:17 | |
| *** artom has joined #openstack-nova | 22:18 | |
| *** artom has quit IRC | 22:18 | |
| *** artom has joined #openstack-nova | 22:19 | |
| *** ijw has joined #openstack-nova | 22:22 | |
| *** marst has joined #openstack-nova | 22:22 | |
| dansmith | mriedem: we can't because project-config right? | 22:23 |
| *** ducttape_ has quit IRC | 22:26 | |
| mriedem | dansmith: can't what? | 22:27 |
| mriedem | that's what the d-g patch is for | 22:27 |
| dansmith | mriedem: can't directly with a depends-on in project-config, but as I just indicated in -qa, a hacky alternative is cool | 22:28 |
| mriedem | yeah just to see it run | 22:28 |
| *** ijw has quit IRC | 22:29 | |
| *** Swami has joined #openstack-nova | 22:29 | |
| *** matrohon has quit IRC | 22:29 | |
| *** lamt has quit IRC | 22:31 | |
| *** thumpba has quit IRC | 22:40 | |
| *** thumpba has joined #openstack-nova | 22:40 | |
| *** mlavalle has quit IRC | 22:40 | |
| *** thumpba has quit IRC | 22:42 | |
| *** brian_price has quit IRC | 22:43 | |
| *** zz_dimtruck is now known as dimtruck | 22:48 | |
| *** alee_ has joined #openstack-nova | 22:49 | |
| *** annegent_ has quit IRC | 22:49 | |
| *** ducttape_ has joined #openstack-nova | 22:50 | |
| *** tblakes has quit IRC | 22:51 | |
| *** haplo37 has quit IRC | 22:52 | |
| *** sneti has quit IRC | 22:56 | |
| *** dimtruck is now known as zz_dimtruck | 22:57 | |
| *** Swami_ has joined #openstack-nova | 22:59 | |
| *** haplo37 has joined #openstack-nova | 23:01 | |
| *** edmondsw has quit IRC | 23:01 | |
| *** ducttape_ has quit IRC | 23:01 | |
| *** lucas-afk has quit IRC | 23:02 | |
| *** Swami has quit IRC | 23:03 | |
| *** brian_price has joined #openstack-nova | 23:04 | |
| *** catintheroof has quit IRC | 23:04 | |
| *** lucasagomes has joined #openstack-nova | 23:05 | |
| *** zz_dimtruck is now known as dimtruck | 23:15 | |
| *** ducttape_ has joined #openstack-nova | 23:17 | |
| *** Swami__ has joined #openstack-nova | 23:18 | |
| *** Swami_ has quit IRC | 23:22 | |
| *** thorst has joined #openstack-nova | 23:24 | |
| *** dimtruck is now known as zz_dimtruck | 23:25 | |
| *** ijw has joined #openstack-nova | 23:28 | |
| *** lamt has joined #openstack-nova | 23:28 | |
| *** smatzek has joined #openstack-nova | 23:29 | |
| *** rama_y has quit IRC | 23:31 | |
| *** ijw has quit IRC | 23:32 | |
| *** Apoorva_ has quit IRC | 23:33 | |
| *** salv-orlando has joined #openstack-nova | 23:39 | |
| *** ducttape_ has quit IRC | 23:39 | |
| *** Apoorva has joined #openstack-nova | 23:40 | |
| *** Swami has joined #openstack-nova | 23:40 | |
| *** zz_dimtruck is now known as dimtruck | 23:42 | |
| *** Swami__ has quit IRC | 23:43 | |
| *** sdake has joined #openstack-nova | 23:44 | |
| *** salv-orlando has quit IRC | 23:45 | |
| *** ijw has joined #openstack-nova | 23:48 | |
| *** smatzek has quit IRC | 23:49 | |
| *** thorst has quit IRC | 23:49 | |
| stewie925 | hello all - what parameter does the 'tox -e api-samples' accept ? | 23:49 |
| stewie925 | I understand that the api-ref can be regenerated/updated using the 'tox -e api-samples' command | 23:50 |
| *** haplo37_ has quit IRC | 23:51 | |
| stewie925 | although unfortunately I could not find an example online... | 23:51 |
| *** dimtruck is now known as zz_dimtruck | 23:52 | |
| stewie925 | oops I mean api-sample, not api-ref | 23:52 |
| *** haplo37_ has joined #openstack-nova | 23:53 | |
| *** ijw has quit IRC | 23:53 | |
| *** sdague has quit IRC | 23:53 | |
| *** vishwanathj has joined #openstack-nova | 23:53 | |
| tjones | mriedem: we've fixed some stuff and added more executors, so hopefully we can catch up. | 23:54 |
| *** Sukhdev has joined #openstack-nova | 23:54 | |
| *** 7IZAAPZ3F has joined #openstack-nova | 23:57 | |
| *** ijw has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!