| opendevreview | Goutham Pacha Ravi proposed openstack/kolla-ansible master: [manila] Drop tenant_id templating from v2 endpoint https://review.opendev.org/c/openstack/kolla-ansible/+/802657 | 06:03 |
|---|---|---|
| *** rpittau|afk is now known as rpittau | 07:02 | |
| haerwu | morning | 07:05 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [WIP] config: allow to set group for user https://review.opendev.org/c/openstack/kolla/+/802663 | 07:28 |
| haerwu | for kevko and his haclient stuff | 07:29 |
| mnasiadka | morning | 07:57 |
| haerwu | hi mnasiadka kevko | 08:14 |
| kevko | haerwu: good morning :) | 08:14 |
| haerwu | kevko: 09:28 < opendevreview> Marcin Juszkiewicz proposed openstack/kolla master: [WIP] config: allow to set group for user https://review.opendev.org/c/openstack/kolla/+/802663 | 08:14 |
| haerwu | kevko: check will it help you with haclient stuff | 08:15 |
| kevko | haerwu: i just wanted to implement it :P | 08:15 |
| kevko | haerwu: will try .. btw ..i've added info to bugreport where is a problem with hacluster | 08:16 |
| kevko | have you seen already ? | 08:16 |
| haerwu | kevko: ah, sorry then for being too fast. | 08:16 |
| haerwu | nope | 08:16 |
| haerwu | in a middle of writing something for my blog | 08:16 |
| kevko | haerwu: check https://bugs.launchpad.net/kolla/+bug/1934753 last comment .. | 08:16 |
| kevko | haerwu: there is a diff of users debian-source and second is debian-binary | 08:17 |
| kevko | haerwu: maybe we should also add tcpdump and messagebus to base image to avoid similar issue in future | 08:18 |
| kevko | i mean users | 08:18 |
| haerwu | why not | 08:18 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: DNM: test barbican logging https://review.opendev.org/c/openstack/kolla-ansible/+/802668 | 08:37 |
| haerwu | mgoddard: present? | 08:59 |
| haerwu | anyway found | 09:00 |
| mgoddard | haerwu: hi | 09:06 |
| haerwu | mgoddard: is term 'short people' fine in English? | 09:07 |
| mgoddard | haerwu: for people who are short? | 09:08 |
| mgoddard | yes | 09:08 |
| haerwu | thanks | 09:10 |
| haerwu | It is good to have own space for living. I can alter it in any way I want. | 09:10 |
| haerwu | Without checking will it be useful for short people. | 09:10 |
| haerwu | writing blog post | 09:11 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add hacluster system user in haclient group https://review.opendev.org/c/openstack/kolla/+/802671 | 09:13 |
| kevko | haerwu: https://review.opendev.org/c/openstack/kolla/+/802671 << maybe it would be nice to define also home directory ? or do you think better is to edit user in kolla_start for every image ? | 09:15 |
| haerwu | kevko: feel free to edit my WIP patch to drop both WIP and test1 entry | 09:15 |
| haerwu | kevko: I think that it can be expanded once we are at it. | 09:15 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Fix various exporter tiny issues such as: https://review.opendev.org/c/openstack/kolla/+/800499 | 09:17 |
| haerwu | rebase | 09:17 |
| kevko | haerwu: it's not big problem i think ...but if you've checked diff between hacluster_pacemaker and masakari_hostmonitor ..you can see there are differences in home directories ..and it's because start strict is amending masakari user for example (if i am correct and remember well) | 09:17 |
| kevko | ^^ in LP | 09:18 |
| haerwu | kevko: if defining one homedir will improve situation then go for it? | 09:18 |
| kevko | haerwu: The group `haclient' already exists and is not a system group. Exiting. << | 09:42 |
| haerwu | https://marcin.juszkiewicz.com.pl/2021/07/28/i-moved-to-the-new-flat/ | 09:44 |
| haerwu | kevko: look at 109706aa8b26f6f915e1cadae89de8135c3b68bf commit and do similar stuff for groups? | 09:45 |
| kevko | haerwu: yeah, just looking .. | 09:46 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add hacluster system user in haclient group https://review.opendev.org/c/openstack/kolla/+/802671 | 09:50 |
| opendevreview | Michal Arbet proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones https://review.opendev.org/c/openstack/kolla/+/802676 | 09:50 |
| opendevreview | Verification of a change to openstack/kolla failed: Add boto3 as s3 dependency for Glance container https://review.opendev.org/c/openstack/kolla/+/797355 | 09:51 |
| opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: WIP: Adding support of podman deployment https://review.opendev.org/c/openstack/kolla-ansible/+/799229 | 10:07 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible master: nova: Use cinder user for Ceph https://review.opendev.org/c/openstack/kolla-ansible/+/798851 | 10:08 |
| kevko | haerwu: wondering if script for useradd should be changed and set every home to /var/lib/{username} | 10:41 |
| kevko | haerwu: and add option to override if needed | 10:41 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Fix capitalisation of Ansible task name https://review.opendev.org/c/openstack/kayobe/+/802242 | 10:44 |
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: seed: Allow disabling of SNAT configuration https://review.opendev.org/c/openstack/kayobe/+/802684 | 10:51 |
| opendevreview | Lin PeiWen proposed openstack/kolla-ansible master: Delete haproxy_single_service_listen.cfg.j2 template https://review.opendev.org/c/openstack/kolla-ansible/+/797883 | 10:53 |
| opendevreview | Merged openstack/kolla stable/victoria: Fix prometheus-elasticsearch-exporter container. https://review.opendev.org/c/openstack/kolla/+/802207 | 10:58 |
| opendevreview | Michal Arbet proposed openstack/kolla master: [WIP] config: allow to set group for user https://review.opendev.org/c/openstack/kolla/+/802663 | 11:16 |
| opendevreview | Michal Arbet proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones https://review.opendev.org/c/openstack/kolla/+/802676 | 11:16 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add some system users to fix user uuid inconsistencies https://review.opendev.org/c/openstack/kolla/+/802671 | 11:16 |
| opendevreview | Michal Arbet proposed openstack/kolla master: [WIP] config: Set default home to /var/lib/{user} https://review.opendev.org/c/openstack/kolla/+/802685 | 11:16 |
| opendevreview | Michal Arbet proposed openstack/kolla master: [WIP] config: Set default home to /var/lib/{user} https://review.opendev.org/c/openstack/kolla/+/802685 | 11:18 |
| opendevreview | Michal Arbet proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones https://review.opendev.org/c/openstack/kolla/+/802676 | 11:18 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add some system users to fix user uuid inconsistencies https://review.opendev.org/c/openstack/kolla/+/802671 | 11:18 |
| opendevreview | Merged openstack/kolla-ansible master: Fix variable names in Octavia documentation https://review.opendev.org/c/openstack/kolla-ansible/+/801624 | 11:22 |
| opendevreview | Lin PeiWen proposed openstack/kolla-ansible master: Delete haproxy_single_service_listen.cfg.j2 template https://review.opendev.org/c/openstack/kolla-ansible/+/797883 | 11:46 |
| kevko | haerwu: could you check that reviews ? https://review.opendev.org/q/topic:%22change-802663-1%22+(status:open%20OR%20status:merged) | 12:01 |
| opendevreview | Merged openstack/kolla stable/ussuri: Fix missing templating block for kolla-toolbox. https://review.opendev.org/c/openstack/kolla/+/802354 | 12:25 |
| opendevreview | Merged openstack/kolla stable/victoria: Fix missing templating block for kolla-toolbox. https://review.opendev.org/c/openstack/kolla/+/802353 | 12:26 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible stable/wallaby: Fix variable names in Octavia documentation https://review.opendev.org/c/openstack/kolla-ansible/+/802472 | 12:28 |
| opendevreview | Pierre Riteau proposed openstack/kolla-ansible stable/victoria: Fix variable names in Octavia documentation https://review.opendev.org/c/openstack/kolla-ansible/+/802473 | 12:29 |
| opendevreview | Merged openstack/kolla stable/wallaby: Fix missing templating block for kolla-toolbox. https://review.opendev.org/c/openstack/kolla/+/802352 | 12:43 |
| opendevreview | Merged openstack/kolla master: Add boto3 as s3 dependency for Glance container https://review.opendev.org/c/openstack/kolla/+/797355 | 12:43 |
| opendevreview | Skylar Tristan Kelty proposed openstack/kolla-ansible master: Fix Manila deploy when non-default name used https://review.opendev.org/c/openstack/kolla-ansible/+/802702 | 13:05 |
| opendevreview | Skylar Tristan Kelty proposed openstack/kolla-ansible master: Fix Manila deploy when non-default name used https://review.opendev.org/c/openstack/kolla-ansible/+/802702 | 13:06 |
| opendevreview | Michal Nasiadka proposed openstack/kayobe master: dnf: add metalink argument to custom dnf repository https://review.opendev.org/c/openstack/kayobe/+/802703 | 13:08 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/wallaby: Use ansible_facts to reference facts https://review.opendev.org/c/openstack/kolla-ansible/+/802704 | 13:11 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Add twentyFiveGigE to port descriptions in inspector rules https://review.opendev.org/c/openstack/kayobe/+/802709 | 13:16 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/victoria: docs: Add information on tuning Ansible https://review.opendev.org/c/openstack/kolla-ansible/+/802710 | 13:18 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/victoria: Use ansible_facts to reference facts https://review.opendev.org/c/openstack/kolla-ansible/+/802711 | 13:18 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/ussuri: docs: Add information on tuning Ansible https://review.opendev.org/c/openstack/kolla-ansible/+/802712 | 13:24 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/ussuri: Use ansible_facts to reference facts https://review.opendev.org/c/openstack/kolla-ansible/+/802713 | 13:24 |
| opendevreview | Lin PeiWen proposed openstack/kolla-ansible master: Delete haproxy_single_service_listen.cfg.j2 template https://review.opendev.org/c/openstack/kolla-ansible/+/797883 | 13:33 |
| kevko | it seems grafana oss repo is broken :( | 13:33 |
| parallax | kevko: it's not - but I think infra mirrors has the error cached | 14:05 |
| kevko | parallax: https://packages.grafana.com/oss/rpm/repodata/bae9dd764624ecd656892f9732b9e28b98d0f255214e0b136e3e0c38931f2784-filelists.xml.gz | 14:06 |
| parallax | I just built it successfully https://paste.opendev.org/show/807768/ | 14:07 |
| parallax | Ubuntu works too | 14:08 |
| opendevreview | Merged openstack/kayobe master: Remove stale DIB_DISABLE_KERNEL_CLEANUP references https://review.opendev.org/c/openstack/kayobe/+/799853 | 14:38 |
| opendevreview | Merged openstack/kayobe master: Fix capitalisation of Ansible task name https://review.opendev.org/c/openstack/kayobe/+/802242 | 14:38 |
| mgoddard | #startmeeting kolla | 15:00 |
| opendevmeet | Meeting started Wed Jul 28 15:00:59 2021 UTC and is due to finish in 60 minutes. The chair is mgoddard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'kolla' | 15:00 |
| mgoddard | #topic rollcall | 15:01 |
| mgoddard | \o | 15:01 |
| yoctozepto | o/ | 15:01 |
| priteau | -o | 15:02 |
| haerwu | _[o][o]_ | 15:02 |
| mgoddard | mgoddard mnasiadka hrw egonzalez yoctozepto rafaelweingartne cosmicsound osmanlicilegi bbezak parallax Fl1nt | 15:04 |
| mgoddard | ^ meeting time | 15:04 |
| parallax | o/ | 15:05 |
| mgoddard | #topic agenda | 15:07 |
| mgoddard | * Roll-call | 15:07 |
| mgoddard | * Agenda | 15:07 |
| mgoddard | * Announcements | 15:07 |
| mgoddard | * Review action items from the last meeting | 15:07 |
| mgoddard | * CI status | 15:07 |
| mgoddard | * Release tasks | 15:07 |
| mgoddard | * Xena cycle planning | 15:07 |
| mgoddard | ** Xena feature prioritisation https://docs.google.com/spreadsheets/d/1BuVMwP8eLnOVJDX8f3Nb6hCrNcNpRQl57T2ENU9Xao8/edit?usp=sharing | 15:07 |
| mnasiadka | o/ | 15:07 |
| mgoddard | * Kolla operator pain points https://etherpad.opendev.org/p/pain-point-elimination | 15:07 |
| mgoddard | * Open discussion | 15:07 |
| mgoddard | #topic announcements | 15:07 |
| mgoddard | None from me | 15:07 |
| mgoddard | Anyone else? | 15:07 |
| mgoddard | #topic Review action items from the last meeting | 15:08 |
| mgoddard | None, but I have proposed some stable releases which was a recent action | 15:09 |
| mgoddard | #topic CI status | 15:09 |
| mgoddard | #link https://etherpad.opendev.org/p/KollaWhiteBoard | 15:09 |
| mgoddard | I've seen a few networking issues recently | 15:09 |
| mgoddard | I think intermittent | 15:09 |
| mgoddard | Are there any other issues? | 15:10 |
| parallax | Grafana is still causing issues, probably depending on CI vendor | 15:10 |
| yoctozepto | you mean on infra mirrors? | 15:11 |
| parallax | Some mirrors might still have the erroneous package metadata cached while others don't | 15:11 |
| yoctozepto | could be | 15:11 |
| parallax | yoctozepto: yeah - I'm wasn't sure whom to talk to about it | 15:11 |
| yoctozepto | it could also be CDN issue | 15:11 |
| yoctozepto | #opendev | 15:11 |
| parallax | upstream problem is already resolved | 15:11 |
| haerwu | iirc there is no cache for grafana | 15:12 |
| yoctozepto | yeah, but I mean often it is that some upstream CDN is flaky and some mirrors rely on it (as some mirrors are just caching proxies) | 15:12 |
| haerwu | or is it? | 15:12 |
| yoctozepto | have not checked, relying on parallax here | 15:12 |
| parallax | I'm just saying building it works on my machine :) In CI - intermittent | 15:13 |
| mgoddard | #topic Release tasks | 15:14 |
| mgoddard | I don't think we have anything this week | 15:15 |
| mgoddard | #link https://docs.openstack.org/kolla/latest/contributor/release-management.html | 15:15 |
| mgoddard | (yay, merged) | 15:16 |
| mgoddard | currently we are R-10 | 15:16 |
| mgoddard | so binary xena in 2 weeks time | 15:16 |
| mgoddard | do we have any more TODOs/cleanup left? | 15:16 |
| yoctozepto | no idea, I'm still lagging behind | 15:17 |
| mgoddard | does anyone want to check what we've done and what's left? | 15:17 |
| mgoddard | tough crowd | 15:18 |
| mgoddard | #topic Xena feature prioritisation | 15:19 |
| mgoddard | #link https://docs.google.com/spreadsheets/d/1BuVMwP8eLnOVJDX8f3Nb6hCrNcNpRQl57T2ENU9Xao8/edit?usp=sharing | 15:19 |
| mgoddard | I extracted patches from gerrit, and put them into a sheet | 15:19 |
| yoctozepto | looking | 15:20 |
| mgoddard | I went through and assigned a Type to each | 15:20 |
| mgoddard | so now we can filter for features, bugfixes, etc | 15:20 |
| yoctozepto | wow | 15:20 |
| mgoddard | for kayobe, we went through together and assigned some patches to Xena, and gave them a numeric priority | 15:21 |
| haerwu | grat | 15:21 |
| mgoddard | then they can be ordered and put into the whiteboard for reviewers | 15:21 |
| yoctozepto | it's missing one thing | 15:21 |
| yoctozepto | the actual project | 15:21 |
| mgoddard | ? | 15:21 |
| mgoddard | see tabs at the bottom | 15:22 |
| yoctozepto | ah, I see tabs | 15:22 |
| yoctozepto | yeah | 15:22 |
| mgoddard | I'm not sure that a community-wide priority makes sense for kolla | 15:22 |
| mgoddard | but what we could do is allow people to register interest to | 15:23 |
| haerwu | I think that we can identify quick/trivial changes and get them in to get list shorter | 15:23 |
| mgoddard | 1. contribute | 15:23 |
| mgoddard | 2. review | 15:23 |
| mgoddard | 3. use | 15:23 |
| yoctozepto | interesting approach | 15:23 |
| mgoddard | ultimately resulting in a system where contributors can get reviewers to sign up to review their code | 15:24 |
| yoctozepto | now only need to support payments in bitcoin :-) | 15:24 |
| mgoddard | a big downside is that this is a snapshot, and not easily updated for new patches | 15:24 |
| mgoddard | but it only took a couple of hours, and it could be done manually from there to add new features to a list | 15:25 |
| mgoddard | I suppose the ultimate aim is to allow us to focus on reviewing important patches | 15:26 |
| mgoddard | rather than being gerrit email event driven | 15:26 |
| mgoddard | or small patch dopamine driven | 15:26 |
| mgoddard | if anyone has ideas of how to use this data to help, I'm listening | 15:27 |
| yoctozepto | yeah, we are looking for oxytocin here | 15:27 |
| yoctozepto | well, it could make sense to propose an overlay system on gerrit to better manage these efforts | 15:28 |
| yoctozepto | this looks like the first (manual) PoC on that | 15:28 |
| jovial` | Could you make the script you used to generte it smarter so that priorities, etc. were preserved on update? | 15:28 |
| yoctozepto | mhm, that would be a good progress forward | 15:28 |
| yoctozepto | we can wire something simple up with sqlite | 15:28 |
| yoctozepto | or even just json-file-based database | 15:29 |
| yoctozepto | that would be easiest | 15:29 |
| haerwu | what is wrong with using RP+1/RP+2 for marking trivial/important patches? | 15:29 |
| yoctozepto | haerwu: it doesn't really give enough info | 15:29 |
| mgoddard | how do we decide which are important? | 15:29 |
| yoctozepto | precisely | 15:30 |
| haerwu | ok | 15:30 |
| mgoddard | gerrit doesn't do a lot to help us here | 15:30 |
| parallax | Bugs have their priority | 15:30 |
| yoctozepto | it would be cool to have this flow | 15:30 |
| yoctozepto | change comes in -> gets assigned a type -> gets assigned the priority (or left for discussion on that) -> gets reviewed thoroughly | 15:31 |
| mgoddard | any solution that is itself feature sized would probably negate this work :) | 15:31 |
| haerwu | we have hashtags in gerrit | 15:31 |
| mgoddard | only the author can set them | 15:31 |
| haerwu | shit. | 15:32 |
| mgoddard | yeah | 15:32 |
| yoctozepto | mayhaps that can be fixed? | 15:32 |
| yoctozepto | I'll ask on opendev | 15:33 |
| mnasiadka | I'm not convinced that managing that in a spreadsheet instead of using Gerrit custom labels/topics/commit message ,,metadata'' will help with anything, but I'm listening ;) | 15:33 |
| mgoddard | the problem is that gerrit does not provide a suitable value to configure | 15:34 |
| haerwu | yoctozepto: thanks | 15:34 |
| mnasiadka | Can't we do custom labels? | 15:34 |
| mgoddard | gerrit would be ideal, since it could be queried | 15:34 |
| yoctozepto | hashtags would solve our problem | 15:34 |
| yoctozepto | with custom views etc. | 15:34 |
| yoctozepto | mnasiadka: labels are numeric; hell no, I don't want +2 for bugfix :-) | 15:34 |
| haerwu | https://gerrit.wikimedia.org/r/c/All-Projects/+/416357/4/project.config suggests option for it | 15:34 |
| mnasiadka | yoctozepto: you can set that +2 is CI, +1 is something else, and so on | 15:35 |
| yoctozepto | 17:34:58 <clarkb> yoctozepto: it can be. Zuul and Ironic have done this. I think we might consider allowing it across the server though? | 15:35 |
| yoctozepto | YAY | 15:35 |
| mnasiadka | well, we can just raise a change to allow it in kolla projects? | 15:35 |
| mnasiadka | before it gets done across the server... | 15:35 |
| haerwu | yoctozepto: cool ;) | 15:36 |
| yoctozepto | mnasiadka: yeah, /me on it | 15:36 |
| mgoddard | sounds good | 15:36 |
| yoctozepto | teamwork ftw :-) | 15:36 |
| mgoddard | still, I think it is the wrong part of the problem to focus on | 15:36 |
| yoctozepto | ok, elaborate | 15:37 |
| mgoddard | I'm trying to fill the vacuum left by not having priorities | 15:37 |
| mgoddard | a limitation of those priorities was that many did not get implemented | 15:37 |
| yoctozepto | well, we can manage tags in the spreadsheet instead | 15:38 |
| yoctozepto | topics too | 15:38 |
| mgoddard | so I'm suggesting we look instead at what has been proposed already | 15:38 |
| mgoddard | or at least what someone has promised to propose | 15:38 |
| mgoddard | but the question then is how to we prioritise them | 15:38 |
| mgoddard | since we have many more patches being proposed than we can review | 15:39 |
| mgoddard | we need to make better use of our time, and focus on the 'right' patches | 15:39 |
| haerwu | #cleanup #drop #trivial as things we want to get once and forget | 15:39 |
| yoctozepto | 17:39:18 <mgoddard> we need to make better use of our time, and focus on the 'right' patches | 15:40 |
| yoctozepto | I could not agree more | 15:40 |
| mgoddard | and perhaps create some feedback between contributors and available review bandwidth | 15:40 |
| mnasiadka | how do we get from what we have today, to tidied and properly tagged patches? :) | 15:41 |
| opendevreview | Skylar Tristan Kelty proposed openstack/kolla-ansible master: Update Manila deploy steps for Wallaby https://review.opendev.org/c/openstack/kolla-ansible/+/802743 | 15:41 |
| mgoddard | I might not have made it clear - the spreadsheet is just a tool. I expected (pre-hashtags) the outcome to be a YAML-ish list in the whiteboard | 15:41 |
| mnasiadka | why not a Gerrit dashboard with categorized entries? | 15:43 |
| mgoddard | that would be nice | 15:43 |
| mgoddard | what categories would be useful? | 15:43 |
| mgoddard | #xena | 15:43 |
| mgoddard | #feature | 15:44 |
| mgoddard | #bugfix | 15:44 |
| haerwu | cleanup, trivial, drop, new feature, ci fix, maintaince (with better word for is as I always forget how to write it)? | 15:44 |
| haerwu | #to-backport | 15:44 |
| jovial` | #priority-high, #priority-medium, #priority-low? | 15:44 |
| mgoddard | we would need a patch triage system to make this work | 15:44 |
| haerwu | jovial`: that's handled by RP | 15:44 |
| jovial` | true | 15:44 |
| yoctozepto | haerwu: and backport by backport candidate | 15:45 |
| yoctozepto | anyhow, https://review.opendev.org/c/openstack/project-config/+/802744 | 15:45 |
| haerwu | yoctozepto: "#to-backport #victoria #bugfix" | 15:45 |
| mnasiadka | victoria is a branch, that gets into stable backports category in Gerrit dashboard and is easy to find without hashtags | 15:46 |
| jovial` | If you let anyone edit the hashtags is there a danger of vandalism? | 15:46 |
| yoctozepto | jovial`: I only allow cores | 15:46 |
| mnasiadka | I think we should only let cores edit hashtags | 15:46 |
| yoctozepto | mnasiadka: yup, my thoughts exactly | 15:47 |
| mnasiadka | so, can somebody summarise what we can't set today? backports and priority is possible now | 15:47 |
| haerwu | I have a fix for Victoria. So I send patch to master, add '#victoria #bugfix', set BC+2 and give bug# in commit | 15:47 |
| yoctozepto | mnasiadka: kind, but that is easily solvable with standardised hashtags | 15:47 |
| yoctozepto | which we will be able to modify now | 15:48 |
| haerwu | once it in master, I backport to W and V with same hashtags | 15:48 |
| mnasiadka | yoctozepto: agreed | 15:48 |
| haerwu | and V gets BC-2 | 15:48 |
| mnasiadka | wonder if hashtags are copied on cherry pick ;) | 15:48 |
| haerwu | can be added by core | 15:48 |
| yoctozepto | haerwu: that makes sense to me | 15:48 |
| yoctozepto | mnasiadka: yeah, need to check, I think they are | 15:48 |
| mnasiadka | So let's define some basic ones we start with, and add additional ones on the way? | 15:48 |
| yoctozepto | mnasiadka: +++ | 15:49 |
| mgoddard | before we go and add a bunch of extra admin to an already stretche team... | 15:49 |
| mnasiadka | I can play with Gerrit dashboard | 15:49 |
| mgoddard | ... use cases | 15:49 |
| yoctozepto | and document the official tags | 15:49 |
| mnasiadka | yeah, in contributor guide | 15:49 |
| yoctozepto | so the tags are... | 15:50 |
| mgoddard | if we start from the queries we think would actually be useful | 15:50 |
| mgoddard | we can work backwards to hashtags | 15:50 |
| mgoddard | but there is also the question of how to decide which hashtags to apply to a patch | 15:50 |
| yoctozepto | mhm | 15:50 |
| mgoddard | if the main goal is to tag important patches to focus on, how do we decide which are important? | 15:50 |
| mnasiadka | those related to ,,priority features'' we want to implement in the cycle? | 15:51 |
| haerwu | I think that we will go with tagging patches as we review them too | 15:51 |
| mnasiadka | and those that unbreak CI, as always - for a starter | 15:51 |
| mgoddard | and how do we decide what those are? | 15:51 |
| mnasiadka | mgoddard: that's very interesting question :) | 15:51 |
| mgoddard | I suppose if we had a #feature tag it would be easier to find features | 15:52 |
| yoctozepto | yes, feature is a must | 15:52 |
| mgoddard | and easier to rank them | 15:52 |
| yoctozepto | we can also have a specialised one | 15:52 |
| mnasiadka | yes, but it's also easy to find them today if they have bp/something in topic | 15:52 |
| yoctozepto | for features being our priorities | 15:52 |
| yoctozepto | just that :-) | 15:52 |
| mnasiadka | #priority-feature ? :) | 15:52 |
| mgoddard | how about this | 15:52 |
| yoctozepto | yeah, something like that | 15:52 |
| mnasiadka | or just #feature with RP+2 | 15:52 |
| mgoddard | a feature is a priority for a release if two cores agree to review it | 15:53 |
| yoctozepto | hmm | 15:53 |
| mgoddard | this is one of the problems I'd like to solve | 15:53 |
| yoctozepto | and then we need a system to collect who agrees to overview the work | 15:53 |
| mgoddard | yes | 15:53 |
| mgoddard | I think it's not too hard | 15:54 |
| mgoddard | we document the proces | 15:54 |
| yoctozepto | cores as hashtags anyone? | 15:54 |
| yoctozepto | #yoctozepto | 15:54 |
| mgoddard | patch author or interested party drives the process to become prioritised | 15:54 |
| * yoctozepto worried about getting overtagged | 15:54 | |
| mgoddard | we agree as a team at meetings that criteria are met, and add the tag | 15:55 |
| yoctozepto | mgoddard: I mean change owners can set tags freely | 15:55 |
| yoctozepto | we need a system of punishments then | 15:56 |
| yoctozepto | 1 month of no core reviews for abuses | 15:56 |
| mgoddard | it's not the end of the world if it gets abused | 15:56 |
| mgoddard | someone might review a patch | 15:56 |
| yoctozepto | yeah, I'm just laughing here | 15:56 |
| yoctozepto | I think it really is sensible | 15:56 |
| mgoddard | of course, a future PTL may throw it out | 15:57 |
| yoctozepto | I think mnasiadka likes what we design | 15:57 |
| mgoddard | there's some nice feedback involved | 15:58 |
| mgoddard | contributor proposes patch | 15:58 |
| mgoddard | nobody signs up to review | 15:58 |
| mgoddard | hmm, maybe I need some karma | 15:58 |
| mgoddard | they do some reviews | 15:58 |
| mgoddard | 2 cores sign up | 15:58 |
| mgoddard | patch lands | 15:59 |
| yoctozepto | well, contributors joining our meetings would be cool as well | 15:59 |
| mgoddard | sure | 15:59 |
| yoctozepto | oh my, we ate up the whole meeting | 15:59 |
| mgoddard | but also it may help reviewers focus on patches they signed up for, and not keep contributors hanging | 15:59 |
| yoctozepto | how did THAT happen | 15:59 |
| mgoddard | there is one more topic | 16:00 |
| mgoddard | #topic Kolla operator pain points | 16:00 |
| mgoddard | #link https://etherpad.opendev.org/p/pain-point-elimination | 16:00 |
| mgoddard | This is being mooted as a potential Y cycle priority | 16:00 |
| mgoddard | Operator pain points | 16:00 |
| mgoddard | we would need to decide on 1-2 | 16:01 |
| mgoddard | if anyone has suggestions, please add them to the linked etherpad | 16:01 |
| yoctozepto | need to ask less-internal operators I guess? | 16:01 |
| mgoddard | we can review next week | 16:01 |
| yoctozepto | ++ | 16:01 |
| mgoddard | sure, I could resurrect the kolla-klub mailing list | 16:02 |
| yoctozepto | +++ | 16:02 |
| mnasiadka | we shouldn't be writing what are operators pain points :) | 16:02 |
| mgoddard | #action mgoddard email kolla-klubbers & openstack-discuss about pain points | 16:02 |
| mgoddard | mnasiadka: you're not an operator? | 16:02 |
| mgoddard | but yes, I see your point | 16:02 |
| yoctozepto | I think mnasiadka has a similar stance to mine; we understand the internals and can adapt easily | 16:03 |
| mnasiadka | mgoddard: I'm a special type of an operator, that pushes a patch 15 minutes after finding a pain point :) | 16:03 |
| yoctozepto | :-) | 16:03 |
| mgoddard | if only they were all like mnasiadka | 16:03 |
| mgoddard | anyway, time's up | 16:03 |
| mgoddard | thanks all | 16:03 |
| yoctozepto | thanks mgoddard | 16:03 |
| mnasiadka | ideal world doesn't exist and we wol | 16:03 |
| mgoddard | #endmeeting | 16:03 |
| opendevmeet | Meeting ended Wed Jul 28 16:03:38 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:03 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-07-28-15.00.html | 16:03 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-07-28-15.00.txt | 16:03 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/kolla/2021/kolla.2021-07-28-15.00.log.html | 16:03 |
| mnasiadka | we wouldn't be needed :) | 16:03 |
| mnasiadka | thanks for the meeting! | 16:03 |
| haerwu | this was great meeting | 16:03 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Debian/Ubuntu: handle APT keys in proper way https://review.opendev.org/c/openstack/kolla/+/784923 | 16:04 |
| yoctozepto | mnasiadka: yeah, we can't make kolla-ansible perfect or else we would be risking being obsolete! | 16:04 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: drop leftovers of RHEL support https://review.opendev.org/c/openstack/kolla/+/785569 | 16:04 |
| haerwu | I would love to get rid of both ;D | 16:04 |
| yoctozepto | I can abandon them for you | 16:07 |
| haerwu | haaha | 16:08 |
| haerwu | btw - how to run requirements-check on local system? | 16:08 |
| yoctozepto | hmm, that might be tricky | 16:10 |
| yoctozepto | need to check how it works in the requirements repo | 16:10 |
| haerwu | got it | 16:13 |
| haerwu | fetch openstack/requirements, pip install it, use playbooks/files/*py script | 16:14 |
| yoctozepto | :-) | 16:18 |
| haerwu | ERROR: Requirement for package PrettyTable excludes a version not excluded in the global list. | 16:20 |
| haerwu | Local settings : {'<0.8'} | 16:20 |
| haerwu | Global settings: set() | 16:20 |
| haerwu | ok, time to send patch for it | 16:20 |
| haerwu | Unexpected : {'<0.8'} | 16:20 |
| *** rpittau is now known as rpittau|afk | 16:21 | |
| yoctozepto | interesting | 16:22 |
| haerwu | commit 09b8c864b4656b7a58210d75e54457452aaaecc5 | 16:23 |
| haerwu | Author: Stephen Finucane <sfinucan@redhat.com> | 16:23 |
| haerwu | Date: Wed Feb 10 17:58:41 2021 +0000 | 16:23 |
| haerwu | Remove cap on PrettyTable | 16:23 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: test-requirements: remove cap on PrettyTable https://review.opendev.org/c/openstack/kolla/+/802753 | 16:24 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Update code to Jinja 3.0 style https://review.opendev.org/c/openstack/kolla/+/798263 | 16:25 |
| haerwu | and this on top. should now pass requirements-check | 16:26 |
| haerwu | Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org> | 16:27 |
| haerwu | Date: Tue Dec 19 01:20:33 2017 +0000 | 16:27 |
| haerwu | long time since we got bot on requirements | 16:27 |
| haerwu | was it disabled or sth? | 16:28 |
| yoctozepto | haerwu: no idea really | 16:28 |
| haerwu | looks like it. https://review.opendev.org/q/owner:proposal-bot does not touch projects | 16:29 |
| haerwu | yoctozepto: +2 https://review.opendev.org/c/openstack/kolla/+/802753? | 16:42 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/wallaby: Add boto3 as s3 dependency for Glance container https://review.opendev.org/c/openstack/kolla/+/802755 | 16:46 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/victoria: Add boto3 as s3 dependency for Glance container https://review.opendev.org/c/openstack/kolla/+/802756 | 16:47 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/ussuri: Add boto3 as s3 dependency for Glance container https://review.opendev.org/c/openstack/kolla/+/802757 | 16:48 |
| opendevreview | Merged openstack/kolla-ansible master: Delete haproxy_single_service_listen.cfg.j2 template https://review.opendev.org/c/openstack/kolla-ansible/+/797883 | 17:16 |
| opendevreview | Merged openstack/kolla-ansible master: manila: add glance section in manila-share.conf https://review.opendev.org/c/openstack/kolla-ansible/+/787346 | 17:28 |
| yoctozepto | haerwu: left a comment | 17:54 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: test-requirements: remove cap on PrettyTable https://review.opendev.org/c/openstack/kolla/+/802753 | 18:01 |
| haerwu | yoctozepto: fixed | 18:01 |
| opendevreview | Merged openstack/kolla stable/wallaby: Add masakari-dashboard to debian binary horizon image https://review.opendev.org/c/openstack/kolla/+/802459 | 18:03 |
| opendevreview | Merged openstack/kolla stable/wallaby: Improve offline build scenario. https://review.opendev.org/c/openstack/kolla/+/802458 | 18:10 |
| yoctozepto | haerwu: have you checked if we even use it? | 18:11 |
| yoctozepto | curious | 18:11 |
| haerwu | I plan to check more reqs | 18:13 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: [DNM] remove not needed requirements https://review.opendev.org/c/openstack/kolla/+/802769 | 18:28 |
| haerwu | CI test | 18:28 |
| haerwu | half of entries in test-requirements dropped | 18:29 |
| * haerwu out | 18:29 | |
| haerwu | next step: bump to versions in openstack/requirements:master ;d | 18:35 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Use merge_configs and merge_yaml to generate custom config https://review.opendev.org/c/openstack/kayobe/+/782749 | 18:42 |
| opendevreview | Merged openstack/kolla-ansible stable/wallaby: Fix variable names in Octavia documentation https://review.opendev.org/c/openstack/kolla-ansible/+/802472 | 19:09 |
| opendevreview | Merged openstack/kolla-ansible stable/victoria: Fix variable names in Octavia documentation https://review.opendev.org/c/openstack/kolla-ansible/+/802473 | 19:13 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Use more RMQ flags for less busy wait https://review.opendev.org/c/openstack/kolla-ansible/+/802796 | 19:19 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Slimmer RMQ https://review.opendev.org/c/openstack/kolla-ansible/+/802798 | 19:28 |
| opendevreview | Merged openstack/kolla-ansible stable/wallaby: Reduce RabbitMQ busy waiting, lowering CPU load https://review.opendev.org/c/openstack/kolla-ansible/+/799237 | 21:03 |
| opendevreview | Merged openstack/kolla-ansible stable/victoria: Reduce RabbitMQ busy waiting, lowering CPU load https://review.opendev.org/c/openstack/kolla-ansible/+/799238 | 21:03 |
| opendevreview | Merged openstack/kolla-ansible stable/ussuri: Reduce RabbitMQ busy waiting, lowering CPU load https://review.opendev.org/c/openstack/kolla-ansible/+/799239 | 21:03 |
| opendevreview | David Moreau Simard proposed openstack/kolla-ansible master: DNM: Test ara 1.5.7rc2 with kolla-ansible https://review.opendev.org/c/openstack/kolla-ansible/+/720902 | 23:03 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!