Posted	Nick	Remark
#openstack-nova - 2018-06-20
04:16:21	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709
04:38:11	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.openstack.org/576712
04:57:56	vpc	hi
04:57:58	vpc	how can i do this
04:57:59	vpc	Take snapshot of an instance along with attached volume
05:13:10	openstackgerrit	zhangyangyang proposed openstack/nova master: Add the neutron doc link https://review.openstack.org/576682
05:15:55	openstackgerrit	zhangyangyang proposed openstack/nova master: Add the neutron doc link https://review.openstack.org/576682
05:18:53	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Mention nova-status upgrade check CLI in upgrade doc https://review.openstack.org/576719
05:24:25	openstackgerrit	zhangyangyang proposed openstack/nova master: Remove os_compute_api:os-config-drive https://review.openstack.org/576435
05:32:27	openstackgerrit	jichenjc proposed openstack/nova master: Mention server status in api-ref when rebuild https://review.openstack.org/576438
06:01:48	openstackgerrit	jichenjc proposed openstack/nova master: Adjust log style and remove ocata support https://review.openstack.org/576462
06:06:11	openstackgerrit	zhangyangyang proposed openstack/nova master: Update information of neutron_opts url in conf https://review.openstack.org/576674
06:23:09	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Mention nova-status upgrade check CLI in upgrade doc https://review.openstack.org/576719
06:26:30	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in sec group test and functional tests https://review.openstack.org/576751
06:38:27	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487
07:02:23	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.openstack.org/576758
07:13:37	vpc	anyone here successfully build trove image?
07:27:20	openstackgerrit	Dmitry Tantsur proposed openstack/nova master: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576580
07:50:57	openstackgerrit	jichenjc proposed openstack/nova master: Mention server status in api-ref when rebuild https://review.openstack.org/576438
08:08:00	openstackgerrit	Dinesh Bhor proposed openstack/nova master: NUMACell, InstanceNUMACell: Adopt 'PCPU' changes https://review.openstack.org/576021
08:23:03	openstackgerrit	Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
08:23:58	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: WIP: Add instance.lock notification https://review.openstack.org/576702
08:23:59	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703
08:25:48	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add instance.lock notification https://review.openstack.org/576702
08:26:24	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703
08:34:50	quiquell|rover	Hello, we are trying to debug the following problem at nova http://logs.openstack.org/85/564285/20/check/tripleo-ci-centos-7-scenario004-multinode-oooq-container/5894ea4/logs/subnode-2/var/log/containers/nova/nova-compute.log.txt.gz#_2018-06-20_01_54_41_655
08:35:08	quiquell|rover	Cannot update service status on host "centos-7-ovh-bhs1-0000241067.localdomain" since it is not registered
08:43:04	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703
09:03:39	stephenfin	efried: https://github.com/tox-dev/tox/commit/d5b9c0a93 (although it's going to be a year or so before we can rely on it 😞)
09:25:49	openstackgerrit	Merged openstack/nova master: Don't heal allocations for deleted servers https://review.openstack.org/575222
09:30:44	yikun	https://review.openstack.org/#/c/576702 https://review.openstack.org/#/c/576703 lock/unlock notification
09:30:45	yikun	hi, gibi, you could take a look ^^^ when you have time, thanks
09:31:23	openstackgerrit	Lee Yarwood proposed openstack/nova master: compute: Ensure pre-migrating instances are destroyed during init_host https://review.openstack.org/562284
10:34:18	gibi	yikun: thanks for picking that up, I will queue it up for review
10:55:51	openstackgerrit	yanpuqing proposed openstack/nova master: Rename auth_uri to www_authenticate_uri https://review.openstack.org/576820
11:11:27	openstackgerrit	Chris Dent proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
11:11:28	openstackgerrit	Chris Dent proposed openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324
11:19:23	openstackgerrit	yanpuqing proposed openstack/nova master: Rename auth_uri to www_authenticate_uri https://review.openstack.org/576820
12:21:26	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423
12:21:27	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add VIFMigrateData.get_dest_vif https://review.openstack.org/566931
12:21:28	openstackgerrit	Matt Riedemann proposed openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call https://review.openstack.org/566932
12:21:29	openstackgerrit	Matt Riedemann proposed openstack/nova master: libvirt: use dest host port bindings during pre_live_migration https://review.openstack.org/566933
12:21:30	openstackgerrit	Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370
12:21:31	openstackgerrit	Matt Riedemann proposed openstack/nova master: Handle strings as booleans in nova.network.model.VIF utils https://review.openstack.org/576664
12:21:32	openstackgerrit	Matt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371
12:21:33	openstackgerrit	Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870
12:21:34	openstackgerrit	Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537
13:10:34	openstackgerrit	Merged openstack/nova master: Wait for network-vif-plugged before starting live migration https://review.openstack.org/558001
13:13:21	gibi	yikun: left some comments in the lock/unlock notification patches. Thanks for picking this work up.
13:17:01	efried	stephenfin: Beautiful!
13:18:20	mnaser	a fun one for this morning... we had a user resize a vm, status was "Confirm or Revert Resize/Migrate" inside horizon (so I assume the API status was the same)
13:18:32	mnaser	when they confirm or revert the resize: "HTTP exception thrown: Instance has not been resized."
13:18:39	stephenfin	fwiw, that was a series of lovely commits but they used GitHub's "helpful" squash on merge feature to work around the usually terrible commits allowed by GitHub's "helpful" pull request model
13:19:11	mnaser	the user deleted the vm so i cant reproduce unfortunately.. but i thought i'd bring it up if anyone seens something like this..
13:20:44	bauzas	mnaser: can't see the instance actions API ?
13:20:56	mnaser	oh is that kept after the instance is deleted?
13:21:05	mnaser	good idea
13:21:07	bauzas	mnaser: IIRC, yes
13:21:12	mnaser	if not i can look it up in the db
13:21:39	mnaser	interesting
13:21:53	mnaser	start, resize, two confirmResize's and delete
13:22:11	mnaser	let me check the request id
13:22:11	bauzas	mnaser: tbc, using this API could be giving you the request numbrer
13:22:19	mnaser	yep it does give the request id
13:22:24	bauzas	yup
13:23:10	mnaser	204 for the confirmResize
13:23:25	bauzas	mnaser: https://developer.openstack.org/api-ref/compute/#list-actions-for-server
13:23:30	bauzas	"Action information of deleted instances can be returned for requests starting with microversion 2.21."
13:23:44	bauzas	so, you should get them
13:23:56	bauzas	mnaser: Queens for you, right?
13:24:00	mnaser	yep
13:24:05	bauzas	cool
13:24:19	bauzas	so yeah, try to see the req-id and look at the compute log
13:24:32	bauzas	(for the resize, I mean)
13:25:01	efried	stephenfin: So I'm clear, this considers it a conflict even if the basepython is "inherited" from [testenv] ?
13:25:14	stephenfin	Yup
13:25:56	efried	stephenfin: And when you say it'll be changed from warning to error in the future - will that config option still be available to ignore that error?
13:26:08	mnaser	bauzas: interesting, looks like i twas looping on "Updating from migration 309e439d-0936-4363-ac36-e783fa7afc2b"
13:26:28	bauzas	mmmm
13:27:09	efried	stephenfin: I assume so; otherwise it winds up having not accomplished anything. So, /me happy. Thank you very much for working that.
13:27:15	bauzas	mnaser: you mean the server status or what?
13:27:23	bauzas	or the logs?
13:27:42	stephenfin	efried: Yeah, setting the option to True will allow you to ignore the override, so using 'py27' with '[testenv] basepython = python3' will still run 'python2.7'
13:27:51	efried	beaut
13:28:12	stephenfin	efried: If you leave the option at False (default), that will raise a warning and then error out in the next major version
13:28:16	bauzas	mnaser: ha, nevermind, found the log
13:28:28	bauzas	that's a RT log
13:28:31	mnaser	yep
13:28:36	efried	Guess I should get used to saying "schöne"
13:28:39	mnaser	so it's almost like the migratios was never marked complete
13:28:44	mnaser	yeah it just looped that non-stop, let me try to check logs on the source machine
13:28:47	stephenfin	efried: and np. It's nice to work on other stuff in the Python ecosystem ever now and then, heh
13:29:29	bauzas	mnaser: given it's a RT log, you should get it every periodic time (default is 60 secs)
13:29:45	bauzas	mnaser: until you confirm the resize
13:30:27	mnaser	it looks like the resize confirm just never reached the compute node
13:33:40	mnaser	"Found an unconfirmed migration during delete, id: 14157, status: confirming"
13:34:40	mnaser	weird
13:36:14	deepak_mourya	hi, in this bug https://bugs.launchpad.net/nova/+bug/1770636
13:36:15	deepak_mourya	Do we need to add exception for CannotDeleteParentResourceProvider only in https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/resource_provider.py#L130-L140 and write test case or something else ?
13:36:15	openstack	Launchpad bug 1770636 in OpenStack Compute (nova) "placement API not translating CannotDeleteParentResourceProvider to 409 Conflict" [Low,Triaged] - Assigned to Deepak Mourya (mourya007)

Earlier   Later