Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-18
11:12:17 slaweq sean-k-mooney: ok, so You are aware and it's not related to neutron, right?
11:12:23 sean-k-mooney i think https://review.openstack.org/#/c/583347/2 will fix it
11:13:06 sean-k-mooney the anti affinity filter is kindof broken because we now allow up to n server to land on the same host
11:13:24 slaweq sean-k-mooney: ok, thx for info, that's enough for me :)
11:13:37 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
11:15:56 sean-k-mooney slaweq: there is an elastic recheck query for this here http://status.openstack.org/elastic-recheck/#1781710 matt was workin on this and i said i would try and help reproduce
11:16:23 sean-k-mooney slaweq: we could add a tempory skip if this is blocking some changes in neutron
11:16:50 slaweq sean-k-mooney: no, it's not so big deal for us because it hits us in non voting jobs mostly
11:17:08 slaweq but I saw it quite many times in last week so I just wanted to ask :)
11:18:27 sean-k-mooney good to know. if it does become a problem feel free to flag it to us agian.
11:19:20 slaweq sean-k-mooney: sure, thx
11:40:25 openstackgerrit Merged openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398
11:59:09 openstackgerrit xulei proposed openstack/nova master: Disable limits if force_hosts or force_nodes is set https://review.openstack.org/576693
12:04:32 jaypipes stephenfin: +2 on all in series.
12:05:03 jaypipes stephenfin: thx for spearheading that stuff. it's really not my cup o tea but I'm glad to be aligned with the rest of openstack on it now.
12:24:11 openstackgerrit Merged openstack/nova master: hardware: Start accounting for networks in NUMA fitting https://review.openstack.org/564448
12:24:20 openstackgerrit Merged openstack/nova master: objects: Add RequestSpec.network_metadata https://review.openstack.org/564442
12:25:27 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for bdm extensions https://review.openstack.org/583559
12:28:16 openstackgerrit Merged openstack/os-traits master: tox: Remove '-constraints' targets https://review.openstack.org/583533
12:28:17 openstackgerrit Merged openstack/os-traits master: Follow the new PTI for document build https://review.openstack.org/583534
12:30:59 openstackgerrit Merged openstack/os-traits master: Switch to stestr https://review.openstack.org/581265
12:41:59 openstackgerrit sahid proposed openstack/nova master: libvirt: remove unused attribute driver for LibvirtConfigNodeDevice https://review.openstack.org/583246
12:42:00 openstackgerrit sahid proposed openstack/nova master: libvirt: add format_dom for ConfigNodeDevicePciSubFunctionCap https://review.openstack.org/583247
12:42:01 openstackgerrit sahid proposed openstack/nova master: libvirt: add missing type for LibvirtConfigNodeDevicePciCap https://review.openstack.org/583248
12:42:02 openstackgerrit sahid proposed openstack/nova master: libvirt: add format_dom() for config LibvirtConfigNodeDevicePciCap https://review.openstack.org/583249
12:42:03 openstackgerrit sahid proposed openstack/nova master: libvirt: add format_dom() method for LibvirtConfigNodeDevice https://review.openstack.org/583250
12:42:04 openstackgerrit sahid proposed openstack/nova master: libvirt: add generater of fake pci devices capabilities https://review.openstack.org/583251
12:47:18 sean-k-mooney stephenfin: jaypipes efried so regarding https://review.openstack.org/#/c/583533/1
12:47:37 sean-k-mooney that will break if os-traits is added to upper-constraints
12:47:59 sean-k-mooney for libs you have to preprocess the upper-constratins and remove them from the list
12:51:48 sean-k-mooney ill create a script that works and propose it to both os-vif and os-traits
12:52:26 openstackgerrit sahid proposed openstack/nova master: hardware: fix memory check usage for small/large pages https://review.openstack.org/532168
12:54:29 jaypipes sean-k-mooney: I shall let you and stephenfin fight that out in a death match.
12:55:42 sean-k-mooney hehe well what stephenfin has will work. until the point that os-traits gets added to upperconstraints
12:56:17 sean-k-mooney its exactly how nova works but nova is not a lib
12:59:08 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Handle HostMappingNotFound when deleting a compute service https://review.openstack.org/583566
13:00:07 do3meli someone able to review this one here: https://review.openstack.org/#/c/578040/ ? thanks
13:01:00 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Handle HostMappingNotFound when deleting a compute service https://review.openstack.org/583567
13:27:55 JulienCornuwel Hi, since the deprecation of nova hooks, I wonder what would be the correct way to handle my use case: I'm doing pci_passthrough of NVME drives to instances, and I want to wipe them clean when the customer destroys his instance. I can't use qemu hooks since they are run at stop/start and I don't want to wipe the drive on a simple reboot. Any pointers?
13:34:26 gibi JulienCornuwel: you can wait for instance.delete.end notifications from nova https://docs.openstack.org/nova/latest/reference/notifications.html
13:35:26 pooja_jadhav dansmith: Hello
13:37:01 JulienCornuwel gibi: Is there a way to prevent nova from giving the drive to another customer until I'm done with the wiping? (takes about 30-40 seconds) I've done some tests, and if I run a nova boot during the wipe, nova just takes the device wether I'm done cleaning it or not…
13:38:28 do3meli thx :efried. will work through it now
13:38:28 efried do3meli: Reviewed
13:38:41 efried JulienCornuwel: Are you looking to write code to do this, or are you trying to do it under the existing codebase?
13:38:50 gibi JulienCornuwel: that would require a way to reserve the given PCI device while you are wipeing it
13:39:00 stephenfin sean-k-mooney: I saw that tox install script in a couple of oslo repos but not all of them. I thought mordred or someone had worked around this now, tbh
13:39:32 efried pooja_jadhav: I think Dan is on vacation this week.
13:40:19 pooja_jadhav efried: Ohh.. thanks for update. I want to discuss about the patch https://review.openstack.org/#/c/580720/
13:40:33 mordred stephenfin: we still have tos intall scripts laying around?
13:40:54 stephenfin mordred: aye, lemme root some out
13:40:58 sean-k-mooney stephenfin: maybe but when i made the same change to os-vif it conmplained
13:41:05 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/578261
13:41:08 mgariepy hello, does "nova-manage cell_v2 map_instances" needs to be run on every upgrade ? It ran on my upgrade from newton to ocata and on the ocata to pike taking about 40 minutes each time.
13:41:33 efried pooja_jadhav: Maybe someone here can help. What do you need?
13:41:54 JulienCornuwel efried: We could write some code. But I was wondering if there was already a way that I was missing.
13:42:09 stephenfin mordred: I see oslo.log, oslo.db, and oslo.cache locally
13:42:20 JulienCornuwel gibi: exactly, is there a way to do that.
13:42:28 stephenfin though there could be more that I don't have local copies of
13:42:48 sean-k-mooney stephenfin: as far as i am aware all libs need to have the script
13:43:00 efried JulienCornuwel: I can't think of anything. If you're writing code, you could do as gibi suggests and, in response to the notification, remove the device from inventory reporting (get_available_resource) until you're done with it.
13:43:04 mordred stephenfin: wow. just when I thought we'd gotten rid of all of those :)
13:43:14 gibi JulienCornuwel: I don't think we have current a way to reserver a PCI dev
13:44:13 pooja_jadhav efried: Based on above patch https://review.openstack.org/#/c/580720/ submitted by Dan, When we boot an instance from volume then it should not consider local disk_gb. But I have applied patch and tried to tested the same but i can that local disk gb is getting used by seeing nova hypervisor-stats.
13:44:34 sean-k-mooney stephenfin: mordred this is what i get with the same change to os-vif
13:44:36 sean-k-mooney http://paste.openstack.org/show/726197/
13:44:59 efried pooja_jadhav: libvirt, I assume?
13:45:01 pooja_jadhav see*
13:45:11 stephenfin sean-k-mooney: Stop overriding install_command
13:45:12 pooja_jadhav efried: yes
13:45:59 stephenfin sean-k-mooney: It should simply read 'install_command = pip install {opts} {packages}' (or possibly it can be unset - I'm not sure). mordred can probably explain why that breaks
13:46:08 sean-k-mooney stephenfin: this is what i have install_command = pip install -U -c {env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}
13:46:19 stephenfin sean-k-mooney: Yup, see above
13:46:30 JulienCornuwel efried: gibi: Thanks for the confirmation. We'll start working on a spec and thinking of a way to do that.
13:46:45 efried pooja_jadhav: So the instance is actually booting from volume - i.e. truly not using the local disk - but nova hypervisor-stats is showing that capacity as having been applied from the local disk?
13:46:50 sean-k-mooney oh you put the constatit in deps
13:46:54 tssurya mgariepy: well the aim of map_instances is to just map the instances to the cells, so if you have done it once, and you are sure all instances are mapped, I don't see any reason to keep running it, unless you have new ones to populate
13:47:08 stephenfin sean-k-mooney: Ah, I get it. You don't want to install your local package with constraints but you _do_ want to install your test requirements
13:47:14 stephenfin ...with them
13:47:15 mgariepy tssurya, https://github.com/openstack/openstack-ansible-os_nova/blob/stable/pike/tasks/nova_db_setup.yml#L58-L90
13:47:18 stephenfin sean-k-mooney: yuuup
13:47:47 efried JulienCornuwel: Note that things involving the existing PCI passthrough subsystem are going to be difficult to get approval for, because we're trying to get rid of that whole thing.
13:47:54 sean-k-mooney stephenfin: yep
13:47:56 mgariepy so if the cell_v2 mapping was done in the upgrade, (and the create_cell returns 2, it would be right to skip the map_instances
13:48:39 efried JulienCornuwel: That said, I think you could do what you want without really hitting that code; and in such a way that it would be transferrable to placement-based device passthrough in the future.
13:48:39 JulienCornuwel efried: In favor of? Cyborg?
13:48:58 pooja_jadhav efried: After looking at the code changes done by Dan, its not taking disk_gb resource. When allocation candidates returns the response.
13:49:07 efried JulienCornuwel: Well, yes, but more generally placement.
13:49:17 mordred stephenfin, sean-k-mooney the trick is putting -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} into the deps list
13:49:19 mordred not the install_command
13:49:40 tssurya mgariepy: yes exactly
13:49:41 sean-k-mooney mordred: ya that is sneaky i like it
13:49:41 mordred like http://git.openstack.org/cgit/openstack/openstacksdk/tree/tox.ini
13:49:50 mgariepy thanks :)
13:50:06 mordred sean-k-mooney: it's the necessary trick for the self-constraint problem ;)
13:50:07 tssurya mgariepy: no problem
13:50:10 efried pooja_jadhav: It sounds like you should downvote the patch (it needs a rebase anyway, so you won't be the only thing holding it up) and explain what you're seeing.
13:50:33 efried pooja_jadhav: What would be even better is if you could come up with a functional test that demonstrates the behavior.
13:50:40 mordred tox installs the deps line separate from the current project - but it uses install_command for both things
13:50:52 sean-k-mooney mordred: it should also work on cygwin which the install script version did not

Earlier   Later