| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-03 | |||
| 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 ^ | |
| #openstack-nova - 2020-09-04 | |||
| 00:45:21 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059 | |
| 00:45:36 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: [Trivial] Replace ref of policy.json to policy.yaml https://review.opendev.org/749821 | |
| 06:09:15 | openstackgerrit | Brin Zhang proposed openstack/nova master: Refactor check and exception https://review.opendev.org/749052 | |
| 06:09:40 | openstackgerrit | Wenping Song proposed openstack/nova master: Refactor check and exception https://review.opendev.org/749052 | |
| 07:35:13 | openstackgerrit | Merged openstack/nova master: Handle oslo.messaging ping endpoint https://review.opendev.org/749838 | |
| 07:46:27 | bauzas | good Friday Nova | |
| 07:56:59 | noonedeadpunk | o/ | |
| 07:58:19 | noonedeadpunk | I can recall that I saw somewhere that nova for libvirt driver during resize creates new instance with new flavor and does live migration to it. Was I dreaming or is it real thing? | |
| 08:01:18 | brinzhang0 | bauzas: good moning | |
| 08:01:40 | bauzas | noonedeadpunk: huh ? | |
| 08:01:45 | brinzhang0 | bauzas: how to filters bug fix in Ussuri release in launchpad? | |
| 08:01:59 | bauzas | brinzhang0: good afternoon | |
| 08:02:01 | bauzas | sec | |
| 08:02:26 | bauzas | brinzhang0: you want to know what was merged as a bugfix in the Ussuri cycle ? | |
| 08:02:36 | brinzhang0 | bauzas: yes | |
| 08:03:08 | brinzhang0 | I want to know which and how many bugs we fixed in Ussuri | |
| 08:03:08 | noonedeadpunk | bauzas: sounds like I was dreaming:) | |
| 08:03:45 | brinzhang0 | bauzas: I tried in launchpad by some way, but I can't get what I want :( | |
| 08:03:52 | bauzas | brinzhang0: you can't really see them by using Launchpad | |
| 08:04:08 | bauzas | brinzhang0: you can look at all the closed bugs by https://bugs.launchpad.net/nova/?field.searchtext=&field.status%3Alist=FIXRELEASED&assignee_option=any&field.assignee=&field.bug_reporter=&field.bug_commenter=&field.subscriber=&field.structural_subscriber=&field.tag=&field.tags_combinator=ANY&field.has_cve.used=&field.omit_dupes.used=&field.omit_dupes=on&field.affects_me.used=&field.has_patch.used=&field.has_branches.used=&field | |
| 08:04:08 | bauzas | .has_branches=on&field.has_no_branches.used=&field.has_no_branches=on&field.has_blueprints.used=&field.has_blueprints=on&field.has_no_blueprints.used=&field.has_no_blueprints=on&search=Search&orderby=-datecreated&start=0 | |
| 08:04:15 | bauzas | oh sec | |
| 08:04:55 | bauzas | brinzhang0: but maybe the better is just to use a gerrit query | |
| 08:05:38 | bauzas | brinzhang0: shorturl.at/cdCEQ | |
| 08:05:48 | bauzas | http://shorturl.at/cdCEQ | |
| 08:06:14 | bauzas | but like I said, a Gerrit query is better | |
| 08:06:31 | brinzhang0 | bauzas: I using https://bugs.launchpad.net/nova/+bugs?advanced=1 to filter, but nothing can get | |
| 08:07:23 | bauzas | brinzhang0: lemme provide you the Gerrit query | |
| 08:07:30 | bauzas | it's simple | |
| 08:07:38 | brinzhang0 | ack | |