Posted	Nick	Remark
#openstack-nova - 2017-01-29
15:11:21	openstackgerrit	Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285
15:11:23	openstackgerrit	Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358
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).

Earlier   Later