Thursday, 2023-01-12

*** soniya29 is now known as soniya29|afk07:16
*** jpena|off is now known as jpena08:37
Tengufun..... cache doesn't seem to work for ansible-galaxy, even though the proxy handles everything (locally - I'm digging deeper in the whole config). 11:02
*** rlandy|out is now known as rlandy11:11
ade_leeclarkb, hey - could you take a look at the fips patches please?12:28
ade_leefungi, clarkb - do you guys know what would be causing failures like - https://zuul.opendev.org/t/openstack/build/11c8ca93532c4dd9a4a49ab10a6585d4/log/tox/py310/1-commands[0].log ?12:29
fungiade_lee: are all those keystone versioned endpoint errors earlier in the log expected?12:48
fungiERROR [castellan.key_manager.barbican_key_manager] Error creating Barbican client: Could not find versioned identity endpoints when attempting to authenticate. Please check that your auth_url is correct. Unable to establish connection to http://localhost/identity/v3: HTTPConnectionPool(host='localhost', port=80): Max retries exceeded with url: /identity/v3 (Caused by12:49
fungiNewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fa25f6be2c0>: Failed to establish a new connection: [Errno 111] Connection refused'))12:49
ade_leefungi, yeah - thanks - I think we figured out that the tox change was somehow causing this12:54
ade_leefungi, this should fix it -- https://review.opendev.org/c/openstack/castellan/+/86997012:55
ade_leefungi, saw you reviewed the fips change - thanks!12:55
fungioh, yeah skipsdist under tox v4 causes the project you're testing to not be installed in the venv12:56
ade_leeah, that explains it12:56
fungiit was the same under tox v3, except that if you also turned on usedevelop then tox ended up installing the project anyway (but in editable mode)13:02
fungithe difference under tox v4 is that skipsdist causes the project not to be installed even if usedevelop is on13:03
fungiso lots of openstack projects are seeing this because they set both skipsdist and usedevelop13:03
fungithe tox maintainers consider it an oversight that usedevelop overrode skipsdist and caused the project to get installed under v313:04
fungiso didn't consider that a breaking change, just a correction of earlier undocumented misbehavior13:04
*** blarnath is now known as d34dh0r5313:05
*** dasm|off is now known as dasm13:10
opendevreviewRiccardo Pittau proposed openstack/project-config master: Reduce virtualpdu tests  https://review.opendev.org/c/openstack/project-config/+/86997913:45
Tengufungi: heya! updated the commit message to better reflect the change13:54
Tengushould be better :)13:54
Tengufungi: also, fyi, I'm still fighting my way with the ansible-galaxy proxy, there were still some issues (as discussed during last December). I think I'm almost done, just some small tweaks to make CI pass, but I could get it work locally at least.13:55
fungioh that's awesome!13:58
Tenguit was a nightmare13:59
Tenguwell it's still is13:59
Tengubut I think I'm seeing some light at the end of the tunnel.13:59
Tenguhopefully, it's not a train....... ;)13:59
fungithis would be good feedback for your colleagues who design and operate that service too14:00
Tenguredirections are terrible ^^'.14:00
Tenguhttps://review.opendev.org/c/opendev/system-config/+/86981914:00
fungiyou're in an excellent position to be able to describe to them the use case and potential benefits to the ansible ecosystem of redesigning to be more proxy-friendly14:01
Tenguso, it's not THAT unfriendly. just a bit complicated.14:04
Tengubasically I lost time due to the fact I didn't use httpd for a loooong time, and never saw the mod_substitute (and its limitations).14:06
Tengufungi: regarding the other patch (default branch) - seems we may put the job to voting. But I'll let that up to your team, since I don't really know the actual usage of this job.14:19
fungithe ansible-devel job is used as an indicator of whether upcoming ansible releases are going to break our control plane system deployments, so we keep it non-voting because that tends to happen often enough that we don't want it to block our ability to merge changes14:21
Tengufair enough :). At least, now its result is meaningful.14:21
fungiyep, thanks for the fix!14:21
opendevreviewMerged openstack/project-config master: ARM64 nodes : rename larger types to be more descriptive  https://review.opendev.org/c/openstack/project-config/+/86843914:25
opendevreviewMerged openstack/project-config master: Add new linaro cloud  https://review.opendev.org/c/openstack/project-config/+/86844214:25
*** jpena is now known as jpena|off17:28
*** dasm is now known as dasm|off22:31
*** rlandy is now known as rlandy|out23:31

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