| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-31 | |||
| 21:16:10 | efried | mriedem: /me gets bubbles up nose, sprays | |
| 21:31:41 | openstack | bug 1784705 in OpenStack Compute (nova) queens "ResourceTracker.stats can leak across multiple ironic nodes" [High,Triaged] https://launchpad.net/bugs/1784705 | |
| 21:31:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add recreate test for bug 1784705 https://review.openstack.org/587614 | |
| 21:31:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make ResourceTracker.stats node-specific https://review.openstack.org/587636 | |
| 22:10:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make ResourceTracker.stats node-specific https://review.openstack.org/587636 | |
| 22:10:15 | openstack | bug 1784705 in OpenStack Compute (nova) "ResourceTracker.stats can leak across multiple ironic nodes" [High,In progress] https://launchpad.net/bugs/1784705 - Assigned to Matt Riedemann (mriedem) | |
| 22:10:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add recreate test for RT.stats bug 1784705 https://review.openstack.org/587614 | |
| 22:34:12 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| 23:00:35 | openstackgerrit | Merged openstack/nova master: Remove unnecessary PlacementFixture setups https://review.openstack.org/587490 | |
| #openstack-nova - 2018-08-01 | |||
| 01:22:23 | openstackgerrit | Merged openstack/nova master: Delete checking a bool opt of None condition https://review.openstack.org/390705 | |
| 01:27:17 | openstackgerrit | Merged openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs https://review.openstack.org/567258 | |
| 01:41:41 | openstackgerrit | Merged openstack/nova master: Delete orphan compute nodes before updating resources https://review.openstack.org/579922 | |
| 01:42:47 | openstackgerrit | Merged openstack/nova stable/queens: Fix server_group_members quota check https://review.openstack.org/581846 | |
| 04:04:21 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix comments in _anchors_for_sharing_providers and related test https://review.openstack.org/587679 | |
| 04:49:43 | openstackgerrit | Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751 | |
| 05:02:55 | openstackgerrit | Merged openstack/nova master: Extract _update_to_placement method in resource tracker https://review.openstack.org/587171 | |
| 05:12:42 | gmann | alex_xu: hi | |
| 05:15:42 | alex_xu | gmann: hi | |
| 05:16:40 | gmann | alex_xu: today i do not have much for API office hour. if you have anything we can have otherwise i would like to skip today | |
| 05:16:58 | alex_xu | gmann: nothing special from me | |
| 05:17:24 | gmann | alex_xu: ok, let we will cancel and have next week | |
| 05:17:32 | alex_xu | gmann: for the keypair quota usage, the conclude is just updating the doc I guess? | |
| 05:18:07 | gmann | alex_xu: i have not checked the detail yet but i will do check tomorrow | |
| 05:18:21 | alex_xu | gmann: ok, thanks | |
| 05:18:48 | gmann | alex_xu: thanks | |
| 06:22:50 | openstackgerrit | Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751 | |
| 07:08:46 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Ensure the order of AllocationRequestResources https://review.openstack.org/587700 | |
| 07:22:06 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Expand aggregate_metadata table index to cover 'value' as well https://review.openstack.org/587704 | |
| 08:35:50 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix comments in _anchors_for_sharing_providers and related test https://review.openstack.org/587679 | |
| 08:37:35 | yikun | tetsuro ^ updated, :) | |
| 08:39:31 | tetsuro | yikun: Looks good to me. Thanks! | |
| 08:39:50 | yikun | thanks~ | |
| 08:41:25 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Expand aggregate_metadata table index to cover 'value' as well https://review.openstack.org/587704 | |
| 09:15:50 | openstackgerrit | Pranab proposed openstack/os-vif master: Support for OVS DB TCP socket communication. https://review.openstack.org/587378 | |
| 09:30:10 | openstackgerrit | Merged openstack/nova master: compute node local_gb_used include swap disks https://review.openstack.org/585928 | |
| 09:33:52 | openstackgerrit | Merged openstack/nova master: Define common variables for irrelevant-files https://review.openstack.org/578882 | |
| 09:53:04 | openstackgerrit | Merged openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434 | |
| 09:59:05 | kosamara | Hi bauzas, I was checking your spec numa-topology-with-rps. What's the status on this, are you actively reworking it? | |
| 09:59:36 | bauzas | kosamara: yup, I'm planning to rework on it at the end of August | |
| 10:00:20 | bauzas | I'll be on PTO for 3 weeks starting next Wed | |
| 10:00:31 | bauzas | gibi: stephenfin: cdent: ^ | |
| 10:00:57 | cdent | ✔ | |
| 10:05:29 | kosamara | bauzas: you mention RPs for PCI devices (L139 https://review.openstack.org/#/c/552924/9/specs/rocky/approved/numa-topology-with-rps.rst ). This is what I'm interested in. Nova currently doesn't create them, so I think it makes sense to do so either as part of this or as a new spec (maybe better, since your spec is already complex?) | |
| 10:05:56 | gibi | bauzas: have a nice time off! | |
| 10:09:51 | bauzas | kosamara: the spec is about discussing the topology, not really how to provide those devices | |
| 10:10:10 | bauzas | kosamara: so, yeah maybe a new spec | |
| 10:10:30 | bauzas | kosamara: at least about the upgrade between the existing and what you want | |
| 10:12:33 | gibi | bauzas: FYI, we talked about the PCI device handling with kosamara in openstack-neutron as well http://eavesdrop.openstack.org/irclogs/%23openstack-neutron/%23openstack-neutron.2018-08-01.log.html#t2018-08-01T08:23:33 | |
| 10:14:59 | bauzas | gibi: ack | |
| 10:16:19 | bauzas | oh, fun, GPUs look like a good driver for PCI devices :p | |
| 10:17:07 | gibi | bauzas: fun indeed | |
| 10:22:54 | bauzas | mdbooth: others, I'm torn with https://review.openstack.org/#/c/478330/ | |
| 10:23:56 | lyarwood | oh it's that time in the cycle again? :) | |
| 10:24:11 | lyarwood | I'm sure this has been nack'd in the past | |
| 10:25:46 | bauzas | nvm, found the spec | |
| 10:25:48 | bauzas | https://blueprints.launchpad.net/nova/+spec/volume-backed-server-rebuild | |
| 10:25:58 | bauzas | I'll accordingly -2 it | |
| 10:26:08 | bauzas | and refer to this series | |
| 10:26:13 | lyarwood | https://review.openstack.org/#/c/532407/ yeah | |
| 10:34:49 | openstackgerrit | Zengzhi Wang proposed openstack/nova master: Fix using last request-id for all log when multiple request calls. https://review.openstack.org/587754 | |
| 10:35:09 | mdbooth | bauzas: On a call. Still want me to look? | |
| 10:35:39 | bauzas | mdbooth: nope, I gently -2'd the change :) | |
| 10:35:52 | mdbooth | 'gently' ;) | |
| 10:36:11 | bauzas | I'm French but I can be a gentleman | |
| 10:36:13 | mdbooth | bauzas: You used the fluffy ban hammer | |
| 10:36:38 | bauzas | mdbooth: there is one spec and already an open series | |
| 10:36:45 | bauzas | I just asked the proposer to collaborate | |
| 10:37:03 | bauzas | -2 just means "please stop what you're doing and discuss" | |
| 10:37:32 | bauzas | it's not a "I don't like you, and for god's sake, you'll suffer pain' | |
| 10:37:45 | mdbooth | bauzas: Although I recall the other discussion. I'm not in favour of the approach in this patch, tbh. | |
| 10:38:08 | mdbooth | To me, rebuild on BFV means you keep the original volume and re-image it. | |
| 10:39:01 | sean-k-mooney | mdbooth: is that not want was originally propsoed? | |
| 10:39:25 | mdbooth | sean-k-mooney: I think that was in the other proposal I recall. | |
| 10:39:34 | mdbooth | sean-k-mooney: My memory is notoriously shonky, though. | |
| 10:41:15 | sean-k-mooney | sorry im thinking of something else i was vagly rembering the rescure rescure bfv changes and mixing it up with rebuild | |
| 10:41:18 | bauzas | anyway, this is a behavioural change proposal, hence a spec | |
| 10:41:36 | bauzas | for people wanting to get the last proposal, the spec is thrre | |
| 10:44:06 | openstackgerrit | Pranab proposed openstack/os-vif master: Support for OVS DB TCP socket communication. https://review.openstack.org/587378 | |
| 10:58:33 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Reload oslo_context after calling monkey_patch() https://review.openstack.org/587772 | |
| 11:30:18 | kosamara | bauzas: thanks! As I said in openstack-neutron, I'm examining how to create RPs for GPUs from the pci devices that nova tracks, in order to move the scheduling of PCI passthrough to placement. This should in the future allow quotas on them too, as resource classes. | |
| 11:32:14 | kosamara | Btw, could someone please take another look at https://review.openstack.org/#/c/579897 ? | |
| 11:34:11 | sean-k-mooney | kosamara: i belive that was planned to be started next cycle. not jsut for gpus but for pci devices in general | |
| 11:38:32 | sean-k-mooney | kosamara: can you explain what the vendorid in https://review.openstack.org/#/c/579897/2/nova/virt/libvirt/config.py | |
| 11:39:03 | kosamara | sean-k-mooney: makes sense not to do it just for GPUs. I don't know about the plans though. I was researching for any relevant specs and found the numa-rps and the net-bandwidth-rps. Can you point me to anything else? | |
| 11:41:10 | kosamara | sean-k-mooney: the vendorid can apparently be a random alphanumeric. I've tried it with the one proposed. My reference: https://wiki.archlinux.org/index.php/PCI_passthrough_via_OVMF#Troubleshooting | |
| 11:47:52 | sean-k-mooney | kosamara: the numa aware resouce providers was a prequisit for moving pci tracking to placement | |
| 11:50:04 | kosamara | sean-k-mooney: I'm not sure of that. PCI passthrough now doesn't require information about the NUMA topology. The PCI RPs could initially be children of the compute node RP, and then NUMA topology information can be added as extra layer in between, no? | |
| 11:51:40 | sean-k-mooney | kosamara: that would require the reshpaer work to be completed. but no today in openstack we track numa affinity of all pcidevces | |
| 11:52:47 | kosamara | sean-k-mooney: what's reshpaer ? | |
| 11:52:57 | sean-k-mooney | kosamara: what im guessing confused about with https://review.openstack.org/#/c/579897/2/nova/virt/libvirt/config.py is why you are adding a vendior id element to the hyperv features elememnt i nthe first place if you are trying to hide the fact you are being virutalised. | |
| 11:53:31 | sean-k-mooney | kosamara: https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/reshape-provider-tree.html | |
| 11:54:31 | sean-k-mooney | kosamara: its the ablitiy to reshap an exisiting nested resouce provider tree and move resouce providers form the root node to child nodes | |
| 11:54:32 | kosamara | sean-k-mooney: Maybe I should word this more precisely. We are trying to block detection of hyperv specifically. Making the vendorid anything other than the genuine one should work. | |
| 11:55:03 | sean-k-mooney | kosamara: are you using libvirt to manage hyperv nodes? | |
| 11:57:13 | sean-k-mooney | kosamara: the code you are modifying is changing the hyperv element in https://libvirt.org/formatdomain.html#elementsFeatures | |
| 12:06:05 | lpetrut | kosamara: I guess you're overriding the signature returned through cpuid 0x40000000 | |
| 12:06:43 | lpetrut | sean-k-mooney: qemu can mimic some of the hyper-v enlightments | |
| 12:07:30 | sean-k-mooney | lpetrut: yep but im wondering if we shoudl entirely remove the hyperv element so that if nvidia make there check smarter in the future we dont need to change the code again | |
| 12:09:59 | lpetrut | but then I guess you'd lose all those hyper-v enlighments, which would probably affect Windows guests performance. I think the idea is to keep those, hiding the signature only. AFAIK, those guest features don't rely on the cpuid signature | |
| 12:10:25 | kosamara | sean-k-mooney: This is for kvm nodes. The <hyperv> element appears also on kvm nodes, when creating VMs from images with os_type='windows' | |