*** stewie925 has quit IRC | 00:02 | |
*** baoli has joined #openstack-nova | 00:06 | |
*** baoli has quit IRC | 00:06 | |
*** sthillma has quit IRC | 00:07 | |
*** shaohe_feng has quit IRC | 00:07 | |
*** shaohe_feng has joined #openstack-nova | 00:08 | |
openstackgerrit | Timofey Durakov proposed openstack/nova: rest api version bumped for async pre live migration checks https://review.openstack.org/314932 | 00:09 |
---|---|---|
openstackgerrit | Timofey Durakov proposed openstack/nova: Make live migration checks async https://review.openstack.org/311116 | 00:09 |
*** lucas____ has joined #openstack-nova | 00:09 | |
*** gcb has quit IRC | 00:09 | |
*** sthillma has joined #openstack-nova | 00:09 | |
*** lucas____ has quit IRC | 00:10 | |
*** lucas____ has joined #openstack-nova | 00:10 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for vmwareapi_opts https://review.openstack.org/332503 | 00:13 |
*** stevemar has quit IRC | 00:13 | |
*** stevemar has joined #openstack-nova | 00:14 | |
*** baoli has joined #openstack-nova | 00:14 | |
*** lucas____ has quit IRC | 00:15 | |
*** baoli has quit IRC | 00:15 | |
*** ducttape_ has quit IRC | 00:17 | |
*** shaohe_feng has quit IRC | 00:18 | |
*** stevemar has quit IRC | 00:18 | |
*** shaohe_feng has joined #openstack-nova | 00:18 | |
*** sthillma has quit IRC | 00:21 | |
*** akshai has quit IRC | 00:24 | |
*** hshiina has joined #openstack-nova | 00:27 | |
*** achanda_ has quit IRC | 00:27 | |
*** sdague has joined #openstack-nova | 00:28 | |
*** shaohe_feng has quit IRC | 00:28 | |
*** shaohe_feng has joined #openstack-nova | 00:29 | |
*** stevemar has joined #openstack-nova | 00:29 | |
*** MikeG451 has quit IRC | 00:30 | |
*** baoli has joined #openstack-nova | 00:33 | |
openstackgerrit | Diana Clarke proposed openstack/nova: Delete old imagebackend methods & tests https://review.openstack.org/322974 | 00:34 |
*** baoli has quit IRC | 00:36 | |
openstackgerrit | Timofey Durakov proposed openstack/nova: live_migration_cleanup method introduced https://review.openstack.org/302611 | 00:37 |
openstackgerrit | Timofey Durakov proposed openstack/nova: Remove unused migrate_data param from driver cleanup and destroy https://review.openstack.org/280653 | 00:37 |
openstackgerrit | Timofey Durakov proposed openstack/nova: call pre_live_migration from conductor https://review.openstack.org/291160 | 00:37 |
openstackgerrit | Timofey Durakov proposed openstack/nova: centralize checks before live-migration https://review.openstack.org/332124 | 00:37 |
*** shaohe_feng has quit IRC | 00:38 | |
*** shaohe_feng has joined #openstack-nova | 00:39 | |
*** baoli has joined #openstack-nova | 00:41 | |
*** Hosam has joined #openstack-nova | 00:42 | |
*** baoli has quit IRC | 00:42 | |
*** EinstCrazy has joined #openstack-nova | 00:44 | |
*** Jeffrey4l has joined #openstack-nova | 00:44 | |
*** hoangcx has joined #openstack-nova | 00:45 | |
*** ducttape_ has joined #openstack-nova | 00:46 | |
*** thorst has joined #openstack-nova | 00:48 | |
*** gongysh has quit IRC | 00:48 | |
*** mkoderer___ has quit IRC | 00:48 | |
*** shaohe_feng has quit IRC | 00:48 | |
*** shaohe_feng has joined #openstack-nova | 00:49 | |
*** mkoderer__ has joined #openstack-nova | 00:49 | |
*** stevemar has quit IRC | 00:49 | |
*** ssurana1 has quit IRC | 00:50 | |
*** stevemar has joined #openstack-nova | 00:50 | |
*** sree has joined #openstack-nova | 00:50 | |
*** EinstCrazy has quit IRC | 00:53 | |
*** svenkat has joined #openstack-nova | 00:53 | |
*** baoli has joined #openstack-nova | 00:54 | |
*** CustosLimen has quit IRC | 00:54 | |
*** anthonyper has quit IRC | 00:54 | |
*** raunak has quit IRC | 00:55 | |
*** thorst has quit IRC | 00:56 | |
*** rfolco has quit IRC | 00:56 | |
*** phuongnh has joined #openstack-nova | 00:57 | |
*** cheneydc has joined #openstack-nova | 00:58 | |
*** shaohe_feng has quit IRC | 00:59 | |
*** fragatina has joined #openstack-nova | 00:59 | |
*** anthonyper has joined #openstack-nova | 00:59 | |
*** shaohe_feng has joined #openstack-nova | 01:00 | |
*** stevemar has quit IRC | 01:00 | |
*** stevemar has joined #openstack-nova | 01:00 | |
*** Hosam has quit IRC | 01:01 | |
*** CustosLimen has joined #openstack-nova | 01:02 | |
*** baoli_ has joined #openstack-nova | 01:02 | |
*** sthillma has joined #openstack-nova | 01:04 | |
*** jaugustine has joined #openstack-nova | 01:04 | |
*** pacoblox1 has quit IRC | 01:04 | |
*** baoli has quit IRC | 01:05 | |
*** stevemar has quit IRC | 01:05 | |
*** yassine__ has quit IRC | 01:06 | |
*** fragatina has quit IRC | 01:07 | |
*** fragatina has joined #openstack-nova | 01:08 | |
*** shaohe_feng has quit IRC | 01:09 | |
*** jaugustine has quit IRC | 01:09 | |
*** ducttape_ has quit IRC | 01:09 | |
*** shaohe_feng has joined #openstack-nova | 01:10 | |
*** kevin-run has joined #openstack-nova | 01:10 | |
*** stevemar has joined #openstack-nova | 01:13 | |
*** salv-orlando has joined #openstack-nova | 01:18 | |
*** shaohe_feng has quit IRC | 01:19 | |
*** shaohe_feng has joined #openstack-nova | 01:20 | |
openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 01:24 |
*** salv-orlando has quit IRC | 01:25 | |
*** agireud has quit IRC | 01:26 | |
*** jinxing has joined #openstack-nova | 01:28 | |
*** shaohe_feng has quit IRC | 01:29 | |
*** agireud has joined #openstack-nova | 01:30 | |
*** shaohe_feng has joined #openstack-nova | 01:30 | |
*** sree has quit IRC | 01:30 | |
*** sree has joined #openstack-nova | 01:31 | |
*** raunak has joined #openstack-nova | 01:31 | |
*** browne1 has quit IRC | 01:31 | |
*** hshiina has quit IRC | 01:34 | |
*** sree has quit IRC | 01:35 | |
*** cdelatte has quit IRC | 01:38 | |
*** agireud has quit IRC | 01:39 | |
*** stevemar has quit IRC | 01:40 | |
*** shaohe_feng has quit IRC | 01:40 | |
*** stevemar has joined #openstack-nova | 01:40 | |
*** shaohe_feng has joined #openstack-nova | 01:41 | |
*** agireud has joined #openstack-nova | 01:41 | |
*** sree has joined #openstack-nova | 01:42 | |
*** sree_ has joined #openstack-nova | 01:43 | |
*** baoli_ has quit IRC | 01:43 | |
*** jinxing has quit IRC | 01:44 | |
*** sree_ is now known as Guest9802 | 01:44 | |
*** baoli has joined #openstack-nova | 01:44 | |
*** jinxing has joined #openstack-nova | 01:45 | |
*** sree has quit IRC | 01:47 | |
*** ducttape_ has joined #openstack-nova | 01:47 | |
openstackgerrit | Chen Fan proposed openstack/nova: xvp: change the default xvp conf path to CONF.xvp group https://review.openstack.org/333079 | 01:47 |
*** Guest9802 has quit IRC | 01:48 | |
*** yamahata has quit IRC | 01:48 | |
*** jinxing has quit IRC | 01:49 | |
*** shaohe_feng has quit IRC | 01:50 | |
*** shaohe_feng has joined #openstack-nova | 01:50 | |
*** sree has joined #openstack-nova | 01:51 | |
*** thorst has joined #openstack-nova | 01:53 | |
*** sree has quit IRC | 01:55 | |
*** winston-d_ has joined #openstack-nova | 01:56 | |
*** sree has joined #openstack-nova | 01:57 | |
*** sacharya has joined #openstack-nova | 01:59 | |
*** thorst has quit IRC | 01:59 | |
*** claudiub has quit IRC | 02:00 | |
*** jichen has joined #openstack-nova | 02:00 | |
*** shaohe_feng has quit IRC | 02:00 | |
*** shaohe_feng has joined #openstack-nova | 02:01 | |
*** sree has quit IRC | 02:01 | |
*** sree has joined #openstack-nova | 02:03 | |
*** zul has quit IRC | 02:03 | |
*** sacharya has quit IRC | 02:03 | |
*** zul has joined #openstack-nova | 02:04 | |
*** Apoorva has quit IRC | 02:04 | |
*** sacharya has joined #openstack-nova | 02:04 | |
*** thorst has joined #openstack-nova | 02:05 | |
*** sree has quit IRC | 02:07 | |
*** READ10 has quit IRC | 02:07 | |
*** bkopilov has quit IRC | 02:08 | |
*** sree has joined #openstack-nova | 02:09 | |
*** shaohe_feng has quit IRC | 02:10 | |
*** shaohe_feng has joined #openstack-nova | 02:11 | |
*** thorst has quit IRC | 02:11 | |
*** stevemar has quit IRC | 02:14 | |
*** sree has quit IRC | 02:14 | |
*** sree has joined #openstack-nova | 02:17 | |
*** shaohe_feng has quit IRC | 02:21 | |
*** shaohe_feng has joined #openstack-nova | 02:22 | |
alex_xu | jaypipes: my id is xuhj | 02:22 |
*** sree has quit IRC | 02:22 | |
alex_xu | i have too many names | 02:23 |
*** zul has quit IRC | 02:23 | |
openstackgerrit | Zhihai Song proposed openstack/nova: Transform instance.power_on notifications https://review.openstack.org/329746 | 02:26 |
*** sthillma has quit IRC | 02:26 | |
*** sree has joined #openstack-nova | 02:26 | |
*** greghaynes has quit IRC | 02:27 | |
openstackgerrit | Zhihai Song proposed openstack/nova: Transform instance.restore notifications https://review.openstack.org/331972 | 02:27 |
*** raunak has quit IRC | 02:30 | |
*** sdague has quit IRC | 02:30 | |
*** shaohe_feng has quit IRC | 02:31 | |
*** sree has quit IRC | 02:31 | |
*** shaohe_feng has joined #openstack-nova | 02:32 | |
*** jinxing has joined #openstack-nova | 02:32 | |
*** sree has joined #openstack-nova | 02:33 | |
*** stevemar has joined #openstack-nova | 02:34 | |
*** aysyd has quit IRC | 02:35 | |
*** sthillma has joined #openstack-nova | 02:36 | |
*** zz_dimtruck is now known as dimtruck | 02:37 | |
*** sree_ has joined #openstack-nova | 02:37 | |
*** sree_ is now known as Guest91960 | 02:38 | |
*** gongysh has joined #openstack-nova | 02:38 | |
*** sree has quit IRC | 02:38 | |
*** greghaynes has joined #openstack-nova | 02:38 | |
*** houming has joined #openstack-nova | 02:38 | |
*** browne has joined #openstack-nova | 02:38 | |
*** stevemar has quit IRC | 02:39 | |
*** sree has joined #openstack-nova | 02:41 | |
*** shaohe_feng has quit IRC | 02:41 | |
*** Guest91960 has quit IRC | 02:41 | |
*** shaohe_feng has joined #openstack-nova | 02:42 | |
*** jinxing has quit IRC | 02:44 | |
*** sthillma has quit IRC | 02:44 | |
*** sree has quit IRC | 02:46 | |
*** jinxing has joined #openstack-nova | 02:47 | |
*** sree has joined #openstack-nova | 02:49 | |
*** salv-orlando has joined #openstack-nova | 02:49 | |
*** shaohe_feng has quit IRC | 02:51 | |
*** shaohe_feng has joined #openstack-nova | 02:52 | |
*** sree has quit IRC | 02:53 | |
*** salv-orlando has quit IRC | 02:54 | |
openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 02:55 |
*** stevemar has joined #openstack-nova | 02:57 | |
*** ducttape_ has quit IRC | 02:58 | |
*** shaohe_feng has quit IRC | 03:02 | |
*** achanda has joined #openstack-nova | 03:03 | |
*** raunak has joined #openstack-nova | 03:04 | |
*** shaohe_feng has joined #openstack-nova | 03:04 | |
*** ducttape_ has joined #openstack-nova | 03:05 | |
*** sree has joined #openstack-nova | 03:07 | |
*** ijw has quit IRC | 03:07 | |
*** achanda has quit IRC | 03:08 | |
*** sree has quit IRC | 03:11 | |
*** shaohe_feng has quit IRC | 03:12 | |
*** sree has joined #openstack-nova | 03:14 | |
*** browne has quit IRC | 03:14 | |
*** raunak has quit IRC | 03:14 | |
*** shaohe_feng has joined #openstack-nova | 03:15 | |
*** thorst has joined #openstack-nova | 03:16 | |
*** sajeesh has quit IRC | 03:17 | |
*** sree has quit IRC | 03:18 | |
*** baoli has quit IRC | 03:19 | |
*** raunak has joined #openstack-nova | 03:19 | |
*** stevemar has quit IRC | 03:20 | |
*** thorst has quit IRC | 03:21 | |
openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 03:22 |
*** shaohe_feng has quit IRC | 03:22 | |
*** pwnall has quit IRC | 03:23 | |
*** shaohe_feng has joined #openstack-nova | 03:23 | |
*** baoli has joined #openstack-nova | 03:23 | |
openstackgerrit | bailin.zhang proposed openstack/nova: Delete reduplicate code in test_compute_mgr.py https://review.openstack.org/331108 | 03:25 |
*** tjones has joined #openstack-nova | 03:26 | |
*** stevemar has joined #openstack-nova | 03:28 | |
*** fishcried has joined #openstack-nova | 03:29 | |
*** fishcried has quit IRC | 03:30 | |
*** bkopilov has joined #openstack-nova | 03:30 | |
*** stevemar has quit IRC | 03:32 | |
*** shaohe_feng has quit IRC | 03:32 | |
*** ducttape_ has quit IRC | 03:33 | |
*** shaohe_feng has joined #openstack-nova | 03:33 | |
*** sree has joined #openstack-nova | 03:33 | |
openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 03:33 |
*** stevemar has joined #openstack-nova | 03:35 | |
*** stevemar has quit IRC | 03:35 | |
*** stevemar has joined #openstack-nova | 03:36 | |
*** sree has quit IRC | 03:37 | |
*** raunak has quit IRC | 03:41 | |
*** raunak has joined #openstack-nova | 03:43 | |
*** shaohe_feng has quit IRC | 03:43 | |
*** shaohe_feng has joined #openstack-nova | 03:43 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Return 400 when SecurityGroupCannotBeApplied is raised https://review.openstack.org/323332 | 03:45 |
*** prateek has joined #openstack-nova | 03:51 | |
*** prateek_ has joined #openstack-nova | 03:51 | |
*** links has joined #openstack-nova | 03:51 | |
*** shaohe_feng has quit IRC | 03:53 | |
*** shaohe_feng has joined #openstack-nova | 03:54 | |
*** ijw has joined #openstack-nova | 03:54 | |
openstackgerrit | melanie witt proposed openstack/nova: WIP Make InstanceGroup object favor the API database https://review.openstack.org/330995 | 03:55 |
openstackgerrit | melanie witt proposed openstack/nova: Add instance groups tables to the API database https://review.openstack.org/329719 | 03:55 |
openstackgerrit | melanie witt proposed openstack/nova: WIP Add online migration to move instance groups to API database https://review.openstack.org/333098 | 03:55 |
*** sree has joined #openstack-nova | 03:57 | |
*** josecastroleon has joined #openstack-nova | 03:58 | |
openstackgerrit | Dao Cong Tien proposed openstack/nova: Stop using mox from unit/cells/test_cells_rpc_driver.py https://review.openstack.org/333101 | 03:59 |
*** burgerk has joined #openstack-nova | 04:00 | |
*** salv-orlando has joined #openstack-nova | 04:00 | |
*** sree has quit IRC | 04:02 | |
*** shaohe_feng has quit IRC | 04:03 | |
*** ducttape_ has joined #openstack-nova | 04:03 | |
*** shaohe_feng has joined #openstack-nova | 04:04 | |
*** svenkat has quit IRC | 04:07 | |
*** lucas____ has joined #openstack-nova | 04:08 | |
*** baoli has quit IRC | 04:10 | |
*** lucas____ has quit IRC | 04:10 | |
*** lucas____ has joined #openstack-nova | 04:10 | |
*** ijw has quit IRC | 04:12 | |
*** shaohe_feng has quit IRC | 04:13 | |
*** shaohe_feng has joined #openstack-nova | 04:14 | |
*** salv-orlando has quit IRC | 04:14 | |
*** ratailor has joined #openstack-nova | 04:15 | |
*** ducttape_ has quit IRC | 04:16 | |
*** lucas____ has quit IRC | 04:16 | |
*** sree has joined #openstack-nova | 04:17 | |
*** lucas____ has joined #openstack-nova | 04:18 | |
*** GB21 has joined #openstack-nova | 04:18 | |
*** achanda has joined #openstack-nova | 04:18 | |
*** liverpooler has quit IRC | 04:19 | |
*** thorst has joined #openstack-nova | 04:19 | |
openstackgerrit | jichenjc proposed openstack/nova: WIP: make purge_option to be default False https://review.openstack.org/332777 | 04:20 |
*** dimtruck is now known as zz_dimtruck | 04:21 | |
*** tjones has left #openstack-nova | 04:21 | |
*** shaohe_feng has quit IRC | 04:24 | |
*** shaohe_feng has joined #openstack-nova | 04:24 | |
*** thorst has quit IRC | 04:25 | |
*** amotoki has joined #openstack-nova | 04:25 | |
*** jichen_ has joined #openstack-nova | 04:26 | |
*** jichen__ has joined #openstack-nova | 04:27 | |
*** jichen has quit IRC | 04:27 | |
*** jichen__ is now known as jichen | 04:27 | |
*** Sukhdev has joined #openstack-nova | 04:28 | |
*** sudipto has joined #openstack-nova | 04:29 | |
*** sacharya has quit IRC | 04:30 | |
*** jichen_ has quit IRC | 04:30 | |
*** raunak has quit IRC | 04:31 | |
*** lucas____ has quit IRC | 04:32 | |
*** diga has joined #openstack-nova | 04:33 | |
*** shaohe_feng has quit IRC | 04:34 | |
*** shaohe_feng has joined #openstack-nova | 04:35 | |
*** raunak has joined #openstack-nova | 04:35 | |
*** lucas____ has joined #openstack-nova | 04:36 | |
*** dave-mccowan has quit IRC | 04:41 | |
*** shaohe_feng has quit IRC | 04:44 | |
*** shaohe_feng has joined #openstack-nova | 04:45 | |
*** burgerk has quit IRC | 04:47 | |
openstackgerrit | jichenjc proposed openstack/nova: update servers policy in code to use formats https://review.openstack.org/332995 | 04:49 |
*** salv-orlando has joined #openstack-nova | 04:50 | |
*** shaohe_feng has quit IRC | 04:54 | |
*** Sukhdev has quit IRC | 04:55 | |
*** burgerk has joined #openstack-nova | 04:55 | |
*** shaohe_feng has joined #openstack-nova | 04:55 | |
*** jaosorior has joined #openstack-nova | 04:55 | |
*** tjones has joined #openstack-nova | 04:56 | |
*** zerda2 has joined #openstack-nova | 04:56 | |
*** salv-orl_ has joined #openstack-nova | 04:59 | |
*** burgerk has quit IRC | 05:00 | |
*** salv-orlando has quit IRC | 05:02 | |
*** shaohe_feng has quit IRC | 05:05 | |
*** shaohe_feng has joined #openstack-nova | 05:05 | |
*** yamahata has joined #openstack-nova | 05:08 | |
*** sree_ has joined #openstack-nova | 05:08 | |
*** sree has quit IRC | 05:09 | |
*** gongysh has quit IRC | 05:09 | |
*** sree_ is now known as Guest77814 | 05:09 | |
*** cheneydc has quit IRC | 05:10 | |
*** sudipto has quit IRC | 05:10 | |
openstackgerrit | jichenjc proposed openstack/nova: update servers policy in code to use formats https://review.openstack.org/332995 | 05:13 |
*** lucas____ has quit IRC | 05:15 | |
*** shaohe_feng has quit IRC | 05:15 | |
*** shaohe_feng has joined #openstack-nova | 05:16 | |
*** pbandark has joined #openstack-nova | 05:19 | |
*** lucas____ has joined #openstack-nova | 05:19 | |
*** houming has quit IRC | 05:19 | |
*** lpetrut has joined #openstack-nova | 05:20 | |
*** GB21 has quit IRC | 05:21 | |
*** ecelik has joined #openstack-nova | 05:22 | |
*** thorst has joined #openstack-nova | 05:23 | |
*** shaohe_feng has quit IRC | 05:25 | |
*** shaohe_feng has joined #openstack-nova | 05:26 | |
*** lucas____ has quit IRC | 05:28 | |
*** kevin-run has quit IRC | 05:30 | |
*** kevin-run has joined #openstack-nova | 05:30 | |
*** thorst has quit IRC | 05:31 | |
*** sudipto has joined #openstack-nova | 05:32 | |
*** ijw has joined #openstack-nova | 05:33 | |
*** lucas____ has joined #openstack-nova | 05:33 | |
*** jinxing has quit IRC | 05:34 | |
*** shaohe_feng has quit IRC | 05:35 | |
*** diga has quit IRC | 05:36 | |
*** shaohe_feng has joined #openstack-nova | 05:36 | |
*** GB21 has joined #openstack-nova | 05:37 | |
*** jinxing has joined #openstack-nova | 05:37 | |
*** kaisers1 has quit IRC | 05:38 | |
*** kaisers has joined #openstack-nova | 05:42 | |
*** lucas____ has quit IRC | 05:43 | |
*** Guest77814 has quit IRC | 05:43 | |
*** sree has joined #openstack-nova | 05:44 | |
*** shaohe_feng has quit IRC | 05:46 | |
*** ducttape_ has joined #openstack-nova | 05:46 | |
*** shaohe_feng has joined #openstack-nova | 05:47 | |
*** lucas____ has joined #openstack-nova | 05:47 | |
openstackgerrit | jichenjc proposed openstack/nova: Verify os-aggregates.inc on sample files https://review.openstack.org/333114 | 05:49 |
*** ducttape_ has quit IRC | 05:51 | |
*** raunak has quit IRC | 05:55 | |
*** shaohe_feng has quit IRC | 05:56 | |
*** lucas____ has quit IRC | 05:56 | |
*** shaohe_feng has joined #openstack-nova | 05:57 | |
*** lucas____ has joined #openstack-nova | 06:01 | |
*** liverpooler has joined #openstack-nova | 06:06 | |
*** shaohe_feng has quit IRC | 06:06 | |
*** shaohe_feng has joined #openstack-nova | 06:07 | |
*** Kevin_Zheng has joined #openstack-nova | 06:08 | |
*** lucas____ has quit IRC | 06:10 | |
*** claudiub has joined #openstack-nova | 06:13 | |
openstackgerrit | jichenjc proposed openstack/nova: Remove 400 as expected error https://review.openstack.org/333122 | 06:13 |
*** rcernin has joined #openstack-nova | 06:15 | |
*** lucas____ has joined #openstack-nova | 06:15 | |
*** shaohe_feng has quit IRC | 06:16 | |
*** nkrinner_afk is now known as nkrinner | 06:17 | |
*** shaohe_feng has joined #openstack-nova | 06:17 | |
*** lucas____ has quit IRC | 06:24 | |
*** moshele has joined #openstack-nova | 06:25 | |
openstackgerrit | jichenjc proposed openstack/nova: Remove rate_limit param in builder https://review.openstack.org/333125 | 06:26 |
*** shaohe_feng has quit IRC | 06:27 | |
*** Qiming has quit IRC | 06:28 | |
*** Wenzhi has quit IRC | 06:28 | |
*** chenfei has quit IRC | 06:28 | |
*** shaohe_feng has joined #openstack-nova | 06:28 | |
*** andreas_s has joined #openstack-nova | 06:28 | |
*** Wenzhi has joined #openstack-nova | 06:28 | |
*** thorst has joined #openstack-nova | 06:28 | |
*** Wenzhi has quit IRC | 06:28 | |
*** lucas____ has joined #openstack-nova | 06:29 | |
*** Wenzhi has joined #openstack-nova | 06:30 | |
*** chenfei has joined #openstack-nova | 06:32 | |
*** houming has joined #openstack-nova | 06:32 | |
openstackgerrit | melanie witt proposed openstack/nova: WIP Make InstanceGroup object favor the API database https://review.openstack.org/330995 | 06:34 |
openstackgerrit | melanie witt proposed openstack/nova: WIP Add online migration to move instance groups to API database https://review.openstack.org/333098 | 06:34 |
*** ljxiash has joined #openstack-nova | 06:34 | |
*** sdake has joined #openstack-nova | 06:34 | |
*** Qiming has joined #openstack-nova | 06:35 | |
*** thorst has quit IRC | 06:36 | |
*** shaohe_feng has quit IRC | 06:37 | |
*** shaohe_feng has joined #openstack-nova | 06:38 | |
openstackgerrit | melanie witt proposed openstack/nova: WIP Add online migration to move instance groups to API database https://review.openstack.org/333098 | 06:42 |
*** lucas____ has quit IRC | 06:42 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 06:44 |
*** sree has quit IRC | 06:44 | |
*** sridharg has joined #openstack-nova | 06:44 | |
*** sree has joined #openstack-nova | 06:44 | |
*** nicolasbock has quit IRC | 06:45 | |
*** mpavone has joined #openstack-nova | 06:46 | |
Kevin_Zheng | alex_xu: ping | 06:47 |
*** chenfei has quit IRC | 06:47 | |
*** Qiming has quit IRC | 06:47 | |
*** ljxiash has quit IRC | 06:47 | |
*** Wenzhi has quit IRC | 06:47 | |
*** shaohe_feng has quit IRC | 06:47 | |
alex_xu | Kevin_Zheng: pong | 06:47 |
*** lucas____ has joined #openstack-nova | 06:47 | |
*** shaohe_feng has joined #openstack-nova | 06:48 | |
*** ducttape_ has joined #openstack-nova | 06:48 | |
Kevin_Zheng | alex_xu: Hi, could you have a look on https://review.openstack.org/#/c/326326/ when you got time? | 06:48 |
*** jamielennox is now known as jamielennox|away | 06:48 | |
Kevin_Zheng | hope it can make the non-priority deadline | 06:48 |
*** ijw has quit IRC | 06:48 | |
alex_xu | Kevin_Zheng: yea, add it to my todo list | 06:49 |
*** Wenzhi has joined #openstack-nova | 06:49 | |
*** belmoreira has joined #openstack-nova | 06:49 | |
Kevin_Zheng | Thanks | 06:49 |
*** hoangcx has quit IRC | 06:49 | |
*** ducttape_ has quit IRC | 06:52 | |
*** gongysh has joined #openstack-nova | 06:52 | |
*** chenfei has joined #openstack-nova | 06:53 | |
*** hoangcx has joined #openstack-nova | 06:53 | |
*** Qiming has joined #openstack-nova | 06:54 | |
alex_xu | Kevin_Zheng: np | 06:54 |
openstackgerrit | jichenjc proposed openstack/nova: Not talking about V2 API code in review doc guide https://review.openstack.org/333134 | 06:55 |
*** lucas____ has quit IRC | 06:56 | |
*** achanda has quit IRC | 06:56 | |
*** fragatina has quit IRC | 06:57 | |
*** lucas____ has joined #openstack-nova | 06:57 | |
*** shaohe_feng has quit IRC | 06:57 | |
*** vtech has joined #openstack-nova | 06:58 | |
*** tangchen has quit IRC | 06:58 | |
*** fragatina has joined #openstack-nova | 06:58 | |
*** irenab has quit IRC | 06:58 | |
*** shaohe_feng has joined #openstack-nova | 06:58 | |
*** muawiakhan has joined #openstack-nova | 06:58 | |
*** lpetrut has quit IRC | 06:59 | |
*** irenab has joined #openstack-nova | 07:00 | |
*** roeyc has joined #openstack-nova | 07:01 | |
*** achanda has joined #openstack-nova | 07:01 | |
*** irenab has quit IRC | 07:01 | |
*** rcernin has quit IRC | 07:02 | |
*** achanda has quit IRC | 07:06 | |
*** lucas____ has quit IRC | 07:06 | |
*** shaohe_feng has quit IRC | 07:08 | |
*** shaohe_feng has joined #openstack-nova | 07:08 | |
*** irenab has joined #openstack-nova | 07:08 | |
*** irenab has quit IRC | 07:09 | |
*** hieulq has joined #openstack-nova | 07:09 | |
*** abalutoiu has quit IRC | 07:09 | |
*** tobasco has quit IRC | 07:09 | |
*** irenab has joined #openstack-nova | 07:09 | |
*** sahid has joined #openstack-nova | 07:10 | |
*** lpetrut has joined #openstack-nova | 07:10 | |
*** lucas____ has joined #openstack-nova | 07:11 | |
*** NightKhaos has quit IRC | 07:12 | |
*** chandankumar has joined #openstack-nova | 07:13 | |
*** tesseract- has joined #openstack-nova | 07:13 | |
*** NightKhaos has joined #openstack-nova | 07:14 | |
*** tesseract- has quit IRC | 07:14 | |
*** tesseract- has joined #openstack-nova | 07:14 | |
*** tobasco has joined #openstack-nova | 07:16 | |
*** markus_z has joined #openstack-nova | 07:16 | |
*** irenab has quit IRC | 07:17 | |
*** rcernin has joined #openstack-nova | 07:17 | |
*** salv-orl_ has quit IRC | 07:18 | |
*** shaohe_feng has quit IRC | 07:18 | |
*** shaohe_feng has joined #openstack-nova | 07:18 | |
*** woodster_ has quit IRC | 07:18 | |
*** lucas____ has quit IRC | 07:21 | |
*** sdake_ has joined #openstack-nova | 07:21 | |
*** nicolasbock has joined #openstack-nova | 07:21 | |
*** josecastroleon has quit IRC | 07:23 | |
*** sdake has quit IRC | 07:24 | |
*** lucas____ has joined #openstack-nova | 07:25 | |
*** jlanoux has joined #openstack-nova | 07:26 | |
*** sree has quit IRC | 07:27 | |
*** shaohe_feng has quit IRC | 07:28 | |
*** shaohe_feng has joined #openstack-nova | 07:29 | |
*** jlanoux has quit IRC | 07:29 | |
*** thorst has joined #openstack-nova | 07:34 | |
*** jlanoux has joined #openstack-nova | 07:34 | |
openstackgerrit | Alex Xu proposed openstack/nova: Deprecated networks related APIs https://review.openstack.org/333151 | 07:34 |
openstackgerrit | Alex Xu proposed openstack/nova: Deprecate FixedIPs API https://review.openstack.org/333152 | 07:34 |
*** zenoway has joined #openstack-nova | 07:35 | |
*** irenab has joined #openstack-nova | 07:38 | |
*** shaohe_feng has quit IRC | 07:38 | |
bauzas | good morning Nova | 07:39 |
*** shaohe_feng has joined #openstack-nova | 07:39 | |
jed56 | hello bauzas | 07:40 |
*** paul-carlton2 has joined #openstack-nova | 07:40 | |
*** thorst has quit IRC | 07:41 | |
haypo | bauzas: hello. here you have a first serie of 6 patches for python 3 :-) https://review.openstack.org/#/c/322040/ | 07:43 |
bauzas | kewl | 07:43 |
haypo | bauzas: you told me that python 3 is proposed to new nova contributors. is there a menthorship program for new contributors? | 07:44 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/331996 | 07:44 |
bauzas | haypo: yup indeed | 07:44 |
bauzas | sec | 07:44 |
haypo | bauzas: i would like to propose them to _review_ my changes rather than writing changes | 07:44 |
*** Hosam has joined #openstack-nova | 07:44 | |
bauzas | haypo: https://wiki.openstack.org/wiki/Nova/Mentoring | 07:44 |
bauzas | oh interesting https://wiki.openstack.org/wiki/Nova/Mentoring#Improve_Python_3_Compatibility | 07:44 |
bauzas | tonyb: around ? | 07:45 |
*** lpetrut has quit IRC | 07:45 | |
bauzas | damn Aussies who are not in our TZ :p | 07:45 |
haypo | claudiub: hi. for https://wiki.openstack.org/wiki/Nova/Mentoring#Improve_Python_3_Compatibility i would like to propose to new contributors to review my python 3 changes: https://review.openstack.org/#/c/322040/ | 07:46 |
haypo | claudiub: what do you think of this idea? | 07:46 |
*** matrohon has joined #openstack-nova | 07:47 | |
*** lucas____ has quit IRC | 07:48 | |
*** ducttape_ has joined #openstack-nova | 07:48 | |
*** shaohe_feng has quit IRC | 07:49 | |
*** shaohe_feng has joined #openstack-nova | 07:49 | |
BobBall | Morning all! | 07:51 |
*** lucas____ has joined #openstack-nova | 07:52 | |
*** itamarl has joined #openstack-nova | 07:54 | |
openstackgerrit | jichenjc proposed openstack/nova: Add multiple create in document https://review.openstack.org/333156 | 07:54 |
*** ducttape_ has quit IRC | 07:54 | |
*** zenoway has quit IRC | 07:55 | |
*** gus has quit IRC | 07:57 | |
*** gus has joined #openstack-nova | 07:57 | |
*** shaohe_feng has quit IRC | 07:59 | |
*** shaohe_feng has joined #openstack-nova | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** itamarl has quit IRC | 08:00 | |
*** itamarl has joined #openstack-nova | 08:00 | |
*** sridharg has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:01 | |
*** lucas____ has quit IRC | 08:02 | |
claudiub | haypo: hi. hm, sure reviewing is an important part of development, so it is a good idea. but as far as mentoring goes, I think people would learn and understand more by doing and commiting code. :) | 08:02 |
*** jinxing has quit IRC | 08:03 | |
*** salv-orlando has joined #openstack-nova | 08:05 | |
*** markvoel_ has quit IRC | 08:05 | |
*** lucas____ has joined #openstack-nova | 08:06 | |
*** paul-carlton2 has quit IRC | 08:07 | |
openstackgerrit | sahid proposed openstack/nova: network: handle unauthorized exception from neutron https://review.openstack.org/312014 | 08:09 |
*** shaohe_feng has quit IRC | 08:09 | |
* bauzas bbiab | 08:09 | |
*** shaohe_feng has joined #openstack-nova | 08:10 | |
*** sree has joined #openstack-nova | 08:10 | |
*** luca_____ has joined #openstack-nova | 08:10 | |
*** lucas____ has quit IRC | 08:11 | |
haypo | claudiub: oh. i'm trying to find a solution to get reviews on my python 3 changes :) | 08:11 |
claudiub | haypo: yeeaah... me too. :) | 08:11 |
openstackgerrit | Zhihai Song proposed openstack/nova: Transform instance.restore notifications https://review.openstack.org/331972 | 08:12 |
*** hoangcx has quit IRC | 08:13 | |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/compute/test_compute.py (8) https://review.openstack.org/330948 | 08:14 |
openstackgerrit | Roey Chen proposed openstack/nova: Adding missing log translation hints https://review.openstack.org/332705 | 08:15 |
*** diga has joined #openstack-nova | 08:15 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 08:16 |
openstackgerrit | jichenjc proposed openstack/nova: Remove unnecessary check in api layer https://review.openstack.org/333166 | 08:18 |
*** hoangcx has joined #openstack-nova | 08:18 | |
*** shaohe_feng has quit IRC | 08:19 | |
*** shaohe_feng has joined #openstack-nova | 08:20 | |
*** sfinucan has joined #openstack-nova | 08:20 | |
*** gcb has joined #openstack-nova | 08:21 | |
*** zenoway has joined #openstack-nova | 08:21 | |
*** mvk_ has quit IRC | 08:21 | |
*** aloga has quit IRC | 08:23 | |
*** danpb has joined #openstack-nova | 08:23 | |
*** luca_____ has quit IRC | 08:24 | |
openstackgerrit | jichenjc proposed openstack/nova: Remove comment on v3 API https://review.openstack.org/333171 | 08:24 |
tdurakov | hi everyone | 08:24 |
*** acabot has quit IRC | 08:26 | |
tdurakov | johnthetubaguy:hi, noticed that live-migration job fails sometimes, any ideas how to debug such fails http://logs.openstack.org/11/302611/10/check/gate-tempest-dsvm-multinode-live-migration/d0e28e3/console.html | 08:27 |
*** p01nt3r75 has joined #openstack-nova | 08:27 | |
*** sridharg has joined #openstack-nova | 08:27 | |
*** sree has quit IRC | 08:27 | |
*** takashin has left #openstack-nova | 08:27 | |
*** gongysh has quit IRC | 08:29 | |
*** sree has joined #openstack-nova | 08:29 | |
*** lucas____ has joined #openstack-nova | 08:29 | |
*** lucas-afk is now known as lucasagomes | 08:29 | |
*** acabot has joined #openstack-nova | 08:29 | |
*** irenab_ has joined #openstack-nova | 08:29 | |
*** aloga has joined #openstack-nova | 08:29 | |
*** shaohe_feng has quit IRC | 08:30 | |
*** irenab has quit IRC | 08:30 | |
*** irenab_ is now known as irenab | 08:30 | |
*** shaohe_feng has joined #openstack-nova | 08:30 | |
*** josecastroleon has joined #openstack-nova | 08:30 | |
*** Wenzhi has quit IRC | 08:31 | |
*** chenfei has quit IRC | 08:31 | |
*** Qiming has quit IRC | 08:31 | |
*** daemontool has joined #openstack-nova | 08:31 | |
*** itamarl_ has joined #openstack-nova | 08:33 | |
*** sree has quit IRC | 08:34 | |
*** paul-carlton2 has joined #openstack-nova | 08:35 | |
haypo | claudiub: hi. why did you write a new blueprint for python3? | 08:35 |
claudiub | haypo: because the one you're currently targetting has been closed as completed in mitaka | 08:36 |
*** itamarl has quit IRC | 08:36 | |
*** itamarl_ is now known as itamarl | 08:36 | |
*** sambetts|afk is now known as sambetts | 08:36 | |
claudiub | which ofcourse, is not done yet. | 08:36 |
claudiub | haypo: I've opened it at the suggestion of johnthetubaguy :) | 08:37 |
*** GB21 has quit IRC | 08:37 | |
*** thorst has joined #openstack-nova | 08:39 | |
openstackgerrit | Victor Stinner proposed openstack/nova: Port cinder unit tests to Python 3 https://review.openstack.org/332686 | 08:39 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port test_ipv6 to py3 and simplify to_global() https://review.openstack.org/332701 | 08:39 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port pci unit tests to Python 3 https://review.openstack.org/332791 | 08:39 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port test_metadata to Python 3 https://review.openstack.org/332738 | 08:39 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port test_hacking to Python 3 https://review.openstack.org/332737 | 08:39 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port test_pipelib and test_policy to Python 3 https://review.openstack.org/322040 | 08:39 |
*** irenab has quit IRC | 08:40 | |
*** shaohe_feng has quit IRC | 08:40 | |
*** shaohe_feng has joined #openstack-nova | 08:40 | |
johnthetubaguy | claudiub: haypo: is that the newton specific one right? | 08:41 |
johnthetubaguy | claudiub: haypo: thats because the old one was marked complete for the cycle the patches landed, to help the tracking of where we are at | 08:42 |
*** lucas____ has quit IRC | 08:42 | |
johnthetubaguy | tdurakov: its easier if you have your own setup where you see it happen, which should be possibly by copying the devstack localrc stuff | 08:42 |
johnthetubaguy | tdurakov: that or you just add extra debuging into the logs to try catch things | 08:42 |
johnthetubaguy | tdurakov: are we still seeing that (apparent) lockup in libvirt that is causing the live-migrate to stall? | 08:43 |
claudiub | johnthetubaguy: yep. | 08:43 |
*** ljxiash has joined #openstack-nova | 08:43 | |
johnthetubaguy | claudiub: cool, just making sure we didn't create another another one :) | 08:43 |
*** derekh has joined #openstack-nova | 08:44 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 08:44 |
claudiub | johnthetubaguy: well, we'll probably have to have another another one in O. :) | 08:44 |
tdurakov | johnthetubaguy: yep, thinking about extra-logging, as there is a fail before tests execution | 08:44 |
*** ljxiash has quit IRC | 08:44 | |
*** Qiming has joined #openstack-nova | 08:44 | |
johnthetubaguy | claudiub: true, unless we get finished (somehow...) | 08:44 |
claudiub | johnthetubaguy: btw, there are quite a lot of patches for python 3. can you lend a hand in reviewing them pls? :) | 08:44 |
tdurakov | fyi, looks stability level increased | 08:44 |
johnthetubaguy | tdurakov: ah, interesting a fail before the tests, yeah, extra logging needed | 08:45 |
claudiub | johnthetubaguy: well, we should be at around 12000 / 15000 unit tests with all the patches that are up, or even better. | 08:45 |
*** hieulq has quit IRC | 08:45 | |
haypo | bauzas: python3 changes are denied 3 months on 6 per cycle? wow | 08:45 |
*** chenfei has joined #openstack-nova | 08:45 | |
*** mfedosin has joined #openstack-nova | 08:45 | |
johnthetubaguy | claudiub: right now, its not going to happen, I have a list of requests stretching to the moon already | 08:45 |
haypo | bauzas: since it takes longer than 1 month to get 6 lines merged, i'm not sure that it will be possible to port nova to python 3 :) | 08:45 |
*** yassine__ has joined #openstack-nova | 08:46 | |
*** GB21 has joined #openstack-nova | 08:46 | |
johnthetubaguy | haypo: yeah, we have almost no one doing reviews on patches, so we have to be brutal with the priorities | 08:46 |
*** thorst has quit IRC | 08:46 | |
johnthetubaguy | if we can get more folks doing good reviews on those, they should be way easier to land them | 08:46 |
haypo | johnthetubaguy: well, the problem is not new in nova. you need more core reviewers. from my point of view (maybe naive), the problem and the solution are simple :) | 08:47 |
*** lucas____ has joined #openstack-nova | 08:47 | |
johnthetubaguy | haypo: we need more reviewers before we can get more core reviewers, but agree | 08:47 |
johnthetubaguy | however the solution is not simply sadly | 08:47 |
*** sameo has joined #openstack-nova | 08:47 | |
*** ljxiash has joined #openstack-nova | 08:47 | |
*** Jeffrey4l has quit IRC | 08:47 | |
johnthetubaguy | haypo: current average merge time seems to be 70 days right now: http://russellbryant.net/openstack-stats/nova-openreviews.txt | 08:47 |
openstackgerrit | jichenjc proposed openstack/nova: Remove rate_limit param in builder https://review.openstack.org/333125 | 08:48 |
haypo | the frustrating point is that https://review.openstack.org/#/c/322040/ was approved 23 days ago, but merged failed on a conflict. i fixed immediatly the conflict, but then... no core reviewed it again... and now bauzas says that june 30 is the deadline for python3 :-/ | 08:48 |
johnthetubaguy | haypo: so if you ping us on IRC here, we will fast approve, let me take a peak | 08:48 |
*** Wenzhi has joined #openstack-nova | 08:48 | |
claudiub | haypo: wait, why june 30? | 08:49 |
haypo | johnthetubaguy: haha, 70 days. so yeah, i'm not sure that i will be possible to finish the python 3 port, since i expect between 50 and 100 more changes needed to port the remaining ~3200 tests | 08:49 |
*** salv-orlando has quit IRC | 08:49 | |
tonyb | bauzas: I am now 'sup? | 08:50 |
haypo | bauzas suggested the idea of a python3 sprint. i don't think it would be possible nor if it would help? | 08:50 |
*** rcernin has quit IRC | 08:50 | |
*** salv-orlando has joined #openstack-nova | 08:50 | |
openstackgerrit | Dao Cong Tien proposed openstack/nova: Stop using mox from unit/cells/test_cells_rpc_driver.py https://review.openstack.org/333101 | 08:50 |
johnthetubaguy | its helped the api docs, certainly worth seeing whats possible | 08:50 |
*** gcb has quit IRC | 08:50 | |
*** shaohe_feng has quit IRC | 08:50 | |
johnthetubaguy | we have a nice burndown list of tests, so that seems very possible | 08:50 |
haypo | claudiub: oh wait, it was tony breeds who mentioned the "Nova non-priority feature freeze" "and the python3 port isn't listed as | 08:50 |
haypo | a priority" http://releases.openstack.org/newton/schedule.html#nova-non-priority-feature-freeze & http://specs.openstack.org/openstack/nova-specs/priorities/newton-priorities.html | 08:51 |
*** ducttape_ has joined #openstack-nova | 08:51 | |
*** shaohe_feng has joined #openstack-nova | 08:51 | |
*** prateek_ has quit IRC | 08:52 | |
*** amotoki has quit IRC | 08:53 | |
johnthetubaguy | haypo: thats right, its real soon, same process as last few cycles, slightly different timeline | 08:53 |
*** lpetrut has joined #openstack-nova | 08:53 | |
claudiub | johnthetubaguy: right, I'm surprised, why is the non-priority feature freeze next week? I thought it was supposed to be on N-2 milestone | 08:54 |
*** salv-orlando has quit IRC | 08:55 | |
johnthetubaguy | claudiub: discussed the move at the design summit, basically getting too many exceptions that spilled into N-3, so shortening the timeframe this time around | 08:55 |
*** ducttape_ has quit IRC | 08:55 | |
johnthetubaguy | its also gives lots more time for getting all the bug fixes we have up for review merged | 08:55 |
*** lucas____ has quit IRC | 08:56 | |
haypo | how much core reviewers do you have in nova? | 08:56 |
tonyb | haypo: 13 https://review.openstack.org/#/admin/groups/25,members | 08:58 |
bauzas | haypo: hey, was afk | 08:59 |
bauzas | so, lemme clarify my email : no, the py3 effort should not be impacted by the feature freeze, given those patches are not really features | 09:00 |
*** Jeffrey4l has joined #openstack-nova | 09:00 | |
*** shaohe_feng has quit IRC | 09:00 | |
bauzas | but that's MHO | 09:00 |
bauzas | haypo: now, the point is, we have other things to review at the same time | 09:01 |
bauzas | haypo: given that, merging a long list of changes needs some time | 09:01 |
*** shaohe_feng has joined #openstack-nova | 09:01 | |
bauzas | which is why I think we should discuss on how to help reviewing the long list | 09:02 |
haypo | hum. oslo has 15 core reviewers, cinder has 13, neutron (core) has 19, swift has 16 | 09:02 |
*** rcernin has joined #openstack-nova | 09:02 | |
johnthetubaguy | well, the big issue here is we need lots of good active reviews, some of those I expect to eventually become core reviewers | 09:02 |
*** roeyc has quit IRC | 09:02 | |
johnthetubaguy | we do actively try to reach out to folks and mentor folks to try and get more reviewers | 09:03 |
gibi_ | johnthetubaguy: Hi! Just a frindly reminder that we have some patches in the notifaction transformation that needs some core review. So if you have time, could you please check it? | 09:03 |
*** achanda has joined #openstack-nova | 09:03 | |
johnthetubaguy | gibi_: yeah, thats on my list, been too busy coding this last week, sadly | 09:03 |
gibi_ | johnthetubaguy: You can start here https://review.openstack.org/#/c/321609 this only needs +A | 09:03 |
haypo | bauzas: python3 changes introduce many small changes which can introduce conflicts in other pending changes | 09:03 |
haypo | bauzas: that's the main issues of these changes | 09:03 |
johnthetubaguy | haypo: seems like a really good candidate to do a big sprint of fixes | 09:04 |
bauzas | yup, hence me saying that you should provide us a patches series so we could sprint on it | 09:04 |
bauzas | what johnthetubaguy said | 09:04 |
haypo | johnthetubaguy: i'm open to any idea :) | 09:04 |
bauzas | haypo: but we need a list of patches, if you remember our discussion | 09:04 |
johnthetubaguy | bauzas: so I think we do, in that py34 unit test skip file | 09:05 |
johnthetubaguy | that and the topic that matches the blueprint | 09:05 |
johnthetubaguy | assuming thats where all the patches are right now | 09:05 |
haypo | bauzas: i started with 6 patches. for me, it's hard to have a serie longer than 10 patches. it takes me too much time to update it if it's longer | 09:05 |
bauzas | haypo: okay, the point is, you could get your series very quickly +W'd if we sprint on it | 09:05 |
*** amotoki has joined #openstack-nova | 09:05 | |
johnthetubaguy | bauzas: +1 | 09:05 |
*** markvoelker has joined #openstack-nova | 09:05 | |
bauzas | haypo: that means that if we -1 one of them, we need you to really quickly update it | 09:05 |
johnthetubaguy | bauzas: fells like we pick two days after the midcycle maybe? do you have the time to try organise that? | 09:06 |
*** hieulq has joined #openstack-nova | 09:06 | |
johnthetubaguy | bauzas: good point, it works both ways, we need a fast loop, although for nits the reviewers can always inline edit | 09:06 |
bauzas | johnthetubaguy: well, post-midcycle becomes a defunct time for EU guys :) | 09:07 |
bauzas | because of August :) | 09:07 |
bauzas | at least for French people :' | 09:07 |
bauzas | :( | 09:07 |
bauzas | but we could try to figure out 2 days for that | 09:07 |
johnthetubaguy | bauzas: oh, good point | 09:07 |
bauzas | I should only take 1.5weeks off | 09:07 |
*** syfy323_ has quit IRC | 09:08 | |
*** achanda has quit IRC | 09:08 | |
johnthetubaguy | over here, it depends if you have children at school /nursery or not. | 09:08 |
bauzas | lemme see the agenda | 09:08 |
bauzas | so we meet on R-14 | 09:08 |
* johnthetubaguy points out he cast his vote for remaining in the EU before signing onto IRC | 09:08 | |
haypo | i'm in holiday the first two weeks on august, maybe 3 :) | 09:08 |
bauzas | http://releases.openstack.org/newton/schedule.html | 09:08 |
bauzas | oops | 09:09 |
bauzas | R-11 | 09:09 |
*** dmk0202 has joined #openstack-nova | 09:09 | |
bauzas | but I would only be on PTO by R-9 | 09:09 |
johnthetubaguy | bauzas: that could work, although I don't remember when the API ref one is planned | 09:09 |
bauzas | good point | 09:10 |
bauzas | I heard August | 09:10 |
bauzas | lemme see the logs too | 09:10 |
johnthetubaguy | ah, that rings a bell | 09:10 |
*** markvoelker has quit IRC | 09:10 | |
bauzas | oh, we do freeze on prio specs ? I missed that | 09:10 |
*** jaosorior has quit IRC | 09:11 | |
*** Wenzhi has quit IRC | 09:11 | |
*** Qiming has quit IRC | 09:11 | |
*** ljxiash has quit IRC | 09:11 | |
*** chenfei has quit IRC | 09:11 | |
*** shaohe_feng has quit IRC | 09:11 | |
*** mvk_ has joined #openstack-nova | 09:11 | |
*** jaosorior has joined #openstack-nova | 09:11 | |
*** shaohe_feng has joined #openstack-nova | 09:11 | |
johnthetubaguy | bauzas: yeah, it rumbled on too long last time | 09:11 |
bauzas | that's a good point | 09:12 |
bauzas | anyway | 09:12 |
haypo | johnthetubaguy, bauzas : *if* you want to sprint on python3, i suggest to make in the near future to avoid possible regressions and conflicts near the release (even if i'm confident in my work, and i'm not aware of recent nova issues introduced by py3 work) | 09:12 |
bauzas | R-10 sounds reasonable | 09:12 |
bauzas | haypo: well, the problem is that N-2 is soon | 09:12 |
bauzas | and we have that non-prio FF to honor | 09:13 |
bauzas | which means us needing to review other things impacted by that FF | 09:13 |
haypo | yeah, you don't have enough core reviewers :) | 09:15 |
haypo | and so usual refactoring tasks are always deferred ;) | 09:16 |
*** syfy323 has joined #openstack-nova | 09:16 | |
bauzas | gosh, I wish I would have only done feature develoment in the last 2 years... | 09:19 |
*** irenab has joined #openstack-nova | 09:20 | |
*** lucas____ has joined #openstack-nova | 09:20 | |
*** Qiming has joined #openstack-nova | 09:20 | |
*** chenfei has joined #openstack-nova | 09:21 | |
*** shaohe_feng has quit IRC | 09:21 | |
*** shaohe_feng has joined #openstack-nova | 09:21 | |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/volume/*.py https://review.openstack.org/321467 | 09:22 |
*** zul has joined #openstack-nova | 09:23 | |
*** RuiChen has joined #openstack-nova | 09:24 | |
*** mvk_ has quit IRC | 09:24 | |
*** zhenguo_ has joined #openstack-nova | 09:25 | |
kashyap | johnthetubaguy: How long you had to stand in a line? Or was it super-quick? | 09:27 |
*** Wenzhi has joined #openstack-nova | 09:27 | |
johnthetubaguy | kashyap: I was only there about 5 mins, they were quite efficient thankfully | 09:27 |
johnthetubaguy | kashyap: kinda pleasing to see there was a line! | 09:28 |
kashyap | Yeah. | 09:28 |
*** Jeffrey4l has quit IRC | 09:28 | |
kashyap | And when will the results be announced? /me checks news | 09:28 |
*** lucas____ has quit IRC | 09:29 | |
*** ljxiash has joined #openstack-nova | 09:29 | |
*** amotoki has quit IRC | 09:30 | |
*** hieulq has quit IRC | 09:30 | |
kashyap | Tomorrow breakfast time. | 09:31 |
*** shaohe_feng has quit IRC | 09:31 | |
*** shaohe_feng has joined #openstack-nova | 09:31 | |
*** amotoki has joined #openstack-nova | 09:32 | |
*** ociuhandu has joined #openstack-nova | 09:32 | |
*** yamahata has quit IRC | 09:33 | |
*** amotoki has quit IRC | 09:33 | |
*** lucas____ has joined #openstack-nova | 09:33 | |
openstackgerrit | Dinesh Bhor proposed openstack/nova: [PY3] byte/string conversions and enable PY3 test https://review.openstack.org/333212 | 09:36 |
*** ljxiash has quit IRC | 09:38 | |
*** amotoki has joined #openstack-nova | 09:39 | |
*** shaohe_feng has quit IRC | 09:41 | |
*** Jeffrey4l has joined #openstack-nova | 09:42 | |
*** salv-orlando has joined #openstack-nova | 09:42 | |
*** sdake_ has quit IRC | 09:42 | |
*** shaohe_feng has joined #openstack-nova | 09:42 | |
*** lucas____ has quit IRC | 09:43 | |
*** mvk_ has joined #openstack-nova | 09:43 | |
*** jamielennox|away is now known as jamielennox | 09:43 | |
*** thorst has joined #openstack-nova | 09:44 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 1) https://review.openstack.org/328850 | 09:47 |
*** lucas____ has joined #openstack-nova | 09:47 | |
*** amotoki has quit IRC | 09:50 | |
*** ducttape_ has joined #openstack-nova | 09:51 | |
*** thorst has quit IRC | 09:51 | |
*** pcaruana has joined #openstack-nova | 09:51 | |
*** PaulMurray has joined #openstack-nova | 09:51 | |
*** shaohe_feng has quit IRC | 09:52 | |
*** shaohe_feng has joined #openstack-nova | 09:52 | |
*** GB21 has quit IRC | 09:54 | |
*** ociuhandu has quit IRC | 09:55 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 5) https://review.openstack.org/329979 | 09:56 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2) https://review.openstack.org/329978 | 09:56 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 4) https://review.openstack.org/329227 | 09:56 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 3) https://review.openstack.org/328851 | 09:56 |
*** fragatina has quit IRC | 09:56 | |
*** ducttape_ has quit IRC | 09:57 | |
*** amotoki has joined #openstack-nova | 09:58 | |
*** roeyc has joined #openstack-nova | 09:58 | |
*** gszasz has joined #openstack-nova | 09:58 | |
*** daemontool has quit IRC | 10:00 | |
*** lucas____ has quit IRC | 10:01 | |
*** foexle has joined #openstack-nova | 10:01 | |
*** sdake has joined #openstack-nova | 10:01 | |
openstackgerrit | jichenjc proposed openstack/nova: Remove rate_limit param in builder https://review.openstack.org/333125 | 10:02 |
*** shaohe_feng has quit IRC | 10:02 | |
*** kevin-run has quit IRC | 10:02 | |
*** shaohe_feng has joined #openstack-nova | 10:03 | |
*** jichen has quit IRC | 10:05 | |
*** markvoelker has joined #openstack-nova | 10:07 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 6) https://review.openstack.org/329228 | 10:09 |
*** markvoelker has quit IRC | 10:11 | |
*** vtech has quit IRC | 10:11 | |
*** vtech has joined #openstack-nova | 10:12 | |
*** speller has joined #openstack-nova | 10:12 | |
*** shaohe_feng has quit IRC | 10:12 | |
*** itamarl has quit IRC | 10:12 | |
*** Manikanta has joined #openstack-nova | 10:12 | |
*** shaohe_feng has joined #openstack-nova | 10:12 | |
*** ociuhandu has joined #openstack-nova | 10:13 | |
*** muawiakhan has quit IRC | 10:14 | |
*** hoangcx has quit IRC | 10:14 | |
speller | Found some serious bug when rebooting a cluster - all vms are stuck. Fix was proposed. Anyone has the time to check it out? | 10:16 |
speller | https://bugs.launchpad.net/nova/+bug/1593186 | 10:16 |
openstack | Launchpad bug 1593186 in OpenStack Compute (nova) "Nova instance stuck in powering-off when rebooting all nodes in cluster" [Undecided,In progress] - Assigned to Eyal Posener (eyal-6) | 10:16 |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Introduce utility module for pcp https://review.openstack.org/203163 | 10:17 |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Introduce utility module for pcp https://review.openstack.org/203163 | 10:18 |
openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Add option for running nova independently from hypervisor https://review.openstack.org/332216 | 10:21 |
openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Perform disk operations in dom0 https://review.openstack.org/331254 | 10:21 |
openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Stream config drive to XAPI https://review.openstack.org/332215 | 10:21 |
*** shaohe_feng has quit IRC | 10:22 | |
*** zul has quit IRC | 10:23 | |
*** shaohe_feng has joined #openstack-nova | 10:23 | |
*** achinm has joined #openstack-nova | 10:23 | |
*** GB21 has joined #openstack-nova | 10:24 | |
*** lucas____ has joined #openstack-nova | 10:25 | |
*** sridharg has quit IRC | 10:25 | |
*** itamarl has joined #openstack-nova | 10:25 | |
*** tbachman has quit IRC | 10:28 | |
*** sdake_ has joined #openstack-nova | 10:28 | |
*** dmk0202 has quit IRC | 10:29 | |
andrearosa | gmann_: are you around? | 10:29 |
*** dedery has joined #openstack-nova | 10:31 | |
*** houming has quit IRC | 10:31 | |
*** sdake has quit IRC | 10:32 | |
*** shaohe_feng has quit IRC | 10:33 | |
*** shaohe_feng has joined #openstack-nova | 10:34 | |
*** lucas____ has quit IRC | 10:34 | |
*** phuongnh has quit IRC | 10:35 | |
*** zul has joined #openstack-nova | 10:35 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: os-vif: convert libvirt driver to use os-vif library https://review.openstack.org/269672 | 10:36 |
*** moshele has quit IRC | 10:37 | |
*** zul has quit IRC | 10:38 | |
*** tangchen_ has joined #openstack-nova | 10:38 | |
*** lucas____ has joined #openstack-nova | 10:39 | |
*** moshele has joined #openstack-nova | 10:40 | |
*** sridharg has joined #openstack-nova | 10:41 | |
*** shaohe_feng has quit IRC | 10:43 | |
*** shaohe_feng has joined #openstack-nova | 10:44 | |
*** Hosam_ has joined #openstack-nova | 10:48 | |
*** josecastroleon has quit IRC | 10:48 | |
*** thorst has joined #openstack-nova | 10:49 | |
*** Hosam has quit IRC | 10:50 | |
*** Hosam has joined #openstack-nova | 10:51 | |
*** zul has joined #openstack-nova | 10:52 | |
*** ducttape_ has joined #openstack-nova | 10:52 | |
openstackgerrit | Matthew Booth proposed openstack/nova: Remove max_size argument to images.fetch and fetch_to_raw https://review.openstack.org/326947 | 10:52 |
openstackgerrit | Matthew Booth proposed openstack/nova: Use correct uuids and fnames in test_create_image_initrd https://review.openstack.org/333241 | 10:52 |
openstackgerrit | Matthew Booth proposed openstack/nova: Replace mox with mock in libvirt rescue tests https://review.openstack.org/333242 | 10:52 |
openstackgerrit | Matthew Booth proposed openstack/nova: Pass instance object to _create_images_and_backing in test https://review.openstack.org/333243 | 10:52 |
openstackgerrit | Matthew Booth proposed openstack/nova: Add create_from_image and create_from_func imagebackend methods https://review.openstack.org/333244 | 10:52 |
*** derekh has quit IRC | 10:53 | |
*** winston-d_ has quit IRC | 10:53 | |
*** smatzek has joined #openstack-nova | 10:53 | |
*** shaohe_feng has quit IRC | 10:53 | |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance rescue mode support https://review.openstack.org/217679 | 10:54 |
*** shaohe_feng has joined #openstack-nova | 10:54 | |
*** Hosam_ has quit IRC | 10:55 | |
*** thorst has quit IRC | 10:56 | |
*** danpb has left #openstack-nova | 10:56 | |
*** danpb has joined #openstack-nova | 10:56 | |
*** jistr is now known as jistr|mtg | 10:56 | |
*** ducttape_ has quit IRC | 10:56 | |
*** muawiakhan has joined #openstack-nova | 10:56 | |
*** salv-orl_ has joined #openstack-nova | 10:58 | |
openstackgerrit | Victor Stinner proposed openstack/nova: Port virt test_hardware to Python 3 https://review.openstack.org/333247 | 10:59 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port test_virt_drivers to Python 3 https://review.openstack.org/333248 | 10:59 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port libvirt test_driver to Python 3 https://review.openstack.org/333249 | 10:59 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port ironic test_driver to Python 3 https://review.openstack.org/333250 | 10:59 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port test_fakelibvirt to Python 3 https://review.openstack.org/333251 | 10:59 |
haypo | bauzas: ^^ you asked for me food, here you have ^^ | 10:59 |
haypo | bauzas: so i have 10 pending patches. as i expected, i'm unable to maintain a longer patch serie | 11:00 |
*** salv-orlando has quit IRC | 11:01 | |
*** tbachman has joined #openstack-nova | 11:02 | |
*** moshele has quit IRC | 11:02 | |
*** moshele has joined #openstack-nova | 11:03 | |
*** shaohe_feng has quit IRC | 11:03 | |
*** shaohe_feng has joined #openstack-nova | 11:04 | |
*** achanda has joined #openstack-nova | 11:05 | |
*** sudipto has quit IRC | 11:07 | |
*** markvoelker has joined #openstack-nova | 11:07 | |
*** moshele has quit IRC | 11:08 | |
*** hshiina has joined #openstack-nova | 11:09 | |
*** achanda has quit IRC | 11:09 | |
*** josecastroleon has joined #openstack-nova | 11:12 | |
*** markvoelker has quit IRC | 11:13 | |
*** shaohe_feng has quit IRC | 11:14 | |
*** shaohe_feng has joined #openstack-nova | 11:15 | |
*** kolesovdv has joined #openstack-nova | 11:15 | |
*** zul has quit IRC | 11:16 | |
*** zul has joined #openstack-nova | 11:17 | |
openstackgerrit | Matthew Booth proposed openstack/nova: Remove max_size argument to images.fetch and fetch_to_raw https://review.openstack.org/326947 | 11:19 |
openstackgerrit | Matthew Booth proposed openstack/nova: Add create_from_image and create_from_func imagebackend methods https://review.openstack.org/333244 | 11:19 |
openstackgerrit | Matthew Booth proposed openstack/nova: Use correct uuids and fnames in test_create_image_initrd https://review.openstack.org/333241 | 11:19 |
openstackgerrit | Matthew Booth proposed openstack/nova: Replace mox with mock in libvirt rescue tests https://review.openstack.org/333242 | 11:19 |
openstackgerrit | Matthew Booth proposed openstack/nova: Pass instance object to _create_images_and_backing in test https://review.openstack.org/333243 | 11:19 |
openstackgerrit | Matthew Booth proposed openstack/nova: Pass is_block_dev explicitly from create_from_func https://review.openstack.org/333263 | 11:19 |
*** lucasagomes is now known as lucas-hungry | 11:19 | |
*** lucas____ has quit IRC | 11:20 | |
*** sdake_ has quit IRC | 11:21 | |
*** vtech has quit IRC | 11:22 | |
*** sdague has joined #openstack-nova | 11:22 | |
*** shaohe_feng has quit IRC | 11:24 | |
*** shaohe_feng has joined #openstack-nova | 11:25 | |
*** lucas____ has joined #openstack-nova | 11:25 | |
*** ekarlso has quit IRC | 11:26 | |
*** daemontool has joined #openstack-nova | 11:28 | |
*** lucas____ has quit IRC | 11:29 | |
*** lucas____ has joined #openstack-nova | 11:30 | |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass path to Image base class https://review.openstack.org/333271 | 11:33 |
openstackgerrit | Matthew Booth proposed openstack/nova: Rename snapshot() to from_libvirt_path() https://review.openstack.org/333272 | 11:33 |
*** dmk0202 has joined #openstack-nova | 11:33 | |
*** lucas____ has quit IRC | 11:34 | |
*** shaohe_feng has quit IRC | 11:34 | |
*** lucas____ has joined #openstack-nova | 11:34 | |
markus_z | danpb: I'd love to have your feedback on the virtlogd change https://review.openstack.org/#/c/323765/8 whenever it's convenient for you. | 11:35 |
*** shaohe_feng has joined #openstack-nova | 11:35 | |
*** rtheis has joined #openstack-nova | 11:35 | |
*** bkopilov has quit IRC | 11:38 | |
*** dedery has quit IRC | 11:38 | |
danpb | markus_z: ok | 11:40 |
*** GB21 has quit IRC | 11:40 | |
*** dedery has joined #openstack-nova | 11:41 | |
openstackgerrit | Jens Rosenboom proposed openstack/nova: Add ability to select specific tests for py34 https://review.openstack.org/333274 | 11:41 |
*** dedery_ has joined #openstack-nova | 11:41 | |
*** thorst has joined #openstack-nova | 11:41 | |
*** GB21 has joined #openstack-nova | 11:42 | |
*** dedery__ has joined #openstack-nova | 11:42 | |
openstackgerrit | John Garbutt proposed openstack/nova: Add prototype feature classification matrix https://review.openstack.org/264719 | 11:43 |
*** fragatina has joined #openstack-nova | 11:43 | |
*** lucas____ has quit IRC | 11:43 | |
openstackgerrit | John Garbutt proposed openstack/nova: Expand initial feature classification lists https://review.openstack.org/323932 | 11:44 |
*** shaohe_feng has quit IRC | 11:44 | |
*** dedery___ has joined #openstack-nova | 11:45 | |
*** shaohe_feng has joined #openstack-nova | 11:45 | |
*** dedery has quit IRC | 11:45 | |
*** dedery___ has quit IRC | 11:46 | |
*** dedery_ has quit IRC | 11:46 | |
*** dedery has joined #openstack-nova | 11:46 | |
*** dedery__ has quit IRC | 11:47 | |
*** lucas____ has joined #openstack-nova | 11:48 | |
*** ekarlso has joined #openstack-nova | 11:51 | |
*** armax has joined #openstack-nova | 11:53 | |
*** ducttape_ has joined #openstack-nova | 11:53 | |
*** tbachman has quit IRC | 11:53 | |
*** rfolco has joined #openstack-nova | 11:53 | |
*** salv-orl_ has quit IRC | 11:53 | |
*** lucas____ has quit IRC | 11:54 | |
*** lucas____ has joined #openstack-nova | 11:54 | |
*** sree has joined #openstack-nova | 11:54 | |
*** Hosam_ has joined #openstack-nova | 11:54 | |
*** daemontool_ has joined #openstack-nova | 11:54 | |
*** shaohe_feng has quit IRC | 11:55 | |
*** daemontool has quit IRC | 11:55 | |
*** eharney has joined #openstack-nova | 11:55 | |
*** ducttape_ has quit IRC | 11:56 | |
*** muawiakhan has quit IRC | 11:56 | |
*** ducttape_ has joined #openstack-nova | 11:56 | |
*** Hosam has quit IRC | 11:56 | |
*** Hosam has joined #openstack-nova | 11:57 | |
*** shaohe_feng has joined #openstack-nova | 11:58 | |
*** markvoelker has joined #openstack-nova | 11:59 | |
*** Hosam_ has quit IRC | 12:00 | |
*** tbachman has joined #openstack-nova | 12:01 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 5) https://review.openstack.org/329979 | 12:03 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2) https://review.openstack.org/329978 | 12:03 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 4) https://review.openstack.org/329227 | 12:03 |
*** lucas____ has quit IRC | 12:03 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 6) https://review.openstack.org/329228 | 12:03 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 3) https://review.openstack.org/328851 | 12:03 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 1) https://review.openstack.org/328850 | 12:03 |
*** lucas-hungry is now known as lucas-afk | 12:04 | |
*** DaiDV has joined #openstack-nova | 12:04 | |
haypo | bauzas: ah, the patch i sent 4 hours ago is already in conflict https://review.openstack.org/#/c/332738/ :-) that's why i cannot maintain a patch serie longer than 10 changes ;) | 12:04 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add DeviceTaggingFilter to scheduler https://review.openstack.org/331490 | 12:05 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: attach configdrive after instance XML https://review.openstack.org/303335 | 12:05 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: adding an update method to virtual_interface https://review.openstack.org/310618 | 12:05 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: Add devices_metadata to instance object https://review.openstack.org/327920 | 12:05 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: add a device_metadata column to instance_extra https://review.openstack.org/318883 | 12:05 |
*** shaohe_feng has quit IRC | 12:05 | |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 12:05 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: enhance the DeviceMetadataList object https://review.openstack.org/318884 | 12:05 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 12:05 |
*** shaohe_feng has joined #openstack-nova | 12:05 | |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Compute manager device tagging support https://review.openstack.org/264017 | 12:05 |
*** lchen_ has joined #openstack-nova | 12:05 | |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 12:06 |
*** achanda has joined #openstack-nova | 12:06 | |
*** diga has quit IRC | 12:06 | |
*** GB21 has quit IRC | 12:07 | |
*** roeyc has quit IRC | 12:07 | |
*** fragatina has quit IRC | 12:07 | |
*** lucas____ has joined #openstack-nova | 12:08 | |
*** ducttape_ has quit IRC | 12:10 | |
*** achanda has quit IRC | 12:10 | |
*** vtech has joined #openstack-nova | 12:11 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: numa: Fixes NUMA topology related unit tests https://review.openstack.org/278835 | 12:11 |
openstackgerrit | Claudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests https://review.openstack.org/278834 | 12:11 |
openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 unit tests https://review.openstack.org/278810 | 12:11 |
openstackgerrit | Claudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests https://review.openstack.org/300262 | 12:12 |
*** aysyd has joined #openstack-nova | 12:13 | |
*** prateek has quit IRC | 12:14 | |
*** shaohe_feng has quit IRC | 12:15 | |
*** shaohe_feng has joined #openstack-nova | 12:16 | |
*** ratailor has quit IRC | 12:17 | |
*** tbachman has quit IRC | 12:18 | |
openstackgerrit | Eyal Posener proposed openstack/nova: Fix nova-compute service start order https://review.openstack.org/330556 | 12:19 |
*** pradk has joined #openstack-nova | 12:21 | |
*** zerda2 has quit IRC | 12:22 | |
*** fragatina has joined #openstack-nova | 12:22 | |
*** liverpooler has quit IRC | 12:22 | |
*** amotoki has quit IRC | 12:23 | |
*** GB21 has joined #openstack-nova | 12:24 | |
*** chandankumar has quit IRC | 12:24 | |
*** shaohe_feng has quit IRC | 12:25 | |
*** tbachman has joined #openstack-nova | 12:26 | |
*** shaohe_feng has joined #openstack-nova | 12:26 | |
*** lucas____ has quit IRC | 12:26 | |
openstackgerrit | Sean Dague proposed openstack/nova: remove personality extension https://review.openstack.org/333030 | 12:27 |
openstackgerrit | Sudipta Biswas proposed openstack/nova: Introduce utility module for pcp https://review.openstack.org/203163 | 12:28 |
*** sudipto has joined #openstack-nova | 12:29 | |
*** erlon has joined #openstack-nova | 12:30 | |
*** dedery has quit IRC | 12:31 | |
*** lucas____ has joined #openstack-nova | 12:31 | |
openstackgerrit | Vladyslav Drok proposed openstack/nova: Update instance node on rebuild only when it is recreate https://review.openstack.org/306010 | 12:32 |
*** armax has quit IRC | 12:33 | |
*** lpetrut has quit IRC | 12:33 | |
*** roeyc has joined #openstack-nova | 12:34 | |
*** shaohe_feng has quit IRC | 12:36 | |
sdague | johnthetubaguy: https://review.openstack.org/#/c/333030/ stack gets rid of 3 entry points in our setup.cfg for extensions of servers | 12:36 |
*** shaohe_feng has joined #openstack-nova | 12:36 | |
sdague | would be great to get more eyes on it :) | 12:36 |
*** M00nr41n has quit IRC | 12:37 | |
*** mfedosin has quit IRC | 12:37 | |
*** GB21 has quit IRC | 12:37 | |
*** abalutoiu has joined #openstack-nova | 12:39 | |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Implement stable disk device instance rescues https://review.openstack.org/270289 | 12:39 |
openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Allow volume backed instances to be rescued by capable virt drivers https://review.openstack.org/270288 | 12:39 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Add boot ordering to individual disks https://review.openstack.org/270286 | 12:39 |
openstackgerrit | Lee Yarwood proposed openstack/nova: virt: Provide block_device_info during a rescue https://review.openstack.org/270287 | 12:39 |
openstackgerrit | Lee Yarwood proposed openstack/nova: image_meta: Add hw_rescue_device and hw_rescue_bus https://review.openstack.org/270285 | 12:39 |
johnthetubaguy | sdague: yeah, good point, adding to my list | 12:40 |
*** kylek3h has joined #openstack-nova | 12:40 | |
*** lucas____ has quit IRC | 12:40 | |
*** syfy323 has quit IRC | 12:40 | |
*** ametts has joined #openstack-nova | 12:43 | |
*** lucas___ has joined #openstack-nova | 12:45 | |
*** shaohe_feng has quit IRC | 12:46 | |
*** pradk has quit IRC | 12:46 | |
*** shaohe_feng has joined #openstack-nova | 12:47 | |
*** ljxiash has joined #openstack-nova | 12:47 | |
*** salv-orlando has joined #openstack-nova | 12:47 | |
wznoinsk | hi all | 12:47 |
openstackgerrit | John Garbutt proposed openstack/nova: Expand initial feature classification lists https://review.openstack.org/323932 | 12:48 |
*** sdake has joined #openstack-nova | 12:48 | |
*** tongli has joined #openstack-nova | 12:48 | |
sdague | johnthetubaguy: also the extensions remove and claudiub's policy in code are going to be collision heavy, so we should try to keep on top of those | 12:51 |
*** janonymous has quit IRC | 12:51 | |
sdague | I just +2ed claudiub's stack | 12:51 |
claudiub | \o/ | 12:51 |
sdague | at least the policy definitions into the code | 12:51 |
*** fragatina has quit IRC | 12:51 | |
johnthetubaguy | sdague: yes, let me jump on that now, its high up my list anyways | 12:52 |
sdague | johnthetubaguy / claudiub / laski - we should also get this in https://review.openstack.org/#/c/332995/ | 12:52 |
wznoinsk | is there any news on the feature classification work (I'm reading Matt's email from May with recap of Austin session and https://etherpad.openstack.org/p/newton-nova-feature-classification)... are there any proposals which features will land in which bucket (incomplete, experimental) | 12:52 |
sdague | which will conflict with patch 6 on claudiub's stack, but is easy to resolve | 12:52 |
wznoinsk | johnthetubaguy ^ | 12:52 |
johnthetubaguy | wznoinsk: there are patches up right now | 12:53 |
johnthetubaguy | wznoinsk: raj_singh is taking a good look at the this too | 12:54 |
openstackgerrit | John Garbutt proposed openstack/nova: Adding details in general purpose feature matrix [1] https://review.openstack.org/327326 | 12:54 |
johnthetubaguy | wznoinsk: actually thats one^ | 12:54 |
wznoinsk | looking, thanks | 12:55 |
wznoinsk | given different categories (cloud, nfv etc.) does it mean there will be more (granular) feature groups maybe? | 12:56 |
*** xyang1 has joined #openstack-nova | 12:56 | |
*** shaohe_feng has quit IRC | 12:56 | |
wznoinsk | thinking SR-IOV, numa, pinning (all EPA) here | 12:57 |
*** edmondsw has joined #openstack-nova | 12:57 | |
*** shaohe_feng has joined #openstack-nova | 12:57 | |
*** tblakeslee has joined #openstack-nova | 12:57 | |
johnthetubaguy | wznoinsk: so sfinucan is taking a look at those, I was hoping to group them, and add a new column for those | 12:57 |
johnthetubaguy | I am not sure he had the same idea | 12:57 |
*** syfy323 has joined #openstack-nova | 12:57 | |
wznoinsk | cool, will check with him then | 12:57 |
openstackgerrit | John Garbutt proposed openstack/nova: Copy edit feature classification https://review.openstack.org/323671 | 12:58 |
sdague | ah patch 6 failed anyway because it was relying on _common which is now gone | 12:59 |
*** rlrossit has joined #openstack-nova | 13:00 | |
claudiub | yeah... missed that one. | 13:00 |
*** baoli has joined #openstack-nova | 13:01 | |
claudiub | i'll actually wait a bit before sending another PS. it will end up in a merge conflict anyways. | 13:02 |
alaski | claudiub: you could just tack it on after sdagues | 13:02 |
*** Hosam has quit IRC | 13:02 | |
claudiub | yeah.. | 13:02 |
*** eharney has quit IRC | 13:03 | |
*** Hosam has joined #openstack-nova | 13:03 | |
*** lucas___ has quit IRC | 13:03 | |
*** hshiina has quit IRC | 13:03 | |
*** fragatina has joined #openstack-nova | 13:05 | |
johnthetubaguy | so alaski beat me too it, all good | 13:05 |
*** shaohe_feng has quit IRC | 13:06 | |
johnthetubaguy | claudiub: whats the plan for "cells_scheduler_filter:TargetCellFilter" and "network:attach_external_network"? | 13:07 |
*** shaohe_feng has joined #openstack-nova | 13:07 | |
*** Hosam has quit IRC | 13:07 | |
*** lucas___ has joined #openstack-nova | 13:08 | |
claudiub | johnthetubaguy: no idea. | 13:08 |
johnthetubaguy | claudiub: seems like they need pulling into code in the exact same way | 13:08 |
*** mdrabe has joined #openstack-nova | 13:09 | |
johnthetubaguy | claudiub: also, can't we also delete these ones now? https://review.openstack.org/#/c/328850/13/etc/nova/policy.json@4 | 13:09 |
*** ociuhandu has quit IRC | 13:09 | |
openstackgerrit | Daniel Berrange proposed openstack/nova: os-vif: convert libvirt driver to use os-vif library https://review.openstack.org/269672 | 13:10 |
*** BjoernT has joined #openstack-nova | 13:11 | |
alaski | johnthetubaguy: if I could comment in the policy.json I would have. I purposely left those because they're still referenced in the file | 13:11 |
alaski | they're not needed, but I thought it would be odd to have them used but not defined | 13:11 |
*** sree has quit IRC | 13:11 | |
johnthetubaguy | alaski: fair enough, I wondering if that was why. They go once we remove the stragglers right? | 13:12 |
speller | Found some serious bug when rebooting a cluster - all vms are stuck. Fix was proposed. Anyone has the time to check it out? | 13:12 |
speller | https://bugs.launchpad.net/nova/+bug/1593186 | 13:12 |
openstack | Launchpad bug 1593186 in OpenStack Compute (nova) "Nova instance stuck in powering-off when rebooting all nodes in cluster" [Undecided,In progress] - Assigned to Eyal Posener (eyal-6) | 13:12 |
alaski | however you're like the fourth person to ask about dropping them so maybe I should have just removed them :) | 13:12 |
alaski | johnthetubaguy: yep, they'll be the last to go | 13:12 |
sdague | johnthetubaguy: at the end of claudio's stack, they are the last ones there. | 13:13 |
sdague | I guess a question on that .... | 13:13 |
johnthetubaguy | sdague: yeah, that was where my question came from | 13:13 |
sdague | alaski / claudiub how are we handling known vs. unknown RHS values | 13:14 |
*** kfarr has joined #openstack-nova | 13:14 | |
sdague | if I have a file that is only | 13:14 |
*** bkopilov has joined #openstack-nova | 13:14 | |
sdague | "os_compute_api:servers:migrations:show": "admin_or_owner" | 13:14 |
*** zz_dimtruck is now known as dimtruck | 13:15 | |
sdague | I assume we correctly resolve "admin_or_owner" in our engine? | 13:15 |
* bauzas loves to be remote, but rather in winter | 13:15 | |
sdague | also, if they specify "admim_or_owner" do we throw an exception or warning | 13:15 |
johnthetubaguy | its "rule:admin_or_owner" I guess, I assume that just gets resolved at the end? | 13:15 |
johnthetubaguy | hmm, good point | 13:15 |
sdague | johnthetubaguy: right, my bad on that | 13:16 |
alaski | sdague: nothing about that changes from how it would be handled with pure file loading | 13:16 |
alaski | which is that no exception is raised until it's checked | 13:16 |
sdague | alaski: what happens today? | 13:16 |
sdague | ah | 13:16 |
sdague | that might be a nice future enhancement | 13:16 |
alaski | policy loading is just building a dict of policy definitions | 13:16 |
sdague | right | 13:16 |
alaski | all resolution happens at enforcement time | 13:17 |
*** shaohe_feng has quit IRC | 13:17 | |
sdague | ok | 13:17 |
alaski | yeah, would definitely be a good enhancement | 13:17 |
johnthetubaguy | we need to do the same for config, based on the operator discussion | 13:17 |
johnthetubaguy | at the summit | 13:17 |
danpb | lyarwood: ok, your rescue dev series looks fine to me now - just need to find another willing nova-core to review it now.... | 13:17 |
*** shaohe_feng has joined #openstack-nova | 13:17 | |
alaski | at that point we need to think of it more as a graph then a dict, and maybe use a different datastructure | 13:17 |
sdague | ok, cool, well ocata improvements | 13:18 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 1) https://review.openstack.org/328850 | 13:18 |
sdague | alaski: ++ | 13:18 |
sdague | I'm just super excited that we're going to be able to run without a policy file :) | 13:18 |
danpb | for those listening in.... https://review.openstack.org/#/q/topic:bp/virt-rescue-stable-disk-devices+status:open+project:openstack/nova | 13:18 |
lyarwood | danpb: ah great thanks! Does it also need a releasenote somewhere or can I add that later? | 13:18 |
*** bkopilov has quit IRC | 13:19 | |
danpb | oh yes, actually it does - you should add tat as part of the last patch | 13:19 |
* danpb keeps forgetting we have reno now | 13:19 | |
lyarwood | danpb: ack, I'll look into that now, thanks again | 13:19 |
*** pwnall has joined #openstack-nova | 13:20 | |
*** lucas___ has quit IRC | 13:22 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 5) https://review.openstack.org/329979 | 13:25 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2) https://review.openstack.org/329978 | 13:25 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 4) https://review.openstack.org/329227 | 13:25 |
claudiub | got to run at the moment... dentist time.. | 13:25 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 3) https://review.openstack.org/328851 | 13:25 |
*** akshai has joined #openstack-nova | 13:25 | |
*** lucas___ has joined #openstack-nova | 13:26 | |
*** shaohe_feng has quit IRC | 13:27 | |
*** lucas___ has quit IRC | 13:27 | |
*** lucas___ has joined #openstack-nova | 13:27 | |
*** sree has joined #openstack-nova | 13:28 | |
*** shaohe_feng has joined #openstack-nova | 13:28 | |
*** kevin-run has joined #openstack-nova | 13:31 | |
*** claudiub has quit IRC | 13:31 | |
*** jistr|mtg is now known as jistr | 13:31 | |
*** mriedem has joined #openstack-nova | 13:32 | |
*** david-lyle_ has joined #openstack-nova | 13:32 | |
*** catintheroof has joined #openstack-nova | 13:32 | |
*** prateek has joined #openstack-nova | 13:32 | |
*** baoli_ has joined #openstack-nova | 13:33 | |
*** eharney has joined #openstack-nova | 13:33 | |
*** baoli has quit IRC | 13:35 | |
*** achanda has joined #openstack-nova | 13:35 | |
*** kevin-run has quit IRC | 13:35 | |
*** david-lyle has quit IRC | 13:36 | |
BobBall | johnthetubaguy: Are you happy for auto_config_disk to not be supported with independent_compute? | 13:36 |
*** kevin-run has joined #openstack-nova | 13:36 | |
*** mriedem1 has joined #openstack-nova | 13:36 | |
johnthetubaguy | BobBall: seems like an OK starting point, I would rather we discussed removing it instead | 13:36 |
*** shaohe_feng has quit IRC | 13:37 | |
BobBall | johnthetubaguy: Happy to remove / discuss removing it for sure; but I guess we'd need another BP for that, and definitely would need a deprecation cycle so doesn't help this BP really. OK - will not permit auto_config_disk when using independent_compute. | 13:37 |
*** luis5tb has quit IRC | 13:38 | |
*** shaohe_feng has joined #openstack-nova | 13:38 | |
*** mriedem1 has left #openstack-nova | 13:38 | |
johnthetubaguy | BobBall: yeah, its a tricky one talking with our product managers really, its something we should have killed ages ago, that and resize down | 13:38 |
*** mriedem has quit IRC | 13:39 | |
*** claudiub has joined #openstack-nova | 13:39 | |
*** salv-orlando has quit IRC | 13:40 | |
*** salv-orlando has joined #openstack-nova | 13:41 | |
*** mriedem has joined #openstack-nova | 13:41 | |
*** cdent has joined #openstack-nova | 13:41 | |
*** mfedosin has joined #openstack-nova | 13:41 | |
openstackgerrit | Sean Dague proposed openstack/nova: Add note about preserve_ephemeral limitations https://review.openstack.org/333358 | 13:42 |
lyarwood | Any cores able to review the following bugfix? https://review.openstack.org/#/c/318266/ - I'd also like to get this into stable/mitaka if possible. | 13:42 |
sdague | that is a pretty quick doc add based on digging through the extensions yesterday | 13:43 |
*** dimtruck is now known as zz_dimtruck | 13:44 | |
mriedem | lyarwood: dropped a quick -1 on that :) | 13:45 |
lyarwood | mriedem: kk checking, thanks! | 13:45 |
mriedem | lyarwood: is that safe for vmware? | 13:45 |
openstackgerrit | Pavel Kholkin proposed openstack/nova: Microversion 2.31 adds keypairs pagination support https://review.openstack.org/284868 | 13:46 |
*** achanda has quit IRC | 13:46 | |
*** foexle has quit IRC | 13:46 | |
*** danpb is now known as danpb_votey_mcvo | 13:46 | |
lyarwood | mriedem: AFAIK yeah, the instance.host still points to the nova-compute host not the actual hypervisor | 13:46 |
danpb_votey_mcvo | hmm, silly nick name limits on this irc server :-( | 13:46 |
lyarwood | mriedem: I might be wrong, so happy to be corrected | 13:46 |
*** danpb_votey_mcvo is now known as danpb | 13:46 | |
mriedem | lyarwood: i'd like to just see a ci run | 13:47 |
mriedem | trying to find the recheck command for vmware | 13:47 |
mriedem | which is always impossible | 13:47 |
mriedem | tjones: can't the nsx ci leave a comment with the recheck command to use? | 13:47 |
dansmith | mriedem: yeah, instance.host is the compute hostname even in vmware, AFAIK | 13:47 |
dansmith | mriedem: instance.node could have been different in some models before, but I think that was all removed | 13:47 |
*** shaohe_feng has quit IRC | 13:47 | |
dansmith | mriedem: I believe only ironic can have a different .node now, but .host is still the compute service host | 13:48 |
*** prateek has quit IRC | 13:48 | |
mriedem | ok | 13:48 |
mriedem | just rechecked it for vmware | 13:48 |
*** shaohe_feng has joined #openstack-nova | 13:48 | |
*** ctrath has joined #openstack-nova | 13:49 | |
*** roeyc has quit IRC | 13:49 | |
mriedem | lyarwood: https://bugs.launchpad.net/nova/+bug/1583284 doesn't say which release you recreated this in | 13:49 |
openstack | Launchpad bug 1583284 in OpenStack Compute (nova) "disconnect_volume calls are made during a remote rebuild of a volume backed instance" [Undecided,In progress] - Assigned to Lee Yarwood (lyarwood) | 13:49 |
mriedem | liberty? mitaka? | 13:49 |
lyarwood | mriedem: ah fail, I'll add that now, given the date this was reproduced after mitaka GA but we've had reports against mitaka, liberty and kilo downstream. | 13:51 |
*** syjulian has joined #openstack-nova | 13:54 | |
diana_clarke | dansmith: Good news - Virtuozzo CI is green now (re: procedural -2 on the image backend patch series). cc: mdbooth | 13:54 |
dansmith | diana_clarke: cool, and the later ones are all reviewed and ready? | 13:55 |
dansmith | 'cause mdbooth said they should land as a set, and I agree | 13:55 |
*** dave-mccowan has joined #openstack-nova | 13:55 | |
*** liverpooler has joined #openstack-nova | 13:55 | |
*** lpetrut has joined #openstack-nova | 13:56 | |
*** pradk has joined #openstack-nova | 13:57 | |
*** diga has joined #openstack-nova | 13:57 | |
diana_clarke | dansmith: Most of the earlier patches in this series have merged already. Next up, are these individual image backend patches. None of them have really been reviewed yet. | 13:58 |
*** shaohe_feng has quit IRC | 13:58 | |
*** shaohe_feng has joined #openstack-nova | 13:58 | |
*** smatzek has quit IRC | 13:58 | |
*** zul_ has joined #openstack-nova | 13:59 | |
*** zul has quit IRC | 14:00 | |
mriedem | diana_clarke: anything happen with tempest testing for resize? | 14:00 |
mriedem | re: the live migration meeting on tuesday | 14:00 |
*** rbradfor has joined #openstack-nova | 14:00 | |
*** tblakeslee has quit IRC | 14:00 | |
diana_clarke | mriedem: I'm hoping to get tempest figured out today (it's new to me). My goal is to increase the coverage around resize. | 14:01 |
*** sree has quit IRC | 14:01 | |
*** zul_ has quit IRC | 14:01 | |
mriedem | diana_clarke: ok, let me know if you have questions | 14:02 |
diana_clarke | mriedem: thanks, much appreciated | 14:02 |
mriedem | should be able to start from an existing resize scenario test but add ssh to it | 14:02 |
*** sree has joined #openstack-nova | 14:02 | |
*** zul has joined #openstack-nova | 14:02 | |
openstackgerrit | Gergely Csatari proposed openstack/nova: api-ref: Removing host_status from Update Server response parameter list https://review.openstack.org/333365 | 14:02 |
mriedem | there is an existing volume-backed resize scenario test, but we don't ssh into the guest to verify the size of the disks before and after the resize | 14:02 |
*** rbradfor has quit IRC | 14:02 | |
mriedem | i need to review the virtuozzo resize/rescue patches again today now that they have ci working | 14:03 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [2/3] https://review.openstack.org/331632 | 14:03 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [3/3] https://review.openstack.org/331633 | 14:03 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [1/3] https://review.openstack.org/280301 | 14:03 |
*** sree has quit IRC | 14:03 | |
diana_clarke | mriedem: cCol, that helps. I'll look at that test when I start to write new tests (or modify existing ones). | 14:03 |
*** sree has joined #openstack-nova | 14:04 | |
diana_clarke | s/cCol/Cool/ | 14:04 |
*** achanda has joined #openstack-nova | 14:05 | |
*** rbradfor has joined #openstack-nova | 14:05 | |
*** lucas___ has quit IRC | 14:06 | |
*** sree has quit IRC | 14:06 | |
*** achanda has quit IRC | 14:07 | |
*** sree has joined #openstack-nova | 14:07 | |
*** tbachman_ has joined #openstack-nova | 14:07 | |
*** shaohe_feng has quit IRC | 14:08 | |
*** Dinesh_Bhor has joined #openstack-nova | 14:09 | |
*** shaohe_feng has joined #openstack-nova | 14:09 | |
mriedem | sdague: +2 on https://review.openstack.org/#/c/330207/ now | 14:09 |
*** tblakeslee has joined #openstack-nova | 14:11 | |
*** lucas___ has joined #openstack-nova | 14:11 | |
*** nkrinner is now known as nkrinner_afk | 14:11 | |
*** tbachman has quit IRC | 14:11 | |
*** tbachman_ is now known as tbachman | 14:11 | |
*** sree has quit IRC | 14:11 | |
*** MikeG451 has joined #openstack-nova | 14:12 | |
sdague | mriedem: great | 14:12 |
danpb | artom: vladikr so the device tagging looks basically good to me - the only thing remaining i don't like is the way we attach the config drive to xml afterwards | 14:12 |
sdague | that whole stack gets rid of all the non create entry points | 14:12 |
danpb | if you can get that sorted, hopefully we might be able to find a second +2/+W before freeze | 14:12 |
*** diga has quit IRC | 14:12 | |
*** woodster_ has joined #openstack-nova | 14:12 | |
*** rlrossit has quit IRC | 14:12 | |
*** achinm has quit IRC | 14:13 | |
*** sdake has quit IRC | 14:13 | |
dansmith | mriedem: don't you have an experimental job that uses lvm or something for testing that libvirt image stack? | 14:14 |
mriedem | dansmith: yeah | 14:15 |
mriedem | check experimental | 14:15 |
*** lucas___ has quit IRC | 14:15 | |
mriedem | dansmith: we have a DNM devstack change to test the flat backend, https://review.openstack.org/#/c/318262/ | 14:15 |
dansmith | mriedem: ah, I see one experimental job is still running on this | 14:15 |
*** sdake has joined #openstack-nova | 14:16 | |
diana_clarke | dansmith: Yup, check experimental was already passing last week, that running job is just me double checking. | 14:16 |
dansmith | mriedem: cool, thanks | 14:16 |
*** kfox1111 is now known as kfox1111_away | 14:17 | |
vladikr | danpb, thanks. I was trying to sort this w/o attaching the config disk after the the define. We've ran into issues, specifically with rbd, where some methods are checking if the image is actually present, if it's not there, it drops back to a flat configuration.. are you suggesting to generate the image and import it twice? Once during a regular flow and then after the define? | 14:18 |
*** shaohe_feng has quit IRC | 14:18 | |
*** shaohe_feng has joined #openstack-nova | 14:18 | |
*** baoli_ has quit IRC | 14:19 | |
danpb | vladikr: what methods are doing such checks ? code in nova, or code in libvirt ? | 14:19 |
danpb | we really ought to be able to create whatever XML we like regardless of whether the things it references currently exist on disk | 14:19 |
*** ljxiash has quit IRC | 14:19 | |
*** ljxiash has joined #openstack-nova | 14:20 | |
vladikr | danpb, yeah.. it's in nova, let me look it up | 14:20 |
* bauzas this is a gentle notice that if people want to review python-novaclient changes, they could look at https://review.openstack.org/#/q/status:open+project:openstack/python-novaclient+branch:master+topic:bp/check-destination-on-migrations-newton | 14:20 | |
*** baoli has joined #openstack-nova | 14:21 | |
*** zz_dimtruck is now known as dimtruck | 14:23 | |
*** vtech has quit IRC | 14:23 | |
*** amotoki has joined #openstack-nova | 14:23 | |
*** ctrath has quit IRC | 14:24 | |
Dinesh_Bhor | oomichi: Hi, Could you please take a look at it ? https://review.openstack.org/#/c/315964/ | 14:24 |
*** josecastroleon has quit IRC | 14:26 | |
*** foexle has joined #openstack-nova | 14:26 | |
*** ctrath has joined #openstack-nova | 14:26 | |
*** shaohe_feng has quit IRC | 14:28 | |
openstackgerrit | Sean Dague proposed openstack/nova: api-ref: merge multiple create to servers.inc https://review.openstack.org/333378 | 14:28 |
*** shaohe_feng has joined #openstack-nova | 14:29 | |
*** rlrossit has joined #openstack-nova | 14:30 | |
*** mmandel has joined #openstack-nova | 14:30 | |
*** claudiub has quit IRC | 14:31 | |
*** ducttape_ has joined #openstack-nova | 14:31 | |
*** amotoki has quit IRC | 14:32 | |
cdent | mriedem: there's log history that suggests this placement api debate still has more debate pending. What needs to happen to land, somewhere, anywhere? | 14:33 |
*** lucas___ has joined #openstack-nova | 14:34 | |
mriedem | cdent: i was out most of yesterday so haven't gotten caught up on all of the thread. we didn't talk about it in the api meeting. are you going to be in the nova meeting tomorrow? | 14:34 |
mriedem | cdent: honestly i've been staying out of it really because i don't have any experience with flask or any of the other frameworks, i'm only used to what nova has. | 14:35 |
*** PaulMurray has quit IRC | 14:35 | |
*** salv-orlando has quit IRC | 14:35 | |
mriedem | i think it's important that the nova team work on whatever we use, so anything too unique would not be a good fit imo | 14:36 |
*** salv-orlando has joined #openstack-nova | 14:36 | |
mriedem | i'm also not crazy about more new dependencies in g-r | 14:36 |
mriedem | i realize a lot of the existing wsgi stack in nova is not great, but a lot of the stuff that's going on now is making that a lot better, like the ec2 removal, legacy v2 removal, extension folding/removal, etc | 14:37 |
cdent | do you mean today's meeting? I can be there, if needed. I've figured out how to get rid of the selector dependency, leaving what's there (which would basically mean raw-wsgi + Routes) | 14:37 |
mriedem | cdent: yeah :) today is thursday isn't it. | 14:37 |
*** ayoung has quit IRC | 14:37 | |
*** jistr is now known as jistr|mtg | 14:38 | |
mriedem | i'd also like the schema validation and microversion support in the new placement api to not be drastically different from nova-api if possible | 14:38 |
mriedem | that doesn't probably have anything to do with the underlying framework though | 14:38 |
*** lucas___ has quit IRC | 14:38 | |
*** shaohe_feng has quit IRC | 14:39 | |
cdent | my feeling is that the way that nova does stuff is way way over abstracted | 14:39 |
*** burgerk has joined #openstack-nova | 14:39 | |
*** shaohe_feng has joined #openstack-nova | 14:39 | |
dansmith | mriedem: I've been staying out of this conversation, but I agree with everything you just said, FYI | 14:40 |
mriedem | dansmith: whew :) | 14:40 |
mriedem | cdent: so, examples in the ML on this would be helpful | 14:40 |
mriedem | if they aren't already | 14:40 |
sdague | mriedem: right, schema validation and microversions basically have to be custom built anyway | 14:40 |
*** amotoki has joined #openstack-nova | 14:40 | |
mriedem | like, explicitly saying, today we have x which sucks for $reasons, and with this other new thing, we could do it this other way, which i think is better and this is why | 14:41 |
*** links has quit IRC | 14:41 | |
mriedem | otherwise i'm really just an uninformed consumer with competing sales people :) | 14:41 |
*** josecastroleon has joined #openstack-nova | 14:41 | |
*** pcaruana has quit IRC | 14:41 | |
mriedem | for a product i don't really want to buy, but have to | 14:41 |
mriedem | like insurance | 14:41 |
mriedem | :( | 14:41 |
cdent | mriedem: I was rather hoping that the code, which is dead simple, would operate as working example of the alternate | 14:42 |
vladikr | danpb, I think the problem started here: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3330 , but I think the more problematic issue was somewhere in the image.cache.. I'll try to sort it out w/o the device_attach | 14:42 |
sdague | cdent: I think the problem is you thought you were starting a conversation on selector vs. routes, but then it got hijacked into wsgi frameworks. | 14:42 |
mriedem | cdent: so what is way way over abstracted today? | 14:42 |
*** claudiub has joined #openstack-nova | 14:43 | |
mriedem | cdent: keeping in mind the extension stuff is going away | 14:43 |
cdent | one mo | 14:43 |
*** baoli has quit IRC | 14:43 | |
danpb | vladikr: if we can't easily address it, my preference would be to create an initial config drive, and then recreate it later with updated data | 14:44 |
danpb | vladikr: and then figure out how to remove the duplicated create later on once its merged | 14:44 |
vladikr | danpb, sounds good, thanks | 14:44 |
danpb | if you do that though, make sure to put a TODO(xxx) item in the code to remind us | 14:45 |
vladikr | danpb, sure. I'll try to sort it without duplicating it first | 14:45 |
bauzas | cdent: mriedem: I think I expressed the same concerns here, I want to have our placement API code as close as possible as what we have in Nova | 14:46 |
bauzas | cdent: because don't forget we have a long tail of one-off contributors | 14:46 |
mriedem | bauzas: however, | 14:46 |
mriedem | one off contributors probably get super lost in nova-api | 14:46 |
mriedem | hell i get lost | 14:46 |
mriedem | i ask sdague stuff all the time | 14:46 |
cdent | bauzas: that's the argumet that works in my favor. The nova api is completely inscrutable (more on that in a moment) so one off people will never understand it | 14:47 |
mriedem | so i appreciate wanting to not replicate a bunch of the existing garbage | 14:47 |
bauzas | cdent: that's where I agree to disagree with you :) | 14:47 |
cdent | bauzas: I'm sorry, but you're just wrong about this. | 14:47 |
cdent | I want to be polite about this. | 14:47 |
cdent | I really do | 14:47 |
mriedem | but if the major confusion is in the things that we're removing, then it's less of an issue | 14:47 |
*** pbandark has quit IRC | 14:47 | |
bauzas | cdent: because I think that if something is bad, then we should fix that rather than getting rid of it | 14:48 |
*** baoli has joined #openstack-nova | 14:48 | |
*** kolesovdv1 has joined #openstack-nova | 14:48 | |
bauzas | cdent: here, the problem would be that we would use a totally different framework and leave nova in a bad shape, if you consider nova being in a bad shape | 14:48 |
cdent | bauzas: If the placement api wasn't in its own domain, then I would agree that using nova-api's structures and fixing them might be a good idea | 14:48 |
cdent | but since we are not in the same domain, it simply isn't worth repeating the errors of history | 14:49 |
mriedem | cdent: so a question on something you said in the thread | 14:49 |
*** foexle has quit IRC | 14:49 | |
mriedem | " I really think Routes has very bad taste when it comes to both WSGI and Python, probably at least in part because of its inspiration in Rails. " | 14:49 |
*** shaohe_feng has quit IRC | 14:49 | |
*** kolesovdv has quit IRC | 14:49 | |
mriedem | cdent: are you referring to the internals of Routes? | 14:49 |
mriedem | or it's API or? | 14:49 |
cdent | mriedem: both | 14:49 |
bauzas | cdent: which domain are you referring to ? | 14:49 |
*** shaohe_feng has joined #openstack-nova | 14:50 | |
cdent | bauzas: the placement is not the nova service, it is its own service, so it doesn't need to be like nova | 14:50 |
bauzas | cdent: oh the fact that we run a different service | 14:50 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [2/3] https://review.openstack.org/331632 | 14:50 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [3/3] https://review.openstack.org/331633 | 14:50 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [1/3] https://review.openstack.org/280301 | 14:50 |
*** jlanoux has quit IRC | 14:50 | |
bauzas | cdent: well, I tend to be very careful of this, because that's still the same nova tree, right? | 14:50 |
sdague | so, lets separate the concerns. Because I think there are a few of them. | 14:50 |
cdent | yes, that's very true sdague | 14:51 |
*** GB21 has joined #openstack-nova | 14:51 | |
sdague | 1) what structure is easiest to review given the current review team, so we can get a thing landed that's working | 14:51 |
sdague | 2) what things do we really want to avoid to make long term maintenance of this new stack easier | 14:52 |
*** baoli has quit IRC | 14:52 | |
*** jaosorior has quit IRC | 14:53 | |
*** kevin-run has quit IRC | 14:53 | |
cdent | assuming you're doine sdague I think there is a 3 | 14:54 |
bauzas | I'm also very concerned by using dependencies that would pull implicit API contracts | 14:54 |
sdague | cdent: go for it | 14:54 |
cdent | 3) what structure allows us to make the most effective and correct use of HTTP and WSGI in a readable and maintainable fashion | 14:54 |
bauzas | (API in terms of interfaces, not REST stuff)Ã | 14:54 |
*** baoli has joined #openstack-nova | 14:54 | |
bauzas | 3) is subjective | 14:54 |
sdague | cdent readable and maintainable by who? | 14:54 |
*** kolesovdv1 has quit IRC | 14:55 | |
*** kolesovdv has joined #openstack-nova | 14:55 | |
cdent | sdague: I would argue: by people who are not infected by Nova | 14:55 |
sdague | everyone is sure their own code is the most readable in the world. But code is not readable or maintainable in a vacuum. | 14:55 |
bauzas | I'm about to ragequit when I hear "infected", but I guess it's ironic | 14:55 |
*** pilgrimstack has joined #openstack-nova | 14:55 | |
cdent | sure, but there's different levels of experience and context | 14:55 |
dansmith | bauzas: same | 14:55 |
*** rlrossit has quit IRC | 14:55 | |
sdague | cdent: the whole placement service is going to be "injected" by nova people for a long time | 14:56 |
cdent | but nova's api is really werd | 14:56 |
cdent | weird | 14:56 |
*** kevin-run has joined #openstack-nova | 14:56 | |
sdague | even once it's a separate thing, I'd expect that 50% or more of the core team is going to be nova cores for a bunch of cycles | 14:56 |
cdent | I think it is fair to be critical of Nova's style. It works. That's great. It is a _huge_ accomplishment and people should be very proud of it. | 14:56 |
cdent | But it is not authoritative. | 14:56 |
*** rrecio has joined #openstack-nova | 14:57 | |
*** sacharya has joined #openstack-nova | 14:57 | |
bauzas | infected is not a synonym to authoritative to me, but whatever | 14:57 |
mriedem | cdent: can we get back to examples? | 14:57 |
*** armax has joined #openstack-nova | 14:57 | |
mriedem | nova's api is really weird, can you give an example? we realize extensions suck | 14:58 |
sdague | mriedem: right, which is why we are actively deleting that code :) | 14:58 |
*** rrecio_ has joined #openstack-nova | 14:58 | |
cdent | However, as I've tried to make very very clear: These are my opinions and I'm extremely happy to do otherwise. I keep getting drawn back into this conversation because each time it appears to be concluded someone brings something back up or makes an assertion of "let's do this". I really just want to get it done. | 14:58 |
mriedem | i would just like to have an informed opinion, | 14:59 |
*** efried has joined #openstack-nova | 14:59 | |
mriedem | i agree that when we do new things, it's a good time for evaluating doing those in different/better ways | 14:59 |
*** david-lyle_ is now known as david-lyle | 14:59 | |
*** shaohe_feng has quit IRC | 14:59 | |
mriedem | but as nova core i need to be comfortable with those and the tradeoffs | 14:59 |
sdague | cdent: right, I think the frustration point that many us are having is that you mostly just keep saying "this all sucks", and mriedem is asking for specific examples of "it would be better if we did this bit like X instead" | 14:59 |
sdague | mriedem: right, exactly | 15:00 |
*** shaohe_feng has joined #openstack-nova | 15:00 | |
cdent | Right, I think I've done that in the code and the comments on the code, but here are the two main things: | 15:00 |
*** rlrossit_ has joined #openstack-nova | 15:00 | |
sdague | one specific bit was Routes over selector, but the judgement call there was that for all it's warts Routes is a known quantity in the openstack community at large, and it's not clear that selector is even maintained any more (3 years since last commit, marked alpha in pypi) | 15:01 |
*** ijw has joined #openstack-nova | 15:01 | |
cdent | 1) It is extremely difficult, to me, to look at the nova api code and find how urls are associated with code and trace those pathways. The reason for using selector (or a declarative mode of using Routes) is to make that _explicit_: this path goes to this code (this is also a reason for not wanting to use pecan). | 15:01 |
*** rrecio has quit IRC | 15:01 | |
*** mvk_ has quit IRC | 15:02 | |
*** tblakeslee has quit IRC | 15:02 | |
mriedem | i guess i've just always looked up the alias, | 15:02 |
mriedem | but that's what i'm used to | 15:02 |
sdague | right, and a lot of #1 is due to the extensions framework, once we tear that out, we can do better there | 15:02 |
cdent | (how do you even know the alias?) | 15:02 |
*** dave-mccowan has quit IRC | 15:02 | |
sdague | and I agree with #1 | 15:02 |
* jaypipes reads back... | 15:03 | |
*** markvoelker_ has joined #openstack-nova | 15:03 | |
*** amotoki has quit IRC | 15:03 | |
*** markvoelker has quit IRC | 15:03 | |
*** tblakeslee has joined #openstack-nova | 15:03 | |
sdague | johnthetubaguy and I talked about restructing the API tree to look like the url resource structure once we got all the extension gorp removed | 15:04 |
openstackgerrit | Tom Patzig proposed openstack/python-novaclient: Add option to disable local disks in flavors https://review.openstack.org/300541 | 15:04 |
cdent | 2) The use of classes for handlers, especially with inheritance, makes it challenging to understand the mechanics of single request method on a single request handler is going to behave and where that behavior is happening. Using function-based wsgi apps or or not-inheriting class-based callables which compose with other things for functionality is much more readable. | 15:04 |
bauzas | I think it's a good idea, and I'd love to review that once the extensions go bye | 15:04 |
mriedem | e.g. GET http://127.0.0.1:8774/v2.1/os-networks | 15:04 |
mriedem | but yeah things like os-multiple-create is an example that updates what you can pass in the request for servers | 15:04 |
mriedem | but as sean said we're removing / folding the extensions in | 15:05 |
cdent | If you have both 1 and 2 together the code...simple | 15:05 |
cdent | Both 1 and 2 are derivatives of "wow there's a lot going on that I can't see, that is not explicit" | 15:06 |
*** fragatina has quit IRC | 15:06 | |
mriedem | cdent: for #2 like this? class FloatingIPActionController(wsgi.Controller): | 15:06 |
mriedem | class FloatingIps(extensions.V21APIExtensionBase): | 15:06 |
cdent | pretty much | 15:06 |
cdent | when making a rest api there is very little reason for the code to be much more than some declaration of URL with handlers and for those handlers to be essentially imperative | 15:07 |
cdent | do x, then y, then z, then return a response | 15:07 |
*** aspiers has joined #openstack-nova | 15:08 | |
*** amotoki has joined #openstack-nova | 15:08 | |
cdent | mriedem: the method at line 69 here https://review.openstack.org/#/c/329151/11/nova/api/openstack/placement/handlers/resource_provider.py | 15:08 |
*** kfarr has quit IRC | 15:09 | |
cdent | the dispatch dispatches to that method, that method run, that's the whole thing | 15:09 |
*** shaohe_feng has quit IRC | 15:09 | |
aspiers | hi all, is use of the mark-host-down API enough to prevent nova-scheduler assigning new VMs to that compute node? | 15:09 |
mriedem | cdent: so e.g. what's handling the 404 here? https://review.openstack.org/#/c/329151/11/nova/api/openstack/placement/handlers/resource_provider.py@73 | 15:09 |
*** burt has joined #openstack-nova | 15:10 | |
mriedem | or is that just missing from the patch? | 15:10 |
*** shaohe_feng has joined #openstack-nova | 15:10 | |
cdent | the dispatcher is, which I don't like very much | 15:10 |
cdent | it is here: | 15:10 |
*** josecastroleon has quit IRC | 15:11 | |
mriedem | fwiw, i do like the @extensions.expected_errors((400, 403, 404)) we have today | 15:12 |
cdent | line 81 at https://review.openstack.org/#/c/329151/11/nova/api/openstack/placement/handler.py | 15:12 |
bauzas | aspiers: yup, if you make sure you still have the ComputeFilter running | 15:12 |
mriedem | so we can't just return whatever errors we want when we want | 15:12 |
kashyap | haypo: When you get a sec: [stable/liberty] https://review.openstack.org/#/c/333403/ - Bump min 'oslo.concurrency' version to 2.6.1 | 15:12 |
*** smatzek has joined #openstack-nova | 15:12 | |
*** josecastroleon has joined #openstack-nova | 15:12 | |
aspiers | bauzas: thanks! | 15:12 |
*** speller has quit IRC | 15:12 | |
*** luis5tb has joined #openstack-nova | 15:12 | |
cdent | mriedem: part of the complexity there is because I was instructed (based on conversation at the bristol midcycle) to avoid using nova bits as much as possible, so I'm trying to limit the surface area where nova.exception is used and imported | 15:13 |
*** rlrossit_ has quit IRC | 15:13 | |
eantyshev | hemna: May I ask you to remove -1 on https://review.openstack.org/190843 as Virtuozzo Storage CI reported success. We have unusually high load in these days so the delay | 15:14 |
*** bkopilov has joined #openstack-nova | 15:14 | |
*** sdake_ has joined #openstack-nova | 15:14 | |
bauzas | cdent: I think I only heard in Bristol about the placement API using a different service type, that's it, not us preventing to use nova code | 15:14 |
bauzas | but I leave others confirm or infirm | 15:15 |
*** pilgrimstack has quit IRC | 15:15 | |
cdent | bauzas: I don't know what mid-cycle you went to, but the one I went to, I came out think that it was important to minimize nova dependencies. | 15:15 |
*** sheel has quit IRC | 15:15 | |
doffm | Everyone remembers a unique bristol midcycle. | 15:15 |
*** baoli has quit IRC | 15:15 | |
doffm | We should record the next one. | 15:15 |
*** sdake has quit IRC | 15:16 | |
mriedem | i only remember clotted cream | 15:16 |
cdent | of course I reckon that's a point 4) in sdague's list of separated concerns that we should try not to let overlap with the others: What is the eventual disposition of the placement API. | 15:16 |
alaski | can we just all agree that go is the answer | 15:16 |
cdent | alaski++ | 15:16 |
bauzas | nope, ciao rather | 15:16 |
bauzas | things would be sooooo easy | 15:17 |
doffm | alaski: Sorry we missed the meeting yesterday. There was a work outing. (Baseball game) | 15:17 |
doffm | We didn't get back in time. | 15:17 |
alaski | no worries, it was quite a short meeting | 15:17 |
doffm | Go is the answer to everything these days. We will all be out of a job soon without 5+ years of go on our resume. | 15:17 |
alaski | tl;dr need moar reviews | 15:17 |
kashyap | mriedem: Are we going to "reap the whirlwind" or something with that bump for stable/liberty...? | 15:18 |
alaski | also dansmith got a virtual cookie for his review work | 15:18 |
doffm | Heh. :) | 15:18 |
alaski | mriedem was eligible as well, but wasn't there to receive it | 15:18 |
aspiers | bauzas: and if nova-compute is restarted on a host which was previously forced down, is it automatically considered up? | 15:18 |
doffm | He probably got a baseball hotdog instead. | 15:18 |
*** amotoki has quit IRC | 15:18 | |
*** hshiina has joined #openstack-nova | 15:18 | |
cdent | bauzas, mriedem, sdague: Maybe we should address these concerns one at time. One that is easy to address is "adding selector doesn't make people happy". I can remove that, with a smallish amount of work. Then people can look and think again? Or it better to hash the bigger picture out? | 15:19 |
sdague | cdent: so, if your stack is redone with Routes instead of selector, it's not a huge change to the pattern, right, just the top level application has a different call | 15:19 |
cdent | that's correct sdague | 15:19 |
bauzas | aspiers: yup, the down state is persisted in DB | 15:19 |
*** dave-mccowan has joined #openstack-nova | 15:19 | |
sdague | honestly, other than just that paste detail, the stack was looking reasonable to me | 15:19 |
bauzas | aspiers: oooh oops | 15:19 |
bauzas | aspiers: the contrary rather | 15:20 |
aspiers | bauzas: right :) | 15:20 |
*** shaohe_feng has quit IRC | 15:20 | |
*** sdake_ has quit IRC | 15:20 | |
*** haypo_ has joined #openstack-nova | 15:20 | |
bauzas | aspiers: ie. something which was marked as down needs to be marked as up manually | 15:20 |
aspiers | so it requires a reversal of the force_down? | 15:20 |
aspiers | yup, thought so. thanks a lot! | 15:20 |
mriedem | kashyap: i'd like to see what the changelog is between 2.3.0 and 2.6.1 | 15:20 |
*** shaohe_feng has joined #openstack-nova | 15:20 | |
mriedem | but that's a pretty large min bump | 15:20 |
mriedem | s/but/because/ | 15:20 |
mriedem | alaski: damn | 15:20 |
bauzas | aspiers: sorry, I did read too fast, yeah a restart doesn't change the compute status | 15:20 |
mriedem | alaski: i got ice cream last night so that makes up for the cookie | 15:20 |
*** jlanoux has joined #openstack-nova | 15:20 | |
cdent | brb | 15:20 |
*** haypo has left #openstack-nova | 15:21 | |
mriedem | ice cream and tacos | 15:21 |
aspiers | bauzas: in that sense it's kind of similar to service-disable, right? | 15:21 |
melwitt | alaski: tbh I was a bit confused by the -1 jenkins status of some patches up for review. is that expected? | 15:21 |
sdague | cdent: the other question is the final exception translation layer. That's been a pattern in Nova so that we can keep detailed errors right up until the surface for the user. Which has turned out to be useful because we can do recovery in some situations. | 15:21 |
*** haypo_ is now known as haypo | 15:21 | |
*** haypo has quit IRC | 15:21 | |
*** haypo has joined #openstack-nova | 15:21 | |
bauzas | aspiers: I'm not 100% sure but yes | 15:21 |
aspiers | bauzas: except I guess it's intended more for use by automatic workflows based on health monitoring, whereas service-disable is more for manual use by cloud operators? | 15:21 |
alaski | mriedem: that sounds more fulfilling than the cookie | 15:21 |
bauzas | aspiers: zactly | 15:21 |
*** tbachman has quit IRC | 15:22 | |
alaski | melwitt: on which series? if it's on my deleting things series then yeah. I keep thinking I have it sorted out and then jenkins/zuul calls me a liar | 15:22 |
mriedem | sdague: are you talking about https://review.openstack.org/#/c/329151/11/nova/api/openstack/placement/handler.py@81 ? | 15:22 |
kaisers | Hi Novas! Quick q: Is there a established way to detect if nova is running on a systemd host? Perhaps via an oslo lib? | 15:22 |
sdague | mriedem: yeh | 15:22 |
*** rtheis_ has joined #openstack-nova | 15:23 | |
*** baoli has joined #openstack-nova | 15:23 | |
sdague | mriedem: but more - https://review.openstack.org/#/c/329151/11/nova/api/openstack/placement/handlers/resource_provider.py@78 | 15:23 |
*** belmoreira has quit IRC | 15:23 | |
melwitt | alaski: yeah, I guess so. I was just going through the list on the priorities etherpad | 15:23 |
*** baoli has quit IRC | 15:23 | |
*** penick has quit IRC | 15:24 | |
cdent | sdague: somewhere there is a commit message in that stack that says "we're going to need more dedicated exceptions" | 15:24 |
openstackgerrit | Andrey Volkov proposed openstack/nova: Fix host mapping saving https://review.openstack.org/332017 | 15:24 |
kashyap | mriedem: I'll try to generate tha, and post it there. | 15:25 |
*** baoli has joined #openstack-nova | 15:25 | |
*** rtheis has quit IRC | 15:25 | |
cdent | what you're looking at now is: get something out relatively quickly so people can't start panicking about it early | 15:25 |
kashyap | s/tha/that/ | 15:25 |
*** rtheis has joined #openstack-nova | 15:25 | |
mriedem | cdent: which i think is fair, | 15:25 |
mriedem | we can fine tooth comb it in review | 15:25 |
sdague | cdent: yep, that's fair, just providing the feedback on the patterns we probably want in there | 15:25 |
cdent | yeah, totes | 15:26 |
sdague | you wanted feedback, right. :) | 15:26 |
alaski | melwitt: I'll update that list for my stuff. I started a new series at https://review.openstack.org/#/c/332478/1 and am working on porting things over from the old series. I had to step back and do some foundational work to simplify upgrade handling. | 15:26 |
openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips-bulk https://review.openstack.org/333075 | 15:26 |
*** ijw has quit IRC | 15:27 | |
*** Guest35964 has joined #openstack-nova | 15:27 | |
*** rtheis__ has joined #openstack-nova | 15:27 | |
*** rtheis_ has quit IRC | 15:27 | |
*** JoseMello has joined #openstack-nova | 15:28 | |
cdent | sdague: yes, very much | 15:28 |
*** kevin-run has quit IRC | 15:28 | |
cdent | but I'm still trying to figure out if it is premature or not | 15:28 |
mriedem | ok, i was at least able to follow https://review.openstack.org/#/c/329149/11/etc/nova/api-paste.ini@7 | 15:29 |
cdent | If people are going to reject the structure, then the alternate structure will be different | 15:29 |
cdent | mriedem: irony, I think that part should go away :) | 15:29 |
melwitt | alaski: cool, thanks | 15:29 |
sdague | johnthetubaguy: on https://review.openstack.org/#/c/333014/1/nova/api/openstack/compute/helpers.py - the attribute is only allowed on rebuild in the jsonschema | 15:29 |
mriedem | cdent: yeah i know, was just jumping back there from the back and forth between you and sean the later change | 15:29 |
*** shaohe_feng has quit IRC | 15:30 | |
*** markus_z has quit IRC | 15:30 | |
*** rtheis has quit IRC | 15:30 | |
doffm | jaypipes, bauzas: I'm going to move the aggregate_db tests out of the unit tests in to a functional test, and mock out database access in the unit tests. | 15:30 |
doffm | Turn them in to something more like the current nova unit tests. | 15:30 |
*** shaohe_feng has joined #openstack-nova | 15:31 | |
sdague | mriedem: right, we should definitely not have that line 7, that causes issues in our application defined paths being in paste.ini | 15:31 |
doffm | As you both have the same opinion about them being sorta 'wrong'. :) | 15:31 |
sdague | but, that's solvable | 15:31 |
*** sdake has joined #openstack-nova | 15:31 | |
*** Guest35964 has quit IRC | 15:32 | |
bauzas | doffm: that sounds a good idea | 15:32 |
*** salv-orlando has quit IRC | 15:33 | |
jaypipes | doffm: rock on. | 15:33 |
*** lucas___ has joined #openstack-nova | 15:33 | |
johnthetubaguy | sdague: true, I was just thinking if we added this now, we might put that in here directly. | 15:33 |
sdague | johnthetubaguy: I can restack patches if you like, it is addressed in the *next* patch though | 15:33 |
kashyap | mriedem: I posted the condensed outputs of: `git log --oneline 2.3.0 2.6.1 | wc -l` and `git diff 2.3.0 2.6.1 --stat-width=72` | 15:33 |
*** sdake_ has joined #openstack-nova | 15:34 | |
*** ljxiash has quit IRC | 15:34 | |
*** rtheis has joined #openstack-nova | 15:34 | |
kashyap | Missed to subtract the merge commits; added that too. | 15:34 |
*** luca_____ has joined #openstack-nova | 15:35 | |
*** rtheis__ has quit IRC | 15:35 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 15:36 |
mriedem | sdague: because it makes the placement api app just resource_providers right? which would be weird when you want to do things on inventories and allocations | 15:36 |
*** lucas___ has quit IRC | 15:36 | |
*** sdake has quit IRC | 15:36 | |
mriedem | kashyap: i'd just do git log --oneline --no-merges 2.3.0..2.6.1 | 15:36 |
mriedem | seems easier | 15:37 |
cdent | mriedem: all that section is doing is setting a prefix on everything that is underneath it | 15:37 |
kashyap | mriedem: Yep, that's what I did, minus the list of commits | 15:37 |
cdent | because it was easier to declare noauth for / | 15:37 |
*** READ10 has joined #openstack-nova | 15:37 | |
*** luca_____ has quit IRC | 15:37 | |
cdent | by having a different wsgi handler at / from /resource_provider | 15:37 |
*** ayoung has joined #openstack-nova | 15:37 | |
kashyap | mriedem: Would you want me to enumerate all the 266 changes (without merge commits) there? | 15:37 |
cdent | because / should be a jsonhome application that anything can request, without auth | 15:37 |
mriedem | kashyap: yikes | 15:38 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance rescue mode support https://review.openstack.org/217679 | 15:38 |
kashyap | mriedem: Guessed you'd say that... | 15:38 |
*** ssurana has joined #openstack-nova | 15:38 | |
sdague | mriedem: the resource names should stay in the app, not in the paste config | 15:39 |
mriedem | kashyap: you're missing a bump in upper-constraints to test with 2.6.1 | 15:39 |
sdague | because in the paste config means they are effective immutable and exist outside of microversion changing resource names | 15:39 |
pkholkin | alaski, mriedem, sdague: hi folks! please start looking this when you have time https://review.openstack.org/#/c/284868, it seems to be almost done (except reviews), please be strict to tests :) thanks! | 15:39 |
mriedem | kashyap: which effectively means the next liberty release requires at a minimum the upper-constraint for all of liberty up to this point | 15:40 |
mriedem | which scares me | 15:40 |
sdague | cdent: where do you plan to expose min / max ver? That should be on /, right? | 15:40 |
kashyap | mriedem: Ah, I should temporarily do that, but later on it'll (the upper-constraints thing) be automatically handled. | 15:40 |
*** shaohe_feng has quit IRC | 15:40 | |
mriedem | pkholkin: yeah, there is similar review for paging on hypervisors that i've been reviewing this week | 15:40 |
*** ctrath has quit IRC | 15:40 | |
mriedem | kashyap: no it wont | 15:40 |
mriedem | kashyap: we disable the u-c update bot in stable branches | 15:41 |
kashyap | Gotcha | 15:41 |
mriedem | b/c it f*s everything up | 15:41 |
kashyap | Got it. | 15:41 |
cdent | sdague: yeah, right now the stack doesn't reflect that | 15:41 |
mriedem | brb | 15:41 |
*** shaohe_feng has joined #openstack-nova | 15:41 | |
*** josecastroleon has quit IRC | 15:42 | |
*** zenoway has quit IRC | 15:42 | |
*** rtheis_ has joined #openstack-nova | 15:42 | |
*** zenoway has joined #openstack-nova | 15:42 | |
kashyap | upper-constraints as of now is at 2.6.0. Interesting... | 15:43 |
*** rtheis has quit IRC | 15:43 | |
pkholkin | mriedem: yes, I found it too, will look | 15:44 |
*** danpb has quit IRC | 15:44 | |
mriedem | kashyap: yeah that's what scares me | 15:45 |
mriedem | about bumping the min to 2.6.1 | 15:45 |
mriedem | anyway bump u-c in the change and see if anyting blows up | 15:45 |
*** ctrath has joined #openstack-nova | 15:45 | |
dansmith | johnthetubaguy: excellent job on splitting out individual improvements from your neutron set, really | 15:45 |
kashyap | mriedem: I'll put the upper-constraints.txt change too, and see what breaks with Jenkins? | 15:45 |
kashyap | Yeah | 15:45 |
*** rtheis has joined #openstack-nova | 15:45 | |
cdent | sdague, mriedem: Do we have any immediate strategy or plan? I'm not actually really working this week, just trying to check back in to make sure the process keeps moving. | 15:46 |
mriedem | i would not be surprised if there is a backward incompat change in oslo.concurrency between 2.3 and 2.6 | 15:46 |
*** dmk0202 has quit IRC | 15:46 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/333181 | 15:46 |
kashyap | mriedem: Please don't be so enthusiatic to predict the doom | 15:46 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: Improve nova.rpc conf options documentation https://review.openstack.org/302678 | 15:47 |
*** zenoway has quit IRC | 15:47 | |
kashyap | :-) | 15:47 |
mriedem | cdent: so it sounds like a no on selector | 15:47 |
openstackgerrit | Balazs Gibizer proposed openstack/nova: doc: Add guidline about notification payload https://review.openstack.org/308393 | 15:47 |
*** rtheis_ has quit IRC | 15:47 | |
*** sridharg has quit IRC | 15:47 | |
cdent | ✔ | 15:47 |
mriedem | cdent: i've only briefly skimmed some of the changes | 15:47 |
mriedem | and noted some of my concerns in there | 15:47 |
sdague | cdent: 1) redo the stack with routes. 2) add intermediate exceptions 3) address the paste issue 4) make sure we have a root version doc | 15:47 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/333451 | 15:48 |
sdague | that's about the feedback I've got so far | 15:48 |
cdent | The remaining question is how does this deal with or relate to dhellmann's concerns about building yet more stuff with custom code? | 15:48 |
*** armax has quit IRC | 15:48 | |
openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Add option for running nova independently from hypervisor https://review.openstack.org/332216 | 15:49 |
openstackgerrit | Julian proposed openstack/nova: WIP Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2) https://review.openstack.org/325435 | 15:49 |
openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Perform disk operations in dom0 https://review.openstack.org/331254 | 15:49 |
openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Stream config drive to XAPI https://review.openstack.org/332215 | 15:49 |
tpatzig | sdague: johnthetubaguy: i think i addressed all of your comments. can you please have another look https://review.openstack.org/#/c/280301 | 15:49 |
*** dillaman has quit IRC | 15:49 | |
tpatzig | just the tests need to pass | 15:49 |
sdague | I honestly think we basically need to decide if we want an API stack merged this cycle, or we want to switch out the API stack to something like flask. Because I just can't really imagine enough folks skilling up on a new API framework at this point in time to get us merged. | 15:49 |
*** itamarl has quit IRC | 15:49 | |
sdague | that's back to original concern #1 | 15:49 |
* cdent nods | 15:50 | |
*** shaohe_feng has quit IRC | 15:50 | |
*** rtheis_ has joined #openstack-nova | 15:51 | |
*** shaohe_feng has joined #openstack-nova | 15:51 | |
mriedem | so far from what i've looked at, which doesn't include the microversion change in detail, i can say i don't hate this | 15:51 |
mriedem | if that helps :) | 15:51 |
mriedem | i don't know how different it would look with flask | 15:52 |
mriedem | w/o being through this first series in detail first | 15:52 |
*** rtheis has quit IRC | 15:53 | |
*** luis5tb has quit IRC | 15:54 | |
*** rtheis has joined #openstack-nova | 15:56 | |
*** andreas_s has quit IRC | 15:56 | |
mriedem | sdague: re: what you just said, "I just can't really imagine enough folks skilling up on a new API framework at this point in time to get us merged." - new framework being what cdent has today and/or flask, right? | 15:57 |
*** ijw has joined #openstack-nova | 15:57 | |
cdent | mriedem, sdague: Part of what has happened here is that in a semi-greenfield context it was faster for me to just do raw wsgi than to figure out all the ins and outs of the the nova-way end-to-end. That is: raw wsgi didn't require skilling up. | 15:58 |
*** rtheis_ has quit IRC | 15:58 | |
mriedem | cdent: how many other resources does the placement api need to support right now? you have resource_providers, what else? allocations? | 15:58 |
*** diga has joined #openstack-nova | 15:58 | |
*** josecastroleon has joined #openstack-nova | 15:58 | |
cdent | mriedem: the only thing missing from that current stack is a handler for /resource_providers/{id}/aggregates | 15:59 |
*** anush__ has joined #openstack-nova | 16:00 | |
cdent | there's nothing planned, immediately, for beyond the /resource_providers base url except for the version info at / | 16:00 |
cdent | longer term there is talk of dynamic resource classes | 16:00 |
*** shaohe_feng has quit IRC | 16:01 | |
*** rtheis_ has joined #openstack-nova | 16:01 | |
*** GB21 has quit IRC | 16:02 | |
*** shaohe_feng has joined #openstack-nova | 16:02 | |
*** mpavone has quit IRC | 16:03 | |
*** markvoelker has joined #openstack-nova | 16:03 | |
mriedem | sdague: so the flip side is if we want the placement api merged in newton and cdent is the one writing it and he's not familiar/comfortable with what nova does today, that will also take longer... | 16:03 |
*** diogogmt has joined #openstack-nova | 16:03 | |
*** rtheis has quit IRC | 16:03 | |
*** markvoelker_ has quit IRC | 16:04 | |
mriedem | cdent: how much time is "1) redo the stack with routes. 2) add intermediate exceptions 3) address the paste issue 4) make sure we have a root version doc" going to take you? | 16:04 |
*** anush__ has quit IRC | 16:04 | |
*** READ10 has quit IRC | 16:04 | |
mriedem | i think 1, 3 and 4 being most important right now | 16:04 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Implement stable disk device instance rescues https://review.openstack.org/270289 | 16:05 |
*** pilgrimstack has joined #openstack-nova | 16:05 | |
cdent | mriedem: not really all that long, the complexity is I'm trying to be out of action for the next week or so (in the states for wedding and family visit) | 16:05 |
openstackgerrit | Andrew Laski proposed openstack/nova: Add policy sample generation https://review.openstack.org/332473 | 16:05 |
*** rtheis has joined #openstack-nova | 16:06 | |
*** rtheis_ has quit IRC | 16:08 | |
pkholkin | mriedem: I leave my comments to hypervisors pagination patch | 16:09 |
pkholkin | and also some to my own after that :) | 16:10 |
dansmith | jaypipes: on this serialization thing and the later one: https://review.openstack.org/#/c/318884/11 | 16:10 |
*** rtheis_ has joined #openstack-nova | 16:10 | |
pkholkin | anyway please look my too | 16:10 |
dansmith | jaypipes: they are using the object serialization and they're mirroring the crap we already have | 16:10 |
dansmith | jaypipes: I'm going to follow up and remove that to_json() junk after this, is that okay with you? | 16:11 |
*** shaohe_feng has quit IRC | 16:11 | |
mriedem | artom: vladikr: is there a tempest test lined up for testing https://review.openstack.org/#/c/304510/ ? | 16:11 |
*** sacharya has quit IRC | 16:12 | |
*** shaohe_feng has joined #openstack-nova | 16:12 | |
*** sacharya_ has joined #openstack-nova | 16:12 | |
*** rtheis has quit IRC | 16:12 | |
*** READ10 has joined #openstack-nova | 16:12 | |
*** muawiakhan has joined #openstack-nova | 16:12 | |
vladikr | mriedem, iirc artom wrote one, but I can't seems to find it.. | 16:12 |
*** rlrossit has joined #openstack-nova | 16:12 | |
mriedem | https://review.openstack.org/#/c/305120/ | 16:12 |
openstackgerrit | STEW TY proposed openstack/nova: Complete verification for os-floating-ips https://review.openstack.org/329704 | 16:12 |
mriedem | ? | 16:12 |
doffm | rlrossit: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L47 | 16:12 |
vladikr | mriedem, seems like it, but not updated | 16:13 |
*** p01nt3r75 has quit IRC | 16:14 | |
sneti | Hi jroll, I've a quick question. I'm looking at the config option for compute manager in https://github.com/openstack/nova/blob/master/nova/conf/service.py#L62-67. | 16:14 |
sneti | I see that ironic is using the short term workaround with ClusteredComputeManager. I just want to confirm if ironic will still be using this. If not I can remove the option. | 16:14 |
jroll | sneti: yes, some people still use that | 16:15 |
*** yamahata has joined #openstack-nova | 16:16 | |
sneti | jroll, ok..thanks for confirming..I will not remove the option.. | 16:16 |
jroll | sneti: apparently launchpad is having issues, but this blueprint will allow us to remove that. https://blueprints.launchpad.net/nova/+spec/ironic-multiple-compute-hosts | 16:16 |
*** DaiDV has quit IRC | 16:17 | |
*** tesseract- has quit IRC | 16:17 | |
*** READ10 has quit IRC | 16:18 | |
sneti | jroll, ya..unable to open the link... | 16:18 |
*** dosaboy has quit IRC | 16:18 | |
mriedem | easy +W https://review.openstack.org/#/c/333358/ | 16:19 |
*** raunak has joined #openstack-nova | 16:19 | |
*** raunak has quit IRC | 16:20 | |
*** sdake_ is now known as sdake | 16:21 | |
*** shaohe_feng has quit IRC | 16:21 | |
*** shaohe_feng has joined #openstack-nova | 16:22 | |
*** ctrath has quit IRC | 16:22 | |
*** sheel has joined #openstack-nova | 16:23 | |
*** paul-carlton2 has quit IRC | 16:23 | |
*** amotoki has joined #openstack-nova | 16:24 | |
*** READ10 has joined #openstack-nova | 16:24 | |
*** paul-carlton2 has joined #openstack-nova | 16:24 | |
*** muawiakhan has quit IRC | 16:25 | |
*** raunak has joined #openstack-nova | 16:26 | |
*** ctrath has joined #openstack-nova | 16:26 | |
*** Apoorva has joined #openstack-nova | 16:27 | |
*** gszasz has quit IRC | 16:27 | |
*** pilgrimstack has quit IRC | 16:27 | |
artom | mriedem, yeah, I saw your comments on that test, I'll make sure it passes | 16:27 |
mriedem | artom: i'm not sure that needs to be a scenario test | 16:28 |
mriedem | artom: you should probably ask mtreinish if that should just be a compute api test | 16:28 |
artom | mriedem, so, I don't know much about Tempest, what's the alternative to a scenario test? | 16:28 |
mriedem | compute api test | 16:28 |
mriedem | like we don't really need a special port created for this do we? | 16:29 |
mriedem | i haven't been through the nova series | 16:29 |
artom | mriedem, I believe the reason for the port is to then SSH into the instance and check metadata API and configdrive | 16:29 |
mriedem | there just seemed like a hell of a lot of test setup for booting a server with a network and volumes that are tagged | 16:29 |
*** hshiina has quit IRC | 16:29 | |
mriedem | artom: the test skips if config drive is disabled | 16:29 |
mriedem | is it actually checking the metadata api? | 16:30 |
artom | mriedem, yes | 16:30 |
artom | Line 129 | 16:30 |
artom | https://review.openstack.org/#/c/305120/9/tempest/scenario/test_device_tagging.py@129 | 16:30 |
mriedem | you don't need a port to ssh into the instance though, there are compute api tests that do ssh testing, it's the run_validation flag or something, but maybe the ssh portion should make it a scenario test | 16:30 |
artom | Or 148 if you prefer | 16:30 |
*** josecastroleon has quit IRC | 16:31 | |
*** sahid has quit IRC | 16:31 | |
*** shaohe_feng has quit IRC | 16:31 | |
mriedem | artom: hmm, ok | 16:32 |
mriedem | i think the metadata service is not run in all jobs | 16:33 |
mriedem | only postgres i think | 16:33 |
*** cdent has quit IRC | 16:33 | |
mriedem | http://logs.openstack.org/20/305120/9/check/gate-tempest-dsvm-full/e10d986/logs/screen-n-api.txt.gz?level=TRACE | 16:33 |
mriedem | maybe that's old | 16:33 |
artom | mriedem, yeah, I pushed a fix to the DeviceTaggingFilter for that | 16:34 |
mriedem | looks like ^ feel into the dreaded type(block_device_mapping) trap | 16:34 |
mriedem | ok L) | 16:34 |
mriedem | :) | 16:34 |
mriedem | s/feel/fell/ | 16:34 |
artom | Speaking of which, is bauzas still around? | 16:34 |
*** shaohe_feng has joined #openstack-nova | 16:34 | |
artom | As Scheduler Main Dude, maybe you can take a look at https://review.openstack.org/#/c/331490/? | 16:35 |
*** matrohon has quit IRC | 16:35 | |
artom | mriedem, anyways, much thanks for the reviews on the Tempest test :) I was AFK and sat down quickyl since I saw I'd been pinged, I need to take care of a couple of things around here, then I can address all the feedback properly and make sure the test runs | 16:36 |
*** josecastroleon has joined #openstack-nova | 16:36 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support https://review.openstack.org/318953 | 16:36 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance https://review.openstack.org/316275 | 16:36 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add REST API support for get me a network https://review.openstack.org/316398 | 16:36 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Handle UnableToAutoAllocateNetwork in _build_and_run_instance https://review.openstack.org/333480 | 16:36 |
artom | How critical is it to un-make it a scenario? | 16:36 |
mriedem | artom: sure, np - to be honest i'm amazed there is actually a tempest test up already | 16:36 |
mriedem | artom: it's not, but i'd ask mtreinish | 16:36 |
mriedem | bbiab | 16:36 |
* artom records for the, err, record, that he has amazed mriedem :D | 16:37 | |
dansmith | artom: definitely frame that one | 16:37 |
* dansmith has yet to amaze mriedem | 16:37 | |
*** syjulian has quit IRC | 16:38 | |
*** dosaboy has joined #openstack-nova | 16:39 | |
dansmith | extremely easy +W: https://review.openstack.org/#/c/333480/1 | 16:39 |
*** mriedem1 has joined #openstack-nova | 16:40 | |
*** pilgrimstack has joined #openstack-nova | 16:40 | |
*** shaohe_feng has quit IRC | 16:42 | |
*** jistr|mtg is now known as jistr | 16:42 | |
artom | dansmith, perhaps you have a Beanie Baby collection to show him? ;) | 16:42 |
dansmith | heh | 16:43 |
*** shaohe_feng has joined #openstack-nova | 16:43 | |
*** mriedem has quit IRC | 16:44 | |
*** lucas____ has joined #openstack-nova | 16:44 | |
*** luca_____ has joined #openstack-nova | 16:46 | |
openstackgerrit | Rawlin Peters proposed openstack/os-vif: Don't create extraneous linux bridge/veth pair for VIFOpenVSwitch https://review.openstack.org/333486 | 16:48 |
*** lucas____ has quit IRC | 16:48 | |
*** yassine__ has quit IRC | 16:49 | |
openstackgerrit | Rawlin Peters proposed openstack/os-vif: Don't create extraneous linux bridge/veth pair for VIFOpenVSwitch https://review.openstack.org/333486 | 16:50 |
*** luca_____ has quit IRC | 16:51 | |
*** tblakeslee has quit IRC | 16:51 | |
*** akshai has quit IRC | 16:51 | |
*** shaohe_feng has quit IRC | 16:52 | |
*** shaohe_feng has joined #openstack-nova | 16:52 | |
*** tblakeslee has joined #openstack-nova | 16:53 | |
mtreinish | mriedem1: which test? | 16:53 |
dansmith | mtreinish: https://review.openstack.org/#/c/305120 | 16:53 |
*** mriedem1 is now known as mriedem | 16:54 | |
*** lucas____ has joined #openstack-nova | 16:54 | |
mtreinish | dansmith: oh, mriedem told me having a tempest test for that wasn't worthwhile | 16:55 |
*** pilgrimstack has quit IRC | 16:55 | |
*** penick has joined #openstack-nova | 16:55 | |
*** lucas____ has quit IRC | 16:55 | |
mriedem | mtreinish: different tags | 16:55 |
*** luca_____ has joined #openstack-nova | 16:55 | |
mriedem | not confusing at all | 16:55 |
mtreinish | haha, ok | 16:55 |
*** browne has joined #openstack-nova | 16:56 | |
openstackgerrit | Peter Hamilton proposed openstack/nova: Add certificate signature verification https://review.openstack.org/331823 | 16:56 |
openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for interval_opts https://review.openstack.org/331761 | 16:57 |
*** efried has quit IRC | 16:57 | |
*** akshai has joined #openstack-nova | 16:58 | |
*** fragatina has joined #openstack-nova | 16:58 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 5) https://review.openstack.org/329979 | 16:59 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2) https://review.openstack.org/329978 | 16:59 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 4) https://review.openstack.org/329227 | 16:59 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 1) https://review.openstack.org/330033 | 16:59 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 6) https://review.openstack.org/329228 | 16:59 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 2) https://review.openstack.org/330034 | 16:59 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 3) https://review.openstack.org/328851 | 16:59 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 3) https://review.openstack.org/330035 | 16:59 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 1) https://review.openstack.org/328850 | 16:59 |
*** luca_____ has quit IRC | 16:59 | |
cfriesen | Any chance a few others could look at this bugfix? (not mine, but I've hit it) https://review.openstack.org/#/c/200870/ | 17:00 |
cfriesen | (Fix evaluation of host disk usage by volume-backed instances) | 17:01 |
*** shaohe_feng has quit IRC | 17:02 | |
*** M00nr41n has joined #openstack-nova | 17:02 | |
*** pilgrimstack has joined #openstack-nova | 17:02 | |
*** shaohe_feng has joined #openstack-nova | 17:03 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support https://review.openstack.org/318953 | 17:04 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance https://review.openstack.org/316275 | 17:04 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add REST API support for get me a network https://review.openstack.org/316398 | 17:04 |
mriedem | alex_xu: dansmith: comments are handled ^ | 17:04 |
*** tblakeslee has quit IRC | 17:04 | |
*** josecastroleon has quit IRC | 17:05 | |
*** daemontool_ has quit IRC | 17:06 | |
*** pilgrimstack has quit IRC | 17:06 | |
*** lucas___ has joined #openstack-nova | 17:07 | |
openstackgerrit | Julian proposed openstack/nova: WIP Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2) https://review.openstack.org/325435 | 17:08 |
*** efried has joined #openstack-nova | 17:08 | |
*** paul-carlton2 has quit IRC | 17:08 | |
*** paul-carlton2 has joined #openstack-nova | 17:09 | |
*** ijw has quit IRC | 17:09 | |
*** M00nr41n has quit IRC | 17:09 | |
*** Jeffrey4l has quit IRC | 17:11 | |
BobBall | alaski: I was wondering if you might be up for a little horse trading? I have uploaded the implementation of xenapi-independent-nova and have some time to offer to help free up your time if you would be happy to give it a look :) | 17:11 |
*** TravT has joined #openstack-nova | 17:11 | |
*** shaohe_feng has quit IRC | 17:12 | |
*** shaohe_feng has joined #openstack-nova | 17:13 | |
alaski | BobBall: excellent, if you could just figure out how to properly handle cellsv2 instance deletion and scheduling in a mixed version environment that would free up huge amounts of time :) | 17:14 |
BobBall | pffft... isn't there something harder you need help with? | 17:15 |
*** efried has quit IRC | 17:16 | |
alaski | I could very likely come up with something :) like pagination and sorting instance lists across multiple cells | 17:17 |
BobBall | But, more seriously, if there are patch series that you need to review that I could do some early leg work on then it might save you some time - just point me at 'em otherwise I'll just pick random things | 17:17 |
alaski | but I'm happy to take a look | 17:17 |
alaski | BobBall: there's a list we try to keep up to date at https://etherpad.openstack.org/p/newton-nova-priorities-tracking | 17:17 |
BobBall | yeah - that's what I've been looking at too | 17:17 |
alaski | any reviews there are always helpful | 17:17 |
BobBall | Will do. | 17:17 |
openstackgerrit | Rawlin Peters proposed openstack/os-vif: Ensure the OVS bridge exists when plugging https://review.openstack.org/330818 | 17:17 |
*** kolesovdv has quit IRC | 17:17 | |
*** devananda has quit IRC | 17:17 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: [WIP] Improve help test for wsgi options https://review.openstack.org/333504 | 17:18 |
alaski | the nova-manage ones especially have been up for a while and could use a once over | 17:18 |
alaski | BobBall: do you have a link for the xenapi work? | 17:19 |
*** DaiDV has joined #openstack-nova | 17:20 | |
BobBall | That would have been helpful to give... yes. Three patches, with the base at https://review.openstack.org/#/c/331254/ - if you could look at some point it'd be awesome :) | 17:20 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 17:21 |
*** pushkaru has joined #openstack-nova | 17:21 | |
*** M00nr41n has joined #openstack-nova | 17:22 | |
*** raildo is now known as raildo-afk | 17:22 | |
*** sdake has quit IRC | 17:22 | |
*** tbachman has joined #openstack-nova | 17:23 | |
alaski | BobBall: cool, on the list | 17:23 |
*** shaohe_feng has quit IRC | 17:23 | |
BobBall | Thanks! | 17:23 |
*** shaohe_feng has joined #openstack-nova | 17:23 | |
BobBall | Time to go see the wife. | 17:23 |
*** BobBall is now known as BobBall_AWOL | 17:23 | |
mriedem | BobBall_AWOL: the bp link in that series is busted, it should be xenapi-independent-nova | 17:25 |
*** GB21 has joined #openstack-nova | 17:26 | |
*** tbachman_ has joined #openstack-nova | 17:26 | |
sudipto | mriedem, i responded to your question on this: https://review.openstack.org/#/c/270341/10 - Also the xen CI passed post your recheck, wondering if you are happy about it yet... | 17:27 |
mriedem | sudipto: i'm never happy about anything :) | 17:27 |
mriedem | i have some other stuff in the queue first | 17:27 |
*** ayoung has quit IRC | 17:28 | |
sudipto | mriedem, heh sure :) Just wanted to bring it to your notice that's all :) | 17:28 |
openstackgerrit | Pavel Kholkin proposed openstack/nova: Microversion 2.31 adds keypairs pagination support https://review.openstack.org/284868 | 17:28 |
*** josecastroleon has joined #openstack-nova | 17:28 | |
*** tbachman has quit IRC | 17:29 | |
*** tbachman_ is now known as tbachman | 17:29 | |
*** lpetrut has quit IRC | 17:29 | |
*** ctrath has quit IRC | 17:30 | |
*** DaiDV has quit IRC | 17:30 | |
*** kfarr has joined #openstack-nova | 17:32 | |
*** abalutoiu has quit IRC | 17:32 | |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: add ploop support into qemu-img info https://review.openstack.org/260636 | 17:33 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: fix disk size calculation for VZ container instances https://review.openstack.org/222567 | 17:33 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 17:33 |
*** anush__ has joined #openstack-nova | 17:33 | |
*** shaohe_feng has quit IRC | 17:33 | |
*** ctrath has joined #openstack-nova | 17:33 | |
openstackgerrit | Timofey Durakov proposed openstack/nova: live_migration_cleanup method introduced https://review.openstack.org/302611 | 17:33 |
openstackgerrit | Timofey Durakov proposed openstack/nova: Remove unused migrate_data param from driver cleanup and destroy https://review.openstack.org/280653 | 17:33 |
openstackgerrit | Timofey Durakov proposed openstack/nova: call pre_live_migration from conductor https://review.openstack.org/291160 | 17:33 |
openstackgerrit | Timofey Durakov proposed openstack/nova: centralize checks before live-migration https://review.openstack.org/332124 | 17:33 |
*** shaohe_feng has joined #openstack-nova | 17:34 | |
jaypipes | dansmith: yes, absolutely OK on the to_json() post-patch removal (sorry, had to read back a bunch after some meetings...). | 17:36 |
dansmith | jaypipes: okay cool thanks | 17:36 |
dansmith | vladikr: ^ | 17:36 |
*** gabriel-bezerra has quit IRC | 17:37 | |
*** rcernin has quit IRC | 17:38 | |
*** rlrossit has quit IRC | 17:38 | |
*** gabriel-bezerra has joined #openstack-nova | 17:39 | |
*** armax has joined #openstack-nova | 17:39 | |
*** rlrossit has joined #openstack-nova | 17:41 | |
*** salv-orlando has joined #openstack-nova | 17:42 | |
*** mriedem1 has joined #openstack-nova | 17:43 | |
*** shaohe_feng has quit IRC | 17:43 | |
*** mriedem has quit IRC | 17:44 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 4) https://review.openstack.org/330036 | 17:44 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Replaces 'authorize' in nova-api (part 5) https://review.openstack.org/333514 | 17:44 |
*** shaohe_feng has joined #openstack-nova | 17:44 | |
*** anush__ has quit IRC | 17:44 | |
*** pushkaru has quit IRC | 17:46 | |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: add ploop support into qemu-img info https://review.openstack.org/260636 | 17:46 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: fix disk size calculation for VZ container instances https://review.openstack.org/222567 | 17:46 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 17:46 |
*** anush__ has joined #openstack-nova | 17:46 | |
*** ijw has joined #openstack-nova | 17:47 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Add DeviceTaggingFilter to scheduler https://review.openstack.org/331490 | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: libvirt: attach configdrive after instance XML https://review.openstack.org/303335 | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: objects: adding an update method to virtual_interface https://review.openstack.org/310618 | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: objects: Add devices_metadata to instance object https://review.openstack.org/327920 | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: db: add a device_metadata column to instance_extra https://review.openstack.org/318883 | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: objects: enhance the DeviceMetadataList object https://review.openstack.org/318884 | 17:48 |
artom | vladikr, FYI ^^^ , pushed a new series to address comments from edleafe in the filter patch | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Compute manager device tagging support https://review.openstack.org/264017 | 17:48 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 17:49 |
vladikr | dansmith, jaypipes, thanks | 17:49 |
*** salv-orl_ has joined #openstack-nova | 17:49 | |
*** salv-orlando has quit IRC | 17:49 | |
vladikr | artom, cool | 17:49 |
vladikr | artom, thanks, there is also the tempest test, could you please look at it? | 17:50 |
openstackgerrit | Vladyslav Drok proposed openstack/nova: Update instance node on rebuild only when it is recreate https://review.openstack.org/306010 | 17:50 |
artom | vladikr, yeah, was next on my list :) | 17:50 |
vladikr | artom, :) | 17:51 |
*** anush__ has quit IRC | 17:51 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support https://review.openstack.org/318953 | 17:52 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance https://review.openstack.org/316275 | 17:52 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add REST API support for get me a network https://review.openstack.org/316398 | 17:52 |
auggy | hey folks i'm running into a problem... it looks like api 2.26, specifically the addition of tags as a response parameter, didn't have sufficient test coverage and now that i'm adding it for 2.31, a bunch of things are failing | 17:53 |
*** shaohe_feng has quit IRC | 17:53 | |
claudiub | have you tried turning it off and on again? :) | 17:54 |
auggy | right now i'm trying to add unit test coverage to rebuilds and 2.26 breaks a few of the tests | 17:54 |
auggy | claudiub: lol | 17:54 |
*** shaohe_feng has joined #openstack-nova | 17:54 | |
artom | Maybe 2.42 will magically fix everything... | 17:54 |
auggy | yeah i guess the best thing i can do for now is just comment out my test and make a comment and then point it out when i submit the change | 17:55 |
artom | auggy, I seem to recall Tempest made it work... | 17:55 |
* artom goes digging... | 17:55 | |
*** paul-carlton2 has quit IRC | 17:56 | |
auggy | artom: this is specifically adding a microversion test to the unit tests | 17:56 |
auggy | i'm not sure if tempest does that | 17:56 |
claudiub | alaski: hi. I've added the "soft_authorization" thingy: https://review.openstack.org/#/c/330033/4/nova/context.py | 17:56 |
*** jlanoux has quit IRC | 17:57 | |
claudiub | should probably add a docstring at this point. | 17:57 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: fix disk size calculation for VZ container instances https://review.openstack.org/222567 | 17:57 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 17:57 |
artom | auggy, ah, looks like I misunderstood you then - I though you were saying 2.26 had some bugs that you found by writing tests | 17:57 |
alaski | claudiub: that would be nice, but looks good. I'm just waiting on tests to run before rereviewing | 17:57 |
*** josecastroleon has quit IRC | 17:58 | |
auggy | artom: no i'm specifically subclassing 2.31 in the unit tests for rebuild server and in doing so some tests broke that had nothing to do with anything i changed | 17:58 |
openstackgerrit | Timofey Durakov proposed openstack/nova: live_migration_cleanup method introduced https://review.openstack.org/302611 | 17:58 |
openstackgerrit | Timofey Durakov proposed openstack/nova: Remove unused migrate_data param from driver cleanup and destroy https://review.openstack.org/280653 | 17:58 |
openstackgerrit | Timofey Durakov proposed openstack/nova: call pre_live_migration from conductor https://review.openstack.org/291160 | 17:58 |
openstackgerrit | Timofey Durakov proposed openstack/nova: centralize checks before live-migration https://review.openstack.org/332124 | 17:58 |
openstackgerrit | Timofey Durakov proposed openstack/nova: Rename driver method check_can_live_migrate_destination_cleanup https://review.openstack.org/333519 | 17:58 |
auggy | so when i changed the api version, 2.25 passed but 2.26 failed | 17:58 |
*** dave-mccowan has quit IRC | 17:58 | |
mriedem1 | how much should i hammer on using the new style config option template https://review.openstack.org/#/c/190843/28/nova/conf/libvirt.py | 17:59 |
*** mriedem1 is now known as mriedem | 17:59 | |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass path to Image base class https://review.openstack.org/333271 | 18:00 |
openstackgerrit | Matthew Booth proposed openstack/nova: Add create_from_image and create_from_func imagebackend methods https://review.openstack.org/333244 | 18:00 |
openstackgerrit | Matthew Booth proposed openstack/nova: Rename Backend snapshot and image to by_libvirt_path and by_name https://review.openstack.org/333272 | 18:00 |
openstackgerrit | Matthew Booth proposed openstack/nova: Pass is_block_dev explicitly from create_from_func https://review.openstack.org/333263 | 18:00 |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Improve mocking of Image objects in test_driver https://review.openstack.org/333242 | 18:00 |
openstackgerrit | Matthew Booth proposed openstack/nova: Pass instance object to _create_images_and_backing in test https://review.openstack.org/333243 | 18:00 |
openstackgerrit | Matthew Booth proposed openstack/nova: Rename the import of nova.virt.disk.api from disk to disk_api https://review.openstack.org/333521 | 18:00 |
openstackgerrit | Matthew Booth proposed openstack/nova: Add Backend from_image and from_func https://review.openstack.org/333522 | 18:00 |
*** sfinucan has quit IRC | 18:01 | |
*** shaohe_feng has quit IRC | 18:04 | |
*** GB21 has quit IRC | 18:04 | |
*** BjoernT is now known as Bjoern_zZzZzZzZ | 18:04 | |
*** shaohe_feng has joined #openstack-nova | 18:06 | |
*** akshai has quit IRC | 18:06 | |
*** fragatina has quit IRC | 18:08 | |
jaypipes | cfriesen: reviewed 200870. | 18:08 |
artom | edleafe, looks like we need to bring in an arbitrator - I still think it's better to have other virt drivers explicitly declare their support of tagging in the tests :) | 18:09 |
*** yamahata has quit IRC | 18:10 | |
artom | Granted, it's unlikely, but if the tests reuse the enum from the code, someone could potentially add a virt driver in the latter and nothing would pick that up. | 18:10 |
jaypipes | artom: did you see dansmith's comment on 310618 about needing to rev the version of VirtualInterface object since the patch adds a new @remotable method (save())? | 18:10 |
artom | jaypipes, yeah - I think vladikr is handling that one? | 18:11 |
vladikr | jaypipes, yea, I'm on it | 18:11 |
*** sdake has joined #openstack-nova | 18:11 | |
*** sudipto has quit IRC | 18:11 | |
jaypipes | vladikr: rock on. happy to +2 that once fixed. | 18:11 |
*** sthillma has joined #openstack-nova | 18:12 | |
vladikr | jaypipes, cool, thanks | 18:12 |
artom | jaypipes, so, you're target of convenience since you're currently around, could you weigh in on https://review.openstack.org/#/c/331490/ ? | 18:13 |
*** shaohe_feng has quit IRC | 18:14 | |
*** tblakeslee has joined #openstack-nova | 18:14 | |
mriedem | mnestratov|2: a few small issues to address in https://review.openstack.org/#/c/190843/ | 18:14 |
*** shaohe_feng has joined #openstack-nova | 18:14 | |
mriedem | mnestratov|2: one more thing on the vz storage ci, it looks like it's still running with os-brick from source rather than g-r | 18:17 |
mriedem | http://openstack-3rd-party-storage-ci-logs.virtuozzo.com/43/190843/28/check/dsvm-tempest-kvm/56fe9e7/logs/pip2-freeze.txt.gz | 18:17 |
*** armax has quit IRC | 18:18 | |
*** dave-mccowan has joined #openstack-nova | 18:19 | |
*** zul has quit IRC | 18:20 | |
*** akshai has joined #openstack-nova | 18:24 | |
mnestratov|2 | mriedem: regarding os-brick - yes it runs from source | 18:24 |
*** shaohe_feng has quit IRC | 18:24 | |
*** zul has joined #openstack-nova | 18:24 | |
*** shaohe_feng has joined #openstack-nova | 18:25 | |
mnestratov|2 | mriedem: have global requirements changed already ? | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Add DeviceTaggingFilter to scheduler https://review.openstack.org/331490 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: attach configdrive after instance XML https://review.openstack.org/303335 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: adding an update method to virtual_interface https://review.openstack.org/310618 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: Add devices_metadata to instance object https://review.openstack.org/327920 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: add a device_metadata column to instance_extra https://review.openstack.org/318883 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: enhance the DeviceMetadataList object https://review.openstack.org/318884 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Compute manager device tagging support https://review.openstack.org/264017 | 18:25 |
openstackgerrit | Vladik Romanovsky proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 18:26 |
dansmith | jaypipes: ready for your +W https://review.openstack.org/#/c/310618/22 | 18:26 |
*** Sukhdev has joined #openstack-nova | 18:27 | |
*** salv-orl_ has quit IRC | 18:28 | |
*** M00nr41n has quit IRC | 18:29 | |
jaypipes | dansmith: done. | 18:29 |
dansmith | jaypipes: sweet. sounds like you were close on a few above that one.. are you free to walk up that stack a few? | 18:30 |
* dansmith bows to his BFDness | 18:30 | |
mnestratov|2 | mriedem: and thanks for review, will address asap | 18:31 |
artom | Is that like the BFG from Doom? | 18:31 |
dansmith | artom: the BFD carries a BFG | 18:31 |
artom | LFG BDG w/ BFG @ BRD | 18:33 |
dansmith | um | 18:34 |
* dansmith looks for his decoder ring | 18:34 | |
jaypipes | dansmith: yeah, reviewing the whole series again now. but only if you walk back your BFD comments. | 18:34 |
*** shaohe_feng has quit IRC | 18:34 | |
artom | dansmith, don't - aside from the fact that I made a type, it's a jumble of acronyms from my gaming days, mostly WoW. | 18:35 |
jaypipes | dansmith: walk back the comments while bowing at my feet, of course. | 18:35 |
openstackgerrit | Sarafraj Singh proposed openstack/nova: WIP - Improve help text of crypto.py https://review.openstack.org/332935 | 18:35 |
*** akshai has quit IRC | 18:35 | |
mriedem | mnestratov|2: the vz connector got into os-brick a long time ago | 18:35 |
*** shaohe_feng has joined #openstack-nova | 18:35 | |
mriedem | mnestratov|2: so the minimum os-brick in global-requirements will work for the vz storage ci | 18:35 |
*** rrecio_ has quit IRC | 18:35 | |
dansmith | jaypipes: I already know you like it, so no real point in acting like you don't anymore :) | 18:36 |
*** rrecio has joined #openstack-nova | 18:36 | |
*** salv-orlando has joined #openstack-nova | 18:36 | |
dansmith | jaypipes: of course | 18:37 |
jaypipes | dansmith: :P | 18:37 |
mnestratov|2 | mriedem: yeah, I know, but nonetheless it seemed to work only from sources two days ago | 18:38 |
mnestratov|2 | i'll check again | 18:38 |
*** mnestratov|2 is now known as mnestratov | 18:39 | |
*** Bjoern_zZzZzZzZ is now known as BjoernT | 18:39 | |
dansmith | mnestratov: ideally we run the tests with what is in u-c of course | 18:39 |
*** PaulMurray has joined #openstack-nova | 18:39 | |
dansmith | so we know it's likely to work with what we tell people to use | 18:39 |
*** tongli has quit IRC | 18:40 | |
dansmith | claudiub: I didn't follow all of your conversation with artom the other day, but did everything point to you being able to implement what they have in your driver? | 18:40 |
*** rodrigods has quit IRC | 18:40 | |
mnestratov | dansmith: :) and what is u-c ? | 18:40 |
*** rodrigods has joined #openstack-nova | 18:40 | |
dims | upper-constraints.txt from openstack/requirements repo | 18:40 |
dansmith | mnestratov: upper-constraints, or global-requirements.. two sets of package versions we tell people we know work | 18:41 |
artom | dansmith, did you see claudiub's POC? | 18:41 |
mnestratov | thanks, got it | 18:41 |
*** anush__ has joined #openstack-nova | 18:41 | |
dansmith | artom: oh, no.. linky? | 18:41 |
artom | dansmith, https://review.openstack.org/#/c/331889/ | 18:41 |
dansmith | mnestratov: again, thanks for being responsive to all these requests :) | 18:41 |
dansmith | artom: bitchin' | 18:42 |
artom | dansmith, word. | 18:42 |
mnestratov | dansmith: no problem | 18:43 |
claudiub | dansmith: yeah. I have to write some unit tests and address artom's comments | 18:43 |
*** mfedosin has quit IRC | 18:43 | |
dansmith | claudiub: sweet | 18:43 |
*** ctrath has quit IRC | 18:44 | |
artom | Is this what we do now, just talk in street slang, homies? | 18:44 |
claudiub | word | 18:44 |
claudiub | microsoft word | 18:44 |
dansmith | fo sho | 18:44 |
dansmith | haha | 18:44 |
*** ctrath has joined #openstack-nova | 18:44 | |
*** shaohe_feng has quit IRC | 18:45 | |
*** shaohe_feng has joined #openstack-nova | 18:45 | |
auggy | claudiub: i see you Excel at street slang ;) | 18:45 |
*** lpetrut has joined #openstack-nova | 18:45 | |
dansmith | auggy: that's a powerful point | 18:46 |
claudiub | auggy: indeed, it is probably my best power point. :) | 18:46 |
dansmith | damn, that's better | 18:46 |
auggy | it's important to have Access to all the information ;) | 18:46 |
artom | My outlook on this channel is getting worse with every terrible pun you guys make. | 18:46 |
claudiub | :)) | 18:46 |
dansmith | quite a funny MS SQL Server | 18:47 |
dansmith | no, wait, that doesn't work | 18:47 |
auggy | artom: maybe you should Exchange this channel for another one | 18:47 |
*** Prophidys_ has quit IRC | 18:47 | |
*** ijw has quit IRC | 18:47 | |
*** stewie925 has joined #openstack-nova | 18:48 | |
claudiub | let's try not to hyper-v-entilate too much from so many puns. :) | 18:48 |
*** ijw has joined #openstack-nova | 18:48 | |
*** anush__ has quit IRC | 18:48 | |
* auggy hands claudiub the trophy | 18:48 | |
artom | Let's open some windows for fresh air. | 18:48 |
sdague | claudiub: I had one question on - https://review.openstack.org/#/c/329978/9/nova/tests/unit/api/openstack/compute/test_extended_server_attributes.py | 18:49 |
*** Prophidys has joined #openstack-nova | 18:49 | |
sdague | I get why cells needed to be admin calls | 18:49 |
sdague | but I'm not sure I know why ext attributes needed to be | 18:49 |
*** TravT has quit IRC | 18:49 | |
*** alexpilotti has joined #openstack-nova | 18:49 | |
*** tonytan4ever has joined #openstack-nova | 18:49 | |
*** Raildo has joined #openstack-nova | 18:50 | |
mriedem | dansmith: i'm right here, right? https://review.openstack.org/#/c/326940/11/nova/objects/compute_node.py | 18:51 |
dansmith | mriedem: yup | 18:51 |
mriedem | dansmith: g-r doesn't tell you things work at the min b/c we don't test the min :) | 18:51 |
*** Sukhdev has quit IRC | 18:51 | |
dansmith | mriedem: right, but I said u-c and just referenced g-r for the context | 18:52 |
mnestratov | can anyone clarify, is it planned to implement and merge storage pools this cycle? | 18:52 |
dansmith | mnestratov: hoping to yeah | 18:53 |
claudiub | sdague: yeah, I think that stuck from a previous patchset. sorry. I was also removing rules from fake_policy.py, which was making the api looser, while by default those apis needed admin context. | 18:54 |
mnestratov | does it make sense to refresh https://review.openstack.org/#/c/314134/ then? | 18:54 |
mnestratov | I could help with that | 18:54 |
*** Sukhdev has joined #openstack-nova | 18:54 | |
claudiub | sdague: but that can be done in another pathces. | 18:54 |
sdague | claudiub: ah, so this is all because fake_policy is bogus and changing things in silly ways? | 18:55 |
*** shaohe_feng has quit IRC | 18:55 | |
sdague | that makes sense | 18:55 |
sdague | I just need a reason in my head about why we were changing tests here | 18:55 |
*** thorst is now known as thorst_away | 18:55 | |
claudiub | sdague: yeah. it proved quite time consuming to remove all those rules and fix the unit tests. | 18:55 |
*** shaohe_feng has joined #openstack-nova | 18:55 | |
sdague | claudiub: yep, no worries | 18:56 |
sdague | ok +2 on that one | 18:56 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2) https://review.openstack.org/329978 | 18:56 |
claudiub | removed the unit tests. | 18:56 |
dansmith | mnestratov: meaning rebase? I doubt anyone is going to complain if you help rebase their patch :D | 18:56 |
sdague | claudiub: ok, that will still pass? | 18:56 |
claudiub | sdague: yep, tested it locally | 18:56 |
mnestratov | dansmith: rebase, fixing unit tests, imlementing ploop part | 18:56 |
sdague | claudiub: great | 18:56 |
openstackgerrit | Sean Dague proposed openstack/nova: api-ref: add not about os-certificates API https://review.openstack.org/333538 | 18:57 |
*** ssurana has quit IRC | 18:57 | |
sdague | claudiub: yeh, fake policy was one of the sillier things we ever did in the unit tests | 18:57 |
*** ssurana has joined #openstack-nova | 18:57 | |
mriedem | sdague: pkholkin makes a good point here https://review.openstack.org/#/c/284868/9/nova/api/openstack/compute/keypairs.py@227 - if we change expected_errors in a microversion, it should probably be a separate method right? | 18:58 |
dansmith | mnestratov: I doubt anyone is going to complain.. You could start with a patch on top of that and then offer to squash it if the owner wants | 18:59 |
mnestratov | dansmith: ok then | 18:59 |
dansmith | mnestratov: and when I say "doubt anyone is going to complain" I mean "I'm sure that would be appreciated" :D | 18:59 |
*** akshai has joined #openstack-nova | 18:59 | |
mnestratov | just wanted to make sure it is actual | 18:59 |
sdague | mriedem: yes, that's a good catch | 18:59 |
claudiub | sdague: yeah, I'd say about a third of fake_policy can be simply removed, it contains default rules anyways. | 19:00 |
sdague | claudiub: right, it also contains some totally made up rules iirc | 19:00 |
sdague | right, after your code is in, we'll have defaults there | 19:00 |
sdague | previously, that wasn't the case | 19:01 |
*** raunak has quit IRC | 19:01 | |
sdague | we should probably have a background task of removing fake_policy all together early in the next cycle | 19:01 |
*** sambetts is now known as sambetts|afk | 19:01 | |
sdague | which also lets us get rid of the real_policy fixture | 19:01 |
*** ptm_away has joined #openstack-nova | 19:01 | |
sdague | which was there to use our actual policy during tests | 19:01 |
*** TravT has joined #openstack-nova | 19:01 | |
*** TravT has quit IRC | 19:01 | |
claudiub | sounds like a good task for people who wants to get into nova | 19:01 |
claudiub | for mentoring and stuff. | 19:02 |
*** PaulMurray has quit IRC | 19:02 | |
sdague | yeh | 19:02 |
*** TravT has joined #openstack-nova | 19:02 | |
*** mdrabe has quit IRC | 19:03 | |
sdague | claudiub: https://review.openstack.org/#/c/329227 seems to have a unit test fail | 19:03 |
claudiub | sdague: did disk_config get removed by any chance? | 19:04 |
sdague | claudiub: yes | 19:04 |
claudiub | sdague: oh, ok. that's why. I was still adding disk_config policies. | 19:05 |
sdague | ah, yeh, we're going to have some collisions there as my extensions removal patches land | 19:05 |
sdague | disk_config is the only one gone now | 19:05 |
*** shaohe_feng has quit IRC | 19:05 | |
sdague | access_ips is the next one up, but I think your patch will land first for that | 19:05 |
*** sheel has quit IRC | 19:05 | |
claudiub | sdague: aham. any others? so I can remove them now? | 19:06 |
sdague | I think disk_config is the only one currently removed | 19:06 |
claudiub | it's still probably to end up in merge confilict though.. | 19:06 |
*** shaohe_feng has joined #openstack-nova | 19:06 | |
sdague | https://review.openstack.org/#/c/333030/ is the stack where others are getting removed | 19:06 |
sdague | claudiub: yes, but I think the conflict will be on my side to resolve after this | 19:07 |
sdague | assuming that's the case, I'll stack on your patch stack when I fix them | 19:07 |
sdague | yeh https://review.openstack.org/#/c/328850/15 is about to pass check | 19:08 |
openstackgerrit | Sarafraj Singh proposed openstack/nova: Improve help text of s3 options https://review.openstack.org/332977 | 19:08 |
sdague | and I approved it, so you're patch series will end up the winners in the merge race | 19:08 |
sdague | s/you're/your/ | 19:09 |
mnestratov | hmm, there are two blueprints https://blueprints.launchpad.net/nova/+spec/libvirt-instance-storage and https://blueprints.launchpad.net/nova/+spec/use-libvirt-storage-pools | 19:09 |
mnestratov | the first one is approved and the second isn't, should all storage pool reviews reference approved one? | 19:09 |
dansmith | jaypipes: on that metadata list object, did you see they're dropping the list base mixin? that means they don't have to implement the objects field | 19:09 |
*** mdrabe has joined #openstack-nova | 19:09 | |
dansmith | jaypipes: I guess that's kindof vestigal from when they had that other field on there though, so hmmm | 19:10 |
*** josecastroleon has joined #openstack-nova | 19:10 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 2) https://review.openstack.org/329978 | 19:10 |
*** mat128 is now known as mat128|away | 19:12 | |
*** ctrath has quit IRC | 19:12 | |
jaypipes | dansmith: I would greatly prefer it to just be a List mixin and look/feel identical to all other List objects. | 19:13 |
jaypipes | dansmith: sorry for being a stickler on that. | 19:13 |
dansmith | jaypipes: but this one is being serialized as a whole, not as a list of things, so I don't think it makes sense | 19:13 |
dansmith | jaypipes: commenting, if you can look at my comments in context in a sec | 19:13 |
jaypipes | kk | 19:13 |
openstackgerrit | Carl Baldwin proposed openstack/nova: Enable deferred IP on Neutron ports https://review.openstack.org/299591 | 19:14 |
*** lucas___ has quit IRC | 19:14 | |
*** lucas___ has joined #openstack-nova | 19:15 | |
*** shaohe_feng has quit IRC | 19:15 | |
*** shaohe_feng has joined #openstack-nova | 19:16 | |
*** lucas___ has quit IRC | 19:20 | |
*** efried has joined #openstack-nova | 19:20 | |
dansmith | jaypipes: see if that makes sense: https://review.openstack.org/#/c/318884/17 | 19:21 |
sdague | claudiub: you'll need to rebase the whole stack | 19:21 |
dansmith | I think the confusion stems from the incorrect early supposition that this was going to be a regular list object and the -List suffix added thusly | 19:21 |
sdague | otherwise the patches on top are going to depend on the broken patch #2 | 19:21 |
sdague | or, from patch 2 up that is need rebasing | 19:21 |
openstackgerrit | Paul Murray proposed openstack/nova: Add periodic task to clean expired console tokens https://review.openstack.org/325381 | 19:22 |
openstackgerrit | Paul Murray proposed openstack/nova: Add console auth token records to the database https://review.openstack.org/325414 | 19:22 |
openstackgerrit | Paul Murray proposed openstack/nova: Add console auth object https://review.openstack.org/320063 | 19:22 |
openstackgerrit | Paul Murray proposed openstack/nova: Add instance uuid to console URLs https://review.openstack.org/325382 | 19:22 |
openstackgerrit | Paul Murray proposed openstack/nova: Add console auth tokens db api methods https://review.openstack.org/319964 | 19:22 |
*** htruta is now known as henrique | 19:22 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 3) https://review.openstack.org/328851 | 19:23 |
*** lucas___ has joined #openstack-nova | 19:23 | |
*** henrique is now known as Guest78419 | 19:23 | |
*** Guest78419 is now known as htruta | 19:23 | |
*** ptm_away is now known as PaulMurray | 19:25 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 4) https://review.openstack.org/329227 | 19:25 |
tpatzig | alaski: are you around? | 19:25 |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 5) https://review.openstack.org/329979 | 19:25 |
*** ctrath has joined #openstack-nova | 19:25 | |
openstackgerrit | Claudiu Belu proposed openstack/nova: policy: Add defaults in code (part 6) https://review.openstack.org/329228 | 19:25 |
*** shaohe_feng has quit IRC | 19:26 | |
PaulMurray | ok, why does jenkins find one patch - right in the middle of my series - and decide that that one depends on something that hasn't merged ????..!! | 19:26 |
tpatzig | alaski: as johnthetubaguy suggested i tried without the local_disks column in the old instance_types table; but it does not work | 19:26 |
PaulMurray | not the others, just that one | 19:26 |
*** welldannit has joined #openstack-nova | 19:26 | |
*** shaohe_feng has joined #openstack-nova | 19:26 | |
*** ctrath has quit IRC | 19:27 | |
*** lucas___ has quit IRC | 19:27 | |
*** tbachman_ has joined #openstack-nova | 19:27 | |
alaski | tpatzig: I'm around | 19:28 |
mriedem | PaulMurray: i've been hitting that also | 19:28 |
mriedem | it's some timeout on the clone/rebase | 19:28 |
*** tbachman has quit IRC | 19:28 | |
*** tbachman_ is now known as tbachman | 19:28 | |
PaulMurray | and interestingly that one is out of order in the above printout from openstackgerrit | 19:28 |
mriedem | PaulMurray: you just have to recheck it | 19:28 |
*** ctrath has joined #openstack-nova | 19:28 | |
*** lucas___ has joined #openstack-nova | 19:28 | |
migi | PaulMurray: could you point to the review? | 19:29 |
PaulMurray | mriedem, yes, got that - after noticing the printout I wonder if there is something non-determinstic going | 19:29 |
PaulMurray | on | 19:29 |
alaski | tpatzig: is this in reference to a specific patch? I haven't looked at that series in a while and don't have the context anymore | 19:29 |
PaulMurray | https://review.openstack.org/325381 | 19:29 |
PaulMurray | its the first one in the list above - it should be in the middle of the list | 19:29 |
migi | PaulMurray: btw jenkins was replaced by zuul so there is no jenkins anymore | 19:29 |
tpatzig | alaski: tests like nova.tests.unit.compute.test_compute.ComputeAPITestCase.test_create_bdm_from_flavor create instance_types and objects/flavor.py(262)_from_db_object compares db_flavor from instance_types with new model from flavor, where the new key is missing then | 19:30 |
*** salv-orlando has quit IRC | 19:30 | |
PaulMurray | migi - ok, referring to the voting name really, but good point | 19:30 |
mriedem | random question, do we get default ordering in sql results by id, or do you need to specify? | 19:30 |
tpatzig | alaski: e.g. http://logs.openstack.org/01/280301/23/check/gate-nova-python27-db/d513a32/console.html#_2016-06-23_15_25_46_180125 | 19:30 |
dansmith | PaulMurray: migi cleverly hiding behind the account of another system is a good deflection strategy | 19:30 |
*** ayoung has joined #openstack-nova | 19:31 | |
PaulMurray | I always thought naming a CI system after the bringer of chaos and destruction was kind of appropriate | 19:32 |
*** akshai has quit IRC | 19:32 | |
openstackgerrit | Julian proposed openstack/nova: WIP Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2) https://review.openstack.org/325435 | 19:32 |
PaulMurray | jenkins sounds more like it would serve tea in the afternoon | 19:32 |
*** lucas___ has quit IRC | 19:33 | |
tpatzig | alaski: i only get all tests happy when i create the new column in both tables instance_types and the new flavors | 19:33 |
mriedem | jaypipes: do you know if mysql will default sort by id if you don't specify an ordering? | 19:33 |
migi | PaulMurray: maybe I am blind but I don't see any issue with the above patch | 19:33 |
dansmith | mriedem: I thought it's db-native order if unspecified | 19:34 |
dansmith | mriedem: which will often be id order if using autoincrement | 19:34 |
dansmith | but I don't think any sorting work will be done if you don't ask for it for performance | 19:34 |
mriedem | hmm | 19:34 |
mriedem | if we're supporting paging in the API, it seems we should be explicit | 19:34 |
mriedem | since we have several paging microversions up right now | 19:34 |
migi | PaulMurray: you have 5 with same topic, there are some conflicts (which is normal) but I don't see any patch that your one depends on | 19:34 |
dansmith | mriedem: well the order should still be stable I think, and the paging stuff supports sort/page right? | 19:35 |
mriedem | dansmith: no | 19:35 |
*** lucas___ has joined #openstack-nova | 19:35 | |
mriedem | not for these new ones | 19:35 |
mriedem | we don't want to allow sort_keys | 19:35 |
dansmith | mriedem: well, I still think the order coming from the db will be stable for things like mysql at least | 19:35 |
mriedem | but our api should be the same regardless of backend db right? | 19:35 |
PaulMurray | migi, thanks for looking, it just seems to happen sometimes, don't know enough to know why | 19:36 |
dansmith | mriedem: what kind of backend wouldn't be stable? | 19:36 |
mriedem | redis? :) idk | 19:36 |
*** shaohe_feng has quit IRC | 19:36 | |
dansmith | mriedem: I mean, yes, but not sure it's worth the sorting penalty, tbh, and paging is often lossy anyway | 19:36 |
*** lucas___ has quit IRC | 19:36 | |
*** haypo has left #openstack-nova | 19:36 | |
*** lucas___ has joined #openstack-nova | 19:36 | |
dansmith | mriedem: you could also say that the db_api needs to be stable, so if your backend isn't stable, then sort it first | 19:36 |
*** shaohe_feng has joined #openstack-nova | 19:37 | |
dansmith | mriedem: this is for keypair and stuff? probably small amounts of data there, so maybe not a big deal to always sort, I just don't know that I'd say that across the board | 19:37 |
mriedem | i'm just looking at keypairs and hypervisors (compute nodes table) | 19:37 |
sdague | mriedem: the back end will be stable for any relational db | 19:38 |
alaski | tpatzig: just pulled the code down to take a look | 19:38 |
sdague | it will be row insert order, which is not meaningful to anyone, but it's stable | 19:38 |
alaski | tpatzig: okay, there seems to be two issues here. one is that the tests are inserting flavors directly into the cell db rather than the api db, you can probably ignore that for now. | 19:39 |
*** akshai has joined #openstack-nova | 19:39 | |
alaski | but that's the reason for your issue | 19:39 |
tpatzig | alaski: thanks. i think the problem is because tests still create instance_types | 19:39 |
alaski | which is that the cell db doesn't have the new column | 19:40 |
*** josecastroleon has quit IRC | 19:40 | |
tpatzig | yes | 19:40 |
PaulMurray | alaski, quick question - is it safe to assume that instance mappings are always there in the api db now ? | 19:40 |
tpatzig | instance_types does'nt have the new column | 19:40 |
alaski | http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/unit/compute/test_compute.py#n327 | 19:41 |
mriedem | the other wierd thing is for keypairs we're sorting on name and the marker is the name, but for compute nodes we're using the id as the marker | 19:41 |
alaski | tpatzig: right, and this test is inserting entries in the wrong db | 19:41 |
alaski | tpatzig: so you could fix that, or have the object ignore that missing entry | 19:41 |
alaski | PaulMurray: not yet | 19:41 |
*** ctrath has quit IRC | 19:41 | |
alaski | PaulMurray: only if a deployer has run the nova-manage cells migration commands | 19:42 |
sdague | mriedem: because we don't expose id in keypairs anywhere | 19:42 |
PaulMurray | alaski, so if I'm running mitaka and I have run the migration it will ? | 19:42 |
*** tbachman_ has joined #openstack-nova | 19:43 | |
mriedem | sdague: ok good point | 19:43 |
alaski | PaulMurray: doesn't look that made it into Mitaka, so only if you're running Newton code | 19:44 |
*** tbachman has quit IRC | 19:44 | |
*** tbachman_ is now known as tbachman | 19:44 | |
alaski | PaulMurray: currently you can just lookup instances normally, all those lookups will get fixed to account for instance_mappings later | 19:45 |
tpatzig | alaski: so instead of db.flavor_create we need to do 'nova.objects.flavor._flavor_create', right? | 19:45 |
alaski | tpatzig: yep. It should go through the object though, objects.Flavor._flavor_create. that odd redirect is from a fixtures testing issue that's now fixed and should go away | 19:46 |
*** shaohe_feng has quit IRC | 19:46 | |
tpatzig | ok. i try | 19:47 |
*** shaohe_feng has joined #openstack-nova | 19:47 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to server metadata unit tests https://review.openstack.org/333551 | 19:47 |
PaulMurray | alaski, I'm going to put a patch up tomorrow that modifies a console proxy to use the db instead of consoleauth | 19:47 |
PaulMurray | alaski, I'm trying to work out how it all fits with the ongoing cells work | 19:48 |
PaulMurray | alaski, I'll see what I can make work - will be good to get your feedback though - its getting tight now with FF looming | 19:48 |
alaski | for now cell db lookups are just regular db lookups with the right db context | 19:49 |
alaski | Nova is still configured with the cell db as the main db, so nothing special needs to be done for cell db queries | 19:49 |
PaulMurray | alaski, ok - so sound like I'm just setting things up so they will work when cells does kick in | 19:49 |
alaski | PaulMurray: yes | 19:50 |
PaulMurray | cool | 19:50 |
*** SamYaple has joined #openstack-nova | 19:50 | |
*** derekh has joined #openstack-nova | 19:50 | |
PaulMurray | mriedem, I haven't left any live migration notes for nova meeting - is that ok? Thought you would be able to cover it anyway | 19:52 |
*** salv-orlando has joined #openstack-nova | 19:52 | |
*** _ducttape_ has joined #openstack-nova | 19:53 | |
*** PaulMurray has quit IRC | 19:53 | |
*** ctrath has joined #openstack-nova | 19:53 | |
mriedem | PaulMurray: noooo! | 19:54 |
mriedem | paul always says something to me and then immediately leaves | 19:54 |
mriedem | 'btw, go f yourself' | 19:54 |
mriedem | *Paul left the room* | 19:54 |
wznoinsk | hi all | 19:55 |
wznoinsk | mriedem et al, what's the best way to ask for voting permissions for a 3rdparty CI in Nova? I'm think Intel NFV CI here... is it here on the #, attend a meeting or start a thread on ML? | 19:56 |
wznoinsk | s/think/thinking | 19:56 |
*** ducttape_ has quit IRC | 19:56 | |
SamYaple | can someone help me figure out why the transport_url does not work with nova-conductor when using more that one thread? | 19:56 |
*** shaohe_feng has quit IRC | 19:56 | |
jaypipes | mriedem: you shouldn't count on an order if it's not specified in the ORDER BY clause. | 19:56 |
SamYaple | it seems like as soon as oslo.service gets involved it doesnt properly parse the transport_url anymore | 19:57 |
mriedem | jaypipes: good, thanks, i didn't want to trust that | 19:57 |
SamYaple | reverts to using the default 'guest' username | 19:57 |
wznoinsk | jaypipes, johnthetubaguy ^ | 19:57 |
*** mfedosin has joined #openstack-nova | 19:57 | |
*** shaohe_feng has joined #openstack-nova | 19:57 | |
mriedem | wznoinsk: probably ML right now | 19:57 |
mriedem | wznoinsk: i don't really have time to stop and check results on that | 19:57 |
jaypipes | wznoinsk: yeah, ML is appropriate I think. | 19:58 |
wznoinsk | thanks | 19:58 |
*** fragatina has joined #openstack-nova | 19:59 | |
*** penick has quit IRC | 20:01 | |
*** penick has joined #openstack-nova | 20:01 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys https://review.openstack.org/329697 | 20:02 |
jaypipes | dansmith: hmm... I'm still not sold on this. :( Gimme a little while to stew on this patch. | 20:02 |
dansmith | jaypipes: hmm, okay | 20:03 |
mriedem | pkholkin: you're unit test is testing microversion 2.28 | 20:04 |
mriedem | so something is very wrong if those test are passing | 20:04 |
*** ayoung has quit IRC | 20:04 | |
dansmith | jaypipes: happy to talk through it too if you want | 20:04 |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Restrict Metadata to Lower Case Keys https://review.openstack.org/329697 | 20:05 |
*** shaohe_feng has quit IRC | 20:07 | |
*** shaohe_feng has joined #openstack-nova | 20:08 | |
*** ssurana has quit IRC | 20:08 | |
*** ssurana has joined #openstack-nova | 20:09 | |
mtreinish | auggy: ^^^ depends-on is only for cross repo dependencies | 20:10 |
auggy | mriedem: ah ok, thanks | 20:11 |
auggy | mriedem: these unit tests are making my brain mushy | 20:11 |
mtreinish | if its in the same tree just make a branch and put your patch on top | 20:11 |
auggy | mtreinish: i mean jeez | 20:11 |
mtreinish | man, mriedem gets all the glory | 20:11 |
auggy | mtreinish: lol i can't read anymore | 20:11 |
dansmith | mriedem: mtreinish: I've used it for in-tree stuff before where I didn't want to get in the middle of a set and just wanted to run tests on top of a thing | 20:11 |
*** fragatina has quit IRC | 20:11 | |
mriedem | i'm not part of this | 20:12 |
mriedem | i've also done what dan did | 20:12 |
dansmith | damn you two and your colors | 20:13 |
mtreinish | dansmith: everyone is green to me | 20:13 |
dansmith | heh | 20:13 |
mtreinish | I'm sure zuul does the right thing. But, I was just pointing out that mechanism was added for cross-repo stuff. Branches are much easier to follow in repo | 20:14 |
mtreinish | at least for me | 20:14 |
auggy | yeah i even tagged them with the same blueprint so it's not like i'm going to lose track or anything | 20:14 |
auggy | i'm seriously wondering if i should split the tests out into their own change | 20:15 |
auggy | that change is so huge and it's mostly test coverage | 20:15 |
auggy | but then philosophically, the tests *are* part of the change | 20:15 |
*** _ducttape_ has quit IRC | 20:15 | |
alaski | mriedem: I may not make the Nova meeting today for the cells update. Not much of note happened unless you want to mention the cookies. We need reviews, that was about the extent of it. | 20:16 |
jaypipes | dansmith: yeah, a hangout you and me would be useful. on a meeting for next ten minutes. after that? | 20:16 |
dansmith | jaypipes: okay | 20:16 |
*** ducttape_ has joined #openstack-nova | 20:16 | |
*** gyee has joined #openstack-nova | 20:16 | |
*** lucas___ has quit IRC | 20:16 | |
mriedem | alaski: ok | 20:17 |
*** shaohe_feng has quit IRC | 20:17 | |
tpatzig | alaski: is working fine with nova.objects.flavor._flavor_create | 20:17 |
*** shaohe_feng has joined #openstack-nova | 20:18 | |
mriedem | auggy: the tests kind of need to stay with the api change... | 20:18 |
*** ctrath has quit IRC | 20:18 | |
alaski | tpatzig: great | 20:18 |
tpatzig | alaski: but, nova.tests.functional.db.test_flavor_model.FlavorTablesCompareTestCase.test_tables_flavors_instance_types is failing (of course) ;) | 20:18 |
mriedem | auggy: but yes it's a ridiculous size of a change | 20:18 |
*** ayoung has joined #openstack-nova | 20:18 | |
auggy | mriedem: yeah that's how i feel about it too, i just wasn't sure if folks would prefer if i split it up | 20:18 |
*** ctrath has joined #openstack-nova | 20:18 | |
*** armax has joined #openstack-nova | 20:18 | |
tpatzig | alaski: it compares that both tables have the same columns | 20:19 |
alaski | tpatzig: yeah. we've put exceptions in those tests for other tables that are in both dbs | 20:19 |
auggy | so many tests T_T | 20:19 |
tpatzig | should we remove that one? | 20:20 |
alaski | oh yeah, that's one of the ones that already has exceptions. just remove it like instance_type_id is handled | 20:21 |
alaski | err, flavor_id rather | 20:21 |
tpatzig | ok | 20:21 |
*** tbachman_ has joined #openstack-nova | 20:24 | |
openstackgerrit | Carl Baldwin proposed openstack/nova: Enable deferred IP on Neutron ports https://review.openstack.org/299591 | 20:24 |
mriedem | carl_baldwin: aren't there patches up for this? https://blueprints.launchpad.net/nova/+spec/neutron-routed-networks | 20:25 |
mriedem | thought i saw one from before | 20:25 |
pkholkin | mriedem: I wasn't sure in some tests, so asked to look them strictly:) thanks a lot for review! Will look tomorrow and work on this | 20:25 |
carl_baldwin | mriedem: ^ Look one line above your question. | 20:26 |
mriedem | carl_baldwin: ha | 20:27 |
dansmith | mriedem: I hope you've got aloe vera | 20:27 |
carl_baldwin | mriedem: That's the only one so far. | 20:27 |
*** tbachman has quit IRC | 20:27 | |
*** tonytan4ever has quit IRC | 20:27 | |
*** Prophidys has quit IRC | 20:27 | |
*** zul has quit IRC | 20:27 | |
*** rbradfor has quit IRC | 20:27 | |
*** syfy323 has quit IRC | 20:27 | |
*** mkoderer__ has quit IRC | 20:27 | |
*** tojuvone_ has quit IRC | 20:27 | |
*** leifz has quit IRC | 20:27 | |
*** melwitt has quit IRC | 20:27 | |
*** toan has quit IRC | 20:27 | |
*** NobodyCam has quit IRC | 20:27 | |
*** sgordon has quit IRC | 20:27 | |
*** zhangjn has quit IRC | 20:27 | |
*** yhvh has quit IRC | 20:27 | |
*** pumaranikar has quit IRC | 20:27 | |
*** aignatov has quit IRC | 20:27 | |
*** Anticimex has quit IRC | 20:27 | |
*** mrda has quit IRC | 20:27 | |
*** diana_clarke has quit IRC | 20:27 | |
*** McNinja has quit IRC | 20:27 | |
*** ericksonsantos has quit IRC | 20:27 | |
*** smcginnis has quit IRC | 20:27 | |
*** liusheng has quit IRC | 20:27 | |
*** dimtruck has quit IRC | 20:27 | |
*** ZZelle has quit IRC | 20:27 | |
*** zigo has quit IRC | 20:27 | |
*** patrickeast has quit IRC | 20:27 | |
*** pkholkin has quit IRC | 20:27 | |
*** DuncanT has quit IRC | 20:27 | |
*** mgagne has quit IRC | 20:27 | |
*** junjie has quit IRC | 20:27 | |
*** BobBall_AWOL has quit IRC | 20:27 | |
*** cburgess has quit IRC | 20:27 | |
*** morgabra has quit IRC | 20:27 | |
*** phschwartz has quit IRC | 20:27 | |
*** swamireddy has quit IRC | 20:27 | |
*** jdurgin has quit IRC | 20:27 | |
*** mat128|away has quit IRC | 20:27 | |
*** _fortis has quit IRC | 20:27 | |
*** dutsmoc has quit IRC | 20:27 | |
*** belliott has quit IRC | 20:27 | |
*** lbragstad has quit IRC | 20:27 | |
*** j_king has quit IRC | 20:27 | |
*** jamielennox has quit IRC | 20:27 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to server actions unit tests https://review.openstack.org/333568 | 20:27 |
mriedem | carl_baldwin: need the bp link in the commit message i think? https://review.openstack.org/#/c/299591/ | 20:27 |
*** tbachman_ is now known as tbachman | 20:27 | |
*** mat128 has joined #openstack-nova | 20:27 | |
*** shaohe_feng has quit IRC | 20:27 | |
mriedem | to link it to bp | 20:27 |
*** j_king has joined #openstack-nova | 20:27 | |
*** lbragstad_ has joined #openstack-nova | 20:27 | |
*** BobBall_AWOL has joined #openstack-nova | 20:27 | |
*** belliott has joined #openstack-nova | 20:27 | |
*** phschwartz has joined #openstack-nova | 20:27 | |
*** aignatov has joined #openstack-nova | 20:27 | |
*** sgordon has joined #openstack-nova | 20:27 | |
*** Anticimex has joined #openstack-nova | 20:27 | |
*** zhangjn has joined #openstack-nova | 20:27 | |
*** smcginnis has joined #openstack-nova | 20:27 | |
carl_baldwin | But, to be fair, I didn't have the right topic on it. | 20:27 |
*** tojuvone_ has joined #openstack-nova | 20:27 | |
*** Prophidys has joined #openstack-nova | 20:27 | |
*** mrda has joined #openstack-nova | 20:27 | |
*** cburgess has joined #openstack-nova | 20:27 | |
*** ericksonsantos has joined #openstack-nova | 20:27 | |
*** ZZelle has joined #openstack-nova | 20:27 | |
*** mkoderer__ has joined #openstack-nova | 20:27 | |
*** leifz- has joined #openstack-nova | 20:28 | |
*** liusheng has joined #openstack-nova | 20:28 | |
*** shaohe_feng has joined #openstack-nova | 20:28 | |
*** jdurgin has joined #openstack-nova | 20:28 | |
*** tonytan4ever has joined #openstack-nova | 20:28 | |
*** rbradfor has joined #openstack-nova | 20:28 | |
*** melwitt has joined #openstack-nova | 20:28 | |
*** leifz- is now known as leifz | 20:28 | |
*** diana_clarke has joined #openstack-nova | 20:28 | |
*** zul has joined #openstack-nova | 20:28 | |
*** yhvh has joined #openstack-nova | 20:28 | |
*** melwitt is now known as Guest13191 | 20:28 | |
*** morgabra has joined #openstack-nova | 20:29 | |
*** toan has joined #openstack-nova | 20:29 | |
*** zigo has joined #openstack-nova | 20:29 | |
*** syfy323 has joined #openstack-nova | 20:29 | |
carl_baldwin | mriedem: The rest of the work is waiting behind the scheduling and johnthetubaguy 's blueprint. | 20:29 |
*** McNinja has joined #openstack-nova | 20:29 | |
*** woodster_ has quit IRC | 20:29 | |
*** pkholkin has joined #openstack-nova | 20:30 | |
*** NobodyCam has joined #openstack-nova | 20:30 | |
*** _fortis has joined #openstack-nova | 20:30 | |
*** mgagne has joined #openstack-nova | 20:30 | |
*** mgagne is now known as Guest20454 | 20:30 | |
*** swamireddy has joined #openstack-nova | 20:30 | |
*** dimtruck has joined #openstack-nova | 20:30 | |
*** tonytan4ever has quit IRC | 20:30 | |
*** dougwig has quit IRC | 20:30 | |
carl_baldwin | mriedem: Yeah, that too. I'll add it. | 20:30 |
*** junjie has joined #openstack-nova | 20:31 | |
*** patrickeast has joined #openstack-nova | 20:31 | |
*** Guest13191 is now known as melwitt | 20:31 | |
*** fyxim has quit IRC | 20:31 | |
carl_baldwin | mriedem: I missed it because the patch was up for a long time as a WIP. | 20:31 |
*** DuncanT has joined #openstack-nova | 20:31 | |
*** comstud has joined #openstack-nova | 20:32 | |
rlrossit | mriedem: dansmith: I'm taking a look at the instance.delete versioned notification patch, and I noticed that in order to get the IP info on the instance, it has to parse it like a dict (https://review.openstack.org/#/c/313654/35/nova/compute/utils.py) | 20:32 |
rlrossit | and that's because the instance info cache holds the network info as a fields.NetworkModel | 20:32 |
rlrossit | which in it's coercion/hydration is backed by the network model | 20:32 |
rlrossit | which is dict-like | 20:32 |
rlrossit | mriedem: dansmith: so instead of having a field that is backed by a dict-like object, do we just want to make that a versioned object? | 20:33 |
openstackgerrit | Claudiu Belu proposed openstack/nova: Added support for new block device format in vmops https://review.openstack.org/246299 | 20:33 |
openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for ISO images https://review.openstack.org/333569 | 20:33 |
openstackgerrit | Carl Baldwin proposed openstack/nova: Enable deferred IP on Neutron ports https://review.openstack.org/299591 | 20:33 |
mriedem | rlrossit: list of dict-like objects | 20:33 |
carl_baldwin | mriedem: Your timing couldn't be better though. I really just picked this up this morning. | 20:33 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [2/3] https://review.openstack.org/331632 | 20:34 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [3/3] https://review.openstack.org/331633 | 20:34 |
mriedem | rlrossit: we've talked about this before | 20:34 |
mriedem | but i don't remember why we didn't convert the network model to versioned objects yet, maybe because no one cared enough | 20:35 |
*** lbragstad_ is now known as lbragstad | 20:35 | |
mriedem | rlrossit: i thought beagles had a change going in that direction in the long ago | 20:35 |
mriedem | dansmith might remember | 20:35 |
*** rfolco has quit IRC | 20:35 | |
mriedem | plus until nova-network is dropped it's probably weird | 20:35 |
*** fyxim has joined #openstack-nova | 20:35 | |
*** rfolco has joined #openstack-nova | 20:35 | |
*** _fortis has quit IRC | 20:35 | |
*** tonytan4ever has joined #openstack-nova | 20:36 | |
rlrossit | mriedem: but if both nova-net and neutron can hydrate the same network model, we should be able to hydrate a versioned object with the same info right? | 20:36 |
*** pumaranikar has joined #openstack-nova | 20:36 | |
*** shaohe_feng has quit IRC | 20:37 | |
*** esmiurium has joined #openstack-nova | 20:38 | |
*** shaohe_feng has joined #openstack-nova | 20:38 | |
*** jamielennox has joined #openstack-nova | 20:38 | |
openstackgerrit | Ryan Rossiter proposed openstack/nova: WIP: Shared state host manager https://review.openstack.org/332463 | 20:39 |
mriedem | rlrossit: probably | 20:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support https://review.openstack.org/318953 | 20:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance https://review.openstack.org/316275 | 20:40 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add REST API support for get me a network https://review.openstack.org/316398 | 20:40 |
openstackgerrit | Sean Dague proposed openstack/nova: collapse servers.ViewBuilderV21 into servers.ViewBuilder https://review.openstack.org/333574 | 20:40 |
mriedem | dansmith: i had to fix a test in the bottom of that ^ | 20:40 |
mriedem | b/c of rebase | 20:40 |
mriedem | rlrossit: this is what i was talking about https://review.openstack.org/#/c/167287/ | 20:41 |
openstackgerrit | Matthew Treinish proposed openstack/nova: Add nova-manage command to lower case metadata https://review.openstack.org/329737 | 20:43 |
*** efried has quit IRC | 20:43 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova: api-ref: added docs for microversion 2.26 https://review.openstack.org/327089 | 20:44 |
sdague | alaski: https://review.openstack.org/#/c/329978/11 is the only patch in claudiub's policy in code definitions without a second +2 | 20:44 |
sdague | with luck we could get that whole stack merged tonight | 20:44 |
openstackgerrit | Baodong (Robert) Li proposed openstack/nova: [WIP] Perform local delete for VMs in error https://review.openstack.org/333579 | 20:45 |
*** tbachman_ has joined #openstack-nova | 20:45 | |
*** tbachman has quit IRC | 20:45 | |
*** tbachman_ is now known as tbachman | 20:45 | |
claudiub | this also: https://review.openstack.org/#/c/329228/11 | 20:45 |
*** tblakeslee has quit IRC | 20:47 | |
*** dougwig has joined #openstack-nova | 20:47 | |
*** tblakeslee has joined #openstack-nova | 20:47 | |
*** shaohe_feng has quit IRC | 20:48 | |
*** shaohe_feng has joined #openstack-nova | 20:48 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to server actions unit tests https://review.openstack.org/333568 | 20:51 |
*** ayoung has quit IRC | 20:51 | |
jaypipes | vladikr: k, new comment/review on 318884. sorry for all the back and forth on this! | 20:52 |
*** tblakeslee has quit IRC | 20:52 | |
vladikr | jaypipes, great, thanks! | 20:53 |
*** tblakeslee has joined #openstack-nova | 20:53 | |
*** cdent has joined #openstack-nova | 20:55 | |
alaski | sdague: got it | 20:55 |
*** diga has quit IRC | 20:55 | |
*** browne has quit IRC | 20:57 | |
*** shaohe_feng has quit IRC | 20:58 | |
*** shaohe_feng has joined #openstack-nova | 20:59 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to server metadata unit tests https://review.openstack.org/333551 | 20:59 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [2/3] https://review.openstack.org/331632 | 20:59 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [3/3] https://review.openstack.org/331633 | 20:59 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [1/3] https://review.openstack.org/280301 | 20:59 |
*** _ducttape_ has joined #openstack-nova | 20:59 | |
tonyb | meeting? | 21:00 |
*** lchen_ has quit IRC | 21:00 | |
*** eharney has quit IRC | 21:00 | |
mriedem | adrian_otto akuriata alevine alexpilotti aloga andreykurilin anteaya artom auggy | 21:00 |
mriedem | bauzas belliott belmoreira bobball cburgess claudiub danpb dguitarbite _diana_ | 21:00 |
mriedem | diana_clarke dims duncant edleafe efried flip214 funzo garyk gcb gjayavelu | 21:00 |
mriedem | irina_pov jaypipes jcookekhugen jgrimm jichen jlvillal jroll kashyap klindgren | 21:00 |
mriedem | krtaylor lbeliveau lxsli macsz markus_z mdorman med_ mikal mjturek mnestratov | 21:00 |
mriedem | moshele mrda nagyz ndipanov neiljerram nic Nisha PaulMurray raildo rgeragnov | 21:00 |
mriedem | sc68cal scottda sdague sileht sorrison swamireddy thorst tjones tonyb | 21:00 |
mriedem | tpatil tpatzig xyang rdopiera raj_singh woodster sahid rbradfor junjie gsilvis | 21:00 |
mriedem | mtanino | 21:00 |
mriedem | nova meeting in #openstack-meeting | 21:01 |
rlrossit | mriedem is getting flustered | 21:01 |
*** sean-k-mooney has joined #openstack-nova | 21:01 | |
mriedem | oh my god oh my god oh my god | 21:01 |
*** akshai has quit IRC | 21:01 | |
*** woodster_ has joined #openstack-nova | 21:01 | |
*** sacharya has joined #openstack-nova | 21:01 | |
*** mvk_ has joined #openstack-nova | 21:02 | |
*** ducttape_ has quit IRC | 21:02 | |
*** alexpilotti has quit IRC | 21:02 | |
*** kfarr has quit IRC | 21:03 | |
*** penick has quit IRC | 21:03 | |
*** rfolco has quit IRC | 21:03 | |
*** sacharya_ has quit IRC | 21:05 | |
*** lchen_ has joined #openstack-nova | 21:05 | |
*** foexle has joined #openstack-nova | 21:07 | |
*** ozialien10 has quit IRC | 21:08 | |
*** shaohe_feng has quit IRC | 21:08 | |
*** akshai has joined #openstack-nova | 21:08 | |
*** shaohe_feng has joined #openstack-nova | 21:09 | |
*** thorst_away has quit IRC | 21:10 | |
*** thorst_away has joined #openstack-nova | 21:13 | |
*** smatzek has quit IRC | 21:14 | |
*** akshai has quit IRC | 21:14 | |
*** thorst_a_ has joined #openstack-nova | 21:15 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to flavors extraspecs unit test https://review.openstack.org/333590 | 21:15 |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Improve help test for wsgi options https://review.openstack.org/333504 | 21:16 |
*** thorst_away has quit IRC | 21:17 | |
*** shaohe_feng has quit IRC | 21:18 | |
*** shaohe_feng has joined #openstack-nova | 21:19 | |
*** irenab has quit IRC | 21:19 | |
*** thorst_a_ has quit IRC | 21:19 | |
*** aysyd has quit IRC | 21:20 | |
*** catintheroof has quit IRC | 21:21 | |
*** tbachman has quit IRC | 21:23 | |
anteaya | edleafe: does anyone self identifying as part of the scientific working group ever communicate on scheduler issues or concerns? | 21:26 |
edleafe | anteaya: Not that I'm aware of | 21:26 |
anteaya | edleafe: scheduler has come up several times when the scientific working group folks gather | 21:27 |
anteaya | hmmmm | 21:27 |
edleafe | anteaya: but they may not have self-identified to me :) | 21:27 |
anteaya | I've offered to help them find you | 21:27 |
anteaya | edleafe: that is true, they might be participating and they didn't say | 21:27 |
anteaya | okay thanks | 21:27 |
edleafe | sure - have them bring up any issues they may have | 21:27 |
anteaya | edleafe: ever feel like attending a scientific working group irc meeting? | 21:28 |
anteaya | edleafe: thanks | 21:28 |
anteaya | edleafe: the next meeting is next Tuesday right after the tc meeting in -meeting | 21:28 |
doffm | anteaya: In what context has it come up? | 21:28 |
anteaya | if you feel like it | 21:28 |
anteaya | doffm: they want the scheduler to do certain things | 21:28 |
doffm | I guess we should just attend and see. | 21:28 |
anteaya | but I can't clearly understand what | 21:28 |
anteaya | so I fail about being a conduit here | 21:29 |
*** sthillma has quit IRC | 21:29 | |
edleafe | anteaya: wasn't aware of it | 21:29 |
*** shaohe_feng has quit IRC | 21:29 | |
anteaya | doffm: thanks that would be great | 21:29 |
anteaya | edleafe: cool, great group of folks | 21:29 |
edleafe | anteaya: added to my schedule | 21:29 |
anteaya | just getting the hang of irc meetings and such | 21:29 |
anteaya | university types and nasa and hpc | 21:29 |
anteaya | edleafe: thank you, the meetings rotate, this weeks was wednesday at 0800 utc | 21:30 |
doffm | FPGA! | 21:30 |
*** shaohe_feng has joined #openstack-nova | 21:30 | |
anteaya | http://eavesdrop.openstack.org/#Scientific_Working_Group | 21:30 |
*** ctrath has quit IRC | 21:30 | |
anteaya | doffm: https://en.wikipedia.org/wiki/Field-programmable_gate_array ? | 21:31 |
edleafe | doffm: what about FPGA? | 21:31 |
*** rlrossit has quit IRC | 21:31 | |
*** lchen_ has quit IRC | 21:32 | |
*** cdent has quit IRC | 21:32 | |
edleafe | anteaya: yeah, 0800 ain't happening for me :) | 21:32 |
bauzas | I can attend those EU timeslots | 21:33 |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: "nova list-extensions" not showing summary for all https://review.openstack.org/286130 | 21:33 |
bauzas | but what do they want to discuss in particular? | 21:33 |
*** lchen_ has joined #openstack-nova | 21:33 | |
bauzas | anteaya: ^ | 21:33 |
*** alexpilotti has joined #openstack-nova | 21:33 | |
bauzas | anteaya: do you have any idea of what are their concerns with the n-sch ? | 21:34 |
anteaya | edleafe: yeah me either | 21:34 |
anteaya | bauzas: no I don't unfortunately | 21:34 |
bauzas | okay | 21:34 |
anteaya | and I don't even know enough about the design or intended design to ask a reasonable question | 21:34 |
anteaya | bauzas: thanks that would be awesome | 21:35 |
bauzas | anteaya: you can put my name on those folks for the odd-time meetings | 21:35 |
*** sthillma has joined #openstack-nova | 21:35 | |
anteaya | I just know several folks have said more than once they are interested | 21:35 |
bauzas | I guess some of them are CERN | 21:35 |
anteaya | bauzas: thanks so much, I really appreciate that | 21:35 |
anteaya | yup, tim is there | 21:35 |
bauzas | okay, I can sort that out with those folks, I know them :) | 21:36 |
anteaya | and a guy from Finland, I can't recall his name, Nolle? | 21:36 |
*** sthillma has quit IRC | 21:36 | |
anteaya | probably spelled wrong | 21:36 |
*** rtheis_ has quit IRC | 21:36 | |
bauzas | k | 21:36 |
anteaya | bauzas: awesome thank you | 21:36 |
*** _ducttape_ has quit IRC | 21:36 | |
anteaya | I would just feel aweful if they missed the boat on these conversations when it sounds like they have real use cases that would be valuable to know about | 21:36 |
*** ducttape_ has joined #openstack-nova | 21:37 | |
anteaya | thanks for understanding | 21:37 |
anteaya | I appreciate it | 21:37 |
bauzas | k | 21:37 |
*** alexpilotti has quit IRC | 21:38 | |
*** thorst_away has joined #openstack-nova | 21:38 | |
*** sdake has quit IRC | 21:38 | |
*** shaohe_feng has quit IRC | 21:39 | |
*** shaohe_feng has joined #openstack-nova | 21:39 | |
*** tblakeslee has quit IRC | 21:41 | |
*** irenab has joined #openstack-nova | 21:41 | |
*** sdake has joined #openstack-nova | 21:42 | |
*** thorst_away has quit IRC | 21:43 | |
*** claudiub has quit IRC | 21:43 | |
*** thorst_away has joined #openstack-nova | 21:43 | |
*** armax has left #openstack-nova | 21:43 | |
doffm | anteaya: At the summit there were a number of people talking about scheduling based on availablity of attached FPGA, which might be a requirement for sceintific communities. | 21:46 |
*** mriedem has quit IRC | 21:46 | |
doffm | Sorry edleafe ^^ | 21:46 |
anteaya | ah, okay thank you I missed the conversation but it is entirely likely the scientific community might have a use case | 21:47 |
*** thorst_away has quit IRC | 21:47 | |
*** sthillma has joined #openstack-nova | 21:47 | |
openstackgerrit | sean mooney proposed openstack/os-vif: revert removal of create_ovs_vif_port timeout https://review.openstack.org/333602 | 21:48 |
edleafe | doffm: yes, I was part of that discussion | 21:48 |
edleafe | doffm: but it was more of a discussion clarifying that handling programming an FPGA was out of scope for Nova | 21:49 |
*** shaohe_feng has quit IRC | 21:49 | |
edleafe | doffm: and that was the "inspiration" for the development of dynamic resource classes | 21:50 |
anteaya | edleafe: ah thank you | 21:50 |
sean-k-mooney | jaypipes: if you get a chance can you take a look at https://review.openstack.org/333602 | 21:50 |
anteaya | edleafe: bauzas I've emailed stig (oneswig) and blair (b1airo) your irc nicks and the log from our discussion so they'll know to look out for you, and thanks again | 21:51 |
*** shaohe_feng has joined #openstack-nova | 21:51 | |
bauzas | k | 21:51 |
sean-k-mooney | jaypipes: the ovs plugin in os-vif is currently broken on master. that fixes the issue and adds tests to make sure it does not happen again | 21:51 |
edleafe | anteaya: kewl | 21:52 |
anteaya | :) | 21:52 |
*** tonytan4ever has quit IRC | 21:54 | |
openstackgerrit | Timofey Durakov proposed openstack/nova: Rename driver method check_can_live_migrate_destination_cleanup https://review.openstack.org/333519 | 21:54 |
openstackgerrit | Timofey Durakov proposed openstack/nova: centralize checks before live-migration https://review.openstack.org/332124 | 21:56 |
openstackgerrit | Timofey Durakov proposed openstack/nova: live_migration_cleanup method introduced https://review.openstack.org/302611 | 21:57 |
*** baoli has quit IRC | 21:57 | |
*** kylek3h has quit IRC | 21:57 | |
openstackgerrit | Timofey Durakov proposed openstack/nova: Remove unused migrate_data param from driver cleanup and destroy https://review.openstack.org/280653 | 21:57 |
openstackgerrit | Timofey Durakov proposed openstack/nova: call pre_live_migration from conductor https://review.openstack.org/291160 | 21:58 |
*** pilgrimstack has joined #openstack-nova | 21:58 | |
*** TravT has quit IRC | 21:58 | |
*** shaohe_feng has quit IRC | 21:59 | |
*** mmandel has quit IRC | 22:00 | |
jaypipes | sean-k-mooney: on it. | 22:00 |
*** shaohe_feng has joined #openstack-nova | 22:00 | |
*** burt has quit IRC | 22:00 | |
jaypipes | sean-k-mooney: done. | 22:01 |
*** ametts has quit IRC | 22:02 | |
sean-k-mooney | thanks i dont like breaking things :) | 22:02 |
*** dimtruck is now known as zz_dimtruck | 22:04 | |
sean-k-mooney | jaypipes: i think there is a bug in how we are using privsep also. or rather the logging | 22:05 |
sean-k-mooney | https://github.com/openstack/os-vif/blob/master/vif_plug_ovs/linux_net.py#L74-L77 here we are loging at debug level but it shows up as a warning in the compute agent logs | 22:06 |
openstackgerrit | Anush Krishnamurthy proposed openstack/nova: Improve consistency in cert https://review.openstack.org/332650 | 22:06 |
*** xyang1 has quit IRC | 22:07 | |
*** lpetrut has quit IRC | 22:07 | |
*** JoseMello has quit IRC | 22:09 | |
*** shaohe_feng has quit IRC | 22:10 | |
*** shaohe_feng has joined #openstack-nova | 22:10 | |
*** derekh has quit IRC | 22:11 | |
*** davidlenwell has quit IRC | 22:12 | |
*** wznoinsk is now known as wznoinsk_ | 22:13 | |
*** davidlenwell has joined #openstack-nova | 22:13 | |
*** edmondsw has quit IRC | 22:16 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Add DeviceTaggingFilter to scheduler https://review.openstack.org/331490 | 22:18 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: libvirt: attach configdrive after instance XML https://review.openstack.org/303335 | 22:18 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: objects: adding an update method to virtual_interface https://review.openstack.org/310618 | 22:18 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: objects: Add devices_metadata to instance object https://review.openstack.org/327920 | 22:18 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: db: add a device_metadata column to instance_extra https://review.openstack.org/318883 | 22:18 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 22:18 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: objects: enhance the DeviceMetadataList object https://review.openstack.org/318884 | 22:18 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 22:18 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Compute manager device tagging support https://review.openstack.org/264017 | 22:18 |
*** mdrabe has quit IRC | 22:19 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 22:19 |
*** shaohe_feng has quit IRC | 22:20 | |
*** pradk has quit IRC | 22:21 | |
*** shaohe_feng has joined #openstack-nova | 22:21 | |
openstackgerrit | Sarafraj Singh proposed openstack/nova: Improve help text of crypto.py https://review.openstack.org/332935 | 22:21 |
*** wznoinsk has joined #openstack-nova | 22:22 | |
*** wznoinsk_ has quit IRC | 22:22 | |
jaypipes | sean-k-mooney: hmm, weird... | 22:23 |
*** penick has joined #openstack-nova | 22:25 | |
*** pilgrimstack has quit IRC | 22:25 | |
sean-k-mooney | jaypipes: http://paste.openstack.org/show/521750/ | 22:25 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [2/3] https://review.openstack.org/331632 | 22:25 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [3/3] https://review.openstack.org/331633 | 22:25 |
openstackgerrit | Tom Patzig proposed openstack/nova: Add option to disable local disks in flavors [1/3] https://review.openstack.org/280301 | 22:25 |
*** erlon has quit IRC | 22:26 | |
sean-k-mooney | jaypipes: it looks like all log output form the privsep daemon is treated as a warning. as a result the log level in the code is ignored | 22:27 |
sean-k-mooney | in production debug loging would not be enabled so that message would never be printed anyway so it would not get elevate to a warning | 22:28 |
sean-k-mooney | it seems to be a limitation of privsep or we have not configured someting correctly | 22:28 |
*** Sukhdev has quit IRC | 22:29 | |
*** Sukhdev has joined #openstack-nova | 22:30 | |
*** shaohe_feng has quit IRC | 22:30 | |
*** shaohe_feng has joined #openstack-nova | 22:31 | |
jaypipes | sean-k-mooney: interesting. that's not really my area of expertise unfortunately. perhaps danpb might know somehting about that in the morning? | 22:32 |
*** mfedosin has quit IRC | 22:32 | |
*** diogogmt has quit IRC | 22:33 | |
*** tbachman has joined #openstack-nova | 22:33 | |
sean-k-mooney | jaypipes: ya ill pop back on tomorrow. it 11:30 here so i probably should get some sleep anyway. | 22:34 |
*** Sukhdev has quit IRC | 22:35 | |
*** Sukhdev has joined #openstack-nova | 22:35 | |
sean-k-mooney | gus: is this the gus that wrote the privsep code by any change? if so do you know if privesep can preserve the log level of logs from privalaged code? | 22:36 |
*** stewie925 has quit IRC | 22:40 | |
*** Sukhdev has quit IRC | 22:40 | |
*** shaohe_feng has quit IRC | 22:40 | |
*** sean-k-mooney is now known as sean-k-mooney_aw | 22:41 | |
*** yamahata has joined #openstack-nova | 22:41 | |
*** shaohe_feng has joined #openstack-nova | 22:41 | |
*** Raildo has quit IRC | 22:42 | |
openstackgerrit | Julian proposed openstack/nova: Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 2) https://review.openstack.org/325435 | 22:42 |
*** diogogmt has joined #openstack-nova | 22:44 | |
*** dingboopt has joined #openstack-nova | 22:46 | |
*** shaohe_feng has quit IRC | 22:51 | |
*** shaohe_feng has joined #openstack-nova | 22:51 | |
*** MikeG451 has quit IRC | 22:53 | |
*** Sukhdev has joined #openstack-nova | 22:54 | |
*** ducttape_ has quit IRC | 22:56 | |
*** fragatina has joined #openstack-nova | 22:57 | |
*** Sukhdev has quit IRC | 22:59 | |
*** salv-orl_ has joined #openstack-nova | 22:59 | |
*** sdague has quit IRC | 22:59 | |
*** sameo has quit IRC | 23:01 | |
*** shaohe_feng has quit IRC | 23:01 | |
*** salv-orlando has quit IRC | 23:02 | |
*** shaohe_feng has joined #openstack-nova | 23:02 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Improve help text for vmware https://review.openstack.org/333616 | 23:02 |
*** fragatina has quit IRC | 23:03 | |
*** Sukhdev has joined #openstack-nova | 23:03 | |
*** alexpilotti has joined #openstack-nova | 23:03 | |
*** fragatina has joined #openstack-nova | 23:03 | |
*** Sukhdev has quit IRC | 23:07 | |
*** shaohe_feng has quit IRC | 23:11 | |
*** Sukhdev has joined #openstack-nova | 23:11 | |
*** shaohe_feng has joined #openstack-nova | 23:12 | |
*** rcernin has joined #openstack-nova | 23:16 | |
*** sdague has joined #openstack-nova | 23:20 | |
*** shaohe_feng has quit IRC | 23:21 | |
*** vtech has joined #openstack-nova | 23:22 | |
*** shaohe_feng has joined #openstack-nova | 23:22 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to image metadata unit tests https://review.openstack.org/333625 | 23:24 |
*** foexle has quit IRC | 23:26 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to server actions unit tests https://review.openstack.org/333568 | 23:28 |
*** sheel has joined #openstack-nova | 23:29 | |
*** fragatina has quit IRC | 23:30 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to flavors extraspecs unit test https://review.openstack.org/333590 | 23:30 |
*** sdake_ has joined #openstack-nova | 23:31 | |
*** shaohe_feng has quit IRC | 23:32 | |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to image metadata unit tests https://review.openstack.org/333625 | 23:32 |
jaypipes | dansmith: \o/ 0 test failures. ResourceTracker.nodename attribute fully removed. | 23:32 |
*** shaohe_feng has joined #openstack-nova | 23:32 | |
dansmith | o.my | 23:32 |
*** bswartz has quit IRC | 23:32 | |
jaypipes | dansmith: it might be beer thirty now... | 23:33 |
*** zz_dimtruck is now known as dimtruck | 23:33 | |
dansmith | totes | 23:33 |
*** sdake has quit IRC | 23:35 | |
gmann_ | andrearosa: hi, sorry i was away yesterday | 23:39 |
openstackgerrit | Augustina Ragwitz proposed openstack/nova: Add microversion support to images unit tests https://review.openstack.org/333626 | 23:40 |
*** shaohe_feng has quit IRC | 23:42 | |
*** shaohe_feng has joined #openstack-nova | 23:43 | |
*** amotoki has quit IRC | 23:48 | |
*** shaohe_feng has quit IRC | 23:52 | |
*** shaohe_feng has joined #openstack-nova | 23:52 | |
*** sdake_ has quit IRC | 23:53 | |
*** daemontool has joined #openstack-nova | 23:53 | |
*** BjoernT has quit IRC | 23:53 | |
*** mriedem has joined #openstack-nova | 23:55 | |
*** alexpilotti has quit IRC | 23:56 | |
*** sdake has joined #openstack-nova | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!