| jroll | mgagne: whoa, that does look broken | 00:00 |
|---|---|---|
| *** signed8bit is now known as signed8bit_ZZZzz | 00:00 | |
| mgagne | jroll: should I just fix it or open a spec? =) | 00:00 |
| *** haomaiwa_ has quit IRC | 00:01 | |
| jroll | mgagne: probably a bug | 00:01 |
| mgagne | jroll: problem I see is that unit test are passing | 00:01 |
| jroll | mgagne: I think it works now because the last version is liberty, but as soon as we add another version it will work | 00:01 |
| mgagne | jroll: and I don't know how I could make them better | 00:01 |
| *** haomaiwang has joined #openstack-nova | 00:01 | |
| mgagne | jroll: file is showing in latest but not liberty | 00:01 |
| jroll | mgagne: even on master? | 00:02 |
| mgagne | jroll: I'm not running master | 00:02 |
| jroll | mgagne: right, so I'm wondering if it's a current bug on master or will present later | 00:03 |
| *** zhangjn has joined #openstack-nova | 00:03 | |
| jroll | now that I read more it looks like it is | 00:03 |
| *** mdorman has quit IRC | 00:04 | |
| *** heyongli has quit IRC | 00:05 | |
| *** heyongli has joined #openstack-nova | 00:05 | |
| jroll | mgagne: I'm having trouble finding a test that ensures the right stuff is in the right version | 00:06 |
| *** markvoelker has quit IRC | 00:06 | |
| mgagne | jroll: https://github.com/openstack/nova/commit/db1e92c2c854125f9bca3fd98654172fc5967637 | 00:06 |
| mgagne | jroll: nova/tests/unit/test_metadata.py | 00:06 |
| mgagne | jroll: test_returns_apis_supported_in_liberty_version | 00:07 |
| mgagne | jroll: but I think it's testing the "wrong" thing | 00:07 |
| *** baoli has quit IRC | 00:07 | |
| mgagne | jroll: there is an endpoint that returns the list of file found in the version but not the files themselves | 00:07 |
| jroll | mgagne: feels like test_network_data_presence should cover it, hrm | 00:08 |
| mgagne | jroll: I guess that could be it | 00:08 |
| jroll | oh maybe | 00:08 |
| mgagne | jroll: but you are right, test_network_data_presence should cover it, it's using the same method | 00:08 |
| *** stackdump has quit IRC | 00:09 | |
| *** otter768 has quit IRC | 00:10 | |
| mgagne | jroll: ok so | 00:10 |
| *** mtanino has quit IRC | 00:11 | |
| mgagne | jroll: it returns the list of files if you request a version. it returns the content if you request a filename | 00:11 |
| *** xyang1 has quit IRC | 00:11 | |
| jroll | mgagne: uh huh | 00:12 |
| jroll | mgagne: now that I look again, this bug is in metadata_for_config_drive so I have a feeling the metadata API doesn't hit this | 00:13 |
| mgagne | jroll: so test is using lookup which will return all your need if you request a filename. The actual injection is not done for the reasons we know. | 00:13 |
| jroll | and it appears there's no tests around metadata_for_config_drive there | 00:13 |
| mgagne | jroll: imo, there is nothing testing the actual injection, only the lookup mechanism | 00:13 |
| jroll | yep | 00:13 |
| jroll | agree | 00:13 |
| jroll | unless that's done elsewhere | 00:14 |
| *** dboik has joined #openstack-nova | 00:14 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova: Properly inject network_data.json in configdrive https://review.openstack.org/241824 | 00:14 |
| mgagne | so I'm not sure how people were using it :-/ | 00:14 |
| *** dboik has quit IRC | 00:14 | |
| jroll | well presumably we were always looking at latest | 00:15 |
| mgagne | unless they were reading from latest/ which is just a moving target and could be any version | 00:15 |
| *** stackdump has joined #openstack-nova | 00:15 | |
| *** heyongli has quit IRC | 00:15 | |
| jroll | did you see how long this took to land originally? slow moving target :) | 00:15 |
| mgagne | too slow if you want my opinion | 00:15 |
| mgagne | I implemented it myself before it got merged | 00:15 |
| jroll | lol :( | 00:16 |
| *** heyongli has joined #openstack-nova | 00:16 | |
| mgagne | now I'm working on cloud-init support which I fear will be soon thrown in the garbage anyway | 00:16 |
| jroll | why thrown in the garbage? because v1 not v2 | 00:16 |
| jroll | ? | 00:16 |
| jroll | also I feel like we have a thing for that | 00:17 |
| mgagne | jroll: yea https://github.com/jayofdoom/cloud-init-debian-pkg | 00:17 |
| jroll | heh ok | 00:17 |
| mgagne | jroll: yea, because cloud-init is being rewritten | 00:17 |
| jroll | right | 00:17 |
| jroll | so we're working on this upstream in v2 | 00:17 |
| jroll | we being onmetal people | 00:18 |
| mgagne | jroll: you know how it is, feature is due for yesterday | 00:18 |
| mriedem | mnestratov: what's the difference between virtuozzo CT and VM in the hypervisor support matrix? | 00:18 |
| jroll | mgagne: yeah :/ | 00:18 |
| mgagne | jroll: I wouldn't mind using and working on cloud-init v2 as long as it's prod ready and there are packages for all OS versions on earth | 00:19 |
| jroll | mgagne: we'll have it in prod soon(TM) | 00:19 |
| jroll | can't speak to the latter | 00:19 |
| mgagne | jroll: yea, everybody supports the latest LTS and what not. try packaging it for precise or some old centos6 =) | 00:20 |
| jroll | yep | 00:20 |
| jroll | we do support both of those, so maybe it'll happen, dunno | 00:20 |
| mgagne | packaging is the bane of my existence so when people suggests using the latest version of whatever software, it often comes down to having to package it yourself :-/ | 00:21 |
| *** dave-mccowan has joined #openstack-nova | 00:22 | |
| *** pratikma_ has quit IRC | 00:23 | |
| *** amotoki has quit IRC | 00:23 | |
| mriedem | mnestratov: i guess container and vm, but i thought they were all VMs I guess | 00:23 |
| mriedem | *containers i mean | 00:24 |
| *** su_zhang has quit IRC | 00:24 | |
| *** heyongli has quit IRC | 00:25 | |
| *** EinstCrazy has quit IRC | 00:25 | |
| *** zhangjn has quit IRC | 00:25 | |
| *** heyongli has joined #openstack-nova | 00:26 | |
| *** gonzalo2kx has quit IRC | 00:26 | |
| *** ccarmack has quit IRC | 00:27 | |
| *** eliqiao has joined #openstack-nova | 00:30 | |
| *** tongli has quit IRC | 00:30 | |
| *** jerrygb has joined #openstack-nova | 00:31 | |
| *** achanda has quit IRC | 00:32 | |
| *** jerrygb has quit IRC | 00:32 | |
| *** takedakn has quit IRC | 00:35 | |
| *** heyongli has quit IRC | 00:35 | |
| *** lbelivea has quit IRC | 00:36 | |
| *** heyongli has joined #openstack-nova | 00:36 | |
| *** ccarmack has joined #openstack-nova | 00:37 | |
| *** Marga_ has quit IRC | 00:39 | |
| *** erhudy has quit IRC | 00:39 | |
| *** Marga_ has joined #openstack-nova | 00:40 | |
| *** diogogmt has quit IRC | 00:42 | |
| *** nihilifer has quit IRC | 00:44 | |
| *** haomaiwang has quit IRC | 00:45 | |
| *** heyongli has quit IRC | 00:46 | |
| *** heyongli has joined #openstack-nova | 00:46 | |
| *** markvoelker has joined #openstack-nova | 00:47 | |
| *** armax has quit IRC | 00:47 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add soft deleting instance actions and events https://review.openstack.org/231887 | 00:48 |
| *** sileht has quit IRC | 00:48 | |
| *** armax has joined #openstack-nova | 00:49 | |
| *** armax has quit IRC | 00:50 | |
| *** beagles has quit IRC | 00:51 | |
| *** willsama has joined #openstack-nova | 00:52 | |
| *** b3nt_pin has joined #openstack-nova | 00:53 | |
| *** dave-mccowan has quit IRC | 00:53 | |
| *** b3nt_pin is now known as Guest46631 | 00:53 | |
| *** heyongli has quit IRC | 00:56 | |
| *** heyongli has joined #openstack-nova | 00:57 | |
| *** dave-mccowan has joined #openstack-nova | 00:59 | |
| *** hemna is now known as hemnafk | 00:59 | |
| *** spandhe has quit IRC | 01:00 | |
| *** zhangjn has joined #openstack-nova | 01:00 | |
| *** EinstCrazy has joined #openstack-nova | 01:01 | |
| *** heyongli has quit IRC | 01:06 | |
| *** Jeffrey4l has joined #openstack-nova | 01:06 | |
| *** heyongli has joined #openstack-nova | 01:07 | |
| *** stackdump has quit IRC | 01:07 | |
| *** IanGovett has joined #openstack-nova | 01:08 | |
| *** pratikmallya has joined #openstack-nova | 01:08 | |
| *** jmccrory has quit IRC | 01:09 | |
| *** zhenguo has joined #openstack-nova | 01:11 | |
| mriedem | rpodolyaka: snikitin: is there an oslo spec i can read for this https://review.openstack.org/#/c/221310/? | 01:11 |
| mriedem | it's not linked from the nova bp | 01:11 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id and user-id when list server-groups https://review.openstack.org/209018 | 01:12 |
| mriedem | rpodolyaka: snikitin: i guess it's this? http://specs.openstack.org/openstack/oslo-specs/specs/kilo/make-enginefacade-a-facade.html | 01:12 |
| *** y_sawai has joined #openstack-nova | 01:12 | |
| mriedem | or this http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/oslo_db-enginefacade.html | 01:13 |
| *** boris-42 has joined #openstack-nova | 01:13 | |
| *** eliqiao has left #openstack-nova | 01:13 | |
| *** aloga has quit IRC | 01:13 | |
| *** jmccrory has joined #openstack-nova | 01:13 | |
| *** jmccrory has quit IRC | 01:14 | |
| *** emagana_ has quit IRC | 01:14 | |
| *** ssurana has quit IRC | 01:14 | |
| *** aloga has joined #openstack-nova | 01:15 | |
| *** spandhe has joined #openstack-nova | 01:16 | |
| *** heyongli has quit IRC | 01:16 | |
| *** willsama has quit IRC | 01:17 | |
| *** electrocucaracha has quit IRC | 01:17 | |
| *** rfolco has joined #openstack-nova | 01:17 | |
| *** dave-mccowan has quit IRC | 01:17 | |
| *** y_sawai has quit IRC | 01:17 | |
| *** heyongli has joined #openstack-nova | 01:17 | |
| *** electrocucaracha has joined #openstack-nova | 01:19 | |
| *** ctrath has joined #openstack-nova | 01:19 | |
| alex_xu | morning~ | 01:22 |
| *** angdraug has quit IRC | 01:24 | |
| *** heyongli has quit IRC | 01:27 | |
| *** heyongli has joined #openstack-nova | 01:27 | |
| *** ByPasS has quit IRC | 01:28 | |
| *** ByPasS has joined #openstack-nova | 01:29 | |
| *** pratikmallya has quit IRC | 01:32 | |
| mriedem | alex_xu: hi | 01:33 |
| *** diogogmt has joined #openstack-nova | 01:33 | |
| alex_xu | mriedem: hi | 01:33 |
| *** electrocucaracha has quit IRC | 01:34 | |
| *** tjones has quit IRC | 01:35 | |
| *** heyongli has quit IRC | 01:37 | |
| *** IanGovett has quit IRC | 01:37 | |
| *** heyongli has joined #openstack-nova | 01:38 | |
| *** baoli has joined #openstack-nova | 01:39 | |
| *** josecastroleon has quit IRC | 01:40 | |
| *** rfolco has quit IRC | 01:42 | |
| *** su_zhang has joined #openstack-nova | 01:43 | |
| *** tjones has joined #openstack-nova | 01:45 | |
| *** heyongli has quit IRC | 01:47 | |
| *** heyongli has joined #openstack-nova | 01:48 | |
| *** smatzek has joined #openstack-nova | 01:49 | |
| *** dims has joined #openstack-nova | 01:49 | |
| *** pixelbeat_ has quit IRC | 01:51 | |
| *** baoli has quit IRC | 01:51 | |
| *** ndipanov has joined #openstack-nova | 01:52 | |
| *** tjones has quit IRC | 01:55 | |
| *** heyongli has quit IRC | 01:57 | |
| *** junjie has joined #openstack-nova | 01:57 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 01:58 | |
| *** ashyoung has quit IRC | 01:58 | |
| *** heyongli has joined #openstack-nova | 01:58 | |
| *** tjones has joined #openstack-nova | 01:59 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/python-novaclient: Add v2 support for optional project_id in version discovery urls https://review.openstack.org/238366 | 02:00 |
| *** tjones has quit IRC | 02:04 | |
| *** dave-mccowan has joined #openstack-nova | 02:07 | |
| *** heyongli has quit IRC | 02:08 | |
| *** jmccrory has joined #openstack-nova | 02:08 | |
| *** breitz1 has joined #openstack-nova | 02:08 | |
| *** heyongli has joined #openstack-nova | 02:08 | |
| *** seanw2 has joined #openstack-nova | 02:09 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/python-novaclient: Add v2 support for optional project_id in version discovery urls https://review.openstack.org/238366 | 02:09 |
| *** seanw1 has quit IRC | 02:10 | |
| *** su_zhang has quit IRC | 02:11 | |
| *** breitz has quit IRC | 02:11 | |
| *** breitz2 has joined #openstack-nova | 02:11 | |
| *** otter768 has joined #openstack-nova | 02:11 | |
| *** seanw1 has joined #openstack-nova | 02:12 | |
| *** breitz1 has quit IRC | 02:13 | |
| *** seanw2 has quit IRC | 02:14 | |
| *** pratikmallya has joined #openstack-nova | 02:15 | |
| *** otter768 has quit IRC | 02:16 | |
| *** ndipanov has quit IRC | 02:18 | |
| *** heyongli has quit IRC | 02:18 | |
| *** heyongli has joined #openstack-nova | 02:19 | |
| *** achanda has joined #openstack-nova | 02:27 | |
| *** heyongli has quit IRC | 02:28 | |
| *** heyongli has joined #openstack-nova | 02:29 | |
| mriedem | need another core to get this in (stable core) https://review.openstack.org/#/c/234100/ | 02:34 |
| *** vilobhmm11 has quit IRC | 02:35 | |
| *** spandhe has quit IRC | 02:35 | |
| *** inteq has joined #openstack-nova | 02:36 | |
| *** heyongli has quit IRC | 02:38 | |
| *** heyongli has joined #openstack-nova | 02:39 | |
| *** otter768 has joined #openstack-nova | 02:39 | |
| mriedem | johnthetubaguy: we might be better off trying to get an lxc job working with nbd rather than lvm | 02:49 |
| mriedem | https://review.openstack.org/#/c/219448/ | 02:49 |
| *** heyongli has quit IRC | 02:49 | |
| mriedem | ^ has only 3 failures http://logs.openstack.org/48/219448/5/check/gate-tempest-dsvm-full/6b6aa85/console.html.gz compared to like 35 in the lvm job | 02:49 |
| *** heyongli has joined #openstack-nova | 02:49 | |
| *** armax has joined #openstack-nova | 02:51 | |
| *** oomichi has joined #openstack-nova | 02:55 | |
| *** haomaiwang has joined #openstack-nova | 02:55 | |
| *** dave-mcc_ has joined #openstack-nova | 02:55 | |
| *** dave-mccowan has quit IRC | 02:56 | |
| *** dave-mccowan has joined #openstack-nova | 02:58 | |
| *** heyongli has quit IRC | 02:59 | |
| mriedem | jdurgin: cburgess: can someone amend this spec with the comments from jaypipes about the glance v2 dependency? https://review.openstack.org/#/c/188244/ | 02:59 |
| *** heyongli has joined #openstack-nova | 03:00 | |
| *** haomaiwang has quit IRC | 03:00 | |
| flyingtt_ | hi, all, how can i create a new domain? | 03:01 |
| *** dave-mcc_ has quit IRC | 03:01 | |
| flyingtt_ | i have found the cli reference, but got nothing. | 03:01 |
| *** ctrath has quit IRC | 03:04 | |
| *** Jeffrey4l has quit IRC | 03:05 | |
| *** mriedem has quit IRC | 03:07 | |
| *** heyongli has quit IRC | 03:09 | |
| *** mc_nair has quit IRC | 03:10 | |
| *** heyongli has joined #openstack-nova | 03:10 | |
| *** armax has quit IRC | 03:10 | |
| *** otter768 has quit IRC | 03:11 | |
| *** zhangjn_ has joined #openstack-nova | 03:15 | |
| *** zhangjn has quit IRC | 03:17 | |
| *** haomaiwang has joined #openstack-nova | 03:19 | |
| *** armax has joined #openstack-nova | 03:19 | |
| *** heyongli has quit IRC | 03:19 | |
| *** heyongli has joined #openstack-nova | 03:20 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id and user-id when list server-groups https://review.openstack.org/209018 | 03:25 |
| *** Sree has joined #openstack-nova | 03:25 | |
| *** Sree has quit IRC | 03:26 | |
| *** Sree has joined #openstack-nova | 03:26 | |
| *** tobe has joined #openstack-nova | 03:27 | |
| *** heyongli has quit IRC | 03:30 | |
| *** heyongli has joined #openstack-nova | 03:31 | |
| *** armax has quit IRC | 03:36 | |
| *** yangyapeng has joined #openstack-nova | 03:40 | |
| *** heyongli has quit IRC | 03:40 | |
| *** oomichi_ has joined #openstack-nova | 03:40 | |
| *** heyongli has joined #openstack-nova | 03:41 | |
| *** sahid has joined #openstack-nova | 03:41 | |
| *** oomichi has quit IRC | 03:42 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 03:42 | |
| *** smatzek has quit IRC | 03:42 | |
| *** dave-mccowan has quit IRC | 03:44 | |
| *** pratikmallya has quit IRC | 03:44 | |
| *** hemna has joined #openstack-nova | 03:47 | |
| *** haomaiwang has quit IRC | 03:48 | |
| *** heyongli has quit IRC | 03:50 | |
| *** heyongli has joined #openstack-nova | 03:51 | |
| *** markvoelker has quit IRC | 03:54 | |
| *** eliqiao has joined #openstack-nova | 03:57 | |
| *** r-daneel has quit IRC | 03:58 | |
| *** pratikmallya has joined #openstack-nova | 03:59 | |
| *** klkumar has joined #openstack-nova | 03:59 | |
| *** heyongli has quit IRC | 04:00 | |
| *** heyongli has joined #openstack-nova | 04:01 | |
| *** baoli has joined #openstack-nova | 04:01 | |
| *** spandhe has joined #openstack-nova | 04:01 | |
| *** unicell has joined #openstack-nova | 04:02 | |
| *** unicell1 has quit IRC | 04:02 | |
| *** pratikmallya has quit IRC | 04:03 | |
| *** armax has joined #openstack-nova | 04:04 | |
| *** venkat_p has joined #openstack-nova | 04:04 | |
| *** venkat_p has quit IRC | 04:05 | |
| *** sahid has quit IRC | 04:05 | |
| *** edtubill has joined #openstack-nova | 04:08 | |
| *** Jeffrey4l has joined #openstack-nova | 04:09 | |
| *** heyongli has quit IRC | 04:11 | |
| *** heyongli has joined #openstack-nova | 04:12 | |
| *** vilobhmm11 has joined #openstack-nova | 04:14 | |
| *** eliqiao1 has joined #openstack-nova | 04:14 | |
| *** eliqiao has quit IRC | 04:16 | |
| *** armax has quit IRC | 04:17 | |
| *** heyongli has quit IRC | 04:21 | |
| *** achanda has quit IRC | 04:22 | |
| *** heyongli has joined #openstack-nova | 04:22 | |
| *** A110602877_ has quit IRC | 04:26 | |
| *** links has joined #openstack-nova | 04:26 | |
| *** garyk1 has joined #openstack-nova | 04:31 | |
| *** heyongli has quit IRC | 04:31 | |
| *** heyongli has joined #openstack-nova | 04:32 | |
| *** haomaiwa_ has joined #openstack-nova | 04:32 | |
| *** signed8bit_ZZZzz has quit IRC | 04:32 | |
| *** su_zhang has joined #openstack-nova | 04:36 | |
| *** heyongli has quit IRC | 04:41 | |
| *** heyongli has joined #openstack-nova | 04:42 | |
| *** baoli has quit IRC | 04:44 | |
| *** sileht has joined #openstack-nova | 04:45 | |
| *** heyongli has quit IRC | 04:52 | |
| *** heyongli has joined #openstack-nova | 04:53 | |
| *** markvoelker has joined #openstack-nova | 04:55 | |
| *** augiemena3 has joined #openstack-nova | 04:57 | |
| *** hemna has quit IRC | 05:00 | |
| *** markvoelker has quit IRC | 05:00 | |
| *** haomaiwa_ has quit IRC | 05:01 | |
| *** haomaiwa_ has joined #openstack-nova | 05:01 | |
| *** su_zhang has quit IRC | 05:02 | |
| *** heyongli has quit IRC | 05:02 | |
| *** klkumar has quit IRC | 05:02 | |
| *** heyongli has joined #openstack-nova | 05:03 | |
| *** jwcroppe has joined #openstack-nova | 05:11 | |
| *** otter768 has joined #openstack-nova | 05:11 | |
| *** heyongli has quit IRC | 05:12 | |
| *** heyongli has joined #openstack-nova | 05:13 | |
| *** klkumar has joined #openstack-nova | 05:14 | |
| *** otter768 has quit IRC | 05:16 | |
| *** achanda has joined #openstack-nova | 05:22 | |
| *** heyongli has quit IRC | 05:22 | |
| *** heyongli has joined #openstack-nova | 05:23 | |
| *** irenab has quit IRC | 05:24 | |
| *** achanda has quit IRC | 05:30 | |
| *** klkumar has quit IRC | 05:30 | |
| *** heyongli has quit IRC | 05:33 | |
| *** markmc has quit IRC | 05:33 | |
| *** heyongli has joined #openstack-nova | 05:34 | |
| *** zhangjn_ has quit IRC | 05:39 | |
| *** nihilifer has joined #openstack-nova | 05:39 | |
| *** zhangjn has joined #openstack-nova | 05:41 | |
| *** achanda has joined #openstack-nova | 05:42 | |
| *** spandhe has quit IRC | 05:42 | |
| *** heyongli has quit IRC | 05:43 | |
| *** fawadkhaliq has quit IRC | 05:43 | |
| *** haomaiwa_ has quit IRC | 05:44 | |
| *** heyongli has joined #openstack-nova | 05:44 | |
| *** fawadkhaliq has joined #openstack-nova | 05:44 | |
| *** edleafe has quit IRC | 05:45 | |
| *** edleafe has joined #openstack-nova | 05:46 | |
| *** salv-orlando has quit IRC | 05:46 | |
| *** klkumar has joined #openstack-nova | 05:46 | |
| *** salv-orlando has joined #openstack-nova | 05:46 | |
| *** klkumar has quit IRC | 05:52 | |
| *** klkumar has joined #openstack-nova | 05:52 | |
| *** heyongli has quit IRC | 05:53 | |
| *** heyongli has joined #openstack-nova | 05:54 | |
| *** jwcroppe has quit IRC | 05:54 | |
| *** salv-orl_ has joined #openstack-nova | 05:55 | |
| *** salv-orlando has quit IRC | 05:57 | |
| *** boris-42 has quit IRC | 05:58 | |
| *** Daisy has joined #openstack-nova | 05:58 | |
| *** heyongli has quit IRC | 06:03 | |
| *** heyongli has joined #openstack-nova | 06:04 | |
| *** abhishekk has joined #openstack-nova | 06:09 | |
| *** heyongli has quit IRC | 06:14 | |
| *** heyongli has joined #openstack-nova | 06:15 | |
| *** venkat_p has joined #openstack-nova | 06:19 | |
| *** rajesht has joined #openstack-nova | 06:19 | |
| *** venkat_p has left #openstack-nova | 06:19 | |
| *** vilobhmm111 has joined #openstack-nova | 06:20 | |
| *** deepakcs has joined #openstack-nova | 06:21 | |
| *** vilobhmm11 has quit IRC | 06:23 | |
| *** chinmay_ has joined #openstack-nova | 06:23 | |
| *** heyongli has quit IRC | 06:24 | |
| *** sileht has quit IRC | 06:25 | |
| *** heyongli has joined #openstack-nova | 06:25 | |
| *** irenab has joined #openstack-nova | 06:26 | |
| *** heyongli has quit IRC | 06:34 | |
| *** heyongli has joined #openstack-nova | 06:35 | |
| *** sileht has joined #openstack-nova | 06:42 | |
| *** heyongli has quit IRC | 06:44 | |
| *** heyongli has joined #openstack-nova | 06:45 | |
| *** spandhe has joined #openstack-nova | 06:47 | |
| *** heyongli has quit IRC | 06:55 | |
| *** baoli has joined #openstack-nova | 06:56 | |
| *** heyongli has joined #openstack-nova | 06:56 | |
| *** markvoelker has joined #openstack-nova | 06:56 | |
| *** Daisy has quit IRC | 06:58 | |
| *** Daisy has joined #openstack-nova | 06:59 | |
| *** cfriesen_ has quit IRC | 07:00 | |
| *** baoli has quit IRC | 07:00 | |
| *** gjayavel_ has joined #openstack-nova | 07:00 | |
| *** eliqiao1 is now known as eliqiao | 07:01 | |
| *** markvoelker has quit IRC | 07:01 | |
| *** gjayavel_ has quit IRC | 07:01 | |
| *** heyongli has quit IRC | 07:05 | |
| *** heyongli has joined #openstack-nova | 07:06 | |
| *** josecastroleon has joined #openstack-nova | 07:06 | |
| *** Daisy has quit IRC | 07:09 | |
| *** Daisy has joined #openstack-nova | 07:09 | |
| *** haomaiwang has joined #openstack-nova | 07:11 | |
| *** otter768 has joined #openstack-nova | 07:12 | |
| *** gongysh has quit IRC | 07:13 | |
| *** gongysh has joined #openstack-nova | 07:13 | |
| *** fawadkhaliq has quit IRC | 07:14 | |
| *** Nathan has joined #openstack-nova | 07:14 | |
| *** sileht has quit IRC | 07:15 | |
| *** gongysh has quit IRC | 07:15 | |
| *** heyongli has quit IRC | 07:15 | |
| *** gongysh has joined #openstack-nova | 07:15 | |
| *** heyongli has joined #openstack-nova | 07:16 | |
| *** otter768 has quit IRC | 07:17 | |
| *** longsube has joined #openstack-nova | 07:17 | |
| *** moshele has joined #openstack-nova | 07:18 | |
| *** moshele has quit IRC | 07:19 | |
| *** moshele has joined #openstack-nova | 07:19 | |
| *** edtubill has quit IRC | 07:20 | |
| *** lsmola has joined #openstack-nova | 07:20 | |
| *** alex_klimov has joined #openstack-nova | 07:22 | |
| *** ZZelle_away is now known as ZZelle | 07:23 | |
| *** jwcroppe has joined #openstack-nova | 07:25 | |
| *** tojuvone has joined #openstack-nova | 07:25 | |
| *** heyongli has quit IRC | 07:25 | |
| *** heyongli has joined #openstack-nova | 07:26 | |
| *** fawadkhaliq has joined #openstack-nova | 07:29 | |
| *** alex_klimov has quit IRC | 07:31 | |
| *** alex_klimov has joined #openstack-nova | 07:31 | |
| *** zhangjn has quit IRC | 07:33 | |
| *** heyongli has quit IRC | 07:36 | |
| *** fawadkhaliq has quit IRC | 07:36 | |
| *** fawadkhaliq has joined #openstack-nova | 07:36 | |
| *** heyongli has joined #openstack-nova | 07:36 | |
| *** nkrinner has joined #openstack-nova | 07:40 | |
| *** Marga_ has quit IRC | 07:40 | |
| *** Daisy has quit IRC | 07:41 | |
| *** Daisy has joined #openstack-nova | 07:41 | |
| *** Marga_ has joined #openstack-nova | 07:44 | |
| *** Daisy has quit IRC | 07:46 | |
| *** heyongli has quit IRC | 07:46 | |
| *** heyongli has joined #openstack-nova | 07:47 | |
| *** vilobhmm111 has quit IRC | 07:47 | |
| *** y_sawai has joined #openstack-nova | 07:47 | |
| *** e0ne has joined #openstack-nova | 07:51 | |
| *** romainh has joined #openstack-nova | 07:51 | |
| *** alexschm has joined #openstack-nova | 07:51 | |
| *** Jeffrey4l has quit IRC | 07:53 | |
| *** zigo has quit IRC | 07:53 | |
| *** jamielennox is now known as jamielennox|away | 07:54 | |
| *** jamielennox|away is now known as jamielennox | 07:54 | |
| *** zigo has joined #openstack-nova | 07:56 | |
| *** heyongli has quit IRC | 07:56 | |
| *** heyongli has joined #openstack-nova | 07:57 | |
| *** jamielennox is now known as jamielennox|away | 07:57 | |
| *** rotbeard has joined #openstack-nova | 07:58 | |
| *** sileht has joined #openstack-nova | 07:59 | |
| *** e0ne has quit IRC | 08:00 | |
| *** zhangjn has joined #openstack-nova | 08:00 | |
| *** Sree_ has joined #openstack-nova | 08:00 | |
| *** Sree_ is now known as Guest95305 | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** rk4n has joined #openstack-nova | 08:02 | |
| *** Sree has quit IRC | 08:03 | |
| *** shuquan has joined #openstack-nova | 08:03 | |
| *** Sree has joined #openstack-nova | 08:03 | |
| *** e0ne has joined #openstack-nova | 08:04 | |
| *** Guest95305 has quit IRC | 08:05 | |
| *** Jeffrey4l has joined #openstack-nova | 08:05 | |
| *** markus_z has joined #openstack-nova | 08:05 | |
| *** heyongli has quit IRC | 08:06 | |
| *** heyongli has joined #openstack-nova | 08:07 | |
| *** shuquan has quit IRC | 08:09 | |
| *** rdopiera has joined #openstack-nova | 08:09 | |
| *** Jeffrey4l has quit IRC | 08:10 | |
| *** Daisy has joined #openstack-nova | 08:11 | |
| *** fawadkhaliq has quit IRC | 08:15 | |
| *** fawadkhaliq has joined #openstack-nova | 08:15 | |
| *** gszasz has joined #openstack-nova | 08:15 | |
| *** openstackgerrit has quit IRC | 08:16 | |
| *** jichen has joined #openstack-nova | 08:16 | |
| *** heyongli has quit IRC | 08:17 | |
| *** openstackgerrit has joined #openstack-nova | 08:17 | |
| *** heyongli has joined #openstack-nova | 08:17 | |
| *** ihrachys has joined #openstack-nova | 08:19 | |
| bauzas | good morning Nova | 08:19 |
| *** josecastroleon has quit IRC | 08:20 | |
| *** edleafe has quit IRC | 08:22 | |
| *** Jeffrey4l has joined #openstack-nova | 08:23 | |
| *** achanda has quit IRC | 08:25 | |
| *** fawadk has joined #openstack-nova | 08:26 | |
| *** heyongli has quit IRC | 08:27 | |
| *** Marga_ has quit IRC | 08:27 | |
| *** heyongli has joined #openstack-nova | 08:28 | |
| *** Marga_ has joined #openstack-nova | 08:28 | |
| *** Jeffrey4l has quit IRC | 08:29 | |
| *** fawadkhaliq has quit IRC | 08:29 | |
| andrearosa | morning | 08:31 |
| *** openstack has joined #openstack-nova | 08:34 | |
| tangchen | Hi, morning :) | 08:35 |
| *** rk4n has quit IRC | 08:37 | |
| *** heyongli has quit IRC | 08:37 | |
| *** ZZelle has quit IRC | 08:38 | |
| *** heyongli has joined #openstack-nova | 08:38 | |
| *** ZZelle has joined #openstack-nova | 08:38 | |
| *** shuquan has joined #openstack-nova | 08:39 | |
| *** huats has joined #openstack-nova | 08:39 | |
| *** Daisy has quit IRC | 08:39 | |
| *** Daisy has joined #openstack-nova | 08:40 | |
| *** Daisy has joined #openstack-nova | 08:40 | |
| *** Jeffrey4l has joined #openstack-nova | 08:41 | |
| *** jlanoux has joined #openstack-nova | 08:43 | |
| *** e0ne has quit IRC | 08:45 | |
| *** EinstCrazy has quit IRC | 08:45 | |
| *** Daisy has quit IRC | 08:45 | |
| *** spandhe has quit IRC | 08:45 | |
| *** Daisy has joined #openstack-nova | 08:46 | |
| *** e0ne has joined #openstack-nova | 08:46 | |
| *** amotoki has joined #openstack-nova | 08:46 | |
| *** heyongli has quit IRC | 08:47 | |
| *** amotoki has quit IRC | 08:48 | |
| *** e0ne has quit IRC | 08:48 | |
| *** heyongli has joined #openstack-nova | 08:48 | |
| bauzas | johnthetubaguy: morning, when you have time, I'd like to discuss about the new relnotes and how to write them correctly for https://review.openstack.org/#/c/197912/35 | 08:49 |
| *** Daisy has quit IRC | 08:50 | |
| *** EinstCrazy has joined #openstack-nova | 08:50 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Versioned notification API https://review.openstack.org/224755 | 08:52 |
| *** cdent has joined #openstack-nova | 08:54 | |
| *** markvoelker has joined #openstack-nova | 08:57 | |
| *** spandhe has joined #openstack-nova | 08:57 | |
| *** heyongli has quit IRC | 08:58 | |
| *** heyongli has joined #openstack-nova | 08:58 | |
| *** shuquan has quit IRC | 08:59 | |
| *** josecastroleon has joined #openstack-nova | 09:00 | |
| *** Sree_ has joined #openstack-nova | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** Sree_ is now known as Guest61212 | 09:01 | |
| *** markvoelker has quit IRC | 09:01 | |
| *** matrohon has joined #openstack-nova | 09:02 | |
| *** lyarwood has joined #openstack-nova | 09:02 | |
| *** Jeffrey4l has quit IRC | 09:02 | |
| *** Sree has quit IRC | 09:02 | |
| *** haomaiwa_ has joined #openstack-nova | 09:07 | |
| *** danpb has joined #openstack-nova | 09:08 | |
| *** heyongli has quit IRC | 09:08 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: doc: update threading.rst https://review.openstack.org/199013 | 09:08 |
| *** heyongli has joined #openstack-nova | 09:09 | |
| *** browne has quit IRC | 09:09 | |
| *** derekh has joined #openstack-nova | 09:09 | |
| *** shuquan has joined #openstack-nova | 09:12 | |
| *** otter768 has joined #openstack-nova | 09:13 | |
| *** Guest61212 has quit IRC | 09:13 | |
| *** MarcF has joined #openstack-nova | 09:14 | |
| *** Nathan has quit IRC | 09:14 | |
| *** rk4n has joined #openstack-nova | 09:17 | |
| *** heyongli has quit IRC | 09:18 | |
| *** otter768 has quit IRC | 09:18 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova-specs: virtozzo storage volume driver in libvirt https://review.openstack.org/184295 | 09:18 |
| *** heyongli has joined #openstack-nova | 09:19 | |
| *** Jeffrey4l has joined #openstack-nova | 09:19 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Use EngineFacade from oslo_db.enginefacade https://review.openstack.org/221310 | 09:20 |
| *** yassine__ has joined #openstack-nova | 09:21 | |
| *** salv-orlando has joined #openstack-nova | 09:21 | |
| *** salv-orlando has quit IRC | 09:22 | |
| *** jistr has joined #openstack-nova | 09:24 | |
| *** salv-orl_ has quit IRC | 09:24 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: add nova volume driver for vzstorage https://review.openstack.org/190843 | 09:25 |
| *** heyongli has quit IRC | 09:28 | |
| *** heyongli has joined #openstack-nova | 09:29 | |
| *** sahid has joined #openstack-nova | 09:30 | |
| *** longsube has quit IRC | 09:35 | |
| *** longsube has joined #openstack-nova | 09:35 | |
| openstackgerrit | huan proposed openstack/nova: Resolve Nova/Neutron race condition https://review.openstack.org/241127 | 09:36 |
| *** spandhe has quit IRC | 09:36 | |
| *** shuquan has quit IRC | 09:38 | |
| *** heyongli has quit IRC | 09:38 | |
| *** heyongli has joined #openstack-nova | 09:39 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add project-id and user-id when list server-groups https://review.openstack.org/209018 | 09:39 |
| *** rha has joined #openstack-nova | 09:42 | |
| *** pratikmallya has quit IRC | 09:42 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'instance_info' and 'instance_extra' https://review.openstack.org/225065 | 09:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'cell_mapping' https://review.openstack.org/226788 | 09:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'instance_mapping' https://review.openstack.org/228979 | 09:44 |
| alex_xu | bauzas: or we should keep decorator 'compat_legacy_props' for a release? | 09:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'key_pair' and 'cell' https://review.openstack.org/239915 | 09:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'network' https://review.openstack.org/240253 | 09:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'certificate' and 'pci_device' https://review.openstack.org/240275 | 09:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: enginefacade: 'floating_ip' https://review.openstack.org/240513 | 09:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: enginefacade: 'fixed_ip' https://review.openstack.org/240811 | 09:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'service' https://review.openstack.org/241178 | 09:44 |
| bauzas | alex_xu: I don't like leaving a decorator only for out-of-tree | 09:45 |
| bauzas | alex_xu: but I can write some relnote docs saying how to primitive the spec object if needed | 09:45 |
| alex_xu | bauzas: ok, 'out-of-tree' is a point | 09:45 |
| *** haomaiwa_ has quit IRC | 09:46 | |
| *** haomaiwang has joined #openstack-nova | 09:46 | |
| johnthetubaguy | bauzas: I am going to fetch a coffee, then lets discuss | 09:46 |
| johnthetubaguy | alex_xu: bauzas: whats "compat_legacy_props" ? | 09:47 |
| bauzas | johnthetubaguy: sure, take time | 09:47 |
| bauzas | that discussion probably needs some other coffee for me | 09:48 |
| *** Jeffrey4l has quit IRC | 09:48 | |
| alex_xu | johnthetubaguy: a decorator to keep the old scheduler filter work with old filter_properties and request_spec dict | 09:48 |
| *** heyongli has quit IRC | 09:48 | |
| *** heyongli has joined #openstack-nova | 09:49 | |
| johnthetubaguy | alex_xu: oh right, I think the issue here is having a mechanism for the correct set of filters and weights dictating the correct validation, or something like that, its a bit messy, but anyways, something for next release I feel | 09:51 |
| * johnthetubaguy actually gets a coffee... | 09:51 | |
| bauzas | johnthetubaguy: so, my point is that I feel https://review.openstack.org/#/c/197912/35 needs an UpgradeImpact since it changes the interface for custom filters | 09:52 |
| bauzas | johnthetubaguy: in order to run the existing filters without changing them, I provided a compat decorator but that's an implementation detail | 09:53 |
| *** e0ne has joined #openstack-nova | 09:53 | |
| bauzas | any other out-of-tree filter could decide either to write its own decorator, or change the method attributes | 09:53 |
| bauzas | that's his choice | 09:53 |
| bauzas | I don't want to be opianiated and say 'you need to use that compat deco for your own filter' since it's something for in-tree only that's removed once the filter are all updated | 09:54 |
| johnthetubaguy | bauzas: oh, thats a fair point | 09:54 |
| bauzas | but I want to make it clear that custom filters would be broken unless the operator does a compat layer | 09:55 |
| *** jwcroppe has quit IRC | 09:55 | |
| bauzas | in that relnote, I would use the compat deco as an example, but just as an example, not more than that | 09:55 |
| johnthetubaguy | bauzas: I think it would be nice to keep that decorator in the tree for at least one cycle, with big warnings | 09:56 |
| bauzas | for example, if RAX decides to just modify their custom filters without using the deco, I'm fine | 09:56 |
| *** jwcroppe has joined #openstack-nova | 09:56 | |
| johnthetubaguy | bauzas: well big warnings in the release notes | 09:56 |
| bauzas | johnthetubaguy: AFAICT, jaypipes was a bit concerned with keeping that deco in-tree | 09:56 |
| bauzas | lemme doublecheck | 09:56 |
| johnthetubaguy | bauzas: agreed, FWIW I think we only have custom cells filters/weights, but anyways | 09:56 |
| johnthetubaguy | bauzas: seems like a small price to pay, to make it easier for folks | 09:57 |
| *** salv-orlando has joined #openstack-nova | 09:57 | |
| bauzas | johnthetubaguy: that's why I thought providing that decorator in a doc example for the relnote, not in-tree, see ? | 09:57 |
| *** markvoelker has joined #openstack-nova | 09:57 | |
| johnthetubaguy | alex_xu: so I was talking about the API validation before, for some reasons, ignore those comments! | 09:57 |
| johnthetubaguy | bauzas: oh, that could work | 09:58 |
| alex_xu | johnthetubaguy: np | 09:58 |
| *** browne has joined #openstack-nova | 09:58 | |
| *** zhenguo has quit IRC | 09:58 | |
| *** heyongli has quit IRC | 09:58 | |
| johnthetubaguy | alex_xu: I shouldn't try to think pre coffee, it never goes well | 09:59 |
| alex_xu | :) | 09:59 |
| bauzas | johnthetubaguy: have you seen http://lists.openstack.org/pipermail/openstack-dev/2015-November/078301.html yet ? | 09:59 |
| *** heyongli has joined #openstack-nova | 09:59 | |
| bauzas | johnthetubaguy: that looks fine to ship the relnote in https://review.openstack.org/#/c/197912/35 | 10:00 |
| bauzas | johnthetubaguy: but I need to dig into that to understand the semantics | 10:00 |
| johnthetubaguy | bauzas: hmm, nope, I guess my filter is not working | 10:00 |
| *** jwcroppe has quit IRC | 10:00 | |
| *** salv-orlando has quit IRC | 10:00 | |
| bauzas | johnthetubaguy: so it's crucial bits :) | 10:00 |
| *** haomaiwang has quit IRC | 10:01 | |
| johnthetubaguy | bauzas: a link to git once we merged to code seems fine | 10:01 |
| *** haomaiwang has joined #openstack-nova | 10:01 | |
| bauzas | johnthetubaguy: read dhellmann's post, and you'll see the new tool, called Reno | 10:01 |
| *** jamielennox|away is now known as jamielennox | 10:01 | |
| *** Jeffrey4l has joined #openstack-nova | 10:01 | |
| bauzas | that allows to add relnotes YAML files in the changes themselves | 10:02 |
| *** markvoelker has quit IRC | 10:02 | |
| *** salv-orlando has joined #openstack-nova | 10:02 | |
| bauzas | http://docs.openstack.org/developer/reno/usage.html#creating-new-release-notes | 10:02 |
| johnthetubaguy | bauzas: yeah, looks good, I remember some talk of that | 10:02 |
| *** browne has quit IRC | 10:03 | |
| bauzas | for sure, I can edit the Relnotes wikipage and leave a link to the change saying "if Reno doesn't work, please bear in mind" | 10:03 |
| bauzas | that kind of thing when you're changing a process and people not yet aware of the new stuff :) | 10:03 |
| bauzas | johnthetubaguy: that just means tho that we need https://review.openstack.org/#/c/241323/ for Nova | 10:04 |
| bauzas | I can handle that | 10:04 |
| *** haomaiwang has quit IRC | 10:04 | |
| *** jwcroppe has joined #openstack-nova | 10:05 | |
| johnthetubaguy | bauzas: if you could sort out those, that would be awesome, ping me and I can get those reviewed | 10:05 |
| *** jwcroppe has quit IRC | 10:06 | |
| bauzas | johnthetubaguy: seems like I'm a guinea pig | 10:06 |
| bauzas | anyway, I need to have relnotes for that change, so anyway... | 10:06 |
| *** njirap has joined #openstack-nova | 10:06 | |
| bauzas | I can defer some of my work to add reno support to nova | 10:06 |
| *** heyongli has quit IRC | 10:08 | |
| *** heyongli has joined #openstack-nova | 10:09 | |
| *** fawadk has quit IRC | 10:10 | |
| *** sahid has quit IRC | 10:11 | |
| *** MarcF has quit IRC | 10:11 | |
| rha | Hi. I am currently to run the python-novaclient functional test against a setup that use neutron instead of nova-network. Seems some of the test cases are specific to nova-network (e.g. "test_fixedip_get" and "test_admin_dns_domain"). What would be the best way to tell the testsuite to skip those tests in a neutron deployment? | 10:11 |
| *** sahid has joined #openstack-nova | 10:13 | |
| *** yangyapeng has quit IRC | 10:14 | |
| johnthetubaguy | rha: unsure actually, I guess we should follow what tempest does with marking some tests with feature flags, but don't think we have done that in there yet, but I am not an expert in that area | 10:15 |
| *** rushiagr_away is now known as rushiagr | 10:17 | |
| *** salv-orlando has quit IRC | 10:18 | |
| *** heyongli has quit IRC | 10:18 | |
| *** salv-orlando has joined #openstack-nova | 10:18 | |
| *** heyongli has joined #openstack-nova | 10:19 | |
| *** amotoki has joined #openstack-nova | 10:20 | |
| *** zhangjn has quit IRC | 10:20 | |
| *** EinstCrazy has quit IRC | 10:20 | |
| *** zhangjn has joined #openstack-nova | 10:23 | |
| *** blackjack1 has left #openstack-nova | 10:24 | |
| *** heyongli has quit IRC | 10:29 | |
| *** sahid has quit IRC | 10:29 | |
| *** heyongli has joined #openstack-nova | 10:29 | |
| *** sahid has joined #openstack-nova | 10:31 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'compute_node' https://review.openstack.org/241189 | 10:32 |
| *** hparekh2 has quit IRC | 10:33 | |
| johnthetubaguy | bauzas: thanks for the release heads up, wow what a lot of things to read, all good stuff though | 10:34 |
| bauzas | johnthetubaguy: sounds pretty easy to implement, I'm just deploying the new sphinx conf | 10:35 |
| johnthetubaguy | cool | 10:35 |
| *** zhangjn_ has joined #openstack-nova | 10:35 | |
| openstackgerrit | Dongcan Ye proposed openstack/nova: VMware: use oslo_config new type PortOpt and IPOpt https://review.openstack.org/241976 | 10:35 |
| *** zhangjn has quit IRC | 10:36 | |
| *** pixelbeat_ has joined #openstack-nova | 10:37 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Replaced deprecated timeutils methods. https://review.openstack.org/241179 | 10:37 |
| rha | johnthetubaguy: That bears the question how/where to tell the tests that nova-network in not available. Via clouds.yaml? | 10:38 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Use EngineFacade from oslo_db.enginefacade https://review.openstack.org/221310 | 10:38 |
| *** heyongli has quit IRC | 10:39 | |
| *** heyongli has joined #openstack-nova | 10:40 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'instance_info' and 'instance_extra' https://review.openstack.org/225065 | 10:41 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'cell_mapping' https://review.openstack.org/226788 | 10:42 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'instance_mapping' https://review.openstack.org/228979 | 10:42 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'key_pair' and 'cell' https://review.openstack.org/239915 | 10:42 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'network' https://review.openstack.org/240253 | 10:42 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'certificate' and 'pci_device' https://review.openstack.org/240275 | 10:42 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: WIP: enginefacade: 'floating_ip' https://review.openstack.org/240513 | 10:42 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: WIP: enginefacade: 'fixed_ip' https://review.openstack.org/240811 | 10:42 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'service' https://review.openstack.org/241178 | 10:42 |
| *** jwcroppe has joined #openstack-nova | 10:43 | |
| *** pratikmallya has joined #openstack-nova | 10:43 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'compute_node' https://review.openstack.org/241189 | 10:43 |
| johnthetubaguy | rha: sorry, not sure how we do the settings there, I was expecting you pass a flag into tox when running the tests | 10:44 |
| rha | johnthetubaguy: Actually the tests read ~/.config/clouds.yaml but that currently almost only contains info for authentication. Not sure if it would be the right place to add such a flag. | 10:47 |
| *** EinstCrazy has joined #openstack-nova | 10:48 | |
| *** pratikmallya has quit IRC | 10:48 | |
| *** heyongli has quit IRC | 10:49 | |
| *** heyongli has joined #openstack-nova | 10:50 | |
| johnthetubaguy | rha: unsure, feels like that should go in tox, but sdague will know more, he is usally online in an hour or two | 10:52 |
| *** sahid has quit IRC | 10:52 | |
| *** longsube has quit IRC | 10:54 | |
| *** takedakn has joined #openstack-nova | 10:57 | |
| *** sahid has joined #openstack-nova | 10:58 | |
| *** jerrygb has joined #openstack-nova | 10:58 | |
| *** heyongli has quit IRC | 10:59 | |
| openstackgerrit | huan proposed openstack/nova: Resolve Nova/Neutron race condition https://review.openstack.org/241127 | 11:00 |
| *** heyongli has joined #openstack-nova | 11:00 | |
| *** yassine__ has quit IRC | 11:03 | |
| *** aysyd has joined #openstack-nova | 11:06 | |
| *** jwcroppe has quit IRC | 11:09 | |
| *** mmdurrant has quit IRC | 11:09 | |
| *** heyongli has quit IRC | 11:10 | |
| *** heyongli has joined #openstack-nova | 11:10 | |
| *** takedakn has quit IRC | 11:12 | |
| *** markvoelker has joined #openstack-nova | 11:13 | |
| *** smatzek has joined #openstack-nova | 11:14 | |
| *** otter768 has joined #openstack-nova | 11:14 | |
| *** hparekh has joined #openstack-nova | 11:15 | |
| *** alex_klimov has quit IRC | 11:16 | |
| *** markvoelker has quit IRC | 11:18 | |
| *** romainh has quit IRC | 11:19 | |
| *** venkat_p has joined #openstack-nova | 11:19 | |
| *** otter768 has quit IRC | 11:19 | |
| *** heyongli has quit IRC | 11:20 | |
| *** heyongli has joined #openstack-nova | 11:21 | |
| *** venkat_p has left #openstack-nova | 11:21 | |
| *** Daisy has joined #openstack-nova | 11:27 | |
| *** heyongli has quit IRC | 11:30 | |
| *** Daisy has quit IRC | 11:31 | |
| *** heyongli has joined #openstack-nova | 11:31 | |
| *** Daisy has joined #openstack-nova | 11:31 | |
| *** y_sawai has quit IRC | 11:32 | |
| *** jwcroppe has joined #openstack-nova | 11:35 | |
| *** Daisy has quit IRC | 11:35 | |
| *** josecastroleon has quit IRC | 11:36 | |
| sdague | rha: clouds.yaml is only for cloud credentials. The reality is that if we have APIs that look different based on backend, which we can't tell via the API, we've got bigger issues. | 11:39 |
| sdague | so we should figure out some way to discover that fact over the API, even if we need to invent some API for that | 11:40 |
| *** Daisy has joined #openstack-nova | 11:40 | |
| *** heyongli has quit IRC | 11:40 | |
| *** heyongli has joined #openstack-nova | 11:41 | |
| *** Daisy has quit IRC | 11:45 | |
| *** alex_klimov has joined #openstack-nova | 11:45 | |
| *** Daisy has joined #openstack-nova | 11:45 | |
| *** venkat_p has joined #openstack-nova | 11:47 | |
| johnthetubaguy | sdague: you are right, some of those network APIs are very broken, that needs a close look at some point (thinking DNS in particular) | 11:48 |
| *** augiemena3 has quit IRC | 11:49 | |
| openstackgerrit | Feng Xi Yan proposed openstack/nova: WIP:(VMWARE)Add power state cache for host init and power state sync. https://review.openstack.org/242006 | 11:49 |
| *** ijw has joined #openstack-nova | 11:49 | |
| *** ijw_ has joined #openstack-nova | 11:50 | |
| *** heyongli has quit IRC | 11:51 | |
| *** alex_klimov has quit IRC | 11:51 | |
| *** Nathan has joined #openstack-nova | 11:51 | |
| *** alex_klimov has joined #openstack-nova | 11:51 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add reno for release notes management https://review.openstack.org/242007 | 11:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: add first reno-based release note https://review.openstack.org/242008 | 11:51 |
| *** heyongli has joined #openstack-nova | 11:51 | |
| bauzas | johnthetubaguy: ^ here are my attempts for reno | 11:52 |
| bauzas | johnthetubaguy: working on the CI patch now but I guess dhellmann will be needed for reviewing | 11:52 |
| *** alex_klimov has quit IRC | 11:52 | |
| bauzas | I'd be pretty far more comfortable by having him +1'ing my changes | 11:52 |
| *** alex_klimov has joined #openstack-nova | 11:53 | |
| *** ijw has quit IRC | 11:53 | |
| *** ssurana has joined #openstack-nova | 11:55 | |
| *** jwcroppe has quit IRC | 11:55 | |
| johnthetubaguy | bauzas: sounds good | 11:55 |
| *** ijw_ has quit IRC | 11:55 | |
| bauzas | johnthetubaguy: since reno uses the git history to get all the commits, I had one bug when trying to build the html output when doing a single change | 11:56 |
| *** Marga_ has quit IRC | 11:57 | |
| bauzas | so there are probably some quirks that need to be whacked | 11:57 |
| bauzas | in the meantime, I'll write my old-style relnotes in the wiki | 11:58 |
| *** rfolco has joined #openstack-nova | 11:58 | |
| bauzas | PaulMurray: around ? do you know how I could get timetables for Paddington/Bristol train ? | 11:59 |
| danpb | bauzas: http://traintimes.org.uk/ | 12:00 |
| bauzas | PaulMurray: and if also there are other options than either BRS or LHR ? | 12:00 |
| bauzas | danpb: ack thanks | 12:00 |
| danpb | bauzas: for general train guidance see http://seat61.com/UK-train-travel.htm | 12:00 |
| *** sai has joined #openstack-nova | 12:00 | |
| *** heyongli has quit IRC | 12:01 | |
| sai | Hello !! | 12:01 |
| danpb | that points you to the right sites for booking, to avoid being charged extra booking fees (if you wish to book in advance) | 12:01 |
| sai | i tried installing liberty devstack today.. | 12:01 |
| *** yassine__ has joined #openstack-nova | 12:01 | |
| alex_xu | johnthetubaguy: sdague oomichi_, hi, just work out the api doc tracking etherpad https://etherpad.openstack.org/p/nova-v2.1-api-doc | 12:01 |
| sai | initially it was working fine, but when i performed unstack && rejoin-stack, | 12:02 |
| bauzas | danpb: okay, lots of information to eat, but that's fair | 12:02 |
| openstackgerrit | Feng Xi Yan proposed openstack/nova: WIP:(VMWARE)Add power state cache for host init and power state sync. https://review.openstack.org/242006 | 12:02 |
| *** heyongli has joined #openstack-nova | 12:02 | |
| PaulMurray | bauzas, hi - yes, hold on | 12:02 |
| sai | i am ending up with an error in n-api not importing a file saying "no such option in group libvirt: rb d_user" | 12:02 |
| alex_xu | johnthetubaguy: sdague oomichi_ gmann, for the api reference doc, basically focus on improve the parts which will be migrated to nova code | 12:02 |
| alex_xu | let me know if this is we expected work on the api ref and right workflow | 12:03 |
| PaulMurray | bauzas, http://www.nationalrail.co.uk/ | 12:03 |
| *** achanda has joined #openstack-nova | 12:03 | |
| johnthetubaguy | alex_xu: thats probably all of it I guess? | 12:03 |
| PaulMurray | bauzas, Note: "bristol parkway" is nearest our site and the hotel | 12:04 |
| bauzas | ok thanks, will look that with also http://seat61.com/UK-train-travel.htm from danpb | 12:04 |
| sai | i wanted to know exactly when that error arises.. can anyone help me out here | 12:04 |
| johnthetubaguy | alex_xu: do you mean this: http://developer.openstack.org/api-ref-compute-v2.1.html I guess? | 12:04 |
| alex_xu | johnthetubaguy: yea, we already fix the structure of api ref in previous release. the only thing we can do for api ref is put more detail and catch more bug in detail | 12:04 |
| alex_xu | johnthetubaguy: yea | 12:04 |
| PaulMurray | bauzas, you could rent a car of course - be daring | 12:05 |
| *** venkat_p has quit IRC | 12:05 | |
| johnthetubaguy | alex_xu: yeah, bugs and the completeness are the key bits | 12:05 |
| *** achanda has quit IRC | 12:05 | |
| bauzas | so, the train ticket is like 50EUR + Heathrow Express (20EUR if booked 1 month in advance) | 12:05 |
| johnthetubaguy | alex_xu: honestly, I think the API reference and the swagger bits are probably higher priority than getting the existing reference perfect, at least thats my gut feel | 12:06 |
| bauzas | that makes 70EUR for getting to Bristol from LHR | 12:06 |
| *** Marga_ has joined #openstack-nova | 12:06 | |
| bauzas | plus any local transportation from hotel to venue | 12:06 |
| bauzas | well, I need to see | 12:06 |
| johnthetubaguy | bauzas: it might be dependent on the time of day, but thats about what I expected, its worth checking out the bus from LHR to bristol | 12:06 |
| danpb | bauzas: yep, sounds about right for train cost if bought in advance | 12:07 |
| alex_xu | johnthetubaguy: does the 'API reference' you mentioned == existing reference? | 12:07 |
| bauzas | johnthetubaguy: oh, there is a direct bus from LHR to Bristol ? | 12:07 |
| danpb | if yuou buy it on the day it'll be x2-x3 more expensive | 12:07 |
| links | hi everyone, im facing issue with resize . Does nova perform scp/rsync of disk before '_clean_shutdown' ? also i cant find evidence that nova 'tried' to scp to 2nd compute from secure logs . | 12:07 |
| bauzas | or does that still need to go to Paddington ? | 12:07 |
| johnthetubaguy | bauzas: yeah, I think PaulMurray mentioned a good coach link | 12:07 |
| johnthetubaguy | bauzas: coach was direct I think | 12:07 |
| johnthetubaguy | bauzas: I guess its "national express" coach, but I am not sure | 12:08 |
| *** zhangjn_ has quit IRC | 12:08 | |
| *** sai has left #openstack-nova | 12:08 | |
| danpb | bauzas: search bus info here http://www.nationalexpress.com/airports/heathrow-airport.aspx | 12:08 |
| bauzas | johnthetubaguy: alex_xu: I know rdopiera was interested in helping with the api bits | 12:08 |
| *** zhangjn has joined #openstack-nova | 12:08 | |
| danpb | they'll go direct to the airport, so no need to go via london in that case | 12:08 |
| alex_xu | bauzas: yea, rdopiera already talk with me | 12:08 |
| *** klkumar has quit IRC | 12:09 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'dnsdomain' and 'ec2' https://review.openstack.org/242019 | 12:09 |
| alex_xu | really thanks rdopiera want to help on doc | 12:09 |
| bauzas | well, the subteam asked during the summit for help, so I pointed that out to rdopiera | 12:10 |
| *** jerrygb has quit IRC | 12:10 | |
| alex_xu | bauzas: thanks :) | 12:11 |
| *** heyongli has quit IRC | 12:11 | |
| johnthetubaguy | raildo-afk: hey, just a bit confused with the API piece in the nested quota spec, I added a comment | 12:11 |
| *** heyongli has joined #openstack-nova | 12:12 | |
| *** deepakcs has quit IRC | 12:14 | |
| *** raildo-afk is now known as raildo | 12:15 | |
| raildo | johnthetubaguy: looking | 12:16 |
| johnthetubaguy | raildo: I am about to run off for lunch I am afraid, but shouldn't be too long | 12:17 |
| *** zhangjn has quit IRC | 12:17 | |
| raildo | no problem, we can talk when you back :) | 12:18 |
| *** regebro has joined #openstack-nova | 12:21 | |
| *** heyongli has quit IRC | 12:21 | |
| regebro | I have a big question about the error message here: https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L203 | 12:22 |
| regebro | The warning (not error, sorry) seems to happen when exactly the opposite is true of what the warning says. | 12:22 |
| *** heyongli has joined #openstack-nova | 12:22 | |
| *** dims_ has joined #openstack-nova | 12:22 | |
| regebro | Can anybody clarify what is correct behavior here? We had half an hour of confusion here because of this. :-) | 12:23 |
| *** dims has quit IRC | 12:25 | |
| chinmay_ | regebro : seems correct no ? look at the comment written above that line | 12:25 |
| chinmay_ | " # can occur when an instance in database is not on host" | 12:26 |
| *** romainh has joined #openstack-nova | 12:26 | |
| regebro | chinmay_: But it's showing it when least gb is BIGGER than free_db. | 12:26 |
| bauzas | regebro: no it's correct | 12:27 |
| regebro | But the message is saying that the host has MORE disk than expected. | 12:27 |
| *** 17SADZP6R has joined #openstack-nova | 12:27 | |
| bauzas | least_gb is the physical size, reported by the virt driver | 12:27 |
| chinmay_ | free_gb is calculated by the flavors of the instances | 12:28 |
| bauzas | while free_gb is the size left (decreased by the size of all the instances) which is based on the DB | 12:28 |
| bauzas | yeah that | 12:28 |
| chinmay_ | right | 12:28 |
| regebro | bauzas: OK, so it's not the least_db at all, OK. | 12:28 |
| bauzas | free_gb = total_gb (reported by the virt driver) - sum(instances) | 12:28 |
| *** sileht has quit IRC | 12:28 | |
| *** Daisy has quit IRC | 12:29 | |
| regebro | bauzas: But then this warning will happen every time there is any instance. | 12:29 |
| bauzas | it depends on the driver | 12:29 |
| *** Daisy has joined #openstack-nova | 12:29 | |
| regebro | Ah well, we have one instance per host, though. | 12:30 |
| chinmay_ | disk_available_least is not applicable for vmware, i guess. whats the driver are you using | 12:30 |
| bauzas | eg for libvirt https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5087 | 12:30 |
| regebro | ironic | 12:30 |
| rajesht | dansmith: could you please review https://review.openstack.org/#/c/215483/ | 12:30 |
| regebro | The first warnings we had was even more fun, becayse free_gb was negative. :-) | 12:31 |
| bauzas | regebro: see for ironic, that's indeed about the VM size https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L366 | 12:31 |
| regebro | bauzas: But normally, free_db would be zero, right? If you have flavours that are auto-matched and hence as big as the disks? | 12:31 |
| *** heyongli has quit IRC | 12:32 | |
| bauzas | regebro: that's... complicated :) | 12:32 |
| bauzas | conceptually, yes because with ironic the winner takes all | 12:32 |
| rajesht | oomichi_: could you please review https://review.openstack.org/#/c/215483/ | 12:32 |
| *** heyongli has joined #openstack-nova | 12:32 | |
| bauzas | regebro: but speaking of Ironic, there is an IronicHostManager which doesn't check the above | 12:33 |
| openstackgerrit | Dongcan Ye proposed openstack/nova: VMware: use oslo_config new type PortOpt and IPOpt https://review.openstack.org/241976 | 12:33 |
| bauzas | so if you're using Ironic, you shouldn't use the regular HostManager you mentioned and thus not getting that warning | 12:33 |
| regebro | God why is everything also so complicated. :-) | 12:33 |
| *** josecastroleon has joined #openstack-nova | 12:33 | |
| bauzas | https://github.com/openstack/nova/blob/master/nova/scheduler/ironic_host_manager.py#L63 | 12:33 |
| *** Daisy has quit IRC | 12:33 | |
| *** IanGovett has joined #openstack-nova | 12:34 | |
| bauzas | ^ is the way Ironic overrides the calculations | 12:34 |
| regebro | bauzas: Aha, but we have a mix, so we can't use IronicHostManager, I think. Ho hum | 12:34 |
| bauzas | well then I guess it's a non-supported model but I'd defer to jroll for that | 12:35 |
| regebro | OK, I'm going for lunch at the moment. Thanks for the help so far! | 12:35 |
| *** regebro is now known as regebro|lunch | 12:35 | |
| bauzas | that's actually a very good question, I dunno if a libvirt+ironic deployment is something we support | 12:36 |
| bauzas | that's doable using aggregates, but the ironic host manager makes things a little bit complicated | 12:36 |
| regebro|lunch | Well, support or not, it works. :-) Except for weird error messages. | 12:36 |
| bauzas | jroll: ^ when you wake-up, I'd be interested in knowing your opinion on that :) | 12:36 |
| *** cdent has quit IRC | 12:37 | |
| bauzas | regebro|lunch: well, if you don't use the ironic host manager, then the calculations are incorrect - that should indeed work, but till you're not up to a certain limit | 12:37 |
| * bauzas also faces severe starvation, heading off | 12:38 | |
| *** zhangjn has joined #openstack-nova | 12:39 | |
| *** sileht has joined #openstack-nova | 12:39 | |
| *** zhangjn has quit IRC | 12:39 | |
| *** zhangjn has joined #openstack-nova | 12:40 | |
| *** dave-mccowan has joined #openstack-nova | 12:42 | |
| *** yassine__ has quit IRC | 12:42 | |
| *** heyongli has quit IRC | 12:42 | |
| *** salv-orlando has quit IRC | 12:42 | |
| *** heyongli has joined #openstack-nova | 12:43 | |
| *** yamahata has joined #openstack-nova | 12:45 | |
| *** thorst has joined #openstack-nova | 12:46 | |
| *** inteq has quit IRC | 12:46 | |
| *** Daisy has joined #openstack-nova | 12:48 | |
| *** daemontool has quit IRC | 12:49 | |
| openstackgerrit | Michal Dubiel proposed openstack/nova: libvirt, vhostuser: Add support for OpenContrail vRouter https://review.openstack.org/241652 | 12:50 |
| *** Jeffrey4l has quit IRC | 12:50 | |
| *** Daisy has quit IRC | 12:51 | |
| *** Daisy has joined #openstack-nova | 12:51 | |
| *** heyongli has quit IRC | 12:52 | |
| *** jistr has quit IRC | 12:52 | |
| *** jwcroppe has joined #openstack-nova | 12:53 | |
| *** heyongli has joined #openstack-nova | 12:53 | |
| *** yassine__ has joined #openstack-nova | 12:54 | |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'host_mapping' https://review.openstack.org/242035 | 12:54 |
| rha | sdague: So for some of the tests (e.g. dns-domains) nova will return 501 (HTTPNotImplemented) when used with neutron, I guess we could treat that as a success result for the tests, no? But others don't fail that grateful. | 12:57 |
| *** Daisy has quit IRC | 12:57 | |
| *** Daisy has joined #openstack-nova | 12:57 | |
| rha | sdague: Don't know if there is a good way to discover what backend is used. (Other than probably looking at the keystone endpoints). | 12:57 |
| *** Nathan has quit IRC | 12:58 | |
| sdague | keystone endpoints are probably reasonable | 12:58 |
| sdague | at least for our testing | 12:58 |
| *** mmdurrant has joined #openstack-nova | 12:59 | |
| *** Sree has joined #openstack-nova | 12:59 | |
| *** subscope has joined #openstack-nova | 12:59 | |
| *** su_zhang has joined #openstack-nova | 12:59 | |
| *** 17SADZP6R has quit IRC | 13:01 | |
| *** chinmay_ has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-nova | 13:01 | |
| *** klkumar has joined #openstack-nova | 13:01 | |
| *** Daisy has quit IRC | 13:02 | |
| *** y_sawai has joined #openstack-nova | 13:02 | |
| *** heyongli has quit IRC | 13:02 | |
| *** tobe has quit IRC | 13:03 | |
| *** Marga_ has quit IRC | 13:03 | |
| *** heyongli has joined #openstack-nova | 13:03 | |
| *** jwcroppe has quit IRC | 13:07 | |
| *** jamielennox is now known as jamielennox|away | 13:08 | |
| *** Jeffrey4l has joined #openstack-nova | 13:08 | |
| *** tobe has joined #openstack-nova | 13:10 | |
| *** jistr has joined #openstack-nova | 13:11 | |
| *** jerrygb has joined #openstack-nova | 13:11 | |
| *** jwcroppe has joined #openstack-nova | 13:11 | |
| *** heyongli has quit IRC | 13:13 | |
| *** heyongli has joined #openstack-nova | 13:13 | |
| *** jwcroppe has quit IRC | 13:14 | |
| *** markvoelker has joined #openstack-nova | 13:14 | |
| *** tobe has quit IRC | 13:14 | |
| *** tobe has joined #openstack-nova | 13:14 | |
| *** otter768 has joined #openstack-nova | 13:15 | |
| *** baoli has joined #openstack-nova | 13:16 | |
| *** jerrygb has quit IRC | 13:16 | |
| johnthetubaguy | raildo: hi, happy to chat through that spec when its a good time | 13:17 |
| raildo | johnthetubaguy: ++ | 13:18 |
| *** markvoelker_ has joined #openstack-nova | 13:18 | |
| *** markvoelker has quit IRC | 13:18 | |
| *** zhangjn has quit IRC | 13:18 | |
| johnthetubaguy | regebro|lunch: bauzas: I had some ideas on having a mix of ironic and non-ironic hosts in a single deploy if thats interesting | 13:19 |
| raildo | johnthetubaguy: if we make every clear now, it'll be easier approve later :D | 13:19 |
| johnthetubaguy | raildo: true | 13:19 |
| *** tobe has quit IRC | 13:19 | |
| johnthetubaguy | raildo: I was thinking the existing API would be enough, just using the ?tenant_id=foo | 13:19 |
| johnthetubaguy | raildo: was there a reason to move away from that, I am wondering what was done for cinder | 13:19 |
| *** otter768 has quit IRC | 13:20 | |
| *** tobe has joined #openstack-nova | 13:20 | |
| johnthetubaguy | raildo: I was really wondering if this bit turned out to be true, roughly: http://specs.openstack.org/openstack/cinder-specs/specs/liberty/cinder-nested-quota-driver.html#rest-api-impact | 13:20 |
| *** burgerk has joined #openstack-nova | 13:21 | |
| raildo | johnthetubaguy: in cinder doen't have API impact since they alredy have the target_id in the quota API calls | 13:21 |
| johnthetubaguy | raildo: so does nova | 13:21 |
| johnthetubaguy | raildo: at least thats what the docs claim | 13:22 |
| raildo | johnthetubaguy: as you can see here | 13:22 |
| raildo | http://developer.openstack.org/api-ref-blockstorage-v2.html#updateQuota | 13:22 |
| raildo | in the nova, we just have the first part of the query, and that first "tenat_id" is the project that the project that the token was scoped | 13:23 |
| *** heyongli has quit IRC | 13:23 | |
| raildo | johnthetubaguy: and we need to verify if the target_project is a subproject fo the project that the token was scoped | 13:23 |
| johnthetubaguy | raildo: similar here, except its a query parameter: http://developer.openstack.org/api-ref-compute-v2.1.html#forceupdatequota | 13:23 |
| *** heyongli has joined #openstack-nova | 13:24 | |
| johnthetubaguy | raildo: well, I mean this one I guess: http://developer.openstack.org/api-ref-compute-v2.1.html#updatesquotatenant | 13:24 |
| *** tobe has quit IRC | 13:25 | |
| raildo | johnthetubaguy: you're saying that we already can do PUT /v2.1/{tenant_id}/os-quota-sets?tenant_id={tenant_id}? | 13:25 |
| johnthetubaguy | raildo: yes | 13:25 |
| raildo | johnthetubaguy: awesome! so I'll remove this API impact | 13:26 |
| johnthetubaguy | raildo: cool, I think at that point I am +2 | 13:26 |
| raildo | johnthetubaguy: I think that will send a patch do make the docs more clear about this part haha | 13:27 |
| raildo | johnthetubaguy: thanks! | 13:27 |
| *** signed8bit has joined #openstack-nova | 13:28 | |
| johnthetubaguy | raildo: np | 13:28 |
| *** njirap has quit IRC | 13:29 | |
| sdague | alex_xu: on your comments on - https://review.openstack.org/#/c/233076/ that's just for the view builder, and I don't think it is problematic in this case | 13:29 |
| johnthetubaguy | danpb: there is a spec that seems related to OF VIF lib again, its about the linux-net code: https://review.openstack.org/#/c/234116/1/specs/mitaka/approved/linux_net_refactor.rst,cm | 13:30 |
| openstackgerrit | Raildo Mascena de Sousa Filho proposed openstack/nova-specs: Moving Nested Quota Driver to Mitaka https://review.openstack.org/209969 | 13:31 |
| *** heyongli has quit IRC | 13:33 | |
| *** rk4n has quit IRC | 13:33 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Moves Hyper-V vTPM instances spec to Mitaka https://review.openstack.org/231418 | 13:34 |
| *** heyongli has joined #openstack-nova | 13:34 | |
| *** edmondsw has joined #openstack-nova | 13:34 | |
| *** rk4n has joined #openstack-nova | 13:34 | |
| *** abhishekk has quit IRC | 13:37 | |
| *** y_sawai has quit IRC | 13:40 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Moving Nested Quota Driver to Mitaka https://review.openstack.org/209969 | 13:40 |
| johnthetubaguy | raildo: I edited the commit message slightly, but +2 on that now, thank you for pushing on that | 13:42 |
| raildo | johnthetubaguy: thank you for your patience with us :D | 13:43 |
| *** heyongli has quit IRC | 13:43 | |
| *** salv-orlando has joined #openstack-nova | 13:44 | |
| *** heyongli has joined #openstack-nova | 13:45 | |
| *** pkoniszewski has quit IRC | 13:47 | |
| *** edleafe has joined #openstack-nova | 13:50 | |
| openstackgerrit | Moshe Levi proposed openstack/nova: Fix pci_stats logging in resource tracker https://review.openstack.org/227116 | 13:50 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: Libvirt hardware policy from libosinfo https://review.openstack.org/218962 | 13:51 |
| *** rushiagr is now known as rushiagr_away | 13:52 | |
| *** rodrigods has joined #openstack-nova | 13:52 | |
| *** rlrossit has joined #openstack-nova | 13:52 | |
| johnthetubaguy | markus_z: your config option work, do we have a blueprint approved for that yet? | 13:52 |
| markus_z | johnthetubaguy: Not yet, mriedem has some doubts I couldn't solve yet. | 13:53 |
| johnthetubaguy | markus_z: for the record, totally want to help review that, and get that in ASAP, its important stuff | 13:53 |
| johnthetubaguy | markus_z: ah, do you have a link? | 13:53 |
| markus_z | johnthetubaguy: https://review.openstack.org/#/c/227948/ | 13:53 |
| *** heyongli has quit IRC | 13:54 | |
| johnthetubaguy | markus_z: sweet, thanks, I will take a look | 13:54 |
| markus_z | johnthetubaguy: thanks! | 13:54 |
| johnthetubaguy | markus_z: lets ping mriedem when he comes online | 13:54 |
| *** heyongli has joined #openstack-nova | 13:55 | |
| *** mdrabe has joined #openstack-nova | 13:56 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: config options: remove invalid help text assignment https://review.openstack.org/242064 | 13:57 |
| alex_xu | sdague: you are right, I misse understand that code | 13:57 |
| *** xyang1 has joined #openstack-nova | 13:57 | |
| *** salv-orlando has quit IRC | 13:59 | |
| *** salv-orlando has joined #openstack-nova | 13:59 | |
| *** daemontool has joined #openstack-nova | 13:59 | |
| sdague | johnthetubaguy: so it does feel like for - https://review.openstack.org/#/c/218962 we should document that direct gobject use in nova code is a short term gap filler until the stable xml API exists in the project in the next cycle | 14:00 |
| sdague | and that we intend to remove that code in N | 14:00 |
| *** haomaiwang has quit IRC | 14:01 | |
| sdague | because I think we all agreed on that approach in the room, and it would be good to record | 14:01 |
| *** links has quit IRC | 14:01 | |
| *** haomaiwa_ has joined #openstack-nova | 14:01 | |
| *** y_sawai has joined #openstack-nova | 14:02 | |
| *** unicell1 has joined #openstack-nova | 14:02 | |
| *** unicell has quit IRC | 14:03 | |
| *** jerrygb has joined #openstack-nova | 14:04 | |
| *** heyongli has quit IRC | 14:04 | |
| danpb | sdague: yep, we can write a followup patch to add that note | 14:04 |
| *** signed8bit is now known as signed8bit_ZZZzz | 14:04 | |
| *** ctrath has joined #openstack-nova | 14:04 | |
| bauzas | johnthetubaguy: interested in getting your thoughts about the ironic vs. other drivers mix | 14:04 |
| *** mnestratov has quit IRC | 14:05 | |
| bauzas | that said, the BP about ironic providing multi-nodes support should make the ironic host manager deprecated | 14:05 |
| *** heyongli has joined #openstack-nova | 14:05 | |
| johnthetubaguy | bauzas: so you know my idea for cells, and picking between schedulers based on the flavor? | 14:05 |
| bauzas | yup | 14:05 |
| johnthetubaguy | bauzas: I think it works for ironic and non-iroinc in the same cell | 14:05 |
| bauzas | I see | 14:06 |
| *** lpetrut has joined #openstack-nova | 14:07 | |
| *** david-lyle has joined #openstack-nova | 14:08 | |
| sdague | danpb: cool, follow up is fine with me | 14:08 |
| *** garyk1 has quit IRC | 14:09 | |
| *** sneti has joined #openstack-nova | 14:10 | |
| *** diogogmt has quit IRC | 14:13 | |
| *** ssurana1 has joined #openstack-nova | 14:14 | |
| bauzas | could someone just push the button for getting rid of the main obfuscating piece of ERT like it was discussed at the summit? https://review.openstack.org/#/c/240159/2 | 14:14 |
| *** heyongli has quit IRC | 14:14 | |
| johnthetubaguy | bauzas: basically split the hosts between two schedulers, and have a third one that just routes between them, or something a bit like that | 14:14 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: config options: add hacking check for help text length https://review.openstack.org/242078 | 14:14 |
| bauzas | johnthetubaguy: yup, I understood your point, proxying the schedulers | 14:15 |
| *** ssurana has quit IRC | 14:15 | |
| bauzas | given the agreement we had at the summit, we can wait until end of Mitaka to see if it's needed | 14:15 |
| *** heyongli has joined #openstack-nova | 14:15 | |
| bauzas | (well, the consensus was a bit discussed because of some out-of-tree stuff...) | 14:15 |
| *** daemontool has quit IRC | 14:15 | |
| *** ssurana1 has quit IRC | 14:15 | |
| johnthetubaguy | bauzas: so my vidyo of that how to get your idea upstream is on youtube, raildo found it, so I tweeted it out | 14:16 |
| raildo | :D | 14:16 |
| bauzas | he he :) | 14:16 |
| *** daemontool has joined #openstack-nova | 14:16 | |
| bauzas | still actively chasing mockingbirds | 14:16 |
| bauzas | :p | 14:16 |
| johnthetubaguy | hehe | 14:16 |
| *** diogogmt has joined #openstack-nova | 14:16 | |
| *** dave-mccowan has quit IRC | 14:17 | |
| johnthetubaguy | bauzas: I am tempted to add docimpact to that git commit, what do you think? | 14:18 |
| bauzas | which one ? | 14:18 |
| *** mrkz has joined #openstack-nova | 14:18 | |
| bauzas | oh PaulMurray's one | 14:18 |
| johnthetubaguy | the remove ERT thingy | 14:18 |
| johnthetubaguy | yeah | 14:18 |
| bauzas | that's fair enough | 14:18 |
| bauzas | we could communicate about that | 14:18 |
| bauzas | that actually reminds me that we should be very clear about the ERT deprecation | 14:19 |
| bauzas | because people could have written some new ERT plugins | 14:19 |
| *** jichen has quit IRC | 14:19 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Remove vcpu resource from extensible resource tracker https://review.openstack.org/240159 | 14:19 |
| *** mriedem has joined #openstack-nova | 14:21 | |
| jroll | bauzas: regebro|lunch: so, we've always said host aggregates covers ironic+libvirt in the same deployment, and I know some people do it, but I'm not sure how their scheduler is set up. need to dig into it again. | 14:22 |
| jroll | there was a talk on this, I also need to watch that | 14:22 |
| johnthetubaguy | bauzas : yeah, I was thinking also deprecate that config value, etc, but we could do that in a follow up | 14:22 |
| johnthetubaguy | jroll: yeah, I heard that two, not sure how they deal with the host managers being different | 14:22 |
| bauzas | johnthetubaguy: I was rather thinking of logging that ERT is deprecated | 14:23 |
| johnthetubaguy | bauzas: yeah, totally. | 14:23 |
| jroll | johnthetubaguy: yeah, though regebro|lunch implies the regular host manager works, is just warn-spam | 14:23 |
| johnthetubaguy | jroll: ah, I missed that | 14:23 |
| johnthetubaguy | jroll: that would do it | 14:24 |
| bauzas | jroll: like I said, it's doable - it's just a matter of accuracy | 14:24 |
| jroll | right | 14:24 |
| jroll | I'm inclined to focus on the new world instead :) | 14:24 |
| jroll | also cells totally mostly kinda works :) | 14:24 |
| mriedem | huzzah | 14:24 |
| *** heyongli has quit IRC | 14:24 | |
| jroll | (keypairs are still busted last I checked though) | 14:25 |
| bauzas | jroll: (btw. sorry I apologize about my participation in the ironic/nova joint session, I'm just a douchebag when talking in english so I'm sometimes getting confused) | 14:25 |
| mriedem | jroll: keypair support was added to cells v1 in liberty | 14:25 |
| jroll | \o/ | 14:25 |
| bauzas | orly? | 14:25 |
| pkholkin | mriedem: Good day Matt! Thanks for you review in enginefacade patch, could you please look if we understand and change it correctly? https://review.openstack.org/#/c/221310/ | 14:25 |
| *** heyongli has joined #openstack-nova | 14:26 | |
| jroll | bauzas: I don't remember anything you should be apologizing before, so either you're fine or my memory sucks :) | 14:26 |
| mriedem | pkholkin: sure, after my next meeting | 14:26 |
| bauzas | jroll: you don't remember me talking like for a minute and stopping at the end of that very verbose phrase? | 14:26 |
| *** markus_z has quit IRC | 14:26 | |
| pkholkin | mriedem: ok | 14:26 |
| *** markus_z has joined #openstack-nova | 14:26 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 14:26 | |
| *** lbelivea has joined #openstack-nova | 14:26 | |
| mriedem | bauzas: as long as you're talking about tasty cheese for like 10 minutes straight, it doesn't bother me | 14:27 |
| bauzas | which made the whole audience just confused about what I was trying to explain and me saying "nvm" ? | 14:27 |
| pkholkin | mriedem: and what about keypairs pagination? I've created such spec one week ago | 14:27 |
| bauzas | mriedem: gosh, I wish communication could be over IRC for the whole time | 14:27 |
| jroll | bauzas: ah yeah, it's coming back. don't worry about it, it happens. if you remember your concern and want to chat about it here sometime I'm happy to do so | 14:28 |
| jroll | ¯\_(ツ)_/¯ | 14:28 |
| bauzas | mriedem: http://www.thelocal.fr/20151104/cheese-worth-40000-stolen-in-dairy-heist | 14:28 |
| markus_z | bauzas: :D | 14:28 |
| mriedem | heh | 14:28 |
| mriedem | pkholkin: is keypair pagination something separate from engine facade? | 14:28 |
| johnthetubaguy | mriedem: jroll: yeah, there x509 stuff broke keypairs and cells, there is a bug open for that somwhere | 14:28 |
| pkholkin | mriedem: yes yes, just a question, this one https://review.openstack.org/#/c/239663/ | 14:29 |
| mriedem | ok, hadn't seen it | 14:29 |
| mriedem | one thing at a time, and it's meeting time for me | 14:29 |
| *** mriedem is now known as mriedem_meeting | 14:29 | |
| bauzas | jroll: nah that's fine, we had a consensus at the end of the session - which is good | 14:30 |
| jroll | johnthetubaguy: I think mriedem_meeting is implying it works now | 14:30 |
| *** regebro|lunch is now known as regebro | 14:30 | |
| pkholkin | mriedem_meeting: ok, no problem, leave your comments any time | 14:30 |
| jroll | bauzas: cool | 14:30 |
| regebro | Oh, there's a "new world" coming, Scary. :-) | 14:30 |
| edleafe | None2soon | 14:31 |
| mriedem_meeting | jroll: johnthetubaguy: https://review.openstack.org/#/c/197514/ | 14:31 |
| *** diogogmt has quit IRC | 14:31 | |
| jroll | mriedem_meeting: woooo. thanks | 14:31 |
| *** mtanino has joined #openstack-nova | 14:31 | |
| johnthetubaguy | mriedem_meeting: sweet, I totally missed that | 14:31 |
| johnthetubaguy | mostly loving the diversity of folks now working and using cells | 14:32 |
| jroll | now I just need to figure out a decent way to deal with https://review.openstack.org/#/c/198896/ | 14:32 |
| jroll | I think that makes cells work in devstack for ironic | 14:32 |
| diana_clarke | regebro: Fun seeing you here :) I just pulled out your book we branded for PyCon 2014 a couple of hours ago to look something up. | 14:33 |
| regebro | diana_clarke: Oh, hi! Cool. :-) | 14:33 |
| mriedem_meeting | jroll: nova-manage cells sub-command to sync ironic flavors? | 14:33 |
| *** ijuwang has joined #openstack-nova | 14:33 | |
| openstackgerrit | Dongcan Ye proposed openstack/nova: VMware: use oslo_config new type PortOpt and IPOpt https://review.openstack.org/241976 | 14:33 |
| regebro | Didn't know edleafe was doing Nova either, hi! | 14:34 |
| *** Jeffrey4l has quit IRC | 14:34 | |
| diana_clarke | regebro: And I was regretting not doing something similar for PyCon 2015. Ah well, there's only so much time in a day. Anyhoo, hi! | 14:34 |
| *** heyongli has quit IRC | 14:35 | |
| jroll | mriedem_meeting: yeah, that's probably the right thing to do | 14:35 |
| * jroll is lazy | 14:35 | |
| *** heyongli has joined #openstack-nova | 14:36 | |
| mriedem_meeting | jroll: somewhere in here https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1112 | 14:36 |
| regebro | diana_clarke: Yeah, I was also busy moving it to github then. | 14:36 |
| dansmith | mriedem_meeting: so what are you thinking about your cells fix? | 14:36 |
| mriedem_meeting | dansmith: i haven't checked results yet | 14:36 |
| dansmith | mriedem_meeting: bauzas was hitting it over and over again on the same patch yesterday and I haven't seen yours hit it | 14:36 |
| dansmith | I just rechecked again | 14:36 |
| *** dave-mccowan has joined #openstack-nova | 14:36 | |
| *** vladikr has joined #openstack-nova | 14:36 | |
| jroll | mriedem_meeting: indeed, thanks | 14:37 |
| mriedem_meeting | wow, rechecked the UC change 8 times and never hit the cells failure | 14:37 |
| bauzas | yup | 14:37 |
| mriedem_meeting | dansmith: bauzas: so you guys are talking about making this a thing? https://review.openstack.org/#/c/241742/ | 14:37 |
| bauzas | like me unlucky on the other hand, I had so many retirs | 14:37 |
| mriedem_meeting | yeah my change below the UC change has hit it several times | 14:37 |
| *** cfriesen_ has joined #openstack-nova | 14:37 | |
| *** mriedem_meeting is now known as mriedem | 14:38 | |
| mriedem | who am i kidding, i'm not paying attention to this meeting | 14:38 |
| bauzas | well, I'm a bit stuck | 14:38 |
| dansmith | mriedem: well, that's what I'm asking about | 14:38 |
| bauzas | we know the problem, but now I wonder what is the proper way to fix it | 14:38 |
| bauzas | and I'd like to avoid an UC | 14:38 |
| mriedem | yeah i'd like to avoid the UC | 14:39 |
| bauzas | because of $operators | 14:39 |
| mriedem | also, i didn't expect the UC to fix anything, i expected it to trigger a db failure that we could track | 14:39 |
| bauzas | so, conceptually, we have 2 saves, one being at the compute manager level, and one being at the virt driver level, right? | 14:39 |
| *** dustins has joined #openstack-nova | 14:39 | |
| mriedem | at least, yeah | 14:39 |
| bauzas | and the problem is that both are using that awesome update_or_create | 14:40 |
| bauzas | which is racy | 14:40 |
| mriedem | b/c they go through save, yes | 14:40 |
| *** dims has joined #openstack-nova | 14:40 | |
| bauzas | because it means that you could find a situation where it thinks it's a create while it should be an update | 14:40 |
| mriedem | both have device_name set so it passes create=None to bdm_update_or_create_at_top | 14:40 |
| bauzas | but my concern is about the need to use that method or just call create for the first time and update for the second one | 14:41 |
| openstackgerrit | dstepanenko proposed openstack/nova: WIP: Implementation of the first part of pci-generate-stats blueprint https://review.openstack.org/242087 | 14:41 |
| bauzas | like we could do in the second stuff : | 14:41 |
| bauzas | try: update() except: create() | 14:41 |
| bauzas | instead of that piece of logic being shipped in the model | 14:41 |
| mriedem | you know, quotas has a thing just like this in the api code, it does an update and if that fails with NotFound it does a create | 14:42 |
| bauzas | what I don't like is using that facade for either getting a create or an update | 14:42 |
| *** maurosr has quit IRC | 14:43 | |
| bauzas | well | 14:43 |
| *** flyingtt_ has quit IRC | 14:43 | |
| *** dims_ has quit IRC | 14:43 | |
| jroll | mriedem: is there an existing flavor sync thing? I'm not seeing it but I swear I have in the past | 14:43 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/quota_sets.py#L139-L144 | 14:43 |
| *** maurosr has joined #openstack-nova | 14:43 | |
| mriedem | jroll: probably a question for alaski, but it sounds familiar to me also | 14:43 |
| jroll | ya, grep isn't helping :/ | 14:43 |
| openstackgerrit | Dongcan Ye proposed openstack/nova: VMware: use oslo_config new type PortOpt https://review.openstack.org/241976 | 14:43 |
| bauzas | jroll: that's here https://review.openstack.org/#/c/197514/16/nova/api/metadata/base.py,cm | 14:44 |
| *** pradk has joined #openstack-nova | 14:44 | |
| alaski | jroll: what sort of flavor sync? | 14:44 |
| mriedem | bauzas: he's asking about syncing flavors, not keypairs | 14:44 |
| bauzas | oh my bad | 14:44 |
| *** tziarko_ has quit IRC | 14:44 | |
| *** rha has quit IRC | 14:44 | |
| bauzas | well, I remember some tooling about flavor sync | 14:44 |
| bauzas | like the ones from CERN | 14:45 |
| *** electrocucaracha has joined #openstack-nova | 14:45 | |
| jroll | alaski: syncing flavors from top to child | 14:45 |
| cfriesen_ | mriedem: dansmith: continuing the issue from yesterday about deleting a resize-in-progress instance. In the delete code why do we only do the resize confirmation if instance.vm_state == vm_states.RESIZED? It seems like the resize on the source host is basically done once we set instance.task_state = task_states.RESIZE_MIGRATED | 14:45 |
| *** smcginnis has quit IRC | 14:45 | |
| jroll | alaski: a better https://review.openstack.org/#/c/198896/1/lib/ironic | 14:45 |
| bauzas | jroll: https://github.com/cernops/nova-flavor-create/blob/master/cern-flavor-create | 14:45 |
| *** heyongli has quit IRC | 14:45 | |
| bauzas | not sure it answers your question tho | 14:45 |
| *** tziarko has joined #openstack-nova | 14:45 | |
| *** irenab has quit IRC | 14:46 | |
| *** heyongli has joined #openstack-nova | 14:46 | |
| jroll | bauzas: aha, that's where it was, thanks. now the question of can I drag that into devstack or nova tree :) | 14:46 |
| *** gjayavelu has quit IRC | 14:46 | |
| *** smcginnis has joined #openstack-nova | 14:46 | |
| bauzas | jroll: we already do that for devstack | 14:46 |
| *** apuimedo has quit IRC | 14:46 | |
| *** hwoarang has quit IRC | 14:47 | |
| *** jaypipes has joined #openstack-nova | 14:47 | |
| jroll | well then | 14:47 |
| jroll | thanks :) | 14:47 |
| dansmith | cfriesen_: I would assume it's to avoid the source deleting things before the instance is started on the destination successfully.. it's a two-phase commit for safety, AFAIK, and the user may not grok the danger of confirming early | 14:47 |
| bauzas | jroll: not for flavors tho, sorry I'm unclear | 14:48 |
| *** hwoarang has joined #openstack-nova | 14:48 | |
| *** hwoarang has quit IRC | 14:48 | |
| *** hwoarang has joined #openstack-nova | 14:48 | |
| bauzas | https://github.com/openstack-dev/devstack/blob/master/lib/nova#L713-L714 | 14:48 |
| *** Jeffrey4l has joined #openstack-nova | 14:48 | |
| *** rha has joined #openstack-nova | 14:48 | |
| alaski | jroll: nova relies on the terrible implicit behavior in https://github.com/openstack-dev/devstack/blob/master/lib/nova#L673, the fact that db sync creates some initial flavors | 14:48 |
| *** rha has quit IRC | 14:48 | |
| *** rha has joined #openstack-nova | 14:48 | |
| bauzas | mriedem: so, the logic you showed me with quotas is the reverse one I was thinking but okay | 14:48 |
| alaski | jroll: I mean devstack and cells | 14:48 |
| jroll | ah, right. | 14:48 |
| bauzas | right | 14:49 |
| jroll | alaski: I assume dragging cern's sync thing into nova tree won't happen since v1 is on life support? | 14:49 |
| bauzas | jroll: I wouldn't go that way but I leave alaski answer | 14:50 |
| jroll | bauzas: what would you recommend then? | 14:50 |
| jroll | pip install cernops seems wrong :P | 14:50 |
| mriedem | jroll: why can't you add a simple thing to nova-manage cells sub-command to do what you needed in devstack? | 14:51 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Hyper-V: Implement nova rescue https://review.openstack.org/127159 | 14:51 |
| mriedem | nova-manage has direct to db access | 14:51 |
| jroll | mriedem: I can if that's cool. it's basically going to be https://github.com/cernops/nova-flavor-create/blob/master/cern-flavor-create | 14:51 |
| jroll | but in a nova-manage-y way | 14:51 |
| jroll | and copy rather than create | 14:51 |
| * jroll jfdi | 14:52 | |
| bauzas | jroll: well, I don't want to propose yet-another-flavor-migration but I would consider passing a message to the child cells for creating flavors | 14:52 |
| *** sneti has quit IRC | 14:52 | |
| *** sneti_ has joined #openstack-nova | 14:52 | |
| dansmith | ugh :( | 14:53 |
| jroll | mehh | 14:53 |
| dansmith | so, one of the things we said we weren't going to do, | 14:53 |
| *** dims has quit IRC | 14:53 | |
| dansmith | was make cellsv1 easier or more attractive for people to deploy that currently aren't doing so | 14:53 |
| bauzas | yeah | 14:54 |
| dansmith | having a bunch of scattered tools required to make it work is a nice barrier, and adding them to nova-manage makes it look a lot more mainline | 14:54 |
| dansmith | I know that sucks | 14:54 |
| dansmith | but... | 14:54 |
| jroll | right | 14:55 |
| *** heyongli has quit IRC | 14:55 | |
| jroll | also it's going to be a pita to make nova-manage connect to both dbs, it seems | 14:56 |
| alaski | yeah, so I like the idea of pulling it into a tools dir or something, or just linking to it from docs. but not really pulling it into nova-manage | 14:56 |
| *** heyongli has joined #openstack-nova | 14:56 | |
| lpetrut | Hi guys, could someone please take a look over the os-win dependency spec? https://review.openstack.org/#/c/227827/ It has a +2 already. We are quite stuck until this spec and blueprint get approved. | 14:56 |
| johnthetubaguy | mriedem: markus_z: just wanted to quickly checkin about the centralise config options, I like the group into a few centrally located files, seems the best trade off | 14:56 |
| dansmith | alaski: link, but not pull, IMHO | 14:56 |
| jroll | so it would have to be a "add flavor to child cell" thing if we did nova-manage | 14:57 |
| jroll | alaski: dansmith: what I'm trying to do is have devstack do this if ironic and cells is enabled | 14:57 |
| jroll | so like, links don't help | 14:57 |
| dansmith | heh | 14:57 |
| johnthetubaguy | markus_z: I am +2 on that, minus the few little nits that need fixing | 14:57 |
| alaski | bauzas: we've discussed message passing for flavor create, but it breaks down when you consider a cell could be down when a flavor is created. we don't have the infra in Nova to handle that well | 14:57 |
| jroll | dansmith: and if we just say "this is never supported", that's fine and I'll sit over here and sync manually and be sad | 14:58 |
| bauzas | fair point, but I'm mostly in favor of dansmith's approach :) | 14:58 |
| alaski | can you create a flavor with nova-manage? | 14:58 |
| jroll | in a child? nope | 14:58 |
| jroll | not that I can tell anyway | 14:58 |
| alaski | just in general | 14:58 |
| *** burgerk has quit IRC | 14:58 | |
| markus_z | johnthetubaguy: Thanks, I'm going to change that by end of the day. | 14:59 |
| alaski | you can pass a config to nova-manage to have it do whatever it does in a different db | 14:59 |
| jroll | alaski: not that I can tell | 14:59 |
| alaski | jroll: okay | 14:59 |
| jroll | right, that clicked late | 14:59 |
| bauzas | jroll: the problem I see is that ironic will pull an hard dependency on something considered experimental in nova | 14:59 |
| jroll | bauzas: no, we aren't depending on it | 14:59 |
| jroll | bauzas: for development I want to support this | 14:59 |
| dansmith | he just wants it to work if configured | 14:59 |
| jroll | turns out my production deployment uses cells | 14:59 |
| bauzas | the if configured concerns me | 14:59 |
| jroll | and sometimes it has bugs | 15:00 |
| jroll | and I like being able to deploy it in devstack | 15:00 |
| openstackgerrit | Monty Taylor proposed openstack/nova: Change image to instance in comment https://review.openstack.org/242102 | 15:00 |
| dansmith | bauzas: well, we support cells in devstack today | 15:00 |
| bauzas | for nova only :) | 15:00 |
| jroll | this is for nova | 15:00 |
| dansmith | right, but ironic is a nova driver | 15:00 |
| jroll | with a given virt driver | 15:00 |
| dansmith | this is not about making ironic support cells | 15:00 |
| bauzas | a-ha I see | 15:00 |
| dansmith | jroll: have you been told you can't do it in devstack if you have to install cernops? | 15:00 |
| *** haomaiwa_ has quit IRC | 15:01 | |
| *** baoli has quit IRC | 15:01 | |
| jroll | dansmith: I haven't asked, don't feel like taking that smackdown | 15:01 |
| dansmith | jroll: and even still, we do this today without this flavor sync, are you hitting something blocking, or just saying it doesn't run well because you don't have something happening? | 15:01 |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| jroll | dansmith: it's just an extra manual thing | 15:01 |
| jroll | dansmith: today I run https://review.openstack.org/#/c/198896/1/lib/ironic | 15:01 |
| bauzas | correct me if I'm wrong but since child cells do copy the api cell db, is it also getting the default flavors, right? | 15:02 |
| jroll | without doing that, nova boot --flavor baremetal doesn't work at all | 15:02 |
| jroll | bauzas: it gets the default flavors when db sync happens | 15:02 |
| bauzas | the problem is when creating a new flavor which is not nova default, right? | 15:02 |
| jroll | right | 15:02 |
| jroll | (man, I did not expect to eat up this much of y'alls time with this) | 15:03 |
| dansmith | jroll: I see.. I'd rather your hack be in devstack than us bless it in nova | 15:03 |
| dansmith | which is selfish, I know :) | 15:03 |
| jroll | dansmith: and vice versa from devstack folks | 15:03 |
| bauzas | since ironic is a nova driver, couldn't we branch in nova devstack to add a new baremetal flavor if the driver is asked ? | 15:03 |
| jroll | :) | 15:03 |
| bauzas | (tbc, I see the change in devstack, not in nova) | 15:03 |
| *** pratikmallya has joined #openstack-nova | 15:04 | |
| dansmith | jroll: yeah, well, the sucky life of being an experimental and frozen feature :/ | 15:04 |
| *** tonytan4ever has joined #openstack-nova | 15:04 | |
| jroll | yeah | 15:04 |
| jroll | alright, I give up I think | 15:05 |
| mriedem | jroll: is this for an ironic job? | 15:05 |
| dansmith | sdague: have you already opined on this? | 15:05 |
| mriedem | jroll: maybe have a devstack plugin in ironic that hooks into devstack? | 15:05 |
| mriedem | that does your thing | 15:05 |
| jroll | mriedem: probably not | 15:05 |
| *** heyongli has quit IRC | 15:05 | |
| *** david-lyle has quit IRC | 15:06 | |
| mriedem | dansmith: no, https://review.openstack.org/#/c/198896/ ian was just concerned that it's mysql only | 15:06 |
| mriedem | and direct sql to the db, which isn't great | 15:06 |
| jroll | mriedem: yeah, I thought about that too, but doing that to pull in code that devstack folks say is terrible (which it is) | 15:06 |
| jroll | seems wrong | 15:06 |
| sdague | dansmith: which thing? | 15:06 |
| *** heyongli has joined #openstack-nova | 15:06 | |
| mriedem | sdague: https://review.openstack.org/#/c/198896/ | 15:06 |
| jroll | sdague: a better way to do 198896 | 15:06 |
| jroll | again, I'm totally fine with "that's dumb, just deal with it manually" | 15:07 |
| * jroll feels like he has stolen far too many man hours this morning | 15:07 | |
| sdague | so... raw sql in devstack seems like not fun for anyone right? | 15:07 |
| dansmith | sdague: I don't want to bless cells further by adding that to nova-manage. people have scripts to do this manually | 15:07 |
| dansmith | jroll: it's fine | 15:07 |
| mriedem | jroll: is this needed for a ci job? or just manual local testing? | 15:08 |
| mriedem | jroll: or your production cells baremetal cloud based on devstack? :) | 15:08 |
| jroll | mriedem: I don't currently have plans to make a job for this, but I also don't think that's a terrible idea | 15:08 |
| jroll | duh, don't you know rackspace runs devstack? | 15:08 |
| *** pratikma_ has joined #openstack-nova | 15:08 | |
| mriedem | *rax would run a fork of devstack | 15:09 |
| dansmith | I'm not sure I see a job for this being terrible worthwhile to me | 15:09 |
| jroll | hahaha | 15:09 |
| mriedem | dansmith: agreed | 15:09 |
| *** baoli has joined #openstack-nova | 15:09 | |
| dansmith | terribly | 15:09 |
| jroll | yeah, I'm not worried about a job for it | 15:09 |
| mriedem | cells + neutron would be a better investment at least if we're talking new cells jobs | 15:09 |
| dansmith | I agree it sucks to not have it just work, but | 15:09 |
| dansmith | mriedem: yeah, I don't really want that can of worms opened either, as you know, but... | 15:09 |
| mriedem | cells + neutron + vmware + containers + nfv | 15:09 |
| mriedem | that's the target golden job | 15:10 |
| *** eglynn has joined #openstack-nova | 15:10 | |
| dansmith | + docker | 15:10 |
| sdague | jroll: so, if ironic moves into a devstack plugin, you can do anything you like in it, including this | 15:10 |
| mriedem | well, that was the containers part | 15:10 |
| bauzas | + iot | 15:10 |
| *** w_verdugo has quit IRC | 15:10 | |
| jroll | sdague: yeah, and I plan to do that at some point. but doing the same hack there is just as terrible | 15:10 |
| sdague | jroll: it is, but you won't have to fight me about it :) | 15:11 |
| jroll | it's just that I don't need devstack-core blessing to do it | 15:11 |
| jroll | yeah | 15:11 |
| *** pratikmallya has quit IRC | 15:11 | |
| sdague | right, then it's on your own conscience | 15:11 |
| jroll | :/ | 15:11 |
| *** pumaranikar has joined #openstack-nova | 15:12 | |
| *** w_verdugo has joined #openstack-nova | 15:12 | |
| *** _gryf is now known as inc0 | 15:12 | |
| *** inc0 is now known as _gryf | 15:12 | |
| jroll | yeah, I'll probably just drop this and play with v2 eventually | 15:12 |
| cfriesen_ | dansmith: sorry, was busy in another window. I'm talking about autoconfirming the resize if the end-user deletes the instance. As far as I can tell we should be okay to auto-confirm on the source host (which cleans up the source resources) as soon as we set instance.task_state = task_states.RESIZE_MIGRATED (since the next call is a cast to finish_resize() on the dest host.) | 15:13 |
| *** _gryf is now known as dejson | 15:13 | |
| *** dejson is now known as _gryf | 15:13 | |
| *** apuimedo has joined #openstack-nova | 15:13 | |
| dansmith | cfriesen_: if you prove it happens on upstream code then I'm willing to believe that we should do that | 15:14 |
| *** cdent has joined #openstack-nova | 15:14 | |
| *** ccarmack has quit IRC | 15:14 | |
| mriedem | couldn't you just put a sleep somewhere in the resize code and delete the instance concurrently while it's sleeping to recreate? | 15:15 |
| mriedem | jaypipes: can you run through this again, i'm nearly +2 https://review.openstack.org/#/c/221310/ | 15:15 |
| mriedem | but you were on it earlier | 15:16 |
| *** heyongli has quit IRC | 15:16 | |
| *** otter768 has joined #openstack-nova | 15:16 | |
| jaypipes | mriedem: sure, will do shortly. | 15:16 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Deprecate Extensible Resource Tracker https://review.openstack.org/242110 | 15:16 |
| *** heyongli has joined #openstack-nova | 15:17 | |
| bauzas | johnthetubaguy: ^ that's I feel needed to do that before actually removing it | 15:17 |
| bauzas | just because that's a stevedore entrypoint | 15:17 |
| *** ccarmack has joined #openstack-nova | 15:17 | |
| *** mc_nair has joined #openstack-nova | 15:18 | |
| bauzas | so we could drop ERT once Nutella opens | 15:18 |
| mriedem | PaulMurray: ^ | 15:18 |
| cfriesen_ | mriedem: yes, that's how I reproduced it. I'm just not sure how to verify the downsides of the missing unplug with stock vswitch | 15:18 |
| *** cdent has quit IRC | 15:19 | |
| andrearosa | bauzas: Nutella is the best name ever! | 15:19 |
| *** cdent has joined #openstack-nova | 15:19 | |
| bauzas | that's a strange feeling. Once in my whole life, someone thinks that I picked a good name | 15:20 |
| johnthetubaguy | bauzas: +1 | 15:21 |
| dansmith | +1 for nutella | 15:21 |
| *** otter768 has quit IRC | 15:21 | |
| johnthetubaguy | although I would be hungry for the whole cycle... | 15:21 |
| *** elenhil has joined #openstack-nova | 15:21 | |
| * dansmith goes to get some nutella | 15:21 | |
| elenhil | hello, guys! | 15:22 |
| gibi | dansmith: hi! You were originaly involved in the Remove shared storage flag from evacuate API https://review.openstack.org/#/c/183033/ spec. Both sdauge and alex_xu already +1 it. Could you take a look on it if you have time? | 15:22 |
| elenhil | can anyone help me? i've got installation of kilo+nova network, and suddenly all newly booted instances stopped acquiring ip address from dhcp until nova-network is restarted | 15:22 |
| elenhil | simply /var/lib/nova/networks/nova-br1004.conf file isn't reloaded | 15:22 |
| elenhil | it happens on every compute node except controller | 15:22 |
| dansmith | gibi: okay | 15:22 |
| gibi | dansmith: thanks! | 15:22 |
| *** apuimedo has quit IRC | 15:22 | |
| *** sneti_ has quit IRC | 15:24 | |
| *** jaypipes has quit IRC | 15:25 | |
| dansmith | gibi: can you link me the review where we made it optional? that should probably be in the spec to drop this.. | 15:25 |
| *** heyongli has quit IRC | 15:26 | |
| *** heyongli has joined #openstack-nova | 15:27 | |
| *** jwcroppe has joined #openstack-nova | 15:28 | |
| mriedem | pkholkin: why not register options in nova.cmd.cells? https://review.openstack.org/#/c/221310/ | 15:28 |
| *** apuimedo has joined #openstack-nova | 15:28 | |
| gibi | dansmith: this is the code patch to make it optional in the compute side https://review.openstack.org/#/c/197951/ | 15:28 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Use EngineFacade from oslo_db.enginefacade https://review.openstack.org/221310 | 15:29 |
| *** mdrabe has quit IRC | 15:29 | |
| *** markvoelker has joined #openstack-nova | 15:30 | |
| *** amotoki has quit IRC | 15:30 | |
| *** markvoelker_ has quit IRC | 15:31 | |
| mriedem | snikitin: ^ | 15:31 |
| *** ijuwang has quit IRC | 15:32 | |
| mriedem | pkholkin: snikitin: it'd be nice if there were a way to automatically detect if we're hitting the database directly and don't have sqlalchemy_api.configure(CONF) set | 15:32 |
| mriedem | otherwise i'm just having to eye ball these | 15:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Deprecate Extensible Resource Tracker https://review.openstack.org/242110 | 15:32 |
| mriedem | bauzas: throw an UpgradeImpact on that and i'm +2 | 15:33 |
| bauzas | oh right, I was lazt | 15:34 |
| bauzas | lazy | 15:34 |
| bauzas | mriedem: btw. FWIW https://review.openstack.org/#/c/242007/ | 15:34 |
| *** salv-orlando has quit IRC | 15:34 | |
| bauzas | mriedem: re: http://lists.openstack.org/pipermail/openstack-dev/2015-November/078301.html | 15:35 |
| *** salv-orlando has joined #openstack-nova | 15:35 | |
| *** markvoelker_ has joined #openstack-nova | 15:35 | |
| bauzas | btw. does someone know how I could whitelist a generated sphinx conf file for PEP8? | 15:36 |
| *** heyongli has quit IRC | 15:36 | |
| mriedem | yeah sec | 15:37 |
| *** diogogmt has joined #openstack-nova | 15:37 | |
| dansmith | gibi: a couple things to fix. if you turn that around quickly I'll +2 | 15:37 |
| *** heyongli has joined #openstack-nova | 15:37 | |
| mriedem | bauzas: just add releasenotes here https://github.com/openstack/nova/blob/master/tox.ini#L126 | 15:37 |
| dansmith | gibi: unfortunately your patch to optionalize the flag wasn't complete enough to let you finish this in mitaka, but the remaining thing will be small | 15:37 |
| sdague | super easy +2 - https://review.openstack.org/#/c/242102/ | 15:37 |
| *** markvoelker has quit IRC | 15:37 | |
| *** juzuluag has joined #openstack-nova | 15:38 | |
| sdague | monty found a typo in a comment when I was showing him how to optimize their nodepool polling of resources | 15:38 |
| *** ctrath has quit IRC | 15:39 | |
| *** ijuwang has joined #openstack-nova | 15:40 | |
| bauzas | mriedem: thanks I was blind | 15:40 |
| snikitin | mriedem: Hm. I think we missed cells :( At the beginning we wanted to use "sqlalchemy_api.configure(CONF) " in nova.config.parse_args() to configure it in all services. But we decided to use this line just in several services. I agree that it'd be nice if there were a way to automatically detect if we're hitting the database directly. But we didn't find a way how to do these. | 15:41 |
| * bauzas facepalm | 15:41 | |
| gibi | dansmith: thanks. I'm updating the spec... | 15:41 |
| mriedem | bauzas: can we do that config in the doc/source/conf.py? | 15:42 |
| bauzas | mriedem: that's not the recommended approach by dhellmann | 15:42 |
| bauzas | but I'm open to anything :) | 15:42 |
| mriedem | oh | 15:42 |
| mriedem | can we put the release notes conf.py in doc/source though? | 15:42 |
| mriedem | that would fix the pep8 issue | 15:42 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Use EngineFacade from oslo_db.enginefacade https://review.openstack.org/221310 | 15:42 |
| bauzas | I was actually just needing that for a change that I wanted to add a relnotes :) | 15:42 |
| bauzas | so since I'm the guinea pig, it's better to write stuff rather than waiting someone to step up - I'm not *that* lazy | 15:43 |
| mriedem | dansmith: do you have any ideas about a better way to scrub the places that hit the db directly so we make sure we have the proper services covered? https://review.openstack.org/#/c/221310/ | 15:43 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Remove shared storage flag in evacuate API https://review.openstack.org/183033 | 15:43 |
| gibi | dansmith: it ^^ is up | 15:43 |
| bauzas | mriedem: well, this is a sphinx problem (about the conf.py location) | 15:44 |
| dansmith | mriedem: I'll look in a sec | 15:44 |
| mriedem | bauzas: i can't remember if you can only specify one config file | 15:44 |
| mriedem | i know it by default looks for doc/source/conf.py | 15:44 |
| mriedem | or pbr does | 15:44 |
| mriedem | but i thought there was a way to override that | 15:44 |
| mriedem | anyway, not a huge deal | 15:44 |
| *** ctrath has joined #openstack-nova | 15:44 | |
| bauzas | mriedem: tbc, https://review.openstack.org/#/c/242007/1/tox.ini,cm does look at relnotes/source | 15:45 |
| dansmith | gibi: you removed the hint on line 65 that the user can still hit the password api, which was critical for selling me on removing it.. was that intentional? | 15:45 |
| jroll | bauzas: probably going to want a releasenotes job that builds those, too | 15:45 |
| bauzas | jroll: I wrote that too in project-config | 15:45 |
| jroll | nice | 15:45 |
| * jroll about to do this for ironic | 15:45 | |
| *** rlrossit has quit IRC | 15:45 | |
| *** heyongli has quit IRC | 15:46 | |
| bauzas | mriedem: so about the conf.py merge with the doc one, I'd defer that to dhellmann | 15:47 |
| *** heyongli has joined #openstack-nova | 15:47 | |
| *** gszasz has quit IRC | 15:47 | |
| gibi | dansmith: no, it wasn't intentional. I mixed up my repo. I'm putting that back | 15:47 |
| *** mdrabe has joined #openstack-nova | 15:47 | |
| bauzas | mriedem: in the glance example, he provided a separate namespace for relnotes | 15:48 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Remove shared storage flag in evacuate API https://review.openstack.org/183033 | 15:48 |
| gibi | dansmith: now it is fixed | 15:48 |
| dansmith | gibi: now it is +2d | 15:48 |
| gibi | dansmith: thanks! | 15:48 |
| dansmith | mriedem: relatively easy spec ^ | 15:49 |
| *** diogogmt has quit IRC | 15:49 | |
| mriedem | is anyone working on creating a ConsolePool object? | 15:49 |
| mriedem | dansmith: i'll take a look | 15:49 |
| mriedem | nova.console.xvp hits the db directly for console pools | 15:49 |
| mriedem | seems we should objectify that | 15:49 |
| *** diogogmt has joined #openstack-nova | 15:49 | |
| *** rlrossit has joined #openstack-nova | 15:51 | |
| dansmith | mriedem: hmm, yeah | 15:51 |
| dansmith | assuming it's used | 15:51 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Deprecate Extensible Resource Tracker https://review.openstack.org/242110 | 15:51 |
| mriedem | would be a question for the xen boyz | 15:51 |
| dansmith | alaski: johnthetubaguy: rax doesn't use that do they? I wonder if it's known to work such that we should fix it, or just remove? | 15:51 |
| mriedem | johnthetubaguy: ^ does anyone use xvp consoles? | 15:51 |
| mriedem | btw, dibs on creating the ConsolePool object if that is a needed thing | 15:52 |
| alaski | we don't use xvp consoles | 15:52 |
| dansmith | bah | 15:52 |
| dansmith | I wasnted it | 15:52 |
| dansmith | we, wanted | 15:52 |
| * dansmith can't type today | 15:53 | |
| mriedem | nutella fingers | 15:53 |
| *** edtubill has joined #openstack-nova | 15:53 | |
| mriedem | mash the keypad now if you need a special dialing wand | 15:53 |
| mriedem | rlrossit: ^ | 15:53 |
| *** jistr is now known as jistr|afkmtg | 15:53 | |
| *** Jeffrey4l has quit IRC | 15:53 | |
| dansmith | more like got-sick-in-japan-head-swollen-can't-see | 15:53 |
| *** edtubill has quit IRC | 15:54 | |
| rlrossit | mriedem: what's going on? | 15:54 |
| *** edtubill has joined #openstack-nova | 15:54 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Deprecate Extensible Resource Tracker https://review.openstack.org/242110 | 15:55 |
| *** bnemec has quit IRC | 15:55 | |
| *** pumaranikar has quit IRC | 15:55 | |
| rlrossit | mriedem: oh the simpsons reference? | 15:55 |
| mriedem | yes | 15:55 |
| dansmith | rlrossit: mriedem wants to use your dialing wand | 15:55 |
| *** Sree has quit IRC | 15:55 | |
| mriedem | it's the only thing we actually talk about | 15:55 |
| *** pumaranikar has joined #openstack-nova | 15:55 | |
| *** salv-orl_ has joined #openstack-nova | 15:56 | |
| rlrossit | dansmith: gross | 15:56 |
| *** ihrachys has quit IRC | 15:56 | |
| *** yamahata has quit IRC | 15:56 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova-specs: Centralize Config Options https://review.openstack.org/227948 | 15:56 |
| *** heyongli has quit IRC | 15:57 | |
| *** burgerk has joined #openstack-nova | 15:57 | |
| *** angdraug has joined #openstack-nova | 15:57 | |
| *** heyongli has joined #openstack-nova | 15:58 | |
| *** salv-orlando has quit IRC | 15:59 | |
| PaulMurray | bauzas, johnthetubaguy, you guys been completing my patches for me again? | 15:59 |
| bauzas | PaulMurray: not really | 15:59 |
| bauzas | PaulMurray: just adding a deprecation notice | 15:59 |
| bauzas | because we need to wait 3 months and a release | 16:00 |
| bauzas | before removing the code | 16:00 |
| bauzas | in particular since we have some stevedore entrypoints | 16:00 |
| *** haomaiwang has quit IRC | 16:01 | |
| bauzas | that way, people can still use custom plugins w/ ERT for Mitaka but they know they have to kick'em by Nutella | 16:01 |
| *** haomaiwang has joined #openstack-nova | 16:01 | |
| PaulMurray | I know, that's why I said it in the commit message | 16:02 |
| PaulMurray | I was about to do that patch | 16:02 |
| PaulMurray | you know, I told the guys here that I was going to try pushing a commit message and see if anyone completes the patch | 16:02 |
| PaulMurray | But now I just mentioned a patch in another commit message and it got done! :) | 16:03 |
| bauzas | oh I hope you din't found yourself hijacked, that was not my purpose | 16:03 |
| bauzas | I saw your commit msg notice but I thought you were writing a removal patch | 16:03 |
| markus_z | mriedem: Would you +2 this https://review.openstack.org/#/c/215102/ danpb already blessed it. | 16:04 |
| *** pumaranikar has quit IRC | 16:04 | |
| bauzas | PaulMurray: does that work for you ? I can also abandon mine, I just want to get the deprecation notice asap in tree | 16:04 |
| mriedem | geez, no foreplay or anything | 16:04 |
| johnthetubaguy | PaulMurray: :) | 16:05 |
| markus_z | mriedem: no time for that :D | 16:05 |
| *** jlanoux has quit IRC | 16:05 | |
| * johnthetubaguy hopes people don't search for too many keywords in the IRC logs | 16:06 | |
| mriedem | they would find worse than that | 16:06 |
| PaulMurray | bauzas, I quote "a follow on patch will add deprecation notices..." | 16:06 |
| PaulMurray | bauzas, don't worry | 16:07 |
| PaulMurray | bauzas, think of it as a prediction | 16:07 |
| markus_z | I'm already censoring myself | 16:07 |
| *** emagana has joined #openstack-nova | 16:07 | |
| *** heyongli has quit IRC | 16:07 | |
| *** dshaw has joined #openstack-nova | 16:07 | |
| bauzas | PaulMurray: oh my bad - I'm certainly a douchebag | 16:07 |
| *** pumaranikar has joined #openstack-nova | 16:08 | |
| *** heyongli has joined #openstack-nova | 16:08 | |
| mriedem | bauzas: you add more things, i comment on more things https://review.openstack.org/#/c/242110/4/nova/compute/resource_tracker.py | 16:08 |
| bauzas | mriedem: don't tell me I'm verbose :) | 16:09 |
| *** alex_klimov has quit IRC | 16:09 | |
| bauzas | PaulMurray: happy with updating my change with mriedem's concern ? | 16:10 |
| mriedem | i'd update the config option help with the same deprecation warning that's in the init method | 16:10 |
| *** salv-orl_ has quit IRC | 16:10 | |
| bauzas | that's a valid point | 16:10 |
| dshaw | Given this doc bug affects nova, I wasn't sure if there was any input on this low-hanging-fruit bug https://bugs.launchpad.net/openstack-manuals/+bug/1512978 | 16:11 |
| openstack | Launchpad bug 1512978 in openstack-manuals "Specifying Wrong file to edit on Ubuntu 14..04" [Undecided,Incomplete] - Assigned to Darren Shaw (ds354m) | 16:11 |
| *** mrkz has quit IRC | 16:11 | |
| PaulMurray | bauzas, I'll do it - that way I can add my name to the commit message and we will win the stats race H ahahaha ahah aha a a aaaa aa.... | 16:11 |
| bauzas | :) | 16:12 |
| *** sahid has quit IRC | 16:12 | |
| PaulMurray | bauzas, while you look for somewhere near Austin call Nutella | 16:12 |
| bauzas | that way, you'll get half the ranting of operators wanting the ERT back | 16:12 |
| bauzas | :) | 16:12 |
| PaulMurray | oh| | 16:12 |
| PaulMurray | hadn't thought of that | 16:12 |
| *** alexschm has quit IRC | 16:12 | |
| PaulMurray | at least there will be one less of them by then | 16:13 |
| *** hemnafk is now known as hemna | 16:13 | |
| *** sahid has joined #openstack-nova | 16:13 | |
| *** jerrygb has quit IRC | 16:13 | |
| PaulMurray | of course you are assuming widespred use of the feature | 16:13 |
| mriedem | dansmith: you don't have any issues with the admin password stuff in this onSharedStorage evacuation bp? | 16:13 |
| cfriesen_ | Maybe we can steal the Nutella release from Android. | 16:13 |
| *** tongli has joined #openstack-nova | 16:14 | |
| *** baoli has quit IRC | 16:14 | |
| bauzas | I also considered naming the release with a cheese name, but for some unknown reason, there is no french cheese starting with N. I guess it's a plot | 16:14 |
| *** baoli has joined #openstack-nova | 16:14 | |
| *** pratikma_ has quit IRC | 16:15 | |
| markus_z | bauzas: I thought you guys have 20 different names for cheese? ;) | 16:15 |
| *** jerrygb has joined #openstack-nova | 16:15 | |
| *** setuid has quit IRC | 16:15 | |
| PaulMurray | 63000 cheeses and no N? | 16:15 |
| bauzas | my bad, there is a Neufchatel one | 16:16 |
| bauzas | https://en.wikipedia.org/wiki/List_of_French_cheeses | 16:16 |
| bauzas | but I diverge | 16:16 |
| PaulMurray | I was just looking at that :) | 16:16 |
| markus_z | bauzas: oh man, how do I pronounce that? | 16:17 |
| *** heyongli has quit IRC | 16:17 | |
| bauzas | markus_z: nee-cha-tel | 16:17 |
| markus_z | bauzas: Please record it and update the wikipedia entry with it :) | 16:18 |
| *** heyongli has joined #openstack-nova | 16:18 | |
| johnthetubaguy | it kinda looks like a german cheese with 9 things in it, but anyways... | 16:18 |
| *** browne has joined #openstack-nova | 16:19 | |
| edleafe | ...and a German name | 16:19 |
| *** inteq has joined #openstack-nova | 16:19 | |
| markus_z | http://www.quickmeme.com/img/29/29db411914f980415f23fad69311c356278a6ab55117f4e7d939da2a934d4c4a.jpg | 16:19 |
| *** baoli has quit IRC | 16:21 | |
| *** setuid has joined #openstack-nova | 16:21 | |
| johnthetubaguy | markus_z: heh | 16:21 |
| *** baoli has joined #openstack-nova | 16:21 | |
| *** dshaw has quit IRC | 16:22 | |
| *** pixelbeat_ has quit IRC | 16:22 | |
| johnthetubaguy | mriedem: dansmith: for the nova meeting later, are one of you OK running that, not had chance to see if mikal wanted to run it, the daylight savings change means I am totally unable to get there now :( | 16:23 |
| mriedem | gibi: dansmith: i have questions inline about this before i can score it https://review.openstack.org/#/c/183033/ | 16:23 |
| mriedem | i think it's 3pm for me so i can do it | 16:23 |
| dansmith | johnthetubaguy: +1 for mriedem | 16:23 |
| *** dave-mccowan has quit IRC | 16:24 | |
| edleafe | yeah - it'll be a 10-minute meeting with mriedem | 16:24 |
| mriedem | tpeoples: jwcroppe: you guys should probably review this spec https://review.openstack.org/#/c/183033/ | 16:24 |
| johnthetubaguy | mriedem: thanks, that would be great | 16:24 |
| markus_z | :D | 16:24 |
| johnthetubaguy | mriedem: I have put lots of crud in the agenda, but a lot of that I just sent out to the ML (mitaka dates and stuff) https://wiki.openstack.org/wiki/Meetings/Nova | 16:25 |
| mriedem | ok, i'll skim the crud | 16:25 |
| mriedem | alex_xu: you might want to skim this per the api impact https://review.openstack.org/#/c/183033/ | 16:26 |
| *** heyongli has quit IRC | 16:27 | |
| tpeoples | thanks mriedem, will take a peek in a bit | 16:27 |
| *** pixelbeat_ has joined #openstack-nova | 16:28 | |
| *** dave-mccowan has joined #openstack-nova | 16:28 | |
| *** heyongli has joined #openstack-nova | 16:28 | |
| *** Guest46631 has quit IRC | 16:29 | |
| jwcroppe | mriedem: will take a look | 16:29 |
| *** burt has joined #openstack-nova | 16:29 | |
| *** jaypipes has joined #openstack-nova | 16:30 | |
| *** Piet has joined #openstack-nova | 16:30 | |
| mriedem | rlrossit: https://www.youtube.com/watch?v=Ti5AkLup1mI | 16:30 |
| *** jerrygb has quit IRC | 16:31 | |
| *** jerrygb has joined #openstack-nova | 16:31 | |
| rlrossit | mriedem: A+ | 16:31 |
| rlrossit | what in the world provoked that? | 16:31 |
| mriedem | you don't want to kno | 16:32 |
| mriedem | *know | 16:32 |
| mriedem | or i don't want to say | 16:32 |
| *** b3nt_pin has joined #openstack-nova | 16:36 | |
| *** b3nt_pin is now known as Guest91242 | 16:36 | |
| *** mnestratov has joined #openstack-nova | 16:36 | |
| xek | dansmith, Hi, can you point me to the person you mentioned at the summit, that can help with introduction of multi-node grenade to projects CI? | 16:36 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs: WIP: Add concept of resource providers https://review.openstack.org/225546 | 16:37 |
| dansmith | xek: sdague | 16:38 |
| *** heyongli has quit IRC | 16:38 | |
| jaypipes | mriedem: review done on that enginefacade patch. bug in unit tests.. | 16:38 |
| jaypipes | snikitin: ^^ | 16:38 |
| xek | dansmith, thanks | 16:38 |
| *** heyongli has joined #openstack-nova | 16:39 | |
| snikitin | jaypipes: thanks I saw | 16:39 |
| jaypipes | snikitin: push a quick fix and I will re-review promptly (and the followup patches0 | 16:39 |
| mriedem | jaypipes: yeah, it's also missing the conf setup stuff in some other services | 16:40 |
| *** pratikmallya has joined #openstack-nova | 16:40 | |
| mriedem | which is very hit or miss, i wish we had a way to force a failure to detect if that wasn't configured properly | 16:40 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Use EngineFacade from oslo_db.enginefacade https://review.openstack.org/221310 | 16:40 |
| snikitin | mriedem, jaypipes: done | 16:40 |
| *** moshele has quit IRC | 16:41 | |
| *** electrocucaracha has quit IRC | 16:41 | |
| jaypipes | snikitin: sweet :) will re-review shortly. | 16:41 |
| jaypipes | snikitin: thx for your prompt attention to it! | 16:41 |
| snikitin | jaypipes: thanks for review :) | 16:41 |
| jaypipes | snikitin: lemme know if/when you push rebases for the dependent patches in the series. | 16:42 |
| *** y_sawai has quit IRC | 16:42 | |
| *** pratikma_ has joined #openstack-nova | 16:43 | |
| sdague | xek: what's the question? | 16:43 |
| *** sahid has quit IRC | 16:43 | |
| dansmith | mriedem: what part of that enginefacade thing did you want me to look at? | 16:43 |
| dansmith | mriedem: I wish we could just deprecate the use_local thing and not add any of that to the non-db services :/ | 16:43 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'instance_info' and 'instance_extra' https://review.openstack.org/225065 | 16:43 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'cell_mapping' https://review.openstack.org/226788 | 16:43 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'instance_mapping' https://review.openstack.org/228979 | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'key_pair' and 'cell' https://review.openstack.org/239915 | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'network' https://review.openstack.org/240253 | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'certificate' and 'pci_device' https://review.openstack.org/240275 | 16:44 |
| mriedem | dansmith: just hte pattern here for registering the options https://review.openstack.org/#/c/221310/28/nova/cmd/all.py | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: enginefacade: 'floating_ip' https://review.openstack.org/240513 | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: enginefacade: 'fixed_ip' https://review.openstack.org/240811 | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'service' https://review.openstack.org/241178 | 16:44 |
| mriedem | dansmith: in all of the commands that can possibly touch the db | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'host_mapping' https://review.openstack.org/242035 | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'dnsdomain' and 'ec2' https://review.openstack.org/242019 | 16:44 |
| mriedem | i was wondering if there was an easier way than me eyeballing the code | 16:44 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'compute_node' https://review.openstack.org/241189 | 16:44 |
| snikitin | jaypipes: rebased :) | 16:44 |
| *** pratikmallya has quit IRC | 16:45 | |
| mriedem | snikitin: https://github.com/openstack/nova/blob/master/tools/xenserver/vm_vdi_cleaner.py#L32 :( | 16:46 |
| *** mgoddard_ has joined #openstack-nova | 16:46 | |
| jaypipes | snikitin: спасибо! | 16:46 |
| *** ccarmack has quit IRC | 16:46 | |
| mriedem | i think that's the last one i found | 16:47 |
| dansmith | mriedem: yeah, it's rather verbose | 16:47 |
| *** Guest91242 is now known as beagles | 16:47 | |
| *** mleroy has quit IRC | 16:47 | |
| mriedem | snikitin: why didn't we just process the oslo_db options in nova.config.parse_args? | 16:47 |
| mriedem | globally | 16:47 |
| mriedem | i kind of hate copy/pasting this everywhere | 16:47 |
| *** heyongli has quit IRC | 16:48 | |
| dansmith | yeah | 16:48 |
| *** sfinucan has joined #openstack-nova | 16:48 | |
| *** elenhil_ has joined #openstack-nova | 16:49 | |
| dansmith | mriedem: alternately, we may want to just pull out the service startup logic that checks use_local, and either does the conductor stuff, blocks db, or does this config | 16:49 |
| *** heyongli has joined #openstack-nova | 16:49 | |
| dansmith | we already have that replicated in a few places which could use cleaning up | 16:49 |
| *** mgoddard has quit IRC | 16:49 | |
| dansmith | but, like I said, it would be nice to get rid of the local case entirely | 16:49 |
| dansmith | johnthetubaguy: how do you feel about us deprecating and eventually removing local conductor mode? | 16:49 |
| *** kylek3h_ has joined #openstack-nova | 16:49 | |
| dansmith | johnthetubaguy: any results from using pymysql to see if it addresses your performance issues enough? | 16:49 |
| *** cdent_ has joined #openstack-nova | 16:50 | |
| johnthetubaguy | dansmith: given the bug belliott spotted with it the other week, its very tempting | 16:50 |
| dansmith | johnthetubaguy: which bug? | 16:50 |
| *** subscope has quit IRC | 16:50 | |
| johnthetubaguy | dansmith: I don't remember now, something about an exception changing from a messagetimeout to something else in local mode | 16:51 |
| dansmith | ah | 16:51 |
| dansmith | yeah | 16:51 |
| johnthetubaguy | I think when you loose connection to the DB | 16:51 |
| alaski | https://review.openstack.org/#/c/233914/4 | 16:51 |
| *** jgriffith has joined #openstack-nova | 16:51 | |
| *** jgriffith is now known as Guest88047 | 16:51 | |
| johnthetubaguy | alaski: thanks, that it | 16:51 |
| alaski | I was just looking at it | 16:51 |
| *** cfriesen__ has joined #openstack-nova | 16:51 | |
| dansmith | we mostly had that for the early days of no-db-compute, but supporting both paths is asking for pain now that we use it so transparently now | 16:51 |
| *** alex_xu_ has joined #openstack-nova | 16:51 | |
| *** su_zhang has quit IRC | 16:52 | |
| johnthetubaguy | dansmith: yeah, I am tempted by deprecating it | 16:52 |
| *** mari0jv has joined #openstack-nova | 16:52 | |
| johnthetubaguy | dansmith: our big issues were scaling up the rabbit cluster AFAIK, but I haven't caught up with those folks recently | 16:52 |
| johnthetubaguy | s/cluster/nodes/ | 16:52 |
| *** ccarmack has joined #openstack-nova | 16:52 | |
| regebro | I thought you might enjoy this: "Host has more disk space than database expected (278gb > -3gb)" | 16:53 |
| dansmith | johnthetubaguy: okay | 16:53 |
| alaski | we also got bit by a local patch that was doing direct db access | 16:53 |
| *** mrkz has joined #openstack-nova | 16:53 | |
| alaski | which started failing | 16:53 |
| dansmith | heh | 16:53 |
| *** ameade has quit IRC | 16:53 | |
| johnthetubaguy | funny | 16:53 |
| *** sc`_ has joined #openstack-nova | 16:53 | |
| *** dosaboy_ has joined #openstack-nova | 16:54 | |
| openstackgerrit | Derek Higgins proposed openstack/nova: Handle a NeutronClientException 404 Error for floating ips https://review.openstack.org/242158 | 16:54 |
| *** tangchen_ has joined #openstack-nova | 16:54 | |
| *** StevenK_ has joined #openstack-nova | 16:54 | |
| *** yassine__ has quit IRC | 16:54 | |
| *** mriedem1 has joined #openstack-nova | 16:55 | |
| openstackgerrit | Paul Murray proposed openstack/nova: Deprecate Extensible Resource Tracker https://review.openstack.org/242110 | 16:55 |
| snikitin | mriedem: because engine facade could not be configered twice. https://github.com/openstack/oslo.db/blob/master/oslo_db/sqlalchemy/enginefacade.py#L220 | 16:55 |
| snikitin | I tried to configure it in parse_args and got a lot of problems with fixtures. For example here https://github.com/openstack/nova/blob/master/nova/tests/unit/conf_fixture.py#L59 | 16:55 |
| snikitin | I argee that it would be best solution to configure it in psrce_args(). But i need to use some global flags IS_ENGINE_FACADE_CONFIGURED or some new arguments in parse_args() like parse_args(....., configure_db = True) to disable configuration if fixtures. will it be ok? | 16:55 |
| *** beagles has quit IRC | 16:56 | |
| johnthetubaguy | dansmith: one less test combination, and one less config, so its deprecating time I think | 16:56 |
| *** codebaus1 has joined #openstack-nova | 16:57 | |
| *** beagles has joined #openstack-nova | 16:57 | |
| *** ameade has joined #openstack-nova | 16:57 | |
| snikitin | * in fixtures | 16:57 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs: Add concept of resource providers https://review.openstack.org/225546 | 16:57 |
| jaypipes | alex_xu, johnthetubaguy, PaulMurray, edleafe, dansmith: I pushed an updated rev for the resource providers spec ^^. Please review at your convenience. thx in advance. | 16:57 |
| dansmith | johnthetubaguy: yeah, we have a bunch of stuff in tests for testing both cases | 16:57 |
| mriedem1 | snikitin: a configure_db=True kwarg to parse_args seems cleaner | 16:57 |
| mriedem1 | pass False for the tests | 16:57 |
| dansmith | johnthetubaguy: if you're cool with it, I'll add a deprecation log message | 16:57 |
| snikitin | mriedem1: ok great! will do | 16:58 |
| *** slDabbler has joined #openstack-nova | 16:58 | |
| dansmith | johnthetubaguy: at least propose it and let it sit for comment | 16:58 |
| *** e0ne has quit IRC | 16:58 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: add first reno-based release note https://review.openstack.org/242008 | 16:58 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add reno for release notes management https://review.openstack.org/242007 | 16:58 |
| mriedem1 | could also see what dansmith and jaypipes think about that | 16:58 |
| *** heyongli has quit IRC | 16:58 | |
| *** ihrachys has joined #openstack-nova | 16:58 | |
| *** barra204 has joined #openstack-nova | 16:58 | |
| dansmith | mriedem1: I also commented about unifying all that service startup logic | 16:58 |
| dansmith | which would make it easier for me to log a deprecation warning :) | 16:59 |
| dansmith | but I could do that after this set if snikitin doesn't want to | 16:59 |
| *** dave-mccowan has quit IRC | 16:59 | |
| dansmith | when I go to add it | 16:59 |
| *** heyongli has joined #openstack-nova | 16:59 | |
| *** dhellmann_ has joined #openstack-nova | 17:00 | |
| dansmith | johnthetubaguy: alaski: also curious what you think about enabling conductor mode for API, but opt-in instead of opt-out, so you could make API use conductor during an upgrade | 17:00 |
| mriedem1 | back in a sec | 17:00 |
| *** mriedem1 has quit IRC | 17:00 | |
| *** mriedem1 has joined #openstack-nova | 17:00 | |
| *** arif-ali_ has joined #openstack-nova | 17:00 | |
| *** cburgess_ has joined #openstack-nova | 17:00 | |
| edleafe | jaypipes: coolness. Added to my list. | 17:01 |
| *** haomaiwang has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-nova | 17:01 | |
| *** sirushti_ has joined #openstack-nova | 17:01 | |
| *** adreznec has quit IRC | 17:01 | |
| PaulMurray | jaypipes, re: https://review.openstack.org/#/c/242110/4/nova/compute/resources/__init__.py that's really involved message - do you think its necessary? | 17:02 |
| *** lsmola has quit IRC | 17:02 | |
| *** Bent0_ has joined #openstack-nova | 17:02 | |
| *** pushkaru has joined #openstack-nova | 17:02 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 17:02 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 17:02 | |
| *** pumaranikar has quit IRC | 17:02 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 17:03 | |
| mriedem1 | dansmith: ok, wfm | 17:03 |
| mriedem1 | lunch time | 17:03 |
| *** mriedem1 is now known as mriedem_lunch | 17:03 | |
| jaypipes | PaulMurray: maybe in the help text only? Basically, we need a big bright warning, though :) which is why I thought it would be good. | 17:03 |
| *** cdent has quit IRC | 17:04 | |
| *** elenhil has quit IRC | 17:04 | |
| *** cfriesen_ has quit IRC | 17:04 | |
| *** mriedem has quit IRC | 17:04 | |
| *** ericksonsantos has quit IRC | 17:04 | |
| *** alex_xu has quit IRC | 17:04 | |
| *** Guest90242 has quit IRC | 17:04 | |
| *** dhellmann has quit IRC | 17:04 | |
| *** kylek3h has quit IRC | 17:04 | |
| *** dougwig has quit IRC | 17:04 | |
| *** arif-ali has quit IRC | 17:04 | |
| *** andrewbogott has quit IRC | 17:04 | |
| *** codebauss has quit IRC | 17:04 | |
| *** tangchen has quit IRC | 17:04 | |
| *** q3k has quit IRC | 17:04 | |
| *** lxsli has quit IRC | 17:04 | |
| *** StevenK has quit IRC | 17:04 | |
| *** cburgess has quit IRC | 17:04 | |
| *** sc` has quit IRC | 17:04 | |
| *** dosaboy has quit IRC | 17:04 | |
| *** mariojv has quit IRC | 17:04 | |
| *** raorn has quit IRC | 17:04 | |
| *** shakamunyi has quit IRC | 17:04 | |
| *** sirushti has quit IRC | 17:04 | |
| *** Bent0 has quit IRC | 17:04 | |
| *** sirushti_ is now known as sirushti | 17:04 | |
| *** arif-ali_ is now known as arif-ali | 17:04 | |
| *** sc`_ is now known as sc` | 17:05 | |
| *** dhellmann_ is now known as dhellmann | 17:05 | |
| *** cdent_ is now known as cdent | 17:05 | |
| *** dougwig_ has joined #openstack-nova | 17:05 | |
| jaypipes | wheeeeeeee, net splits. | 17:05 |
| *** mgoddard_ has quit IRC | 17:05 | |
| *** mgoddard has joined #openstack-nova | 17:05 | |
| *** ericksonsantos has joined #openstack-nova | 17:05 | |
| *** agireud has quit IRC | 17:05 | |
| *** stackdump has joined #openstack-nova | 17:05 | |
| *** adreznec has joined #openstack-nova | 17:05 | |
| *** rushiagr_away is now known as rushiagr | 17:06 | |
| *** lxsli has joined #openstack-nova | 17:06 | |
| *** agireud has joined #openstack-nova | 17:07 | |
| *** q3k has joined #openstack-nova | 17:07 | |
| *** rlrossit has quit IRC | 17:07 | |
| *** ctrath has quit IRC | 17:07 | |
| *** raorn has joined #openstack-nova | 17:08 | |
| PaulMurray | jaypipes, I guess a message to dev and ops MLs would be in order as well | 17:08 |
| *** heyongli has quit IRC | 17:08 | |
| PaulMurray | ? | 17:09 |
| *** heyongli has joined #openstack-nova | 17:09 | |
| *** nkrinner has quit IRC | 17:10 | |
| *** bnemec has joined #openstack-nova | 17:10 | |
| sdague | I'm pro big warning that jaypipes put in there, I think it's good to let people know stuff like that pretty explicitly | 17:12 |
| *** josecastroleon has quit IRC | 17:13 | |
| *** rotbeard has quit IRC | 17:13 | |
| *** andrewbogott has joined #openstack-nova | 17:13 | |
| jaypipes | PaulMurray: yeah, agreed. you want to send that? | 17:13 |
| jaypipes | sdague: it's like the Big Red Button in Ren and Stimpy. | 17:13 |
| PaulMurray | sdague, jaypipes sure - I'll do both | 17:13 |
| PaulMurray | message in patch and email | 17:14 |
| *** jistr|afkmtg is now known as jistr | 17:14 | |
| PaulMurray | sdague, BTW - pain the neck changing email address | 17:14 |
| *** andrewbogott has quit IRC | 17:14 | |
| *** andrewbogott has joined #openstack-nova | 17:14 | |
| *** cdent has left #openstack-nova | 17:15 | |
| PaulMurray | that's when you realise you never understood what was being authenticated | 17:15 |
| sdague | heh | 17:15 |
| dansmith | jaypipes: ah, Ren and Stimpy... | 17:15 |
| sdague | this is why I used my personal address in all my patches | 17:16 |
| jaypipes | PaulMurray: cheers | 17:17 |
| *** otter768 has joined #openstack-nova | 17:17 | |
| *** dshaw has joined #openstack-nova | 17:17 | |
| *** pradk has quit IRC | 17:18 | |
| *** heyongli has quit IRC | 17:19 | |
| *** heyongli has joined #openstack-nova | 17:20 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Deprecate local conductor mode https://review.openstack.org/242168 | 17:21 |
| dansmith | johnthetubaguy: lightning rod ^ | 17:21 |
| *** pradk has joined #openstack-nova | 17:21 | |
| *** otter768 has quit IRC | 17:21 | |
| *** ZZelle_ has joined #openstack-nova | 17:22 | |
| *** ctrath has joined #openstack-nova | 17:23 | |
| dansmith | gibi: if you want to respin to move the link to the other blueprint into refs I will re-+2 quickly | 17:26 |
| *** achanda has joined #openstack-nova | 17:26 | |
| gibi | dansmith: OK I will do it quickly.. | 17:27 |
| *** heyongli has quit IRC | 17:29 | |
| *** dshaw has quit IRC | 17:30 | |
| *** heyongli has joined #openstack-nova | 17:30 | |
| *** matrohon has quit IRC | 17:31 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs: Remove shared storage flag in evacuate API https://review.openstack.org/183033 | 17:31 |
| *** smurke has joined #openstack-nova | 17:31 | |
| gibi | dansmith: done ^^ | 17:32 |
| dansmith | got it, thanks | 17:32 |
| gibi | dansmith: thank you | 17:32 |
| *** lpetrut has quit IRC | 17:33 | |
| *** unicell1 has quit IRC | 17:33 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Build timeouts task filters deleting instances https://review.openstack.org/240017 | 17:34 |
| *** ccarmack1 has joined #openstack-nova | 17:34 | |
| *** subscope has joined #openstack-nova | 17:34 | |
| *** ccarmack has quit IRC | 17:34 | |
| *** nik_ has joined #openstack-nova | 17:37 | |
| johnthetubaguy | dansmith: speed of light is low today, taking a look now | 17:37 |
| *** heyongli has quit IRC | 17:39 | |
| dansmith | I could also post to the ops list to see if there's major opposition to this | 17:39 |
| dansmith | if there is, we probably already have a problem though | 17:40 |
| *** heyongli has joined #openstack-nova | 17:40 | |
| *** smatzek has quit IRC | 17:41 | |
| *** e0ne has joined #openstack-nova | 17:41 | |
| *** slDabbler has quit IRC | 17:41 | |
| dansmith | heyongli: do you know what the status of the pci ci job is? seems like it has been immediately failing for quite some time now | 17:41 |
| *** longsube has joined #openstack-nova | 17:42 | |
| *** pratikmallya has joined #openstack-nova | 17:42 | |
| *** achanda has quit IRC | 17:43 | |
| alaski | dansmith: what does having the API use conductor buy us? Objects protect us from db schema changes, and I typically think of conductor as protecting against object version changes. | 17:44 |
| *** pratikma_ has quit IRC | 17:44 | |
| *** pratikma_ has joined #openstack-nova | 17:44 | |
| *** ssurana has joined #openstack-nova | 17:44 | |
| openstackgerrit | Paul Murray proposed openstack/nova: Deprecate Extensible Resource Tracker https://review.openstack.org/242110 | 17:45 |
| *** ssurana has quit IRC | 17:45 | |
| dansmith | alaski: objects don't protect us from schema changes if we're running old objects.. they protect us because conductor has the newest set of objects and knows how to backport | 17:45 |
| *** achanda has joined #openstack-nova | 17:45 | |
| alaski | ahh, right | 17:46 |
| dansmith | alaski: so the case would be that we want to restart api after we've upgraded all the computes, or just not have to restart api and conductor together | 17:46 |
| alaski | so it would allow updating the db before api | 17:46 |
| PaulMurray | sdague, jaypipes mriedem_lunch changed the wording in https://review.openstack.org/#/c/242110/ - see what you think | 17:46 |
| dansmith | alaski: can upgrade the db before API right now, we just can't upgrade conductor separate from api, because conductor will start moving things | 17:46 |
| dansmith | alaski: we can (and should) also try to gate online migrations to not happen until everything that talks to the DB is upgraded, but we need service records for API for that, and we'll probably get it wrong now and then | 17:47 |
| dansmith | alaski: being able to cut the api over to conductor for an hour while you shuffle things would just be easier to get right I think | 17:47 |
| *** subscope has quit IRC | 17:47 | |
| johnthetubaguy | dansmith: +1 for pausing the data migration while the APIs are switched from old to new | 17:47 |
| dansmith | johnthetubaguy: yeah, we either need to do that automatically with service records for api and making the objects not start migrating until all conductors and APIs are upgraded, or with a manual switch | 17:47 |
| dansmith | I'd prefer not to have a manual switch | 17:48 |
| *** derekh has quit IRC | 17:48 | |
| *** mriedem_lunch is now known as mriedem | 17:48 | |
| *** pratikmallya has quit IRC | 17:48 | |
| johnthetubaguy | dansmith: yeah +1 for api service records | 17:48 |
| johnthetubaguy | (or something similar that does the same thing) | 17:48 |
| alaski | dansmith: okay, I'm starting to get this all organized in my head again. | 17:49 |
| mriedem | PaulMurray: jaypipes: +2 | 17:49 |
| *** heyongli has quit IRC | 17:49 | |
| mriedem | on https://review.openstack.org/#/c/242110/ | 17:49 |
| *** heyongli has joined #openstack-nova | 17:50 | |
| *** romainh has left #openstack-nova | 17:51 | |
| alaski | dansmith: building that as an online switch seems useful. having to restart nova-api to switch between conductor and db would just lead to using conductor from the api I think | 17:51 |
| *** e0ne has quit IRC | 17:51 | |
| dansmith | alaski: meaning leading to people always using api against conductor? | 17:52 |
| *** ssurana has joined #openstack-nova | 17:53 | |
| alaski | yeah. I think right now we would choose that, and a performance hit, if it helped live upgrades. | 17:53 |
| *** e0ne has joined #openstack-nova | 17:54 | |
| johnthetubaguy | I was thinking we just hold off on any data migrations based on the min_service version of all API and conductors, then we can still have the APIs directly access the DB? | 17:54 |
| mriedem | tpeoples: jwcroppe: did you have any objections to https://review.openstack.org/#/c/183033 before I approve it? | 17:55 |
| *** armax has joined #openstack-nova | 17:56 | |
| smurke | https://bugs.launchpad.net/nova/+bug/1387552 Just wanted to understand exactly whether i need to optimize the function by adding new method or is the request for adding new method is to have a relevant name for the behavior ? | 17:56 |
| openstack | Launchpad bug 1387552 in OpenStack Compute (nova) "Should add a new method to get volumes from block_device_info" [Wishlist,Confirmed] - Assigned to Sumant Murke (sumant-murke) | 17:56 |
| *** e0ne has quit IRC | 17:56 | |
| mriedem | smurke: not really a bug | 17:57 |
| mriedem | smurke: not really worth the time either imo | 17:57 |
| jwcroppe | mriedem: I was about 75% way thru it - it looks good to me so far. I wanted to see what kyle3h_ thought too | 17:57 |
| *** danpb has quit IRC | 17:57 | |
| smurke | yeah just new to NOVA and wanted to start with something of ease | 17:57 |
| dansmith | alaski: well, I dunno, it seems reasonable to switch to conductor only during the upgrade, no reason to leave it that way always | 17:58 |
| mriedem | jwcroppe: any idea if he's going to review it soonish? | 17:58 |
| mriedem | jwcroppe: i have my finger on the +W | 17:58 |
| dansmith | johnthetubaguy: yes, that's the mainline plan, it's just easier to mess that up I think | 17:58 |
| dansmith | johnthetubaguy: but I've had a todo to add service records for API for a while, so if you're good with that, I can do it | 17:58 |
| jwcroppe | kylek3h_: ^^^ see https://review.openstack.org/#/c/183033/ | 17:58 |
| johnthetubaguy | dansmith: very true | 17:58 |
| *** ihrachys has quit IRC | 17:58 | |
| *** rlrossit has joined #openstack-nova | 17:58 | |
| johnthetubaguy | dansmith: I am good with that idea | 17:59 |
| jwcroppe | mriedem: if you don't see any update by EOD, I think it's fine. I just +1'd too. I think it's a good change to simplify | 17:59 |
| *** marcusvrn_ has joined #openstack-nova | 17:59 | |
| mriedem | +W | 17:59 |
| mriedem | we can amend if needed | 17:59 |
| tpeoples | jwcroppe I had a ping out to Christine too, but it looked OK to me | 17:59 |
| dansmith | johnthetubaguy: okay | 17:59 |
| smurke | @mriedem : I am fairly new to NOVA and openstack so wanted to start with assigning myself with something much easy | 17:59 |
| johnthetubaguy | smurke: there are some ideas on good things to try out here: https://wiki.openstack.org/wiki/Nova/Mentoring#What_should_I_work_on.3F | 17:59 |
| *** ctrath has quit IRC | 17:59 | |
| *** heyongli has quit IRC | 18:00 | |
| *** jistr has quit IRC | 18:00 | |
| *** longsube has quit IRC | 18:00 | |
| *** haomaiwang has quit IRC | 18:01 | |
| *** heyongli has joined #openstack-nova | 18:01 | |
| *** haomaiwang has joined #openstack-nova | 18:01 | |
| alaski | dansmith: right. I'm just thinking that we don't gain much if we have to restart nova-api to switch to conductor mode, and back off. But I think I need to draw this out to visualize the upgrade workflow. | 18:01 |
| *** smatzek has joined #openstack-nova | 18:01 | |
| *** spandhe has joined #openstack-nova | 18:02 | |
| dansmith | alaski: I dunno, seems okay to me, but... | 18:02 |
| dansmith | alaski: we should do the service record thing regardless | 18:03 |
| alaski | agreed | 18:03 |
| dansmith | alaski: if it proves to be hard to get right, maybe we also allow the api switch | 18:03 |
| *** dave-mccowan has joined #openstack-nova | 18:03 | |
| *** ccarmack1 has quit IRC | 18:04 | |
| *** ctrath has joined #openstack-nova | 18:04 | |
| *** e0ne has joined #openstack-nova | 18:04 | |
| *** browne has quit IRC | 18:05 | |
| *** mnestratov|2 has joined #openstack-nova | 18:05 | |
| *** ccarmack has joined #openstack-nova | 18:07 | |
| alaski | sounds good. I may be over thinking it but I've been away from it for a bit and can't remember what the process is for a live-upgrade. | 18:07 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Use EngineFacade from oslo_db.enginefacade https://review.openstack.org/221310 | 18:07 |
| *** e0ne has quit IRC | 18:08 | |
| *** setuid has left #openstack-nova | 18:08 | |
| *** mnestratov has quit IRC | 18:09 | |
| *** unicell has joined #openstack-nova | 18:09 | |
| *** daemontool has quit IRC | 18:09 | |
| nik_ | mriedem: Matt, I updated patch. Could you look at it, please? | 18:10 |
| *** heyongli has quit IRC | 18:10 | |
| nik_ | mriedem: https://review.openstack.org/221310 | 18:10 |
| *** mnestratov has joined #openstack-nova | 18:10 | |
| *** nik_ is now known as snikitin1 | 18:10 | |
| *** heyongli has joined #openstack-nova | 18:11 | |
| alaski | mriedem: dansmith sooo, it turns out we are using xvp :( | 18:12 |
| alaski | we are also using novnc and trying to get over there completely, but still have users that rely on the old console access | 18:13 |
| dansmith | ugh | 18:13 |
| openstackgerrit | Dan Smith proposed openstack/nova: Deprecate local conductor mode https://review.openstack.org/242168 | 18:13 |
| *** mriedem has quit IRC | 18:13 | |
| *** mnestratov|2 has quit IRC | 18:14 | |
| *** salv-orlando has joined #openstack-nova | 18:15 | |
| *** mnestratov|2 has joined #openstack-nova | 18:15 | |
| openstackgerrit | dane-fichter proposed openstack/nova-specs: Nova Support of Glance Image Signing https://review.openstack.org/188874 | 18:16 |
| *** kylek3h_ has quit IRC | 18:16 | |
| *** kylek3h has joined #openstack-nova | 18:17 | |
| *** mnestratov has quit IRC | 18:19 | |
| *** achanda has quit IRC | 18:19 | |
| *** su_zhang has joined #openstack-nova | 18:20 | |
| *** heyongli has quit IRC | 18:20 | |
| *** mnestratov has joined #openstack-nova | 18:20 | |
| *** mriedem has joined #openstack-nova | 18:21 | |
| *** heyongli has joined #openstack-nova | 18:21 | |
| *** tonytan4ever has quit IRC | 18:23 | |
| *** mnestratov|2 has quit IRC | 18:24 | |
| mriedem | snikitin: looks good | 18:25 |
| *** mnestratov|2 has joined #openstack-nova | 18:25 | |
| mriedem | alaski: why do you have to go and break my heart wrt xvp? | 18:26 |
| *** smurke_ has joined #openstack-nova | 18:27 | |
| *** smurke has quit IRC | 18:27 | |
| alaski | in my heart I just wanted us to be off of it | 18:27 |
| alaski | although I really thought we were | 18:27 |
| *** haomaiwa_ has joined #openstack-nova | 18:27 | |
| *** haomaiwang has quit IRC | 18:28 | |
| dansmith | heh | 18:28 |
| *** mnestratov has quit IRC | 18:29 | |
| mriedem | so you guys plan on moving off | 18:29 |
| mriedem | would a deprecation timer help with that :) | 18:29 |
| alaski | heh, it might. although it may just lead to us holding the code internally, which really wouldn't be too bad. | 18:30 |
| *** heyongli has quit IRC | 18:30 | |
| mriedem | ugh | 18:31 |
| *** jichen has joined #openstack-nova | 18:31 | |
| alaski | we are trying to move off of it, because there's a jar file needed in our control panel to use it which is a great source of pain occasionally | 18:31 |
| dansmith | I think a timer is a good thing | 18:31 |
| *** heyongli has joined #openstack-nova | 18:31 | |
| mriedem | 15.0.0? | 18:31 |
| dansmith | if they don't move off by that timer, then they deserve the pain they get :) | 18:31 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Change logging level for 'oslo_db' https://review.openstack.org/240271 | 18:31 |
| mriedem | i'd prefer 14.0.0 | 18:32 |
| dansmith | 15 is a long way off | 18:32 |
| mriedem | i know | 18:32 |
| alaski | we are running novnc as well right now so it wouln't remove all of our consoles | 18:32 |
| mriedem | i could be dead | 18:32 |
| dansmith | I say go for 14 knowing you'll go for 15 if people cry | 18:32 |
| dansmith | like buying a car | 18:32 |
| mriedem | final offer? | 18:32 |
| *** READ10 has joined #openstack-nova | 18:32 | |
| mriedem | it's my final offer, larry | 18:32 |
| mriedem | let me see what the manager says | 18:33 |
| mriedem | and don't try to sell me on that gd undercoating | 18:33 |
| dansmith | heh | 18:33 |
| dansmith | it's been a while since we got a gd in here | 18:33 |
| mriedem | that can be changed | 18:34 |
| *** vilobhmm11 has joined #openstack-nova | 18:34 | |
| cburgess_ | mriedem I'll look at the comments from jaypipes. I'm not 100% sure I understand the need for v2 so I want to check with nic who isn't back till next week. But I've got it as an action item to follow-up on the spec. | 18:35 |
| *** salv-orlando has quit IRC | 18:35 | |
| mriedem | cburgess_: ok, i didn't understand it either, but it's not my spec :) | 18:37 |
| mriedem | jaypipes: ^ on the rbd spec for faster snapshots using rbd clones | 18:37 |
| mriedem | you made a comment about upload with glance v2 | 18:37 |
| cburgess_ | mriedem Yeah nic wrote the code (and we have been using it since grizzly) so I want to confirm with him and make sure we put the right thing in the spec. I was just baby sitting it for him while he was on vacation. | 18:37 |
| cburgess_ | I *think* I understand what jaypipes meant but I just want to be sure. | 18:39 |
| *** heyongli has quit IRC | 18:41 | |
| *** achanda has joined #openstack-nova | 18:41 | |
| *** heyongli has joined #openstack-nova | 18:42 | |
| *** dave-mccowan has quit IRC | 18:44 | |
| *** pixelbeat_ has quit IRC | 18:44 | |
| *** mwhahaha has joined #openstack-nova | 18:45 | |
| *** rdo has quit IRC | 18:45 | |
| *** rdo has joined #openstack-nova | 18:46 | |
| mwhahaha | hey nova folks, does anyone know where the nova-api-INPUT iptables rules might be managed? Is that usually an external thing or does the nova-api or something directly manipulate those rules? | 18:46 |
| *** regebro has quit IRC | 18:46 | |
| *** tonytan4ever has joined #openstack-nova | 18:47 | |
| vilobhmm11 | dansmith : ping | 18:47 |
| *** pixelbeat_ has joined #openstack-nova | 18:48 | |
| *** armax has quit IRC | 18:49 | |
| *** browne has joined #openstack-nova | 18:50 | |
| *** heyongli has quit IRC | 18:51 | |
| *** heyongli has joined #openstack-nova | 18:52 | |
| mriedem | russellb: what am i missing in this spec that we would need to change nova's attach interface API to take a new port-type parameter? https://review.openstack.org/#/c/139910/ | 18:52 |
| mriedem | the actual request params i mean | 18:52 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'instance_info' and 'instance_extra' https://review.openstack.org/225065 | 18:53 |
| *** dave-mccowan has joined #openstack-nova | 18:53 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'cell_mapping' https://review.openstack.org/226788 | 18:53 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'instance_mapping' https://review.openstack.org/228979 | 18:53 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'key_pair' and 'cell' https://review.openstack.org/239915 | 18:53 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'network' https://review.openstack.org/240253 | 18:53 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'certificate' and 'pci_device' https://review.openstack.org/240275 | 18:53 |
| russellb | mriedem: on the surface, i don't know why that would be required | 18:54 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: enginefacade: 'floating_ip' https://review.openstack.org/240513 | 18:54 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: WIP: enginefacade: 'fixed_ip' https://review.openstack.org/240811 | 18:54 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'service' https://review.openstack.org/241178 | 18:54 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'dnsdomain' and 'ec2' https://review.openstack.org/242019 | 18:54 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'host_mapping' https://review.openstack.org/242035 | 18:54 |
| mriedem | russellb: let it be known that you're my goto neutron API guy from now on | 18:54 |
| russellb | :) | 18:54 |
| russellb | mriedem: you got me in trouble yesterday | 18:54 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: enginefacade: 'compute_node' https://review.openstack.org/241189 | 18:54 |
| mriedem | i liked that little back and forth | 18:54 |
| mriedem | muwahahhaa | 18:54 |
| russellb | all good, we're working through that vlan stuff | 18:54 |
| russellb | re: sr-iov, i haven't looked at that in a long time | 18:55 |
| *** ctrath has quit IRC | 18:55 | |
| mriedem | i'm hoping ndipanov steers that when he's back | 18:56 |
| russellb | mriedem: the spec seems off | 18:56 |
| russellb | sounds like something you do at port create time | 18:56 |
| russellb | not attach | 18:56 |
| russellb | so your comments seem sane | 18:56 |
| russellb | i'll watch the spec | 18:57 |
| *** mari0jv is now known as mariojv | 18:57 | |
| mriedem | sdague: jaypipes: easy win for functional test coverage https://review.openstack.org/#/c/234370/ | 18:57 |
| *** jerrygb has quit IRC | 18:57 | |
| mriedem | there is a bp change lined up to hit the same API so i wanted to get that in first | 18:57 |
| *** electrocucaracha has joined #openstack-nova | 18:58 | |
| *** jerrygb has joined #openstack-nova | 18:58 | |
| russellb | mriedem: a lot of the nova-neutron interaction is wonky :( | 18:58 |
| russellb | which is news to nobody i think | 18:58 |
| *** ccarmack has quit IRC | 18:59 | |
| dansmith | wait, WAT? | 18:59 |
| russellb | :-p | 18:59 |
| dansmith | vilobhmm11: on a call, ping me in an hour | 18:59 |
| russellb | i should have just said "water is wet" | 18:59 |
| *** ctrath has joined #openstack-nova | 19:00 | |
| dansmith | heh | 19:00 |
| *** haomaiwa_ has quit IRC | 19:01 | |
| *** mriedem has quit IRC | 19:01 | |
| *** heyongli has quit IRC | 19:01 | |
| *** haomaiwang has joined #openstack-nova | 19:01 | |
| *** mriedem has joined #openstack-nova | 19:01 | |
| *** heyongli has joined #openstack-nova | 19:02 | |
| *** slDabbler has joined #openstack-nova | 19:02 | |
| *** jerrygb has quit IRC | 19:03 | |
| jaypipes | mriedem: done. | 19:05 |
| *** ccarmack has joined #openstack-nova | 19:06 | |
| *** mrkz has quit IRC | 19:08 | |
| *** dustins has quit IRC | 19:08 | |
| *** READ10 has quit IRC | 19:10 | |
| vilobhmm11 | dansmith : sure | 19:11 |
| *** heyongli has quit IRC | 19:11 | |
| *** heyongli has joined #openstack-nova | 19:12 | |
| *** klkumar has quit IRC | 19:13 | |
| *** rk4n has quit IRC | 19:15 | |
| *** salv-orlando has joined #openstack-nova | 19:17 | |
| *** otter768 has joined #openstack-nova | 19:18 | |
| *** dougwig_ is now known as dougwig | 19:18 | |
| *** rlrossit has quit IRC | 19:19 | |
| mriedem | interesting, default console driver is nova.console.xvp.XVPConsoleProxy | 19:21 |
| *** heyongli has quit IRC | 19:22 | |
| *** otter768 has quit IRC | 19:22 | |
| *** rushiagr is now known as rushiagr_away | 19:22 | |
| *** heyongli has joined #openstack-nova | 19:23 | |
| *** rlrossit has joined #openstack-nova | 19:23 | |
| *** pixelbeat_ has quit IRC | 19:25 | |
| *** fawadkhaliq has joined #openstack-nova | 19:27 | |
| *** rdopiera has quit IRC | 19:28 | |
| *** Marga_ has joined #openstack-nova | 19:30 | |
| *** heyongli has quit IRC | 19:32 | |
| *** stackdump has quit IRC | 19:32 | |
| *** heyongli has joined #openstack-nova | 19:33 | |
| *** ZZelle_ has quit IRC | 19:33 | |
| *** edtubill has quit IRC | 19:34 | |
| *** edtubill has joined #openstack-nova | 19:35 | |
| bauzas | dhellmann: around for discussing about https://review.openstack.org/#/c/242008/2 ? | 19:38 |
| bauzas | dhellmann: I'd like to have relnotes also for the master branch | 19:38 |
| bauzas | and not only for the stable branch | 19:39 |
| *** smurke_ has quit IRC | 19:39 | |
| *** pixelbeat has joined #openstack-nova | 19:39 | |
| *** salv-orlando has quit IRC | 19:39 | |
| *** elenhil_ has quit IRC | 19:41 | |
| mriedem | dansmith: add docimpact or upgradeimpact so we update the release notes for https://review.openstack.org/#/c/242168/ and then i'm +2 | 19:41 |
| dansmith | heh, yeah, I always forget that on my own patches | 19:42 |
| dansmith | I'll update that in a few | 19:42 |
| *** heyongli has quit IRC | 19:42 | |
| *** heyongli has joined #openstack-nova | 19:43 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: add first reno-based release note https://review.openstack.org/242008 | 19:44 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add reno for release notes management https://review.openstack.org/242007 | 19:44 |
| dansmith | I guess I can just do it in gerrit | 19:45 |
| openstackgerrit | Dan Smith proposed openstack/nova: Deprecate local conductor mode https://review.openstack.org/242168 | 19:47 |
| *** gjayavelu has joined #openstack-nova | 19:47 | |
| *** dustins has joined #openstack-nova | 19:51 | |
| *** heyongli has quit IRC | 19:52 | |
| *** jerrygb has joined #openstack-nova | 19:53 | |
| *** alex_klimov has joined #openstack-nova | 19:53 | |
| *** heyongli has joined #openstack-nova | 19:53 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 19:53 | |
| *** salv-orlando has joined #openstack-nova | 20:00 | |
| *** dave-mccowan has quit IRC | 20:00 | |
| *** haomaiwang has quit IRC | 20:01 | |
| *** nic has joined #openstack-nova | 20:01 | |
| *** nic has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-nova | 20:01 | |
| *** nic has joined #openstack-nova | 20:01 | |
| *** alejandrito has joined #openstack-nova | 20:02 | |
| *** Marga_ has quit IRC | 20:03 | |
| *** heyongli has quit IRC | 20:03 | |
| *** Marga_ has joined #openstack-nova | 20:03 | |
| *** heyongli has joined #openstack-nova | 20:04 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add service records for nova-api services https://review.openstack.org/242206 | 20:04 |
| *** brad[] has joined #openstack-nova | 20:05 | |
| *** salv-orlando has quit IRC | 20:05 | |
| brad[] | Is there a way to redefine an existing instance's assigned SSH key other than mucking around in the SQL database? | 20:06 |
| *** salv-orlando has joined #openstack-nova | 20:06 | |
| *** eglynn has quit IRC | 20:06 | |
| brad[] | I'm aware I can just edit authorized_keys but I'd prefer a more complete method if there is one | 20:06 |
| dansmith | brad[]: see topic please | 20:07 |
| brad[] | Ah strange, thought this was a user facing channel for some reason - is there one? | 20:07 |
| dansmith | brad[]: as it says in the topic, #openstack | 20:08 |
| *** rfolco has quit IRC | 20:08 | |
| *** fawadkhaliq has quit IRC | 20:10 | |
| sdague | mriedem: not that I want to go around in circles a ton on - https://review.openstack.org/#/c/240271, but verbose=True is only supposed to make INFO go on by default | 20:10 |
| sdague | mriedem: https://github.com/openstack/oslo.log/blob/7c9bececd70205a96fd6efb521b8a36e989b6b2f/oslo_log/_options.py#L23-L33 | 20:11 |
| mriedem | sdague: there are a couple others with INFO in there, should those all be changed to WARN? | 20:11 |
| sdague | mriedem: no, so verbose is true by default | 20:11 |
| sdague | now | 20:11 |
| mriedem | yeah | 20:11 |
| sdague | which would mean INFO | 20:11 |
| sdague | but debug is false by default | 20:11 |
| sdague | so something else is turning it on | 20:11 |
| sdague | and that is the thing we should probably sort out bypassing | 20:12 |
| vilobhmm11 | dansmith : hope you are free now…john gave a +2 on servicegroup spec…wanted to check if you are ok with it..https://review.openstack.org/#/c/222423/ | 20:12 |
| dansmith | sigh | 20:12 |
| dansmith | vilobhmm11: you know I get email every time that spec changes, right? | 20:12 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add service records for nova-api services https://review.openstack.org/242206 | 20:12 |
| *** heyongli has quit IRC | 20:13 | |
| *** heyongli has joined #openstack-nova | 20:14 | |
| *** dave-mccowan has joined #openstack-nova | 20:14 | |
| mriedem | sdague: hmm, yeah, want to -1 that change? | 20:15 |
| mriedem | saying the same | 20:15 |
| sdague | mriedem: sure | 20:15 |
| *** mrkz has joined #openstack-nova | 20:16 | |
| *** nic has quit IRC | 20:16 | |
| *** dave-mccowan has quit IRC | 20:18 | |
| *** dave-mccowan has joined #openstack-nova | 20:19 | |
| *** Marga_ has quit IRC | 20:20 | |
| dansmith | mriedem: still haven't seen a fail on the update-once patch | 20:21 |
| *** Marga_ has joined #openstack-nova | 20:21 | |
| *** daemontool has joined #openstack-nova | 20:22 | |
| bauzas | jaypipes: around for a quick q ? | 20:23 |
| *** heyongli has quit IRC | 20:23 | |
| bauzas | jaypipes: re: https://review.openstack.org/#/c/202676/28/nova/tests/unit/scheduler/filters/test_exact_core_filter.py,cm | 20:23 |
| bauzas | jaypipes: you missed test_exact_core_filter_passes_no_instance_type | 20:23 |
| bauzas | jaypipes: so I can surely remove the conditional for hte filter,but then I also need to remove this unittest | 20:24 |
| *** heyongli has joined #openstack-nova | 20:24 | |
| *** electrocucaracha has quit IRC | 20:24 | |
| *** pixelbeat has quit IRC | 20:24 | |
| mmmpork | sdague: for https://review.openstack.org/#/c/233076/, do you want me to just piggyback onto your change or submit it as a seperate change? | 20:24 |
| *** electrocucaracha has joined #openstack-nova | 20:24 | |
| bauzas | jaypipes: while I agree with you having no reason to provide a spec wihtout a flavor, it means my change will remove one test - which could be confusing | 20:25 |
| sdague | mmmpork: it has to go into the same change | 20:25 |
| sdague | because we have to bump the microversion when we make a change like this atomically | 20:25 |
| mmmpork | yeah i figured, but thought i'd double check :) | 20:25 |
| mmmpork | i've got the microversion change done, i'm just running tests to make sure i didn't miss anything | 20:26 |
| *** Marga_ has quit IRC | 20:26 | |
| *** ctrath has quit IRC | 20:26 | |
| sdague | mmmpork: cool, we'll probably need some additional tests on this as well to test the routes with and without project_id | 20:26 |
| sdague | but we can look at that once the patch is up for review | 20:27 |
| *** ctrath has joined #openstack-nova | 20:31 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Scheduler RPC API select_dests() https://review.openstack.org/199205 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify left filters for RequestSpec https://review.openstack.org/202678 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Prepare filters for using RequestSpec object https://review.openstack.org/197912 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify conductor to use RequestSpec object https://review.openstack.org/203078 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify metric-related filters for RequestSpec https://review.openstack.org/202676 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec in the ChanceScheduler https://review.openstack.org/199051 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify NUMA, PCI and num_instances filters for RequestSpec https://review.openstack.org/202677 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Aggregate filters for RequestSpec https://review.openstack.org/202674 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify default filters for RequestSpec https://review.openstack.org/202675 | 20:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Affinity filter for RequestSpec https://review.openstack.org/202673 | 20:32 |
| mriedem | dansmith: me neither | 20:33 |
| *** heyongli has quit IRC | 20:33 | |
| *** heyongli has joined #openstack-nova | 20:34 | |
| *** achanda has quit IRC | 20:35 | |
| mriedem | crap nova meeting in 24 minutes | 20:36 |
| * mriedem prepares mentally | 20:36 | |
| *** slDabbler has quit IRC | 20:36 | |
| dansmith | the "crap nova meeting" is on tuesdays I thought | 20:37 |
| dansmith | today is the normal one | 20:37 |
| *** slDabbler has joined #openstack-nova | 20:39 | |
| cburgess_ | "crap nova meetings"? | 20:39 |
| *** pratikma_ has quit IRC | 20:39 | |
| mriedem | crap, nova meeting | 20:39 |
| dansmith | oooohhhh. | 20:39 |
| openstackgerrit | jichenjc proposed openstack/nova: Make lock policy default to admin or owner https://review.openstack.org/242213 | 20:40 |
| *** pratikma_ has joined #openstack-nova | 20:41 | |
| edleafe | mriedem: let's eat grandma! | 20:43 |
| *** heyongli has quit IRC | 20:44 | |
| *** pixelbeat has joined #openstack-nova | 20:44 | |
| mriedem | done and done | 20:45 |
| *** heyongli has joined #openstack-nova | 20:45 | |
| *** spandhe has quit IRC | 20:45 | |
| openstackgerrit | jichenjc proposed openstack/nova: scheduler: refactory pci consume_resource https://review.openstack.org/242215 | 20:46 |
| *** rk4n has joined #openstack-nova | 20:48 | |
| *** rk4n has quit IRC | 20:48 | |
| *** jichen has quit IRC | 20:49 | |
| *** pixelbeat has quit IRC | 20:53 | |
| *** heyongli has quit IRC | 20:54 | |
| *** heyongli has joined #openstack-nova | 20:55 | |
| mikal | Nova meeting in five minutes? | 20:56 |
| *** rk4n has joined #openstack-nova | 20:57 | |
| dansmith | mikal: john told us mriedem was running it today | 20:57 |
| mikal | Oh interesting | 20:57 |
| mikal | I'm off the hook! | 20:58 |
| dansmith | was there some question of whether you were going to be around? | 20:58 |
| mikal | This is a development I support | 20:58 |
| mikal | Herm, I might have missed an email on this... | 20:58 |
| mriedem | no need for pants after all | 20:58 |
| mikal | I've been drowning the last few days | 20:58 |
| edleafe | mikal: you swam back from Tokyo? | 20:58 |
| mikal | Heh | 20:59 |
| *** haomaiwang has quit IRC | 21:01 | |
| *** pratikma_ has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-nova | 21:01 | |
| *** pratikmallya has joined #openstack-nova | 21:01 | |
| *** heyongli has quit IRC | 21:04 | |
| *** heyongli has joined #openstack-nova | 21:05 | |
| *** aysyd has quit IRC | 21:09 | |
| *** armax has joined #openstack-nova | 21:10 | |
| *** heyongli has quit IRC | 21:14 | |
| *** rawlin has joined #openstack-nova | 21:14 | |
| rawlin | Any cores out there want to take a look at https://review.openstack.org/#/c/198054/ ? It's been waiting for approval for quite some time | 21:15 |
| *** heyongli has joined #openstack-nova | 21:15 | |
| *** pratikma_ has joined #openstack-nova | 21:18 | |
| *** otter768 has joined #openstack-nova | 21:19 | |
| *** pratikmallya has quit IRC | 21:19 | |
| *** armax has quit IRC | 21:20 | |
| *** lbelivea has quit IRC | 21:21 | |
| *** smatzek has quit IRC | 21:21 | |
| *** snikitin1 has quit IRC | 21:21 | |
| *** otter768 has quit IRC | 21:23 | |
| *** heyongli has quit IRC | 21:25 | |
| *** mwhahaha has left #openstack-nova | 21:25 | |
| *** heyongli has joined #openstack-nova | 21:25 | |
| *** achanda has joined #openstack-nova | 21:28 | |
| *** stackdump has joined #openstack-nova | 21:29 | |
| mmmpork | sdague: i'm still working on fixing some tests i have broken :) | 21:30 |
| sdague | mmmpork: no prob | 21:31 |
| mmmpork | omg i swear sometimes | 21:35 |
| mmmpork | i updated the version in the test data but forgot to save | 21:35 |
| *** pratikma_ has quit IRC | 21:35 | |
| *** heyongli has quit IRC | 21:35 | |
| * mmmpork slaps forehead | 21:35 | |
| *** lbelivea has joined #openstack-nova | 21:36 | |
| *** heyongli has joined #openstack-nova | 21:36 | |
| pushkaru | Hi, Can you please let me know if is there any configuration option for snapshot timeout like we have for building instance (build_instance_timeout)? working on a bug https://bugs.launchpad.net/nova/+bug/991941 | 21:42 |
| openstack | Launchpad bug 991941 in OpenStack Compute (nova) "Restrating the compute service while taking snapshot of vm makes the snapshot stuck in saving state" [Medium,Confirmed] - Assigned to Pushkar Umaranikar (pushkar-umaranikar) | 21:42 |
| *** armax has joined #openstack-nova | 21:44 | |
| *** heyongli has quit IRC | 21:45 | |
| *** heyongli has joined #openstack-nova | 21:46 | |
| *** su_zhang has quit IRC | 21:50 | |
| *** su_zhang has joined #openstack-nova | 21:51 | |
| *** edmondsw has quit IRC | 21:52 | |
| *** thorst has quit IRC | 21:54 | |
| *** tjones has joined #openstack-nova | 21:55 | |
| *** heyongli has quit IRC | 21:55 | |
| *** heyongli has joined #openstack-nova | 21:56 | |
| dansmith | mriedem: http://logs.openstack.org/85/235485/4/check/gate-tempest-dsvm-full/ccee55b/logs/screen-q-agt.txt.gz?level=TRACE#_2015-10-27_21_23_58_856 | 21:58 |
| dansmith | mriedem: the neutron agent seems to be failing on everything, so probably zero networking on the guests | 21:58 |
| dansmith | mriedem: you also need to disable the neutron event callback if you haven't already, | 21:59 |
| dansmith | since that's not plumbed through which means nova will wait and never receive it | 21:59 |
| dansmith | that's another feature we have to add to cells, else this job will be flaky like our regular neutron jobs were before we added that | 21:59 |
| *** smatzek has joined #openstack-nova | 21:59 | |
| mriedem | dansmith: http://logs.openstack.org/85/235485/4/check/gate-tempest-dsvm-full/ccee55b/logs/screen-q-agt.txt.gz?level=TRACE#_2015-10-27_21_23_58_856 is what that log looks like in a normal run | 21:59 |
| mriedem | b/c $neutron | 21:59 |
| mriedem | but good call on the vif callback | 22:00 |
| dansmith | wtf? really? that's .. special :) | 22:00 |
| mriedem | yeah man | 22:00 |
| dansmith | then it might just be the callback | 22:00 |
| jroll | wow | 22:00 |
| dansmith | I'm not sure how that's allowed to be a thing :) | 22:00 |
| *** haomaiwang has quit IRC | 22:01 | |
| mriedem | because | 22:01 |
| mriedem | SDN | 22:01 |
| mriedem | NFV | 22:01 |
| mriedem | enterprise | 22:01 |
| *** haomaiwang has joined #openstack-nova | 22:01 | |
| * dansmith makes a farting noise | 22:01 | |
| *** unicell1 has joined #openstack-nova | 22:01 | |
| *** rk4n has quit IRC | 22:01 | |
| *** unicell has quit IRC | 22:02 | |
| mriedem | real mature | 22:02 |
| mriedem | i try to keep it classy around here | 22:02 |
| sdague | yeh, so knowing how much time was spent to get the cells job to half reasonableness, this seems like it's going to be a big rabbit hole | 22:04 |
| bauzas | mriedem: dansmith: so about the cells job | 22:04 |
| bauzas | I was thinking about doing a quick chnage | 22:04 |
| dansmith | I agree, but if we're going to, I want to require that events get implemented for cells so that we can hope to have the same level of stability as our other neutron jobs | 22:04 |
| bauzas | https://github.com/openstack/nova/blob/master/nova/objects/block_device.py#L195-L196 | 22:04 |
| sdague | I'd much rather say we'll merge low risk looking patches without a real test job | 22:04 |
| sdague | mostly just as a way to keep all the ops from carrying large out of tree patches | 22:05 |
| bauzas | and instead of using the create=create stuff, just do something like | 22:05 |
| dansmith | sdague: and when it breaks, merge patches to "fix it" based on anecdotal feedback of one person's environment? | 22:05 |
| bauzas | try: create=False except: create=True | 22:05 |
| *** heyongli has quit IRC | 22:06 | |
| sdague | dansmith: yeh, our cells config is so barely a thing anyway | 22:06 |
| *** salv-orlando has quit IRC | 22:06 | |
| *** salv-orlando has joined #openstack-nova | 22:06 | |
| sdague | because we don't ever want this supported for reals, we want to move past it | 22:06 |
| *** heyongli has joined #openstack-nova | 22:06 | |
| dansmith | sdague: I don't see the point in making an exception.. the current state of cells minimal testing is only because we wanted to prevent backslide and focus on replacing it | 22:06 |
| mriedem | bauzas: i think i'd rather just do that in cells_api.bdm_update_or_create_at_top if create=None | 22:07 |
| dansmith | sdague: right, so merging things to keep it on life support doesn't seem to be helpful to me | 22:07 |
| sdague | dansmith: that's fine too | 22:07 |
| bauzas | mriedem: right, I just saw the except Exception which eats | 22:07 |
| *** baoli_ has joined #openstack-nova | 22:07 | |
| mriedem | so we aren't merging anything today | 22:07 |
| sdague | but I do think that if the effort requires getting a reasonable neutron job running, it's just delayed cells v2 by at least another cycle | 22:07 |
| bauzas | man, I hate those kind of catchall exceptions | 22:07 |
| sdague | because of which people are going to get sucked down into this | 22:08 |
| mriedem | sdague: this is all hypothetical at this point | 22:08 |
| *** tjones has quit IRC | 22:08 | |
| mriedem | if it's like me poking at test results a few hours per week, i don't tink that kills anyone but me | 22:08 |
| sdague | mriedem: sure, that's fine | 22:08 |
| *** pratikmallya has joined #openstack-nova | 22:08 | |
| *** baoli has quit IRC | 22:08 | |
| mriedem | plus, we also wanted to get cells CI going to prevent backslide as dan said and to have a baseline for testing v2 | 22:09 |
| sdague | except we know it's going to be a high failure rate without the events interface | 22:09 |
| mriedem | i think there is value in that for cells + neutron also | 22:09 |
| sdague | so that's more than your 2 hrs a week | 22:09 |
| dansmith | yeah, we can't have the test without the events, IMHO | 22:09 |
| *** tjones has joined #openstack-nova | 22:09 | |
| mriedem | yeah maybe | 22:09 |
| *** pratikmallya has quit IRC | 22:10 | |
| mriedem | but i don't really know right now b/c i literally just threw this up quick before summit | 22:10 |
| *** unicell1 has quit IRC | 22:10 | |
| *** pratikma_ has joined #openstack-nova | 22:10 | |
| mriedem | and haven't gotten back to it | 22:10 |
| *** unicell has joined #openstack-nova | 22:10 | |
| mriedem | *before the magic kingdom i should say | 22:10 |
| sdague | ok, well I think we should put a date on the wall as a go / no go for the rest of the effort | 22:10 |
| dansmith | I think if the pressure isn't on for v2 we're never going to get there in a reasonable amount of time | 22:10 |
| dansmith | I was feeling pressured to spend serious time this cycle on v2 | 22:10 |
| dansmith | but maybe I should just do events for v1 and have a beer | 22:11 |
| bauzas | heh, I have a vision of a summit in a magic kingdom | 22:11 |
| bauzas | like the nova room happening on the BTM | 22:12 |
| mriedem | i don't think me tinkering on the ci job detracts any from the v2 priority | 22:12 |
| mriedem | chances are i don't make any progress and it dies and no one cares | 22:12 |
| *** Piet has quit IRC | 22:14 | |
| *** heyongli has quit IRC | 22:16 | |
| *** su_zhang has quit IRC | 22:16 | |
| *** su_zhang has joined #openstack-nova | 22:16 | |
| *** smatzek has quit IRC | 22:16 | |
| *** heyongli has joined #openstack-nova | 22:17 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: DNM: Making project_id optional in v2.1 urls https://review.openstack.org/233076 | 22:18 |
| mmmpork | sdague: ^^ | 22:18 |
| *** Marga_ has joined #openstack-nova | 22:20 | |
| *** daemontool has quit IRC | 22:21 | |
| *** achanda has quit IRC | 22:22 | |
| *** pradk has quit IRC | 22:25 | |
| *** heyongli has quit IRC | 22:26 | |
| *** dustins has quit IRC | 22:26 | |
| *** heyongli has joined #openstack-nova | 22:27 | |
| mriedem | alaski: how does rax deal with cells + neutron w/o the vif plugging callbacks? | 22:29 |
| alaski | mriedem: by not using libvirt, if I'm thinking of the correct thing | 22:29 |
| mriedem | oh right | 22:29 |
| mriedem | yeah only libvirt waits on those | 22:30 |
| dansmith | mriedem: there you go distracting alaski from working on cellsv2 | 22:31 |
| dansmith | mriedem: I hear that people with real SDNs under neutron don't usually have problems requiring the events because they're so fast | 22:31 |
| *** slDabbler has left #openstack-nova | 22:31 | |
| *** gyee has joined #openstack-nova | 22:32 | |
| *** krtaylor has quit IRC | 22:32 | |
| mriedem | baby laski is probably more of a distraction than i am | 22:33 |
| *** alex_klimov has quit IRC | 22:33 | |
| alaski | not today fortunately | 22:34 |
| dansmith | don't get me started on baby laski | 22:34 |
| mriedem | heh, that guy, i tell you what | 22:34 |
| mriedem | real piece of work | 22:34 |
| *** tjones has left #openstack-nova | 22:34 | |
| alaski | heh | 22:35 |
| alaski | I will say that he's been slacking on any real contributions around here | 22:35 |
| alaski | it's all take take take these days | 22:35 |
| mriedem | he gives | 22:35 |
| mriedem | in bodily fluid form | 22:36 |
| alaski | very true | 22:36 |
| mriedem | like me in college | 22:36 |
| mriedem | alright, time to squash this bdm change | 22:36 |
| *** heyongli has quit IRC | 22:36 | |
| *** heyongli has joined #openstack-nova | 22:37 | |
| alaski | mriedem: not sure if you saw my review, but the comment needs to be updated on that change | 22:37 |
| alaski | since I didn't -1 it, and some people miss those reviews | 22:38 |
| *** pratikma_ has quit IRC | 22:38 | |
| mriedem | i saw it | 22:38 |
| *** alejandrito has quit IRC | 22:41 | |
| *** su_zhang has quit IRC | 22:42 | |
| *** stackdump has quit IRC | 22:42 | |
| *** su_zhang has joined #openstack-nova | 22:42 | |
| *** mc_nair has quit IRC | 22:43 | |
| *** stackdump has joined #openstack-nova | 22:44 | |
| *** diogogmt has quit IRC | 22:44 | |
| *** krtaylor has joined #openstack-nova | 22:45 | |
| *** heyongli has quit IRC | 22:47 | |
| *** baoli_ has quit IRC | 22:47 | |
| *** heyongli has joined #openstack-nova | 22:47 | |
| *** mdrabe_ has joined #openstack-nova | 22:48 | |
| *** jhesketh has quit IRC | 22:48 | |
| *** mrkz has quit IRC | 22:50 | |
| *** dgonzalez has quit IRC | 22:51 | |
| *** gjayavel_ has joined #openstack-nova | 22:51 | |
| *** mdrabe has quit IRC | 22:51 | |
| *** pratikmallya has joined #openstack-nova | 22:52 | |
| *** jhesketh has joined #openstack-nova | 22:52 | |
| *** pixelbeat has joined #openstack-nova | 22:53 | |
| mriedem | dansmith: so this was the thing i was wondering about in the meeting | 22:54 |
| mriedem | http://logs.openstack.org/42/241742/1/check/gate-nova-python27/aefcec7/console.html#_2015-11-05_20_36_07_475 | 22:54 |
| mriedem | that test is doing the first save() but create is passed as False | 22:54 |
| mriedem | even though it should be a first time create | 22:55 |
| *** gjayavelu has quit IRC | 22:55 | |
| dansmith | you can't .save() it the first time, right? | 22:55 |
| dansmith | you have to .create() it | 22:55 |
| mriedem | with cells there is no create() | 22:56 |
| dansmith | on the bdm? | 22:56 |
| mriedem | b/c https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L201 | 22:57 |
| *** burt has quit IRC | 22:57 | |
| *** heyongli has quit IRC | 22:57 | |
| *** su_zhang has quit IRC | 22:57 | |
| *** gyee has quit IRC | 22:57 | |
| *** rlrossit has left #openstack-nova | 22:57 | |
| dansmith | where is your test? | 22:57 |
| *** su_zhang has joined #openstack-nova | 22:58 | |
| *** heyongli has joined #openstack-nova | 22:58 | |
| dansmith | oh this is an existing one I guess | 22:58 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_block_device.py#L98 | 22:58 |
| *** achanda has joined #openstack-nova | 22:59 | |
| *** jamielennox|away is now known as jamielennox | 22:59 | |
| mriedem | reserve_block_device_name in the compute manager must be creating it | 23:00 |
| *** haomaiwang has quit IRC | 23:01 | |
| mmmpork | sdague: i'm a little confused, i see there's a pci-test failure, but when i look at the logs i'm not seeing anything useful | 23:01 |
| *** haomaiwa_ has joined #openstack-nova | 23:01 | |
| mriedem | actually no, reserve_block_device_name is only called by the compute api when attaching a volume to an existing instance | 23:02 |
| dansmith | mriedem: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L4618-L4618 | 23:02 |
| mriedem | dansmith: yeah, that's only called from compute api attach_volume | 23:02 |
| mriedem | which is not what this test is doing, it's boot from volume | 23:02 |
| dansmith | surely not | 23:02 |
| mriedem | don't call me shirly | 23:03 |
| dansmith | bdm.save() doesn't work if it's not already created | 23:03 |
| dansmith | for non-cells | 23:03 |
| dansmith | so there has to be a bdm.create going on somewhere | 23:03 |
| mriedem | yeah i saw it pukes if the update doens't return anything | 23:03 |
| mriedem | there is also bdm.update_or_create | 23:03 |
| dansmith | compute_api calls that | 23:04 |
| *** mdrabe_ has quit IRC | 23:04 | |
| *** ccarmack has quit IRC | 23:04 | |
| *** dgonzalez has joined #openstack-nova | 23:04 | |
| *** mc_nair has joined #openstack-nova | 23:05 | |
| mriedem | dansmith: but cells API overrides that method | 23:05 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L201 | 23:05 |
| dansmith | right | 23:05 |
| *** dgonzalez has quit IRC | 23:06 | |
| *** heyongli has quit IRC | 23:07 | |
| mriedem | so.. | 23:07 |
| mriedem | i don't know then | 23:07 |
| *** dgonzalez has joined #openstack-nova | 23:07 | |
| *** jerrygb has quit IRC | 23:08 | |
| *** ccarmack has joined #openstack-nova | 23:08 | |
| *** rmstar has quit IRC | 23:08 | |
| *** rmstar has joined #openstack-nova | 23:08 | |
| *** heyongli has joined #openstack-nova | 23:08 | |
| dansmith | oh, so, | 23:09 |
| dansmith | in cells, the top level overrides compute_api, | 23:09 |
| dansmith | but the cell does not | 23:09 |
| dansmith | so we still call compute_api for that method normally in the child cell | 23:09 |
| dansmith | which ends up doing the create there | 23:09 |
| dansmith | and then we forward update_or_create() back to the top | 23:09 |
| *** pushkaru has quit IRC | 23:12 | |
| *** dgonzalez has quit IRC | 23:12 | |
| *** inteq has quit IRC | 23:12 | |
| *** edtubill has quit IRC | 23:12 | |
| *** dgonzalez has joined #openstack-nova | 23:12 | |
| mriedem | oy | 23:13 |
| mriedem | ok | 23:13 |
| mriedem | so, why doesn't bdm.save() just always pass create=False? | 23:13 |
| dansmith | maybe because you can have multiple levels? | 23:14 |
| *** rawlin has left #openstack-nova | 23:14 | |
| dansmith | wait, no | 23:14 |
| dansmith | so the child creates, | 23:14 |
| dansmith | the parent must create (but doesn't do it with the object) | 23:15 |
| dansmith | the child saves, so the parent must update | 23:15 |
| dansmith | so it can't always pass create=False | 23:15 |
| dansmith | it needs to pass create if the parent doesn't have it in its database | 23:15 |
| *** ccarmack has quit IRC | 23:16 | |
| dansmith | and I think it should be able to pass create=None, except that we don't have a clear identifier like a uuid for it | 23:16 |
| dansmith | actually seems like create=None should always do the right thing | 23:17 |
| dansmith | because it tries to query by instance and device_name | 23:17 |
| dansmith | and if it finds it, it updates | 23:17 |
| dansmith | and if not, it creates | 23:17 |
| *** heyongli has quit IRC | 23:17 | |
| dansmith | there must be some reason why that's not okay to always do | 23:17 |
| *** heyongli has joined #openstack-nova | 23:18 | |
| dansmith | must be something related to situations where you don't have device name, and thus you don't have an identifier or something | 23:19 |
| openstackgerrit | Andrew Bogott proposed openstack/nova: Add disk_minimum_physical_free_space_ratio option to disk filter. https://review.openstack.org/242251 | 23:19 |
| dansmith | maybe the reserve case does that without allocating a device name? | 23:19 |
| *** tjones has joined #openstack-nova | 23:19 | |
| *** tjones has left #openstack-nova | 23:19 | |
| *** burgerk has quit IRC | 23:19 | |
| *** gyee has joined #openstack-nova | 23:19 | |
| *** otter768 has joined #openstack-nova | 23:19 | |
| dansmith | ah | 23:20 |
| dansmith | we have to support changing the name | 23:20 |
| dansmith | eff | 23:20 |
| mriedem | yeah ndipanov's libvirt changes change the passed in device name | 23:21 |
| *** StevenK_ is now known as StevenK | 23:21 | |
| mriedem | so it would be set, then nulled out, then reset in the libvirt driver when it picks the name to use | 23:21 |
| *** signed8bit has quit IRC | 23:22 | |
| *** IanGovett has quit IRC | 23:23 | |
| *** otter768 has quit IRC | 23:24 | |
| dansmith | the ones we merged a while ago? | 23:24 |
| *** eglynn has joined #openstack-nova | 23:24 | |
| *** mrkz has joined #openstack-nova | 23:25 | |
| *** heyongli has quit IRC | 23:28 | |
| *** pixelbeat has quit IRC | 23:28 | |
| *** heyongli has joined #openstack-nova | 23:28 | |
| *** achanda has quit IRC | 23:31 | |
| *** sfinucan has quit IRC | 23:31 | |
| *** armax has left #openstack-nova | 23:32 | |
| mriedem | in liberty yeah | 23:32 |
| *** achanda has joined #openstack-nova | 23:32 | |
| *** ctrath has quit IRC | 23:32 | |
| mriedem | anywho, i have to get going | 23:33 |
| mriedem | i'm in a holding pattern on this squashed change | 23:33 |
| *** jerrygb has joined #openstack-nova | 23:33 | |
| mriedem | i think that test_save test is a bit wonked anyway, b/c it doesn't call create first | 23:34 |
| mriedem | it mocks out db.block_device_mapping_update and returns a thing but in reality it wouldn't return anything | 23:34 |
| mriedem | and would raise an exception | 23:34 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: cells: create bdm at top if device_name is set for first time in save() https://review.openstack.org/241742 | 23:36 |
| *** Marga__ has joined #openstack-nova | 23:37 | |
| *** mriedem has quit IRC | 23:37 | |
| *** salv-orlando has quit IRC | 23:37 | |
| *** heyongli has quit IRC | 23:38 | |
| *** ujjain- has quit IRC | 23:38 | |
| gmann | alex_xu_: cool. nice tracking. Thanks | 23:39 |
| *** heyongli has joined #openstack-nova | 23:39 | |
| *** ujjain has joined #openstack-nova | 23:39 | |
| *** ujjain has quit IRC | 23:39 | |
| *** ujjain has joined #openstack-nova | 23:39 | |
| gmann | alex_xu_: for me too plan sounds fine. | 23:39 |
| *** Marga_ has quit IRC | 23:40 | |
| *** pixelbeat has joined #openstack-nova | 23:40 | |
| *** pratikmallya has quit IRC | 23:42 | |
| *** ijuwang has quit IRC | 23:43 | |
| *** daemontool has joined #openstack-nova | 23:46 | |
| *** tongli has quit IRC | 23:48 | |
| *** heyongli has quit IRC | 23:48 | |
| *** heyongli has joined #openstack-nova | 23:49 | |
| *** mc_nair has quit IRC | 23:49 | |
| *** jerrygb_ has joined #openstack-nova | 23:49 | |
| *** angdraug has quit IRC | 23:49 | |
| *** davideagnello has quit IRC | 23:50 | |
| *** jerrygb has quit IRC | 23:50 | |
| *** tonytan4ever has quit IRC | 23:52 | |
| *** ameade has quit IRC | 23:54 | |
| *** ameade has joined #openstack-nova | 23:55 | |
| *** heyongli has quit IRC | 23:58 | |
| *** heyongli has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
