| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-04 | |||
| 01:57:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299 | |
| 01:58:07 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344 | |
| 01:59:23 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673 | |
| 01:59:41 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676 | |
| 01:59:58 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689 | |
| 02:00:15 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709 | |
| 02:00:27 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.openstack.org/576712 | |
| 02:44:50 | openstackgerrit | Brin Zhang proposed openstack/nova master: Remove the string check of the flavor attribute 'swap' https://review.openstack.org/639012 | |
| 03:23:24 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add descriptions of numbered resource classes and traits https://review.openstack.org/621494 | |
| 05:55:25 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix an error when generating a host ID https://review.openstack.org/639936 | |
| 07:39:50 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.openstack.org/621476 | |
| 08:01:44 | openstackgerrit | Michael Still proposed openstack/nova master: Implement simple prometheus metrics for nova. https://review.openstack.org/640688 | |
| 08:10:27 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.openstack.org/621476 | |
| 08:14:05 | openstackgerrit | Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.openstack.org/621476 | |
| 08:56:11 | openstackgerrit | melanie witt proposed openstack/nova master: Add user_id field to InstanceMapping https://review.openstack.org/633350 | |
| 08:56:11 | openstackgerrit | melanie witt proposed openstack/nova master: Populate InstanceMapping.user_id during migrations and schedules https://review.openstack.org/638574 | |
| 08:56:12 | openstackgerrit | melanie witt proposed openstack/nova master: Add get_counts() to InstanceMappingList https://review.openstack.org/638072 | |
| 08:56:12 | openstackgerrit | melanie witt proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351 | |
| 08:56:13 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Count instances from mappings and cores/ram from placement https://review.openstack.org/638073 | |
| 08:56:13 | openstackgerrit | melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.openstack.org/638324 | |
| 08:56:33 | tssurya | melwitt: you are online still ? :o | |
| 08:57:12 | melwitt | tssurya: yeah :( wanted to get this uploaded before mriedem morning | |
| 08:57:24 | melwitt | going to go to bed now | |
| 08:57:31 | tssurya | yea good night! thanks | |
| 08:57:32 | gibi | sean-k-mooney: hi! I can start digging the pf interface name autodetection now. If you have any extra info top of what we discussed on Friday then hit me with it | |
| 08:57:40 | gibi | melwitt: good night! | |
| 08:58:10 | melwitt | goodnight everyone :) | |
| 09:10:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311 | |
| 09:10:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543 | |
| 09:10:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support server create with ports having resource request https://review.openstack.org/636360 | |
| 09:15:00 | openstackgerrit | Michael Still proposed openstack/nova master: Implement simple prometheus metrics for nova. https://review.openstack.org/640688 | |
| 09:45:50 | openstackgerrit | Lajos Katona proposed openstack/python-novaclient master: Add support for microversion v2.72 https://review.openstack.org/637234 | |
| 10:13:49 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271 | |
| 10:37:01 | kashyap | stephenfin: Hi, when you get a moment, I am sure a bug here is required (it might only cause confusion): https://review.openstack.org/#/c/637578/1/nova/virt/libvirt/driver.py ("Omit needless check on 'CONF.serial_console'") | |
| 10:37:35 | kashyap | (And the small refactorings we discussed there should be done as follow-ups.) | |
| 10:38:31 | aspiers | morning kashyap :) | |
| 10:38:41 | kashyap | aspiers: Morning | |
| 10:38:58 | lyarwood | mdbooth: https://review.openstack.org/#/q/topic:bug/1803961+(status:open+OR+status:merged) - can you take a look at this today if you have time? | |
| 10:39:31 | aspiers | kashyap: https://review.openstack.org/#/c/640483/1 is ready for review now | |
| 10:39:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix wrong consumer type in logging https://review.openstack.org/640723 | |
| 10:42:02 | kashyap | aspiers: Thanks for reworking: https://review.openstack.org/#/c/633855/7/nova/virt/libvirt/host.py | |
| 10:42:46 | kashyap | aspiers: Yep, looking at the 'emulator' one | |
| 11:01:14 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add nits from Id2beaa7c4e5780199298f8e58fb6c7005e420a69 https://review.openstack.org/640729 | |
| 11:01:14 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework 'config-drive' user doc https://review.openstack.org/640730 | |
| 11:01:45 | gibi | sean-k-mooney: I made some progress with the pf_interface_name auto detection but I realized that this makes the feature virt driver dependent. As far as I see xenapi also supports VF and PF passthrough but I don't know how to autodetect pf_interface_name in xenapi :/ | |
| 11:04:01 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add compute service support for attach/detach root volume https://review.openstack.org/614750 | |
| 11:05:44 | kashyap | gibi: It is OK to update a spec file retroactively, yes? Specifically something cosmetic like changing the name of config attribute | |
| 11:06:10 | kashyap | gibi: E.g. I'm suggesting the patch poster to change from 'cpu_models' to 'cpu_model_list' (to better distinguish from the existing 'cpu_model') | |
| 11:06:10 | gibi | kashyap: it is even encuraged to update the spec file if the design diverged | |
| 11:06:17 | kashyap | https://review.openstack.org/#/c/637834/4/nova/conf/libvirt.py@548 | |
| 11:06:26 | kashyap | gibi: Good, thought so. Because that's the most sensible thing. | |
| 11:06:30 | HD|Laptop | hey all | |
| 11:06:44 | HD|Laptop | is there a way to force nova to only use Cinder for all kinds of block storage, both for the images it fetches from Glance as well as for "ordinary" servers created via a simple "openstack server create" without manually allocating/attaching a Cinder volume? | |
| 11:07:57 | lyarwood | HD|Laptop: no, it has been discussed at recent Forum / PTGs etc but nothing has been worked on AFAIK. | |
| 11:08:41 | HD|Laptop | oh damn. 'cause all my compute nodes have 128GB SSDs only and that's it, for storage I have a nice 80TB QNAP NAS :'D | |
| 11:09:06 | HD|Laptop | another thing: can I somehow specify additional parameters for qemu/kvm? I need to match https://github.com/kholia/OSX-KVM/blob/master/boot-macOS.sh | |
| 11:09:18 | HD|Laptop | (and yes I do only run this on original Macs, 9x 2010-era Mac Pro cheesegraters, so I don't violate licenses) | |
| 11:10:06 | lyarwood | HD|Laptop: you can supply additional QEMU params via libvirt but I don't recall a way of doing that part via Nova | |
| 11:10:35 | HD|Laptop | thanks :) | |
| 11:14:12 | lyarwood | HD|Laptop: yeah I can't see a way of passing extra args from Nova to libvirt and down to QEMU tbh | |
| 11:14:56 | kashyap | HD|Laptop: lyarwood: There is no way that Nova provides it either. | |
| 11:15:28 | kashyap | If you go behind libvirt (or Nova's) back and use QEMU command-line options, and if something breaks, you get to keep the pieces :-) | |
| 11:15:32 | lyarwood | kashyap: could you use a custom machine type for this? | |
| 11:16:29 | kashyap | lyarwood: Machine types won't help in this case. (They are a particular chipset with additional devices) | |
| 11:17:00 | lyarwood | kashyap: yeah just for the additional devices I mean | |
| 11:17:19 | kashyap | That's a bit too heavy-weight :-) | |
| 11:17:27 | kashyap | (Earlier typo: s/additional/default/) | |
| 11:17:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Add initial version of metadata configuration https://review.openstack.org/640733 | |
| 11:19:31 | kashyap | lyarwood: Aside: there's already some discusions upstream to propose a separate machine type for 'x86' that is "legacy-free" | |
| 11:19:53 | kashyap | (Just like how AArch64 has a machine type called 'virt', which is expressely designed for virtual machines.) | |
| 11:23:26 | HD|Laptop | kashyap: ah damn. How is qemu called in nova, anyway? Tried to read myself through the libvirt driver, but I couldn't even find out where it gets "-machine pc-i440fx-3.1" from, there isn't even any single mention of that machine type anywhere in the nova git repository | |
| 11:23:52 | HD|Laptop | Maybe I can do a hot-patch in there... | |
| 11:25:17 | kashyap | HD|Laptop: Nova models every guest feature via libvirt classes / objects, which lets libvirt build an opinionated QEMU command-line. | |
| 11:26:26 | kashyap | HD|Laptop: No specific machine types are hard-coded in Nova, obviously. They're selected by what your host hypervisor supports | |
| 11:26:43 | mdbooth | zhubx: Are you the author of https://review.openstack.org/#/c/640271/ ? | |
| 11:26:55 | mdbooth | lyarwood: Looking now. | |
| 11:27:21 | sean-k-mooney | gibi: you are correct that it would make it virt driver dependent in a way but no more then manually adding the pf_interface_name. for xenapi the compute service i belive will be runnin in dom0 i have not deployed xen in quite a long time but i think you could use sysfs the same way. after all if the sriov_nic agent is running in dom0 and can see the interface netdevs then you can also see the vf and | |
| 11:27:22 | HD|Laptop | kashyap: ah, ok. where do I have to look for the final call to libvirt for starting the qemu process? | |
| 11:27:23 | sean-k-mooney | lookup the pf netdev the same way | |
| 11:27:46 | mdbooth | zhubx: If so, I just hit that again. I think this is the right approach and it'll be good to go if you can fix up a few things and add unit tests. | |
| 11:27:56 | kashyap | HD|Laptop: For machine type handling in Nova, see this: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4221,L4248 | |
| 11:28:12 | stephenfin | kashyap: Is required or is not required? | |
| 11:28:20 | kashyap | HD|Laptop: There is no "final call" in Nova; libvirt takes care of it. | |
| 11:28:28 | kashyap | stephenfin: Not required. | |
| 11:29:01 | kashyap | HD|Laptop: If you want additional command-line options that Nova currently doesn't have; then you have to model that into Nova by providing libvirt config class objects. | |
| 11:30:15 | kashyap | stephenfin: (Sorry, saw the typo in my sentence in the scrollback :-)) | |
| 11:30:23 | sean-k-mooney | kashyap: that is allowed to a point. we do not allow the use of the qemu:args element to be genearted to allow raw qemu commands to be added | |
| 11:30:41 | mdbooth | lyarwood: Did you speak to mriedem about removing the timestamp check from the retype tempest test? | |
| 11:30:55 | sean-k-mooney | kashyap: i have tried in the past to enable features that are not supported by libvirt yet or on older versions | |
| 11:31:33 | mdbooth | I'll take your work for it that they're flaky, but IIUC the purpose was to assert functionally that the volume contains the same data after retype. | |
| 11:31:56 | mdbooth | lyarwood: IIRC mriedem was especially keen on this. | |
| 11:32:51 | lyarwood | mdbooth: I had no idea he was attached to it tbh | |
| 11:32:57 | lyarwood | mdbooth: I'll bring it up with him this afternoon | |
| 11:33:13 | lyarwood | mdbooth: and yeah it has been pretty awful for me locally testing this | |
| 11:33:40 | mdbooth | Yeah, I can imagine. Seems to work ok in the gate, though. | |
| 11:34:07 | kashyap | sean-k-mooney: Yeah, that's what I meant: Nova not allowing libvirt's <qemu:commandline> element which allows arbitrary pass-through. | |
| 11:34:32 | sean-k-mooney | yep that is the element i was thinking of | |
| 11:34:34 | kashyap | sean-k-mooney: (And yeah, I've used it for one-off hacks in Nova in the past.) | |
| 11:35:37 | sean-k-mooney | ya i used it to demonstarte that you can run ovs-dpdk without hugepages before i got funding to have the libvirt support for file backed memory developed | |
| 11:36:28 | sean-k-mooney | i never got time to add the nova support for file backed memory the way i wanted it but redhat did it a release or two later for other uses | |