| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-07-15 | |||
| 22:52:03 | melwitt | prometheanfire: will look | |
| 22:53:12 | melwitt | I wrote that test 😝 | |
| 22:57:47 | prometheanfire | :D | |
| #openstack-nova - 2020-07-16 | |||
| 00:42:37 | rouk | i cant seem to make nova-serialproxy work correctly past stein, on train it logs that its connecting to the backend, but my client gets punted. with "remote host closed connection: Connection is already closed" | |
| 00:43:13 | rouk | dont see any bugreports on it, and the logs are pretty silent, can anyone verify that it works for them on train? | |
| 01:02:44 | openstackgerrit | melanie witt proposed openstack/nova master: Use unittest.mock instead of mock in test_websocketproxy https://review.opendev.org/741351 | |
| 01:45:03 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.PY2 and six.PY3 https://review.opendev.org/726512 | |
| 01:45:26 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.add_metaclass https://review.opendev.org/726565 | |
| 01:45:37 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898 | |
| 01:53:37 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224 | |
| 01:53:58 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757 | |
| 01:54:07 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777 | |
| 01:54:18 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094 | |
| 01:54:32 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109 | |
| 01:54:50 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | |
| 06:45:05 | gibi | jsuchome: will check | |
| 07:17:55 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936 | |
| 07:17:56 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: release note: add new features introduces by use-pcpu-and-vcpu-in-one-instance https://review.opendev.org/736997 | |
| 07:17:56 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: test: add some notes and new tests for mixed instance https://review.opendev.org/741395 | |
| 07:45:31 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: migrate testing to ubuntu focal https://review.opendev.org/740863 | |
| 08:39:54 | bauzas | good morning Nova | |
| 08:40:05 | bauzas | (after a few PTOs) | |
| 08:43:14 | aarents | good morning | |
| 08:44:29 | gibi | ||
| 08:44:34 | gibi | bauzas, aarents o/ | |
| 08:46:36 | huaqiang | gibi: morning | |
| 08:47:48 | huaqiang | can you review https://review.opendev.org/728480, stephen and alex mentioned that it's better to let you have a look. | |
| 08:48:26 | gibi | huaqiang: ack, I will check | |
| 08:48:36 | huaqiang | thanks! | |
| 08:54:39 | aarents | bauzas: FYI, I updated https://etherpad.opendev.org/p/nova-libvirt-subteam, I re-propose something to fix this bug: https://bugs.launchpad.net/nova/+bug/1732428 here: https://review.opendev.org/#/c/696084/3 I implemented rebase of backing file suggested by lyarwood kashyap | |
| 08:54:39 | openstack | Launchpad bug 1732428 in OpenStack Compute (nova) "Unshelving a VM breaks instance metadata when using qcow2 backed images" [Medium,In progress] - Assigned to Alexandre arents (aarents) | |
| 08:54:59 | kashyap | aarents: Hi | |
| 08:55:35 | bauzas | aarents: ack, will look | |
| 08:55:47 | aarents | kashyap: Hi | |
| 08:55:51 | aarents | bauzas: thks | |
| 08:55:58 | kashyap | Thx for the reminder; been buried w/ other stuff. WIll look | |
| 08:56:27 | aarents | kashyap: ok cool | |
| 08:57:00 | gibi | huaqiang: approved the patch | |
| 08:57:26 | huaqiang | gibi: cool, appreciate! | |
| 09:01:02 | gibi | huaqiang, stephenfin, alex_xu: extended the runway slot of bp/use-pcpu-and-vcpu-in-one-instance with an additional week as I see good progress there | |
| 09:03:24 | huaqiang | gibi: nice to hear this! | |
| 09:04:39 | gibi | nice to see the progress :) | |
| 09:42:50 | stephenfin | bauzas: I'm working on https://review.opendev.org/#/c/741282/ and noticed a potential bug in one of the vGPU functional tests | |
| 09:43:31 | bauzas | stephenfin: a-ha ? | |
| 09:43:41 | stephenfin | sec, getting the links | |
| 09:43:42 | bauzas | yeah, i tried to provide some methods for the func tests | |
| 09:43:54 | stephenfin | bauzas: very small one | |
| 09:44:08 | stephenfin | these tests use v2.1 + latest microversion https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_vgpu.py#L132-L137 | |
| 09:44:17 | stephenfin | these tests do not https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_vgpu.py#L300 | |
| 09:44:40 | stephenfin | both are inheriting from nova.tests.functional.test_servers.ServersTestBase https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_vgpu.py#L41 | |
| 09:44:49 | stephenfin | which uses API version 2 (.0) by default | |
| 09:45:03 | stephenfin | any reason those later tests shouldn't be using v2.1 + latest microversion too? | |
| 09:45:23 | stephenfin | you need the latest microversion so you can request a host https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_vgpu.py#L330 | |
| 09:45:30 | stephenfin | well, a reasonably new one | |
| 09:46:25 | bauzas | stephenfin: yeah, we need a specific microversion for at least one test AFAICU | |
| 09:47:27 | bauzas | stephenfin: at least for the resize tests : https://github.com/openstack/nova/commit/d2e0afc1f72db5cb56ed987e2873798fc1e89545 | |
| 09:49:01 | stephenfin | bauzas: okay, so I'm going to move this https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_vgpu.py#L134-L137 | |
| 09:49:14 | stephenfin | to the base class https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_vgpu.py#L42 | |
| 09:49:36 | stephenfin | doesn't sound like you wanted to use the legact API v2 for the VGPUMultipleTypesTests tests | |
| 09:50:40 | bauzas | stephenfin: I just wanted to provide some tests for vGPU by using the v2 API, but then when I wanted to provide the resize ones, it needed the microversion | |
| 09:50:58 | bauzas | stephenfin: so, no worries if you want to use the latest API for all of them, it should work | |
| 09:51:09 | stephenfin | okay, sweet. Thanks | |
| 09:52:34 | bauzas | the main issue I had with the tests was the fact that I needed to move some methods to the InstanceHelperMixin one | |
| 09:52:59 | stephenfin | You'll probably be interested in https://review.opendev.org/#/c/741283/1 so | |
| 09:53:12 | bauzas | stephenfin: hence the https://github.com/openstack/nova/blob/d2e0afc1f72db5cb56ed987e2873798fc1e89545/nova/tests/functional/integrated_helpers.py#L538 | |
| 09:53:45 | stephenfin | bauzas: You're *definitely* interested in the above so | |
| 09:55:41 | bauzas | stephenfin: :) | |
| 11:06:15 | openstackgerrit | Merged openstack/nova master: notifications: add the pcpuset info to instance numa cell payload https://review.opendev.org/728480 | |
| 11:06:23 | openstackgerrit | Merged openstack/nova master: objects: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267 | |
| 11:18:57 | openstackgerrit | Merged openstack/nova master: hardware: refactor the code for CPU policy sanity check https://review.opendev.org/740925 | |
| 11:21:06 | lyarwood | stephenfin / gibi ; https://review.opendev.org/#/c/699291/ - Would you mind taking a look at this fix from mriedem that I've been trying to push over the line for a while? | |
| 11:22:07 | stephenfin | sure | |
| 11:24:47 | lyarwood | thanks, I'll get back to your series later today btw | |
| 11:26:07 | gibi | lyarwood: I will check it | |
| 11:56:20 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Regenerate device metadata during interface detach https://review.opendev.org/741436 | |
| 12:10:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: [WIP] Support SRIOV interface attach and detach https://review.opendev.org/740995 | |
| 12:10:50 | stephenfin | gibi, sean-k-mooney: Do either of you know why we do this in the resize operation instead of accessing instance.image_meta? https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3886-L3887 | |
| 12:11:15 | stephenfin | I suspect it's something to do with filtering out undesirable image metadata properties but I'm not sure | |
| 12:11:50 | sean-k-mooney | let me check | |
| 12:12:23 | sean-k-mooney | stephenfin: we are using the cached metadta | |
| 12:12:50 | sean-k-mooney | we store the image metadta in the system metadata table | |
| 12:13:46 | sean-k-mooney | if we looked up the image by its uuid and got the metadata it could have change or the image could have been deleted | |
| 12:15:33 | sean-k-mooney | stephenfin: we only have the image uuid in the instance https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L127 | |
| 12:15:41 | sean-k-mooney | thre is no image_meta | |
| 12:16:00 | sean-k-mooney | but if you want to add a property that just calls that function i dont have any issue with that | |
| 12:17:32 | stephenfin | Hmm, I think they may be equivalent. 'Instance.image_meta' calls 'ImageMeta.from_instance' https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L250-L251 | |
| 12:18:00 | sean-k-mooney | oh we do have a property https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L127 | |
| 12:18:00 | stephenfin | which calls 'get_image_from_system_metadata' https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py#L127 | |
| 12:18:26 | stephenfin | hmm, those sound equivalent so. Probably legacy stuff | |
| 12:18:26 | sean-k-mooney | yes https://github.com/openstack/nova/blob/057891ac247fadfb9637d3e834f8b1ef3e39bc27/nova/objects/image_meta.py#L126-L128 | |
| 12:18:38 | gibi | seem equivalent, yes | |
| 12:18:48 | stephenfin | So my series is gaining yet another tech debt reduction patch \o/ | |
| 12:18:51 | stephenfin | every. damn. time. | |
| 12:19:18 | sean-k-mooney | you could pretend you did not see it :P | |
| 12:19:29 | sean-k-mooney | also this seams backwards to me | |
| 12:20:00 | sean-k-mooney | i would have had instance.image_meta just call utils.get_image_from_system_metadata(sysmeta) | |
| 12:20:24 | gibi | stephenfin: you are a hero removing those depts | |
| 12:20:28 | gibi | debts | |
| 12:20:36 | sean-k-mooney | and had image_meta.from_instance(instance) return instance.image_meta | |
| 12:21:20 | stephenfin | fair point | |