| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-23 | |||
| 09:45:24 | mikal | sean-k-mooney: so should I move the heading in the etherpad? | |
| 10:00:04 | sean-k-mooney | sorry which etherpad? | |
| 10:00:31 | sean-k-mooney | mikal: was distracted downstream | |
| 10:31:14 | mikal | sean-k-mooney: no worries. The dalmatian etherpad (https://etherpad.opendev.org/p/nova-dalmatian-status) lists the SPICE stuff under "to be determined" not "needs code review". Should I just change that, or is it done by a designated person? | |
| 10:40:18 | sean-k-mooney | mikal: oh ya you can move it to needs core review since its approved and you have code not in WIP state | |
| 10:52:45 | mikal | sean-k-mooney: thanks done. I'm hesitant to un-WIP the next few patches because I'll have to redo CI on them, but I guess I'll do that too. | |
| 10:54:20 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | |
| 10:54:20 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow concurrent access to SPICE consoles. https://review.opendev.org/c/openstack/nova/+/922546 | |
| 10:54:21 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Add guest devices to support SPICE USB. https://review.opendev.org/c/openstack/nova/+/922547 | |
| 10:54:21 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally enable SPICE debug logging. https://review.opendev.org/c/openstack/nova/+/922548 | |
| 10:54:22 | opendevreview | Michael Still proposed openstack/nova master: WIP: libvirt: Optionally support sound when using SPICE. https://review.opendev.org/c/openstack/nova/+/922549 | |
| 10:58:51 | opendevreview | Alexander Hild proposed openstack/nova master: feat: Include "vm uuid" in baseBoard asset tag in libvirt domain xml https://review.opendev.org/c/openstack/nova/+/924717 | |
| 11:02:21 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for assisted volume snapshots APIs https://review.opendev.org/c/openstack/nova/+/924588 | |
| 11:05:21 | opendevreview | Alexander Hild proposed openstack/nova master: feat: Include "vm uuid" in baseBoard asset tag in libvirt xml https://review.opendev.org/c/openstack/nova/+/924720 | |
| 11:06:07 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Fix typo https://review.opendev.org/c/openstack/nova/+/924722 | |
| 11:08:32 | opendevreview | Alexander Hild proposed openstack/nova master: feat: "vm uuid" in baseBoard asset tag https://review.opendev.org/c/openstack/nova/+/924723 | |
| 11:10:42 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for port interface APIs https://review.opendev.org/c/openstack/nova/+/924589 | |
| 11:11:14 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for availability zone APIs https://review.opendev.org/c/openstack/nova/+/924590 | |
| 11:12:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for bare metal node APIs https://review.opendev.org/c/openstack/nova/+/924591 | |
| 11:12:09 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for console auth token APIs https://review.opendev.org/c/openstack/nova/+/924592 | |
| 11:12:15 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavor access API https://review.opendev.org/c/openstack/nova/+/924593 | |
| 11:12:20 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavor extra specs APIs https://review.opendev.org/c/openstack/nova/+/924594 | |
| 11:12:24 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for flavors APIs https://review.opendev.org/c/openstack/nova/+/924595 | |
| 11:12:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP pool APIs https://review.opendev.org/c/openstack/nova/+/924596 | |
| 11:12:36 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add contributor docs for response body validation https://review.opendev.org/c/openstack/nova/+/924597 | |
| 13:03:06 | elodilles_ooo | bauzas_: fyi, i'm off this week, so i won't participate on today's meeting | |
| 13:04:23 | bauzas_ | elodilles_ooo: ack thanks | |
| 13:04:35 | bauzas_ | have a good rest | |
| 13:13:14 | sean-k-mooney | elodilles_ooo: in that case please ignore irc and email until your pto is over :) you deserve a break | |
| 13:14:36 | opendevreview | Julien Le Jeune proposed openstack/nova master: Fix test_vmdk_bad_descriptor_mem_limit and test_vmdk_bad_descriptor_mem_limit_stream_optimized https://review.opendev.org/c/openstack/nova/+/924148 | |
| 14:01:07 | opendevreview | sean mooney proposed openstack/nova master: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924731 | |
| 14:01:23 | jlejeune | sean-k-mooney: hello, what du you mean by your remark on my proposal in https://review.opendev.org/c/openstack/nova/+/924148 ? | |
| 14:01:47 | opendevreview | sean mooney proposed openstack/nova stable/2024.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924732 | |
| 14:02:34 | opendevreview | sean mooney proposed openstack/nova stable/2023.2: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924733 | |
| 14:03:01 | opendevreview | sean mooney proposed openstack/nova stable/2023.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924734 | |
| 14:04:55 | opendevreview | sean mooney proposed openstack/nova stable/2024.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924732 | |
| 14:05:27 | opendevreview | sean mooney proposed openstack/nova stable/2023.2: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924733 | |
| 14:05:54 | opendevreview | sean mooney proposed openstack/nova stable/2023.1: Change force_format strategy to catch mismatches https://review.opendev.org/c/openstack/nova/+/924734 | |
| 14:06:59 | sean-k-mooney | jlejeune: i woudl prefer not to try and do the convert and catch the exepction and instead check via the help text if qemu image is present and supprots vmdk | |
| 14:10:18 | jlejeune | hm ok, so a check with --help option before running the qemu-img command, just to verify that qemu(img binary is present or not, right ? | |
| 14:10:31 | sean-k-mooney | jlejeune: basically i dont want to do any of the extra setup in that function if its not installed and does not supprot the format | |
| 14:11:44 | dansmith | personally I'd just go for consistency with the one above (which the current version pretty much does) and not worry about it as much since this is all moving to oslo | |
| 14:12:11 | dansmith | I'd also say that I think a dependency on qemu-img being in path and available for tests is very reasonable | |
| 14:12:24 | dansmith | ...for anything in nova for sure | |
| 14:12:36 | dansmith | and definitely for oslo.utils, since it has a bunch of tests for qemu-img info already | |
| 14:12:42 | jlejeune | dansmith: that's was my first proposal, do the same as other tests and waiting for a global move in oslo | |
| 14:12:55 | jlejeune | :/ | |
| 14:13:13 | dansmith | jlejeune: yep, that's fine with me fwiw.. I don't think we need to probe --help, but maybe sean-k-mooney can explain more | |
| 14:14:13 | sean-k-mooney | our downsstream version dont have some format compiled in and as you are aware many disues have an unsafe qemu image so i didnt want to run it even if its just on our local system | |
| 14:14:46 | sean-k-mooney | what proaposed is fine i just wanted to aovid calllign convert/create ectra if we coudl | |
| 14:15:19 | dansmith | changing FileNotFound to Exception (as the previous one you pointed to does) will catch both cases right? | |
| 14:15:50 | sean-k-mooney | yep and i dont think we want to differenciate | |
| 14:16:02 | sean-k-mooney | ill admign because of the other things i just pushed i only quickly skimmed the patch | |
| 14:16:10 | sean-k-mooney | so i might have missed somethign | |
| 14:17:13 | dansmith | yeah, so I think just using bare exception like the case above makes the most sense, jlejeune bauzas probably just didn't realize (as mentioned by the comment above) that it covers format issues and missing file issues | |
| 14:18:04 | jlejeune | dansmith: totally agree, that was my opinion at first | |
| 14:19:44 | bauzas | yeah I missed the first catch | |
| 14:20:07 | bauzas | what I explained is that I prefer to catch specific exceptions mostly | |
| 14:20:19 | bauzas | but we have some exceptions in code where we bare-catch | |
| 14:20:39 | bauzas | here, sean-k-mooney is just pointing out an already existing pattern, so I'm cool with it | |
| 14:20:51 | bauzas | sorry if that created some misunderstanding | |
| 14:21:01 | dansmith | yep, honestly we should raise on FileNotFound and call qemu-img a requirement, IMHO, and skip on anything else, but this code shouldn't be staying in nova so let's just do the consistent thing for now | |
| 14:21:38 | jlejeune | dansmith: ok, so let's go back on my patchset1 :) | |
| 14:21:55 | bauzas | agreed with dansmith here | |
| 14:22:16 | bauzas | keep consistency for now, but I'd appreciate some further rework on that whole crazy module :) | |
| 14:22:26 | jlejeune | :) | |
| 14:27:04 | opendevreview | Julien Le Jeune proposed openstack/nova master: Fix test_vmdk_bad_descriptor_mem_limit and test_vmdk_bad_descriptor_mem_limit_stream_optimized https://review.opendev.org/c/openstack/nova/+/924148 | |
| 14:29:16 | sean-k-mooney | jlejeune: we have some thing in ci that we dont want this to conflict with so we will likely loop back to your review tomorrow or later today | |
| 14:31:20 | jlejeune | sean-k-mooney: no problem | |
| 14:31:40 | dansmith | sean-k-mooney: ++ | |
| 14:31:46 | bauzas | sean-k-mooney: about that, I missed some comment, but could you please rebase the stable patches by adding [stable-only] tag so the order wouldn't be important for merging ? | |
| 14:32:14 | dansmith | that's not necessary, IMHO | |
| 14:32:21 | dansmith | they're up, we can land them in normal order now | |
| 14:32:26 | sean-k-mooney | bauzas: i spoke to gibi about that and came to the same conlcution | |
| 14:32:30 | sean-k-mooney | we dont need to do that in this case | |
| 14:32:43 | dansmith | they just need to be posted so we have URLs for the advisory, but no need to force fast merge them or anything | |
| 14:32:59 | bauzas | well, sure | |
| 14:33:13 | sean-k-mooney | i also want a full set of check results so dont want to promot to gate directly | |
| 14:33:16 | bauzas | I'm saying that because our SLURP branches could merge before the Bobcat one | |
| 14:33:39 | sean-k-mooney | bauzas: that techncially breaking policy | |
| 14:33:40 | bauzas | but we can proceed by order, I'm fine | |
| 14:33:52 | sean-k-mooney | we could but they are equally supported while instable | |
| 14:34:23 | sean-k-mooney | we shoudl not priorities slurps over supported non slurps | |
| 14:34:47 | bauzas | it was just a pragmatic question in terms of merging | |
| 14:34:50 | bauzas | but I'm cool | |
| 14:35:17 | sean-k-mooney | with that said i have promoated zuul to its own monitor which i will be watching until thsee land :) | |
| 14:35:50 | bauzas | ++ | |
| 14:38:02 | frickler | sean-k-mooney that might be a night shift ;-/ | |
| 14:39:17 | frickler | depending on the state of the gate I would still promote them to the head of gate if needed. also ping me if re-gates are needed due to unrelated failures | |
| 14:40:12 | dansmith | we should at least wait until we see a clean run in check, but yeah, thanks | |
| 14:40:53 | frickler | yes, I won't submit them to gate before that | |
| 14:48:17 | sean-k-mooney | frickler: ya we can consider that once the iniall run is complete. at least this time its one commit per branch | |
| 14:56:15 | sean-k-mooney | awsome post failure in one of the jobs.... | |
| 14:56:21 | sean-k-mooney | and its related. | |
| 14:56:32 | sean-k-mooney | well i know what im doing for the next little while | |
| 14:56:39 | sean-k-mooney | https://4b125a6587520ffd7605-3e126156a22dab1f4ba4bdb0e9add880.ssl.cf2.rackcdn.com/924731/1/check/nova-live-migration/a83a99e/testr_results.html | |
| 14:57:19 | dansmith | oye | |
| 14:57:42 | dansmith | sean-k-mooney: thanks for doing that, I gotta run in a sec, but I'll be back later | |
| 14:58:12 | dansmith | sean-k-mooney: in glance we found some tempest jobs were uploading qcows as AMI because AMI is the first format in the disk_image_formats list, so that could be related | |
| 14:58:41 | sean-k-mooney | maybe btu theses are migration test failrues | |