| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-18 | |||
| 20:40:54 | dansmith | or just log a better error message and not reinvent everything in the bowels | |
| 20:42:20 | leakypipes | mriedem: hey, I hate launchpad bug search :( do you remember the bug report about rebuild getting a NoValidHost cuz the rebuild wasn't taking into account the original instance's resources properly? | |
| 20:44:10 | fried_rice | leakypipes: Thanks. mriedem, you got https://review.openstack.org/#/q/topic:static-rt-in-compute-mgr in your queue (somewhere :P ) ? | |
| 20:53:34 | mriedem | leakypipes: that doesn't ring a bell - we'd only get novalidhost on rebuild if the image changes | |
| 20:53:49 | mriedem | are you saying something with the scheduler filters in that case isn't getting the original request spec or something? | |
| 20:53:55 | mriedem | like project/user or image id or something? | |
| 20:54:19 | mriedem | fried_rice: oh boy... | |
| 20:54:19 | mriedem | umm | |
| 21:07:11 | leakypipes | mriedem: I was certain there was a patch I reviewed recently-ish that had to do with capacity of a host being falsely exceeded when trying to rebuild a host. maybe it's just old age and I'm exhibiting dementia. | |
| 21:07:57 | mriedem | rebuild or evacuate? | |
| 21:10:50 | mriedem | fried_rice: i don't suppose cern was able to verify that series on their ironic cluster yet right? | |
| 21:11:15 | mriedem | i think their ironic compute service code is still queens | |
| 21:11:19 | fried_rice | mriedem: I don't think so, no. | |
| 21:11:24 | fried_rice | they were gonna, soon. | |
| 21:23:23 | openstackgerrit | Paul-Emile Element proposed openstack/nova master: Add emulated TPM support to Nova https://review.openstack.org/631363 | |
| 21:42:47 | mriedem | fried_rice: ok done | |
| 21:42:54 | mriedem | i will have to review the rest of the series next week | |
| 22:25:25 | fried_rice | mriedem_away: Thanks. There's more to the series, if you can. But in particular the top patch (https://review.openstack.org/#/c/620711/) is right near some of the report client singleton stuff you did. | |
| 22:46:53 | fried_rice | leakypipes: a moment of your time? | |
| 22:47:40 | fried_rice | On this RMD thing, I agree as written it's way overblown and crosses a number of lines in the sand. | |
| 22:48:13 | fried_rice | However, I think there are some nuggets of possibilities in there if it is scaled way back and framed in terms of the architecture we've got in place (and forthcoming). | |
| 22:52:33 | fried_rice | - update_provider_tree creates inventory in appropriate providers for ^ (for Train, since we don't have numa modeling, "appropriate providers" == the compute node RP) based on querying RMD or whatever | |
| 22:52:33 | fried_rice | - flavor can contain resource requests for those resource classes in the same way it does today | |
| 22:52:33 | fried_rice | - spawn can pass RMD the allocation information to get the real resources attached to the VM | |
| 22:52:33 | fried_rice | So like: | |
| 22:52:33 | fried_rice | - standard resource classes for just the hard guaranteed minimums, e.g. MIN_LL_CACHE_MB and MIN_MEMORY_BW_MBYTES_SEC | |
| 22:54:19 | fried_rice | None of this dynamic allocation BS; no "burst max" values; communication with RMD via something other than a REST interface to a new service (like via a special file system; maybe I'm misunderstanding, but I think that's a thing). | |
| 22:54:36 | fried_rice | Effectively it's just enabling a couple of new resource classes, plain and simple. | |
| #openstack-nova - 2019-01-19 | |||
| 04:22:27 | openstackgerrit | Hesam Chobanlou proposed openstack/nova master: unused images are always deleted (add to in-tree hper-v code) https://review.openstack.org/628793 | |
| 18:21:56 | lvdombrkr | hello folks | |
| 18:22:10 | lvdombrkr | trying inject root password to instance | |
| 18:22:14 | lvdombrkr | nova config | |
| 18:22:35 | lvdombrkr | [libvirt] | |
| 18:22:35 | lvdombrkr | cpu_mode = host-passthrough | |
| 18:22:35 | lvdombrkr | hw_disk_bus_model=virtio-scsi | |
| 18:22:35 | lvdombrkr | inject_partition = -1 | |
| 18:22:35 | lvdombrkr | inject_password = True | |
| 18:22:38 | lvdombrkr | inject_key = True | |
| 18:22:55 | lvdombrkr | but when booting instance get errir on nova logs: | |
| 18:23:09 | lvdombrkr | artition: Please change permissions on /boot/vmlinuz-4.20.0-1.el7.elrepo.x86_64 to 0x644 | |
| 18:23:09 | lvdombrkr | 2019-01-19 17:46:09.189 8 WARNING nova.virt.disk.api [req-1de3da23-d4a9-4b6a-bc34-38382d3923d9 a083e68210b6444e8acb66c40d79960b 01ee870b0de94b79a93d49fb43e0784b - default default] Ignoring error injecting data into image <LocalFileImage:{'path': u'/var/lib/nova/instances/ef82f997-2024-4cf0-ac40-0be30b47ba7b/disk', 'format': 'qcow2'}> (Please change permissions on /boot/vmlinuz-4.20.0-1.el7.elrepo.x86_64 to 0x644): LibguestfsCannotReadKernelinject_p | |
| 18:23:17 | lvdombrkr | any ideas how to reslove it? | |
| 23:03:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup soft (anti)affinity weight multiplier options https://review.openstack.org/628205 | |
| #openstack-nova - 2019-01-20 | |||
| 02:12:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add microversion to expose virtual device tags https://review.openstack.org/631948 | |
| #openstack-nova - 2019-01-21 | |||
| 02:01:02 | 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:20 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add support matrix for Delete (Abort) on-going live migration https://review.openstack.org/625781 | |
| 03:51:48 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163 | |
| 07:03:04 | openstackgerrit | Sagar Waghmare proposed openstack/nova master: Added new repo https://review.openstack.org/632031 | |
| 08:46:14 | openstackgerrit | Bhagyashri Shewale proposed openstack/os-resource-classes master: Add PCPU standard resource class https://review.openstack.org/631711 | |
| 08:53:32 | openstackgerrit | Merged openstack/gantt master: Retire gantt https://review.openstack.org/630138 | |
| 08:53:45 | openstackgerrit | Merged openstack/python-ganttclient master: Retire python-ganttclient https://review.openstack.org/630154 | |
| 12:01:42 | openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: do not set mac when unplugging macvtap vf https://review.openstack.org/624842 | |
| 12:01:42 | openstackgerrit | sean mooney proposed openstack/nova master: Add free for claimed, allocated devices https://review.openstack.org/616120 | |
| 12:01:43 | openstackgerrit | sean mooney proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929 | |
| 12:01:43 | openstackgerrit | sean mooney proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365 | |
| 12:01:44 | openstackgerrit | sean mooney proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115 | |
| 12:01:45 | openstackgerrit | sean mooney proposed openstack/nova master: [WIP] libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | |
| 12:05:01 | openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: do not set mac when unplugging macvtap vf https://review.openstack.org/624842 | |
| 12:05:02 | openstackgerrit | sean mooney proposed openstack/nova master: Add free for claimed, allocated devices https://review.openstack.org/616120 | |
| 12:05:02 | openstackgerrit | sean mooney proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365 | |
| 12:05:03 | openstackgerrit | sean mooney proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929 | |
| 12:05:03 | openstackgerrit | sean mooney proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115 | |
| 12:05:04 | openstackgerrit | sean mooney proposed openstack/nova master: [WIP] libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | |
| 12:38:54 | 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:52:51 | 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:57:47 | ralonsoh | jaypipes, hi. I added the cleanup to avoid leaving any device created in the system if the test goes wrong | |
| 13:02:55 | jaypipes | ralonsoh: morning! :) | |
| 13:03:22 | jaypipes | ralonsoh: was I correct though, that line 183/4 is doing that delete? | |
| 13:03:58 | ralonsoh | jaypipes, yes, the device is being deleted in this line, but this cleanup is just for precaution | |
| 13:04:10 | 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:36 | jaypipes | ralonsoh: sure, since the _delete_device() method is idempotent, it won't hurt. | |
| 13:05:35 | ralonsoh | jaypipes, yes, that's the point | |
| 13:09:09 | jaypipes | ralonsoh: ack, ok, I'll push it through. | |
| 13:10:04 | jaypipes | ralonsoh: done | |
| 13:10:21 | ralonsoh | jaypipes, thanks! | |
| 13:15:30 | 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:40 | sean-k-mooney | ralonsoh: jaypipes ^ | |
| 13:16:50 | openstackgerrit | Natal Ngétal proposed openstack/nova master: [Configuration] Clean up .gitignore references to personal tools https://review.openstack.org/632086 | |
| 13:16:50 | ralonsoh | sean-k-mooney, yes, to ensure, in case of any problem, we delete both devices | |
| 13:16:53 | 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:17:14 | sean-k-mooney | actully i guess that is just a convention | |
| 13:17:26 | sean-k-mooney | it will actully be called test_dev_9 | |
| 13:17:29 | sean-k-mooney | ok | |
| 13:17:49 | ralonsoh | sean-k-mooney, well yes, and the same in test_add_vlan | |
| 13:18:38 | 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:21:58 | 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:24:38 | 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:51 | sean-k-mooney | *are you free | |
| 13:40:25 | openstackgerrit | Merged openstack/os-resource-classes master: Add PCPU standard resource class https://review.openstack.org/631711 | |
| 13:52:14 | edleafe | I know we skipped one conference; can't remember which | |
| 13:52:32 | edleafe | wrong window | |
| 13:55:46 | efried | n-sch meeting in 5 minutes in #openstack-meeting-alt. Expect it to be a short one. | |
| 14:03:26 | openstackgerrit | Natal Ngétal proposed openstack/nova master: [Configuration] Clean up .gitignore references to personal tools https://review.openstack.org/632086 | |
| 14:37:50 | sean-k-mooney | jaypipes: thanks for the +w i forgot stephenfin is on PTO today | |
| 14:38:41 | jaypipes | sean-k-mooney: no problemo. | |
| 14:46:32 | 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:49:20 | mriedem | gibi: so a consumer record for allocations but no generation? | |
| 14:49:24 | mriedem | i don't think that's possible... | |
| 14:49:34 | mriedem | the consumer generation would start at 0 if it's new | |
| 14:50:12 | 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 | |