| *** kumarmn has joined #openstack-nova | 00:03 | |
| *** kumarmn has quit IRC | 00:05 | |
| *** kumarmn has joined #openstack-nova | 00:05 | |
| *** bigdogstl has quit IRC | 00:06 | |
| *** superdan is now known as dansmith | 00:11 | |
| *** yingjun has joined #openstack-nova | 00:12 | |
| *** fragatina has joined #openstack-nova | 00:13 | |
| *** kumarmn has quit IRC | 00:15 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:17 | |
| *** bigdogstl has joined #openstack-nova | 00:18 | |
| *** markvoelker has joined #openstack-nova | 00:20 | |
| *** edmondsw has joined #openstack-nova | 00:24 | |
| *** markvoelker has quit IRC | 00:24 | |
| *** yangyapeng has quit IRC | 00:28 | |
| *** edmondsw has quit IRC | 00:28 | |
| *** yangyapeng has joined #openstack-nova | 00:29 | |
| *** claudiub has quit IRC | 00:31 | |
| *** fragatina has quit IRC | 00:31 | |
| *** yangyapeng has quit IRC | 00:33 | |
| *** john51 has quit IRC | 00:37 | |
| *** john51 has joined #openstack-nova | 00:37 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:42 | |
| *** Dinesh_Bhor has quit IRC | 00:44 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:44 | |
| *** Dinesh_Bhor has quit IRC | 00:45 | |
| prometheanfire | nova doesn't look like it supports the new os_vif https://review.openstack.org/532987 | 00:49 |
|---|---|---|
| *** TTris has joined #openstack-nova | 00:50 | |
| *** auggy has quit IRC | 00:55 | |
| *** auggy has joined #openstack-nova | 00:55 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:56 | |
| *** hoangcx has joined #openstack-nova | 00:59 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix allocation_candidates not to ignore shared RPs https://review.openstack.org/533396 | 01:01 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support relay RP for allocation candidates https://review.openstack.org/533437 | 01:01 |
| *** zhaochao has joined #openstack-nova | 01:03 | |
| *** phuongnh has joined #openstack-nova | 01:07 | |
| *** jichen has joined #openstack-nova | 01:08 | |
| *** lyan has quit IRC | 01:09 | |
| *** yangyapeng has joined #openstack-nova | 01:13 | |
| *** eharney has quit IRC | 01:14 | |
| *** kumarmn has joined #openstack-nova | 01:15 | |
| *** hongbin has joined #openstack-nova | 01:17 | |
| *** tetsuro_ has joined #openstack-nova | 01:18 | |
| *** yangyapeng has quit IRC | 01:21 | |
| *** yangyapeng has joined #openstack-nova | 01:21 | |
| *** kumarmn has quit IRC | 01:21 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: [placement] Fix resource provider delete https://review.openstack.org/529519 | 01:25 |
| *** Dinesh_Bhor has quit IRC | 01:28 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:30 | |
| *** Dinesh_Bhor has quit IRC | 01:34 | |
| openstackgerrit | Wangpan proposed openstack/nova stable/pike: Handle glance exception during rotating instance backup https://review.openstack.org/533439 | 01:36 |
| *** Dinesh_Bhor has joined #openstack-nova | 01:40 | |
| *** threestrands_ has joined #openstack-nova | 01:40 | |
| *** threestrands has quit IRC | 01:43 | |
| *** tiendc has joined #openstack-nova | 01:43 | |
| *** armax has joined #openstack-nova | 01:44 | |
| *** Dinesh_Bhor has quit IRC | 01:52 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:52 | |
| *** namnh has joined #openstack-nova | 01:56 | |
| *** yassine has joined #openstack-nova | 01:59 | |
| *** yassine is now known as Guest54669 | 01:59 | |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 02:02 |
| *** Dinesh_Bhor has quit IRC | 02:03 | |
| *** itlinux has joined #openstack-nova | 02:07 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 02:09 |
| *** Dinesh_Bhor has joined #openstack-nova | 02:14 | |
| *** Dinesh_Bhor has quit IRC | 02:23 | |
| *** zhurong has joined #openstack-nova | 02:28 | |
| *** itlinux has quit IRC | 02:34 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:36 | |
| *** bigdogstl has quit IRC | 02:40 | |
| *** hshiina has joined #openstack-nova | 02:45 | |
| *** elmaciej has joined #openstack-nova | 02:46 | |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 02:47 |
| *** itlinux has joined #openstack-nova | 02:48 | |
| *** kwathore has quit IRC | 02:48 | |
| *** dkliban has joined #openstack-nova | 02:50 | |
| *** sree has joined #openstack-nova | 02:51 | |
| *** markvoelker has joined #openstack-nova | 02:51 | |
| *** dkliban has left #openstack-nova | 02:53 | |
| *** annp has joined #openstack-nova | 02:54 | |
| *** tbachman has quit IRC | 02:54 | |
| *** elmaciej has quit IRC | 03:01 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201 | 03:03 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520 | 03:04 |
| *** abhishekk has joined #openstack-nova | 03:04 | |
| *** Dinesh_Bhor has quit IRC | 03:04 | |
| *** bigdogstl has joined #openstack-nova | 03:06 | |
| *** kumarmn has joined #openstack-nova | 03:10 | |
| *** jappleii__ has joined #openstack-nova | 03:12 | |
| *** jappleii__ has quit IRC | 03:13 | |
| *** jappleii__ has joined #openstack-nova | 03:13 | |
| *** threestrands_ has quit IRC | 03:15 | |
| *** bigdogstl has quit IRC | 03:18 | |
| *** kumarmn has quit IRC | 03:19 | |
| *** markvoelker has quit IRC | 03:25 | |
| *** bigdogstl has joined #openstack-nova | 03:27 | |
| *** bigdogstl has quit IRC | 03:32 | |
| *** namnh has quit IRC | 03:34 | |
| *** namnh has joined #openstack-nova | 03:35 | |
| *** kumarmn has joined #openstack-nova | 03:45 | |
| *** sdake has quit IRC | 03:46 | |
| *** sdake has joined #openstack-nova | 03:47 | |
| *** sdake has quit IRC | 03:47 | |
| *** sdake has joined #openstack-nova | 03:47 | |
| *** psachin has joined #openstack-nova | 03:47 | |
| *** bigdogstl has joined #openstack-nova | 03:55 | |
| *** edmondsw has joined #openstack-nova | 04:00 | |
| *** bigdogstl has quit IRC | 04:03 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:04 | |
| *** yamamoto has joined #openstack-nova | 04:05 | |
| *** edmondsw has quit IRC | 04:05 | |
| *** claudiub has joined #openstack-nova | 04:08 | |
| *** bigdogstl has joined #openstack-nova | 04:08 | |
| *** hiro-kobayashi has quit IRC | 04:10 | |
| *** Dinesh_Bhor has quit IRC | 04:13 | |
| openstackgerrit | Chen Hanxiao proposed openstack/nova master: test: Use assertEqual() instead of assertXXXEqual() https://review.openstack.org/533493 | 04:16 |
| *** itlinux has quit IRC | 04:17 | |
| *** bigdogstl has quit IRC | 04:18 | |
| *** yingjun has quit IRC | 04:21 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:36 | |
| *** zhurong has quit IRC | 04:39 | |
| *** kumarmn has quit IRC | 04:42 | |
| *** hiro-kobayashi has joined #openstack-nova | 04:46 | |
| -openstackstatus- NOTICE: The logs.openstack.org filesystem has been restored to full health. We are attempting to keep logs uploaded between the prior alert and this one, however if your job logs are missing please issue a recheck. | 04:49 | |
| *** ChanServ changes topic to "The logs.openstack.org filesystem has been restored to full health. We are attempting to keep logs uploaded between the prior alert and this one, however if your job logs are missing please issue a recheck." | 04:49 | |
| *** bigdogstl has joined #openstack-nova | 04:51 | |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Queens_Release_Schedule" | 04:53 | |
| *** Dinesh_Bhor has quit IRC | 04:59 | |
| *** janki has joined #openstack-nova | 04:59 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:00 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 05:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 05:09 |
| *** bigdogstl has quit IRC | 05:10 | |
| *** links has joined #openstack-nova | 05:11 | |
| *** suresh12 has joined #openstack-nova | 05:14 | |
| *** bigdogstl has joined #openstack-nova | 05:17 | |
| *** markvoelker has joined #openstack-nova | 05:22 | |
| *** pcaruana has joined #openstack-nova | 05:23 | |
| *** Eran_Kuris has joined #openstack-nova | 05:24 | |
| *** zhurong has joined #openstack-nova | 05:24 | |
| *** bigdogstl has quit IRC | 05:26 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506 | 05:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506 | 05:27 |
| *** pcaruana has quit IRC | 05:32 | |
| *** ratailor has joined #openstack-nova | 05:32 | |
| *** gouthamr has joined #openstack-nova | 05:35 | |
| *** kumarmn has joined #openstack-nova | 05:43 | |
| *** hongbin has quit IRC | 05:43 | |
| *** xinliang has quit IRC | 05:46 | |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 05:47 |
| *** kumarmn has quit IRC | 05:48 | |
| *** edmondsw has joined #openstack-nova | 05:48 | |
| *** sridharg has joined #openstack-nova | 05:51 | |
| *** edmondsw has quit IRC | 05:53 | |
| *** gouthamr has quit IRC | 05:54 | |
| *** yamamoto_ has joined #openstack-nova | 05:55 | |
| *** yamamoto_ has quit IRC | 05:55 | |
| *** markvoelker has quit IRC | 05:55 | |
| *** bigdogstl has joined #openstack-nova | 05:56 | |
| *** yamamoto has quit IRC | 05:59 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334 | 06:00 |
| *** bigdogstl has quit IRC | 06:00 | |
| *** xinliang has joined #openstack-nova | 06:01 | |
| *** Tom-Tom has joined #openstack-nova | 06:06 | |
| *** Dinesh_Bhor has quit IRC | 06:07 | |
| *** moshele has joined #openstack-nova | 06:07 | |
| *** janki is now known as janki|afk | 06:10 | |
| *** janki|afk has quit IRC | 06:10 | |
| *** janki has joined #openstack-nova | 06:11 | |
| *** bigdogstl has joined #openstack-nova | 06:12 | |
| *** janki is now known as janki|afk | 06:12 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:12 | |
| *** yamamoto has joined #openstack-nova | 06:16 | |
| *** bigdogstl has quit IRC | 06:22 | |
| *** shaohe_feng has quit IRC | 06:27 | |
| *** suresh12 has quit IRC | 06:29 | |
| *** Eran_Kuris has quit IRC | 06:29 | |
| *** suresh12 has joined #openstack-nova | 06:31 | |
| *** bigdogstl has joined #openstack-nova | 06:34 | |
| *** goutham has joined #openstack-nova | 06:37 | |
| *** felipemonteiro_ has joined #openstack-nova | 06:37 | |
| *** liuzz has joined #openstack-nova | 06:37 | |
| *** abhishekk has quit IRC | 06:37 | |
| *** bigdogstl has quit IRC | 06:39 | |
| *** shaohe_feng has joined #openstack-nova | 06:40 | |
| *** suresh12 has quit IRC | 06:40 | |
| *** hieulq has quit IRC | 06:41 | |
| *** kumarmn has joined #openstack-nova | 06:44 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Add support for the 2.57 microversion https://review.openstack.org/528128 | 06:45 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601 | 06:45 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168 | 06:45 |
| *** kumarmn has quit IRC | 06:49 | |
| *** abhishekk has joined #openstack-nova | 06:50 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Add zvm CI information https://review.openstack.org/533512 | 06:56 |
| *** felipemonteiro_ has quit IRC | 06:58 | |
| *** liuzz has quit IRC | 06:59 | |
| *** liuzz has joined #openstack-nova | 07:00 | |
| *** lajoskatona has joined #openstack-nova | 07:02 | |
| *** jappleii__ has quit IRC | 07:02 | |
| *** bigdogstl has joined #openstack-nova | 07:03 | |
| *** MasterofJOKers_ has joined #openstack-nova | 07:07 | |
| *** bigdogstl has quit IRC | 07:07 | |
| *** slaweq has joined #openstack-nova | 07:10 | |
| *** MasterofJOKers has quit IRC | 07:10 | |
| *** MasterofJOKers_ is now known as MasterofJOKers | 07:10 | |
| *** sapd_ has quit IRC | 07:16 | |
| *** r-daneel has quit IRC | 07:17 | |
| *** sapd has joined #openstack-nova | 07:17 | |
| *** r-daneel has joined #openstack-nova | 07:17 | |
| *** goutham has quit IRC | 07:19 | |
| *** sapd_ has joined #openstack-nova | 07:20 | |
| *** sapd has quit IRC | 07:22 | |
| *** alexchadin has joined #openstack-nova | 07:25 | |
| *** maciejjozefczyk_ is now known as maciejjozefczyk | 07:29 | |
| *** Dinesh_Bhor has quit IRC | 07:31 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117 | 07:33 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429 | 07:33 |
| *** Dinesh_Bhor has joined #openstack-nova | 07:33 | |
| *** Dinesh_Bhor has quit IRC | 07:33 | |
| *** sapd has joined #openstack-nova | 07:34 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.59 - Migrations list pagination https://review.openstack.org/516168 | 07:35 |
| *** edmondsw has joined #openstack-nova | 07:36 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:37 | |
| *** bigdogstl has joined #openstack-nova | 07:39 | |
| *** edmondsw has quit IRC | 07:41 | |
| *** kumarmn has joined #openstack-nova | 07:45 | |
| *** janki|afk is now known as janki | 07:46 | |
| *** bigdogstl has quit IRC | 07:48 | |
| *** alexchadin has quit IRC | 07:49 | |
| *** kumarmn has quit IRC | 07:49 | |
| *** Spazmotic has joined #openstack-nova | 07:51 | |
| *** markvoelker has joined #openstack-nova | 07:53 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:54 | |
| *** hoonetorg has quit IRC | 07:58 | |
| *** matrohon has joined #openstack-nova | 07:58 | |
| *** rcernin has quit IRC | 07:59 | |
| *** Eran_Kuris has joined #openstack-nova | 08:00 | |
| *** kholkina has joined #openstack-nova | 08:02 | |
| *** alexchadin has joined #openstack-nova | 08:05 | |
| ameeda | Hi | 08:06 |
| ameeda | how can I update image properties in regression test function ? "self.image["properties"] = {key: value}" | 08:06 |
| *** hoonetorg has joined #openstack-nova | 08:11 | |
| *** bigdogstl has joined #openstack-nova | 08:17 | |
| *** zhurong has quit IRC | 08:18 | |
| *** ralonsoh has joined #openstack-nova | 08:19 | |
| *** avolkov has joined #openstack-nova | 08:20 | |
| *** zhurong has joined #openstack-nova | 08:20 | |
| *** hiro-kobayashi has quit IRC | 08:23 | |
| *** ragiman has joined #openstack-nova | 08:25 | |
| *** jichen has quit IRC | 08:26 | |
| *** bigdogstl has quit IRC | 08:26 | |
| *** markvoelker has quit IRC | 08:26 | |
| *** fragatina has joined #openstack-nova | 08:26 | |
| *** jichen has joined #openstack-nova | 08:26 | |
| *** Dinesh_Bhor has quit IRC | 08:30 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:32 | |
| *** sahid has joined #openstack-nova | 08:34 | |
| *** moshele has quit IRC | 08:39 | |
| *** kumarmn has joined #openstack-nova | 08:46 | |
| *** moshele has joined #openstack-nova | 08:46 | |
| *** jpena|off is now known as jpena | 08:49 | |
| *** kumarmn has quit IRC | 08:50 | |
| *** fragatina has quit IRC | 08:50 | |
| *** moshele has quit IRC | 08:52 | |
| *** bauwser is now known as bauzas | 08:57 | |
| bauzas | morning Novaers | 08:57 |
| bauzas | busy days | 08:58 |
| gibi | morning bauzas | 08:58 |
| *** jaianshu has joined #openstack-nova | 08:59 | |
| bauzas | :) | 08:59 |
| *** psachin has quit IRC | 08:59 | |
| *** moshele has joined #openstack-nova | 09:00 | |
| *** alexchadin has quit IRC | 09:04 | |
| *** alexchadin has joined #openstack-nova | 09:05 | |
| *** moshele has quit IRC | 09:05 | |
| *** ociuhandu_ has quit IRC | 09:10 | |
| *** lucas-afk is now known as lucasagomes | 09:15 | |
| *** hshiina has quit IRC | 09:15 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/python-novaclient master: Add missing spaces in `nova list --changes-since` help https://review.openstack.org/533575 | 09:16 |
| *** alexchadin has quit IRC | 09:17 | |
| *** jafeha__ is now known as jafeha | 09:17 | |
| *** tssurya has joined #openstack-nova | 09:17 | |
| *** Dinesh_Bhor has quit IRC | 09:18 | |
| ameeda | bauzas: how can I update image properties in regression test function ? "self.image["properties"] = {key: value}" | 09:18 |
| ameeda | gibi: how can I update image properties in regression test function ? "self.image["properties"] = {key: value}" | 09:19 |
| *** Dinesh_Bhor has joined #openstack-nova | 09:19 | |
| *** ttsiouts has joined #openstack-nova | 09:21 | |
| *** bigdogstl has joined #openstack-nova | 09:22 | |
| *** edmondsw has joined #openstack-nova | 09:25 | |
| bauzas | ameeda: you'd like to test some specific image properties ? | 09:25 |
| bauzas | is that in a unit test or a functional test ? | 09:25 |
| bauzas | if so, you'd mock the image | 09:26 |
| ameeda | bauzas: this is functional test, I want to update image properties then check if the value is changes "truncates" or not. | 09:28 |
| bauzas | ameeda: if that's for a functional test, you should just pass a specific image | 09:29 |
| *** edmondsw has quit IRC | 09:29 | |
| openstackgerrit | Édouard Thuleau proposed openstack/nova master: Update plugs Contrail methods to work with privsep https://review.openstack.org/533212 | 09:29 |
| *** bigdogstl has quit IRC | 09:29 | |
| ameeda | bauzas: you can see what I do here "http://paste.openstack.org/show/644825/" | 09:31 |
| ameeda | bauzas: I have to check if image properties of original image is same as image properties of snapshot image. | 09:32 |
| ameeda | bauzas: so I have to update image['properties'] , but I did that like this " image['properties']={key: value}, it doesn't works | 09:33 |
| Spazmotic | morning friends | 09:35 |
| bauzas | ameeda: I'd rather generate an image that would contain the property I'd like to test, and check eventually what I give | 09:35 |
| bauzas | s/give/get | 09:35 |
| *** alexchadin has joined #openstack-nova | 09:36 | |
| bauzas | morning | 09:36 |
| *** derekh has joined #openstack-nova | 09:36 | |
| *** takashin has left #openstack-nova | 09:37 | |
| ameeda | bauzas: when I tried to do client api.post_image(image) that give me API not found | 09:37 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Handle glance exception during rotating instance backup https://review.openstack.org/533439 | 09:37 |
| bauzas | ameeda: there is a fake Glance service that you can use for your purpose https://github.com/openstack/nova/blob/master/nova/tests/unit/image/fake.py | 09:37 |
| bauzas | then, you need to stub that service in the functional test | 09:37 |
| ameeda | bauzas: thank you for your time. do you have an example of stub ? | 09:38 |
| bauzas | like https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L1369 | 09:38 |
| ameeda | bauzas: thanks! | 09:39 |
| bauzas | that should give you a "fake Glance" where you could create an image like you wish and pass the image ref to the Nova call you wish | 09:39 |
| bauzas | ameeda: HTH | 09:40 |
| *** larsks has quit IRC | 09:40 | |
| ameeda | bauzas: let me check , I will ask you if I have another question :$ | 09:40 |
| ameeda | Thanks | 09:40 |
| *** moshele has joined #openstack-nova | 09:40 | |
| bauzas | ameeda: np and good luck | 09:42 |
| *** larsks has joined #openstack-nova | 09:43 | |
| ratailor | Can anyone review https://review.openstack.org/#/c/498334/ | 09:46 |
| ratailor | It goes in merge conflict every now and then.. | 09:46 |
| *** zhurong has quit IRC | 09:46 | |
| *** kumarmn has joined #openstack-nova | 09:46 | |
| *** sapd has quit IRC | 09:46 | |
| mdbooth | ratailor: I'll look fwiw. | 09:49 |
| *** kumarmn has quit IRC | 09:51 | |
| *** xinliang has quit IRC | 09:52 | |
| *** lpetrut has joined #openstack-nova | 09:52 | |
| *** moshele has quit IRC | 09:53 | |
| *** moshele has joined #openstack-nova | 09:54 | |
| *** slaweq has quit IRC | 09:54 | |
| *** slaweq has joined #openstack-nova | 09:55 | |
| Spazmotic | whew... that was a good meal | 09:55 |
| *** gszasz has joined #openstack-nova | 09:56 | |
| *** Dinesh_Bhor has quit IRC | 09:57 | |
| Spazmotic | Did Gerritt go through any major changes in the past year? I logged in to make my first commit in a long time and old commits were gone | 09:57 |
| Spazmotic | wondering if i just accidentallied an ew user ㅋㅋㅋㅋ | 09:58 |
| Spazmotic | Not that it matters too terribly.. just curious | 09:58 |
| *** sapd has joined #openstack-nova | 09:59 | |
| *** hieulq has joined #openstack-nova | 09:59 | |
| *** slaweq has quit IRC | 09:59 | |
| *** sahid has quit IRC | 10:02 | |
| *** jaosorior has joined #openstack-nova | 10:03 | |
| *** jichen has quit IRC | 10:04 | |
| *** xinliang has joined #openstack-nova | 10:05 | |
| *** slaweq has joined #openstack-nova | 10:06 | |
| *** sahid has joined #openstack-nova | 10:07 | |
| mdbooth | Spazmotic: Don't think so. Were they just abandoned for you? | 10:08 |
| Spazmotic | Possible.. was also a whole different release so maybe just something there. | 10:08 |
| *** bigdogstl has joined #openstack-nova | 10:08 | |
| Spazmotic | i'll pull up the commit some time later to see just how dumb I am.. too lazy and full now :D | 10:10 |
| Spazmotic | Cheat day.. so many carbs.. so many | 10:10 |
| *** namnh has quit IRC | 10:11 | |
| *** matrohon has quit IRC | 10:12 | |
| *** matrohon has joined #openstack-nova | 10:12 | |
| *** mvk has joined #openstack-nova | 10:13 | |
| *** bigdogstl has quit IRC | 10:19 | |
| *** elmaciej has joined #openstack-nova | 10:22 | |
| *** markvoelker has joined #openstack-nova | 10:24 | |
| *** psachin has joined #openstack-nova | 10:24 | |
| *** sambetts|afk is now known as sambetts | 10:26 | |
| *** rabel_b1 has joined #openstack-nova | 10:26 | |
| *** rabel_b1 has quit IRC | 10:32 | |
| *** alexchadin has quit IRC | 10:36 | |
| ameeda | bauzas: Thank you very much, its work !! | 10:36 |
| *** alexchadin has joined #openstack-nova | 10:36 | |
| ameeda | Regarding this gerrit https://review.openstack.org/#/c/528385 | 10:36 |
| bauzas | cool | 10:36 |
| ameeda | I have another issue | 10:37 |
| ameeda | I was added the detach_volume function outside of _cleanup_volumes to avoid regression, | 10:37 |
| ameeda | since this function used in many places. | 10:37 |
| ameeda | as you see, two code-reviewers asked me why I added that outside. | 10:37 |
| ameeda | https://review.openstack.org/#/c/528385/4/nova/compute/manager.py | 10:37 |
| ameeda | So I move my code to be inside of _cleanup_volumes as here | 10:37 |
| ameeda | https://review.openstack.org/#/c/528385/5/nova/compute/manager.py | 10:37 |
| ameeda | but I faced many issues with py27 and I solve them all. now, I faced many issues with functional test, | 10:37 |
| ameeda | what if I return my code to be outside of _cleanup_volumes? | 10:37 |
| *** sree has quit IRC | 10:39 | |
| *** sree has joined #openstack-nova | 10:39 | |
| kashyap | PSA: If you're hitting: "devstackf27 httpd[5096]: AH00534: httpd: Configuration error: More than one MPM loaded." with DevStack, there's apparently a fix merged for Apache this morning | 10:42 |
| kashyap | https://bugs.launchpad.net/devstack/+bug/1740194 | 10:42 |
| openstack | Launchpad bug 1740194 in devstack "Apache2 unable to start as 2 MPM modules enabled on Fedora 27" [Undecided,Fix released] - Assigned to Attila Fazekas (afazekas) | 10:42 |
| *** sapd has quit IRC | 10:45 | |
| *** sree has quit IRC | 10:46 | |
| *** kumarmn has joined #openstack-nova | 10:47 | |
| *** cdent has joined #openstack-nova | 10:47 | |
| *** dtantsur is now known as dtantsur|bbl | 10:48 | |
| *** phuongnh has quit IRC | 10:51 | |
| *** annp has quit IRC | 10:51 | |
| *** abhishekk has quit IRC | 10:51 | |
| elmaciej | Hi! | 10:51 |
| *** kumarmn has quit IRC | 10:53 | |
| *** alexchadin has quit IRC | 10:54 | |
| *** alexchadin has joined #openstack-nova | 10:55 | |
| ameeda | bauzas: can you help me with that ? | 10:56 |
| *** markvoelker has quit IRC | 10:57 | |
| *** alexchadin has quit IRC | 11:00 | |
| *** bigdogstl has joined #openstack-nova | 11:03 | |
| *** bigdogstl has quit IRC | 11:07 | |
| *** alexchadin has joined #openstack-nova | 11:09 | |
| *** cdent has quit IRC | 11:10 | |
| *** yamamoto has quit IRC | 11:11 | |
| *** cdent has joined #openstack-nova | 11:11 | |
| *** yamamoto has joined #openstack-nova | 11:11 | |
| *** bigdogstl has joined #openstack-nova | 11:12 | |
| *** edmondsw has joined #openstack-nova | 11:13 | |
| *** tbachman has joined #openstack-nova | 11:13 | |
| *** cdent has quit IRC | 11:16 | |
| *** cdent has joined #openstack-nova | 11:17 | |
| *** edmondsw has quit IRC | 11:17 | |
| *** bigdogstl has quit IRC | 11:17 | |
| *** alexchadin has quit IRC | 11:22 | |
| *** alexchadin has joined #openstack-nova | 11:23 | |
| *** matrohon has quit IRC | 11:26 | |
| *** yamamoto_ has joined #openstack-nova | 11:26 | |
| *** gszasz has quit IRC | 11:27 | |
| *** yamamoto has quit IRC | 11:29 | |
| *** bigdogstl has joined #openstack-nova | 11:33 | |
| lyarwood | https://review.openstack.org/#/c/531407/ - Any stable cores able to +2/+W this? I'd rather not fast approve something with my name all over it :) | 11:34 |
| Spazmotic | It's not so muc hthat it's a problem that it's there, ameeda, but you're making that particular function set have an incorrect purpose by placing it inside. of the build process. | 11:36 |
| Spazmotic | When there is already a cleanup process | 11:36 |
| Spazmotic | Especially since it already calls cleanup, would be a ..cleaner system to let cleanup do the...cleanup.. sorry it's late hehe | 11:37 |
| Spazmotic | If you can provide a good reason to have it out of there i'm sure people will understand, but unfortuantely to appease the unit test heavens is going to be a tough sell | 11:40 |
| *** tiendc has quit IRC | 11:41 | |
| *** bigdogstl has quit IRC | 11:41 | |
| *** sdague has joined #openstack-nova | 11:41 | |
| *** matrohon has joined #openstack-nova | 11:42 | |
| *** bigdogstl has joined #openstack-nova | 11:42 | |
| *** nicolasbock has joined #openstack-nova | 11:44 | |
| *** pcaruana has joined #openstack-nova | 11:44 | |
| *** sree has joined #openstack-nova | 11:46 | |
| *** bigdogstl has quit IRC | 11:46 | |
| *** kumarmn has joined #openstack-nova | 11:48 | |
| *** sree has quit IRC | 11:50 | |
| *** kumarmn has quit IRC | 11:53 | |
| *** alexchadin has quit IRC | 11:55 | |
| ameeda | Spazmotic: I still beginner with Openstack, I am not sure by what I said "to avoid regression, since the _cleanup_volumes used in another places", So you guess that I have to keep the code as its now , I mean inside cleanup and fix tests failures ? | 11:56 |
| *** liuzz has quit IRC | 11:57 | |
| *** liuzz has joined #openstack-nova | 11:57 | |
| *** alexchadin has joined #openstack-nova | 11:57 | |
| kashyap | Does anyone have a link to DevStack Ceph job, please? | 12:00 |
| kashyap | Okay, that's it: "gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial" | 12:01 |
| Spazmotic | I'm sorry, these patch sets are so different from each other i'm struggling to keep up here.. give me a few to read it | 12:02 |
| *** liuzz has quit IRC | 12:03 | |
| ameeda | I'm sorry for that | 12:03 |
| Spazmotic | Ok I've got it. | 12:03 |
| ameeda | great | 12:06 |
| ameeda | so what do you think ? | 12:06 |
| Spazmotic | reading the code | 12:06 |
| ameeda | sorry, take your time | 12:06 |
| *** bigdogstl has joined #openstack-nova | 12:06 | |
| Spazmotic | interesting bug... wouldn't midn testing it myself, but reading | 12:06 |
| *** tetsuro_ has quit IRC | 12:07 | |
| ameeda | sure :) | 12:07 |
| Spazmotic | I think I agree there's no reason to throw if instance in there btw since you have already checked for this in the try/except it should be guarunteed to be there | 12:08 |
| ameeda | ok, I agree with you too | 12:08 |
| Spazmotic | I would also recommend leaving it here in cleanup and not try to move it back out.. this is definately hwere it belongs | 12:09 |
| ameeda | so I have to complete my work to fix all failures of tests | 12:09 |
| Spazmotic | i glanced through the console log of zuul.. some nasty exceptiosn in there you're going to have to paly with. my guess is since this ia pretty big change to the cleanup method you may need to adjust the test a bit.. but remember not to Mock your way to victory.. .make sure it makes sense for your new conditions :) | 12:09 |
| Spazmotic | back in aw hile | 12:09 |
| *** psachin has quit IRC | 12:10 | |
| ameeda | take your time | 12:10 |
| *** bigdogstl has quit IRC | 12:11 | |
| *** lucasagomes is now known as lucas-hungry | 12:13 | |
| mdbooth | ratailor: Hey, just commented on your patch. Did you respond to bauzas comments oob at any point? I was going to make the same point, but saw he'd already done it. | 12:13 |
| *** tesseract has joined #openstack-nova | 12:15 | |
| *** mvk has quit IRC | 12:16 | |
| *** sree has joined #openstack-nova | 12:18 | |
| *** sree_ has joined #openstack-nova | 12:22 | |
| *** sree_ is now known as Guest18249 | 12:23 | |
| *** sree has quit IRC | 12:25 | |
| gmann | alex_xu: can you check this - https://review.openstack.org/#/c/522918/ | 12:29 |
| * efried waves | 12:30 | |
| *** bigdogstl has joined #openstack-nova | 12:33 | |
| cdent | efried: aren't you supposed to be on holiday? | 12:36 |
| efried | cdent gibi Thanks for the rechecks. Zuul had a really bad week last week; let's hope this one is better. | 12:36 |
| efried | cdent No? | 12:36 |
| efried | I believe mriedem is on holiday this week. | 12:36 |
| cdent | It's mlk day. I guess that's not an enforced US holiday. I think my us employer-colleagues are taking it off. I can never remember. | 12:37 |
| efried | ohhh, right. | 12:37 |
| cdent | zuul probably wants sacrifices | 12:37 |
| efried | I don't think this is one of those, but let me check... | 12:37 |
| efried | nope, doesn't look like it. I've got pto scheduled for tomorrow, though. | 12:37 |
| efried | My kids are off. But not me. | 12:37 |
| *** jpena is now known as jpena|lunch | 12:38 | |
| kashyap | mdbooth: Hey, when you get a sec, is there a way I can see a Gate job result for an instance boot with an ephemeral disk? | 12:39 |
| *** andreas_s has joined #openstack-nova | 12:39 | |
| *** bigdogstl has quit IRC | 12:40 | |
| *** bigdogstl has joined #openstack-nova | 12:41 | |
| mdbooth | kashyap: Almost certainly, but I wouldn't know off the top of my head which tempest tests cover ephemerals. | 12:41 |
| *** takashin has joined #openstack-nova | 12:41 | |
| kashyap | mdbooth: No problem, I'll plough through them. | 12:42 |
| *** r-daneel_ has joined #openstack-nova | 12:42 | |
| *** r-daneel has quit IRC | 12:43 | |
| *** r-daneel_ is now known as r-daneel | 12:43 | |
| Spazmotic | My commit handles ephs, one sec | 12:45 |
| *** bigdogstl has quit IRC | 12:46 | |
| Spazmotic | Ah you wanted gate, not initial check didn't you | 12:46 |
| kashyap | Spazmotic: I want to see the libvirt XML on any of the Gate jobs with an instance boot with ephemeral disk | 12:48 |
| kashyap | Okido, my local DevStack run just finished after 24 minutes (real 24m49.794s), so I'll try locally | 12:48 |
| *** smatzek has joined #openstack-nova | 12:49 | |
| Spazmotic | Ahhh I see, I cannot help so much with that, hehe. Sorry i'll go back into the shadows | 12:49 |
| *** kumarmn has joined #openstack-nova | 12:49 | |
| kashyap | Heh, np. | 12:49 |
| *** bigdogstl has joined #openstack-nova | 12:50 | |
| *** andreaf has quit IRC | 12:53 | |
| *** andreaf has joined #openstack-nova | 12:53 | |
| *** kumarmn has quit IRC | 12:54 | |
| *** markvoelker has joined #openstack-nova | 12:54 | |
| *** edmondsw has joined #openstack-nova | 12:55 | |
| *** ragiman has quit IRC | 12:57 | |
| *** dtantsur|bbl is now known as dtantsur | 13:01 | |
| *** bigdogstl has quit IRC | 13:01 | |
| *** vladikr has joined #openstack-nova | 13:02 | |
| *** dikonoor has joined #openstack-nova | 13:03 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Make sure that functional test triggered on sample changes https://review.openstack.org/533210 | 13:03 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: Testing if funct test is triggered https://review.openstack.org/533211 | 13:04 |
| *** tetsuro has joined #openstack-nova | 13:04 | |
| *** sahid has quit IRC | 13:05 | |
| *** jaypipes has joined #openstack-nova | 13:08 | |
| elmaciej | Hi! Maybe someone can point me how to start contributing to openstack. I made some fix to allow live migration on lvm based shared storage and don't know if it's worthly to put it official or not. Where should I get started. | 13:10 |
| zioproto | sdague: ping, is it business hours in your timezone now ? :) | 13:10 |
| zioproto | elmaciej: you need to "git review" your patch. More info here https://docs.openstack.org/infra/manual/developers.html | 13:11 |
| elmaciej | zioproto - so I can create my own branch, push to it and call the review? just like that? | 13:11 |
| zioproto | elmaciej: you need to "Sign the appropriate Individual Contributor License Agreement" | 13:11 |
| zioproto | elmaciej: it is exaplained in that page | 13:12 |
| zioproto | then you can push | 13:12 |
| zioproto | and somebody will review your patch | 13:12 |
| zioproto | you dont need to create a branch | 13:12 |
| zioproto | all the work should be in 1 single commit | 13:12 |
| zioproto | then on your shell | 13:12 |
| zioproto | just type | 13:12 |
| zioproto | git review | 13:12 |
| zioproto | this will automatically create a branch | 13:12 |
| elmaciej | ok,got this, thanks, will take a look at evening then. | 13:13 |
| *** mvk has joined #openstack-nova | 13:13 | |
| *** lucas-hungry is now known as lucasagomes | 13:13 | |
| zioproto | elmaciej: thanks for contributing to openstack | 13:13 |
| *** bigdogstl has joined #openstack-nova | 13:13 | |
| Spazmotic | Really no need to create a local branch? have always done it :D | 13:13 |
| elmaciej | it's a pleasure :) I needed live migration on lvm :) buisness requirment | 13:13 |
| *** ratailor has quit IRC | 13:13 | |
| *** danpawlik_ has quit IRC | 13:17 | |
| *** bigdogstl has quit IRC | 13:17 | |
| *** openstackgerrit has quit IRC | 13:18 | |
| *** sahid has joined #openstack-nova | 13:19 | |
| *** alexchadin has quit IRC | 13:19 | |
| *** r-daneel has quit IRC | 13:21 | |
| cdent | bauzas, gibi: 500 fix, already one +2, about 6 months old, would be nice to have: https://review.openstack.org/#/c/389798/ | 13:23 |
| *** lajoskatona has quit IRC | 13:25 | |
| *** yangyapeng has quit IRC | 13:26 | |
| *** yangyapeng has joined #openstack-nova | 13:27 | |
| *** bigdogstl has joined #openstack-nova | 13:27 | |
| *** tetsuro has quit IRC | 13:27 | |
| *** jdillaman has joined #openstack-nova | 13:27 | |
| *** markvoelker has quit IRC | 13:28 | |
| jaypipes | cdent: done. | 13:28 |
| cdent | danke jaypipes wasn't sure if you were a) still alive, b) here today, c) here yet | 13:29 |
| cdent | feeling better? | 13:29 |
| *** jpena|lunch is now known as jpena | 13:29 | |
| jaypipes | cdent: much, thank you. I'm back around 75% I think. | 13:29 |
| cdent | flu sucks | 13:30 |
| cdent | (I think it was diana on much of that thing, I just did a rebase towards the end) | 13:30 |
| *** janki has quit IRC | 13:31 | |
| *** jaianshu has quit IRC | 13:31 | |
| *** janki has joined #openstack-nova | 13:32 | |
| *** yangyapeng has quit IRC | 13:32 | |
| gibi | jaypipes: welcome back! | 13:32 |
| *** bigdogstl has quit IRC | 13:32 | |
| *** Eran_Kuris has quit IRC | 13:32 | |
| jaypipes | gibi: thanks mate :) | 13:32 |
| jaypipes | feels good to be able to work again. | 13:32 |
| lyarwood | bauzas: so https://review.openstack.org/#/c/531407/ wasn't really my change FWIW, mdbooth and melwitt rewrote most of it in master so we should be good to +W this now. | 13:33 |
| *** krtaylor has quit IRC | 13:33 | |
| *** lennyb_ has joined #openstack-nova | 13:33 | |
| *** lennyb_ has quit IRC | 13:34 | |
| efried | jaypipes Howdy sir, welcome back to the land of the living. | 13:35 |
| *** Guest18249 has quit IRC | 13:35 | |
| *** sree has joined #openstack-nova | 13:36 | |
| *** matrohon has quit IRC | 13:36 | |
| *** matrohon has joined #openstack-nova | 13:36 | |
| jaypipes | efried: thanks :) | 13:37 |
| *** danpawlik has joined #openstack-nova | 13:37 | |
| *** r-daneel has joined #openstack-nova | 13:38 | |
| *** esberglu has joined #openstack-nova | 13:40 | |
| *** Guest54669 has quit IRC | 13:40 | |
| mdbooth | lyarwood: IIUC we're expecting failures in https://review.openstack.org/#/c/523958/ because we're assuming c-vol has done its thing, which it hasn't, right? | 13:41 |
| mdbooth | lyarwood: i.e. A lack of tempest failures would indicate a lack of test coverage? | 13:42 |
| mdbooth | And we need a couple of Depends-On: for c-vol patches which don't exist, yet? | 13:42 |
| * mdbooth would like to make it clear that he considers ^^^ to be a very good thing | 13:43 | |
| *** Guest54669 has joined #openstack-nova | 13:43 | |
| lyarwood | mdbooth: yes and yes, there's some additional coverage in the barbican plugin but there's also alot missing from standard tempest | 13:43 |
| *** openstackgerrit has joined #openstack-nova | 13:43 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: create vGPU for instance https://review.openstack.org/528832 | 13:43 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | 13:43 |
| bauzas | AFAIK, mriedem is not around today, right? | 13:43 |
| mdbooth | lyarwood: Ok, thanks. | 13:43 |
| *** sree has quit IRC | 13:44 | |
| *** bigdogstl has joined #openstack-nova | 13:44 | |
| *** Eran_Kuris has joined #openstack-nova | 13:47 | |
| *** esberglu has quit IRC | 13:48 | |
| *** sree has joined #openstack-nova | 13:48 | |
| *** Guest54669 has quit IRC | 13:48 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | 13:48 |
| *** tbachman has quit IRC | 13:49 | |
| *** bigdogstl has quit IRC | 13:49 | |
| *** yassine has joined #openstack-nova | 13:49 | |
| cdent | edleafe, efried: can I assume we are having the usual scheduler meeting? | 13:49 |
| *** yassine is now known as Guest55219 | 13:49 | |
| jaypipes | cdent: yes, but edleafe is commuting. | 13:50 |
| *** kumarmn has joined #openstack-nova | 13:50 | |
| jaypipes | cdent: unless he's off for MLK day? | 13:50 |
| jaypipes | not sure.. | 13:50 |
| *** finucannot is now known as stephenfin | 13:50 | |
| efried | IBM doesn't have MLK as a site holiday. But edleafe asked me to run it, so either he's off anyway, or he's commuting. | 13:51 |
| *** kumarmn has quit IRC | 13:54 | |
| efried | cdent You didn't find the first paragraph of the commit message adequately explanatory? https://review.openstack.org/#/c/533244/ | 13:56 |
| cdent | efried: imagine you didn't know what a _Provider is? | 13:56 |
| cdent | That's what I mean by "in isolation" | 13:56 |
| *** sree has quit IRC | 13:57 | |
| *** zhaochao has quit IRC | 13:57 | |
| efried | okay. | 13:57 |
| *** krtaylor has joined #openstack-nova | 13:57 | |
| cdent | efried: I know it is tedious, and I know I dont do that to everyone, but a) I should, b) I feel like you'll take it in good intent intended | 13:58 |
| cdent | (also I didn't -1 it, just commented) | 13:58 |
| efried | cdent I'm planning to slot this change into the existing series, so that would give it some context. But I'll add some words. | 13:59 |
| *** yangyapeng has joined #openstack-nova | 13:59 | |
| efried | Scheduler meeting NOW in #openstack-meeting-alt | 13:59 |
| Spazmotic | johnthetubaguy I was going to hit you up about a commit the other day but realized I no longer work with you.. too sad. | 13:59 |
| cdent | efried: imagine yourself in the shoes of a drunk, stupid, tired, angry git bisector | 14:00 |
| Spazmotic | I made a commit with some code against the resize_up xenapi stuff if you want to take a look, you're probably one of the best to review it and offer me some suggestions if you find the chance :) https://review.openstack.org/#/c/533168/ | 14:00 |
| Spazmotic | back after a while, stupid.. house.. not self cleaning.. | 14:00 |
| *** gszasz has joined #openstack-nova | 14:03 | |
| *** liverpooler has joined #openstack-nova | 14:04 | |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 14:05 |
| *** lyan has joined #openstack-nova | 14:05 | |
| *** bigdogstl has joined #openstack-nova | 14:05 | |
| *** eharney has joined #openstack-nova | 14:06 | |
| ameeda | alex_xu: are you around ? | 14:06 |
| *** sahid has quit IRC | 14:06 | |
| *** Eran_Kuris has quit IRC | 14:07 | |
| *** esberglu has joined #openstack-nova | 14:07 | |
| openstackgerrit | Merged openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | 14:10 |
| ameeda | gmann: Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 14:10 |
| ameeda | gmann: can you please review the code ? | 14:10 |
| *** andreas_s has quit IRC | 14:15 | |
| openstackgerrit | Merged openstack/nova master: console: Provide an RFB security proxy implementation https://review.openstack.org/345399 | 14:15 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | 14:15 |
| *** bigdogstl has quit IRC | 14:15 | |
| stephenfin | #success After 3 years and 7 (?) releases, encryption between nova's consoleproxy service and compute nodes is finally possible ✌️ | 14:16 |
| openstackstatus | stephenfin: Added success to Success page | 14:16 |
| cdent | stephenfin: | 14:17 |
| cdent | ++ | 14:17 |
| *** Eran_Kuris has joined #openstack-nova | 14:18 | |
| ameeda | stephenfin: Congratulations | 14:19 |
| *** andreas_s has joined #openstack-nova | 14:19 | |
| *** r-daneel has quit IRC | 14:23 | |
| *** andreas_s has quit IRC | 14:25 | |
| *** andreas_s has joined #openstack-nova | 14:25 | |
| *** sahid has joined #openstack-nova | 14:25 | |
| *** andreas__ has joined #openstack-nova | 14:26 | |
| *** cdent has quit IRC | 14:28 | |
| jaypipes | it's breakfast time. | 14:28 |
| *** kumarmn has joined #openstack-nova | 14:30 | |
| *** andreas_s has quit IRC | 14:30 | |
| *** kumarmn_ has joined #openstack-nova | 14:30 | |
| gibi | stephenfin: nice! | 14:31 |
| ildikov | stephenfin: yaaay! nice job! congrats! :) | 14:33 |
| stephenfin | jaypipes: I'll have you know it's basically supper time, actually ;) | 14:34 |
| stephenfin | johnthetubaguy: You're hardly about, are you? | 14:34 |
| *** kumarmn has quit IRC | 14:34 | |
| *** tssurya has quit IRC | 14:35 | |
| * stephenfin wants openstack-doc-core added to the os-api-ref-core Gerrit group | 14:35 | |
| *** ratailor has joined #openstack-nova | 14:38 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 14:40 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | 14:40 |
| *** matrohon has quit IRC | 14:41 | |
| *** matrohon has joined #openstack-nova | 14:41 | |
| *** gcb has joined #openstack-nova | 14:44 | |
| openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.59 - Add trusted_image_certificates https://review.openstack.org/500396 | 14:44 |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 14:48 |
| *** tssurya has joined #openstack-nova | 14:50 | |
| *** eharney has quit IRC | 14:52 | |
| *** elmaciej has quit IRC | 14:52 | |
| *** dulek has joined #openstack-nova | 14:52 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove unnecessary execute permissions in files https://review.openstack.org/533686 | 14:54 |
| *** awaugama has joined #openstack-nova | 14:54 | |
| *** tbh_ has joined #openstack-nova | 14:55 | |
| *** mgariepy has quit IRC | 14:55 | |
| *** matrohon has quit IRC | 14:58 | |
| *** yamahata has joined #openstack-nova | 14:59 | |
| *** elmaciej has joined #openstack-nova | 15:01 | |
| *** ratailor has quit IRC | 15:03 | |
| *** takashin has left #openstack-nova | 15:04 | |
| *** mgariepy has joined #openstack-nova | 15:05 | |
| *** eharney has joined #openstack-nova | 15:05 | |
| *** cdent has joined #openstack-nova | 15:05 | |
| *** yamamoto_ has quit IRC | 15:06 | |
| bauzas | dansmith: mdbooth: FYI, just posted a new revision for the VGPU series starting with https://review.openstack.org/#/c/528832/8 that now lookups existing instances when restarting nova-compute | 15:10 |
| bauzas | dansmith: mdbooth: that said, I still need to work around the XML removal when rebooting/restarting/etc. | 15:10 |
| bauzas | wondering if that could be a follow-up patch | 15:11 |
| dansmith | bauzas: yeah I think we just can't merge the first one until that follow-up is ready behind it | 15:11 |
| bauzas | k | 15:12 |
| mdbooth | bauzas: What dansmith said. | 15:12 |
| *** tssurya has quit IRC | 15:15 | |
| *** armax has quit IRC | 15:17 | |
| *** r-daneel has joined #openstack-nova | 15:17 | |
| openstackgerrit | Carlos Goncalves proposed openstack/os-vif master: Git ignore .stestr https://review.openstack.org/533699 | 15:19 |
| *** links has quit IRC | 15:20 | |
| *** kumarmn_ has left #openstack-nova | 15:23 | |
| *** bigdogstl has joined #openstack-nova | 15:24 | |
| *** moshele has quit IRC | 15:25 | |
| *** markvoelker has joined #openstack-nova | 15:25 | |
| *** tssurya has joined #openstack-nova | 15:28 | |
| *** gszasz has quit IRC | 15:28 | |
| *** matrohon has joined #openstack-nova | 15:28 | |
| *** hongbin has joined #openstack-nova | 15:29 | |
| *** archit has joined #openstack-nova | 15:33 | |
| *** r-daneel has quit IRC | 15:34 | |
| *** elmaciej has quit IRC | 15:34 | |
| *** r-daneel has joined #openstack-nova | 15:35 | |
| *** slaweq has quit IRC | 15:40 | |
| *** mlavalle has joined #openstack-nova | 15:40 | |
| *** slaweq has joined #openstack-nova | 15:40 | |
| *** bigdogstl has quit IRC | 15:41 | |
| *** andreas__ has quit IRC | 15:42 | |
| *** andreas_s has joined #openstack-nova | 15:43 | |
| *** tidwellr has joined #openstack-nova | 15:43 | |
| *** slaweq has quit IRC | 15:44 | |
| *** andreas_s has quit IRC | 15:46 | |
| *** andreas_s has joined #openstack-nova | 15:46 | |
| *** armax has joined #openstack-nova | 15:47 | |
| lyarwood | stephenfin: https://review.openstack.org/#/c/527046/ - any chance of a +W? Had to rebase over the weekend. | 15:49 |
| *** elmaciej has joined #openstack-nova | 15:49 | |
| *** felipemonteiro has joined #openstack-nova | 15:50 | |
| dansmith | bauzas: that recreate on boot patch is nice.. and pretty simple.. too simple not to do :) | 15:50 |
| bauzas | dansmith: I'll need to fully rewrite the bottom change | 15:51 |
| stephenfin | lyarwood: Sure, done | 15:51 |
| dansmith | bauzas: because why? | 15:51 |
| bauzas | dansmith: given I'm changing all the interfaces by not passing the allocations to _get_guest_xml() but rather asks to allocate a mdev for spawn() and passing it to _get_guest_xml() | 15:52 |
| dansmith | bauzas: ah okay | 15:52 |
| lyarwood | stephenfin: thanks | 15:52 |
| openstackgerrit | Carlos Goncalves proposed openstack/os-vif master: WIP: vif_plug_ovs: add should_provide_port https://review.openstack.org/533713 | 15:52 |
| bauzas | dansmith: so for hard_reboot(), I'll instead ask for the current mdev for passing it to _get_guest_xml() | 15:52 |
| dansmith | bauzas: aye | 15:53 |
| *** gcb has quit IRC | 15:54 | |
| *** felipemonteiro_ has joined #openstack-nova | 15:54 | |
| *** kholkina has quit IRC | 15:58 | |
| edleafe | efried: thanks for that huge fix on https://review.openstack.org/#/c/526436/ :) | 15:58 |
| *** tssurya has left #openstack-nova | 15:58 | |
| *** felipemonteiro has quit IRC | 15:58 | |
| efried | It was an enormous sacrifice of time and mental anguish, but you're worth it edleafe | 15:58 |
| *** tssurya has joined #openstack-nova | 15:58 | |
| edleafe | Your suffering has not gone unnoticed | 15:58 |
| efried | edleafe Caught up on sched meeting yet? | 15:59 |
| *** markvoelker has quit IRC | 15:59 | |
| edleafe | no, just poured me a cup of coffee and was getting ready to read back | 15:59 |
| efried | ight. There was some question of how you felt about ProviderTree.snapshot (formerly known as ReadOnlyProvider) | 16:00 |
| bauzas | jaypipes: I think we already discussed about that previously, but someone pinged me about reviewing https://review.openstack.org/#/c/526900/19/nova/db/sqlalchemy/models.py | 16:01 |
| bauzas | jaypipes: would it be a performance issue ? | 16:01 |
| jaypipes | bauzas: no | 16:01 |
| *** chyka has joined #openstack-nova | 16:01 | |
| jaypipes | bauzas: shouldn't be a performance issue at all. | 16:02 |
| bauzas | ok, I remembered your point, but I wasn't sure it was for VARCHAR(255) > TEXT | 16:02 |
| *** chyka has quit IRC | 16:02 | |
| bauzas | ok, will review it when I have time then | 16:02 |
| *** chyka has joined #openstack-nova | 16:02 | |
| jaypipes | bauzas: that said, doesn't the REST API limit the length of the metadata value? | 16:03 |
| jaypipes | bauzas: in which case a microversion is gonna be necessary | 16:03 |
| bauzas | jaypipes: yup, possibly | 16:03 |
| bauzas | tbh, I haven't reviewed it | 16:04 |
| bauzas | just looking at the main modification | 16:04 |
| bauzas | jaypipes: for your point, that'll depend if we cut the string by the API or not | 16:04 |
| bauzas | but yeah, if it's all about having the API giving more than 255 chars for that, then yes, it's a microversion needed | 16:05 |
| *** jackie-truong has joined #openstack-nova | 16:06 | |
| *** yamamoto has joined #openstack-nova | 16:07 | |
| * efried hands gibi a cookie | 16:09 | |
| efried | namedtuple. Fabulous. | 16:09 |
| *** hoonetorg has quit IRC | 16:09 | |
| *** slaweq has joined #openstack-nova | 16:11 | |
| *** itlinux has joined #openstack-nova | 16:12 | |
| prometheanfire | nova doesn't look like it supports the new os_vif https://review.openstack.org/532987 | 16:12 |
| prometheanfire | will bug the list if you want | 16:12 |
| edleafe | efried: not that fabulous | 16:12 |
| efried | edleafe You don't like? | 16:12 |
| edleafe | efried: see the comment I just posted | 16:12 |
| *** yamamoto has quit IRC | 16:14 | |
| efried | edleafe You set an attribute on the class, not an instance thereof. | 16:14 |
| edleafe | ugh, was just thinking that through more. Coffee hasn't fully kicked in yet | 16:15 |
| openstackgerrit | Merged openstack/nova master: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/389798 | 16:15 |
| openstackgerrit | Merged openstack/nova master: Add aggregates check in allocation candidates https://review.openstack.org/522407 | 16:15 |
| *** slaweq has quit IRC | 16:16 | |
| * gibi takes the cookie and goes home with it | 16:17 | |
| *** bigdogstl has joined #openstack-nova | 16:18 | |
| stephenfin | prometheanfire: Oh, looking | 16:20 |
| stephenfin | sean-k-mooney, ralonsoh: Seen https://review.openstack.org/#/c/532987/ ? | 16:20 |
| sean-k-mooney | stephenfin: looking now | 16:21 |
| prometheanfire | stephenfin: cool | 16:21 |
| prometheanfire | hopefully it's just a overly specific test | 16:21 |
| ralonsoh | stephenfin: I'm taking a look now | 16:21 |
| sean-k-mooney | stephenfin: at first glance these see unrelated to os-vif | 16:22 |
| stephenfin | sean-k-mooney: Quite possibly. I haven't looked into it in detail | 16:23 |
| stephenfin | Just saw the multiple rechecks. Usually a sign of something more sinister | 16:23 |
| sean-k-mooney | stephenfin: the network basic opts test were failing due to a an ip confict | 16:23 |
| *** bigdogstl has quit IRC | 16:23 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | 16:24 |
| sean-k-mooney | stephenfin: yes however that fact that tempest.scenario.test_network_basic_ops.TestNetworkBasicOps passes on multinode (one of the failing test) means its likely not a os-vif issue | 16:25 |
| *** bigdogstl has joined #openstack-nova | 16:25 | |
| sean-k-mooney | stephenfin: ill dig a little deeper | 16:25 |
| *** dave-mccowan has joined #openstack-nova | 16:27 | |
| sean-k-mooney | stephenfin: there are a whole bunch of Too many open files errors in n-cpu logs | 16:27 |
| *** sridharg has quit IRC | 16:27 | |
| sean-k-mooney | e.g. http://logs.openstack.org/87/532987/1/check/legacy-tempest-dsvm-neutron-full/cb9c0bb/logs/screen-n-cpu.txt.gz?level=ERROR#_Jan_12_14_00_56_222328 | 16:27 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_traits_for_provider https://review.openstack.org/532564 | 16:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Track associated sharing RPs in report client https://review.openstack.org/526539 | 16:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | 16:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 16:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | 16:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | 16:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 16:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.snapshot => ProviderSnapshot https://review.openstack.org/533244 | 16:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 16:29 |
| efried | jaypipes cdent edleafe ^ -- and FYI, the bottom patch is also needed by mgoddard to get ironic traits moving. | 16:29 |
| jaypipes | efried: sure, just finishing up a rebase | 16:29 |
| *** dave-mcc_ has joined #openstack-nova | 16:30 | |
| efried | gibi also ^ -- thanks for the comments on the ProviderTree.snapshot patch. I've now slotted it into the series where it's gonna wind up. | 16:30 |
| edleafe | efried: ack | 16:30 |
| cdent | noted | 16:31 |
| *** suresh12 has joined #openstack-nova | 16:31 | |
| *** dave-mccowan has quit IRC | 16:32 | |
| *** READ10 has joined #openstack-nova | 16:32 | |
| *** andreas_s has quit IRC | 16:33 | |
| *** andreas_s has joined #openstack-nova | 16:33 | |
| *** itlinux has quit IRC | 16:34 | |
| *** AlexeyAbashkin has quit IRC | 16:39 | |
| *** AlexeyAbashkin has joined #openstack-nova | 16:39 | |
| *** tesseract has quit IRC | 16:40 | |
| artom | Pci knowledgable folks, if I have a PciDevice (a <hostdev> in the libvirt xml), what would be the best way to find its MAC? | 16:42 |
| *** gouthamr has joined #openstack-nova | 16:42 | |
| artom | sahid, stephenfin ^^ ? | 16:42 |
| *** andreas_s has quit IRC | 16:42 | |
| sean-k-mooney | stephenfin: ill stack with the latest release of os-vif and see if i can replicate the ci failure on my end | 16:43 |
| *** gyee has joined #openstack-nova | 16:43 | |
| sahid | artom: not sure i understand, we do set the mac address when configuring the libvirt <interface>, no? | 16:44 |
| *** jpena is now known as jpena|brb | 16:44 | |
| artom | sahid, I mean when passing through an entire PF | 16:44 |
| artom | It's a <hostdev> in the XML, not an <interface> | 16:44 |
| artom | So there's no MAC | 16:44 |
| sahid | oh i see | 16:44 |
| stephenfin | sean-k-mooney: Sounds good. I'll take a look myself tomorrow | 16:45 |
| stephenfin | See if we can figure out what's going on | 16:45 |
| sahid | artom: no idea sorry | 16:45 |
| artom | sahid, no worries :) | 16:45 |
| artom | Who's the most knowledgeable PCI person we have now that vladikr and nikola aren't with us anymore? | 16:46 |
| *** itlinux has joined #openstack-nova | 16:46 | |
| *** bigdogstl has quit IRC | 16:46 | |
| *** sree has joined #openstack-nova | 16:46 | |
| sean-k-mooney | stephenfin: it could be litally that we are not hitting the openfile limit but it could be something else too. if its ulimit then it something we need to fix in devstack | 16:47 |
| *** markvoelker has joined #openstack-nova | 16:47 | |
| *** AlexeyAbashkin has quit IRC | 16:47 | |
| *** nicolasbock has quit IRC | 16:48 | |
| sahid | artom: in such situation it's the guest os which have the device, the host can't really see it | 16:48 |
| sahid | i mean i guess it can't | 16:48 |
| sean-k-mooney | stephenfin: fyi this local.conf should allow you to reproduce http://paste.openstack.org/show/645256/ | 16:49 |
| sahid | so probably your change is qemu-guest-agent | 16:49 |
| stephenfin | sean-k-mooney: Funny how it would trigger now. Surely we were using privsep before? | 16:49 |
| sahid | s/have the device/own the device | 16:49 |
| *** gouthamr has quit IRC | 16:50 | |
| sean-k-mooney | stephenfin: we were but maybe with the change to useing the python libs we are a using more filedicriptors or b we are leaking them? | 16:50 |
| *** nicolasbock has joined #openstack-nova | 16:50 | |
| stephenfin | We could validate that by turning on warnings. Python 3 warns for open fds | 16:50 |
| *** sree has quit IRC | 16:51 | |
| *** liverpooler has quit IRC | 16:53 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/533733 | 16:54 |
| *** itlinux has quit IRC | 16:55 | |
| sean-k-mooney | stephenfin: it could possibly be a provider difference? the test that were failing on rackspace were passing on inap when run as part of the grenade multinode job. anyway ill see if i can reproduce locally | 16:55 |
| *** itlinux has joined #openstack-nova | 16:57 | |
| *** gouthamr has joined #openstack-nova | 16:57 | |
| sean-k-mooney | stephenfin: apparently https://review.openstack.org/508498 broke Kuryr-Kubernetes so we might need to do a 1.8.1/1.9.0 release to fix them though im not conviced the fix should not be on there end yet | 16:58 |
| stephenfin | sean-k-mooney: Oh, someone else is using os-vif. Interesting... | 16:59 |
| sean-k-mooney | ya i was aware kuryr was using it but i did not think they were ever saving os-vif objects | 16:59 |
| *** itlinux has quit IRC | 17:03 | |
| *** tbh_ has quit IRC | 17:05 | |
| *** bigdogstl has joined #openstack-nova | 17:06 | |
| *** dave-mcc_ has quit IRC | 17:08 | |
| *** andreas_s has joined #openstack-nova | 17:09 | |
| *** felipemonteiro_ has quit IRC | 17:10 | |
| *** itlinux has joined #openstack-nova | 17:11 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | 17:12 |
| *** bigdogstl has quit IRC | 17:12 | |
| *** hoonetorg has joined #openstack-nova | 17:14 | |
| *** suresh12 has quit IRC | 17:16 | |
| *** janki has quit IRC | 17:17 | |
| *** andreas_s has quit IRC | 17:19 | |
| *** efried is now known as efried_rollin | 17:19 | |
| *** links has joined #openstack-nova | 17:21 | |
| jaypipes | sean-k-mooney: I'm currently replying to mdulko about that... | 17:23 |
| jaypipes | sean-k-mooney: bottom line, even if we did do obj_make_compatible(), that wouldn't have magically caused the client side to not break. | 17:23 |
| *** matrohon has quit IRC | 17:24 | |
| jaypipes | sean-k-mooney: they still will need to advertise what versions of the objects they need (Nova does it via the nova-specific oslo versionedobject registry, etc) | 17:24 |
| jaypipes | sean-k-mooney: there's no free lunch here... | 17:24 |
| sean-k-mooney | jaypipes: right v2 of the patch had the make compatible code but you would have to do the down grade and i think the compat code just created an empty list of ip | 17:24 |
| jaypipes | right. | 17:25 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | 17:25 |
| jaypipes | anyway, writing an answer... | 17:25 |
| jaypipes | dansmith might be interested in this thread, too.. | 17:25 |
| sean-k-mooney | the main point i wanted to get across was the lib freeze is thusday so if we can fix it by then it needs to be fixed in kuryr kubernetes | 17:25 |
| jaypipes | sean-k-mooney: "fix" is relative here, which is my point :) | 17:26 |
| sean-k-mooney | we could revert the patch i guess since nova is not using that field | 17:26 |
| jaypipes | sean-k-mooney: "fix" might be just a flat revert, yeah | 17:26 |
| *** jpena|brb is now known as jpena | 17:27 | |
| sean-k-mooney | well we could certenly do that but they are using the object i a way we specifcially intoduced that patch to prevent so longterm we would have to adress the migration issue | 17:27 |
| *** elmaciej has quit IRC | 17:28 | |
| *** lucasagomes is now known as lucas-afk | 17:30 | |
| jaypipes | sean-k-mooney: did you even know that kuryr-kubernetes was consuming os-vif objects? I certainly didn't... | 17:31 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | 17:35 |
| *** andreas_s has joined #openstack-nova | 17:36 | |
| *** avolkov has quit IRC | 17:38 | |
| melwitt | bauzas: this was previously approved by mriedem, so I think we can re-approve it after the rebase https://review.openstack.org/#/c/531407 | 17:38 |
| *** bigdogstl has joined #openstack-nova | 17:39 | |
| *** bigdogstl has quit IRC | 17:44 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: create vGPU for instance https://review.openstack.org/528832 | 17:45 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | 17:45 |
| bauzas | melwitt: okie | 17:46 |
| * bauzas moves out for a while but will back tonight | 17:46 | |
| melwitt | thanks bauzas | 17:47 |
| *** andreas_s_ has joined #openstack-nova | 17:47 | |
| *** links has quit IRC | 17:50 | |
| *** andreas_s has quit IRC | 17:50 | |
| *** gszasz has joined #openstack-nova | 17:50 | |
| *** hemna_ has joined #openstack-nova | 17:51 | |
| *** dikonoor has quit IRC | 17:52 | |
| *** bigdogstl has joined #openstack-nova | 17:52 | |
| itlinux | hello nova team.. I have a OOO deployment looking to implement cells2, by default cells2 has one already any tips on how to expand that? Thanks | 17:54 |
| sean-k-mooney | jaypipes: i knew that kuryr was for port plugging but i did not think they were ever storing if the os-vif objects | 17:55 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 17:56 |
| sean-k-mooney | jaypipes: the ips field that we moved form the subnet object to the vif object is never used in os-vif or nova hence why i taught it was safe to move it | 17:56 |
| *** bigdogstl has quit IRC | 17:58 | |
| sean-k-mooney | stephenfin: by the way looking at the nova unit test failuer in https://review.openstack.org/#/c/532987 i dont think we should ever be comparein the serialised form of os-vif objects in nova. we can add the datapath_type field to allow this to pass but i think we should reduce the specificity of the test in the long run | 18:01 |
| *** derekh has quit IRC | 18:02 | |
| stephenfin | sean-k-mooney: Agreed | 18:03 |
| stephenfin | Just about to leave the office, but I can tackle that first thing in the morning | 18:03 |
| stephenfin | Soon as I finish mriedem's comments on the websocket security series | 18:03 |
| stephenfin | o/ | 18:03 |
| sean-k-mooney | stephenfin: ill give it a go tonight but if i have not pushed anything then please do | 18:04 |
| *** ralonsoh has quit IRC | 18:07 | |
| *** gszasz has quit IRC | 18:08 | |
| *** bigdogstl has joined #openstack-nova | 18:09 | |
| melwitt | itlinux: do you mean you want to know how to add an additional cells v2 cell to a deployment? | 18:09 |
| *** mvk has quit IRC | 18:10 | |
| itlinux | thanks melwitt: I need to add other cell2 to the env. | 18:10 |
| itlinux | since OOO has a default but I need to include other DC | 18:10 |
| *** jackie-truong has quit IRC | 18:10 | |
| *** andreas_s_ has quit IRC | 18:11 | |
| melwitt | itlinux: sorry what's DC? | 18:13 |
| itlinux | Datacenter. | 18:13 |
| melwitt | itlinux: we have docs for adding a new cell here https://docs.openstack.org/nova/latest/user/cells.html#adding-a-new-cell-to-an-existing-deployment | 18:14 |
| itlinux | thanks | 18:14 |
| itlinux | this is for cell2 right? | 18:14 |
| melwitt | you'll have to set up a database, message queue, etc for the new cell first, then you'll use nova-manage commands to create_cell and discover_hosts | 18:14 |
| *** sambetts is now known as sambetts|afk | 18:14 | |
| melwitt | yes. the doc doesn't cover how to deploy the new message queue etc | 18:15 |
| itlinux | ok so I could run another OOO and then merge them? | 18:15 |
| itlinux | but it also says . Note, however, that this command is deprecated as of the 16.0.0 Pike release. | 18:15 |
| itlinux | I am in Pike.. | 18:15 |
| melwitt | hm, no, I don't think you'd want to do that because OOO is going to create new controllers and you don't want that | 18:15 |
| itlinux | ok | 18:16 |
| itlinux | that's right.. | 18:16 |
| *** dtantsur is now known as dtantsur|afk | 18:16 | |
| itlinux | so do we have steps on how to create the Q | 18:16 |
| *** bigdogstl has quit IRC | 18:16 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] use simple FaultWrapper https://review.openstack.org/533752 | 18:17 |
| *** jpena is now known as jpena|off | 18:18 | |
| *** kmalloc has joined #openstack-nova | 18:19 | |
| *** Swami_ has joined #openstack-nova | 18:20 | |
| *** gouthamr has quit IRC | 18:20 | |
| *** Tom-Tom has quit IRC | 18:21 | |
| *** corvus is now known as jeblair | 18:21 | |
| *** jeblair is now known as corvus | 18:21 | |
| *** sahid has quit IRC | 18:21 | |
| *** lpetrut has quit IRC | 18:22 | |
| -openstackstatus- NOTICE: Zuul has been restarted and has lost queue contents; changes in progress will need to be rechecked. | 18:23 | |
| cdent | do I recall correctly that changing etc/nova/api-paste.ini is not something we can do easily? | 18:27 |
| melwitt | itlinux: no, I don't think so. and I don't think OOO yet has the ability to add a new cells v2 cell | 18:27 |
| itlinux | ahh I heard that cells2 is enabled .. so just want to check how to find out | 18:28 |
| *** elmaciej has joined #openstack-nova | 18:28 | |
| melwitt | it's enabled for one cell already, I just don't think they have a way of adding more cells | 18:29 |
| itlinux | ok.. | 18:30 |
| itlinux | yes one is there.. | 18:30 |
| itlinux | so how do we find that out? thanks melwitt: | 18:30 |
| melwitt | itlinux: I would try asking in #tripleo about if there's support for multiple cells or if it's in the works | 18:32 |
| *** felipemonteiro has joined #openstack-nova | 18:34 | |
| *** moshele has joined #openstack-nova | 18:34 | |
| *** felipemonteiro_ has joined #openstack-nova | 18:35 | |
| *** tidwellr has quit IRC | 18:35 | |
| *** tidwellr has joined #openstack-nova | 18:36 | |
| *** felipemonteiro has quit IRC | 18:39 | |
| *** moshele has quit IRC | 18:44 | |
| itlinux | how about regions.. | 18:46 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | 18:48 |
| *** AlexeyAbashkin has joined #openstack-nova | 18:50 | |
| openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.60 - Add trusted_image_certificates https://review.openstack.org/500396 | 18:55 |
| *** AlexeyAbashkin has quit IRC | 18:55 | |
| *** corvus is now known as jeblair | 18:58 | |
| *** jeblair is now known as corvus | 18:58 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: func tests for nested providers in alloc candidate https://review.openstack.org/531443 | 19:11 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add _has_provider_trees() utility function https://review.openstack.org/531474 | 19:11 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: _get_trees_matching_all() https://review.openstack.org/531512 | 19:11 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add tests for _get_trees_matching_all() with trait https://review.openstack.org/531899 | 19:11 |
| melwitt | jaypipes: o/ hope you're feeling better | 19:11 |
| jaypipes | melwitt: thanks Melanie, I am :) | 19:13 |
| *** suresh12 has joined #openstack-nova | 19:13 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | 19:13 |
| *** suresh12 has quit IRC | 19:13 | |
| melwitt | jaypipes: glad to hear it :) | 19:15 |
| prometheanfire | sean-k-mooney: for the os-vif thing, is it gating related (open files) or neutron related (db deadlock)? | 19:15 |
| melwitt | itlinux: you'll want to consult tripleo folks for questions about managing regions in a OOO deployment. this is a nova development channel | 19:16 |
| sean-k-mooney | prometheanfire: i ran the senario test locally and all the failing test passed so im guessing the openfile issue is the real reason for the tempest failure | 19:16 |
| sean-k-mooney | prometheanfire: ill be pushing a patch for the nova unit tests in a few seconds | 19:16 |
| *** felipemonteiro__ has joined #openstack-nova | 19:17 | |
| prometheanfire | sean-k-mooney: cool, if that works we can just depend on that | 19:17 |
| *** edleafe- has joined #openstack-nova | 19:17 | |
| sean-k-mooney | prometheanfire: ya i think so i was going to update https://review.openstack.org/#/c/532987/1 also with a depends on. just need to copy some ssh keys as the vm im using has not been used for dev before. | 19:18 |
| prometheanfire | sean-k-mooney: wgm | 19:18 |
| prometheanfire | wfm | 19:18 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | 19:19 |
| melwitt | dansmith: do you think multiple cells with a global MQ could be a thing? I can't recall if you ever mentioned that in your cells talks | 19:19 |
| melwitt | asking in reference to this review https://review.openstack.org/#/c/518556 | 19:19 |
| *** edleafe has quit IRC | 19:19 | |
| *** edleafe- is now known as edleafe | 19:19 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | 19:19 |
| *** felipemonteiro_ has quit IRC | 19:20 | |
| dansmith | melwitt: you can't really because a compute will ask for conductor to do something, and you might get a conductor from your cell and you might not | 19:20 |
| melwitt | dansmith: oh, yeah ... I didn't think of that. okay, cool | 19:20 |
| *** bigdogstl has joined #openstack-nova | 19:20 | |
| openstackgerrit | sean mooney proposed openstack/nova master: make unit tests compatible with os-vif 0.8.0 https://review.openstack.org/533766 | 19:22 |
| sean-k-mooney | ... that should be 1.8.0 | 19:23 |
| openstackgerrit | sean mooney proposed openstack/nova master: make unit tests compatible with os-vif 1.8.0 https://review.openstack.org/533766 | 19:23 |
| *** andreas_s has joined #openstack-nova | 19:23 | |
| *** moshele has joined #openstack-nova | 19:25 | |
| sean-k-mooney | prometheanfire: is it Depend-On or Depends-On: | 19:26 |
| *** bigdogstl has quit IRC | 19:28 | |
| sean-k-mooney | prometheanfire: its Depends-On: .. one sec | 19:28 |
| sean-k-mooney | stephenfin: prometheanfire https://review.openstack.org/#/c/533766/ + https://review.openstack.org/#/c/532987/ should now hopefully work | 19:30 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: SEA https://review.openstack.org/523216 | 19:30 |
| prometheanfire | cool | 19:31 |
| sean-k-mooney | prometheanfire: the issue was only in the unit tests. really execution paths would have worked so this is just a minor change. | 19:32 |
| *** fragatina has joined #openstack-nova | 19:33 | |
| *** andreas_s has quit IRC | 19:38 | |
| edleafe | golang flag parse | 19:39 |
| edleafe | wrong channel? | 19:39 |
| *** chyka has quit IRC | 19:42 | |
| cdent | no edleafe, we've started, nova is going go | 19:49 |
| *** AlexeyAbashkin has joined #openstack-nova | 19:51 | |
| sean-k-mooney | cdent: is that toung and cheek or have we decided nova is not hipster enought lol | 19:53 |
| cdent | if we wanted to be hipster, go is surely too mainstream | 19:54 |
| edleafe | nova is hipster only ironically | 19:54 |
| sean-k-mooney | cdent: but the alternitive would be rust and that too painful | 19:55 |
| *** efried_rollin is now known as efried | 19:55 | |
| edleafe | Ironic should be written in rust | 19:55 |
| *** AlexeyAbashkin has quit IRC | 19:55 | |
| cdent | of course | 19:56 |
| sean-k-mooney | edleafe: are you a parent because that right there is a dad joke | 19:56 |
| edleafe | oh, I could dad-joke you to death | 19:56 |
| * cdent died long ago | 19:57 | |
| * edleafe thought cdent smelled funny | 19:58 | |
| *** moshele has quit IRC | 20:02 | |
| *** suresh12 has joined #openstack-nova | 20:03 | |
| *** tidwellr_ has joined #openstack-nova | 20:06 | |
| *** tidwellr has quit IRC | 20:06 | |
| *** slaweq has joined #openstack-nova | 20:15 | |
| jaypipes | edleafe: is it wrong that I dad-joke but have no children (that I know of) | 20:16 |
| edleafe | jaypipes: without kids it | 20:17 |
| edleafe | it's just bad-joke | 20:17 |
| cdent | jaypipes: your brother is your twin, yeah? maybe that's it? | 20:17 |
| *** lpetrut has joined #openstack-nova | 20:19 | |
| *** bigdogstl has joined #openstack-nova | 20:20 | |
| *** dansmith has quit IRC | 20:22 | |
| *** markvoelker has quit IRC | 20:22 | |
| jaypipes | cdent: yes. yes, that's it. I'll blame him. | 20:25 |
| cdent | always happy so sow brotherly discord | 20:26 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: Add taskflow to requirements https://review.openstack.org/533779 | 20:27 |
| *** bigdogstl has quit IRC | 20:28 | |
| *** jdillaman has quit IRC | 20:29 | |
| *** matrohon has joined #openstack-nova | 20:34 | |
| *** jdillaman has joined #openstack-nova | 20:34 | |
| *** fragatina has quit IRC | 20:35 | |
| *** Guest49522 has joined #openstack-nova | 20:35 | |
| *** Guest49522 is now known as dansmith | 20:40 | |
| *** awaugama has quit IRC | 20:42 | |
| *** matrohon has quit IRC | 20:45 | |
| *** matrohon has joined #openstack-nova | 20:46 | |
| *** sree has joined #openstack-nova | 20:47 | |
| *** AlexeyAbashkin has joined #openstack-nova | 20:50 | |
| *** sree has quit IRC | 20:52 | |
| *** AlexeyAbashkin has quit IRC | 20:54 | |
| *** lpetrut has quit IRC | 20:56 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | 20:58 |
| *** bigdogstl has joined #openstack-nova | 20:58 | |
| *** chyka has joined #openstack-nova | 20:59 | |
| *** vivsoni_ has joined #openstack-nova | 21:03 | |
| *** tidwellr_ has quit IRC | 21:06 | |
| *** suresh12 has quit IRC | 21:06 | |
| *** tidwellr has joined #openstack-nova | 21:06 | |
| *** vivsoni has quit IRC | 21:06 | |
| *** suresh12 has joined #openstack-nova | 21:07 | |
| *** bigdogstl has quit IRC | 21:10 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | 21:10 |
| *** prometheanfire has left #openstack-nova | 21:10 | |
| cdent | efried, edleafe, jaypipes: you might find that ^ interesting. Not done yet, but a potentially useful tidy up | 21:11 |
| cdent | will get back to it tomorrow | 21:11 |
| *** flwang has quit IRC | 21:11 | |
| jaypipes | bauzas: https://review.openstack.org/#/c/528832/9/nova/virt/libvirt/config.py :) | 21:13 |
| *** fragatina has joined #openstack-nova | 21:13 | |
| jaypipes | cdent: ack | 21:13 |
| jaypipes | bauzas: maybe 10th time is the charm? ;P | 21:13 |
| jaypipes | ok, now on to efried' | 21:15 |
| jaypipes | s series :) | 21:15 |
| * efried waits, with breath abated... | 21:16 | |
| bauzas | jaypipes: oh fuuuuuuu | 21:20 |
| bauzas | childs is because I stupidly copied some other patch :p | 21:20 |
| bauzas | I have two children | 21:21 |
| bauzas | so I know that :p | 21:21 |
| *** suresh12 has quit IRC | 21:21 | |
| jaypipes | bauzas: :) | 21:22 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: create vGPU for instance https://review.openstack.org/528832 | 21:24 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | 21:24 |
| bauzas | hop là | 21:24 |
| *** flwang has joined #openstack-nova | 21:25 | |
| bauzas | dansmith: just in case you're still around, I'm thinking about hard reboot | 21:25 |
| bauzas | dansmith: I could just ask for the existing mdev before destroying the guest, and then pass it back to _get_guest_xml() like I said | 21:26 |
| *** nicolasbock has quit IRC | 21:26 | |
| *** fragatina has quit IRC | 21:26 | |
| bauzas | dansmith: but then, that means that for example, if we have a lot of months in between stopping the instance and restarting it, then we won't verify if that's still okay | 21:27 |
| bauzas | the reason actually why we try to retry the xml | 21:28 |
| bauzas | and not just reusing it | 21:28 |
| bauzas | if we agree with that, fine | 21:28 |
| bauzas | if we don't want that, then we would need to pass again the allocation | 21:28 |
| bauzas | thoughts ? | 21:28 |
| sean-k-mooney | jaypipes: bauzas melwitt dansmith when ye get a chance can ye take a look at https://review.openstack.org/#/c/533766/ to unblock https://review.openstack.org/#/c/532987/ | 21:29 |
| bauzas | sean-k-mooney: easy +2 | 21:29 |
| bauzas | oh wait | 21:30 |
| bauzas | sean-k-mooney: what is the min os-vif version for Nova ? | 21:30 |
| *** pcaruana has quit IRC | 21:30 | |
| sean-k-mooney | currently 1.6 i think let me check | 21:30 |
| *** dave-mccowan has joined #openstack-nova | 21:31 | |
| sean-k-mooney | 1.7.0 | 21:31 |
| *** jappleii__ has joined #openstack-nova | 21:32 | |
| *** dgonzalez has quit IRC | 21:33 | |
| *** bigdogstl has joined #openstack-nova | 21:33 | |
| bauzas | sean-k-mooney: -1 because https://github.com/openstack/os-vif/blob/1.7.0/os_vif/objects/vif.py | 21:34 |
| jaypipes | why do we have open-ended dependencies like >=1.7.0 again? :( | 21:34 |
| sean-k-mooney | well we have a catch 22 then | 21:35 |
| sean-k-mooney | the change that increases the upper constriants need this change | 21:35 |
| sean-k-mooney | jaypipes: they are closed by upper-constratints | 21:36 |
| bauzas | sean-k-mooney: before merging the nova change, we should IMHO first bump the min version | 21:36 |
| bauzas | sean-k-mooney: is g-r accepting >= 1.8.0 ? | 21:36 |
| bauzas | if so, just change directly nova requirements.txt | 21:36 |
| bauzas | in the same change | 21:37 |
| bauzas | if not, you need first to modify g-r | 21:37 |
| sean-k-mooney | bauzas: no g-r is os-vif>=1.7.0 u-c is os-vif===1.7.0 | 21:37 |
| sean-k-mooney | we cant increase g-r without first increaseing u-c and u-c need that patch | 21:38 |
| *** bigdogstl has quit IRC | 21:38 | |
| *** dgonzalez has joined #openstack-nova | 21:42 | |
| bauzas | sean-k-mooney: mmm, looks like a Depends-On change then | 21:42 |
| sean-k-mooney | the depends-on is currently from https://review.openstack.org/#/c/532987/ to https://review.openstack.org/#/c/533766/2 | 21:43 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Refactor _build_device_metadata https://review.openstack.org/533804 | 21:43 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Consider hostdev devices when building metadata https://review.openstack.org/533805 | 21:43 |
| bauzas | sean-k-mooney: I guess the gate is not accepting to upgrade upper-constraints because it tests nova ? | 21:43 |
| sean-k-mooney | yep | 21:43 |
| bauzas | shit | 21:43 |
| *** lbragstad has quit IRC | 21:44 | |
| sean-k-mooney | we could pin os-vif in nova then increase the upperconstraints then fix the test and bump minium | 21:44 |
| sean-k-mooney | and unpin | 21:44 |
| bauzas | sean-k-mooney: I think we can possibly ask nova to bump os-vif to 1.8.0 and in the meantime have the g-r change that bumps both global-reqs.txt and upper-constraints.txt | 21:45 |
| bauzas | since there will be a depends-on, that should gate correctly | 21:45 |
| bauzas | but I'm not a requirements expert | 21:46 |
| bauzas | tonyb: around ? | 21:46 |
| bauzas | tonyb: questions for a requirements upgrade loop | 21:46 |
| sean-k-mooney | the change that fixes the tests for 1.8 also works on 1.7 because of weird ness with ovo hence why the patch is currently passing zuul without using os-vif 1.8.0 | 21:46 |
| *** lbragstad has joined #openstack-nova | 21:47 | |
| *** matrohon has quit IRC | 21:48 | |
| bauzas | sean-k-mooney: no I think the nova change uses 1.8.0 | 21:48 |
| bauzas | sean-k-mooney: because pypi will give you the last version | 21:48 |
| sean-k-mooney | basically https://github.com/openstack/nova/blob/6a5673a16f8329007880b67a6dbecbe62ac368d7/nova/tests/unit/network/test_os_vif_util.py#L34-L38 means that the extra datapath_type field is discarded | 21:48 |
| sean-k-mooney | bauzas: no i checked locally it was using 1.7.0 | 21:49 |
| sean-k-mooney | let me check the test logs | 21:49 |
| bauzas | oh fun, you're right http://logs.openstack.org/66/533766/2/check/openstack-tox-py27/45f0378/job-output.txt.gz#_2018-01-15_20_03_47_088918 | 21:50 |
| *** AlexeyAbashkin has joined #openstack-nova | 21:50 | |
| *** chyka_ has joined #openstack-nova | 21:50 | |
| sean-k-mooney | yep so normally this would be a pain in the ass and we would have to bump the minium but in this case we dont | 21:51 |
| bauzas | :/ | 21:51 |
| bauzas | wait | 21:51 |
| bauzas | that works because it's using o.vo so maybe it's just downgrading the version | 21:52 |
| bauzas | I mean the object version | 21:52 |
| bauzas | hence why it works | 21:52 |
| *** chyka has quit IRC | 21:53 | |
| sean-k-mooney | well im adding a new filed to the expected value so use if it downgraded it would discard it but os-vif 1.7.0 does not have explcit down grade logic for 1.8.0 objects | 21:53 |
| *** AlexeyAbashkin has quit IRC | 21:54 | |
| sean-k-mooney | bauzas: basically it works because of this https://github.com/openstack/oslo.versionedobjects/blob/1.18.0/oslo_versionedobjects/base.py#L514-L520 | 21:54 |
| sean-k-mooney | obj_to_primitive loops over the ovo fields and uses that to generate the dictionary retrunt by obj_to_primitive | 21:55 |
| bauzas | sean-k-mooney: right, it works thanks of the ovo framework, that's it | 21:56 |
| bauzas | soooooo... | 21:56 |
| sean-k-mooney | since datapath_type is not in the 1.7.0 field list it just gets discared | 21:56 |
| bauzas | what to do ? | 21:56 |
| bauzas | I think I'll +2 | 21:56 |
| bauzas | it's harmless | 21:56 |
| bauzas | and unblocks things | 21:56 |
| sean-k-mooney | out side of unit tests we have defencive logic which will only populate the datapath_type if it is defined in the vif object | 21:57 |
| sean-k-mooney | and in os-vif we also default to the original hardcoded values if the datapath_type is not set in the profile | 21:58 |
| sean-k-mooney | so in real code this is safe | 21:58 |
| *** bigdogstl has joined #openstack-nova | 21:58 | |
| bauzas | right | 21:58 |
| sean-k-mooney | anyway its almost 10 so its 2 hours past when i should have gone home. catch ye tommorow o/ | 21:59 |
| *** archit has quit IRC | 22:00 | |
| bauzas | \o | 22:00 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: _get_trees_matching_all() https://review.openstack.org/531512 | 22:01 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add tests for _get_trees_matching_all() with trait https://review.openstack.org/531899 | 22:01 |
| *** dillaman has joined #openstack-nova | 22:01 | |
| *** bigdogstl has quit IRC | 22:03 | |
| *** matrohon has joined #openstack-nova | 22:03 | |
| *** bigdogstl has joined #openstack-nova | 22:09 | |
| *** jackie-truong has joined #openstack-nova | 22:09 | |
| *** suresh12 has joined #openstack-nova | 22:09 | |
| *** lbragstad has quit IRC | 22:10 | |
| *** mvk has joined #openstack-nova | 22:11 | |
| *** suresh12 has quit IRC | 22:16 | |
| *** fragatina has joined #openstack-nova | 22:16 | |
| *** lbragstad has joined #openstack-nova | 22:19 | |
| *** matrohon has quit IRC | 22:21 | |
| *** rcernin has joined #openstack-nova | 22:21 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_traits_for_provider https://review.openstack.org/532564 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Track associated sharing RPs in report client https://review.openstack.org/526539 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.snapshot => ProviderSnapshot https://review.openstack.org/533244 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_aggregates_for_provider https://review.openstack.org/533808 | 22:22 |
| efried | jaypipes I removed that 'generation' param in the snapshot patch (where I also cleaned up the other _Provider-accessing cheats). | 22:23 |
| *** bigdogstl has quit IRC | 22:23 | |
| efried | jaypipes If that's too messy for ya, I'll reshuffle the whole series. | 22:23 |
| *** slaweq has quit IRC | 22:24 | |
| *** smatzek has quit IRC | 22:24 | |
| *** slaweq has joined #openstack-nova | 22:24 | |
| tonyb | bauzas: you need 1 review to bump u-c; a second that bumps g-r and have the nova change gate on that 2nd chnage ... which I think is basically what you said | 22:27 |
| *** slaweq has quit IRC | 22:29 | |
| *** bigdogstl has joined #openstack-nova | 22:30 | |
| *** jackie-truong has quit IRC | 22:33 | |
| *** felipemonteiro__ has quit IRC | 22:33 | |
| bauzas | tonyb: the problem is that it seems the u-c change is gated by nova which needs the test modification | 22:34 |
| bauzas | as per sean-k-mooney | 22:35 |
| *** moshele has joined #openstack-nova | 22:37 | |
| efried | jaypipes I'm not convinced that ProviderTree.delta(other_provider_tree) is the way to go, because whatever that guy returns is going to be an ugly blob of unstructured data. I was thinking to do the diff-and-update logic in a single method either in report client or resource tracker. | 22:38 |
| *** slaweq has joined #openstack-nova | 22:39 | |
| *** bigdogstl has quit IRC | 22:39 | |
| *** suresh12 has joined #openstack-nova | 22:40 | |
| *** lyan has quit IRC | 22:43 | |
| *** slaweq has quit IRC | 22:44 | |
| *** tidwellr has quit IRC | 22:44 | |
| *** fragatina has quit IRC | 22:45 | |
| *** suresh12 has quit IRC | 22:45 | |
| *** moshele has quit IRC | 22:48 | |
| *** bigdogstl has joined #openstack-nova | 22:51 | |
| *** markvoelker has joined #openstack-nova | 22:53 | |
| *** chyka_ has quit IRC | 22:55 | |
| *** chyka has joined #openstack-nova | 22:56 | |
| *** eharney has quit IRC | 22:57 | |
| *** markvoelker has quit IRC | 22:58 | |
| *** john51 has quit IRC | 22:59 | |
| *** lbragstad has quit IRC | 23:00 | |
| *** john51 has joined #openstack-nova | 23:00 | |
| *** bigdogstl has quit IRC | 23:01 | |
| *** john51 has quit IRC | 23:02 | |
| *** hieulq has quit IRC | 23:02 | |
| *** hieulq has joined #openstack-nova | 23:03 | |
| *** john51 has joined #openstack-nova | 23:05 | |
| *** esberglu has quit IRC | 23:10 | |
| *** esberglu has joined #openstack-nova | 23:11 | |
| *** esberglu has quit IRC | 23:16 | |
| *** cdent has quit IRC | 23:17 | |
| *** bigdogstl has joined #openstack-nova | 23:17 | |
| jaypipes | efried: I'd be fine with that (single method diff-and-update thing) | 23:20 |
| efried | jaypipes coo, working on it. | 23:20 |
| jaypipes | efried: and will re-review your bottom series tomrrow morn. | 23:20 |
| efried | ack | 23:21 |
| jaypipes | efried: signing off for the night. | 23:21 |
| efried | sleep tight | 23:21 |
| jaypipes | danke | 23:21 |
| *** takashin has joined #openstack-nova | 23:22 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | 23:25 |
| *** bigdogstl has quit IRC | 23:25 | |
| *** esberglu has joined #openstack-nova | 23:27 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818 | 23:27 |
| *** esberglu has quit IRC | 23:28 | |
| *** hongbin has quit IRC | 23:28 | |
| gmann | ameeda: sure, ll check that | 23:29 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 23:30 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.snapshot => ProviderSnapshot https://review.openstack.org/533244 | 23:30 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 23:31 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.update_from_provider_tree https://review.openstack.org/533821 | 23:31 |
| *** itlinux has quit IRC | 23:33 | |
| *** r-daneel has quit IRC | 23:33 | |
| *** bigdogstl has joined #openstack-nova | 23:40 | |
| *** chyka_ has joined #openstack-nova | 23:44 | |
| *** nicolasbock has joined #openstack-nova | 23:47 | |
| *** chyka has quit IRC | 23:49 | |
| *** suresh12 has joined #openstack-nova | 23:51 | |
| *** lbragstad has joined #openstack-nova | 23:53 | |
| *** armax has quit IRC | 23:54 | |
| *** bigdogstl has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!