| *** tetsuro has joined #openstack-nova | 00:00 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201 | 00:01 |
|---|---|---|
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520 | 00:02 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718 | 00:02 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: trivialfix: cleanup _pack_instance_onto_cores() https://review.openstack.org/538698 | 00:04 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add unit tests for EmulatorThreadsTestCase https://review.openstack.org/538699 | 00:04 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Not use thread alloc policy for emulator thread https://review.openstack.org/538700 | 00:04 |
| *** liverpooler has joined #openstack-nova | 00:13 | |
| *** lbragstad has joined #openstack-nova | 00:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/538473 | 00:21 |
| Spazmotic | Morning | 00:27 |
| *** salv-orlando has quit IRC | 00:29 | |
| *** yangyapeng has quit IRC | 00:29 | |
| *** yangyapeng has joined #openstack-nova | 00:30 | |
| *** hiro-kobayahsi has joined #openstack-nova | 00:34 | |
| *** yangyapeng has quit IRC | 00:34 | |
| *** sorrison_laptop has joined #openstack-nova | 00:42 | |
| *** zhaochao has joined #openstack-nova | 00:49 | |
| *** gcb has joined #openstack-nova | 00:53 | |
| *** hemna has quit IRC | 00:55 | |
| *** hieulq__ has quit IRC | 00:55 | |
| *** tuanla____ has joined #openstack-nova | 01:00 | |
| *** hemna has joined #openstack-nova | 01:01 | |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| Spazmotic | Off to the gym. Back in a bit | 01:04 |
| *** hoangcx has joined #openstack-nova | 01:05 | |
| *** tiendc has joined #openstack-nova | 01:07 | |
| *** hieulq__ has joined #openstack-nova | 01:11 | |
| *** yangyapeng has joined #openstack-nova | 01:16 | |
| *** tovin07 has joined #openstack-nova | 01:16 | |
| *** Efried_ has joined #openstack-nova | 01:18 | |
| Efried_ | Is there a core in the house? | 01:19 |
| *** gcb has quit IRC | 01:21 | |
| Efried_ | Could please get re+Ws on six previously-approved patches series starting | 01:21 |
| Efried_ | at https://review.openstack.org/#/c/533808/ | 01:21 |
| Efried_ | Wow, freenode webchat is hard on teeny iPhone. | 01:22 |
| Efried_ | Anyway, trivial manual rebases on a couple of those kicked out my +Ws | 01:23 |
| *** gongysh has joined #openstack-nova | 01:29 | |
| *** salv-orlando has joined #openstack-nova | 01:29 | |
| *** Efried_ has quit IRC | 01:32 | |
| *** salv-orlando has quit IRC | 01:35 | |
| *** mriedem has quit IRC | 01:37 | |
| *** hiro-kobayahsi has quit IRC | 01:39 | |
| *** yikun_jiang has quit IRC | 02:01 | |
| *** yikun has joined #openstack-nova | 02:02 | |
| *** hiro-kobayashi has joined #openstack-nova | 02:05 | |
| *** hongbin has joined #openstack-nova | 02:06 | |
| *** sapcc-bot1 has quit IRC | 02:11 | |
| *** sapcc-bot has joined #openstack-nova | 02:12 | |
| *** armax has joined #openstack-nova | 02:14 | |
| *** phuongnh has quit IRC | 02:15 | |
| *** phuongnh has joined #openstack-nova | 02:15 | |
| Spazmotic | Man that poor patchset still struggling its way through the gate efried ? | 02:18 |
| *** tbachman_ has joined #openstack-nova | 02:20 | |
| *** tbachman has quit IRC | 02:21 | |
| *** tbachman_ is now known as tbachman | 02:21 | |
| *** tuanla____ has quit IRC | 02:27 | |
| *** tuanla____ has joined #openstack-nova | 02:28 | |
| *** salv-orlando has joined #openstack-nova | 02:31 | |
| *** salv-orlando has quit IRC | 02:35 | |
| *** huanxie has joined #openstack-nova | 02:38 | |
| *** sdague has quit IRC | 02:52 | |
| *** Tom-Tom has joined #openstack-nova | 02:55 | |
| *** yamamoto has joined #openstack-nova | 02:58 | |
| openstackgerrit | Spencer Yu proposed openstack/nova master: Use defusedxml.ElementTree function instead of lxml.etree https://review.openstack.org/519291 | 03:02 |
| *** gongysh has quit IRC | 03:03 | |
| *** rcernin has quit IRC | 03:08 | |
| *** harlowja has joined #openstack-nova | 03:11 | |
| *** xinliang has quit IRC | 03:16 | |
| *** xinliang has joined #openstack-nova | 03:16 | |
| Spazmotic | jianghuaw, huanxie I'll be on vacation after this week, so we should find some time before the weekend to have a group chat. | 03:16 |
| huanxie | sure Spazmotic | 03:17 |
| Spazmotic | It's monday so i won't bother you two with it today and it's not something we need to push hard on at the moment yet anyway | 03:17 |
| *** harlowja has quit IRC | 03:18 | |
| Spazmotic | But I think we're closing in one a solution here that we can all agree on | 03:18 |
| *** dave-mccowan has quit IRC | 03:20 | |
| *** yikun_jiang has joined #openstack-nova | 03:20 | |
| *** stakeda has joined #openstack-nova | 03:21 | |
| *** yikun has quit IRC | 03:23 | |
| *** jgwentworth is now known as melwitt | 03:26 | |
| *** salv-orlando has joined #openstack-nova | 03:31 | |
| *** salv-orlando has quit IRC | 03:36 | |
| *** annp has joined #openstack-nova | 03:40 | |
| huanxie | Agree with you Spazmotic | 03:45 |
| jianghuaw | Spazmotic, huanxie Looked at the patch. I have the same concerns as Huan. I think the point to use XCP version than product version is because the there is some other variants basing on Xen which will have the same issue. | 03:53 |
| *** Tom-Tom has quit IRC | 03:54 | |
| jianghuaw | It looks good to me if we can change to use "XCP version" for a quick fix. Later we need check if we can prepare the volume SR earlier in the dest host. | 03:54 |
| jianghuaw | Preparing the SR earlier requires much refactor work to make it looks reasonable. But I think that's the right way which is worthy for us to look at. | 03:56 |
| *** elmaciej has quit IRC | 03:57 | |
| *** blkart_ has joined #openstack-nova | 03:59 | |
| *** sapcc-bot has quit IRC | 04:04 | |
| *** sapcc-bot has joined #openstack-nova | 04:04 | |
| *** rcernin has joined #openstack-nova | 04:04 | |
| *** hongbin has quit IRC | 04:04 | |
| *** owalsh_ has joined #openstack-nova | 04:13 | |
| openstackgerrit | Merged openstack/nova master: libvirt: don't attempt to live snapshot paused instances https://review.openstack.org/532214 | 04:13 |
| openstackgerrit | Dao Cong Tien proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 04:15 |
| *** owalsh has quit IRC | 04:17 | |
| *** psachin has joined #openstack-nova | 04:17 | |
| *** hiro-kobayashi has quit IRC | 04:18 | |
| *** sree has joined #openstack-nova | 04:19 | |
| *** hiro-kobayashi has joined #openstack-nova | 04:19 | |
| *** Tom-Tom has joined #openstack-nova | 04:21 | |
| *** hoangcx has quit IRC | 04:21 | |
| *** bhagyashris has quit IRC | 04:22 | |
| *** Tom-Tom has quit IRC | 04:26 | |
| *** blkart has joined #openstack-nova | 04:29 | |
| *** blkart has quit IRC | 04:30 | |
| *** salv-orlando has joined #openstack-nova | 04:32 | |
| *** bhagyashris has joined #openstack-nova | 04:33 | |
| *** liuyulong has joined #openstack-nova | 04:33 | |
| *** hoangcx has joined #openstack-nova | 04:34 | |
| *** salv-orlando has quit IRC | 04:37 | |
| *** hoangcx has quit IRC | 04:38 | |
| *** hoangcx has joined #openstack-nova | 04:40 | |
| openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: add Linux distribution guest only description for inject_xxx options https://review.openstack.org/528441 | 04:41 |
| *** armax has quit IRC | 04:49 | |
| *** abhishekk has joined #openstack-nova | 05:04 | |
| *** AlexeyAbashkin has joined #openstack-nova | 05:06 | |
| *** claudiub|2 has joined #openstack-nova | 05:07 | |
| *** hongbin has joined #openstack-nova | 05:09 | |
| *** AlexeyAbashkin has quit IRC | 05:10 | |
| *** abhishekk has quit IRC | 05:15 | |
| *** Tom-Tom has joined #openstack-nova | 05:19 | |
| *** diga has joined #openstack-nova | 05:21 | |
| *** ratailor has joined #openstack-nova | 05:24 | |
| *** hongbin has quit IRC | 05:28 | |
| *** AlexeyAbashkin has joined #openstack-nova | 05:31 | |
| *** salv-orlando has joined #openstack-nova | 05:33 | |
| *** janki has joined #openstack-nova | 05:37 | |
| *** salv-orlando has quit IRC | 05:37 | |
| *** moshele has joined #openstack-nova | 05:38 | |
| *** sridharg has joined #openstack-nova | 05:39 | |
| *** pooja_jadhav has joined #openstack-nova | 05:45 | |
| *** AlexeyAbashkin has quit IRC | 05:46 | |
| *** mdnadeem has joined #openstack-nova | 05:59 | |
| *** xinliang has quit IRC | 06:02 | |
| *** mdnadeem has quit IRC | 06:04 | |
| *** diga has quit IRC | 06:05 | |
| *** xinliang has joined #openstack-nova | 06:14 | |
| *** xinliang has joined #openstack-nova | 06:14 | |
| *** mdnadeem has joined #openstack-nova | 06:16 | |
| *** AlexeyAbashkin has joined #openstack-nova | 06:17 | |
| *** MikeG451 has quit IRC | 06:26 | |
| *** syjulian has quit IRC | 06:27 | |
| *** Eran_Kuris has joined #openstack-nova | 06:30 | |
| openstackgerrit | Hoang Trung Hieu proposed openstack/python-novaclient master: Zuul: Remove project name https://review.openstack.org/538777 | 06:30 |
| *** salv-orlando has joined #openstack-nova | 06:34 | |
| *** ameeda has quit IRC | 06:35 | |
| *** ameeda has joined #openstack-nova | 06:36 | |
| *** phuongnh has quit IRC | 06:37 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:37 | |
| *** salv-orlando has quit IRC | 06:38 | |
| *** salv-orlando has joined #openstack-nova | 06:38 | |
| *** lajoskatona has joined #openstack-nova | 06:43 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451 | 06:44 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove unnecessary arguments in notification methods https://review.openstack.org/535599 | 06:48 |
| *** sorrison_laptop has quit IRC | 06:57 | |
| *** Tengu has quit IRC | 06:58 | |
| *** hiro-kobayashi has quit IRC | 07:02 | |
| *** Dinesh_Bhor has quit IRC | 07:10 | |
| *** MikeG451 has joined #openstack-nova | 07:10 | |
| *** rcernin has quit IRC | 07:11 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:11 | |
| Spazmotic | jianghuaw, I think you're right, that was my first attempt, getting the SR prepped earlier, but would definately take some refactor and thought | 07:20 |
| Spazmotic | Will look at your multinode env later tonight :) | 07:20 |
| *** namnh has joined #openstack-nova | 07:21 | |
| Spazmotic | Ah looks like you already got the whole gang to +1 it, not much for me to do there hehe | 07:21 |
| *** Dinesh_Bhor has quit IRC | 07:23 | |
| Spazmotic | Also, what version of XCP would you say to target for this? | 07:24 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Fix some reST field lists in docstrings https://review.openstack.org/538809 | 07:24 |
| Spazmotic | huanxie, could use your input on that one too, since you two are tied in closer to the XCP versioning then I am | 07:25 |
| *** Dinesh_Bhor has joined #openstack-nova | 07:25 | |
| *** kevinz has joined #openstack-nova | 07:29 | |
| *** andreas_s has joined #openstack-nova | 07:32 | |
| *** salv-orlando has quit IRC | 07:33 | |
| *** slaweq has joined #openstack-nova | 07:34 | |
| *** Dinesh_Bhor has quit IRC | 07:34 | |
| *** hoonetorg has quit IRC | 07:35 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:36 | |
| *** slaweq has quit IRC | 07:38 | |
| *** Dinesh_Bhor has quit IRC | 07:38 | |
| *** slaweq has joined #openstack-nova | 07:38 | |
| *** gcb has joined #openstack-nova | 07:39 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:39 | |
| *** slaweq has quit IRC | 07:40 | |
| *** slaweq has joined #openstack-nova | 07:40 | |
| *** moshele has quit IRC | 07:41 | |
| *** pcaruana has joined #openstack-nova | 07:44 | |
| *** fragatina has joined #openstack-nova | 07:44 | |
| *** Dinesh_Bhor has quit IRC | 07:45 | |
| *** belmoreira has joined #openstack-nova | 07:47 | |
| jianghuaw | Spazmotic, the platform version is 2.1.0 for XS 7.0. | 07:48 |
| Spazmotic | Thanks sir | 07:49 |
| *** Dinesh_Bhor has joined #openstack-nova | 07:50 | |
| jianghuaw | np:-) | 07:50 |
| *** hoonetorg has joined #openstack-nova | 07:52 | |
| *** salv-orlando has joined #openstack-nova | 07:55 | |
| *** AlexeyAbashkin has quit IRC | 07:55 | |
| *** sahid has joined #openstack-nova | 07:55 | |
| *** sahid has joined #openstack-nova | 07:55 | |
| *** gongysh has joined #openstack-nova | 08:03 | |
| *** kholkina has joined #openstack-nova | 08:07 | |
| *** gongysh has quit IRC | 08:08 | |
| *** links has joined #openstack-nova | 08:09 | |
| *** zhenguo has joined #openstack-nova | 08:16 | |
| *** ralonsoh has joined #openstack-nova | 08:20 | |
| *** tssurya has joined #openstack-nova | 08:23 | |
| *** alexchadin has joined #openstack-nova | 08:25 | |
| *** tesseract has joined #openstack-nova | 08:26 | |
| *** hrw has joined #openstack-nova | 08:28 | |
| *** Dinesh_Bhor has quit IRC | 08:29 | |
| *** AlexeyAbashkin has joined #openstack-nova | 08:35 | |
| ameeda | Morning | 08:39 |
| *** AlexeyAbashkin has quit IRC | 08:40 | |
| ameeda | when I try to alter sqlite column data type I got error "mismatch models and migration script", Please check this to see the details http://paste.openstack.org/show/656589/ | 08:40 |
| ameeda | what should I do with this case ? | 08:40 |
| *** Dinesh_Bhor has joined #openstack-nova | 08:41 | |
| ameeda | Spazmotic: can you please help ? | 08:41 |
| *** masber has quit IRC | 08:41 | |
| ameeda | jianghuaw: what do you think sir ? | 08:42 |
| *** masber has joined #openstack-nova | 08:42 | |
| *** ttsiouts has joined #openstack-nova | 08:44 | |
| *** salv-orlando has quit IRC | 08:45 | |
| *** mgoddard_ has joined #openstack-nova | 08:45 | |
| *** Dinesh_Bhor has quit IRC | 08:46 | |
| ameeda | Ok , thanks I fix the issue, I have to use reference to instances(uuid) rather than compute_node(id) | 08:46 |
| *** ttsiouts has quit IRC | 08:46 | |
| *** ttsiouts has joined #openstack-nova | 08:47 | |
| *** jpena|off is now known as jpena | 08:48 | |
| *** aloga has quit IRC | 08:49 | |
| *** aloga has joined #openstack-nova | 08:49 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:50 | |
| *** Dinesh_Bhor has quit IRC | 08:52 | |
| *** belmorei_ has joined #openstack-nova | 08:52 | |
| *** amoralej|off is now known as amoralej | 08:53 | |
| *** belmoreira has quit IRC | 08:53 | |
| *** fragatina has quit IRC | 08:55 | |
| *** alexchadin has quit IRC | 08:58 | |
| *** alexchadin has joined #openstack-nova | 08:58 | |
| openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Do not normalize allocation_ratios https://review.openstack.org/532924 | 08:59 |
| maciejjozefczyk | Kevin_Zheng: thx for review :) | 09:00 |
| *** salv-orlando has joined #openstack-nova | 09:02 | |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 09:04 |
| Kevin_Zheng | maciejjozefczyk YW :) | 09:04 |
| *** hrw has quit IRC | 09:04 | |
| *** saulo has joined #openstack-nova | 09:04 | |
| giblet | efried: I shoveled the nrp patches back to the gate as your rebase looks clean | 09:04 |
| *** Dinesh_Bhor has joined #openstack-nova | 09:06 | |
| *** giblet is now known as gibi_ | 09:07 | |
| *** mgoddard_ has quit IRC | 09:07 | |
| *** saulo is now known as saugusto | 09:07 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008 | 09:08 |
| *** saugusto is now known as sauloaugusto | 09:08 | |
| *** finucannot is now known as sfinucan | 09:12 | |
| *** sfinucan is now known as stephenfin | 09:12 | |
| sauloaugusto | Hi , anyone can help me with "ERROR nova.api.openstack.extensions TypeError: Can't upgrade a READER transaction to a WRITER mid-transaction" | 09:16 |
| sauloaugusto | after pike upgrade procedure | 09:16 |
| *** hrw has joined #openstack-nova | 09:16 | |
| sauloaugusto | I saw that discussion at history logs , but I did not find out the solution . | 09:16 |
| sauloaugusto | I am using centos 7 | 09:16 |
| *** jafeha has joined #openstack-nova | 09:17 | |
| *** owalsh_ has quit IRC | 09:17 | |
| *** owalsh has joined #openstack-nova | 09:17 | |
| sauloaugusto | same error occurs with 'openstack server list --all-projects' command | 09:19 |
| hrw | can someone take a look at https://review.openstack.org/#/c/538003 and tell me is it proper way? | 09:21 |
| *** AlexeyAbashkin has joined #openstack-nova | 09:22 | |
| hrw | this code takes care of usb host being present if usbtablet is requested (not all archs have usb host by default) | 09:22 |
| *** derekh has joined #openstack-nova | 09:23 | |
| bauzas | good morning novaers | 09:24 |
| bauzas | back from organizing the SnowCamp :) | 09:25 |
| gibi_ | bauzas: welcome back | 09:28 |
| *** yamahata has quit IRC | 09:28 | |
| bauzas | saw the gate problems and the milestone delay | 09:29 |
| maciejjozefczyk | bauzas: heyho | 09:30 |
| *** yamamoto has quit IRC | 09:30 | |
| *** takashin has left #openstack-nova | 09:31 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: reno for notification-transformation-queens https://review.openstack.org/518018 | 09:31 |
| *** hrw has quit IRC | 09:32 | |
| *** kholkina has quit IRC | 09:35 | |
| *** hrw has joined #openstack-nova | 09:36 | |
| *** liuyulong has quit IRC | 09:39 | |
| *** Dinesh_Bhor has quit IRC | 09:42 | |
| *** hrw has quit IRC | 09:42 | |
| *** mgoddard_ has joined #openstack-nova | 09:43 | |
| *** yamamoto has joined #openstack-nova | 09:47 | |
| *** elmaciej has joined #openstack-nova | 09:48 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: refactors and cleans up test_rbd.py https://review.openstack.org/538022 | 09:49 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | 09:49 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 09:49 |
| *** hrw has joined #openstack-nova | 09:50 | |
| *** aloga has quit IRC | 09:52 | |
| *** aloga has joined #openstack-nova | 09:52 | |
| *** hrw has quit IRC | 09:55 | |
| *** sree has quit IRC | 10:01 | |
| *** sree has joined #openstack-nova | 10:02 | |
| *** tuanla_____ has joined #openstack-nova | 10:03 | |
| *** hrw has joined #openstack-nova | 10:03 | |
| *** moshele has joined #openstack-nova | 10:03 | |
| *** tuanla____ has quit IRC | 10:05 | |
| *** hrw has quit IRC | 10:08 | |
| *** namnh has quit IRC | 10:09 | |
| *** namnh has joined #openstack-nova | 10:09 | |
| *** sree has quit IRC | 10:11 | |
| *** elmaciej has quit IRC | 10:12 | |
| *** AlexeyAbashkin has quit IRC | 10:13 | |
| *** AlexeyAbashkin has joined #openstack-nova | 10:13 | |
| *** stakeda has quit IRC | 10:21 | |
| *** hrw has joined #openstack-nova | 10:25 | |
| *** lpetrut has joined #openstack-nova | 10:26 | |
| *** gcb has quit IRC | 10:29 | |
| *** hrw has quit IRC | 10:30 | |
| *** threestrands_ has joined #openstack-nova | 10:33 | |
| *** sdague has joined #openstack-nova | 10:35 | |
| *** threestrands has quit IRC | 10:36 | |
| *** Tom-Tom has quit IRC | 10:36 | |
| *** lucas-afk is now known as lucasagomes | 10:37 | |
| *** tiendc has quit IRC | 10:37 | |
| *** jappleii__ has joined #openstack-nova | 10:37 | |
| *** jappleii__ has quit IRC | 10:38 | |
| *** jappleii__ has joined #openstack-nova | 10:39 | |
| *** Tom-Tom has joined #openstack-nova | 10:40 | |
| *** threestrands_ has quit IRC | 10:41 | |
| *** Tom-Tom has quit IRC | 10:44 | |
| *** belmorei_ has quit IRC | 10:46 | |
| *** danpawlik has quit IRC | 10:46 | |
| *** hrw has joined #openstack-nova | 10:50 | |
| *** jaosorior has joined #openstack-nova | 10:52 | |
| *** namnh has quit IRC | 10:52 | |
| *** gcb has joined #openstack-nova | 10:53 | |
| *** alexchadin has quit IRC | 10:55 | |
| *** alexchadin has joined #openstack-nova | 10:55 | |
| *** gszasz has joined #openstack-nova | 10:57 | |
| *** Efried_ has joined #openstack-nova | 10:57 | |
| *** elmaciej has joined #openstack-nova | 10:58 | |
| ameeda | when I try "self.main_db = self.useFixture(fixtures.Database(database='main'))" I got this error " .. (sqlite3.OperationalError) no such table: services ... " | 10:58 |
| *** gcb has quit IRC | 10:59 | |
| *** alexchadin has quit IRC | 10:59 | |
| *** hrw has quit IRC | 10:59 | |
| Efried_ | gibi Just popping in to say thanks for shepherding my series. | 10:59 |
| Efried_ | Be back for real in a couple hours. | 11:00 |
| *** hrw has joined #openstack-nova | 11:01 | |
| *** annp has quit IRC | 11:01 | |
| -openstackstatus- NOTICE: Zuul is currently under heavy load. Do not *recheck* or *approve* any changes. | 11:03 | |
| *** alexchadin has joined #openstack-nova | 11:04 | |
| *** hrw has quit IRC | 11:06 | |
| *** Efried_ has quit IRC | 11:07 | |
| *** alexchadin has quit IRC | 11:07 | |
| *** Tom-Tom has joined #openstack-nova | 11:09 | |
| *** danpawlik has joined #openstack-nova | 11:13 | |
| *** Tom-Tom has quit IRC | 11:13 | |
| *** gcb has joined #openstack-nova | 11:17 | |
| *** gcb has quit IRC | 11:18 | |
| *** elmaciej has quit IRC | 11:20 | |
| *** AlexeyAbashkin has quit IRC | 11:23 | |
| *** sambetts|afk is now known as sambetts| | 11:24 | |
| *** sambetts| is now known as sambetts | 11:24 | |
| *** andreas_s has quit IRC | 11:25 | |
| *** andreas_s has joined #openstack-nova | 11:25 | |
| *** hrw has joined #openstack-nova | 11:28 | |
| *** alexchadin has joined #openstack-nova | 11:30 | |
| *** alexchadin has quit IRC | 11:33 | |
| *** andreas_s has quit IRC | 11:35 | |
| *** tuanla_____ has quit IRC | 11:36 | |
| *** andreas_s has joined #openstack-nova | 11:40 | |
| *** alexchadin has joined #openstack-nova | 11:41 | |
| *** alexchadin has quit IRC | 11:43 | |
| *** tetsuro has left #openstack-nova | 11:47 | |
| *** pooja-jadhav has joined #openstack-nova | 11:49 | |
| *** bhagyashri_s has joined #openstack-nova | 11:50 | |
| *** pooja_jadhav has quit IRC | 11:51 | |
| *** dklyle has quit IRC | 11:52 | |
| *** bhagyashris has quit IRC | 11:53 | |
| *** david-lyle has joined #openstack-nova | 11:53 | |
| *** andreas_s has quit IRC | 11:55 | |
| *** hrw has quit IRC | 11:58 | |
| *** pooja_jadhav has joined #openstack-nova | 11:59 | |
| *** AlexeyAbashkin has joined #openstack-nova | 11:59 | |
| *** pooja_jadhav has quit IRC | 11:59 | |
| *** andreas_s has joined #openstack-nova | 12:00 | |
| *** Supun has joined #openstack-nova | 12:01 | |
| *** bhagyashri_s has quit IRC | 12:01 | |
| *** pooja-jadhav has quit IRC | 12:01 | |
| *** pooja_jadhav has joined #openstack-nova | 12:01 | |
| *** bhagyashris has joined #openstack-nova | 12:02 | |
| *** alexchadin has joined #openstack-nova | 12:02 | |
| *** mvk has quit IRC | 12:02 | |
| *** fragatina has joined #openstack-nova | 12:02 | |
| *** andreas_s has quit IRC | 12:03 | |
| *** andreas_s has joined #openstack-nova | 12:03 | |
| *** ratailor has quit IRC | 12:05 | |
| *** belmoreira has joined #openstack-nova | 12:05 | |
| *** amorin has joined #openstack-nova | 12:08 | |
| amorin | hey all | 12:09 |
| amorin | can someone help me with this/ | 12:09 |
| amorin | https://review.openstack.org/#/c/508640/6 | 12:09 |
| *** takashin has joined #openstack-nova | 12:09 | |
| *** liverpooler has quit IRC | 12:09 | |
| *** sree has joined #openstack-nova | 12:10 | |
| *** salv-orlando has quit IRC | 12:14 | |
| sauloaugusto | Someone can help me with this : Can't upgrade a READER transaction to a WRITER mid-transation after upgrade openstack nova . | 12:16 |
| *** salv-orlando has joined #openstack-nova | 12:22 | |
| *** sree_ has joined #openstack-nova | 12:27 | |
| *** sree_ is now known as Guest40215 | 12:27 | |
| *** salv-orlando has quit IRC | 12:29 | |
| *** sree has quit IRC | 12:30 | |
| *** salv-orlando has joined #openstack-nova | 12:32 | |
| *** jpena is now known as jpena|lunch | 12:33 | |
| *** janki has quit IRC | 12:48 | |
| *** moshele has quit IRC | 12:52 | |
| bauzas | amorin: what's your problem with the above change ? | 12:55 |
| bauzas | amorin: that's because you recheck? | 12:55 |
| bauzas | the gate has some problems atm | 12:55 |
| *** mvenesio has joined #openstack-nova | 12:57 | |
| *** Eran_Kuris has quit IRC | 12:58 | |
| *** Eran_Kuris has joined #openstack-nova | 12:59 | |
| amorin | bauzas: yes, I am rechecking | 13:05 |
| amorin | but I was wondering, | 13:05 |
| amorin | because it's still failing at the same place (some kind of timeout) | 13:05 |
| amorin | is the solution to continue recheck? | 13:05 |
| *** amoralej is now known as amoralej|lunch | 13:07 | |
| bauzas | not really | 13:07 |
| bauzas | the gate has major performance issues AFAICS | 13:08 |
| bauzas | plus the fact we're close to the extended FF deadline makes a lot of changes being tested at the same time | 13:08 |
| amorin | ok | 13:09 |
| *** Guest40215 has quit IRC | 13:13 | |
| *** jaypipes has joined #openstack-nova | 13:14 | |
| *** mvk has joined #openstack-nova | 13:17 | |
| *** READ10 has joined #openstack-nova | 13:20 | |
| *** alexchad_ has joined #openstack-nova | 13:22 | |
| *** liverpooler has joined #openstack-nova | 13:23 | |
| *** vivsoni has quit IRC | 13:23 | |
| *** alexchadin has quit IRC | 13:24 | |
| *** vivsoni has joined #openstack-nova | 13:24 | |
| *** vladikr has joined #openstack-nova | 13:28 | |
| *** takedakn has joined #openstack-nova | 13:28 | |
| *** takedakn has quit IRC | 13:29 | |
| *** gszasz has quit IRC | 13:29 | |
| *** gszasz has joined #openstack-nova | 13:30 | |
| bauzas | wow, the gate is really super slow | 13:31 |
| bauzas | more than I thought I mean | 13:31 |
| *** jpena|lunch is now known as jpena | 13:33 | |
| -openstackstatus- NOTICE: Zuul is currently under heavy load. Do not *recheck* or *approve* any changes until we give the go ahead. | 13:33 | |
| *** ChanServ changes topic to "Zuul is currently under heavy load. Do not *recheck* or *approve* any changes until we give the go ahead." | 13:33 | |
| *** Eran_Kuris has quit IRC | 13:33 | |
| *** alexchadin has joined #openstack-nova | 13:34 | |
| bauzas | oh heh ^ | 13:37 |
| *** alexchad_ has quit IRC | 13:37 | |
| *** cleong has joined #openstack-nova | 13:38 | |
| gibi_ | this will be a slow monday | 13:38 |
| *** bkopilov has quit IRC | 13:40 | |
| *** edmondsw has joined #openstack-nova | 13:41 | |
| *** gszasz has quit IRC | 13:41 | |
| *** alexchadin has quit IRC | 13:43 | |
| efried | gibi_ Howdy, thanks for "shoveling" :) | 13:45 |
| *** hemna_ has joined #openstack-nova | 13:45 | |
| *** rmart04 has joined #openstack-nova | 13:46 | |
| *** moshele has joined #openstack-nova | 13:47 | |
| *** alexchadin has joined #openstack-nova | 13:47 | |
| *** Eran_Kuris has joined #openstack-nova | 13:48 | |
| gibi_ | efried: no problme. My effor might mean nothing considering the state of the gatea | 13:49 |
| efried | gibi_ ikr | 13:49 |
| gibi_ | anyhow it was a good warmup for the day :) | 13:49 |
| *** Tom-Tom has joined #openstack-nova | 13:51 | |
| *** gszasz has joined #openstack-nova | 13:54 | |
| *** cdent has joined #openstack-nova | 13:57 | |
| edleafe | crap, just rechecked a patch before I saw #topic | 13:58 |
| *** sree has joined #openstack-nova | 13:58 | |
| bauzas | edleafe: boooooooo | 13:58 |
| edleafe | Any way to un-recheck it? | 13:58 |
| bauzas | edleafe: just trigger a new patchset | 13:59 |
| Spazmotic | Man these XenAPI drivers just get worse the more I read | 13:59 |
| edleafe | Won't that start a check? | 13:59 |
| Spazmotic | Like that malcom in the middle gif where he goes to change the lightbulb.. | 13:59 |
| bauzas | edleafe: ahem, you're right | 13:59 |
| Spazmotic | Just keep patching? hehe | 14:00 |
| edleafe | Scheduler subteam meeting starting now in #openstack-meeting-alt | 14:00 |
| bauzas | edleafe: looks like per looking at -infra that they're restarting zuul anyway | 14:01 |
| *** yamamoto has quit IRC | 14:01 | |
| bauzas | edleafe: so your recheck should be harmless :p | 14:01 |
| * edleafe breathes a sigh of relief | 14:01 | |
| *** slaweq has quit IRC | 14:02 | |
| *** slaweq has joined #openstack-nova | 14:02 | |
| *** mriedem has joined #openstack-nova | 14:03 | |
| *** sree has quit IRC | 14:03 | |
| *** artom_ has joined #openstack-nova | 14:04 | |
| *** artom has quit IRC | 14:04 | |
| andreykurilin | mriedem: hi! do you know someone who use nova+openvz? | 14:04 |
| mriedem | hmmm | 14:05 |
| Spazmotic | I.. do... | 14:05 |
| Spazmotic | But not really.. nevermidn i take it back don't ask me about it hehe | 14:05 |
| mriedem | andreykurilin: mnestratov and the vz people i guess... | 14:05 |
| *** zhenguo has quit IRC | 14:05 | |
| mriedem | i don't know about actual users | 14:05 |
| Spazmotic | I do know a group of folk that use it but it's fairly custom at this point as the openstack openvz drivers are long ded | 14:06 |
| *** purplerbot has quit IRC | 14:06 | |
| *** purplerbot has joined #openstack-nova | 14:06 | |
| *** purplerbot has quit IRC | 14:07 | |
| *** purplerbot has joined #openstack-nova | 14:07 | |
| *** slaweq has quit IRC | 14:07 | |
| andreykurilin | Spazmotic: ha. so nothing alive in upstream? | 14:09 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: test if tempest-full is triggered or not https://review.openstack.org/537936 | 14:09 |
| mriedem | virtuozzo containers are supported via the libvirt driver in nova | 14:10 |
| mriedem | if we're talking about the same thing | 14:10 |
| *** fragatina has quit IRC | 14:10 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/conf/libvirt.py#L105 | 14:11 |
| *** fragatina has joined #openstack-nova | 14:11 | |
| mriedem | you'd use 'parallels' there | 14:11 |
| mriedem | andreykurilin: ^ | 14:11 |
| Spazmotic | Yeah mriedem is the man to know best there, I Think it used to have its own driver some time back but was deprecated by the Libvirt extension? | 14:11 |
| Spazmotic | something like that | 14:11 |
| Spazmotic | Any chance if some gracious core gets some free cycle they could check out this review? I'm heading to Korea in like 5 days and don't want to let it sit for 2 weeks :) https://review.openstack.org/#/c/533168/ | 14:12 |
| andreykurilin | so actually I do not know the "use-case". I was asked about the actual support and it looks like I found the answers. thanks mriedem and Spazmotic | 14:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: test if tempest-full is triggered or not https://review.openstack.org/537936 | 14:14 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM: test if tempest-full is triggered or not https://review.openstack.org/537936 | 14:14 |
| *** david-lyle has quit IRC | 14:15 | |
| *** psachin has quit IRC | 14:15 | |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 14:16 |
| mriedem | andreykurilin: mnestratov is the person to ask from the vz team about their status | 14:17 |
| mriedem | they are the maintainers of that support | 14:17 |
| *** dmsimard is now known as dmsimard|afk | 14:17 | |
| mriedem | maxim nestratov | 14:17 |
| andreykurilin | thanks! | 14:17 |
| *** yamamoto has joined #openstack-nova | 14:18 | |
| *** lyan has joined #openstack-nova | 14:19 | |
| *** r-daneel has joined #openstack-nova | 14:20 | |
| *** lyan has quit IRC | 14:24 | |
| *** edmondsw has quit IRC | 14:25 | |
| artom_ | stephenfin, kashyap, can I ask you guys to take a look at https://review.openstack.org/#/c/533805/2 when you get a chance? I *think* I got the libvirt XML part right, but would like expert second opinions :) | 14:28 |
| *** artom_ is now known as artom | 14:28 | |
| *** superdan is now known as dansmith | 14:28 | |
| *** dave-mccowan has joined #openstack-nova | 14:28 | |
| *** edmondsw has joined #openstack-nova | 14:28 | |
| artom | It's a device tagging bug :( | 14:28 |
| kashyap | artom: Will try, "try" this week is bit stressful and jam-packed | 14:29 |
| artom | kashyap, no worries, appreciate it | 14:29 |
| kashyap | Err, /me rephrases that odd sentence | 14:29 |
| kashyap | s/"try"/"try" because/ | 14:29 |
| *** belmorei_ has joined #openstack-nova | 14:29 | |
| *** yikun_jiang has quit IRC | 14:30 | |
| -openstackstatus- NOTICE: we've been able to restart zuul, and re-enqueue changes for gate. Please hold off on recheck or approves, we are still recovering. More info shortly. | 14:30 | |
| *** yikun has joined #openstack-nova | 14:30 | |
| ameeda | gibi_: please check this https://review.openstack.org/#/c/526900/, I found way to get instance_system_metadata from here https://review.openstack.org/#/c/7045/5/nova/db/api.py | 14:31 |
| *** ralonsoh_ has joined #openstack-nova | 14:32 | |
| *** belmoreira has quit IRC | 14:32 | |
| *** tssurya has quit IRC | 14:32 | |
| *** dave-mccowan has quit IRC | 14:33 | |
| *** dave-mccowan has joined #openstack-nova | 14:33 | |
| mriedem | artom: how latent is that? not a regression in queens right? | 14:34 |
| *** tssurya has joined #openstack-nova | 14:35 | |
| artom | mriedem, the PFs not being tagged thing? | 14:35 |
| *** ralonsoh has quit IRC | 14:35 | |
| artom | mriedem, it was reported to us in Newton | 14:35 |
| mriedem | artom: ok | 14:35 |
| mriedem | just weighing relative severity | 14:36 |
| mriedem | *urgency | 14:36 |
| artom | mriedem, I don't think it was ever working | 14:36 |
| artom | Based on the code | 14:36 |
| mriedem | was that a separate bp from tagged attach? | 14:37 |
| mriedem | must have been | 14:37 |
| artom | mriedem, it was the original device tagging | 14:37 |
| mriedem | oh | 14:37 |
| artom | Well | 14:37 |
| mriedem | i thought that was just ports and volumes? | 14:37 |
| artom | I don't think we ever explicitly decided what to do about PFs | 14:38 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/sriov-pf-passthrough-neutron-port-vlan.html | 14:38 |
| artom | But since a PF can be a neutron port, they're implicitly taggable? | 14:38 |
| mriedem | ^ is what i'm thinking of | 14:38 |
| artom | That's vlans, slightly different | 14:38 |
| artom | But... I don't think those are working either, btw | 14:38 |
| mriedem | great! | 14:38 |
| artom | vladikr had a thing up to fix it | 14:38 |
| artom | And his approach never made sense to me | 14:39 |
| artom | Maybe this is why | 14:39 |
| mriedem | red hat QE strikes again 2 years late | 14:39 |
| artom | (Speaking of QE, we found a problem with the scheduler bypass CVE fix for the fix) | 14:39 |
| artom | (Barely a week later!) | 14:39 |
| mriedem | the error case? | 14:39 |
| mriedem | that we fixed last week? | 14:39 |
| artom | mriedem, yeah, if you rebuild again with the same image it goes through | 14:40 |
| artom | Because there's a new image == old image check in the API | 14:40 |
| artom | I don't have an upstream bug filed yet, but here: https://bugzilla.redhat.com/show_bug.cgi?id=1539703 | 14:40 |
| openstack | bugzilla.redhat.com bug 1539703 in openstack-nova "By rebuilding twice with the same "forbidden" image one can circumvent scheduler rebuild restrictions" [Unspecified,New] - Assigned to eglynn | 14:40 |
| mriedem | oh b/c we didn't reset the image on failure in the scheduler | 14:41 |
| artom | mriedem, https://review.openstack.org/#/c/468758/ vladikr's PF vlan thing | 14:41 |
| mriedem | which is what int32bit's patch was originally doing until i talked him out of it | 14:41 |
| artom | His solution was to refresh info_cache, which seems weird | 14:41 |
| artom | mriedem, well, properly rolling the changes back isn't trivial | 14:42 |
| mriedem | artom: yeah i know, which was my argument in the patch. | 14:42 |
| mriedem | let's just revert rebuild | 14:42 |
| artom | Our of the nova api entirely? 100% behind you :) | 14:42 |
| *** amoralej|lunch is now known as amoralej | 14:42 | |
| mriedem | i'll add that bz to the queens rc etherpad | 14:43 |
| artom | mriedem, I'll file a proper launchpad soon | 14:43 |
| mriedem | ok thanks. i think this is an easy, albeit slightly ugly fix. | 14:44 |
| *** Eran_Kuris has quit IRC | 14:44 | |
| artom | We've said that, what, 3 times now? | 14:44 |
| mriedem | jianghuaw: just to confirm, basic vgpu support for xenapi is merged, correct? the remaining patches are future work for integrating with nested resource providers? | 14:45 |
| mriedem | artom: but this time it's for real | 14:45 |
| mriedem | melwitt: some updates on the cellsv1 job in stable issues over the weekend - i finally opened a bug to track it https://bugs.launchpad.net/nova/+bug/1745838 | 14:46 |
| openstack | Launchpad bug 1745838 in OpenStack Compute (nova) "legacy-tempest-dsvm-cells constantly failing on stable pike and ocata due to libvirt connection reset" [Undecided,New] | 14:46 |
| *** janki has joined #openstack-nova | 14:46 | |
| mriedem | melwitt: and it gets a bit messy, but we need to merge the changes to bump the nova-functional job timeouts, but for that to help, we need to drop the openstack-tox-functional jobs from running on nova, which depends on these changes in stable https://review.openstack.org/#/q/Ia684786d1622da7af31aa4479fc883a7c65848ff | 14:47 |
| mriedem | those changes in stable kept failing the cellsv1 job, so i stopped running the cellsv1 job on stable: https://review.openstack.org/#/c/538619/ | 14:47 |
| jianghuaw | mriedem, yes for basic vgpu support. | 14:47 |
| mriedem | jianghuaw: thanks | 14:48 |
| *** hoangcx has quit IRC | 14:49 | |
| *** salv-orlando has quit IRC | 14:49 | |
| *** hoangcx has joined #openstack-nova | 14:50 | |
| *** salv-orlando has joined #openstack-nova | 14:50 | |
| *** burt has joined #openstack-nova | 14:50 | |
| bauzas | mriedem: jianghuaw: I'm working on a matrix patch that'll add the vgpu support | 14:51 |
| bauzas | mriedem: jianghuaw: I'll CC you the patch once I'm done | 14:51 |
| jianghuaw | bauzas, thanks. | 14:51 |
| *** lyan has joined #openstack-nova | 14:51 | |
| *** esberglu has joined #openstack-nova | 14:51 | |
| bauzas | I also have in my pipe a possible functional test for testing server actions with VGPUs | 14:51 |
| bauzas | but it requires me some fixture investigation | 14:52 |
| jianghuaw | bauzas, I think display head is not in queens' plan. right? Although I have a patch for that. | 14:52 |
| jianghuaw | https://review.openstack.org/#/c/523342 | 14:52 |
| *** sahid has quit IRC | 14:52 | |
| bauzas | jianghuaw: Queens is done now | 14:52 |
| bauzas | for features I mean | 14:52 |
| *** gcb has joined #openstack-nova | 14:52 | |
| sauloaugusto | Is there any channel for mova upgrade support ? | 14:53 |
| jianghuaw | I see. the freeze-date has passed. | 14:53 |
| jianghuaw | bauzas, I don't expect that can be merged:-) | 14:54 |
| mriedem | melwitt: and at this point, ironically the nova-functional patches on stable are failing b/c of the openstack-tox-functional job timeouts, which we're trying to fix here, but we can't recheck anything until zuul is fixed | 14:54 |
| *** salv-orlando has quit IRC | 14:55 | |
| *** mvenesio has quit IRC | 14:55 | |
| *** Eran_Kuris has joined #openstack-nova | 14:57 | |
| mriedem | melwitt: oh btw, the cellsv1 job failures started around 1/18 which is around the same time as that python2.7 update in xenial repos, which was causing other gate instability related to eventlet not monkey patching os.open, | 14:57 |
| mriedem | so i wondered if maybe there is some issue in libvirt-python or something in the older versions we're using on stable | 14:57 |
| *** sahid has joined #openstack-nova | 15:00 | |
| *** takashin has left #openstack-nova | 15:00 | |
| efried | cdent You've got me confused. | 15:01 |
| jaypipes | efried, cdent: reading the vmware mail thread now.. | 15:01 |
| *** yangyapeng has quit IRC | 15:01 | |
| *** lucasagomes is now known as lucas-hungry | 15:01 | |
| cdent | efried: that's my goal in life | 15:01 |
| efried | jaypipes cdent Note there's a followup that did go to the -dev ML. | 15:01 |
| *** gibi_ is now known as gibi | 15:01 | |
| efried | cdent The part I'm confused about is: where were we ever talking about "negative traits"? | 15:02 |
| cdent | we weren't | 15:02 |
| jaypipes | efried: there's no such thing as a negative trait. | 15:02 |
| *** r-daneel_ has joined #openstack-nova | 15:02 | |
| jaypipes | efried: something either has or doesn't have a trait. | 15:02 |
| *** zhaochao has quit IRC | 15:02 | |
| cdent | we were talking about how to not consume resources that have a trait when we don't care about that trait | 15:02 |
| efried | We were? | 15:03 |
| jaypipes | cdent: resources don't have traits. providers have traits. | 15:03 |
| cdent | jaypipes: I know, but naming it hard | 15:03 |
| jaypipes | cdent: did you mean consuming resource from a provider that doesn't have a trait? | 15:03 |
| cdent | jaypipes: no | 15:03 |
| cdent | let me explain | 15:03 |
| cdent | given two providers with equal inventory, one has trait X | 15:04 |
| jaypipes | efried: do you want me to respond to the ML part and not the private email? | 15:04 |
| *** r-daneel has quit IRC | 15:04 | |
| *** r-daneel_ is now known as r-daneel | 15:04 | |
| cdent | we make a query for allocation candidates that both poviders can satisfy | 15:04 |
| *** slaweq has joined #openstack-nova | 15:05 | |
| cdent | how do we say "don't use the provider that has trait X" | 15:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova stable/pike: tests: Use correct response type in tests https://review.openstack.org/538938 | 15:05 |
| efried | Perhaps it would be best if cdent replied to the ML one with whatever's still relevant from the private thread; then you can respond to that. Keep the thread linear-ish. | 15:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova stable/pike: Handle images with no data https://review.openstack.org/538939 | 15:05 |
| gibi | ameeda: sure, you can get sysmeta from the db, but I thought, based on the comments, that you are trying to find a way to get sysmeta via the REST API | 15:05 |
| jaypipes | cdent: you don't. you just don't include the trait in required_traits= param. | 15:05 |
| jaypipes | cdent: it's a sorting/weighing/preference problem, not a filtering problem. | 15:05 |
| *** awaugama has joined #openstack-nova | 15:05 | |
| cdent | jaypipes: right, that's basically what I say at the end of my message | 15:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova stable/pike: Handle images with no data https://review.openstack.org/538939 | 15:05 |
| jaypipes | gotcha | 15:05 |
| efried | cdent But I don't see how that's relevant to the discussion | 15:06 |
| efried | I totally didn't pick up on any part of rado's original note that implied we needed/wanted to do that. | 15:06 |
| mriedem | stephenfin: question in that backport | 15:07 |
| cdent | efried: rado was expressing concern about allocations being made to the wrong pool when not expressing the trait | 15:07 |
| jaypipes | efried: the fundamental problem with the vmware conversation is that resource pools are being represented with nested providers instead of aggregates. aggregates are the thing that provides the "grouping" concept that the resource pools represent. nested providers represent a different relationship: that of dependent trees of providers. | 15:07 |
| *** mdnadeem has quit IRC | 15:07 | |
| cdent | "is there a way to make the scheduler allocate only from one specific RP if no custom trait is specified in the request" | 15:07 |
| efried | Heh. Oh. I took a much simpler approach to that question. It went like this: "No." | 15:08 |
| cdent | The complicating factor here is that I think nested is _exactly_ the model for how resource pool are supposed to work | 15:08 |
| efried | jaypipes ++ and in fact trying to use NRP for pools will break horribly | 15:08 |
| efried | oh, wow, okay, clearly starting on opposite ends of the rope. | 15:09 |
| efried | cdent Can you describe that hierarchy please? | 15:09 |
| cdent | because they represent a hierarcy of containment | 15:09 |
| *** slaweq has quit IRC | 15:10 | |
| *** yamahata has joined #openstack-nova | 15:10 | |
| cdent | so, nrp came about in part because of numa and pci relationships, right? | 15:10 |
| efried | cdent Have you seen the response I made to the ML yet? I want to baseline and make sure I've got my idea of the hierarchy down before we go any further. | 15:10 |
| cdent | efried: yes, I've read it twice and still trying to make sure i understand it | 15:10 |
| efried | k | 15:10 |
| *** moshele has quit IRC | 15:11 | |
| *** r-daneel has quit IRC | 15:11 | |
| stephenfin | mriedem: Reply left. tl;dr: I think we're fine for stable/pike | 15:11 |
| *** r-daneel has joined #openstack-nova | 15:11 | |
| stephenfin | assuming I'm reading g-r correctly | 15:11 |
| mriedem | we just need 2.8.0? | 15:12 |
| *** hongbin has joined #openstack-nova | 15:12 | |
| cdent | the thing about your model which is not necessarily desirable is the way in which esxi hosts are managed as a concrete resources. vcenter doesn't really want to do that (although it is possible and would be a reasonable fallback) | 15:12 |
| stephenfin | mriedem: 2.6.0+ | 15:12 |
| *** gus has quit IRC | 15:13 | |
| efried | jaypipes The aggregate model won't allow you to request a deploy to a specific resource pool. But maybe I'm misunderstanding the original intent, which I thought was to do just that. | 15:13 |
| *** Eran_Kuris has quit IRC | 15:13 | |
| cdent | the intent is to deploy to a specific resource pool | 15:13 |
| *** eharney has joined #openstack-nova | 15:13 | |
| cdent | if you have an aggregate for each specific resource pool, that might be workabe | 15:14 |
| *** gus has joined #openstack-nova | 15:14 | |
| efried | But you still can't deploy to a specific resource pool with just that. | 15:14 |
| cdent | brb | 15:14 |
| efried | You would have to have some other mechanism in place. | 15:14 |
| mriedem | can you add a custom filter and custom scheduler hint which is just 'make it go directly here' | 15:14 |
| jaypipes | efried: uhm, yeah you could request a specific aggregate... | 15:15 |
| efried | jaypipes How? | 15:15 |
| efried | With some mechanism yet to be designed? | 15:15 |
| jaypipes | efried: we already have the member_of param. we can work that in to alloc candidates very easily. | 15:15 |
| *** lajoskatona has quit IRC | 15:15 | |
| efried | okay, agree, but not yet designed or started. | 15:16 |
| jaypipes | efried: of course, that would require the users knowing about the aggregate UUID/names... | 15:16 |
| jaypipes | efried: which generally we have not encouraged. | 15:16 |
| mriedem | stephenfin: got it | 15:16 |
| mriedem | thanks | 15:16 |
| mriedem | there is no aggregate name https://developer.openstack.org/api-ref/placement/#list-resource-provider-aggregates | 15:17 |
| jaypipes | mriedem: sorry, UUID... | 15:17 |
| jaypipes | efried: still, what is the root of the problem that is trying to be solved? | 15:17 |
| cdent | It remains unclear to me why (eventually) nested isn't the ideal solution for this stuff? A compute-node is an rp, it cotains resource pool rps and maybe even some exsi rp, a hybrd representing the stuff managed by the conpute node | 15:18 |
| jaypipes | efried: I mean, are we doing scheduling here or are we just doing resource tracking and vcenter basically takes over the scheduling/placement decisions (that's what I *thought* the situation was currently...) | 15:18 |
| mriedem | sounds like the latter | 15:18 |
| stephenfin | mriedem: Is this something that should wait for a few weeks? https://review.openstack.org/#/c/534371/ | 15:19 |
| mriedem | stephenfin: is that just cleanup? if so, yes | 15:19 |
| efried | sorry guys, pulled into a quick meeting, back in a bit. | 15:19 |
| mriedem | stephenfin: we're in bug fix only mode at this point | 15:19 |
| stephenfin | mriedem: (y) | 15:19 |
| mriedem | or tests, docs, etc | 15:19 |
| *** hoangcx has quit IRC | 15:20 | |
| *** hoangcx has joined #openstack-nova | 15:21 | |
| *** bhagyashris has quit IRC | 15:21 | |
| *** bhagyashris has joined #openstack-nova | 15:21 | |
| *** pooja-jadhav has joined #openstack-nova | 15:22 | |
| cdent | jaypipes: the underlying feature that rado wants to expose here are resource pools and he wants to do it in a way that allows placement to be aware of them | 15:22 |
| *** Supun has quit IRC | 15:22 | |
| *** awaugama has quit IRC | 15:22 | |
| jaypipes | cdent: will the *end user* be capable of a) understanding what these resource pools are and b) asking for a particular resource pool to be used by an instance? | 15:22 |
| cdent | jaypipes: yes. "tenant virtual data center" | 15:23 |
| *** hemna_ has quit IRC | 15:23 | |
| *** stvnoyes has joined #openstack-nova | 15:23 | |
| stephenfin | lyarwood (or another stable-core): fancy hitting these? https://review.openstack.org/#/c/538939/ https://review.openstack.org/#/c/538938/ | 15:23 |
| *** pooja_jadhav has quit IRC | 15:23 | |
| jaypipes | cdent: why not have multiple nova-compute services, each managing a single "resource pool"? | 15:24 |
| *** Supun has joined #openstack-nova | 15:25 | |
| *** awaugama has joined #openstack-nova | 15:26 | |
| cdent | because the clusters behind a nova-compute service are intentionally dynamic: it's desirable to be able to create destroy and change resource pools in the cluster without messing with the openstack control plane much | 15:26 |
| *** moshele has joined #openstack-nova | 15:26 | |
| efried | I'm back. | 15:27 |
| efried | cdent Do these clusters / resource pools (are those different things) contain single compute-ish nodes to which instances get deployed, OR do they contain some general pool of resources from which an instance could be deployed using some resource from one member and some from another? | 15:27 |
| jaypipes | cdent: that's problematic for many reasons, as you know. | 15:28 |
| *** tidwellr has joined #openstack-nova | 15:28 | |
| *** mlavalle has joined #openstack-nova | 15:28 | |
| cdent | jaypipes: yes, but problematic is how we push the ball forward, no? efried cluster and resource pools are different, yes. logically an instance is deployed to a resource pool and not a specific host (although physically they are, of course) | 15:29 |
| cdent | a cluster contains 1 or more resource pools | 15:30 |
| cdent | a resource pool has access to some (dynamic) subsection of a clusters inventory | 15:30 |
| efried | Are physical hosts the underlying members of a resource pool? | 15:31 |
| cdent | efried: that's not really the way it is modeled, no, but I'm not sure what you're actually asking | 15:32 |
| efried | cdent I'm just not understanding where the physical resources are here. | 15:32 |
| jaypipes | cdent: what does the vmware virt driver *currently* do? does it report the entire amount of VCPU, memory, etc for all resource pools and all clusters that vCenter knows about? | 15:32 |
| *** liangy has joined #openstack-nova | 15:33 | |
| *** mylu has joined #openstack-nova | 15:33 | |
| jaypipes | cdent: and the scheduler always picks the single nova-compute service daemon and basically all scheduling/placement is simply deferred to the vCenter API? | 15:33 |
| cdent | right now: the vcenter represents one cluster, and reports all vcpu, memory, etc that it knows abouts | 15:33 |
| cdent | but there are a multiple nova-copute services, each for a given cluster | 15:34 |
| jaypipes | cdent: I'm not seeing the benefit of modeling things using placement's resource providers data modeling if deployments using vCenter won't actually end up using any of that information when doing placement/scheduling decisions anyway. | 15:34 |
| *** lyan has quit IRC | 15:35 | |
| *** alexchadin has quit IRC | 15:37 | |
| cdent | jaypipes: two reasons a) if you have multiple clusters, and thus multiple nova-compues, and those clusters differet in various ways (for example by having different tenant virtual data centers) you need to scheduler differently, b) despite our troubles, we've made a very good resource/provider/inventory/etc modelling system with placement and it is a good place to track and select where to place things | 15:37 |
| cdent | "this cluster over there is the one with all the GPUs" | 15:38 |
| *** pooja-jadhav has quit IRC | 15:38 | |
| efried | cdent It'll help me a lot to understand what is underneath a resource pool. If a resource pool reports a total of 8 VCPUs available, does that mean I am guaranteed to be able to satisfy a deploy requesting 8 VCPUs? (All other things being available) | 15:38 |
| efried | Cause I got the impression that those 8 VCPUs might be spread across say four physical hosts, and an instance could only go to one of those, so the biggest one you could handle might only be 2 VCPUs or whatever. | 15:39 |
| *** r-daneel_ has joined #openstack-nova | 15:39 | |
| *** r-daneel has quit IRC | 15:40 | |
| *** r-daneel_ is now known as r-daneel | 15:40 | |
| * efried pauses to give props to r-daneel for having an awesome nick | 15:40 | |
| jaypipes | cdent: for a) if the end user is just saying "I'm in tenant X, so launch an instance on virtual data center X", that is a stupid-simple scheduling decision that needs nothing other than a single trait (CUSTOM_TENANT_X decorates a single resource provider representing the nova-compute service that manages that tenant virtual data center). For b) again, if vCenter isn't going to use that information in its intra-cluster scheduling decisions, I | 15:40 |
| jaypipes | don't see the point. | 15:40 |
| *** bhagyashris has quit IRC | 15:40 | |
| jaypipes | cdent: another question for you.. | 15:42 |
| cdent | jaypipes: can I pause you for a minute while I answer efried, then your current question? | 15:42 |
| *** sree has joined #openstack-nova | 15:43 | |
| jaypipes | cdent: even if placement ended up getting proper inventory information for these vCenter clusters, would VMWare OpenStack users actually be able looking at that inventory information? Or would they be using VMWare GUIs to look at inventory information for their specific tenant virtual datacenter? | 15:43 |
| *** efried1 has joined #openstack-nova | 15:43 | |
| jaypipes | cdent: of course, no worries | 15:43 |
| cdent | efried: a while back there were some adjustment made to the vmware driver to make sure that the max_unit was no greater than what any individual host could provide. so your second statement about "the biggest one" is an issue in some cases, but it's squishy because of the DRS (dynamic resource scheduler thing which can do unattended live migrations and what not, magic stuff I haven't dug into) | 15:44 |
| efried | So it is in fact the case that there are some number of physical hosts behind a resource pool, and an instance can only be deployed to one, not spread out across more than one. True statement? | 15:46 |
| cdent | efried: so, yes, there is a fair bit of complexity which is _why_ being able to manipulate the entire provider tree from within the virt driver has been of such interest to rado (and me). | 15:46 |
| *** claudiub has joined #openstack-nova | 15:46 | |
| jaypipes | maciejjozefczyk: you around? wanted to chat with you about https://bugs.launchpad.net/nova/+bug/1742747 | 15:46 |
| openstack | Launchpad bug 1742747 in OpenStack Compute (nova) "RT overrides default allocation_ratios for ram cpu and disk" [Undecided,In progress] - Assigned to Maciej Jozefczyk (maciej.jozefczyk) | 15:46 |
| cdent | efried: it can move and you can never know that it did | 15:46 |
| efried | "it" the physical host? "can move" to another resource pool? and "you" the virt driver or you the operator? | 15:47 |
| *** gcb has quit IRC | 15:47 | |
| jaypipes | cdent: with the shared-memory solution, can't VMs literally be on multiple hypervisor hosts at once? | 15:48 |
| maciejjozefczyk | jaypipes: yes, hey! | 15:48 |
| jaypipes | maciejjozefczyk: you see my note on that bug? | 15:48 |
| *** claudiub|3 has joined #openstack-nova | 15:48 | |
| *** esberglu has quit IRC | 15:48 | |
| *** pooja_jadhav has joined #openstack-nova | 15:49 | |
| *** moshele has quit IRC | 15:49 | |
| *** hemna_ has joined #openstack-nova | 15:49 | |
| *** claudiub|2 has quit IRC | 15:49 | |
| maciejjozefczyk | jaypipes: reading | 15:49 |
| *** esberglu has joined #openstack-nova | 15:49 | |
| cdent | jaypipes: I've done my damnedest thus far to stay out of the guts of what vcenter gets up to, but there's plenty of magical knobs, but I'm trying to avoid knowing about them. there's already too many things to know | 15:49 |
| *** namnh has joined #openstack-nova | 15:49 | |
| jaypipes | cdent: ack | 15:49 |
| cdent | efried: it is the instance | 15:50 |
| *** bhagyashris has joined #openstack-nova | 15:50 | |
| efried | okay. "move" where, inter-pool or just intra-pool? | 15:50 |
| efried | not that it actually matters. | 15:50 |
| *** salv-orlando has joined #openstack-nova | 15:51 | |
| *** claudiub has quit IRC | 15:51 | |
| maciejjozefczyk | jaypipes: so we should also check aggregation metadatas if there is already definition of allocation ratio defined? | 15:51 |
| maciejjozefczyk | if zeros are in config? | 15:51 |
| jaypipes | maciejjozefczyk: yeah. check out that operators mailing list thread I linked in there... | 15:51 |
| cdent | efried: intra-cluster. | 15:52 |
| efried | cdent So the thing is, we didn't have a way to model multiple providers before. Now we do. But it sounds to me like nested is *not* the multiple-provider model we want to use here. It just doesn't make sense in light of recently-merged capabilities for a single provider *or* tree to "own" inventory spanning more than one actual compute host. (Unless, as jaypipes implies, there's a memory sharing thing - we can talk abou | 15:52 |
| efried | t that later.) | 15:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add regression test wrinkle for rebuilding twice with invalid image https://review.openstack.org/538955 | 15:52 |
| mriedem | artom: ^ | 15:52 |
| *** mylu has quit IRC | 15:53 | |
| efried | cdent If you do it in a single provider OR a tree, you'll always have to reconcile a request for some subset of the total resources that you can't actually satisfy from any one host. | 15:54 |
| artom | mriedem, I think you'd have to get the server out of ERROR first | 15:54 |
| *** hemna_ has quit IRC | 15:54 | |
| * artom files the launchpad | 15:54 | |
| *** mylu has joined #openstack-nova | 15:55 | |
| efried | cdent With the single-provider model, which I gather is how it has needed to be done to this point, it sounds like some of that reconciliation was that max-unit hack. | 15:55 |
| jaypipes | efried: yeah. I think it's a much better solution for vCenter to model their resource tracking in the vCenter virt driver in a similar fashion to how Ironic's virt driver works. | 15:55 |
| *** salv-orlando has quit IRC | 15:55 | |
| cdent | efried: I have no issue with the inventory for hardware things being pushed all the way down the real hardware, but I _do_ think that representing the hierarchical relationships between things using a tool we've created that has hierarchy makes sense (in this case cluster->resource-pool->physical host) | 15:55 |
| jaypipes | efried: that wouldn't *preclude* using nested providers if, say, certain compute nodes had PCI devices or NUMA nodes, etc. But I would use a new root provider for each compute node in the cluster and represent the cluster as a placement aggregate, not as a child node under a "fake root" | 15:56 |
| efried | jaypipes ++ exactly | 15:56 |
| *** david-lyle has joined #openstack-nova | 15:56 | |
| jaypipes | cdent: I think we're mostly agreeing. just disagreeing on the "single root vs multiple roots" thing. | 15:57 |
| *** sree_ has joined #openstack-nova | 15:57 | |
| gibi | mriedem: quick question: There is a bugfix https://review.openstack.org/#/c/525242 that needs rpc version bump. Is it something that needs to wait for Rocky? | 15:57 |
| *** felipemonteiro has joined #openstack-nova | 15:57 | |
| efried | cdent A placement tree is more than just a way to represent a hierarchy. It is also allows resource requests to span a tree. Which doesn't make sense in the vcenter model as I'm undestanding it. | 15:57 |
| *** sree_ is now known as Guest64456 | 15:57 | |
| mriedem | artom: you can rebuild an ERROR server | 15:57 |
| cdent | efried: then I think it probably hasn't been explained well enough | 15:58 |
| *** eharney has quit IRC | 15:58 | |
| artom | mriedem, hah, that makes it even easier to hit | 15:58 |
| mriedem | gibi: that doesn't necessarily mean it has to wait for rocky, | 15:58 |
| mriedem | gibi: but it does mean we wouldn't backport it | 15:58 |
| maciejjozefczyk | jaypipes: I'm ok with the idea to check aggregation metadatas first | 15:58 |
| *** ircuser-1 has joined #openstack-nova | 15:58 | |
| gibi | mriedem: OK. Then I'm going to rebase the above patch | 15:59 |
| gibi | mriedem: the backport impossibility was agreed earlier so that is clear | 15:59 |
| jaypipes | maciejjozefczyk: ok. with your operator hat on, what do you think about my proposal, considering it involves a technically not backwards-compatible behaviour change? | 15:59 |
| *** sree has quit IRC | 15:59 | |
| *** belmorei_ has quit IRC | 15:59 | |
| efried | cdent I think I get that logically, conductor < cluster <=> n-cpu < resource pool < compute hosts (right?) and from that perspective you want to be able to "model" that structure as a tree. | 15:59 |
| *** lucas-hungry is now known as lucasagomes | 15:59 | |
| cdent | efried, jaypipes: I'd be much better at this conversation if I wasn't still fighting off this cold, but it's not just the desired to model, there's also a desire to do some accounting | 16:00 |
| * cdent tries to decode efried's symbols | 16:00 | |
| *** moshele has joined #openstack-nova | 16:00 | |
| * cdent fails | 16:00 | |
| efried | cdent 1 < many and 1 <=> 1 | 16:00 |
| *** eharney has joined #openstack-nova | 16:01 | |
| *** tidwellr has quit IRC | 16:01 | |
| maciejjozefczyk | jaypipes: From my perspective it would not change anyting, I set allocation_ratio per host not per aggregate, so my opinion seems not be helpful | 16:01 |
| *** felipemonteiro has quit IRC | 16:01 | |
| jaypipes | mgagne: hey! :) | 16:01 |
| mgagne | jaypipes: hi! | 16:02 |
| jaypipes | mgagne: we're talking about your favorite new anti-feature regarding host aggregates and allocation ratios :) | 16:02 |
| mgagne | jaypipes: I just saw your email =) | 16:02 |
| jaypipes | mgagne: cool. I'm hoping to commandeer maciejjozefczyk's patch to add this fix for you. | 16:02 |
| jaypipes | of course, I'm trying to convince maciejjozefczyk that this would be of benefit ;) | 16:03 |
| mgagne | jaypipes: I have yet to properly analyze your suggestion and related bug report. | 16:03 |
| jaypipes | mgagne: no worries, would be good to get your feedback some time today. | 16:03 |
| *** tidwellr has joined #openstack-nova | 16:03 | |
| jaypipes | mgagne: I'm going to ping melwitt on it when she's up, too. | 16:03 |
| mgagne | jaypipes: sure | 16:03 |
| *** tesseract has quit IRC | 16:03 | |
| sauloaugusto | Is it possible to setup pike without cells ? | 16:04 |
| openstackgerrit | Sen Yang proposed openstack/python-novaclient master: Implement hypervisor hostname exact patten match for server cold migrate https://review.openstack.org/520187 | 16:04 |
| jaypipes | sauloaugusto: nope. | 16:04 |
| *** namnh has quit IRC | 16:04 | |
| efried | cdent So I think where we're getting to is this: You could have your nice satisfying model, and have to hack the inventory distribution to make it work; or you could have a flatter model that's less closely representative of the "real world", but your inventorying is simple, and accounting flows automatically from scheduling/allocations. | 16:04 |
| *** jappleii__ has quit IRC | 16:05 | |
| efried | cdent ...except for the part where vcenter is gonna move an instance from one host to another. Yeah, that gets a tad hairy without having done a real live migration or whatever. | 16:05 |
| sauloaugusto | jaypipes: Is there any clue about the error when you migrate from a no cell env to pike and you can only list instances from admin tenant ? | 16:05 |
| dansmith | sauloaugusto: that doesn't sound like a cells-related thing | 16:06 |
| *** hemna_ has joined #openstack-nova | 16:06 | |
| *** jmlowe_ has quit IRC | 16:07 | |
| dansmith | (the cells code doesn't do anything special with users/tenants) | 16:07 |
| cdent | efried: having allocations against physical hosts (instead of the cluster) cannot work because instances move and we've declared the virt driver is not allowed to manipulate allocations | 16:07 |
| efried | cdent Yeah, I see where that's a problem. I don't have an answer for that. | 16:07 |
| *** jmlowe has joined #openstack-nova | 16:07 | |
| cdent | thus inventory on resource pools | 16:08 |
| efried | and resultant hackage. | 16:08 |
| efried | which is unfortunate. | 16:08 |
| maciejjozefczyk | jaypipes mgagne: Need to go. Let me know in bug report about what you decided okey ;) I'll update the patch then | 16:08 |
| cdent | because that is the level at which accounting is desired | 16:08 |
| *** fragatina has quit IRC | 16:08 | |
| jaypipes | maciejjozefczyk: will do, thanks mate :) | 16:08 |
| *** lyan has joined #openstack-nova | 16:08 | |
| *** maciejjozefczyk is now known as maciejjozefczyk_ | 16:08 | |
| efried | cdent Well, at the very least the resource pools should not be children in the same tree. They should be roots. | 16:09 |
| * efried1 discovers that Thunderbird has an IRC client. | 16:10 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Rollback instance.image_ref on failed rebuild https://review.openstack.org/538961 | 16:10 |
| mriedem | artom: here is the fix ^ let me know when you have an LP bug number so i can update the references in here | 16:10 |
| artom | mriedem, yep, writing it up as we speak | 16:10 |
| cdent | efried: remind me (because I'm half asleep) of the mechanism to go from a root (when in nova-scheduler) to the compute node where the spawn can be called? | 16:11 |
| sauloaugusto | dansmith: I think that is related , because the error start after I do themigration steps from nova . After that I start to see the Can't upgrade a READER transaction to a WRITER mid-transaction error on nova-api.log . | 16:11 |
| cdent | efried: if the root is in fact not that compute-monde | 16:11 |
| artom | mriedem, so we're rolling back the image but not other things that could change, like keypair? | 16:11 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | 16:11 |
| openstack | bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer) | 16:11 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242 | 16:11 |
| mriedem | artom: yeah - which was my original complaint in the earlier fix | 16:11 |
| dansmith | sauloaugusto: the migration steps from no cells to one cell were required before pike | 16:12 |
| efried | cdent Not sure we quite have that mechanism yet. | 16:12 |
| mriedem | artom: we can't really roll everything back with what we have today in the conductor code | 16:12 |
| mriedem | we'd either have to change the rpc cast to a call, or move the code that changes the instance from the api to conductor | 16:12 |
| cdent | efried: indeed, thus why I've been modelling it the way I've been | 16:12 |
| efried | cdent Yeah, a way to tie these brother-roots back to the compute host for purposes of allocation and deploy. | 16:12 |
| cdent | if it is a child you know where to go | 16:12 |
| mriedem | hell, maybe we should have just moved the instance.save() from api to conductor... | 16:12 |
| artom | mriedem, yeah... I guess it's fine as a bandaid for now? Ideally we need to come back to this and fix it properly, maybe like what Andrey was proposing | 16:12 |
| cdent | because | 16:12 |
| cdent | it is a child | 16:12 |
| mriedem | artom: yes this is a "tactical fix" | 16:13 |
| cdent | a resource provider withing a compute-node is _obviously_ a child | 16:13 |
| efried | cdent But if it's a child, the scheduler will happily deploy an instance with VCPU from pool1, MEMORY_MB from pool2, and DISK_GB from pool3. | 16:13 |
| cdent | and it seems to me that the ironic model might be the thing that's weird | 16:13 |
| artom | mriedem, heh, code for "someone else can deal with it once I'm retired" ;) | 16:13 |
| efried | cdent Unless you *always* use a numbered granular group. Which doesn't seem like the right answer. | 16:13 |
| mriedem | maybe - it's just something one of my old managers at ibm always said, | 16:13 |
| cfriesen | jaypipes: for https://bugs.launchpad.net/nova/+bug/1742747 do we look at the cpu_allocation_ratio of the first host aggregate, or do we look for the first host aggregate that has cpu_allocation_ratio set? | 16:13 |
| openstack | Launchpad bug 1742747 in OpenStack Compute (nova) "RT overrides default allocation_ratios for ram cpu and disk" [Undecided,In progress] - Assigned to Maciej Jozefczyk (maciej.jozefczyk) | 16:13 |
| mriedem | "do we have a tactical fix while we work on the long-term strategic fix" | 16:13 |
| *** slaweq has joined #openstack-nova | 16:14 | |
| artom | mriedem, not a bad way of thinking | 16:14 |
| efried | I was doing a word puzzle the other day where the clue was "strategic" and the answer was "tactical". I thought how the IBM ppt-jockeys would flip out at that. | 16:14 |
| *** slaweq has quit IRC | 16:14 | |
| artom | awaugama, you around? Since you're the cause of all this, if we build you our proposed tactical fix, you want to try and break it again? | 16:14 |
| sauloaugusto | dansmith: Yes I did that, and I get get list o admin instances , and also create new instances at that . The problem that I can not do nothing at all other tenants . | 16:14 |
| artom | Maybe if we let QE loose on it *before* merging it, we'll avoid the pain? | 16:14 |
| awaugama | artom: I can do some sanity checks on it | 16:15 |
| mriedem | artom: that never seems to happen | 16:15 |
| mriedem | until it's in product | 16:15 |
| *** ircuser-1 has quit IRC | 16:15 | |
| awaugama | artom, and *Technically* I'm just the warning light, i'm not the cause of the issue | 16:15 |
| artom | awaugama, hah, sorry, should have added a ;) or /s in there somewher e:) | 16:15 |
| awaugama | ;) | 16:15 |
| cdent | efried: these issues are why I pressed rado to start talking about and experimenting with this stuff. I think the sort of gravitational mechanics of nested providers is going to have lots of weird | 16:15 |
| cdent | and I'd much prefer to see us working from concrete situations then not | 16:16 |
| efried | cdent Agree. At the moment I'm actually leaning towards the thing I originally said was the worst idea: always using a single numbered request group. | 16:16 |
| *** mylu has quit IRC | 16:17 | |
| Spazmotic | Always good when the best idea is your worst idea | 16:17 |
| *** felipemonteiro has joined #openstack-nova | 16:18 | |
| *** links has quit IRC | 16:20 | |
| cdent | efried: I guess we have time to experiement... | 16:21 |
| *** felipemonteiro_ has joined #openstack-nova | 16:21 | |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Queens_Release_Schedule" | 16:21 | |
| -openstackstatus- NOTICE: zuul.o.o is back online, feel free to recheck / approve patches. | 16:21 | |
| *** moshele has quit IRC | 16:22 | |
| *** chyka has joined #openstack-nova | 16:22 | |
| *** chyka has quit IRC | 16:22 | |
| *** chyka has joined #openstack-nova | 16:23 | |
| *** felipemonteiro has quit IRC | 16:25 | |
| artom | mriedem, https://bugs.launchpad.net/nova/+bug/1746032 | 16:25 |
| openstack | Launchpad bug 1746032 in OpenStack Compute (nova) "By rebuilding twice with the same "forbidden" image one can circumvent scheduler rebuild restrictions" [Undecided,New] | 16:25 |
| *** pcaruana has quit IRC | 16:25 | |
| mriedem | artom: L145 https://etherpad.openstack.org/p/nova-ptg-rocky | 16:27 |
| *** yamamoto has quit IRC | 16:28 | |
| artom | mriedem, oh hey and I'll actually be there this time :) | 16:28 |
| mriedem | we can finally meet, face to face, and settle all the scores | 16:28 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.openstack.org/512552 | 16:28 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env https://review.openstack.org/512553 | 16:28 |
| openstack | bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Undecided,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer) | 16:28 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor https://review.openstack.org/512623 | 16:28 |
| mriedem | there can be only one | 16:28 |
| artom | ... one of *what*? | 16:28 |
| mriedem | idk, i'm just thinking about the quickening | 16:29 |
| mriedem | even though that was scotland | 16:29 |
| artom | Man, your references are out of control! | 16:29 |
| *** yangyapeng has joined #openstack-nova | 16:30 | |
| *** rmcall has joined #openstack-nova | 16:31 | |
| edleafe | cdent: I've been following along, but don't have a clear picture of how things are arranged. Is there a diagram or something that would illustrate this? | 16:31 |
| cdent | edleafe: a) which "this"?, b) I wish I never showed up today because I simply don't have the capacity to actual engage well with this topic | 16:32 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Rollback instance.image_ref on failed rebuild https://review.openstack.org/538961 | 16:32 |
| mriedem | artom: ok here we go, single patch ^ | 16:32 |
| edleafe | cdent: the arrangement of entities in vmware | 16:32 |
| edleafe | and which is the "root" that placement would be selecting | 16:33 |
| mriedem | stable/newton is still open....but not sure we want to hold up EOL for this fix | 16:34 |
| *** yangyapeng has quit IRC | 16:34 | |
| *** slaweq has joined #openstack-nova | 16:34 | |
| Spazmotic | Korea in.. 6 days | 16:35 |
| Spazmotic | Whew | 16:35 |
| dansmith | man, newton is like the undead release | 16:36 |
| efried | edleafe I was considering drawing that up, along with an executive summary of this conversation, and putting it all in a response to the ML thread. | 16:36 |
| artom | Can we call U 'undead'? As an hommage? | 16:36 |
| cdent | edleafe: not that I'm aware of at this time, but I'll check with rado. if efried is able to do something (even if it has errors), that provides at least a starting point | 16:36 |
| *** sahid has quit IRC | 16:37 | |
| cdent | the "root" I was thinking of was "compute-node" | 16:37 |
| mriedem | i'll say it again, the path to LTS upstream is just never EOL'ing anything | 16:37 |
| edleafe | cdent: ok, thanks. I assumed that there must be some internal documentation that would be helpful | 16:37 |
| *** sahid has joined #openstack-nova | 16:37 | |
| *** andreas_s has quit IRC | 16:37 | |
| *** andreas_s has joined #openstack-nova | 16:38 | |
| cdent | edleafe: could very well be, but if there are pretty pictures I haven't seen them. That doesn't mean it doesn't exist. I interact _very_ little on that side of things. | 16:38 |
| Spazmotic | Anyone know any of the cores familiar with XenAPI drivers? Besides Garbutt that is, I got his attention on but was curious if any others were familiar that I could poke :D | 16:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add type annotations to 'nova.pci' https://review.openstack.org/538224 | 16:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: mypy: Add 'types' module https://review.openstack.org/538225 | 16:39 |
| edleafe | cdent: ok, cool. If you come across something, post it so we can all better understand what is needed | 16:39 |
| *** felipemonteiro_ has quit IRC | 16:40 | |
| efried | Spazmotic Maybe bauzas ? | 16:41 |
| artom | Wasn't Bob Ball xenapi? Is he still around? | 16:41 |
| Spazmotic | I know the driver isn't looked at too frequently by folks, so just didn't want to bark up the wrong trees wwhen I had questions or was looking for some eyeballs. | 16:41 |
| Spazmotic | Yeah bob's still around but he's no longer core | 16:41 |
| *** andreas_s has quit IRC | 16:41 | |
| Spazmotic | He always gives me good ideas though so.. love me some Bob | 16:42 |
| *** andreas_s has joined #openstack-nova | 16:42 | |
| *** andreas_s has quit IRC | 16:42 | |
| *** jose-phillips has quit IRC | 16:43 | |
| *** andreas_s has joined #openstack-nova | 16:43 | |
| *** yamamoto has joined #openstack-nova | 16:43 | |
| *** jose-phillips has joined #openstack-nova | 16:44 | |
| *** jpena is now known as jpena|brb | 16:44 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Rollback instance.image_ref on failed rebuild https://review.openstack.org/538961 | 16:47 |
| mriedem | artom: jaypipes: added a variable for the original image ref and a comment about CastAsCall | 16:47 |
| *** slaweq_ has joined #openstack-nova | 16:47 | |
| *** tesseract has joined #openstack-nova | 16:47 | |
| mriedem | Spazmotic: johnthetubaguy knows the xenapi stuff | 16:47 |
| mriedem | and is still core, but not working as much upstream these days | 16:48 |
| *** burt has quit IRC | 16:48 | |
| artom | mriedem, and defiantly ignored my grammar nit ;) | 16:48 |
| Spazmotic | Yeah he's been busy with family stuff.. I got in touch with him the other day though, but figured i'd ask since we need 2 cores | 16:48 |
| *** yamamoto has quit IRC | 16:48 | |
| johnthetubaguy | yeah, I am around for some questions, much of the time | 16:48 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Rollback instance.image_ref on failed rebuild https://review.openstack.org/538961 | 16:48 |
| mriedem | artom: there you go | 16:48 |
| Spazmotic | Now look what you did | 16:48 |
| johnthetubaguy | (still trying to juggle back more reviewing time) | 16:48 |
| mriedem | it's is always something i'll think of since school as a possessive | 16:49 |
| artom | mriedem, cheers :) | 16:49 |
| Spazmotic | hehe spend time with your family john, more important than reviews :D | 16:49 |
| jaypipes | mriedem: danke | 16:49 |
| *** sahid has quit IRC | 16:50 | |
| mriedem | johnthetubaguy: if you are around, i'll badger you about https://review.openstack.org/#/c/527541/ | 16:51 |
| *** salv-orlando has joined #openstack-nova | 16:51 | |
| *** yamamoto has joined #openstack-nova | 16:51 | |
| *** yamamoto has quit IRC | 16:51 | |
| johnthetubaguy | mriedem: sure thing | 16:51 |
| *** slaweq_ has quit IRC | 16:52 | |
| *** andreas_s has quit IRC | 16:52 | |
| *** Guest64456 has quit IRC | 16:53 | |
| johnthetubaguy | mriedem: is the TODO about jay's join magic important? | 16:53 |
| mriedem | no | 16:54 |
| *** salv-orlando has quit IRC | 16:54 | |
| *** salv-orlando has joined #openstack-nova | 16:54 | |
| jaypipes | quoi? | 16:54 |
| *** eharney_ has joined #openstack-nova | 16:54 | |
| johnthetubaguy | https://review.openstack.org/#/c/527541/4/nova/cmd/status.py@413 | 16:54 |
| mriedem | https://review.openstack.org/#/c/527541/4/nova/cmd/status.py@413 | 16:54 |
| mriedem | yeah | 16:54 |
| Spazmotic | someone owes someone a coke dependant upon latency and such | 16:55 |
| mriedem | jaypipes: it's a nova-status check for ironic / instance flavor migration | 16:55 |
| mriedem | something we should have done in pike, so i plan on backporting | 16:55 |
| *** sauloaugusto has quit IRC | 16:55 | |
| jaypipes | ah, k | 16:56 |
| *** andreas_s has joined #openstack-nova | 16:57 | |
| *** rmart04 has quit IRC | 16:58 | |
| *** eharney has quit IRC | 16:58 | |
| mriedem | gibi: speaking of things i don't think we want to backport if we can help it https://review.openstack.org/#/c/522112/ | 16:59 |
| mriedem | ^ adds the ability to specify service/admin user creds for cinder operations, | 16:59 |
| mriedem | like neutron | 16:59 |
| mriedem | so things like periodic tasks can actually detach/delete volumes once the user token is gone | 16:59 |
| *** andreas_s has quit IRC | 17:01 | |
| *** fragatina has joined #openstack-nova | 17:01 | |
| *** janki has quit IRC | 17:04 | |
| *** janki has joined #openstack-nova | 17:05 | |
| *** janki has quit IRC | 17:09 | |
| *** eharney_ is now known as eharney | 17:12 | |
| *** armax has joined #openstack-nova | 17:14 | |
| *** Supun has quit IRC | 17:20 | |
| stephenfin | jaypipes: Any chance you could take a look at this today? https://review.openstack.org/#/c/537363/ | 17:21 |
| *** Supun has joined #openstack-nova | 17:21 | |
| *** r-daneel_ has joined #openstack-nova | 17:21 | |
| *** r-daneel has quit IRC | 17:23 | |
| *** r-daneel_ is now known as r-daneel | 17:23 | |
| cfriesen | mriedem: is there a reason not to use a service token here? (other than that they're experimental) | 17:24 |
| melwitt | mriedem: I wondered about that too (the eventlet os.open thing). I didn't realize the timing coincided with that update | 17:25 |
| melwitt | *the timing of the constant fails | 17:26 |
| *** lpetrut has quit IRC | 17:27 | |
| *** gyee has joined #openstack-nova | 17:29 | |
| Spazmotic | aigoo.. that's enough computers for today | 17:31 |
| Spazmotic | Have a good night everyone. | 17:31 |
| *** jpena|brb is now known as jpena | 17:31 | |
| *** lyan has quit IRC | 17:31 | |
| *** yangyapeng has joined #openstack-nova | 17:31 | |
| *** mgoddard_ has quit IRC | 17:33 | |
| *** AlexeyAbashkin has quit IRC | 17:34 | |
| *** yangyapeng has quit IRC | 17:36 | |
| *** slaweq_ has joined #openstack-nova | 17:36 | |
| *** AlexeyAbashkin has joined #openstack-nova | 17:37 | |
| mriedem | cfriesen: are you conflating the service user thing here? | 17:40 |
| mriedem | cfriesen: different issues | 17:40 |
| *** slaweq_ has quit IRC | 17:40 | |
| *** mgoddard_ has joined #openstack-nova | 17:40 | |
| mriedem | the service user thing is for re-auth if a user token expires | 17:40 |
| mriedem | this is different, it's just creds to be able to have nova do stuff when there is no user token | 17:41 |
| mriedem | the service user token stuff for re-auth should probably no longer be called experimental | 17:41 |
| *** AlexeyAbashkin has quit IRC | 17:41 | |
| *** Supun has quit IRC | 17:41 | |
| mriedem | i don't know of anyone that's done performance testing with it at scale, but haven't heard anyone complain about it either, and it's optional | 17:41 |
| *** mvenesio has joined #openstack-nova | 17:42 | |
| *** Supun has joined #openstack-nova | 17:42 | |
| mriedem | cfriesen: speaking of performance, did you ever get any updates on the perf regression you reported last week/ | 17:42 |
| mriedem | ? | 17:42 |
| jaypipes | stephenfin: on it. | 17:43 |
| melwitt | mriedem: this looks good, but are we supposed to not approve things yet because of zuul? https://review.openstack.org/#/c/538961 | 17:47 |
| *** fragatina has quit IRC | 17:47 | |
| cfriesen | mriedem: we've got some additional data, but no smoking guns. the cpu usage just seems higher overall, with the cpu usage of most services looking more spread-out and less spiky. | 17:48 |
| Spazmotic | zuul is all better ap parently | 17:48 |
| Spazmotic | Was restarted and caught up | 17:48 |
| Spazmotic | Was it's zuul so all better is relative.. but yeah anyway | 17:48 |
| melwitt | Spazmotic: ah, thanks. I see the status alert now in the backscroll | 17:49 |
| *** sridharg has quit IRC | 17:49 | |
| Spazmotic | np melwitt. johnthetubaguy if you can get a cahhance to review my notes on that commit today i'd appreciate it, but don't stress over it if not. If you decide to +2 it i'll see if some of the other UK guys can take a look tomorrow to finish it off :) | 17:51 |
| Spazmotic | Night folks, have a good rest of your day | 17:51 |
| *** sree has joined #openstack-nova | 17:51 | |
| *** yamamoto has joined #openstack-nova | 17:52 | |
| *** yangyapeng has joined #openstack-nova | 17:52 | |
| *** slaweq_ has joined #openstack-nova | 17:55 | |
| *** sree has quit IRC | 17:55 | |
| *** mvk has quit IRC | 17:56 | |
| *** yangyapeng has quit IRC | 17:56 | |
| cfriesen | mriedem: when authenticating a token, is it possible for services to explicitly check whether a valid service token was attached? (This is related to the "glance won't allow update of ceph image location during nova snapshot" issue.) | 17:58 |
| *** slaweq_ has quit IRC | 17:59 | |
| *** JunOcataGuy is now known as penick | 18:00 | |
| *** derekh has quit IRC | 18:00 | |
| *** yamamoto has quit IRC | 18:01 | |
| *** ircuser-1 has joined #openstack-nova | 18:01 | |
| mriedem | cfriesen: that sounds like a question for edmondsw or lbragstad | 18:01 |
| *** sambetts is now known as sambetts|afk | 18:01 | |
| cfriesen | mriedem: cool, will ping them | 18:02 |
| edmondsw | cfriesen not sure I understood the question | 18:02 |
| cfriesen | edmondsw: there's an issue (https://bugs.launchpad.net/openstack-ansible/+bug/1639940) where with ceph-backed instances/images nova tries to make a new image and then update the location later | 18:03 |
| openstack | Launchpad bug 1639940 in openstack-ansible "Snapshots of instances launched from images fails with Ceph as storage." [Medium,Incomplete] - Assigned to Logan V (loganv) | 18:03 |
| cfriesen | edmondsw: this used to work with glance v1, but with v2 it fails because glance doesn't allow updating the image location | 18:03 |
| cfriesen | edmondsw: we were wondering whether we could use service tokens to allow glance to determine that the request is coming from another openstack service rather than a "normal" user | 18:03 |
| *** sree has joined #openstack-nova | 18:04 | |
| edmondsw | ah, I see | 18:04 |
| edmondsw | today, service tokens are only used re: expirations | 18:04 |
| edmondsw | i.e., if the user token has expired, it'll still allow the operation if the service token isn't expired | 18:04 |
| edmondsw | we have talked about going beyond that, and actually checking policy/RBAC based on service token instead of user token, but that has not been implemented | 18:05 |
| *** mgoddard_ has quit IRC | 18:07 | |
| *** slaweq has quit IRC | 18:07 | |
| cfriesen | do the APIs exist to allow glance to extract/validate the user token separately? or is that something that would need to be added in keystone? | 18:07 |
| cfriesen | bah, service token, not user token | 18:07 |
| *** slaweq has joined #openstack-nova | 18:08 | |
| edmondsw | cfriesen looking | 18:08 |
| melwitt | cfriesen: it still fails after setting 'show_multiple_locations = True' as mentioned in comment 7? | 18:08 |
| cfriesen | melwitt: presumably it works, but there are big security warnings about not setting that to True | 18:08 |
| *** sree has quit IRC | 18:09 | |
| mriedem | cfriesen: this sounds very similar to a bug we have with glance v2 and shelve where the shelved snapshot image has a different set of project_id/user_id than the admin token that tries to get the image when unshelving, | 18:09 |
| mriedem | and i thought we could use the member stuff with glancev2 | 18:09 |
| *** weshay|ruck is now known as weshay|ruck|brb | 18:09 | |
| cfriesen | melwitt: the current recommendation from the glance people was to have a whole separate glance node with a different config file just for nova to talk to it. | 18:09 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1675791 | 18:09 |
| openstack | Launchpad bug 1675791 in OpenStack Compute (nova) "Instance created by demo user(non-admin), shelved by admin and unshelved by demo user --> ends up in error state " [Medium,Triaged] - Assigned to Damini Chopra (damini) | 18:09 |
| melwitt | cfriesen: yeah. from what I understand, the fast clone has the caveat of the security issues | 18:09 |
| *** david-lyle has quit IRC | 18:09 | |
| *** yamahata has quit IRC | 18:10 | |
| melwitt | that is, you have to be in an environment where exposure of the image location urls isn't a problematic in order to use COW clone | 18:11 |
| mriedem | err in the case of shelve, the elevated admin context creates the snapshot, and when the user goes to unshelve the instance, it fails b/c the non-admin user doesn't have access to the image created for them | 18:11 |
| *** ralonsoh_ has quit IRC | 18:12 | |
| *** tidwellr has quit IRC | 18:12 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510 | 18:12 |
| mriedem | gibi: cdent: ^ hark back to an old conversation about testing for allocation cleanups | 18:12 |
| * cdent has a flash back | 18:13 | |
| mriedem | dansmith: ^ we talked about that in denver i think | 18:15 |
| *** rmcall has quit IRC | 18:17 | |
| *** Swami has joined #openstack-nova | 18:17 | |
| dansmith | I believe you | 18:18 |
| *** fragatina has joined #openstack-nova | 18:18 | |
| *** jpena is now known as jpena|off | 18:18 | |
| *** rmcall has joined #openstack-nova | 18:18 | |
| edmondsw | cfriesen looks like the context object should have info about the service_token if one was used | 18:18 |
| edmondsw | though I think you're probably treading on thin ice trying to use those in a way that they were not really intended to be used | 18:19 |
| edmondsw | better run what you are thinking by lbragstad | 18:19 |
| *** weshay|ruck|brb is now known as weshay | 18:20 | |
| *** weshay is now known as weshay|ruck | 18:20 | |
| *** slaweq has quit IRC | 18:23 | |
| *** slaweq has joined #openstack-nova | 18:23 | |
| *** felipemonteiro has joined #openstack-nova | 18:24 | |
| *** felipemonteiro_ has joined #openstack-nova | 18:25 | |
| *** cdent has quit IRC | 18:26 | |
| efried | edleafe Be careful what you ask for. See -dev ML. | 18:27 |
| *** felipemonteiro has quit IRC | 18:29 | |
| *** efried is now known as efried_hexchat | 18:29 | |
| *** efried1 is now known as efried | 18:29 | |
| * efried is gonna try this Thunderbird client for a while... | 18:29 | |
| *** tidwellr has joined #openstack-nova | 18:31 | |
| lbragstad | cfriesen: edmondsw summed it up pretty well, the service token work is kind of a long road.. it sounds like you want to use service tokens to validation/determine more than just a "yes, this token is valid" or "no, it isn't"? | 18:31 |
| *** r-daneel_ has joined #openstack-nova | 18:33 | |
| *** r-daneel has quit IRC | 18:34 | |
| edleafe | efried: thanks for that. So if I'm understanding things correctly, the "res.pool" is the root RP that placement would return | 18:35 |
| *** mvk has joined #openstack-nova | 18:35 | |
| efried | edleafe: Depends which model we're going with. | 18:35 |
| efried | edleafe: That would be model (C) | 18:36 |
| *** burt has joined #openstack-nova | 18:37 | |
| *** r-daneel has joined #openstack-nova | 18:37 | |
| *** r-daneel_ has quit IRC | 18:38 | |
| edleafe | efried: (C) makes the most sense to me, based on my limited familiarity | 18:39 |
| efried | edleafe: But it suffers from at least two drawbacks. | 18:39 |
| edleafe | efried: we don't want to fall into the trap of making everything fit the Nova model | 18:39 |
| *** jose-phillips has quit IRC | 18:40 | |
| efried | edleafe: What do you mean? This *is* nova. | 18:40 |
| efried | edleafe: You mean the libvirt model? | 18:40 |
| edleafe | efried: yeah, like we tried to do with ironic | 18:41 |
| edleafe | efried: from a placement POV, all we care about are the things that the consumer wants us to track. For generic libvirt nova, that would be compute nodes. For vmware nova, it would be resource pools | 18:42 |
| efried | Dig. So we should look toward closing those gaps. In general, we should be working to make nova + placement + <virt> work smoothly, for all values of <virt>. | 18:44 |
| edleafe | yeah | 18:44 |
| edleafe | Placement should always be returning the thing that the consumer needs to proceed with a request | 18:45 |
| *** mgoddard_ has joined #openstack-nova | 18:46 | |
| efried | I need to close the time box on this. I'm going to make sure the PTG etherpad has an entry for this, and then get back to my regularly-scheduled programming. | 18:46 |
| edleafe | efried: cool. Thanks for writing up that email | 18:46 |
| efried | yahyoubetcha | 18:46 |
| *** jaosorior has quit IRC | 18:48 | |
| *** hemna_ has quit IRC | 18:52 | |
| *** yamahata has joined #openstack-nova | 18:53 | |
| *** yangyapeng has joined #openstack-nova | 18:54 | |
| *** gjayavelu has joined #openstack-nova | 18:56 | |
| *** yangyapeng has quit IRC | 18:58 | |
| mriedem | jaypipes: melwitt: i'm going to undo the exception code handline refactor in https://review.openstack.org/#/c/538961/ - didn't mean to include that in this patch and it muddies the backpot | 19:00 |
| mriedem | *backport | 19:00 |
| *** Tom-Tom has quit IRC | 19:00 | |
| melwitt | ack | 19:01 |
| *** slaweq_ has joined #openstack-nova | 19:01 | |
| openstackgerrit | Sen Yang proposed openstack/python-novaclient master: Implement hypervisor hostname exact pattern match https://review.openstack.org/520187 | 19:04 |
| *** slaweq_ has quit IRC | 19:05 | |
| *** tbtestaccount has joined #openstack-nova | 19:06 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Rollback instance.image_ref on failed rebuild https://review.openstack.org/538961 | 19:07 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Collapse duplicate error handling in rebuild_instance https://review.openstack.org/539001 | 19:07 |
| *** lucasagomes is now known as lucas-afk | 19:08 | |
| *** rmcall has quit IRC | 19:08 | |
| *** AlexeyAbashkin has joined #openstack-nova | 19:11 | |
| jaypipes | mriedem: +Wallaby'd | 19:11 |
| mriedem | thanks | 19:12 |
| *** rmcall has joined #openstack-nova | 19:13 | |
| efried | jaypipes: Can we continue the discussion as to whether or not we should in fact be requiring update_provider_tree to return True/False? | 19:14 |
| *** yangyapeng has joined #openstack-nova | 19:14 | |
| *** mgariepy has quit IRC | 19:15 | |
| *** tbtestaccount has quit IRC | 19:16 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Rollback instance.image_ref on failed rebuild https://review.openstack.org/539003 | 19:18 |
| cfriesen | lbragstad: just back from lunch. yes, there's a scenario where we would like to answer the question "did this request come from another openstack service, not a 'normal' user". was hoping to use service tokens for this. | 19:18 |
| *** fragatina has quit IRC | 19:18 | |
| *** fragatina has joined #openstack-nova | 19:18 | |
| *** yangyapeng has quit IRC | 19:19 | |
| jaypipes | efried: sure | 19:20 |
| *** arvindn051 has quit IRC | 19:21 | |
| efried | jaypipes: So first off, it's trivial and inexpensive for report client to figure it out, so there's no real *need* for virt to tell us. | 19:21 |
| efried | jaypipes: And after talking through a couple of potential impls from VMWare, it became clear that there's certainly the possibility that it would be awkward for the virt driver to figure it out. | 19:22 |
| efried | jaypipes: For example, one viable implementation is to say, "I don't care what you gave me, I'm going to delete everything and build the ProviderTree as I know it from scratch" | 19:23 |
| *** edmondsw has quit IRC | 19:23 | |
| lbragstad | cfriesen: that kinda sounds like federation? | 19:23 |
| openstackgerrit | melanie witt proposed openstack/nova stable/pike: Stop globally caching host states in scheduler HostManager https://review.openstack.org/539005 | 19:23 |
| *** mgariepy has joined #openstack-nova | 19:24 | |
| efried | jaypipes: Softer than that, it's IMO a source of extra unnecessary bugs to ask the virt driver to make sure they get that bool return correct. | 19:24 |
| cfriesen | lbragstad: not sure federation applies since it's all within the same cloud. We just want glance to be able to special-case a request to change an image location if it comes from nova, but not if it comes from a regular user. | 19:25 |
| ameeda | gibi: now its okay, can you please merge the bug ? or it need something else ? | 19:25 |
| *** harlowja has joined #openstack-nova | 19:25 | |
| efried | jaypipes: Put another way: why have two chunks of code doing the same thing when one will do? | 19:25 |
| jaypipes | efried: ok | 19:26 |
| jaypipes | efried: I just thought it would make the RT's life easier if it could say "ok, no changes from virt driver... just move on" | 19:27 |
| lbragstad | cfriesen: oh - sorry, for some reason i was thinking of different services | 19:27 |
| efried | jaypipes: Definitely could have worked out that way. | 19:27 |
| *** david-lyle has joined #openstack-nova | 19:27 | |
| lbragstad | cfriesen: that sounds like new territory for service tokens, the first thing we started working on with them was the whole long running operation issue.. it'd be good to sync with jamielennox though | 19:29 |
| lbragstad | cfriesen: he was one of the original people driving the effort, so i wouldn't be surprised if he's ventured down a couple different paths similar to what you're describing | 19:30 |
| *** john51 has quit IRC | 19:30 | |
| *** tesseract has quit IRC | 19:31 | |
| *** john51 has joined #openstack-nova | 19:31 | |
| cfriesen | lbragstad: move it over to the keystone channel? | 19:31 |
| lbragstad | cfriesen: sure | 19:31 |
| *** yangyapeng has joined #openstack-nova | 19:35 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Rollback instance.image_ref on failed rebuild https://review.openstack.org/539008 | 19:36 |
| *** mgoddard_ has quit IRC | 19:39 | |
| *** yangyapeng has quit IRC | 19:40 | |
| *** tidwellr has quit IRC | 19:41 | |
| ameeda | is this error caused by me ? "http://logs.openstack.org/00/526900/26/check/openstack-tox-functional/c777a93/testr_results.html.gz" | 19:51 |
| ameeda | from this gerrit "https://review.openstack.org/#/c/526900/" | 19:51 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove redundant call to add_instance_fault_from_exc in rebuild_instance https://review.openstack.org/539011 | 19:51 |
| *** mriedem1 has joined #openstack-nova | 19:52 | |
| ameeda | jaypipes: please check this for me when you available https://review.openstack.org/#/c/526900/ | 19:52 |
| *** mriedem has quit IRC | 19:52 | |
| *** yangyapeng has joined #openstack-nova | 19:56 | |
| mriedem1 | ameeda: instance_system_metadata is a table where one row is a key/value pair for a single instance, and we can have a lot of sysmeta per instance, and a ton of instances, | 19:56 |
| mriedem1 | in what world do we have a system metadata value that needs to be length TEXT? | 19:57 |
| *** mriedem1 is now known as mriedem | 19:57 | |
| ameeda | mriedem: so I did something wrong ? | 19:58 |
| *** jose-phillips has joined #openstack-nova | 19:59 | |
| mriedem | no, it's just, this is potentially a very large change in storage for that talbe | 19:59 |
| mriedem | *Table | 19:59 |
| mriedem | i see you're trying to match the nova table to glance https://github.com/openstack/glance/blob/master/glance/db/sqlalchemy/models.py#L159 | 20:00 |
| mriedem | correct ^ ? | 20:00 |
| *** yangyapeng has quit IRC | 20:00 | |
| mriedem | an alternative solution would be to add an image_props column to the instance_extra table and just store the serialized image properties in there, rather than the instance_system_metadata table | 20:02 |
| mriedem | but then we could have n * TEXT entries in that json blob | 20:02 |
| mriedem | jaypipes: TEXT is not preallocated right? | 20:03 |
| ameeda | mriedem: I am new on openstack , I just follow the bug to fix it, I need to merge this bug. I worked on it a lot as you see :( | 20:04 |
| *** pramodrj07 has joined #openstack-nova | 20:04 | |
| jaypipes | mriedem: correct. | 20:04 |
| mriedem | ameeda: do you have a customer hitting htis? | 20:05 |
| mriedem | *this | 20:05 |
| jaypipes | ameeda: reviewed. | 20:06 |
| ameeda | I am working at local company here, and the customer ask us to fix bugs in openstack. so I have to get points | 20:07 |
| ameeda | jaypipes: Thank you | 20:07 |
| *** slaweq_ has joined #openstack-nova | 20:08 | |
| mriedem | ameeda: hmm, well, there are probably less controversial bugs to fix :) | 20:08 |
| mriedem | if you're just looking for something to get into stackalytics | 20:08 |
| *** Swami has quit IRC | 20:09 | |
| *** eharney has quit IRC | 20:09 | |
| mriedem | reading the history on this bug, from lbragstad - probably due to some issue with storing powervc configuration strategy xml in image metadata back in 2013 - it sounds like the 255 length restriction was from when the image service was passing properties as http headers | 20:10 |
| ameeda | mriedem: I still learn about nova and try to find medium bugs to fix . if my current opened bugs not merged , they will fire me :( | 20:10 |
| mriedem | ameeda: sounds like a terrible employer | 20:10 |
| ameeda | mriedem: yes it was sending by header and now sending on body, so the limitation is gone | 20:11 |
| ameeda | mriedem: no comments ... | 20:11 |
| openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Stop globally caching host states in scheduler HostManager https://review.openstack.org/539013 | 20:12 |
| *** slaweq_ has quit IRC | 20:12 | |
| *** Swami has joined #openstack-nova | 20:15 | |
| *** yangyapeng has joined #openstack-nova | 20:16 | |
| *** tidwellr has joined #openstack-nova | 20:20 | |
| ameeda | mriedem: regarding to your last comment, is this something should I do ? | 20:21 |
| *** yangyapeng has quit IRC | 20:21 | |
| mriedem | ameeda: up to you, it could be done in devstack if you use the nova fake virt driver ,and then just create 1000 instances | 20:22 |
| mriedem | then apply your change and run the migration and see how long it takes | 20:22 |
| *** eharney has joined #openstack-nova | 20:22 | |
| ameeda | sure that will take long time for large data, but what I should I do after what I have done until now :( | 20:23 |
| *** AlexeyAbashkin has quit IRC | 20:25 | |
| *** jose-phillips has quit IRC | 20:25 | |
| *** amoralej is now known as amoralej|off | 20:26 | |
| openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | 20:27 |
| *** gszasz has quit IRC | 20:28 | |
| ameeda | jaypipes: I uploaded the patch set | 20:28 |
| *** edmondsw has joined #openstack-nova | 20:30 | |
| *** Swami has quit IRC | 20:31 | |
| *** edmondsw has quit IRC | 20:32 | |
| *** edmondsw_ has joined #openstack-nova | 20:32 | |
| ameeda | jaypipes mriedem, kindly, can you please review those also https://review.openstack.org/#/c/528069/ and https://review.openstack.org/#/c/528385/ | 20:34 |
| ameeda | and please let me know what do you think about this bug "https://bugs.launchpad.net/nova/+bug/1737708" | 20:35 |
| openstack | Launchpad bug 1737708 in OpenStack Compute (nova) "create instance failed when the userdata size is larger than 64k" [Undecided,New] - Assigned to Deepak Mourya (mourya007) | 20:35 |
| *** namnh has joined #openstack-nova | 20:35 | |
| ameeda | sorry for inconvenience, Thank you for your time | 20:36 |
| *** yangyapeng has joined #openstack-nova | 20:37 | |
| *** jose-phillips has joined #openstack-nova | 20:38 | |
| mriedem | ameeda: melwitt has been working on a duplicate of one of those for a long time https://review.openstack.org/#/c/340614/ | 20:39 |
| *** READ10 has quit IRC | 20:40 | |
| melwitt | yeah, took many iterations to get it right back when the earlier versions caused gate failures in the postgres job. looks like it needs another rebase | 20:41 |
| *** yangyapeng has quit IRC | 20:41 | |
| *** namnh has quit IRC | 20:41 | |
| melwitt | we still have customers that need a fix for it | 20:43 |
| ameeda | mriedem: yes, but my bug scenario doesn't hit _local_delete, so we have to cover all the cases, right ? | 20:43 |
| mriedem | melwitt's patch is fixing a local delete case | 20:44 |
| mriedem | where the instance isn't on a compute host, | 20:44 |
| mriedem | so it is either shelved offloaded or failed during scheduling | 20:44 |
| melwitt | it is on a compute host, it's just in ERROR state | 20:44 |
| mriedem | 'local delete' == delete from the API, not compute | 20:44 |
| melwitt | there's code in compute manager that sets host = None while setting to ERROR state if something fails. IIRC | 20:45 |
| *** tssurya_ has joined #openstack-nova | 20:45 | |
| melwitt | like this https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1941-L1943 | 20:46 |
| ameeda | I am not sure about the case , but what about this fix https://review.openstack.org/#/c/528385/ | 20:48 |
| ameeda | this detach volume when instance creation fails | 20:49 |
| *** jose-phillips has quit IRC | 20:50 | |
| mriedem | ameeda: -1 on that one too | 20:51 |
| mriedem | ameeda: we already attempt to detach volumes when deleting a server | 20:51 |
| mriedem | so if that's failing, i don't think your patch is going to fix it | 20:51 |
| ameeda | what is the case !, I just follow the bugs to fix them , after that I found that the bug is invalid ? | 20:53 |
| *** Swami has joined #openstack-nova | 20:53 | |
| mriedem | ameeda: well, you could be trying to fix a very old bug that's already been fixed | 20:54 |
| mriedem | or the recreate scenario from the bug is not clear | 20:54 |
| mriedem | so the thing to do here, is probably go through the recreate steps in the bug (which are pretty clear) and see if you can reproduce it | 20:55 |
| mriedem | and if so, see where things are failing | 20:55 |
| mriedem | the bug report was written over a year ago and doesn't say which version of nova they were using when they hit this, | 20:56 |
| mriedem | so it might be fixed | 20:56 |
| mriedem | i'm sorry that you invested time in a patch, but we need to make sure it's still a valid bug first | 20:56 |
| ameeda | mriedem: but I reproduce the bug and fix it , then test it again to make sure that I fix the issue, and there are reviewers ask me to make changes on it :( | 20:56 |
| ameeda | I am sure that the bug is still valid , I reproduce it | 20:57 |
| mriedem | ok, so then why is the original detach failing in _shutdown_instance? | 20:58 |
| mriedem | because that's what should be detaching the volume | 20:58 |
| *** yangyapeng has joined #openstack-nova | 21:01 | |
| melwitt | mriedem: I think it's the same bug I'm fixing. I think he means volume quota was exceeded (it's checked in compute). compute catches the exceptions, nulls out the instance.host, then sets it to ERROR state | 21:02 |
| melwitt | then when you go to delete the instance, because compute nulled out the host, the compute api thinks there's no resources to free up because it assumes instance.host = None means failed to schedule | 21:03 |
| melwitt | so I think my patch will take care of it. I'm rebasing it now and rewriting the commit message to be clearer | 21:03 |
| ameeda | my case when vm creations fails when there are no enough quota for volumes. as this https://review.openstack.org/#/c/528069/ scenario. | 21:03 |
| melwitt | I know, that's what I just said | 21:03 |
| ameeda | melwitt: you fix solve the issue when there is no hosts, my fix solve the issue in another case ! | 21:05 |
| ameeda | mriedem: what do you think ? | 21:05 |
| *** yangyapeng has quit IRC | 21:05 | |
| melwitt | it's not another case. compute will set instance.host = None because of the volume quota exceeded | 21:05 |
| *** Sukhdev_ has joined #openstack-nova | 21:07 | |
| ameeda | ok great !! | 21:07 |
| ameeda | duplicated bugs , and my work and time have gone in vain | 21:08 |
| *** liverpooler has quit IRC | 21:10 | |
| *** mvenesio has quit IRC | 21:11 | |
| *** mvenesio has joined #openstack-nova | 21:12 | |
| *** mvenesio has quit IRC | 21:16 | |
| *** Supun has quit IRC | 21:18 | |
| *** yangyapeng has joined #openstack-nova | 21:22 | |
| *** slaweq_ has joined #openstack-nova | 21:22 | |
| *** yangyapeng has quit IRC | 21:26 | |
| *** slaweq_ has quit IRC | 21:27 | |
| efried | mriedem: Any tips for how I can check in nova.tests.functional.test_servers that a particular exception got raised up from _update() ? | 21:29 |
| *** READ10 has joined #openstack-nova | 21:29 | |
| efried | So far my mechanism for getting _update() to run is self._run_periodics() | 21:30 |
| *** r-daneel has quit IRC | 21:33 | |
| *** r-daneel has joined #openstack-nova | 21:33 | |
| *** vivsoni has quit IRC | 21:36 | |
| *** vivsoni has joined #openstack-nova | 21:36 | |
| melwitt | efried: I think that's the usual way to run it | 21:37 |
| efried | melwitt: I think I'm going to have to deconstruct it a bit and run rt.update_available_resource directly. | 21:38 |
| *** cleong has quit IRC | 21:38 | |
| melwitt | efried: any reason you're trying to verify that in a func test instead of a unit test? | 21:38 |
| efried | melwitt: A reasonable question. | 21:39 |
| melwitt | if there's no way to detect what you expect from the instance or other things you can inspect in the func test | 21:39 |
| efried | melwitt: I was hoping to avoid unit test altogether. | 21:39 |
| melwitt | so there's an existing func test and you want to add an assertion to it? | 21:39 |
| efried | melwitt: I'm writing a new func suite. Stand by, I'll put up what I've got. | 21:40 |
| *** eharney has quit IRC | 21:43 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648 | 21:43 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | 21:43 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 21:43 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | 21:43 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | 21:43 |
| *** takashin has joined #openstack-nova | 21:43 | |
| efried | melwitt: Right here is where I want to make sure that exception happened: https://review.openstack.org/#/c/520246/41/nova/tests/functional/test_servers.py@1615 | 21:44 |
| efried | melwitt: And similar for other kinds of error conditions. | 21:44 |
| efried | melwitt: I mean, I'm at least proving the whole thing doesn't blow up when I inject error conditions, which is good. | 21:44 |
| *** moshele has joined #openstack-nova | 21:47 | |
| melwitt | I see. I suppose you could monkey patch _update (or whatever) to call the real _update inside self.assertRaises. I wonder if that would work | 21:49 |
| efried | melwitt: Well, I don't see _run_periodicals really doing anything that adds value to my narrow little test case. | 21:51 |
| melwitt | so you don't want to assert a particular exception is raised? I'm confused :) | 21:52 |
| efried | I do, and the only reason I can't do assertRaises(self._run_periodicals) is because _run_periodicals invokes x, which invokes y, which invokes z, which swallows exceptions. | 21:53 |
| efried | So I'm going to dig in and invoke z directly, so I can assertRaises(z) | 21:53 |
| efried | ...which I just did, and it seems to work a treat! melwitt thanks for sounding boarding :) | 21:53 |
| efried | Now everyone will want to do what I just did | 21:54 |
| melwitt | yeah, that's what I was trying to say, monkey patch the method that will raise it and just call the real method inside self.assertRaises | 21:54 |
| efried | The monkey patching isn't needed | 21:56 |
| melwitt | oh, so run that instead of periodics | 21:56 |
| melwitt | I see | 21:56 |
| efried | just so. | 21:56 |
| efried | Just needed a few extra LOC to dig into the call stack a couple more levels. | 21:57 |
| *** burt has quit IRC | 22:04 | |
| *** threestrands has joined #openstack-nova | 22:05 | |
| *** threestrands has quit IRC | 22:05 | |
| *** threestrands has joined #openstack-nova | 22:05 | |
| *** threestrands_ has joined #openstack-nova | 22:07 | |
| *** threestrands has quit IRC | 22:08 | |
| *** threestrands_ has quit IRC | 22:08 | |
| *** threestrands has joined #openstack-nova | 22:08 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899 | 22:09 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add test for scenario with sum of child resources https://review.openstack.org/534339 | 22:09 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: _get_trees_matching_all_resources() sum children https://review.openstack.org/534866 | 22:09 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add _get_usages_from_trees() function https://review.openstack.org/534967 | 22:09 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: validate provider summary structure https://review.openstack.org/536516 | 22:09 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - populate alloc cands for trees https://review.openstack.org/534968 | 22:09 |
| *** dtruong has quit IRC | 22:13 | |
| *** threestrands_ has joined #openstack-nova | 22:21 | |
| *** jaypipes has quit IRC | 22:21 | |
| *** felipemonteiro_ has quit IRC | 22:22 | |
| *** tidwellr has quit IRC | 22:23 | |
| *** threestrands has quit IRC | 22:23 | |
| *** tidwellr has joined #openstack-nova | 22:24 | |
| *** edmondsw_ is now known as edmondsw | 22:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: reno for notification-transformation-queens https://review.openstack.org/518018 | 22:26 |
| *** tssurya_ has quit IRC | 22:27 | |
| *** tidwellr has quit IRC | 22:29 | |
| mriedem | dansmith: want to hit this pike backport? https://review.openstack.org/#/c/536981/ | 22:30 |
| mgagne | what's the use case for aggregates in Placement API? | 22:30 |
| *** dave-mccowan has quit IRC | 22:31 | |
| *** rmcall has quit IRC | 22:31 | |
| mriedem | mgagne: the primary one is modeling things where one provider shares a resource with another, like a shared storage pool | 22:32 |
| mgagne | mriedem: thanks for the clarification, I tried to read/understand https://developer.openstack.org/api-ref/placement/#resource-provider-aggregates | 22:33 |
| mgagne | and I'm not sure the doc fully explain its purpose, I would have never guess what you said from the doc, it mainly explains how it's not nova aggregates | 22:34 |
| mriedem | mgagne: is the spec that talks about aggregates the most i think https://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/generic-resource-pools.html | 22:34 |
| mriedem | mgagne: heh yeah that's what i was thinking | 22:34 |
| mgagne | how is Generic Resource Pools == aggregates? =) | 22:34 |
| *** namnh has joined #openstack-nova | 22:35 | |
| *** jappleii__ has joined #openstack-nova | 22:35 | |
| mgagne | will read and see if I can get more answers | 22:35 |
| *** jappleii__ has quit IRC | 22:36 | |
| *** jappleii__ has joined #openstack-nova | 22:37 | |
| mgagne | thanks for the link btw ;) | 22:37 |
| *** threestrands_ has quit IRC | 22:37 | |
| *** namnh has quit IRC | 22:40 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: provide more detail on what a provider aggregate is https://review.openstack.org/539033 | 22:41 |
| mriedem | mgagne: see if ^ helps wording-wise | 22:41 |
| *** jose-phillips has joined #openstack-nova | 22:41 | |
| mriedem | sorry about ending on a preposition | 22:41 |
| *** mylu has joined #openstack-nova | 22:42 | |
| mriedem | maybe it's not, me fail english | 22:42 |
| mgagne | mriedem: I think it's enough to give a hint that aggregates do have a purpose other than not being nova aggregates =) | 22:42 |
| mgagne | I'm still trying to digest the spec you linked above. | 22:43 |
| mgagne | would it be ok to update past spec to reflect current implement? Ex.: openstack resource-provider is now openstack resource provider | 22:45 |
| *** mylu has quit IRC | 22:46 | |
| *** rcernin has joined #openstack-nova | 22:46 | |
| *** mylu has joined #openstack-nova | 22:46 | |
| *** liangy has quit IRC | 22:49 | |
| *** mylu has quit IRC | 22:50 | |
| mriedem | mgagne: if it's not too invasive, then that's probably fine | 22:52 |
| mriedem | especially for CLI examples | 22:52 |
| openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Update references to OSC resource provider command https://review.openstack.org/539038 | 22:55 |
| *** liangy has joined #openstack-nova | 22:56 | |
| *** liverpooler has joined #openstack-nova | 22:59 | |
| efried | mgagne mriedem Sharing providers aren't the only use case FWIW. | 23:00 |
| *** slaweq_ has joined #openstack-nova | 23:00 | |
| mriedem | efried: i'm not up to date on the new aggregate hotness | 23:00 |
| *** moshele has quit IRC | 23:00 | |
| mriedem | but yes i know sean-k-mooney has a thing | 23:00 |
| mgagne | efried: please let me know if there is a doc I can read =) | 23:00 |
| mriedem | mgagne: left some comments in there | 23:00 |
| mriedem | mgagne: that spec was written in i think mitaka or newton, so the cli stuff was all future ideas | 23:01 |
| efried | mgagne: Not really a single doc, no. But as an example, see this ML thread... | 23:01 |
| mgagne | mriedem: and now I find that aggregates ARE nova aggregates but not totally :O | 23:01 |
| mgagne | mriedem: yea found the Other end user impact section | 23:01 |
| efried | mgagne: http://lists.openstack.org/pipermail/openstack-dev/2018-January/126685.html | 23:01 |
| mriedem | it might be easier to just add a note that any CLIs are subject to change and the definitive CLI guide is in the osc-placement docs <link> | 23:01 |
| mgagne | efried: ok, I'm a n00b when it comes to placement, I'm trying to catch up very fast so I can answer to jaypipes but it isn't easy to find all the relevant info | 23:02 |
| efried | mgagne: Specifically (6) and its subsequent references in the models | 23:02 |
| efried | mgagne: Answer what from jaypipes? | 23:02 |
| mgagne | mriedem: if there is no doc, I read specs, they often contain more info although some bit might be outdated. | 23:02 |
| mgagne | efried: http://lists.openstack.org/pipermail/openstack-dev/2018-January/126283.html | 23:03 |
| efried | mgagne: Yeah, had a feeling it was that :) | 23:03 |
| *** slaweq_ has quit IRC | 23:04 | |
| *** awaugama has quit IRC | 23:04 | |
| *** dtruong has joined #openstack-nova | 23:04 | |
| *** awaugama_ has joined #openstack-nova | 23:04 | |
| mgagne | efried: so English is not my first language and I often have a hard time to grasp new concepts without visual aids. But once I understand, it's all fine =) | 23:05 |
| efried | Je pourrais t'aider en français si tu veux, mais je crois que la langue n'est pas la problème :) | 23:05 |
| efried | Sauf, la langue naturelle... | 23:06 |
| *** edmondsw has quit IRC | 23:06 | |
| mgagne | hehe, c'est quoi l'affaire de placement :O ;) | 23:06 |
| efried | Seulement pour enregistrer les resources, rien d'autre | 23:07 |
| efried | (pas vraiement) | 23:07 |
| *** weshay|ruck is now known as weshay|ruck|afk | 23:07 | |
| *** yangyapeng has joined #openstack-nova | 23:07 | |
| mgagne | yea, I got the idea. just trying to understand all the fine prints | 23:07 |
| *** liangy has quit IRC | 23:08 | |
| *** salv-orlando has quit IRC | 23:08 | |
| efried | Definitely. I admit I haven't kept thoroughly updated with that particular ML thread, but if you have a couple of minutes to hang about while I read it, I may be able to answer your questions more easily. | 23:08 |
| mriedem | maciejjozefczyk_: looks like your change https://review.openstack.org/#/c/529236/ | 23:08 |
| *** salv-orlando has joined #openstack-nova | 23:09 | |
| efried | The problem I have (and the reason I wasn't following to closely) is that I don't really know about this Filter business, or what compute aggregates are/do (just that they aren't placement aggregates). | 23:09 |
| efried | s/to/too/ | 23:09 |
| mgagne | efried: I'm trying to finish my email now. I do have some questions but tbh, it's mostly missing bits so one could transition from an aggregate based solutions to placement | 23:09 |
| *** claudiub|3 has quit IRC | 23:10 | |
| *** namnh has joined #openstack-nova | 23:11 | |
| efried | mgagne: Are you trying to make an existing deployment work on a particular version of openstack? | 23:11 |
| *** yangyapeng has quit IRC | 23:12 | |
| mgagne | efried: yes, you guessed right. in fact, it has been my job for the last year, upgrading, discovering major changes, cursing and adapting =) | 23:13 |
| *** salv-orlando has quit IRC | 23:13 | |
| efried | mgagne: Okay, which version of openstack? | 23:13 |
| mgagne | efried: We are running Mitaka, trying to move to Newton (currently on pause due to our friends meltdown and spectre) | 23:15 |
| *** namnh has quit IRC | 23:15 | |
| mgagne | efried: but I'm still monitoring ML and now I'm answering the email because I discovered the mentioned change that would break/change our current flow/ways of doing thing. | 23:15 |
| efried | Oh, okay, wow. And Jay is talking about implementing these hacks all the way back to Newton? (which we're trying to EOL afaik) | 23:16 |
| mgagne | efried: so I'm not ready for Ocata yet but I know I will have to cross that river someday | 23:16 |
| mgagne | efried: no ^ | 23:16 |
| *** yangyapeng has joined #openstack-nova | 23:16 | |
| *** Sukhdev_ has quit IRC | 23:16 | |
| mgagne | efried: but the challenge is that once I'm ready to cross that river, it will be too late to complain about changes in Ocata I wasn't aware about. I'm playing catch up here. | 23:17 |
| *** felipemonteiro_ has joined #openstack-nova | 23:18 | |
| mgagne | efried: I don't know if you understand in what kind of boat I'm in. | 23:18 |
| efried | mgagne: Well, I think the first thing that'll help you is to get placement aggregates out of your head. They're not related to host aggregates, and they're not fully implemented in Newton (or Ocata, or Pike, or Queens). | 23:18 |
| *** namnh has joined #openstack-nova | 23:19 | |
| efried | It sounds to me like all you need is a way to customize allocation ratios. Which can be done via placement calls, as long as there's *some* switch to turn off the automatic setting from the conf, as Jay suggests. | 23:19 |
| mgagne | efried: ok, I found that it's not the answer to my problem. But I still need to educate myself on the concept | 23:19 |
| *** bnemec has quit IRC | 23:20 | |
| *** mlavalle has quit IRC | 23:20 | |
| mgagne | efried: yes, that's the conclusion I ended up with too | 23:20 |
| *** bnemec has joined #openstack-nova | 23:20 | |
| efried | mgagne: Has anyone proposed a patch that would do that yet? | 23:20 |
| *** READ10 has quit IRC | 23:20 | |
| mgagne | not that I'm aware of | 23:20 |
| *** edmondsw has joined #openstack-nova | 23:20 | |
| *** yangyapeng has quit IRC | 23:21 | |
| mriedem | efried: this? https://review.openstack.org/#/c/532924/ | 23:22 |
| efried | mriedem: Certainly looks related, at a glance. | 23:23 |
| *** namnh has quit IRC | 23:23 | |
| *** namnh has joined #openstack-nova | 23:24 | |
| efried | mgagne: Have you tried patching ^ in? | 23:25 |
| mgagne | efried: I'm not running master | 23:26 |
| efried | mgagne: Yeah, I get that, I'm asking if you've tried patching that into your running environment to see if it resolves the issue. | 23:27 |
| efried | or rather, allows you to resolve it via conf. | 23:27 |
| mgagne | I'm not sure I understand what you are asking | 23:27 |
| *** yangyapeng has joined #openstack-nova | 23:29 | |
| *** stakeda has joined #openstack-nova | 23:30 | |
| efried | mgagne: IIUC, the point of that patch will be to backport it to where you can use it (which I guess would be Newton? Though again, I thought we were EOLing that guy) | 23:31 |
| *** awaugama_ has quit IRC | 23:31 | |
| efried | mgagne: So I'm suggesting you could do that backport locally and see if it works for you. | 23:31 |
| mgagne | I'm really confused. what's "it" ? =) | 23:31 |
| efried | Though I suppose things have probably changed quite a bit in those areas, so the backport may not be trivial. | 23:32 |
| mgagne | the bug is very technical and I don't fully understand the problem and proposed solution | 23:32 |
| efried | Sorry, the patch mriedem mentioned above - https://review.openstack.org/#/c/532924/ | 23:32 |
| efried | The proposed solution would allow you to set your allocation ratios via conf, end of story. | 23:32 |
| efried | Nothing else would change; in particular, you would continue to be able to use your filtery thingies as you did before. | 23:33 |
| mriedem | efried: we don't backport config option default changes | 23:33 |
| efried | oh | 23:33 |
| *** yangyapeng has quit IRC | 23:33 | |
| mriedem | since, you know, changing defaults on people in stable is kind of mean | 23:34 |
| efried | what about the rest of that patch? | 23:34 |
| efried | yeah, makes sense when you put it like that. | 23:34 |
| mriedem | :) | 23:34 |
| mriedem | i haven't looked at the rest of the patch in a few weeks | 23:34 |
| mgagne | efried: what's via conf for you? Isn't already the case where you can provide allocation ratio on compute node in nova.conf? I'm totally lost | 23:34 |
| mriedem | and i don't think i have the brain power to do so right now | 23:34 |
| efried | mgagne: Yeah, you're right. And I'm probably just confusing the issue even more. I should stop. | 23:35 |
| mgagne | efried: my issue is I'm being told to stop using aggregates (and allocation ratio in metadata) because they are broken. | 23:35 |
| mgagne | efried: and to use nova.conf itself on compute node because "Config management is the solution to your problem" | 23:36 |
| efried | I can understand where that's a problem. | 23:36 |
| mgagne | efried: that's not what I want, this is not my flow, far from it. | 23:37 |
| *** namnh has quit IRC | 23:38 | |
| mgagne | efried: so that's basically it. Something I could do through an API now needs to be done through our config system and the person performing this task might not have access to our config system. | 23:38 |
| mgagne | and you lose "agility" in your operations | 23:39 |
| *** namnh has joined #openstack-nova | 23:42 | |
| *** ekhugen- has joined #openstack-nova | 23:44 | |
| mgagne | time to leave for me, thanks for the talk ;) | 23:44 |
| *** mhenkel_ has joined #openstack-nova | 23:45 | |
| *** NightKhaos has quit IRC | 23:45 | |
| *** mhenkel has quit IRC | 23:45 | |
| *** ekhugen_alt has quit IRC | 23:45 | |
| *** raorn has quit IRC | 23:45 | |
| *** logan- has quit IRC | 23:45 | |
| *** Oku_OS-away has quit IRC | 23:45 | |
| *** StevenK has quit IRC | 23:45 | |
| *** jlvillal has quit IRC | 23:45 | |
| *** fnordahl has quit IRC | 23:45 | |
| *** jistr has quit IRC | 23:45 | |
| *** dtantsur|afk has quit IRC | 23:45 | |
| *** r-daneel has quit IRC | 23:45 | |
| *** edmondsw has quit IRC | 23:46 | |
| *** edmondsw has joined #openstack-nova | 23:47 | |
| *** mgoddard has quit IRC | 23:48 | |
| *** yangyapeng has joined #openstack-nova | 23:49 | |
| *** NightKhaos has joined #openstack-nova | 23:51 | |
| *** raorn has joined #openstack-nova | 23:51 | |
| *** logan- has joined #openstack-nova | 23:51 | |
| *** Oku_OS-away has joined #openstack-nova | 23:51 | |
| *** StevenK has joined #openstack-nova | 23:51 | |
| *** jlvillal has joined #openstack-nova | 23:51 | |
| *** fnordahl has joined #openstack-nova | 23:51 | |
| *** jistr has joined #openstack-nova | 23:51 | |
| *** dtantsur|afk has joined #openstack-nova | 23:51 | |
| *** mylu has joined #openstack-nova | 23:51 | |
| *** edmondsw has quit IRC | 23:51 | |
| *** dave-mccowan has joined #openstack-nova | 23:53 | |
| *** yangyapeng has quit IRC | 23:53 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 23:54 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | 23:54 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | 23:54 |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle IpAddressAlreadyAllocated exception https://review.openstack.org/535532 | 23:55 |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: Use revision_number on adding/removing security group https://review.openstack.org/535510 | 23:56 |
| *** mgoddard has joined #openstack-nova | 23:56 | |
| *** sapd_ has quit IRC | 23:57 | |
| *** hongbin has quit IRC | 23:57 | |
| *** _pewp_ has quit IRC | 23:58 | |
| *** namnh has quit IRC | 23:58 | |
| *** namnh has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!