Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-09-03
16:01:02 artom lyarwood, ... everything?
16:01:04 sean-k-mooney lyarwood: i think lnav support direct links
16:01:31 sean-k-mooney it would automaticaly syntax highlith and alinge all the logs by time stamp too if that worked
16:02:44 lyarwood artom: well not everything but the stuff I typically look at, so n-* logs, tempest logs etc
16:02:59 sean-k-mooney anything ending in log
16:03:21 sean-k-mooney oh i guess most end in .txt
16:04:23 sean-k-mooney ah the gzip compression mess up usin lnav and likely vim with the raw urls
16:08:43 lyarwood sean-k-mooney: vim WORKSFORME with gzip FWIW
16:08:58 sean-k-mooney lyarwood: gzip is not the issue
16:09:09 sean-k-mooney https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_557/746986/3/check/nova-live-migration/557b715/logs/screen-n-cpu.txt
16:09:17 sean-k-mooney us being served compressed
16:09:31 lyarwood ah yeah that borks vim
16:09:37 sean-k-mooney yep
16:10:01 sean-k-mooney vim and lnave are not reading the hearer or it is not serving it with the writh header
16:10:36 lyarwood I think vim just calls out to wget tbh
16:10:42 sean-k-mooney ya its being served tih text/plain and Content-encoding gzip
16:10:46 lyarwood at least it does for me but that could be a plugin
16:11:18 sean-k-mooney yep so lnav and vim would have to use teh Accepts-encode header to not allow copression
16:11:34 sean-k-mooney or read the respocne head and do the right thing
16:12:11 sean-k-mooney that said just localy donloadingthm to somewhere in /tmp and then passing the paths to vim or lnav works
16:31:36 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reproducer for bug #1894095 https://review.opendev.org/749713
16:31:36 openstack bug 1894095 in OpenStack Compute (nova) "Running periodic task during live migration results in incorrect usage" [Medium,Confirmed] https://launchpad.net/bugs/1894095 - Assigned to Stephen Finucane (stephenfinucane)
16:37:09 gibi sean-k-mooney: can I pull the trigger on the os-vif release https://review.opendev.org/#/c/749535/ ?
16:40:48 artom lyarwood, so actually our volume thing with whitebox is different - it's a test for detaching a volume with libvirt stopped, so we expect the volume to remain in-use until libvirt starts up again and the detach completes. Did that behaviour change?
16:45:03 sean-k-mooney gibi: ya i think so
16:45:10 gibi sean-k-mooney: ack
16:45:21 sean-k-mooney the two patches i was waiting for are still not merged but they need rework
16:45:27 sean-k-mooney so we will just backport them later
16:45:32 gibi OK, done
16:45:35 lyarwood artom: when you say completes is the disk detached from the domain?
16:45:40 lyarwood artom: or just detached in cinder
16:46:14 artom lyarwood, cinder
16:46:19 lyarwood artom: if libvirt is stopped the call to detach should fail but AFAIK we should then rollback the volume state to in-use
16:46:27 lyarwood artom: if we don't that's a bug
16:47:00 artom lyarwood, ok, I'll dig some more then - looks like it's only happening with focal
16:47:09 artom For all I know we're stopping libvirt wrong :P
16:47:22 lyarwood huh, might be the error raised from the libvirt python lib has changed
16:47:29 lyarwood when it can't talk to libvirtd
16:47:33 lyarwood or some weirdness like that
16:54:27 gibi dansmith: lyarwood: thanks for taking care of the rbd documentation
16:54:53 gmann lyarwood: you can use this patch also which unskip the tests - https://review.opendev.org/#/c/734029/
16:55:02 gmann devstack-platform-focal job run those test on focal
16:55:40 artom lyarwood, ah, no, looks like systemctl is configured in focal to restart libvirtd if it dies?
16:55:51 artom In any case, something starts it back up again, 3 seconds afer we stop it
16:55:55 gibi artom: thanks for the review on https://review.opendev.org/#/c/748453/ I can try to add a functional test for that later, if that is OK
16:56:16 artom gibi, I think we're better off reviewing stephenfin's series first
16:56:25 artom Then adding on to it
16:56:51 lyarwood gmann: ack thanks
16:56:54 lyarwood artom: kk
16:56:57 lyarwood gibi: np
16:59:34 gibi artom: make sense
16:59:44 gibi artom: I will try to get to that series
16:59:57 artom gibi, same here
17:00:05 artom There's a lot there to unpack
17:03:53 sean-k-mooney atuclly i was hoping we could merge the sriov feature first
17:04:03 sean-k-mooney because im not sure we will get both done by m3
17:04:35 sean-k-mooney so put stephens changes on top of gibis?
17:05:27 sean-k-mooney that way we can avoid the merge conflitcs and still review them all in context?
17:05:31 artom sean-k-mooney, yeah, I was saying the same thing
17:05:48 gibi I would not rush with that move, as I lost my whole afternoon downstream today. I think if we approve both and then there is a conflict we can decide which one to move
17:05:54 artom sean-k-mooney, gibi, 1. merge SRIOV 2. merge stephenfin func tests 3. improve func tests to cover the case gibi is fixing
17:06:19 gibi in general I agree
17:06:30 artom And 4. convert SRIOV live migration to use claims like NUMA live migration, giving sean-k-mooney an aneurysm in the process ;)
17:06:38 gibi It is just hard to commit to 1. right now as I still have a list of func test to write
17:06:53 sean-k-mooney :)
17:07:16 gibi please don't kill each other over it, I need both of you in this team!
17:07:36 artom Nah, we'll just have an honest drinking competition for it
17:07:53 artom I'm only saying that because I believe sean-k-mooney is one of the very few Irish people I might actually beat
17:08:04 sean-k-mooney if we create cliams in the conductor/scheduler it would impove my opipion of them
17:08:24 sean-k-mooney hehe that because i dont drink that often
17:09:20 sean-k-mooney does not mean i will let you win too easy :)
17:09:47 gibi I can be the judge of that drinking contest after the pandemic
17:10:20 sean-k-mooney gibi: for what its worth we both agree we shoudl have 1 code path for the migration stuff
17:10:29 gibi agree
17:10:42 sean-k-mooney im just not comfortably with the level of tech debt that claims have
17:11:50 sean-k-mooney if we make claims better and more useful by creating claims for resouce eairler then i would be more comfortable adtoping them for sriov migration
17:12:32 sean-k-mooney but that one of those things like move neutron port creation to the conductor
17:13:12 sean-k-mooney it would be good to do but not something we can sell to customers or PMs easily.
17:17:57 gibi yeah, it is tech debt removal, and that is hard to sell
17:28:56 sean-k-mooney also in this case hard work with all the other things that have built up around it
17:43:44 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION and NEXT_MIN_{LIBVIRT,QEMU}_VERSION https://review.opendev.org/746981
17:43:45 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/746982
17:43:45 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_NATIVE_TLS_VERSION https://review.opendev.org/746983
17:43:46 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_BETTER_SIGKILL_HANDLING https://review.opendev.org/746984
17:43:46 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_VIDEO_MODEL_VERSIONS https://review.opendev.org/746985
17:43:47 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_PMEM_SUPPORT https://review.opendev.org/746986
17:56:50 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: compute: refactor volume bdm rollback error handling https://review.opendev.org/748512
17:56:53 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: Add regression tests for bug #1889108 https://review.opendev.org/748513
17:56:53 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: compute: Don't delete the original attachment during pre LM rollback https://review.opendev.org/748514
17:56:54 openstack bug 1889108 in OpenStack Compute (nova) stein "failures during driver.pre_live_migration remove source attachments during rollback" [Undecided,In progress] https://launchpad.net/bugs/1889108 - Assigned to Lee Yarwood (lyarwood)
18:22:30 openstackgerrit Mamduh proposed openstack/os-vif master: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/746673
19:28:25 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059
19:47:49 openstackgerrit Rodrigo Barbieri proposed openstack/nova master: Allow tap interface with multiqueue https://review.opendev.org/748533
19:49:46 openstackgerrit Ghanshyam Mann proposed openstack/nova master: [Trivial] Replace ref of policy.json to policy.yaml https://review.opendev.org/749821
19:57:59 openstackgerrit Merged openstack/nova master: Adapt the admin guide to describe the direct RBD export https://review.opendev.org/728411
20:54:22 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP/DNM libvirt: Increase device detach wait times https://review.opendev.org/749827
21:17:46 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059
21:18:33 openstackgerrit Ghanshyam Mann proposed openstack/nova master: [Trivial] Replace ref of policy.json to policy.yaml https://review.opendev.org/749821
22:06:08 openstackgerrit Sean McGinnis proposed openstack/nova master: Handle oslo.messaging ping endpoint https://review.opendev.org/749838
22:07:21 smcginnis Needed for the global requirements update patch for the latest oslo.messaging ^

Earlier   Later