| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-18 | |||
| 15:51:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use cell targeted context to query instance metadata https://review.opendev.org/752459 | |
| 15:53:19 | gibi | that was all from me for this week. See you on Monday | |
| 15:53:21 | gibi | o/ | |
| 16:12:06 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyperv: Configures chassis asset tags for VMs. https://review.opendev.org/752723 | |
| 16:23:47 | sean-k-mooney | stephenfin: look like this issue im facing is that the fakelibvirt dirver is incomplete | |
| 16:24:45 | sean-k-mooney | stephenfin: when we generate teh fake instance domain its not provided with the info related to the vm pci devices just that the nic is type direct or whatever. | |
| 16:25:06 | stephenfin | Hmm, yeah, that's not surprising | |
| 16:25:25 | sean-k-mooney | we are also not generating network node_dev elements for any of the pci PF/VFs | |
| 16:25:51 | sean-k-mooney | im going to hard code a few thigs to see if i can make some minimal chagnes to get it to pass | |
| 16:26:03 | sean-k-mooney | but we likely need to flesh this out more in the future | |
| 16:26:41 | stephenfin | yeah, I did the minimal amount necessary to get the sr-iov migration tests working | |
| 16:27:02 | stephenfin | but it's full of holes, for sure. I just didn't have SR-IOV hardware handy to inspect the flow and copy the XML from | |
| 16:27:32 | sean-k-mooney | so the issue is i need to backprot this to queens | |
| 16:27:43 | sean-k-mooney | so im hesitent to depend on your refactors | |
| 16:27:53 | sean-k-mooney | but they might also fix some of the issues | |
| 16:28:37 | sean-k-mooney | this is what causes the detach to fail to find the device currently | |
| 16:28:40 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/libvirt/fakelibvirt.py#L1826-L1830 | |
| 16:28:52 | sean-k-mooney | that ses the souce_dev to fake_pf_interface_name | |
| 16:29:08 | sean-k-mooney | but in https://review.opendev.org/#/c/602432/17/nova/tests/unit/virt/libvirt/fakelibvirt.py | |
| 16:29:30 | sean-k-mooney | both before and after my change we dont set source_dev when generatign the vm domain | |
| 16:30:10 | sean-k-mooney | im going to hard code if for now but im concerned then next thing ill hit is the pci adress dont match adresses form the host pci_info object | |
| 16:30:24 | sean-k-mooney | since in both cases we are just incrementing the adresses | |
| 16:30:46 | sean-k-mooney | ill corss that bridge later i guess. | |
| 16:34:12 | sean-k-mooney | the issue with that fixture is that the comment and assumtion is wrong | |
| 16:34:26 | sean-k-mooney | we also use this with macvtal to get the vf name not jsut the parent | |
| 16:34:57 | sean-k-mooney | and we also dont have the required info to work this out when we generate the domain stuff currently. | |
| 17:03:16 | openstackgerrit | Merged openstack/nova master: test_evacuate: Wait until subnode is down before starting tests https://review.opendev.org/751065 | |
| 17:14:07 | sean-k-mooney | stephenfin: ... https://github.com/openstack/nova/blob/master/nova/tests/unit/virt/libvirt/fakelibvirt.py#L1087-L1093 | |
| 17:14:27 | sean-k-mooney | detach device in the fake libvirt fixture only support disks | |
| 17:14:57 | sean-k-mooney | so i dont know how/why gibis func test could be passing on master. | |
| 19:19:51 | lyarwood | gmann: sorry was afk, are you still around? | |
| 19:20:31 | lyarwood | gmann: AFAIK the plan is to still move to focal in V and skip the tests | |
| 19:20:51 | lyarwood | gmann: in the meantime we wanted to try to move bionic onto the train UCA to allow us to move the MIN_ versions in the libvirt driver | |
| 19:21:10 | lyarwood | gmann: https://review.opendev.org/#/c/752641/ needs to land first in stable/ussuri and then master to allow grenade to work correct | |
| 19:21:14 | lyarwood | correctly* | |
| 19:21:24 | lyarwood | then the rest of https://review.opendev.org/#/q/topic:bump-libvirt-qemu-victoria+(status:open+OR+status:merged) can land | |
| 19:22:16 | lyarwood | gmann: when do you think th focal change is actually going to land btw? | |
| 19:56:57 | gmann | lyarwood: hi, sorry i was away | |
| 19:59:05 | gmann | lyarwood: ok, and I am keeping tempest-integrated-compute job on bionic to have those test coverage running somewhere | |
| 19:59:34 | gmann | lyarwood: let me review the devstack patches | |
| 20:00:12 | gmann | plan for Focal migration is pending on Barbican issue which is the only blocker left https://storyboard.openstack.org/#!/story/2007732 | |
| 20:09:34 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: testing Train UCA https://review.opendev.org/752759 | |
| 21:40:12 | openstackgerrit | Merged openstack/nova stable/ussuri: Correctly disable greendns https://review.opendev.org/751621 | |
| 23:03:02 | openstackgerrit | Merged openstack/nova master: test_evacuate.sh: Support libvirt-bin and libvirtd systemd services https://review.opendev.org/752650 | |
| 23:03:12 | openstackgerrit | Merged openstack/nova stable/rocky: Improve CinderFixtureNewAttachFlow https://review.opendev.org/749222 | |
| #openstack-nova - 2020-09-19 | |||
| 01:38:16 | gmann | lyarwood: +2 on both devstack patches, | |
| 01:42:27 | openstackgerrit | Merged openstack/nova stable/queens: libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live migration https://review.opendev.org/747976 | |
| 14:20:08 | openstackgerrit | Merged openstack/nova stable/rocky: Robustify attachment tracking in CinderFixtureNewAttachFlow https://review.opendev.org/749223 | |
| 16:58:08 | openstackgerrit | Merged openstack/nova stable/rocky: compute: Use source_bdms to reset attachment_ids during LM rollback https://review.opendev.org/749224 | |
| #openstack-nova - 2020-09-20 | |||
| 08:42:44 | openstackgerrit | Mamduh proposed openstack/os-vif master: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/745895 | |
| 11:50:36 | aj_mailing | join #openstack | |
| 21:59:03 | openstackgerrit | Merged openstack/nova stable/rocky: compute: refactor volume bdm rollback error handling https://review.opendev.org/749225 | |
| 21:59:11 | openstack | bug 1889108 in OpenStack Compute (nova) rocky "failures during driver.pre_live_migration remove source attachments during rollback" [Undecided,In progress] https://launchpad.net/bugs/1889108 - Assigned to Lee Yarwood (lyarwood) | |
| 21:59:11 | openstackgerrit | Merged openstack/nova stable/rocky: Add regression tests for bug #1889108 https://review.opendev.org/749226 | |
| 21:59:14 | openstackgerrit | Merged openstack/nova stable/rocky: compute: Don't delete the original attachment during pre LM rollback https://review.opendev.org/749227 | |
| #openstack-nova - 2020-09-21 | |||
| 01:10:54 | openstackgerrit | Yao wang proposed openstack/nova master: Use VIR_DOMAIN_XML_INACTIVE for detach volume https://review.opendev.org/749489 | |
| 01:40:33 | openstackgerrit | Yingji Sun proposed openstack/nova stable/train: Set different VirtualDevice.key https://review.opendev.org/752843 | |
| 02:37:18 | openstackgerrit | Yao wang proposed openstack/nova master: Use VIR_DOMAIN_XML_INACTIVE for detach volume https://review.opendev.org/749489 | |
| 08:01:16 | bauzas | good morning Nova | |
| 08:10:47 | gibi | good morning bauzas | |
| 08:23:12 | openstackgerrit | Elod Illes proposed openstack/nova stable/pike: libvirt: Provide VIR_MIGRATE_PARAM_PERSIST_XML during live migration https://review.opendev.org/747978 | |
| 08:27:39 | kashyap | bauzas: When is the deadline? 01-Oct? | |
| 08:27:47 | bauzas | yes... | |
| 08:35:59 | brinzhang | gibi: can you review this patch https://review.opendev.org/#/c/749052? | |
| 09:09:03 | kashyap | lyarwood: Morning, I've tried a rapid succession of attaching 20 disks (via `attach-device`) while there's consistent 40% of load on the CPUs ... can't reproduce so far. | |
| 09:09:18 | kashyap | lyarwood: If you don't mind holding your nose tightly ... here's the repro: https://kashyapc.fedorapeople.org/device-attach-and-detach.bash | |
| 09:10:24 | kashyap | (Fun aside: changing the target dev from sdX to vdX increases the execution of the above script by at least 20 seconds. So attaching 'virtio' disks seems to "slow"? Can't be so) | |
| 09:20:37 | kashyap | That said, I should rewrite it in Python and make it as close to what Nova is doing as possible | |
| 09:23:13 | kashyap | Because, I learnt that if we suspsect a timing issue, 'virsh' has a lot more overhead. A naive connection-based approach is better | |
| 09:50:06 | hemanth_n | sean-k-mooney: Appreciate if you can have a final look on this patch https://review.opendev.org/#/c/749175/ .. thank you | |
| 09:55:26 | lyarwood | kashyap:k~. | |
| 09:55:37 | lyarwood | ops weird connection issue | |
| 09:56:05 | lyarwood | kashyap: ack yeah that sounds like a sane approach, I don't think it's the rapid attach/detach that's the issue btw, just the overall load on the underlying host and libvirtd | |
| 09:58:42 | kashyap | lyarwood: Oh, right; perhaps I should increase the load to 80% and do a detach? | |
| 09:58:55 | kashyap | I'll vary the load (by measuring it in parallel with `sar -u 5`) and see | |
| 10:00:52 | lyarwood | yup sounds good | |
| 10:01:57 | tosky | lyarwood: nice work on the live migration/evacuate stuff, it seems that part is fine | |
| 10:02:05 | tosky | now only the grenade one is missing | |
| 10:10:26 | lyarwood | tosky: np thanks for leading the effort :) | |
| 10:10:53 | lyarwood | gibi / stephenfin ; https://review.opendev.org/#/q/owner:self+topic:native-zuulv3-migration+status:open+project:openstack/nova ^ re the above, the live-migration and new evacuate job should be ready for review now. | |
| 10:14:25 | lyarwood | actually let me update the TODO for the ceph jobs | |
| 10:14:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Introduce nova-evacuate https://review.opendev.org/744883 | |
| 10:14:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/752557 | |
| 10:15:19 | lyarwood | there we go, should be good now if the gate behaves | |
| 10:16:16 | tosky | gates and Monday, uhm | |
| 10:16:40 | tosky | there are only 90 (check) + 16 (gate) jobs, so we may be lucky | |
| 10:19:50 | lyarwood | hmmm I wonder if it's actually worth merging the nova-live-migration and nova-evacuate job now that the playbook is written | |
| 10:32:24 | tosky | up to you and the time needed to run the tests vs the time needed to deploy | |
| 10:41:07 | gibi | lyarwood: I have some nits in https://review.opendev.org/#/c/744883 | |
| 10:41:18 | gibi | lyarwood: will you do a follow up or respin it quickly? | |
| 10:42:43 | gibi | brinzhang: sorry I think that refactor needs to wait for W | |
| 10:44:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Enable 'vmcoreinfo' feature by default https://review.opendev.org/752912 | |
| 11:01:28 | sean-k-mooney | hemanth_n: sure ill go over it later today | |
| 11:01:41 | hemanth_n | sean-k-mooney: thanks | |
| 11:01:55 | lyarwood | gibi: ack can do | |
| 11:09:57 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Introduce nova-evacuate https://review.opendev.org/744883 | |
| 11:09:58 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/752557 | |
| 11:22:21 | gibi | lyarwood: thanks | |
| 11:38:29 | openstackgerrit | Yao wang proposed openstack/nova master: Use VIR_DOMAIN_XML_INACTIVE for detach volume https://review.opendev.org/749489 | |
| 11:43:11 | sean-k-mooney | gibi: stephenfin i have figured out why the func test are failing. one issue is i was not adding the 0x prefix if you use "%(func)o" or "%(func)x" it does not genreate the prefix, the main issue however is device detach was only implemented for disk in the fake libvirt driver | |