Posted	Nick	Remark
#openstack-nova - 2017-07-21
03:28:24	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/485634
03:32:50	openstackgerrit	yuanyue proposed openstack/nova master: Whether to use configdrive disk depends on instance.configdrive https://review.openstack.org/485930
03:59:20	openstackgerrit	Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566
04:00:39	openstackgerrit	Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
04:00:39	openstackgerrit	Tony Breeds proposed openstack/nova master: Decouple the os-brick and nova privsep modules. https://review.openstack.org/485908
04:00:40	openstackgerrit	Tony Breeds proposed openstack/nova master: Move the last_bytes util method to the libvirt driver. https://review.openstack.org/472228
04:00:40	openstackgerrit	Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
04:00:41	openstackgerrit	Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
04:00:41	openstackgerrit	Tony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py https://review.openstack.org/485907
04:06:53	openstackgerrit	Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
04:06:53	openstackgerrit	Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
04:06:54	openstackgerrit	Tony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py https://review.openstack.org/485907
04:06:54	openstackgerrit	Tony Breeds proposed openstack/nova master: Move the last_bytes util method to the libvirt driver. https://review.openstack.org/472228
04:06:55	openstackgerrit	Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
04:46:04	openstackgerrit	yuanyue proposed openstack/nova master: Whether to use configdrive disk depends on instance.configdrive https://review.openstack.org/485930
04:46:11	openstackgerrit	OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/485950
04:54:49	openstackgerrit	Merged openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700
04:59:51	openstackgerrit	Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
05:01:28	openstackgerrit	iswarya vakati proposed openstack/nova master: Removed unused 'wrap' property https://review.openstack.org/481465
05:03:15	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/485634
05:23:01	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/485634
05:27:56	openstackgerrit	Tony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
05:27:57	openstackgerrit	Tony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
05:27:57	openstackgerrit	Tony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
06:19:38	openstackgerrit	Merged openstack/nova master: Use plain routes list for os-volume-attachments instead of stevedore https://review.openstack.org/484824
06:20:28	openstackgerrit	Merged openstack/nova master: Use plain routes list for os-snapshots instead of stevedore https://review.openstack.org/484825
06:35:49	openstackgerrit	Merged openstack/nova master: Use plain routes list for extension_info instead of stevedore https://review.openstack.org/484826
06:37:00	trinaths	did any one attend this event, https://www.meetup.com/Openstack-Boston/events/241558725/
06:39:36	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects https://review.openstack.org/474892
06:45:15	anilvenkata	Hi I need help,
06:45:22	anilvenkata	I am trying a multinode devstack setup.
06:45:30	anilvenkata	In controller I have all the services.
06:45:32	anilvenkata	In compute, I am enabling "ENABLED_SERVICES=n-cpu,rabbit,neutron,q-agt,q-dhcp,q-l3" will that be sufficient to spawn vms in compute?
06:46:22	anilvenkata	when I try "nova hypervisor-list" , it is showing only controller and not compute host
06:56:16	trinaths	anilvenkata: is u r compute node able to communicate with services in controller node.
06:56:52	anilvenkata	trinaths, neutron services on compute are able to communicate with controller
06:57:07	trinaths	anilvenkata: nova-compute ?
06:57:13	anilvenkata	trinaths, no
06:57:59	anilvenkata	trinaths, on compute, I am not evening seeing nova service i.e ps -aef | grep nova
06:58:09	anilvenkata	that is empty
06:58:17	trinaths	anilvenkata: mean in u r devstack localrc u did not enable nova ?
06:58:32	anilvenkata	trinaths, I enabled n-cpu
06:58:44	anilvenkata	will that be sufficient?
07:00:12	trinaths	anilvenkata: yes.
07:00:41	trinaths	anilvenkata: use OFFLINE=no and RECLONE=yes and re stack it.
07:00:55	trinaths	anilvenkata: n-cpu must fix that.
07:01:08	anilvenkata	trinaths, sure, thanks trinath
07:01:36	trinaths	anilvenkata: use n-api too
07:01:49	anilvenkata	trinaths, why is it needed?
07:02:09	trinaths	anilvenkata: it will download the nova code. try it
07:02:15	anilvenkata	trinaths, ok
07:07:42	openstackgerrit	junbo.li proposed openstack/nova master: [DOC BLD FIX] Correct reST field lists in docstrings https://review.openstack.org/486010
07:31:37	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
07:35:32	openstackgerrit	Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459
07:54:12	jangutter	Would someone be able to trigger a +W on https://review.openstack.org/#/c/485634/ ?
08:12:20	openstackgerrit	sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522
08:51:21	openstackgerrit	junbo.li proposed openstack/nova master: [DOC BLD FIX] Correct reST field lists in docstrings https://review.openstack.org/486010
09:06:31	openstackgerrit	sahid proposed openstack/nova master: filter: NUMATopologyLimits to a primitve prior to pass it to HostState https://review.openstack.org/486058
09:06:31	openstackgerrit	sahid proposed openstack/nova master: Revert "Convert HostState.limits['numa_topology'] to primitive" https://review.openstack.org/486057
09:18:29	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Destroy the ReqSpec object when deleting the instance https://review.openstack.org/391060
09:24:19	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the unittest for plugin framework https://review.openstack.org/485060
09:24:20	openstackgerrit	Alex Xu proposed openstack/nova master: Use plain routes list for versions instead of stevedore https://review.openstack.org/485011
09:24:20	openstackgerrit	Alex Xu proposed openstack/nova master: Using plain routes for the microversions test https://review.openstack.org/485061
09:24:21	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the code related to extension loading https://review.openstack.org/486070
09:24:21	openstackgerrit	Alex Xu proposed openstack/nova master: Remove the useless extension block_device_mapping_v1 object https://review.openstack.org/486069
09:24:22	openstackgerrit	Alex Xu proposed openstack/nova master: Move the note about '/os-volume_boot' to the correct place https://review.openstack.org/486071
09:25:23	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
09:28:45	alex_xu	gmann: sfinucan ^ last few patches for no more extension
09:28:59	stephenfin	alex_xu: It's stephenfin now ;)
09:29:07	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851
09:29:21	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214
09:29:27	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929
09:29:28	alex_xu	stephenfin: ok :) why change a new name
09:29:33	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
09:29:36	openstackgerrit	Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
09:30:33	stephenfin	alex_xu: So people stop dropping the 'e' from my last name (Finucane) :) Also, so I can use the same name _everywhere_ (GitHub, Twitter, etc.)
09:32:13	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
09:32:19	alex_xu	stephenfin: haha
09:47:04	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
09:51:12	smcginnis	I'm a little confused on a Nova unit test failure we are getting on the version bump of the cinderclient.
09:51:15	smcginnis	http://logs.openstack.org/03/485903/2/check/gate-cross-nova-python27-ubuntu-xenial/bd55208/console.html#_2017-07-21_05_00_42_049094
09:51:35	smcginnis	The Nova test code passes skip_version_check=True
09:51:38	smcginnis	https://github.com/openstack/nova/blob/master/nova/tests/unit/volume/test_cinder.py#L851
09:52:02	smcginnis	But then it blows up on checking the microversion here:
09:52:03	smcginnis	https://github.com/openstack/nova/blob/master/nova/volume/cinder.py#L141
09:53:39	openstackgerrit	Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
10:09:25	frickler	smcginnis: you are looking at the wrong test is seems, this one is failing: https://github.com/openstack/nova/blame/master/nova/tests/unit/volume/test_cinder.py#L826 . does the new python-cinderclient provide a different 3.x range only?
10:11:01	smcginnis	frickler: Shoot, you're right. Don't know how I messed that up when the line number is right in the traceback.
10:11:09	frickler	smcginnis: ah, this is probably the default version changed here: https://github.com/openstack/nova/blame/master/nova/tests/unit/volume/test_cinder.py#L823
10:12:40	smcginnis	frickler: MAX_VERSION is now 3.40. Hmm.
10:13:09	frickler	smcginnis: seems wrong from nova to make assuptions about that being 3.27 permanently
10:14:20	smcginnis	frickler: Well, if I understand this correctly, unless we remove that microversion it should always be there.
10:14:36	smcginnis	frickler: Looking at the check code to see what it's doing now.
10:19:00	smcginnis	Looks like we may still have a bug in python-cinderclient.
10:19:17	smcginnis	It was supposed to have been fixed here it was treating version as a float.
10:19:48	smcginnis	So 3.40 becomes 3.4, which makes the range checking off.
10:21:04	frickler	smcginnis: the test mocks the answer in L823 by using the constant from python-cinderclient, so it will be 3.4. but it compares that result to the fixed string "3.27". so IMO either the mock or the comparison should be fixed
10:22:06	smcginnis	frickler: Ah, you're right. Line 823 calls float() on that.
10:22:17	smcginnis	frickler: I think the test needs to be fixed then.
10:22:31	smcginnis	frickler: Thanks, I can fix that quick.

Earlier   Later