| noonedeadpunk | kleini: we have smth like that to address such issues with files limit https://paste.openstack.org/show/b6bII1zrCGg4j1yMPwaP/ | 08:32 |
|---|---|---|
| noonedeadpunk | oh, wait, you have that for uwsgi.... It's different then. As we had issues with compute side only due to amount of connections to osds for ceph | 08:37 |
| noonedeadpunk | I can recall api part getting ou of control only when we had issues with rabbitmq | 09:14 |
| kleini | Looks like those connections are not correctly closed and laying around. I am unsure how to debug, which connections this are. Is this possible somehow to debug whether that was a MySQL connection or RabbitMQ or whatever? | 09:26 |
| opendevreview | Merged openstack/ansible-role-frrouting master: Make defaults configurable https://review.opendev.org/c/openstack/ansible-role-frrouting/+/937677 | 09:29 |
| opendevreview | Merged openstack/ansible-role-frrouting master: Add support for FRR reload https://review.opendev.org/c/openstack/ansible-role-frrouting/+/937678 | 09:29 |
| jrosser | good morning | 09:45 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-repo_server master: Remove access limitations to repo vhost https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/938440 | 09:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-repo_server master: Remove not used paths on repo-server https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/938268 | 09:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-repo_server master: Fix tags usage for the role https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/938269 | 09:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-repo_server master: Use standalone httpd role https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/938270 | 09:48 |
| noonedeadpunk | o/ | 09:48 |
| jrosser | looks like the molecule stuff conflicts with this https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L231 | 09:55 |
| noonedeadpunk | that should be fine already | 09:57 |
| noonedeadpunk | I think this should be fixing it https://review.opendev.org/c/openstack/openstack-ansible/+/938220/8/test-requirements.txt | 09:58 |
| noonedeadpunk | it's just not respected by depends-on | 09:58 |
| noonedeadpunk | as tox will pull requirements by url | 09:58 |
| jrosser | it is a shame we don't have a molecule job in the integrated repo to validate this | 10:01 |
| jrosser | as it would be easy to break lots of things when bumping the requirements | 10:01 |
| noonedeadpunk | I was testing by placing same requirements file locally and replacing L65 here with local file https://review.opendev.org/c/openstack/ansible-role-pki/+/831236/8/tox.ini | 10:02 |
| noonedeadpunk | oh, yes, true | 10:02 |
| noonedeadpunk | I'm not sure what we should test with molecule in integrated repo though | 10:02 |
| jrosser | maybe just a trivial "noop" type molecule job that refers always to the files in the openstack-ansible repo, rather than urls | 10:02 |
| noonedeadpunk | another thought I have was to add "molecule" scenario to gate-check-commit | 10:03 |
| noonedeadpunk | but then molcule is smth worth using/running locally during development as well | 10:03 |
| noonedeadpunk | so you mean to test that things just installable at all? | 10:05 |
| jrosser | yes, so maybe just that runs one ping task to check that the requirements can be met/installed and that the molecule config is all good | 10:05 |
| noonedeadpunk | aha, okey. this still won't ensure that new ansible won't fail in other repos due to obsolete modules or anyhting like that | 10:06 |
| noonedeadpunk | but yeah. it's better then nothing | 10:06 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_aodh master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/938441 | 10:23 |
| opendevreview | Merged openstack/ansible-role-systemd_networkd master: Restart systemd-networkd on routes changes https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/937618 | 10:33 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_barbican master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/938443 | 10:34 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ceilometer master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/938444 | 10:36 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/938445 | 10:37 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_cloudkitty master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/938446 | 10:37 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_designate master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/938447 | 10:38 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/938448 | 10:39 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_heat master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/938449 | 10:39 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Add noop molecule job to the integrated repo https://review.opendev.org/c/openstack/openstack-ansible/+/938450 | 10:40 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/938451 | 10:41 |
| noonedeadpunk | jrosser: fwiw, there's a topic: https://review.opendev.org/q/topic:%22osa/retire_qdrouterd%22 | 10:41 |
| jrosser | ah ok | 10:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/938452 | 10:43 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/938453 | 10:44 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_manila master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/938454 | 10:45 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_masakari master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/938455 | 10:45 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_mistral master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/938456 | 10:46 |
| noonedeadpunk | for httpd role - seems there's a chicken-egg conflict between upgrade job with https://review.opendev.org/c/openstack/openstack-ansible/+/938275 and https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/938270 intself | 10:50 |
| noonedeadpunk | but otherwise - base role seems to be fine.... | 10:50 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/938457 | 10:50 |
| noonedeadpunk | The only thing I'm struggling with right now - skyline with TLS scenario | 10:52 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_nova master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/938458 | 10:52 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_octavia master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/938459 | 10:53 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_tacker master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/938460 | 10:53 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_trove master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/938461 | 10:54 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_zun master: Remove support for amqp1 https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/938462 | 10:54 |
| noonedeadpunk | but I'm not convinced if it's working right now at all either | 10:57 |
| opendevreview | Merged openstack/openstack-ansible-plugins master: Remove code to deploy qdrouterd https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/938059 | 11:22 |
| opendevreview | Merged openstack/openstack-ansible master: Adjust upgrade jobs to test supported paths https://review.opendev.org/c/openstack/openstack-ansible/+/936659 | 11:25 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Document pretty endpoint namings https://review.opendev.org/c/openstack/openstack-ansible/+/934536 | 11:30 |
| opendevreview | Merged openstack/openstack-ansible-os_skyline master: Ensure proper db connection string with SSL enabled https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/938302 | 11:36 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline stable/2024.2: Ensure proper db connection string with SSL enabled https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/938464 | 11:41 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline stable/2024.1: Ensure proper db connection string with SSL enabled https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/938465 | 11:41 |
| opendevreview | Merged openstack/openstack-ansible master: Bump collection versions used https://review.opendev.org/c/openstack/openstack-ansible/+/936812 | 11:43 |
| opendevreview | Merged openstack/openstack-ansible master: Update test-requirements https://review.opendev.org/c/openstack/openstack-ansible/+/938216 | 11:43 |
| noonedeadpunk | jrosser: do you think this will be enough as a noop molecule job for integrated repo? https://zuul.opendev.org/t/openstack/build/8dfc10d75b334da28a54f29c09a9b22b/log/tox/molecule/1-commands[0].log | 11:48 |
| jrosser | that looks good - it verifies all the requirements can be installed and that the docker pieces are OK | 11:50 |
| noonedeadpunk | potentially we can run even some playbooks from plugins repo.... and do "multinode" aio with molecule... | 11:51 |
| noonedeadpunk | but really not sure to what extend it should be done in integrated repo | 11:52 |
| noonedeadpunk | as I guess nothing stops from replacing gate-check-commit.sh with molecule (it can skip docker part iirc for the scenario) | 11:54 |
| jrosser | perhaps it would be interesting to think about how we finally get rid of the tests repo | 12:12 |
| jrosser | that mostly is unit tests of the connection plugin now | 12:13 |
| noonedeadpunk | and systemd roles as well | 12:13 |
| jrosser | those would be easy to convert to molecule i think | 12:13 |
| noonedeadpunk | but yes, it would be first in the queue to think about | 12:13 |
| jrosser | the lxc connection perhaps more interesting | 12:13 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-repo_server master: Use standalone httpd role https://review.opendev.org/c/openstack/openstack-ansible-repo_server/+/938270 | 12:32 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Add release note for retirement of qdrouterd and amqp1 messaging https://review.opendev.org/c/openstack/openstack-ansible/+/938474 | 13:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-frrouting master: Use centralized requirements for molecule testing https://review.opendev.org/c/openstack/ansible-role-frrouting/+/938222 | 13:17 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-frrouting master: Use FQCN for module calls https://review.opendev.org/c/openstack/ansible-role-frrouting/+/938273 | 13:17 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-apt_package_pinning master: Replace functional tests with molecule https://review.opendev.org/c/openstack/openstack-ansible-apt_package_pinning/+/938476 | 13:31 |
| noonedeadpunk | it would be very helpful to sort out requirements for molecule: https://review.opendev.org/c/openstack/openstack-ansible/+/938220/8/test-requirements.txt | 13:31 |
| noonedeadpunk | as it's quite annoying otherwise... | 13:32 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Add release note for retirement of qdrouterd and amqp1 messaging https://review.opendev.org/c/openstack/openstack-ansible/+/938474 | 13:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-apt_package_pinning master: Replace functional tests with molecule https://review.opendev.org/c/openstack/openstack-ansible-apt_package_pinning/+/938476 | 14:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_mount master: Replace functional tests with molecule https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/938489 | 15:31 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_mount master: Replace functional tests with molecule https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/938489 | 15:34 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Replace functional tests with molecule https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/938492 | 16:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Replace functional tests with molecule https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/938492 | 16:15 |
| jrosser | noonedeadpunk: i was just looking at molecule for config_template btw - did you start that yet? | 16:18 |
| noonedeadpunk | nope | 16:19 |
| noonedeadpunk | not yet | 16:19 |
| noonedeadpunk | looking at networkd right now | 16:20 |
| noonedeadpunk | and then we have only plugins repo left | 16:20 |
| noonedeadpunk | (who's using functional jobs) | 16:21 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Liberalize static_route defenition https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/937623 | 16:24 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Add routing policy management for interfaces https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/937624 | 16:24 |
| *** admin1 is now known as admin0 | 16:44 | |
| *** admin0 is now known as admin1 | 16:44 | |
| jrosser | noonedeadpunk: do you have `requirements-file: ansible-role-requirements.yml` actually working? | 16:54 |
| jrosser | ^ in molecule.yml | 16:54 |
| noonedeadpunk | I think I am | 16:56 |
| noonedeadpunk | but you;d need to have force there | 16:56 |
| noonedeadpunk | when installing from git | 16:56 |
| noonedeadpunk | or well, when versions are not int | 16:56 |
| jrosser | how does it know which directory that file is in? | 16:56 |
| noonedeadpunk | it looks in top dir of the role | 16:57 |
| noonedeadpunk | so like in repo root | 16:57 |
| jrosser | omg /o\ silly typo | 16:58 |
| jrosser | that explains it :) | 16:58 |
| noonedeadpunk | haha | 16:58 |
| jrosser | config template needs some work it seems | 16:58 |
| jrosser | a ton of assumptions that the whole tests run on localhost | 16:58 |
| jrosser | with a bunch of lookup('ini' ..... ) | 16:59 |
| noonedeadpunk | hm. I'd think that it could be fine, as lookups are there for verify more or less | 16:59 |
| noonedeadpunk | or maybe not... | 16:59 |
| jrosser | they try to lookup the output of config template | 17:00 |
| jrosser | i nearly have it fixed now | 17:00 |
| noonedeadpunk | ah... yeah, then it's indeed quite some work... | 17:00 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_service master: Replace functional tests with molecule https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/938492 | 17:01 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-config_template master: Remove support for python2 and ansible < 2.9 https://review.opendev.org/c/openstack/ansible-config_template/+/938507 | 17:16 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Do not try to configure resolved when it's not available https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938512 | 18:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Do not try to configure resolved when it's not available https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938512 | 18:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Do not try to configure resolved when it's not available https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938512 | 18:17 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-config_template master: Replace functional tests with molecule https://review.opendev.org/c/openstack/ansible-config_template/+/938513 | 18:18 |
| jrosser | ^ this fails locally for one test still | 18:19 |
| noonedeadpunk | I'm struggling a bit with networkd as well for idempotence test.... | 18:19 |
| noonedeadpunk | this somehow is always changed: https://opendev.org/openstack/ansible-role-systemd_networkd/src/branch/master/tasks/main.yml#L208-L218 | 18:20 |
| noonedeadpunk | I get why it can be for oneshot, but it should not be for notify-reload type | 18:21 |
| jrosser | this is failing for config_template https://zuul.opendev.org/t/openstack/build/95d5d9fed2014d2abd657ce9e23cc1db/log/job-output.txt#7110 | 18:22 |
| noonedeadpunk | I wonder if that's because of async | 18:22 |
| jrosser | i get baz is undefined when i run it in molecule | 18:23 |
| jrosser | which is true, it is undefined and it should end up as {{ baz }} in the output | 18:23 |
| noonedeadpunk | yeah, but I think this ine was intentional as we change brackets there? | 18:24 |
| noonedeadpunk | yeah | 18:24 |
| jrosser | no i mean the task fails | 18:24 |
| jrosser | it tried to evaulate it | 18:24 |
| noonedeadpunk | I kind of failed to keep 1 tests for systemd_mount as well as failed to find how to run swapon inside docker contsainer | 18:24 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Don't run systemd task in async https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938515 | 18:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Don't run systemd task in async https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938515 | 18:28 |
| noonedeadpunk | will try to see once done with networkd... as it also gives some surprises in docker | 18:47 |
| jrosser | looks like I make a basic error somewhere with the molecule setup | 18:49 |
| noonedeadpunk | well. it could be a bug in it as well. that's kind of a corner case I think, where we re-define tags in jinja which ansible should respect, and not render | 18:51 |
| noonedeadpunk | as it could be not respecting this: https://opendev.org/openstack/ansible-config_template/src/commit/e528ed0e9e9f3d3fcb2f33ddc5d175faf72094ac/tests/templates/test_raw_content.ini#L1 | 18:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Do not try to configure resolved when it's not available https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938512 | 18:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Don't run systemd task in async https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938515 | 18:55 |
| noonedeadpunk | somehow networkd doesn't bring up interfaces in docker.... | 18:58 |
| noonedeadpunk | it's obviously missing smth, but don't get what exactly | 18:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Add routing policy management for interfaces https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/937624 | 19:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Do not try to configure resolved when it's not available https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938512 | 19:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Don't run systemd task in async https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938515 | 19:05 |
| noonedeadpunk | really not sure how to address, except run molecule just on localhost and add another template for that... I don't see what's missing in docker.... | 19:12 |
| noonedeadpunk | like iproute can bring interfaces up and assign ip, not networkd | 19:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_networkd master: Replace functional tests with molecule https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/938517 | 19:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!