| *** tkajinam is now known as Guest162 | 06:33 | |
| *** mgoddard- is now known as mgoddard | 08:11 | |
| Uggla | sean-k-mooney[m], can you please recall me the link to your devstack installation tool using ansible + molecule ? | 08:24 |
|---|---|---|
| whoami-rajat | hi sean-k-mooney , can you take a look at the comment here? basically stephenfin wants to rename the parameter to reimage-boot-volume and add a negative parameter along with it https://review.opendev.org/c/openstack/python-openstackclient/+/831014/7/openstackclient/compute/v2/server.py#3095 | 09:39 |
| zigo | I'm reading Nova release notes for Zed, and saw the thingy about hw:locked_memory. I wonder: what's the point of this feature? In which case one would want the memory to be locked, and to achieve what? Is this a security feature? | 10:32 |
| sean-k-mooney | zigo: o/ | 10:36 |
| zigo | :) | 10:37 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Adds check if blk_dev_info has correct falvor.swap https://review.opendev.org/c/openstack/nova/+/857339 | 10:37 |
| sean-k-mooney | zigo: it was added for the viommu feature not because it requires it but on some aarch64 systems we needed to lock the memoy so that the acclerator could properly do a dma transfer | 10:37 |
| sean-k-mooney | on x86 in general you dont need to do it at least when booting with pci device passhtough libvit will lock enough for the inital devices | 10:38 |
| zigo | Oh, I see... So that's related to HW accelleration. Makes sense. | 10:38 |
| sean-k-mooney | zigo: it was already used implictly when you enabled realtime cpus | 10:39 |
| sean-k-mooney | yep locking the memory marks it as unswappable and unmovable | 10:39 |
| sean-k-mooney | most of the time you do not need that kind of guarentee | 10:44 |
| sean-k-mooney | but if you do now you can | 10:44 |
| sean-k-mooney | anything else catch your eye? | 10:44 |
| opendevreview | Konrad Gube proposed openstack/nova-specs master: Add API for assisted volume extend https://review.opendev.org/c/openstack/nova-specs/+/855490 | 12:25 |
| opendevreview | Bence Romsics proposed openstack/nova stable/victoria: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/857423 | 12:42 |
| opendevreview | Bence Romsics proposed openstack/nova stable/ussuri: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/857427 | 12:58 |
| *** dasm|off is now known as dasm | 13:21 | |
| opendevreview | Konrad Gube proposed openstack/nova-specs master: Add API for assisted volume extend https://review.opendev.org/c/openstack/nova-specs/+/855490 | 13:30 |
| opendevreview | Konrad Gube proposed openstack/nova-specs master: Add API for assisted volume extend https://review.opendev.org/c/openstack/nova-specs/+/855490 | 14:23 |
| bauzas | reminder : nova meeting in 43 mins | 15:17 |
| bauzas | here* | 15:17 |
| elodilles | bauzas: i'll do a quick update now to the stable section if you are not editing right now the agenda on the meeting page | 15:33 |
| bauzas | elodilles: do it now, I was working on the prelude change so I didn't had time yet to edit it | 15:34 |
| elodilles | bauzas: ack, thanks, done | 15:39 |
| bauzas | elodilles: ok, will do it now | 15:39 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Prelude section for Zed release https://review.opendev.org/c/openstack/nova/+/857467 | 15:42 |
| gibi | o/ | 16:00 |
| elodilles | o/ | 16:00 |
| gibi | heh | 16:00 |
| gibi | :) | 16:00 |
| Uggla | o/ | 16:00 |
| sean-k-mooney | o/ | 16:00 |
| gibi | we are too fast | 16:00 |
| bauzas | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Sep 13 16:00:38 2022 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'nova' | 16:00 |
| bauzas | heh | 16:00 |
| dansmith | o/ | 16:00 |
| bauzas | sorry, was finishing to update the agenda :) | 16:00 |
| bauzas | you can raise hands if you want tho | 16:01 |
| bauzas | or meh | 16:01 |
| bauzas | bonjour everyone | 16:01 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:01 |
| bauzas | ok, let's start, I'm pretty sure people who started earlier are not already gone | 16:01 |
| bauzas | #topic Bugs (stuck/critical) | 16:01 |
| bauzas | #info One Critical bug | 16:02 |
| bauzas | #link https://bugs.launchpad.net/nova/+bug/1988311 Now u-c updated https://review.opendev.org/c/openstack/requirements/+/856044 shall we set to High or close the bug ? | 16:02 |
| bauzas | #link https://bugs.launchpad.net/nova/+bug/1988311 Now u-c updated https://review.opendev.org/c/openstack/requirements/+/856044 shall we set to High or close the bug ? | 16:02 |
| bauzas | so, | 16:02 |
| bauzas | oslo.concurrency now is released with 5.0.1 | 16:02 |
| bauzas | upper-constraints are now updated | 16:02 |
| bauzas | #link https://review.opendev.org/c/openstack/requirements/+/856044 | 16:02 |
| bauzas | I verified and both stable/yoga and master ask for oslo.concurrency older than 4.5.0 | 16:03 |
| bauzas | https://review.opendev.org/c/openstack/requirements/+/856044 | 16:03 |
| sean-k-mooney | if the locks are now fixed with the oslo release we could clsoe but we need to raise our min verion | 16:03 |
| bauzas | so technically, I haven't checked but our gate uses the latest version | 16:03 |
| sean-k-mooney | otherwise the problem still exstis and we might need to work around it on stable branches in nvoa | 16:04 |
| bauzas | sean-k-mooney: correct, that's my point | 16:04 |
| sean-k-mooney | gibi i assume we will backprot the eventlet fix in oslo | 16:04 |
| bauzas | so we need to bump our mins for both yoga and master | 16:04 |
| sean-k-mooney | to stable | 16:04 |
| sean-k-mooney | we cant bump mins on stable | 16:04 |
| sean-k-mooney | but we need to backport and ensure its allwoed by upper constratis | 16:04 |
| sean-k-mooney | or we need to patch the lock in nova | 16:05 |
| sean-k-mooney | and backprot that | 16:05 |
| gibi | I assume we backport the fix in oslo | 16:05 |
| sean-k-mooney | that would be my prefernce too | 16:05 |
| gibi | but I did not have time to propose that | 16:05 |
| bauzas | ok, at least I propose to set the bug to High | 16:05 |
| bauzas | the gate isn't longer impact, our distros are tho | 16:05 |
| sean-k-mooney | ack | 16:05 |
| bauzas | impacted* | 16:05 |
| sean-k-mooney | well its cloased for master | 16:05 |
| sean-k-mooney | and high for the older brnaches | 16:06 |
| bauzas | not in the Nova project | 16:06 |
| bauzas | I can set a branch series for the bug | 16:06 |
| sean-k-mooney | you want to keep it up for the min version bump | 16:06 |
| bauzas | lemme do it | 16:06 |
| bauzas | correct | 16:06 |
| gibi | ack | 16:06 |
| bauzas | so, | 16:06 |
| bauzas | set to High, open a Yoga series to the bug report and set it to high too | 16:06 |
| bauzas | and ask for the bump in master and the backport in Yoga | 16:07 |
| bauzas | wfy ? | 16:07 |
| sean-k-mooney | +1 | 16:07 |
| bauzas | ok, let me do the paperwork | 16:07 |
| sean-k-mooney | well backport in oslo for yoga but yes | 16:07 |
| gibi | looks OK | 16:07 |
| bauzas | done | 16:08 |
| bauzas | moving on | 16:08 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 5 new untriaged bugs (-4 since the last meeting) | 16:08 |
| bauzas | kudos to Uggla for this excellent work | 16:09 |
| bauzas | #link https://storyboard.openstack.org/#!/project/openstack/placement 26 open stories (+0 since the last meeting) in Storyboard for Placement | 16:09 |
| bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:09 |
| bauzas | Uggla: do you want to discuss any bug ? | 16:09 |
| Uggla | bauzas, yes | 16:09 |
| Uggla | bauzas, https://bugs.launchpad.net/nova/+bug/1896617 seems valid for me. I have triaged it. | 16:10 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min oslo.concurrencty to >= 5.0.1 https://review.opendev.org/c/openstack/nova/+/857491 | 16:10 |
| Uggla | But I would like your opinion | 16:10 |
| bauzas | shit | 16:11 |
| bauzas | yeah, if we set the perms | 16:12 |
| sean-k-mooney | hum | 16:14 |
| sean-k-mooney | so right now this sound like an rfe to me | 16:14 |
| sean-k-mooney | or at least its a new restriction how what nova ia and is not allwoed to assume | 16:15 |
| sean-k-mooney | normally we assume that libvirt and nova can share a group | 16:15 |
| sean-k-mooney | we do not document that you can harden in this way | 16:16 |
| sean-k-mooney | so to me its not explcitly supported | 16:16 |
| bauzas | sorry, had a doorbell | 16:17 |
| bauzas | I'm back | 16:18 |
| bauzas | yeah, so maybe the bug is valid as a Wishlist ? | 16:18 |
| bauzas | sean-k-mooney: agreed ? | 16:19 |
| sean-k-mooney | i dont think its a bug | 16:19 |
| sean-k-mooney | we could have it be whishlist yes | 16:20 |
| sean-k-mooney | as a mini feature | 16:20 |
| sean-k-mooney | im writign a responce | 16:20 |
| sean-k-mooney | but i think the aswer is to | 16:20 |
| bauzas | cool, can we move on then ? | 16:20 |
| sean-k-mooney | chown nova:$libvirt-group | 16:20 |
| sean-k-mooney | and then libvirt gets read via the group | 16:20 |
| bauzas | if you really think this is a feature, we can set it to Invalid/Wishlist and ask for a blueprint to be filled | 16:20 |
| bauzas | but that's tough for reporters | 16:20 |
| bauzas | sean-k-mooney: yeah, ideally the group shall be libvirt but this sounds distro-oriented | 16:21 |
| sean-k-mooney | ya i think so to as its is not a documented deployment model we support | 16:21 |
| bauzas | problem is, we set perms | 16:21 |
| sean-k-mooney | well it the "libvirt group" but that is disto dependet | 16:21 |
| bauzas | so we're opiniated | 16:21 |
| sean-k-mooney | we can have that be a config option | 16:22 |
| bauzas | sean-k-mooney: yeah that's my point, this is distro-specific | 16:22 |
| bauzas | I honestly don't like us to be opinionated about the perms we shall set | 16:22 |
| bauzas | so, yeah a config knob seems maybe the best | 16:22 |
| bauzas | or... I dunno | 16:22 |
| bauzas | shall we just assume the tempdir is readable by anyone ? | 16:23 |
| sean-k-mooney | we have some requirment around this for vhost-user by the way | 16:23 |
| bauzas | not by *anyone* | 16:23 |
| bauzas | but at least should we just document this is distro-specific and us having a requirement that the directory you set in nova.conf has right perms accordingly? | 16:24 |
| bauzas | and remove this chmod in the code | 16:24 |
| sean-k-mooney | well maybe not | 16:24 |
| bauzas | this would require an upgrade relnote so a spec tho | 16:24 |
| sean-k-mooney | i would have to look at the code but we can take this offline after the meeting | 16:25 |
| bauzas | I don't like the idea of nova managing the underlying OS | 16:25 |
| bauzas | cool | 16:25 |
| Uggla | if I'm not wrong we are setting o+r, I don't really get the diff if we add o+g too. | 16:25 |
| bauzas | Uggla: the point is that I don't see why Nova should do it | 16:25 |
| sean-k-mooney | bauzas: | 16:26 |
| Uggla | but it is already in the driver.py. | 16:26 |
| bauzas | shalln't we just assume the directory are correctly readable by the required users ? | 16:26 |
| sean-k-mooney | we assume tha tnova is in the libvirt group so that we can read the disk to upload the snapshot | 16:26 |
| sean-k-mooney | in there case nova si not in the libvirt-qemu group i think | 16:26 |
| sean-k-mooney | althoguh they apprently worked around this by addign libvirt-qemu user ot nova group | 16:27 |
| sean-k-mooney | which is backwards | 16:27 |
| bauzas | let's put this bug as Opinion then | 16:28 |
| bauzas | this is half distro-specific, half a nova problem | 16:28 |
| bauzas | and people wanting to discuss the resolution are more than happy to engage later on after the meeting | 16:29 |
| bauzas | time flies and we're at the half of the meeting | 16:29 |
| bauzas | agreed ? | 16:29 |
| sean-k-mooney | ack | 16:29 |
| Uggla | ack | 16:30 |
| bauzas | moving on | 16:31 |
| gibi | works for me | 16:31 |
| bauzas | sean-k-mooney: do you have time to do some upstream bug triage this week ? | 16:31 |
| bauzas | keeping in mind this is best-effort | 16:31 |
| sean-k-mooney | i guess so | 16:31 |
| Uggla | bauzas, fyi there are only 4 new bugs remaining now. | 16:32 |
| bauzas | \o/ | 16:32 |
| bauzas | sean-k-mooney: thanks | 16:32 |
| bauzas | and again, if you can't, no worries | 16:32 |
| bauzas | #info bug baton is being passed to sean-k-mooney | 16:34 |
| bauzas | damn | 16:34 |
| bauzas | #info bug baton is being passed to sean-k-mooney | 16:34 |
| bauzas | #topic Gate status | 16:34 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:34 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fplacement&pipeline=periodic-weekly Placement periodic job status | 16:34 |
| bauzas | #link https://zuul.openstack.org/builds?job_name=tempest-integrated-compute-centos-9-stream&project=openstack%2Fnova&pipeline=periodic-weekly Centos 9 Stream periodic job status | 16:34 |
| bauzas | #link https://zuul.opendev.org/t/openstack/builds?job_name=nova-emulation&pipeline=periodic-weekly&skip=0 Emulation periodic job runs | 16:34 |
| bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:35 |
| bauzas | #info STOP DOING BLIND RECHECKS aka. 'recheck' https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures | 16:35 |
| bauzas | all runs are green, nothing to say | 16:35 |
| bauzas | moving on ? | 16:35 |
| bauzas | looks so | 16:36 |
| bauzas | #topic Release Planning | 16:36 |
| bauzas | #link https://releases.openstack.org/zed/schedule.html | 16:36 |
| bauzas | #info RC1 is this Thursday | 16:36 |
| bauzas | this is important ^ | 16:36 |
| sean-k-mooney | yep branches will be cut once rc1 is out | 16:36 |
| bauzas | as a reminder, we'll branch stable/zed from this point in time | 16:36 |
| gibi | I've just pushed the oslo.concurrency min bump https://review.opendev.org/c/openstack/nova/+/857491 that is important for RC! | 16:36 |
| gibi | RC1 | 16:37 |
| bauzas | gibi: you give me a perfect opportunity for the next item | 16:37 |
| bauzas | as a reminder too, after RC1 and before GA, we will only be able to accept stable/zed backports if and only if those are regression bugfixes | 16:38 |
| bauzas | I have a couple of bugs waiting for reviews that help our vgpu state of art, but let's defer them to Antelope | 16:38 |
| bauzas | s/bugs/bugfixes/ | 16:38 |
| bauzas | so those 2 days are crucial for any bugfix requring to be merged | 16:39 |
| bauzas | accordingly, I created an etherpad for tracking | 16:39 |
| bauzas | #link https://etherpad.opendev.org/p/nova-zed-rc-potential Zed RC tracking etherpad | 16:39 |
| * sean-k-mooney man we are expeded to write bugfixes and bugs so un reasonable :P | 16:39 | |
| bauzas | this etherpad will be used for tracking RC1 prep, and other RCs until Zed GA | 16:39 |
| sean-k-mooney | have you created the bug lable yet | 16:40 |
| bauzas | please take a look on it, as I beg for reviewers | 16:40 |
| sean-k-mooney | in launchpad | 16:40 |
| bauzas | reviews* | 16:40 |
| bauzas | sean-k-mooney: you mean the stable tag ? | 16:40 |
| bauzas | yes | 16:40 |
| bauzas | #info people can tag launchpad bug reports with zed-rc-potential tag if they consider the bugfix important for RC1 or later (if regression fix) | 16:41 |
| sean-k-mooney | i mean zed-rc-potential tag yes | 16:41 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=zed-rc-potential | 16:41 |
| bauzas | all of this is in the etherpad | 16:41 |
| bauzas | and accordingly, | 16:41 |
| bauzas | gibi: I just add your https://review.opendev.org/c/openstack/nova/+/857491 to the list of RC1 needed patches | 16:41 |
| gibi | I've already added | 16:42 |
| gibi | L43 | 16:42 |
| bauzas | perfect | 16:42 |
| sean-k-mooney | i just made it an offical tag too so it will come up when you type | 16:42 |
| bauzas | sean-k-mooney: oh good point, I forgot this cycle | 16:43 |
| bauzas | thanks | 16:43 |
| bauzas | so, we have a couple of important patches | 16:43 |
| bauzas | for those which are from the release team, there is a ETD | 16:43 |
| bauzas | which is Friday | 16:43 |
| bauzas | if we don't say anything, those will be merged as it is | 16:43 |
| bauzas | so, reviews. | 16:44 |
| bauzas | I'll ping a couple of cores those two days to ensure we're in a good shape before we branch | 16:44 |
| gibi | ack | 16:44 |
| bauzas | the placement and nova zed branch patches are already there :) | 16:45 |
| bauzas | if you don't wanna freak out :) | 16:45 |
| bauzas | but I'm not happy with the SHA1 so I'll officially -1 the nova one (cc elodilles ;) ) | 16:45 |
| bauzas | ok, now, I have one question | 16:45 |
| elodilles | bauzas: saw that, thanks! | 16:46 |
| bauzas | with https://review.opendev.org/c/openstack/nova/+/855706, sean-k-mooney raised a good point about the min compute version we shall support for Zed | 16:46 |
| bauzas | for the moment, we say Xena | 16:46 |
| elodilles | (and waiting for a response on the placement patch :)) | 16:46 |
| bauzas | in https://github.com/openstack/nova/blob/master/nova/objects/service.py#L233 | 16:46 |
| sean-k-mooney | yep it should by Yoga | 16:46 |
| sean-k-mooney | https://github.com/openstack/governance/blob/master/resolutions/20220210-release-cadence-adjustment.rst#proposed-solution | 16:46 |
| sean-k-mooney | """Y->A should be a "dress rehearsal" where we have the jobs enabled to help smoke out any issues, but where hard guarantees are not yet made.""" | 16:47 |
| bauzas | sean-k-mooney: takashi made a patch accordingly | 16:47 |
| opendevreview | Merged openstack/os-vif stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/os-vif/+/856780 | 16:47 |
| bauzas | #link https://review.opendev.org/c/openstack/nova/+/856895 | 16:47 |
| bauzas | sean-k-mooney: agreed on the resolution so we then need to merge https://review.opendev.org/c/openstack/nova/+/856895 before RC1 | 16:48 |
| bauzas | sean-k-mooney: I personally feel the rpc alias and the min version bump shall continue to be separate patches | 16:48 |
| bauzas | so I'll +2 boht | 16:48 |
| sean-k-mooney | ok | 16:49 |
| sean-k-mooney | if they are both merged for rc then im ok with that | 16:49 |
| sean-k-mooney | i should drop my -1 then on the other patch | 16:49 |
| sean-k-mooney | i would prefer to do these together in general but either works | 16:50 |
| bauzas | sean-k-mooney: previously we kept them separately | 16:50 |
| gibi | I have to review those | 16:50 |
| sean-k-mooney | i set review priorty on both | 16:50 |
| bauzas | as ideally, rpc aliases should be merged just before RC1 while min version bumps shall be merged at the beginning of the release | 16:51 |
| sean-k-mooney | ill look at them agin later and droped my -1 | 16:51 |
| sean-k-mooney | im not convice thats the case | 16:51 |
| sean-k-mooney | or that they need to be seperate | 16:51 |
| bauzas | I'll update the PTL guide to reflect this | 16:52 |
| bauzas | and sean-k-mooney, you're up to debate in such patch | 16:52 |
| sean-k-mooney | we had some discussion about if we shoudl be recording the min or max verison supproted by a release by the way | 16:52 |
| bauzas | (in the PTL guide patch, I mean) | 16:52 |
| bauzas | moving on | 16:52 |
| bauzas | #topic PTG planning | 16:52 |
| bauzas | #link https://etherpad.opendev.org/p/nova-antelope-ptg Antelope PTG etherpad | 16:52 |
| bauzas | as a reminder ^ | 16:52 |
| bauzas | #link https://ptg.opendev.org/ptg.html PTG schedule | 16:52 |
| bauzas | I made the bookings as agreed ^ | 16:53 |
| opendevreview | Merged openstack/os-vif stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/os-vif/+/856781 | 16:53 |
| bauzas | #topic Review priorities | 16:53 |
| bauzas | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | 16:53 |
| bauzas | nothing to say about those | 16:53 |
| bauzas | feel free to engage on review-prio patches during the day | 16:54 |
| bauzas | #topic Stable Branches | 16:54 |
| sean-k-mooney | we should add rc bugs to the review priortes | 16:54 |
| bauzas | elodilles: your 5-min turn | 16:54 |
| bauzas | sean-k-mooney: I can do it | 16:54 |
| elodilles | ack | 16:54 |
| elodilles | #info stable/yoga seems to be blocked by openstacksdk-functional-devstack job | 16:54 |
| elodilles | the py39 issue was resolved, | 16:54 |
| elodilles | but it seems openstacksdk-functional-devstack fails now instead :/ | 16:55 |
| opendevreview | Merged openstack/python-novaclient stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/python-novaclient/+/856788 | 16:55 |
| opendevreview | Merged openstack/python-novaclient stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/python-novaclient/+/856789 | 16:55 |
| elodilles | JayF called our attention yesterday about it | 16:55 |
| bauzas | damn | 16:55 |
| elodilles | i had only a quick look and it fails on other projects gate as well, so it is not just nova | 16:55 |
| elodilles | beyond this | 16:55 |
| elodilles | #info stable/stein (and older) are blocked: grenade and other devstack based jobs fail with the same timeout issue as stable/train was previously | 16:55 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:56 |
| elodilles | the usual things ^^^ | 16:56 |
| elodilles | that's it | 16:56 |
| bauzas | elodilles: fancy writing an email to the world explaining the gate blocker ? | 16:56 |
| elodilles | bauzas: yepp, i can do that | 16:56 |
| bauzas | I just hope someone will just magically wake up and say he'll look | 16:57 |
| bauzas | elodilles: all projects being impacted by the sdk, you said ? | 16:57 |
| elodilles | not all | 16:57 |
| elodilles | but i've seen some others | 16:57 |
| sean-k-mooney | is that the storage cleanpu failure | 16:57 |
| sean-k-mooney | in the functional job | 16:57 |
| sean-k-mooney | where its failing to delete the volume | 16:58 |
| sean-k-mooney | becuase the snapshot is not deleted | 16:58 |
| sean-k-mooney | ah it is test_block_storage_cleanup | 16:58 |
| elodilles | yepp, test_block_storage_cleanup | 16:58 |
| sean-k-mooney | ya so they trided to fix it by doing it in a loop twice | 16:58 |
| sean-k-mooney | but that obviouly does not work | 16:58 |
| sean-k-mooney | so the test is flaky | 16:59 |
| elodilles | oh, so it's a known issue :-o | 16:59 |
| bauzas | => remove the flakey test then | 16:59 |
| sean-k-mooney | ill see if i can get the patch | 16:59 |
| bauzas | and unblock the gate | 16:59 |
| sean-k-mooney | we cant currently filter those | 16:59 |
| sean-k-mooney | via the job config | 16:59 |
| bauzas | anyway, we're on time | 17:00 |
| sean-k-mooney | efoley and i https://review.opendev.org/c/openstack/openstacksdk/+/856509 tried to fix it by backporting that | 17:00 |
| sean-k-mooney | but then we realised it always uses master | 17:00 |
| bauzas | let's wrap this meeting and continue the stable/yoga discussion right after if you want | 17:00 |
| bauzas | #topic Open discussion | 17:00 |
| bauzas | nothing | 17:01 |
| sean-k-mooney | so https://review.opendev.org/c/openstack/openstacksdk/+/852992 was ment to fix it but does not | 17:01 |
| JayF | Might be worth noting for meeting notes purposes that folks running Ironic back as far as train should update their Nova checkout | 17:01 |
| JayF | as all known fixes for the Ironic driver have been backported | 17:01 |
| opendevreview | Merged openstack/osc-placement stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/osc-placement/+/856784 | 17:01 |
| opendevreview | Merged openstack/osc-placement stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/osc-placement/+/856785 | 17:01 |
| JayF | (and I know for em branches we don't do releases) | 17:01 |
| bauzas | JayF: want me to offically set an #info ? | 17:01 |
| bauzas | :) | 17:01 |
| JayF | bauzas: if you think anyone who cares would read it :D | 17:02 |
| sean-k-mooney | elodilles: i just opened a revert https://review.opendev.org/c/openstack/openstacksdk/+/857471 | 17:02 |
| sean-k-mooney | the https://review.opendev.org/c/openstack/openstacksdk/+/852992 fixes an issue and intoduces the flaky test | 17:02 |
| sean-k-mooney | so maybe a partial revert | 17:02 |
| sean-k-mooney | keep the fix but revert the new test caes | 17:03 |
| bauzas | #info Ironic operators running train or later are more than welcome to upgrade their nova checkout with latest stable releases since bugfixes are released now | 17:03 |
| bauzas | JayF: ^ happy ? :) | 17:03 |
| JayF | thank you :D | 17:03 |
| elodilles | sean-k-mooney: ack, thanks for the info! i'll look at them then | 17:03 |
| gibi | (it feels like we have to parallel meeting both overrun its time) | 17:03 |
| bauzas | JayF: people reading our notes are beasts I don't know | 17:03 |
| gibi | *two | 17:03 |
| bauzas | gibi: well, I'll offload you some task | 17:04 |
| bauzas | thanks all | 17:04 |
| bauzas | #endmeeting | 17:04 |
| opendevmeet | Meeting ended Tue Sep 13 17:04:20 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:04 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-13-16.00.html | 17:04 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-13-16.00.txt | 17:04 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-13-16.00.log.html | 17:04 |
| elodilles | thanks bauzas o/ | 17:04 |
| gibi | bauzas, sean-k-mooney: fyi I -1 on https://review.opendev.org/c/openstack/nova/+/856895 It think that patch does more what we need | 17:04 |
| gibi | but I have to drop now | 17:04 |
| gibi | ping me tomorrow if the comment is not clear | 17:04 |
| gibi | o/ | 17:05 |
| sean-k-mooney | afck | 17:05 |
| sean-k-mooney | ack | 17:05 |
| elodilles | sean-k-mooney: if the original fix did not merge into yoga, then how is it causing failing tests? :-o or is that only some part of the fix? | 17:06 |
| elodilles | sean-k-mooney: sorry, it seems i'm a bit lost there o:) | 17:07 |
| sean-k-mooney | we proposed it in yoga but then realised that there is a branch overried | 17:07 |
| sean-k-mooney | so emmas patch is not needed | 17:07 |
| sean-k-mooney | even though the sdk is branched | 17:07 |
| elodilles | sean-k-mooney: oh, i see! as sdk is taken from master branch | 17:08 |
| sean-k-mooney | the sdk jobs always use master sdk on any stable branch | 17:08 |
| sean-k-mooney | yep | 17:08 |
| sean-k-mooney | so that is why a mager change broke stable | 17:08 |
| elodilles | sean-k-mooney: sorry :S now i get it o:) | 17:08 |
| sean-k-mooney | so we might want to drop https://review.opendev.org/c/openstack/openstacksdk/+/857471/1/openstack/block_storage/v3/_proxy.py but https://review.opendev.org/c/openstack/openstacksdk/+/857471/1/openstack/tests/functional/cloud/test_project_cleanup.py is the flaky test that we should revert out | 17:11 |
| sean-k-mooney | ill do that and rebase it to the tip of master | 17:12 |
| elodilles | sean-k-mooney: thanks! | 17:14 |
| sean-k-mooney | https://review.opendev.org/c/openstack/openstacksdk/+/857471 | 17:20 |
| sean-k-mooney | ok so that is just removing the test cases now | 17:20 |
| sean-k-mooney | the retry they added is preserved to not regress https://storyboard.openstack.org/#!/story/2010217 | 17:20 |
| elodilles | sean-k-mooney: ++ | 17:23 |
| sean-k-mooney | bauzas: by the way while im going to attent the ptg i dont plan on addign any topics this time to the adgenda | 17:30 |
| sean-k-mooney | dansmith: by the way does your n-2 greade job still work | 17:32 |
| sean-k-mooney | if so we should likely add that to the check pipline or the perodic pipelien one RC1 is out using yoga as a base | 17:33 |
| sean-k-mooney | so weekly at a minium but we could run it on each patch if we wanted too | 17:33 |
| sean-k-mooney | thats proably a ptg topic i guess | 17:34 |
| sean-k-mooney | i.e. how and what level of testing we wil do in A as part of the dress rehersal for C | 17:34 |
| sean-k-mooney | is it the grenade-skip-level: jobs | 17:36 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/.zuul.yaml#L705-L706 | 17:36 |
| sean-k-mooney | ya it is https://opendev.org/openstack/grenade/src/branch/master/.zuul.yaml#L377 | 17:37 |
| sean-k-mooney | so we wil need a variant for tha tthat is based on yoga as the base and master as the target for A | 17:38 |
| sean-k-mooney | its proably best to do that in hte grenade repo | 17:38 |
| dansmith | sean-k-mooney: we won't know until we start running it again, but it's supposed to | 17:42 |
| dansmith | but we shouldn't be running it on zed-rc right? just on master (antelope) right? | 17:42 |
| dansmith | oh, "once rc1 is out" yeah | 17:42 |
| sean-k-mooney | yep | 17:42 |
| sean-k-mooney | so from friday or next week | 17:42 |
| sean-k-mooney | we can trun it back on on master | 17:43 |
| sean-k-mooney | and pin to yoga | 17:43 |
| sean-k-mooney | as teh from branch | 17:43 |
| sean-k-mooney | its currently wallaby | 17:43 |
| sean-k-mooney | for xena | 17:43 |
| sean-k-mooney | sorry for yoga | 17:43 |
| sean-k-mooney | letters are hard hehe | 17:44 |
| dansmith | yeah | 17:44 |
| opendevreview | Merged openstack/python-novaclient master: Update master for stable/zed https://review.opendev.org/c/openstack/python-novaclient/+/856790 | 18:27 |
| *** dasm is now known as dasm|off | 22:28 | |
| *** dasm|off is now known as Guest305 | 23:03 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!