| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-09-14 | |||
| 16:02:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add test for SR-IOV neutron ports https://review.opendev.org/746948 | |
| 16:02:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Expand SR-IOV live migration tests with NUMA https://review.opendev.org/749360 | |
| 16:02:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Add live migration tests for PCI, SR-IOV servers https://review.opendev.org/746950 | |
| 16:11:52 | elod | stephenfin: sure, I might arrive to those tomorrow, but I'll review them :] | |
| 16:20:25 | gmann | stephenfin: gibi dansmith can you review this which will help to add ceph job on devstack gate too - https://review.opendev.org/#/c/750078/ | |
| 16:33:00 | melwitt | gmann: I got that one | |
| 16:33:18 | gmann | melwitt: stephenfin thanks | |
| 16:55:36 | dansmith | sorry, was on a call | |
| 18:12:09 | openstackgerrit | Merged openstack/nova master: Run voting jobs in gate pipeline also https://review.opendev.org/750078 | |
| 18:23:33 | tosky | lyarwood: I left a comment in the nova-evacuate review, but I may not have been clear enough, so in case please ask | |
| 18:29:19 | lyarwood | tosky: yeah understood, was hoping I wouldn't have to port this now but I'll give it a go this evening | |
| 19:31:53 | openstackgerrit | Merged openstack/nova master: Remove write_to_file. https://review.opendev.org/645087 | |
| 19:32:00 | openstackgerrit | Merged openstack/nova master: conductor: Don't use setattr https://review.opendev.org/746947 | |
| 19:32:08 | openstackgerrit | Merged openstack/nova master: functional: Add test for SR-IOV neutron ports https://review.opendev.org/746948 | |
| 21:12:47 | openstackgerrit | Adam Harwell proposed openstack/nova master: Add scheduler filter for routed_segments https://review.opendev.org/709280 | |
| 21:40:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883 | |
| 22:03:39 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883 | |
| 22:53:06 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883 | |
| 23:05:06 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Do not reference VIR_ERR_DEVICE_MISSING when libvirt is < v4.1.0 https://review.opendev.org/747361 | |
| #openstack-nova - 2020-09-15 | |||
| 02:07:27 | openstackgerrit | Hemanth Nakkina proposed openstack/nova master: Update pci stat pools based on PCI device changes https://review.opendev.org/749175 | |
| 07:39:18 | openstackgerrit | Federico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain https://review.opendev.org/750652 | |
| 08:03:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: virt: Remove 'reset_network' API https://review.opendev.org/749315 | |
| 08:03:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: virt: Remove 'change_instance_metadata' API https://review.opendev.org/749316 | |
| 08:08:57 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883 | |
| 08:16:14 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883 | |
| 08:34:25 | bauzas | whoops my manners | |
| 08:34:31 | bauzas | good morning Nova | |
| 08:34:54 | lyarwood | Morning | |
| 08:44:47 | yingjisun | I found thousands of "2020-09-14 19:56:17.152 1869 INFO nova.metadata.wsgi.server [-] 192.168.131.34,192.168.131.68 "GET /openstack/latest/vendor_data.json HTTP/1.1" status: 200 len: 128 time: 0.0072370" | |
| 08:45:33 | yingjisun | is there a periodic task that query the metadata ? | |
| 08:48:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: test_evacuate: Wait until subnode is down before starting tests https://review.opendev.org/751065 | |
| 09:07:03 | bauzas | yingjisun: well, I'd expect cloud-init to fetch this vendordata | |
| 09:07:16 | bauzas | so, litterally every instance which is booting | |
| 09:08:07 | yingjisun | this is the strange point. I see thousands of such log with 15 minutes. I do not think there are so much instance booting | |
| 09:08:18 | bauzas | sean-k-mooney: stephenfin: I honestly think we shouldn't deprecate stuff post-FF | |
| 09:08:23 | bauzas | sean-k-mooney: stephenfin: https://review.opendev.org/#/c/745605/ | |
| 09:09:06 | lyarwood | bauzas: For what reason? | |
| 09:09:27 | stephenfin | It seemed super low risk | |
| 09:09:33 | yingjisun | I only see 68 "Starting instance ..." in the log. | |
| 09:09:45 | bauzas | lyarwood: because we have something called "Feature Freeze" | |
| 09:09:58 | bauzas | even with low risk deprecations | |
| 09:10:10 | bauzas | that's maybe me who is a bit conservative | |
| 09:10:12 | stephenfin | yingjisun: https://docs.openstack.org/nova/latest/admin/vendordata.html | |
| 09:10:18 | bauzas | but is it really a thing ? | |
| 09:10:26 | stephenfin | yingjisun: The vendordata feature provides a way to pass vendor or deployment-specific information to instances. This can be accessed by users using the metadata service or with config drives. | |
| 09:10:28 | lyarwood | sorry I don't see how feature freeze is even related to deprecations | |
| 09:10:40 | bauzas | lyarwood: you're possibly right | |
| 09:10:45 | bauzas | I just raise a concern | |
| 09:10:55 | lyarwood | ff is for new things, deprecations are just to highlight removal in a future release | |
| 09:10:56 | bauzas | in theory, RCs are there for focusing on bugds | |
| 09:11:13 | bauzas | but I'm pedantic | |
| 09:11:15 | lyarwood | yeah true I just didn't map deprecations to adding risk | |
| 09:11:22 | lyarwood | removing stuff maybe | |
| 09:11:45 | lyarwood | the only reason I'm still going through stephenfin's xenapi removal stuff is because the bulk has already landed | |
| 09:11:53 | lyarwood | so we are only removing stuff that's dead anyway now | |
| 09:12:41 | stephenfin | yingjisun: I suspect you have either something like cloud-init on the guests indexing the metadata service | |
| 09:12:48 | stephenfin | s/either // | |
| 09:14:03 | bauzas | stephenfin: that's my expectations | |
| 09:14:16 | stephenfin | lyarwood, bauzas: Yeah, it would be good to discuss as a team whether we want to proceed with that remove-xenapi series and close it out this cycle, or wait until Wallaby | |
| 09:14:17 | bauzas | that's what I explained | |
| 09:14:24 | stephenfin | bauzas: oh, so you did /o\ | |
| 09:14:29 | bauzas | it's cloud-init thingies | |
| 09:14:30 | stephenfin | sorry, I missed that | |
| 09:14:35 | bauzas | so litterally a guest question | |
| 09:14:38 | stephenfin | yuuup | |
| 09:14:48 | lyarwood | stephenfin: yeah I'd like to remove the already dead stuff before RC if we can | |
| 09:15:05 | lyarwood | or did I miss rc already being cut | |
| 09:15:12 | bauzas | no | |
| 09:15:16 | bauzas | next week | |
| 09:15:23 | bauzas | and https://review.opendev.org/#/c/751045/1 is -W because of this | |
| 09:15:44 | bauzas | I'd certainly prefer if we would stop merging things that would touch nova features | |
| 09:15:52 | lyarwood | k, if we have a week I'd say push ahead removing dead code but would be good to bring up on Thursday if it's still going on | |
| 09:16:10 | lyarwood | bauzas: FWIW the bulk of the removal has already landed so listing it there is valid now | |
| 09:16:21 | bauzas | lyarwood: this is | |
| 09:16:34 | bauzas | my concern was more about any other deprecation or removal we were about to provide | |
| 09:17:06 | lyarwood | kk what's the deadline for the prelude? after rc? | |
| 09:17:32 | lyarwood | IOW can we just set a deadline of rc for deprecations and respin the prelude? | |
| 09:17:44 | bauzas | lyarwood: no, prelude has to merge before we branch | |
| 09:17:52 | stephenfin | it has to merge before the branch | |
| 09:17:54 | stephenfin | yeah | |
| 09:17:58 | bauzas | but again, the whole point of RCs is to focus on bugfixes | |
| 09:18:06 | stephenfin | otherwise it's not included in the 22.0.0 tag | |
| 09:18:12 | lyarwood | yeah understood | |
| 09:18:23 | lyarwood | but we aren't at rc yet soooooo | |
| 09:18:42 | lyarwood | I'd think deprecations are fine until we land the prelude and branch? | |
| 09:18:49 | bauzas | and us pushing things down the line after FF makes me a bit sad since we all communicate on the glorious ideal of whight knights only caring about bugs for the next 2 weeks | |
| 09:19:04 | bauzas | call me an idealist | |
| 09:19:32 | lyarwood | isn't that post rc? | |
| 09:19:37 | stephenfin | bauzas: personally, I can't imagine us adding anything big enough to warrant inclusion in the prelude so I think that could be fixed up to mention XenAPI/VMWareAPI changes and merged | |
| 09:19:40 | lyarwood | I know we do it between ff and rc as well | |
| 09:19:53 | lyarwood | but rc to ga is critical bugfixes etc as well right? | |
| 09:20:34 | bauzas | lyarwood: no, RCs are intended for bugfixes, including the period between M3 and RC1 | |
| 09:20:43 | stephenfin | lyarwood: on an RC-related note, what have we decided on Focal changes for Victoria? Are we simply skipping those tests until the QEMU fix lands in 20.04? | |
| 09:20:51 | bauzas | the period between RC1 and GA being focused on *regressions* | |
| 09:21:06 | stephenfin | yes, I realize I should know this already /o\ | |
| 09:21:17 | lyarwood | stephenfin: I think the TC are pushing ahead with the focal change so yeah | |
| 09:21:23 | bauzas | so in the world of OpenStack, in theory you only focus on bugs for only two weeks in a 6-month timeframe \o/ | |
| 09:21:27 | lyarwood | stephenfin: loads of jobs don't skip them still | |
| 09:25:53 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Victoria 22.0.0 prelude section https://review.opendev.org/751045 | |
| 09:26:23 | gibi | bauzas, stephenfin, lyarwood: regarding deprecatios post FF, I'm OK to finish up the xenapi removal before RC1 but in general please focus on bug triage instead of proposing new deprecations | |