| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-12-09 | |||
| 10:57:49 | sean-k-mooney | ah ok | |
| 10:57:55 | gibi | These APIs are proxy calls to the Network service. Nova has deprecated all the proxy APIs and users should use the native APIs instead. These will fail with a 404 starting from microversion 2.36. See: Relevant Network APIs. | |
| 10:58:03 | bauzas | gibi: yeah I know | |
| 10:58:15 | sean-k-mooney | yep the post action i tough twas only vailid for nova networks | |
| 10:58:15 | gibi | I think this is for the whole /os-security-groups resource | |
| 10:58:20 | bauzas | gibi: I'm just saying we don't document it in our API docs :p | |
| 10:58:56 | bauzas | gibi: well, then the GET /os-security-groups action should be told to by deprecated in our API docs :) | |
| 10:58:59 | gibi | I think the documentation is there in the header | |
| 10:59:13 | bauzas | oh, did I miss something in the docs ? | |
| 10:59:15 | sean-k-mooney | https://docs.openstack.org/api-ref/compute/?expanded=list-security-groups-by-server-detail#list-security-groups | |
| 10:59:30 | sean-k-mooney | bauzas: its in the os-security-groups section | |
| 10:59:38 | sean-k-mooney | These APIs are proxy calls to the Network service. Nova has deprecated all the proxy APIs and users should use the native APIs instead. These will fail with a 404 starting from microversion 2.36. See: Relevant Network APIs. | |
| 10:59:47 | bauzas | I'm blind | |
| 10:59:58 | sean-k-mooney | you were looking at the instance action | |
| 11:00:01 | sean-k-mooney | that does not have the hearder | |
| 11:00:28 | sean-k-mooney | but the os-security-groups endpoint section does | |
| 11:00:35 | bauzas | what the f*** | |
| 11:00:40 | sean-k-mooney | bauzas: https://docs.openstack.org/api-ref/compute/?expanded=list-security-groups-by-server-detail#delete-deallocate-floating-ip-address | |
| 11:00:45 | sean-k-mooney | the big red box ^ | |
| 11:00:49 | bauzas | yeah I get it now | |
| 11:00:56 | sean-k-mooney | :) | |
| 11:01:12 | bauzas | but... can't get why I wasn't able to see it in my own link | |
| 11:01:24 | sean-k-mooney | bauzas: because you were not looking in the right section | |
| 11:01:42 | sean-k-mooney | you were looking at /servers/uuid/os-secuirty-groups | |
| 11:01:49 | sean-k-mooney | that was not deprecated | |
| 11:01:59 | bauzas | Ctrl+F | |
| 11:02:12 | sean-k-mooney | but /os-security-groups and /os-security-group-rules were | |
| 11:02:13 | bauzas | not the right endpoint | |
| 11:02:28 | sean-k-mooney | yep easy mistake to make | |
| 11:02:30 | sean-k-mooney | brb | |
| 11:07:37 | bauzas | gibi: done, you owe me one :p | |
| 11:07:49 | gibi | thanks, and noted :) | |
| 13:40:33 | plibeau | lyarwood: To explain what append in my case: context, the image used during the boot of the instance is now private and the base image on the source compute don't exsit. | |
| 13:40:36 | plibeau | during the resize on the instance in the code we go in "_try_fetch_image_cache" (nova/virt/libvirt/driver.py +10359) and go in except because image.cache can't found image (because now the image is private). In the except image.cache is call again without image in parameter so create_image in call also without image_id. generating is True because image_id not found in kwargs | |
| 13:40:37 | plibeau | and the self.exists because it's a resize so the disk already exist. | |
| 13:40:39 | plibeau | I don't change in image.cache method the line 280 (if os.path.exists(base) and size > self.get_disk_size(base):) because it's the same code for other type of type (qow2,etc) and it's not good idea IMPOV to change that but mayY | |
| 13:40:50 | plibeau | https://review.opendev.org/c/openstack/nova/+/820531 | |
| 13:46:11 | plibeau | *but maybe I'm wrong | |
| 14:02:24 | opendevreview | sean mooney proposed openstack/nova master: This change adds generative envs https://review.opendev.org/c/openstack/nova/+/804292 | |
| 14:04:08 | opendevreview | sean mooney proposed openstack/nova master: This change adds generative envs https://review.opendev.org/c/openstack/nova/+/804292 | |
| 14:41:08 | opendevreview | sean mooney proposed openstack/nova master: [WIP] allow cpus to be externally managed https://review.opendev.org/c/openstack/nova/+/821228 | |
| 14:55:36 | lyarwood | plibeau: Apologies was on calls downstream | |
| 14:59:02 | lyarwood | I think I see what you mean | |
| 15:00:53 | lyarwood | and apologies I was mixing resize and rebuild | |
| 15:13:19 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments. https://review.opendev.org/c/openstack/nova/+/820935 | |
| 15:24:04 | plibeau1 | okay I can copy my detail in the change if you want? And you prefer another change for the refacto? | |
| 15:31:51 | lyarwood | plibeau1: same change, new patchset would be fine I think? | |
| 15:33:14 | plibeau1 | probably yes I will check to apply your proposal on unit test | |
| 15:45:14 | lyarwood | yeah just amend the commit and git review again | |
| 16:31:15 | opendevreview | Pierre Libeau proposed openstack/nova master: Nova resize don't extend disk in one specific case https://review.opendev.org/c/openstack/nova/+/820531 | |
| 16:38:43 | gmann | gibi: sean-k-mooney bauzas pslestang : I am fine not to change API as no one asked it. but +1 on spec to get clear view/documentation | |
| 17:27:59 | sean-k-mooney | gmann: ack | |
| 17:47:18 | opendevreview | Balazs Gibizer proposed openstack/nova master: Log which instance event was timed out https://review.opendev.org/c/openstack/nova/+/819817 | |
| 19:24:53 | mloza | /j #openstack-keystone | |
| 19:28:56 | opendevreview | sean mooney proposed openstack/nova-specs master: add per process healthcheck spec https://review.opendev.org/c/openstack/nova-specs/+/821279 | |
| 21:17:37 | opendevreview | melanie witt proposed openstack/nova master: Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| #openstack-nova - 2021-12-10 | |||
| 01:19:34 | opendevreview | melanie witt proposed openstack/nova stable/xena: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821348 | |
| 01:21:32 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821349 | |
| 01:26:52 | opendevreview | melanie witt proposed openstack/nova stable/victoria: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821350 | |
| 01:28:40 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821351 | |
| 01:30:57 | opendevreview | melanie witt proposed openstack/nova stable/train: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821352 | |
| 03:26:15 | opendevreview | Wenping Song proposed openstack/nova master: Fill the AcceleratorRequestBindingFailed exception msg info https://review.opendev.org/c/openstack/nova/+/817326 | |
| 05:45:45 | opendevreview | Wenping Song proposed openstack/nova master: Generate cyborgclient when there is device_prfoile in instance https://review.opendev.org/c/openstack/nova/+/821368 | |
| 08:00:20 | opendevreview | Wenping Song proposed openstack/nova master: Cleanup guest process when vm evacuate failed and then deleted https://review.opendev.org/c/openstack/nova/+/814746 | |
| 08:14:32 | nautik | If anyone wants to review this MR you are welcome: https://review.opendev.org/c/openstack/nova/+/781076 | |
| 08:15:24 | nautik | There are a few "build failed" messages but I think they are just temporary / testing infra issues | |
| 09:37:31 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Repropose boot a vm with unaddressed port https://review.opendev.org/c/openstack/nova-specs/+/821376 | |
| 10:19:48 | opendevreview | Pierre Libeau proposed openstack/nova master: Nova resize don't extend disk in one specific case https://review.opendev.org/c/openstack/nova/+/820531 | |
| 10:27:45 | opendevreview | Wenping Song proposed openstack/nova master: Fill the exception msg https://review.opendev.org/c/openstack/nova/+/821385 | |
| 10:43:35 | opendevreview | Pierre-Samuel Le Stang proposed openstack/nova-specs master: Implements: blueprint soft-delete-instance-actions https://review.opendev.org/c/openstack/nova-specs/+/821387 | |
| 10:46:51 | pslestang | hello! | |
| 10:51:30 | gibi | o/ | |
| 10:57:56 | kashyap | pslestang: Most ppl (at least in the West) might be off due to the year-end PTO. So expect delayed responses | |
| 11:31:31 | gibi | fyi, it seems that the nova-ceph-multistore job is broken on master https://zuul.opendev.org/t/openstack/builds?job_name=nova-ceph-multistore&project=openstack/nova I will file a bug | |
| 11:31:47 | gibi | https://zuul.opendev.org/t/openstack/build/722c6caf8e454849b897a43bcf617dd2/log/job-output.txt#9419 | |
| 11:31:52 | gibi | example ^^ | |
| 11:32:05 | sean-k-mooney | ROR: Cannot uninstall 'logutils'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall. | |
| 11:32:16 | gibi | yepp that one | |
| 11:32:35 | gibi | the last 4 build hit the same | |
| 11:32:36 | sean-k-mooney | ya so that will proably break other things | |
| 11:32:52 | sean-k-mooney | i guess we just need to update devstack | |
| 11:32:59 | sean-k-mooney | to remove that package or mask it | |
| 11:33:46 | sean-k-mooney | https://packages.ubuntu.com/source/focal/python-logutils | |
| 11:34:47 | sean-k-mooney | looks like its packaged for openstack in debian | |
| 11:35:04 | sean-k-mooney | so i guess it should be safe to remove | |
| 11:37:32 | gibi | filed a bug https://bugs.launchpad.net/nova/+bug/1954427 | |
| 11:41:09 | sean-k-mooney | https://review.opendev.org/c/openstack/devstack/+/821394 | |
| 11:41:25 | sean-k-mooney | not sure if that will work but we can see if it fixes it | |
| 11:41:36 | gibi | sean-k-mooney: awesome, I was still searching for the place in devstack to fix it | |
| 11:42:30 | sean-k-mooney | that might be too early | |
| 11:42:39 | sean-k-mooney | lookign at the logs its installed as part of installing ceph | |
| 11:42:47 | sean-k-mooney | so i might need to fix this in the ceph plugin instead | |
| 11:42:59 | sean-k-mooney | its pulled in by ++ functions-common:apt_get:1207 : sudo DEBIAN_FRONTEND=noninteractive http_proxy= https_proxy= no_proxy= apt-get --option Dpkg::Options::=--force-confold --assume-yes install ceph libnss3-tools python3-rados python3-rbd | |
| 11:43:11 | gibi | that can explain why we only see this breaking the ceph job only | |
| 11:43:22 | sean-k-mooney | ill quickly add the same to the ceph plugin | |
| 11:43:42 | gibi | this is what I call teamwork :) | |
| 11:49:41 | sean-k-mooney | im going to push a patch that may not work as it might unistalll ceph again but if that is what happens we can just remove the egg infor files to work aroudn it as we do in teh fixup fucntion | |
| 11:50:23 | gibi | OK | |
| 11:50:51 | sean-k-mooney | https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/821396 | |
| 11:52:28 | sean-k-mooney | im hoping its just a recommended package rather then a required one for those packages | |