Posted	Nick	Remark
#openstack-nova - 2019-03-06
23:25:13	cfriesen	mikal: did you intentionally omit "mv" or "cp" from privsep?
23:27:14	mikal	cfriesen: privsep only has things which had callers. So an absence either means that the user hasn't merged yet, or there is no user in nova until you came along.
23:28:03	openstackgerrit	melanie witt proposed openstack/nova master: Add user_id field to InstanceMapping https://review.openstack.org/633350
23:28:04	openstackgerrit	melanie witt proposed openstack/nova master: Populate InstanceMapping.user_id during migrations and schedules https://review.openstack.org/638574
23:28:04	openstackgerrit	melanie witt proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351
23:28:05	openstackgerrit	melanie witt proposed openstack/nova master: Add get_counts() to InstanceMappingList https://review.openstack.org/638072
23:28:05	openstackgerrit	melanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement https://review.openstack.org/638073
23:28:06	openstackgerrit	melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.openstack.org/638324
23:29:02	cfriesen	mikal: I need to copy a subtree under /var/lib/libvirt/swtpm/<uuid> and recursively chmod or chown it. On the dest I need to do the opposite.
23:29:30	cfriesen	I'm wondering whether from a security standpoint it'd make more sense to create these under libvirt, or as generic operations in path.py
23:30:49	mikal	cfriesen: I think I'd put it under path, it seems generically useful.
23:31:03	mikal	cfriesen: there's no security difference between either location, its more about who uses it.
23:37:01	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
23:37:33	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125
23:45:53	openstackgerrit	melanie witt proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351
23:45:54	openstackgerrit	melanie witt proposed openstack/nova master: Add get_counts() to InstanceMappingList https://review.openstack.org/638072
23:45:54	openstackgerrit	melanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement https://review.openstack.org/638073
23:45:55	openstackgerrit	melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.openstack.org/638324
23:52:24	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1818914 https://review.openstack.org/641521
23:52:26	openstack	bug 1818914 in OpenStack Compute (nova) "Hypervisor resource usage on source still shows old flavor usage after resize confirm until update_available_resource periodic runs" [Low,Triaged] https://launchpad.net/bugs/1818914
23:52:26	mriedem	cfriesen: ^ recreate test at least
23:59:43	melwitt	mriedem: I think I'm not going to be able to be around tomorrow... :( on that note, I'm not sure how helpful it'd be to land the migration without the user-impacting changes,
#openstack-nova - 2019-03-07
00:00:14	melwitt	there's an open question on the bottom patch about what table index(es) we should add, need jay's input for that
00:01:11	mriedem	melwitt: ok...who is dealing with final client release and any tags we need (don't think we do milestone tags for nova anymore)
00:01:47	mriedem	shit this reminds me that we need a python-novaclient bump for 2.72 if we want to supported bw provider ports with nova boot...
00:01:49	mriedem	gibi: ^
00:02:02	melwitt	mriedem: I hoped to get https://review.openstack.org/640898 out in the final osc-placement release. I can propose the novaclient release tonight, same with osc-placement if you think the allocation ratio change won't make it
00:02:36	mriedem	i haven't reviewed that allocation ratio thing, it's late for that imo
00:03:04	mriedem	the instance mapping user_id is valuable in stein if we can get it because it means less data to migrate in train
00:03:20	mriedem	i'm also assuming cern is going to be backporting this stuff...
00:03:26	efried	cfriesen: still around?
00:03:28	melwitt	ok. that's too bad. cdent and I have tested it locally a bit and it has a func test, but yeah
00:03:28	mriedem	i.e. the counting quota stuff from train to stein
00:03:46	mriedem	well, it was posted 2 days ago so...
00:04:20	mriedem	anyway, i can do a novaclient release tomorrow since we need 2.72 for gibi's stuff
00:04:28	melwitt	yeah, I spent a long time writing the func test for it. I guess I could have uploaded it in two stages
00:04:32	efried	cfriesen: going once... okay, I'm going to update the commit message on https://review.openstack.org/#/c/641480/ and +2 it.
00:04:34	mriedem	and i'd like to ge the instance mapping user_id stuff in at least for the 'less data to migrate' thing
00:04:52	openstackgerrit	Eric Fried proposed openstack/nova master: Add missing libvirt exception during device detach https://review.openstack.org/641480
00:05:00	melwitt	ok. I'll propose the osc-placement release tonight then
00:05:14	melwitt	feel free to make whatever changes are needed to the counting quotas migration stuff tomorrow
00:05:28	melwitt	(anyone feel free)
00:06:16	melwitt	I guess the osc-placement thing isn't so big a deal because could have another release soon-ish and not hard for people to upgrade
00:06:28	melwitt	client stuff is a lot easier
00:07:10	sean-k-mooney	melwitt: oh you coded up the aggreate allocation raito feature
00:08:44	melwitt	sean-k-mooney: yeah :) it took a long time because I added a --dry-run option to it where it'll show all the previewed payloads it would send for each resource provider (since you have to get/set to do updates)
00:09:09	melwitt	and I never saw how the command return value formatting works before with cliff, it was an experience xD
00:09:25	sean-k-mooney	melwitt: am one question...
00:09:51	melwitt	mriedem: I'll address any comments on counting quotas that land tonight/before tomorrow morning. I see you have one -1 on there already
00:10:00	sean-k-mooney	the compute service wont stomp on your changes right. did we complete the inial allcoation ratio stuff?
00:10:23	melwitt	sean-k-mooney: right. initial allocation ratios landed
00:10:38	melwitt	awhile back
00:11:22	sean-k-mooney	ah ok cool so as long as you dont set the allcoation ration for cpus exctra in the nova.conf and only set the inital_allocation ratio then this will work as expected
00:13:03	melwitt	yeah, something like that, I thought maybe you have to set to None and also set the initial ratios. to tell it "take the values from placement"
00:13:04	melwitt	I can't remember atm
00:13:26	sean-k-mooney	dont worry about it
00:13:51	sean-k-mooney	there is a way to do it i just wanted to make sure this would not get overriden
00:13:57	mriedem	sean-k-mooney: here you go https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#allocation-ratios
00:14:15	mriedem	sure am glad someone documented all of that...
00:15:26	melwitt	haha yup. I was gonna say, we have docs!
00:15:31	sean-k-mooney	i mean we spent enough time debating it in the spec we might as well record it in the admin docs too so we dont forget
00:16:04	melwitt	yeah, that is not something anyone wants to try and figure out again
00:16:30	sean-k-mooney	we do. we also have docs for the downstream issue we were having with the rdo cloud just beneath that
00:36:28	sean-k-mooney	the subunit parser but is still a thing :(
00:40:06	openstackgerrit	guang-yee proposed openstack/nova master: pass endpoint interface to Ironic client https://review.openstack.org/640879
00:52:12	openstackgerrit	Merged openstack/nova master: Extend volume for libvirt network volumes (RBD) https://review.openstack.org/613039
00:52:20	openstackgerrit	Merged openstack/nova master: doc: cleanup pci.alias references https://review.openstack.org/640882
01:24:54	openstackgerrit	melanie witt proposed openstack/nova master: Populate InstanceMapping.user_id during migrations and schedules https://review.openstack.org/638574
01:24:55	openstackgerrit	melanie witt proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351
01:24:55	openstackgerrit	melanie witt proposed openstack/nova master: Add get_counts() to InstanceMappingList https://review.openstack.org/638072
01:24:56	openstackgerrit	melanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement https://review.openstack.org/638073
01:24:56	openstackgerrit	melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.openstack.org/638324
01:29:07	stephenfin	sean-k-mooney: This is the one and only tm
01:29:17	stephenfin	*time I'll be online this late
01:29:47	stephenfin	(darn waterfall development process...)
01:46:14	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Should not skip volume_size check for bdm.image_id == image_ref case https://review.openstack.org/641282
01:46:24	openstackgerrit	Matt Riedemann proposed openstack/python-novaclient master: Add support for microversion v2.72 https://review.openstack.org/637234
01:47:49	mriedem	stephenfin: takashin: can one of you approve the 2.72 novaclient change https://review.openstack.org/#/c/637234/ ? it's really just docs, but we need to get it in so we can release novaclient tomorrow.
01:48:49	takashin	mriedem: Okay. I will do it.
01:49:50	mriedem	thanks
01:57:08	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
03:58:09	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
05:52:34	openstackgerrit	Merged openstack/nova master: Move arping to privsep. https://review.openstack.org/624593
05:52:41	openstackgerrit	Merged openstack/nova master: Move cleaning conntrack to privsep. https://review.openstack.org/624594
05:52:50	openstackgerrit	Merged openstack/nova master: Move killing processes to privsep. https://review.openstack.org/635432
05:52:58	openstackgerrit	Merged openstack/nova master: Cleanup the exec_ebtables code a little. https://review.openstack.org/635434
06:10:39	openstackgerrit	guang-yee proposed openstack/nova master: pass endpoint interface to Ironic client https://review.openstack.org/640879
06:17:53	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
06:40:52	openstackgerrit	Merged openstack/nova master: fakelibvirt: Add ability to generate fake PCI devices https://review.openstack.org/640409
08:01:25	openstackgerrit	Kevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter https://review.openstack.org/633178
08:29:52	tssurya	good morning nova, if any cores are around could you please reinstate the +W here: https://review.openstack.org/#/c/606995/ , I sort of lost it during rebase and FYI: its only a simple doc update
08:36:43	gibi	tssurya: done
08:37:22	gibi	tssurya: the functional test failure with parser error was really a nasty thing.
08:37:40	gibi	tssurya: got fixed with multiple commit from mriedem
08:47:14	tssurya	gibi: thanks a lot!
08:47:27	tssurya	yea thanks to you and mriedem for fixing that
08:49:04	gibi	tssurya: is there anything you need help before the FF?
08:49:33	tssurya	gibi: not really :) the stuff I was working on is all in
08:50:37	gibi	tssurya: nice :)
09:26:05	openstackgerrit	Adrian Chiris proposed openstack/nova master: Separate methods to free claimed and allocated devs https://review.openstack.org/616120

Earlier   Later