Posted	Nick	Remark
#openstack-nova - 2018-12-13
22:37:03	mriedem	melwitt: i would like to let that bake a bit
22:37:12	melwitt	k
22:37:15	mriedem	dan and i were talking about something in there as an unexpected side effect this morning
22:37:17	mriedem	hence the depends-on
22:37:47	mriedem	so just the docs change and lee's backport
22:37:54	melwitt	ok
22:38:42	mriedem	if you're curious, the side effect is if the cell is configured to hit the api db, which lots of people still do for up-calls to work, and upgrade levels is 'auto', it means all cells will use the minimum compute version even if a particular cell is fully upgraded
22:38:48	mriedem	which is different from before
22:38:51	mriedem	not bad, but different
22:40:13	melwitt	I was, thanks for saving me from having to read back in the irc log
22:43:10	mriedem	Kevin_Zheng: i've got some questions/confusion on yikun's live migration completion timeout action patch if you want to kick him when you're in the office https://review.openstack.org/#/c/619143/
22:57:53	openstackgerrit	Merged openstack/os-resource-classes master: Tune up documentation to be more useful https://review.openstack.org/624384
23:08:34	mriedem	dansmith: you might have an opinion on the os-vif version side-step in this spec https://review.openstack.org/#/c/607610/ but i'm not sure it's really worth holding up on that
23:10:38	dansmith	mriedem: yeah I think I saw talk of that recently
23:11:26	dansmith	I'm ambivalent, although amused/bemused that it's been however many years and we still aren't sharing objects with neutron
23:11:56	Kevin_Zheng	mriedem: ack
23:15:03	mriedem	we didn't do the port binding thing, talked about since ocata, until rocky, so yeah
23:19:54	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Make compute rpcapi version calculation check all cells https://review.openstack.org/625060
#openstack-nova - 2018-12-14
01:06:01	openstackgerrit	Merged openstack/nova master: Only construct SchedulerReportClient on first access from API https://review.openstack.org/623246
01:06:07	openstackgerrit	Merged openstack/nova master: DRY up SchedulerReportClient init https://review.openstack.org/623247
01:06:14	openstackgerrit	Merged openstack/nova stable/rocky: Note the aggregate allocation ratio restriction in scheduler docs https://review.openstack.org/623546
02:54:32	openstackgerrit	Dave McCowan proposed openstack/nova stable/rocky: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623507
02:55:09	openstackgerrit	Dave McCowan proposed openstack/nova stable/queens: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623509
02:57:07	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Don't persist RequestSpec.requested_destination https://review.openstack.org/611945
05:01:37	openstackgerrit	Merged openstack/nova stable/queens: Refix disk size during live migration with disk over-commit https://review.openstack.org/602478
05:16:52	openstackgerrit	Merged openstack/nova stable/queens: Fix stacktraces with redis caching backend https://review.openstack.org/606896
05:17:05	openstack	bug 1797580 in OpenStack Compute (nova) queens "NoValidHost during live migration after cold migrating to a specified host" [High,In progress] https://launchpad.net/bugs/1797580 - Assigned to Matt Riedemann (mriedem)
05:17:05	openstackgerrit	Merged openstack/nova stable/queens: Add regression test for bug 1797580 https://review.openstack.org/611944
05:17:10	openstackgerrit	Merged openstack/nova stable/queens: Not set instance to ERROR if set_admin_password failed https://review.openstack.org/608179
05:17:16	openstackgerrit	Merged openstack/nova stable/queens: Ignore VirtDriverNotReady in _sync_power_states periodic task https://review.openstack.org/607730
06:07:08	izza_	hi...i created instance with VGPU flavor in Openstack TripleO rocky version with Windows Server 2012 R2 guest OS,but as checked, graphics dirver is not visible in the guest OS
06:07:21	izza_	we're using the tesla P100
06:07:33	izza_	can someone here help us about my issue
06:10:46	openstackgerrit	Sujeet Banerjee proposed openstack/nova master: Hot fix for RMQ failures causing nova out of sync https://review.openstack.org/625150
06:59:19	openstackgerrit	Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143
07:20:03	openstackgerrit	Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143
07:20:04	openstackgerrit	Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142
07:51:20	sum12	Hello, I wanted to understand node discovery for cell_v2, so should a hostmapping be already created when nova-manage discover_hosts is run ?
07:55:17	openstackgerrit	Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142
09:10:23	openstackgerrit	Surya Seetharaman proposed openstack/nova master: Add DownCellFixture https://review.openstack.org/614810
09:10:24	openstackgerrit	Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657
09:16:06	tssurya	gibi: good morning! Whenever you have the time could you have a look at https://review.openstack.org/#/c/614810 (after the CI passes)? I have added more comments, thanks!
09:20:11	openstackgerrit	Lee Yarwood proposed openstack/nova stable/rocky: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623507
09:20:41	openstackgerrit	Lee Yarwood proposed openstack/nova stable/queens: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623509
09:21:18	openstackgerrit	Lee Yarwood proposed openstack/nova stable/pike: Add secret=true to fixed_key configuration parameter https://review.openstack.org/623510
09:30:25	openstackgerrit	zhangyujun proposed openstack/nova master: Should not raise while restore power on failed https://review.openstack.org/624854
10:11:43	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/618478
10:18:06	openstackgerrit	Lee Yarwood proposed openstack/nova master: fixtures: Return a mocked class instead of method within fake_imagebackend https://review.openstack.org/619804
10:18:48	lyarwood	mdbooth: ^ rebased this away from either of our work as neither requires it anymore, can you hit it and let me know if it's still useful?
10:19:04	lyarwood	that's just the fixtures change FWIW
10:19:16	lyarwood	the other workaround hasn't really changed but reviews still welcome
10:19:24	mdbooth	lyarwood: ack
10:24:48	openstackgerrit	Vlad Gusev proposed openstack/nova stable/pike: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/604039
10:24:54	openstackgerrit	Vlad Gusev proposed openstack/nova stable/pike: libvirt: Use os.stat and os.path.getsize for RAW disk inspection https://review.openstack.org/607544
10:29:37	lyarwood	^ LGTM btw.
10:29:40	mdbooth	lyarwood: I was just looking at my very similar fake_imagebackend patch
10:29:47	mdbooth	I commented:
10:30:02	mdbooth	+ fake_image_class.side_effect = image_init
10:30:02	mdbooth	+ fake_image_class = mock.create_autospec(imagebackend.Flat)
10:30:02	mdbooth	+ # the subclass constructors have a different signature.
10:30:02	mdbooth	+ # NOTE(mdbooth): we spec a subclass of imagebackend.Image here because
10:30:39	mdbooth	lyarwood: I assume I hit some particular issue to cause me to do that, because my first attempt would almost certainly have been what you did
10:31:54	mdbooth	lyarwood: It makes sense, though, because they do have different signatures.
10:33:11	lyarwood	mdbooth: hmm yeah but we only use the returned mock as a class remember
10:33:38	lyarwood	mdbooth: the side_effect then provides the same signature as the subclass constructor
10:34:21	mdbooth	lyarwood: I can't remember why I did this. I just assume there was a reason for it, because it doesn't seem like the first thing I'd have thought of.
10:34:56	lyarwood	yeah indeed
10:35:50	mdbooth	lyarwood: Anyway, assuming all *current* tests pass I'm happy to put my +1 on your patch as it. Future problems can be addressed if they arise.
10:36:04	lyarwood	mdbooth: ack thanks
10:44:20	lyarwood	https://review.openstack.org/#/q/topic:bug/1785827+(status:open+OR+status:merged) - if any stable cores are online this series is almost finished, just the Pike changes to +W
10:58:58	egonzalez	hi, did something get merged in the last few hours (maybe yesterday evening UTC) which can cause the following error?
10:59:28	egonzalez	http://paste.openstack.org/show/737292/
10:59:49	egonzalez	Error gathering result from cell 00000000-0000-0000-0000-000000000000: ProgrammingError: (pymysql.err.ProgrammingError) (1146, u"Table 'nova_cell0.services' doesn't exist")
11:02:50	egonzalez	added other log at http://paste.openstack.org/show/737293/ while executing nova-api command
11:14:43	mdbooth	lyarwood: I don't understand the revert_resize problem, btw, and without the instance.host check the workaround executes on _hard_reboot
11:15:39	mdbooth	lyarwood: So the revert_resize thing is that we execute cleanup on the dest, right?
11:15:57	lyarwood	mdbooth: argh yeah we'd lose console.log with hard_reboot, the directory is recreated however.
11:16:21	lyarwood	mdbooth: yes, revert resize cleanup is on the dest
11:16:28	mdbooth	console.log, kernel, and ramdisk, but yeah
11:16:41	lyarwood	mdbooth: it should recreate kernel and ramdisk no?
11:16:54	mdbooth	lyarwood: Unless they've been deleted in glance.
11:17:00	lyarwood	true
11:17:53	mdbooth	lyarwood: So, is the revert_resize thing a different bug?
11:18:12	mdbooth	Because your workaround wouldn't trigger on revert_resize.
11:18:27	lyarwood	mdbooth: with the host check removed it should
11:18:41	lyarwood	mdbooth: we call cleanup with destroy_disks=False and no migrate_data
11:18:43	mdbooth	lyarwood: Right, but it didn't before.
11:19:07	mdbooth	So is that a separate latent bug?
11:19:08	lyarwood	mdbooth: yeah, ah crap I didn't add the bug to the commit did I
11:19:25	lyarwood	mdbooth: yes, it's listed in the conf change
11:19:43	lyarwood	mdbooth: I'll respin the commit message now
11:21:52	openstackgerrit	Kashyap Chamarthy proposed openstack/nova master: libvirt: Support native TLS for migration and disks over NBD https://review.openstack.org/625216
11:22:15	kashyap	mdbooth: lyarwood: ^ I surely went overboard with the long commit message (that I spent 2 hours writing)
11:22:41	kashyap	But I made sure they're in readable chunks.
11:24:04	openstackgerrit	zhangyujun proposed openstack/nova master: Should not raise while restore power on failed https://review.openstack.org/624854
11:24:06	kashyap	Oh, it is WIP!
11:26:11	lyarwood	mdbooth: kernel, ramdisk etc are set against the flavor right?
11:27:28	lyarwood	ah, image props, my bad
11:27:41	mdbooth	lyarwood: Right. I think they can also be specified on the command line.
11:28:13	mdbooth	I don't think anybody actually uses them, but we probably shouldn't assume that, especially not for a backport.

Earlier   Later