| *** germs has quit IRC | 00:01 | |
| *** jistr has quit IRC | 00:02 | |
| *** jistr has joined #openstack-nova | 00:03 | |
| *** odyssey4me has quit IRC | 00:14 | |
| *** odyssey4me has joined #openstack-nova | 00:14 | |
| *** hoangcx has quit IRC | 00:17 | |
| *** hoangcx has joined #openstack-nova | 00:18 | |
| *** itlinux has joined #openstack-nova | 00:23 | |
| *** jroll has quit IRC | 00:24 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fix validation for command arguments https://review.openstack.org/558324 | 00:26 |
|---|---|---|
| *** liuzz_ has joined #openstack-nova | 00:29 | |
| *** liuzz has quit IRC | 00:29 | |
| *** sree has joined #openstack-nova | 00:29 | |
| *** sree has quit IRC | 00:34 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fix AttributeError in getting a resource ID https://review.openstack.org/558328 | 00:43 |
| *** fragatina has quit IRC | 00:49 | |
| *** fragatina has joined #openstack-nova | 00:50 | |
| *** fragatina has quit IRC | 00:50 | |
| *** annp has quit IRC | 00:50 | |
| *** annp has joined #openstack-nova | 00:51 | |
| *** yamamoto has joined #openstack-nova | 00:53 | |
| *** yamamoto has quit IRC | 00:59 | |
| *** markvoelker_ has joined #openstack-nova | 01:02 | |
| *** hongbin has joined #openstack-nova | 01:05 | |
| *** gouthamr has joined #openstack-nova | 01:06 | |
| *** markvoelker has quit IRC | 01:06 | |
| *** gjayavelu has quit IRC | 01:07 | |
| *** liuzz_ has quit IRC | 01:07 | |
| *** phuongnh has joined #openstack-nova | 01:10 | |
| *** annp has quit IRC | 01:12 | |
| *** annp has joined #openstack-nova | 01:13 | |
| *** gyankum has joined #openstack-nova | 01:14 | |
| *** zhaochao has joined #openstack-nova | 01:17 | |
| *** tiendc has joined #openstack-nova | 01:17 | |
| *** licanwei has joined #openstack-nova | 01:20 | |
| *** salv-orl_ has quit IRC | 01:21 | |
| *** salv-orlando has joined #openstack-nova | 01:22 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 01:31 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 01:33 |
| *** yangyapeng has joined #openstack-nova | 01:34 | |
| *** felipemonteiro__ has joined #openstack-nova | 01:36 | |
| *** gouthamr has quit IRC | 01:37 | |
| *** yangyapeng has quit IRC | 01:39 | |
| *** sapd has joined #openstack-nova | 01:42 | |
| *** gouthamr has joined #openstack-nova | 01:42 | |
| openstackgerrit | Merged openstack/nova master: doc: Upgrade placement first https://review.openstack.org/556631 | 01:48 |
| openstackgerrit | Merged openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | 01:48 |
| *** harlowja has quit IRC | 01:50 | |
| *** fragatin_ has joined #openstack-nova | 01:51 | |
| *** yamamoto has joined #openstack-nova | 01:55 | |
| *** germs has joined #openstack-nova | 01:57 | |
| *** germs has quit IRC | 01:57 | |
| *** germs has joined #openstack-nova | 01:57 | |
| *** Tom-Tom has joined #openstack-nova | 01:57 | |
| *** amodi has quit IRC | 01:58 | |
| *** yamamoto has quit IRC | 02:00 | |
| *** germs has quit IRC | 02:01 | |
| *** yangyapeng has joined #openstack-nova | 02:02 | |
| *** AlexeyAbashkin has joined #openstack-nova | 02:02 | |
| *** gouthamr_ has joined #openstack-nova | 02:04 | |
| *** liuzz has joined #openstack-nova | 02:05 | |
| *** yangyapeng has quit IRC | 02:06 | |
| *** AlexeyAbashkin has quit IRC | 02:06 | |
| *** gouthamr has quit IRC | 02:08 | |
| *** yangyapeng has joined #openstack-nova | 02:08 | |
| *** yangyapeng has quit IRC | 02:13 | |
| *** zhenguo has joined #openstack-nova | 02:18 | |
| *** fragatin_ has quit IRC | 02:20 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix the request context in ServiceFixture https://review.openstack.org/558160 | 02:20 |
| *** yangyapeng has joined #openstack-nova | 02:21 | |
| *** yangyapeng has quit IRC | 02:25 | |
| *** sapcc-bot has quit IRC | 02:26 | |
| *** sapcc-bot has joined #openstack-nova | 02:26 | |
| *** armaan_ has quit IRC | 02:30 | |
| *** armaan has joined #openstack-nova | 02:31 | |
| *** harlowja has joined #openstack-nova | 02:31 | |
| *** psachin has joined #openstack-nova | 02:36 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:41 | |
| *** mlavalle has joined #openstack-nova | 02:44 | |
| *** bhagyashris has quit IRC | 02:46 | |
| *** bhagyashris has joined #openstack-nova | 02:48 | |
| *** mlavalle has quit IRC | 02:49 | |
| *** zhurong has joined #openstack-nova | 02:55 | |
| *** yamamoto has joined #openstack-nova | 02:56 | |
| *** fragatina has joined #openstack-nova | 02:59 | |
| *** gouthamr_ has quit IRC | 03:02 | |
| *** yamamoto has quit IRC | 03:02 | |
| *** gyan_ has joined #openstack-nova | 03:04 | |
| *** gyankum has quit IRC | 03:04 | |
| *** AlexeyAbashkin has joined #openstack-nova | 03:04 | |
| *** gyan__ has joined #openstack-nova | 03:05 | |
| *** gyan_ has quit IRC | 03:09 | |
| *** AlexeyAbashkin has quit IRC | 03:09 | |
| *** jmlowe has quit IRC | 03:09 | |
| *** sapcc-bot has quit IRC | 03:10 | |
| *** sapcc-bot has joined #openstack-nova | 03:13 | |
| *** links has joined #openstack-nova | 03:31 | |
| *** ratailor has joined #openstack-nova | 03:33 | |
| *** ratailor has quit IRC | 03:33 | |
| *** yamamoto has joined #openstack-nova | 03:33 | |
| *** yangyapeng has joined #openstack-nova | 03:33 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145 | 03:38 |
| *** yangyapeng has quit IRC | 03:38 | |
| *** nicolasbock has quit IRC | 03:40 | |
| *** jmlowe has joined #openstack-nova | 03:41 | |
| *** gjayavelu has joined #openstack-nova | 03:42 | |
| *** felipemonteiro__ has quit IRC | 03:43 | |
| *** zhurong has quit IRC | 03:52 | |
| *** udesale has joined #openstack-nova | 03:53 | |
| *** phuongnh has quit IRC | 03:55 | |
| *** tiendc has quit IRC | 03:55 | |
| *** hoangcx has quit IRC | 03:55 | |
| *** harlowja has quit IRC | 03:55 | |
| *** phuongnh has joined #openstack-nova | 03:56 | |
| *** tiendc has joined #openstack-nova | 03:56 | |
| *** hoangcx has joined #openstack-nova | 03:56 | |
| *** hongbin has quit IRC | 03:56 | |
| *** annp has quit IRC | 03:57 | |
| *** germs has joined #openstack-nova | 03:57 | |
| *** germs has quit IRC | 03:57 | |
| *** germs has joined #openstack-nova | 03:58 | |
| *** annp has joined #openstack-nova | 03:58 | |
| *** germs has quit IRC | 04:02 | |
| *** ratailor has joined #openstack-nova | 04:04 | |
| *** Tom-Tom has quit IRC | 04:11 | |
| *** markvoelker_ has quit IRC | 04:16 | |
| *** yassine has quit IRC | 04:18 | |
| *** fragatina has quit IRC | 04:20 | |
| *** fragatina has joined #openstack-nova | 04:20 | |
| *** zhenguo has quit IRC | 04:27 | |
| *** jichen has joined #openstack-nova | 04:30 | |
| *** Tom-Tom has joined #openstack-nova | 04:30 | |
| *** yassine has joined #openstack-nova | 04:33 | |
| *** Tom-Tom has quit IRC | 04:35 | |
| *** pmannidi has joined #openstack-nova | 04:37 | |
| *** andreas_s has joined #openstack-nova | 04:39 | |
| *** moshele has joined #openstack-nova | 04:41 | |
| *** andreas_s has quit IRC | 04:43 | |
| *** yamamoto has quit IRC | 04:45 | |
| *** yamamoto has joined #openstack-nova | 04:46 | |
| *** abhishekk has joined #openstack-nova | 04:47 | |
| *** abhishekk is now known as akekane|wfh | 04:47 | |
| *** akekane|wfh is now known as abhishekk | 04:47 | |
| *** lpetrut has joined #openstack-nova | 04:50 | |
| *** jaosorior has joined #openstack-nova | 04:57 | |
| *** yangyapeng has joined #openstack-nova | 04:59 | |
| *** Dinesh_Bhor has quit IRC | 04:59 | |
| *** yangyapeng has quit IRC | 05:03 | |
| *** yangyapeng has joined #openstack-nova | 05:04 | |
| *** moshele has quit IRC | 05:05 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: (WIP)xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041 | 05:07 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154 | 05:16 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 05:18 |
| *** abhishekk_ has joined #openstack-nova | 05:21 | |
| *** abhishekk has quit IRC | 05:23 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 05:24 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: (WIP)xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041 | 05:24 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 05:25 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 05:25 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 05:27 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 05:27 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 05:27 |
| *** Tom-Tom has joined #openstack-nova | 05:34 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Replace GB with GiB https://review.openstack.org/558364 | 05:34 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Replace GB with GiB https://review.openstack.org/558364 | 05:36 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041 | 05:36 |
| *** abhishekk_ is now known as abhishekk | 05:36 | |
| *** Eran_Kuris has joined #openstack-nova | 05:44 | |
| *** naichuans has joined #openstack-nova | 05:44 | |
| *** lpetrut has quit IRC | 05:48 | |
| *** lpetrut has joined #openstack-nova | 05:48 | |
| *** lpetrut has quit IRC | 05:54 | |
| *** germs has joined #openstack-nova | 05:58 | |
| *** germs has quit IRC | 05:58 | |
| *** germs has joined #openstack-nova | 05:58 | |
| *** germs has quit IRC | 06:02 | |
| *** damien_r has joined #openstack-nova | 06:03 | |
| *** sridharg has joined #openstack-nova | 06:07 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:07 | |
| *** yangyapeng has quit IRC | 06:11 | |
| *** yangyapeng has joined #openstack-nova | 06:11 | |
| *** kholkina has joined #openstack-nova | 06:11 | |
| *** Kumar has joined #openstack-nova | 06:12 | |
| *** sar has joined #openstack-nova | 06:13 | |
| *** yangyapeng has quit IRC | 06:15 | |
| *** markvoelker has joined #openstack-nova | 06:17 | |
| *** yangyapeng has joined #openstack-nova | 06:18 | |
| *** sahid has joined #openstack-nova | 06:20 | |
| *** fragatina has quit IRC | 06:20 | |
| *** moshele has joined #openstack-nova | 06:20 | |
| *** fragatina has joined #openstack-nova | 06:20 | |
| *** lpetrut has joined #openstack-nova | 06:21 | |
| *** Kumar has quit IRC | 06:29 | |
| *** Tom-Tom_ has joined #openstack-nova | 06:30 | |
| *** damien_r has quit IRC | 06:30 | |
| *** damien_r has joined #openstack-nova | 06:31 | |
| *** yangyapeng has quit IRC | 06:31 | |
| *** lpetrut has quit IRC | 06:33 | |
| *** Tom-Tom has quit IRC | 06:33 | |
| *** lajoskatona has joined #openstack-nova | 06:34 | |
| *** namnh has joined #openstack-nova | 06:34 | |
| *** yangyapeng has joined #openstack-nova | 06:35 | |
| *** damien_r has quit IRC | 06:37 | |
| *** tiendc has quit IRC | 06:41 | |
| *** markvoelker has quit IRC | 06:51 | |
| *** armaan_ has joined #openstack-nova | 06:53 | |
| *** armaan has quit IRC | 06:53 | |
| *** pcaruana has joined #openstack-nova | 06:53 | |
| *** fragatin_ has joined #openstack-nova | 06:54 | |
| *** fragatina has quit IRC | 06:54 | |
| *** fragatin_ has quit IRC | 06:54 | |
| *** fragatina has joined #openstack-nova | 06:55 | |
| *** voelzmo has joined #openstack-nova | 06:57 | |
| *** belmoreira has joined #openstack-nova | 06:58 | |
| *** tesseract has joined #openstack-nova | 07:00 | |
| *** damien_r has joined #openstack-nova | 07:00 | |
| *** afazekas has quit IRC | 07:01 | |
| *** afazekas has joined #openstack-nova | 07:01 | |
| *** gyan_ has joined #openstack-nova | 07:04 | |
| *** imacdonn has quit IRC | 07:04 | |
| *** imacdonn has joined #openstack-nova | 07:04 | |
| *** gyan__ has quit IRC | 07:04 | |
| *** links has quit IRC | 07:05 | |
| *** links has joined #openstack-nova | 07:05 | |
| *** gyan__ has joined #openstack-nova | 07:05 | |
| *** diga has joined #openstack-nova | 07:08 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: trivial: Fix nits in code comments https://review.openstack.org/558133 | 07:08 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Complement tests in allocation candidates https://review.openstack.org/558135 | 07:08 |
| *** pmannidi has quit IRC | 07:08 | |
| *** gyan_ has quit IRC | 07:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | 07:11 |
| *** gjayavelu has quit IRC | 07:13 | |
| *** voelzmo has quit IRC | 07:16 | |
| *** tuanla____ has joined #openstack-nova | 07:17 | |
| *** diga has quit IRC | 07:19 | |
| *** josecastroleon has joined #openstack-nova | 07:22 | |
| *** tssurya has joined #openstack-nova | 07:23 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154 | 07:27 |
| openstackgerrit | Fan Zhang proposed openstack/nova master: [Trivial]Add missing blank space in conf description https://review.openstack.org/558414 | 07:28 |
| *** ccamacho has joined #openstack-nova | 07:30 | |
| *** amoralej|off is now known as amoralej | 07:34 | |
| *** moshele has quit IRC | 07:35 | |
| *** jpena|off is now known as jpena | 07:37 | |
| *** voelzmo has joined #openstack-nova | 07:38 | |
| *** jaosorior has quit IRC | 07:38 | |
| *** andreas_s has joined #openstack-nova | 07:39 | |
| *** yamamoto_ has joined #openstack-nova | 07:42 | |
| *** andreas_s has quit IRC | 07:44 | |
| *** jaosorior has joined #openstack-nova | 07:44 | |
| *** yamamoto has quit IRC | 07:45 | |
| *** markvoelker has joined #openstack-nova | 07:47 | |
| *** voelzmo has quit IRC | 07:50 | |
| *** alexchadin has joined #openstack-nova | 07:51 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:53 | |
| *** hoangcx has quit IRC | 07:59 | |
| *** germs has joined #openstack-nova | 07:59 | |
| *** germs has quit IRC | 07:59 | |
| *** germs has joined #openstack-nova | 07:59 | |
| *** tetsuro has joined #openstack-nova | 08:02 | |
| tetsuro | @efied(fried_bunny), jaypipes, edleafe, alex_xu_ : I read through the IRC log in Monday and evacuated https://review.openstack.org/558133 , and https://review.openstack.org/558135 from the “anchor” branch since they can be merged now. | 08:02 |
| tetsuro | But I think exposing unrequested resources in prov_sum is necessary change for NRP cases, too, if we are going to expose all the NRPs in the tree. | 08:03 |
| *** rubasov has quit IRC | 08:03 | |
| tetsuro | Without this change, how do NRPs that don’t have requested resources appear in prov_sum? Are we going to expose them without resource information? | 08:03 |
| *** germs has quit IRC | 08:03 | |
| tetsuro | Could you tell me why we need all the NRPs in the tree? If there aren’t specific and concrete usecase so far, I even don’t think we have to support that all NRP in prov_sum since the design depends on the usecase. | 08:04 |
| tetsuro | I’m also asking the same question in https://review.openstack.org/#/c/556873/. | 08:04 |
| *** links has quit IRC | 08:04 | |
| *** ccamacho has quit IRC | 08:05 | |
| *** mdnadeem has joined #openstack-nova | 08:08 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] api-ref: Fix parameters https://review.openstack.org/558142 | 08:10 |
| *** lucas-pto is now known as lucasagomes | 08:11 | |
| *** links has joined #openstack-nova | 08:14 | |
| *** gcb has quit IRC | 08:14 | |
| *** zhurong has joined #openstack-nova | 08:16 | |
| *** hiro-kobayashi has joined #openstack-nova | 08:16 | |
| *** avolkov has joined #openstack-nova | 08:18 | |
| *** brault has quit IRC | 08:20 | |
| *** andreas_s has joined #openstack-nova | 08:20 | |
| *** voelzmo has joined #openstack-nova | 08:20 | |
| *** markvoelker has quit IRC | 08:21 | |
| *** FL1SK has quit IRC | 08:22 | |
| *** gcb has joined #openstack-nova | 08:22 | |
| *** rubasov has joined #openstack-nova | 08:22 | |
| *** ccamacho has joined #openstack-nova | 08:23 | |
| *** voelzmo has quit IRC | 08:25 | |
| *** ralonsoh has joined #openstack-nova | 08:29 | |
| bauzas | good morning Novaers | 08:29 |
| *** gcb has quit IRC | 08:30 | |
| *** voelzmo has joined #openstack-nova | 08:30 | |
| *** mgoddard has joined #openstack-nova | 08:31 | |
| *** lpetrut has joined #openstack-nova | 08:35 | |
| fanzhang | oh hi bauzas, sorry to bother. I accidentally found this trivial bug, https://review.openstack.org/#/c/558414/, should I add CI manually to reviewer or just wait? Kind a new to post patch to nova. :) | 08:35 |
| bauzas | fanzhang: in general, we try to avoid small nit fixes like this one | 08:36 |
| fanzhang | bauzas yeah, I do think it's not appropriate. | 08:37 |
| bauzas | fanzhang: to answer your original question, that change is in the queue http://zuul.openstack.org/ | 08:37 |
| bauzas | I can see it there in the check pipeline | 08:38 |
| bauzas | so you just need to wait until all the jobs are done so that you'd get a Zuul review | 08:38 |
| bauzas | fanzhang: you can have some docs in https://docs.openstack.org/infra/manual/developers.html#automated-testing | 08:39 |
| fanzhang | bauzas oh, cool. Thanks. I see there are lots of CI jobs in nova. But we only have a few in trove. I thought maybe nova is not the same as trove. :) | 08:40 |
| *** dklyle has quit IRC | 08:40 | |
| kashyap | alex_xu_: Thanks for reviewing this! https://review.openstack.org/#/c/534384/ | 08:41 |
| *** dklyle has joined #openstack-nova | 08:41 | |
| fanzhang | bauzas as for the nit fix above, should I abandon it or ? | 08:42 |
| *** gcb has joined #openstack-nova | 08:46 | |
| fanzhang | I noticed there was one trivial fix like above, see https://review.openstack.org/#/c/558183/ . So anything I should do you can just help point it out, thanks :) I do want to know how nova team try to avoid small nit fixes. | 08:47 |
| *** damien_r has quit IRC | 08:47 | |
| johnthetubaguy | fanzhang: we have a list of "trival bug fixes" on this etherpad: https://etherpad.openstack.org/p/rocky-nova-priorities-tracking | 08:51 |
| *** Dinesh_Bhor has quit IRC | 08:55 | |
| *** claudiub has joined #openstack-nova | 08:58 | |
| fanzhang | johnthetubaguy thanks, looks great :) | 08:58 |
| *** yangyapeng has quit IRC | 08:59 | |
| kashyap | Is this the magic incantation to "recheck" Citrix XenServer CI? -- 'xenserver: recheck' | 08:59 |
| * kashyap just tries it | 09:01 | |
| kashyap | johnthetubaguy: Meanwhile; this is ready, also has a +2: https://review.openstack.org/#/c/534384/22 | 09:01 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 09:01 |
| *** namnh has quit IRC | 09:04 | |
| *** tuanla____ has quit IRC | 09:04 | |
| *** annp has quit IRC | 09:04 | |
| *** namnh has joined #openstack-nova | 09:05 | |
| *** tuanla____ has joined #openstack-nova | 09:05 | |
| *** annp has joined #openstack-nova | 09:05 | |
| openstackgerrit | Merged openstack/nova master: Fix a missing white space in exception message https://review.openstack.org/558183 | 09:08 |
| openstackgerrit | Merged openstack/os-vif stable/pike: ovs: do not delete port if already exists https://review.openstack.org/550080 | 09:10 |
| *** sambetts|afk is now known as sambetts | 09:14 | |
| *** damien_r has joined #openstack-nova | 09:15 | |
| *** voelzmo has quit IRC | 09:16 | |
| *** voelzmo has joined #openstack-nova | 09:16 | |
| *** damien_r has quit IRC | 09:17 | |
| johnthetubaguy | kashyap: ah, good call out, will take another peak at that ASAP | 09:17 |
| *** damien_r has joined #openstack-nova | 09:17 | |
| *** markvoelker has joined #openstack-nova | 09:18 | |
| *** hiro-kobayashi has quit IRC | 09:19 | |
| *** aloga has quit IRC | 09:21 | |
| *** gcb has quit IRC | 09:21 | |
| *** aloga has joined #openstack-nova | 09:21 | |
| *** gcb has joined #openstack-nova | 09:27 | |
| *** armaan_ has quit IRC | 09:31 | |
| *** armaan has joined #openstack-nova | 09:31 | |
| *** voelzmo has quit IRC | 09:32 | |
| *** voelzmo has joined #openstack-nova | 09:32 | |
| *** voelzmo has quit IRC | 09:33 | |
| *** voelzmo has joined #openstack-nova | 09:33 | |
| *** voelzmo has quit IRC | 09:33 | |
| *** armaan has quit IRC | 09:36 | |
| *** voelzmo has joined #openstack-nova | 09:36 | |
| *** voelzmo has quit IRC | 09:37 | |
| *** sdague has joined #openstack-nova | 09:37 | |
| *** voelzmo has joined #openstack-nova | 09:37 | |
| *** voelzmo has quit IRC | 09:37 | |
| *** damien_r1 has joined #openstack-nova | 09:40 | |
| kashyap | Splendid! | 09:41 |
| *** damien_r has quit IRC | 09:42 | |
| kashyap | johnthetubaguy: Oh, darn, so it is the spacing that is causing the output to be all mangled? -- https://review.openstack.org/#/c/534384/22/nova/conf/libvirt.py@546 | 09:43 |
| kashyap | Good catch! | 09:43 |
| *** alexchadin has quit IRC | 09:45 | |
| kashyap | johnthetubaguy: Any suggestion to fix that? As it stands, indentation doesn't render | 09:45 |
| kashyap | stephenfin: You about? | 09:45 |
| stephenfin | kashyap: I am | 09:45 |
| kashyap | stephenfin: When you are: I wonder if you have any tricks to get this right: https://review.openstack.org/#/c/534384/22/nova/conf/libvirt.py@546 | 09:45 |
| kashyap | See the rendering, the indentation is gone, and the config is highlighted: http://logs.openstack.org/84/534384/22/check/build-openstack-sphinx-docs/016a796/html/configuration/config.html | 09:46 |
| johnthetubaguy | kashyap: some of it used to, got standup right now, so distracted | 09:46 |
| kashyap | stephenfin: `grep` for the the string: "This allows specifying granular CPU feature flags when specifying" to arrive at the right place in the HTML | 09:46 |
| johnthetubaguy | kashyap: hmm, looks like what I had working is totally broken again | 09:46 |
| stephenfin | kashyap: It's a bug in oslo_config.sphinxext https://review.openstack.org/#/q/status:open+project:openstack/oslo.config+branch:master+topic:bug/1755783 | 09:46 |
| johnthetubaguy | "* ..." and "\n" used to render OK, but its a bit broken right now | 09:46 |
| kashyap | stephenfin: Ah, interesting | 09:47 |
| stephenfin | kashyap: ...which should be fixed soon as those patches are merged. This is the one you care about https://review.openstack.org/553860 | 09:47 |
| kashyap | stephenfin: So I'll add a pointer to your patch series (thanks!). I don't need to make any changes yet, do I? | 09:47 |
| stephenfin | kashyap: Once that is merged though, there are a couple of things that still need changing. I'll comment now | 09:48 |
| kashyap | stephenfin: Ah, thank you. I'll subscribe to that 'oslo_config' patch series | 09:49 |
| *** jichen has quit IRC | 09:50 | |
| kashyap | stephenfin: Hmm, hope that'd be stable branches backports compatible; because I'll be backporting that change to two branches | 09:51 |
| *** yangyapeng has joined #openstack-nova | 09:52 | |
| kashyap | (As it's addressing an important guest perf regression) | 09:52 |
| stephenfin | kashyap: You're just going to lose some of the docs formatting there, I'm afraid | 09:52 |
| *** markvoelker has quit IRC | 09:52 | |
| stephenfin | It'll still render but poorly | 09:52 |
| kashyap | stephenfin: For backports, you mean? | 09:52 |
| stephenfin | Indeed | 09:52 |
| kashyap | Maybe I can adjust it according to the equivalent syntax for backports? (Assuming there is such.) | 09:53 |
| * kashyap needs to bike out AFK for lunch; bbiab | 09:53 | |
| stephenfin | kashyap: Not sure there's any way to do it but I guess it can be experimented with, aye | 09:54 |
| stephenfin | o/ | 09:54 |
| *** alexchadin has joined #openstack-nova | 09:57 | |
| *** tianhui has quit IRC | 09:58 | |
| *** takashin has left #openstack-nova | 10:00 | |
| *** Tom-Tom_ has quit IRC | 10:00 | |
| *** damien_r1 has quit IRC | 10:02 | |
| *** damien_r has joined #openstack-nova | 10:02 | |
| *** tianhui has joined #openstack-nova | 10:04 | |
| *** Tom-Tom has joined #openstack-nova | 10:07 | |
| *** voelzmo has joined #openstack-nova | 10:09 | |
| *** Tom-Tom has quit IRC | 10:12 | |
| *** phuongnh has quit IRC | 10:12 | |
| *** voelzmo has quit IRC | 10:14 | |
| *** dtantsur|afk is now known as dtantsur | 10:18 | |
| johnthetubaguy | kashyap: why did we decide to exclude host-model in that patch again? I get excluding host-passthrough. | 10:18 |
| *** cdent has joined #openstack-nova | 10:19 | |
| *** mdnadeem has quit IRC | 10:19 | |
| *** alexchadin has quit IRC | 10:20 | |
| *** mdnadeem has joined #openstack-nova | 10:21 | |
| *** mdnadeem has quit IRC | 10:21 | |
| *** mdnadeem has joined #openstack-nova | 10:21 | |
| sean-k-mooney[m] | o/ | 10:21 |
| *** zhurong has quit IRC | 10:21 | |
| *** alexchadin has joined #openstack-nova | 10:22 | |
| *** alexchadin has quit IRC | 10:22 | |
| *** alexchadin has joined #openstack-nova | 10:23 | |
| *** voelzmo has joined #openstack-nova | 10:26 | |
| *** armaan has joined #openstack-nova | 10:26 | |
| *** alexchadin has quit IRC | 10:27 | |
| *** mdbooth has joined #openstack-nova | 10:27 | |
| *** abhishekk has quit IRC | 10:28 | |
| sean-k-mooney[m] | hi wierd question but would the nova team object to me runnning a personal ci against nova that is triggred by a specific comment. i nolonger have direct control over what is run in the intel nfv ci and there are a few things i would like to test perodically that cannot currently be tested upstream. similarly there are a few things that i think can be tested upstream that i want prototype with zuul v3 in my | 10:29 |
| sean-k-mooney[m] | "spare time" before proposing them upstream. | 10:29 |
| sean-k-mooney[m] | melwitt: stephenfin dansmith jaypipes ^ | 10:30 |
| stephenfin | sean-k-mooney[m]: If it's not voting, I don't see why not | 10:30 |
| sean-k-mooney[m] | stephenfin: ya it would not set the verifed flag to +/-1 just comment back sucess/failure when triggered by a comment | 10:31 |
| sean-k-mooney[m] | i was also going to restict it to people with intel emails/the nova core team. | 10:32 |
| *** FL1SK has joined #openstack-nova | 10:32 | |
| *** zhurong has joined #openstack-nova | 10:32 | |
| *** nicolasbock has joined #openstack-nova | 10:33 | |
| sean-k-mooney[m] | i am setting up some ci infra for internal use and wantted to tag this on the side to do things like test livemigration with cpu pinning/hugepgaes/vhost-user since that is currently disabled in the intel nfv-ci | 10:33 |
| sean-k-mooney[m] | im going to follow up with them to trun that back on but you know its nice to be able to check some of those edgecases in an automated way | 10:34 |
| stephenfin | Indeed it is. That sounds like a sane thing to do from my perspective anyway | 10:34 |
| sean-k-mooney[m] | this is currently an inovation time/personal time thing so i might not have that much time to work on it but i set up nodepool over the weekend and should have zuul runing by the end of the week. with zuul v3 its not that hard to trigger build on both internal stuff and external. i was thinking of trying to add a patch work trigger to zuul too at somepoint but ya just taught i would ask. | 10:38 |
| *** namnh has quit IRC | 10:38 | |
| *** elmaciej has joined #openstack-nova | 10:43 | |
| *** Zames has joined #openstack-nova | 10:48 | |
| *** markvoelker has joined #openstack-nova | 10:48 | |
| *** tbachman has quit IRC | 10:50 | |
| *** Zames has quit IRC | 10:50 | |
| *** alexchadin has joined #openstack-nova | 10:51 | |
| *** sridharg has quit IRC | 10:53 | |
| openstackgerrit | Merged openstack/nova master: [Trivial]Add missing blank space in conf description https://review.openstack.org/558414 | 10:53 |
| *** voelzmo has quit IRC | 10:55 | |
| *** alexchadin has quit IRC | 10:56 | |
| *** yangyapeng has quit IRC | 10:56 | |
| *** alexchadin has joined #openstack-nova | 10:56 | |
| openstackgerrit | Merged openstack/nova master: Fix unit tests to work with new oslo.config https://review.openstack.org/558084 | 10:58 |
| *** voelzmo has joined #openstack-nova | 11:00 | |
| *** licanwei has quit IRC | 11:00 | |
| *** _pewp_ has quit IRC | 11:03 | |
| *** lucasagomes is now known as lucas-hungry | 11:03 | |
| *** _pewp_ has joined #openstack-nova | 11:06 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add --enable and --disable options to nova-manage update_cell https://review.openstack.org/555416 | 11:11 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update the cells FAQs and scheduler maintenance docs. https://review.openstack.org/556459 | 11:11 |
| *** Zames has joined #openstack-nova | 11:17 | |
| *** rcernin has quit IRC | 11:20 | |
| *** Zames has quit IRC | 11:20 | |
| *** markvoelker has quit IRC | 11:22 | |
| *** ratailor has quit IRC | 11:23 | |
| *** xinliang has quit IRC | 11:29 | |
| *** moshele has joined #openstack-nova | 11:31 | |
| *** jafeha has quit IRC | 11:33 | |
| *** voelzmo has quit IRC | 11:36 | |
| *** voelzmo has joined #openstack-nova | 11:37 | |
| *** voelzmo has quit IRC | 11:37 | |
| *** voelzmo has joined #openstack-nova | 11:37 | |
| *** voelzmo has quit IRC | 11:38 | |
| *** voelzmo has joined #openstack-nova | 11:39 | |
| *** voelzmo has quit IRC | 11:39 | |
| *** yamamoto_ has quit IRC | 11:39 | |
| *** voelzmo has joined #openstack-nova | 11:39 | |
| *** voelzmo has quit IRC | 11:39 | |
| *** jafeha has joined #openstack-nova | 11:40 | |
| *** xinliang has joined #openstack-nova | 11:42 | |
| *** elmaciej has quit IRC | 11:45 | |
| *** jpena is now known as jpena|lunch | 11:46 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move create_tap_dev to the new utility method. https://review.openstack.org/557835 | 11:54 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | 11:54 |
| *** lucas-hungry is now known as lucasagomes | 11:59 | |
| *** ralonsoh_ has joined #openstack-nova | 12:01 | |
| *** tuanla____ has quit IRC | 12:01 | |
| jaypipes | sean-k-mooney[m]: I have no issues with that at all. I think it would be great. | 12:03 |
| *** ralonsoh has quit IRC | 12:04 | |
| *** liverpooler has joined #openstack-nova | 12:04 | |
| *** amoralej is now known as amoralej|lunch | 12:06 | |
| *** dtantsur is now known as dtantsur|brb | 12:06 | |
| *** claudiub|2 has joined #openstack-nova | 12:08 | |
| *** tbachman has joined #openstack-nova | 12:10 | |
| *** claudiub has quit IRC | 12:11 | |
| bauzas | sahid: soft -1 for https://review.openstack.org/#/c/511188/ for asking you to be clear about all the possible values for the new option that you're going to add | 12:12 |
| *** voelzmo has joined #openstack-nova | 12:12 | |
| bauzas | sahid: once you're done, ping me and jaypipes and then I'll review it quickly | 12:13 |
| *** tbachman has quit IRC | 12:14 | |
| *** odyssey4me has quit IRC | 12:15 | |
| *** odyssey4me has joined #openstack-nova | 12:15 | |
| *** voelzmo has quit IRC | 12:17 | |
| *** markvoelker has joined #openstack-nova | 12:19 | |
| alex_xu_ | jaypipes: dansmith fried_bunny edleafe bauzas, good morning, Sundar has a summary for the fpga based on our previous discussion at https://etherpad.openstack.org/p/Cyborg-Nova-Multifunction, he is looking for some review and feedback | 12:20 |
| johnthetubaguy | stephenfin: would be great if you could double check my thinking on this runway patch: https://review.openstack.org/#/c/552242 | 12:21 |
| *** tbachman has joined #openstack-nova | 12:21 | |
| *** gyan__ has quit IRC | 12:21 | |
| *** READ10 has joined #openstack-nova | 12:21 | |
| *** psachin has quit IRC | 12:22 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Filter allocation candidates by forbidden traits in db https://review.openstack.org/556660 | 12:22 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Parse forbidden traits in query strings https://review.openstack.org/556819 | 12:22 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Support forbidden traits in API https://review.openstack.org/556820 | 12:22 |
| *** alexchadin has quit IRC | 12:23 | |
| *** markvoelker has quit IRC | 12:23 | |
| *** markvoelker has joined #openstack-nova | 12:24 | |
| openstackgerrit | sean mooney proposed openstack/nova master: add mtu to libvirt xml for ethernet and bridge types https://review.openstack.org/553072 | 12:24 |
| *** voelzmo has joined #openstack-nova | 12:25 | |
| sean-k-mooney[m] | sahid: stephenfin added teh extra unit tests to ^ and rebased it against master. otherwise its the same as the previous version. | 12:25 |
| *** artom has joined #openstack-nova | 12:27 | |
| *** tetsuro has quit IRC | 12:27 | |
| *** alexchadin has joined #openstack-nova | 12:27 | |
| *** edmondsw has quit IRC | 12:28 | |
| *** edmondsw has joined #openstack-nova | 12:28 | |
| *** voelzmo has quit IRC | 12:29 | |
| *** voelzmo has joined #openstack-nova | 12:29 | |
| *** artom has quit IRC | 12:30 | |
| *** sapcc-bot has quit IRC | 12:31 | |
| *** sapcc-bot has joined #openstack-nova | 12:31 | |
| *** sapcc-bot has quit IRC | 12:32 | |
| *** sapcc-bot has joined #openstack-nova | 12:32 | |
| *** Tom-Tom has joined #openstack-nova | 12:32 | |
| *** edmondsw has quit IRC | 12:32 | |
| fried_bunny | alex_xu_: Roger that; it's been on my list. | 12:34 |
| alex_xu_ | fried_bunny: thanks | 12:34 |
| *** fried_bunny is now known as efried | 12:35 | |
| efried | The pagan rites of spring are over. | 12:35 |
| *** Tom-Tom has quit IRC | 12:37 | |
| *** sapcc-bot has quit IRC | 12:37 | |
| *** jroll has joined #openstack-nova | 12:37 | |
| *** sapcc-bot has joined #openstack-nova | 12:38 | |
| *** mchlumsky has joined #openstack-nova | 12:39 | |
| *** naichuans has quit IRC | 12:39 | |
| *** yamamoto has joined #openstack-nova | 12:40 | |
| kashyap | johnthetubaguy: Hi there; was AFK. Will answer on the review, but to answer your question here: | 12:44 |
| kashyap | johnthetubaguy: For 'host-model' + PCID doesn't make sense in this case — If QEMU already supports PCID, it would be enabled by 'host-model'. And if it is not supported, adding it doesn't make it magically appear :-) | 12:45 |
| kashyap | johnthetubaguy: Does the above answer you? | 12:45 |
| *** yamamoto has quit IRC | 12:45 | |
| *** gouthamr has joined #openstack-nova | 12:47 | |
| *** eharney has joined #openstack-nova | 12:47 | |
| *** eharney has quit IRC | 12:47 | |
| *** sridharg has joined #openstack-nova | 12:47 | |
| *** eharney has joined #openstack-nova | 12:48 | |
| *** lpetrut has quit IRC | 12:48 | |
| kashyap | johnthetubaguy: We logged a warning, instead of an _exception_ due to the prior discussion on the review: https://review.openstack.org/#/c/534384/20/nova/virt/libvirt/driver.py | 12:49 |
| *** lyan has joined #openstack-nova | 12:50 | |
| *** udesale has quit IRC | 12:50 | |
| *** jpena|lunch is now known as jpena | 12:50 | |
| *** lbragstad has joined #openstack-nova | 12:50 | |
| *** lyan is now known as Guest9432 | 12:50 | |
| *** mchlumsky has quit IRC | 12:50 | |
| *** udesale has joined #openstack-nova | 12:50 | |
| *** mchlumsky has joined #openstack-nova | 12:52 | |
| *** lpetrut has joined #openstack-nova | 12:52 | |
| bhagyashris | johnthetubaguy, alex_xu_, melwitt: Hi, addressed review comments could you please review the same https://review.openstack.org/#/c/511825/ thank you :) | 12:56 |
| *** artom has joined #openstack-nova | 12:56 | |
| kashyap | stephenfin: Hey there, so ... some OCD discussion on captialization: | 12:58 |
| *** sapcc-bot has quit IRC | 12:58 | |
| *** edmondsw has joined #openstack-nova | 12:58 | |
| kashyap | stephenfin: On your comment here: https://review.openstack.org/#/c/534384/22/nova/conf/libvirt.py@568 | 12:58 |
| *** gouthamr has quit IRC | 12:59 | |
| *** sapcc-bot has joined #openstack-nova | 12:59 | |
| alex_xu_ | bhagyashris: johnthetubaguy melwitt, good to have agreement on this first http://lists.openstack.org/pipermail/openstack-dev/2018-March/128914.html | 12:59 |
| *** gouthamr has joined #openstack-nova | 12:59 | |
| johnthetubaguy | kashyap: I got the impression host model had PCID missing put the CPU might still support it? | 12:59 |
| kashyap | stephenfin: My rule to capitalize the sentence following a colon: if there is just _one_ sentence after colon, use lower case; otherwise upper case. | 12:59 |
| johnthetubaguy | kashyap: maybe we should add a TODO to raise once its not a backport? | 13:00 |
| kashyap | stephenfin: Hope that make sense? | 13:00 |
| kashyap | johnthetubaguy: Hi | 13:00 |
| *** sapcc-bot5 has joined #openstack-nova | 13:00 | |
| *** sapcc-bot has quit IRC | 13:00 | |
| stephenfin | kashyap: Fine by me. That should have been preceded by 'nit'. Does seem it's purely stylistic too (which I didn't know) https://www.grammarly.com/blog/capitalization-after-colons/ | 13:01 |
| kashyap | johnthetubaguy: No-no: With `host-model`, *if* you have the updated microcode, guest/host kernel, QEMU and libvirt, then libvirt will do the Right Thing. | 13:01 |
| kashyap | stephenfin: No worries, I enjoy the English nit-picks. :-) | 13:01 |
| * kashyap clicks | 13:01 | |
| *** jafeha has quit IRC | 13:02 | |
| johnthetubaguy | kashyap: ah, good. I got the wrong end of the stick about that bit then. | 13:02 |
| kashyap | johnthetubaguy: After the backports are done -- we will remove the restriction on 'host-model' | 13:02 |
| kashyap | johnthetubaguy: No worries; can you please add a follow-up comment on the review, so that we're on the same line, same word? :-) | 13:02 |
| johnthetubaguy | kashyap: so I think you need to set the extra_flags to None when you log the warning? | 13:02 |
| kashyap | johnthetubaguy: That's the other bit | 13:03 |
| kashyap | johnthetubaguy: I think you're right there | 13:03 |
| johnthetubaguy | kashyap: no worries, I think we are in sync then, I added a follow up comment | 13:03 |
| johnthetubaguy | kashyap: sorry this is dragging on, tricky to get the right balance and make it backportable, it turns out | 13:03 |
| kashyap | johnthetubaguy: I first actually raised an exception, and then Matt and Dan came along saying: maybe a warning is less "heavy-handed" | 13:04 |
| stephenfin | johnthetubaguy: Ack on https://review.openstack.org/#/c/552242/ | 13:04 |
| stephenfin | sean-k-mooney[m]: Looking | 13:04 |
| johnthetubaguy | kashyap: yeah, for the backport its a good idea | 13:04 |
| kashyap | johnthetubaguy: Yeah, would like to get this rid of my plate; getting a bit too much of it. But yes, it's a tricky change with right balance | 13:04 |
| kashyap | s/with right balance/that needs the right balance/ | 13:05 |
| johnthetubaguy | stephenfin: thanks, wanted to make sure I didn't just misread it an wait for mikal to wake up after I sign off for the day. | 13:05 |
| *** sapcc-bot5 has quit IRC | 13:06 | |
| kashyap | johnthetubaguy: On your other question about 'host-passthrough': Yes, it is invalid. As libvirt wouldn't mangle anything, and if the CPU has it, libvirt will pass it right through to the guest. | 13:06 |
| jaypipes | efried: I'm happy to do a hangout about the consumer generation today if you need to. | 13:06 |
| *** sapcc-bot has joined #openstack-nova | 13:06 | |
| efried | jaypipes: Is there email I need to catch up on first? | 13:07 |
| kashyap | johnthetubaguy: To summarize, I'll quickly add: the 'extra_flags = None', before the warning. | 13:07 |
| jaypipes | efried: no, just saying I left things yesterday afternoon in a bit of a "too tired to talk about this right now" state. | 13:07 |
| *** sapcc-bot has quit IRC | 13:07 | |
| *** sapcc-bot has joined #openstack-nova | 13:08 | |
| johnthetubaguy | kashyap: yeah, +1 | 13:08 |
| *** amoralej|lunch is now known as amoralej | 13:08 | |
| efried | jaypipes: Okay. The open issue is whether we should change the responses to PUT /alloc/{c} and POST /alloc from 204 No Content to 200 with payload. | 13:08 |
| kashyap | johnthetubaguy: Thanks for the eagle eyes. | 13:08 |
| *** pchavva has joined #openstack-nova | 13:09 | |
| jaypipes | efried: ok, I will answer that on the spec if that's what you prefer. | 13:10 |
| *** pchavva has quit IRC | 13:10 | |
| efried | jaypipes: wfm, unless it needs back-and-forth. | 13:11 |
| *** pchavva has joined #openstack-nova | 13:11 | |
| efried | cdent: Wouldn't mind an opinion from you as well. | 13:11 |
| efried | cdent: oh, looks like you rendered one. Reading... | 13:11 |
| *** sapcc-bot has quit IRC | 13:11 | |
| *** sapcc-bot has joined #openstack-nova | 13:11 | |
| kashyap | johnthetubaguy: Hmm, isn't it a bit "dirty" that we're quietly setting it to 'extra_flags' to None? Or is the onus is on the Operator to read the docs to see exactly what is valid? | 13:12 |
| johnthetubaguy | kashyap: its not great, but I think we just change this to a raise in the follow on patch that isn't backported | 13:12 |
| *** trozet has joined #openstack-nova | 13:13 | |
| *** jafeha has joined #openstack-nova | 13:13 | |
| johnthetubaguy | kashyap: so it comes from our usual backport rules that mean no new config options or features, etc | 13:13 |
| kashyap | johnthetubaguy: Raise an "error" you mean? | 13:17 |
| kashyap | johnthetubaguy: (In a follow-up.) | 13:17 |
| kashyap | Okay, noted. | 13:17 |
| * kashyap notes: his `irssi` notifier is temporarily broken; so will see the messages with a bit of delay. | 13:18 | |
| *** abhishekk has joined #openstack-nova | 13:19 | |
| *** dklyle has quit IRC | 13:27 | |
| *** felipemonteiro__ has joined #openstack-nova | 13:29 | |
| kashyap | stephenfin: Hmm, what is the alternative then, just don't indent it?: https://review.openstack.org/#/c/534384/22/nova/conf/libvirt.py@539 | 13:30 |
| stephenfin | kashyap: Correct | 13:30 |
| stephenfin | You indent subsequent lines of the same bullet point, but the bullet points themselves should not be indented | 13:31 |
| *** yamamoto has joined #openstack-nova | 13:32 | |
| kashyap | stephenfin: Okido; like it is for the disk cache modes. | 13:32 |
| kashyap | Thanks | 13:33 |
| kashyap | stephenfin: Just a quick note: I'm using the back ticks for the lower-cased ``pcid``, and for the rest, with upper-cased 'PCID' (leaving it as is) | 13:36 |
| kashyap | That fine with you? | 13:36 |
| stephenfin | kashyap: a-ok by me, yup :) | 13:36 |
| *** yamamoto has quit IRC | 13:36 | |
| kashyap | :-) One upper-case ``PCID`` (because that is talking about valid config value) | 13:36 |
| kashyap | Thanks! | 13:37 |
| *** tbachman has quit IRC | 13:37 | |
| *** sar has quit IRC | 13:40 | |
| kashyap | stephenfin: johnthetubaguy: Before I upload yet-another version, mind having a quick gander: http://paste.openstack.org/show/718287/ | 13:40 |
| kashyap | Or is it hard to read, without colours? (Can understand if so.) | 13:41 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Include only required fields in ironic node cache https://review.openstack.org/539508 | 13:41 |
| * kashyap removes the spurious comment on line 65 | 13:41 | |
| *** tbachman has joined #openstack-nova | 13:41 | |
| kashyap | Also add the "Related options" section | 13:42 |
| *** artom has quit IRC | 13:42 | |
| johnthetubaguy | kashyap: probably easier in context, but that looks about right | 13:42 |
| *** artom has joined #openstack-nova | 13:43 | |
| kashyap | Yeah; will post a patch in a min | 13:43 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Request only instance_uuid in ironic node list https://review.openstack.org/539509 | 13:44 |
| jaypipes | cdent, efried, edleafe: k, commented on the consumer generation spec. | 13:46 |
| efried | jaypipes: ack, thx | 13:46 |
| jaypipes | cdent, efried, edleafe: in short, I don't believe much change is needed from the spec at all. | 13:46 |
| edleafe | jaypipes: thx | 13:47 |
| edleafe | jaypipes: why do you think we shouldn't require sending the consumer gen on a PUT? | 13:47 |
| openstackgerrit | sahid proposed openstack/nova master: network: add command to configure trusted mode for VFs https://review.openstack.org/458513 | 13:47 |
| openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 13:47 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | 13:47 |
| edleafe | jaypipes: How else could it detect a conflict? | 13:48 |
| efried | edleafe: He's suggesting PUT should *always* 409 if the consumer exists. | 13:48 |
| *** zhurong has quit IRC | 13:48 | |
| jaypipes | edleafe: see my response. I believe that for PUT /allocations/{consumer_uuid}, if *any* allocations exist for a consumer with {consumer_uuid}, we should simply return 409 Conflict | 13:48 |
| efried | I.e. PUT is only for first-time create; POST is for modify. | 13:49 |
| cdent | that's a huge change jaypipes | 13:49 |
| cdent | POST exists explicitly for multiple consumers | 13:49 |
| efried | jaypipes: Also, we would still need to support create-via-POST. | 13:49 |
| edleafe | yeah, not for a single consumer | 13:49 |
| cdent | for single consumer PUT should remain the locus | 13:49 |
| efried | ...which still needs an answer to "what should the generation be for a heretofore-nonexistent consumer | 13:50 |
| efried | ?" | 13:50 |
| edleafe | The canonical case we are handling is nova and neutron both allocating resources for an instance at roughly the same time | 13:50 |
| *** yangyapeng has joined #openstack-nova | 13:50 | |
| edleafe | they both get the current allocations, if any, and make their changes, and PUT them back | 13:50 |
| jaypipes | edleafe: no, they aren't doing it "at the same time". | 13:50 |
| edleafe | One will get back a 409, and it will have to re-GET the allocations, add its allocs, and then PUT that with the new gen | 13:51 |
| jaypipes | edleafe: yes. that is precisely the point. | 13:51 |
| edleafe | jaypipes: that exact example is in the Problem Description of the spec | 13:52 |
| jaypipes | edleafe: but in real life, what will happen is neutron will allocate first for the consumer some networky thing. then the claim_resources() will run in the scheduler for the other compute resources, get a 409 Conflict about consumer already existing, and then it will merge the allocations and do a POST /allocations with the updated/merged allocations list. | 13:52 |
| edleafe | jaypipes: so without a consumer gen in the PUT, how would the second actor ever modify allocations? | 13:52 |
| *** esberglu has joined #openstack-nova | 13:53 | |
| jaypipes | edleafe: the second actor does GET /allocations/{consumer_uuid}, then POST /allocations | 13:53 |
| jaypipes | edleafe: as I wrote in the spec response. | 13:53 |
| edleafe | <shudder> | 13:53 |
| edleafe | jaypipes: that would be a PATCH, which I proposed but was argued against | 13:53 |
| cdent | jaypipes: why POST instead of PUT? It is in fact (restully speaking) a PUT | 13:54 |
| edleafe | the POST, as cdent mentioned, is for multiple consumers | 13:54 |
| edleafe | The POST actually does a series of PUTs, one for each consumer | 13:54 |
| *** yangyapeng has quit IRC | 13:54 | |
| jaypipes | cdent: because that ship has already sailed? I argued for PATCH instead of POST /allocations to handle multiple consumers, but we went with POST /allocations to handle the "change multiple allocations at once" scenario for migrations/resizes | 13:55 |
| edleafe | jaypipes: sure, but we aren't changing *multiple* | 13:55 |
| cdent | jaypipes: what edleafe just said | 13:56 |
| jaypipes | edleafe: nothing about POST /allocations implies or denotes changing multiple things. | 13:56 |
| cdent | there's only one consumer uuid. and yes it does | 13:56 |
| cdent | post allocations is for multiple consumers | 13:56 |
| jaypipes | cdent: where? | 13:56 |
| efried | POST is allowed to be used for just one, sure. | 13:57 |
| cdent | that's why we created it and how I wrote it | 13:57 |
| edleafe | jaypipes: the handler for POST loops on consumer_uuid. It's pretty clear that it is designed to handle multiple consumers | 13:57 |
| cdent | it _can_ do one, but that's not it's point | 13:57 |
| jaypipes | so tell me folks, what about my proposal *won't work*? | 13:57 |
| edleafe | jaypipes: it also doesn't *add* them to any existing allocations for a consumer; it overwrites them | 13:57 |
| cdent | there's no reason not to have generation on PUT because what we want to do, on PUT, is to write allocations for the consumer uuid in the URI. That's what it is for. | 13:58 |
| cdent | What's wrong with that | 13:58 |
| jaypipes | edleafe: which is what we want here. | 13:58 |
| cdent | that's what PUT means jaypipes | 13:58 |
| cdent | overwrite allocations for consumer X | 13:58 |
| cdent | the generation is acting a the etag | 13:58 |
| jaypipes | cdent: are you talking about HTTP/REST semantics? | 13:58 |
| cdent | I'm talking about what we already have in put, minus the generation | 13:58 |
| cdent | adding a generation to PUT makes more sense than overloading POST when we already have an existing semantic that is correct | 13:59 |
| edleafe | jaypipes: using the POST code, there is still no way to add allocations to a consumer | 13:59 |
| *** dtantsur|brb is now known as dtantsur | 13:59 | |
| jaypipes | edleafe: ? sure there is... you just list a set of allocations for a consumer. | 13:59 |
| cdent | I wonder if maybe I'm not understanding jay's suggestion properly | 13:59 |
| efried | FWIW, I agree that we should add generation to PUT, even if not doing so would allow us to sidestep the question of what generation should be specified for creation, which it doesn't. | 14:00 |
| cdent | how is post to /allocations different from put to /allocations/consumer_uuid (with a generation)? | 14:00 |
| efried | jaypipes: POST still needs to be *able* to be used to *create*. Because migration scenario. So we still have to address the question of "what generation do we send down for a consumer that doesn't yet exist?" | 14:00 |
| edleafe | jaypipes: changing the POST code to make it additive would break migrations | 14:00 |
| cdent | efried: yeah, as I said on the spec the none thing is meh-ish, but seems kinda gotta with the model being considered | 14:01 |
| *** abhishekk has quit IRC | 14:01 | |
| efried | IMO there's no meh about it. It's ick-but-necessary. | 14:01 |
| *** germs has joined #openstack-nova | 14:01 | |
| cdent | isn't that what meh means? | 14:01 |
| *** germs has quit IRC | 14:01 | |
| *** germs has joined #openstack-nova | 14:01 | |
| cdent | resignation in the face of crushing reality | 14:01 |
| efried | To me "meh" means "could do with or without it" | 14:01 |
| jaypipes | edleafe: sorry, I'm not suggesting modifying POST /allocations to be additive. I am suggesting to keep it as "overwrite the allcations for all consumers listed in the resquest payload" | 14:01 |
| *** yangyapeng has joined #openstack-nova | 14:02 | |
| efried | jaypipes: Cool, so how do you handle the generation for a newly-created consumer in POST? | 14:02 |
| edleafe | jaypipes: so in the example case of two actors creating their subset of allocations to placement, how will the second one ever be able to get theirs in? | 14:02 |
| jaypipes | efried: overwrite. | 14:03 |
| efried | jaypipes: There's nothing to overwrite. The consumer doesn't exist. | 14:03 |
| jaypipes | efried: so it is created? | 14:03 |
| jaypipes | efried: I'm not proposing anything different for that. | 14:03 |
| efried | yes, it is created by the POST (in e.g. the migration scenario). | 14:03 |
| efried | jaypipes: Right, I'm asking, now that we have to specify the generation per consumer chunk in the POST payload: what generatio do we send down for the new consumer? | 14:04 |
| jaypipes | edleafe: the second one gets a 409 Conflict when trying to do the PUT /allocations/{consumer}. It then does a GET /allocations/{consumer} and merges its needed resources into a call to POST /allocations | 14:04 |
| edleafe | for allocations, POST == a set of PUTs | 14:04 |
| jaypipes | efried: we don't. | 14:04 |
| edleafe | jaypipes: so you | 14:04 |
| edleafe | ugh | 14:05 |
| edleafe | jaypipes: so you're saying that POST needs a consumer gen, just to avoid adding one to PUT?? | 14:05 |
| *** awaugama has joined #openstack-nova | 14:05 | |
| efried | edleafe: He also just said leave the consumer generation off of the respective subsection of POST if it's a new consumer. | 14:05 |
| jaypipes | right. | 14:06 |
| *** lajoskatona has quit IRC | 14:06 | |
| *** germs has quit IRC | 14:06 | |
| jaypipes | edleafe: what I'm proposing is the least amount of change to the API. | 14:06 |
| edleafe | efried: if it's a new consumer, there could not be a 409 response | 14:06 |
| *** r-daneel has joined #openstack-nova | 14:06 | |
| edleafe | If you've gotten a 409, the consumer exists. You can still be in danger of overwriting | 14:06 |
| jaypipes | edleafe: no consumer_generation in POST /allocations means "I think this is a new consumer" | 14:06 |
| edleafe | jaypipes: that makes no sense | 14:06 |
| *** yangyapeng has quit IRC | 14:07 | |
| efried | Shrug, it's a coin-toss between that and sending null. | 14:07 |
| jaypipes | edleafe: sure it does. think about the existing use case of POST /allocations (the resize/migrate case) | 14:07 |
| edleafe | You just said that POST would be used *after* a conflict was detected. Ergo, there *is* a consumer | 14:07 |
| efried | ...it's just not what we landed on yesterday. | 14:07 |
| efried | With jaypipes' suggestion, PUT is only for create, but POST is for create *or* modify. | 14:07 |
| *** Eran_Kuris has quit IRC | 14:07 | |
| edleafe | [t 4bj1] | 14:07 |
| purplerbot | <jaypipes> edleafe: the second one gets a 409 Conflict when trying to do the PUT /allocations/{consumer}. It then does a GET /allocations/{consumer} and merges its needed resources into a call to POST /allocations [2018-04-03 14:04:39.482469] [n 4bj1] | 14:07 |
| efried | With things as we left them yesterday, either one can be used for either create or modify. | 14:08 |
| *** Sundar has joined #openstack-nova | 14:08 | |
| jaypipes | edleafe: when POST /allocations is used to reconcile after a 409 Conflict is received from the PUT /allocations/{consumer_uuid}, yes, the consumer_generation would be expected in the request. But POST /allocations is used for migrate/resize, and in the case of that, the migration UUID would be expected as a new consumer. | 14:08 |
| edleafe | efried: how would modify be guaranteed not to race w/o the consumer gen? | 14:08 |
| cdent | I need to do something else for awhile, can someone be sure this gets summarized to the spec, I've got more to say, but it sounds like this needs to play out a bit | 14:08 |
| *** yamamoto has joined #openstack-nova | 14:09 | |
| jaypipes | edleafe, efried: a hangout perchance? | 14:09 |
| efried | edleafe: consumer_generation omitted would behave exactly the same as consumer_generation=null. Either one works to signify "I think the consumer doesn't exist yet". | 14:09 |
| edleafe | jaypipes: well, that just feels really wrong. POST shouldn't behave one way sometimes, and another way others | 14:09 |
| efried | hangout fine by me. | 14:09 |
| *** Sundar has quit IRC | 14:10 | |
| edleafe | efried: so then placement would reject it, for the exact same reason it rejected the consumer gen-less PUT | 14:10 |
| efried | edleafe: Yes. | 14:10 |
| jaypipes | https://hangouts.google.com/call/7cb33WR2UowcbcI_8EdLAAEE | 14:10 |
| *** amodi has joined #openstack-nova | 14:10 | |
| edleafe | efried: why limit it to 2 actors? What about 3? Nova, cinder and neutron all allocating resources | 14:10 |
| *** ralonsoh_ has quit IRC | 14:11 | |
| efried | it's not limited edleafe. Are you joining? | 14:11 |
| sean-k-mooney[m] | jaypipes: why would you do post to /allocations on migrate instead of PUT | 14:11 |
| sean-k-mooney[m] | jaypipes: sorry for resize we use post be cause we are using a migration uuid instead of the instance uuid never mind | 14:12 |
| *** sean-k-mooney has joined #openstack-nova | 14:14 | |
| *** alexchadin has quit IRC | 14:14 | |
| *** alexchadin has joined #openstack-nova | 14:14 | |
| *** alexchadin has quit IRC | 14:15 | |
| sean-k-mooney | jaypipes: efried edleafe so i was wondering why i did not get any responces to my messages for the last 5 mins. aprently my riot.im client never sent them to irc... | 14:15 |
| *** kholkina has quit IRC | 14:15 | |
| *** alexchadin has joined #openstack-nova | 14:15 | |
| sean-k-mooney | if ye get a bunch of out of context messages form sean-k-mooney[m] in then next few minuts thats why | 14:15 |
| *** alexchadin has quit IRC | 14:15 | |
| *** alexchadin has joined #openstack-nova | 14:16 | |
| *** alexchadin has quit IRC | 14:16 | |
| *** hongbin has joined #openstack-nova | 14:16 | |
| *** alexchadin has joined #openstack-nova | 14:16 | |
| *** alexchadin has quit IRC | 14:17 | |
| *** alexchadin has joined #openstack-nova | 14:17 | |
| *** alexchadin has quit IRC | 14:18 | |
| *** mlavalle has joined #openstack-nova | 14:18 | |
| *** alexchadin has joined #openstack-nova | 14:18 | |
| *** yangyapeng has joined #openstack-nova | 14:18 | |
| *** voelzmo has quit IRC | 14:19 | |
| *** voelzmo has joined #openstack-nova | 14:19 | |
| *** gouthamr has quit IRC | 14:19 | |
| *** voelzmo has quit IRC | 14:20 | |
| *** voelzmo has joined #openstack-nova | 14:20 | |
| *** gouthamr has joined #openstack-nova | 14:20 | |
| *** voelzmo has quit IRC | 14:20 | |
| *** voelzmo has joined #openstack-nova | 14:21 | |
| *** voelzmo has quit IRC | 14:21 | |
| *** voelzmo has joined #openstack-nova | 14:21 | |
| *** voelzmo has quit IRC | 14:22 | |
| *** voelzmo has joined #openstack-nova | 14:22 | |
| *** alexchadin has quit IRC | 14:22 | |
| *** voelzmo has quit IRC | 14:23 | |
| *** yangyapeng has quit IRC | 14:23 | |
| *** brault has joined #openstack-nova | 14:24 | |
| *** alexchadin has joined #openstack-nova | 14:27 | |
| *** itlinux has quit IRC | 14:28 | |
| *** fragatina has quit IRC | 14:28 | |
| *** moshele has quit IRC | 14:29 | |
| *** yangyapeng has joined #openstack-nova | 14:29 | |
| *** fragatina has joined #openstack-nova | 14:29 | |
| *** alexchadin has quit IRC | 14:32 | |
| *** yangyapeng has quit IRC | 14:34 | |
| *** links has quit IRC | 14:37 | |
| *** sahid has quit IRC | 14:37 | |
| edleafe | sean-k-mooney: to answer your question, the POST to allocations was added for the migration case, where we needed a set of allocations for multiple consumers to be changed atomically | 14:39 |
| *** munimeha2 has joined #openstack-nova | 14:42 | |
| *** fragatin_ has joined #openstack-nova | 14:46 | |
| *** alexchadin has joined #openstack-nova | 14:47 | |
| *** gmann_ has quit IRC | 14:47 | |
| *** sshwarts has joined #openstack-nova | 14:48 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:48 | |
| openstackgerrit | Simon Dodsley proposed openstack/nova master: Add enhanced KVM storage QoS quotas https://review.openstack.org/558530 | 14:48 |
| *** fragatina has quit IRC | 14:49 | |
| *** gmann_ has joined #openstack-nova | 14:50 | |
| *** david-lyle has joined #openstack-nova | 14:52 | |
| *** felipemonteiro__ has quit IRC | 14:52 | |
| *** tbachman has quit IRC | 14:59 | |
| *** felipemonteiro_ has quit IRC | 15:02 | |
| *** sar has joined #openstack-nova | 15:02 | |
| *** felipemonteiro_ has joined #openstack-nova | 15:02 | |
| *** alexchadin has quit IRC | 15:04 | |
| bauzas | artom: *cough cough* +Wd https://review.openstack.org/#/c/552722/12 ;à | 15:04 |
| bauzas | ;) | 15:04 |
| *** yangyapeng has joined #openstack-nova | 15:04 | |
| *** belmoreira has quit IRC | 15:05 | |
| *** felipemonteiro__ has joined #openstack-nova | 15:07 | |
| *** belmoreira has joined #openstack-nova | 15:08 | |
| *** felipemonteiro_ has quit IRC | 15:11 | |
| jaypipes | cdent, edleafe, efried: k, summary sent to ML. | 15:13 |
| efried | jaypipes: Thanks for doing that. | 15:13 |
| cdent | thanks jaypipes | 15:14 |
| jaypipes | np | 15:14 |
| openstackgerrit | Merged openstack/nova master: Scheduling Optimization: Remove cell0 from the list of candidates https://review.openstack.org/556821 | 15:16 |
| *** _ix has joined #openstack-nova | 15:16 | |
| openstackgerrit | Merged openstack/nova-specs master: NUMA-aware live migration https://review.openstack.org/552722 | 15:18 |
| *** sahid has joined #openstack-nova | 15:19 | |
| *** yangyapeng has quit IRC | 15:20 | |
| *** felipemonteiro has joined #openstack-nova | 15:20 | |
| *** _ix has quit IRC | 15:21 | |
| *** Eran_Kuris has joined #openstack-nova | 15:22 | |
| *** itlinux has joined #openstack-nova | 15:22 | |
| *** rgerganov has quit IRC | 15:22 | |
| *** felipemonteiro__ has quit IRC | 15:23 | |
| *** _ix has joined #openstack-nova | 15:25 | |
| *** felipemonteiro has quit IRC | 15:25 | |
| *** salv-orlando has quit IRC | 15:26 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Filter resource providers by forbidden traits in db https://review.openstack.org/556472 | 15:28 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Filter allocation candidates by forbidden traits in db https://review.openstack.org/556660 | 15:28 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Parse forbidden traits in query strings https://review.openstack.org/556819 | 15:28 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Support forbidden traits in API https://review.openstack.org/556820 | 15:28 |
| *** andreas_s has quit IRC | 15:29 | |
| *** tbachman has joined #openstack-nova | 15:30 | |
| *** tesseract has quit IRC | 15:32 | |
| openstackgerrit | Merged openstack/nova master: Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527 | 15:33 |
| *** andreas_s has joined #openstack-nova | 15:34 | |
| *** Eran_Kuris has quit IRC | 15:34 | |
| *** gouthamr has quit IRC | 15:34 | |
| *** gouthamr has joined #openstack-nova | 15:35 | |
| *** gouthamr has quit IRC | 15:36 | |
| *** itlinux has quit IRC | 15:36 | |
| cfriesen | anyone ever run into problems with privsep failing with a broken pipe? I'm trying to figure out what went wrong. http://paste.openstack.org/show/718307/ | 15:38 |
| *** andreas_s has quit IRC | 15:38 | |
| *** alexchadin has joined #openstack-nova | 15:41 | |
| *** belmoreira has quit IRC | 15:42 | |
| jaypipes | cfriesen: sorry, never seen that :( | 15:42 |
| *** gouthamr has joined #openstack-nova | 15:43 | |
| stephenfin | cfriesen: I usually only see that what a process dies. Other than that, I've no idea | 15:44 |
| *** itlinux has joined #openstack-nova | 15:44 | |
| efried | cfriesen: EPIPE happens when two threads are talking over RPC and the sender shuts down while the receiver is still waiting for stuff. But you probably knew that. | 15:46 |
| *** chyka has joined #openstack-nova | 15:47 | |
| efried | or... it might be vice versa. Point is, where they don't disconnect friendly-like. | 15:47 |
| openstackgerrit | Merged openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/537897 | 15:48 |
| *** jmlowe has quit IRC | 15:49 | |
| cdent | cfriesen: I reckon your problem them is probably eventlet simply because if you've got a chance to blame eventlet for something, maybe you should. | 15:51 |
| cfriesen | lol | 15:52 |
| zzzeek | jaypipes: you should idle on #openstack-oslo :) | 15:52 |
| arvindn05 | jaypipes: can we quickly discuss a review comment? | 15:52 |
| *** amodi has quit IRC | 15:52 | |
| arvindn05 | https://review.openstack.org/#/c/557795/ i replied to the comment on having specific field(required_traits) for traits vs using a dictofstring approach | 15:53 |
| openstackgerrit | Merged openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | 15:54 |
| arvindn05 | i made the changes and i think its ready to merge but wanted to get your thoughts | 15:55 |
| bauzas | FWIW, I'm now done with nova-specs and runways reviews, will work on my own spec | 15:58 |
| bauzas | unless something urgent of course | 15:58 |
| *** hemna_ has joined #openstack-nova | 16:02 | |
| *** mdbooth has quit IRC | 16:02 | |
| *** jafeha__ has joined #openstack-nova | 16:02 | |
| *** germs has joined #openstack-nova | 16:02 | |
| *** germs has quit IRC | 16:02 | |
| *** germs has joined #openstack-nova | 16:02 | |
| *** jafeha has quit IRC | 16:03 | |
| *** Sukhdev has joined #openstack-nova | 16:04 | |
| arvindn05 | dansmith: addressed the comment. Please provide your thoughts on one open question i had as well | 16:07 |
| *** germs has quit IRC | 16:07 | |
| *** alexchadin has quit IRC | 16:08 | |
| *** tssurya has quit IRC | 16:09 | |
| *** mgagne_ has quit IRC | 16:11 | |
| *** mgagne_ has joined #openstack-nova | 16:11 | |
| *** udesale has quit IRC | 16:12 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Multiple Fixed-IPs support in network information https://review.openstack.org/312626 | 16:15 |
| *** sshwarts has quit IRC | 16:15 | |
| efried | jaypipes, cdent, edleafe: We should have a (hopefully quick) chat about the question of including all provider information in provider summaries. | 16:16 |
| *** alexchadin has joined #openstack-nova | 16:17 | |
| cdent | efried: by "all" you mean "everything in this things tree and shared friends"? | 16:17 |
| efried | cdent: At least "everything in this tree". I had thought the sharing would be "only sharing providers providing resource to the requests". | 16:18 |
| efried | But even the former is in question. | 16:18 |
| efried | We at some point decided that we wanted to return all the providers in the tree | 16:18 |
| efried | even the ones not providing resource to the request. | 16:18 |
| efried | Anyone remember why? | 16:18 |
| cdent | I thought we had to do it order for weighers to work? | 16:18 |
| efried | Yeah, weighers would be a good reason I suppose. But... more specifically? | 16:19 |
| efried | like, in what circumstance would a weigher want to look at a provider that's not providing resource to the request? | 16:19 |
| openstackgerrit | Nguyen Hai proposed openstack/nova-specs master: Enhance nova-specs webpage and clean up repo https://review.openstack.org/551802 | 16:19 |
| efried | (Hint: future me will repeat question with "...look at a resource class that's not part of the request") | 16:19 |
| cdent | I'm afraid I will have to defer to others as I have chosen to achieve ignorance on the details of this particular aspect of things to make room for other thoughts | 16:20 |
| *** dtantsur is now known as dtantsur|brb | 16:21 | |
| jaypipes | efried: here's an example... | 16:22 |
| * efried munches popcorn | 16:22 | |
| jaypipes | efried: imagine a NUMA topology weigher that looks at traits associated with NUMA nodes that are not providing resources for the allocation requests but the weigher (or even NUMA topology filter) would like to use the "intermediate" provider information in its decision-making. | 16:23 |
| efried | Okay. So jaypipes what about resource classes? Do we show all of those? | 16:25 |
| efried | for... similar reasons? | 16:25 |
| jaypipes | efried: I don't see why not... | 16:25 |
| jaypipes | efried: the reason we don't currently is mostly an implementation side-effect I think. | 16:26 |
| *** sahid has quit IRC | 16:26 | |
| *** jmlowe has joined #openstack-nova | 16:26 | |
| *** links has joined #openstack-nova | 16:26 | |
| *** trozet has quit IRC | 16:26 | |
| *** alexchadin has quit IRC | 16:26 | |
| *** gouthamr has quit IRC | 16:26 | |
| efried | jaypipes: Well, I don't see why not either, other than the fact that it'll require a microversion, which is okay, but that requires a spec, which is also fine, but now we're looking at a nontrivial chunk of work, on top of an already overloaded release... | 16:27 |
| efried | jaypipes: Anyway, I agree with you that we should do it. So at least https://review.openstack.org/#/c/558045/ and its predecessor are in the running. | 16:27 |
| efried | and in fact intermingled with your nrp-in-alloc-cands work. | 16:27 |
| edleafe | efried: no problem! https://www.youtube.com/watch?v=CJHoPm2d5CY | 16:27 |
| *** alexchadin has joined #openstack-nova | 16:28 | |
| efried | I also suspect that some of the work later on in that series - the stuff about including "anchor" providers - will overlap significantly with "make the rest of the tree show up". | 16:29 |
| *** mvk has quit IRC | 16:30 | |
| efried | jaypipes: cause that was another thing we didn't close on yesterday: what is the fate (at least in Rocky) of that series. | 16:30 |
| *** Sukhdev has quit IRC | 16:30 | |
| efried | Last I heard, you were still going through the ML post. (You might still be. It was a lot of words. Sorry about that.) | 16:31 |
| *** gjayavelu has joined #openstack-nova | 16:31 | |
| *** trozet has joined #openstack-nova | 16:31 | |
| jaypipes | efried: the fate of the nested providers alloc candidates series? | 16:31 |
| efried | jaypipes: No, hopefully that fate is well known. | 16:31 |
| *** jmlowe has quit IRC | 16:31 | |
| efried | The fate of the series starting at https://review.openstack.org/#/c/558044/ | 16:31 |
| efried | jaypipes: which is the subject of said ML post. | 16:31 |
| *** AlexeyAbashkin has quit IRC | 16:31 | |
| *** mgoddard has quit IRC | 16:33 | |
| *** tbachman has quit IRC | 16:33 | |
| *** alexchadin has quit IRC | 16:33 | |
| jaypipes | efried: ok, I apologize, I haven't gotten to that yet. need to do the mirroring aggregates spec first. | 16:34 |
| *** lucasagomes is now known as lucas-afk | 16:38 | |
| *** zhaochao has quit IRC | 16:38 | |
| *** lpetrut has quit IRC | 16:38 | |
| *** afazekas has quit IRC | 16:39 | |
| *** afazekas has joined #openstack-nova | 16:39 | |
| *** alexchadin has joined #openstack-nova | 16:41 | |
| *** jmlowe has joined #openstack-nova | 16:42 | |
| *** jmlowe has quit IRC | 16:43 | |
| *** alexchadin has quit IRC | 16:45 | |
| *** ccamacho has quit IRC | 16:45 | |
| *** sdague has quit IRC | 16:48 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/534384 | 16:49 |
| kashyap | dansmith: johnthetubaguy: When you get a sec, addresesed what you requested. | 16:50 |
| *** amodi has joined #openstack-nova | 16:51 | |
| *** alexchadin has joined #openstack-nova | 16:55 | |
| *** jmlowe has joined #openstack-nova | 16:56 | |
| *** mdnadeem has quit IRC | 17:00 | |
| *** jpena is now known as jpena|off | 17:00 | |
| *** moshele has joined #openstack-nova | 17:00 | |
| *** david-lyle has quit IRC | 17:04 | |
| *** tbachman has joined #openstack-nova | 17:05 | |
| *** links has quit IRC | 17:13 | |
| openstackgerrit | Nguyen Hai proposed openstack/nova-specs master: Enhance nova-specs webpage and clean up repo https://review.openstack.org/551802 | 17:17 |
| *** annp has quit IRC | 17:17 | |
| *** annp has joined #openstack-nova | 17:18 | |
| *** idlemind has quit IRC | 17:18 | |
| *** sambetts is now known as sambetts|afk | 17:20 | |
| *** itlinux has quit IRC | 17:29 | |
| *** amoralej is now known as amoralej|off | 17:31 | |
| *** Nil_ has quit IRC | 17:33 | |
| *** lpetrut has joined #openstack-nova | 17:34 | |
| *** AlexeyAbashkin has joined #openstack-nova | 17:37 | |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: mirror nova host aggregates to placement API https://review.openstack.org/545057 | 17:39 |
| jaypipes | dansmith: ^^ | 17:39 |
| jaypipes | bauzas: ^^ | 17:40 |
| *** moshele has quit IRC | 17:41 | |
| *** mgoddard has joined #openstack-nova | 17:41 | |
| *** AlexeyAbashkin has quit IRC | 17:42 | |
| *** itlinux has joined #openstack-nova | 17:43 | |
| *** felipemonteiro has joined #openstack-nova | 17:43 | |
| *** jackie-truong has joined #openstack-nova | 17:44 | |
| openstackgerrit | Arvind Nadendla proposed openstack/nova master: Update ImageMetaProp object to expose traits https://review.openstack.org/557795 | 17:45 |
| *** r-daneel has quit IRC | 17:50 | |
| *** alexchadin has quit IRC | 17:50 | |
| *** r-daneel has joined #openstack-nova | 17:50 | |
| *** ccamacho has joined #openstack-nova | 17:53 | |
| *** jmlowe has quit IRC | 18:00 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:02 | |
| *** germs has joined #openstack-nova | 18:03 | |
| *** david-lyle has joined #openstack-nova | 18:04 | |
| *** AlexeyAbashkin has quit IRC | 18:06 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 18:06 |
| *** germs has quit IRC | 18:08 | |
| *** efried has quit IRC | 18:14 | |
| *** efried has joined #openstack-nova | 18:14 | |
| *** oomichi has joined #openstack-nova | 18:15 | |
| *** gouthamr has joined #openstack-nova | 18:21 | |
| *** gouthamr has quit IRC | 18:21 | |
| *** moshele has joined #openstack-nova | 18:22 | |
| *** mgoddard has quit IRC | 18:22 | |
| *** gouthamr has joined #openstack-nova | 18:23 | |
| *** hemna_ has quit IRC | 18:24 | |
| *** moshele has quit IRC | 18:25 | |
| *** dtantsur|brb is now known as dtantsur | 18:25 | |
| *** sdague has joined #openstack-nova | 18:29 | |
| *** sridharg has quit IRC | 18:30 | |
| efried | jaypipes: I would like to rebase series https://review.openstack.org/#/c/554529/ on master, so I can start hacking granular on top. That okay with you, or do you have local changes? | 18:31 |
| cdent | efried: do I recall correctly that https://review.openstack.org/#/q/topic:bp/placement-api-error-handling was driving some cleanups you were hoping to do? Or was there an even more pressing concern? | 18:33 |
| *** damien_r has quit IRC | 18:33 | |
| *** damien_r has joined #openstack-nova | 18:33 | |
| efried | cdent: The former, I believe. I can't think of anything super urgent. | 18:34 |
| * cdent nods | 18:35 | |
| efried | cdent: e.g. https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L633 | 18:35 |
| cdent | would be nice to get it through, so it is in there waiting when we want it | 18:35 |
| cdent | actually I can think of a thing where we will want it: | 18:35 |
| efried | foshizzle | 18:35 |
| cdent | we want to know when an allocation fails because of generation versus out of space | 18:35 |
| cdent | (when consumer generations come on the scene) | 18:36 |
| cdent | Basically the same issue as with inventories, but on the flip side | 18:36 |
| efried | As in, "help me decide whether this conflict is because of a generation mismatch or because a resource you're requesting via the allocation is not available"? | 18:37 |
| *** isssp has joined #openstack-nova | 18:38 | |
| *** armaan has quit IRC | 18:38 | |
| *** cdent_ has joined #openstack-nova | 18:38 | |
| *** armaan has joined #openstack-nova | 18:39 | |
| cdent_ | efried: yes to [t qqJ] (I got disconnectioned) | 18:39 |
| purplerbot | <efried> As in, "help me decide whether this conflict is because of a generation mismatch or because a resource you're requesting via the allocation is not available"? [2018-04-03 18:37:41.872921] [n qqJ] | 18:39 |
| efried | We should have some spec cores give that the ol' +A. Seems pretty straightforward to me. dansmith jaypipes care to have a swag? | 18:40 |
| efried | Maybe they were waiting on edleafe's +1 | 18:40 |
| *** gouthamr has quit IRC | 18:40 | |
| *** arvindn051 has joined #openstack-nova | 18:41 | |
| *** tinwood_ has joined #openstack-nova | 18:41 | |
| *** Roamer`_ has joined #openstack-nova | 18:42 | |
| *** fnordahl_ has joined #openstack-nova | 18:42 | |
| *** FoXMaN_ has joined #openstack-nova | 18:43 | |
| *** fragatin_ has quit IRC | 18:43 | |
| *** pcaruana has quit IRC | 18:44 | |
| *** kencjohnston_ has joined #openstack-nova | 18:45 | |
| *** EmilienM_ has joined #openstack-nova | 18:46 | |
| *** gouthamr has joined #openstack-nova | 18:46 | |
| *** ccamacho has quit IRC | 18:47 | |
| *** cdent has quit IRC | 18:47 | |
| *** arvindn05 has quit IRC | 18:47 | |
| *** EmilienM has quit IRC | 18:47 | |
| *** tinwood has quit IRC | 18:47 | |
| *** mordred has quit IRC | 18:47 | |
| *** Roamer` has quit IRC | 18:47 | |
| *** ispp has quit IRC | 18:47 | |
| *** brad[] has quit IRC | 18:47 | |
| *** fnordahl has quit IRC | 18:47 | |
| *** kencjohnston has quit IRC | 18:47 | |
| *** FoXMaN has quit IRC | 18:47 | |
| *** cdent_ is now known as cdent | 18:47 | |
| *** EmilienM_ is now known as EmilienM | 18:48 | |
| *** dtantsur is now known as dtantsur|afk | 18:49 | |
| *** EmilienM has quit IRC | 18:49 | |
| *** EmilienM has joined #openstack-nova | 18:49 | |
| *** vladikr has quit IRC | 18:49 | |
| *** moshele has joined #openstack-nova | 18:50 | |
| *** purplerbot has quit IRC | 18:51 | |
| *** Sukhdev has joined #openstack-nova | 18:51 | |
| *** vladikr has joined #openstack-nova | 18:53 | |
| jaypipes | efried: go for it. | 18:54 |
| *** mordred has joined #openstack-nova | 18:54 | |
| efried | jaypipes: Staring at it some, I'm not sure I can do it until I see how you're wiring in from AllocationCandidates.get_by_requests | 18:55 |
| edleafe | efried: was on a call. What am I supposed to be +1ing? | 18:55 |
| *** moshele has quit IRC | 18:55 | |
| efried | edleafe: The error codes from placement API spec: https://review.openstack.org/#/c/546177/ | 18:55 |
| *** AlexeyAbashkin has joined #openstack-nova | 18:55 | |
| efried | edleafe: Sorry, the spec is here: https://review.openstack.org/#/c/418393/ | 18:55 |
| *** ccamacho has joined #openstack-nova | 18:56 | |
| efried | jaypipes: my bad, I was looking at the code when I asked you to approve the spec, which you did a week ago. Carry on. | 18:56 |
| edleafe | efried: ack. Have another meeting, but will look ASAP | 18:56 |
| efried | edleafe: Looks like you +1ed two patch sets ago, so should be an easy one. | 18:57 |
| *** voelzmo has joined #openstack-nova | 18:58 | |
| *** oomichi has quit IRC | 18:59 | |
| *** AlexeyAbashkin has quit IRC | 19:00 | |
| *** yassine has quit IRC | 19:01 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 19:04 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 19:04 |
| *** avolkov has quit IRC | 19:09 | |
| *** hongbin has quit IRC | 19:14 | |
| *** voelzmo has quit IRC | 19:17 | |
| *** moshele has joined #openstack-nova | 19:18 | |
| *** voelzmo has joined #openstack-nova | 19:19 | |
| *** READ10 has quit IRC | 19:20 | |
| *** hongbin has joined #openstack-nova | 19:21 | |
| *** jmlowe has joined #openstack-nova | 19:23 | |
| *** lpetrut has quit IRC | 19:23 | |
| *** harlowja has joined #openstack-nova | 19:23 | |
| * dansmith feels a cold chill run up his spine | 19:24 | |
| *** yamamoto has quit IRC | 19:30 | |
| *** eharney has quit IRC | 19:30 | |
| efried | ? | 19:32 |
| cdent | you watch many horror movies efried ? | 19:34 |
| efried | clearly not | 19:34 |
| efried | Is it, like, when you're being watched by the guy with the bad acne and the striped shirt? | 19:35 |
| cdent | a chill fills the room when the monster transforms, perhaps? | 19:36 |
| *** yamamoto has joined #openstack-nova | 19:36 | |
| *** mvk has joined #openstack-nova | 19:37 | |
| *** pcaruana has joined #openstack-nova | 19:38 | |
| *** yassine_ has joined #openstack-nova | 19:38 | |
| *** yassine_ has quit IRC | 19:38 | |
| *** Tom-Tom has joined #openstack-nova | 19:40 | |
| efried | Hah! | 19:41 |
| *** yamamoto has quit IRC | 19:42 | |
| *** eharney has joined #openstack-nova | 19:43 | |
| *** Tom-Tom has quit IRC | 19:44 | |
| *** awaugama has quit IRC | 19:45 | |
| *** brad[] has joined #openstack-nova | 19:48 | |
| *** pcaruana has quit IRC | 19:50 | |
| *** yamamoto has joined #openstack-nova | 19:52 | |
| andrewbogott | I've just upgrade my nova from liberty to mitaka (that's the release that added the api_db). I've done my nova-manage api_db sync but now any nova api calls return 404 | 19:56 |
| andrewbogott | I can see the api handling these calls, it just doesn't know anything. | 19:56 |
| andrewbogott | Has anyone thought about that migration recently enough to have suggestions for what I should be looking for? | 19:56 |
| andrewbogott | (I wonder if e.g. every project is now under some default domain or zone or something that I need to be specifying in my API call) | 19:57 |
| *** yamamoto has quit IRC | 19:57 | |
| *** jackie-truong has quit IRC | 19:57 | |
| *** yamamoto has joined #openstack-nova | 19:59 | |
| *** yamamoto has quit IRC | 19:59 | |
| *** yamamoto has joined #openstack-nova | 20:01 | |
| *** david-lyle is now known as dklyle | 20:02 | |
| *** Sukhdev has quit IRC | 20:02 | |
| cdent | jaypipes, efried: can we pretty please get forbidden traits merged before https://review.openstack.org/#/c/554529 ? | 20:05 |
| *** yamamoto has quit IRC | 20:06 | |
| efried | heh | 20:06 |
| efried | cdent: I was actually just working up granular and thinking much the same thing. Would force jaypipes to make nested work for granular, rather than me having to do the reverse :P | 20:06 |
| cdent | 'xactly | 20:07 |
| cfriesen | andrewbogott: see subject line for the channel. that said, did you add the [api_database]/connection information to nova.conf? | 20:09 |
| andrewbogott | cfriesen: Yep! But, I'll try over there. | 20:09 |
| *** hemna_ has joined #openstack-nova | 20:11 | |
| *** sdague has quit IRC | 20:11 | |
| *** efried is now known as efried_schoolrun | 20:12 | |
| *** moshele has quit IRC | 20:13 | |
| *** liverpooler has quit IRC | 20:14 | |
| andrewbogott | cfriesen: is there a doc you can link me to that explains what that new database actually does? Is it just a cache? | 20:16 |
| *** yamamoto has joined #openstack-nova | 20:16 | |
| *** liverpooler has joined #openstack-nova | 20:17 | |
| *** itlinux has quit IRC | 20:19 | |
| *** itlinux has joined #openstack-nova | 20:21 | |
| *** yamamoto has quit IRC | 20:21 | |
| *** yamamoto has joined #openstack-nova | 20:23 | |
| *** yamamoto has quit IRC | 20:23 | |
| cdent | edleafe, jaypipes: i'm pretty sure the data model in https://review.openstack.org/#/c/557958/ (consumer generation) is wrong. even if I'm wrong about being wrong, then we need to make it clear in the commit why we're doing it weirdly. | 20:24 |
| cdent | are we trying to log generation on the allocation as a sort of historical reference? that won't work because we delete all allocations for a cosumer when they change | 20:27 |
| cdent | efried_schoolrun: you'll also be interested in ^ | 20:28 |
| *** moshele has joined #openstack-nova | 20:29 | |
| *** sapd has quit IRC | 20:30 | |
| *** sapd has joined #openstack-nova | 20:30 | |
| *** sapd has quit IRC | 20:30 | |
| *** sapd has joined #openstack-nova | 20:31 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:33 | |
| *** felipemonteiro has quit IRC | 20:36 | |
| *** sar has quit IRC | 20:37 | |
| *** fragatina has joined #openstack-nova | 20:38 | |
| cfriesen | andrewbogott: it's related to the cellsv2 work. the nova DB became per-cell, and the nova-api DB spans the cells and has higher-level stuff | 20:38 |
| *** fragatina has quit IRC | 20:38 | |
| *** Nil_ has joined #openstack-nova | 20:39 | |
| andrewbogott | ah, that makes sense, and might be a clue. thanks. | 20:39 |
| *** fragatina has joined #openstack-nova | 20:39 | |
| *** fragatina has quit IRC | 20:40 | |
| *** fragatina has joined #openstack-nova | 20:41 | |
| *** purplerbot has joined #openstack-nova | 20:41 | |
| *** liverpooler has quit IRC | 20:47 | |
| *** pchavva has quit IRC | 20:47 | |
| *** damien_r has quit IRC | 20:48 | |
| *** moshele has quit IRC | 20:48 | |
| *** damien_r has joined #openstack-nova | 20:55 | |
| *** priteau has joined #openstack-nova | 20:59 | |
| *** edmondsw has quit IRC | 21:09 | |
| *** edmondsw has joined #openstack-nova | 21:09 | |
| *** itlinux has quit IRC | 21:13 | |
| *** edmondsw has quit IRC | 21:14 | |
| *** tbachman has quit IRC | 21:15 | |
| jaypipes | cdent: you're absolutely right. no idea why I hadn't seen that. :( | 21:16 |
| cdent | ETOOMUCHNUMA | 21:16 |
| cdent | I think I'll use that as my cue to retire | 21:18 |
| cdent | night all | 21:18 |
| *** cdent has quit IRC | 21:18 | |
| *** jmlowe has quit IRC | 21:19 | |
| *** Guest9432 has quit IRC | 21:22 | |
| *** yamamoto has joined #openstack-nova | 21:24 | |
| *** yamamoto has quit IRC | 21:29 | |
| openstackgerrit | Merged openstack/nova master: Fix N332 api_version decorator hacking check https://review.openstack.org/557743 | 21:31 |
| arvindn051 | jaypipes: https://review.openstack.org/#/c/557795/ wanted to discuss comments on the review...good time? | 21:33 |
| jaypipes | arvindn051: I need to re-review that one... | 21:34 |
| arvindn051 | k | 21:36 |
| arvindn051 | no rush...i am able to make progress on future patches...but wanted to get feedback on using a list of strings vs dict | 21:36 |
| *** gouthamr has quit IRC | 21:37 | |
| *** munimeha2 has quit IRC | 21:37 | |
| *** damien_r has quit IRC | 21:38 | |
| *** liverpooler has joined #openstack-nova | 21:40 | |
| *** eharney has quit IRC | 21:42 | |
| *** tbachman has joined #openstack-nova | 21:43 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921 | 21:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep. https://review.openstack.org/552241 | 21:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242 | 21:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078 | 21:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605 | 21:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437 | 21:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439 | 21:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438 | 21:45 |
| *** rcernin has joined #openstack-nova | 21:53 | |
| *** efried_schoolrun is now known as efried | 21:54 | |
| *** tbachman has quit IRC | 21:58 | |
| arvindn051 | jaypipes: thx for comments....i can revert the prefix | 21:58 |
| arvindn051 | how do you propose we handle preferred traits, numbered traits etc in future? | 21:59 |
| arvindn051 | seperate fields for each of them? | 21:59 |
| *** AlexeyAbashkin has joined #openstack-nova | 22:00 | |
| jaypipes | arvindn051: well, preferred traits will have a value of "prefer" not "require", right? | 22:01 |
| *** Tom-Tom has joined #openstack-nova | 22:02 | |
| arvindn051 | but we changed the image meta prop to be a list of strings in the object | 22:02 |
| arvindn051 | so no way to introduce prefer for the value... | 22:02 |
| jaypipes | arvindn051: that's just the outward production of the stuff stored in the image_properties table in glance, though... | 22:03 |
| arvindn051 | i am talking about the ojbect definition...not the image property key value | 22:03 |
| jaypipes | arvindn051: the object definition is entirely temporal... it never saves to a DB. | 22:03 |
| arvindn051 | yup...it just parses the properties into usable object format...i got that. | 22:05 |
| *** AlexeyAbashkin has quit IRC | 22:05 | |
| arvindn051 | if we had to introduce preferred traits to this model, we would first add trait:foo=preffered in the image property | 22:06 |
| arvindn051 | when we parse it into the object, we would need to introduce antoher field preferred_traits with these values right? | 22:06 |
| *** Tom-Tom has quit IRC | 22:07 | |
| jaypipes | arvindn051: yes, precisely. | 22:07 |
| arvindn051 | how about if we do numbered traits like trait:CUSTOM_foo, trait1:foo etc...how would we model this here? | 22:07 |
| jaypipes | arvindn051: don't worry about that. | 22:08 |
| arvindn051 | my intial thought was to just parse the image properties as a dict into the object and have the consumer figure out which keys/values makes sense to him | 22:08 |
| jaypipes | arvindn051: for now, all we need to do is produce the non-granular required traits as a list of trait strings. | 22:08 |
| arvindn051 | is there a possiblity to support the need for granular traits in the future? if so, how would we handle is what i am thinking about | 22:10 |
| arvindn051 | if you think there is no possiblity of this, then we can ignore and go ahead with having specific fields | 22:10 |
| jaypipes | arvindn051: like I said, don't worry about it. :) | 22:10 |
| jaypipes | arvindn051: there's nothing about the proposed solution here that prohibits granular traits being parsed from strings like "trait1:HW_CPU_X86_AVX2" in the future. | 22:11 |
| *** burt has quit IRC | 22:12 | |
| arvindn051 | ok :) i guess we could introduce a granular_traits dict of strings fields in the future if needed... | 22:13 |
| arvindn051 | thanks...will make the updates in a couple of mins | 22:13 |
| *** priteau has quit IRC | 22:15 | |
| *** amodi has quit IRC | 22:15 | |
| *** archit has joined #openstack-nova | 22:15 | |
| *** archit is now known as amodi | 22:15 | |
| openstackgerrit | Arvind Nadendla proposed openstack/nova master: Update ImageMetaProp object to expose traits https://review.openstack.org/557795 | 22:23 |
| arvindn051 | jaypipes: made the changes...should be good now | 22:24 |
| *** tbachman has joined #openstack-nova | 22:26 | |
| *** yamamoto has joined #openstack-nova | 22:26 | |
| jaypipes | arvindn051: cool, thank you sir. | 22:28 |
| *** yamamoto has quit IRC | 22:30 | |
| *** voelzmo has quit IRC | 22:31 | |
| *** felipemonteiro_ has quit IRC | 22:35 | |
| *** lbragstad has quit IRC | 22:36 | |
| *** voelzmo has joined #openstack-nova | 22:41 | |
| *** voelzmo has quit IRC | 22:46 | |
| *** voelzmo has joined #openstack-nova | 22:48 | |
| *** lbragstad has joined #openstack-nova | 22:53 | |
| *** voelzmo has quit IRC | 22:53 | |
| *** hongbin has quit IRC | 22:58 | |
| edleafe | jaypipes: sorry, was at a doc appt. So we don't want the consumer_generation in the Allocation at all? That's cool, and I think that was my original idea, as written in the spec. I don't remember the conversation (there are so many) where it was requested. | 23:02 |
| *** itlinux has joined #openstack-nova | 23:07 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Default to py3 for the pep8 tox env because it's stricter https://review.openstack.org/558648 | 23:14 |
| *** r-daneel has quit IRC | 23:18 | |
| *** liverpooler has quit IRC | 23:18 | |
| *** yamamoto has joined #openstack-nova | 23:26 | |
| *** chyka has quit IRC | 23:30 | |
| *** chyka has joined #openstack-nova | 23:30 | |
| *** sdague has joined #openstack-nova | 23:32 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Default to py3 for the pep8 tox env because it's stricter https://review.openstack.org/558648 | 23:32 |
| *** yamamoto has quit IRC | 23:32 | |
| *** chyka has quit IRC | 23:35 | |
| *** threestrands has joined #openstack-nova | 23:36 | |
| *** threestrands has quit IRC | 23:37 | |
| *** threestrands has joined #openstack-nova | 23:38 | |
| *** threestrands has quit IRC | 23:38 | |
| *** threestrands has joined #openstack-nova | 23:38 | |
| *** mlavalle has quit IRC | 23:39 | |
| *** threestrands has quit IRC | 23:39 | |
| *** threestrands has joined #openstack-nova | 23:39 | |
| *** threestrands has quit IRC | 23:39 | |
| *** threestrands has joined #openstack-nova | 23:39 | |
| *** threestrands has quit IRC | 23:40 | |
| *** threestrands has joined #openstack-nova | 23:41 | |
| *** threestrands has quit IRC | 23:41 | |
| *** threestrands has joined #openstack-nova | 23:41 | |
| *** liverpooler has joined #openstack-nova | 23:50 | |
| *** _ix has quit IRC | 23:58 | |
| *** voelzmo has joined #openstack-nova | 23:59 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fix AttributeError in getting a resource ID https://review.openstack.org/558328 | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!