Posted	Nick	Remark
#openstack-nova - 2017-12-17
07:43:55	openstackgerrit	Ade Lee proposed openstack/nova master: Add flag to make glance image verification optional https://review.openstack.org/528017
10:33:21	openstackgerrit	Merged openstack/nova master: [placement] Separate API schemas (inventory) https://review.openstack.org/520613
14:18:17	openstackgerrit	Ahmed Zaid proposed openstack/nova master: Flavors with illegal names. https://review.openstack.org/528563
14:19:03	openstackgerrit	Ahmed Zaid proposed openstack/nova master: Flavors with illegal names. https://review.openstack.org/528563
22:42:45	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
22:42:45	openstackgerrit	Ildiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047
22:47:29	openstackgerrit	Chris Dent proposed openstack/nova master: Implement set_and_clear_allocations in report client https://review.openstack.org/528089
23:28:42	openstackgerrit	Tetsuro Nakamura proposed openstack/nova master: Fix duplicate allocation candidates https://review.openstack.org/528591
#openstack-nova - 2017-12-18
01:32:12	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
01:39:14	openstackgerrit	Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395
02:41:03	openstackgerrit	Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
02:42:08	openstackgerrit	Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
04:27:34	openstackgerrit	ChangBo Guo(gcb) proposed openstack/nova master: Use method validate_integer from oslo.utils https://review.openstack.org/528620
04:31:28	vivsoni	Hi Team
04:31:30	vivsoni	when i was trying to attach a volume to an instance
04:31:37	vivsoni	using 'nova volume-attach <inst_id> <vol_id>'
04:31:45	vivsoni	the volume state went to 'reserved' status
04:32:04	vivsoni	and i could see my drivers 'initialize_connection()' is not called
05:02:25	openstackgerrit	ChangBo Guo(gcb) proposed openstack/nova master: Use method validate_integer from oslo.utils https://review.openstack.org/528620
05:22:11	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_provider) https://review.openstack.org/528629
05:38:15	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (allocation) https://review.openstack.org/528630
05:57:40	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (allocation_candidate) https://review.openstack.org/528635
06:00:13	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell' https://review.openstack.org/513771
06:02:27	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579
06:02:45	openstackgerrit	Takashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007
06:03:12	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix the order of target host checks https://review.openstack.org/526225
06:58:38	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513717
07:07:17	openstackgerrit	OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
07:53:03	openstackgerrit	Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
07:56:44	openstackgerrit	Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
08:08:30	openstackgerrit	Ahmed Zaid proposed openstack/nova master: Flavors with illegal names. https://review.openstack.org/528563
08:12:06	openstackgerrit	Ahmed Zaid proposed openstack/nova master: Flavors with illegal names. https://review.openstack.org/528563
09:01:45	openstackgerrit	Merged openstack/nova stable/pike: Fix CellDatabases fixture swallowing exceptions https://review.openstack.org/519702
09:14:14	openstackgerrit	rahul bardia proposed openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique https://review.openstack.org/527922
09:17:08	openstackgerrit	Lajos Katona proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728
09:23:33	gibi	happy Monday nova!
09:24:03	openstackgerrit	Merged openstack/nova stable/pike: Handle InstanceNotFound when setting password via metadata https://review.openstack.org/519659
09:31:19	maciejjozefczyk	gibi: I wish it could be happy :)
09:31:55	openstackgerrit	Lajos Katona proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728
09:37:07	openstackgerrit	Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
09:41:13	mdbooth	lyarwood: https://review.openstack.org/#/c/460243/6 interesting.
09:41:50	mdbooth	lyarwood: For the other recent encryptor bug, I was thinking of folding attach/detach encryptor into attach/detach volume
09:42:07	mdbooth	So the caller would never actually worry about encryptors at all, just volumes.
09:42:25	mdbooth	Which is nearly where you've gone with that. Any reason not to go all the way?
09:43:23	lyarwood	mdbooth: iirc just booting from a volume wouldn't call attach/detach volume
09:43:32	lyarwood	mdbooth: see _create_domain_and_network
09:44:54	mdbooth	lyarwood: That's a bug, though, right?
09:45:32	mdbooth	In that, that specifically was the cause of the other problem. We worked round it.
09:45:33	lyarwood	mdbooth: why? you can't attach to a domain that isn't running
09:45:34	gibi	maciejjozefczyk: only 3 working day left for me from this year so I this is my last work Monday, therefore I feel happy :)
09:46:28	mdbooth	lyarwood: How do we boot from volume?
09:46:59	mdbooth	Do we not call attach in that flow?
09:49:00	lyarwood	mdbooth: pretty sure we don't, we just connect the volume and encryptors before starting the domain
09:49:16	maciejjozefczyk	gibi: Good for you :)
09:50:24	mdbooth	Although I guess the binding would be specifically to connect/disconnect
09:50:48	kashyap	If anyone looking for a simple +2 to warm up their Monday - https://review.openstack.org/#/c/527691/
09:50:53	mdbooth	Mounting a volume on a host without connecting its encryptor would always be a bug, I think
09:51:06	mdbooth	lyarwood: Anyway, I'm not saddling up for a -1 on your patch for this :)
09:51:40	mdbooth	lyarwood: I'd just been looking at combining this logic recently to make it harder to make mistakes.
09:51:55	mdbooth	And you're simplifying it somewhat here, which is good.
10:00:58	mdbooth	lyarwood: incidentally, I suspect all connect calls go through driver._connect_volume
10:02:12	mdbooth	So if the arguments lined up, I'd put the volume encryptor stuff in there, and remove all other calls.
10:03:07	mdbooth	That makes sense to me, anyway. May require some refactoring to make reality match theory.
10:04:34	lyarwood	mdbooth: urgh that's going to mess with the exception handling we currently have
10:11:47	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
10:13:42	mdbooth	lyarwood: Doesn't need to happen now, I'm just firming up a related idea.
10:13:57	mdbooth	Although I do think it would be much more robust.
10:15:10	lyarwood	mdbooth: kk, tbh my own reason for doing this refactor is to limit the number of conditionals when switching to native LUKS via QEMU
10:15:14	lyarwood	only*
10:59:51	openstackgerrit	Chen Hanxiao proposed openstack/nova master: libvirt: don't call sync_guest_time if qga is not enabled https://review.openstack.org/524836
11:54:24	openstackgerrit	Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
12:22:30	openstackgerrit	Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
12:42:43	openstackgerrit	Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
12:59:30	openstackgerrit	sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008
13:13:59	openstackgerrit	Artom Lifshitz proposed openstack/nova master: DNM: Test making column Text -> MediumText without migration https://review.openstack.org/528712
13:40:18	leakypipes	edleafe: mornin. we doing a scheduler meeting today?
13:41:02	cdent	leakypipes: do you want one, or are you more about trying to manage your schedule?
13:41:14	leakypipes	cdent: more managing my schedule
13:41:24	leakypipes	cdent: and just curious who's here :)
13:41:29	leakypipes	cdent: and good morning sir
13:41:38	cdent	mawrnin'
13:43:02	cdent	I am basically here, but still have not shed my cold.
13:46:02	bauzas	cdent: leakypipes: honestly, I'm working on implementing the VGPU change, so only if needed, I'd prefer to work
13:46:17	leakypipes	ack
13:46:39	bauzas	shit for libvirt btw
13:47:15	bauzas	cdent: <3
13:48:08	cdent	this is what I'm going to do henceforth: hang out in irc and pass people nice stuff.
13:52:20	bauzas	cdent: thanks for the hot cocoa https://photos.app.goo.gl/qLBp2ZnFcJc2pvAp2
13:58:15	edleafe	leakypipes: yes, this is the last meeting of the year
13:58:26	edleafe	and plug your leak
13:58:51	stephenfin	bauzas, leakypipes: I've a question on the PCI NUMA policies spec for the two of you
13:59:09	bauzas	ack
13:59:20	leakypipes	stephenfin: yes>?
13:59:29	stephenfin	Per leakypipes' comments, I've been moving the storing of the resulting policy to the InstancePCIRequests object
13:59:52	alex_xu	leakypipes: morning, do we still need a separate microversionf or the traits parameter in allocationcandidates, or Eric's granular request's patch will cover that?
14:00:17	stephenfin	But that's thrown up a question cfriesen had asked me previously - should we be doing this policy configuration on an instance-level basis or on a PCI request-level basis?
14:00:59	leakypipes	stephenfin: probably instance-level (i.e. the RequestSpec)
14:01:08	leakypipes	alex_xu: scheduler meeting... discuss in there?
14:01:21	alex_xu	leakypipes: ok, got it
14:02:02	leakypipes	stephenfin: unless the policy knob can be toggled per-device, then of course it would be more appropriate to live in InstancePCIRequest I think. thoughts?

Earlier   Later