| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-23 | |||
| 16:26:37 | tojuvone | johnthetubaguy, but in services need to duplicate on each service on host | |
| 16:26:39 | mriedem | sdague: i'm seeing rabbit have memory limit warnings around the time we lose the logs | |
| 16:27:19 | sdague | yeh, there is no individual test timeout, the test is still running and hung | |
| 16:27:19 | mriedem | edmondsw: see my reply to madhaviy above | |
| 16:27:36 | edmondsw | mriedem, ah, didn't realize we'd already had this discussion... | |
| 16:27:37 | mriedem | sdague: doesn't testr have a 500s timeout per test? | |
| 16:27:45 | johnthetubaguy | tojuvone: given what I described, I think it would get its own table, but not sure if everyone likes that direction, I think we should focus on getting the use cases crisp first | |
| 16:28:31 | mriedem | edmondsw: madhaviy: johnthetubaguy: https://openstack.nimeyo.com/87011/openstack-policy-check-network-attach_external_network | |
| 16:28:31 | openstackgerrit | Sean Dague proposed openstack/python-novaclient master: set test timout to 300 seconds https://review.openstack.org/449211 | |
| 16:28:39 | sdague | mriedem: no, testr enforces nothing | |
| 16:28:41 | tojuvone | johnthetubaguy, yes, trying to update a lot including the cases and all. | |
| 16:29:26 | johnthetubaguy | tojuvone: yeah, I would just leave the implementation bit alone for now, lets agree these use cases first, particularly the scheduler related ones | |
| 16:29:34 | sdague | mriedem: all our test timeouts are done via test fixtures | |
| 16:29:39 | mriedem | sdague: ah | |
| 16:29:45 | mriedem | i see we set that for nova but not novaclient | |
| 16:29:48 | sdague | yep | |
| 16:30:01 | johnthetubaguy | edmondsw: there were other ones I think sneti and aunnam found that are accessed outside the API | |
| 16:30:01 | mriedem | self.useFixture(fixtures.Timeout(test_timeout, gentle=True)) | |
| 16:30:01 | mriedem | yar | |
| 16:30:09 | openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | |
| 16:30:09 | openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | |
| 16:30:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Set instance.availability_zone whenever we schedule https://review.openstack.org/446053 | |
| 16:30:11 | openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | |
| 16:30:12 | tojuvone | johnthetubaguy, ok, sounds good to make better progress | |
| 16:30:23 | openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | |
| 16:30:25 | openstackgerrit | Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292 | |
| 16:30:26 | openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | |
| 16:30:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293 | |
| 16:30:28 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861 | |
| 16:30:29 | openstackgerrit | Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | |
| 16:30:31 | openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | |
| 16:30:44 | sdague | mriedem: that should at least give us different results, and not have the big timeout kill us | |
| 16:30:56 | mriedem | sdague: yeah, thanks | |
| 16:31:02 | johnthetubaguy | edmondsw: FWIW, I think the neutron refactor work is heading in a direction that would fix that policy issue, anyways | |
| 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 | |