| opendevreview | Bui Doan Dang proposed openstack/kolla master: Fix cinder-backup: add missing sysfsutils,lsscsi package https://review.opendev.org/c/openstack/kolla/+/927886 | 01:55 |
|---|---|---|
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Add option to enable proxysql's prometheus exporter https://review.opendev.org/c/openstack/kolla-ansible/+/928619 | 02:22 |
| kevko | morning ....what can i do with 'ready to submitt' gerrit change ? | 06:42 |
| kevko | https://review.opendev.org/c/openstack/kolla-ansible/+/928619/ | 06:43 |
| kevko | recheck recheck and recheck ? | 06:43 |
| kevko | beacuse fail in arm seems totally unrelated | 06:43 |
| SvenKieske | morning | 07:11 |
| SvenKieske | I'm confused, the gate pipeline is clean but the arm pipeline is blocking? is that really true? I thought arm was non blocking? | 07:12 |
| priteau | kevko: I have never seen this status before. New Gerrit thing? | 07:16 |
| priteau | You could ask on #opendev | 07:17 |
| kevko | priteau: it's also first time for me ...i have a nightmares with this status ..and woke up at the middle of the night just to check it :D | 07:23 |
| kevko | shouldn't that jobs has non-voting then ? | 07:25 |
| kevko | btw, anybody to check this ? https://review.opendev.org/c/openstack/kolla/+/929516 | 07:28 |
| kevko | and also this https://review.opendev.org/c/openstack/kolla/+/829295/16 ? < also docs added | 07:28 |
| SvenKieske | currently actually reviewing it and the linked reviews | 07:29 |
| SvenKieske | but imho gerrit really has a problem.. | 07:29 |
| kevko | SvenKieske: thank you very much ... | 07:29 |
| kevko | SvenKieske: really ? why do you think ? because of that never seen before status ? | 07:30 |
| SvenKieske | when I look at a comment it has text, e.g. https://review.opendev.org/c/openstack/kolla-ansible/+/773243/14, but when I click "expand this comment (click on it) gerrit just says "undefined"? | 07:30 |
| kevko | SvenKieske: that comment was long (example of changed tasks in ansible play and other stuff to demonstrate what I am trying to say ....) ..so maybe because of it ? ... below i am writing this "I don't know why gerrit can't show my original message but :" ...... but then i just shortened the comment | 07:32 |
| mnasiadka | morning | 07:32 |
| mnasiadka | kevko: arm jobs are non voting, and even if they would be - these are in different pipeline and it doesn't matter | 07:33 |
| * SvenKieske|OSISM[m] uploaded an image: (28KiB) < https://matrix.org/oftc/media/v1/media/download/AUs8mMhKx3UoXu3LBFjQ0UROMbgvxgqos5QPMVGMcVIxXre2Z89pX0zCmND-jnVVvAlrBKpYjZXt1Zaxt9lrD59CeSAJ5TTQAHJlZ2lvLmNoYXQvYVpiYW5IZ0FyaWhPQXJwQmVVdmJqcWNz > | 07:33 | |
| * SvenKieske|OSISM[m] uploaded an image: (21KiB) < https://matrix.org/oftc/media/v1/media/download/AUjsNvqejthzw4G0zMztZK47dwFI9-Yo7Vn_BMocT-A6jQdtvUAKnESPqj5On_uBNczNe-s33WuyqglotT5t7XhCeSAJ54zgAHJlZ2lvLmNoYXQvV2habE9zQ2NNTUpYTUFkcXBLZ2xYUGxr > | 07:33 | |
| mnasiadka | ah, these are voting, but it didn't matter in the past | 07:34 |
| mnasiadka | oh boy, Matrix bridge | 07:34 |
| SvenKieske | sorry, that was the fastest way to share a screenshot, on the second upload you can see the text, on the first one you see only "undefined" | 07:34 |
| mnasiadka | that might be some Zuul bug, if the dashboard did get this major rework, I don't know what changed in Zuul code :) | 07:35 |
| SvenKieske | maybe it's just javascript foo again, I'll try reloading the page.. | 07:35 |
| kevko | mnasiadka: so what to do next with that review ? | 07:36 |
| mnasiadka | ask on #opendev, which you already did | 07:36 |
| kevko | yep | 07:36 |
| mnasiadka | but in that timezone there's only frickler | 07:36 |
| kevko | okay, i will wait | 07:36 |
| mnasiadka | so you need to be very patient :) | 07:36 |
| SvenKieske | not sure if he's working today he got some travelling to do afaik. if you're lucky he's still available, but not sure. | 07:37 |
| kevko | mnasiadka: well, I'm not really good in this discipline :D | 07:37 |
| opendevreview | Ivan Vnučko proposed openstack/kolla-ansible master: Fix Masakari instancemonitor when libvirt TLS enabled https://review.opendev.org/c/openstack/kolla-ansible/+/929594 | 07:38 |
| kevko | mnasiadka: In the top right, where the buttons like rebase, abandon, or edit are, I have a greyed-out 'submit' button. Even you, Michal, don't have the right to click on it? | 07:41 |
| mnasiadka | Submit should be done by Zuul, seems it's stuck ;-) | 07:43 |
| mnasiadka | (I don't have rights) | 07:43 |
| SvenKieske | stackoverflow suggest a gerrit bug at least ;) https://stackoverflow.com/questions/41436931/gerrit-change-is-ready-to-submit-but-submit-is-grey-clicking-the-button-woul | 07:44 |
| kevko | btw, this is history of that job https://zuul.opendev.org/t/openstack/builds?job_name=kolla-ansible-debian-aarch64-podman&skip=0 | 07:44 |
| SvenKieske | my submit button is also greyed out fwiw | 07:44 |
| chembervint | hi, colleagues. could you please review my small but important patch https://review.opendev.org/c/openstack/kolla-ansible/+/928077 ? | 07:46 |
| kevko | it's very interisting for example in this review https://review.opendev.org/c/openstack/kolla-ansible/+/923946 the order of comments is - 1. Build failed (ARM64 pipeline). , 2 . verified +1 , 3. Starting gate jobs , 4. Verified +2 , 5. Merged | 07:47 |
| kevko | but in my patch after step 4 . Verified +2 the next in queue is not 'Merged' ...but failed arm | 07:48 |
| SvenKieske | not sure if there are race conditions lurking in the code if right at that time something incompatible merges? on the surface zuul should just enforce a strict ordering of patches when merging so in theory that should not be a problem | 07:51 |
| SvenKieske | and afaik that is one of zuuls selling points | 07:51 |
| kevko | I have a strong desire to hit recheck again. :D | 07:51 |
| mnasiadka | I doubt it's going to help | 07:52 |
| SvenKieske | chembervint: I'll look into it. may your deity of favor have mercy with the poor souls who have to use multipath :D | 07:53 |
| mnasiadka | actually let's mark arm as non-voting | 07:53 |
| SvenKieske | I still suspect a bug and think this will not help (making arm non-voting) but I don't object as I can easily be wrong. | 07:53 |
| SvenKieske | some day in the future we would like to have arm stable though. :/ | 07:54 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: CI: Mark aarch64 non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/929598 | 07:56 |
| chembervint | mnasiadka: thank you | 07:56 |
| kevko | warning: skipped previously applied commit ea6bd143a | 07:58 |
| kevko | hint: use --reapply-cherry-picks to include skipped commits | 07:58 |
| kevko | hint: Disable this message with "git config advice.skippedCherryPicks false" | 07:58 |
| kevko | Successfully rebased and updated refs/heads/mariadb-proxysql. | 07:58 |
| kevko | will see... | 07:58 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add option to enable proxysql's prometheus exporter https://review.opendev.org/c/openstack/kolla-ansible/+/928619 | 07:58 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix mariadb backup not working https://review.opendev.org/c/openstack/kolla-ansible/+/928960 | 07:58 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Switch mariadb's loadbalancer from HAProxy to ProxySQL https://review.opendev.org/c/openstack/kolla-ansible/+/913724 | 07:58 |
| SvenKieske | chembervint: should merge soon. in general it would be nice if patches/commit messages include reasoning why something needs to be done, not just the bugreport. :) | 08:01 |
| chembervint | SvenKieske: thank you. ok, will take into the account :) | 08:01 |
| SvenKieske | mhm, thinking about the "ready to submit" thing I doubt it's a general bug, because other stuff get's merged, right? | 08:05 |
| kevko | SvenKieske: yeah, in relation chain it was 'merged' for parent commit | 08:08 |
| kevko | now when I rebased against master ...the merged parent commit dissapeared from relation chain ... | 08:09 |
| kevko | so if it will be merged now ...it's weird ..because this should be irrelevant if it is already merged or not | 08:09 |
| kevko | SvenKieske: tobias said "perhaps that you have patchset 3 on the parent patch but patchset 4 is merged, a simple rebase and approve again?" | 08:10 |
| kevko | SvenKieske: but if it is as he said ...i should see '(Indirect relation)' in relation chain ...and I didn't | 08:10 |
| SvenKieske | yeah I read that, currently rereading the stuff wrt to python <3.10 removal | 08:11 |
| opendevreview | Pierre Riteau proposed openstack/kayobe-config-dev stable/2024.1: [2024.1-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config-dev/+/929599 | 08:12 |
| SvenKieske | I somehow find most TC documents very hard to read, they all seem to assume some kind of hidden knowledge.. | 08:14 |
| SvenKieske | is there some definition, what is a "python project"? https://governance.openstack.org/tc/reference/pti/python.html#pti-python | 08:15 |
| SvenKieske | are we a python project? I would argue we are more of a yaml project? :D | 08:15 |
| kevko | we definitely are :D | 08:16 |
| SvenKieske | python? or yaml? :D or both? | 08:16 |
| kevko | (i mean yaml) | 08:16 |
| opendevreview | Pierre Riteau proposed openstack/kayobe-config-dev stable/2023.2: [2023.2-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config-dev/+/929601 | 08:17 |
| opendevreview | Pierre Riteau proposed openstack/kayobe-config-dev stable/2023.1: [2023.1-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config-dev/+/929602 | 08:19 |
| SvenKieske | my line counting program (tokei) even says we are more of an ReStructuredText Project, we have 17k lines of rst but only 12k of python. and 51k lines of yaml :D | 08:21 |
| SvenKieske | in kolla-ansible | 08:21 |
| kevko | i am still saying we shouldn't write docuementation :D | 08:21 |
| SvenKieske | oh I don't know, everytime I read it it's actually rather decent and helpful. _if_ the answer is in the docs I always find it faster than looking at the code. | 08:22 |
| SvenKieske | but yeah, many times I directly look at code tbh. | 08:23 |
| kevko | hh, that was joke ....i never read docs :D | 08:23 |
| kevko | also for other projects of openstack :D | 08:23 |
| kevko | i tried ...on the beginning ... and i end with checking the code :D | 08:24 |
| mnasiadka | we always assumed kolla is a python project, kolla-ansible and kayobe not really - and it's driven by Ansible requirements | 08:27 |
| kevko | +1 | 08:27 |
| opendevreview | Pierre Riteau proposed openstack/kayobe-config-dev master: Use CentOS Stream 9 IPA deployment images https://review.opendev.org/c/openstack/kayobe-config-dev/+/929603 | 08:30 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Test overcloud provisioning with Bifrost https://review.opendev.org/c/openstack/kayobe/+/903249 | 08:31 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Test overcloud provisioning with Bifrost https://review.opendev.org/c/openstack/kayobe/+/903249 | 08:31 |
| opendevreview | Roman Krček proposed openstack/kolla master: Change copy-cacerts behaviour https://review.opendev.org/c/openstack/kolla/+/926882 | 08:36 |
| priteau | mnasiadka: the author of https://review.opendev.org/c/openstack/kolla/+/927886 updated their commit message | 08:38 |
| opendevreview | Merged openstack/kolla-ansible master: Fix os-brick multipathd del map fails for SAN multipath devices https://review.opendev.org/c/openstack/kolla-ansible/+/928077 | 09:19 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/915440 | 09:54 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Test overcloud provisioning with Bifrost https://review.opendev.org/c/openstack/kayobe/+/903249 | 10:03 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/915440 | 10:26 |
| kevko | SvenKieske: please can u check ^^ ...added some unit tests and DocsStrings as you wished ... | 10:26 |
| kevko | SvenKieske: this is also important to move towards check without notifiers | 10:26 |
| kevko | * check/configure | 10:27 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/915440 | 10:33 |
| kevko | and anybody else of course :) | 10:37 |
| opendevreview | Merged openstack/kayobe-config stable/2023.1: [2023.1-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config/+/929559 | 10:41 |
| opendevreview | Merged openstack/kayobe-config stable/2023.2: [2023.2-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config/+/929558 | 10:41 |
| opendevreview | Merged openstack/kayobe-config stable/2024.1: [2024.1-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config/+/929557 | 10:41 |
| opendevreview | Merged openstack/kayobe master: Avoid shared IPA image cache on Ansible control host https://review.opendev.org/c/openstack/kayobe/+/922371 | 11:31 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Backup from active mariadb server https://review.opendev.org/c/openstack/kolla/+/928956 | 11:39 |
| opendevreview | Merged openstack/kolla-ansible master: Add option to enable proxysql's prometheus exporter https://review.opendev.org/c/openstack/kolla-ansible/+/928619 | 11:45 |
| kevko | SvenKieske: around ? | 11:49 |
| opendevreview | Andrey Kurilin proposed openstack/kolla master: Allow to use configure_user macro for external templates https://review.opendev.org/c/openstack/kolla/+/924387 | 11:49 |
| opendevreview | Andrey Kurilin proposed openstack/kolla master: Allow to use configure_user macro for external templates https://review.opendev.org/c/openstack/kolla/+/924387 | 11:56 |
| opendevreview | Andrey Kurilin proposed openstack/kolla master: Allow to use configure_user macro for external templates https://review.opendev.org/c/openstack/kolla/+/924387 | 11:58 |
| tafkamax | Hi, I have a question regarding provider networks. They are meant to be "direct" attachments from the underlying physical network to the VM, but I cannot seem to get OSPF going with the other hosts in the provider network... (full message at <https://matrix.org/oftc/media/v1/media/download/AVVEn17j7_bVAo1LJ6tX7fo25gpCBh_IGrEvW-zqsZrtpM9zl9zWPMERYpn2UsMgSk_sRbQKkNY7jRC64O21l2dCeSAZnLiQAG1hdHJpeC5vcmcvbHNIemJiR3htdEVPUGpSa0tzUkN6eXp6>) | 12:08 |
| opendevreview | Merged openstack/kolla master: Fix cinder-backup: add missing sysfsutils,lsscsi package https://review.opendev.org/c/openstack/kolla/+/927886 | 12:14 |
| tafkamax | nvm, checking if it might be related to the FRR | 12:17 |
| tafkamax | * the FRR version I am running | 12:18 |
| tafkamax | Well I got rid of some errors from FRR side, but still the interface does not return a OSPF hello message back, so it has to be related to the communication from provider :/ | 12:29 |
| opendevreview | Pierre Riteau proposed openstack/kayobe stable/2024.1: Avoid shared IPA image cache on Ansible control host https://review.opendev.org/c/openstack/kayobe/+/929637 | 12:38 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/915440 | 12:42 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Test overcloud provisioning with Bifrost https://review.opendev.org/c/openstack/kayobe/+/903249 | 12:43 |
| tafkamax | aha it seems I needed to add OSPF to default security group, or remove port security from neutron. Explicilty allowing any proto did not work - it needed to be ospf. | 13:37 |
| SvenKieske | kevko: I'm currently more or less full time doing preparations for a private cloud upgrade, so only around in the evening, you can catch me via mail if it helps. | 13:53 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Test overcloud provisioning with Bifrost https://review.opendev.org/c/openstack/kayobe/+/903249 | 13:54 |
| opendevreview | yosef proposed openstack/kolla-ansible master: Add ovn-relay role https://review.opendev.org/c/openstack/kolla-ansible/+/929646 | 13:55 |
| opendevreview | Merged openstack/kolla-ansible master: Fix mariadb backup not working https://review.opendev.org/c/openstack/kolla-ansible/+/928960 | 14:21 |
| opendevreview | Andrey Kurilin proposed openstack/kolla master: Allow to use configure_user macro for external templates https://review.opendev.org/c/openstack/kolla/+/924387 | 14:22 |
| kevko | SvenKieske: well, i've just addressed your comments and that code is tiny :) ... | 14:26 |
| kevko | SvenKieske: but understand ...if there is no time ..okay | 14:26 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/915440 | 14:29 |
| SvenKieske | kevko: I'll have a look later, but thanks already for providing it! I have still a draft comment on that change that I think I can discard now :) | 14:30 |
| kevko | SvenKieske: ok thanks, I need to also prepare downstream gits (upgrading on the end of month ... lab at monday) ...and i am always sleeping better when I know that I've cherry-picked already merged or near to merge commit .... | 14:32 |
| kevko | SvenKieske: so this week I have a marathone of my all patches ... | 14:32 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Test overcloud provisioning with Bifrost https://review.opendev.org/c/openstack/kayobe/+/903249 | 15:01 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Support running without root privileges https://review.opendev.org/c/openstack/kayobe/+/923430 | 15:06 |
| opendevreview | Merged openstack/kayobe-config-dev stable/2024.1: [2024.1-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config-dev/+/929599 | 15:33 |
| opendevreview | Merged openstack/kayobe-config-dev stable/2023.2: [2023.2-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config-dev/+/929601 | 15:36 |
| opendevreview | Merged openstack/kayobe-config-dev stable/2023.1: [2023.1-only] Synchronise with latest kayobe https://review.opendev.org/c/openstack/kayobe-config-dev/+/929602 | 15:36 |
| opendevreview | Will Szumski proposed openstack/kayobe master: kolla-openstack: Add ironic-inspector dir to config https://review.opendev.org/c/openstack/kayobe/+/924163 | 15:43 |
| greatgatsby | Hello. We've been able to tame redis, but gnocchi is not able to keep up with processing metrics, despite 32 workers per controller. I read about changing the "sack" size, but I can't find anywhere that defines what "active metrics" actually means? | 16:06 |
| greatgatsby | from the docs: sacks value = number of **active** metrics / 300 | 16:09 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Test overcloud provisioning with Bifrost https://review.opendev.org/c/openstack/kayobe/+/903249 | 16:17 |
| opendevreview | Merged openstack/kolla-ansible master: CI: Mark aarch64 non-voting https://review.opendev.org/c/openstack/kolla-ansible/+/929598 | 17:19 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Don't notify handlers during copy-cert https://review.opendev.org/c/openstack/kolla-ansible/+/924651 | 20:21 |
| opendevreview | Roman Krček proposed openstack/kolla master: Change copy-cacerts behaviour https://review.opendev.org/c/openstack/kolla/+/926882 | 20:33 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Performance: Don't notify handlers during config https://review.opendev.org/c/openstack/kolla-ansible/+/745164 | 20:43 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Refactor services' check-containers and optimise https://review.opendev.org/c/openstack/kolla-ansible/+/773243 | 20:57 |
| opendevreview | Roman Krček proposed openstack/kolla-ansible master: Fix unintentional trigger of ansible handlers https://review.opendev.org/c/openstack/kolla-ansible/+/924145 | 20:57 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Fix handling configs in base image https://review.opendev.org/c/openstack/kolla/+/915440 | 22:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!