Posted	Nick	Remark
#openstack-nova - 2017-06-08
07:30:04	openstackgerrit	melanie witt proposed openstack/nova master: Add periodic task to clean expired console tokens https://review.openstack.org/325381
07:30:05	openstackgerrit	melanie witt proposed openstack/nova master: Add console connection object https://review.openstack.org/320063
07:30:05	openstackgerrit	melanie witt proposed openstack/nova master: Use ConsoleConnection object to generate authorizations https://review.openstack.org/325414
07:30:06	openstackgerrit	melanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation https://review.openstack.org/333990
07:30:06	openstackgerrit	melanie witt proposed openstack/nova master: Add access_url_base to console_auth_tokens table https://review.openstack.org/334614
07:52:36	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
08:45:16	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524
08:45:17	openstackgerrit	Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625
08:46:29	openstackgerrit	Pooja Jadhav proposed openstack/nova master: Nova flavor-list swap should be 0 when no value is specified https://review.openstack.org/471715
08:50:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489
08:53:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: remove get_nw_info_for_instance from compute.utils https://review.openstack.org/451387
08:55:07	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove useless API tests https://review.openstack.org/461011
08:55:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group https://review.openstack.org/462469
08:56:00	openstackgerrit	Mikhail Feoktistov proposed openstack/nova master: Remove fstype param from ploop init https://review.openstack.org/444970
08:59:14	openstackgerrit	Stephen Finucane proposed openstack/nova master: Convert additional disassociate tests to mock https://review.openstack.org/471408
09:03:59	openstackgerrit	Mikhail Feoktistov proposed openstack/nova master: Add is_vz_container function https://review.openstack.org/445947
09:06:36	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
09:09:22	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Improve assertJsonEqual error reporting https://review.openstack.org/428199
09:11:06	openstackgerrit	Béla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625
09:15:04	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add separate instance.create payload type https://review.openstack.org/463001
09:17:59	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstanceCreatePayload https://review.openstack.org/419730
09:27:07	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add key_name field to InstancePayload https://review.openstack.org/463002
09:30:06	Kevin_Zheng	tests, so, should I add test for PUT also to os-server-tags or move them all to servers?
09:30:06	Kevin_Zheng	Hi, I'm working on https://review.openstack.org/#/c/465842/ which adds tags field to instance list details/update/show API and related functional tests, and tests for list details/show already exists in https://github.com/openstack/nova/tree/master/nova/tests/functional/api_sample_tests/api_samples/os-server-tags/v2.26 under os-server-tags tests, as it is tests for servers API, I'd think it should be under servers
09:33:17	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779
09:34:36	openstackgerrit	jichenjc proposed openstack/nova master: propagate OSError to MigrationPreCheckError https://review.openstack.org/469779
09:37:11	openstackgerrit	Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for aggregates https://review.openstack.org/470932
09:37:11	openstackgerrit	Andrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref for allocations https://review.openstack.org/470933
09:43:33	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210
09:43:47	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211
09:49:02	openstackgerrit	Merged openstack/nova master: Remove useless API tests https://review.openstack.org/461011
09:53:57	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842
09:54:45	openstackgerrit	zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
09:57:55	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Disambiguate resource provider conflict message https://review.openstack.org/468928
09:59:20	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Update placement devref to modern features https://review.openstack.org/469047
10:00:39	openstackgerrit	Chris Dent proposed openstack/nova master: [placement] Use util.extract_json in allocations handler https://review.openstack.org/471067
10:01:37	openstackgerrit	Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
10:12:00	openstackgerrit	sahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller https://review.openstack.org/459741
10:24:50	openstackgerrit	Mikhail Feoktistov proposed openstack/nova master: raise exception if create Virtuozzo container with swap disk https://review.openstack.org/397829
10:38:12	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
10:40:14	Gayi	Hello everyone..
10:48:42	openstackgerrit	falseuser proposed openstack/nova master: Clear instance root_device_name in rebuild operation https://review.openstack.org/464991
10:55:46	openstackgerrit	Ritesh proposed openstack/nova master: rbd: flatten images when unshelving https://review.openstack.org/457886
11:08:54	mikal	mdbooth: You around? Not sure what timezone you're in.
11:19:34	mdbooth	mikal: I'm around. UK.
11:19:40	mdbooth	What's up?
11:20:53	mikal	I was confused about some "pty flush" code in the libvirt driver...
11:21:14	mdbooth	mikal: I'll probably be similarly confused :)
11:21:17	mikal	But I think I've got it now. I'm fiddling with file permissions while I sneak up with privsep and I didn't understand why the flush needed a chown first
11:21:29	mikal	It turns out it doesn't its the call _after_ that which needs the chown
11:21:41	mikal	Sorry
11:21:41	mikal	So yeah, I think I just wasted your time
11:21:42	mdbooth	cdent: Too late: https://goo.gl/photos/31jQijVTf5wsAuFi6 :)
11:21:55	cdent	+many
11:22:30	mdbooth	mikal: Hehe, np.
11:28:22	openstackgerrit	Merged openstack/nova master: Fix html_last_updated_fmt for Python3 https://review.openstack.org/470676
11:54:08	openstackgerrit	Merged openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489
12:08:57	openstackgerrit	Michael Still proposed openstack/nova master: Avoid chowing console logs in libvirt https://review.openstack.org/472229
12:08:57	openstackgerrit	Michael Still proposed openstack/nova master: Move the last_bytes util method to the libvirt driver. https://review.openstack.org/472228
12:13:44	openstackgerrit	Mikhail Feoktistov proposed openstack/nova master: Add is_vz_container function https://review.openstack.org/445947
12:16:30	mriedem	it's p-2 day
12:16:44	mriedem	any burning regressions or bugs we need to get fixed before we tag it?
12:23:31	bauzas	mriedem: honestly, I'm still looking at all the bugs
12:23:38	bauzas	for the moment, none
12:23:49	bauzas	but that's a today triage day for me
12:25:33	mriedem	lyarwood: i just noticed the git author thing again while rebasing https://review.openstack.org/#/c/436094/
12:25:41	mriedem	i pulled that down with git review -d
12:25:47	mriedem	rebased it to master: git rebase -i master
12:26:12	mriedem	resolved merge conflicts, added the resolved files and did git commit, then git rebase --continue
12:26:16	mriedem	and then i was the author
12:27:04	mriedem	i do'nt know what's going on there
12:29:04	openstackgerrit	Matt Riedemann proposed openstack/nova master: Cleanup update_instance cell mapping handling https://review.openstack.org/467380
12:29:05	openstackgerrit	Matt Riedemann proposed openstack/nova master: Remove cells v2 transition code from update_instance https://review.openstack.org/467382
12:29:05	openstackgerrit	Matt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells https://review.openstack.org/467383
12:29:12	kashyap	cfriesen: Have you booted an UEFI image lately?
12:29:17	kashyap	(Or if ever)
12:29:40	openstackgerrit	Matthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready https://review.openstack.org/471773
12:30:11	openstackgerrit	Matt Riedemann proposed openstack/nova master: Pull out code that builds VIF in _build_network_info_model https://review.openstack.org/466449
12:30:40	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: Handle network-changed event for a specific port https://review.openstack.org/465787
12:36:17	openstack	Launchpad bug 1687581 in OpenStack Compute (nova) "Newton > Ocata upgrade. secret_uuid empty" [High,New]
12:36:17	sdague	mriedem: https://bugs.launchpad.net/nova/+bug/1687581 seems like something we should probably fix sooner rather than later
12:37:25	sdague	people upgrading newton -> ocata ceph apparently can't start newton guests
12:37:44	mriedem	i know what caused that
12:38:15	mriedem	https://review.openstack.org/#/c/389399/
12:38:37	mriedem	i think
12:40:51	mriedem	so it looks like in newton they had rbd auth enabled on the cinder side, but didn't have the secret uuid coming from cinder, it was coming from nova.conf
12:42:07	sdague	right
12:42:17	mriedem	i can work up a patch
12:42:24	sdague	mriedem: there is a patch in the bug
12:42:30	sdague	they just didn't gerrit it
12:43:04	sdague	http://paste.ubuntu.com/24497744/
12:45:10	mriedem	i know, but that needs tests and comments, etc
12:45:15	mriedem	i can co-author them
12:48:49	openstack	Launchpad bug 1696001 in OpenStack Compute (nova) ocata "duplicate for #1696685 nova-manage cell_v2 map_cell0 can create invalid connection URL" [Undecided,In progress] - Assigned to Sylvain Bauza (sylvain-bauza)
12:48:49	bauzas	claudiub: see https://bugs.launchpad.net/nova/+bug/1696685
12:49:01	bauzas	claudiub: you can review the stable/ocata change
12:50:02	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384
12:50:14	openstack	Launchpad bug 1696417 in OpenStack Compute (nova) "nova-manage db online_data_migrations can fail when upgrading to newton under certain conditions" [Undecided,Incomplete]
12:50:14	bauzas	mriedem: we have a super weird issue for stable/newton with db migrations when you have different TZs https://bugs.launchpad.net/nova/+bug/1696417
12:50:26	bauzas	mriedem: for sure we could just say "meh"

Earlier   Later