Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-23
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
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

Earlier   Later