Earlier  
Posted Nick Remark
#openstack-nova - 2019-03-07
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
09:26:06 openstackgerrit Adrian Chiris proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929
09:26:06 openstackgerrit Adrian Chiris proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365
09:29:58 HD|Laptop hey all. Assuming I have an existing QEMU VM in qcow2 format that needs to be run inside my Openstack environment, how do I best do this? create an image in glance, then spawn an instance based on it? or directly import it in cinder and create a server with no image and just the cinder volume attached?
09:34:21 bauzas HD|Laptop: see https://docs.openstack.org/image-guide/create-images-manually.html
09:35:26 bauzas HD|Laptop: also, see the channel topic, here is only for development questions
09:38:50 openstackgerrit Adrian Chiris proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929
10:03:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove duplicate cleanup in functional tests https://review.openstack.org/636996
10:35:36 openstackgerrit Merged openstack/nova master: Test proper allocation of devices during reshape https://review.openstack.org/639854
10:56:44 openstackgerrit Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
11:03:19 openstackgerrit Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
11:54:27 mdbooth lyarwood: I've discovered why your test is failing with failure to delete the volume type
11:54:49 mdbooth lyarwood: Although I haven't worked out what to do about it, yet, or why it appears to work in the py2 tests
11:56:02 mdbooth Anyway, when you create a volume from a glance image, it seems that cinder creates a new volume to hold the downloaded glance image with the same type as the volume being created: cinder's image cache
11:56:19 mdbooth The failure to delete the type is because the image cache volume still exists.
11:58:04 mdbooth lyarwood: Probably easiest to go ask some cinder folks how to handle that. I'll see if I can find some.
12:06:38 openstackgerrit Michael Still proposed openstack/nova master: Move slight bonkers IP management to privsep. https://review.openstack.org/624592
12:06:39 openstackgerrit Michael Still proposed openstack/nova master: Move FP device creation and deletion to privsep. https://review.openstack.org/635433
12:06:39 openstackgerrit Michael Still proposed openstack/nova master: Privsepify ipv4 forwarding enablement. https://review.openstack.org/635431
12:06:40 openstackgerrit Michael Still proposed openstack/nova master: Move adding vlans to interfaces to privsep. https://review.openstack.org/635436
12:06:40 openstackgerrit Michael Still proposed openstack/nova master: Privsep the ebtables modification code. https://review.openstack.org/635435
12:06:41 openstackgerrit Michael Still proposed openstack/nova master: Move dnsmasq restarts to privsep. https://review.openstack.org/639280
12:06:41 openstackgerrit Michael Still proposed openstack/nova master: Move iptables rule fetching and setting to privsep. https://review.openstack.org/636508
12:06:42 openstackgerrit Michael Still proposed openstack/nova master: Move calls to ovs-vsctl to privsep. https://review.openstack.org/639282
12:06:42 openstackgerrit Michael Still proposed openstack/nova master: Move router advertisement daemon restarts to privsep. https://review.openstack.org/639281
12:06:43 openstackgerrit Michael Still proposed openstack/nova master: Move final bridge commands to privsep. https://review.openstack.org/639580
12:06:43 openstackgerrit Michael Still proposed openstack/nova master: Move setting of device trust to privsep. https://review.openstack.org/639283
12:06:44 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
12:06:44 openstackgerrit Michael Still proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.openstack.org/639581
12:06:45 openstackgerrit Michael Still proposed openstack/nova master: Cleanup no longer required filters and add a release note. https://review.openstack.org/639826
12:52:04 openstackgerrit Stephen Finucane proposed openstack/nova stable/rocky: tests: Move mocking to setUp https://review.openstack.org/641651
12:52:05 openstackgerrit Stephen Finucane proposed openstack/nova stable/rocky: objects: Store InstancePCIRequest.numa_policy in DB https://review.openstack.org/641653
12:52:05 openstackgerrit Stephen Finucane proposed openstack/nova stable/rocky: tests: Create functional libvirt test base class https://review.openstack.org/641652
12:56:16 artom ~o~
13:00:29 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test macvtap port with resource request https://review.openstack.org/641309
13:09:11 openstackgerrit Surya Seetharaman proposed openstack/nova master: Delete orphaned RS and IM records of archived instances https://review.openstack.org/560042
13:23:56 yaawang kashyap: Hi, will you continue doing https://review.openstack.org/#/c/640866/? I want to fix some pep8 issues to pass the test :)
13:27:31 kashyap yaawang: Oh, yes. Let me fix that right away
13:27:34 kashyap yaawang: Sorry :-)
13:28:16 yaawang kashyap: thk :)
13:31:20 mriedem gibi: are you running today's meeting?
13:31:31 gibi mriedem: yes, I can do that
13:32:11 gibi mriedem: I read that melwitt is not here today, and that you will take care of the FF
13:33:43 mriedem yeah that's why i was wondering who was running the meeting
13:34:23 gibi mriedem: if you would like to run the meeting too then I have no problem with that either
13:34:40 mriedem i shouldn't since i have to get my kid to the bus in about an hour
13:34:51 gibi mriedem: OK, then I will run it
13:35:02 mriedem thanks
13:35:06 gibi np
13:36:04 jaypipes guten morgen
13:37:11 mriedem maybe i should make efried do all the FF stuff today...
13:37:17 mriedem jaypipes: question about indexes for you
13:37:30 mriedem jaypipes: in https://review.openstack.org/#/c/633349/
13:37:40 mriedem need to add just a user_id index or both user_id + project_id
13:37:41 efried mriedem: I would be down, with coaching.
13:37:56 openstackgerrit Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
13:38:28 mriedem efried: well the good news is we don't do milestone tags anymore so there is no need to tag s-3 for nova, right smcginnis?
13:38:37 mriedem mel already released osc-placement last night,
13:38:47 mriedem so i think the only release we have today is novaclient
13:39:17 efried okay, just propose a novaclient release?
13:39:31 mriedem yeah, will need this first https://review.openstack.org/#/c/637234/
13:39:35 efried ...after checking if there's anything... yeah
13:41:36 openstackgerrit Merged openstack/python-novaclient master: Add support for microversion v2.72 https://review.openstack.org/637234
13:42:14 gibi and it is merged ^^
13:43:07 openstackgerrit Kashyap Chamarthy proposed openstack/nova-specs master: cpu-model-selection: s/cpu_models/cpu_model_list/ https://review.openstack.org/640866
13:43:14 kashyap yaawang: ^ Hope that fixes it
13:43:34 jaypipes mriedem: answered.
13:43:49 jaypipes mriedem: fine to put a single index on user_id. was a good catch from dansmith
13:44:08 jaypipes mriedem: in a future patch, we can add a dual index if that is something we see is needed.
13:44:45 jaypipes mriedem: as melwitt stated, the server group members query is on user only, not project, so a dual index on (project_id, user_id) would not help that query.
13:45:21 mriedem jaypipes: but this one does use both https://review.openstack.org/#/c/638072/17/nova/objects/instance_mapping.py@407
13:45:47 jaypipes ffs...
13:46:05 mriedem so i guess we'd want a user_id and user_id+project_id index?
13:46:19 jaypipes mriedem: no, it doesn't actually.
13:46:25 jaypipes mriedem: it's two queries :(
13:46:40 jaypipes mriedem: that ffs wasn't directed at you.
13:46:55 jaypipes mriedem: each .scalar() is a sep query.
13:47:00 mriedem yeah i realize
13:47:11 mriedem this is a pattern in the quota qounting query code...
13:47:13 mriedem in a few places
13:47:19 jaypipes oh, I see..
13:47:25 jaypipes hrmph
13:48:36 jaypipes mriedem: ok, well if the pattern is that user_id is always queries along with project_id, then yes, the index should be on (project_id, user_id)
13:49:06 mriedem well project_id isn't always in there https://review.openstack.org/#/c/638324/19/nova/objects/instance_mapping.py@466
13:49:57 mriedem the first one above is a replacement for the "instances" quota counting which is over project and user,

Earlier   Later