Posted	Nick	Remark
#openstack-nova - 2018-01-04
23:07:25	jaypipes	coolio.
23:08:01	hemna_	lyarwood, https://bugs.launchpad.net/nova/+bug/1741364
23:08:02	openstack	Launchpad bug 1741364 in OpenStack Compute (nova) "ceph ephemeral info not updated during live migrate" [Undecided,New]
23:11:54	efried	jaypipes See response on https://review.openstack.org/#/c/526539/ -- not sure if I've answered the right question there.
23:26:09	mnaser	does nova not set the instance status to ERROR if cinder volume detach fails?
23:26:39	mnaser	I have some cinder volumes which show attached servers, but the server is deleted
23:31:10	mriedem	mnaser: correct
23:31:40	mriedem	normal volume detach? or volume detach during instance delete?
23:31:53	mnaser	mriedem: volume detach during instance delete
23:32:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: [libvirt] Allow multiple volume attachments https://review.openstack.org/267587
23:32:22	openstackgerrit	Matt Riedemann proposed openstack/nova master: Pass multiattach flag to reserve_block_device_name https://review.openstack.org/531244
23:32:24	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: [api] Allow multi-attach in compute api https://review.openstack.org/271047
23:32:34	mriedem	mnaser: yeah that happens here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2391
23:32:35	mnaser	(boot from volume instance)
23:32:49	mriedem	if it fails we log something but keep going
23:32:54	mriedem	because we've already destroyed the guest on the hypervisor
23:33:23	mriedem	this is where we try to delete the volume if delete_on_termination=True https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2440
23:33:59	mnaser	hmmm
23:34:39	mnaser	because now i end up with volumes with attachments (for whatever reason) and there doesn't seem to be a straight forward way of clearing that out
23:34:50	mriedem	force detach in cinder?
23:35:15	mnaser	i see force delete but not force detach
23:35:19	mnaser	(force delete is ok for me though)
23:36:17	mriedem	that might be what i was thinking of
23:36:29	mriedem	smcginnis: knows
23:36:29	openstackgerrit	Andreas Karis proposed openstack/nova master: Add debug output for selected page size https://review.openstack.org/530662
23:36:57	mnaser	Delete for volume 973d5a3d-07f3-4d05-b62c-8dfe746d298c failed: Invalid volume: Volume must not be migrating, attached, belong to a group, have snapshots or be disassociated from snapshots after volume transfer. (HTTP 400) (Request-ID: req-ecfad96c-8125-47e4-8e25-9a9e892139aa)
23:37:01	mnaser	i guess this is in cinder-land
23:37:21	mnaser	(cant force delete an attach volume)
23:37:50	mriedem	mnaser: https://developer.openstack.org/api-ref/block-storage/v2/#force-detach-volume
23:38:22	mordred	wow. y'all are having all the fun
23:38:39	mriedem	mordred: this channel is a barrel of laughs all day every day
23:38:52	mordred	mriedem: that's what I tell people
23:39:15	mordred	"looking for a barrel of laughs? go check out #openstack-nova!", I tell them
23:40:19	mnaser	mriedem: interesting, i could probably even normal detach, but i wonder if the cinder cli client has it
23:41:01	mnaser	hopefully scaling up our cinder api endpoints should mean that we don't see this again
23:41:32	mriedem	i don't see a force detach command in cinderclient
23:46:22	SamYaple	i dont think there is one...
23:46:50	SamYaple	inconsitencies between nova and cinder ive always had to go to the db to solve
23:47:03	mriedem	there is an api
23:47:05	mriedem	so curl should work
23:47:20	mriedem	should also be easy enough to add a cli for this in cinderclient
23:47:38	SamYaple	indeed
#openstack-nova - 2018-01-05
00:00:54	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
00:17:24	openstackgerrit	Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
00:29:02	mriedem	alex_xu: takashin: important question in https://review.openstack.org/#/c/330406/78/api-ref/source/parameters.yaml@4148
00:29:19	mriedem	this microversion adds a uuid field to the response for GET /os-migrations,
00:29:42	mriedem	i suggested we could also take this opportunity to add uuid to the response for GET /servers/{uuid}/migrations as well
00:29:55	mriedem	thoughts on that?
00:30:35	mriedem	comment on the review if you like, i have to make dinner
01:29:56	openstackgerrit	Merged openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
02:34:35	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
03:51:14	openstackgerrit	Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
05:46:57	openstackgerrit	Andreas Jaeger proposed openstack/os-vif master: zuul: Enable functional tests in gate https://review.openstack.org/530961
06:10:59	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299
06:53:28	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove a duplicate line in a unit test https://review.openstack.org/531317
07:04:05	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117
07:24:20	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117
07:37:10	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Add ComputeNode.virt_type field https://review.openstack.org/531327
07:37:11	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Use virt_type in /os-hypervisors/details API https://review.openstack.org/531328
07:37:16	openstackgerrit	Chen Hanxiao proposed openstack/nova master: conf: hyperv: fix a comment typo https://review.openstack.org/531329
07:48:34	openstackgerrit	Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965
07:49:01	hrw	stephenfin: added test for ^^ - can you take a look?
08:09:57	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429
08:14:09	kashyap	hrw: Oh, yes, I thought of the test in your change, but forgot to mention it. Cool that you've worked it out.
08:14:48	hrw	kashyap: I was aware that test is needed. Just had no idea how to write it. Kevin wrote it for me.
08:15:16	kashyap	I see, ah, missed the co-author bit.
08:15:32	kashyap	hrw: I myself struggle with the unit tests as I don't look at it terribly often
08:15:52	hrw	kashyap: Kevin knows Nova code. I don't
08:28:49	hrw	mailed ML
08:57:50	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
09:18:33	stephenfin	hrw: Sure
09:19:28	hrw	stephenfin: thanks.
09:19:53	hrw	stephenfin: hope that someone will take a look too. Not adding reviewers as I do not know who to add
09:29:15	openstackgerrit	ChangBo Guo(gcb) proposed openstack/nova master: Use method validate_integer from oslo.utils https://review.openstack.org/528620
09:50:42	hrw	stephenfin: by 'link to your blog' you mean https://marcin.juszkiewicz.com.pl/2018/01/04/today-i-was-fighting-with-nova-no-idea-who-won/ link?
09:50:56	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Fix hypervisor_type in /os-hypervisors/details https://review.openstack.org/531328
09:50:57	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: [libvirt] Filter hypervisor_type by virt_type https://review.openstack.org/531347
09:51:07	stephenfin	hrw: Yup
09:51:14	hrw	ok
09:53:19	openstackgerrit	Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965
09:53:23	hrw	reno updated, link added
09:54:27	kashyap	hrw: Thanks for the quick feedback
09:54:35	kashyap	s/feedback/response/
09:55:24	hrw	kashyap: it's my job ;D
09:55:58	hrw	kashyap: and I prefer to have patches quickly updated to keep reviewers attention
09:56:12	kashyap	hrw: Is this pressing? If so you have to state it, so reviewers calibrate their queue accordingly :-)
09:56:31	kashyap	Those who can +2, i.e.
09:56:53	hrw	kashyap: it would be lovely to get both my patches into Queens. That level of priority.
09:57:17	kashyap	Yeah, it should. Upstream is not _that_ terribly slow for self-contained stuff like this.
09:57:34	kashyap	This is a straight-forward one. No delay should be done in merging this.
09:57:50	hrw	We have workaround for cpu_mode already merged into Kolla and I know that it is present in some other ways of deploying too.
09:59:09	hrw	UEFI one can not be workarounded
10:01:48	hrw	added some random core reviewers to both patches
10:03:43	sean-k-mooney	stephenfin: o/ happy new year
10:03:51	stephenfin	sean-k-mooney: o/ ditto
10:04:05	sean-k-mooney	see my comment on https://review.openstack.org/#/c/530961/2
10:04:35	stephenfin	sean-k-mooney: Hahaha
10:04:41	stephenfin	This is what I'm currently reading
10:04:46	stephenfin	https://docs.python.org/2/library/subprocess.html#subprocess.Popen.communicate
10:05:04	stephenfin	I was trying to capture stderr/stdout to see if that was indeed the issue (I suspected it was)

Earlier   Later