Posted	Nick	Remark
#openstack-nova - 2019-09-24
17:23:30	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add functional tests for [cinder]/cross_az_attach=False https://review.opendev.org/684141
17:39:11	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Stop filtering out 'accepted' for in-progress migrations https://review.opendev.org/684409
18:14:22	KeithMnemonic	mriedem,sean-k-mooney: What was not fixed is perception. The 3 items needed are not something we do by default in our release
18:14:47	KeithMnemonic	so for the customer since they do they after install, it was their view it was not fixed
18:15:01	mriedem	ack
18:20:25	KeithMnemonic	and ideally I think their "pie in the sky" is for the amount of storage/usage on their backend to be used in anything that shows consumption or checks free space
18:24:18	openstackgerrit	Merged openstack/nova master: Reject migration with QoS port from conductor if RPC pinned https://review.opendev.org/683948
18:24:41	mriedem	KeithMnemonic: let me guess, they are using ceph?
18:25:02	mriedem	and confused about nova saying they have $num_computes x total ceph pool size of storage?
18:25:37	mriedem	KeithMnemonic: or you mean to use cinder for all storage rather than local disk on the compute hosts, e.g. volume-backed everywhere?
18:26:08	mriedem	the latter has details in https://review.opendev.org/#/c/511965/
18:26:26	mriedem	the former https://review.opendev.org/#/c/650188/
18:26:39	KeithMnemonic	second, they needed large root volumes, so until they applied the 3 steps, they did not understand why a backend with 50 TB was complaining about no space ;-)
18:27:39	mriedem	tbc i think the only thing you really need in rocky to properly track disk usage for volume-backed servers in nova is to disable the DiskFilter
18:28:11	mriedem	that removes the root disk filtering and claims during scheduling/build and what the resource tracker reports out of the hypervisors API
18:28:25	mriedem	the zero_disk_flavor policy thing is more tied to a security issue than anything
18:32:49	openstackgerrit	Matt Riedemann proposed openstack/nova master: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/680762
18:32:50	openstackgerrit	Matt Riedemann proposed openstack/nova master: Isolate request spec handling from _cold_migrate https://review.opendev.org/680763
18:32:56	mriedem	gibi: stephenfin: ^ was approved but needed to be rebased if one of you can re-approve
18:33:52	mriedem	ah crap i missed something gibi changed, sec
18:37:26	openstackgerrit	Matt Riedemann proposed openstack/nova master: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/680762
18:37:27	openstackgerrit	Matt Riedemann proposed openstack/nova master: Isolate request spec handling from _cold_migrate https://review.opendev.org/680763
18:59:00	openstackgerrit	Matt Riedemann proposed openstack/nova master: Make nova-next multinode and drop tempest-slow-py3 https://review.opendev.org/683988
19:41:55	openstackgerrit	Akihiro Motoki proposed openstack/nova master: doc: Improve PDF document structure https://review.opendev.org/682746
19:50:38	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add heal_instance_mappings command https://review.opendev.org/655908
19:54:52	openstackgerrit	Walter A. Boring IV (hemna) proposed openstack/nova stable/pike: WIP: Avoid redundant initialize_connection on source post live migration https://review.opendev.org/683008
19:59:06	mriedem	♪ jesus just left chicago ♪
20:01:21	dansmith	I approve.
20:22:25	openstackgerrit	François Palin proposed openstack/nova master: Add retry when attachment_delete or terminate_connection fails https://review.opendev.org/669674
20:34:12	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add heal_instance_mappings command https://review.opendev.org/655908
23:26:19	openstackgerrit	Dustin Cowles proposed openstack/nova master: Provider Config File: Merge provider configs to provider tree https://review.opendev.org/676522
#openstack-nova - 2019-09-25
01:00:37	openstackgerrit	Matt Riedemann proposed openstack/nova master: Make nova-next multinode and drop tempest-slow-py3 https://review.opendev.org/683988
03:55:43	openstackgerrit	Merged openstack/nova master: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/680762
03:55:49	openstackgerrit	Merged openstack/nova master: Isolate request spec handling from _cold_migrate https://review.opendev.org/680763
07:28:26	openstackgerrit	Jacek Tomasiak proposed openstack/nova master: Ensure volume filesystems are mounted on resume https://review.opendev.org/684290
07:29:19	openstackgerrit	Luyao Zhong proposed openstack/nova master: doc: attaching virtual persistent memory to guests https://review.opendev.org/680300
07:42:37	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Reset forced_destination before migration at a proper time https://review.opendev.org/684546
07:42:37	openstack	bug 1845291 in OpenStack Compute (nova) "migration is not recheduled if the server originally booted with --availability-zone :" [Undecided,New] https://launchpad.net/bugs/1845291 - Assigned to Balazs Gibizer (balazs-gibizer)
07:42:37	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Functional reproduction for bug 1845291 https://review.opendev.org/684545
07:42:51	openstackgerrit	Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242
07:42:52	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244
07:42:52	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243
07:42:53	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
07:42:53	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245
07:42:54	openstackgerrit	Sundar Nadathur proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999
07:59:36	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Move HostNameWeigher to a common fixture https://review.opendev.org/683874
09:22:54	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/pike: Delete instance_id_mappings record in instance_destroy https://review.opendev.org/684658
09:28:24	openstackgerrit	Chris Dent proposed openstack/nova master: Correct link to placement upgrade notes https://review.opendev.org/684661
09:30:51	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Mask the token used to allow access to consoles https://review.opendev.org/220622
10:29:21	openstackgerrit	Merged openstack/nova master: Remove upgrade specific info from user facing exception text https://review.opendev.org/684253
10:37:04	openstackgerrit	Merged openstack/python-novaclient master: Add a check for --config-drive option on nova boot https://review.opendev.org/653683
10:37:48	openstackgerrit	Chris Dent proposed openstack/nova master: single pass instance info fetch in host manager https://review.opendev.org/623558
11:30:26	openstackgerrit	zhurong proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999
11:32:04	gibi	stephenfin, alex_xu: do you have ideas how we will do the nova project update in Shanghai?
11:43:03	cdent	stephenfin: this is a nova docs change driven by the structural changes in the placement docs: https://review.opendev.org/#/c/684661/
12:06:44	stephenfin	gibi: Haven't a clue. I was hoping efried would just give us slides and we'd use those \o/
12:07:10	gibi	stephenfin: :)
12:26:57	donnyd	did we ever get that grenade thing sorted out?
12:55:58	mriedem	stephenfin: you might want to propose wording for the pcpu stuff you want to see in https://review.opendev.org/#/c/683327/
12:56:31	mriedem	i guess just something like, "It's now possible to place pinned and unpinned instances on the same host when using the libvirt driver."?
12:56:59	stephenfin	basically, yeah. tbh, I was hoping someone less in the trenches would be able to provide exact wording to make sure I was assuming too much
12:57:01	stephenfin	if that makes sense
12:57:47	mriedem	alex_xu: can you review the vpmem docs today? https://review.opendev.org/#/c/680300/ - rc1 is tomorrow
12:58:08	mriedem	pinned and unpinned + same host + libvirt seems generic enough to me
12:58:20	mriedem	i'd rebase the prelude on the actual admin docs patch and just link to that for details
12:58:28	mriedem	i can make that change
13:06:18	mriedem	stephenfin: who else reviewed the pcpu series that can review those docs?
13:06:19	mriedem	bauzas: ?
13:06:43	stephenfin	bauzas, alex_xu or efried should be able to handle it, I suspect
13:12:56	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add a prelude for the Train release https://review.opendev.org/683327
13:13:00	mriedem	is it a french holiday today?
13:13:12	mriedem	the feast of maximum strikage
13:13:39	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Reset forced_destination before migration at a proper time https://review.opendev.org/684546
13:14:00	bauzas	mriedem: I can do
13:14:13	gibi	mriedem: fixed your comments in ^^
13:15:36	mriedem	sean-k-mooney: stephenfin: how much of sean's comments are going to be addressed in https://review.opendev.org/#/c/683437/ which i thought was going to be more of a refactor to ease where the new pcpu docs go but sounds like it's adding new content as well
13:15:45	mriedem	new content is fine, but not really when it's in a critical path to merging changes we need for rc1
13:16:35	stephenfin	I can fix them the first two now
13:16:40	stephenfin	*the last two
13:17:01	stephenfin	the first one (documenting RT) is much more work and not needed here
13:17:46	mriedem	ok i guess leave a .. todo:: for whatever needs a follow up
13:18:07	stephenfin	there's already one but in another doc (the flavor extra spec one)
13:18:16	stephenfin	so I think we're covered
13:18:21	stephenfin	I'll comment as much
13:18:54	stephenfin	I purposefully wanted to avoid getting into the weeks on the emulator threads stuff. Should have known sean-k-mooney would call me on it, heh
13:25:29	openstackgerrit	Jacek Tomasiak proposed openstack/nova master: Ensure volume filesystems are mounted on resume https://review.opendev.org/684290
13:25:47	openstackgerrit	Ivaylo Mitev proposed openstack/nova master: VMware VMDK detach: get adapter type from instance VM https://review.opendev.org/653738
13:28:47	mriedem	gibi: on https://review.opendev.org/#/c/684545/ i think we should do that outside of the ServerMovingTests class,
13:29:10	mriedem	and make it a standalone test class under regressions - easier to backport that way and we won't run duplicates of it in extended classes
13:29:28	gibi	mriedem: ack. I will move the test under regressions
13:30:36	stephenfin	mriedem: incoming
13:30:36	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Update CPU topologies guide to reflect the new PCPU world https://review.opendev.org/683485
13:30:36	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Clarify everything CPU pinning https://review.opendev.org/683437
13:31:03	stephenfin	bauzas, alex_xu, sean-k-mooney: https://review.opendev.org/683437 https://review.opendev.org/683485
13:31:16	bauzas	stephenfin: I just provided comments for PS3 :)
13:31:28	bauzas	but honestly, it's just a nirt
13:31:49	stephenfin	bauzas: I replied
13:32:03	bauzas	stephenfin: anyway, it's docs
13:32:05	stephenfin	(I already have a glossary at the top of the document explaining the terms)
13:32:30	bauzas	meaning that I'm fine with +2ing the change and just saying that we could provide better explanations in some follow-up

Earlier   Later