Posted	Nick	Remark
#openstack-nova - 2019-09-12
23:23:14	sean-k-mooney	https://review.opendev.org/#/c/548936/102/.zuul.yaml@69
23:23:32	sean-k-mooney	i have it mostly done but i dont know how to change the config on upgrade
23:24:05	sean-k-mooney	i think i might need a greade plugin to do that or and extention to the grenade job
23:51:12	artom	stephenfin, that was my understanding, yeah
23:51:27	artom	I mean, even dansmith was telling me he'd look at it next week
23:51:43	artom	To me that's an implicit "test-only patches can wait until after FF"
23:53:22	sean-k-mooney	speaking of test only patches my uber test of doom https://review.opendev.org/#/c/681771/2 is pendingin the experimenal queue. that will test PMEM PCPUS and numa migration
23:53:31	sean-k-mooney	its curently getting ready to stack
23:53:34	sean-k-mooney	http://zuul.openstack.org/stream/6a67b28d6e9e4b739e7575df7943a69d?logfile=console.log
23:53:42	sean-k-mooney	but im going to go to sleep now
23:54:21	sean-k-mooney	if nothing else merge but that 31 patch chain then there are no conflic between those 3 feature branches
23:54:39	sean-k-mooney	so lets see how that looks in the morning
#openstack-nova - 2019-09-13
00:02:07	openstackgerrit	Merged openstack/nova master: Support reverting migration / resize with bandwidth https://review.opendev.org/676140
00:09:07	donnyd	sean-k-mooney: is FN giving you any issues?
00:09:07	donnyd	looks like you have some jobs running
01:10:19	brinzhang	melwitt: mriedem: Thanks
01:41:45	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Remove an unused file and a related description https://review.opendev.org/681955
03:15:51	openstackgerrit	Merged openstack/nova master: Add nova-status to man-pages list https://review.opendev.org/681733
03:31:26	openstackgerrit	Merged openstack/nova master: trivial: Remove single-use classmethod https://review.opendev.org/681751
06:02:44	gibi	happy rechecking morning nova
06:24:42	gibi	ok I think I rechecked every approved but failed nova patch
06:26:19	gibi	32 approved nova patches in the check queue 3 in the gate queue
06:59:52	gibi	The todays killer bug is: bug 1813147 - p35 jobs are failing with subunit.parser ... FAILED
06:59:53	openstack	bug 1813147 in OpenStack Compute (nova) "p35 jobs are failing with subunit.parser ... FAILED" [High,In progress] https://launchpad.net/bugs/1813147 - Assigned to Balazs Gibizer (balazs-gibizer)
07:15:13	openstackgerrit	Merged openstack/nova master: Add user_id and project_id column to Migration https://review.opendev.org/673990
07:15:39	openstackgerrit	Merged openstack/nova master: Cleanup reno live-migration-with-PCI-device https://review.opendev.org/681942
07:19:33	brinzhang_	good morning gibi
07:20:01	gibi	brinzhang_: good morning
07:45:04	bauzas	good morning folks
07:47:07	gibi	good morning bauzas
07:49:44	openstackgerrit	Takashi NATSUME proposed openstack/nova master: doc: mark the max microversion for train https://review.opendev.org/681982
07:59:13	kashyap	efried: aspiers: I was ill yesterday (and partly today). Reading the relevant bits.
08:00:55	kashyap	aspiers: No, that reference to 'vexpress-a15' is _not_ correct: it should be 'virt' for ARMv7 or AArch64.
08:01:04	kashyap	aspiers: See the releasenotes/notes/Use-virt-as-machine-type-for-ARMv7-cd2c252336057ec8.yaml
08:02:00	kashyap	aspiers: That Glance doc should be updated
08:02:33	bauzas	gibi: I eventually went unable to join yesterday evening
08:02:38	bauzas	gibi: any idea on where we are ?
08:02:48	bauzas	did we cut the cord ?
08:02:58	bauzas	I see a lot of merge conflicts as expected
08:03:15	kashyap	aspiers: For reference, see this: https://opendev.org/openstack/nova/commit/e155baefb0
08:03:28	gibi	bauzas: the bw series is approved. I see vpmem in also approved
08:03:34	bauzas	gibi: actually, nevermind, I'll read the nova meeting logs
08:03:50	bauzas	gibi: the cpu-resources too, but it got conflicts
08:04:07	gibi	bauzas: yepp
08:07:18	bauzas	okay, so tl;dr: let's continue trying to merge stuff that was previously approved
08:07:43	bauzas	gibi: how is your series ? still in the weeds ?
08:09:38	gibi	bauzas: one patch merged during the night, rest is in check queue after recheck
08:11:01	bauzas	ack
08:13:47	kashyap	aspiers: Sent the doc fix to Glance: https://review.opendev.org/681990 useful-image-properties: Update note on machine types
08:33:40	openstackgerrit	Ivaylo Mitev proposed openstack/nova master: VMware: Update flavor-related metadata on resize https://review.opendev.org/681004
08:38:55	luyao	stephenfin: are you around, I see you string the vpmems and cpu resources, they merge conflict now
08:39:58	luyao	stephenfin: I'll rebase to master branch
08:41:52	luyao	stephenfin: this is the last patch , right ? https://review.opendev.org/#/c/674895/41
08:55:46	openstackgerrit	Luyao Zhong proposed openstack/nova master: db: Add resources column in instance_extra table https://review.opendev.org/678447
08:55:46	openstackgerrit	Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResourceList objs https://review.opendev.org/678448
08:55:47	openstackgerrit	Luyao Zhong proposed openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449
08:55:47	openstackgerrit	Luyao Zhong proposed openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450
08:55:48	openstackgerrit	Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
08:55:48	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453
08:55:49	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
08:55:49	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
08:55:50	openstackgerrit	Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
08:55:50	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640
08:55:51	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793
08:55:51	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: '_get_(v|p)cpu_total' to '_get_(v|p)cpu_available' https://review.opendev.org/672693
08:55:52	openstackgerrit	Luyao Zhong proposed openstack/nova master: objects: Add 'InstanceNUMATopology.cpu_pinning' property https://review.opendev.org/680106
08:55:52	openstackgerrit	Luyao Zhong proposed openstack/nova master: Validate CPU config options against running instances https://review.opendev.org/680107
08:55:53	openstackgerrit	Luyao Zhong proposed openstack/nova master: trivial: Use sane indent https://review.opendev.org/680229
08:55:53	openstackgerrit	Luyao Zhong proposed openstack/nova master: objects: Add 'NUMACell.pcpuset' field https://review.opendev.org/680108
08:55:54	openstackgerrit	Luyao Zhong proposed openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800
08:55:54	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Start reporting 'HW_CPU_HYPERTHREADING' trait https://review.opendev.org/675571
08:59:32	bauzas	luyao: AFAICS, looks like stephenfin rebased his series on top of yours to prevent more merge conflicts
08:59:36	bauzas	stephenfin: correct ?
09:01:51	luyao	bauzas: I think so, I handled the merge conflict due to my first two patches
09:05:59	bauzas	cool
09:24:29	aspiers	kashyap: thanks, please see also https://review.opendev.org/#/c/681866/
09:25:56	kashyap	aspiers: Ah, it's the other thing you mentioned; metadata props. Nice
09:30:24	kashyap	aspiers: +1, FWIW. Wauw, given the number of values/props, could spot no typos :D
09:41:53	aspiers	kashyap: why is ARM singled out in these docs when we have other defaults for other archs?
09:43:00	kashyap	aspiers: I don't know who wrote it originally. But I agree, we should be complete and specify the defaults for other arches, too?
09:43:12	aspiers	That was my thought
09:43:44	stephenfin	luyao: Correct. Looks like you have it handled. Thanks :)
09:43:45	kashyap	Mention it in the review, please.
09:43:47	aspiers	kashyap: We probably also need comments in the nova code "# if you change this, don't forget to change useful-image-properties.rst in glance"
09:44:09	kashyap	aspiers: Yes, good reminder, to keep things in sync
09:44:23	aspiers	kashyap: I'll do that now
09:47:12	aspiers	kashyap: hw_machine_type is not the only thing needing a reminder, what else?
09:49:57	aspiers	hw_*_bus is one
09:55:36	kashyap	aspiers: Yeah, hw_*_bus and machine type are the only things that come to mind
10:01:01	openstackgerrit	Adam Spiers proposed openstack/nova master: Add reminder to update corresponding glance docs https://review.opendev.org/682012
10:01:02	aspiers	kashyap: ^^^
10:04:38	kashyap	aspiers: Already has it open
10:10:00	kashyap	aspiers: Reviewed; only thing is - do you want to spell out the name of the document?
10:10:22	aspiers	kashyap: I removed that from the first version, because it could change, and anyway there are multiple files
10:10:38	aspiers	there's the metadefs JSON too
10:10:39	kashyap	aspiers: Okay, thought so.
10:11:44	kashyap	aspiers: And on casing, is my guess correct?
10:22:33	aspiers	kashyap: replied
10:31:29	kashyap	aspiers: So I have to "unlearn" the capitalization everywhere except at the start of sentence.
10:31:35	aspiers	kashyap: IIUC

Earlier   Later