| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Do not duplicate the in-repo example files inside the documentation https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916870 | 07:50 |
|---|---|---|
| jrosser_ | good morning | 09:19 |
| noonedeadpunk | o/ | 09:27 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline master: Do not define a random password for each run https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/912332 | 09:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Move Murano/Senlin/Sahara to Inactive state https://review.opendev.org/c/openstack/openstack-ansible/+/916900 | 09:32 |
| jrosser_ | i would really like to merge the whole stack here asap https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916650/1 | 09:34 |
| jrosser_ | there are people trying to use this and failing because of errors/typos in the docs | 09:34 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server master: Add distro infra jobs https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/914692 | 09:35 |
| jrosser_ | stuart has a follow up patch that needs to go on top of all that as well | 09:35 |
| noonedeadpunk | yeah, I've seen it | 09:35 |
| noonedeadpunk | though didn't vote on that follow up as was not sure about ordering | 09:36 |
| noonedeadpunk | damiandabrowski: would you have a chance to look through these docs? Would ping spotz but don't see around :( | 09:36 |
| noonedeadpunk | or maybe we should different review requirements for ops repo to land just with 1 vote? | 09:37 |
| noonedeadpunk | I can propose patch to reflect that if sounds reasonable | 09:37 |
| noonedeadpunk | And I'd also love to land https://review.opendev.org/c/openstack/openstack-ansible/+/914188/7 asap as well :) | 09:38 |
| jrosser_ | yes i was wondering how we wanted to handle the ops repo | 09:40 |
| noonedeadpunk | we can totally lower the bar for it | 09:40 |
| jrosser_ | its true that stuarts patch will need a giant rebase though | 09:40 |
| jrosser_ | but thats fine | 09:40 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Switch default Ceph deployment to the Reef release https://review.opendev.org/c/openstack/openstack-ansible/+/917014 | 10:06 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ceph_client master: Switch default ceph version to reef https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/917015 | 10:08 |
| noonedeadpunk | jamesdenton: I left a comment for https://review.opendev.org/c/openstack/openstack-ansible/+/917014 | 10:11 |
| noonedeadpunk | ugh, sorry, meant jrosser_ | 10:11 |
| jrosser_ | ok thats i bit weak tbh | 10:13 |
| jrosser_ | becasue in OSA context "community" has specific meaning in the past as being download.ceph.com repo | 10:14 |
| jrosser_ | i think that actually is a term used in ceph-ansible | 10:14 |
| noonedeadpunk | well, I'm also fine to mark that bug as invalid if you can post a comment there :) | 10:14 |
| jrosser_ | and i was just looking into the issue of pinning the repos to the ceph.com ones | 10:15 |
| jrosser_ | and it is totally overridable easily | 10:15 |
| jrosser_ | but maybe we need to put in specific examples of how to switch the pinning from ceph.com to UCA | 10:15 |
| noonedeadpunk | yeah, though I think we can make it conditionally/related to ceph_pkg_source ? | 10:16 |
| noonedeadpunk | just by adding extra condition - `ceph_pkg_source | default('ceph') == 'ceph'` or smth to https://opendev.org/openstack/openstack-ansible-ceph_client/src/branch/master/meta/main.yml#L47 ? | 10:17 |
| noonedeadpunk | (probably default is not even needed though) | 10:17 |
| noonedeadpunk | or ceph_origin... huh, what's the difference... | 10:18 |
| noonedeadpunk | probabl ceph_origin is ceph-ansible thing | 10:19 |
| noonedeadpunk | and we also have this: https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/common-tasks/ceph-server.yml#L41 /o\ | 10:20 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [doc] Update maintenance state for unmaintained releases https://review.opendev.org/c/openstack/openstack-ansible/+/917016 | 10:25 |
| noonedeadpunk | so yeah, I guess it's possible to override all these pinnings today, but probably we can be a bit smarter | 10:27 |
| jrosser_ | i wonder if that pinning in ceph_server is duplicate | 10:27 |
| jrosser_ | as we are already running openstack_hosts and/or ceph_client | 10:27 |
| jrosser_ | and in a bunch of places, both of those | 10:28 |
| noonedeadpunk | well, we don't run ceph_client against mon/osd hosts | 10:29 |
| noonedeadpunk | and I'm not sure we're doing pinning for ceph through openstack_hosts? | 10:29 |
| jrosser_ | oh we sure do https://github.com/openstack/openstack-ansible-openstack_hosts/blob/master/defaults/main.yml#L195-L200 | 10:30 |
| jrosser_ | becasue of lxc having dependancy on librbd | 10:30 |
| jrosser_ | i am i bit nervous about making some change to this as we have really now spread this pinning all over the place | 10:32 |
| jrosser_ | and perhaps it is better to clean that up and make it be in a single well defined place | 10:32 |
| jrosser_ | well - perhaps that is not actually possible though | 10:33 |
| noonedeadpunk | oh, well... yeah... | 10:34 |
| noonedeadpunk | they all serve kinda different purpose | 10:34 |
| noonedeadpunk | and probably we should not merge them too much, as it might be valid to have different client/server versions | 10:34 |
| noonedeadpunk | or at least have a way to do so... | 10:35 |
| jrosser_ | also the semantics are different in the different places, openstack-hosts says "make this least priority" | 10:36 |
| jrosser_ | and other places say "make this highest priority" | 10:36 |
| jrosser_ | so adding logic on top of that is tricky | 10:36 |
| noonedeadpunk | yeah, you're right... | 10:37 |
| noonedeadpunk | but that even we can't asses all places that needs to be overriden - is also a bit wrong | 10:39 |
| jrosser_ | true - i think thats why it points to needing a clean up, rather than make what we have more complicated | 10:40 |
| jrosser_ | tbh - what i think we should do, is make the pinning more explicit | 10:46 |
| jrosser_ | so that everywhere we are completely defining the relative priority of ceph.com, the distro and UCA | 10:48 |
| jrosser_ | and we also previously had a lot of difficulty requiring the mixing of package: "src:ceph" vs. package: "*", release: "ceph.com" | 10:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Fix RabbitMQ vhost defenition for guestagent https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/917024 | 11:08 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Add TLS configuration for guestagent notifications https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/917025 | 11:13 |
| noonedeadpunk | yeah, sure I think you're right as usual :) | 11:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Fix RabbitMQ vhost defenition for guestagent https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/917024 | 11:14 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Fix RabbitMQ vhost defenition for guestagent https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/917024 | 11:18 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Remove openstack-ansible-backup reference https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916650 | 11:51 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Remove readme include from repo docs https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916651 | 11:53 |
| noonedeadpunk | thanks! | 11:59 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Add documentation for ELK to the renderred docs https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916652 | 12:11 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Add ClusterAPI documentation to renderred docs https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916653 | 12:11 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Do not duplicate the in-repo example files inside the documentation https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916870 | 12:11 |
| noonedeadpunk | huh | 12:15 |
| noonedeadpunk | seems we're missing doc promotion or smth like that? | 12:15 |
| noonedeadpunk | ah, no | 12:15 |
| noonedeadpunk | I need to be more patient :D | 12:15 |
| spotz[m] | noonedeadpunk: Which docs do you need? | 12:20 |
| noonedeadpunk | spotz[m]: they all just landed ;) | 12:29 |
| noonedeadpunk | but thanks for checking on that | 12:30 |
| spotz[m] | Ah then yes I wasn't around I was sleeping:) | 12:30 |
| noonedeadpunk | yeah, that;s totaly fair | 12:30 |
| noonedeadpunk | Though I thought you're usually connected through some bouncer | 12:31 |
| noonedeadpunk | anyway :) thanks for being always around! | 12:31 |
| spotz[m] | I've been staying connected through Matrix, so basically the same thing unless the bridge is off like yesterday during the TC meeting | 12:42 |
| noonedeadpunk | yeah, IRC log says you joined channel 30 mins ago | 12:52 |
| noonedeadpunk | so I also was surprised that you saw history/ping | 12:52 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Switch default Ceph deployment to the Reef release https://review.opendev.org/c/openstack/openstack-ansible/+/917014 | 13:17 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Remove caveat around ceph versions in deploy guide documentation. https://review.opendev.org/c/openstack/openstack-ansible/+/917042 | 13:17 |
| jrosser_ | hmm so we have broken the docs in the subdir https://github.com/openstack/openstack-ansible-ops/tree/master/mcapi_vexxhost | 13:21 |
| noonedeadpunk | ah, inlcudes are renderred... | 13:24 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Fix URL to published documentation https://review.opendev.org/c/openstack/openstack-ansible-ops/+/917045 | 13:25 |
| noonedeadpunk | and link here is broken https://github.com/openstack/openstack-ansible-ops/tree/master/mcapi_vexxhost/doc/source | 13:25 |
| jrosser_ | i think maybe we move it all to the top level docs dir and put just a doc with the published URL there instead | 13:26 |
| noonedeadpunk | this is another option ofc | 13:28 |
| noonedeadpunk | I frankly don't have much preference. | 13:30 |
| noonedeadpunk | though having in renderred docs is quite nice | 13:30 |
| noonedeadpunk | as simplifies couple of things, and I guess it should be also better google-able overall | 13:31 |
| opendevreview | Merged openstack/openstack-ansible-rabbitmq_server master: Adjust legacy OpenStack HA policy to make reply queues HA https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916042 | 13:42 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Add independent set of variables for guestagent configuration https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/917047 | 13:45 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/2023.2: Remove RabbitMQ restart when changing policy https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916993 | 13:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/2023.1: Remove RabbitMQ restart when changing policy https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916994 | 13:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/2023.1: Remove RabbitMQ restart when changing policy https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916994 | 13:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/2023.2: Adjust legacy OpenStack HA policy to make reply queues HA https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916995 | 13:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/2023.1: Adjust legacy OpenStack HA policy to make reply queues HA https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916996 | 13:48 |
| andrewbonney | noonedeadpunk: is the track branch in https://review.opendev.org/c/openstack/openstack-ansible/+/914188/7/inventory/group_vars/utility_all/source_git.yml correct? | 13:58 |
| noonedeadpunk | andrewbonney: yeah, we don't want to update tempest/plugins after release | 14:02 |
| noonedeadpunk | and all *_git_track_branch are just for bump tool | 14:02 |
| andrewbonney | Ok | 14:02 |
| opendevreview | Merged openstack/openstack-ansible-ops master: Fix URL to published documentation https://review.opendev.org/c/openstack/openstack-ansible-ops/+/917045 | 14:11 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-ops master: Clarifications to mcapi_vexxhost README https://review.opendev.org/c/openstack/openstack-ansible-ops/+/916817 | 14:15 |
| opendevreview | Merged openstack/openstack-ansible master: Remove caveat around ceph versions in deploy guide documentation. https://review.opendev.org/c/openstack/openstack-ansible/+/917042 | 14:20 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible stable/2023.2: Remove caveat around ceph versions in deploy guide documentation. https://review.opendev.org/c/openstack/openstack-ansible/+/916997 | 14:41 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-uwsgi stable/2023.2: Add Debian 12 distro setup variable https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/916998 | 15:02 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/ansible-role-uwsgi stable/2023.1: Add Debian 12 distro setup variable https://review.opendev.org/c/openstack/ansible-role-uwsgi/+/916999 | 15:03 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Add independent set of variables for guestagent configuration https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/917047 | 15:36 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client stable/2023.1: Add backwards compatibility of ceph_components format https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/917000 | 15:40 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Move Murano/Senlin/Sahara to Inactive state https://review.opendev.org/c/openstack/openstack-ansible/+/916900 | 16:12 |
| jrosser_ | well that is a surprising result for the ceph reef job | 16:20 |
| jrosser_ | somehow still 17.x everywhere | 16:20 |
| * noonedeadpunk haven't looked at results yet | 16:27 | |
| opendevreview | Merged openstack/openstack-ansible master: Add cinderstore glance testing scenario https://review.opendev.org/c/openstack/openstack-ansible/+/901187 | 16:29 |
| opendevreview | Merged openstack/openstack-ansible-os_glance master: Define lock directory for oslo_concurrency https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/914752 | 16:42 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Switch default Ceph deployment to the Reef release https://review.opendev.org/c/openstack/openstack-ansible/+/917014 | 20:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!