Posted	Nick	Remark
#openstack-sdks - 2020-03-26
12:29:10	smcginnis	mordred: Looks like something else going on there - fixtures._fixtures.timeout.TimeoutException
12:29:42	smcginnis	But I can fix up the pep8 issues at least.
12:29:48	mordred	smcginnis: yeah - that seems super unlikely to be caused by flake8 ... but oy
12:30:28	smcginnis	"F821 undefined name 'Optional'" Wasn't that just added?
12:31:49	smcginnis	Hmm, and doesn't fail locally for me.
12:32:12	tremble	smcginnis, I kept hitting those timeout exceptions too, I think there's a flakey test somewhere.
12:32:14	mordred	smcginnis: I LOVE when flake8 is weird like that
12:32:41	smcginnis	At least a few of the things are legitimate. Might need to add some ignores in there for others.
12:32:46	mordred	smcginnis: maybe it has something to do with the python version?
12:33:01	smcginnis	And hopefully another pass through the check queue will get around those UT failures.
12:33:08	mordred	the Optional error would be solved by importing Optional from typing
12:33:39	smcginnis	Ohhhh, interesting...
12:33:59	smcginnis	Current master that states hacking>=2.0.0 results in "hacking==1.1.0" getting installed.
12:34:09	smcginnis	And with the patch just adding a cap, 2.0.0 gets installed.
12:34:21	mordred	aroo?
12:34:36	mordred	can we agree that doesn't make sense?
12:34:37	smcginnis	Glad I just pulled a shot.
12:34:44	smcginnis	Completely.
12:35:01	mordred	tremble: new release of sdk is winding its way through the machinery
12:36:03	smcginnis	OK, I need more coffee.
12:36:14	smcginnis	I am NOT just adding a cap.
12:36:19	sshnaidm	odyssey4me, hi, are you going to backport https://review.opendev.org/#/c/715052/ to stable branches?
12:36:33	smcginnis	I actually am bumping hacking to 2.0, which now makes total sense why it would trigger new errors.
12:36:39	odyssey4me	sshnaidm: yep, happy to once the preceding patches merge
12:36:50	openstackgerrit	Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325
12:36:52	smcginnis	And obviously needed more coffee when I did that in the first place, because I should have expected new things to pop up.
12:38:12	sshnaidm	odyssey4me, preceding patches..?
12:38:20	mordred	smcginnis: haha. well, that at least makes more sense?
12:38:34	odyssey4me	sshnaidm: https://review.opendev.org/#/q/I50b1020c896d7d8a7d58e62de778bb8c2b6e970e - I'll pick the patch on top of those
12:39:05	openstackgerrit	Jesse Pretorius (odyssey4me) proposed openstack/openstacksdk stable/train: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715192
12:39:08	sshnaidm	odyssey4me, I can just abandon mines, you have a full solution in yours
12:40:00	openstackgerrit	Sagi Shnaidman proposed openstack/openstacksdk stable/train: Run fetch-subunit-output role conditionally https://review.opendev.org/714770
12:40:16	openstackgerrit	Igor Malinovskiy proposed openstack/python-openstackclient master: Add 'address_scope' type support to network rbac commands https://review.opendev.org/709124
12:40:37	openstackgerrit	Jesse Pretorius (odyssey4me) proposed openstack/openstacksdk stable/stein: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715193
12:40:40	sshnaidm	odyssey4me, train and stein should be fine, rocky is waiting for pep8 fix
12:41:15	openstackgerrit	Jesse Pretorius (odyssey4me) proposed openstack/openstacksdk stable/train: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715192
12:41:16	openstackgerrit	Igor Malinovskiy proposed openstack/python-openstackclient master: Add 'subnetpool' type support to rbac commands https://review.opendev.org/712705
12:41:31	tremble	mordred, zo/
12:41:32	odyssey4me	sshnaidm: either way works - I've picked them already
12:41:34	tremble	mordred, \o/
12:42:11	tremble	mordred, Anything I can watch to see it go through?
12:42:56	mordred	tremble: https://review.opendev.org/#/c/715090/
12:43:03	mordred	although it looks like it just went through
12:44:03	mordred	tremble: there's release jobs in https://zuul.opendev.org/t/openstack/status for the openstack/releases project that will actually publish to pypi
12:45:44	tremble	Will the test jobs pick it up already?
12:47:26	openstackgerrit	Sean McGinnis proposed openstack/openstacksdk master: Raise hacking to 2.x https://review.opendev.org/715155
12:48:06	smcginnis	tremble: If you filter on "release", there is a job that just started in teh release-post queue.
12:48:19	smcginnis	That will tag the repo, among other things.
12:48:37	smcginnis	Once that tagging happens, that triggers yet another job to run that will package it up and publish to pypi.
12:48:55	smcginnis	So probably 15-20 minutes or so until it's published and available.
12:49:57	tremble	Jobs triggering jobs triggering jobs. That looks like the kind of crazy I'd set up.
12:51:08	smcginnis	;)
12:58:48	sshnaidm	mordred, can we merge? https://review.opendev.org/#/c/714771/ odyssey4me set his backport on top of that
13:00:08	mordred	sshnaidm: done
13:00:29	sshnaidm	mordred, thanks, odyssey4me ^^
13:00:41	odyssey4me	np :)
13:05:13	openstackgerrit	Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512
13:15:01	openstackgerrit	Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_mapping https://review.opendev.org/713643
13:15:02	openstackgerrit	Mark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686
13:15:03	openstackgerrit	Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632
13:15:04	openstackgerrit	Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633
13:15:05	openstackgerrit	Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991
13:15:06	openstackgerrit	Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992
13:28:37	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add link to image sharing api docs https://review.opendev.org/677266
13:47:18	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Add libc6-dev to bindep https://review.opendev.org/715210
13:50:14	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Be explicit about python version in image https://review.opendev.org/715211
13:50:50	mordred	amoralej|lunch: ^^ the libc6-dev patch should fix the https://review.opendev.org/#/c/697444/ patch
13:51:06	mordred	gah
13:51:13	mordred	amotoki: ^^
13:51:24	mordred	amoralej|lunch: sorry - bad tab-complete
13:51:41	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
14:28:51	gtema	nodepool builds seem to be failing everywhere in SDK
14:29:01	gtema	I mean nodepool jobs
14:30:17	mordred	there's an issue in setuptools
14:30:22	mordred	that has been fixed
14:30:25	gtema	ah, ok
14:30:42	mordred	but it's gonna be a minute before it winds its way through the layers of things to dib
14:30:56	gtema	oki, keep waiting ;)
14:30:59	mordred	(the bug caused non-entrypoint scripts to lose their executable bit when they were installed)
14:31:12	gtema	cool
14:31:17	mordred	yeah. so awesome
14:31:51	smcginnis	Hah, oops.
14:31:53	gtema	but from our logs you don't see this really - just that it failed
14:55:38	mordred	gtema_: have a sec for 2 easy patches: https://review.opendev.org/#/c/715210/ https://review.opendev.org/#/c/715211/ ?
14:58:00	gtema	yupp, looking now
14:59:30	mordred	gtema: thanks!
14:59:45	gtema	should I +A for the py3.7?
15:00:03	mordred	gtema: sure. it's a pretty mechanical change
15:00:13	gtema	done
15:00:18	mordred	I think "let's upgrade to 3.8" would be one where we shoudl actually discuss :)
15:00:29	gtema	agree
15:01:12	gtema	btw this libc - today figured out, that in my zuul F30 image does not include by default bzip2 - pretty weird
15:01:34	mordred	gtema: yeah - I think we just had a patch for that in zuul
15:02:11	mordred	oh even weirder ...
15:02:12	gtema	can you please point me to it? I just did a dirty hack to keep it running
15:02:16	mordred	https://review.opendev.org/#/c/715041/
15:02:25	mordred	but the issue was that it was only installing for rpm
15:02:32	mordred	but I'd expect it to be installed on F30
15:02:54	mordred	oh - wait ...
15:03:02	gtema	yeah, should have been there

Earlier   Later