| *** smatzek has quit IRC | 00:02 | |
| *** slaweq has quit IRC | 00:10 | |
| *** slaweq has joined #openstack-nova | 00:11 | |
| *** baoli has joined #openstack-nova | 00:25 | |
| *** baoli_ has joined #openstack-nova | 00:26 | |
| *** baoli has quit IRC | 00:26 | |
| *** baoli has joined #openstack-nova | 00:29 | |
| *** baoli_ has quit IRC | 00:29 | |
| *** penick has joined #openstack-nova | 00:33 | |
| *** Tom_ has joined #openstack-nova | 00:34 | |
| *** dave-mccowan has joined #openstack-nova | 00:36 | |
| *** hieulq has joined #openstack-nova | 00:37 | |
| *** baoli has quit IRC | 00:38 | |
| *** Tom_ has quit IRC | 00:38 | |
| *** slaweq has quit IRC | 00:44 | |
| *** gouthamr has quit IRC | 00:47 | |
| *** slaweq has joined #openstack-nova | 00:48 | |
| *** baoli has joined #openstack-nova | 00:48 | |
| *** hongbin has joined #openstack-nova | 00:58 | |
| *** jichen has joined #openstack-nova | 00:58 | |
| *** litao__ has joined #openstack-nova | 01:01 | |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| *** xinliang has joined #openstack-nova | 01:05 | |
| *** zhenguo has joined #openstack-nova | 01:06 | |
| *** psachin has joined #openstack-nova | 01:19 | |
| *** slaweq has quit IRC | 01:21 | |
| *** edmondsw has joined #openstack-nova | 01:24 | |
| *** liusheng has joined #openstack-nova | 01:27 | |
| *** edmondsw has quit IRC | 01:29 | |
| *** slaweq has joined #openstack-nova | 01:33 | |
| *** trungnv has joined #openstack-nova | 01:34 | |
| *** archit has joined #openstack-nova | 01:36 | |
| *** Shunli has joined #openstack-nova | 01:40 | |
| *** dave-mccowan has quit IRC | 01:49 | |
| *** yamamoto has joined #openstack-nova | 01:51 | |
| *** markvoelker has quit IRC | 01:54 | |
| *** slaweq has quit IRC | 02:03 | |
| *** yamamoto has quit IRC | 02:05 | |
| *** slaweq has joined #openstack-nova | 02:11 | |
| *** ijw_ has joined #openstack-nova | 02:22 | |
| *** ijw has quit IRC | 02:22 | |
| *** Tom has joined #openstack-nova | 02:23 | |
| *** archit has quit IRC | 02:24 | |
| *** yamamoto has joined #openstack-nova | 02:24 | |
| *** Tom has quit IRC | 02:27 | |
| *** yamamoto has quit IRC | 02:30 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Add pagination and changes since filter support for os-instance-action API https://review.openstack.org/507762 | 02:34 |
|---|---|---|
| *** yamamoto has joined #openstack-nova | 02:41 | |
| *** slaweq has quit IRC | 02:42 | |
| *** slaweq has joined #openstack-nova | 02:50 | |
| *** liusheng has quit IRC | 02:58 | |
| *** penick has quit IRC | 03:03 | |
| *** liusheng has joined #openstack-nova | 03:05 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Add pagination and changes since filter support for os-instance-action API https://review.openstack.org/507762 | 03:06 |
| *** yamamoto has quit IRC | 03:18 | |
| *** slaweq has quit IRC | 03:21 | |
| *** slaweq has joined #openstack-nova | 03:23 | |
| *** yamamoto has joined #openstack-nova | 03:31 | |
| *** Tom has joined #openstack-nova | 03:31 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Add pagination and changes since filter support for os-instance-action API https://review.openstack.org/507762 | 03:33 |
| *** gbarros has joined #openstack-nova | 03:35 | |
| *** Tom has quit IRC | 03:35 | |
| *** mdnadeem has joined #openstack-nova | 03:37 | |
| *** udesale has joined #openstack-nova | 03:40 | |
| *** Tom has joined #openstack-nova | 03:41 | |
| *** yamamoto has quit IRC | 03:42 | |
| *** yamamoto has joined #openstack-nova | 03:47 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473 | 03:49 |
| *** yamahata has joined #openstack-nova | 03:50 | |
| *** links has joined #openstack-nova | 03:50 | |
| *** crushil has joined #openstack-nova | 03:50 | |
| *** yamamoto has quit IRC | 03:51 | |
| *** hongbin has quit IRC | 03:54 | |
| *** markvoelker has joined #openstack-nova | 03:55 | |
| *** manasm has joined #openstack-nova | 03:55 | |
| *** slaweq has quit IRC | 03:56 | |
| *** gouthamr has joined #openstack-nova | 03:58 | |
| *** smatzek has joined #openstack-nova | 04:00 | |
| *** dikonoor has joined #openstack-nova | 04:00 | |
| *** yamamoto has joined #openstack-nova | 04:00 | |
| *** Tom has quit IRC | 04:02 | |
| *** smatzek has quit IRC | 04:04 | |
| *** diga has joined #openstack-nova | 04:04 | |
| *** slaweq has joined #openstack-nova | 04:06 | |
| *** jaosorior has joined #openstack-nova | 04:07 | |
| *** felipemonteiro_ has joined #openstack-nova | 04:12 | |
| *** mingyu has quit IRC | 04:12 | |
| *** claudiub has joined #openstack-nova | 04:13 | |
| *** mingyu has joined #openstack-nova | 04:13 | |
| *** felipemonteiro__ has joined #openstack-nova | 04:13 | |
| *** mingyu has quit IRC | 04:17 | |
| *** felipemonteiro_ has quit IRC | 04:17 | |
| *** bhagyashri_s has quit IRC | 04:19 | |
| *** zsli_ has joined #openstack-nova | 04:27 | |
| *** felipemonteiro__ has quit IRC | 04:28 | |
| *** bhagyashris has joined #openstack-nova | 04:28 | |
| *** markvoelker has quit IRC | 04:28 | |
| *** felipemonteiro__ has joined #openstack-nova | 04:29 | |
| *** vks1 has joined #openstack-nova | 04:29 | |
| *** Shunli has quit IRC | 04:29 | |
| *** ratailor has joined #openstack-nova | 04:32 | |
| *** ratailor_ has joined #openstack-nova | 04:33 | |
| *** crushil has quit IRC | 04:34 | |
| *** ratailor_ has quit IRC | 04:35 | |
| *** gbarros has quit IRC | 04:36 | |
| *** edand has joined #openstack-nova | 04:36 | |
| *** slaweq has quit IRC | 04:38 | |
| *** felipemonteiro__ has quit IRC | 04:38 | |
| *** ratailor has quit IRC | 04:40 | |
| *** jwcroppe has joined #openstack-nova | 04:46 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:48 | |
| *** slaweq has joined #openstack-nova | 04:50 | |
| *** krypto has joined #openstack-nova | 04:50 | |
| *** phuongnh has quit IRC | 04:58 | |
| *** edmondsw has joined #openstack-nova | 05:00 | |
| *** phuongnh has joined #openstack-nova | 05:01 | |
| *** yamamoto has quit IRC | 05:02 | |
| *** yamamoto has joined #openstack-nova | 05:03 | |
| *** mikal has quit IRC | 05:04 | |
| *** edmondsw has quit IRC | 05:05 | |
| *** yamamoto has quit IRC | 05:08 | |
| *** sree has joined #openstack-nova | 05:14 | |
| *** slaweq has quit IRC | 05:23 | |
| *** lajoskatona has joined #openstack-nova | 05:23 | |
| *** sridharg has joined #openstack-nova | 05:24 | |
| *** markvoelker has joined #openstack-nova | 05:26 | |
| *** gouthamr has quit IRC | 05:27 | |
| *** slaweq has joined #openstack-nova | 05:28 | |
| *** yamamoto has joined #openstack-nova | 05:39 | |
| *** yamamoto has quit IRC | 05:41 | |
| *** tetsuro has joined #openstack-nova | 05:41 | |
| *** yamamoto has joined #openstack-nova | 05:45 | |
| *** Tom has joined #openstack-nova | 05:47 | |
| *** yamamoto has quit IRC | 05:48 | |
| *** spectr has joined #openstack-nova | 05:54 | |
| *** Tom has quit IRC | 05:56 | |
| *** markvoelker has quit IRC | 05:58 | |
| *** slaweq has quit IRC | 06:00 | |
| *** tetsuro has quit IRC | 06:02 | |
| *** udesale__ has joined #openstack-nova | 06:03 | |
| *** udesale has quit IRC | 06:05 | |
| *** jwcroppe has quit IRC | 06:05 | |
| *** slaweq has joined #openstack-nova | 06:06 | |
| *** vks1 has quit IRC | 06:06 | |
| *** mingyu has joined #openstack-nova | 06:08 | |
| *** jwcroppe has joined #openstack-nova | 06:08 | |
| *** Oku_OS-away is now known as Oku_OS | 06:08 | |
| *** sahid has joined #openstack-nova | 06:16 | |
| *** belmoreira has joined #openstack-nova | 06:18 | |
| *** cheneydc has joined #openstack-nova | 06:20 | |
| *** josecastroleon has joined #openstack-nova | 06:25 | |
| *** jogo has quit IRC | 06:29 | |
| *** vks1 has joined #openstack-nova | 06:31 | |
| *** slaweq has quit IRC | 06:36 | |
| *** trinaths has joined #openstack-nova | 06:38 | |
| *** rcernin has joined #openstack-nova | 06:40 | |
| *** slaweq has joined #openstack-nova | 06:41 | |
| *** igordc has quit IRC | 06:48 | |
| *** yamamoto has joined #openstack-nova | 06:49 | |
| *** igordc has joined #openstack-nova | 06:50 | |
| *** andreas_s has joined #openstack-nova | 06:52 | |
| *** krypto has quit IRC | 06:55 | |
| *** gcb has joined #openstack-nova | 06:55 | |
| *** krypto has joined #openstack-nova | 06:55 | |
| *** markvoelker has joined #openstack-nova | 06:55 | |
| *** sshwarts has joined #openstack-nova | 06:56 | |
| *** yamamoto has quit IRC | 06:58 | |
| *** baoli has quit IRC | 06:59 | |
| *** udesale has joined #openstack-nova | 06:59 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: AZ operations: check host has no instances https://review.openstack.org/509206 | 06:59 |
| *** udesale has quit IRC | 06:59 | |
| *** udesale has joined #openstack-nova | 06:59 | |
| *** udesale has quit IRC | 07:00 | |
| *** udesale__ has quit IRC | 07:01 | |
| *** udesale has joined #openstack-nova | 07:01 | |
| *** krypto has quit IRC | 07:07 | |
| *** pcaruana has joined #openstack-nova | 07:08 | |
| *** krypto has joined #openstack-nova | 07:08 | |
| *** krypto has quit IRC | 07:08 | |
| *** krypto has joined #openstack-nova | 07:08 | |
| *** jdillaman has quit IRC | 07:09 | |
| *** slaweq has quit IRC | 07:14 | |
| *** markus_z has joined #openstack-nova | 07:14 | |
| *** krypto has quit IRC | 07:15 | |
| *** krypto has joined #openstack-nova | 07:17 | |
| openstackgerrit | David Rabel proposed openstack/nova master: Choose VMware datastore by the least provisioned space https://review.openstack.org/402632 | 07:18 |
| *** edmondsw has joined #openstack-nova | 07:18 | |
| *** yingjun has joined #openstack-nova | 07:18 | |
| *** takedakn has joined #openstack-nova | 07:19 | |
| *** edmondsw has quit IRC | 07:22 | |
| *** jdillaman has joined #openstack-nova | 07:23 | |
| *** alexchadin has joined #openstack-nova | 07:25 | |
| *** ratailor has joined #openstack-nova | 07:29 | |
| *** markvoelker has quit IRC | 07:29 | |
| *** takedakn has quit IRC | 07:31 | |
| *** hoonetorg has quit IRC | 07:33 | |
| *** jpena|off is now known as jpena | 07:39 | |
| *** jwcroppe has quit IRC | 07:42 | |
| *** manasm has quit IRC | 07:44 | |
| *** tssurya has joined #openstack-nova | 07:44 | |
| *** slaweq has joined #openstack-nova | 07:45 | |
| *** hoonetorg has joined #openstack-nova | 07:46 | |
| *** seife_ has joined #openstack-nova | 07:49 | |
| *** david_1 has joined #openstack-nova | 07:49 | |
| *** mkoderer_ has joined #openstack-nova | 07:49 | |
| *** tpatzig_ has joined #openstack-nova | 07:49 | |
| *** dgonzalez_ has joined #openstack-nova | 07:49 | |
| *** sree_ has joined #openstack-nova | 07:50 | |
| *** sree has quit IRC | 07:50 | |
| *** seife_ has quit IRC | 07:50 | |
| *** dgonzalez_ has quit IRC | 07:51 | |
| *** tpatzig_ has quit IRC | 07:51 | |
| *** mkoderer_ has quit IRC | 07:51 | |
| *** david_1 has quit IRC | 07:51 | |
| *** sree_ is now known as Guest33221 | 07:51 | |
| *** jdillaman has quit IRC | 07:54 | |
| *** avolkov has joined #openstack-nova | 07:54 | |
| *** elod has joined #openstack-nova | 07:57 | |
| *** krypto has quit IRC | 07:58 | |
| *** krypto has joined #openstack-nova | 07:58 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Fix warning on {'cell_id': 1} is an invalid UUID https://review.openstack.org/505471 | 08:07 |
| *** alexchadin has quit IRC | 08:14 | |
| *** alexchadin has joined #openstack-nova | 08:14 | |
| *** gcb has quit IRC | 08:17 | |
| *** gcb has joined #openstack-nova | 08:17 | |
| *** alexchadin has quit IRC | 08:18 | |
| *** liusheng has quit IRC | 08:18 | |
| *** slaweq has quit IRC | 08:18 | |
| *** yikun has joined #openstack-nova | 08:20 | |
| *** bauwser is now known as bauzas | 08:20 | |
| *** alexchadin has joined #openstack-nova | 08:21 | |
| *** slaweq has joined #openstack-nova | 08:23 | |
| *** krypto has quit IRC | 08:24 | |
| *** lucas-afk is now known as lucasagomes | 08:24 | |
| *** krypto has joined #openstack-nova | 08:25 | |
| *** krypto has quit IRC | 08:25 | |
| *** krypto has joined #openstack-nova | 08:25 | |
| *** markvoelker has joined #openstack-nova | 08:26 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Add pagination and changes since filter support for os-instance-action API https://review.openstack.org/507762 | 08:31 |
| *** derekh has joined #openstack-nova | 08:32 | |
| *** ratailor_ has joined #openstack-nova | 08:34 | |
| *** krypto has quit IRC | 08:35 | |
| *** krypto has joined #openstack-nova | 08:35 | |
| *** ratailor has quit IRC | 08:38 | |
| *** edmondsw has joined #openstack-nova | 08:39 | |
| *** vks1 has quit IRC | 08:41 | |
| *** edmondsw has quit IRC | 08:43 | |
| *** jamesdenton has quit IRC | 08:44 | |
| *** artom has joined #openstack-nova | 08:44 | |
| *** slaweq_ has joined #openstack-nova | 08:48 | |
| *** ArchiFleKs has quit IRC | 08:50 | |
| *** ratailor_ has quit IRC | 08:50 | |
| *** gszasz has joined #openstack-nova | 08:50 | |
| *** ratailor has joined #openstack-nova | 08:50 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add 400 as error code for resource class delete https://review.openstack.org/508080 | 08:52 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add description for reousrce class creation https://review.openstack.org/508083 | 08:53 |
| *** finucannot is now known as stephenfin | 08:54 | |
| *** slaweq has quit IRC | 08:57 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | 08:58 |
| *** trinaths has left #openstack-nova | 08:58 | |
| *** markvoelker has quit IRC | 08:59 | |
| *** abhishekk has joined #openstack-nova | 09:01 | |
| abhishekk | hi are their any guide lines on "When do I need a new Service version"? | 09:02 |
| abhishekk | hi all, ^^^ | 09:05 |
| *** slaweq has joined #openstack-nova | 09:06 | |
| bauzas | abhishekk: mmmmmm | 09:06 |
| *** gcb has quit IRC | 09:06 | |
| *** kaisers1 has quit IRC | 09:06 | |
| bauzas | abhishekk: I don't exactly remember if we wrote when you need a new Service version, but that's basically when you have an upgrade impact | 09:07 |
| bauzas | thinking about rolling updates by example | 09:07 |
| bauzas | abhishekk: what is your usecase? | 09:07 |
| abhishekk | bauzas: ok, thanks for the pointer | 09:07 |
| *** vks1 has joined #openstack-nova | 09:07 | |
| abhishekk | bauzas: no use case, I was going through comments on patch https://review.openstack.org/#/c/409644/, where john g has mentioned to bump the service version | 09:09 |
| openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Add pagination and timestamp filtering support for os-migrations API https://review.openstack.org/506030 | 09:09 |
| abhishekk | so wondering in what all cases I need service version bump (I was planning to document same for internal reference) | 09:09 |
| *** ircuser-1 has joined #openstack-nova | 09:10 | |
| abhishekk | bauzas: thank you for the pointer | 09:11 |
| *** manasm has joined #openstack-nova | 09:15 | |
| *** gcb has joined #openstack-nova | 09:19 | |
| *** psachin has quit IRC | 09:20 | |
| *** yamahata has quit IRC | 09:22 | |
| *** psachin has joined #openstack-nova | 09:23 | |
| *** kaisers has joined #openstack-nova | 09:24 | |
| *** krypto has quit IRC | 09:26 | |
| *** krypto has joined #openstack-nova | 09:26 | |
| *** zsli_ has quit IRC | 09:29 | |
| *** yamamoto has joined #openstack-nova | 09:29 | |
| *** liusheng has joined #openstack-nova | 09:31 | |
| *** yingjun has quit IRC | 09:31 | |
| *** krypto has quit IRC | 09:34 | |
| *** krypto has joined #openstack-nova | 09:34 | |
| *** krypto has quit IRC | 09:34 | |
| *** krypto has joined #openstack-nova | 09:34 | |
| *** ociuhandu has joined #openstack-nova | 09:35 | |
| *** slaweq has quit IRC | 09:38 | |
| *** mingyu has quit IRC | 09:39 | |
| *** mingyu has joined #openstack-nova | 09:39 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473 | 09:41 |
| *** mingyu has quit IRC | 09:42 | |
| *** mingyu has joined #openstack-nova | 09:42 | |
| *** mingyu has quit IRC | 09:47 | |
| *** slaweq has joined #openstack-nova | 09:49 | |
| *** ociuhandu has quit IRC | 09:55 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Cleanup of existing index pages https://review.openstack.org/498819 | 09:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! doc: Add contents page https://review.openstack.org/498820 | 09:55 |
| *** markvoelker has joined #openstack-nova | 09:56 | |
| *** andreas_s has quit IRC | 09:58 | |
| *** cheneydc has quit IRC | 09:58 | |
| *** gcb has quit IRC | 09:59 | |
| *** mvk has quit IRC | 10:00 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Fix indentation of database options https://review.openstack.org/443097 | 10:01 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | 10:01 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove 'db_driver' config opt https://review.openstack.org/508487 | 10:01 |
| *** jichen has quit IRC | 10:02 | |
| *** smatzek has joined #openstack-nova | 10:02 | |
| *** Guest33221 has quit IRC | 10:03 | |
| *** sree has joined #openstack-nova | 10:03 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Convert floating IP tests to ddt https://review.openstack.org/471360 | 10:04 |
| *** sree has quit IRC | 10:04 | |
| *** psachin has quit IRC | 10:05 | |
| *** Oku_OS is now known as Oku_OS-away | 10:06 | |
| *** andreas_s has joined #openstack-nova | 10:06 | |
| *** smatzek has quit IRC | 10:06 | |
| *** sree has joined #openstack-nova | 10:08 | |
| *** edand has quit IRC | 10:09 | |
| *** mvk has joined #openstack-nova | 10:11 | |
| *** sree has quit IRC | 10:12 | |
| *** andreas_s has quit IRC | 10:14 | |
| *** mhenkel has quit IRC | 10:15 | |
| *** andreas_s has joined #openstack-nova | 10:16 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove dead code of api.fault notification sending https://review.openstack.org/505164 | 10:17 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: doc: Adds Hyper-V PCI passthrough details https://review.openstack.org/510467 | 10:19 |
| *** psachin has joined #openstack-nova | 10:20 | |
| *** andreas_s has quit IRC | 10:20 | |
| *** slaweq has quit IRC | 10:23 | |
| mdbooth | lyarwood: Would you mind casting your eye over https://review.openstack.org/#/c/510106/ ? It's your previously approved spec, but with additional api goodness as requested by mriedem in code review. | 10:24 |
| lyarwood | mdbooth: ack, I'll try to take a look today, when is the deadline btw? | 10:25 |
| *** udesale has quit IRC | 10:25 | |
| mdbooth | lyarwood: Soon :) I'm still going to have to hassle some cores. | 10:26 |
| lyarwood | mdbooth: kk, it's the 19th FYI | 10:26 |
| mdbooth | lyarwood: I had that noted down somewhere. Right now it's generally in the 'soon' bucket :) | 10:27 |
| *** edmondsw has joined #openstack-nova | 10:28 | |
| *** phuongnh has quit IRC | 10:28 | |
| *** slaweq has joined #openstack-nova | 10:29 | |
| *** markvoelker has quit IRC | 10:30 | |
| *** edmondsw has quit IRC | 10:32 | |
| gibi | bauzas, stephenfin: hi, if you have some time, there is a not soo complex patch needing a second +2 https://review.openstack.org/#/c/505164/ | 10:33 |
| *** gszasz has quit IRC | 10:33 | |
| *** andreas_s has joined #openstack-nova | 10:34 | |
| *** smatzek has joined #openstack-nova | 10:35 | |
| *** cdent has joined #openstack-nova | 10:37 | |
| *** cdent has left #openstack-nova | 10:38 | |
| *** andreas_s has quit IRC | 10:39 | |
| *** edand has joined #openstack-nova | 10:39 | |
| *** andreas_s has joined #openstack-nova | 10:40 | |
| *** mhenkel has joined #openstack-nova | 10:40 | |
| *** sdague has joined #openstack-nova | 10:41 | |
| *** andreas_s has quit IRC | 10:42 | |
| *** andreas_s has joined #openstack-nova | 10:43 | |
| *** lpetrut has joined #openstack-nova | 10:43 | |
| *** tbachman has quit IRC | 10:43 | |
| stephenfin | gibi: Done. The reno needs a little sprucing up but otherwise +" | 10:50 |
| stephenfin | * +2 | 10:50 |
| gibi | stephenfin: thanks I fill fix that up soon | 10:51 |
| gibi | stephenfin: also thanks for looking at https://review.openstack.org/#/c/483969/ | 10:51 |
| gibi | stephenfin: unfortunately that is the last in the series | 10:52 |
| gibi | stephenfin: the first is https://review.openstack.org/#/c/483324 | 10:52 |
| openstackgerrit | Merged openstack/nova master: Add 400 as error code for resource class delete https://review.openstack.org/508080 | 10:52 |
| *** andreas_s has quit IRC | 10:54 | |
| *** dtantsur|afk is now known as dtantsur | 10:55 | |
| *** gszasz has joined #openstack-nova | 10:56 | |
| *** alexchadin has quit IRC | 10:57 | |
| rgerganov | Does anyone know why Zuul gives -1 on a patch which can be clearly applied on master? https://review.openstack.org/#/c/509417/ | 10:59 |
| *** yassine has quit IRC | 10:59 | |
| rgerganov | I have +1 from Jenkins, so how to "recheck" only Zuul? | 10:59 |
| kashyap | rgerganov: #openstack-infra folks would know | 10:59 |
| rgerganov | kashyap, ok, I will try to ask there | 11:00 |
| *** nicolasbock has joined #openstack-nova | 11:01 | |
| *** vks1 has quit IRC | 11:03 | |
| *** yamamoto has quit IRC | 11:03 | |
| *** slaweq has quit IRC | 11:03 | |
| *** jaosorior has quit IRC | 11:05 | |
| *** jaosorior has joined #openstack-nova | 11:05 | |
| openstackgerrit | sahid proposed openstack/nova master: pci: update PciDevice object field 'address' to accept NULL https://review.openstack.org/508175 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: pci: add for PciDevice object new field mdev https://review.openstack.org/508176 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: pci: generalize object unit-tests for different framework https://review.openstack.org/508177 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: pci: generalize stats unit-tests for different framework https://review.openstack.org/508179 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: pci: add support for mdev device type https://review.openstack.org/508178 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: update PCI node device to report mdev devices https://review.openstack.org/508183 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: report mdev resources https://review.openstack.org/508184 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add support to start vm with using mdev (vGPU) https://review.openstack.org/508185 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: functional: rework fakelibvirt host pci devices https://review.openstack.org/508186 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: functional: resuse SRIOV funtional tests for MDEV devices https://review.openstack.org/508187 | 11:07 |
| openstackgerrit | sahid proposed openstack/nova master: WIP - functional: rewrite class to generate pci devices https://review.openstack.org/508835 | 11:07 |
| *** edmondsw has joined #openstack-nova | 11:10 | |
| *** edmondsw has quit IRC | 11:10 | |
| *** nicolasbock_ has joined #openstack-nova | 11:13 | |
| *** andreas_s has joined #openstack-nova | 11:14 | |
| *** vks1 has joined #openstack-nova | 11:15 | |
| *** manasm has quit IRC | 11:16 | |
| *** nicolasbock has quit IRC | 11:16 | |
| *** andreas_s has quit IRC | 11:18 | |
| *** andreas_s has joined #openstack-nova | 11:18 | |
| *** lucasagomes is now known as lucas-hungry | 11:27 | |
| *** markvoelker has joined #openstack-nova | 11:27 | |
| *** andreas_s has quit IRC | 11:30 | |
| sdague | zuul is in testing mode | 11:31 |
| sdague | don't worry about zuul -1 votes | 11:31 |
| *** vivsoni_ has quit IRC | 11:33 | |
| *** yamamoto has joined #openstack-nova | 11:34 | |
| *** slaweq has joined #openstack-nova | 11:35 | |
| *** diga has quit IRC | 11:37 | |
| *** andreas_s has joined #openstack-nova | 11:38 | |
| *** yamamoto has quit IRC | 11:39 | |
| *** sambetts|afk is now known as sambetts | 11:44 | |
| *** ansiwen[q] has joined #openstack-nova | 11:46 | |
| *** vivsoni has joined #openstack-nova | 11:48 | |
| *** dave-mccowan has joined #openstack-nova | 11:53 | |
| *** dave-mcc_ has joined #openstack-nova | 11:55 | |
| *** jaypipes has joined #openstack-nova | 11:56 | |
| *** josecastroleon has quit IRC | 11:58 | |
| *** dave-mccowan has quit IRC | 11:58 | |
| *** gszasz has quit IRC | 12:00 | |
| *** markvoelker has quit IRC | 12:00 | |
| *** gouthamr has joined #openstack-nova | 12:01 | |
| *** tbachman has joined #openstack-nova | 12:01 | |
| *** yassine has joined #openstack-nova | 12:05 | |
| *** edmondsw has joined #openstack-nova | 12:05 | |
| *** tbachman has quit IRC | 12:06 | |
| *** vivsoni_ has joined #openstack-nova | 12:09 | |
| *** vivsoni has quit IRC | 12:09 | |
| *** tbachman has joined #openstack-nova | 12:11 | |
| *** udesale has joined #openstack-nova | 12:13 | |
| *** vivsoni_ has quit IRC | 12:13 | |
| *** vivsoni has joined #openstack-nova | 12:13 | |
| *** gszasz has joined #openstack-nova | 12:17 | |
| *** liverpooler has joined #openstack-nova | 12:18 | |
| jaypipes | bauzas: can you explain what you meant by "and how Python is bad with module-level functions" please? | 12:18 |
| *** vks1 has quit IRC | 12:20 | |
| *** alexchadin has joined #openstack-nova | 12:21 | |
| *** markvoelker has joined #openstack-nova | 12:21 | |
| *** yamamoto has joined #openstack-nova | 12:23 | |
| *** hoonetorg has quit IRC | 12:23 | |
| *** lucas-hungry is now known as lucasagomes | 12:24 | |
| *** ratailor has quit IRC | 12:24 | |
| *** ratailor has joined #openstack-nova | 12:28 | |
| *** smatzek has quit IRC | 12:31 | |
| *** yassine has quit IRC | 12:31 | |
| *** smatzek has joined #openstack-nova | 12:31 | |
| *** jwcroppe_ has joined #openstack-nova | 12:32 | |
| *** smatzek has quit IRC | 12:33 | |
| *** smatzek has joined #openstack-nova | 12:33 | |
| *** ratailor_ has joined #openstack-nova | 12:34 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Flatten functions https://review.openstack.org/367470 | 12:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: De-duplicate _numa_get_flavor_XXX_map_list https://review.openstack.org/385074 | 12:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Rename '_numa_get_constraints_XXX' functions https://review.openstack.org/385072 | 12:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Standardize '_get_XXX_constraints' functions https://review.openstack.org/385071 | 12:38 |
| *** ratailor has quit IRC | 12:38 | |
| sean-k-mooney | jaypipes: dansmith matt does not seam to be around so ill ask ye. the lab hosting the intel nfv ci has had its firewall fixed to allow ssh again. | 12:38 |
| *** gouthamr has quit IRC | 12:38 | |
| jaypipes | sean-k-mooney: oh, nice. | 12:38 |
| sean-k-mooney | jaypipes: dansmith so the ci is now being triggered again but its broken | 12:38 |
| jaypipes | sean-k-mooney: it's not voting though is it? | 12:39 |
| *** baoli has joined #openstack-nova | 12:39 | |
| *** slaweq has quit IRC | 12:39 | |
| sean-k-mooney | jaypipes: im haveing troble internally getting this proritied to fix. should i ask them to leave it commenting or put it into silent mode | 12:39 |
| sean-k-mooney | its not gateing | 12:40 |
| jaypipes | sean-k-mooney: put it in silent mode until it can be fixed I think. | 12:40 |
| jaypipes | sean-k-mooney: and if you need some muscle to come over and bonk some heads to get it fixed, lemme know ;) | 12:40 |
| bauzas | jaypipes: hola | 12:40 |
| *** mriedem has joined #openstack-nova | 12:40 | |
| sean-k-mooney | jaypipes: ok will do. | 12:40 |
| jaypipes | bauzas: bonjour, monsieur | 12:41 |
| bauzas | jaypipes: using top-level functions is IMHO bizarre | 12:41 |
| jaypipes | bauzas: "and how Python is bad with module-level functions" <-- what do you mean by "bad" though? | 12:42 |
| jaypipes | bauzas: is there something poor-performing or something about module-level functions that I don't know about? | 12:42 |
| *** hoonetorg has joined #openstack-nova | 12:43 | |
| *** catintheroof has joined #openstack-nova | 12:43 | |
| bauzas | jaypipes: the main problem with module-level functions is they are global | 12:43 |
| jaypipes | bauzas: I'm not following you... | 12:44 |
| bauzas | jaypipes: sorry, I'm trying to explain | 12:45 |
| bauzas | jaypipes: I'm not really *against* using module functions | 12:45 |
| bauzas | jaypipes: it's just another design pattern | 12:45 |
| bauzas | jaypipes: but the problem I see with using that pattern is about | 12:46 |
| bauzas | 1/ we could have a long list of functions | 12:46 |
| bauzas | 2/ each function is set globally by python | 12:46 |
| bauzas | which means for 2/ you need to make sure to not use a global variable | 12:47 |
| jaypipes | bauzas: you need to make sure not to use global variables in staticmethods as well... | 12:47 |
| bauzas | not really | 12:48 |
| bauzas | you can use class variables | 12:48 |
| bauzas | the main problem is about the variables scopes | 12:49 |
| jaypipes | bauzas: sorry, I guess I just don't see the difference | 12:49 |
| *** yassine has joined #openstack-nova | 12:50 | |
| Tengu | hello! | 12:50 |
| Tengu | small question: I'd like to get some stats regarding OVS using collectd, I guess I'll need to configure some SNMP plugin in collectd? or are there better way? (note: not sure this is the right channel… ?) | 12:51 |
| *** slaweq has joined #openstack-nova | 12:51 | |
| *** smatzek has quit IRC | 12:52 | |
| *** artom has quit IRC | 12:52 | |
| bauzas | jaypipes: sorry that's where I have some problems explaining | 12:52 |
| jaypipes | Tengu: hi :) definitely not the right channel :) | 12:52 |
| jaypipes | Tengu: are you an OpenStack user? | 12:52 |
| bauzas | jaypipes: honestly, https://softwareengineering.stackexchange.com/questions/171296/staticmethod-vs-module-level-function is what I want to explain | 12:52 |
| bauzas | there are not so much differences in between using the static deco and just a module-scoped function | 12:52 |
| Tengu | jaypipes: yup - sorry, OVS in openstack env - should have precised. | 12:52 |
| bauzas | it's more about what kind of maintainability you want to get, and if you want to make sure to not have problems later | 12:53 |
| jaypipes | bauzas: these functions are operating on the database, not on the objects. Which is why I want to have them in separate module-level functions. | 12:53 |
| jaypipes | Tengu: I think the best thing would be to ask your question on the openstack-operators@ mailing list. | 12:54 |
| Tengu | jaypipes: hmm ok. | 12:54 |
| *** lyan has joined #openstack-nova | 12:54 | |
| jaypipes | bauzas: also... did you read the part in the question that says "I've googled this question, and it seems there's some general agreement that module-level functions are preferred over static methods because it's more pythonic."? :) | 12:55 |
| bauzas | jaypipes: well, the dude is wrong when he's saying ". Static methods have the advantage of being bound to its class" | 12:56 |
| bauzas | because static methods are *unbound* AFAIK | 12:56 |
| bauzas | I was rather pointing the best answer | 12:56 |
| bauzas | tbc, I see the CRUD ops as just being against a specific table | 12:57 |
| jaypipes | bauzas: I think we'll just have to agree to disagree on this one. It's a code style preference, nothing more. | 12:57 |
| bauzas | in that case, I just feel those should be defined as static | 12:57 |
| *** jpena is now known as jpena|lunch | 12:58 | |
| bauzas | jaypipes: agreed, I was about to say the same, it's just a pattern discussion | 12:58 |
| bauzas | nothing wrong from your side | 12:58 |
| bauzas | just stylistic | 12:58 |
| bauzas | that said, I did not -1d for that | 12:58 |
| bauzas | I rather pointed the problem of explicitelly telling which columns to return | 12:58 |
| bauzas | from a maintenability PoV, that is concerning me | 12:59 |
| *** cshen has joined #openstack-nova | 12:59 | |
| *** jwcroppe_ has quit IRC | 12:59 | |
| jaypipes | bauzas: and I responded that I prefer to have the columns be listed explicitly for the reason I outlined in my response to you. I gave you a specific example of what happens when you add a field to the ORM model and that field is a relationship() result and all of a sudden your queries that use that ORM model bloat up unnecessarily. | 13:00 |
| *** edand has quit IRC | 13:00 | |
| bauzas | jaypipes: hum, seems like I missed that point then | 13:03 |
| bauzas | if so, I apologize | 13:03 |
| bauzas | jaypipes: in https://review.openstack.org/#/c/509025/2/nova/objects/resource_provider.py@405 ? | 13:03 |
| jaypipes | yes | 13:03 |
| bauzas | the Gerrit formatting messed up most of your comment | 13:03 |
| jaypipes | bauzas: the comment I made on Oct 6 at 2:50pm | 13:03 |
| jaypipes | bauzas: ugh, yeah :( | 13:04 |
| bauzas | hah, I thought you were telling me *why* to not use ORM objects, which I don't disagree | 13:05 |
| bauzas | I didn't noticed you were giving me an explanation about why it's important to be telling which fields to return | 13:05 |
| jaypipes | bauzas: no worries. /me grabs breakfast and coffee now :) | 13:06 |
| bauzas | jaypipes: /me needs to look again at your comment then | 13:07 |
| *** smatzek has joined #openstack-nova | 13:07 | |
| *** gszasz has quit IRC | 13:08 | |
| *** ratailor_ has quit IRC | 13:10 | |
| *** ratailor_ has joined #openstack-nova | 13:10 | |
| *** tbachman has quit IRC | 13:14 | |
| *** liusheng has quit IRC | 13:15 | |
| *** liusheng has joined #openstack-nova | 13:16 | |
| *** mdnadeem has quit IRC | 13:16 | |
| *** gcb has joined #openstack-nova | 13:17 | |
| *** eharney has joined #openstack-nova | 13:19 | |
| *** tssurya has quit IRC | 13:21 | |
| *** jdwidari has joined #openstack-nova | 13:22 | |
| *** jdwidari has quit IRC | 13:23 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 13:24 |
| efried | mriedem ^ | 13:24 |
| *** slaweq has quit IRC | 13:25 | |
| efried | I worked on it all weekend | 13:25 |
| *** smatzek has quit IRC | 13:26 | |
| *** lbragstad has joined #openstack-nova | 13:26 | |
| *** smatzek has joined #openstack-nova | 13:26 | |
| *** gbarros has joined #openstack-nova | 13:26 | |
| *** pchavva has joined #openstack-nova | 13:26 | |
| *** sree has joined #openstack-nova | 13:27 | |
| mriedem | bauzas: i went through the bottom change in the "target host during cold migration" series again https://review.openstack.org/#/c/408955/ | 13:28 |
| mriedem | fresh -1 | 13:28 |
| efried | jaypipes Will you get a chance to skim https://review.openstack.org/510244 before the scheduler meeting? | 13:28 |
| bauzas | mriedem: ack, needs to review as well | 13:28 |
| *** belmoreira has quit IRC | 13:30 | |
| *** smatzek has quit IRC | 13:31 | |
| *** lajoskatona has quit IRC | 13:31 | |
| *** sree has quit IRC | 13:32 | |
| *** belmoreira has joined #openstack-nova | 13:32 | |
| *** ratailor__ has joined #openstack-nova | 13:35 | |
| dansmith | jaypipes: sdague: bauzas: This and the one right behind it are needed to unblock rally's gate, FYI: https://review.openstack.org/#/c/510203 | 13:35 |
| *** tbachman has joined #openstack-nova | 13:35 | |
| *** tssurya has joined #openstack-nova | 13:36 | |
| bauzas | dansmith: looking | 13:36 |
| sdague | dansmith: +2 | 13:36 |
| sdague | dansmith: also, ug multiple sort queries | 13:37 |
| *** slaweq has joined #openstack-nova | 13:37 | |
| dansmith | sdague: yeah, :/ | 13:37 |
| sdague | that's mostly a good way to use no indexes | 13:37 |
| *** gszasz has joined #openstack-nova | 13:37 | |
| dansmith | that | 13:37 |
| *** ratailor_ has quit IRC | 13:37 | |
| dansmith | is another ibm special feature | 13:37 |
| dansmith | but, the ship has sailed | 13:37 |
| stephenfin | gibi: Single comment on https://review.openstack.org/#/c/483324/ | 13:39 |
| *** dgonzalez has left #openstack-nova | 13:40 | |
| *** ratailor_ has joined #openstack-nova | 13:40 | |
| *** bnemec has joined #openstack-nova | 13:40 | |
| *** gouthamr has joined #openstack-nova | 13:40 | |
| gibi | stephenfin: thanks. | 13:40 |
| *** jdwidari has joined #openstack-nova | 13:40 | |
| *** gouthamr has quit IRC | 13:41 | |
| gibi | stephenfin: I think that part of the code makes sure that the bdm is marked deleted in db | 13:41 |
| stephenfin | gibi: But we still have the object reference that we can show? | 13:41 |
| gibi | stephenfin: as we are at a last step of instance deletion | 13:41 |
| gibi | stephenfin: yes we have the in memory version of that bdm | 13:42 |
| gibi | stephenfin: or somebody could query it out form the db with read_deleted=True | 13:42 |
| gibi | stephenfin: I can move the bdm.destroy() after the delete.end notification if that looks more logical to you | 13:42 |
| *** ratailor__ has quit IRC | 13:43 | |
| mnaser | i'm trying to figure out from the commits if the placement database connection is something that is no longer being used? | 13:45 |
| mnaser | it looks like it was reverted a while back.. but there's an open patch to add it again (that's pretty old): https://review.openstack.org/#/c/362766/ ? | 13:46 |
| *** tbachman has quit IRC | 13:47 | |
| *** coreywright has quit IRC | 13:47 | |
| *** psachin has quit IRC | 13:48 | |
| dansmith | mnaser: it is used by placement | 13:49 |
| dansmith | er, wait | 13:49 |
| *** gouthamr has joined #openstack-nova | 13:49 | |
| mnaser | dansmith: https://review.openstack.org/#/q/39fb302fd9c8fc57d3e4bea1c60a02ad5067163f not sure if that helps you cobble things up together but it's not helping me much | 13:50 |
| mnaser | (im reviewing the puppet change and seeing if we should deprecate that option or not) | 13:50 |
| dansmith | mnaser: so we introduced some things a little too early and backed up a few steps, | 13:50 |
| dansmith | but it's forward-looking not something that is going to go away | 13:50 |
| dansmith | mnaser: we put something back in after that revert I think, just a sec | 13:51 |
| *** sree has joined #openstack-nova | 13:52 | |
| *** krypto has quit IRC | 13:52 | |
| dansmith | or maybe we never put it back | 13:52 |
| *** krypto has joined #openstack-nova | 13:53 | |
| *** crushil has joined #openstack-nova | 13:53 | |
| dansmith | mnaser: well, anyway, "Deprecate" is probably the wrong word for the puppet patch, but it's confusing | 13:53 |
| krypto | hello all any idea why windows vm created with 4 vcpu flavor shows only 2 vcpu inside it? I can see that in vm configuration its 4 cpu but inside the guest its just 2 | 13:54 |
| mriedem | dansmith: looks like unit test failures in https://review.openstack.org/#/c/506416/ are real | 13:55 |
| dansmith | mriedem: yep I know | 13:55 |
| mnaser | dansmith: i think we'll add a warning to mention that it's currently un-used (not deprecated) till https://review.openstack.org/#/c/362766/ is merged | 13:56 |
| mriedem | we never put the placement api stuff back in | 13:56 |
| *** jpena|lunch is now known as jpena | 13:56 | |
| mriedem | after the revert | 13:56 |
| mriedem | s/api/db/ | 13:56 |
| edleafe | Scheduler subteam meeting in 3 minutes in #openstack-meeting-alt | 13:57 |
| mnaser | mriedem so should we deprecate that option in puppet-nova or just warn that it's not used until it gets added again? | 13:57 |
| dansmith | mriedem: I thought we had put an alternate connection url conf option in, that let you separate if not, but apparently we didn't | 13:57 |
| mriedem | mnaser: i'd probably just say it's not used | 13:57 |
| mriedem | otherwise we have to coordinate deprecations | 13:58 |
| mriedem | and once puppet drops it, we might add it back in :) | 13:58 |
| mnaser | alright cool, i'll leave a note mentioning to drop the warning once https://review.openstack.org/#/c/362766/ merges (if it ever does that is)? | 13:58 |
| mriedem | we haven't put serious thought into what the migration / split out is going to look like | 13:58 |
| mnaser | mriedem: gotcha. in that case we'll just leave it a warning and then keep up with nova's changes :> | 13:59 |
| mnaser | thanks mriedem and dansmith :) | 14:00 |
| *** coreywright has joined #openstack-nova | 14:01 | |
| *** links has quit IRC | 14:03 | |
| *** felipemonteiro__ has joined #openstack-nova | 14:03 | |
| *** tbachman has joined #openstack-nova | 14:03 | |
| stephenfin | gibi: Yeah, if you wouldn't mind | 14:04 |
| *** awaugama has joined #openstack-nova | 14:04 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:04 | |
| gibi | stephenfin: sure, I can do that | 14:04 |
| stephenfin | I get that it's safe there, but it seems...weird | 14:04 |
| stephenfin | gibi: Ta :) | 14:04 |
| *** smatzek has joined #openstack-nova | 14:08 | |
| *** felipemonteiro__ has quit IRC | 14:08 | |
| *** david-lyle has joined #openstack-nova | 14:08 | |
| gibi | mriedem: do you know if somebody is working on the spec discussed as a per requisite for the rx/tx queue size support? https://etherpad.openstack.org/p/nova-ptg-queens L239 | 14:09 |
| gibi | mriedem: I reached out to nic but he said no | 14:09 |
| *** ratailor__ has joined #openstack-nova | 14:10 | |
| mriedem | gibi: stephenfin: not sure that change needs to be made | 14:10 |
| mriedem | at least not in the notifications patch | 14:10 |
| *** alexchadin has quit IRC | 14:10 | |
| *** jmlowe has joined #openstack-nova | 14:10 | |
| mriedem | gibi: i thought cburgess and nic were working on that one | 14:11 |
| gibi | mriedem: the notification patch started using the destroyed bdm and therefore introduced the confusion | 14:11 |
| stephenfin | mriedem: It's not unrelated though - we weren't previously referencing bdm later in the function and now we are | 14:11 |
| *** ratailor__ has quit IRC | 14:11 | |
| mriedem | i didn't follow the outcome of the conversation though | 14:11 |
| stephenfin | Yeah, what gibi said | 14:11 |
| gibi | mriedem: I pinged cburgess but no answer yet | 14:11 |
| stephenfin | fwiw, I'm fine with it being done in a follow up, but I think it's confusing that we're "ensur(ing) block device mappings are not leaked", then "leaking" them via notifications | 14:12 |
| gibi | cburgess: ^^ ping me please | 14:12 |
| mriedem | stephenfin: well it means leaked as in not deleted from the db | 14:12 |
| mriedem | thta's probably dead code now anyway | 14:12 |
| mriedem | we delete the bdms when we delete the instance | 14:12 |
| *** ratailor_ has quit IRC | 14:13 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L1891 | 14:13 |
| *** Tom__ has joined #openstack-nova | 14:14 | |
| mriedem | so given that, how about just removing that part in a follow up | 14:14 |
| mriedem | rather than move it in this one | 14:14 |
| stephenfin | Yup, let's do that so | 14:14 |
| gibi | OK, I can push a followup | 14:15 |
| *** mingyu has joined #openstack-nova | 14:16 | |
| *** slaweq has quit IRC | 14:16 | |
| *** Tom___ has joined #openstack-nova | 14:16 | |
| mriedem | also just fyi https://github.com/openstack/nova/commit/5674e7646d106751b27d191e3334d9e6ebe9ab1b was newton, | 14:19 |
| *** esberglu has joined #openstack-nova | 14:19 | |
| mriedem | so it should be safe to make that assumption from the queens compute | 14:19 |
| mriedem | that the db api is doing what we want | 14:19 |
| gibi | mriedem: ack | 14:20 |
| *** Tom__ has quit IRC | 14:20 | |
| mriedem | stephenfin: cool, thanks for hitting those | 14:21 |
| *** gouthamr has quit IRC | 14:23 | |
| *** Tom___ has quit IRC | 14:26 | |
| *** Tom__ has joined #openstack-nova | 14:27 | |
| *** tbachman has quit IRC | 14:29 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416 | 14:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000 https://review.openstack.org/508595 | 14:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | 14:29 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | 14:29 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: PCI NUMA Policies https://review.openstack.org/361140 | 14:30 |
| *** tbachman has joined #openstack-nova | 14:30 | |
| *** spectr has quit IRC | 14:30 | |
| stephenfin | jaypipes: Fancy taking another shot at the PCI NUMA policies spec? ^ | 14:31 |
| jaypipes | stephenfin: I'll take a look in a bit, sure | 14:31 |
| *** markus_z has quit IRC | 14:32 | |
| *** belmoreira has quit IRC | 14:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix typo in test_prep_resize_errors_migration https://review.openstack.org/510591 | 14:33 |
| mriedem | gibi: ^ :) | 14:33 |
| *** jangutter has joined #openstack-nova | 14:34 | |
| *** tssurya has quit IRC | 14:36 | |
| *** belmoreira has joined #openstack-nova | 14:37 | |
| *** jdillaman has joined #openstack-nova | 14:37 | |
| *** gouthamr has joined #openstack-nova | 14:38 | |
| gibi | mriedem: looking... | 14:39 |
| dansmith | that was all gibi's fault anyway | 14:39 |
| *** slaweq has joined #openstack-nova | 14:40 | |
| *** archit has joined #openstack-nova | 14:40 | |
| gibi | but now it is fixed and everybody is happy :) | 14:40 |
| mriedem | heh | 14:40 |
| dansmith | heh | 14:40 |
| mriedem | melwitt: btw, i plan on starting draft slides for the pike update this week | 14:41 |
| *** jdillaman has quit IRC | 14:42 | |
| *** hferenc has quit IRC | 14:42 | |
| *** hferenc has joined #openstack-nova | 14:44 | |
| dansmith | I've seen several volume related issues causing gate resets this morning | 14:45 |
| *** hongbin has joined #openstack-nova | 14:45 | |
| *** gszasz has quit IRC | 14:46 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: placement: add ProviderTree.is_inventory_empty https://review.openstack.org/480957 | 14:48 |
| *** thorst has joined #openstack-nova | 14:49 | |
| *** Tom__ has quit IRC | 14:52 | |
| *** Tom___ has joined #openstack-nova | 14:53 | |
| *** Tom___ has quit IRC | 14:53 | |
| *** belmoreira has quit IRC | 14:54 | |
| *** gyee has joined #openstack-nova | 14:55 | |
| *** links has joined #openstack-nova | 14:56 | |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580 | 14:58 |
| openstackgerrit | Merged openstack/nova master: Fix instance_get_by_sort_filters() for multiple sort keys https://review.openstack.org/510203 | 14:59 |
| mriedem | dansmith: https://review.openstack.org/#/c/509899/ | 14:59 |
| mriedem | but that's not showing up in http://status.openstack.org/elastic-recheck/index.html | 15:00 |
| mriedem | b/c the page is busted for some reason | 15:00 |
| mriedem | probably new gerrit | 15:00 |
| dansmith | mriedem: ah okay I had looked | 15:00 |
| *** jwcroppe has joined #openstack-nova | 15:00 | |
| * bauzas is pissed off by his country, just learnt that tomorrow my daughters won't be in the school | 15:00 | |
| mriedem | they don't send you 10 emails for a month before that happens? | 15:00 |
| *** jdillaman has joined #openstack-nova | 15:00 | |
| mriedem | most of my email now is school spam | 15:00 |
| efried | dansmith To be clear, by "hamstringing" you mean settling on the param being an allocations object versus some other thing we would have more time to think about and design? | 15:00 |
| dansmith | efried: no, passing allocations into the virt driver is not a huge deal, | 15:01 |
| bauzas | mriedem: just one name : strike | 15:01 |
| dansmith | efried: but if libvirt gets an rp uuid and a count into spawn right now, what does it do? | 15:01 |
| mriedem | bauzas: oh i see | 15:01 |
| alex_xu | efried: jaypipes thanks for taking care the trait spec | 15:01 |
| bauzas | grrrrrr | 15:01 |
| efried | dansmith Well, it *can* ignore the new param and just use flavors. | 15:02 |
| efried | dansmith Or, if it has the time and resources, it can implement the thing that consumes the allocations. | 15:02 |
| dansmith | efried: sure, but what purpose have you served by doing that for no reason? | 15:02 |
| efried | Because of the ability to do that second thing. | 15:02 |
| bauzas | efried: dansmith: saw your conversation, yeah I'll just use the flavor for that | 15:02 |
| efried | dansmith PowerVM also has use for it, FWIW. | 15:02 |
| efried | Because we're hamstrung by the PCI manager, and the sooner we can move to generic RPs, the better. | 15:03 |
| dansmith | efried: fine, so add that thing, I'm fine with that, I'd just like not to linearize the libvirt stuff behind it | 15:03 |
| dansmith | efried: go for it | 15:03 |
| *** cfriesen has joined #openstack-nova | 15:03 | |
| efried | dansmith Oh, totally. I don't think it serializes anything. I'll see if I can hit it. Thanks. | 15:03 |
| *** tssurya has joined #openstack-nova | 15:05 | |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22device-mapper%3A%20reload%20ioctl%20on%5C%22%20AND%20%20%20message%3A%5C%22failed%3A%20Invalid%20argument%5C%22%20AND%20%20%20tags%3A%5C%22screen-c-vol.txt%5C%22&from=7d | 15:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: utils: Replace 'utf8' with oslo.utils equivalent https://review.openstack.org/466799 | 15:05 |
| *** shvepsy_ has quit IRC | 15:06 | |
| *** shvepsy has joined #openstack-nova | 15:06 | |
| sean-k-mooney | mriedem: just an fyi, i mentioned this to jay and dan earlier but i have asked for the intel nfv ci to be swapped to silent mode. | 15:07 |
| sean-k-mooney | mriedem: ssh to gerrit seams to have been restored but logs are not being uploaded and jobs are failing so i am guessing we are still being blocked form connecting to some sites | 15:08 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework man pages https://review.openstack.org/502105 | 15:09 |
| sean-k-mooney | mriedem: i have asked for the vpn and ssh logins so i can review the local copy of the log and see if i can determin what is needed to restore the ci to a working state | 15:09 |
| *** mingyu has quit IRC | 15:09 | |
| sean-k-mooney | mriedem: but ya just an fyi i did not want to have it spaming every patchset when i know its currently broken | 15:10 |
| *** mingyu has joined #openstack-nova | 15:10 | |
| mriedem | sean-k-mooney: ok | 15:13 |
| *** mingyu has quit IRC | 15:14 | |
| *** slaweq has quit IRC | 15:14 | |
| *** krypto has quit IRC | 15:14 | |
| dansmith | jaypipes: are you rebasing your nrp set? | 15:17 |
| jaypipes | dansmith: on to the no-orm-resource-providers series, yes. | 15:17 |
| *** slaweq has joined #openstack-nova | 15:18 | |
| jaypipes | dansmith: and in the process I'll be fixing up requested things on the n-r-p patches | 15:18 |
| dansmith | jaypipes: okay, does it have to go at the end? | 15:18 |
| dansmith | there's a ton of stuff to do still in the de-orm set | 15:18 |
| bauzas | jaypipes: I'm tempted to +2 eventually your no-orm bottom change :) | 15:18 |
| dansmith | and it's kinda unfortunate to have to chew all of that before we can make any progress on nrp :/ | 15:18 |
| jaypipes | dansmith: lemme see if there's a middle patch I could go from... one sec | 15:18 |
| dansmith | jaypipes: or maybe a reasonable re-ordering of the lower set so you could tack on earlier or something | 15:19 |
| *** gcb has quit IRC | 15:20 | |
| jaypipes | dansmith: https://review.openstack.org/#/c/509035 is where I could tie on n-r-p. I suppose I could re-order to have that one land earlier in the series. | 15:21 |
| dansmith | might be good if it's not too much work | 15:22 |
| *** Tom___ has joined #openstack-nova | 15:23 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fix sending legacy instance.update notification https://review.openstack.org/510603 | 15:23 |
| *** mlakat has quit IRC | 15:24 | |
| *** cfriesen has quit IRC | 15:24 | |
| sean-k-mooney | johnthetubaguy: o/ | 15:25 |
| johnthetubaguy | sean-k-mooney: hi | 15:25 |
| sean-k-mooney | johnthetubaguy: QQ im just looking at the compute manager class in the pre live migration fuction https://github.com/openstack/nova/blob/664322cae7fcb76ed27b3afd7a7b6434e6e23229/nova/compute/manager.py#L5542-L5557 | 15:26 |
| *** andreas_s has quit IRC | 15:26 | |
| sean-k-mooney | johnthetubaguy: this is where we will have to consume the new neutron port bining api above the virt drivers correct | 15:26 |
| jaypipes | dansmith: ok, I'll see what I can do | 15:26 |
| dansmith | jaypipes: regardless, I've +2d a bunch of the early part of that de-orm set | 15:27 |
| *** Tom___ has quit IRC | 15:27 | |
| dansmith | so if our gate was working at all, we could merge those soonish | 15:27 |
| sean-k-mooney | johnthetubaguy: so instead of getting current bindings with self.network_api.get_instance_nw_info(context, instance) i will need to create the destingation port bindings and the activate them later | 15:28 |
| sean-k-mooney | johnthetubaguy: just trying to make sure https://review.openstack.org/#/c/375580/5/specs/queens/approved/neutron-new-port-binding-api.rst is accurate | 15:28 |
| *** tbachman has quit IRC | 15:28 | |
| johnthetubaguy | sean-k-mooney: it calls through to here today: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L327 | 15:28 |
| johnthetubaguy | https://github.com/openstack/nova/blob/664322cae7fcb76ed27b3afd7a7b6434e6e23229/nova/network/neutronv2/api.py#L327 | 15:28 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove dead code of api.fault notification sending https://review.openstack.org/505164 | 15:30 |
| sean-k-mooney | johnthetubaguy: right so that si called on line https://github.com/openstack/nova/blob/664322cae7fcb76ed27b3afd7a7b6434e6e23229/nova/compute/manager.py#L5556 | 15:30 |
| *** andreas_s has joined #openstack-nova | 15:30 | |
| gibi | stephenfin, mriedem: could you check back to https://review.openstack.org/#/c/505164 I updated the reno based on stephenfin's excellent suggestions | 15:31 |
| johnthetubaguy | sean-k-mooney: yes, that's the one | 15:31 |
| johnthetubaguy | sean-k-mooney: but should be no changes to the compute manager, only changes in the network layer | 15:31 |
| sean-k-mooney | johnthetubaguy: well what i wanted to check is where is the xml generated for the remote node is it is the prelivemigration on line 5547 or after the networking call on 5556 | 15:33 |
| sean-k-mooney | i had the perception that i might need to change the order of those calls also im currently trace through the code to confim though | 15:34 |
| *** hongbin_ has joined #openstack-nova | 15:34 | |
| johnthetubaguy | sean-k-mooney: for the XML stuff, its all about adding the correct details into migrate_data so this method gets it correct: https://github.com/openstack/nova/blob/664322cae7fcb76ed27b3afd7a7b6434e6e23229/nova/virt/libvirt/driver.py#L6357 | 15:35 |
| *** hongbin has quit IRC | 15:36 | |
| *** ijw has joined #openstack-nova | 15:37 | |
| sean-k-mooney | ah cool thanks that is what i was looking for. i can trace out how that gets populated now that i know where to look | 15:37 |
| johnthetubaguy | sean-k-mooney: have a look at this bit: https://github.com/openstack/nova/blob/664322cae7fcb76ed27b3afd7a7b6434e6e23229/nova/virt/libvirt/driver.py#L6930 | 15:39 |
| *** penick has joined #openstack-nova | 15:39 | |
| *** andreas_s has quit IRC | 15:39 | |
| *** ijw_ has quit IRC | 15:40 | |
| *** jgriffith_ is now known as jgriffith | 15:41 | |
| *** liverpooler has quit IRC | 15:41 | |
| *** ijw_ has joined #openstack-nova | 15:41 | |
| *** priteau has joined #openstack-nova | 15:43 | |
| *** krypto has joined #openstack-nova | 15:43 | |
| *** krypto has quit IRC | 15:43 | |
| *** krypto has joined #openstack-nova | 15:43 | |
| *** ijw has quit IRC | 15:44 | |
| *** tbachman has joined #openstack-nova | 15:44 | |
| *** jpena is now known as jpena|brb | 15:45 | |
| *** mvk has quit IRC | 15:45 | |
| *** catintheroof has quit IRC | 15:48 | |
| *** andreas_s has joined #openstack-nova | 15:49 | |
| *** slaweq has quit IRC | 15:51 | |
| *** tbachman has quit IRC | 15:53 | |
| *** gszasz has joined #openstack-nova | 15:54 | |
| *** andreas_s has quit IRC | 15:57 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: use already loaded BDM in instance.<action> https://review.openstack.org/483324 | 15:58 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: use already loaded BDM in instance.<action> (2) https://review.openstack.org/483955 | 15:58 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: use already loaded BDM in instance.create https://review.openstack.org/483969 | 15:58 |
| sean-k-mooney | johnthetubaguy: right so that is part of the virt diriver prelive migration function which is called before setup_networks_on_host which is why i taught i might need to swap the order of those calls | 15:58 |
| *** slaweq has joined #openstack-nova | 15:58 | |
| *** gbarros has quit IRC | 16:00 | |
| gibi | mriedem, stephenfin: I had to rebase https://review.openstack.org/#/c/483324 to adapt some new test cases from master. You were both +2 before so could you check it again? | 16:00 |
| sean-k-mooney | johnthetubaguy: any way you confimed im looking more or less in the right part of the code so ill trace through it and confim the current workflow and what we need for the new workflow | 16:01 |
| *** tbachman has joined #openstack-nova | 16:01 | |
| *** armax has joined #openstack-nova | 16:02 | |
| johnthetubaguy | sean-k-mooney: I guess you want to send the updated port bindings to os-vif on the destination node, before handing control back to the source node? | 16:03 |
| *** tbachman_ has joined #openstack-nova | 16:04 | |
| sean-k-mooney | johnthetubaguy: am well i want to create the binding for the destionation before i create destination xml so that if the interfaces defintion change i can update that section of xml. e.g. linux bridge to ovs or iptables to contrack firewall | 16:05 |
| jaypipes | dansmith: I don't get it... I cherry-pick this patch locally: https://review.openstack.org/#/c/509030/ onto master (trying to re-order the de-orm series) and the cherry-pick applies code that is from the patch underneath it (https://review.openstack.org/#/c/509029/) :( | 16:05 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif https://review.openstack.org/422512 | 16:05 |
| johnthetubaguy | sean-k-mooney: agreed | 16:06 |
| *** tbachman has quit IRC | 16:06 | |
| *** tbachman_ is now known as tbachman | 16:06 | |
| dansmith | jaypipes: if you want to pastebin a list in the right order, I can try to do it | 16:06 |
| jaypipes | dansmith: I thought cherry-pick was supposed to only apply the diff of the specific patch and nothing else? | 16:07 |
| sean-k-mooney | johnthetubaguy: in the current flow i generate teh xml and then bind which mean today when i live migrate form a lb host to ovs it work but livirt still plugs the tap into linux brige and neutron wait for the port to show up on ovs | 16:07 |
| dansmith | jaypipes: not really, but it's only supposed to do the one you ask for, which is what you're saying yeah | 16:07 |
| *** andreas_s has joined #openstack-nova | 16:07 | |
| sean-k-mooney | johnthetubaguy: works was ment to be in qoutes because it live migrate but you networking is screwed untill you hard reboot the vm | 16:08 |
| jaypipes | dansmith: well, this is what I'm *trying* to do... http://paste.openstack.org/show/623122/ | 16:09 |
| dansmith | jaypipes: k, lemme try | 16:09 |
| jaypipes | dansmith: and what I tried was git checkout master && git pull && checkout -b de-orm-rp && git cherry-pick 0c1a5bc | 16:09 |
| jaypipes | dansmith: but that applied changes from ca983c1 for some reason | 16:10 |
| *** mingyu has joined #openstack-nova | 16:10 | |
| *** jmlowe has quit IRC | 16:10 | |
| *** sshwarts has quit IRC | 16:11 | |
| *** gbarros has joined #openstack-nova | 16:11 | |
| *** dave-mcc_ is now known as dave-mccowan | 16:13 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: de-ORM ResourceProvider.get_by_uuid() https://review.openstack.org/509025 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: Move RP._get|set_aggregates() to module scope https://review.openstack.org/509026 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: Remove RP.get_traits() method https://review.openstack.org/509027 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: move RP._set_traits() to module scope https://review.openstack.org/509028 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: remove _HasAResourceProvider mixin https://review.openstack.org/509036 | 16:13 |
| openstackgerrit | Dan Smith proposed openstack/nova master: rp: break functions out of _set_traits() https://review.openstack.org/509908 | 16:13 |
| dansmith | jaypipes: that look right? ^ | 16:13 |
| dansmith | had to re-order one thing from your list, but otherwise that ordering was fine | 16:14 |
| *** mingyu has quit IRC | 16:15 | |
| jaypipes | dansmith: yep, that'll work. thanks man, really appreciated. | 16:15 |
| dansmith | np | 16:15 |
| jaypipes | dansmith: I see you just slid the CRUD one before the Streamline one. k. | 16:15 |
| *** jpena|brb is now known as jpena | 16:15 | |
| dansmith | had to else huge esplosion | 16:15 |
| jaypipes | yes, that's the espolosion I was dealing with.. | 16:15 |
| *** udesale has quit IRC | 16:16 | |
| *** lucasagomes is now known as lucas-afk | 16:21 | |
| *** eharney has quit IRC | 16:21 | |
| *** lpetrut has quit IRC | 16:23 | |
| *** andreas_s has quit IRC | 16:25 | |
| *** mvk has joined #openstack-nova | 16:26 | |
| *** jogo has joined #openstack-nova | 16:28 | |
| *** jogo has quit IRC | 16:28 | |
| *** jogo has joined #openstack-nova | 16:28 | |
| *** cali_boxer has quit IRC | 16:28 | |
| *** fragatina has joined #openstack-nova | 16:29 | |
| *** krypto has quit IRC | 16:30 | |
| *** krypto has joined #openstack-nova | 16:31 | |
| *** krypto has quit IRC | 16:31 | |
| *** krypto has joined #openstack-nova | 16:31 | |
| *** cali_boxer has joined #openstack-nova | 16:31 | |
| *** pcaruana has quit IRC | 16:32 | |
| *** jaosorior has quit IRC | 16:33 | |
| *** jaosorior has joined #openstack-nova | 16:33 | |
| *** andreas_s has joined #openstack-nova | 16:34 | |
| *** chyka has joined #openstack-nova | 16:36 | |
| *** jaosorior has quit IRC | 16:38 | |
| *** jaosorior has joined #openstack-nova | 16:40 | |
| *** chyka has quit IRC | 16:40 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1721652 in the functional test env https://review.openstack.org/510176 | 16:46 |
| openstack | bug 1721652 in OpenStack Compute (nova) pike "Evacuate cleanup fails at _delete_allocation_for_moved_instance" [High,Confirmed] https://launchpad.net/bugs/1721652 | 16:46 |
| gibi | mriedem: I managed to push a functional regression for bug 1721652 https://review.openstack.org/510176 | 16:47 |
| openstack | bug 1721652 in OpenStack Compute (nova) pike "Evacuate cleanup fails at _delete_allocation_for_moved_instance" [High,Confirmed] https://launchpad.net/bugs/1721652 | 16:47 |
| *** liverpooler has joined #openstack-nova | 16:48 | |
| *** rcernin has quit IRC | 16:49 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable PUT dict to /allocations/{consumer_uuid} https://review.openstack.org/510625 | 16:49 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Document the changes to PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | 16:49 |
| mriedem | gibi: ok, cool | 16:50 |
| *** andreas_s has quit IRC | 16:52 | |
| *** jaosorior has quit IRC | 16:53 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase https://review.openstack.org/499539 | 16:54 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest https://review.openstack.org/503037 | 16:54 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete https://review.openstack.org/495159 | 16:54 |
| *** andreas_s has joined #openstack-nova | 16:57 | |
| *** Apoorva has joined #openstack-nova | 16:58 | |
| *** links has quit IRC | 16:58 | |
| *** Apoorva has quit IRC | 16:58 | |
| *** Apoorva has joined #openstack-nova | 16:58 | |
| *** derekh has quit IRC | 17:01 | |
| *** harlowja has joined #openstack-nova | 17:01 | |
| *** hongbin_ has quit IRC | 17:02 | |
| *** slaweq has quit IRC | 17:02 | |
| *** vvargaszte has joined #openstack-nova | 17:03 | |
| *** jdwidari has quit IRC | 17:03 | |
| *** slaweq has joined #openstack-nova | 17:04 | |
| *** hongbin has joined #openstack-nova | 17:04 | |
| *** ijw_ has quit IRC | 17:07 | |
| *** fragatina has quit IRC | 17:09 | |
| *** hongbin has quit IRC | 17:09 | |
| *** Swami has joined #openstack-nova | 17:10 | |
| *** mingyu has joined #openstack-nova | 17:12 | |
| *** hongbin has joined #openstack-nova | 17:12 | |
| *** krtaylor_ has joined #openstack-nova | 17:13 | |
| *** ijw has joined #openstack-nova | 17:13 | |
| *** krtaylor has quit IRC | 17:14 | |
| *** andreas_s has quit IRC | 17:15 | |
| *** cali_boxer has quit IRC | 17:15 | |
| *** dikonoor has quit IRC | 17:15 | |
| *** chyka has joined #openstack-nova | 17:16 | |
| *** ijw has quit IRC | 17:16 | |
| *** mingyu has quit IRC | 17:16 | |
| *** krtaylor_ has quit IRC | 17:17 | |
| *** cali_boxer has joined #openstack-nova | 17:17 | |
| *** krtaylor has joined #openstack-nova | 17:17 | |
| *** jpena is now known as jpena|away | 17:17 | |
| *** chyka has quit IRC | 17:21 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove unnecessary BDM destroy during instance delete https://review.openstack.org/510635 | 17:22 |
| *** gbarros has quit IRC | 17:22 | |
| *** cali_boxer has quit IRC | 17:22 | |
| *** cali_boxer has joined #openstack-nova | 17:23 | |
| *** slaweq has quit IRC | 17:23 | |
| *** slaweq has joined #openstack-nova | 17:24 | |
| *** lbragstad has quit IRC | 17:24 | |
| openstackgerrit | Merged openstack/nova master: Remove useless periodic task that expires quota reservations https://review.openstack.org/506822 | 17:27 |
| openstackgerrit | Merged openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | 17:30 |
| openstackgerrit | Merged openstack/nova master: Fix typo in test_prep_resize_errors_migration https://review.openstack.org/510591 | 17:30 |
| *** slaweq has quit IRC | 17:31 | |
| *** slaweq has joined #openstack-nova | 17:31 | |
| *** slaweq has quit IRC | 17:32 | |
| *** slaweq has joined #openstack-nova | 17:32 | |
| jaypipes | can we please boot the EMC ScaleIO CI system? :( | 17:38 |
| melwitt | mriedem: ack | 17:39 |
| *** sree has quit IRC | 17:39 | |
| *** sree has joined #openstack-nova | 17:40 | |
| *** slaweq has quit IRC | 17:40 | |
| *** slaweq has joined #openstack-nova | 17:41 | |
| openstackgerrit | Merged openstack/nova master: Make setenv consistent for unit, func, and api-samples https://review.openstack.org/507976 | 17:42 |
| *** lbragstad has joined #openstack-nova | 17:42 | |
| *** jwcroppe has quit IRC | 17:43 | |
| *** ijw has joined #openstack-nova | 17:43 | |
| *** sree has quit IRC | 17:44 | |
| *** sambetts is now known as sambetts|afk | 17:45 | |
| *** slaweq has quit IRC | 17:45 | |
| *** ijw has quit IRC | 17:47 | |
| *** gjayavelu has joined #openstack-nova | 17:48 | |
| mriedem | ericyoung: can the emc scaleio ci just be run on the one patch that adds the imagebackend? | 17:50 |
| *** thorst has quit IRC | 17:51 | |
| Tengu | hello! | 17:52 |
| Tengu | anyone can tell me a bit more about this log message? 2017-10-09 19:49:34.201 2665 DEBUG nova.compute.utils [req-08080314-ca07-499b-b559-a30632b6a773 f3cef0caace94cf0af3d7c21b3706583 553ad7544f5a4479b4eb346dc7a76a82 - default default] [instance: 9f503af3-ab45-43e6-be45-b0dbcff60483] Cannot attach VIF 4529caaf-3b69-4314-899c-a2090fe6114b to the node 4aa07b6d-ccf0-4f2b-8938-e20b07ff0156 due to error: | 17:52 |
| Tengu | Unable to attach VIF 4529caaf-3b69-4314-899c-a2090fe6114b, not enough free physical ports. (HTTP 400) notify_about_instance_usage /usr/lib/python2.7/site-packages/nova/compute/utils.py:328 | 17:52 |
| Tengu | I'm trying to deploy an overcloud (tripleO), and I have some issues with the Ceph nodes, the deploy script being unable to find the two nodes. I'm wondering if this log entry can explain why, and what it means. | 17:53 |
| *** gjayavelu has joined #openstack-nova | 17:54 | |
| *** hongbin has quit IRC | 17:54 | |
| *** gbarros has joined #openstack-nova | 17:55 | |
| *** andreas_s has joined #openstack-nova | 17:56 | |
| *** chyka has joined #openstack-nova | 17:57 | |
| *** jmlowe has joined #openstack-nova | 17:57 | |
| *** jwcroppe has joined #openstack-nova | 17:57 | |
| *** mriedem1 has joined #openstack-nova | 17:58 | |
| *** mriedem has quit IRC | 17:59 | |
| *** edmondsw has quit IRC | 18:00 | |
| *** chyka has quit IRC | 18:02 | |
| *** edmondsw has joined #openstack-nova | 18:02 | |
| *** thorst has joined #openstack-nova | 18:04 | |
| *** krypto has quit IRC | 18:05 | |
| *** dtantsur is now known as dtantsur|afk | 18:06 | |
| *** edmondsw has quit IRC | 18:06 | |
| *** slaweq has joined #openstack-nova | 18:07 | |
| *** sridharg has quit IRC | 18:07 | |
| *** Apoorva_ has joined #openstack-nova | 18:08 | |
| *** andreas_s has quit IRC | 18:09 | |
| *** smatzek has quit IRC | 18:10 | |
| *** pcaruana has joined #openstack-nova | 18:11 | |
| *** Apoorva has quit IRC | 18:11 | |
| *** smatzek has joined #openstack-nova | 18:11 | |
| mriedem1 | efried: happy monday | 18:11 |
| *** mriedem1 is now known as mriedem | 18:11 | |
| *** slaweq has quit IRC | 18:12 | |
| *** edmondsw has joined #openstack-nova | 18:12 | |
| *** andreas_s has joined #openstack-nova | 18:14 | |
| *** smatzek has quit IRC | 18:16 | |
| *** avolkov has quit IRC | 18:16 | |
| *** smatzek has joined #openstack-nova | 18:17 | |
| *** edmondsw has quit IRC | 18:17 | |
| *** sambetts|afk has quit IRC | 18:17 | |
| mriedem | sdague: you were +2 on this before, assume you still like it https://review.openstack.org/#/c/505317/6 | 18:17 |
| sdague | mriedem / efried - why did that shuffle get added? | 18:20 |
| mriedem | efried: looks like https://review.openstack.org/#/c/490057/ needs to be rebased and there are quite a few test failures | 18:20 |
| *** sambetts_ has joined #openstack-nova | 18:20 | |
| mriedem | sdague: it's not added | 18:20 |
| sdague | oh, never mind, it was there before the patch is just highlighting weirdly | 18:20 |
| mriedem | yeah | 18:20 |
| *** smatzek has quit IRC | 18:21 | |
| *** smatzek has joined #openstack-nova | 18:21 | |
| *** slaweq has joined #openstack-nova | 18:24 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 18:25 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 18:25 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 18:25 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 18:25 |
| *** smatzek has quit IRC | 18:26 | |
| *** eharney has joined #openstack-nova | 18:27 | |
| *** smatzek has joined #openstack-nova | 18:27 | |
| *** slaweq has quit IRC | 18:28 | |
| *** andreas_s has quit IRC | 18:31 | |
| *** smatzek has quit IRC | 18:31 | |
| *** gjayavelu has quit IRC | 18:31 | |
| jaypipes | dansmith: I'll definitely fix up that unit test miss thing. just in the middle of rebasing the n-r-p series now for the ID vs. uuid stuff. | 18:31 |
| *** gjayavelu has joined #openstack-nova | 18:32 | |
| dansmith | jaypipes: ack | 18:32 |
| *** thorst_ has joined #openstack-nova | 18:36 | |
| *** thorst has quit IRC | 18:37 | |
| *** ejat has quit IRC | 18:38 | |
| *** smatzek has joined #openstack-nova | 18:39 | |
| *** tbachman has quit IRC | 18:39 | |
| *** ejat has joined #openstack-nova | 18:41 | |
| melwitt | sdague, mriedem: my devstack hack patch for the ceph job https://review.openstack.org/#/c/509663/ | 18:46 |
| sdague | melwitt: we typically don't want any of the plugin conditional code to leak into there | 18:47 |
| sdague | melwitt: is there an appropriate plugin phase for the ceph plugin to do it? | 18:48 |
| mriedem | melwitt: so this is progress right? http://logs.openstack.org/63/509663/4/experimental/gate-tempest-dsvm-py35-full-devstack-plugin-ceph-ubuntu-xenial-nv/2b03b3a/logs/devstacklog.txt.gz#_2017-10-06_05_26_37_113 | 18:48 |
| mriedem | sdague: apparently not | 18:48 |
| mriedem | keystone is special as it's loaded early in stack.sh | 18:48 |
| sdague | where is the ceph plugin source? | 18:48 |
| melwitt | sdague: there used to be post-config. but now keystone is started before post-config and keystone start is the thing that's failing in the ceph job. so I need a new phase between pip install and keystone start | 18:48 |
| mriedem | sdague: https://github.com/openstack/devstack-plugin-ceph | 18:48 |
| melwitt | mriedem: that's the py35 job which won't work until the ceph plugin is installing python3-rados and python3-rbd. I have a patch up for that too | 18:50 |
| melwitt | mriedem: but you want to take a look at the py27 job, it passes with that hack | 18:50 |
| mriedem | oh right | 18:50 |
| melwitt | I got confused by the same thing last week | 18:50 |
| mriedem | i know i reported a bug for ceph libradoes in glance-store in lp somewhere | 18:50 |
| mriedem | for py3 | 18:50 |
| mriedem | ah here it is passing http://logs.openstack.org/63/509663/4/experimental/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/af8e25f/ | 18:51 |
| melwitt | ah yup, it's that. glance-store weirdly sort of silently ignores import errors even in the rbd file | 18:51 |
| sdague | melwitt: why can't it run at stack_install ? | 18:51 |
| mriedem | melwitt: yup | 18:51 |
| * dansmith chuckles about the libradoes quailism | 18:51 | |
| melwitt | sdague: is stack_install after pip install and before keystone start? if it is, then it can run there | 18:51 |
| sdague | https://github.com/openstack-dev/devstack/blob/a29329b55983d69c3966e1699f985be8cdf6e59d/stack.sh#L914 | 18:52 |
| mriedem | ah yup | 18:52 |
| sdague | it's definitely before keystone start | 18:52 |
| mriedem | so need something like this https://github.com/openstack/devstack-plugin-ceph/blob/master/devstack/plugin.sh#L6 | 18:53 |
| mriedem | elif [[ "$1" == "stack" && "$2" == "install" ]]; then | 18:53 |
| mriedem | good call | 18:53 |
| melwitt | omg, thanks sdague | 18:53 |
| melwitt | sweet | 18:53 |
| sdague | melwitt: no prob | 18:53 |
| melwitt | I need better devstack-fu | 18:54 |
| sdague | no worries, it's a lot of code | 18:54 |
| sdague | if no appropriate phase existed, I would have created one for it, because the whole plugin abstraction completely breaks if you have plugin names in the main code :) | 18:55 |
| sdague | we've added ones in the past when we realized there was a missing place folks needed to do something | 18:55 |
| sdague | I think I've got +2 on the ceph plugin repo too, so let me know when that's posted and I can help get it in | 18:56 |
| melwitt | awesomesauce | 18:57 |
| *** lpetrut has joined #openstack-nova | 18:58 | |
| *** lpetrut has quit IRC | 19:00 | |
| *** lpetrut has joined #openstack-nova | 19:01 | |
| openstackgerrit | sean mooney proposed openstack/nova master: WIP reorder destination host setup https://review.openstack.org/510658 | 19:04 |
| *** ijw has joined #openstack-nova | 19:06 | |
| *** chyka has joined #openstack-nova | 19:07 | |
| *** slaweq has joined #openstack-nova | 19:08 | |
| *** eharney has quit IRC | 19:08 | |
| *** edmondsw has joined #openstack-nova | 19:09 | |
| melwitt | sdague: posted here https://review.openstack.org/#/c/510660 will need to await test results to be 100% sure it's gonna owrk | 19:11 |
| melwitt | *work | 19:11 |
| sdague | melwitt: cool, I starred it, will come back around when the tests are in | 19:11 |
| melwitt | thx | 19:11 |
| *** chyka has quit IRC | 19:12 | |
| mriedem | so uh https://review.openstack.org/#/c/394808/ | 19:17 |
| mriedem | as far as i can tell, | 19:17 |
| mriedem | that was added in ocata but the libvirt live migration flag it's setting is only available starting in libvirt 2.3.0 | 19:17 |
| mriedem | http://libvirt.org/git/?p=libvirt.git;a=blob;f=docs/news-2016.html.in#l437 | 19:17 |
| mriedem | our minimum is 1.2.9 | 19:18 |
| sean-k-mooney | we can override this from the config no? | 19:18 |
| mriedem | no | 19:19 |
| sdague | mriedem: probably just put a conditional around that? | 19:20 |
| sean-k-mooney | oh so the live_migration_flag option in the libvirt section of the nova-cpu.conf is jsut the default set ? | 19:20 |
| mriedem | oh this might just be some other libvirt backend | 19:20 |
| mriedem | yeah nvm, super old http://libvirt.org/git/?p=libvirt.git;a=commit;h=632be336898c9efef258e1f2d1c3df199a98a4aa | 19:21 |
| *** esberglu has quit IRC | 19:23 | |
| sean-k-mooney | so live_migration_flag was apparently removed in kilo ... i should probably update our internal wiki to not tell you to use those when livemigrating | 19:26 |
| *** hemna_ has joined #openstack-nova | 19:36 | |
| *** esberglu has joined #openstack-nova | 19:37 | |
| *** thorst_ has quit IRC | 19:38 | |
| *** thorst has joined #openstack-nova | 19:39 | |
| *** esberglu has quit IRC | 19:39 | |
| *** esberglu has joined #openstack-nova | 19:39 | |
| *** gszasz has quit IRC | 19:40 | |
| *** esberglu has quit IRC | 19:40 | |
| *** esberglu has joined #openstack-nova | 19:40 | |
| *** esberglu has quit IRC | 19:40 | |
| *** esberglu has joined #openstack-nova | 19:40 | |
| *** mikal has joined #openstack-nova | 19:44 | |
| *** thorst has quit IRC | 19:44 | |
| *** thorst has joined #openstack-nova | 19:45 | |
| *** eharney has joined #openstack-nova | 19:45 | |
| *** awaugama has quit IRC | 19:46 | |
| *** Apoorva_ has quit IRC | 19:47 | |
| *** Apoorva has joined #openstack-nova | 19:57 | |
| *** tbachman has joined #openstack-nova | 19:57 | |
| *** vvargaszte has quit IRC | 19:58 | |
| *** vvargaszte has joined #openstack-nova | 20:00 | |
| *** jpena|away is now known as jpena|off | 20:01 | |
| *** chyka has joined #openstack-nova | 20:01 | |
| *** crushil has quit IRC | 20:03 | |
| *** ijw has quit IRC | 20:03 | |
| *** crushil has joined #openstack-nova | 20:03 | |
| *** chyka has quit IRC | 20:06 | |
| *** ijw has joined #openstack-nova | 20:07 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: remove old code in post_live_migration_at_destination https://review.openstack.org/510680 | 20:11 |
| *** smatzek has quit IRC | 20:12 | |
| openstackgerrit | Merged openstack/nova master: Remove dead code of api.fault notification sending https://review.openstack.org/505164 | 20:12 |
| mriedem | sean-k-mooney: having to re-learn live migration to review this port binding spec again of course | 20:12 |
| *** smatzek has joined #openstack-nova | 20:13 | |
| *** liverpooler has quit IRC | 20:13 | |
| *** fragatina has joined #openstack-nova | 20:17 | |
| *** smatzek has quit IRC | 20:17 | |
| *** pcaruana has quit IRC | 20:18 | |
| *** boris_42_ has joined #openstack-nova | 20:18 | |
| *** vvargaszte has quit IRC | 20:22 | |
| *** jwcroppe has quit IRC | 20:23 | |
| *** jwcroppe has joined #openstack-nova | 20:24 | |
| *** armax has quit IRC | 20:26 | |
| *** jwcroppe has quit IRC | 20:29 | |
| *** sahid has quit IRC | 20:32 | |
| *** priteau_ has joined #openstack-nova | 20:37 | |
| *** priteau has quit IRC | 20:37 | |
| claudiub | mriedem: hellou. just an fyi, and I'm going to send a mail to the MLs and a patch for it, we're deprecating the support for Windows / Hyper-V Server 2012; the acceptable versions will be 2012 r2 and 2016. | 20:45 |
| mriedem | man we do not do a good job of documenting the valid values for filtering instances by status | 20:45 |
| mriedem | claudiub: ok | 20:45 |
| *** felipemonteiro has joined #openstack-nova | 20:45 | |
| *** thorst has quit IRC | 20:46 | |
| *** felipemonteiro_ has quit IRC | 20:46 | |
| *** crushil has quit IRC | 20:47 | |
| *** mingyu has joined #openstack-nova | 20:49 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:52 | |
| *** smatzek has joined #openstack-nova | 20:53 | |
| *** mingyu has quit IRC | 20:53 | |
| *** ijw has quit IRC | 20:54 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967 | 20:55 |
| openstackgerrit | melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968 | 20:55 |
| openstackgerrit | melanie witt proposed openstack/nova master: WIP Improve the CellDatabases test fixture and usage https://review.openstack.org/508432 | 20:55 |
| openstackgerrit | melanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells https://review.openstack.org/510691 | 20:55 |
| *** ijw has joined #openstack-nova | 20:55 | |
| *** felipemonteiro has quit IRC | 20:55 | |
| *** erlon has joined #openstack-nova | 20:57 | |
| efried | mriedem sdague Sorry, I'm back now. Little lunchtime injury to deal with. Anything I need to know right now or will me email/dashboard tell all? | 20:57 |
| *** smatzek has quit IRC | 20:58 | |
| melwitt | mriedem, dansmith: found a couple of cells bugs while working on CellDatabases. bottom two in the stack ^ | 20:58 |
| openstackgerrit | Merged openstack/nova master: Always put 'uuid' into sort_keys for stable instance lists https://review.openstack.org/510140 | 20:59 |
| mriedem | efried: the latter | 21:00 |
| *** crushil has joined #openstack-nova | 21:00 | |
| efried | mriedem Cool, thanks. | 21:00 |
| *** priteau_ has quit IRC | 21:01 | |
| efried | mriedem And to answer your question, the distance between those two patches was significant and we didn't want to collapse them together, though we could have. But you're absolutely correct that the second one just undoes what the first one adds wrt those auth options. | 21:01 |
| *** penick_ has joined #openstack-nova | 21:04 | |
| *** penick has quit IRC | 21:06 | |
| mriedem | melwitt: don't tell me you can't get bandwidth usage with multiple cells?! | 21:07 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix server status values in GET /servers docs https://review.openstack.org/510696 | 21:07 |
| dansmith | melwitt: not possible. | 21:09 |
| melwitt | mriedem: lol. that one is not about a bug | 21:09 |
| *** yamahata has joined #openstack-nova | 21:10 | |
| melwitt | dansmith: true, bugs in cells is unpossible | 21:10 |
| dansmith | melwitt: I feel like we should be able to test that bottom one somehow | 21:10 |
| dansmith | er, | 21:10 |
| dansmith | I mean the scatter gather one | 21:11 |
| *** lpetrut has quit IRC | 21:11 | |
| melwitt | dansmith: yeah, I'm pondering on it | 21:12 |
| melwitt | at first I was thinking there's not a good way to test it but there's probably a way | 21:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update "SHUTOFF" description in API guide https://review.openstack.org/510697 | 21:14 |
| melwitt | just ... want ... to ... save ... comment ... in .... gerrit ... | 21:15 |
| melwitt | nnnnggghh | 21:15 |
| mriedem | it is pretty slow right now | 21:15 |
| melwitt | yeah, some ppl are talking about it in #infra | 21:15 |
| melwitt | probably gonna have to restart it again | 21:16 |
| dansmith | yeah, super slow | 21:16 |
| dansmith | melwitt: a couple questions in the other patch | 21:16 |
| * melwitt looks | 21:16 | |
| *** archit has quit IRC | 21:17 | |
| openstackgerrit | Merged openstack/nova master: use already loaded BDM in instance.<action> https://review.openstack.org/483324 | 21:18 |
| *** ijw has quit IRC | 21:19 | |
| *** ijw has joined #openstack-nova | 21:19 | |
| *** Apoorva has quit IRC | 21:21 | |
| *** awaugama has joined #openstack-nova | 21:21 | |
| *** armax has joined #openstack-nova | 21:21 | |
| melwitt | dansmith: in the test I was trying to assert that the context for the notify was targeted to the same cell the instance was created in. though I'm not sure how we know it's cell1 (there's already an assertion in that test that knows instance is created in cell1) | 21:21 |
| *** Apoorva has joined #openstack-nova | 21:21 | |
| *** slaweq has quit IRC | 21:22 | |
| dansmith | I really wish we had something in the context that identified the cell, because several tests go through obscure hoops to try to verify the cell | 21:22 |
| dansmith | if you're going to leave it that way I think you need a bunch more comments explaining all the assumptions, how the fixture works, etc | 21:24 |
| openstackgerrit | Merged openstack/nova master: use already loaded BDM in instance.<action> (2) https://review.openstack.org/483955 | 21:24 |
| dansmith | personally I think it'd be better to just (re-)mock target_cell so you can get a stable __enter__ result and ensure that its yielded context is used for the instance | 21:25 |
| *** gjayavelu has quit IRC | 21:25 | |
| melwitt | dansmith: it's not relying on the fixture giving the same context for subsequent targets (I removed all of that in the top patch and this is me working backward fixing the bugs I found) | 21:25 |
| dansmith | I mean the same oslo_db context | 21:26 |
| dansmith | or whatever it's called | 21:26 |
| melwitt | I just wanted to verify it's targeted to the right thing | 21:26 |
| dansmith | the thing we yield from our global db facade hacky thing | 21:26 |
| melwitt | yeah, I took out that stuff in the last patch | 21:26 |
| melwitt | I left only default_ctxt_mgr. unless that's what you're talking about | 21:26 |
| dansmith | is that how this works then? | 21:27 |
| dansmith | because you're asserting that something from a freshly created context is the same as an earlier-created one | 21:27 |
| melwitt | oh, I think I understand what you're saying. you're talking about how in the fixture we only create a new context manager once per cell | 21:28 |
| *** gjayavelu has joined #openstack-nova | 21:29 | |
| dansmith | right | 21:29 |
| dansmith | yeah, it's oslo_db context_mgr that I'm trying to say with my words, right? | 21:29 |
| melwitt | okay. yes, oslo.db TransactionContextManager | 21:29 |
| melwitt | well, so we have one for cell0 and one for cell1 in the test env, and I wanted to assert that the notify one is targeted to cell1 and not cell0 | 21:30 |
| melwitt | that's what my thinking was | 21:30 |
| melwitt | to make sure we're targeted to the right cell | 21:30 |
| dansmith | which you're still doing in the new patch, L426 of fixtures.py | 21:31 |
| melwitt | yeah, sorry I misunderstood you. I thought you were talking about the last_ctxt_mgr thing that I took out | 21:31 |
| dansmith | yeah, I understand what you're doing, I just think it's obscure and an artifact of the fixture that is awkward to test for | 21:31 |
| melwitt | I noticed that wasn't actually doing anything anymore so I removed it | 21:31 |
| melwitt | but yeah I think asserting it's not None accomplishes the same thing | 21:32 |
| dansmith | since we fixed a bunch of stuff, I think we don't need that now, yeah | 21:32 |
| dansmith | my instance list stuff could avoid another db hit if I could tell what cell a context was targeted for, so .. we might want to consider doing something other than hackery for detecting that at some point | 21:32 |
| *** penick has joined #openstack-nova | 21:33 | |
| *** pchavva has quit IRC | 21:33 | |
| melwitt | yeah. eons ago I suggested putting cell_uuid in the context for something else. so maybe we could do that? | 21:34 |
| melwitt | when I brought it up it was back when we had the mq client cache thing in compute/rpcapi and we could have done cache lookups with the cell_uuid | 21:34 |
| dansmith | yeah, that's what I meant | 21:35 |
| *** penick_ has quit IRC | 21:36 | |
| melwitt | yeah, I'm +1 on that | 21:37 |
| *** penick has quit IRC | 21:37 | |
| *** penick has joined #openstack-nova | 21:39 | |
| *** felipemonteiro_ has quit IRC | 21:43 | |
| dansmith | gdi gerrit, come on | 21:43 |
| efried | Is there chatter on -infra about it? | 21:44 |
| efried | (I just rejoined) | 21:44 |
| dansmith | yeah | 21:44 |
| *** gouthamr has quit IRC | 21:45 | |
| efried | cause yeah | 21:46 |
| efried | Didn't realize how completely I rely on gerrit to do basically anything (and how much I took it for granted). | 21:46 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for placement conf & requests https://review.openstack.org/492247 | 21:47 |
| dansmith | melwitt: I'm going to run off for a bit but I'll hit those when I'm back if you get them updated | 21:48 |
| melwitt | dansmith: cool, thanks | 21:48 |
| efried | Rebase successful, but still complaining of a merge conflict. That's odd, isn't it? | 21:49 |
| melwitt | if it's Zuul, I think it's posting not-really-merge-conflicts | 21:49 |
| efried | ah | 21:49 |
| melwitt | right now things are back to being gated on Jenkins, so I'd only pay attention to that as far as passing/not | 21:49 |
| openstackgerrit | Kaustubh Phatak proposed openstack/nova master: Fixing: Bug #1722409 vmware network matches wrong substring from vSphere https://review.openstack.org/510702 | 21:50 |
| openstack | bug 1722409 in OpenStack Compute (nova) "nova wrongly attaches different network with similar name" [Undecided,New] https://launchpad.net/bugs/1722409 - Assigned to Kaustubh Phatak (kphatak-pf9) | 21:50 |
| openstackgerrit | Kaustubh Phatak proposed openstack/nova master: Fixing: Bug #1722409 vmware network matches https://review.openstack.org/510703 | 21:51 |
| openstack | bug 1722409 in OpenStack Compute (nova) "nova wrongly attaches different network with similar name" [Undecided,New] https://launchpad.net/bugs/1722409 - Assigned to Kaustubh Phatak (kphatak-pf9) | 21:51 |
| *** slaweq has joined #openstack-nova | 21:53 | |
| *** ijw has quit IRC | 21:53 | |
| *** jmlowe has quit IRC | 22:01 | |
| *** slaweq has quit IRC | 22:02 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Deprecate file injection https://review.openstack.org/509013 | 22:02 |
| mriedem | sdague: for your morrow ^ | 22:02 |
| *** chyka has joined #openstack-nova | 22:05 | |
| *** ijw has joined #openstack-nova | 22:06 | |
| *** chyka has quit IRC | 22:10 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/510324 | 22:11 |
| *** fragatina has quit IRC | 22:13 | |
| *** fragatina has joined #openstack-nova | 22:13 | |
| *** jaypipes has quit IRC | 22:14 | |
| *** gjayavelu has quit IRC | 22:15 | |
| *** hongbin has joined #openstack-nova | 22:15 | |
| *** gjayavelu has joined #openstack-nova | 22:18 | |
| *** thorst has joined #openstack-nova | 22:21 | |
| *** thorst has quit IRC | 22:21 | |
| *** thorst has joined #openstack-nova | 22:22 | |
| *** Sukhdev has joined #openstack-nova | 22:23 | |
| *** boris_42_ has quit IRC | 22:25 | |
| *** lbragstad has quit IRC | 22:25 | |
| *** edmondsw has quit IRC | 22:27 | |
| *** andreas_s has joined #openstack-nova | 22:28 | |
| *** claudiub has quit IRC | 22:28 | |
| *** baoli has quit IRC | 22:29 | |
| *** john51 has quit IRC | 22:32 | |
| *** andreas_s has quit IRC | 22:32 | |
| *** john51 has joined #openstack-nova | 22:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move user_data max length check to schema https://review.openstack.org/510704 | 22:33 |
| *** fragatina has quit IRC | 22:34 | |
| *** john51 has quit IRC | 22:35 | |
| *** john51 has joined #openstack-nova | 22:38 | |
| *** smatzek has joined #openstack-nova | 22:42 | |
| *** lyan has quit IRC | 22:43 | |
| *** smatzek has quit IRC | 22:47 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add alternate hosts https://review.openstack.org/486215 | 22:47 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | 22:47 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | 22:47 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/510159 | 22:47 |
| *** catintheroof has joined #openstack-nova | 22:47 | |
| *** Sukhdev has quit IRC | 22:51 | |
| *** awaugama has quit IRC | 22:53 | |
| *** sdague has quit IRC | 22:56 | |
| melwitt | grr, I thought I came up with a good way to test the scatter gather thing, but SpawnIsSynchronous messes that up. or rather, what I assert is still true without the fix bc it's synchronous | 22:59 |
| *** MasterOfBugs has joined #openstack-nova | 23:06 | |
| *** gbarros_ has joined #openstack-nova | 23:10 | |
| *** mriedem1 has joined #openstack-nova | 23:11 | |
| *** gbarros has quit IRC | 23:12 | |
| melwitt | hm, it works when I remove SpawnIsSynchronous. I dunno if that makes it susceptible to timing bugs | 23:12 |
| melwitt | probably. | 23:12 |
| *** aloga has quit IRC | 23:13 | |
| *** mriedem has quit IRC | 23:13 | |
| *** tbachman has quit IRC | 23:20 | |
| *** crushil has quit IRC | 23:21 | |
| *** crushil has joined #openstack-nova | 23:21 | |
| *** gjayavelu has quit IRC | 23:22 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967 | 23:22 |
| openstackgerrit | melanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells https://review.openstack.org/510691 | 23:22 |
| openstackgerrit | melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968 | 23:22 |
| openstackgerrit | melanie witt proposed openstack/nova master: WIP Improve the CellDatabases test fixture and usage https://review.openstack.org/508432 | 23:22 |
| *** gbarros_ has quit IRC | 23:25 | |
| *** gbarros has joined #openstack-nova | 23:26 | |
| *** ijw has quit IRC | 23:26 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Target context for build notification in conductor https://review.openstack.org/509967 | 23:29 |
| openstackgerrit | melanie witt proposed openstack/nova master: Fix target_cell usage for scatter_gather_cells https://review.openstack.org/510691 | 23:29 |
| openstackgerrit | melanie witt proposed openstack/nova master: Elevate existing RequestContext to get bandwidth usage https://review.openstack.org/509968 | 23:29 |
| openstackgerrit | melanie witt proposed openstack/nova master: WIP Improve the CellDatabases test fixture and usage https://review.openstack.org/508432 | 23:29 |
| *** gbarros has quit IRC | 23:41 | |
| *** MasterOfBugs has quit IRC | 23:45 | |
| *** edmondsw has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!