| opendevreview | likui proposed openstack/kolla-ansible master: update the default value of node_custom_config https://review.opendev.org/c/openstack/kolla-ansible/+/825715 | 08:08 |
|---|---|---|
| jingvar | what python version you use? I have ansible hungs on 20 nodes... | 08:48 |
| priteau | Whatever comes as default with our distributions, e.g. CentOS or Ubuntu | 08:51 |
| priteau | Ansible hanging sounds more like connectivity problems | 08:52 |
| priteau | DNS, MTU, etc. | 08:52 |
| mgoddard | new gerrit UI has swapped code review +2 and backport candidate +2 buttons. Argh! | 09:24 |
| priteau | Do you know if there a way to add shortcuts for CR+2 and W+1? It would be nice not to have to open the commenting UI | 09:40 |
| priteau | https://gerrit-review.googlesource.com/Documentation/images/user-review-ui-change-screen-quick-approve.png | 09:42 |
| hrw | morning | 09:51 |
| hrw | now time to get my patch back to gerrit... | 09:51 |
| hrw | I think that all train issues will be sorted | 09:53 |
| hrw | out | 09:53 |
| opendevreview | Merged openstack/kolla-ansible master: Revert "Use friendly target names in Prometheus" https://review.opendev.org/c/openstack/kolla-ansible/+/822427 | 09:55 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/xena: Revert "Use friendly target names in Prometheus" https://review.opendev.org/c/openstack/kolla-ansible/+/826216 | 09:56 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla stable/train: CI: make it work again https://review.opendev.org/c/openstack/kolla/+/826033 | 09:57 |
| hrw | uf. | 09:57 |
| hrw | priteau: I saw your patch for train a bit late. sorted stuff a bit different ways | 09:58 |
| priteau | Thanks, no problem | 09:59 |
| hrw | now let's see what CI says | 09:59 |
| hrw | locally centos 7/8 binary/source builds | 09:59 |
| priteau | More stuff probably broke since I wrote it | 09:59 |
| hrw | no more monasca-grafana, no more sensu-client | 09:59 |
| hrw | I fixed sensu-server but no idea how to get sensu-client properly fixed | 09:59 |
| jingvar | priteau: dns,mtu checked, works ok on 3 controls and 2 compute nodes | 10:15 |
| priteau | Can you run with -vvv and see what causes ansible to hang | 10:19 |
| hrw | ok. k-a jobs in train fail | 10:21 |
| hrw | ERROR: Package 'websocket-client' requires a different Python: 2.7.16 not in '>=3.6'\n" | 10:22 |
| opendevreview | Merged openstack/kolla stable/wallaby: Mitigate two Log4j vulnerabilities in Apache Storm https://review.opendev.org/c/openstack/kolla/+/825949 | 10:28 |
| frickler | yoctozepto: mgoddard: if you both say that we should be conservative and not change the default, I'll update the patch accordingly, but I'd like to avoid going back and forth on this https://review.opendev.org/c/openstack/kolla-ansible/+/818362/comment/04c3be41_5e7e2fe8/ | 10:33 |
| jingvar | priteau: like https://github.com/ansible/ansible/issues/30411 | 10:33 |
| hrw | shit. looks like train needs pip upgrade | 10:42 |
| hrw | pip 20 installs, pip 8 does not | 10:42 |
| hrw | hm. it worked locally... | 10:43 |
| hrw | INFO:kolla.common.utils.bifrost-base:[91mError: Cannot find a valid baseurl for repo: epel | 10:44 |
| hrw | hm. | 10:44 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla stable/train: CI: make it work again https://review.opendev.org/c/openstack/kolla/+/826033 | 10:49 |
| hrw | priteau: marked you as co-author, added closes-bug and made zuul fetch requirements (from your patch) | 10:50 |
| opendevreview | Merged openstack/kolla stable/xena: Mitigate two Log4j vulnerabilities in Apache Storm https://review.opendev.org/c/openstack/kolla/+/825948 | 10:54 |
| opendevreview | Merged openstack/kolla stable/victoria: Mitigate two Log4j vulnerabilities in Apache Storm https://review.opendev.org/c/openstack/kolla/+/826050 | 10:54 |
| opendevreview | Merged openstack/kolla stable/ussuri: Mitigate two Log4j vulnerabilities in Apache Storm https://review.opendev.org/c/openstack/kolla/+/826051 | 10:54 |
| opendevreview | Merged openstack/kolla-ansible master: Remove custom value for max_allowed_request_size_in_bytes https://review.opendev.org/c/openstack/kolla-ansible/+/824568 | 10:54 |
| opendevreview | Merged openstack/kolla-ansible master: Delete the logrotate files because they do not have separate logs https://review.opendev.org/c/openstack/kolla-ansible/+/822593 | 10:54 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla-ansible stable/train: CI: make it work again (k-a side) https://review.opendev.org/c/openstack/kolla-ansible/+/826269 | 11:08 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla stable/train: CI: make it work again https://review.opendev.org/c/openstack/kolla/+/826033 | 11:09 |
| hrw | train-- | 11:11 |
| opendevreview | Isaac Prior proposed openstack/kolla-ansible master: Fix log rotation for fluentd created files https://review.opendev.org/c/openstack/kolla-ansible/+/797900 | 12:02 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Upgrade pip and friends in source images (part 2) https://review.opendev.org/c/openstack/kolla/+/826273 | 12:09 |
| hrw | found while working on train | 12:11 |
| hrw | Die, train, die! | 12:19 |
| hrw | 000_FAILED_elasticsearch-curator. | 12:20 |
| hrw | log | 12:20 |
| hrw | pip 8 is just stupid | 12:21 |
| hrw | and I prefer not to backport 'upgrade pip and friends' patch over all stable/ branches | 12:21 |
| hrw | but considering doing it in train just because of centos7 | 12:21 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla stable/train: CI: make it work again https://review.opendev.org/c/openstack/kolla/+/826033 | 12:44 |
| hrw | bifrost disabled on centos 8, elasticsearch-curator gets pip 20 | 12:44 |
| opendevreview | Konstantin Yarovoy proposed openstack/kolla master: Refactor image module and transition to DockerClient https://review.opendev.org/c/openstack/kolla/+/825786 | 13:08 |
| *** mrunge_ is now known as mrunge | 13:11 | |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla master: Upgrade pip and friends in source images (part 2) https://review.opendev.org/c/openstack/kolla/+/826273 | 13:21 |
| opendevreview | Konstantin Yarovoy proposed openstack/kolla master: Refactor image module and transition to DockerClient https://review.opendev.org/c/openstack/kolla/+/825786 | 13:33 |
| opendevreview | Merged openstack/kolla-ansible master: Use subnet_dns_publish_fixed_ip Neutron extension https://review.opendev.org/c/openstack/kolla-ansible/+/825822 | 13:40 |
| opendevreview | Konstantin Yarovoy proposed openstack/kolla master: Refactor image module and transition to DockerClient https://review.opendev.org/c/openstack/kolla/+/825786 | 14:01 |
| opendevreview | Konstantin Yarovoy proposed openstack/kolla master: Refactor image module and transition to DockerClient https://review.opendev.org/c/openstack/kolla/+/825786 | 14:03 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla stable/train: CI: make it work again https://review.opendev.org/c/openstack/kolla/+/826033 | 14:29 |
| hrw | mnasiadka, mgoddard, other cores: https://review.opendev.org/c/openstack/kolla/+/826273 is trivial | 14:43 |
| opendevreview | Merged openstack/kolla-ansible master: fluentd: add modern parsing config https://review.opendev.org/c/openstack/kolla-ansible/+/823093 | 14:48 |
| opendevreview | Merged openstack/kayobe stable/victoria: Generate kolla config when running database commands https://review.opendev.org/c/openstack/kayobe/+/823765 | 14:55 |
| yoctozepto | mgoddard, mnasiadka: I see you are adding lots of BC+2 today | 15:01 |
| mgoddard | yoctozepto: I think I have corrected them all, but let me know if not | 15:01 |
| mgoddard | yoctozepto: gerrit UI change | 15:01 |
| mgoddard | moaned about it already | 15:02 |
| mnasiadka | Yes, lovely UI change. | 15:02 |
| yoctozepto | mgoddard: ack, here https://review.opendev.org/c/openstack/kolla-ansible/+/823094 | 15:02 |
| yoctozepto | ah, yeas | 15:02 |
| yoctozepto | so it's not me | 15:03 |
| yoctozepto | I like the new order better, but needs getting used to again | 15:03 |
| hrw | CR back at top is good | 15:04 |
| hrw | k-a in train fails. meh | 15:06 |
| yoctozepto | mgoddard: https://review.opendev.org/c/openstack/kolla-ansible/+/823155 another BC+2 | 15:37 |
| mgoddard | muscle memory is hard to defeat | 15:37 |
| yoctozepto | mgoddard: and one more https://review.opendev.org/c/openstack/kolla-ansible/+/825715 | 15:38 |
| yoctozepto | mgoddard: yeah, it truly is | 15:38 |
| yoctozepto | priteau: I CC'ed you on #opendev; I agree it would be nice to have the CR+2 button up | 15:43 |
| yoctozepto | the W+1 button is there if you leave CR+2 | 15:43 |
| priteau | Thanks | 15:43 |
| priteau | Ah, I knew there was one button but couldn't remember what it was | 15:44 |
| yoctozepto | priteau: clarkb solved the mystery | 15:45 |
| opendevreview | Marcin Juszkiewicz proposed openstack/kolla stable/train: CI: make it work again https://review.opendev.org/c/openstack/kolla/+/826033 | 15:52 |
| opendevreview | Merged openstack/kolla master: Upgrade pip and friends in source images (part 2) https://review.opendev.org/c/openstack/kolla/+/826273 | 16:20 |
| opendevreview | Merged openstack/kolla stable/wallaby: openstack-base: drop anyjson https://review.opendev.org/c/openstack/kolla/+/826052 | 16:21 |
| opendevreview | Merged openstack/kolla stable/victoria: openstack-base: drop anyjson https://review.opendev.org/c/openstack/kolla/+/826053 | 16:21 |
| opendevreview | Merged openstack/kolla stable/ussuri: openstack-base: drop anyjson https://review.opendev.org/c/openstack/kolla/+/826054 | 16:21 |
| opendevreview | Merged openstack/kolla-ansible master: Fluentd cleanup https://review.opendev.org/c/openstack/kolla-ansible/+/823094 | 16:22 |
| opendevreview | Kevin Rasmussen proposed openstack/kolla-ansible master: Fix bad openstack command while registering IDP https://review.opendev.org/c/openstack/kolla-ansible/+/826322 | 16:25 |
| opendevreview | Merged openstack/kolla master: Clean up fluentd image recipe https://review.opendev.org/c/openstack/kolla/+/823071 | 16:38 |
| mnasiadka | hrw: I think you need a depends-on on the Kolla change? | 16:48 |
| hrw | mnasiadka: in k-a? | 16:56 |
| hrw | mnasiadka: feel free to add | 16:57 |
| hrw | I am better at solving kolla part than k-a one | 16:58 |
| yoctozepto | mnasiadka: if k needs k-a then it would be a circular dep | 16:59 |
| opendevreview | Kevin Rasmussen proposed openstack/kolla-ansible master: Fix bad openstack command while registering IDP https://review.opendev.org/c/openstack/kolla-ansible/+/826322 | 17:18 |
| opendevreview | Merged openstack/kolla-ansible master: update the default value of node_custom_config https://review.opendev.org/c/openstack/kolla-ansible/+/825715 | 17:24 |
| mnasiadka | yoctozepto: well, k-a doesn't need k, I just wanted to see the CI in Kolla green :) | 17:56 |
| hrw | mnasiadka: EOL train and you will see ;D | 18:01 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Fix bad openstack command while registering IDP https://review.opendev.org/c/openstack/kolla-ansible/+/826322 | 18:02 |
| mnasiadka | hrw: I don't mind, but there are people using it, so let's wait the usual timeline :) | 18:09 |
| hrw | I know | 18:09 |
| hrw | mnasiadka: kolla part is solved. now time to solve k-a side | 18:10 |
| opendevreview | Konstantin Yarovoy proposed openstack/kolla master: Refactor image module and transition to DockerClient https://review.opendev.org/c/openstack/kolla/+/825786 | 18:23 |
| opendevreview | Buddhika Sanjeewa proposed openstack/kolla-ansible master: Deploy Zun with Cinder Ceph support https://review.opendev.org/c/openstack/kolla-ansible/+/824722 | 19:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
