| opendevreview | Merged openstack/ansible-role-systemd_service stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/798453 | 00:32 |
|---|---|---|
| opendevreview | Merged openstack/ansible-role-systemd_service stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/798454 | 00:32 |
| opendevreview | Merged openstack/ansible-role-systemd_networkd stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/798450 | 00:37 |
| opendevreview | Merged openstack/ansible-role-systemd_networkd stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/ansible-role-systemd_networkd/+/798451 | 00:37 |
| *** odyssey4me is now known as Guest1231 | 01:12 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_mount stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/798447 | 05:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-systemd_mount stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/798448 | 05:31 |
| zbr | noonedeadpunk: someone running linters as root? | 07:37 |
| noonedeadpunk | yeah, might be the case | 08:58 |
| noonedeadpunk | zbr: but eventually we're fighting a bit with the workarounding of _get_role_fqrn logic :( | 09:09 |
| noonedeadpunk | esp assumption about author == namespace if there're no spaces | 09:09 |
| opendevreview | Merged openstack/ansible-role-systemd_mount stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/798447 | 09:10 |
| noonedeadpunk | but well, we already have solution for that, it's just hasn't landed for all our repos | 09:10 |
| opendevreview | Merged openstack/ansible-role-systemd_mount stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/798448 | 09:10 |
| zbr | noonedeadpunk: there were lots of discussions around, author was a workaround to minimize annoyance, but ideally you should put a namespace. | 09:10 |
| noonedeadpunk | well, it's assumption that roles must be in galaxy | 09:11 |
| noonedeadpunk | which is not even near to truth | 09:11 |
| noonedeadpunk | as if they're not - you don't need namespace | 09:11 |
| noonedeadpunk | but yeah, you won't have meta as well I guess... | 09:12 |
| noonedeadpunk | it's really tough to satisfy everyone, I know :) | 09:12 |
| noonedeadpunk | but I wish there would be some setting that would tell ansible-lint current role name | 09:13 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_keystone master: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/799182 | 09:18 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_keystone stable/wallaby: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/799145 | 09:18 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_glance master: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/799184 | 09:20 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/798463 | 09:24 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_glance stable/wallaby: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/799146 | 09:25 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_container_create master: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/799185 | 09:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_container_create stable/wallaby: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/799147 | 09:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_glance stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/798518 | 09:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_keystone stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/798536 | 09:29 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_container_create stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/798475 | 09:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/798465 | 09:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins master: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/799187 | 09:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/wallaby: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/799148 | 09:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/798597 | 09:33 |
| *** frenzy_friday is now known as frenzyfriday|afk | 11:00 | |
| jonher | "Cloudkitty can be installed by defining `cloudkitty_hosts` in the openstack-ansible inventory." would that be enough for a reno in 797030 (backport playbooks to deploy Cloudkitty to Victoria) something else i should also include? | 11:03 |
| jonher | asking here instead of pushing so we save some CI time in case you want something edited | 11:04 |
| noonedeadpunk | jonher: Let's also mention that playbook os-cloudkitty-install.yml has been added and now it's part of setup-openstack.yml. In order to get cloudkitty installed deployers should define cloudkitty_hosts in openstack_user_config or conf.d files. | 11:07 |
| noonedeadpunk | smth like that | 11:07 |
| jonher | Support has been added to allow the deployment of the Cloudkitty service when hosts are present in the host group `cloudkitty_hosts` in openstack_user_config or conf.d files. | 11:13 |
| jonher | `os-cloudkitty-install.yml` has been added and is now part of `setup-openstack.yml` | 11:13 |
| noonedeadpunk | awesome! | 11:14 |
| jonher | "as it's pretty major change that's worth reflecting and incrementing release" do i need to take action for incrementing a version in afile etc. Or is that part of the release process itself? | 11:17 |
| noonedeadpunk | it's part of release process, so no action needed from your side | 11:18 |
| jonher | alright, thanks :) | 11:18 |
| opendevreview | Jonathan Herlin proposed openstack/openstack-ansible stable/victoria: Integrate cloudkitty https://review.opendev.org/c/openstack/openstack-ansible/+/797030 | 11:18 |
| noonedeadpunk | but when we do minor relase it would be useful to see some release note why instead of bugfix it's minor release | 11:18 |
| jonher | yes indeed, i didn't think about the reno since i didn't have that in the merge to master, should i push a separate review to master branch with the same reno? | 11:19 |
| noonedeadpunk | Not master, but stable/wallaby | 11:20 |
| jonher | alright | 11:28 |
| noonedeadpunk | well, with release notes there's a portion of magic regarding how they work and actually close to nobody really knows what is safe thing to do | 11:30 |
| noonedeadpunk | but what you can't do - backport them, otherwise they may broke (we have disaster with train renos) | 11:31 |
| noonedeadpunk | we need another vote for https://review.opendev.org/c/openstack/openstack-ansible/+/798083/2 | 11:36 |
| jonher | ok, good to know | 11:36 |
| noonedeadpunk | as well as https://review.opendev.org/q/topic:%22osa%252Fpki%22+status:open+branch:stable/wallaby | 11:36 |
| noonedeadpunk | ebbex: jamesdenton jrosser ^ | 11:46 |
| noonedeadpunk | ok, so only qrouterd weird stuff left with centos 8.... | 14:42 |
| jrosser | looks like the actual interface is ens3 | 14:47 |
| noonedeadpunk | looks like it... but ubuntu/debian are fine | 14:47 |
| noonedeadpunk | spawning local vm atm | 14:48 |
| jrosser | https://github.com/openstack/ansible-role-qdrouterd/blob/1c06fe38f0aaa251e50c321bdb2f7cb8c5391d17/templates/qdrouterd.conf.j2#L58 | 14:48 |
| noonedeadpunk | wtf is `inventory_hostname < router` ? | 14:49 |
| noonedeadpunk | the way of != ? | 14:49 |
| jrosser | thats special | 14:49 |
| noonedeadpunk | can't really google that... | 14:52 |
| noonedeadpunk | btw, shouldn't it be a mgmt interface instead of eth0? | 14:53 |
| jrosser | this is original code from 3 years ago | 14:54 |
| jrosser | perhaps everything had eth0 then | 14:54 |
| opendevreview | Merged openstack/openstack-ansible-plugins stable/wallaby: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/799148 | 15:01 |
| jrosser | oh well it's in containers in this functional test | 15:02 |
| jrosser | so there will always be eth0 | 15:02 |
| opendevreview | Merged openstack/openstack-ansible-plugins stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/798597 | 15:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Start Xena cycle https://review.opendev.org/c/openstack/openstack-ansible/+/798920 | 15:10 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Revert placement client override https://review.opendev.org/c/openstack/openstack-ansible/+/798955 | 15:10 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Revert placement client override https://review.opendev.org/c/openstack/openstack-ansible/+/798955 | 15:10 |
| noonedeadpunk | jrosser: oh, I think ansible jsut can't gather facts in centos container | 15:18 |
| noonedeadpunk | as there's neither ip nor ifocnfig | 15:18 |
| jrosser | oohh - is this the new with building with dnf, stuff is missing.... | 15:18 |
| noonedeadpunk | ok, installing iproute not enough... | 15:24 |
| noonedeadpunk | hm, might be some facts caching somewhere... | 15:26 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone stable/wallaby: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/799145 | 16:10 |
| opendevreview | Merged openstack/openstack-ansible-os_keystone stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/798536 | 16:10 |
| opendevreview | Merged openstack/openstack-ansible-lxc_container_create stable/wallaby: Add pki role to local a-r-r https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/799147 | 16:11 |
| opendevreview | Merged openstack/openstack-ansible-lxc_container_create stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-lxc_container_create/+/798475 | 16:11 |
| opendevreview | Merged openstack/openstack-ansible stable/victoria: Workaround linting issue https://review.opendev.org/c/openstack/openstack-ansible/+/798083 | 16:46 |
| opendevreview | Merged openstack/openstack-ansible-ceph_client stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/798463 | 16:52 |
| opendevreview | Merged openstack/openstack-ansible-ceph_client stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/798465 | 16:52 |
| opendevreview | Merged openstack/openstack-ansible stable/victoria: Integrate cloudkitty https://review.opendev.org/c/openstack/openstack-ansible/+/797030 | 17:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Add iproute to CentOS 8 LXC https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/799328 | 20:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts master: Add iproute to CentOS LXC https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/799328 | 20:51 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/wallaby: Bump roles for W release https://review.opendev.org/c/openstack/openstack-ansible/+/799329 | 20:52 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-lxc_hosts stable/wallaby: Add iproute to CentOS LXC https://review.opendev.org/c/openstack/openstack-ansible-lxc_hosts/+/799252 | 20:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Start Xena cycle https://review.opendev.org/c/openstack/openstack-ansible/+/798920 | 21:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum stable/ussuri: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/746296 | 21:07 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!