| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-02-29 | |||
| 19:21:38 | JayF | ;) | |
| 19:21:49 | dansmith | gmann: but for py38 that was only unit and functional | |
| 19:21:53 | sean-k-mooney | gmann: well we have 3.12 in centos 9 stream today as a non default | |
| 19:22:04 | gmann | dansmith: py38 was mandatory not in best effort | |
| 19:22:09 | sean-k-mooney | and soon ubuntu 24.04 | |
| 19:22:14 | dansmith | gmann: mandatory but unit and functional only | |
| 19:22:15 | JayF | dansmith: sean-k-mooney: my objection to python 3.12 being listed best effort is no longer valid in light of the new information ... as long as someone is actually going to adapt that interface lol | |
| 19:22:24 | sean-k-mooney | so ya we can tyr to get some teempest testign started when its not FF | |
| 19:22:30 | gmann | dansmith: ah, yes | |
| 19:22:51 | dansmith | gmann: so if the big concern is how best our effort is, then let's use different words, that's fine with me | |
| 19:24:44 | gmann | it says this which I think should be clear ? 'This distribution list is the best effort to test them in OpenStack CI/CD.These may not be part of integrated testing and may be tested periodically only.' | |
| 19:25:22 | gmann | or we can add more description here for example. pre-released things also if available like 24.04 case today | |
| 19:26:00 | dansmith | gmann: I get it. pick different words :) | |
| 19:27:06 | gmann | :) 'Optional Testing', 'Advance Testing' | |
| 19:27:36 | gmann | 'Unstable Testing' | |
| 19:27:52 | sean-k-mooney | im just going to download the iso, install it and see if it works and report back | |
| 19:28:16 | dansmith | gmann: I think optional sounds too unconcerned, but the others are reasonable to me | |
| 19:29:27 | gmann | ack, let me add 'Advance/Unstable Testing' so that we can cover any advance testing for non-released versions or unstable one like Centos Stream | |
| 19:37:43 | dansmith | sean-k-mooney: this is the next one of melwitt's series I think we need your re-+2 on: https://review.opendev.org/c/openstack/nova/+/826755/36 | |
| 19:38:35 | sean-k-mooney | i was looking at https://review.opendev.org/c/openstack/nova/+/910571 just now but sure ill swap to that | |
| 19:38:54 | dansmith | okay just saying that's the next one ready (almost) to go to the gate | |
| 19:39:00 | sean-k-mooney | so ot me that shows i still have a +2 on that | |
| 19:39:03 | sean-k-mooney | on https://review.opendev.org/c/openstack/nova/+/826755/36 | |
| 19:39:28 | sean-k-mooney | so it just need a +w right | |
| 19:39:40 | dansmith | oh sorry, it doesn't for me, but I see now | |
| 19:39:50 | dansmith | it put your +2 in the "and 5 more" overflow meny | |
| 19:39:52 | dansmith | how dumb | |
| 19:39:53 | dansmith | I can +w | |
| 19:40:23 | sean-k-mooney | ya it did the saem for you in one of the other pathces | |
| 19:40:31 | sean-k-mooney | i hate the new way the ui handeles this | |
| 19:40:38 | dansmith | seems like it should sort people with votes tot he top | |
| 19:40:39 | dansmith | tyeah | |
| 19:41:05 | sean-k-mooney | so im +2 back up to the create patch | |
| 19:41:20 | sean-k-mooney | whic i have not re reviewd since we are stoping at the testing one | |
| 19:41:52 | sean-k-mooney | so i think the stack is good for what we neeed today | |
| 19:42:39 | dansmith | from you yeah, I haven't made it through all up to there though | |
| 19:45:08 | sean-k-mooney | im going to try and get a few other bitd done related to down stream and then im going to finish for today and cook food | |
| 19:45:22 | sean-k-mooney | ping me if anything comes up | |
| 19:46:58 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 19:46:58 | opendevreview | Amit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | |
| 19:47:37 | sean-k-mooney | dansmith: actully before i leave the glance multiple location stuff is not done yet right | |
| 19:48:40 | sean-k-mooney | well glance client is not in any case | |
| 19:48:42 | dansmith | officially punted | |
| 19:48:44 | sean-k-mooney | https://review.opendev.org/c/openstack/python-glanceclient/+/890634 | |
| 19:48:50 | dansmith | so no need to look | |
| 19:48:51 | sean-k-mooney | ok | |
| 20:56:34 | gmann | dansmith: sean-k-mooney: updated version for testing runtime https://review.opendev.org/c/openstack/governance/+/908862 | |
| 21:21:45 | dansmith | gmann: yeah it's in my queue | |
| 23:38:42 | opendevreview | melanie witt proposed openstack/nova master: testing: Add ephemeral encryption support to fixtures https://review.opendev.org/c/openstack/nova/+/910571 | |
| #openstack-nova - 2024-03-01 | |||
| 00:20:41 | opendevreview | sean mooney proposed openstack/nova master: add multinode ironic shard job https://review.opendev.org/c/openstack/nova/+/910333 | |
| 06:27:17 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 06:27:18 | opendevreview | Amit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | |
| 10:08:53 | tobias-urdin | sean-k-mooney: when you have time, see if this updated patch is ok now based on your feedback https://review.opendev.org/c/openstack/nova/+/904381 | |
| 10:09:40 | tobias-urdin | also super interested in your ideas on how to solve the bug for this patch https://review.opendev.org/c/openstack/nova/+/875859 that we got hit by some days ago | |
| 10:31:09 | sean-k-mooney[m] | bauzas: can we get https://review.opendev.org/c/openstack/nova/+/896100 in to unblock the uc bump | |
| 10:32:32 | bauzas | sean-k-mooney: I don't have context but I can take a look | |
| 11:22:12 | opendevreview | Merged openstack/nova master: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 11:22:25 | opendevreview | Merged openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 11:25:43 | ratailor__ | Hi cores, can I get reviews on https://review.opendev.org/q/topic:%22list-requested-az%22 | |
| 11:30:33 | ratailor__ | elodilles, gibi sean-k-mooney[m] dansmith please review ^^ | |
| 11:47:55 | gibi | ratailor__: bauzas: I -1 on the AZ series, the mock in the functional tests feels like a paperover of a bug | |
| 11:48:41 | stephenfin | bauzas: Can we please just merge https://review.opendev.org/c/openstack/nova/+/896100 and move on | |
| 11:55:17 | opendevreview | Amit Uniyal proposed openstack/nova master: Refactor vf profile for PCI device https://review.opendev.org/c/openstack/nova/+/905134 | |
| 12:02:20 | sean-k-mooney | bauzas: tl;dr castalan added 2 new required abstract methods to a basecalse we inherit form | |
| 12:02:33 | sean-k-mooney | bauzas: that got reverted on stable last cycle | |
| 12:02:57 | sean-k-mooney | bauzas: but the castalan version that was intened for stabel only got pulled into master by mistake | |
| 12:03:21 | sean-k-mooney | so we were not seing this for the last cycle | |
| 12:03:49 | sean-k-mooney | bauzas: we should have had this code chagne since bobcat | |
| 12:05:37 | sean-k-mooney | bauzas: i dont really think its reasoable to punt the new castalane version for two cycles in a row just because we didnt accept a patch that addes to uncalled functions that raise not implemetned | |
| 12:06:28 | sean-k-mooney | bauzas: there was a way for castalan to not make this a breaking api change | |
| 12:07:04 | sean-k-mooney | but im not sure how fair it is to say to them rework this now and do another release unless we are going to write that patch for them | |
| 12:07:58 | sean-k-mooney | bauzas: im not particaly happy either but apparenty the non-client release deadline is the deadlien for the code to be merged not the deadline for the requirement repo to be bumped | |
| 12:08:31 | sean-k-mooney | hence reluctant +2 from me | |
| 12:34:53 | opendevreview | Merged openstack/nova master: Fix nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/910466 | |
| 12:40:08 | gibi | bauzas: would it make sense to fix the AZ series now, (I can try), or the bug I found means we punt it form the release? | |
| 12:59:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix for showing AZ without reqspec https://review.opendev.org/c/openstack/nova/+/910728 | |
| 13:10:02 | sean-k-mooney | gibi: technially code feeze was yesterday, but we could accpet it today if we wanted too | |
| 13:10:29 | sean-k-mooney | i would not extend to monday however without going though the excption process | |
| 13:11:22 | sean-k-mooney | gibi: what is the behvior with your fix | |
| 13:11:35 | sean-k-mooney | gibi: i would expect the pinned az to be None | |
| 13:11:52 | sean-k-mooney | if its not in the request spec so in json that would be null not '' | |
| 13:12:57 | sean-k-mooney | gibi: could you add an api sample test to show the end result? | |
| 13:13:36 | sean-k-mooney | we coudl go with empty string i guess instead of null | |
| 13:31:00 | stblatzheim | sean-k-mooney: Fix was merged. Any change i can backport this till stable/zed or at least stable/2023.1 ? | |
| 13:31:14 | stblatzheim | sean-k-mooney: Fix was merged. Any chance i can backport this till stable/zed or at least stable/2023.1 ? | |
| 13:32:17 | sean-k-mooney | yes it shoudl be backportable we have to go newest to oldest but i dont see why it could not go back to zed | |
| 13:32:45 | sean-k-mooney | are you familar with how to cerrypick it | |
| 13:33:10 | sean-k-mooney | i doubt it will have merge confict so fi the patch is merged you can do it via the gerrit ui | |
| 13:33:24 | sean-k-mooney | if its not merged the gerrit ui wont add the cherry-picked form line | |
| 13:34:42 | stblatzheim | I'll give it a try :) It's already merged. | |
| 13:35:03 | sean-k-mooney | that was more a meta comment for the older branches | |
| 13:35:17 | sean-k-mooney | you have ot go n to n-1 then n-1 to n-2 | |
| 13:35:21 | opendevreview | Steven Blatzheim proposed openstack/nova stable/2023.2: Fix nova-metadata-api for ovn dhcp native networks https://review.opendev.org/c/openstack/nova/+/910616 | |
| 13:35:24 | sean-k-mooney | not n to n-2 | |
| 13:35:45 | sean-k-mooney | so the simpelstet way to do that is jsut wait for ^ to merge | |
| 13:35:50 | sean-k-mooney | before creating the next one | |
| 13:36:25 | stblatzheim | Ok, so i'll wait for 2023.2 and when it's merged i'll start 2023.1 and after this merge zed | |
| 13:36:41 | sean-k-mooney | cool | |
| 13:37:24 | stblatzheim | thanks for your help with the whole fix | |
| 13:38:19 | sean-k-mooney | no worries, i have +2'd the first backport feel free to ping me if you need reviews on the others | |