Posted	Nick	Remark
#openstack-nova - 2017-11-29
03:40:37	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: [WIP] Add cross cells support for InstanceActionAPI https://review.openstack.org/523674
03:41:07	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
03:42:48	alex_xu	efried: I will check that, to find out which code fix that bug, that really surprise me...
04:10:55	openstackgerrit	Jianghua Wang proposed openstack/nova master: XenAPI: provide VGPU_DISPLAY_HEAD inventory in compute node https://review.openstack.org/523342
04:33:56	openstackgerrit	zhongshengping proposed openstack/nova master: Fix project_domain_name and user_domain_name in doc https://review.openstack.org/523686
05:15:50	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the objects which related to the old v2 API implementation https://review.openstack.org/519997
05:15:51	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the objects for describing the extension for v2.1 API https://review.openstack.org/519998
05:16:01	alex_xu	oomichi_afk: gmann ^ just updated
05:22:12	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
05:40:10	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
05:46:48	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
05:58:27	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
06:05:15	gmann	alex_xu: thanks , ll check
06:18:15	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID https://review.openstack.org/523606
06:26:03	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: [WIP] Add cross cells support for InstanceActionAPI https://review.openstack.org/523674
06:41:59	openstackgerrit	Merged openstack/nova master: PowerVM support matrix update https://review.openstack.org/523140
07:08:50	openstackgerrit	Chen Hanxiao proposed openstack/python-novaclient master: inject file: add description of injecting multiple files. https://review.openstack.org/515655
07:20:38	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
08:11:12	openstackgerrit	jichenjc proposed openstack/nova master: [placement]Enhance doc for placement allocation list https://review.openstack.org/511715
08:15:47	openstackgerrit	jichenjc proposed openstack/nova master: check query param for server groups function https://review.openstack.org/500347
08:25:39	openstackgerrit	licanwei proposed openstack/nova master: Make nova-manage syncing all cells databases https://review.openstack.org/519275
08:30:40	kalyan	Hi All, is openstack/libvirt usb passthrough has a limitation USB protocol <=2.0 ??
08:31:02	kalyan	could someone please correct me on this and provide some information to achieve the same if possible
08:31:10	kalyan	thanks in advance! :)
08:38:11	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
08:57:44	openstackgerrit	Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
09:08:05	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/523765
09:14:00	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/520478
09:14:46	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676
09:32:07	openstackgerrit	Erik Berg proposed openstack/nova master: Cell_v2: unique constraint on database_connection mapping https://review.openstack.org/523817
09:50:55	openstackgerrit	Chen Hanxiao proposed openstack/nova master: libvirt: throw NotImplementedError if qga is not responsive when setting password https://review.openstack.org/523828
10:10:31	openstackgerrit	Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
10:42:22	openstackgerrit	Surya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url https://review.openstack.org/518556
11:40:05	openstackgerrit	Erik Berg proposed openstack/nova master: Cool, wasn't sure about wether the transport_url needed to be unique aswell. The more you know :) https://review.openstack.org/523849
13:15:56	jaypipes	*yawn*
13:15:59	jaypipes	morning supernovas
13:28:43	bauzas	jaypipes: morning
13:54:27	mriedem	lucasagomes: you should take a look at this change https://review.openstack.org/#/c/523559/
13:59:12	efried	jaypipes Looks like some microversion digits were missed in the rebase.
13:59:25	jaypipes	efried: lemme look...
13:59:39	efried	jaypipes I can take care of it - I have some work to do on the series anyway.
13:59:48	lucasagomes	mriedem, hi there! I will take a look, although, I haven't been involved with ironic for awhile. It might be better to get some of the #openstack-ironic folks to take a look as well
14:00:00	efried	jaypipes As long as you don't have local changes?
14:00:13	jaypipes	efried: go for it.
14:00:18	efried	rgr
14:11:42	lucasagomes	mriedem, posted a question re using task_state + vm_state to determine if the rebuild failed or not
14:12:54	mriedem	lucasagomes: heh, did you see my comment in the code?
14:12:59	mriedem	basically the same idea
14:13:16	lucasagomes	mriedem, oh, I looked at the wrong column ahah oh right
14:13:20	lucasagomes	yeah i think that would be more flexible
14:13:24	lucasagomes	if it works
14:15:47	lucasagomes	replied to your comment basically agreeing with the task_state + vm_state idea :-)
14:25:01	stephenfin	jaypipes: I've a question on https://github.com/openstack/nova/commit/15ac5b688bf6d91ac42ca33860d187d80289d82d
14:25:20	jaypipes	stephenfin: yessir?
14:25:58	stephenfin	jaypipes: That contains an online data migration. Would I need to run the `nova-manage db online_data_migrations` command for that to take place, or should it happen any time?
14:26:43	stephenfin	I'd thought the UUID field would be populated any time I loaded an object from the DB, but then I saw that command
14:32:21	jaypipes	stephenfin: there's no nova-manage db stuff in that patch.
14:32:47	jaypipes	stephenfin: all data migrations are done upon load/save of the PciDevice object
14:34:21	jaypipes	stephenfin: one might make an argument that there *should* have been a corresponding patch that added a nova-manage online data migration (that read in PciDevice objects in batches)
14:39:52	mriedem	that's what we did when we added the uuid column to compute_nodes, services and migrations tables
14:40:05	mriedem	you could still add one...
14:42:07	jaypipes	mriedem: ack
14:42:34	jaypipes	mriedem: would need to be done before the PCI device manager is integrated with placement.
14:42:53	stephenfin	jaypipes, mriedem: I can do that today
14:42:57	openstackgerrit	Sylvain Bauza proposed openstack/nova master: libvirt: Provide VGPU inventory for a single GPU type https://review.openstack.org/514763
14:43:12	stephenfin	jaypipes: In the interim, I think there's a bug with that patch. See here
14:43:13	stephenfin	https://bugzilla.redhat.com/show_bug.cgi?id=1516952
14:46:54	openstack	bugzilla.redhat.com bug 1516952 in openstack-nova "Cannot boot vm with sriov port after upgrade OSP11 to OSP12" [Urgent,New] - Assigned to eglynn
14:46:54	mriedem	i see the problem
14:46:54	mriedem	_from_db_object isn't handling the uuid column properly
14:46:54	mriedem	https://review.openstack.org/#/c/469147/2/nova/objects/pci_device.py@194
14:46:54	mriedem	there should be a skip in there
14:46:54	mriedem	if key not in ('extra_info', 'uuid'):
14:48:42	mriedem	stephenfin: i updated https://bugzilla.redhat.com/show_bug.cgi?id=1516952 - open an lp bug and i'll push a fix
14:50:29	jaypipes	dansmith: because we can't make them not-nullable until the fields are populated with something.
14:50:29	dansmith	jaypipes: so leave them unset? a nullable field means it can be set to None, and that's extremely wrong for an id, IMHO
14:50:29	jaypipes	dansmith: it gets set to a non-null value right after there..
14:50:29	stephenfin	Couldn't we just move that non-null setting to the top of the function?
14:50:29	stephenfin	then we don't need to skip anything
14:50:29	jaypipes	dansmith: and then when the version of the object goes to the next major version, we change nullable to False and remove the data migration.
14:50:29	dansmith	okay I'm not sure what you're saying, you want to make it null so that a generic loop can set it to none for a few cycles?
14:50:29	dansmith	-2 on that :)
14:51:11	mriedem	fwiw we've only ever bumped the major version on an object before, it doesn't happen really
14:51:19	mriedem	Instance is the only one
14:51:29	jaypipes	yes, I've complained about that before...
14:51:43	dansmith	jaypipes: so do the work to bump.. it's not very fun :)
14:52:33	jaypipes	well, sounds like I'm in the minority here, stephenfin, so just add a conditional in that setattr loop to check for whether db_dev['uuid'] is None and if so, don't set obj.uuid to None.
14:52:55	mriedem	i've got a patch local
14:53:03	mriedem	just need an lp bug
14:53:07	jaypipes	dansmith: do we have a good guide to doing that work?
14:54:06	openstack	bugzilla.redhat.com bug 1516952 in openstack-nova "Cannot boot vm with sriov port after upgrade OSP11 to OSP12" [Urgent,New] - Assigned to eglynn
14:54:19	stephenfin	mriedem: https://bugs.launchpad.net/nova/+bug/1735188
14:54:22	mriedem	https://bugs.launchpad.net/nova/+bug/1735189
14:54:24	mriedem	you beat me
14:54:29	stephenfin	Ha!
14:54:50	stephenfin	so close
14:55:14	dansmith	jaypipes: nope, it also takes work to put that together and nobody else has ever asked to do it. Happy to help with that, but like many other things, I'd value this well below a lot of the other things we've got going on at the moment
14:55:32	openstack	Launchpad bug 1735188 in OpenStack Compute (nova) "PciDevice UUID field not populated" [Undecided,New]
14:55:33	openstack	Launchpad bug 1735189 in OpenStack Compute (nova) "ValueError: Field `uuid' cannot be None in PciDevice._from_db_object" [Undecided,New]
14:55:42	jaypipes	dansmith: ack. maybe a weekend project for me around the holidays.

Earlier   Later