Posted	Nick	Remark
#openstack-nova - 2018-04-26
08:40:57	kashyap	But I filed it as a mostly "item tracker"
08:41:19	kashyap	Maybe it's a habit out of dealing with Bugzilla
08:42:03	openstackgerrit	Stephen Finucane proposed openstack/nova master: Handle deprecation of inspect.getargspec https://review.openstack.org/530777
09:02:59	bauzas	cores, easy +W for a High priority backport potential bugfix https://review.openstack.org/#/c/564257/
09:03:08	bauzas	gibi, stephenfin: ^
09:03:30	stephenfin	bauzas: Um, I already did that?
09:03:44	bauzas	oops my bad
09:03:57	stephenfin	Some people are so ungrateful
09:04:20	bauzas	stephenfin: <3
09:04:40	bauzas	stephenfin: I can spoil you InfinityWars if you want
09:04:53	stephenfin	bauzas: I'm pretty sure I know where you live
09:09:46	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:09:47	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
09:14:52	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:14:53	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
09:15:57	bauzas	stephenfin: /me is afraid
09:17:23	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:17:24	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
09:27:51	stephenfin	How to confuse oneself: fire up 'tox -e py35' that was recreated a while back and is missing dependencies. Try to figure out what's going on from the wall of text
09:27:58	stephenfin	That's got to be a bug with stestr
09:40:40	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
09:40:41	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
09:40:42	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
09:49:10	lyarwood	stephenfin: thanks again
10:08:30	openstackgerrit	Merged openstack/nova master: libvirt: fix hard reboot issue with mdevs https://review.openstack.org/564257
10:23:28	stephenfin	patchbomb incoming
10:23:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add NUMANetworkTopology object https://review.openstack.org/564439
10:23:52	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440
10:23:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.networks field https://review.openstack.org/564441
10:23:55	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add NetworkRequest.physnet, tunnelled https://review.openstack.org/564442
10:23:56	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Unchain _get_phynet_info from _get_port_vnic_info https://review.openstack.org/564443
10:23:57	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Add 'populate_networks' to network API https://review.openstack.org/564444
10:23:58	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Add '_get_network_tunnel_status' https://review.openstack.org/564445
10:23:59	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Add network_requests to instance_extra https://review.openstack.org/564446
10:24:00	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add Instance.network_requests https://review.openstack.org/564447
10:24:01	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Start accounting for network requests https://review.openstack.org/564448
10:24:02	openstackgerrit	Stephen Finucane proposed openstack/nova master: rt: Start utilizing Instance.network_requests https://review.openstack.org/564449
10:24:03	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add RequestSpec.requested_networks https://review.openstack.org/564450
10:24:04	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Remove unnecessary default parameter https://review.openstack.org/564451
10:24:05	openstackgerrit	Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing network_requests https://review.openstack.org/564452
10:24:47	stephenfin	gibi, bauzas, jaypipes, sean-k-mooney[m]: voilà ^
10:30:39	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: libvirt: fix hard reboot issue with mdevs https://review.openstack.org/564454
11:02:07	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_NUMA_VERSION_PPC https://review.openstack.org/564010
11:02:07	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_BLOCK_LM_WITH_VOLUMES_VERSION https://review.openstack.org/563984
11:02:09	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/564012
11:02:09	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: Drop BAD_LIBVIRT_NUMA_VERSIONS https://review.openstack.org/564011
11:02:10	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD https://review.openstack.org/564013
11:04:08	kashyap	Darn, one more time
11:04:15	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_BLOCK_LM_WITH_VOLUMES_VERSION https://review.openstack.org/563984
11:04:16	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_NUMA_VERSION_PPC https://review.openstack.org/564010
11:04:17	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_NUMA_VERSIONS https://review.openstack.org/564011
11:04:18	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/564012
11:04:19	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD https://review.openstack.org/564013
11:11:02	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Fix dropped check for boot_index 0 in _validate_bdm https://review.openstack.org/564461
11:18:21	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: xenapi: handle InstanceNotFound in detach_interface() https://review.openstack.org/564468
11:36:11	efried	ō/
11:36:31	mriedem	efried: ha, that inspect.getargspec warnings filter explodes on anything using mox
11:36:59	efried	mriedem: Niiice. I didn't look too closely at the failures.
11:37:15	efried	mriedem: We should... get rid of mox.
11:38:18	mriedem	- Failed: 718
11:39:29	mriedem	https://github.com/openstack/mox3/blob/285bc2a1760d460080df79f1ed3b0988cab44f7b/mox3/mox.py#L909
11:39:37	mriedem	could patch mox3
11:40:42	kashyap	efried: Hehe, unrelated: I saw you comment about the typo: "Teset" -- https://review.openstack.org/#/c/554438/3/nova/tests/unit/test_utils.py
11:40:50	kashyap	efried: That is ... much better than the typo mriedem makes :P
11:40:56	kashyap	(You know what it is.)
11:41:02	mriedem	testes
11:41:11	kashyap	Indeed, :D
11:41:54	efried	We *could* patch mox3 -- to use mock.
11:42:11	efried	surely somebody's done that somewhere.
11:42:13	mriedem	it looks like it's usage is best effort and optional
11:42:25	mriedem	for inspect.getargspec
11:42:41	mriedem	so easy thing would just be adding an "if six.PY3: raise ValueError()"
11:42:57	mriedem	anywho, the warnings filter is low priority now
11:43:20	efried	Yeah, totally. But leaving the patch open is goodness. And we can revisit it when mox is all gone.
11:43:38	efried	takashin has been doing a good job there.
11:44:32	openstack	Launchpad bug 1766208 in OpenStack Compute (nova) "Lift the restriction on choices for `cpu_model_extra_flags` config attribute" [Undecided,In progress] - Assigned to Kashyap Chamarthy (kashyapc)
11:44:32	kashyap	mriedem: Morning, meta comment: on your remark here, if you say: "I insist", I'll close it as you prefer. I'm normally mindful of it, but in this instance I'd prefer to track it via LP bug too: https://bugs.launchpad.net/nova/+bug/1766208
11:44:50	kashyap	See my last comment, before you say the phrase :-)
11:45:17	kashyap	And thanks for the actual substantial review, though! _That_ appreciate the most
11:45:34	mriedem	kashyap: i insist
11:46:13	kashyap	Heh, okay; moooooooving on.
11:51:13	openstackgerrit	Matt Riedemann proposed openstack/nova master: Handle deprecation of inspect.getargspec https://review.openstack.org/530777
11:54:18	mriedem	kashyap: i think you can throw https://review.openstack.org/#/c/563926/ into runways
11:54:25	mriedem	pickings in the queue are starting to get slim
11:54:30	mriedem	for non-blocked things
11:55:42	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add a warnings filter for inspect.getargspec usage https://review.openstack.org/564343
11:59:27	kashyap	mriedem: Hi, will click in a min; making some quick late lunch
12:00:07	kashyap	mriedem: Do you have a (non-null) pointer as to how I can "throw it into a runway"? If you have to look up too, disregard; I'll get to it
12:00:34	efried	kashyap: https://etherpad.openstack.org/p/nova-runways-rocky
12:00:50	efried	kashyap: Add it at the bottom of the queue (L89)
12:01:56	kashyap	Gracias
12:02:10	openstackgerrit	Matt Riedemann proposed openstack/nova master: Remove explicit instance.info_cache.delete() https://review.openstack.org/563689
12:03:32	kashyap	Done
12:03:35	kashyap	Thanks, Eric!
12:05:01	efried	yahyoubetcha
12:15:22	kashyap	Has anyone here tinkered with modifying the 'post-config' for Nova here: http://git.openstack.org/cgit/openstack-dev/devstack/tree/.zuul.yaml#n281
12:17:15	mriedem	sean-k-mooney[m]: i think https://review.openstack.org/#/c/551370/ is ready for your review. i've confirmed via the live migration CI run in the patch at the top of the series that enables this code that it's hitting _update_vif_xml correctly and linked the logs inline
12:17:32	mriedem	sean-k-mooney[m]: however, note the TODO i have in here https://review.openstack.org/#/c/551370/19/nova/virt/libvirt/migration.py@252
12:17:36	mriedem	about vif type changing
12:18:01	mriedem	remember we talked about whether we could just ignore the original source_vif and just get a new vif config based on the destination vif to generate the new xml *except* for the pci address?

Earlier   Later