Posted	Nick	Remark
#openstack-sdks - 2020-06-19
15:56:05	AJaeger	could I get a review for https://review.opendev.org/#/c/734852/ - updating docstheme for service-types-authority, please?
15:56:16	AJaeger	mordred ^
15:58:06	mordred	AJaeger: done
15:58:55	AJaeger	thanks!
15:59:00	gtema_	mordred, do you remember what is this networking problem that blocks everything?
16:09:33	openstackgerrit	Merged openstack/service-types-authority master: Switch to newer openstackdocstheme version https://review.opendev.org/734852
17:39:41	holser	mordred Quick question. Why do we install python-dev on Ubuntu and don't install it for CentOS/RedHat? I am talking about https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/ensure-python/tasks/main.yaml#L9
17:48:50	holser	mordred opened https://storyboard.openstack.org/#!/story/2007838
18:35:19	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
21:16:09	openstackgerrit	Sean McGinnis proposed openstack/openstacksdk stable/ussuri: Update docs to work with newer Sphinx https://review.opendev.org/737083
21:16:30	openstackgerrit	Sean McGinnis proposed openstack/openstacksdk stable/ussuri: Don't error if clouds.yaml is not readable https://review.opendev.org/726962
21:17:00	openstackgerrit	Sean McGinnis proposed openstack/openstacksdk stable/ussuri: Fix issues found by latest flake8 https://review.opendev.org/730332
21:17:14	openstackgerrit	Sean McGinnis proposed openstack/openstacksdk stable/ussuri: Don't error if clouds.yaml is not readable https://review.opendev.org/726962
23:25:26	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
#openstack-sdks - 2020-06-20
00:03:15	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
00:26:50	mordred	holser: I believe that's just a point-in-time thing - nobody has gotten around to doing it for not-debuntu. it would be perfectly appropriate for it to support rh distros too - and I think since it was written there is good unittesting support in zuul-jobs, so it should be easy enough to test that the addition of that support works
06:31:15	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
13:22:02	openstackgerrit	Sean McGinnis proposed openstack/openstacksdk stable/ussuri: Update docs to work with newer Sphinx https://review.opendev.org/737083
15:33:41	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures https://review.opendev.org/737135
17:02:22	openstackgerrit	Takashi Kajinami proposed openstack/python-openstackclient master: DNM: Implement unit tests for the parent change https://review.opendev.org/737147
20:06:20	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
20:47:03	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
21:45:58	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
#openstack-sdks - 2020-06-21
04:25:46	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
06:37:16	openstackgerrit	Takashi Kajinami proposed openstack/python-openstackclient master: DNM: Implement unit tests for the parent change https://review.opendev.org/737147
07:47:01	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
10:00:15	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
12:05:19	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
12:49:51	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add volume_info module https://review.opendev.org/735949
13:21:15	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
15:18:54	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
16:18:56	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
19:09:36	openstackgerrit	Maysa de Macedo Souza proposed openstack/ansible-collections-openstack master: Add trunk module https://review.opendev.org/737190
#openstack-sdks - 2020-06-22
00:56:27	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946
07:13:27	openstackgerrit	Lewis Denny proposed openstack/python-openstackclient master: Add API check for server_groups.create https://review.opendev.org/736104
07:46:11	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI https://review.opendev.org/737226
08:01:09	openstackgerrit	jayaditya gupta proposed openstack/cliff master: Output correct json for security groups in 'openstack server show' https://review.opendev.org/735128
08:44:55	openstackgerrit	Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI https://review.opendev.org/737226
11:17:43	dtantsur	mordred: hey, could you please check https://review.opendev.org/#/c/727562/ again when you have a minute?
13:15:49	mordred	dtantsur: ooh, it's green!
13:15:59	dtantsur	:)
13:16:03	dtantsur	happens, yeah?
13:16:51	mordred	:)
13:16:57	mordred	I think that's a super cool patch
13:17:02	gtema	what is green? everything? we can recheck? I don't believe - it's monday
13:19:35	mordred	gtema: quick, just approve things while it lasts ;)
13:20:21	gtema	nah, let's see on my project cleanup filters patch :D
13:22:08	mordred	heh
13:22:54	mordred	gtema: I really wish that a) Resource methods didn't take a session argument - that it was passed to the Resource constructor and b) it wasn't a Proxy being passed to the session parameter but was just the Connection object
13:23:16	mordred	gtema: I do not wish this strongly enough to make that patch though
13:23:24	mordred	because *wow* that would be a big patch
13:23:40	mordred	I think it would make way the hell more sense code-wise though
13:24:35	gtema	hm, you make me think 2 times and yet again and again
13:25:52	gtema	we live in the world of huge patches
13:26:20	gtema	that you simply don't have a place anymore, where a tiny change would be immediately affective everywhere
13:26:42	gtema	we have sich a spaghetti of interfaces
13:26:44	gtema	such
13:30:48	mordred	yeah
13:31:57	mordred	gtema: so - shaohe_feng was asking about that cyborg thing. I officially got nerdsniped, have written most of the patches to switch them off of glanceclient (they use literally one thing, download)
13:32:44	mordred	gtema: *EXCEPT* - they do signature verification, which we don't have (we can create signatures, but have no verification code) ... going down *that* rabbit hole gets all the way to needed barbican api things
13:38:03	gtema	oh this encryption, this is really fun
13:38:49	mordred	yeah
13:39:35	mordred	turns out there is implementation that can be essentially copied for most of it ... but unfortunately can't just be used because it was written as server-side libraries assuming oslo.config and context objects
13:39:37	mordred	BUT
13:40:30	mordred	I think once we have the support added in sdk, we should be able to update castellan and cursive to remove the code we copy from them and just wrap it with oslo.config / context related stuff
13:40:50	gtema	wow
13:40:59	mordred	yeah
13:41:03	mordred	it's quite the rabbit hole
13:41:32	gtema	I knew it's Monday, but not that ...
13:41:37	mordred	haha
13:45:11	gtema	btw mordred, do we have somewhere a Zuul role to upload roles to Galaxy? I didn't find any
13:52:56	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures https://review.opendev.org/737135
13:53:16	mordred	gtema: sshnaidm|ruck is working on some roles to upload collections
13:53:39	gtema	yes, but those are for collections, and I was wondering about roles
13:53:42	mordred	gtema: https://review.opendev.org/#/c/730360/
13:53:47	mordred	gtema: not that I know of
13:54:01	gtema	okay, will need to make own one then
13:54:03	sshnaidm|ruck	gtema, it should be the same
13:54:10	mordred	I imagine it should be easy to adapt the collection ones
13:54:11	sshnaidm|ruck	ansible-galaxy publish ...
13:54:32	gtema	really?
13:54:44	sshnaidm|ruck	gtema, and why to upload roles only if you can make a collection from them
13:55:01	mordred	sshnaidm|ruck: maybe we should make ansible-collection-publish ansible-galaxy-publish -
13:55:23	mordred	(maybe that's not worth it with the world moving to collections though)
13:55:24	sshnaidm|ruck	gtema, mordred, I don't think uploading roles to galaxy is a thing now
13:55:35	sshnaidm|ruck	just format it as a collection and that's it
13:55:38	mordred	sshnaidm|ruck: I don't think they removed it yet did they?
13:55:54	gtema	no, why should they
13:55:54	mordred	2.8 isn't EOL I don't think
13:56:02	sshnaidm|ruck	mordred, no, but it doesn't make sense upload a pure role
13:56:12	mordred	it does if you're not using a collection-aware ansible :)
13:56:27	sshnaidm|ruck	mordred, well, from 2.8
13:56:50	mordred	or if you are already doing a role workflow and haven't gotten a chance to change everything you're doing - there's lots of people with roles out there so far
13:57:10	mordred	I mena - I know we're _eventually_ going to all migrate to collections, but it's going to take some time
13:57:40	gtema	there is half of internet of articles about roles ;-)
13:57:45	sshnaidm|ruck	mordred, yeah, that's about migration, but if you start from scratch - better to create a collection
13:58:29	gtema	it's clear, I wanted to start publishing my roles which exist since years with Zuul, that's why I was asking
13:58:56	sshnaidm|ruck	gtema, worth to add to them galaxy.yml and voila, you have a collection :)
13:59:23	gtema	and all the users not using latest ansible are cut-off
13:59:30	gtema	that's my problem

Earlier   Later