Posted	Nick	Remark
#openstack-nova - 2020-07-30
16:27:38	sean-k-mooney	by default
16:28:43	sean-k-mooney	basically that is what i would like peple to run before pushing
16:29:01	jangutter	I'm in agreement.
16:30:03	sean-k-mooney	ok ill remvoe the functional test from cover so for now at least
16:30:41	sean-k-mooney	stephenfin: jangutter ralonsoh as a head up i also want to change the default ovsdb backend in os-vif this cycle
16:30:54	sean-k-mooney	i ment to do it af ew releases ago
16:31:02	sean-k-mooney	but we are still defaulting to ovs-vsctl
16:31:18	ralonsoh	sean-k-mooney, in os-vif?
16:31:32	sean-k-mooney	ralonsoh: ya
16:31:36	ralonsoh	we removed ovs-vsctl, we only have native now
16:31:51	sean-k-mooney	no
16:31:56	ralonsoh	in neutron
16:32:03	sean-k-mooney	in neutron yes
16:32:08	ralonsoh	just to let you know
16:32:13	sean-k-mooney	we never got around to doing it in os-vif
16:32:22	sean-k-mooney	it was planed for train
16:32:37	sean-k-mooney	but we forot to defualt in trian and hten remove in ussui
16:32:45	ralonsoh	it doesn't makes sense to keep both
16:32:54	sean-k-mooney	so im going to push a patch to do it now in victoria and remove in W
16:32:59	ralonsoh	cool
16:33:22	sean-k-mooney	i would remove it now but we have to have a deprecation cycle
16:33:39	ralonsoh	at least default to native, I think so
16:34:23	sean-k-mooney	yep anyway ill push a patch for that shortly
16:34:40	sean-k-mooney	just tought ye would like to know
16:42:30	sean-k-mooney	is this pep failure new http://paste.openstack.org/show/796470/
16:42:37	sean-k-mooney	*pep8
16:43:13	sean-k-mooney	looks like py38 and pyflakes are unhappy
16:46:39	stephenfin	sean-k-mooney: tox -e pep8 --recreate
16:46:46	stephenfin	you're using an old version of the library, I suspect
16:46:53	stephenfin	that or constraints isn't properly set up
16:47:06	sean-k-mooney	i did recreate it
16:47:15	sean-k-mooney	but i might need to rebase
16:47:39	sean-k-mooney	oh this is the train branch
16:47:43	sean-k-mooney	im backporting
16:47:55	sean-k-mooney	it need python 3.6
16:48:03	sean-k-mooney	on train not 3.8
16:49:05	gmann	sean-k-mooney: for 3.8 you need pyflakes 2.1.1 as min
16:49:46	sean-k-mooney	ya i just happened to use one of my 20.04 ubuntu vms to do the backport and i forgot it does not have a python old enough to run the train env
16:50:04	stephenfin	Fedora has multiple Python versions...
16:50:06	sean-k-mooney	i could update it to 2.1.1
16:50:09	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Raising a failure in case of 403 https://review.opendev.org/735068
16:50:21	sean-k-mooney	ubunutu normally does as well
16:50:27	sean-k-mooney	i just did not want to instal older ones
17:14:37	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Reject requests for no hyperthreads on hosts with HT https://review.opendev.org/744021
17:14:37	openstack	bug 1889633 in OpenStack Compute (nova) ussuri "Pinned instance with thread policy can consume VCPU" [High,Triaged] https://launchpad.net/bugs/1889633
17:14:37	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Add reproducer for bug #1889633 https://review.opendev.org/744020
17:14:40	stephenfin	sean-k-mooney: ^
17:14:58	stephenfin	Needs a unit test, I suspect. Will address tomorrow
17:15:00	stephenfin	For now, run ti,me
17:45:08	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Removed the host FQDN from the exception message https://review.opendev.org/743950
18:10:39	sean-k-mooney	stephenfin: one suggestion inline and a request for a release note
18:11:03	sean-k-mooney	stephenfin: but ya that should work, i think it can be slightly cleaner
18:13:26	sean-k-mooney	are we ment to include a release not with every bug fix by the way
18:13:43	sean-k-mooney	i forget if its all fixes or just some
18:14:16	sean-k-mooney	i kind of feel like i shoudl ask for one but its also a 6 line patch so it also feels like over kill
18:14:18	sean-k-mooney	https://review.opendev.org/#/c/743950/2
18:15:11	openstack	Launchpad bug 1851587 in OpenStack Compute (nova) "HypervisorUnavailable error leaks compute host fqdn to non-admin users" [Undecided,In progress] - Assigned to Harshavardhan Metla (harsha24)
18:15:11	sean-k-mooney	it is for a public security hardening bug however https://bugs.launchpad.net/nova/+bug/1851587 which is why i think operator might care to see it in a release note
18:28:00	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Moved the quoted section https://review.opendev.org/737215
19:16:01	openstackgerrit	Merged openstack/nova stable/ussuri: Handle multiple 'vcpusched' elements during live migrate https://review.opendev.org/743804
19:16:46	openstackgerrit	Lee Yarwood proposed openstack/nova stable/train: Handle multiple 'vcpusched' elements during live migrate https://review.opendev.org/743805
22:18:54	openstackgerrit	sean mooney proposed openstack/nova master: Lookup nic feature by PCI address https://review.opendev.org/739131
22:49:31	openstackgerrit	Ghanshyam Mann proposed openstack/os-vif master: [goal] migrate testing to ubuntu focal https://review.opendev.org/738130
23:16:14	openstackgerrit	Merged openstack/nova master: libvirt: Remove blockjob.end == 0 workaround resolved in libvirt v2.3.0 https://review.opendev.org/729596
23:16:21	openstackgerrit	Merged openstack/nova master: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/724842
23:40:00	openstackgerrit	Merged openstack/nova master: func: Add live migration rollback volume attachment tests https://review.opendev.org/743534
23:40:08	openstack	bug 1889108 in OpenStack Compute (nova) "failures during driver.pre_live_migration remove source attachments during rollback" [High,In progress] https://launchpad.net/bugs/1889108 - Assigned to Lee Yarwood (lyarwood)
23:40:08	openstackgerrit	Merged openstack/nova master: Add regression tests for bug #1889108 https://review.opendev.org/743289
#openstack-nova - 2020-07-31
00:30:05	rambo_li	If anyone can help me review the https://review.opendev.org/#/c/739349/, thank you very much!
02:38:44	openstackgerrit	Merged openstack/nova stable/train: Handle multiple 'vcpusched' elements during live migrate https://review.opendev.org/743805
06:27:50	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Raising a failure in case of 403 https://review.opendev.org/735068
06:36:28	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Moved the quoted section https://review.opendev.org/737215
08:05:03	nightmare_unreal	hello can someone leave their comment on this ticket. https://review.opendev.org/#/c/742210/ . Also suggestion on test cases
08:05:04	nightmare_unreal	thanks
08:26:10	aarents	good morning nova,
08:27:02	aarents	lyarwood thanks for review! I fixed the nit on : https://review.opendev.org/#/c/696084/ and https://review.opendev.org/#/c/743708/ , I will be off next week, so I will be able to re-submit only after if needed.
10:00:06	openstackgerrit	Harshavardhan Metla proposed openstack/nova master: Raising a failure in case of 403 https://review.opendev.org/735068
10:59:44	lyarwood	aarents: ack I'll take a look today
11:20:35	songwenping_	bauzas: hi, i test create server with vgpu, and encounter error with "Verify all devices in group 71 are bound to vfio-<bus> or pci-stub and not already in use". Do you have any idea?
11:21:53	jsuchome	lyarwood: hi, me again ... anything else we need for that rbd-download job? besides https://review.opendev.org/#/c/743220/
11:30:07	openstackgerrit	Takashi Natsume proposed openstack/python-novaclient master: Remove unused code https://review.opendev.org/744136
12:03:29	lyarwood	melwitt: thanks for the master reviews btw, https://review.opendev.org/#/c/743493/ is ready for stable/queens btw if you have time today.
12:05:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Reject requests for no hyperthreads on hosts with HT https://review.opendev.org/744021
12:05:39	openstack	bug 1889633 in OpenStack Compute (nova) "Pinned instance with thread policy can consume VCPU" [High,In progress] https://launchpad.net/bugs/1889633 - Assigned to Stephen Finucane (stephenfinucane)
12:05:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Add reproducer for bug #1889633 https://review.opendev.org/744020
12:05:49	stephenfin	sean-k-mooney, artom: ^
12:05:51	stephenfin	songwenping_: bauzas is on vacation for the next few weeks, unfortunately :-(
12:06:28	songwenping_	ok. thanks stephenfin.
12:12:19	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove warning filters for legacy Python version https://review.opendev.org/743285
12:12:19	openstackgerrit	Stephen Finucane proposed openstack/nova master: trivial: Remove unused attributes https://review.opendev.org/743284
12:20:07	artom	stephenfin, small nit on the test patch, but it'd bug me if it went in as is - just comes off as sloppy
12:21:14	stephenfin	artom: Hmm, I don't understand the first comment. Can you elaborate?
12:21:25	artom	stephenfin, eh?
12:21:35	stephenfin	Since we only have 4 CPUs
12:21:51	stephenfin	where's that coming from?
12:22:14	artom	stephenfin, oh, you kept it as 8
12:22:18	sean-k-mooney	we have 8
12:22:21	artom	Huh, and the pinning assertion still works?
12:22:25	stephenfin	I kept what as 8?
12:22:31	artom	8 CPUs

Earlier   Later