*** jangutter has quit IRC | 00:01 | |
*** janki has quit IRC | 00:01 | |
*** nicolasbock has quit IRC | 00:08 | |
*** jangutter has joined #openstack-nova | 00:25 | |
*** Swami has quit IRC | 00:25 | |
*** savvas has quit IRC | 00:26 | |
*** whoami-rajat has quit IRC | 00:30 | |
*** savvas has joined #openstack-nova | 00:32 | |
*** jangutter has quit IRC | 00:34 | |
openstackgerrit | Merged openstack/nova master: Update RequestSpec.flavor on resize_revert https://review.openstack.org/588689 | 00:35 |
---|---|---|
*** gryf has quit IRC | 00:39 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276 | 00:40 |
*** gryf has joined #openstack-nova | 00:41 | |
*** dkehn has quit IRC | 00:46 | |
*** dkehn has joined #openstack-nova | 00:56 | |
*** gyee has quit IRC | 00:59 | |
*** bhagyashris has quit IRC | 01:03 | |
*** savvas has quit IRC | 01:06 | |
*** slaweq_ has quit IRC | 01:06 | |
*** annp has joined #openstack-nova | 01:11 | |
*** sapd1 has joined #openstack-nova | 01:15 | |
*** Dinesh_Bhor has joined #openstack-nova | 01:18 | |
*** dklyle has joined #openstack-nova | 01:19 | |
*** Dinesh_Bhor has quit IRC | 01:19 | |
*** savvas has joined #openstack-nova | 01:20 | |
*** savvas has quit IRC | 01:25 | |
*** Dinesh_Bhor has joined #openstack-nova | 01:27 | |
*** bhagyashris has joined #openstack-nova | 01:28 | |
*** savvas has joined #openstack-nova | 01:30 | |
*** savvas has quit IRC | 01:34 | |
*** dave-mccowan has quit IRC | 01:36 | |
*** dklyle has quit IRC | 01:39 | |
*** savvas has joined #openstack-nova | 01:40 | |
*** jangutter has joined #openstack-nova | 01:43 | |
*** mschuppe1 has joined #openstack-nova | 01:45 | |
*** mrsoul` has joined #openstack-nova | 01:46 | |
*** brinzhang has joined #openstack-nova | 01:46 | |
*** cdent has joined #openstack-nova | 01:47 | |
*** mrsoul has quit IRC | 01:48 | |
*** mschuppert has quit IRC | 01:48 | |
*** bhagyashris has quit IRC | 01:59 | |
*** hongbin has quit IRC | 02:00 | |
*** jangutter has quit IRC | 02:04 | |
*** tssurya has joined #openstack-nova | 02:04 | |
*** cdent has quit IRC | 02:11 | |
*** tssurya has quit IRC | 02:12 | |
*** savvas has quit IRC | 02:12 | |
*** awaugama has quit IRC | 02:14 | |
*** tssurya has joined #openstack-nova | 02:14 | |
*** brinzh has joined #openstack-nova | 02:14 | |
*** brinzh has quit IRC | 02:15 | |
*** savvas has joined #openstack-nova | 02:18 | |
*** mtreinish has quit IRC | 02:25 | |
*** mtreinish has joined #openstack-nova | 02:26 | |
*** brinzhang_ has joined #openstack-nova | 02:31 | |
openstackgerrit | Chen proposed openstack/nova master: Fix evacuate logging https://review.openstack.org/593055 | 02:31 |
openstackgerrit | Chen proposed openstack/nova master: Fix evacuate logging https://review.openstack.org/593055 | 02:34 |
*** sapd1 has quit IRC | 02:34 | |
*** sapd1 has joined #openstack-nova | 02:36 | |
*** sapd1_ has joined #openstack-nova | 02:36 | |
*** brinzhang__ has joined #openstack-nova | 02:45 | |
*** brinzhang_ has left #openstack-nova | 02:48 | |
*** brinzhang has quit IRC | 02:49 | |
*** brinzhang__ is now known as brinzhang | 02:49 | |
*** Bhujay has joined #openstack-nova | 02:50 | |
*** savvas has quit IRC | 02:51 | |
*** brinzhang has quit IRC | 02:52 | |
*** brinzhang has joined #openstack-nova | 02:52 | |
*** vladikr has quit IRC | 02:53 | |
*** vladikr has joined #openstack-nova | 02:53 | |
*** gbarros has joined #openstack-nova | 02:53 | |
*** tssurya has quit IRC | 02:53 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fix mock.patch usage in unit tests https://review.openstack.org/602198 | 02:54 |
gibi | melwitt: replied in the review | 02:56 |
*** savvas has joined #openstack-nova | 02:58 | |
*** Bhujay has quit IRC | 02:59 | |
*** vladikr has quit IRC | 03:10 | |
*** vladikr has joined #openstack-nova | 03:11 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 03:11 |
*** cfriesen has joined #openstack-nova | 03:12 | |
openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node https://review.openstack.org/602219 | 03:17 |
*** Bhujay has joined #openstack-nova | 03:17 | |
*** 77FAAKC1H has joined #openstack-nova | 03:17 | |
*** 77FAAKC1H has quit IRC | 03:18 | |
*** Bhujay has quit IRC | 03:18 | |
*** Bhujay has joined #openstack-nova | 03:19 | |
*** dpawlik has joined #openstack-nova | 03:23 | |
*** dpawlik has quit IRC | 03:28 | |
*** savvas has quit IRC | 03:30 | |
*** savvas has joined #openstack-nova | 03:36 | |
*** Bhujay has quit IRC | 03:41 | |
*** vladikr has quit IRC | 03:42 | |
*** vladikr has joined #openstack-nova | 03:43 | |
*** slaweq_ has joined #openstack-nova | 03:44 | |
*** gbarros has quit IRC | 03:48 | |
*** Dinesh_Bhor has quit IRC | 03:48 | |
*** sorrison has joined #openstack-nova | 03:49 | |
*** slaweq_ has quit IRC | 03:49 | |
*** med_ has joined #openstack-nova | 03:51 | |
*** sorrison has quit IRC | 03:51 | |
*** jangutter has joined #openstack-nova | 03:54 | |
*** jangutter has quit IRC | 03:58 | |
*** erlon has joined #openstack-nova | 04:08 | |
*** savvas has quit IRC | 04:10 | |
*** ivve has joined #openstack-nova | 04:10 | |
*** med_ has quit IRC | 04:18 | |
*** vladikr has quit IRC | 04:18 | |
*** vladikr has joined #openstack-nova | 04:18 | |
*** nicolasbock has joined #openstack-nova | 04:23 | |
*** savvas has joined #openstack-nova | 04:23 | |
*** savvas has quit IRC | 04:28 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276 | 04:32 |
*** savvas has joined #openstack-nova | 04:33 | |
*** Dinesh_Bhor has joined #openstack-nova | 04:36 | |
*** savvas has quit IRC | 04:37 | |
*** savvas has joined #openstack-nova | 04:42 | |
*** claudiub has joined #openstack-nova | 04:44 | |
*** claudiub|2 has joined #openstack-nova | 04:45 | |
*** krtaylor_ has quit IRC | 04:45 | |
*** krtaylor_ has joined #openstack-nova | 04:46 | |
*** claudiub has quit IRC | 04:48 | |
*** Dinesh_Bhor has quit IRC | 04:51 | |
*** claudiub has joined #openstack-nova | 04:51 | |
*** claudiub|2 has quit IRC | 04:53 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 05:02 |
*** Dinesh_Bhor has joined #openstack-nova | 05:04 | |
*** claudiub|2 has joined #openstack-nova | 05:04 | |
*** sapd1 has quit IRC | 05:06 | |
*** sapd1 has joined #openstack-nova | 05:06 | |
*** claudiub has quit IRC | 05:07 | |
*** claudiub|2 has quit IRC | 05:09 | |
*** erlon has quit IRC | 05:12 | |
*** phuongnh has quit IRC | 05:15 | |
*** savvas has quit IRC | 05:16 | |
*** brinzhang has quit IRC | 05:19 | |
*** brinzhang has joined #openstack-nova | 05:20 | |
*** savvas has joined #openstack-nova | 05:21 | |
*** nicolasbock has quit IRC | 05:27 | |
*** artom has joined #openstack-nova | 05:29 | |
*** luksky11 has joined #openstack-nova | 05:29 | |
*** artom has quit IRC | 05:29 | |
*** artom has joined #openstack-nova | 05:30 | |
openstackgerrit | Tony Breeds proposed openstack/python-novaclient stable/queens: Switch to stestr https://review.openstack.org/601933 | 05:30 |
openstackgerrit | Tony Breeds proposed openstack/python-novaclient stable/queens: import zuul job settings from project-config https://review.openstack.org/601400 | 05:30 |
*** jiaopengju has joined #openstack-nova | 05:38 | |
*** sapd1_ has quit IRC | 05:45 | |
openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node https://review.openstack.org/602219 | 05:47 |
*** savvas has quit IRC | 05:55 | |
*** Eran_Kuris has joined #openstack-nova | 05:56 | |
*** savvas has joined #openstack-nova | 06:01 | |
*** mschuppe1 is now known as mschuppert | 06:13 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 06:14 |
*** cfriesen has quit IRC | 06:22 | |
*** dpawlik has joined #openstack-nova | 06:23 | |
*** sahid has joined #openstack-nova | 06:32 | |
*** savvas has quit IRC | 06:33 | |
*** fanzhang has quit IRC | 06:34 | |
*** cfriesen has joined #openstack-nova | 06:34 | |
*** fanzhang has joined #openstack-nova | 06:34 | |
*** adrianc has joined #openstack-nova | 06:36 | |
*** jangutter has joined #openstack-nova | 06:36 | |
*** Luzi has joined #openstack-nova | 06:36 | |
*** jangutter has quit IRC | 06:37 | |
*** savvas has joined #openstack-nova | 06:39 | |
*** jangutter has joined #openstack-nova | 06:42 | |
*** savvas has quit IRC | 06:44 | |
*** savvas has joined #openstack-nova | 06:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/601047 | 07:01 |
*** rcernin has quit IRC | 07:01 | |
*** tesseract has joined #openstack-nova | 07:01 | |
*** savvas has quit IRC | 07:02 | |
*** jangutter has quit IRC | 07:04 | |
*** savvas has joined #openstack-nova | 07:06 | |
*** ivve has quit IRC | 07:16 | |
*** cfriesen has quit IRC | 07:16 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 07:17 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 07:19 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): support compute node resource provider update https://review.openstack.org/521041 | 07:33 |
*** AlexeyAbashkin has joined #openstack-nova | 07:34 | |
*** Dinesh_Bhor has quit IRC | 07:34 | |
*** kaliya has joined #openstack-nova | 07:38 | |
*** savvas has quit IRC | 07:40 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 07:41 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 07:42 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): support compute node resource provider update https://review.openstack.org/521041 | 07:42 |
*** jpena|off is now known as jpena | 07:44 | |
*** savvas has joined #openstack-nova | 07:46 | |
*** Dinesh_Bhor has joined #openstack-nova | 07:47 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041 | 07:49 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 07:50 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041 | 07:51 |
*** Dinesh_Bhor has quit IRC | 07:54 | |
*** marvin_mhg has quit IRC | 07:59 | |
*** marvin_mhg has joined #openstack-nova | 08:01 | |
*** oanson has quit IRC | 08:04 | |
*** artom has quit IRC | 08:18 | |
*** artom has joined #openstack-nova | 08:18 | |
*** artom has quit IRC | 08:19 | |
*** artom has joined #openstack-nova | 08:19 | |
*** oanson has joined #openstack-nova | 08:19 | |
*** artom has quit IRC | 08:20 | |
*** savvas has quit IRC | 08:20 | |
*** artom has joined #openstack-nova | 08:20 | |
*** artom has quit IRC | 08:21 | |
*** artom has joined #openstack-nova | 08:21 | |
*** artom has quit IRC | 08:22 | |
*** artom_ has joined #openstack-nova | 08:22 | |
*** artom_ is now known as artom | 08:22 | |
*** artom has quit IRC | 08:23 | |
*** savvas has joined #openstack-nova | 08:25 | |
*** Dinesh_Bhor has joined #openstack-nova | 08:26 | |
*** s10 has joined #openstack-nova | 08:31 | |
*** AlexeyAbashkin has quit IRC | 08:38 | |
*** AlexeyAbashkin has joined #openstack-nova | 08:38 | |
*** annp has quit IRC | 08:46 | |
*** jiapei has quit IRC | 08:52 | |
*** trungnv has quit IRC | 08:54 | |
*** trungnv has joined #openstack-nova | 08:54 | |
*** adrianc_ has joined #openstack-nova | 08:56 | |
*** adrianc has quit IRC | 08:57 | |
*** savvas has quit IRC | 08:58 | |
*** savvas has joined #openstack-nova | 09:04 | |
*** dtantsur|afk is now known as dtantsur | 09:04 | |
*** brinzh has joined #openstack-nova | 09:05 | |
*** savvas has quit IRC | 09:37 | |
*** maciejjozefczyk has quit IRC | 09:41 | |
*** dpawlik has quit IRC | 09:43 | |
*** annp has joined #openstack-nova | 09:44 | |
*** Dinesh_Bhor has quit IRC | 09:45 | |
*** savvas has joined #openstack-nova | 09:51 | |
*** maciejjozefczyk has joined #openstack-nova | 09:54 | |
*** dpawlik has joined #openstack-nova | 09:54 | |
*** savvas has quit IRC | 09:56 | |
*** adrianc_ has quit IRC | 09:59 | |
*** savvas has joined #openstack-nova | 10:01 | |
*** Dinesh_Bhor has joined #openstack-nova | 10:04 | |
*** savvas has quit IRC | 10:05 | |
*** adrianc_ has joined #openstack-nova | 10:10 | |
*** savvas has joined #openstack-nova | 10:10 | |
*** Dinesh_Bhor has quit IRC | 10:26 | |
*** annp has quit IRC | 10:28 | |
*** slaweq__ has joined #openstack-nova | 10:36 | |
*** savvas has quit IRC | 10:43 | |
*** savvas has joined #openstack-nova | 10:49 | |
*** slaweq__ has quit IRC | 10:52 | |
*** s1061123 has joined #openstack-nova | 11:15 | |
*** jamesdenton has joined #openstack-nova | 11:40 | |
*** nicolasbock has joined #openstack-nova | 11:40 | |
*** vladikr has quit IRC | 11:48 | |
*** vladikr has joined #openstack-nova | 11:49 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Support initial allocation ratios https://review.openstack.org/552105 | 11:50 |
*** Eran_Kuris has quit IRC | 12:16 | |
*** tbachman has joined #openstack-nova | 12:22 | |
*** szaher_ has joined #openstack-nova | 12:24 | |
*** savvas has quit IRC | 12:25 | |
*** tetsuro has joined #openstack-nova | 12:28 | |
*** tetsuro has quit IRC | 12:29 | |
*** tetsuro has joined #openstack-nova | 12:29 | |
*** munimeha1 has joined #openstack-nova | 12:30 | |
*** savvas has joined #openstack-nova | 12:32 | |
*** belmoreira has quit IRC | 12:33 | |
*** lyan has joined #openstack-nova | 12:36 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276 | 12:36 |
*** savvas has quit IRC | 12:37 | |
*** brinzhang has quit IRC | 12:39 | |
*** brinzh has quit IRC | 12:40 | |
*** savvas has joined #openstack-nova | 12:42 | |
*** savvas has quit IRC | 12:51 | |
*** belmoreira has joined #openstack-nova | 12:51 | |
*** dave-mccowan has joined #openstack-nova | 12:52 | |
*** dave-mccowan has quit IRC | 12:57 | |
*** med_ has joined #openstack-nova | 12:58 | |
*** dave-mcc_ has joined #openstack-nova | 12:59 | |
*** markvoelker has joined #openstack-nova | 12:59 | |
*** savvas has joined #openstack-nova | 13:00 | |
*** adrianc_ has quit IRC | 13:01 | |
*** jaypipes has joined #openstack-nova | 13:01 | |
*** adrianc_ has joined #openstack-nova | 13:02 | |
*** szaher_ has quit IRC | 13:02 | |
*** szaher has joined #openstack-nova | 13:05 | |
*** gbarros has joined #openstack-nova | 13:07 | |
openstackgerrit | Hamdy Khader proposed openstack/nova master: Set defult value of num_nvme_discover_tries=5 https://review.openstack.org/602351 | 13:15 |
*** savvas has quit IRC | 13:18 | |
*** savvas has joined #openstack-nova | 13:18 | |
*** jaypipes has quit IRC | 13:19 | |
*** lbragstad has quit IRC | 13:25 | |
*** Eran_Kuris has joined #openstack-nova | 13:28 | |
*** savvas has quit IRC | 13:30 | |
*** savvas has joined #openstack-nova | 13:31 | |
openstackgerrit | Claudiu Belu proposed openstack/nova master: conductor: add live_resize task https://review.openstack.org/248579 | 13:34 |
openstackgerrit | Claudiu Belu proposed openstack/nova master: API: Adds instance live-resize https://review.openstack.org/184584 | 13:35 |
*** awaugama has joined #openstack-nova | 13:36 | |
*** munimeha1_ has joined #openstack-nova | 13:37 | |
*** erlon has joined #openstack-nova | 13:39 | |
*** jangutter has joined #openstack-nova | 13:48 | |
*** tetsuro has quit IRC | 13:48 | |
*** mvenesio has joined #openstack-nova | 13:50 | |
*** trozet has joined #openstack-nova | 13:56 | |
*** tetsuro has joined #openstack-nova | 13:59 | |
*** tetsuro_ has joined #openstack-nova | 13:59 | |
openstackgerrit | Claudiu Belu proposed openstack/nova master: WIP: replace spec with autospec https://review.openstack.org/557299 | 14:01 |
*** claudiub has joined #openstack-nova | 14:01 | |
*** belmoreira has quit IRC | 14:02 | |
*** jangutter has quit IRC | 14:03 | |
*** erlon has quit IRC | 14:03 | |
*** tetsuro has quit IRC | 14:04 | |
*** mdbooth has joined #openstack-nova | 14:10 | |
*** munimeha1_ has quit IRC | 14:14 | |
*** munimeha1 has quit IRC | 14:14 | |
*** savvas has quit IRC | 14:21 | |
*** maciejjozefczyk has quit IRC | 14:22 | |
*** lbragstad has joined #openstack-nova | 14:23 | |
*** Luzi has quit IRC | 14:27 | |
*** pcaruana has joined #openstack-nova | 14:28 | |
*** savvas has joined #openstack-nova | 14:30 | |
*** slaweq__ has joined #openstack-nova | 14:32 | |
*** savvas has quit IRC | 14:34 | |
*** eharney has joined #openstack-nova | 14:35 | |
litao__ | hi | 14:38 |
litao__ | This is my patch:https://review.openstack.org/#/c/592252/ I need some reviews , thanks | 14:39 |
*** savvas has joined #openstack-nova | 14:39 | |
*** sapd1 has quit IRC | 14:40 | |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 14:40 |
openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 14:40 |
*** mriedem has joined #openstack-nova | 14:41 | |
*** sapd1 has joined #openstack-nova | 14:42 | |
*** hoangcx has joined #openstack-nova | 14:42 | |
kashyap | litao__: Hi, many folks are at a conference this week, so expect some delays in responses | 14:42 |
*** belmoreira has joined #openstack-nova | 14:43 | |
*** savvas has quit IRC | 14:44 | |
*** whoami-rajat has joined #openstack-nova | 14:45 | |
mriedem | dansmith: this was the thing i mentioned at breakfast re resize reschedule up-calls; do the proposed solutions inline make sense? https://blueprints.launchpad.net/nova/+spec/fix-reschedule-up-calls | 14:45 |
*** bhagyashris has joined #openstack-nova | 14:46 | |
*** tetsuro_ has quit IRC | 14:48 | |
*** savvas has joined #openstack-nova | 14:48 | |
*** med_ has quit IRC | 14:49 | |
mriedem | bauzas: here is the fugly cross_az_attach=False bug fix i mentioned yesterday https://review.openstack.org/#/c/469675/ | 14:50 |
mriedem | note that i plan on writing an alternate, less tightly coupled approach to fixing this laid out in https://review.openstack.org/#/c/469675/4/nova/compute/api.py@918 | 14:50 |
*** bnemec has quit IRC | 14:52 | |
*** jangutter has joined #openstack-nova | 14:52 | |
*** tetsuro has joined #openstack-nova | 14:52 | |
*** tetsuro has quit IRC | 14:52 | |
*** tetsuro_ has joined #openstack-nova | 14:53 | |
lyarwood | bauzas: mind taking a look at https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1787606 this morning if you get a chance | 14:53 |
*** slaweq__ has quit IRC | 14:53 | |
bauzas | mriedem: ta, will look at both | 14:53 |
*** tetsuro has joined #openstack-nova | 14:53 | |
mriedem | edleafe: you might want to take a look at https://review.openstack.org/#/c/593074/ from lyarwood above | 14:54 |
openstackgerrit | Claudiu Belu proposed openstack/nova master: conductor: add live_resize task https://review.openstack.org/248579 | 14:54 |
openstackgerrit | Claudiu Belu proposed openstack/nova master: API: Adds instance live-resize https://review.openstack.org/184584 | 14:54 |
bauzas | lyarwood: yup, saw your change yesterday | 14:54 |
*** slaweq__ has joined #openstack-nova | 14:55 | |
*** savvas has quit IRC | 14:55 | |
*** adrianc_ has quit IRC | 14:56 | |
*** efried has joined #openstack-nova | 14:57 | |
lyarwood | bauzas: well rebase, I'll fix the failing func tests if it's a valid fix | 14:57 |
*** tetsuro_ has quit IRC | 14:57 | |
bauzas | lyarwood: mriedem: could we just one day say we deprecate multiple instances creation ? :) | 14:57 |
edleafe | mriedem: One of the early discussions on alternate hosts was the concern about providing an alternate that may have already been consumed by another instance from the request. If that's no longer a concern, then that patch is fine. | 14:58 |
*** mdbooth has quit IRC | 14:58 | |
*** bhagyashris has quit IRC | 14:58 | |
*** savvas has joined #openstack-nova | 14:58 | |
*** mdbooth has joined #openstack-nova | 14:58 | |
bauzas | edleafe: the problem is that we don't know yet if we will consume the host | 14:59 |
mriedem | edleafe: yeah that's what i mentioned to lya | 14:59 |
mriedem | *lyarwood | 14:59 |
*** jaosorior has joined #openstack-nova | 14:59 | |
bauzas | I mean, when it goes to pick a destination, we don't know yet if the allocation is possible | 14:59 |
*** bhagyashris has joined #openstack-nova | 15:02 | |
*** tbachman has quit IRC | 15:02 | |
bauzas | nevermind, we consumed the host before we get the alternates | 15:02 |
*** eharney has quit IRC | 15:03 | |
mriedem | i think the point is we don't want several reschedules trying to clobber the same alternate | 15:03 |
mriedem | maybe we should randomize the alternats? | 15:03 |
edleafe | bauzas: the selected_host has already claimed the resources at that point. Alternates only are provided if something else has prevented the selected host from bulding the instance. The feeling expressed when I wrote that is that we didn't want to include a host that already had some of its resources consumed | 15:03 |
mriedem | *alternates? | 15:03 |
*** abhishekk has joined #openstack-nova | 15:03 | |
melwitt | gibi: where did you send a mail about the minimum bandwidth scheduling demo location? I didn't see it | 15:03 |
*** jiapei has joined #openstack-nova | 15:04 | |
*** cfriesen has joined #openstack-nova | 15:06 | |
gibi | melwitt: http://lists.openstack.org/pipermail/openstack-dev/2018-September/134622.html | 15:06 |
*** owalsh has quit IRC | 15:07 | |
*** owalsh has joined #openstack-nova | 15:07 | |
bauzas | mriedem: edleafe: yeah, the problem is that we don't optimize how we sort the alternates | 15:07 |
melwitt | gibi: oh, yeah, ok. the same one I read /facepalm | 15:08 |
melwitt | yesterday. I was thinking there was some other new one | 15:08 |
*** lei-zh has joined #openstack-nova | 15:08 | |
bauzas | mriedem: edleafe: meaning, instances could be optimized to be claimed spreadly, but then they could have the same host as alternate | 15:08 |
gibi | melwitt: if there was a new one yesterday then I didn't see that :) | 15:09 |
bauzas | so, yeah, we could just randomize the alternates, or find a better strategy | 15:09 |
*** itlinux has quit IRC | 15:09 | |
edleafe | lyarwood: I'm curious as to the motivation for this. Have you seen a number of build failures on hosts that would require more alternates? | 15:09 |
*** erlon has joined #openstack-nova | 15:09 | |
*** takashin has joined #openstack-nova | 15:10 | |
bauzas | edleafe: see the bug report | 15:10 |
bauzas | in a small env, this could fail | 15:10 |
bauzas | 5 instances, 2 nodes | 15:10 |
bauzas | so, yeah, we're over rejecting | 15:10 |
*** tssurya has joined #openstack-nova | 15:10 | |
edleafe | Ah, when you are requesting more instances in a single build than you have available hosts | 15:11 |
*** dklyle has joined #openstack-nova | 15:11 | |
bauzas | we should less reject, but we should also shuffle a little bit to spread more | 15:11 |
*** cdent has joined #openstack-nova | 15:11 | |
bauzas | edleafe: that's correct | 15:11 |
*** damien_r has quit IRC | 15:11 | |
*** sorrison has joined #openstack-nova | 15:11 | |
edleafe | Patient: "Doctor, it hurts when I do this" Doctor: "Then stop doing that!" | 15:12 |
edleafe | :) | 15:12 |
mriedem | then i'd make the logic conditional on whether or not we have num_instances > filtered_hosts | 15:14 |
*** kaliya has quit IRC | 15:14 | |
*** cdent has quit IRC | 15:14 | |
*** imacdonn has quit IRC | 15:14 | |
*** cdent has joined #openstack-nova | 15:15 | |
*** imacdonn has joined #openstack-nova | 15:15 | |
*** holser|ptg has joined #openstack-nova | 15:15 | |
*** gyee has joined #openstack-nova | 15:15 | |
*** nicolasbock has quit IRC | 15:15 | |
*** bnemec has joined #openstack-nova | 15:15 | |
bauzas | mriedem: edleafe: well | 15:16 |
*** savvas has quit IRC | 15:17 | |
*** sorrison has quit IRC | 15:17 | |
*** savvas has joined #openstack-nova | 15:17 | |
*** sorrison has joined #openstack-nova | 15:17 | |
bauzas | mriedem: edleafe: the above is a real bug if and only if what was claimed is not able to be spawned, right? | 15:18 |
bauzas | because we claimed before we look for alternates, so from a pure capacity point of view, we should be good | 15:18 |
edleafe | bauzas: it depends on the reason for the failure. | 15:19 |
bauzas | edleafe: we trigger alternates because something else but a capacitiy issue, right ? | 15:19 |
edleafe | bauzas: if it was because of insufficient resources, then we have a BIG problem | 15:19 |
*** mdbooth has quit IRC | 15:19 | |
edleafe | bauzas: yes, that's the design | 15:20 |
*** mdbooth has joined #openstack-nova | 15:20 | |
*** alexchadin has joined #openstack-nova | 15:20 | |
edleafe | We've already claimed resources before a host is designated as the selected_host | 15:20 |
*** sorrison has quit IRC | 15:21 | |
*** sorrison has joined #openstack-nova | 15:21 | |
bauzas | edleafe: yup, I'm just stating this | 15:22 |
bauzas | edleafe: alternates are for derailed requests that shouldn't be capacity problems | 15:22 |
bauzas | so | 15:22 |
*** med_ has joined #openstack-nova | 15:23 | |
*** sahid has quit IRC | 15:23 | |
bauzas | back to the possible solutions, I think lyarwood's proposal is good with the slight detail that we could just shuffle a bit | 15:23 |
mriedem | i just want to make sure that if i'm spawning 5 instances with 4 hosts, is the first alternate for all instances going to be the same host? | 15:24 |
mriedem | or is the answer "it depends" | 15:24 |
*** eharney has joined #openstack-nova | 15:25 | |
*** hoangcx has quit IRC | 15:25 | |
bauzas | mriedem: there is a TODO on this | 15:26 |
bauzas | mriedem: edleafe: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L411 | 15:27 |
bauzas | mriedem: that's the shuffling bit I'd like to have | 15:27 |
bauzas | we just get the first element and we consider it to be the alternate | 15:27 |
bauzas | that's highly probabilistic that we gonna end on the same alternate | 15:28 |
bauzas | so, just fix the TODO and do a bit of a guess instead of getting the first element of the list | 15:28 |
bauzas | edleafe: alternates are all seen as viable targets, rightN | 15:29 |
bauzas | right? | 15:29 |
bauzas | from a capacity pov | 15:29 |
bauzas | so, there is no harm to pick a random one and not just the first in the list | 15:29 |
bauzas | edleafe: you agree ? | 15:29 |
bauzas | lyarwood: I made some very technical comments on the functional test | 15:30 |
bauzas | lyarwood: if you need help for a respin, lemme know | 15:30 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 15:31 |
edleafe | bauzas: that's a different version of shuffling, for the nested case | 15:31 |
edleafe | IOW, if a single RP has multiple allocation_candidates, just pick the first for now | 15:31 |
edleafe | The TODO is to evaluate each for "best fit" | 15:31 |
*** Eran_Kuris has quit IRC | 15:33 | |
*** savvas has quit IRC | 15:33 | |
*** slaweq__ is now known as slaweq | 15:34 | |
lyarwood | bauzas: thanks, I'll get to it after this session | 15:35 |
edleafe | bauzas: I don't see a problem with a random selection, but in the two-node setup described in the bug, it won't help :) | 15:35 |
bauzas | edleafe: well, it could help or not | 15:36 |
bauzas | edleafe: but at least, we would ensure that if we have more than 1 host as alternate, we don't pick the same for all the instances :) | 15:36 |
edleafe | bauzas: in a two-node setup, there is only one possible alternate :) | 15:36 |
bauzas | edleafe: yeah, sure, no disagreement on it :D | 15:36 |
bauzas | actually, my guess is that the example we have in the bug report will fail anyway | 15:37 |
edleafe | bauzas: the other thing to remember is that the list of hosts have been weighed at this point | 15:37 |
edleafe | So a random choice will ignore all weighting | 15:38 |
bauzas | ah good point | 15:38 |
bauzas | mmmm | 15:38 |
bauzas | ok, let's keep it simple then | 15:38 |
bauzas | if alternates follow the packing/spreading strategy given by the weighers, they're already shuffled | 15:39 |
bauzas | so, what lyarwood did is sufficient | 15:39 |
*** savvas has joined #openstack-nova | 15:41 | |
*** macza has joined #openstack-nova | 15:41 | |
openstackgerrit | Eric Fried proposed openstack/os-traits master: Add OWNERSHIP_$SERVICE traits https://review.openstack.org/602160 | 15:43 |
*** hamzy has quit IRC | 15:43 | |
*** belmoreira has quit IRC | 15:46 | |
*** savvas has quit IRC | 15:46 | |
bauzas | edleafe: wait a sec | 15:47 |
bauzas | edleafe: is https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L417 a weighed list ? | 15:48 |
*** belmoreira has joined #openstack-nova | 15:49 | |
edleafe | ARs are not weighed. The hosts they represent are | 15:49 |
bauzas | that's old greek to me :) | 15:49 |
bauzas | oh shit | 15:50 |
bauzas | I think I misunderstood the logic | 15:51 |
*** Sukhdev has joined #openstack-nova | 15:51 | |
bauzas | we got selected hosts, we iterate over those and see whether they have an AR against each | 15:51 |
*** savvas has joined #openstack-nova | 15:51 | |
bauzas | gotcha | 15:51 |
bauzas | so, yeah, selected hosts is weighed | 15:52 |
edleafe | bauzas: this is where the alternates are created: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L260, and the 'hosts' variable is defined above: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L260 | 15:52 |
*** belmoreira has quit IRC | 15:53 | |
*** betherly_ is now known as betherly | 15:54 | |
bauzas | yeah I got it | 15:54 |
edleafe | bauzas: oops: the second link should be https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L201 | 15:54 |
bauzas | https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L397 is the loop over hosts that are weighed | 15:54 |
bauzas | so, yeah, we basically either spread or pack that list | 15:55 |
bauzas | so, we're fine | 15:55 |
edleafe | yup | 15:55 |
bauzas | no need to shuffle any bit | 15:55 |
bauzas | if the operator chose the packing strategy, that's their decision | 15:55 |
*** med_ has quit IRC | 15:58 | |
*** savvas has quit IRC | 16:00 | |
*** itlinux has joined #openstack-nova | 16:00 | |
openstackgerrit | sean mooney proposed openstack/os-vif master: add support for generic tap device plug https://review.openstack.org/602384 | 16:01 |
*** nicolasbock has joined #openstack-nova | 16:02 | |
*** savvas has joined #openstack-nova | 16:05 | |
bauzas | edleafe: you know what ? I'm horribly confused | 16:08 |
openstackgerrit | Eric Fried proposed openstack/os-traits master: Add OWNERSHIP_$SERVICE traits https://review.openstack.org/602160 | 16:11 |
bauzas | edleafe: how many times are we calling _get_alternate_hosts() if we have multiple instances ? one AFAICS | 16:11 |
*** sorrison has quit IRC | 16:11 | |
mriedem | mdbooth: i'm not sure why but my evacuate script is thinking that we only have 1 hypervisor in this job but we should have 2 https://review.openstack.org/#/c/602174/ - i'm guessing it's a bug in my script doing the word count | 16:13 |
mriedem | oh i see why | 16:14 |
mriedem | well i think | 16:14 |
*** sorrison has joined #openstack-nova | 16:15 | |
*** itlinux is now known as itlinux-away | 16:15 | |
mriedem | $ echo "Missing value auth-url required for auth plugin password" | wc -l | 16:15 |
mriedem | 1 | 16:15 |
*** itlinux-away is now known as itlinux | 16:17 | |
*** med_ has joined #openstack-nova | 16:17 | |
*** itlinux is now known as itlinux-away | 16:17 | |
*** dave-mcc_ has quit IRC | 16:25 | |
*** itlinux-away is now known as itlinux | 16:25 | |
*** itlinux is now known as itlinux-away | 16:25 | |
*** itlinux-away is now known as itlinux | 16:25 | |
*** itlinux is now known as itlinux-away | 16:25 | |
*** mdbooth_ has joined #openstack-nova | 16:27 | |
*** mdbooth_ has quit IRC | 16:27 | |
*** itlinux-away is now known as itlinux | 16:27 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: consumer gen: more tests for delete allocation cases https://review.openstack.org/591811 | 16:29 |
edleafe | bauzas: We call it once for each requested instance | 16:29 |
*** tetsuro has quit IRC | 16:31 | |
*** cfriesen has quit IRC | 16:32 | |
*** munimeha1 has joined #openstack-nova | 16:32 | |
*** awaugama has quit IRC | 16:32 | |
*** tetsuro has joined #openstack-nova | 16:34 | |
*** eharney has quit IRC | 16:35 | |
*** bodgix has left #openstack-nova | 16:35 | |
*** munimeha1_ has joined #openstack-nova | 16:38 | |
*** dave-mcc_ has joined #openstack-nova | 16:38 | |
*** cfriesen has joined #openstack-nova | 16:39 | |
*** alexchadin has quit IRC | 16:40 | |
*** savvas has quit IRC | 16:41 | |
*** munimeha1 has quit IRC | 16:42 | |
*** dtantsur is now known as dtantsur|afk | 16:43 | |
*** adrianc_ has joined #openstack-nova | 16:43 | |
*** savvas has joined #openstack-nova | 16:44 | |
*** s10 has quit IRC | 16:45 | |
*** bhagyashris has quit IRC | 16:46 | |
*** munimeha1_ has quit IRC | 16:47 | |
*** alexchadin has joined #openstack-nova | 16:48 | |
openstackgerrit | Merged openstack/nova master: Fix mock.patch usage in unit tests https://review.openstack.org/602198 | 16:50 |
*** bhagyashris has joined #openstack-nova | 16:51 | |
bauzas | edleafe: I tried to summarize my thoughts in one single comment https://review.openstack.org/#/c/593074/2/nova/scheduler/filter_scheduler.py@397 | 16:55 |
*** erlon has quit IRC | 16:56 | |
*** tesseract has quit IRC | 17:00 | |
*** brault has quit IRC | 17:00 | |
*** brault has joined #openstack-nova | 17:02 | |
*** alexchadin has quit IRC | 17:02 | |
*** brault has quit IRC | 17:05 | |
*** sorrison has quit IRC | 17:05 | |
*** alexchadin has joined #openstack-nova | 17:05 | |
*** sorrison has joined #openstack-nova | 17:05 | |
*** jpena is now known as jpena|off | 17:07 | |
*** sorrison has quit IRC | 17:07 | |
*** sorrison has joined #openstack-nova | 17:08 | |
*** priteau has joined #openstack-nova | 17:10 | |
*** lei-zh has quit IRC | 17:10 | |
*** eharney has joined #openstack-nova | 17:11 | |
*** lei-zh has joined #openstack-nova | 17:11 | |
*** brault has joined #openstack-nova | 17:12 | |
*** erlon has joined #openstack-nova | 17:17 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Update RequestSpec.flavor on resize_revert https://review.openstack.org/602403 | 17:17 |
*** savvas has quit IRC | 17:18 | |
*** hamzy has joined #openstack-nova | 17:18 | |
*** cdent has quit IRC | 17:21 | |
*** savvas has joined #openstack-nova | 17:24 | |
*** itlinux is now known as itlinux-away | 17:27 | |
*** itlinux-away is now known as itlinux | 17:29 | |
*** itlinux is now known as itlinux-away | 17:29 | |
*** sorrison has quit IRC | 17:30 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Noop CantStartEngineError in targets_cell if API DB not configured https://review.openstack.org/581912 | 17:31 |
*** itlinux-away is now known as itlinux | 17:32 | |
melwitt | efried: can you add some notes about the whitelist plan on the etherpad? I wasn't sure which line we're talking | 17:32 |
*** itlinux is now known as itlinux-away | 17:32 | |
*** AlexeyAbashkin has quit IRC | 17:32 | |
*** sorrison has joined #openstack-nova | 17:32 | |
efried | melwitt: roger that. It's here: https://review.openstack.org/#/c/591037/ Where do you want it on which etherpad? :) | 17:33 |
*** itlinux-away is now known as itlinux | 17:34 | |
melwitt | efried: oh, if it's already known, that's cool | 17:34 |
*** itlinux is now known as itlinux-away | 17:34 | |
efried | melwitt: Agree it would be useful to have on one or both etherpads | 17:34 |
melwitt | efried: yeah, I wanted there to be a reference for cyborg team to see how that part will be handled. I don't see a relevant section in the etherpad https://etherpad.openstack.org/p/stein-ptg.cyborg-nova-new so far | 17:35 |
efried | oh, yeah, that spec is in a topic of its own later on.... | 17:35 |
efried | melwitt: L291 of https://etherpad.openstack.org/p/nova-ptg-stein | 17:35 |
melwitt | ah thanks | 17:36 |
mriedem | actually 214 | 17:38 |
mriedem | i see what efried is trying to do | 17:38 |
*** itlinux-away is now known as itlinux | 17:38 | |
*** itlinux is now known as itlinux-away | 17:38 | |
mriedem | oh nvm - different topic :) | 17:38 |
*** cdent has joined #openstack-nova | 17:39 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: cover migration cases with functional tests https://review.openstack.org/602408 | 17:39 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Update RequestSpec.flavor on resize_revert https://review.openstack.org/602409 | 17:39 |
*** vladikr is now known as vladikr_brb | 17:40 | |
*** itlinux-away is now known as itlinux | 17:41 | |
*** luksky11 has quit IRC | 17:41 | |
*** savvas has quit IRC | 17:43 | |
*** savvas has joined #openstack-nova | 17:43 | |
*** bnemec has quit IRC | 17:45 | |
*** alexchadin has quit IRC | 17:48 | |
*** mdbooth has quit IRC | 17:52 | |
lyarwood | edleafe: sorry missed your ping earlier, I'm not sure how that additional conditional would help tbh | 17:53 |
*** mdbooth has joined #openstack-nova | 17:53 | |
bauzas | mriedem: that's the bug right ? https://bugs.launchpad.net/nova/+bug/1784020 | 17:55 |
openstack | Launchpad bug 1784020 in OpenStack Compute (nova) "Shared storage providers are not supported and will break things if used" [Medium,Fix released] | 17:55 |
bauzas | thanks for the heads-up | 17:56 |
openstackgerrit | Merged openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943 | 17:56 |
*** cfriesen has quit IRC | 17:56 | |
mriedem | yes | 17:56 |
*** cfriesen has joined #openstack-nova | 17:56 | |
edleafe | lyarwood: because the bug only is an issue when there aren't enough potential alternate hosts, right? | 17:57 |
lyarwood | edleafe: right and alternates listed are per instance | 17:58 |
lyarwood | alternates are listed* | 17:58 |
*** vladikr_brb is now known as vladikr | 17:58 | |
edleafe | lyarwood: so in the more common case where there are many more alternates available, we would be better off not adding hosts that may not have enoguh resources | 17:59 |
edleafe | lyarwood: only in the case where there aren't enough alternates should we include a host that is actively being built on | 18:00 |
*** tbachman has joined #openstack-nova | 18:01 | |
*** cfriesen has quit IRC | 18:01 | |
lyarwood | edleafe: ah right, got it | 18:02 |
*** cfriesen has joined #openstack-nova | 18:02 | |
*** kaisers has quit IRC | 18:03 | |
openstackgerrit | Eric Fried proposed openstack/os-traits master: Add OWNERSHIP_$SERVICE traits https://review.openstack.org/602160 | 18:04 |
*** luksky has joined #openstack-nova | 18:04 | |
*** abhishekk has quit IRC | 18:06 | |
*** sorrison has quit IRC | 18:10 | |
*** mdbooth has quit IRC | 18:11 | |
*** savvas has quit IRC | 18:16 | |
*** savvas has joined #openstack-nova | 18:17 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add post-test hook for testing evacuate https://review.openstack.org/602174 | 18:18 |
*** kaisers has joined #openstack-nova | 18:20 | |
*** Sukhdev has quit IRC | 18:23 | |
*** AJaeger has joined #openstack-nova | 18:24 | |
*** AJaeger has left #openstack-nova | 18:24 | |
*** slaweq has quit IRC | 18:25 | |
*** priteau has quit IRC | 18:27 | |
*** tssurya has quit IRC | 18:30 | |
*** lbragstad has quit IRC | 18:33 | |
*** hemna_ has joined #openstack-nova | 18:36 | |
*** bhagyashris has quit IRC | 18:37 | |
*** efried has quit IRC | 18:38 | |
*** jangutter has quit IRC | 18:39 | |
*** cdent has quit IRC | 18:39 | |
*** tetsuro has quit IRC | 18:39 | |
*** jangutter has joined #openstack-nova | 18:39 | |
*** jangutter has quit IRC | 18:39 | |
*** tbachman has quit IRC | 18:39 | |
*** mriedem has quit IRC | 18:39 | |
*** lei-zh has quit IRC | 18:41 | |
*** erlon has quit IRC | 18:42 | |
*** takashin has quit IRC | 18:42 | |
*** cfriesen has quit IRC | 18:43 | |
*** burt has joined #openstack-nova | 18:43 | |
*** nicolasbock has quit IRC | 18:51 | |
*** rcernin has joined #openstack-nova | 19:00 | |
*** alexchadin has joined #openstack-nova | 19:00 | |
*** adrianc_ has quit IRC | 19:00 | |
*** med_ has quit IRC | 19:00 | |
*** rtjure has joined #openstack-nova | 19:03 | |
*** geekinutah has joined #openstack-nova | 19:06 | |
*** cdent has joined #openstack-nova | 19:11 | |
*** itlinux is now known as itlinux-away | 19:22 | |
*** itlinux-away is now known as itlinux | 19:23 | |
*** itlinux is now known as itlinux-away | 19:24 | |
*** itlinux-away is now known as itlinux | 19:24 | |
*** itlinux is now known as itlinux-away | 19:24 | |
*** itlinux-away is now known as itlinux | 19:24 | |
*** itlinux is now known as itlinux-away | 19:24 | |
*** itlinux-away is now known as itlinux | 19:25 | |
*** itlinux is now known as itlinux-away | 19:26 | |
*** lbragstad has joined #openstack-nova | 19:26 | |
*** slaweq has joined #openstack-nova | 19:28 | |
*** lbragstad has quit IRC | 19:31 | |
*** alexchadin has quit IRC | 19:31 | |
*** itlinux-away is now known as itlinux | 19:33 | |
*** itlinux is now known as itlinux-away | 19:33 | |
*** gbarros has quit IRC | 19:35 | |
*** itlinux-away is now known as itlinux | 19:36 | |
*** itlinux is now known as itlinux-away | 19:36 | |
*** priteau has joined #openstack-nova | 19:41 | |
*** cfriesen has joined #openstack-nova | 19:45 | |
*** alexchadin has joined #openstack-nova | 19:46 | |
*** itlinux-away is now known as itlinux | 19:49 | |
*** cfriesen has quit IRC | 19:50 | |
*** itlinux is now known as itlinux-away | 19:50 | |
*** lbragstad has joined #openstack-nova | 19:50 | |
*** itlinux-away is now known as itlinux | 19:50 | |
*** bnemec has joined #openstack-nova | 19:54 | |
*** cdent has quit IRC | 19:55 | |
*** awaugama has joined #openstack-nova | 19:55 | |
*** sorrison has joined #openstack-nova | 19:55 | |
*** tetsuro has joined #openstack-nova | 19:55 | |
*** mriedem has joined #openstack-nova | 19:55 | |
mriedem | gibi: for your bw provider demo, osc commands take a wrap option, might be good to use - you're doing this at the summit too i think | 19:56 |
*** cfriesen has joined #openstack-nova | 19:56 | |
*** tetsuro has quit IRC | 19:58 | |
*** vladikr has quit IRC | 20:02 | |
*** vladikr has joined #openstack-nova | 20:03 | |
openstackgerrit | sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.openstack.org/602432 | 20:03 |
*** cdent has joined #openstack-nova | 20:05 | |
*** bhagyashris has joined #openstack-nova | 20:05 | |
*** jangutter has joined #openstack-nova | 20:05 | |
*** slaweq has quit IRC | 20:07 | |
*** gbarros has joined #openstack-nova | 20:08 | |
*** takashin has joined #openstack-nova | 20:08 | |
stephenfin | mriedem, gibi: You can also use '-c' to show only the columns you care out and filter out the noise | 20:08 |
*** tetsuro has joined #openstack-nova | 20:09 | |
*** slaweq has joined #openstack-nova | 20:09 | |
*** jangutter_ has joined #openstack-nova | 20:09 | |
*** jangutter has quit IRC | 20:09 | |
*** tssurya has joined #openstack-nova | 20:09 | |
*** efried has joined #openstack-nova | 20:09 | |
*** med_ has joined #openstack-nova | 20:11 | |
*** erlon has joined #openstack-nova | 20:11 | |
gibi | stephenfin, mriedem: thanks, it was a projector - laptop incompatibilty issue today about screen size. We will do better in Berlin :) | 20:13 |
mriedem | stephenfin: oh i know all about -c | 20:14 |
mriedem | and -f | 20:14 |
*** nicolasbock has joined #openstack-nova | 20:15 | |
*** Sukhdev has joined #openstack-nova | 20:16 | |
melwitt | our cycle highlights say live migration between neutron backends works https://github.com/openstack/releases/blob/master/deliverables/rocky/nova.yaml#L44-L46 | 20:20 |
*** sorrison has quit IRC | 20:21 | |
melwitt | given that, I think filling in the gaps would be good ^ | 20:21 |
*** vladikr has quit IRC | 20:22 | |
*** vladikr has joined #openstack-nova | 20:23 | |
*** sorrison has joined #openstack-nova | 20:25 | |
*** alexchadin has quit IRC | 20:28 | |
*** priteau has quit IRC | 20:30 | |
*** itlinux is now known as itlinux-away | 20:33 | |
*** priteau has joined #openstack-nova | 20:35 | |
*** itlinux-away is now known as itlinux | 20:35 | |
*** pcaruana has quit IRC | 20:38 | |
gibi | bauzas: if you want to try nested scheduling then this is the commit you need https://review.openstack.org/#/c/585672/ (our today's demo was based on it too) | 20:38 |
bauzas | gibi: awesome, ta | 20:38 |
*** sorrison has quit IRC | 20:39 | |
bauzas | gibi: I first need to put my hands on some hardware | 20:39 |
openstackgerrit | Merged openstack/python-novaclient master: Follow the new PTI for document build https://review.openstack.org/602153 | 20:39 |
gibi | bauzas: please be aware that it is on an old base. I'm working on to rebase the whoel series | 20:39 |
*** sorrison has joined #openstack-nova | 20:39 | |
bauzas | but that's something I'd like to do sooner than later clearly | 20:39 |
bauzas | gibi: no worries, you said it's a substantial amount of work | 20:39 |
bauzas | gibi: I guess it's worth it for me to help you if I can | 20:39 |
bauzas | gibi: the only problem is that I was a bit on and off last cycle, so I need to ramp up a little bit | 20:40 |
gibi | bauzas: as soon as I catch up with the rebase you can help a lot by reviewing the series and finding gaps we missed | 20:42 |
gibi | bauzas: until that if you have any questions about the current solution then I'm happy to help | 20:43 |
bauzas | gibi: sure, reviewing is one thing I can surely do | 20:45 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix docs and add functional test for AggregateMultiTenancyIsolation https://review.openstack.org/601835 | 20:47 |
*** hamzy has quit IRC | 20:48 | |
*** rcernin has quit IRC | 20:51 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add post-test hook for testing evacuate https://review.openstack.org/602174 | 20:53 |
gibi | bauzas: this is the complication in the claim code path that I feel will cause trouble in move operations https://github.com/openstack/nova/blob/d8004bddaea167f85d5940e586cb5c9707adb278/nova/scheduler/client/report.py#L1820-L1827 | 20:53 |
gibi | bauzas: fortunately it seems we only hit this in evacuate case | 20:54 |
gibi | bauzas: as the rest of the move uses migration_uuid on the sorce host | 20:54 |
bauzas | looking | 20:55 |
bauzas | gibi: oh shit, good catch | 20:56 |
bauzas | gibi: do we have any way to ask placement whether we have allocations in a subtree ? | 20:57 |
bauzas | something like ?root_rp=<> | 20:57 |
* bauzas is seriously lagging on placement features | 20:57 | |
gibi | bauzas: nothing I know of | 20:58 |
bauzas | http://p6.storage.canalblog.com/62/41/721385/52520059.jpg | 20:59 |
*** itlinux has quit IRC | 20:59 | |
bauzas | I miss sound | 20:59 |
* bauzas looking at the placement apiref | 21:00 | |
bauzas | oh wait | 21:00 |
bauzas | gibi: https://github.com/openstack/nova/blob/d8004bddaea167f85d5940e586cb5c9707adb278/nova/scheduler/client/report.py#L1816 with some luck, we should get the allocation even if that's in a subtree | 21:01 |
bauzas | I shall start looking at the query | 21:01 |
*** gbarros has quit IRC | 21:02 | |
bauzas | efried: btw. I said I have a patch up for resize/cold-migrate about allocations | 21:04 |
gibi | bauzas: one instance still only one allocation even if that allocation contains more than on RP including nested RPs. So from that API you will get the whole allocation | 21:04 |
bauzas | efried: see https://review.openstack.org/#/c/589085/ | 21:04 |
bauzas | gibi: say it again | 21:04 |
bauzas | ah | 21:04 |
bauzas | gibi: I'll get N allocations against N inventories, right? | 21:05 |
efried | bauzas: Ack, I'm on that guy, thx | 21:05 |
gibi | bauzas: GET /allocations/<instance_uuid> returns the allocation of the instance that contains multiple RPs including nested one | 21:05 |
gibi | bauzas: yes. one consumer (the instance_uuid) mapped to a list of allocations | 21:05 |
gibi | bauzas: but you cannot tell placement only return a subset of those allocations | 21:06 |
efried | why would you want to do that? | 21:06 |
gibi | efried: I don't :) | 21:06 |
*** itlinux has joined #openstack-nova | 21:06 | |
bauzas | gibi: I don't get why I would restrict | 21:06 |
efried | we're in violent agreement, then | 21:06 |
*** lbragstad has quit IRC | 21:06 | |
gibi | bauzas: then I missunderstood "22:57 < bauzas> gibi: do we have any way to ask placement whether we have allocations in a subtree " | 21:06 |
bauzas | gibi: if I'm getting all the allocations including the nested ones, that's perfect, nope? | 21:06 |
gibi | bauzas: I thought you only want to get the nested one. But sure you get all of them today | 21:07 |
bauzas | gibi: I meant I wanted to make sure we were getting all the allocations including the "nested" ones | 21:07 |
gibi | bauzas: then you are good with the today's API | 21:07 |
bauzas | then I think we're cool, no ? | 21:07 |
efried | oh - you're asking for some form of /resource_provider/{u}/allocations where we get a subtree? | 21:07 |
bauzas | \o/ | 21:07 |
bauzas | efried: no | 21:07 |
efried | tbc, that route gives you all allocations on the provider (from all consumers) | 21:08 |
bauzas | efried: the context is, what do we need for nested RPs to be correctly scheduled | 21:08 |
efried | mm | 21:08 |
bauzas | efried: and gibi pointed out we were looking up allocations on some cases | 21:08 |
efried | mm | 21:08 |
bauzas | so I just wanted to make sure we were getting'em, including the ones that were against inventories on children | 21:09 |
efried | mm | 21:09 |
efried | and, for that matter, sharing providers | 21:09 |
bauzas | you know, if you have teeth problem, I can help you :p | 21:09 |
bauzas | in my case, for the nested RPs | 21:09 |
bauzas | not for shared | 21:09 |
gibi | bauzas: what you will not get from placement is the mapping between granular request groups in the a_c request and the allocation that fulfills that group | 21:09 |
* bauzas google translates | 21:10 | |
efried | "not for shared" - yes, for shared too. If you had any. Which you kinda shouldn't. | 21:10 |
gibi | bauzas: this is where the spec https://review.openstack.org/#/c/597601/ comes into picture | 21:10 |
*** gbarros has joined #openstack-nova | 21:10 | |
bauzas | nah, kidding | 21:10 |
bauzas | ok | 21:10 |
bauzas | gibi: ok, I'll look at the spec later in the day | 21:11 |
bauzas | naichuans: FWIW, I'm gonna creating a vgpu-stein BP | 21:11 |
gibi | bauzas: will GPUs use granular requet groups? | 21:11 |
bauzas | gibi: it will for NUMA affinity | 21:11 |
*** savvas_ has joined #openstack-nova | 21:12 | |
gibi | bauzas: then for NUMA you might hit the problem I describe in that spec | 21:12 |
bauzas | yup | 21:12 |
bauzas | for the simple case where VGPUs are on a child but other resources are on root RP, we're all good | 21:12 |
bauzas | provided I'm getting RPs that support both requesrs | 21:13 |
efried | gibi: Skimming the spec, I don't think it goes far enough, I'm afraid. | 21:13 |
bauzas | requests, ie. VGPU in one child that's necessarly on the root RP having enough resources | 21:13 |
*** sorrison has quit IRC | 21:13 | |
gibi | efried: comments are very welcome :) | 21:13 |
* bauzas nods | 21:13 | |
bauzas | oh, awesome, there is already a vgpu-stein BP https://blueprints.launchpad.net/nova/+spec/vgpu-stein | 21:14 |
bauzas | naichuans: nevermind then my previous point | 21:14 |
*** itlinux has quit IRC | 21:15 | |
*** sorrison has joined #openstack-nova | 21:15 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: [Stable Only] Remove soft-deleted instances from quota_usages https://review.openstack.org/602459 | 21:15 |
*** itlinux has joined #openstack-nova | 21:15 | |
*** savvas has quit IRC | 21:16 | |
efried | gibi: If I wind up with more than one value in my 'contributes_to' list, I'm right back where I started not knowing which request group corresponds to which chunk of the response. | 21:18 |
*** hshiina has joined #openstack-nova | 21:19 | |
gibi | efried: if you mean group_policy=None and two request groups end up fulfilled by the same RP then you are correct that still sucks | 21:19 |
efried | gibi: So | 21:19 |
gibi | efried: I forget about that case | 21:19 |
efried | The alternative you suggest at the end, breaking the allocation request up... | 21:19 |
bauzas | oh shit, I just realize that the next PTG will litterally be 1 week after Brexit, and I'm usually flying BA | 21:19 |
bauzas | that could become fun | 21:19 |
efried | if you do that keyed by request group instead, you're golden. | 21:19 |
efried | gibi: ^ | 21:19 |
efried | Sucks to have to change the allocation request syntax, for sure, but there's no other complete solution. | 21:20 |
gibi | efried: good point. I will need to work on that spec | 21:20 |
efried | gibi: Even for a granular request you have the problem. | 21:21 |
*** priteau has quit IRC | 21:23 | |
*** slaweq has quit IRC | 21:24 | |
*** bhagyashris has quit IRC | 21:24 | |
*** bhagyashris has joined #openstack-nova | 21:24 | |
*** luksky has quit IRC | 21:25 | |
*** savvas has joined #openstack-nova | 21:25 | |
*** lbragstad has joined #openstack-nova | 21:26 | |
*** mvenesio has quit IRC | 21:26 | |
efried | gibi: e.g. resources2=BW:100&resources3=BW:200&group_policy=none | 21:26 |
efried | today you might get back BW:300 from a single provider, which is :( | 21:26 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: [Stable Only] Remove soft-deleted instances from quota_usages https://review.openstack.org/570653 | 21:26 |
*** s10 has joined #openstack-nova | 21:27 | |
*** rcernin has joined #openstack-nova | 21:29 | |
*** savvas_ has quit IRC | 21:29 | |
gibi | efried: true | 21:30 |
*** priteau has joined #openstack-nova | 21:30 | |
bauzas | gibi: efried: that's the exact problem I hit for VGPU affinity, right? | 21:31 |
*** hshiina has quit IRC | 21:31 | |
bauzas | if I want to have this NUMA node and that VGPU to be on the same tree | 21:31 |
efried | bauzas: It's less of a problem for VGPU, which is single unit | 21:31 |
efried | bauzas: Oh, no, that's a separate problem. | 21:31 |
bauzas | sorry, I learnt German and English at college, not Placement :p | 21:32 |
*** slaweq has joined #openstack-nova | 21:32 | |
*** gbarros has quit IRC | 21:32 | |
*** itlinux has quit IRC | 21:36 | |
*** itlinux has joined #openstack-nova | 21:37 | |
*** med_ has quit IRC | 21:42 | |
*** awaugama has quit IRC | 21:42 | |
*** itlinux has quit IRC | 21:50 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt https://review.openstack.org/602474 | 21:53 |
bauzas | oh, we're not advertising on nova-specs changes ? | 21:53 |
bauzas | I mean, I don't openstackgerrit telling my spec | 21:54 |
openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt https://review.openstack.org/602474 | 21:55 |
bauzas | mmm, strange but ok | 21:56 |
openstackgerrit | sean mooney proposed openstack/os-vif master: add support for generic tap device plug https://review.openstack.org/602384 | 21:58 |
bauzas | mriedem: given what we said yesterday, shall we put a procedural -2 on https://review.openstack.org/#/c/599208/ and say we can't do it until https://review.openstack.org/#/q/topic:use-nested-allocation-candidates+(status:open+OR+status:merged) is merged ? | 21:59 |
mriedem | because once the vgpu is on the child provider, vgpu scheduling would be broken? | 22:01 |
*** itlinux has joined #openstack-nova | 22:01 | |
*** itlinux has quit IRC | 22:02 | |
bauzas | mriedem: yup | 22:03 |
bauzas | no valid hosts | 22:04 |
bauzas | since it wouldn't find any RP that fulfills the VGPU resource class | 22:04 |
mriedem | bauzas: i guess? alternatively, we should have a functional test that does the reshape and then tries to schedule to the vgpu in the child - which we said we wanted as part of the checklist for placement extraction | 22:04 |
bauzas | mriedem: yeah, I discussed that with gibi | 22:04 |
bauzas | mriedem: the functional test should see that | 22:04 |
mriedem | we do have functional tests for reshaper with the fake virt driver, | 22:04 |
mriedem | we just don't try to schedule to that host after the reshape | 22:05 |
*** dpawlik has quit IRC | 22:05 | |
openstackgerrit | Merged openstack/nova stable/rocky: Add functional test for affinity with multiple cells https://review.openstack.org/599731 | 22:05 |
bauzas | mriedem: cool, I know what to do for next week | 22:05 |
mriedem | the functional test for reshaper mocks out the reshape code in the test itself; i suggested to efried that we move that to a fake virt driver - and if we did that, then i think we can just bolt the post-reshape scheduler request onto the same test | 22:05 |
mriedem | maybe we don't need to move that into a fake virt driver, idk | 22:06 |
*** dpawlik has joined #openstack-nova | 22:06 | |
mriedem | but would be nice if we can just add a few lines to that existing test to see if this passes or fails | 22:06 |
bauzas | mriedem: lemme test that | 22:07 |
mriedem | bauzas: https://github.com/openstack/nova/blob/0ba34a818414823eda5e693dc2127a534410b5df/nova/tests/functional/test_servers.py#L1835 | 22:07 |
mriedem | yeah so at the end, just try to create an instance on that host with vgpu and see if it blows up | 22:08 |
mriedem | if it fails, we can assert it should fail until https://review.openstack.org/#/q/topic:use-nested-allocation-candidates+(status:open+OR+status:merged) fixes it | 22:08 |
mriedem | and rebase those on top | 22:08 |
mriedem | LIKE A CHAMP | 22:08 |
efried | mriedem: What ought to happen is: the fake libvirt driver should (by default? based on fixture setup?) call the real libvirt driver's impl of update_provider_tree, which should be modified to do reshaping for vgpus in this cycle (and maybe other things) so we can actually test those things in functional tests. | 22:08 |
bauzas | mriedem: https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L1835 ? | 22:09 |
bauzas | oh shit, sorry | 22:09 |
mriedem | bauzas: yes just linked that above | 22:09 |
mriedem | gawd! | 22:09 |
efried | mriedem: But we should (still) absolutely not move the mocked-up update_provider_tree-of-bizarreness from that mock to the fake libvirt driver. It just doesn't make sense. | 22:09 |
bauzas | I should employ someone to read my IRC | 22:09 |
mriedem | efried: i will cut your damn letter jacket | 22:09 |
mriedem | efried: honestly i don't care how it's done | 22:09 |
efried | Just try it. Actually, the last guy who said that was also named Matt. | 22:09 |
mriedem | just that it's tested | 22:09 |
openstackgerrit | sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.openstack.org/602432 | 22:09 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Refix disk size during live migration with disk over-commit https://review.openstack.org/602477 | 22:15 |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Refix disk size during live migration with disk over-commit https://review.openstack.org/602478 | 22:15 |
jroll | melwitt: https://docs.openstack.org/ironic/latest/install/configure-nova-flavors.html | 22:21 |
melwitt | jroll: yeah, I'm pretty sure that override will _not_ stop quota counting from considering and summing the values in the flavor | 22:23 |
*** sean-k-mooney has joined #openstack-nova | 22:24 | |
*** rtjure has quit IRC | 22:25 | |
sean-k-mooney | mriedem: o/ fixed now on irc | 22:25 |
melwitt | jroll: we count the 'vcpus' and 'memory_mb' columns in the 'instances' table to count as quota usage. and I don't think the extra spec overrides will affect how those columns are populated | 22:26 |
melwitt | and those values come from the flavor | 22:26 |
mriedem | sean-k-mooney: https://etherpad.openstack.org/p/nova-network-removal-rocky | 22:30 |
mriedem | there are still some apis to investigate there | 22:31 |
sean-k-mooney | mriedem: thanks i have bookmarked it ill take a look | 22:31 |
*** savvas has quit IRC | 22:32 | |
*** savvas has joined #openstack-nova | 22:33 | |
*** lyan has quit IRC | 22:34 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: remove virt driver requires_allocation_refresh https://review.openstack.org/602481 | 22:34 |
*** savvas_ has joined #openstack-nova | 22:35 | |
*** savvas has quit IRC | 22:39 | |
jroll | melwitt: awesome, thanks :) | 22:39 |
*** jaosorior has quit IRC | 22:43 | |
*** alexchadin has joined #openstack-nova | 22:43 | |
*** priteau has quit IRC | 22:47 | |
*** eharney has quit IRC | 22:50 | |
*** holser|ptg has quit IRC | 22:50 | |
*** erlon has quit IRC | 22:51 | |
*** tssurya has quit IRC | 22:53 | |
*** macza has quit IRC | 23:08 | |
*** sorrison has quit IRC | 23:10 | |
*** takashin has left #openstack-nova | 23:11 | |
*** bhagyashris has quit IRC | 23:11 | |
*** jangutter_ has quit IRC | 23:12 | |
*** mriedem has quit IRC | 23:12 | |
*** alexchadin has quit IRC | 23:14 | |
openstackgerrit | Chris Dent proposed openstack/nova-specs master: List resource providers having inventory https://review.openstack.org/600016 | 23:17 |
*** lbragstad has quit IRC | 23:17 | |
*** s10 has quit IRC | 23:18 | |
*** s10 has joined #openstack-nova | 23:19 | |
*** cfriesen has quit IRC | 23:19 | |
*** s10 has quit IRC | 23:19 | |
*** s10 has joined #openstack-nova | 23:19 | |
*** s10 has quit IRC | 23:20 | |
*** s10 has joined #openstack-nova | 23:20 | |
*** lbragstad has joined #openstack-nova | 23:21 | |
*** s10 has quit IRC | 23:21 | |
*** s10 has joined #openstack-nova | 23:21 | |
*** sean-k-mooney has quit IRC | 23:21 | |
*** s10 has quit IRC | 23:22 | |
*** sorrison has joined #openstack-nova | 23:22 | |
*** s10 has joined #openstack-nova | 23:22 | |
*** s10 has quit IRC | 23:22 | |
*** slaweq has quit IRC | 23:24 | |
*** Sukhdev has quit IRC | 23:27 | |
*** erlon has joined #openstack-nova | 23:29 | |
*** efried has quit IRC | 23:30 | |
openstackgerrit | Merged openstack/nova stable/rocky: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/599732 | 23:30 |
*** nicolasbock has quit IRC | 23:34 | |
openstackgerrit | Merged openstack/nova stable/rocky: Fix DB archiver AttributeError due to wrong table name attribute used https://review.openstack.org/599881 | 23:35 |
openstackgerrit | Sam Morrison proposed openstack/nova master: Allow ability for non admin users to use all filters on server list. https://review.openstack.org/526558 | 23:37 |
*** tetsuro has quit IRC | 23:39 | |
*** cfriesen has joined #openstack-nova | 23:41 | |
*** lbragstad has quit IRC | 23:44 | |
*** priteau has joined #openstack-nova | 23:44 | |
*** cdent has quit IRC | 23:47 | |
*** mchlumsky has quit IRC | 23:47 | |
*** jangutter has joined #openstack-nova | 23:49 | |
*** jangutter has quit IRC | 23:50 | |
*** priteau has quit IRC | 23:50 | |
*** jangutter has joined #openstack-nova | 23:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!