| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-18 | |||
| 10:39:23 | kashyap | So that's correct based on what was there at that time | |
| 10:39:36 | zigo | Right. | |
| 10:39:53 | kashyap | Yes, it is outdated. Updating these MediaWiki pages are a *pain* in the neck | |
| 10:40:16 | zigo | Yeah, the syntax is ... hum... suboptimal, to use nice words! :) | |
| 10:41:20 | kashyap | zigo: Refresh the page, and check if the Buster ("stable") looks correct now | |
| 10:42:13 | zigo | kashyap: Well, it looks ok to me, though instead of Buster ("sid") you may want to write Bullseye ("sid") and write what's there. | |
| 10:43:55 | kashyap | zigo: Right, let me do that | |
| 10:44:10 | kashyap | I'll update the Buster ("sid") --> Bullseye ("sid") | |
| 10:45:36 | kashyap | zigo: Am I correct in saying that what's at top-left (the 4.1.1) indicates the Bullseye ("sid") version? -- https://tracker.debian.org/pkg/qemu | |
| 10:45:45 | kashyap | Also what does the "1:" mean? | |
| 10:46:26 | zigo | kashyap: That's an "EPOCH".... basically, this is for when we do a mistake with the upstream version, and need to "upgrade" to a lower version. | |
| 10:46:34 | zigo | So, that's distro specific ... | |
| 10:46:40 | kashyap | Ah, right. The same concept exists in Fedora. | |
| 10:46:50 | zigo | 1:2.3.4 is higher that 2.3.5 | |
| 10:46:51 | kashyap | Much frowned upon, though. :-) | |
| 10:47:05 | kashyap | Right. | |
| 10:47:18 | zigo | We try to avoid it as much as possible, because that's very confusing when writting dependency, and a mistake may happen very fast... | |
| 10:47:35 | kashyap | Yeah, it's an emergency hammer | |
| 10:47:53 | zigo | For example, if you want version >= 2.3.5 and forget the 1:, then you get 1:2.2.4 satisfying your dependency and you're screwed. | |
| 10:48:57 | zigo | Does anyone know if Nova fixed the issue of volume live migration in Rocky, and if yes, in what point release? | |
| 10:49:14 | zigo | I've tried upgrading os-brick to 2.5.8 in a compute, it didn't help... | |
| 10:49:21 | zigo | I'll try nova and cinder next... | |
| 10:50:02 | kashyap | zigo: Can you refresh the page to now check if Debian stuff is all kosher? | |
| 10:50:28 | zigo | kashyap: LGTM ! :) | |
| 10:51:07 | kashyap | stephenfin: Any ideas to transfer this to a better place than MediaWiki? -- can we do proper tables in rST? | |
| 10:51:14 | kashyap | https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | |
| 10:51:19 | kashyap | The MediaWiki syntax sucks | |
| 10:51:55 | kashyap | We _can_ do tables in rST; I've done it myself. Perhaps will consider moving this page in-tree | |
| 11:04:06 | openstackgerrit | Merged openstack/nova master: Convert legacy nova-live-migration and nova-multinode-grenade to py3 https://review.opendev.org/692374 | |
| 11:12:20 | zigo | Ok, I found it, the issue is in Nova, and fixed by the point release in Rocky. | |
| 11:37:40 | stephenfin | kashyap: Yeah, that probably makes sense in doc/source/reference | |
| 11:37:47 | stephenfin | and you should be able to do it in tables, yup | |
| 11:40:58 | kashyap | stephenfin: Thanks! I'll get to it. | |
| 11:49:20 | zigo | Does anyone know which commit has the cinder volume live migration fix in Rocky? | |
| 11:50:06 | lyarwood | zigo: That's pretty generic, do you have a launchpad bug refrence I could check? | |
| 11:51:19 | zigo | lyarwood: Basically, in 18.1.0, live migrating a cinder volume never finshes, with 18.2.3 that's fixed... | |
| 11:51:33 | zigo | I don't know about any bug number though, I just tested and saw it was fixed. | |
| 11:51:47 | zigo | I'm trying to lookup commits now ... | |
| 11:52:58 | lyarwood | zigo: ah retyping a volume? | |
| 11:53:31 | zigo | lyarwood: Just live-migrating a volume: | |
| 11:53:31 | zigo | openstack volume migrate --host <volume-hostname> <volume-id> | |
| 11:53:37 | lyarwood | zigo: I landed https://review.opendev.org/#/q/I1bdf3431bda2da98380e0dcaa9f952e6768ca3af to fix that | |
| 11:53:42 | zigo | In 18.1.0, the new volume is re-attached ... | |
| 11:54:17 | zigo | ... but the old one never deleted (and both new and old are still in the bad migration status) | |
| 11:54:46 | lyarwood | ack yeah that came in with 18.2.1 | |
| 11:55:07 | zigo | lyarwood: Thanks, this looks like it. | |
| 11:55:42 | zigo | lyarwood: FYI, I need to have the exact patch, so I get this fast in Buster (upgrading to 18.2.3 has a lot of chances to be unfortunately rejected by the Debian stable release team...) | |
| 11:56:03 | lyarwood | https://review.opendev.org/#/c/657577/ is the change | |
| 11:56:19 | zigo | Yeah, got it, thanks a lot. | |
| 12:00:27 | zigo | lyarwood: I need that one too, right? | |
| 12:00:29 | zigo | https://review.opendev.org/#/c/639331/ | |
| 12:00:39 | zigo | Ah no, that's for tempest ... | |
| 12:04:47 | lyarwood | zigo: yeah you don't technically need that, it will however ensure things are tested correctly if Debian stable has CI for OpenStack | |
| 12:05:17 | zigo | lyarwood: Yeah, I may also patch the tempest package... :) | |
| 12:22:15 | openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other that nova-compute https://review.opendev.org/694756 | |
| 12:27:49 | openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/694756 | |
| 12:28:54 | openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/694756 | |
| 12:43:46 | efried | o/ nova | |
| 12:43:46 | efried | I'm going to be out today. | |
| 12:43:46 | efried | Enjoy. | |
| 13:00:06 | gibi | efried_pto: enjoy your day too | |
| 14:23:50 | mriedem | i wonder, maybe we should drop using the neutron-grenade-multinode job which is 2 node grenade + smoke tests. we already have nova-grenade-multinode which is just live migration tests in both a block storage + lvm and shared storage + ceph setup. if we changed nova-grenade-multinode to run smoke tests + live migration for the block storage + lvm case, before changing the env to ceph, then we'd cover what the neutron-grenade-m | |
| 14:23:50 | mriedem | node job was doing | |
| 14:24:15 | mriedem | oh nova-grenade-multinode also runs a cold migrate test since train | |
| 14:27:15 | sean-k-mooney | mriedem: that sounds resonable. have you looked at https://review.opendev.org/#/c/548936/ it woudl be good to eventually get that merged to have a native grenade job | |
| 14:27:58 | mriedem | nope i haven't. it's huge and it scares me. | |
| 14:29:55 | sean-k-mooney | it is, it does appear to work at least. that said the last ci run was july so it might not anymore | |
| 14:34:43 | mriedem | grenade-core includes devstack-core and the devstack / qa team have done a lot of the initial zuulv3-ification of stuff, starting with devstack and tempest, so i'll defer to them for reviewing that massive zuulv3 chnage | |
| 14:34:47 | mriedem | *change | |
| 14:38:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: FUP to Ie1a0cbd82a617dbcc15729647218ac3e9cd0e5a9 https://review.opendev.org/694787 | |
| 14:50:24 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Drop neutron-grenade-multinode job https://review.opendev.org/694789 | |
| 14:57:36 | artom | If you do a Follow Up Patch to a Follow Up Patch, is it a Follow Up Patch Addendum? | |
| 15:07:26 | mriedem | so making nova-grenade-multinode run smoke tests adds about 14-15 minutes to the job unless we prune down the smoke tests that it runs which we might want to consider doing - to just run compute API and scenario smoke tests, though then we lose some of the coverage that neutron-grenade-multinode gave us, | |
| 15:07:44 | mriedem | though a nova change likely isn't going to break an identity/image/object/network/volume API change that doesn't use nova at all | |
| 15:19:21 | mriedem | i would need some help on the regexing though.. | |
| 15:20:15 | mriedem | artom: you like a good regex test right? | |
| 15:20:20 | mriedem | say i have a test string of: tempest.api.compute.flavors.test_flavors.FlavorsV2TestJSON.test_get_flavor[id-1f12046b-753d-40d2-abb6-d8eb8b30cb2f,smoke] | |
| 15:20:40 | mriedem | i want to match tempest.api.compute.* and tempest.scenario.* but only for those strings containing 'smoke' | |
| 15:21:12 | mriedem | i think (tempest.api.compute.*|tempest.scenario.*)([\bsmoke\b]) might work? | |
| 15:21:58 | artom | mriedem, I think aspiers was the regex master, but I'll try | |
| 15:22:36 | artom | mriedem, that should work | |
| 15:23:19 | artom | I'd have done it as 'tempest\.(api\.compute|scenario)\..*smoke' | |
| 15:25:19 | mriedem | yeah that seems to work as long as smoke comes last | |
| 15:26:02 | mriedem | well, i guess there is no $ on the regex so i guess it's ok | |
| 15:26:23 | artom | So 'tempest\.(api\.compute|scenario)\..*smoke.*' then :) | |
| 15:26:25 | mriedem | tempest.(api.compute|scenario)\..*smoke.* | |
| 15:26:29 | mriedem | yeah | |
| 15:36:34 | sean-k-mooney | bauzas: could you take a look at https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1763766 ideally im aiming to get this merged before m1 | |
| 15:37:20 | sean-k-mooney | stephenfin: similar for https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/vm-scoped-sriov-numa-affinity if i can get that merged before m1 it would make peopel happy so if you could review it would be nice before i start asking others | |
| 15:59:02 | bauzas | sean-k-mooney: ack, will look | |
| 16:10:13 | openstackgerrit | Mark Goddard proposed openstack/nova master: [ironic] Fix node rebalance race issues https://review.opendev.org/694802 | |
| 16:14:21 | mriedem | mgoddard: hi https://review.opendev.org/#/c/684849/ | |
| 16:14:38 | kashyap | Can someone update the openSUSE stuff here? -- https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | |
| 16:14:46 | kashyap | (I've updated for Debian, Ubuntu, RHEL and Fedora.) | |
| 16:14:58 | kashyap | Who's the contact point for Oracle Linux again? | |
| 16:15:46 | mgoddard | mriedem: thanks, I'll check it out | |
| 16:18:31 | artom | kashyap, probably Satan | |
| 16:19:09 | artom | Dammit >< | |
| 16:19:10 | kashyap | artom: Yeah, c'mon | |
| 16:22:07 | mgoddard | mriedem: it could be because I'm using Rocky, but I don't see any errors in RT following rebalance | |