| *** osmanlicilegi is now known as Guest0 | 01:48 | |
| *** amoralej|off is now known as amoralej | 06:16 | |
| mnasiadka | admin1: rhel8 support is deprecated in Xena, that means we don't test it (well, we never tested it) and we don't really support it. | 06:36 |
|---|---|---|
| opendevreview | Verification of a change to openstack/kolla master failed: Remove Keystone admin endpoint https://review.opendev.org/c/openstack/kolla/+/843751 | 08:57 |
| frickler | tooz/protobuf u-c conflict breaking CI ^^ https://8a874fe53877280064a1-1a1dc7cbee41e9efce3274023c92b346.ssl.cf5.rackcdn.com/843751/5/gate/kolla-build-debian/a3dc6e8/kolla/build/000_FAILED_openstack-base.log | 09:23 |
| frickler | will nag oslos | 09:27 |
| hrw | thx | 09:27 |
| hrw | move from protobuf 3 -> 4 was on 20 July | 09:29 |
| hrw | 3.20.1 -> 4.21.2 | 09:29 |
| hrw | tooz 3.0.0 is 4th August | 09:30 |
| mnasiadka | INFO:kolla.common.utils.openstack-base:[91mERROR: Cannot install tooz[consul,etcd,etcd3,etcd3gw,ipc,memcached,mysql,postgresql,redis,zake,zookeeper]==3.0.0 because these package versions have conflicting dependencies. | 10:00 |
| mnasiadka | gate broken :D | 10:00 |
| mnasiadka | https://8a874fe53877280064a1-1a1dc7cbee41e9efce3274023c92b346.ssl.cf5.rackcdn.com/843751/5/gate/kolla-build-debian/a3dc6e8/kolla/build/000_FAILED_openstack-base.log | 10:00 |
| hrw | mnasiadka: look a bit earlier ;D | 10:00 |
| hrw | I blame tooz | 10:00 |
| opendevreview | Stephen Finucane proposed openstack/kolla master: Don't install the tooz etcd3 extra https://review.opendev.org/c/openstack/kolla/+/852534 | 10:01 |
| hrw | yay | 10:03 |
| hrw | INFO:kolla.common.utils.openstack-base:Built at 2022-08-09 12:17:02.795381 (took 0:01:28.258287) | 10:17 |
| hrw | watch that patch | 10:17 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Edit services roles to support database sharding https://review.opendev.org/c/openstack/kolla-ansible/+/770216 | 10:17 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add loadbalancer-config role and wrap haproxy-config role inside https://review.opendev.org/c/openstack/kolla-ansible/+/851004 | 10:17 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Add proxysql-config role and wrap it in loadbalancer-config role https://review.opendev.org/c/openstack/kolla-ansible/+/851442 | 10:17 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [CI] Test ProxySQL with shards in the nova cells scenario https://review.opendev.org/c/openstack/kolla-ansible/+/770621 | 10:17 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: [DNM] Trigger cells job https://review.opendev.org/c/openstack/kolla-ansible/+/838916 | 10:17 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Use Docker healthchecks for mariadb-server service https://review.opendev.org/c/openstack/kolla-ansible/+/805616 | 10:17 |
| kevko | Hi \o/, anybody for second +2 and +w for my proxysql patches guys :) ? | 10:22 |
| hrw | I have to admit that I do not understand those proxysql patches | 10:23 |
| kevko | hrw: well, main logic is already merged ...other patches is only just defaults .. :/ | 10:23 |
| opendevreview | Merged openstack/kolla-ansible stable/yoga: Set the ironic notification level https://review.opendev.org/c/openstack/kolla-ansible/+/850924 | 10:41 |
| opendevreview | Merged openstack/kolla-ansible stable/wallaby: Set the ironic notification level https://review.opendev.org/c/openstack/kolla-ansible/+/850926 | 11:05 |
| opendevreview | Merged openstack/kolla-ansible stable/xena: Set the ironic notification level https://review.opendev.org/c/openstack/kolla-ansible/+/850925 | 11:05 |
| admin1 | mnasiadka, but i am using ubuntu and not rhel8 | 12:43 |
| mnasiadka | admin1: focal? | 12:47 |
| admin1 | jammy | 12:47 |
| admin1 | on master | 12:47 |
| admin1 | jammy support is still not out yet | 12:48 |
| opendevreview | Merged openstack/kolla master: Don't install the tooz etcd3 extra https://review.opendev.org/c/openstack/kolla/+/852534 | 12:48 |
| mnasiadka | admin1: works in CI last time I checked | 12:49 |
| mnasiadka | kevko_: do we test proxysql and sharding in any of the CI jobs? | 12:49 |
| admin1 | that was my assumption ( worked on ci) .. and then tried to test it | 12:49 |
| admin1 | all worked fine, except the nova issue | 12:49 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Remove Keystone admin endpoint https://review.opendev.org/c/openstack/kolla/+/843751 | 12:51 |
| frickler | kevko_: can you please stop doing useless rebases? zuul will rebase the patch automatically for testing, a simple "recheck gate fix merged" comment would have been enough for ^^ | 12:58 |
| frickler | same goes for you proxysql stack | 12:58 |
| frickler | *your | 12:58 |
| *** amoralej is now known as amoralej|lunch | 13:01 | |
| kevko_ | mnasiadka: yes we are testing -> https://review.opendev.org/c/openstack/kolla-ansible/+/770621 << nova_cells scenario | 13:07 |
| kevko_ | mnasiadka: here -> it's green -> https://review.opendev.org/c/openstack/kolla-ansible/+/838916 | 13:07 |
| kevko_ | frickler: hmm, sorry I didn't know that i can just type "recheck gate fix merged" ... i saw there was an error with etcd3 which is now merged ..so i just rebased onto top of that merged commit | 13:08 |
| kevko_ | frickler: for proxysql patch stack i had a merge conflict but i realized that i don't need that patch so i abandoned .. | 13:09 |
| kevko_ | but maybe someone can give me +2 as there are only "simple" patches left as main logic was already merged | 13:10 |
| hrw | kevko_: you do your patch, it is branched and tested in such branch. when final merge happens on zuul then it is merged on top of master branch | 13:10 |
| hrw | kevko_: so as long as master's changes are not needed for you no rebase is needed | 13:11 |
| kevko_ | just recheck .. | 13:11 |
| kevko_ | correct ? | 13:11 |
| hrw | yes | 13:11 |
| kevko_ | ok, i will know | 13:11 |
| kevko_ | thanks | 13:11 |
| hrw | if your patchset needs new code from master then 'git review -d; git up; CHECK; git review' | 13:11 |
| hrw | kevko_: unless you want to beat https://review.opendev.org/c/openstack/kolla/+/430940 which took 50 revisions | 13:13 |
| kevko_ | and what is the difference if i rebase patch in review onto top of master (with merged fix) ...and just type recheck .. zuul will test it in both cases, won't it ? | 13:14 |
| hrw | rebase generates mails | 13:14 |
| kevko_ | understand | 13:14 |
| opendevreview | Merged openstack/kolla-ansible master: Add clouds.yaml file and use it https://review.opendev.org/c/openstack/kolla-ansible/+/452005 | 13:14 |
| opendevreview | Merged openstack/kolla master: Honour the linuxbridge experimental status https://review.opendev.org/c/openstack/kolla/+/852023 | 13:14 |
| hrw | kevko_: and if developerX reviewed revisionY then your rebase generates revisionY+1 which needs review again | 13:15 |
| hrw | unless it is 'gerrit, do rebase' as then votes stay | 13:16 |
| kevko_ | yeah, but only commit message will be changed | 13:17 |
| mnasiadka | kevko_: so let's merge it first and get other patches tested ;-) | 13:19 |
| hrw | kevko_: nope. commit is same | 13:19 |
| kevko_ | mnasiadka: ok, but it's tested as there is dummy patch which is triggering this test -> https://review.opendev.org/c/openstack/kolla-ansible/+/838916 | 13:21 |
| mnasiadka | kevko_: I mean that e.g. this patch - https://review.opendev.org/c/openstack/kolla-ansible/+/770216 - did not run this CI job, so we only know that it hasn't broken the usual scenarios | 13:22 |
| kevko_ | mnasiadka: ok Michal, thank you :) | 13:23 |
| admin1 | mnasiadka, is there a way to debug and check why its not working ? or what can i submit ? | 13:53 |
| *** amoralej|lunch is now known as amoralej | 14:02 | |
| opendevreview | Merged openstack/kolla-ansible master: Honour the linuxbridge experimental status https://review.opendev.org/c/openstack/kolla-ansible/+/852022 | 14:21 |
| kevko_ | hrw, frickler one more question about CI :) , when patch has 2x +2 and +w , i should see this in zuul queue, shouldn't I ? | 14:23 |
| hrw | unless it depends-on other not merged patches | 14:24 |
| * hrw out | 14:27 | |
| frickler | also needs Zuul+1 before entering gate queue | 15:02 |
| SvenKieske | kevko_ you should add a reason to your recheck command afaik; there is a policy somewhere that there should not be rechecks without a reasoning | 15:35 |
| kevko_ | frickler: well it has all -> https://review.opendev.org/c/openstack/kolla-ansible/+/770621 | 15:41 |
| kevko_ | hrw: depends-on only or also if patch is in stack of patches ? | 15:42 |
| frickler | kevko_: nope, it has three patches in the stack below it, those are implicit dependencies and need to be approved/merged first | 15:43 |
| kevko_ | frickler: hmm :( | 15:43 |
| kevko_ | mnasiadka: so, if I understand guys correctly, CI patch for proxysql Cells can't be merged before patches below in stack :/ | 15:44 |
| *** amoralej is now known as amoralej|off | 15:57 | |
| admin1 | mnasiadka, where do I check the kolla-ansible CI logs ? | 16:56 |
| opendevreview | James Kirsch proposed openstack/kolla-ansible master: Add support for LetsEncrypt-managed certs https://review.opendev.org/c/openstack/kolla-ansible/+/741340 | 18:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!