Posted	Nick	Remark
#openstack-nova - 2018-06-26
08:36:08	openstackgerrit	Lee Yarwood proposed openstack/nova stable/pike: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559032
08:40:49	openstackgerrit	Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262
08:58:28	openstackgerrit	Surya Seetharaman proposed openstack/nova-specs master: Handling a down cell https://review.openstack.org/557369
09:04:46	openstackgerrit	do3meli proposed openstack/nova master: docs: add nova host-evacuate command to evacuate documentation https://review.openstack.org/578040
09:26:01	openstackgerrit	Lee Yarwood proposed openstack/nova stable/ocata: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/559034
09:26:30	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] api-ref: add traits patameter https://review.openstack.org/578048
09:27:20	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] api-ref: add traits parameter https://review.openstack.org/578048
10:14:14	frickler	is it intentional or a bug that unit tests require qemu-img being installed? /me tends to assume the latter. getting this error while trying to build pkgs for 16.1.4: http://paste.openstack.org/show/724292/
10:19:45	openstackgerrit	Jens Harbott (frickler) proposed openstack/nova stable/pike: mock utils.execute() in qemu-img unit test https://review.openstack.org/578058
10:21:02	openstack	Launchpad bug 1765122 in OpenStack Compute (nova) "qemu-img execute not mocked in unit tests" [Low,Fix released] - Assigned to Jay Pipes (jaypipes)
10:21:02	frickler	jaypipes: there was a bug already opened, just lacking cherry-pick to stable/pike ^^ https://bugs.launchpad.net/nova/+bug/1765122
10:29:38	openstackgerrit	sahid proposed openstack/nova stable/queens: add mtu to libvirt xml for ethernet and bridge types https://review.openstack.org/578010
10:38:40	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Rename '_get_phynet_info' https://review.openstack.org/577233
10:38:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Rename 'create_pci_requests_for_sriov_ports' https://review.openstack.org/575488
10:38:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Always retrieve network information if available https://review.openstack.org/564444
10:38:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: network: Retrieve tunneled status in '_get_physnet_info' https://review.openstack.org/564445
10:38:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add NUMACell.network_info https://review.openstack.org/564439
10:38:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Start accounting for networks in NUMA placement https://review.openstack.org/564448
10:38:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add NUMATopologyLimits.networks https://review.openstack.org/575486
10:38:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add RequestSpec.numa_networks https://review.openstack.org/564442
10:38:49	openstackgerrit	Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.numa_networks https://review.openstack.org/564452
10:38:50	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440
10:38:51	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field https://review.openstack.org/564441
10:50:52	efried	wwriverrat: Hello? (I'm just checking in, will be back in 2-3h)
10:54:59	ygl	hi all
10:55:19	ygl	can anyone let me know what framework in python does the nova api server use ?
11:01:17	openstackgerrit	Vladyslav Drok proposed openstack/nova master: rbd: flatten images when creating/unshelving an instance https://review.openstack.org/457886
11:17:50	openstackgerrit	Vishakha Agarwal proposed openstack/python-novaclient master: No requirement of –all-tenants while listing servers https://review.openstack.org/569090
12:02:58	openstackgerrit	Merged openstack/nova master: Bump keystoneauth1 minimum to 3.9.0 https://review.openstack.org/577431
12:03:05	openstackgerrit	Merged openstack/nova master: Adapter raise_exc=False by default https://review.openstack.org/577437
12:10:17	openstackgerrit	Chason Chan proposed openstack/nova master: Fix the duplicated config options of api_database and placement_database https://review.openstack.org/577023
12:12:47	jaypipes	frickler: no worries mate :)
12:13:33	jaypipes	efried: morning. hey, so I'm fully back online now. batteries at 100%.
12:13:52	jaypipes	efried: I missed the scheduler meeting yesterday as I was on a plane.
12:14:00	jaypipes	efried: anything pressing to discuss?
12:32:42	jangutter	PSA: when in a meeting with jaypipes, be extra extra careful. He will coerce you into saying the phrase 'a bag of dicts', and it will take a long time to explain python to HR.
12:33:03	gibi	:)
12:33:05	jaypipes	jangutter: *cough* that's "random bag of dicts", sir.
12:33:41	jaypipes	jangutter: also see: why to buy a good pair of headphones while working in an office :P
12:34:15	jangutter	jaypipes: hence your subtle technique of getting me to say it.
12:35:48	jaypipes	jangutter: you've figured me out.
12:37:36	openstackgerrit	Merged openstack/nova master: fix PowerVM get_bootdisk_path docstring https://review.openstack.org/576977
12:37:43	openstackgerrit	Merged openstack/nova master: api-ref: Fix parameters about trusted certificate IDs https://review.openstack.org/576046
12:37:50	openstackgerrit	Merged openstack/nova master: Update links in README https://review.openstack.org/574644
12:41:38	frickler	jaypipes: well actually I am a bit worried about this bug having been able to pass testing. makes me wonder whether execute() should get mocked/otherwise trapped globally in unit tests
13:00:38	efried	jaypipes: Good morning, welcome back.
13:01:49	openstack	Launchpad bug 1777591 in OpenStack Compute (nova) "‘limit’ in allocation_candidates where sometimes make fore_hosts invalid" [High,In progress] - Assigned to xulei (605423512-j)
13:01:49	efried	jaypipes: We mainly talked about https://bugs.launchpad.net/nova/+bug/1777591 -- see the bug for what we concluded, and speak up if you have a dissenting stance.
13:06:39	jaypipes	efried: ahhh, force_hosts...
13:06:58	efried	everyone's favorite, apparently.
13:21:41	openstackgerrit	Matt Riedemann proposed openstack/nova master: Remove support for /os-fixed-ips REST API https://review.openstack.org/568516
13:33:27	mriedem	heh PUT /os-floating-ips-bulk/delete
13:33:29	mriedem	makes total sense
13:39:54	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix API reference for os-floating-ips-bulk https://review.openstack.org/578106
13:41:28	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix API reference for os-floating-ips-bulk https://review.openstack.org/578106
13:41:29	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix API reference for os-floating-ip-dns https://review.openstack.org/578107
13:46:16	openstackgerrit	xulei proposed openstack/nova master: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/576693
13:48:51	openstackgerrit	xulei proposed openstack/nova master: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/576693
13:48:52	openstackgerrit	Matt Riedemann proposed openstack/nova master: Update API reference for os-floating-ip-pools https://review.openstack.org/578108
13:51:42	mriedem	if anybody is looking for something to do, the os-floating-ip-dns and os-floating-ips-bulk APIs are purely nova-network and can be removed following the standard 410 pattern we've established for this kind of work: https://etherpad.openstack.org/p/nova-network-removal-rocky
13:58:40	mriedem	huh https://docs.openstack.org/nova/latest/cli/nova-manage.html#nova-api-database "Sync the api cells database up to the most recent version. This is the standard way to create the db as well." that is both confusing and wrong in the same shot
13:58:55	mriedem	- there is no 'api cells database'
13:58:59	mriedem	- it doesn't create a database
14:13:12	gibi	mriedem: did I understand correnctly that you had strange problem with 'field_name in an_ovo_object' calls?
14:15:11	gibi	mriedem: based on your statement '17:18 < mriedem> >>> obj.obj_attr_is_set('foo')'
14:16:09	gibi	mriedem: I'm troubleshooting my random unit test failures in https://review.openstack.org/#/c/567268/21
14:19:00	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix CLI docs for nova-manage api_db commands https://review.openstack.org/578116
14:19:42	mriedem	gibi: yeah something i was playing with yesterday, but in a venv
14:20:18	mriedem	reminds me i need to check back on that
14:20:22	gibi	mriedem: did you find any specific issue with oslo.versioned object?
14:20:34	mriedem	nope, i didn't figure it out
14:20:44	gibi	mriedem: OK thanks. then I dig forward
14:20:54	mriedem	i had to go a different direction anyway on https://review.openstack.org/#/c/515423/ for the fix i was investigating
14:21:14	gibi	mriedem: I see
14:24:57	stephenfin	Anyone fancy looking at the two of these? Closes out kashyap's "bump libvirt" series https://review.openstack.org/#/q/topic:bump_min_libvirt_and_qemu+status:open
14:25:23	kashyap	stephenfin: Oh, reminds me to finish off this too: https://review.openstack.org/#/c/567258/
14:25:33	kashyap	(Was buried in something else)
14:26:10	kashyap	stephenfin: Zuul is -1 on the VHOSTUSER_MQ: https://review.openstack.org/#/c/565472/
14:26:33	stephenfin	kashyap: zuul lies
14:26:39	kashyap	:-)
14:26:48	stephenfin	But I'll recheck to be safe
14:26:51	kashyap	Oh yeah
14:27:04	kashyap	It's the IBM zKVM that seems to fail
14:27:22	stephenfin	mriedem, gibi: surely not the same thing, but I had issues with 'field_name in an_ovs_object' vs. 'an_obv_obj.obj_attr_is_set(field_name)' https://review.openstack.org/#/c/564439/9/nova/objects/numa.py
14:27:49	stephenfin	the former worked while the latter complained repeatedly about undefined attributes or something
14:28:43	kashyap	stephenfin: Do you want to address Takashi's nit?
14:28:49	gibi	stephenfin: interesting, thanks
14:29:21	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472
14:29:26	stephenfin	kashyap: Sure, done
14:29:35	kashyap	Thx!
14:29:36	gibi	stephenfin: I'm suspecting something wrong with 'in' in my case
14:29:51	stephenfin	gibi: Ah, other way round. Of course :)
14:29:59	mriedem	stephenfin: that's because obj_attr_is_set looks for obj_fields
14:30:21	mriedem	http://git.openstack.org/cgit/openstack/oslo.versionedobjects/tree/oslo_versionedobjects/base.py#n681
14:31:01	mriedem	the 'in' implementation is a bit cleaner http://git.openstack.org/cgit/openstack/oslo.versionedobjects/tree/oslo_versionedobjects/base.py#n321
14:31:05	mriedem	but it hides that AttributeError
14:31:22	gibi	mriedem: thanks, this could be something that helps my investigation
14:31:43	gibi	I thought 'in' and obj_attr_is_set is equivalent :/
14:31:52	stephenfin	mriedem: Oh, good to know. I should probably call that out in the commit message
14:31:55	stephenfin	gibi: As did I

Earlier   Later