Posted	Nick	Remark
#openstack-nova - 2019-09-09
21:51:38	sean-k-mooney	this is a behavior i have been ware of for years so i kind of just assume this was going to be fixed as part of either artoms work or stephens
21:54:47	mriedem	i felt pretty happy with the progress we had by thursday last week, we had a working CI run, you had done manual testing of upgrade paths, etc. dan and i did solid reviews, so i was thinking just fixing up those comments and going over again would do it, but the non-numa scope creep stuff threw a wrench into the mix adding what appears to be unnecessary risk
21:55:33	mriedem	and this isn't the only thing i'm trying to review this week to get in so i'm not 100% focused on this, so frequently changing is costly
21:55:35	sean-k-mooney	mriedem: well we can always jsut revert to that version that had been tested
21:55:45	sean-k-mooney	ack
21:55:53	mriedem	he'll still have to make changes, but yeah i know
21:56:12	mriedem	it's going to need re-test either way i think since there were non-trivial changes outside of the non-numa pinning stuff since last week
21:56:44	mriedem	melwitt: maybe we can get imacdonn to re-test, but he doesn't appear to be around,
21:56:55	mriedem	but fwiw i was able to reproduce in devstack - i think i left notes in the bug report on how to do that
21:57:17	mriedem	so i'm also not against just recreating in devstack w/o the fix, then applying the fix and assert the issue is gone
21:57:32	mriedem	this security groups crap all gets burned out with nova-net at some point anyway
21:57:45	mriedem	the fact we create a default security group when using neutron is pointless anyway since it's not used
22:01:15	sean-k-mooney	artom: dansmith mriedem. if ye come to a decision on a path forward i will retest everything tommorow with whatever ye choose.
22:10:55	mriedem	idk, i hate to make artom undo a bunch of work, but also don't like munging a bunch of other stuff in here just b/c it's convenient though out of scope
22:11:24	mriedem	i'd prefer to separate the non-numa pinning stuff out
22:11:27	mriedem	and deal with that later
22:11:46	sean-k-mooney	i.e. make it a patch at the end
22:11:54	sean-k-mooney	that we can merge when we get to it
22:12:02	mriedem	yes
22:12:38	mriedem	think of it like someone pushing a bug fix that also refactors a bunch of code unnecessarily, we're generally not cool with that
22:13:25	sean-k-mooney	ya its an unrelated or tangentally related change in a non trivial feature
22:20:45	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.80: Add user_id/project_id to migration-list API https://review.opendev.org/675023
22:25:47	openstackgerrit	Merged openstack/nova master: Don't duplicate PlacementFixture in libvirt func tests https://review.opendev.org/681059
22:40:27	melwitt	mriedem: ack, gonna add more comments to the review as far as why this patch gets around the problem
22:40:36	melwitt	(based on my read of the bug and code)
22:42:06	mriedem	gibi: as for your series (when you're back) you've got a real failure in the cold migration revert with a mixed compute grenade scenario https://review.opendev.org/#/c/676140/18
22:42:15	mriedem	good thing you added that test coverage though :)
22:42:47	efried	dansmith: would you please have another (hopefully last) look at the bottom two patches in the vpmem series https://review.opendev.org/#/c/678447/
22:45:03	efried	TIA.
22:45:03	efried	(I would feel better if it was you doing that than me)
22:45:03	efried	stephenfin: If you can hit the currently-bottom SEV patch, the rest is +A https://review.opendev.org/#/c/644565/
23:47:48	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/stein: doc: Fix a broken reference link https://review.opendev.org/681141
23:48:38	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/rocky: doc: Fix a broken reference link https://review.opendev.org/681142
23:49:36	openstackgerrit	Takashi NATSUME proposed openstack/nova stable/queens: doc: Fix a broken reference link https://review.opendev.org/681143
23:49:40	alex_xu	efried: yes, I don't found design problem and upgrade plan makes sense for me also. What I'm just doing is the manual test of the upgrade process, ensure thing works as expected now
#openstack-nova - 2019-09-10
01:59:55	openstackgerrit	Merged openstack/python-novaclient master: Microversion 2.79: Add delete_on_termination to volume-attach API https://review.opendev.org/673485
02:27:30	openstackgerrit	Akira KAMIO proposed openstack/nova master: VMware: disk_io_limits settings are not reflected when resize https://review.opendev.org/680296
03:01:08	openstackgerrit	Artom Lifshitz proposed openstack/nova master: LM: Use Claims to update numa-related XML on the source https://review.opendev.org/635229
03:01:08	openstackgerrit	Artom Lifshitz proposed openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827
03:01:09	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021
03:01:09	openstackgerrit	Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606
03:01:10	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595
03:38:34	Izza_	hello...good day, i'm doing tempest testing on openstack helm environment but i encountered error "Got Server Fault"
03:38:45	Izza_	tempest.lib.exceptions.ServerFault: Got server fault
03:39:03	Izza_	Failed 1 tests - output below:
03:39:04	Izza_	"/usr/lib/python2.7/site-packages/tempest/test.py", line 172, in setUpClass
03:41:26	Izza_	Failed 1 tests - output below:
03:41:27	Izza_	"/usr/lib/python2.7/site-packages/tempest/test.py", line 172, in setUpClass
03:44:19	Izza_	hi can u pls help me on my tempest testing, scenario: tempest.api.compute.servers.test_create_server.ServersTestJSON , error: tempest.lib.exceptions.ServerFault: Got server fault
03:44:29	Izza_	Details: Unexpected API Error. Please report this at http://bugs.launchpad.net/nova/ and attach the Nova API log if possible.
03:59:20	openstackgerrit	Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606
03:59:21	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595
03:59:21	openstackgerrit	Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021
04:30:30	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243
04:30:30	openstackgerrit	Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242
04:30:31	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
04:30:31	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244
04:30:32	openstackgerrit	Sundar Nadathur proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999
04:30:32	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
04:39:04	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244
04:39:04	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243
04:39:05	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
04:39:05	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
04:39:06	openstackgerrit	Sundar Nadathur proposed openstack/nova master: [WIP] add cyborg tempest job https://review.opendev.org/670999
06:13:31	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453
06:13:31	openstackgerrit	Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
06:13:32	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
06:13:32	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
06:13:33	openstackgerrit	Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
06:24:47	arshad777	I have created a sync replicated volume with peer persistence enabled. Attached this volume
07:18:55	openstackgerrit	Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640
07:18:56	openstackgerrit	Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
07:36:57	openstackgerrit	Takashi NATSUME proposed openstack/python-novaclient master: doc: Add support microversions for options https://review.opendev.org/681174
08:21:11	bauzas	stephenfin: once you're there, see dansmith's comments
08:21:27	bauzas	stephenfin: he wants to squash https://review.opendev.org/#/c/680983/
08:21:44	bauzas	stephenfin: so please try to provide the new revisions this morning
08:51:03	stephenfin	sure thing
09:02:24	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Support reverting migration / resize with bandwidth https://review.opendev.org/676140
09:11:36	openstackgerrit	weibin proposed openstack/nova master: Add support for using ceph RBD ereasure code https://review.opendev.org/681188
09:19:10	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972
09:21:29	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980
09:23:48	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497
09:24:40	stephenfin	bauzas: trivial patch with no conflicts here https://review.opendev.org/#/c/679339/
09:24:45	stephenfin	(if you'd be so kind)
09:25:02	bauzas	reminder : I'm French, I'm never kind
09:25:41	stephenfin	bauzas: I'm going to rebase my the cpu-resources series on top of the SEV series to head off the incoming merge conflicts too
09:25:50	stephenfin	while I rework it, that is
09:26:05	bauzas	stephenfin: +Wd FWIW
09:26:08	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize https://review.opendev.org/678827
09:26:17	bauzas	stephenfin: cool, I'll look at gibi's series then
09:28:24	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019
09:30:40	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Extract pf$N literals as constants from func test https://review.opendev.org/680991
09:30:41	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Improve dest service level func tests https://review.opendev.org/680998
09:34:41	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.80: Add user_id/project_id to migration-list API https://review.opendev.org/675023
09:38:26	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.80: Add user_id/project_id to migration-list API https://review.opendev.org/675023
09:48:51	bauzas	gibi: I'm almost done with https://review.opendev.org/#/c/676140 but I'll need to do some checks later today
09:57:18	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.80: Add user_id/project_id to migration-list API https://review.opendev.org/675023
10:09:57	aspiers	stephenfin: I'm here in case you have any last minute questions about https://review.opendev.org/#/c/644565/
10:21:31	gibi	bauzas: thanks
10:31:48	openstackgerrit	Alexandra Settle proposed openstack/nova master: Fixing broken links https://review.opendev.org/681206

Earlier   Later