Earlier  
Posted Nick Remark
#openstack-nova - 2017-02-08
07:38:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix keypair API parameters https://review.openstack.org/420626
07:39:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Use ListOfIntegersField in oslo.versionedobjects https://review.openstack.org/423912
07:39:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix doc generation warnings https://review.openstack.org/425549
07:39:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882
07:39:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738
07:39:40 openstackgerrit Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436
07:39:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
07:39:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
07:40:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
07:45:12 wllabs morning
07:45:34 wllabs good morning
07:54:02 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512
07:55:08 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43 - Enable cold migration with target host https://review.openstack.org/406707
08:14:07 openstackgerrit yuhui proposed openstack/nova master: Fix some type errors https://review.openstack.org/430629
08:38:36 openstackgerrit melanie witt proposed openstack/nova master: Claim and report zero root disk for boot-from-volume instances https://review.openstack.org/428505
08:38:39 openstackgerrit yuhui proposed openstack/nova master: Correct some spelling errors https://review.openstack.org/430646
09:25:22 openstackgerrit John Garbutt proposed openstack/nova master: Stop _undefine_domain erroring if domain not found https://review.openstack.org/430400
09:25:31 openstackgerrit John Garbutt proposed openstack/nova master: Stop _undefine_domain erroring if domain not found https://review.openstack.org/430400
09:25:42 openstackgerrit John Garbutt proposed openstack/nova master: WIP: Stop leaving failed live-migrates in the migrating state https://review.openstack.org/430404
09:31:15 mdbooth http://logs.openstack.org/09/395709/7/check/gate-tempest-dsvm-full-devstack-plugin-nfs-nv/a4c1057/logs/screen-n-cpu.txt.gz?level=ERROR
09:31:23 mdbooth The qemu-img info errors are new
09:31:31 mdbooth Anybody know anything about those?
09:32:56 openstackgerrit keerthivasan selvaraj proposed openstack/python-novaclient master: Adding cpu_allocation_ratio in the python-novaclient https://review.openstack.org/430684
09:41:38 lyarwood mdbooth: morning, nope but that's during swap_volume so I'll take a look
09:42:46 lyarwood mdbooth: ah no it's not, nvm
09:44:13 openstackgerrit Alex Szarka proposed openstack/nova master: Reduce code complexity - servers.py https://review.openstack.org/359861
09:45:00 nikeshm lyarwood: hi
09:46:02 openstack Launchpad bug 1662820 in OpenStack Compute (nova) "test_volume_swap failed in Kaminario Cinder Driver CI" [Undecided,New]
09:46:02 nikeshm lyarwood: https://bugs.launchpad.net/nova/+bug/1662820
09:46:21 lyarwood nikeshm: morning, thanks I'll take a look at that shortly
10:08:50 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660
10:09:52 nikeshm lyarwood: thanks
10:10:00 nikeshm good morning
10:11:43 lyarwood nikeshm: updated with a comment, I'd have to check but I don't think Trusty is supported or used in the gate anymore
10:12:15 lyarwood nikeshm: this smells like a libvirt issue that was fixed in the Xenial version of libvirt btw
10:13:23 openstackgerrit Gábor Antal proposed openstack/nova master: Update hacking version in test-requirements.txt https://review.openstack.org/430702
10:14:29 mdbooth lyarwood: Found the root cause of those qemu-img errors I mentioned earlier, btw
10:14:38 mdbooth Simple race between delete and a periodic task
10:15:07 mdbooth periodic task should probably take its own lock and check the instance still exists, tbh, but it's not really an issue
10:26:47 egonzalez Hi, I have a couple of questions regarding to cells_v2 and upgrades
10:27:06 egonzalez Is there any issue if simple_cell_setup is executed with ocata code, but services still running in newton?
10:28:58 mdbooth lyarwood: Going through this log: http://logs.openstack.org/09/395709/7/check/gate-tempest-dsvm-full-devstack-plugin-nfs-nv/a4c1057/logs/screen-n-cpu.txt.gz?level=ERROR
10:31:20 kashyap mdbooth: From your log, it seems the error is a file not found?
10:31:21 kashyap 2017-02-07 19:10:25.548 1437 ERROR nova.compute.manager Stderr: u"qemu-img: Could not open '/opt/stack/data/nova/instances/60b7cb32-0376-47d6-a431-c1bf85516e2e/disk': Could not open '/opt/stack/data/nova/instances/60b7cb32-0376-47d6-a431-c1bf85516e2e/disk': No such file or directory\n"
10:31:33 openstackgerrit Wang Qing wu proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif https://review.openstack.org/422512
10:31:42 mdbooth kashyap: nm, already found the root cause of both of those
10:38:35 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Deprecate two more nova-net options https://review.openstack.org/412390
10:38:36 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Fix indentation in conf/netconf https://review.openstack.org/430717
10:38:36 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Deprecate 'ipv6_backend' https://review.openstack.org/430716
10:38:37 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: conf: Add 'vif' group https://review.openstack.org/430718
10:40:56 nikeshm lyarwood: ok
10:41:08 nikeshm i am trying to switch to xenial
11:15:26 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43: List/Show all server migration types https://review.openstack.org/430839
11:29:12 sfinucan macsz: Something to look out for in the conf option patches - deprecated_since should now be '16.0.0'
11:29:20 sfinucan johnthetubaguy: likewise ^
11:29:38 johnthetubaguy sfinucan: yeah +1
11:33:39 sfinucan johnthetubaguy: I would be interested in your thoughts on https://review.openstack.org/430718
11:34:53 johnthetubaguy sfinucan: the file inject of network templates will not go away
11:35:10 johnthetubaguy sfinucan: putting that in config drive is the only way to configure networking without DHCP
11:35:33 johnthetubaguy sfinucan: its the more general file injection that would die, AFAIK
11:35:46 sfinucan johnthetubaguy: Aha, yeah I wasn't sure if there was a difference
11:36:15 johnthetubaguy its the open the file system and write files in that really has to die
11:36:30 johnthetubaguy the write files passed in API into config drive is also not brilliant
11:36:47 johnthetubaguy the create files from info we know and put it into config drive, I think has to stay
11:36:50 sfinucan aye, mriedem spoke of doing just that this cycle
11:36:51 johnthetubaguy but I might have that all wrong
11:38:49 sfinucan johnthetubaguy: Out of curiosity, are you aware of any reason /not/ to provide IPv6 information in that injected template if we IPv6 networks available
11:39:00 sfinucan i.e. do we really need to use 'CONF.use_ipv6' here https://github.com/openstack/nova/blob/d48aeb5783ea9bd9495eb2e7c67bc587dfd7d85d/nova/virt/netutils.py#L131-L132
11:39:32 johnthetubaguy good point, that does seem a bit nuts, we should just inject what neutron sends us
11:40:06 sfinucan I'm thinking it's a remnant from nova-network days used as a "quick escape"
11:40:16 sfinucan But I don't see any reason to keep it around now
11:40:29 sfinucan Likewise for libvirt port filters https://github.com/openstack/nova/blob/d48aeb5783ea9bd9495eb2e7c67bc587dfd7d85d/nova/virt/libvirt/firewall.py#L158 and #L172
11:40:45 lyarwood mdbooth: wallofde^C^Ctext added to https://review.openstack.org/#/c/383859/16
11:45:43 johnthetubaguy sfinucan: thats probably right, it sounds broken to me
11:47:19 sfinucan Time to go kill some code so :)
11:47:23 sfinucan Yay
11:48:43 johnthetubaguy sfinucan: :)
11:48:53 openstackgerrit John Garbutt proposed openstack/nova master: Stop failed live-migrates getting stuck migrating https://review.openstack.org/430404
11:55:34 openstack Launchpad bug 1662626 in OpenStack Compute (nova) "live-migrate left in migrating as domain not found" [Medium,In progress] - Assigned to John Garbutt (johngarbutt)
11:55:34 johnthetubaguy sfinucan: if you get chance, this is another nasty live-migrate bug that was blocking the live-migration performance testing OSIC are doing: https://bugs.launchpad.net/nova/+bug/1662626
11:56:08 johnthetubaguy sfinucan: along with this one: https://review.openstack.org/#/c/429798/5
11:56:12 sfinucan johnthetubaguy: so https://review.openstack.org/#/c/430404/ ?
11:56:27 johnthetubaguy sfinucan: both of them
11:56:34 sfinucan gotcha
11:56:34 johnthetubaguy well, all three I guess
11:56:57 johnthetubaguy one fixes the root cause of the error, the other fixes our handing of the error, if that makes sense
11:57:12 sfinucan It does
11:57:19 sfinucan I'll take a look shortly, once I'm done with lyarwood / mdbooth's two fixes
12:01:00 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660
12:04:31 lyarwood sfinucan: we should start a club
12:05:24 lyarwood or a therapy group
12:20:55 mdbooth lyarwood: Awesome, thanks for doing that.
12:21:05 mdbooth lyarwood: Just making the log level change now.
12:21:27 openstackgerrit MarkMielke proposed openstack/nova master: DiskFilter should only apply when storage is requested https://review.openstack.org/430091
12:22:36 lyarwood mdbooth: np
12:22:38 lyarwood gerrit--
12:23:36 mdbooth lyarwood: Incidentally, the patch I posted for the detach race doesn't work, because the race isn't local to the compute host
12:23:49 mdbooth Which is annoying
12:28:09 lyarwood mdbooth: wonderful, so I guess we just remove the detach from the compute layer, see which virt drivers we break and add in an additional detach there?
12:28:56 mdbooth lyarwood: The issue is that the bdm list is fetch on the api node and passed to the compute node, but is out of date by the time it's used.

Earlier   Later