| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-09 | |||
| 14:03:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 14:06:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 15:02:48 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Switch the default video model from 'cirrus' to 'virtio' https://review.opendev.org/c/openstack/nova/+/798680 | |
| 16:17:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 16:17:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 16:17:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interaface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 16:17:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test]Refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 16:19:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 17:09:54 | melwitt | lyarwood: the error in https://bugs.launchpad.net/nova/+bug/1931707 is the same but downstream it's happening during a server show | |
| 18:23:35 | opendevreview | Rodrigo Barbieri proposed openstack/nova stable/stein: Error anti-affinity violation on migrations https://review.opendev.org/c/openstack/nova/+/800114 | |
| 18:53:45 | opendevreview | Rodrigo Barbieri proposed openstack/nova master: Improve prep_resize reschedule unit test https://review.opendev.org/c/openstack/nova/+/800297 | |
| 22:33:33 | opendevreview | Merged openstack/nova stable/train: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787316 | |
| 22:39:48 | opendevreview | melanie witt proposed openstack/nova stable/stein: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/791402 | |
| 22:40:34 | opendevreview | melanie witt proposed openstack/nova master: Make test_archive_task_logs deterministic https://review.opendev.org/c/openstack/nova/+/800313 | |
| #openstack-nova - 2021-07-10 | |||
| 05:44:17 | opendevreview | Merged openstack/nova stable/ussuri: Ignore PCI devices with 32bit domain https://review.opendev.org/c/openstack/nova/+/791771 | |
| 14:38:05 | opendevreview | Takashi Natsume proposed openstack/nova-specs master: Create specs directory for Yoga https://review.opendev.org/c/openstack/nova-specs/+/800324 | |
| #openstack-nova - 2021-07-12 | |||
| 01:08:52 | opendevreview | Shuai Qian proposed openstack/nova master: support evacuating instance in soft_deleted https://review.opendev.org/c/openstack/nova/+/800357 | |
| 06:33:57 | opendevreview | ruishawn proposed openstack/nova master: Add armv6l to arch list https://review.opendev.org/c/openstack/nova/+/799964 | |
| 10:14:44 | sean-k-mooney | stephenfin: can you take a look again at this spec form lyarwood https://review.opendev.org/c/openstack/nova-specs/+/799624 | |
| 10:15:14 | sean-k-mooney | stephenfin: gibi and i are both +2 on it but wanted to give you a chance to take a look before applying +w | |
| 10:15:22 | stephenfin | ack, will do | |
| 10:19:05 | gibi | ack | |
| 10:21:50 | sean-k-mooney | while ye are both here im going to try and poc the claim in the sucuder utils today and update teh pci spec accourdingly. ill proably adress stephens other comment first and then do that update later today or tomorow | |
| 10:22:18 | sean-k-mooney | realistically however im not sure we will have time to agree the direction before thursday so this might just fall to next cycle | |
| 10:26:37 | gibi | sean-k-mooney: ack, I will be available for review | |
| 10:26:44 | gibi | but I agree that this might slip to next | |
| 10:29:21 | sean-k-mooney | trivial yoga spec directory path if people want take a look https://review.opendev.org/c/openstack/nova-specs/+/800324 | |
| 10:30:28 | gibi | sean-k-mooney: done | |
| 10:30:36 | sean-k-mooney | gibi++ | |
| 10:47:32 | opendevreview | Merged openstack/nova-specs master: Create specs directory for Yoga https://review.opendev.org/c/openstack/nova-specs/+/800324 | |
| 11:14:30 | opendevreview | Merged openstack/nova-specs master: Add nova-manage commands to show and refresh connection_info https://review.opendev.org/c/openstack/nova-specs/+/799624 | |
| 12:27:10 | opendevreview | Lee Yarwood proposed openstack/nova-specs master: Store and allow libvirt instance device buses and models to be updated https://review.opendev.org/c/openstack/nova-specs/+/799811 | |
| 12:28:03 | lyarwood | stephenfin sean-k-mooney gibi ; ^ I've submitted to the image-property command given it was 2 vs 1, can you take another swing at that this afternoon if you have time | |
| 12:38:44 | gibi | sure | |
| 12:41:57 | gibi | I'm still +2 :) | |
| 12:49:25 | lyarwood | cool thanks | |
| 13:13:02 | sean-k-mooney | lyarwood: im also +2 on that version ill wait for stephenfin to review | |
| 13:13:44 | sean-k-mooney | the imporant thing for me is if/when we expand the list we dont need to update any docs or add another command other then extending the list | |
| 13:58:17 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Enable auto-generation of migrations https://review.opendev.org/c/openstack/nova/+/800077 | |
| 14:10:20 | sean-k-mooney | ah i was going to ask the docs for that are here https://review.opendev.org/c/openstack/nova/+/800078/ in a followup | |
| 14:10:49 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Add documentation on database migrations https://review.opendev.org/c/openstack/nova/+/800078 | |
| 14:11:38 | sean-k-mooney | oh and you are using the venv target for tox ok i was going to ask if add a specific venv to create migrations | |
| 14:11:52 | sean-k-mooney | venv works we use that for reno already | |
| 14:17:15 | opendevreview | Merged openstack/nova-specs master: Store and allow libvirt instance device buses and models to be updated https://review.opendev.org/c/openstack/nova-specs/+/799811 | |
| 14:34:07 | elodilles | lyarwood gibi bauzas : I've pushed the release patches, please review them when you have time https://review.opendev.org/q/project:openstack/releases+is:open+intopic:nova | |
| 14:34:16 | gibi | elodilles: thanks | |
| 14:34:26 | elodilles | also let me know if we should wait for any patch to be included | |
| 14:36:19 | lyarwood | ack looking | |
| 14:37:01 | opendevreview | Stephen Finucane proposed openstack/nova master: db: Final cleanups https://review.opendev.org/c/openstack/nova/+/800484 | |
| 14:50:51 | opendevreview | Stephen Finucane proposed openstack/nova master: Add 'hw:vif_multiqueue_enabled' flavor extra spec https://review.opendev.org/c/openstack/nova/+/792356 | |
| 14:50:52 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Document virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/792362 | |
| 14:50:53 | opendevreview | Stephen Finucane proposed openstack/nova master: Move 'hw:pmu', 'hw_pmu' parsing to nova.virt.hardware https://review.opendev.org/c/openstack/nova/+/792364 | |
| 15:45:55 | stephenfin | sean-k-mooney: Are you planning to respin https://review.opendev.org/c/openstack/os-vif/+/798055 ? | |
| 16:03:39 | sean-k-mooney | stephenfin: oh am yes, maybe tomorrow | |
| 16:03:57 | sean-k-mooney | ill try and do it either later this eveining or in the morning | |
| 16:07:42 | jgwentworth | stephenfin: hey, I dunno if you saw my review on https://review.opendev.org/c/openstack/nova/+/706295 | |
| 16:09:19 | stephenfin | I did not. Weird. Maybe a merge conflict resolution gone wrong? | |
| 16:12:41 | stephenfin | nope, just copy-paste mistake from the other exceptions in there; will fix | |
| 16:14:05 | opendevreview | Merged openstack/nova master: Add func test for nova-manage db archive_deleted_rows --before https://review.opendev.org/c/openstack/nova/+/796744 | |
| 16:18:06 | stephenfin | actually, I was copy-pasting from something else that was already broken and propagating the mistake \o/ https://github.com/openstack/nova/blob/master/nova/network/neutron.py#L434-L438 | |
| 16:24:56 | opendevreview | Stephen Finucane proposed openstack/nova master: Use neutronclient's port binding APIs https://review.opendev.org/c/openstack/nova/+/706295 | |
| 19:03:29 | opendevreview | sean mooney proposed openstack/os-vif master: add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 19:12:49 | opendevreview | sean mooney proposed openstack/os-vif master: add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 19:14:46 | sean-k-mooney | stephenfin: ^ ok that should have adressed your nits | |
| 20:44:39 | opendevreview | sean mooney proposed openstack/nova master: [WIP] move instance claim to scheduler. https://review.opendev.org/c/openstack/nova/+/800560 | |
| 21:35:19 | opendevreview | melanie witt proposed openstack/nova-specs master: Repropose: Support Consumer Types in placement https://review.opendev.org/c/openstack/nova-specs/+/800569 | |
| 21:59:03 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose nova-audit spec for Xena https://review.opendev.org/c/openstack/nova-specs/+/800570 | |
| 22:01:55 | opendevreview | melanie witt proposed openstack/nova-specs master: Repropose: Support Consumer Types in placement https://review.opendev.org/c/openstack/nova-specs/+/800569 | |
| #openstack-nova - 2021-07-13 | |||
| 06:28:58 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 06:28:58 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 06:29:00 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - build instance with smartnic arqs https://review.opendev.org/c/openstack/nova/+/798249 | |
| 06:29:00 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - create arqs https://review.opendev.org/c/openstack/nova/+/758944 | |
| 06:29:01 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 06:29:01 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | |
| 06:29:02 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 08:41:15 | gibi | melwitt: I just saw your consumer type spec. I think I created a bad precedent with my RP re-parenting spec proposed in the nova-specs repo. | |
| 08:41:27 | gibi | At that time I thought that there is no spec tracking in placement | |
| 08:41:44 | gibi | but it turned out that there is ./doc/source/specs in the placement repository | |
| 08:44:02 | gibi | so we have two ways forward. Make the placement spec directiory freezed and use nova-specs for placement specs too, or duplicate my RP re-parenting spec there and move you consumer type spec to there. | |
| 09:22:09 | kashyap | sean-k-mooney: Hey, so I've got a Red Hat QE to do some migration tests w/ Cirrus and VirtIO - for Windows and Linux | |
| 09:23:18 | kashyap | sean-k-mooney: It all "just works". In brief, this was the test: | |
| 09:25:02 | kashyap | Have a Linux and Windows guest on source with 'cirrus', change the video model to 'virtio'; reboot the guest, and then live-migrate -- it succeeds just fine. | |
| 09:26:03 | kashyap | sean-k-mooney: And thanks for the review; just saw that | |
| 09:48:24 | sean-k-mooney | ok thats good to know for the new nova-manage command | |
| 09:49:29 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI to look up actual remote address https://review.opendev.org/c/openstack/placement/+/800611 | |
| 09:51:25 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI to look up actual remote address https://review.opendev.org/c/openstack/placement/+/800611 | |
| 09:53:06 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | |
| 09:57:55 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | |
| 10:00:30 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | |
| 10:03:35 | opendevreview | Stephen Finucane proposed openstack/nova master: Use neutronclient's port binding APIs https://review.opendev.org/c/openstack/nova/+/706295 | |
| 10:03:39 | stephenfin | gibi: ^ | |
| 10:04:13 | stephenfin | gibi: Fixed the single failing test and addressed the mypy complaint so that should be green now. I'll remind melwitt when she's about too | |
| 10:32:50 | gibi | stephenfin: ack, looking | |
| 11:21:56 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | |
| 12:49:09 | opendevreview | Lee Yarwood proposed openstack/nova master: WIP/DNM nova-manage: Introduce bdm show and refresh commands https://review.opendev.org/c/openstack/nova/+/800634 | |
| 13:31:31 | ganso | lyarwood, bauzas: if you have a minute to spare could you please take a look at this last backport of the anti-affinity series? It already has a +2, just needs a +W: https://review.opendev.org/c/openstack/nova/+/800114 Thanks in advance! | |
| 15:20:13 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | |
| 15:48:04 | gibi | nova meeting starts in 12 minutes here in the channel | |
| 15:52:05 | gibi | stephenfin: you might want to drop your -1 from https://review.opendev.org/c/openstack/project-config/+/798071 as you proposal was accepted last week on the meeting | |
| 15:52:36 | stephenfin | done | |