| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-05-15 | |||
| 19:17:17 | sean-k-mooney | yep im just confirming that the thing we are now string should be still ordered and stable | |
| 19:17:20 | dansmith | yep, sorted() is a good safety measure to keep it stable, but we don't need the extra top-level dict | |
| 19:17:21 | sean-k-mooney | which it should be | |
| 19:17:27 | dansmith | yeah | |
| 19:22:05 | dansmith | gmann: why no functional 3.12 job yet? our functionals pass on 3.12, but based on my experience above, we better go ahead and have a voting job to make sure we don't regress on that | |
| 19:23:02 | dansmith | or, hmm, do we only get functional jobs from our own tree? we don't even have a 311 job in nova | |
| 19:24:55 | dansmith | ah, I guess the base is there and I can add it | |
| 19:26:12 | sean-k-mooney | dansmith: i think gmann had a patch up for this for placement but i have not seen one for nova | |
| 19:26:19 | dansmith | coming | |
| 19:26:27 | sean-k-mooney | i think the definiton of the functional 3.12 job was pending | |
| 19:26:48 | opendevreview | Dan Smith proposed openstack/nova master: Make python 3.12 unit and functional voting https://review.opendev.org/c/openstack/nova/+/919767 | |
| 19:26:54 | dansmith | the base job appears to exist, so I think this ^ should do it | |
| 19:27:55 | sean-k-mooney | oh i was thinking of https://review.opendev.org/c/openstack/osc-placement/+/904612 | |
| 19:28:21 | sean-k-mooney | yep they were added by https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/918946 | |
| 19:28:31 | sean-k-mooney | which looks like it merged last week | |
| 19:29:26 | opendevreview | Dan Smith proposed openstack/nova master: Make python 3.12 unit and functional voting https://review.opendev.org/c/openstack/nova/+/919767 | |
| 19:29:30 | dansmith | this patch removes 38 too by the way | |
| 19:29:34 | dansmith | which I think is legit now | |
| 19:29:43 | sean-k-mooney | yes it is | |
| 19:30:02 | sean-k-mooney | som ehave started blocking install on 38 but i know you dont want to do that in nova | |
| 19:30:23 | dansmith | yeah I wouldn't personally, but I'm just one guy :) | |
| 19:31:24 | sean-k-mooney | so we are missing 3.11 for functional | |
| 19:31:36 | sean-k-mooney | 3.12 is not requried but if its passing and voting then sure | |
| 19:31:45 | dansmith | yeah, missing 3.11 | |
| 19:31:47 | sean-k-mooney | do we want to remove 3.10 or bump it to 3.11 | |
| 19:31:51 | dansmith | I think we should keep 3.12 passing as long as it is | |
| 19:32:14 | dansmith | we could.. testing either end is enough for me | |
| 19:32:15 | sean-k-mooney | yep just wonder if you want 2 jobs or 3 while its working | |
| 19:33:02 | sean-k-mooney | ok im going to finish for today ill take a look at this tomorwow | |
| 19:33:18 | sean-k-mooney | 3.10 may have been the default on ubuntu 22.04 | |
| 19:33:37 | dansmith | it is, and that's still our primary platform for this release | |
| 19:33:38 | sean-k-mooney | so that might have been why we were tsting it and not 3.11 or we just forgot | |
| 19:34:00 | sean-k-mooney | ya so i would be fine not adding 3.11 and keeping 3.10 | |
| 19:34:05 | dansmith | me too | |
| 19:34:46 | sean-k-mooney | in that case ill +2 https://review.opendev.org/c/openstack/nova/+/919767 and gmann or others can disagree | |
| 19:36:16 | dansmith | ack, thanks | |
| 19:36:42 | dansmith | py312 unit job passed now on the patch below | |
| 19:38:11 | sean-k-mooney | oh you mena in zuul, it has not reported yet | |
| 19:38:57 | sean-k-mooney | its failed on the first patch but passing in the second and hopefully 3 | |
| 19:40:51 | sean-k-mooney | ah right the failure in the first patch is the hacking testcase | |
| 19:40:58 | sean-k-mooney | ya that all makes sesnse | |
| 19:42:24 | dansmith | yeah | |
| 20:05:29 | opendevreview | Merged openstack/nova master: do not use str(url) to stringify a URL for subsequent use https://review.opendev.org/c/openstack/nova/+/913910 | |
| 21:06:56 | gmann | dansmith: sean-k-mooney : yes, i was waiting for this to merge https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/918946 | |
| 21:08:18 | gmann | dansmith: sean-k-mooney +A on nova change. can you review osc-placement also where I am making it voting and its base change https://review.opendev.org/c/openstack/osc-placement/+/919101/1 | |
| 21:08:47 | gmann | as soon as we have py3.12 pass we should make it voting to avoid any incompatible change merge | |
| 21:44:53 | dansmith | gmann: yep, agree, got those | |
| #openstack-nova - 2024-05-16 | |||
| 00:52:42 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose specs for ephemeral encryption https://review.opendev.org/c/openstack/nova-specs/+/907654 | |
| 01:46:16 | gmann | thanks | |
| 07:31:32 | opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: Handle MAC dash-addresses during migration https://review.opendev.org/c/openstack/nova/+/919760 | |
| 09:08:32 | opendevreview | Pierre Riteau proposed openstack/nova master: Fix formatting issues in extra-specs docs https://review.opendev.org/c/openstack/nova/+/917462 | |
| 10:03:04 | stephenfin | bauzas: Friendly reminder about that OpenAPI spec needing your attention, please and thank you | |
| 10:03:39 | bauzas | stephenfin: I'm currently working hard on creating my presentation for the next OpenInfra Day in Paris next week, but I'll try | |
| 12:18:29 | opendevreview | Merged openstack/osc-placement master: Update testing of python versions https://review.opendev.org/c/openstack/osc-placement/+/904612 | |
| 12:29:19 | opendevreview | Merged openstack/osc-placement master: Make python 3.12 functional job voting https://review.opendev.org/c/openstack/osc-placement/+/919101 | |
| 13:16:07 | opendevreview | Merged openstack/nova master: Fix formatting issues in extra-specs docs https://review.opendev.org/c/openstack/nova/+/917462 | |
| 13:36:44 | opendevreview | Merged openstack/nova stable/2023.2: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/900339 | |
| 13:42:49 | dansmith | bauzas: this stack fixes python 3.12 functional/unit tests if you could help merge that before we regress :) https://review.opendev.org/c/openstack/nova/+/919763 | |
| 13:43:39 | bauzas | dansmith: okay, I don't really have time but I trust you, +Wd | |
| 13:44:15 | dansmith | bauzas: okay, it's a stack of three.. maybe gibi could look at the other two, or maybe melwitt when she's around | |
| 13:44:35 | sean-k-mooney | only the second one https://review.opendev.org/c/openstack/nova/+/919764/2 | |
| 13:44:42 | sean-k-mooney | is missing a +w at this point | |
| 13:44:45 | bauzas | this is done | |
| 13:44:59 | sean-k-mooney | cool | |
| 13:45:00 | bauzas | the fixes are simple but I don't really have time to think about some concerns | |
| 13:45:04 | dansmith | thanks | |
| 13:45:13 | dansmith | bauzas: yep, safe I think | |
| 13:45:28 | sean-k-mooney | form the nova side yes | |
| 13:46:22 | bauzas | dansmith: glance-api is absolutely blind on the req-id, so how could I see any logs from glance apart from devstack@g-api | |
| 13:46:55 | dansmith | is it/ | |
| 13:47:08 | dansmith | but gateway exception usually means apache not glance | |
| 13:51:16 | dansmith | unless glance is 500'ing or something | |
| 13:53:51 | bauzas | okay, I can look at httpd's logs | |
| 13:58:20 | bauzas | dansmith: can you help me for a sec ? I want to backup my instance but then it's queued on the glance side for a while and eventually I no longer see the image | |
| 13:58:40 | dansmith | about to start a meeting, but can try in parallel | |
| 13:58:44 | dansmith | using backup or snapshot? | |
| 13:58:45 | bauzas | nothing is told on the glance side, only the server event record in nova gives me the exception | |
| 13:58:49 | bauzas | server backup | |
| 13:59:15 | dansmith | because you want to test our backup stuff, or because you want a snapshot of your image? | |
| 13:59:44 | dansmith | I don't know what the plumbing through our server backup stuff looks like in detail, which is why I'm asking | |
| 14:00:07 | dansmith | so you see an image created on the glance side (POST /images) .. do you see PUT/POST /images/$uuid/data ? | |
| 14:02:26 | bauzas | dansmith: yep | |
| 14:02:31 | bauzas | POST /images | |
| 14:02:50 | bauzas | I have an instance, I just want a new image from it | |
| 14:03:23 | dansmith | okay, I'd be using snapshot not backup.. backup is a bit of a different thing.. still results in an image, but snapshot is more straightfortward | |
| 14:04:04 | sean-k-mooney | if using osc i think you shoudl b eusing "openstack server create image" | |
| 14:04:08 | sean-k-mooney | to take the snapshot | |
| 14:04:25 | sean-k-mooney | but ya backup is for snapshot roation really | |
| 14:04:32 | dansmith | sean-k-mooney: exactly | |
| 14:04:50 | sean-k-mooney | also backup does not work for BFV as an fyi | |
| 14:05:07 | sean-k-mooney | not sure why but it does not | |
| 14:05:49 | sean-k-mooney | we call it out in the api ref in anycase https://docs.openstack.org/api-ref/compute/#create-server-back-up-createbackup-action | |
| 14:05:56 | sean-k-mooney | so its a know limitaiton not a bug | |
| 14:06:36 | bauzas | okay, I'll try snapshot then | |
| 14:20:04 | bauzas | doh | |
| 14:20:06 | bauzas | eq-67d7b42c-c6a6-46cc-a86b-7eb66afabb79 demo admin] hit limit for project: [Resource image_size_total is over limit of 1000 due to current usage 5257 and delta 0] {{(pid=2376125) enforce_limits /usr/local/lib/python3.9/site-packages/oslo> | |
| 14:25:00 | sean-k-mooney | is that a 1TB limit ? | |
| 14:25:12 | sean-k-mooney | with a 5.2TB image | |
| 14:25:36 | dansmith | no I think it's KiB | |
| 14:25:38 | sean-k-mooney | image size is in GB right | |