| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-18 | |||
| 06:48:42 | openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833 | |
| 06:51:16 | alex_xu | gmann: thanks :) | |
| 06:54:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix the incorrect description and sample https://review.openstack.org/528180 | |
| 07:56:59 | slaweq | sean-k-mooney: hi | |
| 07:58:01 | slaweq | sean-k-mooney: recently in some neutron jobs I saw failing test tempest.api.compute.admin.test_servers_on_multinodes.ServersOnMultiNodesTest.test_create_server_with_scheduler_hint_group_anti_affinity - it doesn't looks like related to neutron for me | |
| 07:58:35 | slaweq | sean-k-mooney: maybe You are already aware of such issue? Can You take a look on it maybe? | |
| 07:58:56 | slaweq | example of failure is e.g.: http://logs.openstack.org/08/555608/34/check/neutron-tempest-multinode-full/1d3aafb/logs/testr_results.html.gz | |
| 08:03:39 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Fix confusing log message in scheduler https://review.openstack.org/583486 | |
| 08:09:42 | openstackgerrit | jichenjc proposed openstack/nova master: Remove Ocata comments which expires now https://review.openstack.org/583489 | |
| 08:16:09 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Fix confusing log message in scheduler https://review.openstack.org/583486 | |
| 08:22:55 | openstackgerrit | jichenjc proposed openstack/nova master: clarify usage of upgrade_levels group https://review.openstack.org/583502 | |
| 08:24:42 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398 | |
| 08:26:50 | MultipleCrashes | Hello Guys, Looking for review of this to workflow - https://review.openstack.org/#/c/563418/ , Thanks! | |
| 08:52:11 | openstackgerrit | Merged openstack/nova master: Handle HostMappingNotFound when deleting a compute service https://review.openstack.org/582674 | |
| 09:01:22 | tssurya | alex_xu: good morning! are you around ? | |
| 09:02:18 | tssurya | if you have some minutes, could you please have a look at https://review.openstack.org/#/c/581243/ ? its a trivial nit fix for the specs | |
| 09:25:24 | openstackgerrit | Merged openstack/nova master: Transform metrics.update notification https://review.openstack.org/580567 | |
| 09:33:31 | openstackgerrit | Stephen Finucane proposed openstack/os-traits master: Switch to stestr https://review.openstack.org/581265 | |
| 09:33:32 | openstackgerrit | Stephen Finucane proposed openstack/os-traits master: tox: Remove '-constraints' targets https://review.openstack.org/583533 | |
| 09:33:33 | openstackgerrit | Stephen Finucane proposed openstack/os-traits master: Follow the new PTI for document build https://review.openstack.org/583534 | |
| 09:33:59 | stephenfin | jaypipes: When you're about https://review.openstack.org/583533 | |
| 10:06:41 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for config_drive extension https://review.openstack.org/583538 | |
| 10:22:53 | sean-k-mooney | stephenfin: you know that we dont use constraints in os-vif either. want me to fix that. | |
| 10:23:03 | stephenfin | sean-k-mooney: Yeah, go for it | |
| 11:08:35 | slaweq | sean-k-mooney: hi, did You saw my question which I asked earlier? | |
| 11:08:49 | sean-k-mooney | slaweq: no sorry | |
| 11:09:05 | slaweq | sean-k-mooney: no problem, I can write it once again now :) | |
| 11:09:21 | slaweq | so, recently in some neutron jobs I saw failing test tempest.api.compute.admin.test_servers_on_multinodes.ServersOnMultiNodesTest.test_create_server_with_scheduler_hint_group_anti_affinity - it doesn't looks like related to neutron for me | |
| 11:09:26 | slaweq | maybe You are already aware of such issue? Can You take a look on it maybe? | |
| 11:09:30 | slaweq | example of failure is e.g.: | |
| 11:09:34 | slaweq | http://logs.openstack.org/08/555608/34/check/neutron-tempest-multinode-full/1d3aafb/logs/testr_results.html.gz | |
| 11:11:42 | sean-k-mooney | yes so i was talking to matt riedemann about that job last night | |
| 11:11:49 | sean-k-mooney | or rather that test | |
| 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? | |