| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-23 | |||
| 16:31:14 | sneti | johnthetubaguy, yes..this is the one: https://review.openstack.org/#/c/448260/2/nova/policies/cells_scheduler.py | |
| 16:31:40 | sdague | mriedem: is the crash dump testing a thing we do other places? | |
| 16:31:54 | mriedem | sdague: not sure about tempest | |
| 16:31:54 | tojuvone | johnthetubaguy, I should have that part by tomorrow and can continue from that | |
| 16:32:06 | johnthetubaguy | tojuvone: sounds great, ping me when thats up | |
| 16:32:11 | mriedem | sdague: oomichi probably knows, i think he added it | |
| 16:32:18 | mriedem | or was involved | |
| 16:32:28 | tojuvone | johnthetubaguy, thanks, I will | |
| 16:33:03 | johnthetubaguy | edmondsw: so sounds like some scheduler filters need policy too | |
| 16:33:38 | johnthetubaguy | edmondsw: that should probably get fixed too | |
| 16:34:16 | edmondsw | johnthetubaguy I'd say let's open defects for these, so they're not lost... and if they get fixed apart from the defects we can always cancel them. Your thoughts? | |
| 16:34:43 | johnthetubaguy | edmondsw: yeah, I have asked sneti and aunnam to do that as they find these | |
| 16:34:51 | edmondsw | great, tx | |
| 16:35:03 | johnthetubaguy | or at least I mean to... | |
| 16:35:13 | edmondsw | sneti, have you opened one for the external network attach case, or will you, or should I? | |
| 16:35:48 | edmondsw | sneti, if you haven't already, I'd say you open the scheduler one you found and I can open the network one | |
| 16:36:05 | oomichi | mriedem: sdague: temepst doesn't have crash-dump test, I guess cirros cannot take crash-dump because it requires some crash-dump tool | |
| 16:36:11 | johnthetubaguy | edmondsw: +1 | |
| 16:36:19 | oomichi | even if nova triggers it | |
| 16:36:50 | sdague | oomichi: ok, so why are we running those tests? | |
| 16:36:51 | mriedem | oomichi: but we have crash dump tests in novaclient functional tests | |
| 16:37:38 | oomichi | mriedem: oh, I didn't that novaclient has | |
| 16:39:15 | oomichi | sdague: that depends on what kind of test do we want. Cirros is enough if we just want to call the API I feel. but if we want to verify the crashdump file is created in the virtual machine, the tool is required | |
| 16:39:19 | mriedem | oomichi: yeah and they take about 4 minutes to run | |
| 16:39:35 | mriedem | http://logs.openstack.org/57/447657/1/gate/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/b3aaf58/console.html#_2017-03-23_12_21_00_379603 | |
| 16:40:45 | openstackgerrit | Pavel Gluschak (scsnow) proposed openstack/nova master: WIP: honor disk_format=ploop image metadata property https://review.openstack.org/429745 | |
| 16:40:49 | oomichi | mriedem: yeah, because crash dump takes 1) switch to 2nd-kernel, 2) copy system memory of 1st kernel to a file 3) reboot the system | |
| 16:41:07 | oomichi | need much time | |
| 16:47:56 | edmondsw | johnthetubaguy mriedem sneti: https://bugs.launchpad.net/nova/+bug/1675486 | |
| 16:47:56 | openstack | Launchpad bug 1675486 in OpenStack Compute (nova) "network:attach_external_network policy check outside nova-api" [Undecided,New] | |
| 16:48:26 | johnthetubaguy | sweet, good to capture that, not 100% sure how we want to fix that | |
| 16:52:21 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 16:52:22 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 16:52:23 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | |
| 16:52:24 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | |
| 16:52:27 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 16:52:29 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 16:52:29 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 16:53:51 | sneti | edmondsw, sure..I will open a bug for the scheduler one | |
| 16:54:06 | edmondsw | sneti tx | |
| 16:58:53 | harlowja | efried very much agreed | |
| 16:59:10 | harlowja | some included fixtures i think would help | |
| 17:15:19 | sfinucan | johnthetubaguy: Might be interested in https://review.openstack.org/#/c/446898/ | |
| 17:15:31 | sfinucan | if gcb hadn't already mentioned it to you | |
| 17:26:32 | edleafe | mdbooth: kashyap: still around? | |
| 17:27:05 | kashyap | edleafe: Yeah, but just about to bike out to the near by university for a prior appointment | |
| 17:27:13 | kashyap | edleafe: I think you have the logs :-) | |
| 17:27:55 | kashyap | edleafe: If it's a quick question, then I have about 5 mins before I head out the door. Otherwise, do write here, I'll read the scroll when I get back & respond. | |
| 17:27:57 | edleafe | kashyap: wow, you're psychic! :) | |
| 17:28:17 | kashyap | Heh | |
| 17:28:20 | edleafe | kashyap: wondering if you wanted me to send them to you to review | |
| 17:28:54 | edleafe | or should I paste the relevant sections | |
| 17:28:58 | kashyap | edleafe: Sure, do email them; but I won't promise that I'll see them tonight. Tomorrow I can. | |
| 17:29:07 | kashyap | edleafe: No, entire log would be useful. Why? | |
| 17:29:25 | kashyap | Because I can correlate the unique IDs libvirt sends out for each action it tries to coordinate with QEMU | |
| 17:29:34 | edleafe | kashyap: dunno, just being careful | |
| 17:29:40 | kashyap | If you accidentally snip out some request / response, I might lose some context | |
| 17:30:45 | edleafe | kashyap: makes sense | |
| 18:01:35 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read NIC features in libvirt https://review.openstack.org/444415 | |
| 18:01:35 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Modify PciPassthroughFilter to accept lists https://review.openstack.org/444946 | |
| 18:01:36 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | |
| 18:01:48 | mriedem | sdague: yeah those novaclient tests are timing out test_trigger_crash_dump_in_resized_state | |
| 18:03:16 | johnthetubaguy | mriedem: I hit some nasty rebase things on that refactor patch, will have to resolve those in the morning | |
| 18:03:49 | mriedem | johnthetubaguy: ok i'll do them today if i get a chance | |
| 18:08:10 | sdague | mriedem: yeh, I honestly think the crashdump tests probably have to be turned off | |
| 18:08:19 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in cells_scheduler.py https://review.openstack.org/448260 | |
| 18:08:38 | openstackgerrit | John Garbutt proposed openstack/nova master: compute: Move detach logic from manager into driver BDM https://review.openstack.org/439520 | |
| 18:09:39 | mriedem | johnthetubaguy: i guess it wasn't that bad huh :) | |
| 18:09:56 | johnthetubaguy | mriedem: so fixed my nit, but that needs a rebase after those patches we just approved | |
| 18:10:02 | mriedem | ok | |
| 18:12:22 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | |
| 18:12:38 | mriedem | sdague: alex_xu suggested that if we make that single PUT method for updating a service, we also change the response to include the full resource, like a GET call would https://review.openstack.org/#/c/447149/2/specs/pike/approved/service-hyper-uuid-in-api.rst@208 | |
| 18:12:45 | mriedem | sdague: i'd be OK with that, what do you think? | |
| 18:15:44 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | |
| 18:16:44 | openstackgerrit | Vasyl Saienko proposed openstack/nova master: Implement interface attach/detach in ironic virt driver https://review.openstack.org/419975 | |
| 18:17:24 | efried | mriedem ^^ It's been tough getting flake8-import-order, rebasing, and subteam +1s all coordinated, but this one is hopefully ready to go again. | |
| 18:17:42 | efried | [Mine, not Vasyl's ;-)] | |
| 18:18:34 | sdague | mriedem: I am ok with full resource | |
| 18:18:52 | sdague | I also think it's not a terrible idea to fix that os-hypervisor query interface that's crazy at the same time | |
| 18:19:05 | sdague | given that everyone is going to need to adapt code forthis | |
| 18:24:19 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | |
| 18:25:22 | openstackgerrit | Monty Taylor proposed openstack/nova master: WIP Clean up and consolidate client creation https://review.openstack.org/438512 | |
| 18:27:59 | mriedem | sdague: so i should just rename to "complete unf*ch os-services and os-hypervisors" :) | |
| 18:28:11 | sdague | mriedem: ++ | |
| 18:28:25 | mriedem | wow spelling | |
| 18:38:37 | openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/357151 | |
| 18:40:04 | mriedem | sdague: ah it looks like we timeout and never kill a timer http://logs.openstack.org/11/449211/1/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/9839aac/console.html#_2017-03-23_18_04_43_739252 | |
| 18:40:07 | mriedem | waiting for a server to build | |
| 18:40:41 | mriedem | i'll take a look | |
| 18:41:37 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 18:42:47 | sdague | it's kind of suspicious that after a bunch of crash dumping rabbit goes sideways | |
| 18:42:58 | sdague | I'd start with disabling all those crashdump tests first | |
| 18:43:34 | mriedem | i seem to remember andreykurilin fixing something where the crash dump tests started taking a long time | |
| 18:43:44 | mriedem | andreykurilin: if you're around, do you remember something like that? | |
| 18:45:13 | openstackgerrit | Sean Dague proposed openstack/python-novaclient master: set test timout to 300 seconds https://review.openstack.org/449211 | |
| 18:45:31 | sdague | that's got the skip in there | |
| 18:45:35 | mriedem | we can use bug test_trigger_crash_dump_in_shutoff_state | |
| 18:45:39 | mriedem | oops | |
| 18:45:45 | mriedem | bug 1675526 | |
| 18:45:45 | openstack | bug 1675526 in python-novaclient "Trigger crash dump functional tests are timing out and hanging" [High,Triaged] https://launchpad.net/bugs/1675526 | |