| *** dviroel|afk is now known as dviroel | 00:31 | |
| *** dviroel is now known as dviroel|out | 00:35 | |
| admin1 | hi .. i had to delete the rabbitmq containers and create them again .. when i run the setup-openstack playbook, it does not add the vhosts in rabbitmq .. how to force or have it add rabbitmq queues ? | 02:44 |
|---|---|---|
| jamesdenton | i believe the individual roles handle their respective vhosts | 02:54 |
| admin1 | thanks .. so there is no --rabbitmq tag per se ? | 02:55 |
| admin1 | running the playbook again created them .. not sure whey it skipped the first time | 02:56 |
| jamesdenton | *shrug* | 03:00 |
| admin1 | i have another strange issue .. for some instances .. console is stuck, ssh is also stuck ( connection established, key exchanged, but the login is stuck ) .. while other instances are just fine | 03:08 |
| admin1 | ping just works fine .. but nova reboot never reboots .. goes to a reboot status and is just stuck there | 03:09 |
| admin1 | nova-compute log shows nothing | 03:09 |
| admin1 | how to handle a instance that seems to be stuck on powering on process | 03:16 |
| admin1 | shall i set its status to shutdown and then do a virsh shutdown and then try to up again from cli ? | 03:16 |
| jamesdenton | hrmm | 03:21 |
| jamesdenton | i used to see similar ssh issues related to mtu | 03:21 |
| jamesdenton | the console, though, that's a new one | 03:21 |
| jamesdenton | can you find any sort of pattern, like a particular compute that's trouble | 03:22 |
| admin1 | in the same hypervisor, some work fine, some do not | 03:23 |
| admin1 | how safe is a virsh shutdown ? | 03:24 |
| jamesdenton | safe is relative | 03:29 |
| jamesdenton | :) | 03:29 |
| jamesdenton | you might test it with a throwaway instance, but IIRC you can 'virsh destroy' the instance and start it back up | 03:29 |
| admin1 | i rebooted one hypervisor with 1 instance .. nova shows active .. it did not start automatically | 03:53 |
| admin1 | how do i start a instance after hypervisor reboot when its state is set to active | 03:55 |
| admin1 | do i modify mysql to make it shutoff and then issue start command ? | 03:56 |
| jamesdenton | what happens if you just issue an 'openstack server stop'? | 04:13 |
| admin1 | power state: nostate , task_state: powering off , status: Active | 04:14 |
| admin1 | and it remains like that | 04:14 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-tests master: Update ansible lint version https://review.opendev.org/c/openstack/openstack-ansible-tests/+/824454 | 07:20 |
| *** sshnaidm|afk is now known as sshnaidm | 08:49 | |
| jrosser__ | still need to merge centos-8 removal from zun in order to drop it elsewhere https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/820679 | 08:50 |
| opendevreview | Merged openstack/openstack-ansible-os_zun master: Remove testing on Centos-8 https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/820679 | 09:27 |
| jrosser__ | oh thats interesting, there seems to be a category of things that the dashboard does not show, like this https://review.opendev.org/c/openstack/openstack-ansible/+/820982 | 09:32 |
| jrosser__ | things that are +2+w but cannot proceed becasue of a dependancy / depends-on | 09:32 |
| * noonedeadpunk writes down | 09:34 | |
| jrosser__ | i guess they drop out of the "needs approval" section | 09:36 |
| noonedeadpunk | yep, +w means it's approved | 09:37 |
| noonedeadpunk | I guess needs to check for +V label as well | 09:38 |
| jrosser__ | maybe we can have a "needs merge" section for those | 09:39 |
| jrosser__ | andrewbonney: noonedeadpunk also there is a huge pile of review on master and stable branches | 09:44 |
| jrosser__ | i have some time today to keep an eye on those / fix broken things | 09:44 |
| noonedeadpunk | yeah, was about to go through reviews | 09:44 |
| jrosser__ | awesome, thanks | 09:45 |
| jrosser__ | some stuff is breaking a few times on centos 8 so it is good to merge dropping that | 09:45 |
| jrosser__ | and then i also see some repeated failure for distro installs in an unexpected way | 09:45 |
| noonedeadpunk | is it with bionic? | 09:53 |
| jrosser__ | focal | 09:57 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/victoria: Add nfs deployment scenario https://review.opendev.org/c/openstack/openstack-ansible/+/823538 | 09:59 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-systemd_service master: Add 'load' boolean for services. https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/824096 | 10:01 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-tests master: Do not syncronise DB/MQ/Service setup tasks to other roles https://review.opendev.org/c/openstack/openstack-ansible-tests/+/824407 | 10:02 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Ensure bash completions directory is present https://review.opendev.org/c/openstack/openstack-ansible/+/824522 | 10:26 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Do not duplicate packages installed with the venv build role https://review.opendev.org/c/openstack/openstack-ansible/+/824179 | 10:26 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove workaround for pre-release debian bullseye https://review.opendev.org/c/openstack/openstack-ansible/+/823491 | 10:34 |
| opendevreview | Merged openstack/openstack-ansible-tests master: Fix rich version for ansible-lint https://review.opendev.org/c/openstack/openstack-ansible-tests/+/824453 | 10:34 |
| opendevreview | Merged openstack/openstack-ansible-tests master: Update ansible lint version https://review.opendev.org/c/openstack/openstack-ansible-tests/+/824454 | 10:34 |
| noonedeadpunk | jrosser__: did you faced some issue because of https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/824189/1/tasks/python_venv_wheel_build.yml ? | 10:35 |
| noonedeadpunk | s/did/have/ | 10:36 |
| jrosser__ | yes its in the centos-9 set of changes | 10:36 |
| noonedeadpunk | as I don't really think it's incorrect now.... | 10:37 |
| jrosser__ | just let me look | 10:37 |
| noonedeadpunk | or wait... | 10:38 |
| jrosser__ | well one issue is that venv_install_distro_package_list seems completely bogus in the condition | 10:38 |
| jrosser__ | and second, if venv_build_base_distro_package_list has entries, but venv_build_distro_package_list does not, then the task does not run | 10:39 |
| jrosser__ | which was ultimately what broke things on centos-9 | 10:39 |
| noonedeadpunk | I realized that I;'m wrong while writing, but still. The idea was that we don't need to instal build packages when nothing to be installed for installation itself | 10:39 |
| jrosser__ | so really my change makes it evaulate the same list in the when: as it will use for the installation | 10:40 |
| jrosser__ | this looks like the actual place that venv_install_distro_package_list matters https://opendev.org/openstack/ansible-role-python_venv_build/src/branch/master/tasks/python_venv_install.yml#L30-L42 | 10:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Run service_setup only once https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/824526 | 11:10 |
| *** dviroel|out is now known as dviroel | 11:15 | |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_murano master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/824382 | 11:15 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_blazar master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/824403 | 11:24 |
| *** arxcruz is now known as arxcruz|ruck | 11:25 | |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-os_octavia master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/824379 | 11:25 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-os_keystone master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/820999 | 11:26 |
| noonedeadpunk | jrosser__: we need to check https://review.opendev.org/c/openstack/openstack-ansible/+/752040 one day.... | 11:30 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_glance master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/824397 | 11:30 |
| jrosser__ | i was never confident in that | 11:32 |
| jrosser__ | like specifically calling out the [0]th host of the group | 11:32 |
| jrosser__ | what if it is down, or during OS upgrade..... | 11:33 |
| noonedeadpunk | I agree. But overall issue sounds reasonable | 11:33 |
| jrosser__ | somehow the change looks simple but i worry about consequences | 11:33 |
| noonedeadpunk | I don't think solution is valid indeed | 11:34 |
| noonedeadpunk | also serial ['5', '100%'] (;¬_¬) | 11:34 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_mistral master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/824383 | 11:35 |
| noonedeadpunk | maybe in common-playbooks we should check if provided groups are identical or not... | 11:36 |
| noonedeadpunk | and if not run them separately... | 11:36 |
| noonedeadpunk | eventually good these are playbooks, so custom one could be used to overcome that easily... | 11:37 |
| jrosser__ | there is also another use of cinder_api[0] in there | 11:37 |
| noonedeadpunk | um https://opendev.org/openstack/openstack-ansible-os_cinder/src/branch/master/tasks/cinder_db_sync.yml#L24-L27 | 11:38 |
| noonedeadpunk | feels very redundant | 11:38 |
| noonedeadpunk | (in the playbook) | 11:40 |
| noonedeadpunk | or maybe not.... | 11:41 |
| jrosser__ | thats not really in line with what this is trying to do https://codesearch.opendev.org/?q=cinder_all_software_updated&i=nope&literal=nope&files=&excludeFiles=&repos= | 11:41 |
| jrosser__ | i.e wait until everything is done to to the db migrate | 11:41 |
| jrosser__ | this is all messy :/ | 11:41 |
| noonedeadpunk | yeah... | 11:41 |
| jrosser__ | i wonder if it's a sign of 'old ansible', that you just wouldnt do it like this today | 11:42 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-os_nova master: Fix nova-key tag handling https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/824551 | 11:42 |
| jrosser__ | or maybe trying to do too many components scheduler/backup/api all in the same playbook | 11:42 |
| noonedeadpunk | I'm not really sure if anything starting from L47 is needed | 11:47 |
| noonedeadpunk | except one thing.... | 11:48 |
| noonedeadpunk | these conditional likely incorrect https://opendev.org/openstack/openstack-ansible-os_cinder/src/branch/master/tasks/main.yml#L49 | 11:49 |
| noonedeadpunk | We need to create everything with scheduler setup, not with api | 11:50 |
| opendevreview | Merged openstack/openstack-ansible master: Remove CI jobs for centos-8 https://review.opendev.org/c/openstack/openstack-ansible/+/820982 | 12:03 |
| noonedeadpunk | damn, it's a mess anyway. we need somehow to run setup tasks before cinder starts... | 12:04 |
| noonedeadpunk | I guess that's why we have these restart tasks now | 12:05 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_cinder master: Setup db/mq/service during scheduler run https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/824553 | 12:05 |
| noonedeadpunk | because generally you start cinder-volume when you don't have scheduler, db, rabbitmq, even keystone user... | 12:06 |
| opendevreview | Merged openstack/openstack-ansible-tests master: Do not syncronise DB/MQ/Service setup tasks to other roles https://review.opendev.org/c/openstack/openstack-ansible-tests/+/824407 | 12:23 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Fix ceph example https://review.opendev.org/c/openstack/openstack-ansible/+/628625 | 12:38 |
| opendevreview | Merged openstack/openstack-ansible master: Remove centos-8 support https://review.opendev.org/c/openstack/openstack-ansible/+/820854 | 12:46 |
| opendevreview | Merged openstack/openstack-ansible-os_nova master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/824380 | 12:47 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-python_venv_build master: Add per-distro vars files https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/824180 | 12:55 |
| *** anbanerj is now known as frenzyfriday | 12:57 | |
| opendevreview | Merged openstack/openstack-ansible-os_designate master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/824398 | 12:57 |
| opendevreview | Merged openstack/openstack-ansible-os_cinder master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_cinder/+/824400 | 13:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Enable senlin tempest plugin when senlin is deployed https://review.opendev.org/c/openstack/openstack-ansible/+/754105 | 13:02 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_senlin master: DNM - test patch for senlin tempest testing https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/754045 | 13:03 |
| opendevreview | Merged openstack/openstack-ansible-os_ironic master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/824388 | 13:07 |
| *** anbanerj is now known as frenzyfriday | 13:08 | |
| opendevreview | Merged openstack/openstack-ansible-os_heat master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/824389 | 13:12 |
| opendevreview | Merged openstack/openstack-ansible-os_tacker master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/824374 | 13:12 |
| opendevreview | Merged openstack/openstack-ansible-os_swift master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/824376 | 13:15 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Move git_requirements to plugins collection https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824563 | 13:15 |
| opendevreview | Merged openstack/openstack-ansible-os_ceilometer master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/824401 | 13:15 |
| opendevreview | Merged openstack/openstack-ansible-galera_server stable/xena: Allow galera_address to be FQDN https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/823839 | 13:15 |
| opendevreview | Merged openstack/openstack-ansible-os_magnum master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/824386 | 13:16 |
| opendevreview | Merged openstack/openstack-ansible-os_trove master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/824373 | 13:16 |
| opendevreview | Merged openstack/openstack-ansible-os_masakari master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/824384 | 13:20 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_zun stable/xena: Remove testing on Centos-8 https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/824535 | 13:21 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_zun stable/xena: Remove testing on Centos-8 https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/824535 | 13:21 |
| opendevreview | Merged openstack/openstack-ansible-os_barbican master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_barbican/+/824404 | 13:22 |
| opendevreview | Merged openstack/openstack-ansible-os_senlin master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/824377 | 13:26 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_zun stable/xena: Remove testing on Centos-8 https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/824535 | 13:31 |
| jamesdenton | mornin' all | 13:32 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_zun stable/xena: Remove testing on Centos-8 https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/824535 | 13:35 |
| opendevreview | Merged openstack/openstack-ansible-os_manila master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/824385 | 13:35 |
| jrosser__ | hello | 13:36 |
| opendevreview | James Denton proposed openstack/openstack-ansible-ops master: Update MNAIO for Focal https://review.opendev.org/c/openstack/openstack-ansible-ops/+/824486 | 13:36 |
| opendevreview | Merged openstack/openstack-ansible-os_aodh master: Refactor use of include_vars https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/824342 | 13:37 |
| jamesdenton | hello there | 13:40 |
| jamesdenton | noonedeadpunk these mnaio patches are a stopgap until some more modern solution comes along. It's working, though. | 13:41 |
| noonedeadpunk | jamesdenton: it's wip | 13:42 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/824381 | 13:42 |
| jamesdenton | yes - i'm just running it through a couple more tests | 13:42 |
| jamesdenton | added the ability to use no_containers (metal), which speeds things up considerably | 13:43 |
| jamesdenton | i just need to borrow some NVMe | 13:43 |
| noonedeadpunk | I'm really fine to merge them. A bit surprised it's working though hehe | 13:45 |
| jamesdenton | not without complications :D | 13:45 |
| jamesdenton | i saw mention of using terraform over heat yesterday. thoughts? | 13:48 |
| jamesdenton | and are you thinking of deploying a minimal AIO to bootstrap a MNAIO? | 13:48 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/xena: Remove CI jobs for centos-8 https://review.opendev.org/c/openstack/openstack-ansible/+/824567 | 13:49 |
| opendevreview | Merged openstack/openstack-ansible-lxc_hosts master: Use only supported functional jobs https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/824390 | 13:52 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/wallaby: Remove CI jobs for centos-8 https://review.opendev.org/c/openstack/openstack-ansible/+/824569 | 13:56 |
| opendevreview | Merged openstack/openstack-ansible-plugins master: Make plugins usable as part of collection https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824148 | 13:59 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/victoria: Remove CI jobs for centos-8 https://review.opendev.org/c/openstack/openstack-ansible/+/824570 | 14:05 |
| opendevreview | Merged openstack/openstack-ansible stable/ussuri: Remove opensuse jobs https://review.opendev.org/c/openstack/openstack-ansible/+/824206 | 14:06 |
| opendevreview | Merged openstack/openstack-ansible stable/victoria: Fix rich version for ansible-lint https://review.opendev.org/c/openstack/openstack-ansible/+/824456 | 14:07 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/ussuri: Remove CI jobs for centos-8 https://review.opendev.org/c/openstack/openstack-ansible/+/824572 | 14:17 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/ussuri: Remove periodic jobs for extended-maintainance ussuri branch https://review.opendev.org/c/openstack/openstack-ansible/+/824573 | 14:17 |
| *** frenzyfriday is now known as frenzyfriday|ruck | 14:18 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Move git_requirements plugin to collection https://review.opendev.org/c/openstack/openstack-ansible/+/824574 | 14:20 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-tests master: Remove unused centos-8 functional job definitions https://review.opendev.org/c/openstack/openstack-ansible-tests/+/820674 | 14:20 |
| noonedeadpunk | jrosser__: I wonder where we should move https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/roles/system_crontab_coordination | 14:23 |
| noonedeadpunk | I was about to push to plugins but then realized how widely it is used... | 14:23 |
| noonedeadpunk | well, maybe not that much though.... | 14:23 |
| opendevreview | Merged openstack/openstack-ansible-openstack_hosts master: Remove use of advanced-virtualisation repo for centos https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/824202 | 14:24 |
| opendevreview | Merged openstack/ansible-role-python_venv_build master: Fix conditional for installing required wheel build packages https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/824189 | 14:25 |
| jrosser__ | if it's an OSA internal thing rather than a re-usable role we should probably move it | 14:26 |
| jrosser__ | noonedeadpunk: also is this right? https://github.com/openstack/openstack-ansible-lxc_hosts/blob/stable/xena/tests/ansible-role-requirements.yml | 14:26 |
| jrosser__ | that the versions stay as master even on stable branches | 14:27 |
| noonedeadpunk | well, no, but has been always like that ( | 14:27 |
| jrosser__ | ok :) | 14:27 |
| noonedeadpunk | there's no sync bot that would edit that during branching. | 14:28 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-tests stable/xena: Fix rich version for ansible-lint https://review.opendev.org/c/openstack/openstack-ansible-tests/+/824540 | 14:34 |
| jrosser__ | do we need that ^ down all the stable branches? | 14:35 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-tests stable/victoria: Fix rich version for ansible-lint https://review.opendev.org/c/openstack/openstack-ansible-tests/+/824541 | 14:39 |
| noonedeadpunk | jrosser__: till ussuri (not including) | 14:41 |
| jrosser__ | ok | 14:41 |
| jrosser__ | it's needed to fix tests repo -> qdrouterd -> remove centos-8 patches | 14:42 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Move system_crontab_role to collection https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824590 | 14:58 |
| opendevreview | Merged openstack/openstack-ansible-galera_server stable/xena: Enable recursion in combine() filter https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/824412 | 15:00 |
| opendevreview | Merged openstack/ansible-role-systemd_networkd master: Add centos-9 support https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/823419 | 15:03 |
| noonedeadpunk | well, really big part of definitions left in openstack-ansible.rc are related to ceph-ansible only now... | 15:06 |
| noonedeadpunk | I wish we could use their ansible.cfg just for ceph deployment... | 15:07 |
| opendevreview | Merged openstack/openstack-ansible-os_nova stable/xena: Enable recursion in combine() filter https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/824410 | 15:09 |
| opendevreview | Merged openstack/openstack-ansible-os_nova master: Replace genisoimage with xorriso for centos-9 https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/824217 | 15:09 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest stable/xena: Enable recursion in combine() filter https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/824417 | 15:13 |
| opendevreview | Merged openstack/ansible-role-uwsgi stable/xena: Enable recursion in combine() filter https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/824414 | 15:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_placement master: Move system_crontab_coordination role to collection https://review.opendev.org/c/openstack/openstack-ansible-os_placement/+/824592 | 15:15 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Move system_crontab_coordination role to collection https://review.opendev.org/c/openstack/openstack-ansible/+/824593 | 15:15 |
| jrosser__ | qdrouterd role is PITA as well | 15:19 |
| opendevreview | Merged openstack/openstack-ansible-os_swift stable/xena: Enable recursion in combine() filter https://review.opendev.org/c/openstack/openstack-ansible-os_swift/+/824413 | 15:19 |
| jrosser__ | failing everwhere in wierd ways | 15:19 |
| noonedeadpunk | I always wanted to try it out but never managed to.. As atually rabit on itself is kind of pita as well, isn't it? | 15:20 |
| noonedeadpunk | *actually | 15:20 |
| noonedeadpunk | *rabbit | 15:20 |
| noonedeadpunk | to be fair, it jsut works lately... | 15:20 |
| jrosser__ | oh well i mean from CI perspective | 15:22 |
| noonedeadpunk | yeah, and I dunno if anybody use it tbh... | 15:22 |
| jrosser__ | we never commit to it so anything trivial like removing legacy OS turns into a huge debugging session | 15:22 |
| jrosser__ | and it's functional jobs too | 15:23 |
| noonedeadpunk | at the same time a bit pity to drop it :) | 15:23 |
| jrosser__ | i made a topic for dropping centos-8 jobs - i think infra want that done fairly soon https://review.opendev.org/q/topic:%22osa%252Fremove-centos8%22+(status:open%20OR%20status:merged) | 15:25 |
| opendevreview | Merged openstack/ansible-role-systemd_service master: Suppress 'Service' directive if ListenStream is specified https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/824041 | 15:25 |
| opendevreview | Merged openstack/ansible-role-systemd_service master: Add 'load' boolean for services. https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/824096 | 15:25 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use config_template from it's collection rather then ceph-ansible https://review.opendev.org/c/openstack/openstack-ansible/+/824595 | 15:31 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron stable/xena: Enable recursion in combine() filter https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/824415 | 15:36 |
| opendevreview | Merged openstack/openstack-ansible master: Ensure bash completions directory is present https://review.opendev.org/c/openstack/openstack-ansible/+/824522 | 15:52 |
| opendevreview | Merged openstack/openstack-ansible-os_blazar master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_blazar/+/824403 | 15:59 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-config_template master: Copy refactor of code quality issues https://review.opendev.org/c/openstack/ansible-config_template/+/824601 | 16:05 |
| opendevreview | Merged openstack/openstack-ansible master: Disable zun volume test https://review.opendev.org/c/openstack/openstack-ansible/+/823517 | 16:05 |
| *** dviroel is now known as dviroel|lunch | 16:05 | |
| opendevreview | Merged openstack/openstack-ansible master: Use local repositories for ansible collections in CI https://review.opendev.org/c/openstack/openstack-ansible/+/821093 | 16:05 |
| opendevreview | Merged openstack/openstack-ansible master: Fix ceph example https://review.opendev.org/c/openstack/openstack-ansible/+/628625 | 16:05 |
| opendevreview | Merged openstack/openstack-ansible master: docs: indicate how to override upper constraints https://review.opendev.org/c/openstack/openstack-ansible/+/823393 | 16:05 |
| opendevreview | Merged openstack/openstack-ansible master: [doc] Document usage of the user-collection-requirements https://review.opendev.org/c/openstack/openstack-ansible/+/776405 | 16:05 |
| *** sshnaidm is now known as sshnaidm|afk | 16:18 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Move git_requirements to plugins collection https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824563 | 16:28 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone master: Use memcached plugin from collection https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/824183 | 16:44 |
| noonedeadpunk | doh why https://review.opendev.org/c/openstack/openstack-ansible/+/824184/6 don't want to go to gates.... | 16:47 |
| noonedeadpunk | feels like rebase is needed.... | 16:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use plugins repo as collection https://review.opendev.org/c/openstack/openstack-ansible/+/824184 | 16:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Move git_requirements plugin to collection https://review.opendev.org/c/openstack/openstack-ansible/+/824574 | 16:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Move system_crontab_coordination role to collection https://review.opendev.org/c/openstack/openstack-ansible/+/824593 | 16:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Use config_template from it's collection rather then ceph-ansible https://review.opendev.org/c/openstack/openstack-ansible/+/824595 | 16:48 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-pki master: WIP - Initial code to create SSH keypairs and certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/824609 | 16:51 |
| jrosser__ | noonedeadpunk: ^ i made a start on this for keypairs, if you have any thoughts on this i'd be interested | 16:52 |
| jrosser__ | very very simple so far but i will follow how PKI role works for CA / certs | 16:52 |
| noonedeadpunk | jrosser__: I'd allow to provide options for openssh_cert for sure... | 16:54 |
| jrosser__ | sure | 16:55 |
| jrosser__ | example playbook https://paste.opendev.org/show/812100/ | 16:55 |
| noonedeadpunk | I was more thinking of `options` as module argument:) | 16:56 |
| noonedeadpunk | At allows to provide things like `no-port-forwarding` `no-pty` `no-user-rc` etc | 16:57 |
| noonedeadpunk | source_address as well | 16:57 |
| jrosser__ | i was going to default that to "clear" | 16:57 |
| jrosser__ | make it opt-in rather than everything enabled by surprise | 16:58 |
| noonedeadpunk | I'd also would limit IP to the mgmt network for nova for instance (I guess it's used by default for cold migrations?) | 16:58 |
| noonedeadpunk | so good to make it configurable. But I agree on clear by default | 16:59 |
| *** dviroel|lunch is now known as dviroel | 17:00 | |
| noonedeadpunk | also I have no idea about how to rotate that? | 17:00 |
| noonedeadpunk | and another thing - maybe we should have {{ pki_keypair_method }}? As I can imagine valid usecase when some "driver" won't support certificates, so pki_method and pki_keypair_method will differ? | 17:03 |
| jrosser__ | oh yes thats a good idea | 17:03 |
| opendevreview | Merged openstack/openstack-ansible-os_murano master: Use common service setup tasks from a collection rather than in-role https://review.opendev.org/c/openstack/openstack-ansible-os_murano/+/824382 | 17:05 |
| noonedeadpunk | btw regarding plugins repo... I guess it's worth to leave functional tests there, but maybe make them self-contained (not dependant from tests)? | 17:06 |
| noonedeadpunk | and same comes for config_template actually.... or leave tests just for these 2 repos... | 17:06 |
| jrosser__ | config template has some rather complex test setups with containers unfortunatly | 17:07 |
| noonedeadpunk | I'm not sure this is needed though? | 17:08 |
| noonedeadpunk | does running against localhost change anything | 17:08 |
| jrosser__ | maybe i am wrong with this.... i need to look again | 17:09 |
| noonedeadpunk | but logs gathering and ansible setup would be still pita to maintain... | 17:09 |
| noonedeadpunk | need to think more about it... | 17:10 |
| jrosser__ | the first part of an AIO build is kind of what the tests repo does | 17:11 |
| jrosser__ | perhaps thats the route to getting rid of the tests repo, to re-use the AIO deploy but make it break off and run a role specific play for these cases | 17:11 |
| noonedeadpunk | I'm actually not sure we need to bootstrap-aio... bootstrap-ansible could be enough (well if we don't need containers ofc) | 17:13 |
| noonedeadpunk | as well as pre/post tasks | 17:13 |
| noonedeadpunk | btw not sure what that means https://opendev.org/openstack/ansible-config_template/src/branch/master/tests/test.yml#L41-L42 | 17:19 |
| jamesdenton | are the python wheel builds typically restricted to a single infra node? | 17:24 |
| noonedeadpunk | to single repo host to be specific, yes | 17:24 |
| jamesdenton | right, ok | 17:24 |
| jamesdenton | do you know what the circumstances might be where that build is attempted on the other repo hosts? | 17:25 |
| jamesdenton | that might not have gcc+ installed? | 17:25 |
| noonedeadpunk | when you have different arch/operating systems | 17:25 |
| noonedeadpunk | but gcc should be still installed I guess | 17:25 |
| jrosser__ | the python venv build role should install everything it needs | 17:26 |
| noonedeadpunk | but jrosser found bug today that can have effect this | 17:26 |
| jrosser__ | hmm | 17:26 |
| jrosser__ | if its not trying to build on the repo host then thats the first thing to address | 17:27 |
| jamesdenton | well, it's running on all 3 | 17:27 |
| jrosser__ | if it is and the toolchain is missing then thats a different bug | 17:27 |
| jamesdenton | and the toolchain exists on repo1 and not repo2/3 | 17:27 |
| jrosser__ | thats as it should be | 17:27 |
| noonedeadpunk | was thinking about https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/824189 | 17:28 |
| jrosser__ | it should target the 'first' one, or rather the one selected here https://github.com/openstack/ansible-role-python_venv_build/blob/master/defaults/main.yml#L116-L122 | 17:28 |
| jamesdenton | i won't bug too much about this, because this is stable/ussuri installed on metal in MNAIO, but i may try to replicate to master and see | 17:28 |
| noonedeadpunk | and targets get generated here https://github.com/openstack/ansible-role-python_venv_build/blob/master/vars/main.yml#L37-L78 | 17:29 |
| jamesdenton | ok, i may debug venv_build_targets then and see | 17:29 |
| jrosser__ | if you don't have facts that are valid that can make the target fail to work out properly too | 17:30 |
| jrosser__ | we did fix that but not sure how far back it was backported | 17:30 |
| jamesdenton | ok, that makes sense | 17:31 |
| noonedeadpunk | was backported only to V https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/777559 | 17:32 |
| jamesdenton | thank you | 17:38 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Remove references to unsupported operating systems https://review.opendev.org/c/openstack/openstack-ansible/+/782557 | 17:48 |
| opendevreview | James Denton proposed openstack/openstack-ansible-os_ironic master: Remove Unnecessary Test Files https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/824214 | 17:50 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-pki master: WIP - Initial code to create SSH keypairs and certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/824609 | 17:54 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-pki master: WIP - Initial code to create SSH keypairs and certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/824609 | 17:55 |
| opendevreview | Jonathan Rosser proposed openstack/ansible-role-pki master: WIP - Initial code to create SSH keypairs and certificates https://review.opendev.org/c/openstack/ansible-role-pki/+/824609 | 17:59 |
| noonedeadpunk | we should finally merge that I guess https://review.opendev.org/c/openstack/openstack-ansible/+/782557 | 18:22 |
| jrosser__ | it is nearly a year :) | 18:22 |
| noonedeadpunk | it really is( | 18:28 |
| opendevreview | dasm proposed openstack/openstack-ansible-os_tempest stable/wallaby: Added var file for EL9 https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/824639 | 18:35 |
| opendevreview | Merged openstack/openstack-ansible stable/xena: Add Ironic-related bridges to AIO https://review.opendev.org/c/openstack/openstack-ansible/+/824169 | 19:13 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Update provider_networks with latest changes https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824646 | 20:41 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Move git_requirements to plugins collection https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824563 | 20:49 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Fix modules location https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824649 | 21:19 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Move git_requirements to plugins collection https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824563 | 21:20 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Move git_requirements to plugins collection https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824563 | 21:20 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Update provider_networks with latest changes https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/824646 | 21:22 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Use provider_networks from collection https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/824650 | 21:28 |
| *** dviroel is now known as dviroel|out | 21:36 | |
| opendevreview | Merged openstack/openstack-ansible master: Use plugins repo as collection https://review.opendev.org/c/openstack/openstack-ansible/+/824184 | 22:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!