| *** slaweq has joined #openstack-nova | 00:13 | |
| *** erlon_ has joined #openstack-nova | 00:15 | |
| *** markvoelker has joined #openstack-nova | 00:16 | |
| *** hongbin has joined #openstack-nova | 00:18 | |
| *** slaweq has quit IRC | 00:25 | |
| *** erlon_ has quit IRC | 00:48 | |
| *** markvoelker has quit IRC | 00:49 | |
| *** takashin has joined #openstack-nova | 00:55 | |
| *** ileixe has joined #openstack-nova | 00:56 | |
| *** slaweq has joined #openstack-nova | 01:11 | |
| *** tbachman_ has joined #openstack-nova | 01:14 | |
| *** tbachman has quit IRC | 01:17 | |
| *** tbachman_ is now known as tbachman | 01:17 | |
| *** slaweq has quit IRC | 01:25 | |
| *** hongbin has quit IRC | 01:39 | |
| *** hongbin has joined #openstack-nova | 01:41 | |
| *** hongbin_ has joined #openstack-nova | 01:45 | |
| *** hongbin has quit IRC | 01:45 | |
| *** markvoelker has joined #openstack-nova | 01:46 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Don't call begin_detaching when detaching volume from sheveld vm https://review.openstack.org/624959 | 02:01 |
|---|---|---|
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add support matrix for Delete (Abort) on-going live migration https://review.openstack.org/625781 | 02:01 |
| *** Dinesh_Bhor has joined #openstack-nova | 02:06 | |
| *** Dinesh_Bhor has quit IRC | 02:06 | |
| *** sapd1_x has joined #openstack-nova | 02:11 | |
| *** slaweq has joined #openstack-nova | 02:16 | |
| *** markvoelker has quit IRC | 02:19 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:22 | |
| *** slaweq has quit IRC | 02:24 | |
| *** mschuppert has quit IRC | 02:27 | |
| *** tiendc has joined #openstack-nova | 02:36 | |
| *** fanzhang has quit IRC | 03:04 | |
| *** psachin has joined #openstack-nova | 03:04 | |
| *** slaweq has joined #openstack-nova | 03:13 | |
| *** markvoelker has joined #openstack-nova | 03:16 | |
| *** slaweq has quit IRC | 03:24 | |
| *** tbachman_ has joined #openstack-nova | 03:27 | |
| *** tbachman has quit IRC | 03:29 | |
| *** tbachman_ is now known as tbachman | 03:29 | |
| *** ircuser-1 has joined #openstack-nova | 03:31 | |
| *** sapd1_x has quit IRC | 03:38 | |
| *** Dinesh_Bhor has quit IRC | 03:39 | |
| *** Dinesh_Bhor has joined #openstack-nova | 03:43 | |
| *** wolverineav has joined #openstack-nova | 03:47 | |
| *** markvoelker has quit IRC | 03:48 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163 | 03:51 |
| *** wolverineav has quit IRC | 03:51 | |
| *** sapd1_x has joined #openstack-nova | 03:56 | |
| *** udesale has joined #openstack-nova | 04:00 | |
| *** udesale has quit IRC | 04:00 | |
| *** udesale has joined #openstack-nova | 04:01 | |
| *** slaweq has joined #openstack-nova | 04:16 | |
| *** slaweq has quit IRC | 04:24 | |
| *** ileixe has quit IRC | 04:32 | |
| *** lpetrut has joined #openstack-nova | 04:42 | |
| *** Dinesh_Bhor has quit IRC | 04:46 | |
| *** tbachman has quit IRC | 04:52 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:53 | |
| *** ileixe has joined #openstack-nova | 05:02 | |
| *** lpetrut has quit IRC | 05:05 | |
| *** slaweq has joined #openstack-nova | 05:15 | |
| *** Luzi has joined #openstack-nova | 05:19 | |
| *** ratailor has joined #openstack-nova | 05:23 | |
| *** slaweq has quit IRC | 05:24 | |
| *** ratailor_ has joined #openstack-nova | 05:34 | |
| *** ratailor has quit IRC | 05:37 | |
| *** ratailor__ has joined #openstack-nova | 05:37 | |
| *** ratailor_ has quit IRC | 05:40 | |
| *** spsurya has joined #openstack-nova | 05:40 | |
| *** tetsuro has joined #openstack-nova | 05:45 | |
| *** sridharg has joined #openstack-nova | 06:02 | |
| *** tbachman has joined #openstack-nova | 06:08 | |
| *** lpetrut has joined #openstack-nova | 06:10 | |
| *** bhagyashris has joined #openstack-nova | 06:15 | |
| *** Dinesh_Bhor has quit IRC | 06:16 | |
| *** markvoelker has joined #openstack-nova | 06:16 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:17 | |
| *** hongbin has joined #openstack-nova | 06:25 | |
| *** hongbin has quit IRC | 06:25 | |
| *** hongbin_ has quit IRC | 06:27 | |
| *** egonzalez has quit IRC | 06:42 | |
| *** zioproto has quit IRC | 06:42 | |
| *** mrhillsman has quit IRC | 06:42 | |
| *** johnsom has quit IRC | 06:42 | |
| *** seyeongkim has quit IRC | 06:42 | |
| *** rabel has quit IRC | 06:42 | |
| *** icey has quit IRC | 06:42 | |
| *** ebbex has quit IRC | 06:42 | |
| *** rabel has joined #openstack-nova | 06:42 | |
| *** seyeongkim has joined #openstack-nova | 06:42 | |
| *** zioproto has joined #openstack-nova | 06:42 | |
| *** johnsom has joined #openstack-nova | 06:42 | |
| *** mrhillsman has joined #openstack-nova | 06:42 | |
| *** icey has joined #openstack-nova | 06:43 | |
| *** egonzalez has joined #openstack-nova | 06:43 | |
| *** ebbex has joined #openstack-nova | 06:47 | |
| *** markvoelker has quit IRC | 06:48 | |
| *** rcernin has quit IRC | 07:00 | |
| openstackgerrit | Sagar Waghmare proposed openstack/nova master: Added new repo https://review.openstack.org/632031 | 07:03 |
| *** pooja_jadhav has joined #openstack-nova | 07:04 | |
| *** wolverineav has joined #openstack-nova | 07:11 | |
| *** wolverineav has quit IRC | 07:12 | |
| *** dpawlik has joined #openstack-nova | 07:13 | |
| *** wolverineav has joined #openstack-nova | 07:14 | |
| *** ccamacho has joined #openstack-nova | 07:28 | |
| *** avolkov has joined #openstack-nova | 07:41 | |
| *** tetsuro has quit IRC | 07:43 | |
| *** slaweq has joined #openstack-nova | 07:43 | |
| *** tetsuro has joined #openstack-nova | 07:43 | |
| *** markvoelker has joined #openstack-nova | 07:46 | |
| *** tetsuro has quit IRC | 07:47 | |
| *** sapd1_x has quit IRC | 07:47 | |
| *** sapd__x has joined #openstack-nova | 07:47 | |
| *** YaWang has joined #openstack-nova | 07:50 | |
| *** jangutter has joined #openstack-nova | 07:51 | |
| *** rpittau has joined #openstack-nova | 07:54 | |
| *** tetsuro has joined #openstack-nova | 07:55 | |
| *** lpetrut has quit IRC | 07:58 | |
| *** tetsuro has quit IRC | 07:58 | |
| *** tetsuro has joined #openstack-nova | 07:58 | |
| *** takashin has left #openstack-nova | 08:00 | |
| *** ratailor_ has joined #openstack-nova | 08:08 | |
| *** ratailor__ has quit IRC | 08:11 | |
| *** tkajinam has quit IRC | 08:16 | |
| *** wolverineav has quit IRC | 08:18 | |
| *** markvoelker has quit IRC | 08:19 | |
| *** yan0s has joined #openstack-nova | 08:23 | |
| *** helenafm has joined #openstack-nova | 08:24 | |
| *** wolverineav has joined #openstack-nova | 08:26 | |
| *** ralonsoh has joined #openstack-nova | 08:26 | |
| *** wolverineav has quit IRC | 08:28 | |
| *** wolverineav has joined #openstack-nova | 08:29 | |
| *** sapd__x has quit IRC | 08:30 | |
| *** sapd__x has joined #openstack-nova | 08:30 | |
| *** wolverineav has quit IRC | 08:33 | |
| *** xek_ has joined #openstack-nova | 08:42 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/os-resource-classes master: Add PCPU standard resource class https://review.openstack.org/631711 | 08:46 |
| openstackgerrit | Merged openstack/gantt master: Retire gantt https://review.openstack.org/630138 | 08:53 |
| openstackgerrit | Merged openstack/python-ganttclient master: Retire python-ganttclient https://review.openstack.org/630154 | 08:53 |
| *** wolverineav has joined #openstack-nova | 09:05 | |
| *** tetsuro has quit IRC | 09:05 | |
| *** tetsuro has joined #openstack-nova | 09:08 | |
| *** tetsuro has quit IRC | 09:11 | |
| *** tetsuro has joined #openstack-nova | 09:11 | |
| *** tetsuro has quit IRC | 09:14 | |
| *** markvoelker has joined #openstack-nova | 09:16 | |
| *** sapd__x has quit IRC | 09:18 | |
| *** Dinesh_Bhor has quit IRC | 09:21 | |
| *** Dinesh_Bhor has joined #openstack-nova | 09:24 | |
| *** wolverineav has quit IRC | 09:28 | |
| *** panda|off is now known as panda | 09:29 | |
| *** sapd__x has joined #openstack-nova | 09:30 | |
| *** derekh has joined #openstack-nova | 09:38 | |
| *** yan0s has quit IRC | 09:48 | |
| *** markvoelker has quit IRC | 09:49 | |
| *** ratailor_ has quit IRC | 09:52 | |
| *** jaosorior has joined #openstack-nova | 09:53 | |
| *** dtantsur|afk is now known as dtantsur | 09:57 | |
| *** bhagyashris has quit IRC | 09:58 | |
| *** cdent has joined #openstack-nova | 09:59 | |
| *** yan0s has joined #openstack-nova | 10:01 | |
| *** ralonsoh has quit IRC | 10:10 | |
| *** ralonsoh has joined #openstack-nova | 10:11 | |
| *** ratailor has joined #openstack-nova | 10:12 | |
| *** openstackgerrit has quit IRC | 10:21 | |
| *** ratailor has quit IRC | 10:38 | |
| *** Dinesh_Bhor has quit IRC | 10:39 | |
| *** wolverineav has joined #openstack-nova | 10:41 | |
| *** cdent has quit IRC | 10:45 | |
| *** markvoelker has joined #openstack-nova | 10:46 | |
| *** ratailor has joined #openstack-nova | 10:48 | |
| *** tetsuro has joined #openstack-nova | 10:50 | |
| *** ratailor_ has joined #openstack-nova | 10:55 | |
| *** ratailor has quit IRC | 10:57 | |
| *** tssurya has joined #openstack-nova | 10:59 | |
| *** ratailor_ has quit IRC | 11:00 | |
| *** ratailor has joined #openstack-nova | 11:00 | |
| *** ratailor has quit IRC | 11:06 | |
| *** pooja_jadhav has quit IRC | 11:09 | |
| *** poojajadhav has joined #openstack-nova | 11:09 | |
| *** cdent has joined #openstack-nova | 11:10 | |
| *** moshele has joined #openstack-nova | 11:10 | |
| *** ratailor has joined #openstack-nova | 11:13 | |
| *** tiendc has quit IRC | 11:19 | |
| *** markvoelker has quit IRC | 11:19 | |
| *** erlon has joined #openstack-nova | 11:19 | |
| *** udesale has quit IRC | 11:20 | |
| *** shilpasd has joined #openstack-nova | 11:26 | |
| *** poojajadhav has quit IRC | 11:33 | |
| *** poojajadhav has joined #openstack-nova | 11:34 | |
| *** ratailor has quit IRC | 11:36 | |
| *** ratailor has joined #openstack-nova | 11:36 | |
| *** lpetrut has joined #openstack-nova | 11:47 | |
| *** poojajadhav has quit IRC | 11:49 | |
| *** sapd__x has quit IRC | 11:53 | |
| *** yan0s has quit IRC | 11:54 | |
| *** openstackgerrit has joined #openstack-nova | 12:01 | |
| openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: do not set mac when unplugging macvtap vf https://review.openstack.org/624842 | 12:01 |
| openstackgerrit | sean mooney proposed openstack/nova master: Add free for claimed, allocated devices https://review.openstack.org/616120 | 12:01 |
| openstackgerrit | sean mooney proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365 | 12:01 |
| openstackgerrit | sean mooney proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929 | 12:01 |
| openstackgerrit | sean mooney proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115 | 12:01 |
| openstackgerrit | sean mooney proposed openstack/nova master: [WIP] libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | 12:01 |
| *** moshele has quit IRC | 12:04 | |
| openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: do not set mac when unplugging macvtap vf https://review.openstack.org/624842 | 12:05 |
| openstackgerrit | sean mooney proposed openstack/nova master: Add free for claimed, allocated devices https://review.openstack.org/616120 | 12:05 |
| openstackgerrit | sean mooney proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365 | 12:05 |
| openstackgerrit | sean mooney proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929 | 12:05 |
| openstackgerrit | sean mooney proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115 | 12:05 |
| openstackgerrit | sean mooney proposed openstack/nova master: [WIP] libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | 12:05 |
| *** yan0s has joined #openstack-nova | 12:07 | |
| *** cdent has quit IRC | 12:08 | |
| *** moshele has joined #openstack-nova | 12:14 | |
| *** panda is now known as panda|lunch | 12:14 | |
| *** zul has joined #openstack-nova | 12:16 | |
| *** markvoelker has joined #openstack-nova | 12:16 | |
| *** tetsuro has quit IRC | 12:19 | |
| *** udesale has joined #openstack-nova | 12:26 | |
| *** ratailor has quit IRC | 12:38 | |
| *** jistr is now known as jistr|afk | 12:38 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add test to check os_vif.internal.command.ip.exists https://review.openstack.org/632077 | 12:38 |
| *** ratailor has joined #openstack-nova | 12:47 | |
| *** markvoelker has quit IRC | 12:49 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Cleanup device at the end of 'test_iproute_object_closes_correctly' test https://review.openstack.org/629112 | 12:52 |
| *** shilpasd has quit IRC | 12:56 | |
| ralonsoh | jaypipes, hi. I added the cleanup to avoid leaving any device created in the system if the test goes wrong | 12:57 |
| jaypipes | ralonsoh: morning! :) | 13:02 |
| jaypipes | ralonsoh: was I correct though, that line 183/4 is doing that delete? | 13:03 |
| ralonsoh | jaypipes, yes, the device is being deleted in this line, but this cleanup is just for precaution | 13:03 |
| jaypipes | ralonsoh: the delete of the link itself should be done with an addCleanup() call, but AFAICT, the self._del_device() is ensuring that the other device(s) are deleted appropriately, right? | 13:04 |
| *** eharney has joined #openstack-nova | 13:04 | |
| jaypipes | ralonsoh: sure, since the _delete_device() method is idempotent, it won't hurt. | 13:04 |
| ralonsoh | jaypipes, yes, that's the point | 13:05 |
| jaypipes | ralonsoh: ack, ok, I'll push it through. | 13:09 |
| jaypipes | ralonsoh: done | 13:10 |
| ralonsoh | jaypipes, thanks! | 13:10 |
| *** jistr|afk is now known as jistr | 13:10 | |
| *** ShilpaSD has joined #openstack-nova | 13:11 | |
| sean-k-mooney | so looking at https://review.openstack.org/#/c/629112/4..5/os_vif/tests/functional/internal/command/ip/test_impl_pyroute2.py we are just being extra safe by registering both cleanup function | 13:15 |
| sean-k-mooney | ralonsoh: jaypipes ^ | 13:15 |
| openstackgerrit | Natal Ngétal proposed openstack/nova master: [Configuration] Clean up .gitignore references to personal tools https://review.openstack.org/632086 | 13:16 |
| ralonsoh | sean-k-mooney, yes, to ensure, in case of any problem, we delete both devices | 13:16 |
| sean-k-mooney | ralonsoh: jaypipes that said wont hte device created on line 193 actully be called test_dev_9.100 as it is a vlan100 subport | 13:16 |
| sean-k-mooney | actully i guess that is just a convention | 13:17 |
| sean-k-mooney | it will actully be called test_dev_9 | 13:17 |
| sean-k-mooney | ok | 13:17 |
| ralonsoh | sean-k-mooney, well yes, and the same in test_add_vlan | 13:17 |
| sean-k-mooney | ya if we were using ifconfig the dev.vlan name would be required i think but since we are using ip link to create them we are free to call them whatever we like | 13:18 |
| *** ratailor has quit IRC | 13:19 | |
| sean-k-mooney | ill need to resping the change to the release repo so ill pick up the 4 patch in that chain when i do for the 1.14.0 release | 13:21 |
| sean-k-mooney | stephenfin: are you to quckly review some os-vif changes. im +2 on all of them but could you take a look at them https://review.openstack.org/#/c/631829/3 | 13:24 |
| sean-k-mooney | *are you free | 13:24 |
| *** cdent has joined #openstack-nova | 13:33 | |
| *** panda|lunch is now known as panda | 13:38 | |
| openstackgerrit | Merged openstack/os-resource-classes master: Add PCPU standard resource class https://review.openstack.org/631711 | 13:40 |
| *** priteau has joined #openstack-nova | 13:41 | |
| *** yan0s has quit IRC | 13:46 | |
| *** yan0s has joined #openstack-nova | 13:48 | |
| *** yan0s has quit IRC | 13:50 | |
| *** yan0s has joined #openstack-nova | 13:51 | |
| edleafe | I know we skipped one conference; can't remember which | 13:52 |
| edleafe | wrong window | 13:52 |
| *** takashin has joined #openstack-nova | 13:54 | |
| *** nehaalhat has joined #openstack-nova | 13:55 | |
| efried | n-sch meeting in 5 minutes in #openstack-meeting-alt. Expect it to be a short one. | 13:55 |
| *** sum12 has quit IRC | 13:56 | |
| *** ileixe has quit IRC | 13:56 | |
| *** sum12 has joined #openstack-nova | 13:56 | |
| *** tetsuro has joined #openstack-nova | 13:59 | |
| *** mriedem has joined #openstack-nova | 14:02 | |
| openstackgerrit | Natal Ngétal proposed openstack/nova master: [Configuration] Clean up .gitignore references to personal tools https://review.openstack.org/632086 | 14:03 |
| *** udesale has quit IRC | 14:05 | |
| *** udesale has joined #openstack-nova | 14:06 | |
| *** lbragstad has joined #openstack-nova | 14:11 | |
| *** nehaalhat has quit IRC | 14:12 | |
| *** eharney has quit IRC | 14:19 | |
| *** sapd__x has joined #openstack-nova | 14:21 | |
| *** xek_ has quit IRC | 14:25 | |
| *** xek_ has joined #openstack-nova | 14:26 | |
| *** takashin has left #openstack-nova | 14:26 | |
| *** Luzi has quit IRC | 14:26 | |
| *** tetsuro has quit IRC | 14:26 | |
| *** cdent has quit IRC | 14:30 | |
| *** mvkr has quit IRC | 14:33 | |
| sean-k-mooney | jaypipes: thanks for the +w i forgot stephenfin is on PTO today | 14:37 |
| jaypipes | sean-k-mooney: no problemo. | 14:38 |
| *** efried has quit IRC | 14:41 | |
| *** efried has joined #openstack-nova | 14:42 | |
| *** kmalloc has joined #openstack-nova | 14:44 | |
| *** YaWang has quit IRC | 14:44 | |
| *** mchlumsky has joined #openstack-nova | 14:46 | |
| gibi | mriedem: hi, I see that jaypipes already +2 the ensure consumer status check but I have a question in https://review.openstack.org/#/c/631671/7/placement/cmd/status.py@51 | 14:46 |
| mriedem | gibi: so a consumer record for allocations but no generation? | 14:49 |
| mriedem | i don't think that's possible... | 14:49 |
| mriedem | the consumer generation would start at 0 if it's new | 14:49 |
| gibi | mriedem: based on your docstring I felt that you wanted to detect this case as well with the query but I don't think that the current query detects that | 14:50 |
| mriedem | the consumers table defaults generation to 0 so as long as there is a consumers table record, i think it has a generation | 14:50 |
| gibi | mriedem: OK, then I just overreacted. thanks for the explanation | 14:50 |
| mriedem | np | 14:51 |
| mriedem | gibi: while you're here, and you've been on the per-aggregate weights patch, i had a usage/behavior question about one of the weighers which you might have an opinion https://review.openstack.org/#/c/628163/12/nova/tests/unit/scheduler/weights/test_weights_compute.py@95 | 14:52 |
| *** wolverineav has quit IRC | 14:53 | |
| mriedem | i'd ask dan but he's out this week | 14:54 |
| *** efried is now known as efried_mlk | 14:55 | |
| *** dave-mccowan has joined #openstack-nova | 14:55 | |
| mriedem | stephenfin: easy cleanup patch on which you have context https://review.openstack.org/#/c/628205/ | 14:57 |
| sean-k-mooney | mriedem: stephenfin is on PTO today he will be back tomorrow | 14:57 |
| mriedem | ah yes MLK day in ireland... | 14:58 |
| sean-k-mooney | no he just took today off | 14:58 |
| sean-k-mooney | i think he was at a gig over the weekend or something | 14:58 |
| gibi | mriedem: looking... | 15:00 |
| sean-k-mooney | actully he may been skying the cleanups make sense to me. | 15:01 |
| *** maciejjozefczyk has quit IRC | 15:03 | |
| gibi | mriedem: I tend to agree to simply document that min in case of build_failure weigher means the min of the configured weights not the calculated (and negated) weights | 15:04 |
| *** maciejjozefczyk has joined #openstack-nova | 15:05 | |
| *** mlavalle_ has joined #openstack-nova | 15:07 | |
| *** mlavalle_ has quit IRC | 15:08 | |
| *** cdent has joined #openstack-nova | 15:10 | |
| *** mlavalle has quit IRC | 15:11 | |
| *** mlavalle has joined #openstack-nova | 15:11 | |
| *** dave-mccowan has quit IRC | 15:12 | |
| mriedem | gibi: ok so you wouldn't change anything about the docs here? https://review.openstack.org/#/c/628163/13/doc/source/user/filter-scheduler.rst@508 | 15:13 |
| *** szaher has quit IRC | 15:13 | |
| mriedem | i had started commenting on that about whether or not we should mention something about that, or give an example, but when i read what i'd add it just makes it more confusing | 15:13 |
| *** dpawlik has quit IRC | 15:14 | |
| mriedem | i left a comment that i can't come up with a good comment, +2 | 15:15 |
| *** mlavalle has quit IRC | 15:16 | |
| *** mlavalle has joined #openstack-nova | 15:16 | |
| *** moshele has quit IRC | 15:17 | |
| *** szaher has joined #openstack-nova | 15:17 | |
| gibi | mriedem: I think that doc is correct from external perspective | 15:19 |
| gibi | mriedem: and I agree configuring weighers are hard, and this patch now makes it even more complex | 15:19 |
| mriedem | heh | 15:20 |
| mriedem | time for edleafe's scheduler dry run API | 15:20 |
| gibi | is it a test environment for scheduler config? :) | 15:21 |
| edleafe | gibi: the idea was to pass in a request and some target hosts, and have the scheduler return the hosts that would have passed, but not actually build anything on those hosts | 15:23 |
| edleafe | It would have been useful for the Watcher project to know if they try to move an instance to a host whether the new host was a valid target | 15:24 |
| gibi | edleafe: I see. It is interesting | 15:25 |
| *** dpawlik has joined #openstack-nova | 15:26 | |
| edleafe | gibi: now what they do is try to move it and catch the failure. It's slow and expensive | 15:26 |
| cdent | I am slow and expensive | 15:28 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: A few miscellaneous items related to "native TLS" https://review.openstack.org/630980 | 15:30 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: docs: Update references to "QEMU-native TLS" document https://review.openstack.org/631283 | 15:30 |
| *** dpawlik has quit IRC | 15:30 | |
| *** mriedem has quit IRC | 15:32 | |
| gibi | edleafe: in return, such a dryrun might return True for a host but then the actual scheduling will fail a due to races | 15:35 |
| gibi | edleafe: but I see that this is a tradeoff | 15:35 |
| edleafe | gibi: sure, that's always been understood, and things like total available resources are already able to be known. What it *would* do is help with things like affinity/anti-affinity, aggs, etc., that are pure nova-isms | 15:37 |
| gibi | edleafe: yeah, make sense | 15:38 |
| *** TxGirlGeek has joined #openstack-nova | 15:43 | |
| *** temka has quit IRC | 15:50 | |
| *** munimeha1 has joined #openstack-nova | 15:52 | |
| *** whoami-rajat has quit IRC | 15:55 | |
| *** mriedem has joined #openstack-nova | 15:57 | |
| *** macza has joined #openstack-nova | 15:59 | |
| *** udesale has quit IRC | 16:01 | |
| *** psachin has quit IRC | 16:12 | |
| gibi | mriedem: btw, as dansmith is on vacation, I | 16:12 |
| gibi | mriedem: btw, as dansmith is on vacation, I'd like to ask your oppinion on my answer in the bandwidth ML thread | 16:12 |
| gibi | mriedem: of course if your oppinion is to wait for dansmith then I will wiat | 16:12 |
| gibi | wait | 16:13 |
| sean-k-mooney | gibi: by the way i was not arguing that all features reqire move support just that numa is not a good example :) | 16:14 |
| gibi | sean-k-mooney: yeah. I could use other example like virt driver support for certain operations | 16:15 |
| sean-k-mooney | yep ironic does not really have much of a choice | 16:15 |
| *** cfriesen has joined #openstack-nova | 16:16 | |
| sean-k-mooney | gibi: speaking of ironic i assume there is noting in pricipal stopping bw based schduling from being extended to ironic if the appropriate ml2 drivers were extended | 16:17 |
| gibi | sean-k-mooney: yes and no. From nova perspective nothing prevents ironic to handle bandwidth | 16:17 |
| *** hemnaaway is now known as hemna | 16:17 | |
| sean-k-mooney | you would just need to update the ml2 drivers for the tor switch | 16:17 |
| gibi | sean-k-mooney: from neutron perspective I'm not sure bandwidht allocation can be enforced | 16:17 |
| gibi | sean-k-mooney: if you use the tor for it then it is not the same bandwidth as in case of libvirt and sriov | 16:18 |
| *** hongbin has joined #openstack-nova | 16:18 | |
| sean-k-mooney | gibi: yes and no | 16:18 |
| gibi | sean-k-mooney: in theory we can extend the current bandwidth support for the tor switch in in case of libvirt + sriov | 16:19 |
| sean-k-mooney | gibi: well for the libvirt case to include the tor means hierarcical port binding | 16:19 |
| *** TxGirlGeek has quit IRC | 16:19 | |
| sean-k-mooney | but for ironic the tor is the l1 port binding | 16:19 |
| sean-k-mooney | anyway that is for the future | 16:20 |
| gibi | sean-k-mooney: yes :) | 16:20 |
| gibi | sean-k-mooney: my problem with dansmith mail is that I thought that microversion can be used to signal a new, supported use case, if the change affect an API behavior. But from the mail it seems that is only true for complete feature | 16:22 |
| cfriesen | sean-k-mooney: PCI/SRIOV question for you. with the upstream code, how do you ensure that an instance that requests a PCI NIC ends up on a compute node that has physical NICs on the desired physical network? Is that just host aggregates? As I recall, the network-aware scheduling stuff never landed. | 16:22 |
| sean-k-mooney | the pci filter performs that check | 16:22 |
| sean-k-mooney | in the whitelist we declare the physnet that the pf is connected too | 16:23 |
| mriedem | gibi: for supporting numa live migration, i don't think a microversion was going to be added - today it just fails (might be a 400, i'd have to look), | 16:23 |
| mriedem | i see both sides and i think the argument for not adding a microversion for move support is to avoid the ux mess of being able to create a server with one microversion but then you have to use another to do something with it | 16:24 |
| sean-k-mooney | cfriesen: the network aware schdulling stuff was for offloads. idealy we would do this with placement rather then the pcipassthough filter | 16:24 |
| mriedem | i guess that argument could be made for something like volume-backed rebuild as well | 16:25 |
| mriedem | you can create a volume-backed server with 2.1 but can't rebuild and re-image the root volume without a new microversion | 16:25 |
| sean-k-mooney | cfriesen: whenever we get around to modeling PFs/VFs in placemetn we could use either placement aggrates or traits to model physnets | 16:25 |
| *** sapd__x has quit IRC | 16:25 | |
| mriedem | gibi: anyway, if we don't have move support in stein anyway, then the microversion for move is a train discussion and we can defer until then can't we? | 16:25 |
| gibi | mriedem: so in this case discoverability is less important here than ux microversion usage | 16:25 |
| mriedem | unless the argument is being made that we shouldn't ship the create support in stein unless move support is also ready...which i don't think we should do (and that argument could be made about a lot of features we add, like vgpus and trusted certs) | 16:26 |
| mriedem | i.e. resize with vgpus doesn't work completely, you have to rebuild the server after the resize to get the vgpus | 16:27 |
| mriedem | btw, now that i say that, i wonder if that would work for bw resources as a workaround for cold migration ^ | 16:27 |
| gibi | mriedem: OK then I will propose the first microversion top of the create / delete support and deferr the move support microversion question for Train | 16:28 |
| mriedem | gibi: or at least defer for a week until dan is back :) | 16:28 |
| gibi | mriedem: yeah :) | 16:28 |
| gibi | mriedem: does rebuild re-allocate today? | 16:28 |
| sean-k-mooney | gibi: mriedem i was under the impression rebuild would not work either in stien | 16:29 |
| mriedem | gibi: no, since resources shouldn't really change on rebuild since the flavor doesn't change | 16:30 |
| mriedem | so i guess that won't work | 16:30 |
| gibi | mriedem: so it would be an extra check during rebuild, to see if healing is needed | 16:30 |
| sean-k-mooney | its not strictly a move operation but i has understood that effectivly start,stop,pause/unpause,suspend and resume were the only actions that would work | 16:30 |
| mriedem | rebuild is specifically not a move | 16:30 |
| mriedem | there is no resource claim on rebiuld | 16:31 |
| mriedem | *rebuild | 16:31 |
| sean-k-mooney | well unless the image is chaged | 16:31 |
| mriedem | even though we have an outstanding bug about the pci resource usage changing b/c of changing the image | 16:31 |
| sean-k-mooney | in which case we have to go back to the scheuler | 16:31 |
| *** dpawlik has joined #openstack-nova | 16:31 | |
| mriedem | sean-k-mooney: yes but that's for filtering the image, not for claiming resources | 16:31 |
| mriedem | "does the new image work on this existing compute host" | 16:31 |
| sean-k-mooney | well if the image changes the resouces that are need they would have to be claimed/updated | 16:32 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1780441 | 16:32 |
| openstack | Launchpad bug 1780441 in OpenStack Compute (nova) "Rebuild does not respect number of PCIe devices" [Undecided,New] | 16:32 |
| mriedem | sean-k-mooney: normal resources don't change based on the image | 16:32 |
| mriedem | except pci | 16:32 |
| mriedem | hence that bug | 16:32 |
| gibi | mriedem: in theory I can trigger a resource healing check during rebuild, but I guess I should not fail the rebuild if bandwidth resource is not available, should I? | 16:32 |
| sean-k-mooney | well you can enable hugepags and cpu pinning and pinning policies and numa too | 16:32 |
| sean-k-mooney | all of which may change the resoces required | 16:33 |
| sean-k-mooney | but in general yes | 16:33 |
| mriedem | for anything nfv related, the answer is don't ever try to do anythign with the server once you created it | 16:33 |
| sean-k-mooney | gibi: mriedem for rebuild with BW aware schduling you would also need to validate if the port request changed | 16:33 |
| cfriesen | mriedem: wouldn't that fall under https://bugs.launchpad.net/nova/+bug/1763766 ? | 16:33 |
| openstack | Launchpad bug 1763766 in OpenStack Compute (nova) "nova needs to disallow resource consumption changes on image rebuild" [Medium,Triaged] | 16:33 |
| mriedem | because nova doesn't support operations on those servers | 16:33 |
| mriedem | cfriesen: oh i thought that was your bug | 16:34 |
| sean-k-mooney | mriedem: well we have never enforced that | 16:34 |
| cfriesen | yeah, it is. I haven't done anything about it yet. | 16:34 |
| cfriesen | I'm just thinking that PCI would fall under it too | 16:34 |
| sean-k-mooney | mriedem: its good advice but its not explcitly stated as far as i know | 16:34 |
| mriedem | sean-k-mooney: yes i know | 16:35 |
| sean-k-mooney | cfriesen: yes likely. | 16:35 |
| gibi | sean-k-mooney: in general we said that changing the resource request of a port is not supported (yet). I don't see how this is special for rebiuld. The port can be changed any time | 16:35 |
| spotz | Anyone able to ask a quick keypair metadata questioon? | 16:35 |
| spotz | err answer... not enough coffee | 16:35 |
| *** dpawlik has quit IRC | 16:35 | |
| sean-k-mooney | gibi: for rebuild we are recreating the vm from scratch and should validate its resocues | 16:35 |
| gibi | sean-k-mooney: but current nova code assumes that the resource allocation of such VM is unchanged hence no reallocation happen | 16:36 |
| mriedem | correct | 16:36 |
| gibi | sean-k-mooney: so it is not special for banwidth | 16:36 |
| mriedem | because all of the nfv stuff added in juno didn't take rebuild into account | 16:36 |
| *** eharney has joined #openstack-nova | 16:36 | |
| sean-k-mooney | https://bugs.launchpad.net/nova/+bug/1763766 | 16:36 |
| openstack | Launchpad bug 1763766 in OpenStack Compute (nova) "nova needs to disallow resource consumption changes on image rebuild" [Medium,Triaged] | 16:36 |
| mriedem | s/rebuild/any other operation besides delete/ | 16:37 |
| sean-k-mooney | true but the bug cfriesen pointed to would cover it^ | 16:37 |
| cfriesen | mriedem: and resize | 16:37 |
| mriedem | "any other operation besides delete" | 16:37 |
| gibi | sean-k-mooney: OK, so you suggest to fail rebuild if the port's resource request changed | 16:37 |
| sean-k-mooney | gibi: yes | 16:37 |
| gibi | sean-k-mooney: that would need nova to store such request | 16:37 |
| gibi | sean-k-mooney: to have something to compare with | 16:38 |
| mriedem | ummm | 16:38 |
| sean-k-mooney | gibi: it could compare it to the existing placement allocation | 16:38 |
| gibi | sean-k-mooney: I'm more like the idea that neutron rejects the change of a qos rule if the port is bound | 16:38 |
| mriedem | that seems pretty shitty for nova have to do that enforcement, | 16:38 |
| gibi | sean-k-mooney: with multiple port, that comparision is really ahrd | 16:38 |
| gibi | hard | 16:38 |
| sean-k-mooney | gibi: that was brought up before | 16:38 |
| mriedem | neutron should be responsible for not allowing bw policy changes on attached ports | 16:38 |
| sean-k-mooney | gibi: it would be a change in the neutron behaivor but not one that they said the were against | 16:39 |
| mriedem | gibi: yes i agree with what you said, | 16:39 |
| sean-k-mooney | mriedem: well this is special for min mandwith | 16:39 |
| mriedem | "I'm more like the idea that neutron rejects the change of a qos rule if the port is bound" | 16:39 |
| sean-k-mooney | there is no reason max bandwith of dscp cant change | 16:39 |
| gibi | sean-k-mooney: I reacall that this was stated in the spec | 16:39 |
| openstackgerrit | Merged openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677 | 16:39 |
| *** yan0s has quit IRC | 16:39 | |
| sean-k-mooney | gibi: i raised this before yes but i dont think we made a decision to actully modify neutron to do it | 16:40 |
| gibi | sean-k-mooney: I will ping rubasov tomorrow about it, he is leading the neutron work | 16:40 |
| sean-k-mooney | gibi: anyway this is why assumed rebuild would be out of scope until the migration stuff was done as i assuem someone nova or neutron would have to do this type of validaion | 16:41 |
| gibi | sean-k-mooney: https://review.openstack.org/#/c/595243/2/specs/stein/approved/bandwidth-resource-provider.rst@110 here I state in the spec that changing rule is out of scope https://review.openstack.org/#/c/595243/2/specs/stein/approved/bandwidth-resource-provider.rst@110 | 16:41 |
| *** whoami-rajat has joined #openstack-nova | 16:41 | |
| gibi | sean-k-mooney: but rebuild is not the good time to check, the port might changed without anybody calling rebuild on the server the port is bound to | 16:42 |
| sean-k-mooney | gibi: yes i am aware its out of scope but its required for rebuild and move i think | 16:42 |
| sean-k-mooney | gibi: anyway i dont think it should be a blocker for this in stien | 16:43 |
| sean-k-mooney | its just another edgecase to be address in train | 16:43 |
| gibi | sean-k-mooney: move will use the actual state of the port not because it wants to use the updated value but because neutron is considered the source of truth in case of port resource request so nova does not store such request | 16:43 |
| gibi | when somebody try to change a QoS rule on a boun port, neutron should check if there are enough resource before it accepts the rule change | 16:44 |
| sean-k-mooney | gibi: sure is it not logical that rebuild should also use the actual state of the port for the same reason | 16:44 |
| mriedem | i assume this is already a problem in the neutron api which is much more loosey goosey about making changes to bound ports, unlike cinder's strong enforcement of not making changes to attached volumes | 16:44 |
| sean-k-mooney | gibi: yep we discussed that in dublin | 16:44 |
| mriedem | e..g in neutron am i able to change port binding details about a bound port? | 16:45 |
| sean-k-mooney | mriedem: the other qos type dont consome resouces | 16:45 |
| sean-k-mooney | e.g. max bandwith and dscp marking can be changed on the fly | 16:46 |
| gibi | sean-k-mooney: in generan nova will not be able to prevent the user to change a port in a way that leads to inconsistencies so neutron should do the check | 16:46 |
| sean-k-mooney | gibi: yep im not arguing it should try | 16:46 |
| *** smcginnis has joined #openstack-nova | 16:47 | |
| cfriesen | sean-k-mooney: is this still correct as far as the formatting of the PCI whitelist or is there a newer doc somewhere? https://specs.openstack.org/openstack/nova-specs/specs/juno/implemented/pci-passthrough-sriov.html | 16:48 |
| gibi | sean-k-mooney: but re-allocation during rebuild only needed if neutron allowed the bound port to change. | 16:48 |
| sean-k-mooney | there is a new dock but let me check | 16:48 |
| gibi | sean-k-mooney: so if neutron rejects that then reubild does not need to be changed due to bandwidth | 16:49 |
| sean-k-mooney | gibi which it does hand has always supported | 16:49 |
| *** sridharg has quit IRC | 16:49 | |
| gibi | sean-k-mooney: it should not be supported to min bandwidth any more without making sure that the placement allocation is updated | 16:49 |
| gibi | sean-k-mooney: otherwise we end up in resource inconsistencies | 16:50 |
| mriedem | cfriesen: there are examples in https://docs.openstack.org/nova/latest/admin/pci-passthrough.html | 16:50 |
| sean-k-mooney | gibi: sure but doing so was not covered in the spec because it was declared out of scope | 16:50 |
| gibi | sean-k-mooney: exactly | 16:50 |
| mriedem | cfriesen: and https://docs.openstack.org/nova/latest/configuration/config.html#pci.passthrough_whitelist | 16:50 |
| sean-k-mooney | gibi: so in stien it wont be done | 16:50 |
| gibi | sean-k-mooney: so I will ping rubasov tomorrow to make it clear if they will do the reject or not | 16:50 |
| sean-k-mooney | adn we can get out of sync with neutron | 16:50 |
| sean-k-mooney | gibi: that is a neutron api change and would need a neutron spec right | 16:51 |
| gibi | sean-k-mooney: let me check what the neutron spec states about this | 16:51 |
| mriedem | going back to what i said earlier, "here is a thing you can do to create a server, but don't ever touch it after that" :) | 16:52 |
| gibi | sean-k-mooney: neutron spec states it is out of scope too https://specs.openstack.org/openstack/neutron-specs/specs/rocky/minimum-bandwidth-allocation-placement-api.html#out-of-scope | 16:52 |
| sean-k-mooney | mriedem: you mean treat your instances like cattel and not pets | 16:52 |
| gibi | sean-k-mooney: I will clarify with rubasov what does it mean | 16:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Share snapshot image membership with instance owner https://review.openstack.org/630769 | 16:53 |
| sean-k-mooney | gibi: i had assumed that it would be addressed at some point in the future | 16:53 |
| mriedem | i don't see why disallowing making min bw allocation changes on a bound port is tied up in os-vif migration | 16:53 |
| sean-k-mooney | but yes good to check | 16:53 |
| mriedem | seems like an unnecessary dependency | 16:54 |
| sean-k-mooney | its not | 16:54 |
| gibi | sean-k-mooney: yeah supporting the update is future, but rejecting it to prevent incosystencies might be in the current scope | 16:54 |
| sean-k-mooney | does the spec say that? | 16:54 |
| mriedem | i would think, "don't give users a loaded gun to f up resource tracking in their system" as not a feature | 16:54 |
| mriedem | sean-k-mooney: yes | 16:54 |
| mriedem | "This is out of scope in this spec and should be done during the work related to os-vif migration tasks [5]." | 16:54 |
| gibi | yeah I don't see the reason there too ^^ | 16:54 |
| mriedem | in fact it would a bug in neutron | 16:55 |
| sean-k-mooney | mriedem: it was proably related the use of os-vif for port negociation or something its likely out of date | 16:55 |
| *** moshele has joined #openstack-nova | 16:55 | |
| sean-k-mooney | mriedem: again changing qos policy is ment to be allowed in neutron | 16:55 |
| sean-k-mooney | mriedem: and sice min bandwith was best effort before there was noting to validate | 16:56 |
| sean-k-mooney | so the only thing that would have added such a check are these specs | 16:56 |
| mriedem | sure i get it | 16:57 |
| mriedem | in the before times it was the wild west and you get lucky | 16:57 |
| sean-k-mooney | they declared the check/supprot out of scope so i would assume we keep the status quo by default. | 16:57 |
| gibi | sean-k-mooney: I think keeping the resouce view consistent is important enough that this can be reported as a bug agains neutron as soon as nova create/delete support in nova merges | 16:57 |
| mriedem | like mlavalle said on the call on friday, why even go to the trouble of adding placement to this to guarantee minimum bandwidth if we're not going to enforce it and make sure we calculate it properly | 16:57 |
| mriedem | gibi: agree | 16:58 |
| sean-k-mooney | gibi: sure | 16:58 |
| mriedem | if we're going to punt on keeping the system in check, then we shouldn't even add the feature | 16:58 |
| *** priteau has quit IRC | 16:58 | |
| *** moshele has quit IRC | 16:58 | |
| sean-k-mooney | not agruing we should punt | 16:58 |
| mriedem | i have to move on from this | 16:59 |
| *** rpittau has quit IRC | 16:59 | |
| sean-k-mooney | jsut taht we had said we would not do it in the spec. if we and to bring it into scope then greate but its another depency for adding support to nova | 16:59 |
| sean-k-mooney | ok | 16:59 |
| *** priteau has joined #openstack-nova | 17:00 | |
| *** helenafm has quit IRC | 17:01 | |
| gibi | mriedem, sean-k-mooney: thanks for the discussion. talk to you tomorrow | 17:02 |
| *** dpawlik has joined #openstack-nova | 17:02 | |
| cfriesen | sean-k-mooney: am I understanding this right that the "physical_network" name in the pci whitelist will be correlated against the neutron provider network for the passed-in neutron port? | 17:05 |
| sean-k-mooney | cfriesen: it correaltes to the the neutron physenet | 17:05 |
| sean-k-mooney | cfriesen: so yes it corralate to the provider:physical_network in the neutron network object | 17:07 |
| *** dpawlik has quit IRC | 17:07 | |
| cfriesen | cool, thanks | 17:07 |
| *** dtantsur is now known as dtantsur|afk | 17:16 | |
| openstackgerrit | sean mooney proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | 17:25 |
| *** tssurya has quit IRC | 17:28 | |
| *** mlavalle has quit IRC | 17:38 | |
| *** mlavalle has joined #openstack-nova | 17:39 | |
| *** mlavalle has quit IRC | 17:41 | |
| *** mlavalle has joined #openstack-nova | 17:41 | |
| *** mlavalle has quit IRC | 17:42 | |
| *** mlavalle has joined #openstack-nova | 17:42 | |
| *** mlavalle has quit IRC | 17:43 | |
| *** fragatina has joined #openstack-nova | 17:43 | |
| *** mlavalle has joined #openstack-nova | 17:43 | |
| *** fragatina has quit IRC | 17:44 | |
| *** fragatina has joined #openstack-nova | 17:44 | |
| *** mlavalle has quit IRC | 17:47 | |
| *** mlavalle has joined #openstack-nova | 17:48 | |
| *** mriedem has quit IRC | 17:50 | |
| *** mlavalle has left #openstack-nova | 17:50 | |
| *** fragatina has quit IRC | 17:52 | |
| *** logan- has quit IRC | 17:52 | |
| *** fragatina has joined #openstack-nova | 17:52 | |
| *** mlavalle has joined #openstack-nova | 17:53 | |
| *** logan_ has joined #openstack-nova | 17:53 | |
| *** logan_ is now known as logan- | 17:53 | |
| *** fragatina has quit IRC | 17:55 | |
| *** fragatina has joined #openstack-nova | 17:56 | |
| *** fragatina has quit IRC | 17:58 | |
| *** fragatina has joined #openstack-nova | 17:58 | |
| *** moshele has joined #openstack-nova | 18:00 | |
| *** priteau has quit IRC | 18:01 | |
| *** derekh has quit IRC | 18:01 | |
| *** fragatina has quit IRC | 18:03 | |
| *** fragatina has joined #openstack-nova | 18:03 | |
| *** fragatina has quit IRC | 18:05 | |
| *** artom has joined #openstack-nova | 18:05 | |
| *** fragatina has joined #openstack-nova | 18:05 | |
| *** lpetrut has quit IRC | 18:06 | |
| *** openstackgerrit has quit IRC | 18:07 | |
| *** fragatina has quit IRC | 18:07 | |
| *** fragatina has joined #openstack-nova | 18:08 | |
| *** fragatina has quit IRC | 18:08 | |
| *** fragatina has joined #openstack-nova | 18:09 | |
| *** fragatina has quit IRC | 18:09 | |
| *** fragatina has joined #openstack-nova | 18:10 | |
| *** fragatina has quit IRC | 18:11 | |
| *** fragatina has joined #openstack-nova | 18:12 | |
| *** fragatina has quit IRC | 18:13 | |
| sean-k-mooney | hi so quick question. _rollback_live_migration in the compute manager does not currently appear to ever call the virt driver on the source node | 18:13 |
| *** fragatina has joined #openstack-nova | 18:14 | |
| sean-k-mooney | i assume it would be ok to add a _rollback_live_migration_at_source to the base driver interface that was a noop and implement it for the libvirt driver? | 18:14 |
| sean-k-mooney | we have have rollback_live_migration_at_destination but that dose not help me. | 18:15 |
| *** fragatina has quit IRC | 18:15 | |
| *** fragatina has joined #openstack-nova | 18:16 | |
| *** _fragatina_ has joined #openstack-nova | 18:17 | |
| *** fragatina has quit IRC | 18:22 | |
| *** ralonsoh has quit IRC | 18:26 | |
| *** tbachman has quit IRC | 18:27 | |
| *** lbragstad is now known as lbragstad_afk | 18:27 | |
| *** lbragstad_afk is now known as lbragstad_503 | 18:27 | |
| *** ccamacho has quit IRC | 18:33 | |
| cdent | jaypipes: I believe this is for you: http://existentialcomics.com/comic/273 | 18:35 |
| *** _fragatina_ has quit IRC | 18:40 | |
| *** fragatina has joined #openstack-nova | 18:41 | |
| sean-k-mooney | cdent: if only that was the calibar of converstations at a typical bar discussing such matters hehe | 18:43 |
| cdent | i might actually go to bars if that were the case | 18:44 |
| edleafe | sean-k-mooney: I prefer this: http://theincidentaleconomist.com/wordpress/wp-content/uploads/2014/11/sportsball.jpg | 18:45 |
| *** fragatina has quit IRC | 18:47 | |
| sean-k-mooney | it is good but i can feel the heat of the browns burn in cdents link from here and i dont even follow american football | 18:47 |
| *** fragatina has joined #openstack-nova | 18:47 | |
| *** fragatina has quit IRC | 18:47 | |
| *** fragatina has joined #openstack-nova | 18:48 | |
| *** ccamacho has joined #openstack-nova | 18:50 | |
| *** erlon has quit IRC | 18:54 | |
| *** fragatina has quit IRC | 18:56 | |
| *** fragatina has joined #openstack-nova | 18:56 | |
| *** mvkr has joined #openstack-nova | 18:56 | |
| *** fragatina has quit IRC | 18:57 | |
| *** fragatina has joined #openstack-nova | 18:58 | |
| *** openstackgerrit has joined #openstack-nova | 19:00 | |
| openstackgerrit | sean mooney proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | 19:00 |
| *** fragatina has quit IRC | 19:03 | |
| *** dpawlik has joined #openstack-nova | 19:03 | |
| *** fragatina has joined #openstack-nova | 19:04 | |
| *** moshele has quit IRC | 19:04 | |
| *** fragatina has quit IRC | 19:07 | |
| *** fragatina has joined #openstack-nova | 19:07 | |
| *** dpawlik has quit IRC | 19:07 | |
| -openstackstatus- NOTICE: The error causing post failures on jobs has been corrected. It is safe to recheck these jobs. | 19:17 | |
| *** dpawlik has joined #openstack-nova | 19:19 | |
| *** dpawlik has quit IRC | 19:23 | |
| *** fragatina has quit IRC | 19:28 | |
| *** fragatina has joined #openstack-nova | 19:29 | |
| *** mriedem has joined #openstack-nova | 19:35 | |
| jaypipes | cdent: that is friggin brilliant. | 19:38 |
| *** fragatina has quit IRC | 19:39 | |
| *** fragatina has joined #openstack-nova | 19:39 | |
| *** fragatina has quit IRC | 19:42 | |
| *** fragatina has joined #openstack-nova | 19:43 | |
| cdent | jaypipes: isn't it delightful? | 19:44 |
| *** markvoelker has joined #openstack-nova | 19:46 | |
| jaypipes | cdent: it is indeed. | 19:49 |
| cdent | jaypipes: you may be the Übermensch | 19:49 |
| *** Vek has joined #openstack-nova | 19:51 | |
| jaypipes | cdent: heh | 19:58 |
| *** dpawlik has joined #openstack-nova | 19:58 | |
| *** dpawlik has quit IRC | 20:02 | |
| sean-k-mooney | https://review.openstack.org/#/c/631829/ well this is new. all post_failures and all i have check apparently passed... | 20:03 |
| *** xek_ has quit IRC | 20:09 | |
| *** whoami-rajat has quit IRC | 20:15 | |
| *** eharney has quit IRC | 20:16 | |
| *** markvoelker has quit IRC | 20:20 | |
| *** fragatina has quit IRC | 20:20 | |
| *** fragatina has joined #openstack-nova | 20:21 | |
| *** ianw is now known as ianw_pto | 20:26 | |
| *** tbachman has joined #openstack-nova | 20:42 | |
| *** macza has quit IRC | 20:43 | |
| *** macza has joined #openstack-nova | 20:45 | |
| *** priteau has joined #openstack-nova | 20:51 | |
| *** ade_lee has quit IRC | 20:58 | |
| *** avolkov has quit IRC | 21:03 | |
| *** priteau has quit IRC | 21:04 | |
| *** markvoelker has joined #openstack-nova | 21:17 | |
| *** ccamacho has quit IRC | 21:28 | |
| *** ade_lee has joined #openstack-nova | 21:38 | |
| *** cdent has quit IRC | 21:48 | |
| *** markvoelker has quit IRC | 21:49 | |
| *** takashin has joined #openstack-nova | 21:50 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add TargetDBSetupTask https://review.openstack.org/627892 | 21:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.openstack.org/631581 | 21:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask https://review.openstack.org/627890 | 21:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891 | 21:52 |
| *** rcernin has joined #openstack-nova | 21:54 | |
| *** sean-k-mooney has quit IRC | 21:56 | |
| *** sean-k-mooney has joined #openstack-nova | 21:58 | |
| *** dpawlik has joined #openstack-nova | 21:59 | |
| *** dpawlik has quit IRC | 22:03 | |
| *** fragatina has quit IRC | 22:04 | |
| *** fragatina has joined #openstack-nova | 22:04 | |
| zzzeek | mriedem: am I allowed to +2 + workflow my own sqlalhcemy-migrate patch ? | 22:05 |
| mriedem | zzzeek: which one? the mysqlclient one? | 22:06 |
| zzzeek | the "quote=force" one | 22:06 |
| mriedem | i just approved it | 22:06 |
| mriedem | i would like to avoid self approvals, but on that project i think a single +2/+W is probably OK given it's basically just you and me | 22:06 |
| zzzeek | mriedem: that's what i was asking :) | 22:07 |
| zzzeek | er where is verified coming from on that ? is there another zuul job ? | 22:07 |
| *** markvoelker has joined #openstack-nova | 22:08 | |
| mriedem | zuul will run the gate queue jobs on it | 22:08 |
| mriedem | jobs for that project are defined here https://github.com/openstack-infra/project-config/blob/fcc974bf4c0dc324f93a2aec0520e6e4889c106d/zuul.d/projects.yaml#L8106 | 22:09 |
| *** spsurya has quit IRC | 22:09 | |
| *** fragatina has quit IRC | 22:11 | |
| *** fragatina has joined #openstack-nova | 22:12 | |
| *** dpawlik has joined #openstack-nova | 22:15 | |
| *** fragatina has quit IRC | 22:18 | |
| *** fragatina has joined #openstack-nova | 22:19 | |
| *** dpawlik has quit IRC | 22:20 | |
| mriedem | zzzeek: fyi i'm proposing a release for sqla-migrate https://review.openstack.org/632180 | 22:21 |
| zzzeek | mriedem: OK so if this does include the quote=force change, note that I bumped the minimum sqlalchemy vresion to 0.9 | 22:22 |
| mriedem | yup that's why i did a minor version bump | 22:22 |
| zzzeek | mriedem: OK if that's what we need then that's great | 22:23 |
| zzzeek | sqla 0.9 is like 5 years ago | 22:23 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add description about sort order in API ref guideline https://review.openstack.org/627282 | 22:29 |
| jaypipes | mriedem: so I think I remember what that rebuild bug/issue I was asking you about last week was... | 22:35 |
| jaypipes | mriedem: if we rebuild to same host but change the image from the original image, we do a full select_destinations-code-path again, right? | 22:35 |
| *** priteau has joined #openstack-nova | 22:36 | |
| jaypipes | mriedem: but we include a single destination host when doing the select_destinations(), yeah? In order to basically determine if the original destination host is still a match for the new image? | 22:37 |
| jaypipes | mriedem: if this is the case, we're seeing a problem where we're getting NoValidHost back from the scheduler because the amount of resources "used" for the original host doesn't account for the original instance. In other words, we erroneously double-up the amount of requested resources when attempting to ask the scheduler if the original host is still a fit. | 22:38 |
| jaypipes | mriedem: or rather, that we're not correctly subtracting the amount of used original instance resources from the amount we request in the "recreate" scheduler request. | 22:39 |
| *** fragatina has quit IRC | 22:39 | |
| jaypipes | gawd, this stuff is hard to talk about in any sort of reasonably clear language, sorry :( | 22:39 |
| *** fragatina has joined #openstack-nova | 22:40 | |
| *** fragatina has quit IRC | 22:40 | |
| *** fragatina has joined #openstack-nova | 22:40 | |
| *** fragatina has quit IRC | 22:41 | |
| *** fragatina has joined #openstack-nova | 22:42 | |
| *** fragatina has quit IRC | 22:44 | |
| *** fragatina has joined #openstack-nova | 22:45 | |
| melwitt | jaypipes: correct that we go through the scheduler for rebuilds, as of queens. fwiw, it looks like it will say NoValidHost when filters fail https://bugs.launchpad.net/nova/+bug/1744325 but I don't recall issues with doubling up allocations for rebuilds before | 22:47 |
| openstack | Launchpad bug 1744325 in OpenStack Compute (nova) pike "If a rebuild is refused by the scheduler, the instance's imageref is not rolled back" [High,Fix committed] - Assigned to melanie witt (melwitt) | 22:47 |
| melwitt | (that bug shows brief explanation of rebuild going through scheduler and what happens when there's a failure to schedule) | 22:48 |
| *** imacdonn_ has quit IRC | 22:48 | |
| *** imacdonn_ has joined #openstack-nova | 22:48 | |
| mriedem | jaypipes: pretty sure that's been fixed by cfriesen and/or hongbin | 22:50 |
| mriedem | maybe you don't have the backport | 22:50 |
| jaypipes | melwitt: thx Melanie :) as you know, we're on Ocata currently for VMs and I'm seeing that the "solution" internally is "don't change the image when rebuilding"... | 22:50 |
| * mriedem runs to a wake | 22:50 | |
| *** mriedem has quit IRC | 22:51 | |
| jaypipes | Have I mentioned I hate the fact that rebuild is basically a crutch because people can't stand to lose their IP addresses because they hard-code them everywhere? :( | 22:51 |
| *** fragatina has quit IRC | 22:51 | |
| *** fragatina has joined #openstack-nova | 22:52 | |
| melwitt | heh, yeah, good ol rebuild | 22:52 |
| melwitt | so, if this is ocata, you wouldn't be going through scheduler for rebuild right? | 22:53 |
| jaypipes | melwitt: yeah, it's been backported. | 22:54 |
| melwitt | ah ok. I'm looking for a bug fix like mriedem mentioned | 22:54 |
| jaypipes | melwitt: well, at least according to https://bugs.launchpad.net/nova/+bug/1744325 (and we have that code merged in our local ocata branch) | 22:54 |
| openstack | Launchpad bug 1744325 in OpenStack Compute (nova) pike "If a rebuild is refused by the scheduler, the instance's imageref is not rolled back" [High,Fix committed] - Assigned to melanie witt (melwitt) | 22:54 |
| melwitt | jaypipes: here we go https://review.openstack.org/561015 | 22:55 |
| melwitt | good memory mriedem | 22:55 |
| jaypipes | melwitt: that's it! thank you! | 22:57 |
| melwitt | \o/ | 22:57 |
| *** tkajinam has joined #openstack-nova | 23:03 | |
| *** fragatina has quit IRC | 23:07 | |
| *** fragatina has joined #openstack-nova | 23:08 | |
| *** tbachman has quit IRC | 23:08 | |
| *** tbachman has joined #openstack-nova | 23:09 | |
| *** mlavalle has quit IRC | 23:15 | |
| *** slaweq has quit IRC | 23:35 | |
| *** hongbin has quit IRC | 23:42 | |
| *** munimeha1 has quit IRC | 23:53 | |
| *** macza has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
