| noonedeadpunk | like feels that it's tough for it to unmount or smth | 08:01 |
|---|---|---|
| noonedeadpunk | folks, any good guesses wtf could be here? https://bugs.launchpad.net/openstack-ansible/+bug/2095465 | 08:10 |
| noonedeadpunk | as can not imagine anything except weird proxy for "cannot verify github.com's certificate, issued by ‘CN=Example Corp Openstack Infrastructure Intermediate CA,ST=England,C=GB’" | 08:11 |
| f0o | wow yeah that looks a lot like a proxy wrapping TLS | 08:15 |
| noonedeadpunk | ++ thanks! | 08:16 |
| jrosser | how is that even possible | 08:17 |
| noonedeadpunk | that is smth I'm not really sure | 08:17 |
| noonedeadpunk | as you need to work hard to mess up proxy like that I believe | 08:17 |
| jrosser | well and to somehow get a certificate signed by the OSA CA to appear to be coming from somewhere totally else | 08:18 |
| jrosser | unless due to "enterprise reasons" they've had to install our CA into some proxy device and thats gone horriby wrong | 08:18 |
| noonedeadpunk | I'd guess they might wanted to take opprotunity of having CA to use it for proxy tls termination? | 08:18 |
| noonedeadpunk | or that | 08:19 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Add molecule testing for the role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939751 | 08:27 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible master: Add additional commented RabbitMQ policy to manage segment sizes https://review.opendev.org/c/openstack/openstack-ansible/+/939713 | 08:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Add molecule testing for the role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939751 | 08:35 |
| noonedeadpunk | hm, that's weird as well: https://zuul.opendev.org/t/openstack/build/8811c8041d754dd6949c9094816426e0/log/job-output.txt#1293-1295 | 09:10 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Add molecule testing for the role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939751 | 09:11 |
| noonedeadpunk | as python3-setuptools are installed in containers and on localhost... maybe not inside of venv though... hm... | 09:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Install setupttols for molecule venv https://review.opendev.org/c/openstack/openstack-ansible/+/939771 | 09:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Add molecule testing for the role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939751 | 09:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Install setupttols for molecule venv https://review.opendev.org/c/openstack/openstack-ansible/+/939771 | 09:29 |
| opendevreview | Merged openstack/ansible-role-pki master: Install setuptools for noble https://review.opendev.org/c/openstack/ansible-role-pki/+/939752 | 10:17 |
| opendevreview | Merged openstack/ansible-role-pki master: Use OSA_TEST_REQUIREMENTS_FILE for molecule job https://review.opendev.org/c/openstack/ansible-role-pki/+/939301 | 10:17 |
| opendevreview | Merged openstack/openstack-ansible-os_skyline master: Use standalone httpd role https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/938297 | 11:06 |
| opendevreview | Merged openstack/openstack-ansible-os_skyline master: Implement TLS backend coverage for Skyline https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/938298 | 11:24 |
| jrosser | i'm not sure that the replies in https://bugs.launchpad.net/openstack-ansible/+bug/2095465 make much sense at all | 11:46 |
| jrosser | NAT or not has nothing to do with certificates | 11:46 |
| noonedeadpunk | unless it was pointing traffic somehow back to haproxy | 12:05 |
| noonedeadpunk | or smth like that | 12:05 |
| jrosser | port forward for 443 or something | 12:07 |
| noonedeadpunk | yeah, can be... | 12:14 |
| noonedeadpunk | but it's really smth very weird | 12:14 |
| noonedeadpunk | what sucks about molecule - are absent logs as a result | 12:26 |
| noonedeadpunk | from docker, I mean | 12:27 |
| noonedeadpunk | maybe there's some solution exists for that... | 12:27 |
| noonedeadpunk | just trying to understnad wtf is missing in galera role for EL: https://zuul.opendev.org/t/openstack/build/af8d35e5d4994b12b7a0fae677cc31db | 12:38 |
| noonedeadpunk | as it fails to startup with Failed to establish connection: Connection refused | 12:39 |
| noonedeadpunk | but in fact it feels it's just missing some binary | 12:39 |
| noonedeadpunk | https://paste.openstack.org/show/bj1GjDDCUGsJ9GP3dj6y/ | 12:40 |
| jrosser | i am kind of struggling with molecule and host key checking once we use ssh | 13:23 |
| jrosser | the host key changes each time the docker container is re-created | 13:24 |
| noonedeadpunk | ouch, yeah, that;s true | 13:24 |
| noonedeadpunk | but you can disable that through ansible setting | 13:25 |
| jrosser | maybe i'm doing it wrong but `ansible_ssh_extra_args: '-o StrictHostKeyChecking=no'` doesnt seem to help | 13:25 |
| noonedeadpunk | ok, try that | 13:26 |
| noonedeadpunk | host_key_checking: false in here https://opendev.org/openstack/ansible-role-pki/src/branch/master/molecule/default/molecule.yml#L32 | 13:26 |
| jrosser | ah ok will try that | 13:33 |
| noonedeadpunk | so. el fails either to missing `iptuils` or `procps` | 13:36 |
| noonedeadpunk | doh | 13:36 |
| noonedeadpunk | and then EL is quite bad in terms of idempotency | 13:36 |
| noonedeadpunk | it's procps /o\ | 13:44 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Add molecule testing for the role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939751 | 13:46 |
| jrosser | hrrm i wonder if we have some divergence between our ssh plugin and the builtin one | 13:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Use module_hotfixes for yum_repository https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939798 | 13:48 |
| noonedeadpunk | oh, yes, for sure | 13:48 |
| noonedeadpunk | at least there was one very nasty bug regarding the ansible_user or smth like that I've found but never looked into reasons deep enough | 13:48 |
| noonedeadpunk | meaning this one: https://bugs.launchpad.net/openstack-ansible/+bug/2044229 | 13:49 |
| jrosser | well hah that is interesting | 13:52 |
| jrosser | because these moleule tests run a zuul in ci | 13:53 |
| jrosser | and we need to set remote_user to root to connect to the docker container | 13:53 |
| jrosser | and thats basically not working | 13:53 |
| jrosser | until i saw my patch fail i'd been running everything locally as root | 13:54 |
| jrosser | so now i switch to the regular ubuntu user and try to get the remote user correct, just like it would be in the zuul job | 13:54 |
| noonedeadpunk | ah | 13:54 |
| noonedeadpunk | but well | 13:54 |
| jrosser | and there is one simple play i have, that just gathers facts from the docker container | 13:55 |
| jrosser | and for a couple hours now i try to make that work with the osa ssh plugin with no luck | 13:55 |
| jrosser | but switch it to ansible.builtin.ssh and it works first time :( | 13:56 |
| noonedeadpunk | well this sounds exactly like the bug an our plugin ignoring REMOTE_USER | 13:56 |
| noonedeadpunk | as -u is what does not work | 13:57 |
| jrosser | so, remember i had some time ago to fix a parameter on the plugin for new ansible, pkcs11 or something | 13:57 |
| jrosser | and i did a one-line fix for that, but i did think it was pretty unmaintainable | 13:57 |
| jrosser | what would be better is if we had the upstream module parameters copy/pasted in *exactly* as they are in ansible.builtin.ssh | 13:58 |
| noonedeadpunk | yeah, it needs to be added from time to time... | 13:58 |
| jrosser | and then some very obvious different section where we define the unique ones for our plugin | 13:58 |
| noonedeadpunk | I wnder how to do that | 13:58 |
| noonedeadpunk | given that basically the parameter is just declarative | 13:58 |
| jrosser | maybe just space them apart with some comments in the code | 13:58 |
| jrosser | ## these are ours..... # keep these in sync with upstream...... | 13:59 |
| noonedeadpunk | and iirc it as added to "DOCUMENTATION" constant | 13:59 |
| jrosser | yes thats right | 13:59 |
| noonedeadpunk | which is what actually unmaintainable | 13:59 |
| noonedeadpunk | as it's a multiline string var parsed as yaml or smth | 14:00 |
| jrosser | its a shame its not a more natural derived class sort of thing | 14:00 |
| noonedeadpunk | yes | 14:00 |
| jrosser | where that all just comes in magically from the thing you inherit from | 14:00 |
| noonedeadpunk | pretty much the root cause of having plugins like ours | 14:00 |
| noonedeadpunk | and which would make total sense for plenty of things | 14:00 |
| jrosser | i will see if i can do that - make our parameters more obvious from the upstream ones | 14:02 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Add molecule testing for the role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939751 | 14:02 |
| jrosser | and then try to update the whole of the upstream ones | 14:02 |
| noonedeadpunk | I wonder if we can take the contant value from the parent class | 14:02 |
| noonedeadpunk | as I think we should be able to? | 14:03 |
| noonedeadpunk | and then probably merge 2 together or smth... | 14:03 |
| noonedeadpunk | jsut as 2 text strings through new line... | 14:04 |
| jrosser | documentation string seems to live outside the class though | 14:05 |
| noonedeadpunk | but still inside of the import? | 14:09 |
| noonedeadpunk | jsut order should be different then I guess | 14:10 |
| jrosser | also there is this https://docs.ansible.com/ansible/latest/dev_guide/developing_modules_documenting.html#documentation-fragments | 14:10 |
| noonedeadpunk | so like SSH.DOCUMENTATION is preset here?https://opendev.org/openstack/openstack-ansible-plugins/src/branch/master/plugins/connection/ssh.py#L306 | 14:10 |
| noonedeadpunk | oh, well | 14:11 |
| noonedeadpunk | hehe | 14:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Install setupttols for molecule venv https://review.opendev.org/c/openstack/openstack-ansible/+/939771 | 14:21 |
| jrosser | also if you find a way to (for example) hack on the plugins repo whilst trying to fix molecule things for a different repo, that would be interesting | 14:26 |
| jrosser | it seems to re-do the requirements.yml for each time you `molecule converge` which is unhelpful for messing with the dependencies | 14:26 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Install setupttols for molecule venv https://review.opendev.org/c/openstack/openstack-ansible/+/939771 | 14:29 |
| noonedeadpunk | yeah, actually messing with dependencies is a different sort of pita right now :( | 14:38 |
| spatel | noonedeadpunk hey! I am dealing with this issue - https://github.com/vexxhost/magnum-cluster-api/pull/464 | 14:43 |
| spatel | based on this patch all I need cross_az_attach=True right? | 14:44 |
| noonedeadpunk | um, I guess it depends? | 14:44 |
| noonedeadpunk | as True would be pretty much current behaviour | 14:44 |
| noonedeadpunk | So assumption that you alight it what whatever you have set in Nova | 14:44 |
| noonedeadpunk | *align it with | 14:45 |
| noonedeadpunk | in my case - we have different storage clusters per AZ | 14:45 |
| noonedeadpunk | so we need to create volumes only in AZ which was supplied for workers | 14:45 |
| spatel | In my case all compute resources running on AZ1 and Cinder is default to nova | 14:46 |
| spatel | my k8s doesn't like it and failing to attach storage | 14:46 |
| noonedeadpunk | so I'm not sure why you're seeing an issue | 14:46 |
| spatel | To solve my issue I have to edit k8s deployment and set this by hand topology.cinder.csi.openstack.org/zone: nova | 14:47 |
| noonedeadpunk | so that is weird, as it should be using "nova" as default | 14:47 |
| noonedeadpunk | as of today even | 14:47 |
| spatel | when I build application its by default topology.cinder.csi.openstack.org/zone: AZ1 (where my computes running) | 14:48 |
| noonedeadpunk | So in comment I've pointed to the plugin docs: https://github.com/kubernetes/cloud-provider-openstack/blob/d228854cf58e7b4ed93d5e7ba68ab639450e3449/docs/cinder-csi-plugin/using-cinder-csi-plugin.md#supported-parameters | 14:48 |
| spatel | I am running 0.24 | 14:48 |
| spatel | mcapi driver | 14:48 |
| noonedeadpunk | and right now capi doesn't really supply "availability" for storageclass? | 14:49 |
| noonedeadpunk | only type? | 14:49 |
| noonedeadpunk | so if you ask me - I'd say it should be working just fine for you.... | 14:49 |
| noonedeadpunk | as I was trying to solve an opposite problem | 14:50 |
| noonedeadpunk | could it be that you've set default_schedule_zone in cinder.conf? | 14:50 |
| spatel | Let me check.. | 14:50 |
| noonedeadpunk | but potentially it will work for you as well :D | 14:51 |
| spatel | No I don't have anything specific default_schedule_zone | 14:52 |
| spatel | Should I set default_schedule_zone = nova ? | 14:52 |
| noonedeadpunk | that can help... | 14:52 |
| spatel | Let me try and see but I believe cinder default AZ is always nova | 14:53 |
| noonedeadpunk | it should be yes | 14:56 |
| noonedeadpunk | as well as k8s should be doing nova by default | 14:56 |
| spatel | I am telling k8s to use AZ1 to schedule computes | 14:58 |
| noonedeadpunk | so you have in kube.conf az set for cinder? | 14:58 |
| spatel | so my k8s scheduling everything in AZ1 region along with topology.cinder.csi.openstack.org/zone: AZ1 :( | 14:59 |
| spatel | I didn't set anything anywhere. This is all coming automatically | 14:59 |
| noonedeadpunk | yah, but that is defined in config on control plane? | 14:59 |
| spatel | I have this two value in my k8s templates - availability_zone=az1,control_plane_availability_zones=az1 | 15:00 |
| jrosser | the value must com from somewhere :) | 15:01 |
| noonedeadpunk | yeah | 15:01 |
| noonedeadpunk | and there's nothing in code I see which would set it | 15:01 |
| noonedeadpunk | https://github.com/vexxhost/magnum-cluster-api/blob/main/magnum_cluster_api/resources.py#L333-L336 | 15:01 |
| jrosser | must it be in the cloud-init data? | 15:02 |
| noonedeadpunk | but I'm not really an expert in k8s, so no idea... | 15:02 |
| noonedeadpunk | I think it is | 15:03 |
| jrosser | so thats the place to start | 15:03 |
| noonedeadpunk | and then it creates some local config file which is used by controlelr to create resources | 15:03 |
| jrosser | find it there, check how that stuff is generated in magnum-cluster-api | 15:03 |
| jrosser | there will be the source | 15:03 |
| noonedeadpunk | when I needed to fix AZ zones - I backtracked to the linked above code | 15:05 |
| noonedeadpunk | (which resulted in PR) | 15:05 |
| jrosser | though thinking about it i am not sure which way round this is | 15:06 |
| jrosser | does o-c-c-m in the workload cluster say "hey cinder api - give me a volume" | 15:07 |
| noonedeadpunk | I don't think you can do that through magnum api? | 15:07 |
| noonedeadpunk | as then - you can't do that from control cluster | 15:08 |
| jrosser | no, but thats how persistent volume is wired into the workload cluster | 15:08 |
| noonedeadpunk | ah, you mean the "boot" one? | 15:08 |
| jrosser | oh well i don't know | 15:08 |
| jrosser | what are we talking about :) ? | 15:08 |
| noonedeadpunk | I was thinking about additional volumes to be attached to the workload cluster | 15:09 |
| noonedeadpunk | but how workload cluter is spawned - is a different topic/code indeed | 15:09 |
| jrosser | right - and thats openstack-cloud-controller-manager makign requests from in the workload cluster to to cinder or whatever | 15:09 |
| jrosser | so theres two things at play, what is requested, vs what actually is supplied | 15:11 |
| jrosser | and each of those could be not what you expect, i guess | 15:11 |
| jrosser | and heads back to your suggestion about checking the default schedule zone | 15:12 |
| noonedeadpunk | NeilHanlon: are you going to FOSDEM this time around? | 15:30 |
| spatel | sorry folks, I was on another call | 15:33 |
| NeilHanlon | noonedeadpunk: yep! i'll be there | 15:35 |
| * noonedeadpunk thinking | 15:35 | |
| noonedeadpunk | like - plane back is $17 for me | 15:36 |
| noonedeadpunk | the problem is that plane to brussels is $75 :D | 15:36 |
| noonedeadpunk | so it's eating up my beer budget a bit | 15:37 |
| noonedeadpunk | but it sounds like it's gonna be fun there | 15:37 |
| NeilHanlon | heh. i'm flying into amsterdam and taking the train, but not cheaper anyways lol | 15:44 |
| NeilHanlon | i can buy you a couple beers, for sure;) | 15:44 |
| noonedeadpunk | nah, that's not where I was leading towards :D | 15:48 |
| NeilHanlon | I know :P | 15:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add jmespath to test-requirements https://review.opendev.org/c/openstack/openstack-ansible/+/939824 | 16:03 |
| noonedeadpunk | well, with that tempo of molecule adoption, we can quite easily add docker as a container tech, lol | 16:20 |
| noonedeadpunk | just treat docker as lxc | 16:21 |
| noonedeadpunk | though it's a very weird approach for docker world... but it sounds quite trivial overall | 16:23 |
| jrosser | so remote_user is failing becasue this is None https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/connection/ssh.py#L766 | 16:23 |
| jrosser | if i add a debug print there, the value is "root" for the builtin ssh and "None" for our plugin | 16:24 |
| noonedeadpunk | I can recall seeing smth like that, yes | 16:24 |
| noonedeadpunk | I have zero idea why though | 16:25 |
| jrosser | it looks like the keyword remote_user is not working on a play, but ansible_user as a var is OK | 16:33 |
| noonedeadpunk | yup, wrote that in bug report - "Supplying `-e ansible_user=ubuntu` works and is respected" | 16:34 |
| jrosser | oh doh ok :) | 16:34 |
| noonedeadpunk | which made me extra confused of wtf is wrong with remote_user | 16:34 |
| noonedeadpunk | I know - launchpad is not really readable | 16:35 |
| noonedeadpunk | when stuff pasted directly | 16:35 |
| noonedeadpunk | so it does never add -o 'User="ubuntu"' to ssh command as a result | 16:36 |
| jrosser | well, User=root but yes that | 16:36 |
| jrosser | i'm trying to be ubuntu and then connect as root | 16:36 |
| noonedeadpunk | yeah, ok, it doesn't work in any direction - I was root trying to connect as ubuntu | 16:36 |
| jrosser | ah righto | 16:37 |
| jrosser | i'll dig a bit more | 16:37 |
| noonedeadpunk | I think I found that when tried to do deployment as non-root user on hosts | 16:37 |
| noonedeadpunk | and then leverage become | 16:37 |
| jrosser | i am wondering if somehow there is aliasing between the vars of the base/derived classs or something | 16:37 |
| noonedeadpunk | so 2023.1 unmaintained just landed | 17:06 |
| jrosser | noonedeadpunk: do you have a link to your connection plugin bug for remote_user? | 17:06 |
| noonedeadpunk | https://bugs.launchpad.net/openstack-ansible/+bug/2044229 | 17:07 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Fix remote_user with ssh connection plugin https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939829 | 17:07 |
| jrosser | possible \o/ there | 17:08 |
| noonedeadpunk | h | 17:08 |
| noonedeadpunk | *oh | 17:08 |
| noonedeadpunk | that looks too simple lol | 17:08 |
| jrosser | it does :) | 17:08 |
| noonedeadpunk | it wasn't that simple to find I bet | 17:09 |
| jrosser | i put a print of remote_user in our plugin and the base class | 17:09 |
| jrosser | and they were always matching, so None in ours when None in the base plugin | 17:10 |
| jrosser | therefore we were somehow dropping the value of remote_user somewhere, and really there is not too much code in the plugin any more | 17:10 |
| noonedeadpunk | and it was propably not limited to remote_user | 17:12 |
| jrosser | it looks like there is a whole ton of important flags there | 17:12 |
| noonedeadpunk | this one totally worth backporting | 17:12 |
| jrosser | you should test it too though ;) | 17:13 |
| noonedeadpunk | yeah, fair | 17:14 |
| jrosser | i have a very trivial test case for the plugins repo that tests the builtin and our ssh | 17:15 |
| jrosser | so no lxc or anything, just ssh-to-the-host | 17:15 |
| noonedeadpunk | well, integrated test should do some testing ofc... and I think there was also some kind of unit test with python? | 17:16 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-role-qdrouterd unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-role-qdrouterd/+/939830 | 17:20 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-role-pki unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-role-pki/+/939831 | 17:20 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-role-python_venv_build unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/939832 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-role-systemd_mount unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/939833 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-role-systemd_networkd unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/939834 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-role-systemd_service unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/939835 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-role-uwsgi unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/939836 | 17:21 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-role-zookeeper unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-role-zookeeper/+/939837 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/ansible-hardening unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ansible-hardening/+/939838 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-apt_package_pinning unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-apt_package_pinning/+/939839 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-ceph_client unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/939840 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-ceph_client unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/939841 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-galera_server unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939842 | 17:22 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-haproxy_server unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/939843 | 17:23 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-lxc_container_create unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/939844 | 17:23 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-lxc_hosts unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/939845 | 17:23 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-memcached_server unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-memcached_server/+/939846 | 17:23 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-openstack_hosts unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/939847 | 17:23 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-openstack_openrc unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-openstack_openrc/+/939848 | 17:24 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-ops unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-ops/+/939849 | 17:24 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_adjutant unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_adjutant/+/939850 | 17:24 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Install setuptools for molecule venv https://review.opendev.org/c/openstack/openstack-ansible/+/939771 | 17:24 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_aodh unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/939851 | 17:24 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_aodh unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/939852 | 17:24 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_barbican unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/939853 | 17:24 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_blazar unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/939854 | 17:25 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_ceilometer unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/939855 | 17:25 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_ceilometer unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/939856 | 17:25 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add jmespath to test-requirements https://review.opendev.org/c/openstack/openstack-ansible/+/939824 | 17:25 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_cinder unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/939857 | 17:25 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_cloudkitty unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/939858 | 17:25 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_designate unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/939859 | 17:26 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_glance unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/939860 | 17:26 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_glance unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/939861 | 17:26 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_gnocchi unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/939862 | 17:26 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_gnocchi unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/939863 | 17:26 |
| noonedeadpunk | what is the most annoying in this all - are launchpad notifications | 17:26 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_heat unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/939864 | 17:26 |
| noonedeadpunk | like having 300 emails on included fixes to new branch | 17:27 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_horizon unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/939865 | 17:27 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_horizon unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/939866 | 17:27 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_ironic unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/939867 | 17:27 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_keystone unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/939868 | 17:27 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_magnum unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/939869 | 17:27 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_magnum unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/939870 | 17:27 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_manila unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/939871 | 17:28 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_manila unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/939872 | 17:28 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_masakari unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/939873 | 17:28 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_mistral unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/939875 | 17:28 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_neutron unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/939876 | 17:28 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_neutron unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/939877 | 17:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add jmespath to test-requirements https://review.opendev.org/c/openstack/openstack-ansible/+/939824 | 17:29 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_nova unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/939878 | 17:29 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_octavia unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/939879 | 17:29 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_octavia unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/939880 | 17:29 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_placement unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/939881 | 17:29 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_rally unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/939882 | 17:29 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add jmespath to test-requirements https://review.opendev.org/c/openstack/openstack-ansible/+/939824 | 17:29 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_swift unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/939883 | 17:30 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_tacker unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/939884 | 17:30 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_tacker unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/939885 | 17:30 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_tempest unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/939886 | 17:30 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_trove unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/939887 | 17:30 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_trove unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/939888 | 17:31 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_zun unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/939889 | 17:31 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-os_zun unmaintained/2023.1: Update TOX_CONSTRAINTS_FILE for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/939890 | 17:31 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-plugins unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939891 | 17:31 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-rabbitmq_server unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/939892 | 17:31 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-repo_server unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/939893 | 17:31 |
| opendevreview | OpenStack Release Bot proposed openstack/openstack-ansible-tests unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/openstack-ansible-tests/+/939894 | 17:32 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add most basic molecule test for ssh connection plugin https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939898 | 18:00 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Update ssh connection plugin paramters to match upstream https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939899 | 18:00 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Add molecule testing to the role https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/939902 | 18:13 |
| jrosser | looks like the basic molecule job on the plugins repo is passing for debian | 18:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server master: Add molecule testing for the role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/939751 | 18:14 |
| noonedeadpunk | sweet | 18:14 |
| noonedeadpunk | I galera_server was passing for all distros | 18:14 |
| jrosser | so that i think has a better and simpler way to get a ssh based connection, everything is defined in molecule.yml for that now without any external inventory needed | 18:14 |
| noonedeadpunk | and rabbitmq - well, will see | 18:15 |
| jrosser | needs the rocky/centos fixing up i think to skip apparmor and get the sshd service name correct | 18:15 |
| noonedeadpunk | I'm somehow concerned about future chicken-egg due to absent depends-on for requirements | 18:16 |
| noonedeadpunk | probably - we need to supply requirements file through ENV var | 18:16 |
| jrosser | yes with make pre-job to render one? | 18:17 |
| noonedeadpunk | and pass them through zuul pre-tasks | 18:17 |
| jrosser | *maybe | 18:17 |
| noonedeadpunk | yeah | 18:17 |
| jrosser | yup | 18:17 |
| noonedeadpunk | but one step at a time... | 18:17 |
| jrosser | we can use the one in the repo being tested as the source | 18:17 |
| noonedeadpunk | yup | 18:17 |
| jrosser | then adjust the content to point to the on-disk repos, shout be do-able | 18:17 |
| noonedeadpunk | btw, I think we also need to add `roles` support into ansible-collection-requirements... | 18:18 |
| jrosser | indeed | 18:18 |
| noonedeadpunk | but that's a different topic | 18:18 |
| jrosser | i think o'm going to leave the plugins repo for today - had a long day | 18:18 |
| jrosser | if you want to mess with that, go for it | 18:18 |
| noonedeadpunk | ++ thanks for working on that! | 18:19 |
| noonedeadpunk | I've picked easy ones, hehe | 18:19 |
| jrosser | its definatly broken for centos/rocky | 18:19 |
| noonedeadpunk | not today at least | 18:19 |
| jrosser | then we can revisit what i did in lxc_container_create based on the todays stuff for the plugins repo | 18:20 |
| jrosser | that can be simplified now i understand it better | 18:20 |
| jrosser | there is a downside to defining the networks statically in the molecule.yml file though - they will collide if you try to test several things on the same host | 18:21 |
| jrosser | i had to docker network rm <network> a few times today | 18:21 |
| noonedeadpunk | so yes, eventually - you can set them jsut to a subnet and it will pick ip randomly | 18:21 |
| jrosser | hmm | 18:22 |
| noonedeadpunk | but then if you need to know IP somewhere for variables... | 18:22 |
| jrosser | ^ exactly this for anything including the connection plugin | 18:22 |
| noonedeadpunk | I actually don't know how to fetch them in a good enough way | 18:22 |
| jrosser | so i moved to static ip allocation today for that | 18:23 |
| noonedeadpunk | yeah | 18:23 |
| noonedeadpunk | as I also found that ansible_host is not defined there at all | 18:23 |
| jrosser | but i do have a method in the lxc_container_create role that looks it up dynamcially, but it is messier | 18:23 |
| noonedeadpunk | and we rely a lot on it | 18:23 |
| noonedeadpunk | anyway - quite a good progress :) | 18:24 |
| jrosser | at the same time the existing lxc tests look for there being a specific IP on the interfaces | 18:24 |
| jrosser | so i figured that would also be totally difficult if the ip was not known | 18:24 |
| noonedeadpunk | yeah, true | 18:27 |
| opendevreview | Merged openstack/openstack-ansible master: Add additional commented RabbitMQ policy to manage segment sizes https://review.opendev.org/c/openstack/openstack-ansible/+/939713 | 20:13 |
| opendevreview | Merged openstack/openstack-ansible stable/2024.2: Remove ansible-role-qdrouterd from zuul required-projects https://review.opendev.org/c/openstack/openstack-ansible/+/939723 | 21:00 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2024.1: Remove ansible-role-qdrouterd from zuul required-projects https://review.opendev.org/c/openstack/openstack-ansible/+/939914 | 21:08 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/2024.1: Remove ansible-role-qdrouterd from zuul required-projects https://review.opendev.org/c/openstack/openstack-ansible/+/939914 | 21:08 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/2023.2: Remove ansible-role-qdrouterd from zuul required-projects https://review.opendev.org/c/openstack/openstack-ansible/+/939915 | 21:09 |
| jrosser | do we merge all the new 2023.1 branch patches now? | 21:21 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add most basic molecule test for ssh connection plugin https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939898 | 21:33 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Update ssh connection plugin paramters to match upstream https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939899 | 21:33 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add most basic molecule test for ssh connection plugin https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939898 | 21:54 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Update ssh connection plugin paramters to match upstream https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939899 | 21:54 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Add most basic molecule test for ssh connection plugin https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939898 | 22:06 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-plugins master: Update ssh connection plugin paramters to match upstream https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/939899 | 22:06 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Upgrade magnum-cluster-api to 0.24.2 https://review.opendev.org/c/openstack/openstack-ansible-ops/+/936229 | 22:21 |
| noonedeadpunk | jrosser: yeah. I will also push 2023.1 for integrated repo in the morning | 22:24 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Store clusterctl and magnum-system namespace dump if cluster create fails https://review.opendev.org/c/openstack/openstack-ansible-ops/+/936626 | 22:28 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Test all supported versions of k8s workload cluster with magnum-cluster-api https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916649 | 22:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Add molecule testing to the role https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/939902 | 22:28 |
| noonedeadpunk | seems you've decided to prolong your long day a little bit :D | 22:29 |
| * jrosser shakes fist at centos to end the day | 22:37 | |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a restart to put some database compaction config changes into effect, and will return within a few minutes | 22:53 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!