| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-29 | |||
| 11:33:22 | sean-k-mooney | stephenfin: no they dont | |
| 11:33:36 | stephenfin | I'd ask elodilles directly but he's on PTO | |
| 11:34:00 | sean-k-mooney | it ptl or whoever is listed here https://github.com/openstack/releases/blob/master/data/release_liaisons.yaml#L93-L96 | |
| 11:35:08 | stephenfin | ah | |
| 11:35:10 | stephenfin | sweet | |
| 11:35:17 | stephenfin | then I guess you can do it :) | |
| 11:35:38 | sean-k-mooney | im just looking at the changes between the current sha and the old one | |
| 11:38:01 | sean-k-mooney | stephenfin: we might end up doing another release if we land some of the inflight changes | |
| 11:38:11 | sean-k-mooney | but i have no issue with doing this release now | |
| 11:38:14 | stephenfin | that's okay, I think? | |
| 11:38:24 | sean-k-mooney | https://review.opendev.org/c/openstack/releases/+/854362 | |
| 11:38:35 | sean-k-mooney | ptl-approved flag is no set | |
| 11:38:45 | sean-k-mooney | ya its ok | |
| 11:38:53 | sean-k-mooney | release are cheap | |
| 11:39:16 | sean-k-mooney | i assume this is needed for ocs change to merge or simialr | |
| 11:39:59 | sean-k-mooney | i know those micorversion also had osc patches | |
| 11:42:12 | sean-k-mooney | stephenfin: anyway let me know if you need release for anything else in the nova deliverables or ping bauzas tomorrow | |
| 11:42:23 | sean-k-mooney | if im not about | |
| 11:50:00 | gibi | I'm not sure we have rel mgmt up yet as elodilles_pto is on pto today | |
| 11:51:09 | sean-k-mooney | even if we do im not sure if we have two of them | |
| 11:51:22 | sean-k-mooney | given its still pto season | |
| 11:51:27 | gibi | yeah | |
| 11:53:09 | sean-k-mooney | hum https://review.opendev.org/c/openstack/nova/+/851445 helped but i sill have 74 failing tests | |
| 11:53:27 | sean-k-mooney | AttributeError: 'VGPUTests' object has no attribute 'libvirt' | |
| 11:53:38 | sean-k-mooney | so the libvirt fixutre is still not quite working | |
| 11:53:41 | gibi | hm, that is still somehow related to the mock stuf | |
| 11:54:26 | gibi | have you looked at https://review.opendev.org/c/openstack/nova/+/852464 | |
| 11:54:48 | sean-k-mooney | nope bug that looks relevent | |
| 11:55:01 | sean-k-mooney | *but | |
| 11:55:09 | sean-k-mooney | ill pull that in and see if it resolves the issue | |
| 11:55:21 | sean-k-mooney | i wonder if i should squash the too for backport or not | |
| 11:55:30 | gibi | probalby OK to squash | |
| 11:55:58 | sean-k-mooney | ill try adding them seperatly and see if that resovles the issue then i can squash | |
| 11:56:07 | gibi | ack | |
| 11:59:16 | sean-k-mooney | im not actully backporting the mock removal by the way | |
| 11:59:24 | sean-k-mooney | just the double mocking patch | |
| 11:59:32 | sean-k-mooney | so stable will still use mock the lib | |
| 11:59:36 | sean-k-mooney | but without the double mocking | |
| 12:00:09 | gibi | hm, I think that is OK | |
| 12:03:17 | sean-k-mooney | looks like ill need https://opendev.org/openstack/nova/commit/cd03bbc1c33e33872594cf002f0e7011ab8ea047 too | |
| 12:04:39 | sean-k-mooney | that should be ok to backport too | |
| 12:05:14 | sean-k-mooney | thats needed for fake_get_mac_by_pci_address | |
| 12:05:39 | sean-k-mooney | that bug has existied since pf support was added so backporting to wallaby is fine | |
| 12:05:40 | gibi | ahh yeah, that was en extension to the fixture | |
| 12:05:59 | sean-k-mooney | ya i could just pull that in but if this does not cause much issue ill just backport it | |
| 12:06:01 | gibi | but for the MAC update to work we need https://review.opendev.org/c/openstack/neutron/+/829247 on the neutron side too | |
| 12:06:08 | sean-k-mooney | yes | |
| 12:06:15 | gibi | this is getting hairy :) | |
| 12:06:17 | sean-k-mooney | but having the mac info wont break anything | |
| 12:06:22 | gibi | true | |
| 12:06:29 | sean-k-mooney | so we can also backport the neutron patch but its not a blocker | |
| 12:06:30 | gibi | then fine | |
| 12:06:35 | sean-k-mooney | which is why you did not have a depens on | |
| 12:06:57 | sean-k-mooney | if this proves unworkable ill pull in just the test fixture changes | |
| 12:07:12 | gibi | ack | |
| 12:07:14 | gibi | make sense | |
| 12:58:32 | sean-k-mooney | gibi: ok i have a working backport series i need to add cherrypicked lines since i pull them in via interactive rebase to get the order right but ill push them and then do that | |
| 12:58:52 | gibi | cool | |
| 12:59:10 | opendevreview | sean mooney proposed openstack/nova stable/yoga: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/855022 | |
| 12:59:11 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/855023 | |
| 12:59:12 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Remove double mocking https://review.opendev.org/c/openstack/nova/+/855024 | |
| 12:59:13 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/855025 | |
| 12:59:14 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/855026 | |
| 12:59:39 | sean-k-mooney | i was tempted to pull in the vdpa change to stop using sriov | |
| 13:00:00 | sean-k-mooney | but im not sure if that was backportable | |
| 13:00:05 | sean-k-mooney | so i didnt add it | |
| 13:08:35 | gibi | OK I think I finished looking at the latest update of the manila series. what was fixed looks correct to me, but there is still open comment. I know Uggla working on them as we speak | |
| 13:10:13 | gibi | I'm going to switch back to the PCI series adding unit tests. But I will keep an eye of incoming updates on user_data, bfv rebuild and manial | |
| 13:10:24 | gibi | manila | |
| 13:10:26 | gibi | serieses | |
| 13:10:38 | opendevreview | sean mooney proposed openstack/nova stable/yoga: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/855022 | |
| 13:10:39 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/855023 | |
| 13:10:40 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Remove double mocking https://review.opendev.org/c/openstack/nova/+/855024 | |
| 13:10:42 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/855025 | |
| 13:10:44 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/855026 | |
| 13:11:22 | sean-k-mooney | gibi: ack ^ those can wait until after FF but if you get time to look over them many were your patches orginally | |
| 13:11:53 | gibi | sure that is a nice distraction for my afternoon coffee :) | |
| 13:12:06 | gibi | was there any major merge conflict there? | |
| 13:12:33 | sean-k-mooney | no not really. i had to decide when to stop pulling in patches to avoid them | |
| 13:12:45 | sean-k-mooney | but most were trivia | |
| 13:13:19 | Uggla | gibi, thank you for the review. Although I have a lot of work and corrections to do. ;) | |
| 13:13:46 | gibi | Uggla: no worries. FF week is always a bit crazy :D | |
| 13:14:35 | gibi | sean-k-mooney: ack | |
| 13:19:26 | sean-k-mooney | gibi: very odd | |
| 13:19:39 | sean-k-mooney | all the reference commits are not wrong but i copied them from gerrit | |
| 13:19:46 | sean-k-mooney | so i dont knwo why they are wrong | |
| 13:20:31 | gibi | maybe you copied from an older PS? | |
| 13:20:51 | sean-k-mooney | actuly they are not wrong https://opendev.org/openstack/nova/commit/7bba330a1e6c09564b27bbbd85f42519c5b1598e | |
| 13:21:59 | sean-k-mooney | https://opendev.org/openstack/nova/commit/8d2776fb34339b311c713992a39507452c4ae42f | |
| 13:22:02 | sean-k-mooney | they both exist | |
| 13:22:22 | sean-k-mooney | the one i have listed is the merged one | |
| 13:22:33 | sean-k-mooney | the other wone is the one that came form the review | |
| 13:22:49 | sean-k-mooney | gerrit rebases it when its applying it to the repo | |
| 13:23:12 | sean-k-mooney | ill see what the bot says when it runs | |
| 13:23:23 | sean-k-mooney | and ill either try fixing them again or leave them as is | |
| 13:23:49 | sean-k-mooney | i was trying to use the commit form the gitia link | |
| 13:24:44 | sean-k-mooney | if i need too ill just cherry-pick them all again form a clean start | |
| 13:24:56 | sean-k-mooney | i might jsut do that now | |
| 13:26:05 | sean-k-mooney | git review will do it the way the check script expects | |
| 13:29:54 | gibi | yeah the only downside of the current hashes that they are not bringig up the original patch in gerrit ui | |