| opendevreview | Merged openstack/kolla-ansible master: ansible: bump min to 2.13 and max to 2.14 https://review.opendev.org/c/openstack/kolla-ansible/+/877697 | 01:26 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: Bump up Ansible supported versions to 6.x/7.x https://review.opendev.org/c/openstack/kayobe/+/878207 | 07:13 |
| mnasiadka | morning | 07:41 |
| mnasiadka | kevko, frickler: would you be interested in reviewing https://review.opendev.org/c/openstack/kolla-ansible/+/823395? I think it's there for a long time, and would be useful to get it merged this cycle. | 07:42 |
| kevko | mnasiadka: building master stack and i will review more things ..probably today or tomorrov | 08:04 |
| kevko | mnasiadka: https://review.opendev.org/c/openstack/kolla-ansible/+/874729 what about this ..can u ? | 08:04 |
| mnasiadka | if you'll update reno, because now it's misleading - I can +2 ;-) | 08:05 |
| priteau | Good morning. Could we also get a review of https://review.opendev.org/c/openstack/kolla-ansible/+/871279? Thanks kevko, frickler | 08:06 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/zed: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876829 | 08:26 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/yoga: rabbitmq: update to RabbitMQ 3.10 https://review.opendev.org/c/openstack/kolla/+/880799 | 08:26 |
| opendevreview | Merged openstack/kolla-ansible master: mariadb: add mariadb_datadir_volume parameter https://review.opendev.org/c/openstack/kolla-ansible/+/876649 | 08:27 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible stable/yoga: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876830 | 08:30 |
| kevko | priteau: done , | 08:31 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible master: Add precheck to fail if RabbitMQ HA needs configuring https://review.opendev.org/c/openstack/kolla-ansible/+/880274 | 08:45 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Revert back to MichaelRigart.interfaces https://review.opendev.org/c/openstack/kayobe/+/880742 | 08:47 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: opensearch: move to yum/apt repos https://review.opendev.org/c/openstack/kolla/+/880809 | 10:26 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: opensearch: move to yum/apt repos https://review.opendev.org/c/openstack/kolla/+/880809 | 10:53 |
| SvenKieske | did anybody ever try to replace flake8 with ruff (https://github.com/charliermarsh/ruff)? | 11:14 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/yoga failed: opensearch: default dashboards tag to opensearch_tag https://review.opendev.org/c/openstack/kolla-ansible/+/880599 | 11:30 |
| frickler | SvenKieske: I don't think so. it likely would only make sense if it was adopted by all of openstack and the inertia for that is pretty unsurmountable. see e.g. the recent tox vs. nox topic | 12:17 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Revert back to MichaelRigart.interfaces https://review.opendev.org/c/openstack/kayobe/+/880742 | 12:26 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/xena: Fix create sasl account before config file is ready https://review.opendev.org/c/openstack/kolla-ansible/+/880778 | 12:26 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/yoga: Fix create sasl account before config file is ready https://review.opendev.org/c/openstack/kolla-ansible/+/880777 | 12:27 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: opensearch: default dashboards tag to opensearch_tag https://review.opendev.org/c/openstack/kolla-ansible/+/880598 | 12:49 |
| mnasiadka | mgoddard mnasiadka hrw bbezak frickler kevko SvenKieske - meeting in 9 | 12:51 |
| opendevreview | Michal Nasiadka proposed openstack/kolla master: opensearch: move to yum/apt repos https://review.opendev.org/c/openstack/kolla/+/880809 | 12:52 |
| mnasiadka | #startmeeting kolla | 13:00 |
| opendevmeet | Meeting started Wed Apr 19 13:00:01 2023 UTC and is due to finish in 60 minutes. The chair is mnasiadka. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
| opendevmeet | The meeting name has been set to 'kolla' | 13:00 |
| mnasiadka | #topic rollcall | 13:00 |
| mnasiadka | o/ | 13:00 |
| mmalchuk | \o | 13:00 |
| SvenKieske | o/ | 13:00 |
| mattcrees | o/ | 13:00 |
| mhiner | o/ | 13:00 |
| frickler | \o | 13:00 |
| kevko | \o/ | 13:01 |
| mnasiadka | #topic agenda | 13:03 |
| mnasiadka | * CI status | 13:03 |
| mnasiadka | * Release tasks | 13:03 |
| mnasiadka | * Regular stable releases (first meeting in a month) | 13:03 |
| mnasiadka | * Current cycle planning | 13:03 |
| mnasiadka | * Additional agenda (from whiteboard) | 13:03 |
| mnasiadka | * Open discussion | 13:03 |
| mnasiadka | #topic CI status | 13:03 |
| mnasiadka | any other CI issues that the octavia one that needs slight fixing? | 13:04 |
| mnasiadka | hmm, can't see any | 13:04 |
| mnasiadka | #topic Release tasks | 13:05 |
| frickler | final xena release is done | 13:05 |
| frickler | will go into em later this week | 13:05 |
| mnasiadka | good, thanks frickler | 13:06 |
| mmalchuk | sad and cool news at the same time) | 13:06 |
| mnasiadka | once A is out, we might think about marking Wallaby as EOL - given the size of our team | 13:06 |
| frickler | +1 | 13:07 |
| mnasiadka | I think it's time to focus on trying to get podman merged (and other patches with RP+1) - I'll try to make some time this and next week for some focused testing of podman | 13:07 |
| mnasiadka | And will go through the release checklist, to make sure most of the preparation tasks are handled | 13:08 |
| mnasiadka | #topic Current cycle planning | 13:08 |
| mnasiadka | I haven't updated the whiteboard with priorities for Bobcat yet, but the Whiteboard also needs some tidying up after asian translation attack | 13:09 |
| kevko | i've spinned master stack so i will probably review this patches .. podman and letsencrypt | 13:09 |
| frickler | do we also want to have some kind of freeze, so that we can say other new patches will have to wait until after branching? | 13:09 |
| frickler | unless they are bug fixes of course | 13:09 |
| mnasiadka | Yes, basically nothing out of RP+1 should be reviewed and merged | 13:10 |
| mnasiadka | I mean that doesn't have review priority +1 | 13:10 |
| mmalchuk | lets merge backports with fixes and freeze | 13:10 |
| mnasiadka | and we should focus rather on master branch for a couple of weeks than the backports | 13:11 |
| SvenKieske | I see some divergence in opinion here :D | 13:11 |
| mmalchuk | Kayobe still lack of reviewers | 13:11 |
| mnasiadka | Kayobe always trails behind, that's how it is | 13:11 |
| mmalchuk | ok | 13:12 |
| mmalchuk | lets merge k/k-a backports with fixes and freeze | 13:12 |
| mnasiadka | freeze only affects master branch patches - we don't review those that don't have RP+1 | 13:12 |
| frickler | the freeze would be for master, stable branches don't need a freeze | 13:12 |
| mmalchuk | backports made from the master) | 13:13 |
| mmalchuk | some fixes in master still on review | 13:13 |
| mnasiadka | if there is anything important that you want review priority on - please post it on the IRC channel, we'll assess and mark with review priority if needed | 13:13 |
| frickler | if there is nothing urgent in backports, I'd agree with mnasiadka to focus on master for now | 13:14 |
| mnasiadka | so agreed | 13:14 |
| mnasiadka | let's move on | 13:14 |
| frickler | of course everyone has their own opinion on what is urgent ;) | 13:14 |
| mmalchuk | ok, agree | 13:15 |
| mnasiadka | #topic Additional agenda (from whiteboard) | 13:15 |
| frickler | ah, we still have to look at the other stable releases | 13:15 |
| frickler | didn't do them for april yet | 13:15 |
| mnasiadka | Ok, let's have a look on the backports | 13:16 |
| frickler | or shall we agree to skip for this month? | 13:16 |
| mnasiadka | let's skip for this month and revisit in May | 13:16 |
| mnasiadka | there's a lot of movement from what I see | 13:16 |
| mnasiadka | at least in k-a | 13:16 |
| mnasiadka | https://etherpad.opendev.org/p/2023.2-leaderless - retire Sahara, Vitrage, Monasca (already dropped) and Windows support (if we have any remnants of it)? | 13:17 |
| mnasiadka | that's the first additional topic | 13:17 |
| mnasiadka | so, we have some projects that are getting retired | 13:17 |
| mnasiadka | Vitrage is one of them | 13:17 |
| mnasiadka | Sahara seems to have survived, but I don't know if it's usable at all in Kolla | 13:18 |
| frickler | so we can deprecate now and drop in bobcat? | 13:18 |
| mnasiadka | I think so | 13:18 |
| kevko | +1 | 13:18 |
| mmalchuk | lets get rid of them all | 13:18 |
| mnasiadka | third one is Winstackers - we've had some Windows support in the past | 13:18 |
| mnasiadka | But I don't know if it's still there | 13:18 |
| mmalchuk | Windows support useless | 13:19 |
| mnasiadka | Anyone volunteering for the effort of writing deprecation renos? | 13:19 |
| SvenKieske | I only find two mentions of "windows" in conf.py under the deploy guides section | 13:19 |
| SvenKieske | and those are not about windows support, but about windows icon files (.ico) | 13:20 |
| mmalchuk | mnasiadka should we write only renos or with code removeing? | 13:20 |
| mnasiadka | seems no Windows support, I think we removed it some cycles ago | 13:20 |
| frickler | SvenKieske: is that your statement of volunteering? ;) | 13:20 |
| mnasiadka | so we need deprecation renos for now, and drop the code in master after A is branched | 13:21 |
| SvenKieske | I mean, I guess I could write relnotes, my setup for them is still not complete but that should not take much time (fingers crossed) | 13:21 |
| mmalchuk | I can try | 13:21 |
| mnasiadka | I think Sven was faster :) | 13:22 |
| mmalchuk | ok | 13:22 |
| mnasiadka | So, do we want to deprecate Sahara as well? | 13:22 |
| mmalchuk | I'll focus on the backports | 13:23 |
| SvenKieske | did never use it, so I got no opinion | 13:23 |
| mnasiadka | ok, Vitrage for now | 13:23 |
| mnasiadka | #action SvenKieske raise patch to deprecate Vitrage (to be dropped in Bobcat cycle) | 13:23 |
| mmalchuk | Sahara yes | 13:23 |
| frickler | I'd say deprecate sahara, too, wu can still undeprecate if its situation improves | 13:23 |
| mnasiadka | #action SvenKieske deprecate Sahara too | 13:24 |
| SvenKieske | hum, so questions: do we "deprecate" or do we remove support? that's two different things, no? also: do I only write relnotes, or do I remove code? :D | 13:24 |
| mnasiadka | only write renos for now, that it is deprecated and will be dropped in Bobcat cycle | 13:24 |
| SvenKieske | okay | 13:24 |
| mnasiadka | and we need to drop code in Bobcat cycle | 13:24 |
| SvenKieske | because really removing will be a little more work, I see over 300 hits for "sahara" alone | 13:25 |
| mnasiadka | (which is sort of now, but not for Kolla, since we're trailing) | 13:25 |
| frickler | yes, but likely won't help to prepare the removal now, will be too many merge conflicts | 13:25 |
| mnasiadka | Removing code is fun work, don't worry | 13:25 |
| mnasiadka | :) | 13:25 |
| SvenKieske | yeah. I know :) | 13:25 |
| mnasiadka | ok, next topic | 13:25 |
| mnasiadka | drop OpenEuler | 13:25 |
| mmalchuk | agree | 13:26 |
| mnasiadka | so - OpenEuler CI has been failing for a long time | 13:26 |
| kevko | +1 | 13:26 |
| mnasiadka | it was added at the same time as Rocky 8, but since we stopped publishing centos images - that stopped working | 13:26 |
| SvenKieske | no replies on the ML on OpenEuler so far, but you maybe could've mentioned that we're deprecating it in the subject line | 13:26 |
| mnasiadka | and nobody has picked it up | 13:26 |
| mmalchuk | SvenKieske I've replied) | 13:27 |
| mnasiadka | well, I could have, but still if somebody would be interested, they would pick it up earlier than my mail | 13:27 |
| SvenKieske | mmalchuk: apologies, I meant no replies which wanted to take over maintenance | 13:27 |
| mnasiadka | Anyway, surely we should drop the CI job and deprecate | 13:28 |
| mnasiadka | What about dropping? Should we wait for Bobcat cycle as well? | 13:28 |
| mmalchuk | think no | 13:28 |
| SvenKieske | I mean it's broken, so it's practically dead already? | 13:28 |
| mnasiadka | practically yes | 13:28 |
| frickler | https://review.opendev.org/c/openstack/kolla-ansible/+/879129 is the patch to remove the CI job | 13:28 |
| SvenKieske | so +1 for removing | 13:28 |
| mnasiadka | it doesn't even go through bootstrap-servers | 13:28 |
| mmalchuk | +1 for remove | 13:29 |
| SvenKieske | if someone interested shows up it's quickly reverted | 13:29 |
| mnasiadka | ok then | 13:29 |
| mnasiadka | #agreed to remove OpenEuler this cycle (in addition to deprecating) | 13:29 |
| mnasiadka | Anyone wants to remove the code and write the deprecation+drop reno? | 13:30 |
| mmalchuk | SvenKieske may be remove line not add comment? | 13:30 |
| mmalchuk | frickler sorry | 13:31 |
| frickler | mmalchuk: yes, if we want to drop completely, that makes sense | 13:31 |
| frickler | I'll update and also add a reno and see what else could be removed | 13:31 |
| SvenKieske | I just commented this on the patch, let's move patch discussion to gerrit, no? :) | 13:31 |
| mnasiadka | ok then, I'll w-1 | 13:31 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Stop running broken openeuler job https://review.opendev.org/c/openstack/kolla-ansible/+/879129 | 13:31 |
| mnasiadka | ok then, next one | 13:31 |
| mnasiadka | (kyarovoy): help needed with https://review.opendev.org/c/openstack/kolla/+/825786, if someone has time - please look at the problems that appear with kolla-build-ansible | 13:31 |
| mnasiadka | Anybody did have time to have a look since last week? | 13:32 |
| Fl1nt | Hi everyone! | 13:32 |
| SvenKieske | well I did look; I really don't get why we support such an old docker release, does it even work in practice? | 13:32 |
| mmalchuk | mnasiadka tests? | 13:33 |
| mnasiadka | We don't test, so we don't really should support. | 13:33 |
| mnasiadka | (about docker version) | 13:33 |
| SvenKieske | I agree about missing relno though, but there where unadressed comments anyway, so I didn't want to pile up | 13:33 |
| mnasiadka | anyway, this patch broke pushes, so I don't think that without setting up a local env we can fix it | 13:34 |
| mnasiadka | If anybody can have a look and help - then please do, I'll leave it for next week | 13:34 |
| kevko | mnasiadka: i ran recheck because logs missing :/ | 13:34 |
| SvenKieske | yeah, maybe the call for help could also be clarified: is it about the failing CI? I'm a big fan of addressing all comments, then look if the CI still fails. | 13:35 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Bump up Ansible supported versions to 6.x/7.x https://review.opendev.org/c/openstack/kayobe/+/878207 | 13:35 |
| mnasiadka | ok then, no other items in the additional agenda | 13:36 |
| mnasiadka | #topic Open discussion | 13:36 |
| ihalomi | any news on mounting /run directory? | 13:36 |
| kevko | podman ^ ? | 13:37 |
| Fl1nt | Can someone review the designate patch? | 13:37 |
| ihalomi | yes on podman | 13:37 |
| Fl1nt | https://review.opendev.org/c/openstack/kolla-ansible/+/878270 | 13:37 |
| kevko | Fl1nt which one ? | 13:37 |
| Fl1nt | kevko, too quick ^^ | 13:37 |
| mnasiadka | is it a priority for Antelope? | 13:37 |
| kevko | ihalomi: i will review podman today proabbly and i will check | 13:37 |
| SvenKieske | regarding mounting /run: afaik that was fixed by mounting all needed subdirs? I actually asked upstream at podman but afaik that's not gonna get fixed | 13:37 |
| mnasiadka | it's +131, so it's far from being trivial | 13:37 |
| Fl1nt | mnasiadka, as designate is literally broken on any branch yes | 13:38 |
| mnasiadka | Fl1nt: As long as you can find two core reviewers that sign up for reviewing that, then sure - feature freeze granted. | 13:38 |
| Fl1nt | well hence why I'm asking in here during the weekly meeting ^^ | 13:39 |
| mnasiadka | And I'm waiting for those two core reviewers :) | 13:39 |
| kevko | Fl1nt: i will review also this .. adding to my list .. | 13:39 |
| mnasiadka | Do you want to do backports of that to stable branches? | 13:39 |
| frickler | designate works fine for me, so it depends on the deployment scenario I guess | 13:39 |
| Fl1nt | up to stable/wallaby if possible | 13:39 |
| mmalchuk | can someone find two+ core reviewers for Kayobe too? | 13:39 |
| Fl1nt | frickler, it works by a hasardous combination. | 13:39 |
| kevko | what about https://review.opendev.org/c/openstack/kolla-ansible/+/877413 (multiple ceph files) , this i think can have priority https://review.opendev.org/c/openstack/kolla-ansible/+/874729 masakari | 13:39 |
| mnasiadka | Then it needs to be 100% backwards compatible | 13:39 |
| mnasiadka | and by just looking at that patch I see it's not. | 13:40 |
| Fl1nt | kevko, I've already review this one | 13:40 |
| Fl1nt | same feeling as mnasiadka | 13:40 |
| mmalchuk | kevko my comments for ceph review not checked | 13:40 |
| mnasiadka | kevko: update the reno for masakari, then we can merge it, do I need to write it 3 more times? :) | 13:40 |
| ihalomi | okay, so I will create new patchset with mounting of all subdirs needed, should it be defined in one place and then used as variable in all services or should i add it to each service dependent only on what it needs to mount? | 13:40 |
| Fl1nt | TBH: this patch about CEPH is weird as it force cross-az cinder usage where you don't want that on a CEPH cluster | 13:41 |
| kevko | i will post new patch today in hours ..but there will be only tiny changes and i will resolve comments ...but overall patch is good, isn't it ? | 13:41 |
| kevko | mnasiadka, Fl1nt understand , i will ping you in hour maybe | 13:41 |
| kevko | masakari thing ? ^^ | 13:41 |
| Fl1nt | kevko, perfect! I'll be there :D thx! | 13:41 |
| mmalchuk | Fl1nt why? one ceph can have multiple pools | 13:41 |
| mhiner | ihalomi: i think it should be added to existing podman patch | 13:41 |
| kevko | Fl1nt: nope, you just set cross-az: false in config | 13:42 |
| Fl1nt | mmalchuk, yes, but AZ isn't the proper segment, it should segment on backends then. AZ is a logical segment such as AZ-1/2/3 with each zone being an aggregate so can represent zone or hw such as HPE/DELL/SUPERMICRO | 13:42 |
| mnasiadka | ihalomi: I don't think we mount whole /run on a lot of containers | 13:42 |
| Fl1nt | kevko, yes, and it should be the default when using CEPH | 13:43 |
| mmalchuk | Fl1nt It's a formality | 13:43 |
| kevko | it's not weird , juju has the same implementation | 13:44 |
| Fl1nt | mmalchuk, on our setup being HPC/Converged it have impact. | 13:44 |
| mmalchuk | there more setups in the world) | 13:44 |
| ihalomi | mnasiadka: so listing it in defaults of each service using it is fine? | 13:44 |
| SvenKieske | those ceph patches are in general good, I don't get though why they break backward compatibility regarding keyring names and introduce inconsistencies as well. | 13:44 |
| Fl1nt | mmalchuk, only three exist, either CEPH as a full storage cluster or converged or mix of both, in all case you do not define AZ on CEPH but on Compute aggregate, each zone can access all backend pools or a set of pools | 13:45 |
| mnasiadka | ihalomi: you can have a variable in group_vars/all and just reference the variable where you need it | 13:45 |
| mmalchuk | it can break if custom files used before. we shouldn't hardcode order of the backends via list[0] list[1] etc. | 13:46 |
| Fl1nt | mmalchuk, exactly. | 13:46 |
| mmalchuk | my comments in code about them | 13:46 |
| ihalomi | mnasiadka: okay, i will try to do it this week | 13:46 |
| kevko | SvenKieske: beacuse it is from beginning bad ... ceph.client.cinder.keyring is ceph - name of cluster, client - cinder ... if you have azs ..you have to distinguish ceph cluster names | 13:46 |
| kevko | SvenKieske : The $cluster metavariable is your Ceph cluster name as defined by the name of the Ceph configuration file (i.e., ceph.conf means the cluster name is ceph; thus, ceph.keyring). The $name metavariable is the user type and user ID (e.g., client.admin; thus, ceph.client.admin.keyring). | 13:47 |
| SvenKieske | kevko: but it breaks backward compatibility for just a name cleanup, hard disagree with that | 13:47 |
| SvenKieske | kevko: when that should really be fixed, fix it everywhere, not only for 1 variable. | 13:48 |
| kevko | SvenKieske: no, it is backward compatible ... it will brae deployment only if you override default values in global.conf | 13:48 |
| Fl1nt | SvenKieske, kolla/kolla-ansible try to address a "generic" approach, meaning you need to stick with fact that the service you implement need to stick with based service best practices + variabilization for flexibility and maximum scenario support. | 13:48 |
| SvenKieske | the gnocchi key is still named wrong than, anyway let's move patch discussion to gerrit, where it belongs :) | 13:48 |
| mnasiadka | yes, patch discussion in gerrit | 13:49 |
| mnasiadka | :) | 13:49 |
| kevko | SvenKieske and if you overriden defaults by yourself ..you should read reno for upgrades where it is explained ... | 13:49 |
| kevko | agree | 13:49 |
| mmalchuk | SvenKieske we should have _default variable and the _actual one for backward support | 13:49 |
| SvenKieske | kevko: agreed :) | 13:49 |
| kevko | last thing - after ceph merge .. kolla-ansible will support azs out of the box ... only aggreagations will be missing ..but it would be small patch to implement ... | 13:51 |
| kevko | and that is cool thing | 13:51 |
| Fl1nt | kevko, aggregation? | 13:51 |
| SvenKieske | question about meeting process: wasn't it the case in the past that the meeting logs are posted to openstack-discuss? did I just miss that mail? | 13:51 |
| mmalchuk | small patch? | 13:51 |
| ihalomi | about this patchset adding podman to ansible-collection https://review.opendev.org/c/openstack/ansible-collection-kolla/+/852240 do we need more CI test or the ones that are in podman patchset are enough? | 13:51 |
| Fl1nt | AZ is placement aggregate, what did I missed there kevko ? | 13:52 |
| mnasiadka | there is no such thing as a small patch, unless it's a version bump of a minor component in a Kolla image ;-) | 13:52 |
| kevko | yep https://docs.openstack.org/nova/latest/admin/availability-zones.html | 13:52 |
| mnasiadka | ihalomi: I think the ones are enough, but as a final solution we need those podman jobs to be run also in ansible-collection-kolla | 13:52 |
| mmalchuk | mnasiadka right) | 13:52 |
| kevko | Fl1nt: it's more things ..i will propose several patches ..and you will see :) | 13:53 |
| mnasiadka | ok then, patch discussions after the meeting or in Gerrit | 13:53 |
| mnasiadka | Last stupid question - anybody going to Vancouver? | 13:53 |
| Fl1nt | kevko, hum... don't get it, you create an AZ by associating placement metadata with host aggregates. | 13:53 |
| kevko | nope :( | 13:53 |
| SvenKieske | it's sad that there are no virtual tickets for vancouver | 13:53 |
| ihalomi | mnasiadka: okay so no need to work on that for now | 13:53 |
| kevko | Fl1nt: i will write you DM if you are curious | 13:53 |
| Fl1nt | mnasiadka, nope, waiting for EU based one ^^ | 13:53 |
| frickler | it is sad that the foundation still does in-person events at all | 13:53 |
| Fl1nt | kevko, mosdef! | 13:54 |
| Fl1nt | frickler, in-person is cool, but they should do smaller one more often and spread everywhere as GDG | 13:54 |
| mnasiadka | Ok, Kolla User Forum session was accepted, but probably will be lower attendance than last time in Berlin | 13:54 |
| mnasiadka | Anyway - I need to run | 13:54 |
| mnasiadka | thanks for attending and lively conversations | 13:55 |
| Fl1nt | mnasiadka, see ya | 13:55 |
| mnasiadka | See you next week! | 13:55 |
| mnasiadka | #endmeeting | 13:55 |
| opendevmeet | Meeting ended Wed Apr 19 13:55:09 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:55 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-19-13.00.html | 13:55 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-19-13.00.txt | 13:55 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-19-13.00.log.html | 13:55 |
| mmalchuk | mnasiadka thanks | 13:55 |
| mmalchuk | mnasiadka cloul you please also +w https://review.opendev.org/c/openstack/kayobe/+/833634 | 13:56 |
| mmalchuk | could) | 13:56 |
| mnasiadka | done | 13:56 |
| mmalchuk | thanks | 13:56 |
| mnasiadka | kevko: https://review.opendev.org/c/openstack/kolla/+/873913 - do we need that? | 13:56 |
| mmalchuk | cores please revoew backports: https://review.opendev.org/q/topic:glance-api-multipath and https://review.opendev.org/q/topic:dib-yaml-width | 13:57 |
| mmalchuk | k-a ofcource | 13:57 |
| mmalchuk | mnasiadka another one for +w is https://review.opendev.org/c/openstack/kayobe/+/859842 | 14:00 |
| kevko | mnasiadka: well, it is some time ago we discussed this and i don't know why CI is OK to build horizon without that ..but i added this patch to our downstream repo ..because horizon was unbuildable | 14:01 |
| SvenKieske | mhm, we don't even have antelope relnotes just yet, no? | 14:01 |
| kevko | mnasiadka: i don't know what is the situation for now ...but you can try to build horizon locally on laptop from yoga branch and you will see | 14:01 |
| frickler | SvenKieske: current master reno's will become antelope after branching | 14:05 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Pass through kolla inventories as is https://review.opendev.org/c/openstack/kayobe/+/802864 | 14:07 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Use merge_configs and merge_yaml to generate Kolla custom config https://review.opendev.org/c/openstack/kayobe/+/782749 | 14:07 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Add cached plugin https://review.opendev.org/c/openstack/kayobe/+/803064 | 14:07 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Kayobe environment dependencies https://review.opendev.org/c/openstack/kayobe/+/802865 | 14:07 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Use merge_configs and merge_yaml to generate Kolla custom config https://review.opendev.org/c/openstack/kayobe/+/782749 | 14:09 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Add cached plugin https://review.opendev.org/c/openstack/kayobe/+/803064 | 14:09 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Kayobe environment dependencies https://review.opendev.org/c/openstack/kayobe/+/802865 | 14:09 |
| opendevreview | Merged openstack/kolla-ansible stable/yoga: opensearch: default dashboards tag to opensearch_tag https://review.opendev.org/c/openstack/kolla-ansible/+/880599 | 14:13 |
| mmalchuk | mgoddard please review https://review.opendev.org/c/openstack/kayobe/+/845338 and https://review.opendev.org/c/openstack/kayobe/+/861397 | 14:15 |
| opendevreview | Merged openstack/kolla-ansible master: Use the upgraded image to run Nova upgrade checks https://review.opendev.org/c/openstack/kolla-ansible/+/871279 | 14:16 |
| SvenKieske | frickler: yeah so the "unreleased" ones? they are empty as well | 14:30 |
| *** Guest9750 is now known as atmark | 14:32 | |
| frickler | SvenKieske: this looks non-empty to me https://docs.openstack.org/releasenotes/kolla/unreleased.html same for k-a | 14:39 |
| kevko | btw, sorry but i forgot to ask , what are we going to do with reviews which are years old ? in most cases owners will not work with openstack anymore | 14:39 |
| kevko | especially in k-a | 14:40 |
| frickler | kevko: it should still be a case-by-case decision IMHO, patches can still be useful | 14:40 |
| kevko | yeah of course, but i've spent 2 days by reviewing all ...and found that most of the reviews has unanswered comments, or merge conflicts ..or for example xen bugfixes :D | 14:41 |
| kevko | what i remember is that i abandon xen patch myself because we removed long time ago | 14:42 |
| SvenKieske | frickler: weird, just was looking at my local source branch. I guess I need to relearn how the relnotes work in k-a | 14:45 |
| mmalchuk | you forgot to review my patches but promices) | 14:45 |
| frickler | in the PTG said we could spend some time on it each week if the meeting takes less than 60m, didn't happen yet but we'll see | 14:45 |
| frickler | SvenKieske: in a local copy you should be able to run "tox -e releasenotes" to generate them from the sources | 14:46 |
| frickler | but that should be the same for just about any openstack project | 14:47 |
| mmalchuk | frickler better something like "./build/bin/tox -e venv -- reno ..." | 14:47 |
| mmalchuk | using venvs much better | 14:47 |
| frickler | no, better do the same thing that is done for the actual publishing later | 14:47 |
| frickler | tox uses venvs anyway | 14:48 |
| mmalchuk | if -e evenv passed only) | 14:48 |
| mmalchuk | " -e envlist work against specified environments (ALL selects all). (default: None)" | 14:49 |
| mmalchuk | default: None ! | 14:49 |
| SvenKieske | frickler: thx for the reminder! | 14:50 |
| frickler | "tox -e releasenotes" creates its own venv in .tox/releasenotes, still a venv | 14:50 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/zed: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876829 | 14:51 |
| mmalchuk | ok, but you need tox and deps. better to install it in own venv | 14:51 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: Drop support for openEuler https://review.opendev.org/c/openstack/kolla-ansible/+/879129 | 14:52 |
| opendevreview | Gaël THEROND proposed openstack/kolla-ansible master: Improve designate role support. https://review.opendev.org/c/openstack/kolla-ansible/+/878270 | 14:52 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/zed: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876829 | 14:52 |
| SvenKieske | as I promised, need to setup my relnotes stuff again, pip install fails with some pcre header not present for gcc..hope it doesn't need long deprecated pcvre_v1 | 14:52 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/yoga: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876830 | 14:53 |
| frickler | SvenKieske: make sure you use recent pip, else upstream wheels might not get used | 14:53 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/yoga: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876830 | 14:54 |
| mmalchuk | SvenKieske MacOS ? | 14:54 |
| Fl1nt | SvenKieske, patched designate spelling according to your recommandations. | 14:54 |
| SvenKieske | lol, no macos | 14:55 |
| mmalchuk | I have the same issue on MacOS and fixed it | 14:55 |
| SvenKieske | pip 22.2.2 new enough? | 14:55 |
| SvenKieske | python package management still a shitshow :( | 14:55 |
| mmalchuk | fix: export CPPFLAGS="-I/usr/local/include $CPPFLAGS" | 14:56 |
| mmalchuk | and pcre installed | 14:56 |
| mmalchuk | check the paths when pip build binaries | 14:57 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/xena: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/880833 | 14:58 |
| SvenKieske | it was just missing pcre.h headers; fixed by installing pcre-devel | 14:59 |
| mmalchuk | cool | 15:01 |
| kevko | SvenKieske: please check my answers in ceph patch and let me know if you got it | 15:01 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/xena: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/880833 | 15:02 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/yoga: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876830 | 15:03 |
| SvenKieske | kevko: replied | 15:05 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/yoga: opensearch: Drop container_engine from kolla_toolbox arguments during upgrade https://review.opendev.org/c/openstack/kolla-ansible/+/880856 | 15:05 |
| kevko | SvenKieske: cool, agree | 15:06 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Adds support for custom Multipathd configuration. https://review.opendev.org/c/openstack/kayobe/+/879190 | 15:19 |
| opendevreview | Matt Crees proposed openstack/kolla-ansible master: Add check_mode: false to Nova upgrade checks https://review.opendev.org/c/openstack/kolla-ansible/+/880860 | 15:23 |
| kevko | mnasiadka https://review.opendev.org/c/openstack/kolla-ansible/+/874729? ? your comment done .. | 15:38 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Use merge_configs and merge_yaml to generate Kolla custom config https://review.opendev.org/c/openstack/kayobe/+/782749 | 15:40 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Add cached plugin https://review.opendev.org/c/openstack/kayobe/+/803064 | 15:40 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Kayobe environment dependencies https://review.opendev.org/c/openstack/kayobe/+/802865 | 15:40 |
| opendevreview | Gaël THEROND proposed openstack/kolla-ansible master: Improve designate role support. https://review.opendev.org/c/openstack/kolla-ansible/+/878270 | 16:00 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Generate local Kolla Ansible config in check mode https://review.opendev.org/c/openstack/kayobe/+/869986 | 16:01 |
| opendevreview | Mark Goddard proposed openstack/kayobe master: Allow to use own repos.y[a]ml file to build containers https://review.opendev.org/c/openstack/kayobe/+/845338 | 16:06 |
| Fl1nt | SvenKieske, men, this designate patch isn't disruptive and just work out of the box even for existing deployment, I've tested it somewhere around 35 times between our PROD and staging environment on Victoria/Wallaby/Yoga branches, could we move on? | 16:07 |
| SvenKieske | did you also test upgrades? | 16:08 |
| Fl1nt | from victoria to yoga yes | 16:08 |
| SvenKieske | so, as already stated, I really like the support for multiple pools, but I still don't understand what get's fixed here, that should be clearly stated somewhere: bugreport, commit message, code comment | 16:12 |
| Fl1nt | I've explained it on the latest commit, there are 5 bug related to that. | 16:13 |
| Fl1nt | basically, the pools.yaml.j2 miss an ID that prohibit the designate-manage update command to update the existing pools accordingly | 16:15 |
| Fl1nt | plus | 16:15 |
| Fl1nt | the neutron integration wasn't appropriate, which prohibit the correct floating IPs integration and release. It btw means that any current prod deployment using neutron dns external integration is broken and FIP can't be deleted anymore. | 16:16 |
| SvenKieske | I just fear the docs than need an update as well? https://docs.openstack.org/kolla-ansible/latest/reference/networking/designate-guide.html | 16:22 |
| Fl1nt | If it's just that I can do it. | 16:23 |
| opendevreview | Merged openstack/kayobe master: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kayobe/+/833634 | 16:29 |
| opendevreview | Merged openstack/kayobe master: Fix an issue when 'acl' package can be forgotten https://review.opendev.org/c/openstack/kayobe/+/859842 | 16:29 |
| opendevreview | Merged openstack/kolla stable/zed: Replace invisible unicode chars https://review.opendev.org/c/openstack/kolla/+/878665 | 16:44 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Revert back to MichaelRigart.interfaces https://review.opendev.org/c/openstack/kayobe/+/880742 | 16:47 |
| opendevreview | Merged openstack/kolla stable/yoga: Replace invisible unicode chars https://review.opendev.org/c/openstack/kolla/+/878666 | 17:11 |
| opendevreview | Merged openstack/kolla-ansible master: Add precheck to fail if RabbitMQ HA needs configuring https://review.opendev.org/c/openstack/kolla-ansible/+/880274 | 17:11 |
| opendevreview | Merged openstack/kolla-ansible master: Update notes about CentOS support https://review.opendev.org/c/openstack/kolla-ansible/+/878698 | 17:11 |
| opendevreview | Merged openstack/kolla master: Update Stackalytics links https://review.opendev.org/c/openstack/kolla/+/877346 | 17:11 |
| mmalchuk | mnasiadka please +w https://review.opendev.org/c/openstack/kayobe/+/845338 | 18:08 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/zed: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kayobe/+/880875 | 18:14 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/yoga: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kayobe/+/880876 | 18:15 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/xena: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kayobe/+/880877 | 18:15 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/zed: Fix an issue when 'acl' package can be forgotten https://review.opendev.org/c/openstack/kayobe/+/880878 | 18:21 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/yoga: Fix an issue when 'acl' package can be forgotten https://review.opendev.org/c/openstack/kayobe/+/880879 | 18:22 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/xena: Fix an issue when 'acl' package can be forgotten https://review.opendev.org/c/openstack/kayobe/+/880880 | 18:22 |
| opendevreview | Merged openstack/kolla-ansible master: cli: fix find globals.d https://review.opendev.org/c/openstack/kolla-ansible/+/879154 | 18:29 |
| opendevreview | Merged openstack/kayobe master: Adds support for custom Multipathd configuration. https://review.opendev.org/c/openstack/kayobe/+/879190 | 18:48 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/zed: Adds support for custom Multipathd configuration. https://review.opendev.org/c/openstack/kayobe/+/880881 | 18:58 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/yoga: Adds support for custom Multipathd configuration. https://review.opendev.org/c/openstack/kayobe/+/880882 | 19:11 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe stable/xena: Adds support for custom Multipathd configuration. https://review.opendev.org/c/openstack/kayobe/+/880883 | 19:13 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/zed: cli: fix find globals.d https://review.opendev.org/c/openstack/kolla-ansible/+/880885 | 19:26 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/yoga: cli: fix find globals.d https://review.opendev.org/c/openstack/kolla-ansible/+/880886 | 19:27 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Trivial fix - add int filter for rabbitmq definitions https://review.opendev.org/c/openstack/kolla-ansible/+/880887 | 19:28 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible stable/xena: cli: fix find globals.d https://review.opendev.org/c/openstack/kolla-ansible/+/880888 | 19:28 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla-ansible master: octavia: Fix hm-interface after bumping openstack collection https://review.opendev.org/c/openstack/kolla-ansible/+/878209 | 19:56 |
| frickler | mnasiadka: ^^ that's PS26 again, let's see if that works and do the further cleanup in an additional patch | 19:57 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: Fix maximum width of the DIB Multiline-YAML https://review.opendev.org/c/openstack/kolla-ansible/+/880756 | 20:35 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: Remove RabbitMQ ha-all policy when not required https://review.opendev.org/c/openstack/kolla-ansible/+/876829 | 20:35 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: Fix create sasl account before config file is ready https://review.opendev.org/c/openstack/kolla-ansible/+/880776 | 20:35 |
| -opendevstatus- NOTICE: The Etherpad service on etherpad.opendev.org will be offline for the next 90 minutes for a server replacement and operating system upgrade | 21:58 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!