| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-01 | |||
| 13:07:16 | opendevreview | sean mooney proposed openstack/nova master: split monkey_patching form import https://review.opendev.org/c/openstack/nova/+/922425 | |
| 13:07:17 | opendevreview | sean mooney proposed openstack/nova master: Add IO Thread Pool Executor https://review.opendev.org/c/openstack/nova/+/922497 | |
| 13:07:17 | opendevreview | sean mooney proposed openstack/nova master: Remove nova debugger funcitonality https://review.opendev.org/c/openstack/nova/+/922496 | |
| 13:07:18 | opendevreview | sean mooney proposed openstack/nova master: remove eventlet.tpool https://review.opendev.org/c/openstack/nova/+/905287 | |
| 14:20:00 | opendevreview | Merged openstack/nova master: tests: Ensure all APIs have a request body schema https://review.opendev.org/c/openstack/nova/+/915735 | |
| 15:33:08 | opendevreview | sean mooney proposed openstack/nova master: use futureist for scater_gather https://review.opendev.org/c/openstack/nova/+/905284 | |
| 16:14:29 | opendevreview | sean mooney proposed openstack/nova master: use futerist for RBD calls https://review.opendev.org/c/openstack/nova/+/917962 | |
| 16:26:48 | opendevreview | sean mooney proposed openstack/nova master: Add IO Thread Pool Executor https://review.opendev.org/c/openstack/nova/+/922497 | |
| 16:26:48 | opendevreview | sean mooney proposed openstack/nova master: Remove nova debugger funcitonality https://review.opendev.org/c/openstack/nova/+/922496 | |
| 16:26:49 | opendevreview | sean mooney proposed openstack/nova master: use futureist for scater_gather https://review.opendev.org/c/openstack/nova/+/905284 | |
| 16:26:49 | opendevreview | sean mooney proposed openstack/nova master: remove eventlet.tpool https://review.opendev.org/c/openstack/nova/+/905287 | |
| 16:26:50 | opendevreview | sean mooney proposed openstack/nova master: use futerist for RBD calls https://review.opendev.org/c/openstack/nova/+/917962 | |
| #openstack-nova - 2024-07-02 | |||
| 02:24:42 | opendevreview | Merged openstack/nova master: api: Fix bugs in schemas https://review.opendev.org/c/openstack/nova/+/915843 | |
| 02:34:34 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 08:37:02 | opendevreview | Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627 | |
| 11:10:30 | opendevreview | Fabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load https://review.opendev.org/c/openstack/nova-specs/+/922201 | |
| 11:49:42 | opendevreview | Max proposed openstack/nova master: performance: reduce calls to libvirt / add caching https://review.opendev.org/c/openstack/nova/+/922855 | |
| 11:51:32 | opendevreview | Bence Romsics proposed openstack/os-vif master: Do not add taps in trunk bridges to the dead vlan https://review.opendev.org/c/openstack/os-vif/+/923036 | |
| 12:06:38 | opendevreview | Fabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load https://review.opendev.org/c/openstack/nova-specs/+/922201 | |
| 12:52:53 | opendevreview | Merged openstack/nova-specs master: Follow-up for openapi spec https://review.opendev.org/c/openstack/nova-specs/+/919603 | |
| 13:44:07 | opendevreview | Merged openstack/nova master: tests: Ensure API schemas are valid https://review.opendev.org/c/openstack/nova/+/915844 | |
| 13:56:56 | opendevreview | Michal Arbet proposed openstack/nova-specs master: Add availability zone-aware affinity filter to the scheduler https://review.opendev.org/c/openstack/nova-specs/+/890779 | |
| 13:58:13 | opendevreview | Michal Arbet proposed openstack/nova-specs master: Add availability zone-aware affinity filter to the scheduler https://review.opendev.org/c/openstack/nova-specs/+/890779 | |
| 14:09:49 | opendevreview | Dan Smith proposed openstack/nova master: Reject qcow files with data-file attributes https://review.opendev.org/c/openstack/nova/+/923255 | |
| 14:09:50 | opendevreview | Dan Smith proposed openstack/nova master: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923256 | |
| 14:09:52 | opendevreview | Dan Smith proposed openstack/nova master: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923257 | |
| 14:09:54 | opendevreview | Dan Smith proposed openstack/nova master: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923258 | |
| 14:10:38 | opendevreview | Dan Smith proposed openstack/nova stable/2024.1: Reject qcow files with data-file attributes https://review.opendev.org/c/openstack/nova/+/923273 | |
| 14:10:40 | opendevreview | Dan Smith proposed openstack/nova stable/2024.1: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923274 | |
| 14:10:42 | opendevreview | Dan Smith proposed openstack/nova stable/2024.1: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923275 | |
| 14:10:44 | opendevreview | Dan Smith proposed openstack/nova stable/2024.1: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923276 | |
| 14:11:10 | opendevreview | Dan Smith proposed openstack/nova stable/2023.2: Reject qcow files with data-file attributes https://review.opendev.org/c/openstack/nova/+/923284 | |
| 14:11:11 | opendevreview | Dan Smith proposed openstack/nova stable/2023.2: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923285 | |
| 14:11:12 | opendevreview | Dan Smith proposed openstack/nova stable/2023.2: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923286 | |
| 14:11:13 | opendevreview | Dan Smith proposed openstack/nova stable/2023.2: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923287 | |
| 14:11:25 | opendevreview | Dan Smith proposed openstack/nova stable/2023.1: Reject qcow files with data-file attributes https://review.opendev.org/c/openstack/nova/+/923288 | |
| 14:11:26 | opendevreview | Dan Smith proposed openstack/nova stable/2023.1: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923289 | |
| 14:11:27 | opendevreview | Dan Smith proposed openstack/nova stable/2023.1: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923290 | |
| 14:11:28 | opendevreview | Dan Smith proposed openstack/nova stable/2023.1: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923291 | |
| 14:35:03 | mnaser | dansmith: sorry I know its a busy morning but I was working on back porting the patches to unmaintained/zed | |
| 14:35:11 | mnaser | https://review.opendev.org/c/openstack/nova/+/923291/1/nova/tests/unit/virt/libvirt/test_utils.py | |
| 14:35:30 | mnaser | I think this will fail pep8 because of lack of new line above test_create_image_encryption | |
| 14:37:14 | dansmith | yep, looks like it already did | |
| 14:37:15 | dansmith | thanks | |
| 14:37:39 | dansmith | that was last minute yesterday | |
| 14:37:49 | mnaser | yeah it's all been last minute all over the place | |
| 14:37:52 | dansmith | it passed pep8 on master when I tried, so maybe the backport trimmed | |
| 14:38:02 | dansmith | srsly | |
| 14:38:07 | mnaser | I am taking care of zed for all the projects | |
| 14:38:18 | mnaser | https://review.opendev.org/c/openstack/nova/+/923297 if you want to eye it over (and it matters to you that is :p) | |
| 14:38:26 | sean-k-mooney | it passed for me locally too | |
| 14:38:47 | sean-k-mooney | i think its because in the gate its applied to the tip of the branch | |
| 14:38:59 | sean-k-mooney | when we run the jobs | |
| 14:39:06 | mnaser | ahh right maybe it ran it over the last commit that was clean | |
| 14:39:07 | sean-k-mooney | so to see it locally we might need to rebase | |
| 14:45:28 | tkajinam | I downloaded the chain for master but pep8 fails in my local (even after rebasing the chain on the latest master). zuul hasn't run pep8 year over 923258 but I suspect it may detect the same error. | |
| 14:45:40 | tkajinam | s/year/yet/ | |
| 14:46:02 | dansmith | oh it does fail for me on master, | |
| 14:46:09 | dansmith | I ran preivously before the tests were added | |
| 14:46:11 | dansmith | I will fix | |
| 14:47:07 | opendevreview | Dan Smith proposed openstack/nova master: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923258 | |
| 14:47:54 | sean-k-mooney | reappoved | |
| 14:50:02 | opendevreview | Dan Smith proposed openstack/nova stable/2024.1: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923276 | |
| 14:55:26 | opendevreview | Dan Smith proposed openstack/nova stable/2023.2: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923287 | |
| 14:58:43 | dansmith | mnaser: tkajinam sorry about that, thanks for the heads up 2023.1 coming in a sec | |
| 15:00:59 | opendevreview | Dan Smith proposed openstack/nova stable/2023.1: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923291 | |
| 15:09:09 | JayF | melwitt: sean-k-mooney: re: the lock greening errors in python 3.12: https://github.com/eventlet/eventlet/issues/969 is what itamarst tracked it to. | |
| 15:12:11 | frickler | sean-k-mooney: dansmith: do you want us to promote any of the approved patches into gate or do you prefer for all check jobs to finish first? | |
| 15:13:03 | dansmith | frickler: definitely want to see the tests first | |
| 15:19:47 | sean-k-mooney | frickler: based on the quick review we have done we belive the patchs shoudl be correct however we need the wider coverage form check to confirm we are not regressing any edgecases. if we see a timeout ectra or other infa issues we can consier promotiign or requeing | |
| 15:22:17 | frickler | well I'm not talking about force-merging (yet), you'll have the same jobs running in the gate pipeline again (except n-v), but anyway, the decision is up to you | |
| 15:27:44 | opendevreview | Fabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load https://review.opendev.org/c/openstack/nova-specs/+/922201 | |
| 15:32:40 | tkajinam | dansmith, no problem, and thank you ! | |
| 15:54:00 | dansmith | definitely want to see all the scenarios run against this since its so much change and it's nearly impossible to do that ahead of time | |
| 15:54:16 | dansmith | if we hit obviously unrelated things I think promote/force is the tool we can use to expedite | |
| 15:54:22 | dansmith | thanks frickler for checking | |
| 15:56:26 | sean-k-mooney | im checkign now but the nova-live-migration job fails on the second patch | |
| 15:56:28 | sean-k-mooney | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_7fc/923256/1/check/nova-live-migration/7fc79c6/ | |
| 15:56:29 | opendevreview | Fabian Wiesel proposed openstack/nova-specs master: Lazy Metadata Loading in order to Reduce Server Load https://review.opendev.org/c/openstack/nova-specs/+/922201 | |
| 15:56:31 | sean-k-mooney | the ceph one passed | |
| 15:56:52 | dansmith | hmm | |
| 15:57:07 | dansmith | that is odd or sure | |
| 15:57:09 | dansmith | *for | |
| 15:57:20 | dansmith | I have to do a medial thing in a few minutes, so I will be out a bit around there | |
| 15:57:24 | sean-k-mooney | looks like only the blockmigation tests | |
| 15:57:38 | dansmith | right but perhaps something is upset about the disk image? | |
| 15:57:56 | dansmith | Jul 02 15:17:20.783035 np0037859393 nova-compute[41928]: ERROR oslo_messaging.rpc.server nova.exception.ImageUnacceptable: Image 02acc766-7c4b-4ffb-a72d-080ea50527d4 is unacceptable: Image not in a supported format | |
| 15:57:57 | dansmith | eff | |
| 15:58:19 | sean-k-mooney | yep | |
| 15:58:57 | sean-k-mooney | AttributeError: 'NoneType' object has no attribute 'from_file' | |
| 15:59:03 | sean-k-mooney | but i tought we caught that | |
| 16:01:15 | sean-k-mooney | so this is raising https://review.opendev.org/c/openstack/nova/+/923256/1/nova/virt/images.py#152 | |
| 16:01:32 | sean-k-mooney | and we take this else https://review.opendev.org/c/openstack/nova/+/923256/1/nova/virt/images.py#174 | |
| 16:02:07 | sean-k-mooney | oh ok i need to look at the ohte rhost to see it properly | |
| 16:04:12 | sean-k-mooney | Jul 02 15:19:58.679517 np0037859379 nova-compute[85178]: WARNING nova.virt.images [None req-3f27d1d1-b0a1-46a9-ac4b-6b869e49ea15 tempest-LiveMigrationTest-599041496 tempest-LiveMigrationTest-599041496-project-admin] Unable to perform deep image inspection on type 'aki' | |
| 16:04:28 | sean-k-mooney | so we have special cased for ami image but not aki | |
| 16:05:17 | sean-k-mooney | so i think its passing on the ami iamge and failing on the kernel image | |
| 16:06:06 | sean-k-mooney | dansmith: and we are only seeing it in that job becasue tis the only one using uec images | |
| 16:06:08 | sean-k-mooney | https://github.com/openstack/nova/blob/master/.zuul.yaml#L120-L121 | |
| 16:07:20 | sean-k-mooney | so "if disk_format == 'ami':" should be "if disk_format in ('ami','aki'):" | |