openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration https://review.opendev.org/736946 | 00:56 |
---|---|---|
*** __ministry has joined #openstack-sdks | 02:06 | |
*** __ministry has quit IRC | 02:16 | |
*** __ministry has joined #openstack-sdks | 02:31 | |
*** slaweq has joined #openstack-sdks | 04:20 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #openstack-sdks | 04:33 | |
*** rpittau|afk is now known as rpittau | 06:21 | |
*** ttsiouts has joined #openstack-sdks | 07:11 | |
*** gtema has joined #openstack-sdks | 07:13 | |
openstackgerrit | Lewis Denny proposed openstack/python-openstackclient master: Add API check for server_groups.create https://review.opendev.org/736104 | 07:13 |
*** ttsiouts has quit IRC | 07:14 | |
*** ttsiouts has joined #openstack-sdks | 07:16 | |
*** stingrayza has joined #openstack-sdks | 07:23 | |
*** also_stingrayza has quit IRC | 07:25 | |
*** tosky has joined #openstack-sdks | 07:29 | |
*** sshnaidm|off is now known as sshnaidm|ruck | 07:29 | |
*** factor has quit IRC | 07:45 | |
openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI https://review.opendev.org/737226 | 07:46 |
*** jpena|off is now known as jpena | 07:56 | |
*** jpich has joined #openstack-sdks | 07:57 | |
*** nightmare_unreal has joined #openstack-sdks | 07:58 | |
*** ralonsoh has joined #openstack-sdks | 07:59 | |
openstackgerrit | jayaditya gupta proposed openstack/cliff master: Output correct json for security groups in 'openstack server show' https://review.opendev.org/735128 | 08:01 |
*** dtantsur|afk is now known as dtantsur | 08:10 | |
*** BakaKuna has quit IRC | 08:18 | |
*** dayou_ has joined #openstack-sdks | 08:36 | |
*** dayou has quit IRC | 08:39 | |
openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI https://review.opendev.org/737226 | 08:44 |
*** ttsiouts has quit IRC | 09:16 | |
*** ttsiouts has joined #openstack-sdks | 09:18 | |
*** tkajinam has quit IRC | 09:21 | |
*** ttsiouts has quit IRC | 09:55 | |
*** rpittau is now known as rpittau|bbl | 10:15 | |
*** ttsiouts has joined #openstack-sdks | 10:29 | |
*** ttsiouts has quit IRC | 10:30 | |
*** ttsiouts has joined #openstack-sdks | 10:31 | |
*** gtema has quit IRC | 10:50 | |
*** ttsiouts has quit IRC | 10:54 | |
*** ttsiouts has joined #openstack-sdks | 10:55 | |
dtantsur | mordred: hey, could you please check https://review.opendev.org/#/c/727562/ again when you have a minute? | 11:17 |
*** jpena is now known as jpena|lunch | 11:29 | |
*** gtema has joined #openstack-sdks | 11:30 | |
*** gtema has quit IRC | 11:35 | |
*** gtema has joined #openstack-sdks | 11:39 | |
*** ttsiouts has quit IRC | 11:42 | |
*** ttsiouts has joined #openstack-sdks | 11:44 | |
*** enriquetaso has joined #openstack-sdks | 12:05 | |
*** jpich has quit IRC | 12:14 | |
*** jpich has joined #openstack-sdks | 12:15 | |
*** rpittau|bbl is now known as rpittau | 12:22 | |
*** jpena|lunch is now known as jpena | 12:32 | |
*** lbragstad has joined #openstack-sdks | 13:12 | |
mordred | dtantsur: ooh, it's green! | 13:15 |
dtantsur | :) | 13:15 |
dtantsur | happens, yeah? | 13:16 |
mordred | :) | 13:16 |
mordred | I think that's a super cool patch | 13:16 |
gtema | what is green? everything? we can recheck? I don't believe - it's monday | 13:17 |
mordred | gtema: quick, just approve things while it lasts ;) | 13:19 |
gtema | nah, let's see on my project cleanup filters patch :D | 13:20 |
mordred | heh | 13:22 |
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:22 |
mordred | gtema: I do not wish this strongly enough to make that patch though | 13:23 |
mordred | because *wow* that would be a big patch | 13:23 |
mordred | I think it would make way the hell more sense code-wise though | 13:23 |
gtema | hm, you make me think 2 times and yet again and again | 13:24 |
gtema | we live in the world of huge patches | 13:25 |
gtema | that you simply don't have a place anymore, where a tiny change would be immediately affective everywhere | 13:26 |
gtema | we have sich a spaghetti of interfaces | 13:26 |
gtema | such | 13:26 |
mordred | yeah | 13:30 |
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:31 |
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:32 |
gtema | oh this encryption, this is really fun | 13:38 |
mordred | yeah | 13:38 |
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 |
mordred | BUT | 13:39 |
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 |
gtema | wow | 13:40 |
mordred | yeah | 13:40 |
mordred | it's quite the rabbit hole | 13:41 |
gtema | I knew it's Monday, but not that ... | 13:41 |
mordred | haha | 13:41 |
gtema | btw mordred, do we have somewhere a Zuul role to upload roles to Galaxy? I didn't find any | 13:45 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures https://review.opendev.org/737135 | 13:52 |
mordred | gtema: sshnaidm|ruck is working on some roles to upload collections | 13:53 |
gtema | yes, but those are for collections, and I was wondering about roles | 13:53 |
mordred | gtema: https://review.opendev.org/#/c/730360/ | 13:53 |
mordred | gtema: not that I know of | 13:53 |
gtema | okay, will need to make own one then | 13:54 |
sshnaidm|ruck | gtema, it should be the same | 13:54 |
mordred | I imagine it should be easy to adapt the collection ones | 13:54 |
sshnaidm|ruck | ansible-galaxy publish ... | 13:54 |
gtema | really? | 13:54 |
sshnaidm|ruck | gtema, and why to upload roles only if you can make a collection from them | 13:54 |
mordred | sshnaidm|ruck: maybe we should make ansible-collection-publish ansible-galaxy-publish - | 13:55 |
mordred | (maybe that's not worth it with the world moving to collections though) | 13:55 |
sshnaidm|ruck | gtema, mordred, I don't think uploading roles to galaxy is a thing now | 13:55 |
sshnaidm|ruck | just format it as a collection and that's it | 13:55 |
mordred | sshnaidm|ruck: I don't think they removed it yet did they? | 13:55 |
gtema | no, why should they | 13:55 |
mordred | 2.8 isn't EOL I don't think | 13:55 |
sshnaidm|ruck | mordred, no, but it doesn't make sense upload a pure role | 13:56 |
mordred | it does if you're not using a collection-aware ansible :) | 13:56 |
sshnaidm|ruck | mordred, well, from 2.8 | 13:56 |
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:56 |
mordred | I mena - I know we're _eventually_ going to all migrate to collections, but it's going to take some time | 13:57 |
gtema | there is half of internet of articles about roles ;-) | 13:57 |
sshnaidm|ruck | mordred, yeah, that's about migration, but if you start from scratch - better to create a collection | 13:57 |
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 |
sshnaidm|ruck | gtema, worth to add to them galaxy.yml and voila, you have a collection :) | 13:58 |
gtema | and all the users not using latest ansible are cut-off | 13:59 |
gtema | that's my problem | 13:59 |
sshnaidm|ruck | mordred, btw, these patches seem to need more love https://review.opendev.org/#/c/730360/ | 13:59 |
gtema | I am unfortunately having much of those | 13:59 |
sshnaidm|ruck | gtema, using collection is possible from 2.8 | 14:00 |
sshnaidm|ruck | do you have 2.7 still somewhere? | 14:00 |
gtema | no, but in 2.8 it is rather a "hack" | 14:00 |
mordred | yeah - ansible-galaxy install doesn't work in 2.8 | 14:00 |
gtema | I think it was also "ansible-galaxy role install vs ansible-galaxy collection install" | 14:01 |
mordred | I don't expect people to really start using collections until they're on 2.9 at least | 14:01 |
mordred | because of the ansible-galaxy thing | 14:01 |
gtema | and then, there is "ansible-galaxy COLLECTION publish", at least according to 2.10 docs | 14:02 |
gtema | and there is no way to upload role using cli from what I see | 14:03 |
mordred | oh - right | 14:03 |
mordred | that's one of the issues with pre-collection galaxy | 14:03 |
mordred | it's tied to github | 14:03 |
mordred | for publication | 14:03 |
mordred | it can only import from github | 14:03 |
mordred | this is one of the big wins with collections | 14:04 |
gtema | okay, will need to "consider" forcing to move to collections | 14:05 |
gtema | hmm, devstack is still broken? openstacksdk-functional-devstack-networking doesn't pass still | 14:15 |
mordred | gtema: let me look - devstack should work at this point | 14:17 |
gtema | https://f8bf2d4a845608449589-8f1f171af5507b30533733754e7f1ad8.ssl.cf5.rackcdn.com/735336/3/check/openstacksdk-functional-devstack-networking/718c56c/job-output.txt | 14:17 |
frickler | No 'neutron.service_plugins' driver found, looking for 'firewall_v2' | 14:21 |
mordred | oh -didn't fwaas get retired? | 14:24 |
frickler | that has been retired, seems we need to pull it from the job definition, yes | 14:24 |
frickler | slaweq: ^^ | 14:24 |
mordred | https://review.opendev.org/#/c/735858/ | 14:24 |
mordred | frickler, gtema ^^ | 14:24 |
mordred | now - I'm not sure why the nodepool job failed in gtema's change | 14:25 |
mordred | since ensure-virtualenv was added to it | 14:25 |
gtema | didn't I say I hate Mondays? | 14:25 |
mordred | yeah | 14:25 |
mordred | https://review.opendev.org/#/c/737130/ <-- that added ensure-virtualenv in nodepool | 14:26 |
slaweq | frickler: sorry, I'm in the meeting | 14:26 |
slaweq | frickler: as mordred mentioned, there is patch for that already | 14:26 |
mordred | slaweq: now we just have to get it landed :) | 14:26 |
gtema | +A'ed | 14:27 |
slaweq | mordred: gtema thx | 14:27 |
gtema | mordred, in my change nodepool passed in the last recheck (which is still running) | 14:30 |
mordred | gtema: oh good | 14:31 |
gtema | the status is from 19th, and venv merged on 20th | 14:32 |
mordred | gtema: so the recheck of slaweq's change should work | 14:32 |
gtema | yupp, fingers crossed | 14:32 |
openstackgerrit | Merged openstack/keystoneauth master: Implement HTTP Basic client support in keystoneauth1 https://review.opendev.org/727562 | 14:34 |
openstackgerrit | Merged openstack/openstacksdk stable/ussuri: Update docs to work with newer Sphinx https://review.opendev.org/737083 | 14:52 |
mordred | dtantsur: ^^ http basic client landed | 14:59 |
dtantsur | \o/ | 14:59 |
mordred | dtantsur: there's one more patch in the gate (simple one, it's about raising lower constraints) - once that's in, I think we should suggest a release | 15:01 |
dtantsur | +++ | 15:01 |
openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Drop python 3.5 support https://review.opendev.org/737285 | 15:06 |
mordred | lbragstad: ^^ we dropped 3.5 in sdk already, so I think it's fine to land that now | 15:06 |
lbragstad | mordred makes sense, thank you | 15:07 |
*** gtema has quit IRC | 15:20 | |
*** gtema has joined #openstack-sdks | 15:20 | |
*** gtema has quit IRC | 15:49 | |
*** rpittau is now known as rpittau|afk | 16:01 | |
*** dtantsur is now known as dtantsur|afk | 16:10 | |
*** gtema has joined #openstack-sdks | 16:10 | |
*** ttsiouts has quit IRC | 16:10 | |
*** sshnaidm|ruck is now known as sshnaidm|afk | 16:13 | |
openstackgerrit | Merged openstack/keystoneauth master: Update lower-constraints versions https://review.opendev.org/734803 | 16:43 |
openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI https://review.opendev.org/737226 | 16:43 |
*** ttsiouts has joined #openstack-sdks | 16:50 | |
*** diablo_rojo has joined #openstack-sdks | 16:57 | |
*** jpich has quit IRC | 16:58 | |
*** senrique_ has joined #openstack-sdks | 17:05 | |
*** ttsiouts has quit IRC | 17:05 | |
*** enriquetaso has quit IRC | 17:07 | |
*** gtema has quit IRC | 17:16 | |
*** jpena is now known as jpena|off | 17:23 | |
*** ttsiouts has joined #openstack-sdks | 17:37 | |
*** ttsiouts has quit IRC | 17:43 | |
*** ralonsoh has quit IRC | 17:47 | |
*** ttsiouts has joined #openstack-sdks | 18:15 | |
*** senrique__ has joined #openstack-sdks | 18:42 | |
*** senrique_ has quit IRC | 18:44 | |
*** ttsiouts has quit IRC | 18:47 | |
*** gtema has joined #openstack-sdks | 19:17 | |
*** nightmare_unreal has quit IRC | 19:19 | |
openstackgerrit | Merged openstack/keystoneauth master: Drop python 3.5 support https://review.opendev.org/737285 | 19:19 |
*** gtema has quit IRC | 19:22 | |
*** slaweq has quit IRC | 19:46 | |
*** ttsiouts has joined #openstack-sdks | 19:55 | |
*** slaweq has joined #openstack-sdks | 19:58 | |
*** ttsiouts has quit IRC | 20:26 | |
*** factor has joined #openstack-sdks | 21:17 | |
openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI https://review.opendev.org/737226 | 21:30 |
*** ttsiouts has joined #openstack-sdks | 21:32 | |
*** ttsiouts_ has joined #openstack-sdks | 21:35 | |
*** ttsiouts has quit IRC | 21:37 | |
*** ttsiouts_ has quit IRC | 21:39 | |
openstackgerrit | Slawek Kaplonski proposed openstack/openstacksdk master: Remove neutron-fwaas from the jobs' required project https://review.opendev.org/735858 | 21:48 |
*** slaweq has quit IRC | 22:07 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk stable/ussuri: Fix issues found by latest flake8 https://review.opendev.org/730332 | 22:17 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk stable/ussuri: Don't error if clouds.yaml is not readable https://review.opendev.org/726962 | 22:18 |
*** slaweq has joined #openstack-sdks | 22:23 | |
*** tkajinam has joined #openstack-sdks | 22:51 | |
*** senrique__ has quit IRC | 22:58 | |
*** tosky has quit IRC | 22:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!