Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-23
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 openstack Launchpad bug 1675486 in OpenStack Compute (nova) "network:attach_external_network policy check outside nova-api" [Undecided,New]
16:47:56 edmondsw johnthetubaguy mriedem sneti: https://bugs.launchpad.net/nova/+bug/1675486
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: Libvirt support for tagged nic attachment https://review.openstack.org/415520
16:52:29 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
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: Modify PciPassthroughFilter to accept lists https://review.openstack.org/444946
18:01:35 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read NIC features in libvirt https://review.openstack.org/444415
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 openstack bug 1675526 in python-novaclient "Trigger crash dump functional tests are timing out and hanging" [High,Triaged] https://launchpad.net/bugs/1675526
18:45:45 mriedem bug 1675526
18:45:54 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729
18:45:59 sdague ok, feel free to update if you like, got to run away from keyboard for a bit
18:46:04 mriedem ok
18:55:52 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Set test timout to 300 seconds https://review.openstack.org/449211
19:09:37 openstackgerrit Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavor API https://review.openstack.org/449288
19:12:01 openstackgerrit Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavor API https://review.openstack.org/449288

Earlier   Later