| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-29 | |||
| 17:04:40 | superdan | EmilienM: because you don't have cells records in place or something? | |
| 17:04:52 | EmilienM | superdan: we should have | |
| 17:05:06 | EmilienM | dansmith: I filed a bug https://bugs.launchpad.net/tripleo/+bug/1660160 | |
| 17:05:06 | openstack | Launchpad bug 1660160 in tripleo "No host-to-cell mapping found for selected host" [Critical,Triaged] | |
| 17:05:28 | dansmith | EmilienM: that message means you don't | |
| 17:06:47 | EmilienM | we have nova-cell_v2-cell-default | |
| 17:06:57 | EmilienM | http://logs.openstack.org/31/426231/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/915aeba/logs/undercloud/var/log/undercloud_install.txt.gz#_2017-01-27_18_24_40_000 | |
| 17:07:02 | EmilienM | let me check what it executes | |
| 17:07:31 | EmilienM | http://logs.openstack.org/31/426231/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/915aeba/logs/undercloud/var/log/undercloud_install.txt.gz#_2017-01-27_18_24_40_000 | |
| 17:07:37 | EmilienM | damn, wrong link | |
| 17:07:40 | EmilienM | https://github.com/openstack/puppet-nova/blob/master/manifests/cell_v2/cell.pp#L45 | |
| 17:07:44 | EmilienM | this is what we executed | |
| 17:08:00 | EmilienM | nova-manage ${extra_params} cell_v2 create_cell --name=default ${transport_url_real} ${database_connection_real} | |
| 17:08:35 | dansmith | EmilienM: the error message you linked in the bug about no compute node record is resource tracker, nothing to do with cellsv2 | |
| 17:10:17 | EmilienM | dansmith: you know better than me, to be honest I don't know. I just know that something broke again for us with a very recent commit | |
| 17:10:22 | EmilienM | and you can see it here: http://logs.openstack.org/31/426231/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/915aeba/logs/undercloud/var/log/nova/nova-compute.txt.gz#_2017-01-27_19_01_00_621 | |
| 17:12:00 | dansmith | EmilienM: that message comes from resource tracker on compute | |
| 17:12:44 | EmilienM | dansmith: I tried to revert https://review.openstack.org/#/c/319379/ and run experimental pipeline, and tripleo worked again. | |
| 17:13:00 | EmilienM | so we'll just pin nova in RDO for now. | |
| 17:13:50 | dansmith | EmilienM: it passed multinode devstack and grenade, so it must be something in the tripleo job config that isn't set right.. why do you think just reverting the patch is the way to go? | |
| 17:14:30 | EmilienM | dansmith: it is not the way to go | |
| 17:15:17 | EmilienM | and we'll unpin when someone will help to figure what's wrong, we have zero clue at this time. We are not experts enough in Nova | |
| 17:20:05 | EmilienM | enjoy! | |
| 18:25:34 | mriedem | does anyone else enjoy doing taxes as much as i do?! | |
| 18:35:41 | tbachman | mriedem: clearly not, given how early you’ve started them ;) | |
| 18:54:10 | amrith | andreykurilin, ping. please take a look at http://logs.openstack.org/35/426535/3/check/gate-trove-scenario-dsvm-mysql-multi-ubuntu-xenial/16eaf0c/console.html#_2017-01-29_17_52_32_209313. something appears to be getting mixed up between novaclient and keystone and I'm not sure what that could be. this particular failure seems to be out of trove's direct control (I think). but something trove is doing may be the thing | |
| 18:54:10 | amrith | that is exacerbating this. | |
| 20:26:26 | andreykurilin | amrith: will contact you at trove channel | |
| #openstack-nova - 2017-01-30 | |||
| 02:34:11 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Enable cold migration with target host https://review.openstack.org/426636 | |
| 02:38:57 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 02:39:25 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 02:40:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 02:40:22 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 02:41:57 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Enable cold migration with target host https://review.openstack.org/426636 | |
| 02:44:02 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 02:44:08 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 02:44:49 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host https://review.openstack.org/406707 | |
| 02:45:58 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926 | |
| 02:46:13 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926 | |
| 03:54:27 | openstackgerrit | David Medberry proposed openstack/nova: Explain how allow_resize_to_same_host is useful https://review.openstack.org/426531 | |
| 04:03:45 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | |
| 05:07:24 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix doc generation warnings https://review.openstack.org/425549 | |
| 05:07:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Use ListOfIntegersField in oslo.versionedobjects https://review.openstack.org/423912 | |
| 05:07:56 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738 | |
| 05:08:03 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | |
| 05:08:12 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters https://review.openstack.org/420626 | |
| 05:08:19 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436 | |
| 05:08:35 | openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters and examples in aggregate API https://review.openstack.org/425512 | |
| 05:08:43 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876 | |
| 06:19:21 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: xAbort Cold Migration https://review.openstack.org/334732 | |
| 06:19:52 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Abort Cold Migration https://review.openstack.org/334732 | |
| 07:44:51 | openstackgerrit | Andrey Volkov proposed openstack/nova: Update pci device fields from deleted object https://review.openstack.org/426243 | |
| 08:25:27 | zer0c00l | I don't know if i have to get my eyes checked, but it looks like _heal_instance_info_cache just going to refresh cache only once for the instance on top of the list | |
| 08:25:37 | zer0c00l | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5881 | |
| 08:26:06 | zer0c00l | We are getting the first instance that is 'okay' to healed and breaking off that loop | |
| 08:26:23 | zer0c00l | am i missing something that is obvious ? | |
| 08:33:11 | openstackgerrit | Oleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration https://review.openstack.org/246910 | |
| 09:45:58 | openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Change condition of root disk snapshot handling https://review.openstack.org/382030 | |
| 09:45:59 | openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Remove backward compatibility checks for disk snapshots https://review.openstack.org/382031 | |
| 09:45:59 | openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Call snapshot operations for not file based backends only https://review.openstack.org/414710 | |
| 09:46:00 | openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Correct ploop imagebackend attribute https://review.openstack.org/414709 | |
| 09:59:01 | openstackgerrit | Rikimaru Honjo proposed openstack/nova: Add checks for preventing unexpected detaching volume https://review.openstack.org/354617 | |
| 10:07:26 | sfinucan | lyarwood: Morning. Interesting question on https://review.openstack.org/#/c/425087 | |
| 10:07:49 | sfinucan | It appears the test I've added has highlighted a latent Python 3 issue, which has since been fixed on master | |
| 10:08:40 | sfinucan | Do I (a) modify the change to fix said Python issue, (b) backport the Python 3 issue fix first, or (c) remove the test from the patch? | |
| 10:10:17 | lyarwood | sfinucan: I'd go with a and document it as a conflict in the change | |
| 10:10:41 | sfinucan | lyarwood: and that is what I'll do. Cheers :) | |
| 10:10:41 | lyarwood | sfinucan: assuming the fix on master also fixes other py3 issues | |
| 10:10:54 | sfinucan | Yeah, a lot of them | |
| 10:11:17 | lyarwood | sfinucan: yeah cool, then just fix this test up and document, that should be enough IMHO | |
| 10:23:53 | sfinucan | lyarwood: Think that's done now | |
| 10:24:23 | lyarwood | sfinucan: thanks, I'll take a look once CI comes backs. | |
| 10:24:37 | lyarwood | sfinucan: would you mind taking a look at https://review.openstack.org/#/c/399735/ this morning if you have time | |
| 10:24:51 | sfinucan | sure | |
| 10:42:27 | mdbooth | Morning! Now we're out of feature crunch, looking for another +2 on this bugfix which has been sitting a while: https://review.openstack.org/#/c/383859/13 | |
| 10:50:06 | cdent | johnthetubaguy, bauzas : I'm a little lost on the state of things. Any salient updates to share? | |
| 10:50:36 | bauzas | cdent: waiting for +2 on the grenade fix | |
| 10:50:45 | johnthetubaguy | cdent: I think bauzas's change is waiting on grenade | |
| 10:50:46 | johnthetubaguy | yeah, that | |
| 10:51:05 | johnthetubaguy | the ironic one isn't read yet: https://review.openstack.org/#/c/404472 | |
| 10:51:13 | johnthetubaguy | I was thinking about tidying that up right now actually | |
| 10:51:19 | johnthetubaguy | but I suspect its a bit late to get that in now | |
| 10:51:55 | cdent | johnthetubaguy: cool, that was the one I was wondering about most. | |
| 10:52:18 | bauzas | johnthetubaguy: cdent: well, that one seems at risk | |
| 10:52:43 | cdent | I guess jay will be back today and will have some input. It all remains a bit fuzzy about what the actual intent was. | |
| 10:53:11 | bauzas | cdent: question, how can we know it works with the ironic driver ? | |
| 10:53:18 | bauzas | I can only see one ironic job | |
| 10:53:28 | bauzas | which tells me it's green http://logs.openstack.org/72/404472/25/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/0b27bd3/ | |
| 10:53:42 | bauzas | but I'm not sure we're covering the patch with that job | |
| 10:54:04 | cdent | bauzas: That's pretty much the source of my question. Without knowing the overall arc of intent, it is hard to discuss coverage. | |
| 10:54:11 | cdent | Too many people saying "I'm not sure". | |
| 10:54:44 | bauzas | cdent: I'm just trying to explain that I discovered most of the corner cases when I investigated the grenade issue | |
| 10:54:50 | bauzas | wrt the scheduler change | |
| 10:55:27 | johnthetubaguy | bauzas: the follow up patch was trying to get coverage | |
| 10:55:27 | bauzas | I'd be far less concerned if I was somehow seeing that some job is running an ironic cloud that does reporting to nova | |
| 10:55:42 | bauzas | I know the tripleo folks kinda use ironic for their undercloud | |
| 10:55:53 | bauzas | so we could perhaps see if their jobs are happy | |
| 10:56:26 | cdent | Personally, in a perfect universe, I don't think of grenade or tempest as coverage at all. They are smoke. If there aren't functional tests in tree, we're faking it and causing us the problems we encountered at the end of last week. So the _next_ time we come round to this, we should start from tests that do something (not unit tests). | |
| 10:56:55 | cdent | But, given that we are not in a perfect universe, and now is not the next time, it's good you did that digging, and it's what we've got... | |
| 10:57:45 | cdent | I do know that jroll created a dnm devstack change to exercise the ironic-related change. I'm not sure of the current state of that. | |