| *** thorst__ has quit IRC | 00:00 | |
| *** thorst__ has joined #openstack-nova | 00:00 | |
| *** thors____ has quit IRC | 00:01 | |
| *** thorst___ has joined #openstack-nova | 00:01 | |
| *** thor_____ has quit IRC | 00:02 | |
| *** thors____ has joined #openstack-nova | 00:03 | |
| *** paul-carlton has quit IRC | 00:03 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: api-ref: limits.inc validate parameters https://review.openstack.org/315216 | 00:03 |
|---|---|---|
| *** thorst_ has quit IRC | 00:03 | |
| *** fragatina has quit IRC | 00:04 | |
| *** thorst_ has joined #openstack-nova | 00:04 | |
| *** paul-carlton has joined #openstack-nova | 00:04 | |
| *** rbridgeman has quit IRC | 00:05 | |
| *** thor_____ has joined #openstack-nova | 00:05 | |
| *** thorst__ has quit IRC | 00:05 | |
| *** thorst___ has quit IRC | 00:05 | |
| *** thorst__ has joined #openstack-nova | 00:06 | |
| *** mrkz has quit IRC | 00:06 | |
| *** shaohe_feng has quit IRC | 00:06 | |
| *** thorst___ has joined #openstack-nova | 00:07 | |
| *** rama_y has quit IRC | 00:07 | |
| *** tho______ has joined #openstack-nova | 00:08 | |
| *** thors____ has quit IRC | 00:08 | |
| *** thorst_ has quit IRC | 00:08 | |
| *** thorst_ has joined #openstack-nova | 00:08 | |
| *** shaohe_feng has joined #openstack-nova | 00:09 | |
| *** fragatina has joined #openstack-nova | 00:09 | |
| *** thors____ has joined #openstack-nova | 00:09 | |
| *** thor_____ has quit IRC | 00:09 | |
| *** thorst__ has quit IRC | 00:10 | |
| *** jgriffith_ has joined #openstack-nova | 00:11 | |
| *** thorst___ has quit IRC | 00:11 | |
| *** tho______ has quit IRC | 00:12 | |
| *** thorst__ has joined #openstack-nova | 00:12 | |
| *** ivase_ has joined #openstack-nova | 00:12 | |
| *** thorst_ has quit IRC | 00:13 | |
| *** thorst_ has joined #openstack-nova | 00:13 | |
| *** thors____ has quit IRC | 00:14 | |
| *** fragatina has quit IRC | 00:14 | |
| *** dimtruck is now known as zz_dimtruck | 00:14 | |
| *** thorst___ has joined #openstack-nova | 00:14 | |
| *** thors____ has joined #openstack-nova | 00:15 | |
| *** thor_____ has joined #openstack-nova | 00:16 | |
| *** ivase has quit IRC | 00:16 | |
| *** shaohe_feng has quit IRC | 00:17 | |
| *** shaohe_feng has joined #openstack-nova | 00:17 | |
| *** thorst__ has quit IRC | 00:17 | |
| *** thorst__ has joined #openstack-nova | 00:17 | |
| *** thorst_ has quit IRC | 00:18 | |
| *** thorst_ has joined #openstack-nova | 00:18 | |
| *** thorst___ has quit IRC | 00:18 | |
| *** jgriffith has quit IRC | 00:19 | |
| *** thors____ has quit IRC | 00:19 | |
| *** thorst___ has joined #openstack-nova | 00:19 | |
| *** penick has joined #openstack-nova | 00:20 | |
| *** thors____ has joined #openstack-nova | 00:20 | |
| *** thor_____ has quit IRC | 00:20 | |
| *** thor_____ has joined #openstack-nova | 00:21 | |
| *** tho______ has joined #openstack-nova | 00:22 | |
| *** thorst__ has quit IRC | 00:22 | |
| *** thorst_ has quit IRC | 00:23 | |
| *** thorst_ has joined #openstack-nova | 00:23 | |
| *** thorst___ has quit IRC | 00:23 | |
| *** thors____ has quit IRC | 00:24 | |
| *** thorst__ has joined #openstack-nova | 00:24 | |
| *** thorst___ has joined #openstack-nova | 00:25 | |
| *** thor_____ has quit IRC | 00:25 | |
| *** smatzek has joined #openstack-nova | 00:26 | |
| *** tho______ has quit IRC | 00:26 | |
| *** thors____ has joined #openstack-nova | 00:26 | |
| stewie925 | I think I was able to solve the Policy violation issue on running the commands - I ran 'source openrc admin' which switches me to 'admin' role, and I was able to run 'nova service-list' command with no more 'policy' errors | 00:26 |
| *** kmartin has quit IRC | 00:26 | |
| *** claudiub has quit IRC | 00:26 | |
| *** thor_____ has joined #openstack-nova | 00:27 | |
| *** shaohe_feng has quit IRC | 00:27 | |
| *** thorst_ has quit IRC | 00:27 | |
| *** shaohe_feng has joined #openstack-nova | 00:28 | |
| *** thorst_ has joined #openstack-nova | 00:28 | |
| *** diogogmt has joined #openstack-nova | 00:28 | |
| *** thorst__ has quit IRC | 00:29 | |
| *** thorst__ has joined #openstack-nova | 00:29 | |
| *** thorst___ has quit IRC | 00:30 | |
| *** thorst___ has joined #openstack-nova | 00:30 | |
| *** thors____ has quit IRC | 00:31 | |
| *** thors____ has joined #openstack-nova | 00:31 | |
| *** thor_____ has quit IRC | 00:31 | |
| *** thorst_ has quit IRC | 00:32 | |
| *** thorst_ has joined #openstack-nova | 00:32 | |
| *** thor_____ has joined #openstack-nova | 00:33 | |
| *** sajeesh has joined #openstack-nova | 00:34 | |
| *** tho______ has joined #openstack-nova | 00:34 | |
| *** sajeesh has quit IRC | 00:34 | |
| *** thorst__ has quit IRC | 00:34 | |
| *** thorst___ has quit IRC | 00:35 | |
| *** thorst__ has joined #openstack-nova | 00:35 | |
| *** zz_dimtruck is now known as dimtruck | 00:35 | |
| *** thors____ has quit IRC | 00:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Get Me a Network https://review.openstack.org/283206 | 00:36 |
| *** thorst___ has joined #openstack-nova | 00:37 | |
| *** thorst_ has quit IRC | 00:37 | |
| *** shaohe_feng has quit IRC | 00:37 | |
| *** thor_____ has quit IRC | 00:37 | |
| *** thorst_ has joined #openstack-nova | 00:38 | |
| *** tho______ has quit IRC | 00:38 | |
| *** shaohe_feng has joined #openstack-nova | 00:38 | |
| *** thors____ has joined #openstack-nova | 00:39 | |
| *** thorst__ has quit IRC | 00:39 | |
| *** thorst__ has joined #openstack-nova | 00:40 | |
| *** thor_____ has joined #openstack-nova | 00:41 | |
| *** gagehugo has quit IRC | 00:41 | |
| *** thorst___ has quit IRC | 00:41 | |
| *** mtanino has quit IRC | 00:41 | |
| *** tbachman has quit IRC | 00:42 | |
| *** thorst___ has joined #openstack-nova | 00:42 | |
| *** tbachman has joined #openstack-nova | 00:42 | |
| *** thorst_ has quit IRC | 00:42 | |
| *** thorst_ has joined #openstack-nova | 00:43 | |
| *** thors____ has quit IRC | 00:43 | |
| *** thors____ has joined #openstack-nova | 00:44 | |
| *** thorst__ has quit IRC | 00:44 | |
| *** thorst__ has joined #openstack-nova | 00:45 | |
| *** thor_____ has quit IRC | 00:45 | |
| *** baoli has joined #openstack-nova | 00:45 | |
| *** lifeless has quit IRC | 00:46 | |
| *** thorst___ has quit IRC | 00:46 | |
| *** lifeless has joined #openstack-nova | 00:47 | |
| *** thorst___ has joined #openstack-nova | 00:47 | |
| *** derekh has joined #openstack-nova | 00:47 | |
| *** thorst_ has quit IRC | 00:47 | |
| *** shaohe_feng has quit IRC | 00:47 | |
| *** gjayavelu has quit IRC | 00:48 | |
| *** shaohe_feng has joined #openstack-nova | 00:48 | |
| *** thorst_ has joined #openstack-nova | 00:48 | |
| *** thors____ has quit IRC | 00:48 | |
| *** thors____ has joined #openstack-nova | 00:49 | |
| *** jdurgin1 has joined #openstack-nova | 00:49 | |
| *** thorst__ has quit IRC | 00:49 | |
| *** thorst__ has joined #openstack-nova | 00:50 | |
| *** thor_____ has joined #openstack-nova | 00:51 | |
| *** thorst___ has quit IRC | 00:52 | |
| *** thorst___ has joined #openstack-nova | 00:52 | |
| *** dimtruck is now known as zz_dimtruck | 00:52 | |
| *** zz_dimtruck is now known as dimtruck | 00:52 | |
| *** thorst_ has quit IRC | 00:53 | |
| *** thors____ has quit IRC | 00:53 | |
| *** rcernin has quit IRC | 00:54 | |
| *** thorst_ has joined #openstack-nova | 00:54 | |
| *** thorst__ has quit IRC | 00:55 | |
| *** fishcried has joined #openstack-nova | 00:55 | |
| *** thorst__ has joined #openstack-nova | 00:55 | |
| *** thor_____ has quit IRC | 00:56 | |
| *** thors____ has joined #openstack-nova | 00:56 | |
| *** thorst___ has quit IRC | 00:56 | |
| *** thorst___ has joined #openstack-nova | 00:57 | |
| *** shaohe_feng has quit IRC | 00:58 | |
| *** thor_____ has joined #openstack-nova | 00:58 | |
| *** shaohe_feng has joined #openstack-nova | 00:58 | |
| *** fishcried has quit IRC | 00:58 | |
| *** thorst_ has quit IRC | 00:58 | |
| *** thorst_ has joined #openstack-nova | 00:59 | |
| *** thorst__ has quit IRC | 01:00 | |
| *** thorst__ has joined #openstack-nova | 01:01 | |
| *** thors____ has quit IRC | 01:01 | |
| *** thorst___ has quit IRC | 01:01 | |
| *** thorst___ has joined #openstack-nova | 01:01 | |
| *** dimtruck is now known as zz_dimtruck | 01:02 | |
| *** thor_____ has quit IRC | 01:02 | |
| *** thors____ has joined #openstack-nova | 01:03 | |
| *** thorst_ has quit IRC | 01:03 | |
| *** thorst_ has joined #openstack-nova | 01:04 | |
| *** cheneydc has joined #openstack-nova | 01:05 | |
| *** markvoelker_ has joined #openstack-nova | 01:05 | |
| *** thorst__ has quit IRC | 01:05 | |
| *** thor_____ has joined #openstack-nova | 01:05 | |
| *** thorst___ has quit IRC | 01:06 | |
| *** thorst__ has joined #openstack-nova | 01:06 | |
| *** thorst___ has joined #openstack-nova | 01:07 | |
| *** thors____ has quit IRC | 01:08 | |
| *** shaohe_feng has quit IRC | 01:08 | |
| *** thorst_ has quit IRC | 01:09 | |
| *** shaohe_feng has joined #openstack-nova | 01:09 | |
| *** thors____ has joined #openstack-nova | 01:09 | |
| *** derekh has quit IRC | 01:09 | |
| *** thorst_ has joined #openstack-nova | 01:10 | |
| *** thor_____ has quit IRC | 01:10 | |
| *** thorst__ has quit IRC | 01:10 | |
| *** thorst__ has joined #openstack-nova | 01:10 | |
| *** thorst___ has quit IRC | 01:11 | |
| *** thorst___ has joined #openstack-nova | 01:11 | |
| *** thor_____ has joined #openstack-nova | 01:12 | |
| *** tho______ has joined #openstack-nova | 01:13 | |
| *** jinxing has joined #openstack-nova | 01:13 | |
| *** thors____ has quit IRC | 01:13 | |
| *** ozialien10 has quit IRC | 01:14 | |
| *** nic has quit IRC | 01:14 | |
| *** thors____ has joined #openstack-nova | 01:14 | |
| *** thorst_ has quit IRC | 01:14 | |
| *** raunak has joined #openstack-nova | 01:14 | |
| *** mriedem has quit IRC | 01:15 | |
| *** thorst__ has quit IRC | 01:15 | |
| *** thorst_ has joined #openstack-nova | 01:15 | |
| *** thorst___ has quit IRC | 01:16 | |
| *** ijw has joined #openstack-nova | 01:16 | |
| *** thorst__ has joined #openstack-nova | 01:16 | |
| *** thorst__ has quit IRC | 01:16 | |
| *** thorst__ has joined #openstack-nova | 01:16 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Method verification for os-volume_attachments.inc https://review.openstack.org/315289 | 01:17 |
| *** thor_____ has quit IRC | 01:17 | |
| *** winston-d_ has joined #openstack-nova | 01:17 | |
| *** tho______ has quit IRC | 01:18 | |
| *** shaohe_feng has quit IRC | 01:18 | |
| *** thorst__ has quit IRC | 01:19 | |
| *** thors____ has quit IRC | 01:19 | |
| *** stewie925 has quit IRC | 01:19 | |
| *** shaohe_feng has joined #openstack-nova | 01:19 | |
| *** thorst_ has quit IRC | 01:20 | |
| *** iceyao has joined #openstack-nova | 01:22 | |
| *** rama_y has joined #openstack-nova | 01:22 | |
| *** iceyao_ has joined #openstack-nova | 01:23 | |
| *** shakamunyi has joined #openstack-nova | 01:23 | |
| *** ijw has quit IRC | 01:23 | |
| *** iceyao has quit IRC | 01:26 | |
| *** penick has quit IRC | 01:27 | |
| *** shaohe_feng has quit IRC | 01:28 | |
| *** shaohe_feng has joined #openstack-nova | 01:29 | |
| *** EinstCrazy has joined #openstack-nova | 01:30 | |
| *** BjoernT has joined #openstack-nova | 01:31 | |
| *** shaohe_feng has quit IRC | 01:39 | |
| *** shaohe_feng has joined #openstack-nova | 01:39 | |
| openstackgerrit | melanie witt proposed openstack/nova: imagebackend: Check that the RBD image exists before trying to cleanup https://review.openstack.org/315187 | 01:41 |
| openstackgerrit | melanie witt proposed openstack/nova: Rewrite _cleanup_resize and finish_migration unit tests to use mock instead of mox https://review.openstack.org/315241 | 01:41 |
| *** zhangjn has quit IRC | 01:41 | |
| openstackgerrit | melanie witt proposed openstack/nova: Remove mox in test_volume_snapshot_create_outer_success https://review.openstack.org/315309 | 01:41 |
| *** zz_dimtruck is now known as dimtruck | 01:42 | |
| *** jinxing has quit IRC | 01:45 | |
| *** amotoki has joined #openstack-nova | 01:46 | |
| *** zhangjn has joined #openstack-nova | 01:46 | |
| *** ducttape_ has joined #openstack-nova | 01:46 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Optimize creating security_group https://review.openstack.org/315311 | 01:48 |
| *** shaohe_feng has quit IRC | 01:49 | |
| *** shaohe_feng has joined #openstack-nova | 01:49 | |
| *** salv-orlando has joined #openstack-nova | 01:51 | |
| *** tbachman has quit IRC | 01:51 | |
| *** markvoelker_ has quit IRC | 01:51 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Extend ResourceProvider to have resources and aggregates https://review.openstack.org/284963 | 01:52 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 01:52 |
| *** tbachman has joined #openstack-nova | 01:52 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Implement hypervisor notifications https://review.openstack.org/315312 | 01:53 |
| *** cdent has quit IRC | 01:54 | |
| *** jinxing has joined #openstack-nova | 01:55 | |
| *** salv-orlando has quit IRC | 01:56 | |
| *** raunak has quit IRC | 01:57 | |
| *** pwnall has quit IRC | 01:58 | |
| *** ljxiash has joined #openstack-nova | 01:58 | |
| *** markvoelker has joined #openstack-nova | 01:59 | |
| *** shaohe_feng has quit IRC | 01:59 | |
| *** markvoelker_ has joined #openstack-nova | 01:59 | |
| *** shaohe_feng has joined #openstack-nova | 02:00 | |
| *** rooooook has quit IRC | 02:01 | |
| *** jinxing has quit IRC | 02:02 | |
| *** kengo_sakai has joined #openstack-nova | 02:02 | |
| *** markvoelker has quit IRC | 02:03 | |
| *** gjayavelu has joined #openstack-nova | 02:06 | |
| *** hoangcx has joined #openstack-nova | 02:08 | |
| *** ljxiash has quit IRC | 02:09 | |
| *** ljxiash has joined #openstack-nova | 02:09 | |
| *** shaohe_feng has quit IRC | 02:09 | |
| *** shaohe_feng has joined #openstack-nova | 02:10 | |
| *** BjoernT has quit IRC | 02:13 | |
| *** apoorvad has quit IRC | 02:19 | |
| *** shaohe_feng has quit IRC | 02:20 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: api-ref: os-floating-ip-dns.inc method verification https://review.openstack.org/315284 | 02:20 |
| *** shaohe_feng has joined #openstack-nova | 02:21 | |
| *** tonytan4ever has joined #openstack-nova | 02:21 | |
| *** rfolco has quit IRC | 02:22 | |
| *** fishcried has joined #openstack-nova | 02:24 | |
| *** ljxiash has quit IRC | 02:26 | |
| *** ljxiash has joined #openstack-nova | 02:26 | |
| *** ducttape_ has quit IRC | 02:27 | |
| *** houming has joined #openstack-nova | 02:27 | |
| *** jinxing has joined #openstack-nova | 02:28 | |
| *** Daisy has joined #openstack-nova | 02:29 | |
| *** ljxiash has quit IRC | 02:29 | |
| *** ljxiash has joined #openstack-nova | 02:29 | |
| *** shaohe_feng has quit IRC | 02:30 | |
| *** shaohe_feng has joined #openstack-nova | 02:31 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Parameter verification for os-interface.inc https://review.openstack.org/315318 | 02:31 |
| *** markvoelker_ has quit IRC | 02:31 | |
| *** Swami_ has joined #openstack-nova | 02:34 | |
| *** mdrabe has joined #openstack-nova | 02:34 | |
| yonglihe | lbeliveau: no plan before get resources... | 02:35 |
| *** Daisy has quit IRC | 02:37 | |
| *** penick has joined #openstack-nova | 02:37 | |
| *** Swami has quit IRC | 02:38 | |
| *** daemontool has quit IRC | 02:38 | |
| *** markvoelker_ has joined #openstack-nova | 02:38 | |
| *** shaohe_feng has quit IRC | 02:40 | |
| *** shaohe_feng has joined #openstack-nova | 02:41 | |
| *** ivase has joined #openstack-nova | 02:41 | |
| *** ivase_ has quit IRC | 02:44 | |
| *** raunak has joined #openstack-nova | 02:45 | |
| *** raunak has quit IRC | 02:45 | |
| *** raunak has joined #openstack-nova | 02:46 | |
| *** ducttape_ has joined #openstack-nova | 02:46 | |
| *** smatzek has quit IRC | 02:49 | |
| *** nelsnelson has quit IRC | 02:50 | |
| *** shaohe_feng has quit IRC | 02:50 | |
| *** ducttape_ has quit IRC | 02:51 | |
| *** baoli has quit IRC | 02:51 | |
| *** shaohe_feng has joined #openstack-nova | 02:53 | |
| *** ducttape_ has joined #openstack-nova | 02:54 | |
| *** ljxiash has quit IRC | 02:55 | |
| *** ljxiash has joined #openstack-nova | 02:56 | |
| *** rfolco has joined #openstack-nova | 02:59 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/313743 | 02:59 |
| *** smatzek has joined #openstack-nova | 03:00 | |
| *** ljxiash has quit IRC | 03:00 | |
| *** shaohe_feng has quit IRC | 03:01 | |
| *** unicell has quit IRC | 03:01 | |
| *** unicell has joined #openstack-nova | 03:01 | |
| *** shaohe_feng has joined #openstack-nova | 03:01 | |
| *** unicell has quit IRC | 03:01 | |
| *** unicell has joined #openstack-nova | 03:01 | |
| *** Sukhdev has quit IRC | 03:02 | |
| *** tonytan4ever has quit IRC | 03:08 | |
| *** takashin has left #openstack-nova | 03:09 | |
| *** ducttape_ has quit IRC | 03:09 | |
| *** ivase_ has joined #openstack-nova | 03:10 | |
| *** ivase has quit IRC | 03:10 | |
| *** shaohe_feng has quit IRC | 03:11 | |
| *** unicell has quit IRC | 03:11 | |
| *** shaohe_feng has joined #openstack-nova | 03:11 | |
| *** markvoelker_ has quit IRC | 03:13 | |
| *** ducttape_ has joined #openstack-nova | 03:15 | |
| *** diogogmt has quit IRC | 03:18 | |
| *** raunak has quit IRC | 03:19 | |
| *** shaohe_feng has quit IRC | 03:21 | |
| *** shaohe_feng has joined #openstack-nova | 03:22 | |
| *** raunak has joined #openstack-nova | 03:23 | |
| *** dimtruck is now known as zz_dimtruck | 03:29 | |
| *** sacharya has quit IRC | 03:31 | |
| *** shaohe_feng has quit IRC | 03:31 | |
| *** shaohe_feng has joined #openstack-nova | 03:32 | |
| *** links has joined #openstack-nova | 03:34 | |
| *** ducttape_ has quit IRC | 03:39 | |
| *** smatzek has quit IRC | 03:42 | |
| *** shaohe_feng has quit IRC | 03:42 | |
| *** shaohe_feng has joined #openstack-nova | 03:42 | |
| *** fragatina has joined #openstack-nova | 03:43 | |
| *** fragatina has quit IRC | 03:45 | |
| *** fragatina has joined #openstack-nova | 03:46 | |
| *** fragatina has quit IRC | 03:46 | |
| *** fragatina has joined #openstack-nova | 03:47 | |
| *** sree has joined #openstack-nova | 03:49 | |
| *** rfolco has quit IRC | 03:52 | |
| *** shaohe_feng has quit IRC | 03:52 | |
| *** shaohe_feng has joined #openstack-nova | 03:52 | |
| *** EinstCrazy has quit IRC | 04:01 | |
| *** Fdaisuke_ has joined #openstack-nova | 04:02 | |
| *** shaohe_feng has quit IRC | 04:02 | |
| *** EinstCrazy has joined #openstack-nova | 04:03 | |
| *** Fdaisuke has quit IRC | 04:04 | |
| *** shaohe_feng has joined #openstack-nova | 04:05 | |
| *** shaohe_feng has quit IRC | 04:12 | |
| *** prateek has joined #openstack-nova | 04:13 | |
| *** shaohe_feng has joined #openstack-nova | 04:15 | |
| *** shaohe_feng has quit IRC | 04:23 | |
| *** shaohe_feng has joined #openstack-nova | 04:23 | |
| *** pcaruana has joined #openstack-nova | 04:25 | |
| *** palexster has quit IRC | 04:28 | |
| *** palexster has joined #openstack-nova | 04:28 | |
| *** sacharya has joined #openstack-nova | 04:32 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: centralize cache options https://review.openstack.org/315341 | 04:32 |
| *** pcaruana has quit IRC | 04:32 | |
| *** shaohe_feng has quit IRC | 04:33 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: api-ref: parameter verification for os-quota-sets https://review.openstack.org/314833 | 04:33 |
| *** shaohe_feng has joined #openstack-nova | 04:33 | |
| *** jdurgin1 has quit IRC | 04:34 | |
| *** sacharya has quit IRC | 04:37 | |
| *** salv-orlando has joined #openstack-nova | 04:39 | |
| *** raunak has quit IRC | 04:40 | |
| *** prateek has quit IRC | 04:40 | |
| *** shaohe_feng has quit IRC | 04:43 | |
| *** shaohe_feng has joined #openstack-nova | 04:44 | |
| *** raunak has joined #openstack-nova | 04:44 | |
| *** salv-orlando has quit IRC | 04:46 | |
| *** fishcried has quit IRC | 04:52 | |
| *** shaohe_feng has quit IRC | 04:53 | |
| *** shaohe_feng has joined #openstack-nova | 04:55 | |
| *** Fdaisuke_ has quit IRC | 04:55 | |
| *** Sukhdev has joined #openstack-nova | 04:56 | |
| *** Fdaisuke_ has joined #openstack-nova | 04:56 | |
| *** jinxing has quit IRC | 04:57 | |
| *** sdake has quit IRC | 04:57 | |
| *** salv-orlando has joined #openstack-nova | 04:58 | |
| *** penick has quit IRC | 05:00 | |
| *** mdrabe has quit IRC | 05:03 | |
| *** penick has joined #openstack-nova | 05:03 | |
| *** shaohe_feng has quit IRC | 05:04 | |
| *** shaohe_feng has joined #openstack-nova | 05:05 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: centralize smbfs libvirt options (16) https://review.openstack.org/309360 | 05:05 |
| *** houming has quit IRC | 05:08 | |
| *** jinxing has joined #openstack-nova | 05:11 | |
| *** armax has quit IRC | 05:13 | |
| *** ccarmack has joined #openstack-nova | 05:14 | |
| *** shaohe_feng has quit IRC | 05:14 | |
| *** shaohe_feng has joined #openstack-nova | 05:15 | |
| *** rama_y has quit IRC | 05:16 | |
| *** rama_y has joined #openstack-nova | 05:17 | |
| *** woodster_ has quit IRC | 05:18 | |
| *** ccarmack has quit IRC | 05:18 | |
| *** prateek has joined #openstack-nova | 05:21 | |
| *** apoorv has joined #openstack-nova | 05:23 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: centralize remotefs libvirt options (end) https://review.openstack.org/309363 | 05:24 |
| *** shaohe_feng has quit IRC | 05:24 | |
| *** shaohe_feng has joined #openstack-nova | 05:25 | |
| *** kodokuu has quit IRC | 05:30 | |
| *** iceyao_ has quit IRC | 05:33 | |
| *** iceyao has joined #openstack-nova | 05:33 | |
| *** shaohe_feng has quit IRC | 05:34 | |
| *** zenoway has joined #openstack-nova | 05:35 | |
| *** shaohe_feng has joined #openstack-nova | 05:36 | |
| *** amotoki_ has joined #openstack-nova | 05:39 | |
| *** ljxiash has joined #openstack-nova | 05:40 | |
| *** Thelo has joined #openstack-nova | 05:41 | |
| Thelo | Hi, when nova boots a vm from image ( in glance) , why does it need to call cinder API? | 05:42 |
| *** rama_y has quit IRC | 05:43 | |
| *** amotoki has quit IRC | 05:43 | |
| *** ljxiash has quit IRC | 05:44 | |
| *** shaohe_feng has quit IRC | 05:45 | |
| *** shaohe_feng has joined #openstack-nova | 05:46 | |
| *** zenoway has quit IRC | 05:50 | |
| *** ljxiash has joined #openstack-nova | 05:53 | |
| *** rcernin has joined #openstack-nova | 05:54 | |
| *** shaohe_feng has quit IRC | 05:55 | |
| *** shaohe_feng has joined #openstack-nova | 05:56 | |
| *** rama_y has joined #openstack-nova | 05:56 | |
| *** iceyao has quit IRC | 05:59 | |
| *** ljxiash has quit IRC | 05:59 | |
| *** ljxiash has joined #openstack-nova | 06:00 | |
| *** ljxiash has quit IRC | 06:04 | |
| *** shaohe_feng has quit IRC | 06:05 | |
| *** andreas_s has joined #openstack-nova | 06:06 | |
| *** ljxiash has joined #openstack-nova | 06:07 | |
| *** Oku_OS-away is now known as Oku_OS | 06:07 | |
| *** mjura has joined #openstack-nova | 06:07 | |
| *** shaohe_feng has joined #openstack-nova | 06:08 | |
| *** pcaruana has joined #openstack-nova | 06:12 | |
| *** laszloh has joined #openstack-nova | 06:12 | |
| *** houming has joined #openstack-nova | 06:14 | |
| *** sudipto has joined #openstack-nova | 06:15 | |
| *** shaohe_feng has quit IRC | 06:16 | |
| *** shaohe_feng has joined #openstack-nova | 06:16 | |
| *** rama_y has quit IRC | 06:19 | |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Warn about unset cache backend https://review.openstack.org/307718 | 06:20 |
| *** esracelik has joined #openstack-nova | 06:24 | |
| *** shaohe_feng has quit IRC | 06:26 | |
| *** shaohe_feng has joined #openstack-nova | 06:26 | |
| *** raunak has quit IRC | 06:29 | |
| *** nkrinner_afk is now known as nkrinner | 06:33 | |
| *** sacharya has joined #openstack-nova | 06:33 | |
| *** shaohe_feng has quit IRC | 06:36 | |
| *** salv-orlando has quit IRC | 06:36 | |
| *** sacharya has quit IRC | 06:38 | |
| *** shaohe_feng has joined #openstack-nova | 06:39 | |
| *** mpavone has joined #openstack-nova | 06:43 | |
| *** mpavone has quit IRC | 06:43 | |
| *** belmoreira has joined #openstack-nova | 06:44 | |
| *** paul-carlton1 has joined #openstack-nova | 06:45 | |
| *** belmoreira has quit IRC | 06:45 | |
| *** knikolla has quit IRC | 06:45 | |
| *** shaohe_feng has quit IRC | 06:46 | |
| *** Swami_ has quit IRC | 06:47 | |
| *** shaohe_feng has joined #openstack-nova | 06:47 | |
| *** knikolla has joined #openstack-nova | 06:48 | |
| *** nelsnelson has joined #openstack-nova | 06:48 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: libvirt: Do not live migrate if tunneled with disks https://review.openstack.org/310707 | 06:48 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: api-ref: parameter verification for os-quota-sets https://review.openstack.org/314833 | 06:49 |
| *** Swami_ has joined #openstack-nova | 06:50 | |
| *** hoangcx has quit IRC | 06:50 | |
| *** belmoreira has joined #openstack-nova | 06:51 | |
| *** sridharg has joined #openstack-nova | 06:51 | |
| *** nelsnelson has quit IRC | 06:52 | |
| *** palexster has quit IRC | 06:54 | |
| *** palexster has joined #openstack-nova | 06:54 | |
| *** shaohe_feng has quit IRC | 06:57 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:57 | |
| *** knikolla has quit IRC | 06:58 | |
| *** crinkle_ has joined #openstack-nova | 06:59 | |
| *** Sukhdev has quit IRC | 06:59 | |
| *** crinkle has quit IRC | 07:00 | |
| *** StevenK has quit IRC | 07:00 | |
| *** shaohe_feng has joined #openstack-nova | 07:00 | |
| *** crinkle_ is now known as crinkle | 07:00 | |
| *** mmedvede has quit IRC | 07:00 | |
| *** anish has quit IRC | 07:01 | |
| *** knikolla has joined #openstack-nova | 07:01 | |
| *** StevenK has joined #openstack-nova | 07:01 | |
| *** anish has joined #openstack-nova | 07:02 | |
| *** lennyb has quit IRC | 07:03 | |
| *** jaosorior has joined #openstack-nova | 07:03 | |
| *** tesseract has joined #openstack-nova | 07:05 | |
| *** shaohe_feng has quit IRC | 07:07 | |
| *** shaohe_feng has joined #openstack-nova | 07:08 | |
| *** tesseract has quit IRC | 07:09 | |
| *** matrohon has joined #openstack-nova | 07:10 | |
| *** jlanoux has joined #openstack-nova | 07:11 | |
| *** gcb has quit IRC | 07:11 | |
| *** gcb has joined #openstack-nova | 07:12 | |
| *** sahid has joined #openstack-nova | 07:12 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: centralize cache options https://review.openstack.org/315341 | 07:12 |
| *** jichen has joined #openstack-nova | 07:14 | |
| *** iceyao has joined #openstack-nova | 07:15 | |
| *** lennyb has joined #openstack-nova | 07:16 | |
| *** shaohe_feng has quit IRC | 07:17 | |
| *** shaohe_feng has joined #openstack-nova | 07:18 | |
| *** jed56 has joined #openstack-nova | 07:19 | |
| *** daemontool has joined #openstack-nova | 07:19 | |
| *** lennyb has quit IRC | 07:20 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added better error messages during (un)pinning CPUs https://review.openstack.org/305654 | 07:22 |
| *** ygl has joined #openstack-nova | 07:26 | |
| ygl | hi all | 07:26 |
| ygl | i need some information | 07:26 |
| ygl | what is the difference between booting a vm from volume and booting it from image ? | 07:27 |
| *** mmedvede has joined #openstack-nova | 07:27 | |
| *** danpb has joined #openstack-nova | 07:27 | |
| *** shaohe_feng has quit IRC | 07:27 | |
| *** shaohe_feng has joined #openstack-nova | 07:28 | |
| ygl | can anyone help me here please ? | 07:28 |
| *** Dinesh_Bhor has quit IRC | 07:30 | |
| *** yamahata has joined #openstack-nova | 07:31 | |
| *** lennyb has joined #openstack-nova | 07:32 | |
| *** sacharya has joined #openstack-nova | 07:34 | |
| ygl | what is the difference between booting a vm from volume and booting it from image ? | 07:34 |
| andrearosa | ygl: if you boot from an image you get your OS image from an image server (glance), Nova contacts the image server download the image and tries to boot the VM. If you boot from a volume you need to install the image into the volume then nova won't try to contact the image server but it exxpects to find a valid image to boot from burned in the volume | 07:36 |
| *** salv-orlando has joined #openstack-nova | 07:37 | |
| ygl | andrearosa: why is it that boot fromimage takes longer time than boot from volume with huge images ? | 07:37 |
| andrearosa | ygl: for these kind of questions is better to ask in #openstack or #openstack-101 channels | 07:37 |
| *** shaohe_feng has quit IRC | 07:38 | |
| andrearosa | ygl: I'd suspect because of the download time | 07:38 |
| *** shaohe_feng has joined #openstack-nova | 07:38 | |
| ygl | andrearosa: if i spin up a vm with no volume by default, where does it store files if we create in it ? | 07:39 |
| *** sacharya has quit IRC | 07:39 | |
| *** dmk0202 has joined #openstack-nova | 07:40 | |
| *** paul-carlton1 has quit IRC | 07:41 | |
| *** jlanoux has quit IRC | 07:42 | |
| *** zenoway has joined #openstack-nova | 07:42 | |
| *** zenoway has quit IRC | 07:42 | |
| *** salv-orlando has quit IRC | 07:42 | |
| *** zenoway has joined #openstack-nova | 07:42 | |
| Thelo | andrearosa: DO you know why nova stills calls cinder-api while booting from image? | 07:43 |
| ygl | andrearosa: without a hard disk image where will nova install the image for a no volume vm launch ? | 07:46 |
| *** shaohe_feng has quit IRC | 07:48 | |
| *** shaohe_feng has joined #openstack-nova | 07:48 | |
| andrearosa | ygl: it depends on your condfiguration, how the storage is set in your env. Please move the discussion to #openstack | 07:48 |
| *** Swami_ has quit IRC | 07:49 | |
| openstackgerrit | Paul Carlton proposed openstack/nova: Add a lock() context manager to image backend https://review.openstack.org/279625 | 07:52 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Rename Raw backend to NoBacking https://review.openstack.org/279626 | 07:52 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Implementation of Libvirt Storage Pools https://review.openstack.org/314134 | 07:52 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Introduce ImageCacheLocalPool https://review.openstack.org/279669 | 07:52 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Remove fake_imagebackend.Raw and cleanup dependent tests https://review.openstack.org/267661 | 07:52 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Rename Image.check_image_exists to Image.exists() https://review.openstack.org/270998 | 07:52 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Remove deprecated option libvirt.remove_unused_kernels https://review.openstack.org/265886 | 07:52 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Only attempt to inject files if the injection disk exists https://review.openstack.org/250872 | 07:52 |
| bauzas | good morning Nova | 07:54 |
| lyarwood_ | morning | 07:54 |
| *** gsilvis has quit IRC | 07:55 | |
| *** paul-carlton1 has joined #openstack-nova | 07:55 | |
| *** gsilvis has joined #openstack-nova | 07:56 | |
| *** fragatina has quit IRC | 07:56 | |
| *** shaohe_feng has quit IRC | 07:58 | |
| *** shaohe_feng has joined #openstack-nova | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** paul-carlton1 has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:01 | |
| *** mvk_ has joined #openstack-nova | 08:04 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: api-ref: parameter verification for os-server-groups https://review.openstack.org/315394 | 08:05 |
| *** mvk has quit IRC | 08:08 | |
| *** cheneydc has quit IRC | 08:08 | |
| *** shaohe_feng has quit IRC | 08:08 | |
| *** ociuhandu has quit IRC | 08:09 | |
| *** shaohe_feng has joined #openstack-nova | 08:09 | |
| *** cheneydc has joined #openstack-nova | 08:09 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| *** jamielennox is now known as jamielennox|away | 08:10 | |
| *** gcb has quit IRC | 08:11 | |
| *** gcb has joined #openstack-nova | 08:12 | |
| *** efoley_ has joined #openstack-nova | 08:15 | |
| *** apoorv has quit IRC | 08:16 | |
| *** hoangcx has joined #openstack-nova | 08:16 | |
| *** lucas-dinner is now known as lucasagomes | 08:17 | |
| *** shaohe_feng has quit IRC | 08:19 | |
| *** shaohe_feng has joined #openstack-nova | 08:19 | |
| *** Jeffrey4l has quit IRC | 08:22 | |
| *** MarcF has joined #openstack-nova | 08:25 | |
| *** Jeffrey4l has joined #openstack-nova | 08:26 | |
| *** shaohe_feng has quit IRC | 08:29 | |
| *** shaohe_feng has joined #openstack-nova | 08:29 | |
| *** heyongli has quit IRC | 08:30 | |
| *** gjayavelu has quit IRC | 08:30 | |
| *** gjayavelu has joined #openstack-nova | 08:31 | |
| *** heyongli has joined #openstack-nova | 08:31 | |
| *** penick has quit IRC | 08:33 | |
| *** penick has joined #openstack-nova | 08:34 | |
| *** jistr has joined #openstack-nova | 08:35 | |
| *** paul-carlton1 has joined #openstack-nova | 08:38 | |
| *** yamahata has quit IRC | 08:38 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove the auto discovery of glance client version https://review.openstack.org/315419 | 08:39 |
| *** shaohe_feng has quit IRC | 08:39 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: centralize cache options https://review.openstack.org/315341 | 08:39 |
| *** shaohe_feng has joined #openstack-nova | 08:39 | |
| *** yassine_ has joined #openstack-nova | 08:40 | |
| *** paul-carlton1 has quit IRC | 08:42 | |
| *** apoorv has joined #openstack-nova | 08:42 | |
| *** jamiehan_ has joined #openstack-nova | 08:43 | |
| *** jamiehan_ has quit IRC | 08:44 | |
| *** jamie_h_ has joined #openstack-nova | 08:44 | |
| *** jamie_h_ has quit IRC | 08:48 | |
| *** jamie_h_ has joined #openstack-nova | 08:48 | |
| *** jamie_h_ has quit IRC | 08:48 | |
| *** jamie_h_ has joined #openstack-nova | 08:49 | |
| *** jamie_h has quit IRC | 08:49 | |
| *** shaohe_feng has quit IRC | 08:49 | |
| *** jamie_h_ has quit IRC | 08:49 | |
| *** jamie_h has joined #openstack-nova | 08:50 | |
| *** ljxiash has quit IRC | 08:50 | |
| *** shaohe_feng has joined #openstack-nova | 08:50 | |
| *** gcb has quit IRC | 08:50 | |
| *** ljxiash has joined #openstack-nova | 08:51 | |
| *** ljxiash has quit IRC | 08:51 | |
| *** ljxiash has joined #openstack-nova | 08:51 | |
| *** ljjjustin has quit IRC | 08:52 | |
| *** ljjjustin has joined #openstack-nova | 08:53 | |
| *** gjayavelu has quit IRC | 08:54 | |
| *** salv-orlando has joined #openstack-nova | 08:56 | |
| *** shaohe_feng has quit IRC | 09:00 | |
| mdbooth | paul-carlton paul-carlton2: Did you deliberately rebase https://review.openstack.org/#/c/250872/ btw? | 09:00 |
| *** jamie_h has quit IRC | 09:00 | |
| *** shaohe_feng has joined #openstack-nova | 09:01 | |
| *** pcaruana is now known as pcaruana|afk| | 09:01 | |
| *** salv-orlando has quit IRC | 09:02 | |
| *** efoley__ has joined #openstack-nova | 09:03 | |
| *** gcb has joined #openstack-nova | 09:04 | |
| *** openstackgerrit has quit IRC | 09:04 | |
| *** mvk_ has quit IRC | 09:04 | |
| *** openstackgerrit has joined #openstack-nova | 09:04 | |
| *** efoley_ has quit IRC | 09:06 | |
| *** shaohe_feng has quit IRC | 09:10 | |
| *** yassine_ has quit IRC | 09:11 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: use_neutron_default_nets: StrOpt ->BoolOpt https://review.openstack.org/243061 | 09:12 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: test:clean up wrong value in method CONF.set_override https://review.openstack.org/242416 | 09:12 |
| *** shaohe_feng has joined #openstack-nova | 09:12 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Add distinct-subset-shard-scheduler spec https://review.openstack.org/313519 | 09:19 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: Refactor the image transfer https://review.openstack.org/281134 | 09:19 |
| *** shaohe_feng has quit IRC | 09:20 | |
| *** shaohe_feng has joined #openstack-nova | 09:21 | |
| paul-carlton2 | mdbooth, nope I did review -d 314134 and then committed some changes but because it was dependant on your changes it rebased yours too | 09:21 |
| paul-carlton2 | hope that did not cause any issues? | 09:21 |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: remove import_opts from floating_ips section https://review.openstack.org/315443 | 09:21 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add support matrix for attach and detach interfaces https://review.openstack.org/312801 | 09:22 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add check of capabilities for detach interface https://review.openstack.org/308115 | 09:22 |
| mdbooth | paul-carlton2: I try not to rebase them unless I need to, tbh. It gives Jenkins another excuse to break :) Also I have to merge with local. | 09:23 |
| openstackgerrit | John Garbutt proposed openstack/nova: config options: centralize default flavor option https://review.openstack.org/314123 | 09:24 |
| *** sfinucan has joined #openstack-nova | 09:24 | |
| openstackgerrit | John Garbutt proposed openstack/nova: deprecate "default_flavor" config option https://review.openstack.org/314124 | 09:25 |
| paul-carlton2 | ok, I can probably rebase my change on master removing the dependency on your stuff for now, the code I'm working on is not actually dependant on your stuff yet | 09:25 |
| mdbooth | paul-carlton2: I'm sure I have some local config which tells git review not to rebase implicitly. Can't find it right now, though. | 09:25 |
| paul-carlton2 | I'll ask one of the git gurus here | 09:25 |
| mdbooth | paul-carlton2: It's ok to base it on my patches, as long as you don't rebase the series. | 09:25 |
| openstackgerrit | John Garbutt proposed openstack/nova: Config options: centralize smbfs libvirt options (16) https://review.openstack.org/309360 | 09:26 |
| openstackgerrit | John Garbutt proposed openstack/nova: Config options: centralize remotefs libvirt options (end) https://review.openstack.org/309363 | 09:26 |
| paul-carlton2 | It might be simpler to remove dependency for now, your stuff should start merging soon now the spec is approved | 09:26 |
| mdbooth | paul-carlton2: Yeah, hopefully | 09:27 |
| *** RuiChen has joined #openstack-nova | 09:27 | |
| mdbooth | Something looks borked in the series, though | 09:27 |
| paul-carlton2 | as I said the work I'm doing in this patch is almost all new stuff to do with storgage pool management | 09:27 |
| paul-carlton2 | btw, could you and maybe Diana too review my specs https://review.openstack.org/#/c/310505/ and https://review.openstack.org/#/c/310538/ | 09:29 |
| paul-carlton2 | Yep, Jenkins says This change depends on a change that failed to merge. | 09:30 |
| *** shaohe_feng has quit IRC | 09:30 | |
| *** shaohe_feng has joined #openstack-nova | 09:31 | |
| paul-carlton2 | but not clear to me what change it is referring to | 09:31 |
| openstackgerrit | John Garbutt proposed openstack/nova: Move config options from nova/api directory (5) https://review.openstack.org/314290 | 09:32 |
| paul-carlton2 | mdbooth, have a meeting now, back soon | 09:32 |
| *** lappy-rook has joined #openstack-nova | 09:32 | |
| *** yassine has joined #openstack-nova | 09:33 | |
| *** yassine is now known as Guest27233 | 09:33 | |
| paul-carlton2 | cancel that, no meeting after all | 09:33 |
| *** sacharya has joined #openstack-nova | 09:35 | |
| openstackgerrit | John Garbutt proposed openstack/nova: centralized conf: nova/network/rpcapi.py https://review.openstack.org/274210 | 09:35 |
| *** mvk_ has joined #openstack-nova | 09:35 | |
| *** jamie_h has joined #openstack-nova | 09:36 | |
| openstackgerrit | John Garbutt proposed openstack/nova: config options: centralize section "database" + "api_database" https://review.openstack.org/263804 | 09:36 |
| *** gszasz has joined #openstack-nova | 09:38 | |
| *** jamie_h has quit IRC | 09:38 | |
| *** sacharya has quit IRC | 09:39 | |
| *** shaohe_feng has quit IRC | 09:41 | |
| openstackgerrit | John Garbutt proposed openstack/nova: deprecate "file transfer" feature for Glance images https://review.openstack.org/314669 | 09:41 |
| *** shaohe_feng has joined #openstack-nova | 09:42 | |
| paul-carlton2 | johnthetubaguy , in https://review.openstack.org/#/c/308198 I'm working on allowing live migration of instances in a rescued state | 09:43 |
| paul-carlton2 | it occurs to me that changing the wrapper to allow this is ok for libvirt driver because I'm doing changes to address issues with live migration of rescued instances | 09:44 |
| paul-carlton2 | but other drivers might have isssues, any thoughts on how xenapi driver will be impacted, can it migrate rescued instances? | 09:45 |
| johnthetubaguy | paul-carlton2: yeah, this would break xenapi I think, as we leave a fake VM in the xenapi database, in a similar way to the libvirt xml file | 09:45 |
| paul-carlton2 | So I'd need to fix that too, abandon change or do something to block this on a per driver basis | 09:46 |
| johnthetubaguy | paul-carlton2: we honestly don't (yet) have a good way of handling those kind of issues, beyond the compute manage pre-live-migration check doing a virt driver capabilities check to see if its allowed in rescue, maybe? | 09:47 |
| johnthetubaguy | actually, thats not too shoddy in this case | 09:47 |
| paul-carlton2 | ironic and vmware drivers don't do live migration I think but I need to check with hyperv people too | 09:48 |
| openstackgerrit | Jens Rosenboom proposed openstack/nova: Warn about unset cache backend https://review.openstack.org/307718 | 09:48 |
| *** syfy323 has quit IRC | 09:48 | |
| *** hshiina has joined #openstack-nova | 09:49 | |
| *** abalutoiu has joined #openstack-nova | 09:49 | |
| johnthetubaguy | paul-carlton2: yeah, I know they have been re-working rescue recently, I can't remember quite how they implemented that, I don't see them online right now | 09:49 |
| *** abalutoiu has quit IRC | 09:50 | |
| *** abalutoiu has joined #openstack-nova | 09:50 | |
| paul-carlton2 | johnthetubaguy, where do I find the virt driver capabilities stuff? | 09:50 |
| *** shaohe_feng has quit IRC | 09:51 | |
| paul-carlton2 | ah, I see it line 135 of virt/driver.py | 09:51 |
| *** amotoki has joined #openstack-nova | 09:52 | |
| *** syfy323 has joined #openstack-nova | 09:52 | |
| *** shaohe_feng has joined #openstack-nova | 09:53 | |
| *** amotoki_ has quit IRC | 09:54 | |
| johnthetubaguy | paul-carlton2: cool, yeah, thats it | 09:54 |
| johnthetubaguy | paul-carlton2: I can't quite think of a direct parallel, but he important thing is you can't do the check in the API, as clearly we support multiple hypervisors in a single cloud | 09:55 |
| *** ociuhandu has joined #openstack-nova | 09:57 | |
| paul-carlton2 | johnthetubaguy, Thanks, I'll add "supports_live_migrate_rescued" to capabilities and work on figuring out how to use this in the drivers | 09:58 |
| paul-carlton2 | ah, I get it | 09:59 |
| paul-carlton2 | if not self.driver.capabilities["supports_recreate"]: | 09:59 |
| paul-carlton2 | raise exception.InstanceRecreateNotSupported | 09:59 |
| paul-carlton2 | in the compute manager | 10:00 |
| *** kmartin has joined #openstack-nova | 10:00 | |
| paul-carlton2 | i.e. I check for supports_live_migrate_rescued in live_migration method | 10:00 |
| *** shaohe_feng has quit IRC | 10:01 | |
| bauzas | jaypipes: when you're around, please ping me, I have a question for you about some related BP I'm working on (check-destinations) | 10:01 |
| *** shaohe_feng has joined #openstack-nova | 10:02 | |
| *** cheneydc has quit IRC | 10:02 | |
| bauzas | jaypipes: given our discussion yesterday, I wonder if I should rather change https://review.openstack.org/#/c/312757/2/nova/objects/request_spec.py@58 to be a nested Service object | 10:02 |
| *** rmart04 has joined #openstack-nova | 10:03 | |
| bauzas | given also the silly 1:N relationship between a service and a compute node | 10:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: virt/hardware: Check for threads when "required" https://review.openstack.org/285232 | 10:04 |
| openstackgerrit | Sudipta Biswas proposed openstack/nova: Remove the auto discovery of glance client version https://review.openstack.org/315419 | 10:05 |
| bauzas | johnthetubaguy: in case you want to provide your own opinion ^ | 10:05 |
| *** jlanoux has joined #openstack-nova | 10:05 | |
| *** kmartin has quit IRC | 10:08 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 10:10 |
| *** shaohe_feng has quit IRC | 10:11 | |
| *** shaohe_feng has joined #openstack-nova | 10:12 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Remove deprecated "memcached_server" in Default section https://review.openstack.org/315463 | 10:15 |
| *** apoorv has quit IRC | 10:17 | |
| *** __szilard_cserey has joined #openstack-nova | 10:21 | |
| *** apoorv has joined #openstack-nova | 10:21 | |
| *** shaohe_feng has quit IRC | 10:22 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: centralize cache options https://review.openstack.org/315341 | 10:22 |
| *** shaohe_feng has joined #openstack-nova | 10:22 | |
| *** hoangcx has quit IRC | 10:23 | |
| *** haukebruno has joined #openstack-nova | 10:23 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/nova: libvirt: add nova volume driver for vzstorage https://review.openstack.org/190843 | 10:24 |
| *** efoley__ has quit IRC | 10:24 | |
| *** haukebruno has quit IRC | 10:27 | |
| *** haukebruno has joined #openstack-nova | 10:27 | |
| *** EinstCrazy has quit IRC | 10:30 | |
| *** EinstCrazy has joined #openstack-nova | 10:31 | |
| *** jinxing has quit IRC | 10:32 | |
| *** shaohe_feng has quit IRC | 10:32 | |
| *** jamie_h has joined #openstack-nova | 10:32 | |
| *** jamie_h has quit IRC | 10:32 | |
| *** apoorv has quit IRC | 10:32 | |
| *** jamie_h has joined #openstack-nova | 10:33 | |
| *** efoley__ has joined #openstack-nova | 10:34 | |
| *** sdague has joined #openstack-nova | 10:34 | |
| *** shaohe_feng has joined #openstack-nova | 10:35 | |
| *** EinstCrazy has quit IRC | 10:35 | |
| *** kolesovdv has joined #openstack-nova | 10:36 | |
| *** cdent has joined #openstack-nova | 10:36 | |
| bauzas | johnthetubaguy: correct me if I'm wrong but Ironic and Hyper-V (both using multi-nodes per service) don't have evacuate and live-migrate ? | 10:40 |
| bauzas | http://docs.openstack.org/developer/nova/support-matrix.html#operation_live_migrate tells me that for Ironic | 10:41 |
| bauzas | same goes for evacuate... http://docs.openstack.org/developer/nova/support-matrix.html#operation_evacuate | 10:42 |
| *** mah_ has joined #openstack-nova | 10:42 | |
| *** shaohe_feng has quit IRC | 10:42 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add Allocation and AllocationList objects https://review.openstack.org/282442 | 10:42 |
| openstackgerrit | Chris Dent proposed openstack/nova: Extend ResourceProvider to have resources and aggregates https://review.openstack.org/284963 | 10:42 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a get_by_uuid for aggregates https://review.openstack.org/284895 | 10:42 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP Correct the Inventory object https://review.openstack.org/315288 | 10:42 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 10:42 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 10:42 |
| *** alexpilotti has joined #openstack-nova | 10:42 | |
| mah_ | any help please, I have the vms not assigned by IPs .. after stacking directly it works fine for some minutes but later it doesnot work fine :S | 10:43 |
| *** shaohe_feng has joined #openstack-nova | 10:43 | |
| alexpilotti | bauzas: Hyper-V has live migrate | 10:43 |
| alexpilotti | bauzas: evacuate is somewhere on the review queue | 10:44 |
| *** josecastroleon has quit IRC | 10:47 | |
| bauzas | alexpilotti: okay, my question is then, does Hyper-V supports multi-compute nodes per compute service ? | 10:48 |
| *** smatzek has joined #openstack-nova | 10:48 | |
| bauzas | AFAICS, no | 10:48 |
| *** gcb has quit IRC | 10:48 | |
| alexpilotti | bauzas: nova-compute runs on each Hyper-V node | 10:51 |
| bauzas | alexpilotti: nvm, I have my answer https://github.com/openstack/nova/blob/master/nova/virt/hyperv/driver.py#L178-L179 | 10:51 |
| *** shaohe_feng has quit IRC | 10:52 | |
| *** shaohe_feng has joined #openstack-nova | 10:53 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova-specs: VMware: add support for live migration https://review.openstack.org/299207 | 10:53 |
| *** abalutoiu_ has joined #openstack-nova | 10:54 | |
| *** josecastroleon has joined #openstack-nova | 10:55 | |
| *** abalutoiu has quit IRC | 10:58 | |
| *** rtheis has joined #openstack-nova | 10:59 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: utils: Move is_volume_backed_instance into compute_utils https://review.openstack.org/281444 | 11:00 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: WIP - libvirt: Implement stable device rescues https://review.openstack.org/270289 | 11:00 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Allow instances with bdms to be rescued by capable virt drivers https://review.openstack.org/270288 | 11:00 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Add boot ordering to individual disks https://review.openstack.org/270286 | 11:00 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: virt: Provide the instance bdm during a rescue https://review.openstack.org/270287 | 11:00 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: image_meta: Add hw_rescue_device and hw_rescue_bus https://review.openstack.org/270285 | 11:00 |
| *** sree has quit IRC | 11:00 | |
| *** sree has joined #openstack-nova | 11:01 | |
| *** gcb_ has joined #openstack-nova | 11:01 | |
| *** shaohe_feng has quit IRC | 11:03 | |
| *** shaohe_feng has joined #openstack-nova | 11:03 | |
| *** abalutoiu__ has joined #openstack-nova | 11:03 | |
| *** mah_ has left #openstack-nova | 11:04 | |
| *** apoorv has joined #openstack-nova | 11:05 | |
| *** ygl has quit IRC | 11:05 | |
| *** sree has quit IRC | 11:05 | |
| *** amotoki has quit IRC | 11:06 | |
| *** claudiub has joined #openstack-nova | 11:06 | |
| *** abalutoiu_ has quit IRC | 11:06 | |
| sdague | johnthetubaguy: on https://review.openstack.org/#/c/314198/2/api-ref/source/os-networks.inc what does the host value really mean? Is that the host the network node is on? | 11:07 |
| johnthetubaguy | sdague: honestly, I was wondering what that meant as well | 11:07 |
| sdague | as far as I can tell, it no longer does anything other than modify data models | 11:07 |
| sdague | it's not plumbed to neutron | 11:08 |
| sdague | https://github.com/openstack/nova/blob/25ac5a2756b36efa2da7d183a97200f0016c355c/nova/network/api.py#L352-L370 is where it stops | 11:08 |
| johnthetubaguy | yeah, its nova-net only, but I don't remember why we have a mapping to host | 11:08 |
| johnthetubaguy | I guess the vlan might be restricted to some set of hosts, in theory | 11:08 |
| sdague | host is a foreign key though | 11:08 |
| sdague | it's a singleton | 11:08 |
| sdague | oh, you think you might have multiple network nodes and some have only some vlans? | 11:09 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: handle reserved pages size https://review.openstack.org/292500 | 11:09 |
| openstackgerrit | sahid proposed openstack/nova: virt: reserved number of mempages on compute host https://review.openstack.org/292499 | 11:09 |
| *** amotoki has joined #openstack-nova | 11:10 | |
| johnthetubaguy | sdague: yeah, something like that | 11:10 |
| sdague | given the state of nova network, I'm not sure that we should spend a ton of brain power getting this bit right. What if I update this patch and just mark all 3 of these as warning, these methods are historical and should not be used | 11:10 |
| johnthetubaguy | https://github.com/openstack/nova/blob/master/nova/network/manager.py#L225 | 11:11 |
| johnthetubaguy | sdague: yeah, some kinda nova-network only warning | 11:11 |
| *** hamzy has quit IRC | 11:12 | |
| johnthetubaguy | hmm, so much code to read to see if this still does anything, yuck | 11:12 |
| *** shaohe_feng has quit IRC | 11:13 | |
| sdague | johnthetubaguy: right, we should probably mark our losses on some of this | 11:13 |
| *** shaohe_feng has joined #openstack-nova | 11:13 | |
| johnthetubaguy | sdague: yeah, its going away soon, I guess thats the key bit! | 11:13 |
| sdague | because having servers and flavors right is way more important than this :) | 11:14 |
| *** amotoki has quit IRC | 11:14 | |
| johnthetubaguy | sdague: +100 | 11:14 |
| *** dansmith has quit IRC | 11:14 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: numa: Fixes NUMA topology related unit tests https://review.openstack.org/278835 | 11:15 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: libvirt: fixes python 3 related unit tests https://review.openstack.org/278834 | 11:15 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 unit tests https://review.openstack.org/278810 | 11:15 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes python 3 urllib quote / unquote usage https://review.openstack.org/278809 | 11:15 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop https://review.openstack.org/174854 | 11:16 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 11:16 |
| jroll | this is a missing context thing, right? http://logs.openstack.org/17/314917/2/check/gate-tempest-dsvm-ironic-pxe_ssh/71d0687/logs/screen-n-cpu.txt.gz#_2016-05-12_10_09_27_393 | 11:17 |
| jroll | apparently https://review.openstack.org/#/c/312750/ broke us here somehow | 11:17 |
| *** dansmith has joined #openstack-nova | 11:17 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Method verification of os-security-group-default-rules https://review.openstack.org/315257 | 11:17 |
| *** dansmith is now known as Guest51926 | 11:17 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Complete method verification of os-networks https://review.openstack.org/314198 | 11:20 |
| johnthetubaguy | jroll: maybe try the thing mriedem suggested in that patch, I suspect that will pre-load the key and fix your issue | 11:22 |
| johnthetubaguy | jroll: I am guessing you end up caching an object in a way it gets disconnected from the context, and it has data missing? | 11:23 |
| jroll | johnthetubaguy: yeah, looping call launches a thread, was thinking just add an update_store() there | 11:23 |
| *** shaohe_feng has quit IRC | 11:23 | |
| jroll | old trick bearhands taught me | 11:23 |
| jroll | I can also propose that thing matt had if you like | 11:23 |
| johnthetubaguy | jroll: heh, that could do that, but it sounds like a lazy load you don't want | 11:24 |
| jroll | right | 11:24 |
| jroll | should probably do both | 11:24 |
| johnthetubaguy | but looking more, I am not sure matt's suggestion will cut it | 11:24 |
| *** shaohe_feng has joined #openstack-nova | 11:24 | |
| johnthetubaguy | yeah, both might make sense, see which fixes it first | 11:24 |
| jroll | I mean, we should have the correct context there anyway, so this doesn't happen again and such | 11:24 |
| *** cdent has quit IRC | 11:24 | |
| johnthetubaguy | oh right, so I see "Lazy-loading 'keypairs' on Instance uuid" | 11:24 |
| johnthetubaguy | that would do it | 11:24 |
| jroll | we call refresh() in the thread | 11:24 |
| johnthetubaguy | but yeah, I like the extra defense | 11:25 |
| jroll | but eager loading keypairs will also fix this case and be better | 11:25 |
| jroll | cool thanks | 11:25 |
| openstackgerrit | Sean Dague proposed openstack/nova: Complete method verification of os-networks https://review.openstack.org/314198 | 11:25 |
| johnthetubaguy | jroll: np | 11:25 |
| * johnthetubaguy goes for some food | 11:25 | |
| sdague | johnthetubaguy: how about that ^^^ | 11:25 |
| sdague | johnthetubaguy: damn it, check that out first :) | 11:25 |
| johnthetubaguy | heh, looking... | 11:26 |
| sdague | as I was hoping to get it landed before rbradfor comes in, so he can finish up the next stages on those files | 11:26 |
| johnthetubaguy | sdague: micro nit, I think a header got too long | 11:27 |
| johnthetubaguy | sdague: but generally, yeah, I like that | 11:27 |
| sdague | johnthetubaguy: ah, let me run local tests | 11:28 |
| sdague | well if you want to stick a +2 on it, I can fix the nit, then approve it in | 11:28 |
| sdague | alex_xu: already had a +2 on the earlier version | 11:28 |
| openstackgerrit | John Garbutt proposed openstack/nova: Complete method verification of os-networks https://review.openstack.org/314198 | 11:28 |
| sdague | or that :) | 11:28 |
| *** jaosorior has quit IRC | 11:28 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova: Ironic: update context in looping call https://review.openstack.org/315491 | 11:29 |
| sdague | yeh, I missed hitting `ctrl =` in one place I guess :) | 11:29 |
| *** jaosorior has joined #openstack-nova | 11:29 | |
| johnthetubaguy | sdague: looks good now :) | 11:29 |
| johnthetubaguy | sdague: I think thats a good approach, spend our time on the non-deprecated stuff, at least to start with! | 11:30 |
| sdague | yeh | 11:30 |
| * johnthetubaguy bravely runs away | 11:30 | |
| *** salv-orlando has joined #openstack-nova | 11:30 | |
| sdague | :) | 11:30 |
| *** apoorv has quit IRC | 11:30 | |
| tojuvone | Hi, guys. Does not look for me, I am sick, Fever what ever.. Anyhow would like to share something. | 11:31 |
| *** apoorv has joined #openstack-nova | 11:31 | |
| *** diogogmt has joined #openstack-nova | 11:33 | |
| *** shaohe_feng has quit IRC | 11:33 | |
| *** haukebruno has quit IRC | 11:34 | |
| tojuvone | sdague, dansmith: As of discussion so far, I think to abandon this: https://review.openstack.org/276671 | 11:34 |
| *** sudipto has quit IRC | 11:34 | |
| sdague | tojuvone: ok, hope you feel better soon | 11:35 |
| *** salv-orlando has quit IRC | 11:35 | |
| tojuvone | sdague: And for this I'll do something like commented: https://gerrit.opnfv.org/gerrit/13825 | 11:35 |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova: Eager load keypairs in instance metadata https://review.openstack.org/315495 | 11:35 |
| *** sacharya has joined #openstack-nova | 11:35 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova: Ironic: update context in looping call https://review.openstack.org/315491 | 11:36 |
| jroll | johnthetubaguy: ^ there's both | 11:36 |
| tojuvone | sdague: Thanks :) Cannot wait to get well to do something! | 11:37 |
| sdague | tojuvone: ok, so instead you are just listening for notifications on the changes? | 11:37 |
| sdague | trying to make sense of the opnvf review | 11:37 |
| sdague | it feels like a notification based model is much better in these cases anyway, the polling is going to be expensive even if the host status was around | 11:38 |
| *** shaohe_feng has joined #openstack-nova | 11:38 | |
| *** gcb_ has quit IRC | 11:39 | |
| *** ivase_ has quit IRC | 11:40 | |
| *** sacharya has quit IRC | 11:40 | |
| *** ducttape_ has joined #openstack-nova | 11:41 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/nova: libvirt: add nova volume driver for vzstorage https://review.openstack.org/190843 | 11:42 |
| *** ZZelle has joined #openstack-nova | 11:42 | |
| tojuvone | Well one would need perhaps still something like maintenance_reason. I was thinking API change to set it and another to expose to server(s) | 11:43 |
| tojuvone | but | 11:43 |
| *** shaohe_feng has quit IRC | 11:44 | |
| tojuvone | That is surely that one can get all the "state" information related to server in one place | 11:44 |
| tojuvone | then again like you said, polling is expensive | 11:44 |
| *** shaohe_feng has joined #openstack-nova | 11:44 | |
| tojuvone | so surely notification for any changes to maintenance_reason for server owner would do the same. | 11:45 |
| *** iceyao_ has joined #openstack-nova | 11:46 | |
| *** iceyao has quit IRC | 11:46 | |
| *** baoli has joined #openstack-nova | 11:46 | |
| *** baoli has quit IRC | 11:46 | |
| *** baoli has joined #openstack-nova | 11:47 | |
| tojuvone | services is already exposed to server, so new param in there wouldn't cost so much | 11:47 |
| tojuvone | and also service is exposed to hypervisors API, so jsut one more param to expose there | 11:49 |
| *** baoli_ has joined #openstack-nova | 11:49 | |
| sdague | tojuvone: yep, sounds reasonable to put that in notifications instead, I agree, it's pretty cheap to go grab that other data at that point | 11:51 |
| *** gcb_ has joined #openstack-nova | 11:51 | |
| *** baoli has quit IRC | 11:52 | |
| *** shaohe_feng has quit IRC | 11:54 | |
| *** hamzy has joined #openstack-nova | 11:54 | |
| *** shaohe_feng has joined #openstack-nova | 11:54 | |
| tojuvone | sdaque: so if try something like having maintenance_reason + notification. | 11:54 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests https://review.openstack.org/300262 | 11:56 |
| *** ttx has quit IRC | 11:56 | |
| *** salv-orlando has joined #openstack-nova | 11:59 | |
| *** alexpilotti has quit IRC | 12:01 | |
| *** fragatina has joined #openstack-nova | 12:01 | |
| *** ducttape_ has quit IRC | 12:02 | |
| *** alexpilotti has joined #openstack-nova | 12:03 | |
| *** shaohe_feng has quit IRC | 12:04 | |
| *** shaohe_feng has joined #openstack-nova | 12:05 | |
| *** ttx has joined #openstack-nova | 12:07 | |
| *** raildo-afk is now known as raildo | 12:10 | |
| *** efried has joined #openstack-nova | 12:13 | |
| *** shaohe_feng has quit IRC | 12:14 | |
| *** shaohe_feng has joined #openstack-nova | 12:15 | |
| *** kengo_sakai has quit IRC | 12:15 | |
| *** winston-d_ has quit IRC | 12:15 | |
| *** rodrigods has quit IRC | 12:15 | |
| *** rodrigods has joined #openstack-nova | 12:16 | |
| *** pradk has joined #openstack-nova | 12:17 | |
| *** ccarmack has joined #openstack-nova | 12:18 | |
| *** RuiChen has quit IRC | 12:22 | |
| *** gcb_ has quit IRC | 12:23 | |
| *** ccarmack has quit IRC | 12:23 | |
| *** apoorv has quit IRC | 12:24 | |
| *** ljxiash has quit IRC | 12:24 | |
| Yingxin | jaypipes: I'm wondering if the resource provider related objects are represented correctly in the logic layer. | 12:24 |
| openstackgerrit | Paul Carlton proposed openstack/nova-specs: Automatic Live Migration Completion https://review.openstack.org/306561 | 12:24 |
| Yingxin | jaypipes: see my comments in https://review.openstack.org/#/c/284963/ | 12:24 |
| *** shaohe_feng has quit IRC | 12:25 | |
| *** shaohe_feng has joined #openstack-nova | 12:25 | |
| *** aysyd has joined #openstack-nova | 12:27 | |
| *** [1]Thelo has joined #openstack-nova | 12:27 | |
| *** Thelo has quit IRC | 12:30 | |
| *** [1]Thelo is now known as Thelo | 12:30 | |
| openstackgerrit | Paul Carlton proposed openstack/nova-specs: Live Migration of Rescued Instances https://review.openstack.org/307131 | 12:30 |
| *** houming has quit IRC | 12:30 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: WIP api-ref: parameter verification for os-aggregates https://review.openstack.org/315517 | 12:31 |
| *** efoley__ has quit IRC | 12:32 | |
| *** zz_dimtruck is now known as dimtruck | 12:33 | |
| *** shaohe_feng has quit IRC | 12:35 | |
| *** shaohe_feng has joined #openstack-nova | 12:35 | |
| *** snikitin has joined #openstack-nova | 12:38 | |
| *** nelsnelson has joined #openstack-nova | 12:39 | |
| *** haukebruno has joined #openstack-nova | 12:42 | |
| *** nelsnelson has quit IRC | 12:43 | |
| *** shaohe_feng has quit IRC | 12:45 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: api-ref: parameter verification for os-quota-sets https://review.openstack.org/314833 | 12:46 |
| *** shaohe_feng has joined #openstack-nova | 12:46 | |
| *** sree has joined #openstack-nova | 12:46 | |
| *** andymaier has joined #openstack-nova | 12:47 | |
| *** rfolco has joined #openstack-nova | 12:48 | |
| *** andymaier has quit IRC | 12:49 | |
| *** sree has quit IRC | 12:50 | |
| *** abalutoiu__ has quit IRC | 12:53 | |
| *** shaohe_feng has quit IRC | 12:55 | |
| *** shaohe_feng has joined #openstack-nova | 12:55 | |
| *** sree has joined #openstack-nova | 13:00 | |
| *** pcaruana|afk| is now known as pcaruana | 13:01 | |
| *** haukebruno has quit IRC | 13:02 | |
| *** pwnall has joined #openstack-nova | 13:02 | |
| *** alexpilotti has quit IRC | 13:02 | |
| *** abalutoiu has joined #openstack-nova | 13:03 | |
| *** alexpilotti has joined #openstack-nova | 13:03 | |
| *** nelsnelson has joined #openstack-nova | 13:04 | |
| *** mriedem has joined #openstack-nova | 13:04 | |
| *** coreycb has quit IRC | 13:04 | |
| *** rlrossit has joined #openstack-nova | 13:04 | |
| *** edmondsw has joined #openstack-nova | 13:04 | |
| *** sree__ has joined #openstack-nova | 13:05 | |
| *** crito__ has joined #openstack-nova | 13:05 | |
| *** shaohe_feng has quit IRC | 13:06 | |
| *** shaohe_feng has joined #openstack-nova | 13:06 | |
| *** andymaier has joined #openstack-nova | 13:06 | |
| *** josecastroleon has quit IRC | 13:07 | |
| *** sfinucan has quit IRC | 13:08 | |
| *** sree has quit IRC | 13:08 | |
| *** tbachman has quit IRC | 13:08 | |
| *** sfinucan has joined #openstack-nova | 13:08 | |
| *** pwnall has quit IRC | 13:09 | |
| *** rmart04 has quit IRC | 13:09 | |
| *** josecastroleon has joined #openstack-nova | 13:09 | |
| *** pwnall has joined #openstack-nova | 13:09 | |
| *** ctrath has joined #openstack-nova | 13:10 | |
| openstackgerrit | Clenimar Filemon proposed openstack/nova: Use Identity v3 params when connecting to Ironic https://review.openstack.org/300154 | 13:10 |
| *** mdrabe has joined #openstack-nova | 13:10 | |
| *** rmart04 has joined #openstack-nova | 13:11 | |
| *** mrkz has joined #openstack-nova | 13:11 | |
| *** tbachman has joined #openstack-nova | 13:12 | |
| *** abalutoiu has quit IRC | 13:13 | |
| *** abalutoiu has joined #openstack-nova | 13:14 | |
| *** links has quit IRC | 13:14 | |
| *** shaohe_feng has quit IRC | 13:16 | |
| *** shaohe_feng has joined #openstack-nova | 13:17 | |
| mriedem | no dansmith yet? what's the deal. | 13:17 |
| *** xyang1 has joined #openstack-nova | 13:18 | |
| *** ljxiash has joined #openstack-nova | 13:18 | |
| *** toure_ is now known as toure | 13:20 | |
| *** toure is now known as toure_ | 13:21 | |
| *** amotoki has joined #openstack-nova | 13:21 | |
| openstackgerrit | jichenjc proposed openstack/nova: api-ref keypair update (os-keypairs.inc) https://review.openstack.org/314502 | 13:22 |
| jaypipes | Yingxin: I will take a look at those comments shortly. trying to get cdent's revision of the generic-resource-pools spec reviewed :) | 13:22 |
| *** ljxiash has quit IRC | 13:22 | |
| *** atuvenie_ has joined #openstack-nova | 13:23 | |
| mriedem | jaypipes: it's on my list for this morning | 13:23 |
| Yingxin | jaypipes: np, I'm actually still working on that | 13:23 |
| jaypipes | mriedem, bauzas: I managed to extricate the (host, node) mess from the compute manager / resource tracker last night. | 13:24 |
| jroll | mriedem: are there any intel rates or whatever yet for portland? | 13:24 |
| jroll | mriedem: also dan is hiding as Guestsomething | 13:24 |
| jaypipes | jroll: dansmith said he'll put up to 300 people up at his place. | 13:24 |
| bauzas | jaypipes: saw my ping earlier ? | 13:24 |
| jroll | jaypipes: perfect | 13:24 |
| jaypipes | bauzas: no, sorry... could you repeat? | 13:24 |
| mriedem | jroll: not that i know of yet, i need to send them the rsvp info | 13:24 |
| johnthetubaguy | gibi did you see the comments in your versioned notifications spec? | 13:25 |
| jroll | mriedem: right on, thanks | 13:25 |
| bauzas | jaypipes: so, re: our yesterday's discussion, I was thinking of https://review.openstack.org/#/c/312757/2/nova/objects/request_spec.py@58 to be changed into a Service object | 13:25 |
| jaypipes | bauzas: looking at it now... | 13:25 |
| *** Guest51926 is now known as dansmith | 13:26 | |
| mriedem | Guest51926: want to get this and the things below it? https://review.openstack.org/#/c/315187/ the unskipped tempest test is passing | 13:26 |
| *** shaohe_feng has quit IRC | 13:26 | |
| mriedem | dansmith: ^ | 13:26 |
| mriedem | then i'll work on the backports | 13:26 |
| jaypipes | bauzas: is requested_destination the new equivalent of force_hosts=[host]? | 13:26 |
| *** shaohe_feng has joined #openstack-nova | 13:26 | |
| dansmith | mriedem: yeah | 13:27 |
| *** jichen has quit IRC | 13:27 | |
| jaypipes | bauzas: is requested_destination the new equivalent of force_hosts=[host]? | 13:29 |
| bauzas | jaypipes: sorry, I'm on the call | 13:30 |
| *** ivase has joined #openstack-nova | 13:30 | |
| jaypipes | bauzas: np | 13:31 |
| openstackgerrit | Sharat Sharma proposed openstack/nova: Adding "security_group_default_rules" parameter https://review.openstack.org/315557 | 13:31 |
| *** smatzek has quit IRC | 13:31 | |
| *** iceyao_ has quit IRC | 13:31 | |
| mriedem | jaypipes: you'll be happy to know you're still the only vegan that rsvp'ed for the midcycle | 13:31 |
| jaypipes | mriedem: sweet. I'll bring my own meat. | 13:32 |
| sdague | I thought jaypipes was going with the sushi only option | 13:32 |
| mriedem | is that an option? | 13:32 |
| *** ccarmack has joined #openstack-nova | 13:32 | |
| mriedem | because i want it | 13:32 |
| *** fragatina has quit IRC | 13:32 | |
| mriedem | dansmith: what is abnormal poultry? | 13:33 |
| mriedem | quail? | 13:33 |
| mriedem | osterich? | 13:34 |
| dansmith | mriedem: yes, those | 13:34 |
| *** dimtruck is now known as zz_dimtruck | 13:34 | |
| *** zz_dimtruck is now known as dimtruck | 13:34 | |
| jaypipes | sdague: only if the sushi has eel on it. | 13:35 |
| jaypipes | sdague: a lovely sight I had to watch my wife eat last night. | 13:36 |
| johnthetubaguy | dansmith: just wanted to nudge on the neutron refactor spec: https://review.openstack.org/#/c/313001/ | 13:36 |
| *** sacharya has joined #openstack-nova | 13:36 | |
| sahid | mriedem, jaypipes any chance to make this https://review.openstack.org/#/c/312014/ - it's related to network | 13:36 |
| dansmith | johnthetubaguy: ah, thanks I'll look in a sec | 13:36 |
| *** shaohe_feng has quit IRC | 13:36 | |
| bauzas | jaypipes: so, to answer your question, nope, requested_dest != force_hosts | 13:37 |
| bauzas | because force_hosts doesn't check the scheduler filters | 13:37 |
| *** shaohe_feng has joined #openstack-nova | 13:37 | |
| bauzas | jaypipes: http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/check-destination-on-migrations-newton.html | 13:37 |
| mriedem | sahid: yeah i just need to get back to reviewing that one | 13:38 |
| sahid | mriedem: cool i tried to address all your comments | 13:38 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add a RequestSpec generation migration script https://review.openstack.org/302828 | 13:40 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Verify requested_destination in the scheduler https://review.openstack.org/314509 | 13:40 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Resize API operation passing down original RequestSpec https://review.openstack.org/312009 | 13:40 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add requested_destination field to RequestSpec https://review.openstack.org/312757 | 13:40 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Cold migrate using the RequestSpec object https://review.openstack.org/284974 | 13:40 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: API changes for verifying the scheduler when migrating https://review.openstack.org/315572 | 13:40 |
| dansmith | mriedem: so are you going to backport all those cleanups before the actual fix or what? | 13:40 |
| mriedem | dansmith: hells naw | 13:40 |
| mriedem | just the fix | 13:40 |
| *** fragatina has joined #openstack-nova | 13:41 | |
| openstackgerrit | Andrey Pavlov proposed openstack/nova-specs: Added a spec for disable-termination and shutdown-termination flags https://review.openstack.org/234219 | 13:41 |
| jaypipes | bauzas: is this blueprint/spec already approved as-is? | 13:41 |
| mriedem | dansmith: since they are new tests in the fix patch, backporting shouldn't be an issue | 13:41 |
| bauzas | jaypipes: yup | 13:41 |
| jaypipes | sahid: I'll add it to my queue :) | 13:41 |
| *** sacharya has quit IRC | 13:41 | |
| jaypipes | bauzas: k, thx for the info... lemme read this through and get back to you on it. | 13:41 |
| sahid | jaypipes: yes i understand, thanks | 13:42 |
| jaypipes | bauzas: your question above was about a Service object though, right? How does that have to do with the requested_destination thing? | 13:42 |
| dansmith | mriedem: ah, I see yeah, I misread, cool | 13:42 |
| *** efried has quit IRC | 13:42 | |
| bauzas | jaypipes: because the admin wants to provide a destination | 13:42 |
| bauzas | jaypipes: there are 2 possibilities | 13:42 |
| bauzas | given it's a string he gives | 13:42 |
| jaypipes | bauzas: right, but the service has nothing to do with the destination. the compute node is the destination, not the service. | 13:43 |
| bauzas | either it's just an host name, and then it means that we need to figure out which nodes he wants to reach | 13:43 |
| *** BjoernT has joined #openstack-nova | 13:43 | |
| bauzas | okay, so I'll leave with my proposal | 13:43 |
| jaypipes | bauzas: sorry, what am I missing here? is this just terminology? or am I missing something obvious? :) | 13:43 |
| bauzas | I just made a clear message that since Ironic multi-nodes doesn't provide evacuate/live-mig, then we'll figure out the ComputeNode object related to the host name | 13:44 |
| *** dimtruck is now known as zz_dimtruck | 13:44 | |
| bauzas | jaypipes: just that because our silly relationship, giving you an host name doesn't really provide the destination | 13:44 |
| jaypipes | bauzas: gotcha. ok, lemme finish reading this and we'll discuss in a bit, yeh? | 13:44 |
| bauzas | k | 13:44 |
| bauzas | jaypipes: patches are up, you can take those to understand | 13:45 |
| jaypipes | will do. | 13:45 |
| jroll | are these 401 things a known issue? can't find anything in e-r: http://logs.openstack.org/95/315495/1/check/gate-tempest-dsvm-full/930526b/console.html | 13:45 |
| *** slaweq has joined #openstack-nova | 13:45 | |
| bauzas | jaypipes: https://review.openstack.org/#/q/topic:bp/check-destination-on-migrations | 13:45 |
| jaypipes | cdent is first though :) | 13:45 |
| jaypipes | bauzas: cheers | 13:45 |
| mriedem | jroll: yes | 13:45 |
| mriedem | jroll: since 4/29 when fernet was made the default token provider in devstack | 13:45 |
| mriedem | all hell broke loose | 13:45 |
| bauzas | jaypipes: tbc, I want to be done very early with that in this cycle, so I could bite into your list of BPs to help you, guys :) | 13:45 |
| mriedem | http://status.openstack.org//elastic-recheck/index.html#1577558 | 13:46 |
| mriedem | http://status.openstack.org//elastic-recheck/index.html#1578866 | 13:46 |
| jroll | mriedem: aha, thanks. /me rechecks | 13:46 |
| *** shaohe_feng has quit IRC | 13:47 | |
| *** erhudy has joined #openstack-nova | 13:47 | |
| *** eharney has joined #openstack-nova | 13:47 | |
| *** rmart04 has quit IRC | 13:48 | |
| *** BjoernT has quit IRC | 13:49 | |
| jroll | johnthetubaguy: alaski: as a note, those patches I had to unbreak ironic didn't do the job :/ | 13:49 |
| sdague | mriedem: I'm building something to check the sort order of the parameters file, but, we're going to need to keep it non enforcing until we want to ruin everyone's outstanding patches | 13:49 |
| *** shaohe_feng has joined #openstack-nova | 13:50 | |
| jroll | alaski: notably the eager load doesn't seem to actually load it http://logs.openstack.org/95/315495/1/check/gate-tempest-dsvm-ironic-pxe_ipa-nv/cb93016/logs/screen-n-cpu.txt.gz#_2016-05-12_12_17_53_806 | 13:50 |
| mriedem | sdague: s/until/unless/ ? | 13:50 |
| mriedem | dansmith: oh fun, something in nic's test must be goofing up global state and causing other tests to fail in stable/mitaka | 13:51 |
| dansmith | mriedem: really? | 13:51 |
| mriedem | or, these tests are just different between mitaka and newton | 13:51 |
| mriedem | looking | 13:51 |
| dansmith | mriedem: you mean in here? https://review.openstack.org/#/c/315187/8/nova/tests/unit/virt/libvirt/test_driver.py | 13:51 |
| sdague | mriedem: yeh | 13:52 |
| mriedem | dansmith: no actually https://review.openstack.org/#/c/315241/5/nova/tests/unit/virt/libvirt/test_driver.py and _test_finish_revert_migration_after_crash | 13:52 |
| mriedem | i'll just have to add a mock in there in stable/mitaka | 13:52 |
| mriedem | sdague: i'm fine with people having to fix their existing unapproved patches | 13:52 |
| johnthetubaguy | jroll: dang it, must be some other call path we are hitting there, I guess | 13:53 |
| mriedem | dansmith: because in mitaka; | 13:53 |
| mriedem | imagebackend.Backend.image(mox.IgnoreArg(), 'disk').AndReturn( | 13:53 |
| mriedem | fake_imagebackend.Raw()) | 13:53 |
| mriedem | and in newton: | 13:53 |
| mriedem | drvr.image_backend = mock.Mock() | 13:53 |
| mriedem | drvr.image_backend.image.return_value = drvr.image_backend | 13:53 |
| mriedem | the fake Raw() mock in mitaka doesn't have some methods on it, which pass just fine in newton because it's using a mock now | 13:54 |
| mriedem | tarnation | 13:54 |
| mriedem | fixable, but annoying | 13:54 |
| dansmith | mriedem: oh I see | 13:54 |
| jroll | johnthetubaguy: yeah, the more curious one is the context thing, I've no clue why that doesn't solve it | 13:54 |
| *** smatzek has joined #openstack-nova | 13:54 | |
| *** atuvenie_ has quit IRC | 13:54 | |
| dansmith | jroll: I have a meeting a few minutes, but I can help look when I'm done if you're still broken | 13:55 |
| jroll | dansmith: I'm going to step away for a bit myself, so that'd be awesome, thanks | 13:55 |
| dansmith | okay | 13:55 |
| jroll | dansmith: for context if you get bored in the meantime: 315491 and 315495 | 13:56 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Rename Raw backend to NoBacking https://review.openstack.org/279626 | 13:56 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove fake_imagebackend.Raw and cleanup dependent tests https://review.openstack.org/267661 | 13:56 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Rename Image.check_image_exists to Image.exists() https://review.openstack.org/270998 | 13:56 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove deprecated option libvirt.remove_unused_kernels https://review.openstack.org/265886 | 13:56 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Only attempt to inject files if the injection disk exists https://review.openstack.org/250872 | 13:56 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Remove unused arguments to images.fetch and images.fetch_to_raw https://review.openstack.org/315582 | 13:56 |
| *** andymaier has quit IRC | 13:56 | |
| *** shaohe_feng has quit IRC | 13:57 | |
| *** shaohe_feng has joined #openstack-nova | 13:57 | |
| *** pushkaru has joined #openstack-nova | 13:59 | |
| *** belmoreira has quit IRC | 13:59 | |
| *** fragatina has quit IRC | 13:59 | |
| sdague | mriedem: well, I'm more concerned the merge conflicts | 14:00 |
| mriedem | sdague: yeah, ok | 14:00 |
| sdague | mriedem: here is the current set of faults - http://paste.openstack.org/show/496907/ | 14:00 |
| dansmith | jroll: ah, laski already made me abandon my patch, which was already ding 315495 | 14:00 |
| dansmith | er, doing | 14:00 |
| *** penick has quit IRC | 14:00 | |
| mriedem | honestly i'm suprised people haven't already hit a bunch of those in parameters.yaml | 14:00 |
| mriedem | sdague: ok, if you have a tool already, we can just save it for the end | 14:01 |
| sdague | mriedem: it's a big enough file, it tends to not be an issue | 14:01 |
| mriedem | and people can keep in mind to try and sort them while doing this, but not required | 14:01 |
| sdague | well, it's actually going to be part of the sphinx extension | 14:01 |
| *** atuvenie has joined #openstack-nova | 14:01 | |
| *** nkrinner is now known as nkrinner_afk | 14:02 | |
| *** ducttape_ has joined #openstack-nova | 14:03 | |
| alaski | jroll: dansmith just got off a call, looking now | 14:04 |
| *** whenry has joined #openstack-nova | 14:04 | |
| *** sdake has joined #openstack-nova | 14:05 | |
| *** efried has joined #openstack-nova | 14:06 | |
| *** daemontool has left #openstack-nova | 14:06 | |
| *** shaohe_feng has quit IRC | 14:07 | |
| alaski | jroll: this looks like it's a different location than the metadata lookup | 14:07 |
| *** _ducttape_ has joined #openstack-nova | 14:07 | |
| *** shaohe_feng has joined #openstack-nova | 14:08 | |
| *** zz_dimtruck is now known as dimtruck | 14:08 | |
| *** ducttape_ has quit IRC | 14:09 | |
| mriedem | oh way to blow up my stats alaski! https://review.openstack.org/#/c/315201/ | 14:10 |
| * mriedem is ruined | 14:10 | |
| *** doug-fish has joined #openstack-nova | 14:11 | |
| alaski | heh. I did actually think about that for a split second. You'll have to blame dansmith for not seeing my feedback unless I -1 | 14:11 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: api-ref: parameter verification for os-server-groups https://review.openstack.org/315394 | 14:11 |
| mriedem | alaski: fair point | 14:11 |
| mriedem | he's conditioned you | 14:11 |
| mriedem | jroll: so, it scares me that that broke ironic... | 14:11 |
| mriedem | is there a bug in ironic for that? | 14:12 |
| *** sree has joined #openstack-nova | 14:12 | |
| mriedem | because as far as i know we should just have been lazy loading instance.keypairs | 14:12 |
| *** ccarmack has quit IRC | 14:12 | |
| mriedem | which wouldn't be in instance_extra, so we'd generate a KeyPairList with a single entry using the instance.key_name | 14:12 |
| *** sudipto has joined #openstack-nova | 14:13 | |
| openstackgerrit | Sean Dague proposed openstack/nova: report info if parameters are out of order https://review.openstack.org/315592 | 14:13 |
| sdague | mriedem: that's the detector for issues | 14:13 |
| mriedem | huh, the ironic job even failed in that original change http://logs.openstack.org/50/312750/5/check/gate-tempest-dsvm-ironic-pxe_ipa-nv/a3d58f4/logs/screen-n-cpu.txt.gz?level=TRACE | 14:13 |
| mriedem | but since it's non-voting we didn't notice | 14:13 |
| *** Guest27233 has quit IRC | 14:14 | |
| sdague | mriedem: well, can we do a quick revert? | 14:15 |
| mriedem | quick revert of what? | 14:15 |
| alaski | this looks odd to me http://logs.openstack.org/95/315495/1/check/gate-tempest-dsvm-ironic-pxe_ipa-nv/cb93016/logs/screen-n-cond.txt.gz?level=WARNING#_2016-05-12_12_18_21_489 | 14:15 |
| sdague | the break? | 14:15 |
| mriedem | sdague: no | 14:15 |
| *** sree__ has quit IRC | 14:15 | |
| alaski | it's trying to lazy load, but there's an obj_attr_is_set check right before it | 14:15 |
| mriedem | sdague: it's the bottom patch in the series to move keypairs to the api db | 14:15 |
| alaski | oh | 14:16 |
| sdague | mriedem: sure, but how many have landed on top yet? https://review.openstack.org/#/c/312751 got race failed | 14:16 |
| *** pwnall has quit IRC | 14:16 | |
| alaski | http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/instance.py#n757 | 14:17 |
| *** shaohe_feng has quit IRC | 14:17 | |
| mriedem | sdague: https://review.openstack.org/#/c/312750/5/nova/objects/instance.py also has an rpc version bump | 14:18 |
| sdague | ah | 14:18 |
| mriedem | so we have the immediate fix approved that jroll pushed | 14:18 |
| *** shaohe_feng has joined #openstack-nova | 14:18 | |
| mriedem | but as alaski points out, we have a death trap still | 14:18 |
| dansmith | it concerns me that ironic is doing a thing that depends on this like this | 14:19 |
| sdague | well, maybe we want to hold https://review.openstack.org/#/c/312751 and above until this gets figured out | 14:19 |
| dansmith | I haven't looked at what is really going on yet, but .. eesh | 14:19 |
| mriedem | i would like to look into why https://review.openstack.org/#/c/312751 keeps failing grenade | 14:19 |
| dansmith | mriedem: it's failing on the old side | 14:19 |
| *** pwnall has joined #openstack-nova | 14:20 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Add a lock() context manager to image backend https://review.openstack.org/279625 | 14:20 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Introduce ImageCacheLocalPool https://review.openstack.org/279669 | 14:20 |
| *** sridharg has quit IRC | 14:20 | |
| sdague | mriedem: http://logs.openstack.org/51/312751/10/check/gate-grenade-dsvm-multinode/f45d049/console.html (last fail) is bad networking | 14:20 |
| alaski | I'm a bit confused on this, the failure seems to be instance.refresh() trying to lazy load something. though it doesn't log what | 14:20 |
| sdague | rax-ord might just be having issues | 14:20 |
| *** laszloh has quit IRC | 14:20 | |
| mriedem | dansmith: yeah looks like ansible | 14:20 |
| alaski | however, keypairs should definitely be in expected_attrs for that call | 14:21 |
| mriedem | alaski: yeah we needed the metadata change anyway, but right i'm concerned about that refresh failure | 14:21 |
| *** pwnall has quit IRC | 14:21 | |
| alaski | I can't see how they're related yet | 14:22 |
| sdague | mriedem: yeh, they are both rax-ord not having multinode networking | 14:22 |
| melwitt | mriedem: on the tests messing up the global state, that's what my patch https://review.openstack.org/#/c/315309/ was about. you're saying even with that, something is failing in stable/mitaka? | 14:23 |
| sdague | so unrelated | 14:23 |
| mriedem | melwitt: no, it was something else | 14:23 |
| *** Vek has quit IRC | 14:23 | |
| *** josecastroleon has quit IRC | 14:23 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: api-ref: image.inc - Update method validation https://review.openstack.org/314268 | 14:24 |
| melwitt | mriedem: hm, tarnation | 14:24 |
| dansmith | alaski: it must just be that the call instance is not loading keypairs when queried that way I think | 14:24 |
| *** flaper87 has quit IRC | 14:24 | |
| *** josecastroleon has joined #openstack-nova | 14:24 | |
| mriedem | melwitt: also, good early morning to you | 14:25 |
| mriedem | you must be scrumming | 14:25 |
| *** paul-carlton1 has joined #openstack-nova | 14:25 | |
| mriedem | melwitt: this is the mitaka backport https://review.openstack.org/#/c/315596/ | 14:25 |
| melwitt | mriedem: good morning! maybe. | 14:25 |
| *** paul-carlton1 has left #openstack-nova | 14:25 | |
| *** josecastroleon has quit IRC | 14:26 | |
| *** Vek has joined #openstack-nova | 14:26 | |
| melwitt | (maybe scrumming) :) | 14:26 |
| *** flaper87 has joined #openstack-nova | 14:27 | |
| *** flaper87 has quit IRC | 14:27 | |
| *** flaper87 has joined #openstack-nova | 14:27 | |
| dansmith | actually, I think I know what it is | 14:27 |
| *** xwizard has joined #openstack-nova | 14:27 | |
| dansmith | it's a bug in refresh | 14:27 |
| jaypipes | sahid: done | 14:27 |
| *** shaohe_feng has quit IRC | 14:28 | |
| dansmith | not sure why we're poking it now because of keypairs, but | 14:28 |
| mriedem | extra = [field for field in INSTANCE_OPTIONAL_ATTRS | 14:28 |
| mriedem | if self.obj_attr_is_set(field)] | 14:28 |
| mriedem | need to pop 'keypairs' from that? | 14:28 |
| dansmith | no | 14:28 |
| dansmith | when mriedem's fix to o.vo is available I think we'd have more detail | 14:28 |
| *** ljxiash has joined #openstack-nova | 14:28 | |
| *** prateek has quit IRC | 14:28 | |
| *** iceyao has joined #openstack-nova | 14:28 | |
| *** p01nt3r75 has joined #openstack-nova | 14:28 | |
| *** sree has quit IRC | 14:29 | |
| mriedem | dansmith: that should be in 1.9.1 | 14:30 |
| mriedem | it was in 1.9.0 | 14:30 |
| dansmith | right | 14:30 |
| alaski | afaict current isn't looked up with a keypair, but it should be in extra there | 14:30 |
| *** mdrabe has quit IRC | 14:30 | |
| mriedem | ovo 1.9.1 is in this test run | 14:30 |
| mriedem | http://logs.openstack.org/95/315495/1/check/gate-tempest-dsvm-ironic-pxe_ipa-nv/cb93016/logs/ | 14:30 |
| dansmith | oh, didn't think that g-r had merged yet | 14:31 |
| *** rmart04 has joined #openstack-nova | 14:31 | |
| dansmith | really wish I wasn't on this call and could focus :/ | 14:31 |
| *** ccarmack has joined #openstack-nova | 14:31 | |
| *** shaohe_feng has joined #openstack-nova | 14:32 | |
| dansmith | I guess it's because we fail early due to the orphaning, yeah | 14:32 |
| *** pwnall has joined #openstack-nova | 14:32 | |
| mriedem | the ironic driver is calling instance.refresh() | 14:33 |
| mriedem | ok, i think i see the issue | 14:35 |
| mriedem | i can push a patch quick to test it out | 14:35 |
| alaski | I see a another potential bug, in that _from_db_object might not set a field even if it's in expected_attrs. if the db doesn't doesn't return 'extra' | 14:35 |
| *** sdake has quit IRC | 14:35 | |
| dansmith | alaski: that's intentional | 14:36 |
| *** links has joined #openstack-nova | 14:36 | |
| dansmith | not ideal, but I think you'll see a bunch of stuff fail if you don't | 14:37 |
| *** jwcroppe_ has joined #openstack-nova | 14:37 | |
| alaski | I'm wondering why we don't fail intentionally when that's the case | 14:37 |
| dansmith | disable it for all the fields and I think you'll see | 14:37 |
| dansmith | I remember it being something we had to do in the flavors work | 14:37 |
| *** shaohe_feng has quit IRC | 14:38 | |
| dansmith | probably around the fact that we had flavor loading coming from sysmeta and/or extra without knowing ahead of time | 14:38 |
| dansmith | so maybe we don't need to do it for keypairs, but still, | 14:38 |
| dansmith | the bug would be actually why we're not loading it, not that we're not failing there | 14:39 |
| *** yassine has joined #openstack-nova | 14:39 | |
| alaski | yeah, not really related. I was just thinking that we could have expected_attrs set fine and still not get a keypair leading to a later lazy load. but as long as the lazy load works it's not really a problem | 14:39 |
| *** yassine is now known as Guest84855 | 14:40 | |
| *** jwcroppe has quit IRC | 14:40 | |
| dansmith | if nothing else we could log there so we know that it happened, when we see this downstream | 14:40 |
| *** jdurgin1 has joined #openstack-nova | 14:40 | |
| *** mjura has quit IRC | 14:40 | |
| alaski | yeah, that could be useful | 14:41 |
| *** shaohe_feng has joined #openstack-nova | 14:41 | |
| bauzas | alaski: just to make sure, you had concerns on https://review.openstack.org/#/c/284974/12/nova/compute/api.py because only of the comment, or because you don't think the code will work with cellsv1 ? | 14:42 |
| openstackgerrit | Paul Carlton proposed openstack/nova-specs: Live Migration of Rescued Instances https://review.openstack.org/307131 | 14:42 |
| alaski | bauzas: the comment | 14:42 |
| bauzas | alaski: k, will remove it then | 14:43 |
| *** mdrabe has joined #openstack-nova | 14:43 | |
| bauzas | alaski: thanks | 14:43 |
| jroll | yikes, looks at all this chatter I generated | 14:44 |
| dansmith | ironic ci breaks often for no reason, right? it would be nice to gate on this, but ISTR the job being pretty flaky in the last few weeks at least | 14:46 |
| johnthetubaguy | I think JayF is looking at the gate right now, not sure how that is going | 14:47 |
| *** shaohe_feng has quit IRC | 14:48 | |
| *** shaohe_feng has joined #openstack-nova | 14:48 | |
| JayF | I'm more just gluing pieces together other people are working on | 14:50 |
| JayF | but it's very clearly the #1 thing being worked on in Ironic since the summit | 14:50 |
| *** tonytan4ever has joined #openstack-nova | 14:51 | |
| jroll | dansmith: right, we used to gate on it, flakiness made it non-voting, our #1 prio this cycle is upgrade testign and generally making our gate more reliable | 14:52 |
| dansmith | okay cool | 14:52 |
| *** abalutoiu has quit IRC | 14:52 | |
| jroll | the one true answer is getting bare metal resources in nodepool | 14:52 |
| jroll | this stuff runs in like 10 minutes there | 14:53 |
| *** jlanoux has quit IRC | 14:53 | |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add test fixture for cell0. https://review.openstack.org/301864 | 14:54 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Map instances to cell0 on failed schedule https://review.openstack.org/267827 | 14:54 |
| mriedem | from https://review.openstack.org/#/c/312750/ i don't really see how the instance that the ironic driver is refreshing has a keypairs attribute set | 14:54 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Manage db sync command for cell0 https://review.openstack.org/274885 | 14:54 |
| *** akijak has left #openstack-nova | 14:55 | |
| sahid | jaypipes: i replied to you | 14:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs: resource-providers: Move allocation fields https://review.openstack.org/300177 | 14:57 |
| jroll | mriedem: this boot call has a key pair specified http://logs.openstack.org/50/312750/5/check/gate-tempest-dsvm-ironic-pxe_ipa-nv/a3d58f4/console.html#_2016-05-11_19_33_14_115 | 14:57 |
| jaypipes | mriedem, dansmith, edleafe, bauzas: review (and hopefull approval) of above spec would be appreciated. thx in advance! | 14:57 |
| mriedem | jroll: yeah but we don't set the keypairs on the instance when creating it until a later patch in the series | 14:58 |
| *** rama_y has joined #openstack-nova | 14:58 | |
| bauzas | jaypipes: in my todo bag | 14:58 |
| *** slaweq has quit IRC | 14:58 | |
| *** shaohe_feng has quit IRC | 14:58 | |
| jroll | mriedem: is that not something that happens today? | 14:59 |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: reorder parameters.yaml https://review.openstack.org/315617 | 14:59 |
| mriedem | jroll: instance.keypairs isn't set during create until the next change in the series https://review.openstack.org/#/c/312751/ | 14:59 |
| dansmith | mriedem: ah, heh, good point | 14:59 |
| *** shaohe_feng has joined #openstack-nova | 14:59 | |
| jroll | mriedem: oh, they're in sys meta or something before this eh | 14:59 |
| mriedem | jroll: no | 14:59 |
| *** READ10 has joined #openstack-nova | 15:00 | |
| mriedem | jroll: we build instance.keypairs from instance.key_name if we don't have instance.keypairs | 15:00 |
| jroll | ah | 15:00 |
| mriedem | until keypairs are in instance_extra | 15:00 |
| dansmith | remember I said I had rebased a piece into the wrong patch? :) | 15:00 |
| mriedem | yes you did | 15:00 |
| jroll | so anyway, this patch in question still does the query no? | 15:00 |
| jroll | whether a keypair exists or not | 15:00 |
| mriedem | dansmith: https://review.openstack.org/#/c/312750/5/nova/objects/instance.py@469 | 15:00 |
| dansmith | ironic is passing on the next patch, yes? | 15:01 |
| mriedem | yes | 15:01 |
| mriedem | so it was https://review.openstack.org/#/c/312751/10/nova/objects/instance.py | 15:01 |
| jroll | heh | 15:02 |
| dansmith | jroll: so that was my bad, but as soon as we land that, I think we're probably good :D | 15:02 |
| jroll | dansmith: awesome | 15:02 |
| mriedem | so we were storing an empty keypair list in instance_extra? | 15:02 |
| *** thorst_ has joined #openstack-nova | 15:02 | |
| dansmith | no, not storing anything yet right? | 15:02 |
| mriedem | and tha's why obj_attr_is_set in refresh was passing? | 15:02 |
| jroll | dansmith: you win easiest fix ever | 15:02 |
| dansmith | and then trying to load it | 15:02 |
| dansmith | and per alaski since it wasn't set, we didn't load it, hence the lazy load that failed | 15:02 |
| mriedem | i also notice that 'keypairs': fields.ObjectField('KeyPairList') isn't nullable=True | 15:03 |
| *** josecastroleon has joined #openstack-nova | 15:03 | |
| dansmith | nor should it be | 15:03 |
| mriedem | like other things in here | 15:03 |
| dansmith | once we're past this, we always have a list in there, which may be empty if you didn't ask for any keypairs | 15:03 |
| dansmith | None means not migrated yet | 15:04 |
| *** haplo37 has joined #openstack-nova | 15:04 | |
| mriedem | dansmith: see my comment here? https://review.openstack.org/#/c/312750/5/nova/objects/instance.py@469 | 15:04 |
| openstackgerrit | Alexis Lee proposed openstack/nova: Make 'debug' option reconfigurable in Nova https://review.openstack.org/280851 | 15:05 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: api-ref: Method verification for servers-actions https://review.openstack.org/315220 | 15:06 |
| mriedem | i'm not seeing how if instance.keypairs wasn't set originally, in refresh() it would get into that extra list built here: | 15:06 |
| mriedem | http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/instance.py#n752 | 15:06 |
| mriedem | which is then passed to get_by_uuid to get the current state from the db, and expected_attrs there shouldn't have 'keypairs' in it, so current shouldn't have keypairs set on it | 15:07 |
| mriedem | and if that's true, then when we check | 15:07 |
| mriedem | if self.obj_attr_is_set(field): | 15:07 |
| mriedem | for 'keypairs' it should be False | 15:07 |
| alaski | yeah, I still don't see the actual break | 15:08 |
| mriedem | good, i thought it was just me :) | 15:08 |
| dansmith | before the next patch, | 15:08 |
| dansmith | we always have instance_extra.keypairs=None because we're not creating instances with it | 15:09 |
| *** shaohe_feng has quit IRC | 15:09 | |
| mriedem | so obj_attr_is_set == True, even though the value is None | 15:09 |
| dansmith | which means if you try to dereference instance.keypairs, we'll try a load, which won't do anything since there's nothing to load | 15:09 |
| mriedem | right? | 15:09 |
| dansmith | because the actual use of that field wasn't supposed to be until the next patch where we actually ever have something to load | 15:09 |
| *** shaohe_feng has joined #openstack-nova | 15:09 | |
| dansmith | right, he passed it, so it was set, but never created in the db, | 15:10 |
| dansmith | but set on the object all the way down to compute | 15:10 |
| mriedem | and we always have instance_extra.keypairs=None because 'keypairs': fields.ObjectField('KeyPairList') isn't nullable=True | 15:10 |
| dansmith | no | 15:10 |
| dansmith | we always have it None because we weren't creating it in the DB until the next patch | 15:10 |
| dansmith | the nullability of the field has nothing to do with the database | 15:10 |
| mriedem | so on refresh we have 'keypairs' in expected_attrs when we get here http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/instance.py#n347 | 15:11 |
| mriedem | but http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/instance.py#n349 will be None | 15:11 |
| mriedem | because it's not in the db | 15:11 |
| mriedem | so current doesn't have keypairs set in refresh | 15:12 |
| mriedem | so we lazy load and kaboom | 15:12 |
| dansmith | it's actually that we have it set in refresh() not from_db_obj() but otherwise yes | 15:12 |
| mriedem | if that's wrong, then i give up and am going to move on | 15:12 |
| dansmith | if I hadn't stupidly rebased that piece into the wrong place in the stack we'd have been fine | 15:13 |
| dansmith | and/or if jroll wasn't so quick to notice | 15:13 |
| alaski | shouldn't we be blowing up at http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/instance.py#n887 then? | 15:14 |
| mriedem | alaski: i think db_keypairs is None | 15:14 |
| mriedem | so we don't get in there | 15:14 |
| mriedem | i think None is passed to _load_keypairs so it's a total noop | 15:14 |
| alaski | gah, yeah I was seeing that as else. nvm | 15:14 |
| mriedem | i still don't see where instance.keypairs starts out as None | 15:15 |
| dansmith | mriedem: you mean instance_extra? | 15:15 |
| mriedem | no | 15:15 |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: reorder parameters.yaml https://review.openstack.org/315617 | 15:15 |
| openstackgerrit | Sean Dague proposed openstack/nova: report info if parameters are out of order https://review.openstack.org/315592 | 15:15 |
| dansmith | instance.keypairs cannot be none | 15:15 |
| dansmith | it can be unset, but not None | 15:16 |
| alaski | mriedem: same thing http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/instance.py#n349 is a noop and returns None | 15:16 |
| *** yamahata has joined #openstack-nova | 15:16 | |
| alaski | er, doesn't set it | 15:16 |
| dansmith | right | 15:16 |
| dansmith | None in the DB means it doesn't set it | 15:16 |
| dansmith | which is what alaski was complaining about earlier | 15:16 |
| dansmith | makes it more confusing when it fails late | 15:16 |
| alaski | yep, just about to say that | 15:17 |
| *** kmartin has joined #openstack-nova | 15:17 | |
| mriedem | how does self.obj_attr_is_set(field) evaluate to True in refresh then? | 15:17 |
| mriedem | if it's not set | 15:17 |
| *** sdake has joined #openstack-nova | 15:17 | |
| bauzas | alaski: just a point about http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/check-destination-on-migrations-newton.html#work-items | 15:17 |
| alaski | mriedem: it's set on instance, just not in the db | 15:18 |
| bauzas | alaski: I was planning to add initally instance actions | 15:18 |
| mriedem | alaski: but where is it initially set on instance? | 15:18 |
| bauzas | alaski: but I wonder if it's still a good option | 15:18 |
| alaski | mriedem: in compute/api early on | 15:18 |
| mriedem | alaski: not in https://review.openstack.org/#/c/312750/ | 15:18 |
| mriedem | that's the next patch | 15:18 |
| alaski | I'm assuming, I should actually check | 15:18 |
| mriedem | i don't see what actually sets instance.keypairs in https://review.openstack.org/#/c/312750/ | 15:19 |
| *** shakamunyi has quit IRC | 15:19 | |
| dansmith | mriedem: one lazy load would get it from keypairs right? | 15:19 |
| *** shaohe_feng has quit IRC | 15:19 | |
| mriedem | which would cause self.obj_attr_is_set(field) evaluate to True in refresh | 15:19 |
| dansmith | mriedem: if we refresh, the new instance would have it missing, but we orphan it so we fail before we look it up in keypairs again | 15:19 |
| mriedem | yes a lazy load would do it | 15:19 |
| *** shaohe_feng has joined #openstack-nova | 15:20 | |
| mriedem | which i guess would have to be in the metadata service in https://review.openstack.org/#/c/312750/5/nova/api/metadata/base.py | 15:20 |
| dansmith | wish grenade wasn't being a punk so we could land the next patch before this conversation is over :) | 15:20 |
| openstackgerrit | Paul Carlton proposed openstack/nova: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 15:20 |
| mriedem | is the ironic job running the metadata service? | 15:20 |
| dansmith | it can't have been the metadata service | 15:20 |
| dansmith | that would be a new/different instance object | 15:20 |
| mriedem | true | 15:20 |
| jroll | we do not afaik | 15:21 |
| *** hemnafk is now known as hemna | 15:21 | |
| mriedem | but i don't see what else would be lazy-loading 'keypairs' then in that first patch | 15:21 |
| jroll | configdrive generation | 15:21 |
| jroll | ? | 15:21 |
| alaski | bauzas: an instance action for select_destinations? | 15:21 |
| nikhil | mriedem: sdague : g'morning. We're discussing nova v1, v2 work in #openstack-dev; your input would be appreciated. | 15:21 |
| *** sacharya has joined #openstack-nova | 15:21 | |
| jroll | mriedem: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L652 | 15:22 |
| bauzas | alaski: nope, for saying 'forced_evacuate' | 15:22 |
| bauzas | it was a ndipanov's point | 15:22 |
| bauzas | "[Bypass the scheduler if the flag is set and ] log either FORCED_REBUILD or FORCED_MIGRATE action." | 15:22 |
| *** jdillaman has joined #openstack-nova | 15:23 | |
| mriedem | jroll: yup, that would do it | 15:24 |
| alaski | bauzas: I think it's more important that the evacuate and live-migrate are recorded as instance actions, if that's the case I'm not really concerned about capturing forced with them | 15:24 |
| *** jlanoux has joined #openstack-nova | 15:24 | |
| bauzas | alaski: they are already marked as instance actions, AFAIR | 15:24 |
| jroll | mriedem: though, it seems like that succeeds, is that what you expect? | 15:24 |
| bauzas | alaski: but I'll doublecheck | 15:24 |
| mriedem | jroll: the lazy load will work | 15:24 |
| alaski | it's logged at http://logs.openstack.org/95/315495/1/check/gate-tempest-dsvm-ironic-pxe_ipa-nv/cb93016/logs/screen-n-cpu.txt.gz#_2016-05-12_12_17_53_806 | 15:24 |
| alaski | just before the blowup | 15:25 |
| mriedem | yeah, so we lazy load keypairs on instance, and then it does a refresh, and since it's not in the db yet, that's why refresh fails | 15:25 |
| dansmith | regardless, refresh should be smarter about trying to load a thing it can't load, so I'll put up something to help with that | 15:25 |
| jroll | right, okay | 15:25 |
| dansmith | that would have made this clearer | 15:25 |
| jroll | clicking for me now | 15:25 |
| *** sacharya_ has joined #openstack-nova | 15:26 | |
| mriedem | +1000 for making refreshing smarter | 15:26 |
| *** catintheroof has joined #openstack-nova | 15:26 | |
| dansmith | luckily, | 15:27 |
| dansmith | ironic is using the right interface to do what they're doing, which means that instead of this complexity being pushed into the driver, it's all in our facade, | 15:27 |
| *** catintheroof has quit IRC | 15:27 | |
| dansmith | which while needing some help to be more obvious, means that it's not fixing in the ironic driver that we need | 15:28 |
| dansmith | which seems like a win from our internal modeling perspective | 15:28 |
| dansmith | if this was that other n-project, we'd have to fix 20 drivers I think | 15:28 |
| *** sacharya has quit IRC | 15:28 | |
| jroll | lol | 15:29 |
| *** shaohe_feng has quit IRC | 15:29 | |
| *** catintheroof has joined #openstack-nova | 15:29 | |
| *** shaohe_feng has joined #openstack-nova | 15:29 | |
| jroll | dansmith: want me to leave around https://review.openstack.org/#/c/315495/ ? | 15:30 |
| dansmith | jroll: I would prefer to un-abandon mine and get credit for it in the first place, but whatever you want :P | 15:30 |
| dansmith | that makes it sound like I care more than I do | 15:30 |
| dansmith | I don't | 15:30 |
| dansmith | so yes | 15:30 |
| dansmith | it's in the gate already | 15:31 |
| dansmith | so definitely don't disturb it | 15:31 |
| jroll | dansmith: cmon, I need something in case there's an election next round | 15:31 |
| *** hshiina has quit IRC | 15:31 | |
| jroll | :) | 15:31 |
| dansmith | even better reasoning :) | 15:31 |
| *** links has quit IRC | 15:32 | |
| jaypipes | danpb: sahid's https://review.openstack.org/#/c/292499/ looks A-OK to me. Feel free to wallaby it. | 15:32 |
| jroll | dansmith: do you think https://review.openstack.org/#/c/315491/ is valuable even though it doesn't fix this particular issue then? | 15:34 |
| dansmith | jroll: I have no idea what that is doing | 15:34 |
| *** sacharya_ has quit IRC | 15:34 | |
| jroll | heh | 15:34 |
| *** armax has joined #openstack-nova | 15:34 | |
| jroll | comstud taught me to always update the context when in a greenthread :P | 15:34 |
| dansmith | because why? | 15:35 |
| jroll | one reason is log.foo(instance=bar) | 15:35 |
| jroll | but anyway, the reason I put that up was a thought that we had dropped the context from the instance and that would handle it | 15:35 |
| *** dmk0202 has quit IRC | 15:35 | |
| melwitt | how does the instance not have a context? | 15:36 |
| jroll | I guess there's no reason to change it though, I can drop it | 15:36 |
| dansmith | okay, I don't know wht update_store() does | 15:36 |
| dansmith | it definitely doesn't give instance context though | 15:36 |
| *** dmk0202 has joined #openstack-nova | 15:36 | |
| jroll | right, I think it gives the context its instance, now that I think about it | 15:36 |
| rlrossit | alaski: I don't want to go against the +1 train on your policy-in-code spec, so can I ask you a few questions here to hopefully get them answered? | 15:36 |
| jroll | melwitt: given the exception that was my first thought, but I think that's wrong | 15:37 |
| * jroll abandons | 15:37 | |
| melwitt | oh, instance not in db yet (was reading mriedem's message earlier) | 15:37 |
| alaski | rlrossit: sure | 15:37 |
| dansmith | jroll: it stores the context in threading.local() | 15:37 |
| alaski | jroll: it adds the context to a thread local store, which is where logging looks for it | 15:37 |
| alaski | it's how the request-id is added to logs | 15:38 |
| jroll | right, got it | 15:38 |
| rlrossit | alaski: how is this new stuff going to handle something along the lines of default: "!" in an existing policy file today? | 15:38 |
| *** kodokuu has joined #openstack-nova | 15:39 | |
| dansmith | alaski: so do you know under what circumstances we need to do that? | 15:39 |
| jroll | alaski: so I think it is actually needed, because there's some logging there, but that also needs clean up and I'll deal with it later | 15:39 |
| *** iceyao has quit IRC | 15:39 | |
| alaski | dansmith: used to be after spawn, spawn_n. but the utils.* methods take care of it now | 15:39 |
| *** shaohe_feng has quit IRC | 15:39 | |
| alaski | if you do an explicit eventlet.spawn[_n] you'll need it | 15:40 |
| dansmith | okay | 15:40 |
| jaypipes | Yingxin: I really like how you've modeled http://paste.openstack.org/show/496904/ | 15:40 |
| dansmith | jroll: fwiw, comstud told me to do lots of things that turned out to be bad ideas | 15:40 |
| *** shaohe_feng has joined #openstack-nova | 15:40 | |
| jaypipes | dansmith, cdent, bauzas: can you take a look at Yingxin's proposed modeling here? http://paste.openstack.org/show/496904/ | 15:40 |
| dansmith | jroll: like listen to steel panther songs | 15:41 |
| jroll | alaski: so this is an oslo.something.loopingcall, probably needs it (or I could make a utils wrapper for that if you like) | 15:41 |
| jroll | heh | 15:41 |
| jaypipes | Yingxin: the only thing missing there would be "backref" fields in Inventory and Allocation to refer to the ResourceProvider. Otherwise, I think that is excellent. | 15:41 |
| *** mtanino has joined #openstack-nova | 15:41 | |
| openstackgerrit | Gage Hugo proposed openstack/nova: api-ref: ips.inc example verification https://review.openstack.org/315252 | 15:41 |
| alaski | jroll: I'm not familiar with that implementation, but if it spawns a greenthread yeah. http://git.openstack.org/cgit/openstack/nova/tree/nova/utils.py#n1035 is how it's handled in other places | 15:42 |
| kodokuu | Hi, Anyone can help me with vmware-nsx driver ? | 15:42 |
| bauzas | jaypipes: that looks good indeed, but I should take time to review into details | 15:42 |
| jaypipes | Yingxin: and you can't set default=[] for an ObjectField('ResourceList') or ListOfObjectsField('Aggregate') but I understand what you were going for there :) | 15:42 |
| jaypipes | bauzas: for sure. | 15:42 |
| alaski | rlrossit: depends on what exactly you want to accomplish. but if all policies are registered before use there's essentially no default rule anymore | 15:43 |
| alaski | so it won't work as a way to disable a bunch of things | 15:43 |
| *** sacharya has joined #openstack-nova | 15:43 | |
| bauzas | jaypipes: is it just the object modeling corresponding to the SQL schema ? if so, that's kinda agreed | 15:43 |
| alaski | rlrossit: any unregistered/undefined rules would still use it | 15:43 |
| rlrossit | alaski: so if Nova adds a new API, it's an opt-out thing because if I don't update my policy file I'll just get it? | 15:44 |
| dansmith | jaypipes: we have most of those models already, yes? seeing it without context here I'm not sure what you're getting at | 15:44 |
| jroll | alaski: cool, https://github.com/openstack/oslo.service/blob/master/oslo_service/loopingcall.py#L118 is the thing here | 15:44 |
| *** ljxiash has quit IRC | 15:44 | |
| alaski | rlrossit: right | 15:44 |
| rlrossit | alaski: I'm just wondering if we can get some sort of functionality to say "we're adding a new api, heads up here you go" | 15:44 |
| alaski | rlrossit: we'd probably want to control that with microversions | 15:45 |
| jroll | I see lots of loopingcalls in libvirt driver too | 15:45 |
| * jroll makes a note to deal with this | 15:45 | |
| alaski | rlrossit: maybe a config option to limit the max accepted microversion, no matter what the code can deal with? | 15:45 |
| rlrossit | alaski: that's what I was just going to ask | 15:45 |
| *** andreas_s has quit IRC | 15:46 | |
| rlrossit | so it'll be a max available microversion kinda thing | 15:46 |
| melwitt | rlrossit, alaski: I think I asked something similar in a summit session and it was thought the release notes are very discoverable now | 15:46 |
| alaski | rlrossit: yeah. I'd run it by sdague, alex_xu, and other api folks | 15:46 |
| rlrossit | melwitt: release notes are difficult to automate when you may-or-may-not-be deploying from HEAD :) | 15:46 |
| alaski | I think they're pretty discoverable, but people may want to deploy continuously but control API movements | 15:47 |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs: XenAPI: support VGPU via passthrough PCI https://review.openstack.org/280099 | 15:47 |
| melwitt | for signaling that we're changing a thing | 15:47 |
| rlrossit | alaski: +1 | 15:47 |
| sdague | I'm really anti putting a max api version config | 15:48 |
| *** josecastroleon has quit IRC | 15:48 | |
| sdague | this is monotonic improvement | 15:48 |
| sdague | and these things often come coupled with rpc versions | 15:48 |
| sdague | so unless we are going to put in a max rpc version config, it doesn't make any sense | 15:48 |
| alaski | we have that right? | 15:49 |
| dansmith | we do | 15:49 |
| *** matrohon has quit IRC | 15:49 | |
| sdague | so then we need to coordinate it? | 15:49 |
| sdague | rlrossit: what is the core of the issue you are running into? | 15:49 |
| *** shaohe_feng has quit IRC | 15:50 | |
| sdague | just that your policy wasn't updated? | 15:50 |
| dansmith | we control that max automatically based on a lookup table (if so configured), but agree that coordinating it also with api version is non-trivial | 15:50 |
| *** shaohe_feng has joined #openstack-nova | 15:50 | |
| dansmith | especially in the face of something like cells | 15:50 |
| sdague | rlrossit: because, the policy issue is being solved this cycle | 15:50 |
| rlrossit | sdague: the scenario in my head is that Nova adds an API that we don't want to expose to customers, but Nova does admin_or_owner on it. Now suddenly we have a mismatch in performing actions | 15:50 |
| alaski | sdague: they benefit from not having default policies in code. by setting a default of "!" all new things are off until turned on | 15:51 |
| rlrossit | and I have no automated way of figuring out that the API was added | 15:51 |
| mriedem | alaski: you can't override that still in the policy.json regardless of defaults in code? | 15:52 |
| mriedem | i guess that would disable everything.... | 15:52 |
| rlrossit | mriedem: you can, but if something is added, I won't have it in my policy | 15:52 |
| alaski | mriedem: you'd have to override each new policy. there's no fallback to default because everything is set | 15:52 |
| sdague | rlrossit: so 2 things - https://github.com/openstack/nova/blob/89fdbc51c91dd6a46a26659f5918dd3a0c0a5ab0/nova/api/openstack/rest_api_version_history.rst#L1 | 15:52 |
| rlrossit | alaski: so I thought of a way I can work around this with what you're giving me | 15:53 |
| sdague | and, the whole point is we're not actually trying to make it easy for you to have a different API than others | 15:53 |
| *** claudiub has quit IRC | 15:53 | |
| mriedem | with policy defaults in code, we're going to have a tool to dump the effective policy | 15:53 |
| rlrossit | alaski: you're making a policy generator in oslo right? | 15:53 |
| rlrossit | yeah | 15:53 |
| alaski | yep | 15:53 |
| mriedem | if you're doing CD, save that between deploys to prod | 15:53 |
| sdague | yeh | 15:53 |
| *** Swami has joined #openstack-nova | 15:53 | |
| rlrossit | I can check that thing to see if something is added to the policy | 15:53 |
| mriedem | if the policy changes in your testing, fail the test | 15:53 |
| alaski | https://review.openstack.org/#/c/314244/ | 15:53 |
| mriedem | we did this with building packages when requirements.txt would change | 15:54 |
| mriedem | the build would fail and we'd have to manually inspect what deps changed | 15:54 |
| rlrossit | badda bing, badda boom | 15:54 |
| melwitt | nice idea | 15:54 |
| *** zenoway has quit IRC | 15:54 | |
| sdague | it might also honestly just be worth failing the test every time https://github.com/openstack/nova/blob/89fdbc51c91dd6a46a26659f5918dd3a0c0a5ab0/nova/api/openstack/rest_api_version_history.rst#L1 changes until you white list moving forward | 15:54 |
| * rlrossit takes notes so he doesn't forget | 15:54 | |
| *** josecastroleon has joined #openstack-nova | 15:54 | |
| sdague | if you are concerned about new API features hitting the wire that you didn't think about | 15:54 |
| sdague | because that's the real concern, right? That a new API exposes and no one noticed? | 15:55 |
| *** zenoway has joined #openstack-nova | 15:55 | |
| *** zenoway has quit IRC | 15:55 | |
| rlrossit | sdague: it can be that, but I suppose it may also be changes in the default policy too that we might need to watch out for | 15:55 |
| sdague | rlrossit: right, sure | 15:55 |
| rlrossit | like mriedem's swapping volumes policy update | 15:55 |
| *** cdent has joined #openstack-nova | 15:55 | |
| sdague | yeh, that's a pretty reasonable approach | 15:55 |
| *** kmartin has quit IRC | 15:57 | |
| rlrossit | I can sleep slightly better at night now knowing things hopefully don't blow up underneath me | 15:57 |
| rlrossit | thanks a bunch everyone | 15:57 |
| mriedem | i sleep terribly every night regardless | 15:57 |
| mriedem | :) | 15:57 |
| rlrossit | alaski: I'll add myself to the pile of +1 on that spec now :) | 15:57 |
| alaski | rlrossit: \o/ | 15:58 |
| *** shaohe_feng has quit IRC | 16:00 | |
| *** kodokuu has quit IRC | 16:00 | |
| *** alexpilotti has quit IRC | 16:01 | |
| *** shaohe_feng has joined #openstack-nova | 16:01 | |
| rlrossit | alaski: actually is the removal of a configurable default mentioned anywhere in the spec? | 16:01 |
| rlrossit | if not, that may be something to consider for an upgrade impact | 16:01 |
| *** doug-fis_ has joined #openstack-nova | 16:01 | |
| *** alexpilotti has joined #openstack-nova | 16:02 | |
| *** Oku_OS is now known as Oku_OS-away | 16:03 | |
| mriedem | if you're removing policy, you're removing the check, | 16:03 |
| *** doug-fi__ has joined #openstack-nova | 16:03 | |
| mriedem | so if you have policy set for that rule, it's just useless since nothing is checking it | 16:03 |
| mriedem | then it's really just cruft in your policy.json | 16:04 |
| mriedem | but not a failure | 16:04 |
| rlrossit | mriedem: but today if something is unlisted, it takes the default | 16:04 |
| mriedem | yeah, and if that is removed | 16:04 |
| mriedem | why would you care? | 16:04 |
| rlrossit | or if we add an API in Newton, it'll default to Nova's when the deployer upgrades | 16:04 |
| mriedem | rlrossit: yeah, but your test will catch that and fail so you can check it out | 16:05 |
| *** doug-fish has quit IRC | 16:05 | |
| rlrossit | mriedem: my test, I'm just saying this as an operators-in-general sense | 16:05 |
| mriedem | adding new policy is an upgrade impact, i don't see how removing policy is | 16:05 |
| *** doug-fis_ has quit IRC | 16:06 | |
| rlrossit | mriedem: if an operator is missing something right now, it takes their configured default. If they upgrade to Newton, they will suddenly be getting Nova's default, not theirs | 16:06 |
| *** jaosorior has quit IRC | 16:06 | |
| *** rmart04 has quit IRC | 16:07 | |
| *** jdurgin1 has quit IRC | 16:07 | |
| *** __szilard_cserey has quit IRC | 16:07 | |
| *** doug-fi__ has quit IRC | 16:08 | |
| *** rbridgeman has joined #openstack-nova | 16:08 | |
| *** p01nt3r75 has quit IRC | 16:08 | |
| mriedem | rlrossit: ? | 16:09 |
| mriedem | 'their configured default' in policy.json? | 16:09 |
| mriedem | for a given rule | 16:09 |
| mriedem | ? | 16:09 |
| rlrossit | https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L4 | 16:09 |
| rlrossit | mriedem: this is now ignored | 16:10 |
| rlrossit | (or will be rather) | 16:10 |
| *** shaohe_feng has quit IRC | 16:10 | |
| mriedem | hmm | 16:11 |
| rlrossit | see also: https://github.com/openstack/oslo.policy/blob/master/oslo_policy/policy.py#L207-L216 | 16:11 |
| *** shaohe_feng has joined #openstack-nova | 16:11 | |
| mriedem | -1 the sh*t out of alaski's spec then :) | 16:11 |
| mriedem | i need lunch before my noon cinder love fest | 16:11 |
| *** sdake has quit IRC | 16:13 | |
| *** mkoderer__ has quit IRC | 16:14 | |
| *** diogogmt has quit IRC | 16:15 | |
| *** dmk0202 has quit IRC | 16:15 | |
| *** whenry has quit IRC | 16:18 | |
| *** apoorvad has joined #openstack-nova | 16:19 | |
| *** atuvenie has quit IRC | 16:20 | |
| *** shaohe_feng has quit IRC | 16:20 | |
| *** esracelik has quit IRC | 16:21 | |
| Yingxin | dansmith: the context is in the comments of https://review.openstack.org/#/c/284963/ | 16:21 |
| *** kolesovdv has quit IRC | 16:21 | |
| dansmith | okay | 16:22 |
| Yingxin | bauzas: yes, it's the object modeling in the logical layer(model the has-a relationship of resource provider) | 16:22 |
| *** andymaier has joined #openstack-nova | 16:22 | |
| *** raunak has joined #openstack-nova | 16:23 | |
| *** shaohe_feng has joined #openstack-nova | 16:23 | |
| *** josecastroleon has quit IRC | 16:24 | |
| *** abalutoiu has joined #openstack-nova | 16:25 | |
| Yingxin | jaypipes: what's the point to add the backref? it has a risk if there's a bug to point the backref to another provider. | 16:26 |
| *** cdent has quit IRC | 16:26 | |
| *** fragatina has joined #openstack-nova | 16:27 | |
| *** mkoderer__ has joined #openstack-nova | 16:27 | |
| *** danpb has quit IRC | 16:28 | |
| *** shaohe_feng has quit IRC | 16:31 | |
| *** shaohe_feng has joined #openstack-nova | 16:31 | |
| *** gjayavelu has joined #openstack-nova | 16:32 | |
| *** jlanoux has quit IRC | 16:34 | |
| *** rmart04 has joined #openstack-nova | 16:36 | |
| * Yingxin zzZ | 16:38 | |
| *** shaohe_feng has quit IRC | 16:41 | |
| *** shaohe_feng has joined #openstack-nova | 16:41 | |
| *** abalutoiu_ has joined #openstack-nova | 16:42 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: fix disk size calculation for VZ container instances https://review.openstack.org/222567 | 16:42 |
| *** atuvenie has joined #openstack-nova | 16:43 | |
| jaypipes | Yingxin: yeah, I suppose you're right on that. Especially with cdent's latest REST API proposal, where there isn't a way to just operate on a single Resource or Aggregate. | 16:43 |
| *** doug-fish has joined #openstack-nova | 16:43 | |
| *** andymaier has quit IRC | 16:44 | |
| *** Jeffrey4l has quit IRC | 16:45 | |
| *** josecastroleon has joined #openstack-nova | 16:45 | |
| *** abalutoiu has quit IRC | 16:45 | |
| *** doug-fish has quit IRC | 16:48 | |
| jaypipes | dansmith: Yingxin's paste is in relation to Chris' patch here: https://review.openstack.org/#/c/284963/ | 16:48 |
| dansmith | jaypipes: yeah, he said that above | 16:48 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 16:49 |
| krotscheck | auggy, sdague, anyone who knows the answer: Just had a question for the internal HPE UX team: If Nova is running with a non-metal driver, what does power state represent? | 16:49 |
| *** sdake has joined #openstack-nova | 16:49 | |
| *** alex_xu has quit IRC | 16:49 | |
| *** thorst_ has quit IRC | 16:50 | |
| *** gzhai2 has joined #openstack-nova | 16:50 | |
| mriedem | krotscheck: https://github.com/openstack/nova/blob/master/nova/compute/power_state.py#L19 | 16:50 |
| *** Yingxin has quit IRC | 16:50 | |
| krotscheck | Yeah, reading that... | 16:50 |
| *** gzhai1 has quit IRC | 16:50 | |
| *** _xin1 has joined #openstack-nova | 16:50 | |
| *** heyongli has quit IRC | 16:50 | |
| *** Yingxin_ has joined #openstack-nova | 16:50 | |
| *** eliqiao has quit IRC | 16:50 | |
| *** Yingxin_ is now known as Yingxin | 16:50 | |
| *** _xin has quit IRC | 16:50 | |
| *** gjayavelu has quit IRC | 16:51 | |
| *** eliqiao has joined #openstack-nova | 16:51 | |
| *** shaohe_feng has quit IRC | 16:51 | |
| *** mageshgv has joined #openstack-nova | 16:51 | |
| *** mageshgv has quit IRC | 16:51 | |
| *** shaohe_feng has joined #openstack-nova | 16:51 | |
| alaski | rlrossit: yeah, good call on the upgrade impact. I can update that | 16:51 |
| *** heyongli has joined #openstack-nova | 16:52 | |
| *** woodster_ has joined #openstack-nova | 16:52 | |
| rlrossit | \o/ I helped! | 16:52 |
| krotscheck | mriedem: I think we figured it out, thanks :) | 16:52 |
| *** alex_xu has joined #openstack-nova | 16:52 | |
| alaski | rlrossit: we're not going to be able to honor current defaults though. I think we just need to make a lot of noise about it so deployers see it coming | 16:54 |
| *** lucasagomes is now known as lucas-dinner | 16:54 | |
| *** sambetts is now known as sambetts|afk | 16:55 | |
| *** antonyfm has joined #openstack-nova | 16:55 | |
| rlrossit | alaski: so that's why I'm thinking that if they have a default: "<rule>" in their policy.json, we pretty much have to ignore all of the in-code defaults right? | 16:55 |
| *** thorst_ has joined #openstack-nova | 16:56 | |
| *** mkoderer__ has quit IRC | 16:57 | |
| *** gjayavelu has joined #openstack-nova | 16:57 | |
| alaski | the way this is implemented that's not really feasible. I think we really just need to be clear that default becomes useless | 16:58 |
| openstackgerrit | Paul Carlton proposed openstack/nova-specs: Automatic Live Migration Completion https://review.openstack.org/306561 | 16:58 |
| *** thorst_ has quit IRC | 17:01 | |
| *** shaohe_feng has quit IRC | 17:01 | |
| *** diogogmt has joined #openstack-nova | 17:01 | |
| *** shaohe_feng has joined #openstack-nova | 17:02 | |
| *** ccarmack has quit IRC | 17:02 | |
| *** ccarmack has joined #openstack-nova | 17:02 | |
| edleafe | jaypipes: will review a little later when (if) I escape from meeting hell. | 17:03 |
| *** nic has joined #openstack-nova | 17:03 | |
| *** thorst_ has joined #openstack-nova | 17:03 | |
| *** mvk_ has quit IRC | 17:04 | |
| *** abalutoiu has joined #openstack-nova | 17:04 | |
| *** mkoderer__ has joined #openstack-nova | 17:04 | |
| *** ijw has joined #openstack-nova | 17:05 | |
| *** lakshmiS has joined #openstack-nova | 17:06 | |
| *** thorst_ has quit IRC | 17:07 | |
| *** abalutoiu_ has quit IRC | 17:08 | |
| *** ijw has quit IRC | 17:10 | |
| *** abalutoiu_ has joined #openstack-nova | 17:10 | |
| *** shaohe_feng has quit IRC | 17:12 | |
| *** abalutoiu__ has joined #openstack-nova | 17:12 | |
| *** ijw has joined #openstack-nova | 17:12 | |
| *** atuvenie_ has joined #openstack-nova | 17:12 | |
| *** shaohe_feng has joined #openstack-nova | 17:12 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance rescue mode support https://review.openstack.org/217679 | 17:13 |
| *** abalutoiu has quit IRC | 17:14 | |
| *** davideagnello has joined #openstack-nova | 17:14 | |
| *** josecastroleon has quit IRC | 17:15 | |
| sdague | mriedem: so - https://review.openstack.org/#/c/315617/ is already in merge conflict :) | 17:15 |
| sdague | that was first attempt at fixing the param ordering | 17:15 |
| *** abalutoiu_ has quit IRC | 17:15 | |
| *** atuvenie has quit IRC | 17:15 | |
| sdague | if you want to review that and the one below it, when I find a window where we've got a nearly empty gate (like maybe monday morning) I'll just do the fix and fast approve with all the parameters | 17:16 |
| rlrossit | alaski: I don't disagree with it would be nice to have it that way, but I think that'll break a lot of deployers (but break is a different case here, more "unexpected behavior") | 17:16 |
| *** unicell has joined #openstack-nova | 17:17 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add resource provider tables to the api database https://review.openstack.org/315681 | 17:19 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Make Inventory and ResourceProvider objects use the API DB instead https://review.openstack.org/315682 | 17:19 |
| dansmith | jaypipes: alaski: FYI, if you decide you want to move them back to the cell someday, you will have to find someone else to do the work | 17:20 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova-specs: VMware: Enable memory, disk and vnic limits https://review.openstack.org/315685 | 17:20 |
| dansmith | hopefully I will be a grizzly bearded mountain man by that point | 17:20 |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: finish validation for os-server-external-events.inc https://review.openstack.org/315212 | 17:20 |
| *** shaohe_feng has quit IRC | 17:22 | |
| *** doug-fish has joined #openstack-nova | 17:22 | |
| *** shaohe_feng has joined #openstack-nova | 17:22 | |
| *** doug-fish has quit IRC | 17:23 | |
| jaypipes | dansmith: :( sorry duder. | 17:23 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/313743 | 17:23 |
| *** doug-fish has joined #openstack-nova | 17:24 | |
| alaski | dansmith: heh. I don't want them ever moved back. I say we split every table into it's own database | 17:24 |
| *** jistr has quit IRC | 17:24 | |
| * dansmith throws a chair at alaski | 17:24 | |
| *** salv-orlando has quit IRC | 17:24 | |
| alaski | :) | 17:24 |
| sdague | I think every instance needs it's own database | 17:26 |
| * dansmith throws alaski at sdague | 17:26 | |
| edleafe | alaski: now that's the best data design idea I've heard in a while! | 17:26 |
| alaski | sdague: love it! | 17:26 |
| *** gszasz has quit IRC | 17:28 | |
| *** doug-fish has quit IRC | 17:28 | |
| *** shaohe_feng has quit IRC | 17:32 | |
| *** rmart04 has quit IRC | 17:33 | |
| *** shaohe_feng has joined #openstack-nova | 17:33 | |
| *** gagehugo has joined #openstack-nova | 17:34 | |
| *** astupnikov has joined #openstack-nova | 17:34 | |
| *** rfolco has quit IRC | 17:35 | |
| *** astupnikov has quit IRC | 17:35 | |
| *** rfolco has joined #openstack-nova | 17:35 | |
| *** NellyK has joined #openstack-nova | 17:36 | |
| *** Yingxin has quit IRC | 17:37 | |
| *** _xin1 has quit IRC | 17:38 | |
| *** alex_xu has quit IRC | 17:38 | |
| *** rdo has quit IRC | 17:38 | |
| mriedem | tdurakov: looks like the live migration job on the experimental queue has an issue with setting up ceph? http://logs.openstack.org/73/312773/1/experimental/gate-tempest-dsvm-multinode-live-migration/c57f6b9/console.html#_2016-05-08_09_18_59_360 | 17:40 |
| mriedem | jbernard: ^ | 17:40 |
| dansmith | and looks like it's using the infra mirror, | 17:41 |
| dansmith | so assume it's not related to some new ceph packages | 17:41 |
| *** markvoelker has joined #openstack-nova | 17:41 | |
| *** Sukhdev has joined #openstack-nova | 17:42 | |
| jbernard | looking | 17:42 |
| *** sahid has quit IRC | 17:42 | |
| *** shaohe_feng has quit IRC | 17:42 | |
| jbernard | oh yeah, missing a param there | 17:43 |
| *** shaohe_feng has joined #openstack-nova | 17:43 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/live_migration/hooks/ceph.sh | 17:43 |
| jbernard | mriedem: i suppose RULE_ID is unset | 17:44 |
| mriedem | yes it's not set | 17:44 |
| jbernard | so awk/cut is not parsing the output correctly | 17:45 |
| mriedem | here right? https://github.com/openstack/nova/blob/master/nova/tests/live_migration/hooks/ceph.sh#L109 | 17:45 |
| *** READ10 has quit IRC | 17:45 | |
| jbernard | yep | 17:45 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: api-ref: os-floating-ip-dns.inc method verification https://review.openstack.org/315284 | 17:46 |
| mriedem | https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/lib/ceph#L316 | 17:46 |
| jbernard | id like to see the output before the pipe | 17:46 |
| *** rdo has joined #openstack-nova | 17:46 | |
| *** _ducttape_ has quit IRC | 17:46 | |
| jbernard | one sec | 17:48 |
| *** doug-fis_ has joined #openstack-nova | 17:48 | |
| mriedem | what sucks is we have the code duplicated between the devstack plugin and this nova hook | 17:48 |
| jbernard | yeah, that's broken | 17:50 |
| jbernard | atleast for gnu awk | 17:50 |
| jbernard | s/$3/$2 | 17:50 |
| mriedem | it's the same in the plugin https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/lib/ceph#L316 | 17:51 |
| *** jbernard has left #openstack-nova | 17:51 | |
| *** jbernard has joined #openstack-nova | 17:51 | |
| jbernard | mriedem: yep, need to fix that in both places | 17:52 |
| mriedem | jbernard: but the ceph job in nova is working... | 17:52 |
| mriedem | ooo | 17:52 |
| mriedem | CEPH_REPLICAS | 17:52 |
| *** rlrossit has quit IRC | 17:52 | |
| mriedem | in the live migration job that's 2 | 17:52 |
| *** shaohe_feng has quit IRC | 17:53 | |
| jbernard | yup :) | 17:53 |
| mriedem | the plugin job defaults to 1 | 17:53 |
| mriedem | https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/lib/ceph#L82 | 17:53 |
| jbernard | we dont execute that block normally | 17:53 |
| *** NellyK has quit IRC | 17:53 | |
| mriedem | jbernard: want to push that change? | 17:53 |
| mriedem | we can test it out in nova which is nice | 17:53 |
| jbernard | mriedem: will do | 17:53 |
| mriedem | thanks! | 17:53 |
| jbernard | mriedem: ill update both the plugin and nova | 17:53 |
| mriedem | <3 | 17:53 |
| mriedem | above and beyond | 17:53 |
| *** shaohe_feng has joined #openstack-nova | 17:53 | |
| jbernard | mriedem: any chance to share that logic with the plugin? | 17:54 |
| jbernard | mriedem: that seems like a better solution moving forward | 17:54 |
| mriedem | you'd have to ask tdurakov | 17:54 |
| *** doug-fis_ has quit IRC | 17:54 | |
| jbernard | i didn't know there was duplicated code there | 17:54 |
| jbernard | tdurakov: ^ | 17:54 |
| mriedem | i'll open a bug quick | 17:55 |
| jbernard | cool, ill reference that in the commits | 17:56 |
| *** ljxiash has joined #openstack-nova | 17:56 | |
| mriedem | jbernard: https://bugs.launchpad.net/nova/+bug/1581159 | 17:57 |
| openstack | Launchpad bug 1581159 in OpenStack Compute (nova) "ceph testing: parsing osd crush rule dump devstack output with awk is wrong" [Undecided,New] | 17:57 |
| *** zenoway has joined #openstack-nova | 17:57 | |
| *** pcaruana has quit IRC | 17:58 | |
| *** ljxiash has quit IRC | 18:00 | |
| *** zenoway has quit IRC | 18:02 | |
| *** alexpilotti has quit IRC | 18:02 | |
| *** shaohe_feng has quit IRC | 18:03 | |
| *** shaohe_feng has joined #openstack-nova | 18:03 | |
| *** alexpilotti has joined #openstack-nova | 18:03 | |
| *** mriedem has quit IRC | 18:03 | |
| *** mriedem has joined #openstack-nova | 18:03 | |
| *** andymaier has joined #openstack-nova | 18:04 | |
| *** doug-fish has joined #openstack-nova | 18:05 | |
| *** dave-mccowan has quit IRC | 18:05 | |
| *** sfinucan has quit IRC | 18:05 | |
| *** jichen has joined #openstack-nova | 18:06 | |
| *** ri0 has joined #openstack-nova | 18:07 | |
| *** doug-fis_ has joined #openstack-nova | 18:08 | |
| *** rcernin has quit IRC | 18:09 | |
| openstackgerrit | Jon Bernard proposed openstack/nova: live-migration ceph: fix typo in ruleset parsing https://review.openstack.org/315703 | 18:10 |
| *** doug-fish has quit IRC | 18:10 | |
| *** andymaier has quit IRC | 18:11 | |
| *** doug-fis_ has quit IRC | 18:13 | |
| *** shaohe_feng has quit IRC | 18:13 | |
| *** sudipto has quit IRC | 18:13 | |
| *** shaohe_feng has joined #openstack-nova | 18:14 | |
| *** jistr has joined #openstack-nova | 18:15 | |
| *** jistr is now known as jistr|afk | 18:15 | |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** openstackgerrit has joined #openstack-nova | 18:18 | |
| *** markvoelker_ has joined #openstack-nova | 18:19 | |
| *** andymaier has joined #openstack-nova | 18:20 | |
| *** markvoelker has quit IRC | 18:22 | |
| *** shaohe_feng has quit IRC | 18:23 | |
| *** shaohe_feng has joined #openstack-nova | 18:24 | |
| *** atuvenie_ has quit IRC | 18:26 | |
| *** ri0 has quit IRC | 18:27 | |
| *** pwnall has quit IRC | 18:27 | |
| *** ducttape_ has joined #openstack-nova | 18:28 | |
| *** abalutoiu__ has quit IRC | 18:28 | |
| *** ri0 has joined #openstack-nova | 18:29 | |
| *** claudiub has joined #openstack-nova | 18:31 | |
| *** erhudy has quit IRC | 18:31 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: limits.inc example verification https://review.openstack.org/315714 | 18:32 |
| *** ri0 has quit IRC | 18:33 | |
| *** cdent has joined #openstack-nova | 18:33 | |
| *** BjoernT has joined #openstack-nova | 18:33 | |
| *** shaohe_feng has quit IRC | 18:34 | |
| *** shaohe_feng has joined #openstack-nova | 18:34 | |
| *** belmoreira has joined #openstack-nova | 18:34 | |
| mriedem | dansmith: +1 from jenkins on the ceph fix backport https://review.openstack.org/#/c/315596/ | 18:35 |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: complete verification for extensions resource https://review.openstack.org/314101 | 18:36 |
| *** rlrossit has joined #openstack-nova | 18:36 | |
| *** rchurch has quit IRC | 18:36 | |
| *** kmartin has joined #openstack-nova | 18:36 | |
| cdent | p!hist | 18:37 |
| sdague | jroll: https://review.openstack.org/#/c/311865/ is all broken on grenade testing because of the not yet landed nova patch right? | 18:38 |
| dansmith | mriedem: cool | 18:39 |
| *** abalutoiu__ has joined #openstack-nova | 18:40 | |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: support limits via image meta data https://review.openstack.org/180942 | 18:41 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: configure limits during resize operation https://review.openstack.org/204219 | 18:41 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object https://review.openstack.org/188321 | 18:41 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: create vif with resource limitations https://review.openstack.org/196171 | 18:41 |
| *** BjoernT has quit IRC | 18:42 | |
| cdent | jaypipes, Yingxin, dansmith: Can you guys summarize the discussion you had earlier about the resource provider modelling. I've tried to read the log but I'm unable to extract a summary. Also, in terms of the models, as they currently are, it probably helps to look at the web API WIP, which drove most of the modeling (that plus my ignorance of ovo): https://review.openstack.org/#/c/293104/ | 18:43 |
| dansmith | I had no discussion | 18:43 |
| mriedem | rlrossit: http://developer.openstack.org/api-ref/compute/ | 18:44 |
| *** shaohe_feng has quit IRC | 18:44 | |
| rlrossit | I know how to google mriedem | 18:44 |
| rlrossit | jeez | 18:44 |
| mriedem | no | 18:44 |
| *** shaohe_feng has joined #openstack-nova | 18:44 | |
| cdent | sorry dansmith, I guess you were invited, didn't participate? | 18:44 |
| mriedem | because docs.o.o will take you to the api-site docs which are old | 18:44 |
| *** sdake has quit IRC | 18:44 | |
| dansmith | cdent: if there was a thing for which I was invited, I don't know what that wa | 18:45 |
| dansmith | was | 18:45 |
| cdent | dansmith: no worries, just trying extract meaning from the log, you can safely ignore me (this time) | 18:45 |
| *** prateek has joined #openstack-nova | 18:46 | |
| *** rchurch has joined #openstack-nova | 18:46 | |
| *** dmk0202 has joined #openstack-nova | 18:47 | |
| mriedem | rlrossit: https://review.openstack.org/#/c/314101/ | 18:47 |
| *** lpetrut has joined #openstack-nova | 18:48 | |
| rlrossit | mriedem: I'm just wondering what the difference between https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L111 and https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L304 | 18:49 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add a RequestSpec generation migration script https://review.openstack.org/302828 | 18:50 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Verify requested_destination in the scheduler https://review.openstack.org/314509 | 18:50 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: API changes for verifying the scheduler when migrating https://review.openstack.org/315572 | 18:50 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Resize API operation passing down original RequestSpec https://review.openstack.org/312009 | 18:50 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add requested_destination field to RequestSpec https://review.openstack.org/312757 | 18:50 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Cold migrate using the RequestSpec object https://review.openstack.org/284974 | 18:50 |
| mriedem | rlrossit: compute_extension: prefix is v2 api | 18:50 |
| mriedem | os_compute_api: is v2.1 | 18:50 |
| rlrossit | mriedem: does network: stuff go away after the proxying is gone? | 18:50 |
| mriedem | rlrossit: no | 18:50 |
| mriedem | let me gerrit that for you | 18:50 |
| mriedem | rlrossit: https://review.openstack.org/#/c/312209/ | 18:51 |
| mriedem | rlrossit: we can't drop the network stuff until nova-network is dead | 18:51 |
| bauzas | dansmith: jaypipes: mriedem: in case you folks wanna review an already +2d https://review.openstack.org/302828 :) | 18:51 |
| bauzas | (just a rebase) | 18:51 |
| rlrossit | mriedem: so, I mean, there's os-networks stuff, but then there's https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L205-L249 | 18:52 |
| rlrossit | or is that all in the nova-network umbrella too? | 18:52 |
| jaypipes | cdent: reviewing 293104 now. | 18:52 |
| cdent | jaypipes, responsding to your spec comments now | 18:52 |
| rlrossit | and then there's the compute: prefix | 18:52 |
| mriedem | rlrossit: that's v2 api | 18:52 |
| cdent | jaypipes, 293104 is a mess, but intentionally so, it's sort of like an RFC | 18:52 |
| jaypipes | cdent: unfortunately, I am going to be afk for a few hours at least :( | 18:52 |
| mriedem | rlrossit: the network: stuff is checked in the network rpc api | 18:52 |
| mriedem | i think | 18:52 |
| cdent | jaypipes: and I'm out all day tomorrow | 18:52 |
| jaypipes | cdent: and no worries on 293104. it's expected as we work through all this. | 18:53 |
| cdent | but here all evening | 18:53 |
| mriedem | rlrossit: i asked the same exact thing the other day to alaski | 18:53 |
| mriedem | it's legacy v2 api cruft | 18:53 |
| mriedem | sec | 18:53 |
| jaypipes | cdent: ok, forgot about that. do your best to respond async on the reviews then. | 18:53 |
| * cdent nods | 18:53 | |
| *** abalutoiu_ has joined #openstack-nova | 18:54 | |
| *** shaohe_feng has quit IRC | 18:54 | |
| *** crito__ has quit IRC | 18:54 | |
| *** shaohe_feng has joined #openstack-nova | 18:55 | |
| mriedem | rlrossit: e.g https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L73 | 18:55 |
| mriedem | rlrossit: is used here https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2616 | 18:55 |
| openstackgerrit | Clenimar Filemon proposed openstack/nova: Use Identity v3 params when connecting to Ironic https://review.openstack.org/300154 | 18:55 |
| mriedem | rlrossit: but only if skip_policy_check=False | 18:55 |
| mriedem | rlrossit: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L213 | 18:56 |
| mriedem | see https://github.com/openstack/nova/blob/master/nova/compute/api.py#L162 | 18:56 |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref: servers-action-evacuate.inc https://review.openstack.org/314776 | 18:56 |
| mriedem | rlrossit: because the legacy v2 rest apis weren't checking these | 18:56 |
| sdague | mriedem: https://review.openstack.org/#/c/315212/ is also updated with your comments | 18:56 |
| mriedem | rlrossit: but in v2.1 the policy check happens in the v2.1 api | 18:56 |
| mriedem | so v2.1 REST API passes skip_policy_check=True when instantiating the compute API | 18:56 |
| mriedem | sdague: yeah i was getting there but then got rlrossit bombed | 18:56 |
| *** abalutoiu__ has quit IRC | 18:57 | |
| rlrossit | :( | 18:57 |
| dansmith | is that a thing now? | 18:58 |
| *** zenoway has joined #openstack-nova | 18:58 | |
| dansmith | rlrossit-bombed | 18:58 |
| mriedem | rlrossit: there is where the network policy checks happen for nova-network https://github.com/openstack/nova/blob/master/nova/network/api.py#L41 | 18:58 |
| mriedem | rlrossit: but only for v2 api | 18:58 |
| mriedem | not an issue if you're using neutron because neutron controls it's own policy | 18:59 |
| jroll | sdague: re 311865, yes the jobs that normally pass are broken due to the nova break, the grenade stuff timed out in smoke tests and didn't make it to resource creation | 18:59 |
| mriedem | except for what nova has in it's rest api for proxies | 18:59 |
| * jroll checks again | 18:59 | |
| *** sambetts|afk has quit IRC | 18:59 | |
| mriedem | dansmith: when he's across the table from me, it is | 18:59 |
| *** rlrossit has quit IRC | 19:00 | |
| mriedem | dansmith: and liberty https://review.openstack.org/#/c/315694/ | 19:01 |
| *** antonyfm has quit IRC | 19:01 | |
| *** omnipresent has joined #openstack-nova | 19:01 | |
| mriedem | mtreinish: you should +W these so we can merge that tempest skip revert https://review.openstack.org/#/q/I25f65bcc76b83f31a8fce77c2b751d2d167ffc7e,n,z | 19:01 |
| *** zenoway has quit IRC | 19:02 | |
| mtreinish | mriedem: done | 19:02 |
| *** alexpilotti has quit IRC | 19:02 | |
| *** alexpilotti has joined #openstack-nova | 19:02 | |
| *** sambetts has joined #openstack-nova | 19:03 | |
| *** shaohe_feng has quit IRC | 19:04 | |
| *** shaohe_feng has joined #openstack-nova | 19:05 | |
| mriedem | sdague: one little thing in https://review.openstack.org/#/c/315212/3/api-ref/source/parameters.yaml | 19:05 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: api-ref: finish validation for os-server-external-events.inc https://review.openstack.org/315212 | 19:06 |
| *** armax has quit IRC | 19:07 | |
| *** fragatina has quit IRC | 19:08 | |
| *** kfarr has joined #openstack-nova | 19:09 | |
| *** shaohe_feng has quit IRC | 19:15 | |
| *** shaohe_feng has joined #openstack-nova | 19:15 | |
| *** salv-orlando has joined #openstack-nova | 19:16 | |
| bauzas | dansmith: correct me if I'm wrong but instance.image_meta is the new access method for getting the image ? | 19:21 |
| dansmith | bauzas: the image ... metadata, yeah | 19:21 |
| bauzas | dansmith: okay, while it's different with utils.get_image_from_system_metadata(instance.system_metadata) ? | 19:22 |
| bauzas | because it would give us the image ? not the metadata of it ? | 19:22 |
| sdague | mriedem: ah, good catch | 19:22 |
| dansmith | bauzas: no, that returns metadata too right? | 19:22 |
| bauzas | dansmith: I think so | 19:23 |
| bauzas | dansmith: because for getting the image, we need to call the image API I think | 19:23 |
| bauzas | (using the image metadata... :) ) | 19:23 |
| dansmith | ...right... so I'm confused what you're asking for | 19:23 |
| bauzas | oh, my bad | 19:23 |
| bauzas | lemme give you a bit of explanation | 19:24 |
| mriedem | bauzas: actually | 19:24 |
| mriedem | bauzas: before you start | 19:24 |
| bauzas | dansmith: https://review.openstack.org/#/c/302828/8/nova/objects/request_spec.py@503 | 19:24 |
| mriedem | bauzas: look at what instance.image_meta does | 19:24 |
| dansmith | mriedem: ...right :) | 19:24 |
| bauzas | mriedem: it's a property | 19:24 |
| mriedem | to call the same utility you're asking about replacing it with | 19:24 |
| *** shaohe_feng has quit IRC | 19:25 | |
| bauzas | grmblbl | 19:25 |
| mriedem | so if all you need is the dict form, then i don't think you need instance.image_meta | 19:26 |
| *** ccarmack has quit IRC | 19:26 | |
| dansmith | you should use instance.image_meta if you have instance | 19:26 |
| mriedem | yeah | 19:26 |
| mriedem | regardless, | 19:27 |
| bauzas | dansmith: right, but mriedem made a valid point on lazy-loading the field | 19:27 |
| mriedem | RequestSpec._image_meta_from_image | 19:27 |
| mriedem | handles either | 19:27 |
| mriedem | i made an awesome point | 19:27 |
| mriedem | in the qa channel | 19:27 |
| *** shaohe_feng has joined #openstack-nova | 19:27 | |
| *** r-daneel has joined #openstack-nova | 19:28 | |
| mriedem | there are some other lazy-loaded fields here, like pci_requests, numa_topology and flavor | 19:29 |
| *** vladikr has quit IRC | 19:30 | |
| *** Sukhdev has quit IRC | 19:32 | |
| *** raunak has quit IRC | 19:32 | |
| *** Sukhdev has joined #openstack-nova | 19:32 | |
| *** prateek has quit IRC | 19:34 | |
| bauzas | mriedem: yeah I noticed that | 19:35 |
| *** shaohe_feng has quit IRC | 19:35 | |
| *** shaohe_feng has joined #openstack-nova | 19:36 | |
| bauzas | mriedem: to answer your question on https://review.openstack.org/#/c/302828/8/nova/objects/request_spec.py@493 I actually verified that calling delete() on a null row is fine | 19:36 |
| *** apoorvad has quit IRC | 19:36 | |
| *** atuvenie_ has joined #openstack-nova | 19:37 | |
| mriedem | yeah it must else the test would fail | 19:37 |
| mriedem | just seems weird | 19:37 |
| auggy | sdague: i'm working on an option for your reviews.py script that might show a little more info on what's been touched so far to help avoid people stepping on each other | 19:38 |
| auggy | sdague: i have a spreadsheet i've been using to manually track which is kind of a pain | 19:38 |
| auggy | so i'm hoping by updating the review script maybe it will help me to be able to update that spreadsheet a little easier | 19:39 |
| *** bswartz has quit IRC | 19:39 | |
| *** mtecer has joined #openstack-nova | 19:39 | |
| bauzas | mriedem: I can make it more explicit tho | 19:40 |
| bauzas | or safier rather | 19:40 |
| mriedem | meh, or just add a comment | 19:40 |
| auggy | sdague: this is what i have been updating - https://goo.gl/2VQPtu | 19:40 |
| *** antonyfm has joined #openstack-nova | 19:40 | |
| *** mtecer has quit IRC | 19:40 | |
| bauzas | thanks for reviewing anyway :) | 19:40 |
| auggy | sdague: i mean this is the spreadsheet | 19:40 |
| mriedem | bauzas: you poked the bear :) | 19:42 |
| *** ociuhandu has quit IRC | 19:43 | |
| bauzas | that's fine, I prefer by far poking a bear than awaking a kracken if that is going to a production system with 1M+ instances | 19:43 |
| bauzas | rather* than | 19:44 |
| bauzas | or those guys would literally call the FBI to arrest me if I'm going to PDX | 19:44 |
| *** shaohe_feng has quit IRC | 19:45 | |
| mriedem | sdague: do you know if the live migration job is setup to stop if one of the runs fails? | 19:45 |
| mriedem | looks like it is | 19:46 |
| *** shaohe_feng has joined #openstack-nova | 19:46 | |
| *** fragatina has joined #openstack-nova | 19:46 | |
| *** rlrossit has joined #openstack-nova | 19:49 | |
| *** matrohon has joined #openstack-nova | 19:51 | |
| *** jistr|afk has quit IRC | 19:52 | |
| *** yamahata has quit IRC | 19:52 | |
| *** sacharya_ has joined #openstack-nova | 19:53 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/313743 | 19:53 |
| *** sacharya has quit IRC | 19:53 | |
| *** sacharya has joined #openstack-nova | 19:53 | |
| *** belmoreira has quit IRC | 19:54 | |
| *** Swami has quit IRC | 19:54 | |
| *** sacharya_ has quit IRC | 19:54 | |
| *** rbridgeman has quit IRC | 19:54 | |
| *** apoorvad has joined #openstack-nova | 19:55 | |
| *** shaohe_feng has quit IRC | 19:56 | |
| *** mtecer has joined #openstack-nova | 19:56 | |
| *** shaohe_feng has joined #openstack-nova | 19:57 | |
| *** Swami has joined #openstack-nova | 19:57 | |
| *** yamahata has joined #openstack-nova | 19:58 | |
| *** pwnall has joined #openstack-nova | 20:00 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add resource provider tables to the api database https://review.openstack.org/315681 | 20:01 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Make Inventory and ResourceProvider objects use the API DB instead https://review.openstack.org/315682 | 20:01 |
| *** josecastroleon has joined #openstack-nova | 20:02 | |
| *** rbridgeman has joined #openstack-nova | 20:04 | |
| *** ctrath has quit IRC | 20:04 | |
| *** shaohe_feng has quit IRC | 20:06 | |
| *** shaohe_feng has joined #openstack-nova | 20:06 | |
| rlrossit | jaypipes: alaski: I'm looking at the nova policy file and I notice that os-server-tags is totally unrestricted for all CRUD operations in the default policy. Should that be changed over to admin_or_owner? | 20:06 |
| mriedem | sdague: ripping and tearing on this keypairs.inc one | 20:07 |
| rlrossit | https://github.com/openstack/nova/blob/master/etc/nova/policy.json#L448-L453 for reference | 20:07 |
| *** ctrath has joined #openstack-nova | 20:07 | |
| alaski | rlrossit: yeah. there may be something in the db lookups to prevent cross tenant operations, but policy should reflect and enforce that | 20:08 |
| rlrossit | alaski: want me to open a bug + patch? | 20:08 |
| alaski | rlrossit: that would be lovely | 20:08 |
| rlrossit | <3 | 20:08 |
| *** yamahata has quit IRC | 20:09 | |
| mriedem | hmm | 20:14 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/keypairs.py#L147-L157 | 20:14 |
| mriedem | why? | 20:14 |
| *** shaohe_feng has quit IRC | 20:16 | |
| *** belmoreira has joined #openstack-nova | 20:16 | |
| *** mtecer has quit IRC | 20:16 | |
| mriedem | sdague: so on the deleted/deleted_at fields in a response, when some things move to the api db, like keypairs, they don't have those fields in the data model anymore, but parameters.yaml says they are required. i guess it just means the keys still have to be in the response dict but they'll be deleted=False and deleted_at=None | 20:18 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Update os-server-tags in default policy https://review.openstack.org/315757 | 20:18 |
| rlrossit | alaski: ^ | 20:18 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: support limits via image meta data https://review.openstack.org/180942 | 20:19 |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object https://review.openstack.org/188321 | 20:19 |
| *** shaohe_feng has joined #openstack-nova | 20:19 | |
| *** raunak has joined #openstack-nova | 20:19 | |
| *** ayoung has quit IRC | 20:20 | |
| alaski | rlrossit: you're going to run afoul of test_policy.py | 20:22 |
| rlrossit | wait there's actually unit tests on the default policy? | 20:22 |
| alaski | yeah. I'm not quite sure why, but yes | 20:23 |
| *** Sukhdev has quit IRC | 20:23 | |
| *** rbridgeman_ has joined #openstack-nova | 20:23 | |
| *** Sukhdev has joined #openstack-nova | 20:24 | |
| *** rcernin has joined #openstack-nova | 20:24 | |
| *** smatzek has quit IRC | 20:26 | |
| *** shaohe_feng has quit IRC | 20:26 | |
| *** rbridgeman has quit IRC | 20:26 | |
| tonyb | mriedem, mikal: I have a busy monrnign with extra events at my kids school. so I'll be late and probably miss the nova meeting :( | 20:26 |
| mriedem | unacceptable | 20:28 |
| *** shaohe_feng has joined #openstack-nova | 20:29 | |
| *** rama_y has quit IRC | 20:31 | |
| *** josecastroleon has quit IRC | 20:32 | |
| *** wznoinsk has quit IRC | 20:32 | |
| *** yamahata has joined #openstack-nova | 20:33 | |
| *** belmoreira has quit IRC | 20:34 | |
| *** ericksonsantos has quit IRC | 20:35 | |
| sdague | mriedem: hmmmm.... right. We're going to need a story on that api change, even if it's just document that those fields are optional | 20:35 |
| *** hamzy has quit IRC | 20:36 | |
| *** lpetrut has quit IRC | 20:36 | |
| *** shaohe_feng has quit IRC | 20:37 | |
| mriedem | sdague: well, the keys will be in the response body | 20:38 |
| mriedem | so that doesn't change | 20:38 |
| mriedem | which makes me wonder | 20:38 |
| mriedem | can you read deleted keypairs from the cell db / api today? | 20:38 |
| sdague | read deleted is usually admin | 20:38 |
| sdague | I'd have to look exactly | 20:39 |
| *** shaohe_feng has joined #openstack-nova | 20:39 | |
| sdague | rlrossit: the unrestricted policy is a bit of a red herring, because there remains project scoping on tenant for most things | 20:39 |
| mriedem | i don't see read_deleted='yes' when getting keypairs | 20:40 |
| *** lpetrut has joined #openstack-nova | 20:40 | |
| mriedem | so it's just odd that we leak keypair data fields through the api | 20:40 |
| mriedem | like the timestamp mixin fields and the id | 20:40 |
| mriedem | which.... | 20:40 |
| mriedem | when we online data migrate keypairs to the api db they have to retain their id | 20:40 |
| sdague | mriedem: all predates me | 20:40 |
| rlrossit | sdague: well a red herring is still a case to get it fixed :) | 20:40 |
| mriedem | dansmith: ^ did you factor that into the keypair migration to the api db? | 20:41 |
| sdague | rlrossit: sure, it's just not the security issue you might think it is | 20:41 |
| mriedem | rlrossit: http://docs.openstack.org/developer/reno/usage.html | 20:41 |
| dansmith | mriedem: meaning are the timestamps copied? | 20:41 |
| *** raildo is now known as raildo-afk | 20:41 | |
| mriedem | http://lmgtfy.com/?q=docs+openstack+reno | 20:42 |
| mriedem | dansmith: well, the id | 20:42 |
| *** eharney has quit IRC | 20:42 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/keypairs.py#L43 | 20:42 |
| mriedem | ^ builds the keypair response body | 20:42 |
| *** thorst_ has joined #openstack-nova | 20:42 | |
| mriedem | attrs is defined here https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/keypairs.py#L214 | 20:43 |
| dansmith | mriedem: no, I'm not copying the id | 20:43 |
| mriedem | didn't we do that for flavors? | 20:43 |
| mriedem | because the db id is exposed in the api | 20:44 |
| dansmith | mriedem: they can't reference the keypair by id anywhere that I know of, so I don't think that they need to care whether it's stable or not | 20:44 |
| dansmith | mriedem: because flavors can be referenced by lots of things | 20:44 |
| dansmith | mriedem: and aggregates will be the same | 20:44 |
| *** rama_y has joined #openstack-nova | 20:44 | |
| *** gyee has joined #openstack-nova | 20:44 | |
| dansmith | if we do the same for keypair, we have to ban creation until they're moved, | 20:45 |
| *** wznoinsk has joined #openstack-nova | 20:45 | |
| dansmith | which is very bad for keypairs | 20:45 |
| dansmith | and since they can't refer to them by id, I don't think it matters | 20:45 |
| sdague | it's super weird that it's in the get call | 20:45 |
| sdague | but it's not referencable, so I agree with dansmith, it doesn't need to be kept the same | 20:45 |
| *** Sukhdev has quit IRC | 20:46 | |
| *** shaohe_feng has quit IRC | 20:47 | |
| *** shaohe_feng has joined #openstack-nova | 20:48 | |
| sdague | we should microversion the rando fields away | 20:48 |
| mriedem | you'd have to query on user_id and key name to replace id | 20:48 |
| mriedem | but that's fine | 20:49 |
| dansmith | mriedem: are you saying you *can* query on id? | 20:49 |
| dansmith | the method doesn't exist in the DB, AFAIK | 20:49 |
| mriedem | via the keypairs api | 20:49 |
| mriedem | well, sec | 20:49 |
| sdague | dansmith: no, just the get call returns it | 20:49 |
| sdague | the get is by keypair name | 20:49 |
| dansmith | thought so | 20:49 |
| *** armax has joined #openstack-nova | 20:50 | |
| mriedem | ah ok this was confusing me https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/keypairs.py#L213 | 20:50 |
| mriedem | id == name | 20:50 |
| dansmith | heh | 20:50 |
| sdague | yeh, the code in the api layer there is confusing | 20:51 |
| * mriedem changes that quick | 20:51 | |
| sdague | but once you get to the compute.api the parameter naming is less weird | 20:51 |
| *** andymaier has quit IRC | 20:51 | |
| alaski | melwitt: did you want to push up another rev for https://review.openstack.org/#/c/244046 ? | 20:52 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Update os-server-tags in default policy https://review.openstack.org/315757 | 20:52 |
| *** takashin has joined #openstack-nova | 20:52 | |
| *** jichen has quit IRC | 20:53 | |
| melwitt | alaski: yeah, sure | 20:54 |
| mriedem | sdague: yeah the keypair api uses key_name so it makes more sense | 20:54 |
| mriedem | i'll still update the rest api | 20:54 |
| mriedem | meeting in 5 minutes | 20:54 |
| dansmith | ah crap | 20:55 |
| mriedem | and we have a lot to talk about... | 20:56 |
| mriedem | so go to the bathroom now | 20:56 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add a RequestSpec generation migration script https://review.openstack.org/302828 | 20:57 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Verify requested_destination in the scheduler https://review.openstack.org/314509 | 20:57 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: API changes for verifying the scheduler when migrating https://review.openstack.org/315572 | 20:57 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Resize API operation passing down original RequestSpec https://review.openstack.org/312009 | 20:57 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add requested_destination field to RequestSpec https://review.openstack.org/312757 | 20:57 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Cold migrate using the RequestSpec object https://review.openstack.org/284974 | 20:57 |
| *** shaohe_feng has quit IRC | 20:57 | |
| *** shaohe_feng has joined #openstack-nova | 20:58 | |
| *** smatzek has joined #openstack-nova | 20:59 | |
| *** smatzek has quit IRC | 21:00 | |
| *** pushkaru has quit IRC | 21:00 | |
| *** jwcroppe_ has quit IRC | 21:01 | |
| *** sdake has joined #openstack-nova | 21:04 | |
| openstackgerrit | melanie witt proposed openstack/nova: Add a functional test for instance fault message with retry https://review.openstack.org/244046 | 21:05 |
| *** shaohe_feng has quit IRC | 21:07 | |
| *** ctrath has quit IRC | 21:08 | |
| *** ctrath has joined #openstack-nova | 21:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/313743 | 21:08 |
| *** shaohe_feng has joined #openstack-nova | 21:08 | |
| *** ducttape_ has quit IRC | 21:09 | |
| *** baoli_ has quit IRC | 21:10 | |
| *** burt has joined #openstack-nova | 21:11 | |
| *** xek has quit IRC | 21:11 | |
| *** pushkaru has joined #openstack-nova | 21:12 | |
| *** xek has joined #openstack-nova | 21:12 | |
| openstackgerrit | Emile Snyder proposed openstack/nova-specs: Attach/detach instance access from Manila shares https://review.openstack.org/310050 | 21:12 |
| *** lakshmiS has quit IRC | 21:13 | |
| *** ctrath has quit IRC | 21:13 | |
| *** ctrath has joined #openstack-nova | 21:14 | |
| *** ducttape_ has joined #openstack-nova | 21:14 | |
| *** shaohe_feng has quit IRC | 21:18 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds HostCapabilities object model https://review.openstack.org/285856 | 21:18 |
| *** Sukhdev has joined #openstack-nova | 21:18 | |
| *** shaohe_feng has joined #openstack-nova | 21:18 | |
| *** sdake_ has joined #openstack-nova | 21:24 | |
| *** thorst_ has quit IRC | 21:27 | |
| *** thorst_ has joined #openstack-nova | 21:27 | |
| *** shaohe_feng has quit IRC | 21:28 | |
| *** haplo37 has quit IRC | 21:28 | |
| *** sdake has quit IRC | 21:28 | |
| *** shaohe_feng has joined #openstack-nova | 21:28 | |
| *** sdake has joined #openstack-nova | 21:31 | |
| *** thorst_ has quit IRC | 21:31 | |
| *** sdake_ has quit IRC | 21:32 | |
| *** ctrath has quit IRC | 21:32 | |
| *** thorst_ has joined #openstack-nova | 21:33 | |
| *** gagehugo has quit IRC | 21:33 | |
| *** ametts has quit IRC | 21:33 | |
| *** takashin has left #openstack-nova | 21:34 | |
| *** dedery has joined #openstack-nova | 21:36 | |
| *** thorst_ has quit IRC | 21:36 | |
| *** thorst_ has joined #openstack-nova | 21:36 | |
| *** rlrossit has quit IRC | 21:37 | |
| *** dedery has quit IRC | 21:38 | |
| *** shaohe_feng has quit IRC | 21:38 | |
| *** shaohe_feng has joined #openstack-nova | 21:38 | |
| *** rtheis has quit IRC | 21:43 | |
| *** doug-fish has joined #openstack-nova | 21:43 | |
| *** ivase has quit IRC | 21:45 | |
| *** earlephilhower has quit IRC | 21:46 | |
| *** dedery has joined #openstack-nova | 21:47 | |
| *** admin0 has joined #openstack-nova | 21:47 | |
| admin0 | anyone around ? | 21:47 |
| admin0 | :D | 21:47 |
| mriedem | no | 21:47 |
| admin0 | if i see a req-XX on nova.instance_actions table and a stop command, do you know how do I trace the source of that request ?what provides that req-XX ? where is it logged(horizon or api , which IP etc ) | 21:47 |
| admin0 | :D | 21:47 |
| *** dedery has quit IRC | 21:47 | |
| *** bswartz has joined #openstack-nova | 21:47 | |
| admin0 | trying to figure out this | 21:48 |
| *** doug-fish has quit IRC | 21:48 | |
| mriedem | you can trace it through the nova logs | 21:48 |
| *** shaohe_feng has quit IRC | 21:48 | |
| dansmith | jroll: the patch you need just merged | 21:48 |
| mriedem | admin0: see http://specs.openstack.org/openstack/openstack-specs/specs/return-request-id.html | 21:49 |
| jroll | dansmith: yep, apparently people have been staring at zuul | 21:49 |
| alaski | mikal: https://review.openstack.org/#/c/244046 could use your eyes again, minor fixup | 21:49 |
| *** shaohe_feng has joined #openstack-nova | 21:49 | |
| dansmith | jroll: hah, sorry, I suck | 21:49 |
| *** ivase has joined #openstack-nova | 21:49 | |
| *** matrohon has quit IRC | 21:49 | |
| jroll | dansmith: lol | 21:50 |
| admin0 | mriedem: its the nova conductor right ? not th escheduler | 21:51 |
| admin0 | scheduler is internal | 21:51 |
| *** pushkaru has quit IRC | 21:52 | |
| *** lpetrut has quit IRC | 21:52 | |
| *** ivase has quit IRC | 21:53 | |
| admin0 | grep does not find anything :( | 21:54 |
| *** lpetrut has joined #openstack-nova | 21:55 | |
| mriedem | dansmith: https://developer.ibm.com/open/disaster-recovery-for-cloud-dragon/ | 21:55 |
| mriedem | i can just point the quiesce spec at that | 21:55 |
| *** earlephilhower has joined #openstack-nova | 21:55 | |
| dansmith | mriedem: fine with me | 21:56 |
| * dansmith & | 21:56 | |
| *** burt has quit IRC | 21:56 | |
| mikal | Doh, forgot it was nova meeting week | 21:57 |
| *** mdrabe has quit IRC | 21:57 | |
| melwitt | I'm having a hell of a time trying to find the original spec | 21:57 |
| melwitt | finally found via gerrit search | 21:58 |
| *** shaohe_feng has quit IRC | 21:59 | |
| *** thorst_ has quit IRC | 21:59 | |
| *** shaohe_feng has joined #openstack-nova | 21:59 | |
| mriedem | "In the described solution, we make no attempt to save the memory state of a workload. We assume that the workloads can be restarted and continue to run in the backup data center without restoring their memory state. This is a valid assumption for a large class of cloud-based services." | 22:00 |
| *** dmk0202 has quit IRC | 22:00 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Extend ResourceProvider to have resources and aggregates https://review.openstack.org/284963 | 22:00 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 22:00 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Remove mox from compute/test_compute_cells.py https://review.openstack.org/308021 | 22:01 |
| *** alexpilotti has quit IRC | 22:01 | |
| *** alexpilo_ has joined #openstack-nova | 22:01 | |
| mriedem | dansmith: ok, commented in the spec | 22:02 |
| mriedem | basically, the dragon project relies on drbd | 22:03 |
| mriedem | so, don't rely on nova to be managing everything for you | 22:03 |
| mriedem | but groups resources into a heat template | 22:03 |
| mriedem | so the workload can be moved to another DC | 22:03 |
| mriedem | and does periodic backups | 22:03 |
| mriedem | anyway, they should go talk to those guys and sort it out | 22:03 |
| mriedem | i don't want to add this to nova when there are existing solutions | 22:04 |
| *** cdelatte has joined #openstack-nova | 22:04 | |
| melwitt | mriedem: ah, that's the bit I was interested in. from what I understand, there isn't a way to do something like quiesce/freeze standalone on VMs other than something manual like running virsh commands on hypervisors | 22:04 |
| mriedem | melwitt: as the docs for that other DR project mention, they don't rely on freeze | 22:05 |
| mriedem | they assume your apps are written to handle this | 22:05 |
| mriedem | but they also attempt to minimize data loss with drbd | 22:05 |
| *** thorst_ has joined #openstack-nova | 22:05 | |
| *** thorst_ has quit IRC | 22:05 | |
| *** thorst_ has joined #openstack-nova | 22:06 | |
| *** Sukhdev has quit IRC | 22:07 | |
| melwitt | mriedem: yeah, that's what I was trying to say. I was going through reading their docs and you mentioned the part I was looking for | 22:08 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/313743 | 22:08 |
| *** Guest84855 has quit IRC | 22:08 | |
| dansmith | melwitt: right, it's very doable at the actual application layer | 22:08 |
| *** shaohe_feng has quit IRC | 22:09 | |
| *** shaohe_feng has joined #openstack-nova | 22:09 | |
| *** dedery has joined #openstack-nova | 22:09 | |
| *** dedery has quit IRC | 22:11 | |
| mriedem | however, | 22:11 |
| mriedem | they have legacy applications | 22:11 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: WIP Don't check cinder volume states during attach https://review.openstack.org/315789 | 22:11 |
| mriedem | and won't rewrite them | 22:11 |
| mriedem | so the iaas has to handle it | 22:12 |
| dansmith | oh, well, in that case... | 22:12 |
| mriedem | that was the argument for multiattach too | 22:12 |
| *** jwcroppe has joined #openstack-nova | 22:12 | |
| dansmith | ... | 22:12 |
| mriedem | this is a bridge for legacy apps to the cloud | 22:12 |
| mriedem | plus, $$$!!! | 22:12 |
| *** dimtruck is now known as zz_dimtruck | 22:12 | |
| dansmith | I do love $$$ | 22:12 |
| *** aysyd has quit IRC | 22:13 | |
| melwitt | dansmith: if you back nova with the right thing? just trying to understand this | 22:14 |
| dansmith | melwitt: to accomplish the consistent snapshot of multiple things? | 22:15 |
| melwitt | dansmith: yeah | 22:15 |
| dansmith | melwitt: you can flush and hold writes from inside the guests and coordinate to do a snapshot | 22:15 |
| *** thorst_ has quit IRC | 22:16 | |
| dansmith | the point of this is that you have say two VMs that are running each a half of an application, maybe each with their own db | 22:16 |
| dansmith | and you want to be able to snapshot both volumes together in case you have to roll back, so that one is not ahead or behind the other when you restore | 22:17 |
| dansmith | so all you have to do is coordinate that at the application layer | 22:17 |
| dansmith | it can be done a lot better there too if you save some point in your data and keep rolling so that if you need to roll back you can, but without freezing everything together to accomplish it | 22:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/313743 | 22:19 |
| melwitt | dansmith: thanks, that explains it well | 22:19 |
| *** shaohe_feng has quit IRC | 22:19 | |
| *** shaohe_feng has joined #openstack-nova | 22:20 | |
| mriedem | dansmith: you might want to mention that in reply to the 2 guys in the spec reviews | 22:20 |
| mriedem | *review | 22:20 |
| mriedem | in reply to the question of what is 'cloudy' | 22:22 |
| mriedem | and why this isn't it | 22:22 |
| *** thorst_ has joined #openstack-nova | 22:22 | |
| mriedem | and if you don't want to mention it, i can :) | 22:22 |
| melwitt | yeah, I must admit I was looking for "what should they do then" in the Alternatives in the spec or in a comment by someone. the Dragon project seems a perfect fit. assuming they don't have to be perfectly synced I guess | 22:22 |
| *** pushkaru has joined #openstack-nova | 22:22 | |
| *** andymaier has joined #openstack-nova | 22:22 | |
| *** jfcastro has joined #openstack-nova | 22:23 | |
| mriedem | melwitt: they'll say they have to be perfectly synced | 22:23 |
| *** imcsk8 has joined #openstack-nova | 22:23 | |
| mriedem | and dragon won't work for them | 22:23 |
| mriedem | because they have special unicorn legacy applications | 22:23 |
| melwitt | but it didn't seem like the quiesce API would accomplish a perfect sync either, would it? | 22:23 |
| *** ducttape_ has quit IRC | 22:23 | |
| imcsk8 | hello, when working with mitaka the endopint version should be 2 or 2.1? | 22:24 |
| *** lakshmiS has joined #openstack-nova | 22:24 | |
| *** jamielennox|away is now known as jamielennox | 22:24 | |
| *** ayoung has joined #openstack-nova | 22:25 | |
| dansmith | melwitt: it does because it freezes the vm at the point of quiesce | 22:25 |
| dansmith | and requires them to freeze one at a time, then snapshot them all, then unfreeze one at a time | 22:25 |
| jfcastro | any pointer to how nova manage swap, plz? | 22:25 |
| *** markvoelker_ has quit IRC | 22:25 | |
| mriedem | imcsk8: 2.1 | 22:25 |
| mriedem | v2.0 is deprecated | 22:26 |
| mriedem | *deprecated | 22:26 |
| mriedem | yeah they still have to do some hand-holding in the app calling the api | 22:26 |
| mriedem | 1. freeze vm1, wait for it to change state, 2. freeze vm2, wait for state change, etc | 22:26 |
| melwitt | dansmith: I mean if they call quiesce per VM (I thought it said the API is not taking a list) then they wouldn't all quiesce at the same time is what I was thinking | 22:26 |
| mriedem | rather than say sending a list of instance uuids to do in batch or something | 22:26 |
| mriedem | melwitt: they want them to go in order, not at the same time | 22:27 |
| imcsk8 | mriedem: thanks! | 22:27 |
| *** thorst_ has quit IRC | 22:27 | |
| *** [1]Thelo has joined #openstack-nova | 22:27 | |
| mriedem | "Usually Nova API will manipulate one VM per action. One proposal is to expose quiesce, unquiesce single API action on multiple VMs in order, this will break Nova API style and leads to implementation complexity." | 22:27 |
| dansmith | melwitt: they wouldn't all quiesce at the same time, right, but the snapshots would all be done while all of them are frozen | 22:27 |
| dansmith | mriedem: right my first complaint last time was that they wanted a "snapshot all these vms, kthx" operation | 22:28 |
| melwitt | okay, my bad. I thought they wanted to have the whole app across VMs snapshot at the same time | 22:28 |
| dansmith | which requires a lot of orchestration in nova | 22:28 |
| mriedem | make a server group and snapshot that :) | 22:28 |
| dansmith | melwitt: well, they want that, but that's really not doable | 22:28 |
| *** shaohe_feng has quit IRC | 22:29 | |
| dansmith | mriedem: oh wait, I know.. put all the instances on one compute node and just checkpoint the host! | 22:29 |
| *** shaohe_feng has joined #openstack-nova | 22:30 | |
| mriedem | so what we need is, snapshot in ironic | 22:30 |
| melwitt | I see. it seems like the quiesce API isn't a great solution, it sort of does it but not to what they really want | 22:30 |
| *** Thelo has quit IRC | 22:30 | |
| *** [1]Thelo is now known as Thelo | 22:30 | |
| dansmith | melwitt: that's why this is a poor analog for what they actually want, and not really that useful, IMHO | 22:30 |
| melwitt | dansmith: yeah, I see that point | 22:31 |
| mriedem | what i really want is food, so, time to leave | 22:32 |
| melwitt | food API spec | 22:32 |
| mriedem | if you're my wife, you eat the food in order | 22:33 |
| mriedem | sort of like this spec | 22:33 |
| melwitt | legacy eating | 22:33 |
| *** omnipresent has quit IRC | 22:33 | |
| mriedem | ha, i'll point that out to her | 22:33 |
| mriedem | and then explain it | 22:33 |
| mriedem | there will be much eye rolling | 22:34 |
| melwitt | :D | 22:34 |
| *** mriedem has quit IRC | 22:34 | |
| dansmith | heh | 22:34 |
| *** dave-mccowan has joined #openstack-nova | 22:34 | |
| *** mriedem has joined #openstack-nova | 22:34 | |
| *** mriedem has quit IRC | 22:35 | |
| *** dave-mcc_ has joined #openstack-nova | 22:36 | |
| imcsk8 | mriedem: the proper version is v2.25 right? | 22:37 |
| openstackgerrit | Michael Still proposed openstack/nova: Power on failures should put an instance into error https://review.openstack.org/258243 | 22:37 |
| *** mrkz has quit IRC | 22:38 | |
| *** shaohe_feng has quit IRC | 22:40 | |
| *** pwnall has quit IRC | 22:40 | |
| *** dave-mccowan has quit IRC | 22:40 | |
| *** shaohe_feng has joined #openstack-nova | 22:40 | |
| *** gjayavelu has quit IRC | 22:41 | |
| *** kfarr has quit IRC | 22:41 | |
| *** cdent has quit IRC | 22:41 | |
| *** lpetrut has quit IRC | 22:43 | |
| *** admin0 has quit IRC | 22:45 | |
| *** pushkaru has quit IRC | 22:46 | |
| *** rama_y has quit IRC | 22:46 | |
| *** shaohe_feng has quit IRC | 22:50 | |
| *** shaohe_feng has joined #openstack-nova | 22:50 | |
| *** rbridgeman_ has quit IRC | 22:53 | |
| *** gjayavelu has joined #openstack-nova | 22:55 | |
| *** shaohe_feng has quit IRC | 23:00 | |
| *** atuvenie_ has quit IRC | 23:00 | |
| *** lakshmiS has quit IRC | 23:01 | |
| *** shaohe_feng has joined #openstack-nova | 23:01 | |
| *** sdague has quit IRC | 23:01 | |
| *** jwcroppe_ has joined #openstack-nova | 23:03 | |
| *** ayoung has quit IRC | 23:03 | |
| *** zhangjn has quit IRC | 23:04 | |
| *** jwcroppe has quit IRC | 23:05 | |
| *** dave-mcc_ has quit IRC | 23:06 | |
| *** unicell has quit IRC | 23:06 | |
| *** xyang1 has quit IRC | 23:06 | |
| *** unicell has joined #openstack-nova | 23:07 | |
| *** zhangjn has joined #openstack-nova | 23:07 | |
| *** Sukhdev has joined #openstack-nova | 23:09 | |
| *** tonytan4ever has quit IRC | 23:09 | |
| *** shaohe_feng has quit IRC | 23:10 | |
| *** shaohe_feng has joined #openstack-nova | 23:13 | |
| *** andymaier has quit IRC | 23:16 | |
| *** shaohe_feng has quit IRC | 23:21 | |
| *** shaohe_feng has joined #openstack-nova | 23:21 | |
| *** gjayavelu has quit IRC | 23:22 | |
| *** Swami has quit IRC | 23:24 | |
| *** cdelatte has quit IRC | 23:25 | |
| *** markvoelker has joined #openstack-nova | 23:26 | |
| *** r-daneel has quit IRC | 23:27 | |
| *** j_king has quit IRC | 23:29 | |
| *** abalutoiu_ has quit IRC | 23:29 | |
| *** j_king has joined #openstack-nova | 23:29 | |
| *** markvoelker has quit IRC | 23:31 | |
| *** shaohe_feng has quit IRC | 23:31 | |
| *** shaohe_feng has joined #openstack-nova | 23:31 | |
| *** claudiub has quit IRC | 23:31 | |
| *** nic has quit IRC | 23:34 | |
| *** gjayavelu has joined #openstack-nova | 23:35 | |
| *** liusheng has quit IRC | 23:36 | |
| *** antonyfm has quit IRC | 23:36 | |
| *** liusheng has joined #openstack-nova | 23:36 | |
| *** testttttt has joined #openstack-nova | 23:38 | |
| *** shaohe_feng has quit IRC | 23:41 | |
| *** shaohe_feng has joined #openstack-nova | 23:42 | |
| *** testttttt has quit IRC | 23:44 | |
| *** mriedem has joined #openstack-nova | 23:46 | |
| mriedem | jaypipes: well, i got home, and mortimer had eaten laura's purse | 23:47 |
| *** Sukhdev has quit IRC | 23:48 | |
| *** epheo has quit IRC | 23:49 | |
| *** mdrabe has joined #openstack-nova | 23:49 | |
| *** shaohe_feng has quit IRC | 23:51 | |
| *** shaohe_feng has joined #openstack-nova | 23:52 | |
| *** gjayavelu has quit IRC | 23:54 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/313743 | 23:54 |
| *** pwnall has joined #openstack-nova | 23:58 | |
| *** tpepper has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!