| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-12-09 | |||
| 10:55:14 | gibi | when we finally meet again | |
| 10:55:43 | bauzas | well, if I was paying for a vine in 2019 for the next meeting, then it would be a very nice one :) | |
| 10:56:05 | bauzas | after 3 years | |
| 10:56:42 | sean-k-mooney | gibi: what is the invalid input in this case by the way | |
| 10:57:00 | bauzas | hmmm, https://docs.openstack.org/api-ref/compute/?expanded=list-security-groups-by-server-detail#servers-security-groups-servers-os-security-groups | |
| 10:57:26 | bauzas | unfortunately, we don't document the fact that the POST action is now deprecated | |
| 10:57:37 | gibi | sean-k-mooney: security_group is expected to be a dict not a string | |
| 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 | |