Posted	Nick	Remark
#openstack-nova - 2017-08-10
23:30:36	efried	mtreinish ack.
23:31:33	efried	mtreinish Just got restacked with mod_wsgi, though, so I may push it out a bit.
23:48:37	mtreinish	efried: no worries, I don't think it works anyway. It's still failing the gate
23:48:49	mtreinish	I'll have to play with it some more tomorrow
#openstack-nova - 2017-08-11
00:32:44	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/489029
00:33:00	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/489031
00:33:47	openstackgerrit	Takashi NATSUME proposed openstack/nova-specs master: Update a URL https://review.openstack.org/489028
01:29:30	dims	alex_xu : hope you feel better
01:29:54	dims	soon :)
01:52:25	openstackgerrit	jichenjc proposed openstack/nova master: Remove RamFilter and DiskFilter in default filter https://review.openstack.org/492765
02:29:51	openstackgerrit	Merged openstack/nova master: Remove provider allocs in confirm/revert resize https://review.openstack.org/488510
04:31:15	openstackgerrit	Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
04:31:16	openstackgerrit	Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
04:31:16	openstackgerrit	Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190
04:31:17	openstackgerrit	Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
04:31:17	openstackgerrit	Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438
04:31:18	openstackgerrit	Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
04:31:19	openstackgerrit	Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
04:31:20	openstackgerrit	Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
04:31:21	openstackgerrit	Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486
04:31:22	openstackgerrit	Michael Still proposed openstack/nova master: Refactor libvirt.utils.execute() away. https://review.openstack.org/489816
07:06:49	openstackgerrit	Merged openstack/nova master: Resource tracker compatibility with Ocata and Pike https://review.openstack.org/491012
07:50:51	openstackgerrit	jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458
07:51:10	openstackgerrit	Markus Zoeller (markus_z) proposed openstack/nova master: docs: Explain the flow of the "serial console" feature https://review.openstack.org/476188
08:24:49	openstackgerrit	Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589
09:27:21	openstackgerrit	yfzhao proposed openstack/python-novaclient master: Fix wrong links https://review.openstack.org/492921
09:34:41	gibi	good morning
09:39:41	bauzas	gibi: good lunch :p
09:41:47	gibi	bauzas: the day started earlier a bit but I was occupied with internal meetings
09:42:03	gibi	bauzas: but yes, a lunch is about to happen too
09:42:42	gibi	bauzas: did Matt cut rc1 last night?
09:43:12	bauzas	gibi: no
09:43:23	bauzas	gibi: https://review.openstack.org/#/c/492788/2
09:43:39	bauzas	he was waiting for the merge IIRC
09:43:53	bauzas	gibi: now, the main problem is about the Ironic job
09:43:57	bauzas	dtantsur: around ?
09:44:25	bauzas	dtantsur: context is https://review.openstack.org/#/c/487954/14
09:44:37	openstackgerrit	Balazs Gibizer proposed openstack/nova master: replace chance with filter scheduler in func tests https://review.openstack.org/491529
09:44:52	bauzas	dtantsur: that change is atm a RC1 blocker but I don't see the Ironic job providing the log saying that the flavor is modified
09:45:06	gibi	bauzas: thanks for the info
09:45:33	bauzas	dtantsur: if we cut RC1 without ^, that could possibly be an upgrade problem for Queens
09:45:55	bauzas	so I would appreciate any ironic folks support now :)
09:46:06	bauzas	TheJulia: around too ?
09:47:58	smcginnis	bauzas: Do you need to hold that RC request?
09:49:34	smcginnis	bauzas: Oh, nevermind. Now I see your comment there.
09:50:04	bauzas	smcginnis: yeah, I'd like to discuss with matt about whether we cut RC1 without the change I provided in my comment
09:50:21	bauzas	smcginnis: do you accept that ? is it possible to cut RC1 only on Friday?
09:50:30	bauzas	I mean, it's also a branch
09:50:41	smcginnis	bauzas: I think we're probably fine waiting until Monday if we have to.
09:50:49	smcginnis	bauzas: You're just waiting on one patch, right?
09:51:05	bauzas	smcginnis: yup, I just rebased the releases tag with the merge sha1
09:51:32	bauzas	smcginnis: so if we just cut the tag without the above change, a +W would only be needed
09:51:51	bauzas	the only blocker for us is https://review.openstack.org/#/c/487954/
09:52:18	bauzas	other bugs would be treated in other RCs
09:52:34	bauzas	HTH
09:52:36	smcginnis	bauzas: Cool, I think we're fine waiting a little then.
09:55:27	dtantsur	morning bauzas
09:55:52	dtantsur	I'll start looking in it in a few minutes
10:00:23	openstackgerrit	Balazs Gibizer proposed openstack/nova master: test server evacuation with placement https://review.openstack.org/492548
10:03:15	bauzas	dtantsur: FWIW, I'm forging a DNM patch for adding more voerbosity
10:04:29	dtantsur	cool
10:16:55	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Make compute log less verbose with allocs autocorrection https://review.openstack.org/492571
10:18:56	dtantsur	bauzas: I don't really understand the CI failure there.. it seems unrelated to me
10:19:19	bauzas	dtantsur: hence me logging more
10:19:27	bauzas	dtantsur: I'm even not sure we run the thread
10:19:43	bauzas	given it's in a conditional
10:20:55	dtantsur	we should not, it only runs for nodes with resource_class != None
10:21:25	dtantsur	bauzas: I can make https://review.openstack.org/#/c/491777/ depends-on this patch, to see how it behaves with resource_classes. wdyt?
10:22:14	bauzas	dtantsur: sure
10:22:43	bauzas	dtantsur: if that doesn't trample you waiting for devstack patch being merged
10:23:00	bauzas	dtantsur: I mean, if you can wait for that devstack patch to be merged, that's fine to me
10:23:01	dtantsur	we need to make sure it won't blow up after merging the both changes :)
10:23:06	bauzas	yeah
10:23:32	bauzas	anyway, just doing an urgent internal bug thingy and then I'm back to the DNM patch
10:23:43	dtantsur	ack
10:27:44	dtantsur	ok, both patches updated, waiting for the CI
10:41:06	openstackgerrit	Dmitry Tantsur proposed openstack/nova master: Deprecate bare metal filters https://review.openstack.org/492563
10:42:04	dtantsur	bauzas: meanwhile, do you think we can also get ^^ in?
10:53:11	bauzas	dtantsur: looks to me hard for RC1
10:53:21	bauzas	dtantsur: even if we haven't yet merged it
10:53:26	bauzas	tagged it, sorry
10:53:50	dtantsur	ok, that's fine. I just have an ironic docs patch depending on it, I may need to split it
10:53:53	bauzas	dtantsur: we're already past the deadline but I leave matt make the hard call :)
10:54:29	dtantsur	vdrok: first of all, please review https://review.openstack.org/#/c/487954/
10:54:52	vdrok	dtantsur: looking
10:56:09	dtantsur	vdrok: it fails the ironic CI for some reason. I see network connection problems between various services, so it is not necessary related to the patch itself
10:56:36	vdrok	dtantsur: yup, there is some socket error in the vbmc log as well
10:56:51	dtantsur	I've rechecked it, let's see
10:57:16	dtantsur	vdrok: our next step would be to make https://review.openstack.org/491777 and https://review.openstack.org/476968 pass the CI - reviews welcome there too
10:57:26	bauzas	dtantsur: I'm more concerned by the fact I don't see the logs mentioning the flavor update rather than the Ironic job giving us -1 :)
10:57:57	bauzas	dtantsur: in other words, I feel brave enough to +2 some ironic change if I'm sure the job issues are unrelated
10:58:17	dtantsur	bauzas: why should we see any updates, given that the nodes don't have resource classes yet?
10:58:38	bauzas	oh f****
10:59:01	bauzas	dtantsur: you killed me :p
10:59:22	dtantsur	bear metal powerzzz!
10:59:25	bauzas	dtantsur: those ironic nodes aren't having resource classes
10:59:26	bauzas	?
10:59:31	bauzas	yet, I mean ?
11:00:04	dtantsur	bauzas: yep. your logging line should show up in https://review.openstack.org/491777 instead - hence I made it depends-on the nova patch
11:01:20	bauzas	dtantsur: oh snap https://review.openstack.org/#/c/491777/9/devstack/lib/ironic@1821 right?

Earlier   Later