Monday, 2020-06-22

openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: New CI jobs configuration  https://review.opendev.org/73694600:56
*** __ministry has joined #openstack-sdks02:06
*** __ministry has quit IRC02:16
*** __ministry has joined #openstack-sdks02:31
*** slaweq has joined #openstack-sdks04:20
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-sdks04:33
*** rpittau|afk is now known as rpittau06:21
*** ttsiouts has joined #openstack-sdks07:11
*** gtema has joined #openstack-sdks07:13
openstackgerritLewis Denny proposed openstack/python-openstackclient master: Add API check for server_groups.create  https://review.opendev.org/73610407:13
*** ttsiouts has quit IRC07:14
*** ttsiouts has joined #openstack-sdks07:16
*** stingrayza has joined #openstack-sdks07:23
*** also_stingrayza has quit IRC07:25
*** tosky has joined #openstack-sdks07:29
*** sshnaidm|off is now known as sshnaidm|ruck07:29
*** factor has quit IRC07:45
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI  https://review.opendev.org/73722607:46
*** jpena|off is now known as jpena07:56
*** jpich has joined #openstack-sdks07:57
*** nightmare_unreal has joined #openstack-sdks07:58
*** ralonsoh has joined #openstack-sdks07:59
openstackgerritjayaditya gupta proposed openstack/cliff master: Output correct json for security groups in 'openstack server show'  https://review.opendev.org/73512808:01
*** dtantsur|afk is now known as dtantsur08:10
*** BakaKuna has quit IRC08:18
*** dayou_ has joined #openstack-sdks08:36
*** dayou has quit IRC08:39
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI  https://review.opendev.org/73722608:44
*** ttsiouts has quit IRC09:16
*** ttsiouts has joined #openstack-sdks09:18
*** tkajinam has quit IRC09:21
*** ttsiouts has quit IRC09:55
*** rpittau is now known as rpittau|bbl10:15
*** ttsiouts has joined #openstack-sdks10:29
*** ttsiouts has quit IRC10:30
*** ttsiouts has joined #openstack-sdks10:31
*** gtema has quit IRC10:50
*** ttsiouts has quit IRC10:54
*** ttsiouts has joined #openstack-sdks10:55
dtantsurmordred: 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|lunch11:29
*** gtema has joined #openstack-sdks11:30
*** gtema has quit IRC11:35
*** gtema has joined #openstack-sdks11:39
*** ttsiouts has quit IRC11:42
*** ttsiouts has joined #openstack-sdks11:44
*** enriquetaso has joined #openstack-sdks12:05
*** jpich has quit IRC12:14
*** jpich has joined #openstack-sdks12:15
*** rpittau|bbl is now known as rpittau12:22
*** jpena|lunch is now known as jpena12:32
*** lbragstad has joined #openstack-sdks13:12
mordreddtantsur: ooh, it's green!13:15
dtantsur:)13:15
dtantsurhappens, yeah?13:16
mordred:)13:16
mordredI think that's a super cool patch13:16
gtemawhat is green? everything? we can recheck? I don't believe - it's monday13:17
mordredgtema: quick, just approve things while it lasts ;)13:19
gtemanah, let's see on my project cleanup filters patch :D13:20
mordredheh13:22
mordredgtema: 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 object13:22
mordredgtema: I do not wish this strongly enough to make that patch though13:23
mordredbecause *wow* that would be a big patch13:23
mordredI think it would make way the hell more sense code-wise though13:23
gtemahm, you make me think 2 times and yet again and again13:24
gtemawe live in the world of huge patches13:25
gtemathat you simply don't have a place anymore, where a tiny change would be immediately affective everywhere13:26
gtemawe have sich a spaghetti of interfaces13:26
gtemasuch13:26
mordredyeah13:30
mordredgtema: 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
mordredgtema: *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 things13:32
gtemaoh this encryption, this is really fun13:38
mordredyeah13:38
mordredturns 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 objects13:39
mordredBUT13:39
mordredI 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 stuff13:40
gtemawow13:40
mordredyeah13:40
mordredit's quite the rabbit hole13:41
gtemaI knew it's Monday, but not that ...13:41
mordredhaha13:41
gtemabtw mordred, do we have somewhere a Zuul role to upload roles to Galaxy? I didn't find any13:45
openstackgerritMonty Taylor proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures  https://review.opendev.org/73713513:52
mordredgtema: sshnaidm|ruck is working on some roles to upload collections13:53
gtemayes, but those are for collections, and I was wondering about roles13:53
mordredgtema: https://review.opendev.org/#/c/730360/13:53
mordredgtema: not that I know of13:53
gtemaokay, will need to make own one then13:54
sshnaidm|ruckgtema, it should be the same13:54
mordredI imagine it should be easy to adapt the collection ones13:54
sshnaidm|ruckansible-galaxy publish ...13:54
gtemareally?13:54
sshnaidm|ruckgtema, and why to upload roles only if you can make a collection from them13:54
mordredsshnaidm|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|ruckgtema, mordred, I don't think uploading roles to galaxy is a thing now13:55
sshnaidm|ruckjust format it as a collection and that's it13:55
mordredsshnaidm|ruck: I don't think they removed it yet did they?13:55
gtemano, why should they13:55
mordred2.8 isn't EOL I don't think13:55
sshnaidm|ruckmordred, no, but it doesn't make sense upload a pure role13:56
mordredit does if you're not using a collection-aware ansible :)13:56
sshnaidm|ruckmordred, well, from 2.813:56
mordredor 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 far13:56
mordredI mena - I know we're _eventually_ going to all migrate to collections, but it's going to take some time13:57
gtemathere is half of internet of articles about roles ;-)13:57
sshnaidm|ruckmordred, yeah, that's about migration, but if you start from scratch - better to create a collection13:57
gtemait's clear, I wanted to start publishing my roles which exist since years with Zuul, that's why I was asking13:58
sshnaidm|ruckgtema, worth to add to them galaxy.yml and voila, you have a collection :)13:58
gtemaand all the users not using latest ansible are cut-off13:59
gtemathat's my problem13:59
sshnaidm|ruckmordred, btw, these patches seem to need more love https://review.opendev.org/#/c/730360/13:59
gtemaI am unfortunately having much of those13:59
sshnaidm|ruckgtema, using collection is possible from 2.814:00
sshnaidm|ruckdo you have 2.7 still somewhere?14:00
gtemano, but in 2.8 it is rather a "hack"14:00
mordredyeah - ansible-galaxy install doesn't work in 2.814:00
gtemaI think it was also "ansible-galaxy role install vs ansible-galaxy collection install"14:01
mordredI don't expect people to really start using collections until they're on 2.9 at least14:01
mordredbecause of the ansible-galaxy thing14:01
gtemaand then, there is "ansible-galaxy COLLECTION publish", at least according to 2.10 docs14:02
gtemaand there is no way to upload role using cli from what I see14:03
mordredoh - right14:03
mordredthat's one of the issues with pre-collection galaxy14:03
mordredit's tied to github14:03
mordredfor publication14:03
mordredit can only import from github14:03
mordredthis is one of the big wins with collections14:04
gtemaokay, will need to "consider" forcing to move to collections14:05
gtemahmm, devstack is still broken? openstacksdk-functional-devstack-networking doesn't pass still14:15
mordredgtema: let me look - devstack should work at this point14:17
gtemahttps://f8bf2d4a845608449589-8f1f171af5507b30533733754e7f1ad8.ssl.cf5.rackcdn.com/735336/3/check/openstacksdk-functional-devstack-networking/718c56c/job-output.txt14:17
fricklerNo 'neutron.service_plugins' driver found, looking for 'firewall_v2'14:21
mordredoh -didn't fwaas get retired?14:24
fricklerthat has been retired, seems we need to pull it from the job definition, yes14:24
fricklerslaweq: ^^14:24
mordredhttps://review.opendev.org/#/c/735858/14:24
mordredfrickler, gtema ^^14:24
mordrednow - I'm not sure why the nodepool job failed in gtema's change14:25
mordredsince ensure-virtualenv was added to it14:25
gtemadidn't I say I hate Mondays?14:25
mordredyeah14:25
mordredhttps://review.opendev.org/#/c/737130/ <-- that added ensure-virtualenv in nodepool14:26
slaweqfrickler: sorry, I'm in the meeting14:26
slaweqfrickler: as mordred mentioned, there is patch for that already14:26
mordredslaweq: now we just have to get it landed :)14:26
gtema+A'ed14:27
slaweqmordred: gtema thx14:27
gtemamordred, in my change nodepool passed in the last recheck (which is still running)14:30
mordredgtema: oh good14:31
gtemathe status is from 19th, and venv merged on 20th14:32
mordredgtema: so the recheck of slaweq's change should work14:32
gtemayupp, fingers crossed14:32
openstackgerritMerged openstack/keystoneauth master: Implement HTTP Basic client support in keystoneauth1  https://review.opendev.org/72756214:34
openstackgerritMerged openstack/openstacksdk stable/ussuri: Update docs to work with newer Sphinx  https://review.opendev.org/73708314:52
mordreddtantsur: ^^ http basic client landed14:59
dtantsur\o/14:59
mordreddtantsur: 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 release15:01
dtantsur+++15:01
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Drop python 3.5 support  https://review.opendev.org/73728515:06
mordredlbragstad: ^^ we dropped 3.5 in sdk already, so I think it's fine to land that now15:06
lbragstadmordred makes sense, thank you15:07
*** gtema has quit IRC15:20
*** gtema has joined #openstack-sdks15:20
*** gtema has quit IRC15:49
*** rpittau is now known as rpittau|afk16:01
*** dtantsur is now known as dtantsur|afk16:10
*** gtema has joined #openstack-sdks16:10
*** ttsiouts has quit IRC16:10
*** sshnaidm|ruck is now known as sshnaidm|afk16:13
openstackgerritMerged openstack/keystoneauth master: Update lower-constraints versions  https://review.opendev.org/73480316:43
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI  https://review.opendev.org/73722616:43
*** ttsiouts has joined #openstack-sdks16:50
*** diablo_rojo has joined #openstack-sdks16:57
*** jpich has quit IRC16:58
*** senrique_ has joined #openstack-sdks17:05
*** ttsiouts has quit IRC17:05
*** enriquetaso has quit IRC17:07
*** gtema has quit IRC17:16
*** jpena is now known as jpena|off17:23
*** ttsiouts has joined #openstack-sdks17:37
*** ttsiouts has quit IRC17:43
*** ralonsoh has quit IRC17:47
*** ttsiouts has joined #openstack-sdks18:15
*** senrique__ has joined #openstack-sdks18:42
*** senrique_ has quit IRC18:44
*** ttsiouts has quit IRC18:47
*** gtema has joined #openstack-sdks19:17
*** nightmare_unreal has quit IRC19:19
openstackgerritMerged openstack/keystoneauth master: Drop python 3.5 support  https://review.opendev.org/73728519:19
*** gtema has quit IRC19:22
*** slaweq has quit IRC19:46
*** ttsiouts has joined #openstack-sdks19:55
*** slaweq has joined #openstack-sdks19:58
*** ttsiouts has quit IRC20:26
*** factor has joined #openstack-sdks21:17
openstackgerritSagi Shnaidman proposed openstack/ansible-collections-openstack master: DNM test new CI  https://review.opendev.org/73722621:30
*** ttsiouts has joined #openstack-sdks21:32
*** ttsiouts_ has joined #openstack-sdks21:35
*** ttsiouts has quit IRC21:37
*** ttsiouts_ has quit IRC21:39
openstackgerritSlawek Kaplonski proposed openstack/openstacksdk master: Remove neutron-fwaas from the jobs' required project  https://review.opendev.org/73585821:48
*** slaweq has quit IRC22:07
openstackgerritMonty Taylor proposed openstack/openstacksdk stable/ussuri: Fix issues found by latest flake8  https://review.opendev.org/73033222:17
openstackgerritMonty Taylor proposed openstack/openstacksdk stable/ussuri: Don't error if clouds.yaml is not readable  https://review.opendev.org/72696222:18
*** slaweq has joined #openstack-sdks22:23
*** tkajinam has joined #openstack-sdks22:51
*** senrique__ has quit IRC22:58
*** tosky has quit IRC22:59

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!