| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-06-22 | |||
| 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 | mordred | 2.8 isn't EOL I don't think | |
| 13:55:54 | gtema | no, why should they | |
| 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 | |
| 13:59:51 | sshnaidm|ruck | mordred, btw, these patches seem to need more love https://review.opendev.org/#/c/730360/ | |
| 13:59:54 | gtema | I am unfortunately having much of those | |
| 14:00:10 | sshnaidm|ruck | gtema, using collection is possible from 2.8 | |
| 14:00:21 | sshnaidm|ruck | do you have 2.7 still somewhere? | |
| 14:00:38 | gtema | no, but in 2.8 it is rather a "hack" | |
| 14:00:53 | mordred | yeah - ansible-galaxy install doesn't work in 2.8 | |
| 14:01:17 | gtema | I think it was also "ansible-galaxy role install vs ansible-galaxy collection install" | |
| 14:01:19 | mordred | I don't expect people to really start using collections until they're on 2.9 at least | |
| 14:01:41 | mordred | because of the ansible-galaxy thing | |
| 14:02:01 | gtema | and then, there is "ansible-galaxy COLLECTION publish", at least according to 2.10 docs | |
| 14:03:15 | gtema | and there is no way to upload role using cli from what I see | |
| 14:03:35 | mordred | oh - right | |
| 14:03:44 | mordred | that's one of the issues with pre-collection galaxy | |
| 14:03:47 | mordred | it's tied to github | |
| 14:03:51 | mordred | for publication | |
| 14:03:57 | mordred | it can only import from github | |
| 14:04:07 | mordred | this is one of the big wins with collections | |
| 14:05:28 | gtema | okay, will need to "consider" forcing to move to collections | |
| 14:15:54 | gtema | hmm, devstack is still broken? openstacksdk-functional-devstack-networking doesn't pass still | |
| 14:17:29 | mordred | gtema: let me look - devstack should work at this point | |
| 14:17:39 | gtema | https://f8bf2d4a845608449589-8f1f171af5507b30533733754e7f1ad8.ssl.cf5.rackcdn.com/735336/3/check/openstacksdk-functional-devstack-networking/718c56c/job-output.txt | |
| 14:21:22 | frickler | No 'neutron.service_plugins' driver found, looking for 'firewall_v2' | |
| 14:24:15 | mordred | oh -didn't fwaas get retired? | |
| 14:24:27 | frickler | that has been retired, seems we need to pull it from the job definition, yes | |
| 14:24:30 | frickler | slaweq: ^^ | |
| 14:24:32 | mordred | https://review.opendev.org/#/c/735858/ | |
| 14:24:35 | mordred | frickler, gtema ^^ | |
| 14:25:05 | mordred | now - I'm not sure why the nodepool job failed in gtema's change | |
| 14:25:16 | mordred | since ensure-virtualenv was added to it | |
| 14:25:54 | gtema | didn't I say I hate Mondays? | |
| 14:25:57 | mordred | yeah | |
| 14:26:04 | mordred | https://review.opendev.org/#/c/737130/ <-- that added ensure-virtualenv in nodepool | |
| 14:26:12 | slaweq | frickler: sorry, I'm in the meeting | |
| 14:26:26 | slaweq | frickler: as mordred mentioned, there is patch for that already | |
| 14:26:52 | mordred | slaweq: now we just have to get it landed :) | |
| 14:27:06 | gtema | +A'ed | |
| 14:27:14 | slaweq | mordred: gtema thx | |
| 14:30:51 | gtema | mordred, in my change nodepool passed in the last recheck (which is still running) | |
| 14:31:37 | mordred | gtema: oh good | |
| 14:32:02 | gtema | the status is from 19th, and venv merged on 20th | |
| 14:32:09 | mordred | gtema: so the recheck of slaweq's change should work | |
| 14:32:20 | gtema | yupp, fingers crossed | |
| 14:34:21 | openstackgerrit | Merged openstack/keystoneauth master: Implement HTTP Basic client support in keystoneauth1 https://review.opendev.org/727562 | |
| 14:52:39 | openstackgerrit | Merged openstack/openstacksdk stable/ussuri: Update docs to work with newer Sphinx https://review.opendev.org/737083 | |
| 14:59:09 | mordred | dtantsur: ^^ http basic client landed | |