Posted	Nick	Remark
#openstack-nova - 2019-05-23
23:57:55	openstackgerrit	Merged openstack/python-novaclient stable/stein: Set the lower limit of api_version for volume_type https://review.opendev.org/661024
#openstack-nova - 2019-05-24
02:03:02	openstackgerrit	melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.opendev.org/638324
02:03:02	openstackgerrit	melanie witt proposed openstack/nova master: Add documentation for counting quota usage from placement https://review.opendev.org/653845
02:42:38	openstackgerrit	Merged openstack/nova stable/stein: Disable limit if affinity(anti)/same(different)host is requested https://review.opendev.org/659239
03:23:19	yaawang	bauzas: Hi, in spec expose-auto-converge-post-copy(https://review.opendev.org/#/c/651681/), what do you mean by "preffering"?
04:58:25	openstackgerrit	Sundar Nadathur proposed openstack/nova-specs master: Nova Cyborg interaction specification. https://review.opendev.org/603955
05:10:07	openstackgerrit	Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for cyborg access https://review.opendev.org/631242
05:10:07	openstackgerrit	Sundar Nadathur proposed openstack/nova master: WIP: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243
05:10:08	openstackgerrit	Sundar Nadathur proposed openstack/nova master: WIP: Create and bind Cyborg ARQs. https://review.opendev.org/631244
05:10:09	openstackgerrit	Sundar Nadathur proposed openstack/nova master: WIP: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245
05:49:11	eandersson	What is the general recommended value for heal_instance_info_cache_interval on a large deployment (~1k nodes)?
05:50:01	eandersson	With the default we couldn't scale neutron fast enough to actually handle the load.
06:18:08	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Add compute_nodes_uuid field to Destination object https://review.opendev.org/661188
06:31:39	mnaser	melwitt, efried: wrt nova-consoleauth, we've killed it in production (and we've killed it in OSA and looks like Matt proposed a change to remove the cleanup code) -- https://review.opendev.org/#/c/661126/
06:42:30	openstackgerrit	Leehom Li proposed openstack/nova master: RT keep report usage in case CPUPinning conflict https://review.opendev.org/661208
06:44:40	openstackgerrit	Leehom Li proposed openstack/nova master: RT keep report usage in case CPUPinning conflict https://review.opendev.org/661208
07:24:38	jangutter	If only this comic came out at the start of the month: https://xkcd.com/2153/
07:39:44	openstackgerrit	Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765
08:25:55	lyarwood	ganso: LGTM but as tonyb said, as this is stable-only we typically get full Nova Core members to also sign off on things
08:26:55	lyarwood	dansmith / efried ; https://review.opendev.org/#/c/659338 - as above, as this is stable-only would you mind giving this a once over?
08:28:16	lyarwood	mdbooth: https://review.opendev.org/#/c/658903/ - tgif, am I missing the point here?
08:28:54	mdbooth	lyarwood: looking
08:31:33	lyarwood	ah nvm I think I see it now
08:31:56	lyarwood	they want to pull directly from rbd when using a file based backend
08:34:17	mdbooth	lyarwood: Right
08:35:41	mdbooth	lyarwood: I hesitate to say it, but I think we want an os-glance library for this
08:36:07	mdbooth	Otherwise we're embedding details of glance backend drivers in nova
08:36:20	mdbooth	I suspect cinder would also use it
08:36:25	lyarwood	we already have rbd_utils.py
08:36:38	mdbooth	That's generic
08:36:48	mdbooth	And glance presumably has other backend drivers
08:36:51	lyarwood	as is this stuff tbh
08:37:00	mdbooth	which aren't ceph
08:37:07	lyarwood	there's nothing glance specific you'd need here
08:37:13	lyarwood	you already have the location
08:37:28	lyarwood	if anything rbd_utils etc could all live in os-brick
08:38:50	mdbooth	lyarwood: See: file, http, rbd, swift, sheepdog, cinder, vmware
08:42:07	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Add valid compute nodes with host and/or node in api layer https://review.opendev.org/661237
08:53:00	lyarwood	mdbooth: right, I just don't think we need yet another lib for this
10:08:34	openstackgerrit	Silvan Kaiser proposed openstack/nova stable/stein: Added mount fstype based validation of Quobyte mounts https://review.opendev.org/660706
10:23:32	openstackgerrit	Merged openstack/nova stable/stein: Fix target used in nova.policy.check_is_admin https://review.opendev.org/660330
10:23:45	openstackgerrit	Merged openstack/nova stable/queens: Define irrelevant-files for tempest-full-py3 job https://review.opendev.org/650460
10:23:52	openstackgerrit	Merged openstack/nova stable/queens: Error out migration when confirm_resize fails https://review.opendev.org/652150
10:32:37	openstackgerrit	Merged openstack/nova stable/queens: Teardown networking when rolling back live migration even if shared disk https://review.opendev.org/658149
10:32:47	openstackgerrit	Merged openstack/nova master: Link versioned notification talk into docs https://review.opendev.org/661115
10:57:51	openstackgerrit	Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500
11:07:42	openstackgerrit	Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500
11:39:15	lyarwood	has anyone ever seen LM try to claim resources on the source at the start of a migration?
11:39:36	lyarwood	IHAC suggesting that's happening but that seems totally wrong
12:03:59	openstackgerrit	Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500
12:09:02	efried	stephenfin: in case you didn't see http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-05-24.log.html#t2019-05-24T06:31:39
12:09:41	openstackgerrit	Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500
12:15:18	aspiers	god sometimes Python really pisses me off
12:16:45	aspiers	Python 2/3 unicode stuff is a never-ending nightmare
12:17:05	aspiers	I'm literally seeing heisenbugs with it
12:17:18	aspiers	one minute everything's fine, the next it magically breaks
12:17:23	cdent	i've never met a language that does it in a way that makes me happy. python is one of many that make me want to flip tables
12:17:59	cdent	aspiers: bugs on your own machine or remote nodes? if the latter there are so many variables involved (system encoding, python build, blah blah)
12:18:08	aspiers	local
12:18:32	aspiers	I'm on the verge
12:18:54	aspiers	I just want to slurp in the contents of __file__ (sounds weird I know, but there is a good reason) and then do a substring match on it
12:19:28	aspiers	if I try to decode('utf-8') on each line I get UnicodeDecodeError: 'utf8' codec can't decode byte 0xf3 in position 1: unexpected end of data
12:19:54	aspiers	despite it being an ASCII file (nova/nova/tests/unit/test_test.py)
12:20:34	aspiers	ascii is no better: UnicodeDecodeError: 'ascii' codec can't decode byte 0xf3 in position 1: ordinal not in range(128)
12:24:34	aspiers	WTF!! I literally ran the test twice in a row and it only failed the second time!!
12:27:35	aspiers	OK, I've spotted a pattern
12:27:41	aspiers	It always passes on py36
12:28:02	aspiers	It *only* passes on py27 if I run it after editing and saving the file
12:28:11	aspiers	If I re-run it, subsequent runs fail
12:28:39	aspiers	WT actual F
12:31:31	cdent	if __file__ always the .py file and never the .pyc file?
12:31:38	aspiers	ohhhhhhh
12:31:39	cdent	s/if/is/
12:31:42	aspiers	you genius
12:31:44	aspiers	that will be it
12:31:50	cdent	it was a guess
12:32:01	aspiers	that would perfectly explain it though
12:32:06	cdent	my genius is never expecting things to be sensible
12:32:10	aspiers	haha
12:32:19	cdent	"what if it's this totally stupid thing that shouldn't happen"
12:32:20	aspiers	my folly is always expecting things to be sensible
12:32:36	cdent	how long have you been in this business ? :)
12:32:38	aspiers	(except in politics. I'm not *that* stupid)
12:35:21	aspiers	cdent: works! __file__.rstrip('c')
12:35:32	cdent	wacky
12:35:44	aspiers	you just saved me bashing my head over this for hours - THANK YOU
12:35:53	cdent	you're very welcome
12:36:12	aspiers	no amount of rubber-ducking would have helped on that one, I fear
12:36:49	aspiers	:)
12:38:30	openstackgerrit	Adam Spiers proposed openstack/nova master: Move selective patching of open() to nova.test for reuse https://review.opendev.org/661266
12:38:47	aspiers	cdent: ^^^
12:39:02	aspiers	BTW I think I fixed the contextlib issue in the parent of that one
12:41:17	cdent	\o/
12:51:53	efried	tonyb: done
12:54:59	aspiers	cdent: just noticed some functional tests are calling os.path.exists('/etc/policy.d')
12:55:13	aspiers	presumably that's not good in a functional test environment
12:55:42	cdent	There's a few bleeds like that in the nova functional tests
12:55:59	aspiers	also .netrc, yikes
12:56:14	cdent	I remember there were some (since fixed) I think, which would be change behavior by using the /etc/nova/nova.conf
12:58:38	aspiers	my new helpers might help clean some of those cases up

Earlier   Later