| *** ssurana has joined #openstack-nova | 00:02 | |
| *** gongysh has joined #openstack-nova | 00:03 | |
| *** gszasz has quit IRC | 00:04 | |
| *** gcb has quit IRC | 00:05 | |
| sbezverk | hello, do I need both cell0 and cell1 in ocata or just cell0 is sufficient? | 00:06 |
|---|---|---|
| *** mhenkel has joined #openstack-nova | 00:08 | |
| *** mriedem has quit IRC | 00:08 | |
| *** Jack_Iv has joined #openstack-nova | 00:14 | |
| *** Jack_Iv has quit IRC | 00:19 | |
| *** masber has quit IRC | 00:21 | |
| *** baoli_ has joined #openstack-nova | 00:21 | |
| *** dtp has quit IRC | 00:24 | |
| *** baoli has quit IRC | 00:25 | |
| *** liangy has joined #openstack-nova | 00:26 | |
| *** gouthamr has joined #openstack-nova | 00:28 | |
| *** adisky_ has quit IRC | 00:29 | |
| *** adrian_otto has joined #openstack-nova | 00:33 | |
| *** mnestratov|2 has quit IRC | 00:33 | |
| *** Jack_Iv has joined #openstack-nova | 00:35 | |
| *** chenrong has joined #openstack-nova | 00:35 | |
| *** Jack_Iv has quit IRC | 00:40 | |
| *** zhurong has joined #openstack-nova | 00:41 | |
| *** mhenkel has quit IRC | 00:42 | |
| *** takashin has joined #openstack-nova | 00:43 | |
| *** liverpooler has joined #openstack-nova | 00:47 | |
| *** mhenkel has joined #openstack-nova | 00:51 | |
| *** tovin07_ has joined #openstack-nova | 00:54 | |
| openstackgerrit | Bing Li proposed openstack/python-novaclient master: Remove log translations https://review.openstack.org/448336 | 00:55 |
| *** adrian_otto has quit IRC | 00:59 | |
| *** lifeless_ is now known as lifeless | 01:00 | |
| *** phuongnh has joined #openstack-nova | 01:00 | |
| lifeless | efried: is this you rbug? https://github.com/testing-cabal/mock/issues/398 | 01:01 |
| *** baoli has joined #openstack-nova | 01:02 | |
| *** baoli_ has quit IRC | 01:05 | |
| *** kenperkins has quit IRC | 01:08 | |
| *** kevinz has joined #openstack-nova | 01:08 | |
| *** armax has quit IRC | 01:08 | |
| *** READ10 has quit IRC | 01:09 | |
| *** esberglu has quit IRC | 01:13 | |
| *** gouthamr has quit IRC | 01:16 | |
| *** READ10 has joined #openstack-nova | 01:19 | |
| *** zhurong has quit IRC | 01:23 | |
| *** yamahata has quit IRC | 01:27 | |
| openstackgerrit | Duan Jiong proposed openstack/nova master: Fix some reST field lists in docstrings https://review.openstack.org/448841 | 01:27 |
| *** masber has joined #openstack-nova | 01:29 | |
| *** kaisers1 has joined #openstack-nova | 01:32 | |
| *** gjayavelu has quit IRC | 01:33 | |
| *** kaisers has quit IRC | 01:34 | |
| *** Jack_Iv has joined #openstack-nova | 01:36 | |
| *** Apoorva_ has joined #openstack-nova | 01:36 | |
| *** liangy has quit IRC | 01:37 | |
| *** dave-mccowan has quit IRC | 01:37 | |
| *** dave-mccowan has joined #openstack-nova | 01:39 | |
| *** chenrong has quit IRC | 01:39 | |
| *** Apoorva has quit IRC | 01:40 | |
| *** gongysh has quit IRC | 01:40 | |
| *** Apoorva_ has quit IRC | 01:40 | |
| *** Jack_Iv has quit IRC | 01:40 | |
| *** zhurong has joined #openstack-nova | 01:42 | |
| *** pumaranikar has joined #openstack-nova | 01:44 | |
| *** manjeets has joined #openstack-nova | 01:46 | |
| *** edmondsw has quit IRC | 01:46 | |
| *** raj_sing- has joined #openstack-nova | 01:46 | |
| *** litao has joined #openstack-nova | 01:48 | |
| *** masber has quit IRC | 01:49 | |
| openstackgerrit | Huan Xie proposed openstack/nova master: Create ovs port with other params together https://review.openstack.org/444230 | 01:52 |
| *** masber has joined #openstack-nova | 01:53 | |
| *** dharinic- has joined #openstack-nova | 01:53 | |
| *** hieulq has quit IRC | 01:56 | |
| *** Sukhdev_ has quit IRC | 01:56 | |
| *** ssurana has quit IRC | 01:57 | |
| *** hieulq has joined #openstack-nova | 01:59 | |
| *** dave-mcc_ has joined #openstack-nova | 02:08 | |
| *** dave-mccowan has quit IRC | 02:10 | |
| *** jdurgin has joined #openstack-nova | 02:12 | |
| *** dave-mccowan has joined #openstack-nova | 02:13 | |
| *** unicell has quit IRC | 02:15 | |
| *** dave-mcc_ has quit IRC | 02:16 | |
| *** rfolco has quit IRC | 02:19 | |
| *** owalsh has quit IRC | 02:20 | |
| *** owalsh has joined #openstack-nova | 02:21 | |
| *** shaohe_feng has joined #openstack-nova | 02:22 | |
| *** READ10 has quit IRC | 02:22 | |
| *** hongbin has joined #openstack-nova | 02:23 | |
| *** NostawRm has quit IRC | 02:23 | |
| *** NostawRm has joined #openstack-nova | 02:23 | |
| openstackgerrit | Danfly proposed openstack/nova master: remove mox from unit/virt/xenapi/image/test_bittorrent.py https://review.openstack.org/448855 | 02:25 |
| openstackgerrit | Alex Xu proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625 | 02:26 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201 | 02:26 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | 02:26 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Trait and TraitList objects https://review.openstack.org/376199 | 02:26 |
| openstackgerrit | Alex Xu proposed openstack/nova master: api-ref: Complete all the verifications of remote consoles https://review.openstack.org/338051 | 02:32 |
| *** gongysh has joined #openstack-nova | 02:36 | |
| *** Jack_Iv has joined #openstack-nova | 02:36 | |
| *** dharinic- is now known as dharinic | 02:37 | |
| *** hieulq has quit IRC | 02:39 | |
| *** hieulq has joined #openstack-nova | 02:41 | |
| *** Jack_Iv has quit IRC | 02:41 | |
| *** eliqiao has joined #openstack-nova | 02:43 | |
| openstackgerrit | Danfly proposed openstack/nova master: remove mox from tests/unit/compute/test_compute_utils.py https://review.openstack.org/448862 | 02:44 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Add description to policies in admin_actions https://review.openstack.org/440829 | 02:47 |
| *** unicell has joined #openstack-nova | 02:48 | |
| *** edmondsw has joined #openstack-nova | 02:48 | |
| *** edmondsw has quit IRC | 02:53 | |
| *** ayogi has joined #openstack-nova | 02:55 | |
| *** masber has quit IRC | 02:56 | |
| *** Danfly has joined #openstack-nova | 02:57 | |
| *** awaugama has quit IRC | 02:59 | |
| *** gouthamr has joined #openstack-nova | 02:59 | |
| *** tuanluong has joined #openstack-nova | 03:00 | |
| *** ayogi has quit IRC | 03:06 | |
| *** nicolasbock has quit IRC | 03:06 | |
| *** ayogi has joined #openstack-nova | 03:06 | |
| *** gjayavelu has joined #openstack-nova | 03:10 | |
| Danfly | who has time to review my patch? tks a lot. https://review.openstack.org/#/c/445504/ | 03:13 |
| *** liverpooler has quit IRC | 03:15 | |
| *** gouthamr has quit IRC | 03:19 | |
| *** baoli has quit IRC | 03:22 | |
| *** Sukhdev has joined #openstack-nova | 03:33 | |
| *** ayogi has quit IRC | 03:34 | |
| *** hongbin has quit IRC | 03:37 | |
| *** Jack_Iv has joined #openstack-nova | 03:37 | |
| *** links has joined #openstack-nova | 03:37 | |
| *** amotoki has quit IRC | 03:37 | |
| *** Jack_Iv has quit IRC | 03:42 | |
| *** kenperkins has joined #openstack-nova | 03:43 | |
| *** zhurong has quit IRC | 03:48 | |
| *** masber has joined #openstack-nova | 03:48 | |
| *** armax has joined #openstack-nova | 03:49 | |
| *** dave-mccowan has quit IRC | 03:50 | |
| *** amotoki has joined #openstack-nova | 03:50 | |
| *** hieulq has quit IRC | 03:50 | |
| *** hieulq has joined #openstack-nova | 03:53 | |
| *** fragatin_ has joined #openstack-nova | 03:54 | |
| *** amotoki has quit IRC | 03:55 | |
| *** fragatina has quit IRC | 03:55 | |
| *** fragatin_ has quit IRC | 03:59 | |
| *** amotoki has joined #openstack-nova | 03:59 | |
| *** kaisers has joined #openstack-nova | 03:59 | |
| *** gjayavelu has quit IRC | 04:00 | |
| *** kaisers has quit IRC | 04:03 | |
| *** ratailor has joined #openstack-nova | 04:05 | |
| *** ratailor has quit IRC | 04:05 | |
| *** udesale has joined #openstack-nova | 04:07 | |
| *** sree has joined #openstack-nova | 04:08 | |
| *** jdurgin has quit IRC | 04:12 | |
| *** jdurgin1 has joined #openstack-nova | 04:12 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936 | 04:16 |
| *** guchihiro has joined #openstack-nova | 04:17 | |
| *** NikhilS has joined #openstack-nova | 04:19 | |
| *** amotoki has quit IRC | 04:19 | |
| *** adisky_ has joined #openstack-nova | 04:20 | |
| *** jdurgin1 has quit IRC | 04:20 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add deprecate-nova-cert spec https://review.openstack.org/448882 | 04:22 |
| *** tovin07_ has quit IRC | 04:25 | |
| *** psachin has joined #openstack-nova | 04:28 | |
| *** amotoki has joined #openstack-nova | 04:28 | |
| *** vks1 has joined #openstack-nova | 04:32 | |
| *** gongysh has quit IRC | 04:34 | |
| *** tovin07_ has joined #openstack-nova | 04:36 | |
| *** ratailor has joined #openstack-nova | 04:36 | |
| *** tovin07_ has quit IRC | 04:38 | |
| *** masber has quit IRC | 04:40 | |
| *** kenperkins has quit IRC | 04:43 | |
| *** tovin07_ has joined #openstack-nova | 04:44 | |
| *** armax has quit IRC | 04:44 | |
| *** armax has joined #openstack-nova | 04:44 | |
| *** yamahata has joined #openstack-nova | 04:48 | |
| *** fragatina has joined #openstack-nova | 04:49 | |
| *** nmathew has joined #openstack-nova | 04:52 | |
| *** Jeffrey4l_ has quit IRC | 04:55 | |
| *** Jeffrey4l has joined #openstack-nova | 04:56 | |
| *** ayogi has joined #openstack-nova | 05:02 | |
| *** Jack_Iv has joined #openstack-nova | 05:02 | |
| *** gongysh has joined #openstack-nova | 05:03 | |
| *** Jack_Iv has quit IRC | 05:06 | |
| *** markvoelker has quit IRC | 05:08 | |
| *** markvoelker has joined #openstack-nova | 05:09 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 05:09 |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add deprecate-nova-cert spec https://review.openstack.org/448882 | 05:12 |
| *** markvoelker has quit IRC | 05:13 | |
| *** gcb has joined #openstack-nova | 05:14 | |
| *** prateek has joined #openstack-nova | 05:18 | |
| *** diga has joined #openstack-nova | 05:22 | |
| *** Jack_Iv has joined #openstack-nova | 05:23 | |
| *** baoli has joined #openstack-nova | 05:24 | |
| *** Jack_Iv has quit IRC | 05:28 | |
| *** baoli has quit IRC | 05:29 | |
| *** venki has quit IRC | 05:32 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | 05:35 |
| *** Shunli has joined #openstack-nova | 05:40 | |
| *** aunnam has joined #openstack-nova | 05:42 | |
| *** kaisers has joined #openstack-nova | 05:47 | |
| *** venki has joined #openstack-nova | 05:47 | |
| *** bmace has quit IRC | 05:47 | |
| *** bmace has joined #openstack-nova | 05:48 | |
| *** zhurong has joined #openstack-nova | 05:53 | |
| *** Sukhdev has quit IRC | 05:53 | |
| *** venki has quit IRC | 05:54 | |
| *** sree_ has joined #openstack-nova | 05:54 | |
| *** sree_ is now known as Guest90664 | 05:54 | |
| *** jamespd has quit IRC | 05:56 | |
| *** tjones has joined #openstack-nova | 05:56 | |
| *** tjones1 has quit IRC | 05:56 | |
| *** jamespd has joined #openstack-nova | 05:56 | |
| *** sree has quit IRC | 05:56 | |
| *** Jack_Iv has joined #openstack-nova | 05:58 | |
| *** irenab has joined #openstack-nova | 05:58 | |
| *** armax has quit IRC | 06:03 | |
| *** armax has joined #openstack-nova | 06:03 | |
| *** armax has quit IRC | 06:04 | |
| *** armax has joined #openstack-nova | 06:05 | |
| *** armax has quit IRC | 06:05 | |
| *** haplo37_ has joined #openstack-nova | 06:06 | |
| *** tonyb_ has joined #openstack-nova | 06:06 | |
| *** pabardina has quit IRC | 06:07 | |
| *** snikitin_ has joined #openstack-nova | 06:07 | |
| *** venki has joined #openstack-nova | 06:07 | |
| *** slaweq__ has joined #openstack-nova | 06:08 | |
| *** nkorabli has joined #openstack-nova | 06:08 | |
| *** nkorabli has quit IRC | 06:08 | |
| *** markvoelker has joined #openstack-nova | 06:09 | |
| *** dutsmoc is now known as comstud | 06:09 | |
| *** tjones has quit IRC | 06:09 | |
| *** Jack_Iv has quit IRC | 06:10 | |
| *** unicell1 has joined #openstack-nova | 06:10 | |
| *** gcb has quit IRC | 06:11 | |
| *** unicell has quit IRC | 06:11 | |
| *** gcb has joined #openstack-nova | 06:11 | |
| *** Jack_Iv has joined #openstack-nova | 06:12 | |
| *** hemna_ has joined #openstack-nova | 06:12 | |
| *** tonyb has quit IRC | 06:13 | |
| *** flaper87 has quit IRC | 06:13 | |
| *** haplo37 has quit IRC | 06:13 | |
| *** snikitin has quit IRC | 06:13 | |
| *** slaweq has quit IRC | 06:13 | |
| *** hemna has quit IRC | 06:13 | |
| *** haplo37_ is now known as haplo37 | 06:13 | |
| *** markvoelker has quit IRC | 06:13 | |
| *** pabardina has joined #openstack-nova | 06:14 | |
| *** john51 has quit IRC | 06:14 | |
| *** john51 has joined #openstack-nova | 06:15 | |
| *** namnh has joined #openstack-nova | 06:17 | |
| *** venki has quit IRC | 06:18 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 06:18 |
| *** gjayavelu has joined #openstack-nova | 06:19 | |
| *** masber has joined #openstack-nova | 06:22 | |
| *** edmondsw has joined #openstack-nova | 06:25 | |
| *** tovin07_ has quit IRC | 06:26 | |
| *** edmondsw has quit IRC | 06:29 | |
| *** jaosorior has joined #openstack-nova | 06:30 | |
| openstackgerrit | Danfly proposed openstack/nova master: remove mox from unit/compute/test_compute_utils.py https://review.openstack.org/448862 | 06:33 |
| *** abalutoiu_ has joined #openstack-nova | 06:33 | |
| *** venki has joined #openstack-nova | 06:37 | |
| *** abalutoiu has quit IRC | 06:37 | |
| *** Danfly has quit IRC | 06:37 | |
| *** abalutoiu_ has quit IRC | 06:38 | |
| *** fragatina has quit IRC | 06:39 | |
| *** masber has quit IRC | 06:39 | |
| *** venki has quit IRC | 06:41 | |
| *** nkorabli has joined #openstack-nova | 06:42 | |
| *** satyar has joined #openstack-nova | 06:48 | |
| *** amotoki_ has joined #openstack-nova | 06:52 | |
| *** udesale__ has joined #openstack-nova | 06:53 | |
| openstackgerrit | Huan Xie proposed openstack/nova master: Create ovs port with other params together https://review.openstack.org/444230 | 06:54 |
| *** amotoki has quit IRC | 06:55 | |
| *** venki has joined #openstack-nova | 06:55 | |
| *** udesale has quit IRC | 06:56 | |
| *** gszasz has joined #openstack-nova | 06:59 | |
| *** tovin07_ has joined #openstack-nova | 07:01 | |
| *** isq has quit IRC | 07:01 | |
| *** sbezverk_ has joined #openstack-nova | 07:01 | |
| *** isq has joined #openstack-nova | 07:02 | |
| *** tbachman has joined #openstack-nova | 07:02 | |
| *** jpena|off has quit IRC | 07:02 | |
| *** dmellado has quit IRC | 07:03 | |
| *** rodrigods has quit IRC | 07:03 | |
| *** Prophidys has quit IRC | 07:03 | |
| *** vladikr has quit IRC | 07:04 | |
| *** namnh has quit IRC | 07:04 | |
| *** vladikr has joined #openstack-nova | 07:05 | |
| *** tbachman_ has quit IRC | 07:05 | |
| *** sbezverk has quit IRC | 07:05 | |
| *** rodrigods has joined #openstack-nova | 07:06 | |
| *** dmellado has joined #openstack-nova | 07:08 | |
| *** jpena|off has joined #openstack-nova | 07:08 | |
| *** andreas_s has joined #openstack-nova | 07:08 | |
| *** Prophidys has joined #openstack-nova | 07:08 | |
| *** zsli_ has joined #openstack-nova | 07:10 | |
| *** Jack_Iv has quit IRC | 07:10 | |
| *** Jack_Iv has joined #openstack-nova | 07:10 | |
| *** Shunli has quit IRC | 07:12 | |
| *** moshele has joined #openstack-nova | 07:13 | |
| *** bkopilov has joined #openstack-nova | 07:18 | |
| *** udesale has joined #openstack-nova | 07:19 | |
| *** moshele has quit IRC | 07:22 | |
| *** udesale__ has quit IRC | 07:23 | |
| *** nkrinner_afk is now known as nkrinner | 07:25 | |
| *** baoli has joined #openstack-nova | 07:25 | |
| *** huanxie has joined #openstack-nova | 07:27 | |
| *** markus_z has joined #openstack-nova | 07:30 | |
| *** baoli has quit IRC | 07:31 | |
| *** gcb has quit IRC | 07:31 | |
| *** ratailor has quit IRC | 07:31 | |
| *** gcb has joined #openstack-nova | 07:34 | |
| *** Jack_Iv has quit IRC | 07:40 | |
| *** Jack_Iv has joined #openstack-nova | 07:40 | |
| *** amoralej|off is now known as amoralej | 07:43 | |
| *** tesseract has joined #openstack-nova | 07:44 | |
| *** sridharg has joined #openstack-nova | 07:46 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Stop using mox stubs in nova/tests/unit/cells/fakes.py https://review.openstack.org/342043 | 07:47 |
| *** mlakat has quit IRC | 07:49 | |
| *** gcb has quit IRC | 07:50 | |
| *** tobasco has quit IRC | 07:57 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** tuanluong has quit IRC | 08:00 | |
| *** gcb has joined #openstack-nova | 08:02 | |
| *** rmart04 has joined #openstack-nova | 08:03 | |
| *** pcaruana has joined #openstack-nova | 08:06 | |
| *** flaper87 has joined #openstack-nova | 08:06 | |
| *** flaper87 has quit IRC | 08:07 | |
| *** flaper87 has joined #openstack-nova | 08:07 | |
| *** rmart04 has quit IRC | 08:07 | |
| *** venki has quit IRC | 08:08 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Remove fstype param from ploop init https://review.openstack.org/444970 | 08:09 |
| *** markvoelker has joined #openstack-nova | 08:10 | |
| *** diga has quit IRC | 08:10 | |
| *** diga has joined #openstack-nova | 08:12 | |
| *** edmondsw has joined #openstack-nova | 08:13 | |
| *** venki has joined #openstack-nova | 08:13 | |
| *** markvoelker has quit IRC | 08:14 | |
| *** felipemonteiro_ has quit IRC | 08:14 | |
| *** felipemonteiro_ has joined #openstack-nova | 08:15 | |
| *** zsli_ has quit IRC | 08:16 | |
| *** tobasco has joined #openstack-nova | 08:17 | |
| *** edmondsw has quit IRC | 08:17 | |
| *** mkoderer has joined #openstack-nova | 08:17 | |
| *** moshele has joined #openstack-nova | 08:17 | |
| *** Shunli has joined #openstack-nova | 08:17 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Raise general error message when volume attach fails https://review.openstack.org/320163 | 08:18 |
| openstackgerrit | Danfly proposed openstack/nova master: remove mox from unit/api/openstack/compute/test_flavor_manage.py https://review.openstack.org/448581 | 08:19 |
| *** Shunli has quit IRC | 08:21 | |
| *** jichen has joined #openstack-nova | 08:21 | |
| *** Shunli has joined #openstack-nova | 08:21 | |
| *** zsli_ has joined #openstack-nova | 08:23 | |
| *** guchihiro has quit IRC | 08:24 | |
| Kevin_Zheng | https://bugs.launchpad.net/nova/+bug/1675276 seems quite a Bug | 08:25 |
| openstack | Launchpad bug 1675276 in OpenStack Compute (nova) "Volumes attached to shelved instance may contain incorrect device_name" [Undecided,New] - Assigned to Zhenyu Zheng (zhengzhenyu) | 08:25 |
| *** Shunli has quit IRC | 08:25 | |
| *** zsli_ has quit IRC | 08:25 | |
| *** mnestratov has joined #openstack-nova | 08:29 | |
| *** rmart04 has joined #openstack-nova | 08:30 | |
| *** Shunli has joined #openstack-nova | 08:31 | |
| *** rmart04 has quit IRC | 08:35 | |
| *** ratailor has joined #openstack-nova | 08:35 | |
| *** ratailor has quit IRC | 08:36 | |
| *** ratailor has joined #openstack-nova | 08:37 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova-specs master: Fix rendering sub-directories in README https://review.openstack.org/449000 | 08:38 |
| *** takashin has left #openstack-nova | 08:39 | |
| *** ralonsoh has joined #openstack-nova | 08:43 | |
| *** yamahata has quit IRC | 08:46 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 08:47 |
| openstackgerrit | sahid proposed openstack/nova master: numa: fit instance NUMA node with cpus reserved onto host NUMA node https://review.openstack.org/400875 | 08:47 |
| openstackgerrit | sahid proposed openstack/nova master: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 08:47 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 08:47 |
| openstackgerrit | sahid proposed openstack/nova master: numa: take into account cpus reserved https://review.openstack.org/400876 | 08:47 |
| openstackgerrit | sahid proposed openstack/nova master: numa: update numa usage to include reserved CPUs https://review.openstack.org/400877 | 08:47 |
| *** moshele has quit IRC | 08:47 | |
| Jack_Iv | Why does trove rewrite PG access rules? | 08:49 |
| Jack_Iv | postgres@psql8:~$ psql | 08:49 |
| Jack_Iv | psql: FATAL: role "postgres" is not permitted to log in | 08:49 |
| *** jpena|off is now known as jpena | 08:51 | |
| *** sridharg has quit IRC | 08:51 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Add is_vz_container function https://review.openstack.org/445947 | 08:53 |
| *** jaosorior is now known as jaosorior_brb | 08:55 | |
| *** ltomasbo|away is now known as ltomasbo | 08:55 | |
| *** snecklifter has joined #openstack-nova | 08:59 | |
| *** openstackgerrit has quit IRC | 09:03 | |
| *** openstackgerrit has joined #openstack-nova | 09:03 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Remove fstype param from ploop init https://review.openstack.org/444970 | 09:03 |
| *** lucas-afk is now known as lucasagomes | 09:08 | |
| *** david-lyle has quit IRC | 09:11 | |
| *** david-lyle has joined #openstack-nova | 09:13 | |
| sfinucan | Jack_Iv: That sounds like a question of #openstack-trove or #openstack | 09:14 |
| Jack_Iv | oh, sorry, wrong channel | 09:14 |
| *** esberglu has joined #openstack-nova | 09:16 | |
| *** diga has quit IRC | 09:17 | |
| *** esberglu has quit IRC | 09:20 | |
| *** gabor_antal has quit IRC | 09:23 | |
| *** gabor_antal has joined #openstack-nova | 09:24 | |
| *** david-lyle_ has joined #openstack-nova | 09:27 | |
| *** david-lyle has quit IRC | 09:27 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Remove fstype param from ploop init https://review.openstack.org/444970 | 09:29 |
| *** venki has quit IRC | 09:30 | |
| *** kaisers1 has quit IRC | 09:32 | |
| *** jichen has quit IRC | 09:32 | |
| *** sridharg has joined #openstack-nova | 09:32 | |
| *** gongysh has quit IRC | 09:38 | |
| *** venki has joined #openstack-nova | 09:38 | |
| *** derekh has joined #openstack-nova | 09:38 | |
| *** links has quit IRC | 09:39 | |
| *** karimb has joined #openstack-nova | 09:39 | |
| *** ociuhandu has joined #openstack-nova | 09:40 | |
| *** amotoki_ has quit IRC | 09:40 | |
| *** zsli_ has joined #openstack-nova | 09:42 | |
| *** zsli_ has quit IRC | 09:43 | |
| *** rmart04 has joined #openstack-nova | 09:43 | |
| *** zsli_ has joined #openstack-nova | 09:44 | |
| *** cdent has joined #openstack-nova | 09:45 | |
| *** Shunli has quit IRC | 09:46 | |
| *** kaisers1 has joined #openstack-nova | 09:47 | |
| *** rmart04 has quit IRC | 09:47 | |
| *** udesale has quit IRC | 09:48 | |
| openstackgerrit | Yoshihiko Atsumi proposed openstack/nova master: Add confirm resized server functional negative tests https://review.openstack.org/421074 | 09:49 |
| *** lpetrut has joined #openstack-nova | 09:53 | |
| *** links has joined #openstack-nova | 09:56 | |
| *** avolkov has joined #openstack-nova | 09:58 | |
| *** amotoki has joined #openstack-nova | 09:59 | |
| *** satyar has quit IRC | 10:00 | |
| *** edmondsw has joined #openstack-nova | 10:01 | |
| kevinbenton | johnthetubaguy: yo | 10:02 |
| *** edmondsw has quit IRC | 10:05 | |
| *** gcb has quit IRC | 10:08 | |
| kevinbenton | johnthetubaguy: nova/neutron vif_type issue if you have some time https://bugs.launchpad.net/nova/+bug/1675343 | 10:10 |
| openstack | Launchpad bug 1675343 in OpenStack Compute (nova) "vif_type='tap' fails with permission error on /dev/net/tun" [Undecided,New] | 10:10 |
| *** markvoelker has joined #openstack-nova | 10:11 | |
| *** links has quit IRC | 10:11 | |
| *** venki has quit IRC | 10:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Imported Translations from Zanata https://review.openstack.org/449041 | 10:11 |
| *** Jack_Iv has quit IRC | 10:11 | |
| *** nicolasbock has joined #openstack-nova | 10:12 | |
| *** vsaienk01 has left #openstack-nova | 10:16 | |
| *** markvoelker has quit IRC | 10:16 | |
| *** rmart04 has joined #openstack-nova | 10:21 | |
| *** links has joined #openstack-nova | 10:23 | |
| *** rmart04 has quit IRC | 10:25 | |
| *** felipemonteiro__ has joined #openstack-nova | 10:26 | |
| *** phuongnh has quit IRC | 10:27 | |
| *** felipemonteiro_ has quit IRC | 10:28 | |
| openstackgerrit | Danfly proposed openstack/nova master: remove mox from unit/api/openstack/compute/test_disk_config.py https://review.openstack.org/449054 | 10:30 |
| *** kevinz has quit IRC | 10:31 | |
| *** links has quit IRC | 10:35 | |
| *** Jack_Iv has joined #openstack-nova | 10:35 | |
| *** tbachman has quit IRC | 10:44 | |
| *** amotoki has quit IRC | 10:46 | |
| johnthetubaguy | kevinbenton: hey, just taking a peak | 10:46 |
| snecklifter | Hello, any chance of getting this nudged along: https://review.openstack.org/#/c/393489 | 10:47 |
| snecklifter | Thanks | 10:47 |
| snecklifter | CI is green | 10:47 |
| johnthetubaguy | snecklifter: I haven't done my look through stuff that needs a nudge, that looks like one I should take a peak at | 10:48 |
| *** amotoki has joined #openstack-nova | 10:49 | |
| johnthetubaguy | kevinbenton: I don't think I have the context around that one, I remember some chatter yesterday | 10:49 |
| snecklifter | johnthetubaguy: many thanks | 10:49 |
| *** links has joined #openstack-nova | 10:51 | |
| *** gjayavelu has quit IRC | 11:01 | |
| *** jaosorior_brb is now known as jaosorior | 11:02 | |
| johnthetubaguy | mdbooth: are you are around, there was that patch I was looking at late yesterday I wanted to ask you about | 11:03 |
| *** nkorabli has quit IRC | 11:03 | |
| mdbooth | johnthetubaguy: Sorry, I've been very downstream focussed for a few days | 11:04 |
| *** nkorabli has joined #openstack-nova | 11:04 | |
| johnthetubaguy | mdbooth: no worries, that happens | 11:04 |
| mdbooth | Got a link? | 11:04 |
| johnthetubaguy | mdbooth: I think its this one: https://review.openstack.org/#/c/437597/ | 11:04 |
| johnthetubaguy | mdbooth: I couldn't think when we would fetch by attachment_id, I think I am missing something | 11:04 |
| *** esberglu has joined #openstack-nova | 11:05 | |
| mdbooth | johnthetubaguy: It just seems inevitable to me if we've got apis creating by attachment | 11:05 |
| *** jistr is now known as jistr|afk | 11:06 | |
| mdbooth | However, my point was really higher level than that | 11:06 |
| mdbooth | Which is that I don't think it's worth adding a db schema change for code we haven't seen yet | 11:06 |
| johnthetubaguy | mdbooth: so if the index didn't have a cost, I would totally be up for adding it anyways | 11:06 |
| mdbooth | I don't see a problem merging the code before the user | 11:06 |
| mdbooth | I just think it's premature to merge it when we haven't even seen the user | 11:07 |
| mdbooth | There's no reason we can't merge it quickly later | 11:07 |
| mdbooth | Merging it early just risks potentially needing another db migration | 11:07 |
| mdbooth | And it has (a small) cost when we're not using it | 11:08 |
| johnthetubaguy | thats totally fair, no harm in waiting for that | 11:08 |
| mdbooth | I was really just -1 because there's no follow up. The patch may be good. | 11:08 |
| johnthetubaguy | I think we pass instance_uuid and volume_uuid to look up a BDM, then we check if the attachment_id is present or not, and use the new flow or not, but it would be good to see that code first | 11:09 |
| *** Guest90664 has quit IRC | 11:09 | |
| johnthetubaguy | mdbooth: yeah, thats totally fair, I like your idea of waiting for the follow, it should answer our questions for us | 11:09 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Idempotent PUT /resource_classes/{name} https://review.openstack.org/448791 | 11:09 |
| *** esberglu has quit IRC | 11:10 | |
| johnthetubaguy | mdbooth: I think I managed to miss that bit of your comment, my bad | 11:10 |
| *** Drankis has joined #openstack-nova | 11:13 | |
| *** nkorabli has quit IRC | 11:14 | |
| *** nkorabli has joined #openstack-nova | 11:15 | |
| *** ekuris has quit IRC | 11:15 | |
| *** tovin07_ has quit IRC | 11:15 | |
| *** tovin07 has quit IRC | 11:19 | |
| *** edmondsw has joined #openstack-nova | 11:19 | |
| *** nkorabli has quit IRC | 11:19 | |
| *** ababich has quit IRC | 11:22 | |
| *** cdent has quit IRC | 11:22 | |
| *** satyar has joined #openstack-nova | 11:23 | |
| *** ekuris has joined #openstack-nova | 11:31 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP/DNM: Destroy the ReqSpec object when deleting the instance https://review.openstack.org/391060 | 11:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Destroy the ReqSpec object when deleting the instance https://review.openstack.org/391060 | 11:33 |
| *** masber has joined #openstack-nova | 11:33 | |
| *** andreykurilin_ has quit IRC | 11:36 | |
| *** andreykurilin has joined #openstack-nova | 11:36 | |
| *** mvk has quit IRC | 11:36 | |
| *** dharinic has quit IRC | 11:36 | |
| *** raj_sing- has quit IRC | 11:36 | |
| *** pumaranikar has quit IRC | 11:36 | |
| *** aunnam has quit IRC | 11:36 | |
| *** manjeets has quit IRC | 11:36 | |
| *** Jack_Iv has quit IRC | 11:41 | |
| *** Jack_Iv has joined #openstack-nova | 11:41 | |
| *** ratailor has quit IRC | 11:42 | |
| tojuvone | johnthetubaguy, Hi, do you have a minute | 11:43 |
| *** amotoki has quit IRC | 11:45 | |
| *** Jack_Iv has quit IRC | 11:45 | |
| tojuvone | johnthetubaguy, Just wondering new host_maintenance_events table vs. using services table | 11:47 |
| *** rfolco has joined #openstack-nova | 11:48 | |
| tojuvone | johnthetubaguy, services table already exposed to servers that suits me, but event would need duplicate on each service | 11:49 |
| *** vladikr has quit IRC | 11:50 | |
| *** nmathew has quit IRC | 11:50 | |
| tojuvone | brb in an hour or get hit by major traffic jam | 11:54 |
| *** gcb has joined #openstack-nova | 11:55 | |
| *** aysyd has joined #openstack-nova | 11:55 | |
| *** cdent has joined #openstack-nova | 11:56 | |
| *** Jack_Iv has joined #openstack-nova | 11:58 | |
| *** esberglu has joined #openstack-nova | 11:59 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova master: WIP: honor disk_format=ploop image metadata property https://review.openstack.org/429745 | 12:02 |
| *** hshiina has joined #openstack-nova | 12:02 | |
| *** jpena is now known as jpena|lunch | 12:03 | |
| mnestratov | mdbooth: hi, could you please take a look at ^ ^ and say what you think now? | 12:03 |
| *** esberglu has quit IRC | 12:04 | |
| *** karimb has quit IRC | 12:06 | |
| *** mvk has joined #openstack-nova | 12:06 | |
| *** tblakes has joined #openstack-nova | 12:06 | |
| *** nkorabli has joined #openstack-nova | 12:07 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Remove fstype param from ploop init https://review.openstack.org/444970 | 12:10 |
| *** bkopilov has quit IRC | 12:12 | |
| *** markvoelker has joined #openstack-nova | 12:13 | |
| *** tbachman has joined #openstack-nova | 12:13 | |
| *** jjung has joined #openstack-nova | 12:15 | |
| *** jjung has quit IRC | 12:16 | |
| *** markvoelker has quit IRC | 12:17 | |
| openstackgerrit | Alexander Baranin proposed openstack/nova master: Add allocation ratios to hypervisor api. https://review.openstack.org/449107 | 12:18 |
| sbezverk_ | johnthetubaguy: ping | 12:19 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova master: tests: fix uefi testcases https://review.openstack.org/449109 | 12:22 |
| johnthetubaguy | tojuvone: sbezverk_: sorry, on a call, about to grab lunch, I should be around in 30 mins or so if thats OK | 12:22 |
| *** zhurong has quit IRC | 12:22 | |
| *** pratapagoutham has joined #openstack-nova | 12:23 | |
| *** pratapagoutham has quit IRC | 12:25 | |
| sbezverk_ | johnthetubaguy: sure thing, bon apetit!! | 12:26 |
| *** jistr|afk is now known as jistr | 12:27 | |
| *** kornica has quit IRC | 12:28 | |
| *** priteau has joined #openstack-nova | 12:28 | |
| *** vks1 has quit IRC | 12:29 | |
| *** jbernard has quit IRC | 12:30 | |
| *** jbernard has joined #openstack-nova | 12:31 | |
| *** liverpooler has joined #openstack-nova | 12:35 | |
| *** dane-fichter has joined #openstack-nova | 12:38 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Don't use floats in microversion handling https://review.openstack.org/449119 | 12:39 |
| cdent | johnthetubaguy, bauzas, jaypipes : that's ^ probably reasonably important enough to merge soonish | 12:40 |
| *** catintheroof has joined #openstack-nova | 12:44 | |
| sbezverk_ | cdent: do you have a second for qq? | 12:44 |
| cdent | sbezverk_: I can try | 12:45 |
| *** markvoelker has joined #openstack-nova | 12:46 | |
| sbezverk_ | cdent: in ocata, the doc sais that we need to have two cells cell0 and cell1, but on the other hand I am running ocata with just cell0 and everything is working. SO there is an inconsistency. Could you please clarify? | 12:46 |
| tojuvone | johnthetubaguy, No problem :) | 12:46 |
| sdague | cdent: could you put a test that tries to set the decorator with a float to ensure that explodes really early? | 12:46 |
| cdent | sdague: sure | 12:46 |
| *** ayogi has quit IRC | 12:46 | |
| sdague | because I would be concerned by someone doing that accidentally | 12:46 |
| sbezverk_ | cdent: is it really mandatory to have both cells? | 12:47 |
| Dinesh_Bhor | Hi all, can someone take a look at a small patch? https://review.openstack.org/#/c/445782/ It's already having a +2. | 12:47 |
| cdent | sbezverk_: As I understand it, yes, so something is likely not what it appears, but that's an area I'm not very familiar with | 12:47 |
| *** mkoderer has quit IRC | 12:47 | |
| sbezverk_ | cdent: could you suggest local cell expert ;) ?? | 12:48 |
| *** dane-fichter has quit IRC | 12:48 | |
| sbezverk_ | cdent: cause as I said I have a test bed functioning normal with single cell0. | 12:48 |
| cdent | sbezverk_: dansmith is probably the final word there and melwitt as well but I think neither of them is around yet | 12:49 |
| EmilienM | jroll: fyi, the ironic/nova thing is fixed for us. Thanks again for your patch! | 12:49 |
| cdent | sbezverk_: are you sure you haven't named the thing which would be cell1 'cell0'? I believe you should have three database tables? | 12:49 |
| EmilienM | dansmith, bauzas ^ | 12:49 |
| bauzas | EmilienM: coolio | 12:50 |
| sbezverk_ | cdent: positive, I wrote a job and it creates a single cell0 | 12:50 |
| sbezverk_ | cdent: so I have now 3 databases only nova, nova-api and cell0 | 12:51 |
| sbezverk_ | cdent: I will follow up with these folks to confirm, thank you very much for your help | 12:51 |
| cdent | sbezverk_: I think what you've got there is correct | 12:52 |
| cdent | but like I said, I've not done much poking in this area, and I don't currently have a running system | 12:52 |
| *** psachin has quit IRC | 12:52 | |
| *** liverpooler has quit IRC | 12:52 | |
| cdent | sbezverk_: actually, i do have database tables though and I have 'nova', 'nova_api' and 'nova_cell0'. only | 12:52 |
| *** liverpooler has joined #openstack-nova | 12:53 | |
| cdent | the table named 'nova' is cell1 | 12:53 |
| sbezverk_ | cdent: ahhh, so nova became kind of cell1, got it | 12:54 |
| *** dave-mccowan has joined #openstack-nova | 12:54 | |
| bauzas | cdent: https://review.openstack.org/#/c/449119/1 had sdague's comment that I agree with | 12:54 |
| cdent | bauzas: yeah, just working on that now, thanks. | 12:54 |
| bauzas | just add this and I'm +2 | 12:54 |
| bauzas | cdent: FWIW, thanks for fixing that | 12:55 |
| *** belmoreira has joined #openstack-nova | 12:55 | |
| *** zsli_ has quit IRC | 12:56 | |
| *** raj_singh has quit IRC | 12:56 | |
| *** raj_singh has joined #openstack-nova | 12:57 | |
| *** nmathew has joined #openstack-nova | 12:57 | |
| *** nmathew has quit IRC | 12:57 | |
| *** gszasz has quit IRC | 13:00 | |
| *** udesale has joined #openstack-nova | 13:01 | |
| *** gszasz has joined #openstack-nova | 13:01 | |
| jroll | EmilienM: \o/ | 13:02 |
| *** rodrigods has quit IRC | 13:03 | |
| *** rodrigods has joined #openstack-nova | 13:03 | |
| *** catintheroof has quit IRC | 13:03 | |
| *** jpena|lunch is now known as jpena | 13:03 | |
| *** burt has joined #openstack-nova | 13:05 | |
| cdent | sdague, bauzas: If you pass in a float on that thing it will raise an AtributeError, at import time. Is that sufficient, or would you prefer that exception(s) raised from parse_version_string be more custom? I tend to prefer built in exceptions for this kind of thing. | 13:06 |
| *** amoralej is now known as amoralej|lunch | 13:07 | |
| *** awaugama has joined #openstack-nova | 13:07 | |
| bauzas | cdent: I think sdague asked for an unittest verifying that behaviour | 13:07 |
| cdent | yes, i've written the test | 13:07 |
| bauzas | not what happens if someone uses that deco incorrectly | 13:07 |
| bauzas | cdent: tbc, the former is enough from my PoV | 13:07 |
| bauzas | if we have a docstring that explains the deco is awaiting for a string, and then the developer is using it incorrectly, then meh | 13:08 |
| bauzas | it's like not following a manual when using a chainsaw | 13:09 |
| sdague | cdent: as long as that would fail a commit I'm good with it | 13:11 |
| *** haplo37 has quit IRC | 13:11 | |
| cdent | sdague: yeah, code will boom | 13:11 |
| sdague | cdent: wfm | 13:11 |
| *** mriedem has joined #openstack-nova | 13:13 | |
| *** baoli has joined #openstack-nova | 13:13 | |
| cdent | sdague: yeah: AttributeError: 'float' object has no attribute 'split'\nThe test run didn't actually run any tests" | 13:13 |
| *** amotoki has joined #openstack-nova | 13:13 | |
| cdent | (that's on the functional tests) | 13:13 |
| sdague | cool | 13:13 |
| *** mdrabe has joined #openstack-nova | 13:13 | |
| *** esberglu has joined #openstack-nova | 13:15 | |
| *** snecklifter has left #openstack-nova | 13:16 | |
| *** esberglu has quit IRC | 13:19 | |
| *** haplo37 has joined #openstack-nova | 13:20 | |
| *** gouthamr has joined #openstack-nova | 13:21 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Don't use floats in microversion handling https://review.openstack.org/449119 | 13:22 |
| *** cleong has joined #openstack-nova | 13:23 | |
| *** crushil has joined #openstack-nova | 13:25 | |
| *** priteau has quit IRC | 13:26 | |
| cdent | sdague, bauzas: that ^ should do it | 13:27 |
| sdague | cdent: frickler has a comment on the last patch | 13:29 |
| *** sree has joined #openstack-nova | 13:29 | |
| *** jdurgin has joined #openstack-nova | 13:30 | |
| bauzas | sdague: frickler missed the transformation into a tuple | 13:31 |
| *** eharney has joined #openstack-nova | 13:31 | |
| cdent | yeah, it's a tuple by then | 13:32 |
| sdague | so, is there a reason not to move APIVersionRequest into the common library and reuse all of that? | 13:33 |
| cdent | sdague: no reason other than time | 13:33 |
| cdent | sdague: I also thought that writing something fresh would allow for some compare and contrast so we could have a mix of both in whatever the common thing was | 13:34 |
| cdent | there are some differences in approach | 13:35 |
| cdent | (nothing major) | 13:35 |
| sdague | yeh, it feels like mostly we're just rediscovering all the bugs again :) | 13:35 |
| cdent | heh | 13:35 |
| sdague | I honestly remember this exact issue when we hit 2.10 | 13:35 |
| efried | lifeless Yes, https://github.com/testing-cabal/mock/issues/398 is me. | 13:36 |
| efried | mriedem I've got a doc appt during the nova meeting today. I left my update in the agenda. | 13:36 |
| cdent | sdague: sure. the other two issues were: didn't want to reuse nova code because I thought we'd be moving soon; and when I first did this I couldn't seem to untangle the way in which nova was doing version_handler from other parts of the wsgi stack. | 13:38 |
| *** esberglu has joined #openstack-nova | 13:38 | |
| cdent | In other words I found the code too hard to read, so wrote some more. Which is sad but true. | 13:38 |
| cdent | (it's the stuff in Controller that bit me) | 13:39 |
| *** mlavalle has joined #openstack-nova | 13:41 | |
| *** krtaylor has joined #openstack-nova | 13:43 | |
| mriedem | efried: ok | 13:44 |
| sdague | cdent: ok, but you've effectively replicated the APIVersionRequest object as a tuple now. It feels like that's going to be a pretty direct transition | 13:46 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | 13:46 |
| openstackgerrit | Hironori Shiina proposed openstack/nova-specs master: Ironic: Cold migration support https://review.openstack.org/449155 | 13:46 |
| cdent | sdague; that part is not the interesting/useful/verbose part of the handling though. It is the users of the Version concept, and those parts between placement and nova are much different (simply because the framework is too) | 13:47 |
| cdent | I agree with you that there's some duplication, and that's unfortunate, but it's where we are right now | 13:47 |
| cdent | and I'm not too het up about repeating bug fixes, it is good for learning, for refactoring, and refreshing, especially when they are caught before they leak | 13:48 |
| cdent | we had the review process for all this work, and it missed some things, that's okay, that's life. | 13:48 |
| *** vks1 has joined #openstack-nova | 13:49 | |
| *** takashin has joined #openstack-nova | 13:52 | |
| *** hshiina has quit IRC | 13:53 | |
| *** awaugama has quit IRC | 13:54 | |
| *** eharney has quit IRC | 13:55 | |
| *** NikhilS has quit IRC | 13:59 | |
| mriedem | nova meeting time | 14:00 |
| *** annegentle has joined #openstack-nova | 14:00 | |
| *** liangy has joined #openstack-nova | 14:01 | |
| *** hongbin has joined #openstack-nova | 14:02 | |
| *** cdent has quit IRC | 14:02 | |
| *** cdent has joined #openstack-nova | 14:02 | |
| *** READ10 has joined #openstack-nova | 14:04 | |
| *** vks1 has quit IRC | 14:05 | |
| *** annegentle has quit IRC | 14:07 | |
| *** Jack_Iv has quit IRC | 14:07 | |
| *** Jack_Iv has joined #openstack-nova | 14:07 | |
| *** lucasxu has joined #openstack-nova | 14:08 | |
| *** moshele has joined #openstack-nova | 14:10 | |
| *** eharney has joined #openstack-nova | 14:11 | |
| *** sree has quit IRC | 14:11 | |
| moshele | sean-k-mooney: hi, can you review the https://review.openstack.org/#/c/398277/ | 14:11 |
| *** prateek has quit IRC | 14:11 | |
| *** Jack_Iv has quit IRC | 14:12 | |
| *** karimb has joined #openstack-nova | 14:13 | |
| *** fragatina has joined #openstack-nova | 14:14 | |
| *** fengzhr has quit IRC | 14:15 | |
| *** fengzhr has joined #openstack-nova | 14:15 | |
| *** Jack_Iv has joined #openstack-nova | 14:18 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 14:18 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | 14:18 |
| *** armax has joined #openstack-nova | 14:18 | |
| *** kenperkins has joined #openstack-nova | 14:18 | |
| jgriffith | lyarwood sorry, I thought you were indicating that your patches that replaced mine had merged; still waiting before refactoring everything | 14:19 |
| jgriffith | lyarwood looks like there's still some dialogue to hash out | 14:20 |
| *** vks1 has joined #openstack-nova | 14:20 | |
| johnthetubaguy | I think lyarwood is out for a week, not 100% sure | 14:21 |
| *** felipemonteiro has joined #openstack-nova | 14:21 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 14:22 |
| *** amoralej|lunch is now known as amoralej | 14:22 | |
| *** Jack_Iv has quit IRC | 14:24 | |
| *** sc68cal has joined #openstack-nova | 14:25 | |
| *** fengzhr1 has joined #openstack-nova | 14:25 | |
| *** gcb has quit IRC | 14:26 | |
| *** fengzhr has quit IRC | 14:27 | |
| *** fengzhr1 has quit IRC | 14:28 | |
| *** madhaviy has joined #openstack-nova | 14:28 | |
| *** jianghuaw has joined #openstack-nova | 14:29 | |
| *** fengzhr has joined #openstack-nova | 14:29 | |
| *** adisky_ has quit IRC | 14:29 | |
| *** annegentle has joined #openstack-nova | 14:29 | |
| *** xinliang has quit IRC | 14:30 | |
| *** awaugama has joined #openstack-nova | 14:35 | |
| *** kenperkins has quit IRC | 14:37 | |
| *** links has quit IRC | 14:38 | |
| *** fandi has joined #openstack-nova | 14:42 | |
| *** xinliang has joined #openstack-nova | 14:44 | |
| *** xinliang has quit IRC | 14:44 | |
| *** xinliang has joined #openstack-nova | 14:44 | |
| *** gouthamr has quit IRC | 14:44 | |
| *** sneti has joined #openstack-nova | 14:45 | |
| *** kevinz has joined #openstack-nova | 14:46 | |
| *** kenperkins has joined #openstack-nova | 14:48 | |
| *** sneti has quit IRC | 14:48 | |
| *** sneti has joined #openstack-nova | 14:50 | |
| *** hemna_ is now known as hemna | 14:50 | |
| *** awaugama_ has joined #openstack-nova | 14:51 | |
| *** takashin has left #openstack-nova | 14:53 | |
| *** awaugama has quit IRC | 14:54 | |
| sdague | where are servers assigned to server_groups? | 14:55 |
| *** kevinz has quit IRC | 14:55 | |
| dansmith | sdague: hint group=foo | 14:56 |
| sdague | oh, it's buried inside hints? | 14:56 |
| gibi | mriedem: during the nova meeting I got the info from sjmc7 about the needed transformations | 14:57 |
| sdague | which aren't documented... that's the awesome :) | 14:57 |
| *** cdent has quit IRC | 14:58 | |
| *** fandi has quit IRC | 14:58 | |
| gibi | mriedem: < sjmc7> gibi: the only notification we rely on that's not implemented is compute.instance.volume.attach/detach | 14:58 |
| gibi | mriedem: so we are on a good track as those notifications already in focus | 14:58 |
| Kevin_Zheng | https://bugs.launchpad.net/nova/+bug/1675276 | 14:59 |
| openstack | Launchpad bug 1675276 in OpenStack Compute (nova) "Volumes attached to shelved instance may contain incorrect device_name" [Undecided,In progress] - Assigned to Zhenyu Zheng (zhengzhenyu) | 14:59 |
| mriedem | Kevin_Zheng: the device_name is never guaranteed | 15:00 |
| bauzas | sdague: yup | 15:00 |
| dansmith | sdague: yep, create a group foo, then boot an instance with hint group=foo | 15:00 |
| mriedem | Kevin_Zheng: we've talked about just removing the device_name from the os-volume_attachments API in the past | 15:00 |
| bauzas | sdague: you create a group thru the API | 15:00 |
| bauzas | and what dansmith said | 15:00 |
| bauzas | no way to remove an instance from a group, long-lived story | 15:01 |
| *** belmorei_ has joined #openstack-nova | 15:01 | |
| Kevin_Zheng | mriedem: Hmm OK，what I dicovered is | 15:01 |
| mriedem | there have been specs for that though, adding and removing members | 15:01 |
| mriedem | i remember those from kilo | 15:01 |
| bauzas | yup | 15:01 |
| bauzas | and I can name the person :) | 15:02 |
| bauzas | who is now a Watcher core :) | 15:02 |
| bauzas | anyway, my point is | 15:02 |
| Kevin_Zheng | cinder can save wrong device name for shelved instance | 15:02 |
| *** belmoreira has quit IRC | 15:02 | |
| Kevin_Zheng | could you check for detail？ on my bug report？ | 15:02 |
| bauzas | I'm not sure the servergroup API is the good abstract model for describing colocation rules for placement between instances | 15:02 |
| sdague | yeh, it's just without reading nova code you can't use any of this, there is nothing in the API ref that connects all this together (slight asside, but a little relevant for figuring out a path forward) | 15:03 |
| mriedem | Kevin_Zheng: yeah i just read it | 15:03 |
| mriedem | Kevin_Zheng: i think we have to pass a mountpoint to the cinder os-attach API | 15:03 |
| *** marst has joined #openstack-nova | 15:03 | |
| *** adrian_otto has joined #openstack-nova | 15:03 | |
| Kevin_Zheng | device name didnt got checked for shelved instance | 15:04 |
| Kevin_Zheng | so I can | 15:04 |
| Kevin_Zheng | pass vda and cinder will save vda | 15:04 |
| mriedem | Kevin_Zheng: right we don't check the device name for the instance during attach to a shelved offloaded instance because the instance does not have a host, so we can't call the compute to reserve the device name, like during normal volume attach | 15:05 |
| Kevin_Zheng | yeah，I was thinking do | 15:05 |
| mriedem | Kevin_Zheng: that's why we create the BDM with device_name=None here http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n3678 | 15:05 |
| Kevin_Zheng | do something once we got the right name | 15:05 |
| mriedem | otherwise we do http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n3687 | 15:05 |
| mriedem | Kevin_Zheng: are you proposing that we set the device_name in the BDM after unshelve? | 15:06 |
| madhaviy | @mriedem, one question on nova/network/neutronv2/api's _check_external_network_attach() method. This is checking whether user/context can attach a network which is marked as external. so when this code is executed during deploy/spawn, policy checking is been done on compute host. Do we need to validate this even before | 15:06 |
| Kevin_Zheng | yeah I did some learning this afternoon | 15:06 |
| gibi | bauzas: there was a proposal from jaypipes to revisit the server group concept here https://review.openstack.org/#/c/183837/4/specs/liberty/approved/generic-scheduling-policies.rst | 15:06 |
| mriedem | Kevin_Zheng: i think that might still be a lie though, | 15:06 |
| mriedem | because just because the user asked for /dev/vda doesn't mean that's where it will show up on the guest | 15:06 |
| mriedem | even though we stored that in cinder | 15:06 |
| Kevin_Zheng | yeah | 15:07 |
| mriedem | this is really why we need to just have a microversion to take device_name out of the volume attach API :) | 15:07 |
| *** dimtruck is now known as zz_dimtruck | 15:07 | |
| mriedem | because we can't honor it | 15:07 |
| mriedem | we don't even try with the libvirt driver | 15:07 |
| mriedem | we ignore it | 15:07 |
| Kevin_Zheng | yeah | 15:07 |
| Kevin_Zheng | I will try to put up one | 15:07 |
| Kevin_Zheng | if nobody does | 15:07 |
| *** efried has quit IRC | 15:07 | |
| mriedem | thanks, this is something we've talked about in previous summits and meetups and just never had someone write a spec | 15:08 |
| mriedem | but it comes up as a problem about every 3 months :) | 15:08 |
| Kevin_Zheng | ：） | 15:08 |
| Kevin_Zheng | but still | 15:08 |
| Kevin_Zheng | we will have none in cinder if not provide one | 15:08 |
| bauzas | gibi: yup, I reviewed it even :) | 15:09 |
| Kevin_Zheng | anyway to sync it？ | 15:09 |
| gibi | bauzas: I think that spec contains a lot of nice idea and discussion on this matter | 15:09 |
| *** armax has quit IRC | 15:09 | |
| mriedem | Kevin_Zheng: let me check the cinder api | 15:11 |
| mriedem | if mountpoint is required then we'd have an issue | 15:11 |
| Kevin_Zheng | mriedem: Im using phone so I can't send you the code，but there is a if condation in | 15:12 |
| Kevin_Zheng | nova's attach function | 15:13 |
| Kevin_Zheng | it checked volume.attach_status =detached | 15:13 |
| *** john5223 has joined #openstack-nova | 15:14 | |
| Kevin_Zheng | maybe we can also check device name there，if none，send again the | 15:14 |
| Kevin_Zheng | right one | 15:14 |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add deprecate-nova-cert spec https://review.openstack.org/448882 | 15:15 |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add deprecate-nova-cert spec https://review.openstack.org/448882 | 15:16 |
| *** zz_dimtruck is now known as dimtruck | 15:17 | |
| *** efried has joined #openstack-nova | 15:18 | |
| mriedem | Kevin_Zheng: looks like the mountpoint is required in the os-attach API in cinder | 15:19 |
| mriedem | in fact, it looks like i can trigger a 500 error if i don't include it in the request body | 15:19 |
| *** liangy has quit IRC | 15:19 | |
| *** felipemonteiro has quit IRC | 15:21 | |
| *** cdent has joined #openstack-nova | 15:25 | |
| mriedem | melwitt: dansmith: I think Kevin_Zheng found another fun local delete cell0 issue :) | 15:26 |
| mriedem | not totally sure yet, but we don't call http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n2144 when doing the local delete case where the instance isn't in a cell | 15:26 |
| *** moshele has quit IRC | 15:26 | |
| mriedem | so the instance is in error state but has volumes attached, and we don't cleanup the volumes | 15:26 |
| mriedem | i think this would be (1) create server, (2) shelve it, (3) attach volume (4) unshelve - fails, puts instance in error state (5) delete | 15:27 |
| mriedem | the question is, does the instance got to cell0 on the unshelve failure in step 4? | 15:27 |
| dansmith | shouldn't, | 15:28 |
| dansmith | because you can't move the instance, so it wouldn't reschedule to cell0 if it's already in cell1, which it has to be because it was shelved in the first place | 15:28 |
| mriedem | madhaviy: yes the policy check happens on the compute, which is odd, i've brought this up in the mailing list before, i'd have to dig that up | 15:29 |
| mriedem | dansmith: but on shelve offload the instance no longer has a host right? | 15:29 |
| mriedem | althought, | 15:29 |
| mriedem | it should have an instance mapping | 15:29 |
| mriedem | back to the cell it's in | 15:29 |
| dansmith | mriedem: sure, but it's in a cell | 15:29 |
| mriedem | ok | 15:29 |
| dansmith | you can't move the instance record from one db to the other Imeant | 15:29 |
| mriedem | but a shelved offloaded instance will have an instance mapping, | 15:29 |
| mriedem | so it's in a cell | 15:30 |
| mriedem | even if instance.host is NOne | 15:30 |
| madhaviy | mriedem: sure | 15:30 |
| dansmith | right | 15:30 |
| *** felipemonteiro__ has quit IRC | 15:30 | |
| mriedem | so in that case, | 15:31 |
| mriedem | on delete, we still get here http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n1847 | 15:31 |
| mriedem | and the cell could be cell1 or whatever | 15:31 |
| mriedem | and we just destroy the instance here http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n1887 | 15:31 |
| *** nicolasbock has quit IRC | 15:32 | |
| mriedem | which bypasses http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n2206 | 15:32 |
| mriedem | if i'm following this correctly | 15:32 |
| *** liangy has joined #openstack-nova | 15:32 | |
| mriedem | basically, for local delete we've duplicated the paths, and are missing some things from the case that we don't have instance.host but it's in a cell | 15:33 |
| mriedem | i think i could write a functional test for this pretty easily, | 15:34 |
| mriedem | and verify if the bdm was destroyed when the instance was deleted or not | 15:34 |
| *** crushil has quit IRC | 15:36 | |
| *** slaweq_ has quit IRC | 15:41 | |
| *** slaweq has joined #openstack-nova | 15:41 | |
| *** pumaranikar has joined #openstack-nova | 15:42 | |
| *** armax has joined #openstack-nova | 15:42 | |
| *** felipemonteiro has joined #openstack-nova | 15:42 | |
| *** hshiina has joined #openstack-nova | 15:44 | |
| *** ttx has quit IRC | 15:44 | |
| *** ttx has joined #openstack-nova | 15:46 | |
| *** sneti_ has joined #openstack-nova | 15:47 | |
| *** dimtruck is now known as zz_dimtruck | 15:48 | |
| *** felipemonteiro has quit IRC | 15:48 | |
| *** nicolasbock has joined #openstack-nova | 15:49 | |
| *** Swami has joined #openstack-nova | 15:49 | |
| *** karimb has quit IRC | 15:50 | |
| *** vgadiraj has joined #openstack-nova | 15:50 | |
| *** crushil has joined #openstack-nova | 15:52 | |
| *** aunnam has joined #openstack-nova | 15:53 | |
| *** karimb has joined #openstack-nova | 15:53 | |
| *** amoralej is now known as amoralej|brb | 15:54 | |
| *** peter-hamilton has joined #openstack-nova | 15:54 | |
| *** tjones has joined #openstack-nova | 15:56 | |
| *** nkrinner is now known as nkrinner_afk | 15:57 | |
| *** dharinic- has joined #openstack-nova | 15:58 | |
| *** david-lyle_ is now known as david-lyle | 15:58 | |
| *** belmorei_ has quit IRC | 15:59 | |
| *** zz_dimtruck is now known as dimtruck | 16:00 | |
| *** jdurgin has quit IRC | 16:07 | |
| *** unicell has joined #openstack-nova | 16:07 | |
| mriedem | weird, the python-novaclient jobs are timing out since 3/21 | 16:07 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Killed%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20timeout%20-s%209%5C%22%20AND%20tags%3A%5C%22console%5C%22%20AND%20project%3A%5C%22openstack%2Fpython-novaclient%5C%22&from=7d | 16:07 |
| *** unicell1 has quit IRC | 16:09 | |
| *** awaugama_ has quit IRC | 16:10 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add remove-nova-cert spec https://review.openstack.org/448882 | 16:10 |
| *** belmoreira has joined #openstack-nova | 16:10 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add remove-nova-cert spec https://review.openstack.org/448882 | 16:10 |
| *** bkopilov has joined #openstack-nova | 16:11 | |
| *** awaugama has joined #openstack-nova | 16:11 | |
| sdague | mriedem: well there are some crash dump tests in there that are taking 4 minutes to run | 16:11 |
| sdague | mriedem: http://logs.openstack.org/41/449041/1/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/191f2b4/console.html#_2017-03-23_11_04_17_455195 | 16:12 |
| *** udesale has quit IRC | 16:12 | |
| sdague | something else is going on, the tests passed, then zuul forgot about it for an hour | 16:13 |
| *** belmoreira has quit IRC | 16:13 | |
| mriedem | yeah | 16:14 |
| mriedem | 2017-03-23 11:04:17.455195 | 2017-03-23 11:04:17.449 | Finished | 16:14 |
| mriedem | 2017-03-23 12:00:48.459991 | /home/jenkins/workspace/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/devstack-gate/functions.sh: line 1074: 3542 Killed | 16:14 |
| mriedem | so 56 minutes of nothing | 16:14 |
| sdague | I saw a thing like that before and brought it up in infra, I can't remember now what was getting lost | 16:14 |
| *** eharney has quit IRC | 16:17 | |
| *** nic has joined #openstack-nova | 16:18 | |
| *** eharney has joined #openstack-nova | 16:19 | |
| *** dtp has joined #openstack-nova | 16:19 | |
| *** markus_z has quit IRC | 16:20 | |
| *** mdrabe has quit IRC | 16:21 | |
| efried | harlowja One thing I'm finding with TaskFlow is it's kinda tricky to unit test around. | 16:21 |
| efried | Especially with nested flows and such. | 16:22 |
| efried | If I mock the whole TaskFlow apparatus, I have to test the individual tasks separately and can't get good verification of the overall flow with dependencies and such. | 16:23 |
| johnthetubaguy | tojuvone: are you still about, I forgot to ping you when I got back | 16:23 |
| tojuvone | hieulq, yes, still here | 16:24 |
| *** jamesden_ has joined #openstack-nova | 16:24 | |
| edmondsw | johnthetubaguy are you aware that there is a policy rule (network:attach_external_network) being checked by nova-compute? | 16:24 |
| efried | harlowja But if I don't, I better not try to mock any of the individual tasks - too much internal stuff is expected. | 16:24 |
| efried | Not a show-stopper. And not sure what could be done to mitigate - maybe taskflow lib providing some stock fixtures? | 16:25 |
| sdague | mriedem: are there no individual test timeouts on these? | 16:25 |
| edmondsw | johnthetubaguy just found that doesn't work if I don't put a policy file on a host... ugh... we should only be checking policy in nova-api | 16:25 |
| johnthetubaguy | edmondsw: you reminded me, I forgot about that one again, there are a few strange stragglers like that | 16:25 |
| tojuvone | johnthetubaguy, yes, so was wondering the maintenance_events. Wither own table or in services table | 16:25 |
| tojuvone | johnthetubaguy, services exposed to servers so that works | 16:26 |
| mriedem | sdague: i'm not seeing any | 16:26 |
| edmondsw | johnthetubaguy that's gotta be a defect, right? There's even talk about nova-compute being split out from nova. It shouldn't be relying on nova's policy file | 16:26 |
| tojuvone | johnthetubaguy, but in services need to duplicate on each service on host | 16:26 |
| mriedem | sdague: i'm seeing rabbit have memory limit warnings around the time we lose the logs | 16:26 |
| *** sridharg has quit IRC | 16:26 | |
| sdague | yeh, there is no individual test timeout, the test is still running and hung | 16:27 |
| mriedem | edmondsw: see my reply to madhaviy above | 16:27 |
| *** slaweq has quit IRC | 16:27 | |
| edmondsw | mriedem, ah, didn't realize we'd already had this discussion... | 16:27 |
| mriedem | sdague: doesn't testr have a 500s timeout per test? | 16:27 |
| johnthetubaguy | tojuvone: given what I described, I think it would get its own table, but not sure if everyone likes that direction, I think we should focus on getting the use cases crisp first | 16:27 |
| *** jamesden_ has quit IRC | 16:28 | |
| mriedem | edmondsw: madhaviy: johnthetubaguy: https://openstack.nimeyo.com/87011/openstack-policy-check-network-attach_external_network | 16:28 |
| openstackgerrit | Sean Dague proposed openstack/python-novaclient master: set test timout to 300 seconds https://review.openstack.org/449211 | 16:28 |
| sdague | mriedem: no, testr enforces nothing | 16:28 |
| tojuvone | johnthetubaguy, yes, trying to update a lot including the cases and all. | 16:28 |
| *** jamesden_ has joined #openstack-nova | 16:28 | |
| johnthetubaguy | tojuvone: yeah, I would just leave the implementation bit alone for now, lets agree these use cases first, particularly the scheduler related ones | 16:29 |
| sdague | mriedem: all our test timeouts are done via test fixtures | 16:29 |
| mriedem | sdague: ah | 16:29 |
| mriedem | i see we set that for nova but not novaclient | 16:29 |
| sdague | yep | 16:29 |
| johnthetubaguy | edmondsw: there were other ones I think sneti and aunnam found that are accessed outside the API | 16:30 |
| mriedem | self.useFixture(fixtures.Timeout(test_timeout, gentle=True)) | 16:30 |
| mriedem | yar | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Set instance.availability_zone whenever we schedule https://review.openstack.org/446053 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | 16:30 |
| tojuvone | johnthetubaguy, ok, sounds good to make better progress | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 16:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | 16:30 |
| sdague | mriedem: that should at least give us different results, and not have the big timeout kill us | 16:30 |
| mriedem | sdague: yeah, thanks | 16:30 |
| johnthetubaguy | edmondsw: FWIW, I think the neutron refactor work is heading in a direction that would fix that policy issue, anyways | 16:31 |
| sneti | johnthetubaguy, yes..this is the one: https://review.openstack.org/#/c/448260/2/nova/policies/cells_scheduler.py | 16:31 |
| *** jamesden_ has quit IRC | 16:31 | |
| sdague | mriedem: is the crash dump testing a thing we do other places? | 16:31 |
| mriedem | sdague: not sure about tempest | 16:31 |
| tojuvone | johnthetubaguy, I should have that part by tomorrow and can continue from that | 16:31 |
| johnthetubaguy | tojuvone: sounds great, ping me when thats up | 16:32 |
| mriedem | sdague: oomichi probably knows, i think he added it | 16:32 |
| mriedem | or was involved | 16:32 |
| tojuvone | johnthetubaguy, thanks, I will | 16:32 |
| *** lucasxu has quit IRC | 16:32 | |
| johnthetubaguy | edmondsw: so sounds like some scheduler filters need policy too | 16:33 |
| johnthetubaguy | edmondsw: that should probably get fixed too | 16:33 |
| edmondsw | johnthetubaguy I'd say let's open defects for these, so they're not lost... and if they get fixed apart from the defects we can always cancel them. Your thoughts? | 16:34 |
| johnthetubaguy | edmondsw: yeah, I have asked sneti and aunnam to do that as they find these | 16:34 |
| *** ssurana has joined #openstack-nova | 16:34 | |
| edmondsw | great, tx | 16:34 |
| johnthetubaguy | or at least I mean to... | 16:35 |
| edmondsw | sneti, have you opened one for the external network attach case, or will you, or should I? | 16:35 |
| edmondsw | sneti, if you haven't already, I'd say you open the scheduler one you found and I can open the network one | 16:35 |
| *** slaweq has joined #openstack-nova | 16:36 | |
| oomichi | mriedem: sdague: temepst doesn't have crash-dump test, I guess cirros cannot take crash-dump because it requires some crash-dump tool | 16:36 |
| johnthetubaguy | edmondsw: +1 | 16:36 |
| oomichi | even if nova triggers it | 16:36 |
| *** liangy has quit IRC | 16:36 | |
| sdague | oomichi: ok, so why are we running those tests? | 16:36 |
| mriedem | oomichi: but we have crash dump tests in novaclient functional tests | 16:36 |
| oomichi | mriedem: oh, I didn't that novaclient has | 16:37 |
| *** hshiina has quit IRC | 16:38 | |
| *** felipemonteiro has joined #openstack-nova | 16:38 | |
| *** mdrabe has joined #openstack-nova | 16:39 | |
| oomichi | sdague: that depends on what kind of test do we want. Cirros is enough if we just want to call the API I feel. but if we want to verify the crashdump file is created in the virtual machine, the tool is required | 16:39 |
| mriedem | oomichi: yeah and they take about 4 minutes to run | 16:39 |
| mriedem | http://logs.openstack.org/57/447657/1/gate/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/b3aaf58/console.html#_2017-03-23_12_21_00_379603 | 16:39 |
| *** jianghuaw-m has joined #openstack-nova | 16:40 | |
| openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: WIP: honor disk_format=ploop image metadata property https://review.openstack.org/429745 | 16:40 |
| oomichi | mriedem: yeah, because crash dump takes 1) switch to 2nd-kernel, 2) copy system memory of 1st kernel to a file 3) reboot the system | 16:40 |
| oomichi | need much time | 16:41 |
| *** amoralej|brb is now known as amoralej | 16:41 | |
| *** Apoorva has joined #openstack-nova | 16:46 | |
| *** corey_ has joined #openstack-nova | 16:46 | |
| *** adrian_otto1 has joined #openstack-nova | 16:46 | |
| *** andreas_s has quit IRC | 16:47 | |
| *** liangy has joined #openstack-nova | 16:47 | |
| *** cleong has quit IRC | 16:47 | |
| edmondsw | johnthetubaguy mriedem sneti: https://bugs.launchpad.net/nova/+bug/1675486 | 16:47 |
| openstack | Launchpad bug 1675486 in OpenStack Compute (nova) "network:attach_external_network policy check outside nova-api" [Undecided,New] | 16:47 |
| johnthetubaguy | sweet, good to capture that, not 100% sure how we want to fix that | 16:48 |
| *** adrian_otto has quit IRC | 16:49 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 16:52 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | 16:52 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 16:52 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | 16:52 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 16:52 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 16:52 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 16:52 |
| sneti | edmondsw, sure..I will open a bug for the scheduler one | 16:53 |
| edmondsw | sneti tx | 16:54 |
| *** Apoorva_ has joined #openstack-nova | 16:56 | |
| *** felipemonteiro_ has joined #openstack-nova | 16:56 | |
| *** ltomasbo is now known as ltomasbo|away | 16:57 | |
| harlowja | efried very much agreed | 16:58 |
| harlowja | some included fixtures i think would help | 16:59 |
| *** Apoorva has quit IRC | 17:00 | |
| *** jianghuaw-m has quit IRC | 17:00 | |
| *** ltomasbo|away is now known as ltomasbo | 17:06 | |
| *** unicell has quit IRC | 17:06 | |
| *** READ10 has quit IRC | 17:06 | |
| *** catintheroof has joined #openstack-nova | 17:07 | |
| *** migi is now known as migi_afk | 17:07 | |
| *** migi_afk is now known as migi | 17:07 | |
| *** nkorabli has quit IRC | 17:07 | |
| *** nicolasbock has quit IRC | 17:09 | |
| *** nicolasbock has joined #openstack-nova | 17:14 | |
| *** amotoki has quit IRC | 17:14 | |
| *** pcaruana has quit IRC | 17:14 | |
| *** crushil has quit IRC | 17:15 | |
| sfinucan | johnthetubaguy: Might be interested in https://review.openstack.org/#/c/446898/ | 17:15 |
| sfinucan | if gcb hadn't already mentioned it to you | 17:15 |
| *** crushil has joined #openstack-nova | 17:17 | |
| *** mvk has quit IRC | 17:17 | |
| *** lpetrut has quit IRC | 17:18 | |
| *** gyee has joined #openstack-nova | 17:20 | |
| *** nkorabli has joined #openstack-nova | 17:20 | |
| *** vladikr has joined #openstack-nova | 17:23 | |
| *** vladikr has quit IRC | 17:23 | |
| *** fragatina has quit IRC | 17:24 | |
| edleafe | mdbooth: kashyap: still around? | 17:26 |
| kashyap | edleafe: Yeah, but just about to bike out to the near by university for a prior appointment | 17:27 |
| kashyap | edleafe: I think you have the logs :-) | 17:27 |
| *** vladikr has joined #openstack-nova | 17:27 | |
| *** tbachman has quit IRC | 17:27 | |
| kashyap | edleafe: If it's a quick question, then I have about 5 mins before I head out the door. Otherwise, do write here, I'll read the scroll when I get back & respond. | 17:27 |
| edleafe | kashyap: wow, you're psychic! :) | 17:27 |
| kashyap | Heh | 17:28 |
| edleafe | kashyap: wondering if you wanted me to send them to you to review | 17:28 |
| edleafe | or should I paste the relevant sections | 17:28 |
| kashyap | edleafe: Sure, do email them; but I won't promise that I'll see them tonight. Tomorrow I can. | 17:28 |
| kashyap | edleafe: No, entire log would be useful. Why? | 17:29 |
| kashyap | Because I can correlate the unique IDs libvirt sends out for each action it tries to coordinate with QEMU | 17:29 |
| edleafe | kashyap: dunno, just being careful | 17:29 |
| kashyap | If you accidentally snip out some request / response, I might lose some context | 17:29 |
| *** jaosorior has quit IRC | 17:30 | |
| edleafe | kashyap: makes sense | 17:30 |
| *** lucasxu has joined #openstack-nova | 17:32 | |
| *** ociuhandu has quit IRC | 17:33 | |
| *** gyee has quit IRC | 17:33 | |
| *** gyee has joined #openstack-nova | 17:34 | |
| *** tbachman has joined #openstack-nova | 17:35 | |
| *** jianghuaw-m has joined #openstack-nova | 17:36 | |
| *** kenperkins has quit IRC | 17:37 | |
| *** gyee has quit IRC | 17:39 | |
| *** slaweq_ has joined #openstack-nova | 17:39 | |
| *** lucasagomes is now known as lucas-afk | 17:41 | |
| *** jianghuaw-m has quit IRC | 17:41 | |
| *** madhaviy has quit IRC | 17:41 | |
| *** slaweq has quit IRC | 17:41 | |
| *** krtaylor has quit IRC | 17:42 | |
| *** derekh has quit IRC | 17:43 | |
| *** fragatina has joined #openstack-nova | 17:45 | |
| *** karimb has quit IRC | 17:46 | |
| *** adrian_otto1 has quit IRC | 17:52 | |
| *** links has joined #openstack-nova | 17:52 | |
| *** mvk has joined #openstack-nova | 17:53 | |
| *** slaweq_ has quit IRC | 17:54 | |
| *** slaweq has joined #openstack-nova | 17:56 | |
| *** adrian_otto has joined #openstack-nova | 17:58 | |
| *** links has quit IRC | 17:58 | |
| *** Drankis has quit IRC | 17:59 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read NIC features in libvirt https://review.openstack.org/444415 | 18:01 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Modify PciPassthroughFilter to accept lists https://review.openstack.org/444946 | 18:01 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 18:01 |
| mriedem | sdague: yeah those novaclient tests are timing out test_trigger_crash_dump_in_resized_state | 18:01 |
| johnthetubaguy | mriedem: I hit some nasty rebase things on that refactor patch, will have to resolve those in the morning | 18:03 |
| mriedem | johnthetubaguy: ok i'll do them today if i get a chance | 18:03 |
| *** ralonsoh has quit IRC | 18:04 | |
| *** ZZelle has joined #openstack-nova | 18:06 | |
| *** catintheroof has quit IRC | 18:07 | |
| sdague | mriedem: yeh, I honestly think the crashdump tests probably have to be turned off | 18:08 |
| *** ltomasbo is now known as ltomasbo|away | 18:08 | |
| *** jpena is now known as jpena|away | 18:08 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in cells_scheduler.py https://review.openstack.org/448260 | 18:08 |
| openstackgerrit | John Garbutt proposed openstack/nova master: compute: Move detach logic from manager into driver BDM https://review.openstack.org/439520 | 18:08 |
| mriedem | johnthetubaguy: i guess it wasn't that bad huh :) | 18:09 |
| johnthetubaguy | mriedem: so fixed my nit, but that needs a rebase after those patches we just approved | 18:09 |
| mriedem | ok | 18:10 |
| * johnthetubaguy goes for food and band practice | 18:11 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | 18:12 |
| mriedem | sdague: alex_xu suggested that if we make that single PUT method for updating a service, we also change the response to include the full resource, like a GET call would https://review.openstack.org/#/c/447149/2/specs/pike/approved/service-hyper-uuid-in-api.rst@208 | 18:12 |
| mriedem | sdague: i'd be OK with that, what do you think? | 18:12 |
| *** ujjain has quit IRC | 18:13 | |
| *** gjayavelu has joined #openstack-nova | 18:14 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | 18:15 |
| *** vks1 has quit IRC | 18:15 | |
| *** adrian_otto has quit IRC | 18:16 | |
| openstackgerrit | Vasyl Saienko proposed openstack/nova master: Implement interface attach/detach in ironic virt driver https://review.openstack.org/419975 | 18:16 |
| *** adrian_otto has joined #openstack-nova | 18:17 | |
| efried | mriedem ^^ It's been tough getting flake8-import-order, rebasing, and subteam +1s all coordinated, but this one is hopefully ready to go again. | 18:17 |
| *** Sukhdev has joined #openstack-nova | 18:17 | |
| efried | [Mine, not Vasyl's ;-)] | 18:17 |
| *** fengzhr has quit IRC | 18:18 | |
| sdague | mriedem: I am ok with full resource | 18:18 |
| sdague | I also think it's not a terrible idea to fix that os-hypervisor query interface that's crazy at the same time | 18:18 |
| sdague | given that everyone is going to need to adapt code forthis | 18:19 |
| *** fengzhr has joined #openstack-nova | 18:20 | |
| *** nkorabli has quit IRC | 18:22 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | 18:24 |
| openstackgerrit | Monty Taylor proposed openstack/nova master: WIP Clean up and consolidate client creation https://review.openstack.org/438512 | 18:25 |
| mriedem | sdague: so i should just rename to "complete unf*ch os-services and os-hypervisors" :) | 18:27 |
| sdague | mriedem: ++ | 18:28 |
| mriedem | wow spelling | 18:28 |
| *** nicolasbock has quit IRC | 18:28 | |
| *** dtp has quit IRC | 18:33 | |
| *** nkorabli has joined #openstack-nova | 18:33 | |
| *** nicolasbock has joined #openstack-nova | 18:37 | |
| *** lucasxu has quit IRC | 18:38 | |
| openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/357151 | 18:38 |
| mriedem | sdague: ah it looks like we timeout and never kill a timer http://logs.openstack.org/11/449211/1/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/9839aac/console.html#_2017-03-23_18_04_43_739252 | 18:40 |
| mriedem | waiting for a server to build | 18:40 |
| mriedem | i'll take a look | 18:40 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | 18:41 |
| sdague | it's kind of suspicious that after a bunch of crash dumping rabbit goes sideways | 18:42 |
| sdague | I'd start with disabling all those crashdump tests first | 18:42 |
| mriedem | i seem to remember andreykurilin fixing something where the crash dump tests started taking a long time | 18:43 |
| mriedem | andreykurilin: if you're around, do you remember something like that? | 18:43 |
| *** unicell has joined #openstack-nova | 18:43 | |
| openstackgerrit | Sean Dague proposed openstack/python-novaclient master: set test timout to 300 seconds https://review.openstack.org/449211 | 18:45 |
| sdague | that's got the skip in there | 18:45 |
| mriedem | we can use bug test_trigger_crash_dump_in_shutoff_state | 18:45 |
| mriedem | oops | 18:45 |
| mriedem | bug 1675526 | 18:45 |
| openstack | bug 1675526 in python-novaclient "Trigger crash dump functional tests are timing out and hanging" [High,Triaged] https://launchpad.net/bugs/1675526 | 18:45 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729 | 18:45 |
| sdague | ok, feel free to update if you like, got to run away from keyboard for a bit | 18:45 |
| mriedem | ok | 18:46 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Set test timout to 300 seconds https://review.openstack.org/449211 | 18:55 |
| *** baoli has quit IRC | 19:00 | |
| *** baoli has joined #openstack-nova | 19:01 | |
| *** sneti has quit IRC | 19:07 | |
| *** sneti has joined #openstack-nova | 19:07 | |
| *** aunnam_ has joined #openstack-nova | 19:08 | |
| openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavor API https://review.openstack.org/449288 | 19:09 |
| *** crushil has quit IRC | 19:11 | |
| openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavor API https://review.openstack.org/449288 | 19:12 |
| *** sree has joined #openstack-nova | 19:12 | |
| *** Jack_Iv has joined #openstack-nova | 19:14 | |
| *** krtaylor has joined #openstack-nova | 19:15 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | 19:16 |
| *** sree has quit IRC | 19:17 | |
| *** baoli has quit IRC | 19:17 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228 | 19:20 |
| *** adrian_otto has quit IRC | 19:24 | |
| *** adrian_otto has joined #openstack-nova | 19:26 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402 | 19:26 |
| *** baoli has joined #openstack-nova | 19:26 | |
| *** fragatina has quit IRC | 19:27 | |
| *** tesseract has quit IRC | 19:28 | |
| *** eharney has quit IRC | 19:33 | |
| *** baoli has quit IRC | 19:34 | |
| *** adrian_otto has quit IRC | 19:34 | |
| *** mlavalle has quit IRC | 19:34 | |
| *** kenperkins has joined #openstack-nova | 19:37 | |
| *** baoli has joined #openstack-nova | 19:44 | |
| *** awaugama has quit IRC | 19:45 | |
| *** suresh12 has joined #openstack-nova | 19:45 | |
| *** amoralej is now known as amoralej|off | 19:46 | |
| *** eharney has joined #openstack-nova | 19:49 | |
| *** xyang1 has joined #openstack-nova | 19:57 | |
| *** annegentle has quit IRC | 20:00 | |
| *** annegentle has joined #openstack-nova | 20:00 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix description of volumeAttachment for attach/swap-volume https://review.openstack.org/449302 | 20:02 |
| *** Jack_Iv has quit IRC | 20:03 | |
| *** Jack_Iv has joined #openstack-nova | 20:04 | |
| *** Jack_Iv has quit IRC | 20:04 | |
| *** suresh12 has quit IRC | 20:06 | |
| *** avolkov has quit IRC | 20:07 | |
| *** belmoreira has joined #openstack-nova | 20:07 | |
| *** suresh12 has joined #openstack-nova | 20:07 | |
| *** belmoreira has quit IRC | 20:07 | |
| *** fragatina has joined #openstack-nova | 20:09 | |
| *** fragatina has quit IRC | 20:10 | |
| *** fragatina has joined #openstack-nova | 20:10 | |
| *** baoli has quit IRC | 20:12 | |
| *** ekuris has quit IRC | 20:17 | |
| *** satyar has quit IRC | 20:21 | |
| *** ujjain has joined #openstack-nova | 20:22 | |
| *** ujjain has quit IRC | 20:22 | |
| *** ujjain has joined #openstack-nova | 20:22 | |
| *** jpena|away is now known as jpena|off | 20:24 | |
| *** awaugama has joined #openstack-nova | 20:26 | |
| *** dtp has joined #openstack-nova | 20:27 | |
| *** mlavalle has joined #openstack-nova | 20:29 | |
| *** liangy has quit IRC | 20:29 | |
| *** eharney has quit IRC | 20:30 | |
| *** baoli has joined #openstack-nova | 20:32 | |
| *** ekuris has joined #openstack-nova | 20:33 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | 20:33 |
| *** corey_ has quit IRC | 20:38 | |
| *** karimb has joined #openstack-nova | 20:40 | |
| mriedem | sdague: in the api-ref for server details we have os-extended-volumes:volumes_attached which just says it's an array of attached volumes. at 2.1 it's a list of dicts with a single 'id' key which is the volume id. after 2.3 the list of dicts can also contain the 'delete_on_termination' key. | 20:42 |
| mriedem | we don't describe the actual contents of that dict in the list, | 20:42 |
| mriedem | i.e. the id and delete_on_termination keys, | 20:42 |
| mriedem | i'm trying to figure out what's the best way to represent that, | 20:42 |
| mriedem | would the name be os-extended-volumes:volumes_attached.id? | 20:42 |
| mriedem | os-extended-volumes:volumes_attached.delete_on_termination? | 20:42 |
| sdague | mriedem: that has been the pattern we were using | 20:42 |
| *** karimb has quit IRC | 20:43 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in suspend_server.py https://review.openstack.org/447722 | 20:44 |
| mriedem | sdague: ok, alternatively to avoid the turducken of nested parameters, i could just describe those keys in the description for the os-extended-volumes:volumes_attached parameter? | 20:44 |
| *** mtanino has joined #openstack-nova | 20:45 | |
| sdague | you could, but it sort of breaks the flow | 20:45 |
| sdague | I would honestly do the nested thing | 20:45 |
| mriedem | i'll just do the normal pattern | 20:45 |
| mriedem | ok | 20:45 |
| mriedem | danke | 20:45 |
| mriedem | writing functional regression tests that actually use our api makes you have to actually read the api docs :) | 20:46 |
| * cdent wonders if mriedem is always hungry | 20:46 | |
| mriedem | it's amazing | 20:46 |
| sdague | mriedem: yeh, and it's close enough now that it's worth fixing when you find a weird issue we missed | 20:46 |
| mriedem | yeah that's true | 20:46 |
| mriedem | if it were a needle in a haystack i'd be much less motivated | 20:47 |
| *** cdent has quit IRC | 20:48 | |
| *** lpetrut has joined #openstack-nova | 20:51 | |
| *** mtanino has quit IRC | 20:56 | |
| *** slaweq_ has joined #openstack-nova | 20:56 | |
| *** liangy has joined #openstack-nova | 20:57 | |
| cfriesen | I'm looking at the "realtime" handling of emupcpus in _get_guest_numa_config() and I'm confused about the case where the instance isn't pinned | 20:58 |
| mriedem | cfriesen: sahid and sfinucan are your people | 20:58 |
| mriedem | but aren't around probably | 20:58 |
| *** yamahata has joined #openstack-nova | 20:58 | |
| *** adrian_otto has joined #openstack-nova | 20:58 | |
| cfriesen | yeah...I'll ask tomorrow | 20:59 |
| *** suresh12 has quit IRC | 21:01 | |
| *** slaweq has quit IRC | 21:02 | |
| *** slaweq_ is now known as slaweq | 21:02 | |
| *** suresh12 has joined #openstack-nova | 21:02 | |
| *** gjayavelu has quit IRC | 21:02 | |
| *** gjayavelu has joined #openstack-nova | 21:03 | |
| *** rfolco has quit IRC | 21:06 | |
| *** dimtruck is now known as zz_dimtruck | 21:07 | |
| *** krtaylor has quit IRC | 21:08 | |
| *** sdague has quit IRC | 21:10 | |
| *** aysyd has quit IRC | 21:11 | |
| *** lpetrut has quit IRC | 21:12 | |
| *** slaweq has quit IRC | 21:15 | |
| *** mtanino has joined #openstack-nova | 21:16 | |
| *** slaweq has joined #openstack-nova | 21:18 | |
| *** suresh12 has quit IRC | 21:19 | |
| *** suresh12 has joined #openstack-nova | 21:23 | |
| *** suresh12 has quit IRC | 21:28 | |
| *** esberglu has quit IRC | 21:29 | |
| *** gjayavelu has quit IRC | 21:29 | |
| *** annegentle has quit IRC | 21:31 | |
| *** suresh12 has joined #openstack-nova | 21:33 | |
| *** karimb has joined #openstack-nova | 21:34 | |
| *** pbandark has joined #openstack-nova | 21:34 | |
| mriedem | dansmith: melwitt: just verified that local delete regression when deleting a shelved offloaded server with volumes attached | 21:36 |
| mriedem | will push up the test to show it shortly | 21:36 |
| mriedem | writing the bug report | 21:36 |
| dansmith | uh yeahokay | 21:36 |
| dansmith | upps | 21:36 |
| dansmith | wtf | 21:36 |
| *** zz_dimtruck is now known as dimtruck | 21:36 | |
| *** mtanino has quit IRC | 21:36 | |
| * dansmith is retarded today | 21:36 | |
| melwitt | regression :( | 21:37 |
| mriedem | mfing local delete | 21:38 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1675570 | 21:38 |
| openstack | Launchpad bug 1675570 in OpenStack Compute (nova) "Volumes are not detached when deleting shelved offloaded servers" [High,Triaged] | 21:38 |
| bauzas | mriedem: dansmith: hola, FYI https://review.openstack.org/#/c/391060/ is ready for reviews :) | 21:39 |
| *** peter-hamilton has quit IRC | 21:39 | |
| *** lucasxu has joined #openstack-nova | 21:40 | |
| *** tblakes has quit IRC | 21:40 | |
| melwitt | mriedem: we have the _local_delete function that does all the volume detach stuff, but we never incorporated calling it into the cells v2 short circuits of delete. I think because we were only thinking of the "not yet scheduled" case | 21:41 |
| mriedem | yup | 21:41 |
| mriedem | hold gentle caller, | 21:41 |
| mriedem | for the test | 21:41 |
| melwitt | thanks for the writing the test. *cues karate kid montage again* | 21:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334 | 21:42 |
| mriedem | melwitt: dansmith: ^ | 21:42 |
| mriedem | Kevin_Zheng gets all of the credit for reporting this to me this morning | 21:42 |
| melwitt | *plays montage for Kevin_Zheng too* | 21:42 |
| mriedem | we've never done any decent volume-related functional testing because our cinder fixture wasn't good enough | 21:42 |
| mriedem | that patch builds up the cinder fixture too | 21:43 |
| mriedem | to actually track attach/detach | 21:43 |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in used_limits https://review.openstack.org/447710 | 21:43 |
| melwitt | mriedem: ah, yeah. I remember adding some stuff to that in my famous boot-from-volume patch. and I kept thinking to break it out into a separate thing but didn't | 21:44 |
| melwitt | probably because it would have been adding stuff that nothing was using | 21:44 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334 | 21:45 |
| mriedem | is that like melwitt's famous bbq? | 21:45 |
| melwitt | nevermind, what you did is different than what I was thinking. I had done this https://review.openstack.org/#/c/355091/8/nova/tests/unit/volume/fake.py | 21:45 |
| melwitt | haha | 21:45 |
| *** adrian_otto has quit IRC | 21:47 | |
| *** awaugama has quit IRC | 21:47 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies to availability_zone https://review.openstack.org/440836 | 21:49 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334 | 21:51 |
| mriedem | melwitt: yeah btw if we would have called that _local_delete method, we could have gotten the instance.delete.start/end notifications too | 21:52 |
| mriedem | which was the other regression | 21:52 |
| melwitt | /facepalm | 21:52 |
| mriedem | yuup | 21:53 |
| Apoorva_ | NFS based Cinder connections seem to be broken by Nova changes. Our CI logs http://openstack-ci.tintri.com/tintri/refs-changes-59-448159-4/logs/screen-n-cpu.log.txt | 21:53 |
| Apoorva_ | http://paste.openstack.org/show/603992/ | 21:55 |
| *** suresh12 has quit IRC | 21:56 | |
| openstackgerrit | Sujitha proposed openstack/nova master: [WIP] Add BDM to InstancePayload https://review.openstack.org/448779 | 21:59 |
| openstackgerrit | Sujitha proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 21:59 |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 21:59 |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228 | 21:59 |
| *** jdurgin has joined #openstack-nova | 21:59 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add descripiton to policies in virtual_interfaces.py https://review.openstack.org/445652 | 22:00 |
| Apoorva_ | Can someone help me with this? http://paste.openstack.org/show/603992/ | 22:00 |
| tjones | so i was sitting in openstack-meeting today at 2pm and no one was there… but i see i missed the meeting. so - did it move??? | 22:00 |
| mriedem | tjones: wrong week, today was the 7pm PT | 22:01 |
| mriedem | *7am | 22:01 |
| *** Swami has quit IRC | 22:01 | |
| *** scottda has quit IRC | 22:04 | |
| *** gjayavelu has joined #openstack-nova | 22:05 | |
| *** burt has quit IRC | 22:07 | |
| *** mdrabe has quit IRC | 22:07 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228 | 22:07 |
| *** suresh12 has joined #openstack-nova | 22:07 | |
| mriedem | Apoorva_: have you diffed the _connect_volume method to see what changed the signature? | 22:08 |
| Apoorva_ | lyarwood mriedem would this https://review.openstack.org/#/c/437665/ cause Cinder nfs based driver connections to fail? | 22:08 |
| mriedem | on | 22:08 |
| mriedem | *no | 22:08 |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228 | 22:09 |
| openstackgerrit | Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298 | 22:09 |
| mriedem | Apoorva_: we have an NFS CI job running in the community infra, are you running a 3rd party Cinder NFS driver CI? | 22:09 |
| mriedem | this is the failure | 22:09 |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 22:09 |
| mriedem | TypeError: connect_volume() takes exactly 3 arguments (4 given) | 22:09 |
| Apoorva_ | mriedem our driver inherits nfs driver. | 22:10 |
| melwitt | I see something, hang on | 22:10 |
| mriedem | my guess would be https://github.com/openstack/nova/commit/f0153fa4c87528e1dcbda8cba55d34ab7aec4e7a | 22:10 |
| melwitt | https://review.openstack.org/#/c/438970/ | 22:11 |
| melwitt | yeah, I was trying to find the review | 22:11 |
| mriedem | yeah, so mdbooth missed the nfs driver | 22:12 |
| mriedem | in that change, | 22:12 |
| mriedem | probably because he's handling it in https://review.openstack.org/#/c/383859/ which isn't merged | 22:12 |
| mriedem | Apoorva_: please open a bug | 22:12 |
| Apoorva_ | mriedem will do that | 22:12 |
| mriedem | Apoorva_: thanks, post it here and we'll get it fixed | 22:15 |
| openstackgerrit | Sujitha proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | 22:17 |
| openstackgerrit | Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146 | 22:18 |
| Apoorva_ | mriedem melwitt https://bugs.launchpad.net/nova/+bug/1675584 | 22:19 |
| openstack | Launchpad bug 1675584 in OpenStack Compute (nova) "TypeError: connect_volume() takes exactly 3 arguments (4 given) for NFS Cinder drivers" [Undecided,New] | 22:19 |
| openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in cells_scheduler.py https://review.openstack.org/448260 | 22:28 |
| *** baoli has quit IRC | 22:30 | |
| *** nkorabli has quit IRC | 22:30 | |
| *** jdurgin has quit IRC | 22:30 | |
| tjones | mriedem: thanks - my schedule is messed up. stupid outloook…. i hate it | 22:31 |
| *** sneti has quit IRC | 22:32 | |
| *** dave-mcc_ has joined #openstack-nova | 22:33 | |
| *** xyang1 has quit IRC | 22:33 | |
| *** liangy has quit IRC | 22:36 | |
| *** dave-mccowan has quit IRC | 22:36 | |
| *** dave-mccowan has joined #openstack-nova | 22:38 | |
| *** lucasxu has quit IRC | 22:39 | |
| *** dave-mcc_ has quit IRC | 22:40 | |
| *** nic has quit IRC | 22:46 | |
| *** unicell1 has joined #openstack-nova | 22:46 | |
| *** unicell has quit IRC | 22:46 | |
| *** marst has quit IRC | 22:50 | |
| *** jamesden_ has joined #openstack-nova | 22:50 | |
| *** jamesden_ has quit IRC | 22:50 | |
| *** tjones has quit IRC | 22:50 | |
| mriedem | i also hate outlook | 22:54 |
| mriedem | Apoorva_: thanks, will push a fix shortly | 22:54 |
| Apoorva_ | mriedem appreciate it! | 22:55 |
| mriedem | gluster is also broken | 22:56 |
| *** ujjain has quit IRC | 22:57 | |
| mriedem | and quobyte | 22:57 |
| *** felipemonteiro has quit IRC | 22:59 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 23:00 |
| kaisers | mriedem: oh, good this is in the works. Saw the fails earlier and did not come around to take a more detailed look so far. Thanks! | 23:00 |
| kaisers | *ka-ching* and it's midnight | 23:00 |
| *** ujjain has joined #openstack-nova | 23:01 | |
| *** ujjain has quit IRC | 23:01 | |
| *** ujjain has joined #openstack-nova | 23:01 | |
| *** efried has quit IRC | 23:01 | |
| mriedem | np | 23:01 |
| mriedem | i'm making these abc's, they should have been anyway | 23:01 |
| mriedem | i have no idea why scality doesn't implement disconnect_volume | 23:04 |
| *** aunnam_ has quit IRC | 23:06 | |
| kaisers | hmm, same here. But i'm beyond useful anyways. Off for a few hrs of zzzzZZzz | 23:10 |
| mriedem | man the scality driver was added 4 years ago | 23:10 |
| *** efried has joined #openstack-nova | 23:11 | |
| *** suresh12 has quit IRC | 23:11 | |
| *** kaisers has quit IRC | 23:13 | |
| *** catintheroof has joined #openstack-nova | 23:13 | |
| *** sree has joined #openstack-nova | 23:13 | |
| *** sree has quit IRC | 23:18 | |
| *** catintheroof has quit IRC | 23:20 | |
| *** catintheroof has joined #openstack-nova | 23:21 | |
| mriedem | weird, the rbd driver doesn't implement connect_volume | 23:25 |
| *** takashin has joined #openstack-nova | 23:26 | |
| *** karimb has quit IRC | 23:27 | |
| *** pbandark has quit IRC | 23:31 | |
| *** catintheroof has quit IRC | 23:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Revert "libvirt: Pass instance to connect_volume and ..." https://review.openstack.org/449362 | 23:33 |
| mriedem | melwitt: dansmith: witness https://review.openstack.org/#/c/449362/ | 23:34 |
| dansmith | omy | 23:34 |
| mriedem | i was fixing it locally but decided, you know what, nothing uses this, it's 6:34pm, and i have other things to do, | 23:35 |
| mriedem | so fuck it | 23:35 |
| dansmith | oh I was going to +W it for you but.. I guess you did :) | 23:35 |
| mriedem | feel free to pile on | 23:35 |
| *** tjones has joined #openstack-nova | 23:36 | |
| *** mlavalle has quit IRC | 23:36 | |
| mriedem | Apoorva_: https://review.openstack.org/#/c/449362/ is the fix for now | 23:38 |
| *** suresh12 has joined #openstack-nova | 23:38 | |
| *** catintheroof has joined #openstack-nova | 23:38 | |
| *** hongbin has quit IRC | 23:38 | |
| mriedem | oh also the scality volume driver was removed from cinder in ocata, | 23:39 |
| mriedem | so the libvirt volume driver for scality does nothing right now anyway | 23:39 |
| mriedem | so i'll follow up on removing that too | 23:39 |
| *** nicolasbock has quit IRC | 23:40 | |
| Apoorva_ | mriedem thanks! | 23:40 |
| *** kaisers has joined #openstack-nova | 23:40 | |
| melwitt | revert makes sense to me too | 23:41 |
| *** suresh12 has quit IRC | 23:42 | |
| *** adrian_otto has joined #openstack-nova | 23:42 | |
| *** vishwanathj has quit IRC | 23:43 | |
| *** tjones has quit IRC | 23:44 | |
| *** kaisers has quit IRC | 23:44 | |
| *** catintheroof has quit IRC | 23:45 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | 23:47 |
| *** efried has quit IRC | 23:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: remove scality volume driver https://review.openstack.org/449364 | 23:55 |
| mriedem | jgriffith: there you go ^ | 23:55 |
| *** armax has quit IRC | 23:55 | |
| *** efried has joined #openstack-nova | 23:58 | |
| jgriffith | mriedem you have my 2cents | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!