| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-08 | |||
| 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 | nikeshm | lyarwood: https://bugs.launchpad.net/nova/+bug/1662820 | |
| 09:46:02 | openstack | Launchpad bug 1662820 in OpenStack Compute (nova) "test_volume_swap failed in Kaminario Cinder Driver CI" [Undecided,New] | |
| 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: conf: Deprecate 'ipv6_backend' https://review.openstack.org/430716 | |
| 10:38:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Fix indentation in conf/netconf https://review.openstack.org/430717 | |
| 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 | 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: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: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 | johnthetubaguy | well, all three I guess | |
| 11:56:34 | sfinucan | gotcha | |
| 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. | |
| 12:29:21 | mdbooth | So we can take a lock on the compute node, but we're still operating on a list which was generated outside of that lock. | |
| 12:29:53 | mdbooth | It would also race with anything else which modified bdms other than the compute node | |
| 12:31:39 | lyarwood | mdbooth: isn't the compute the only thing modifying bdms for running instances? | |
| 12:32:04 | mdbooth | lyarwood: I doubt it, tbh. | |
| 12:32:32 | mdbooth | I wouldn't be surprised to discover that api and conductor also touch them at various times. | |