| opendevreview | wangjiaqi proposed openstack/kolla master: Drop support for python 3.6 and 3.7 https://review.opendev.org/c/openstack/kolla/+/850838 | 05:50 |
|---|---|---|
| opendevreview | Merged openstack/kolla master: rabbitmq: Update to 3.10 https://review.opendev.org/c/openstack/kolla/+/853318 | 06:58 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/yoga: Pin docker to <6 https://review.opendev.org/c/openstack/kolla/+/855120 | 06:59 |
| opendevreview | Rafal Lewandowski proposed openstack/kayobe master: bumped up stackhpc.os-images version https://review.opendev.org/c/openstack/kayobe/+/854359 | 07:03 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/yoga: Pin docker to <6 https://review.opendev.org/c/openstack/kolla/+/855120 | 07:19 |
| frickler | being helpful and welcoming to new contributors gets really difficult when they don't seem to understand what one is writing | 07:48 |
| opendevreview | wangjiaqi proposed openstack/kolla master: Drop support for python 3.6 and 3.7 https://review.opendev.org/c/openstack/kolla/+/850838 | 08:01 |
| frickler | yoctozepto: just noticed that there is also a typo in the filename https://review.opendev.org/c/openstack/kolla/+/850838/10/releasenotes/notes/drop-pyhton-3-6-and-3-7-6cc88979aad423e8.yaml but I'd tend to ignore that, wdyt? | 08:13 |
| yoctozepto | frickler: yeah, nothin cares about these filenames except for uniqueness | 08:17 |
| yoctozepto | there are lots of typos already | 08:17 |
| opendevreview | wu.chunyang proposed openstack/kolla master: Bump prometheus services to latest version https://review.opendev.org/c/openstack/kolla/+/854347 | 08:25 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla master: Drop centos8s jobs from experimental pipeline https://review.opendev.org/c/openstack/kolla/+/855128 | 08:32 |
| frickler | mnasiadka: ^^ | 08:32 |
| frickler | also I feel like we should repair the prometheus job before we merge prometheus patches | 08:33 |
| mnasiadka | frickler: zuul doesn't like this change ;) | 08:33 |
| frickler | meh | 08:33 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla master: Drop centos8s jobs from experimental pipeline https://review.opendev.org/c/openstack/kolla/+/855128 | 08:36 |
| opendevreview | wu.chunyang proposed openstack/kolla-ansible master: [doc] Run docker registry with host network https://review.opendev.org/c/openstack/kolla-ansible/+/854637 | 08:44 |
| *** brinzhang_ is now known as brinzhang | 09:09 | |
| mnasiadka | bloody hell, need to combine docker <6 patch with mariadb patch | 10:03 |
| yoctozepto | bloody indeed | 10:13 |
| yoctozepto | and also tweak the global-reqs | 10:14 |
| yoctozepto | or just stop caring | 10:14 |
| opendevreview | Michal Nasiadka proposed openstack/kolla stable/yoga: mariadb: Fix RPM repository URLs https://review.opendev.org/c/openstack/kolla/+/854546 | 10:22 |
| kevko | mnasiadka: which patch to combine with mariadb ? can I help ? | 11:47 |
| mnasiadka | kevko: you can help with fixing prometheus-efk job that seems to fail after proxysql implementation :) | 11:49 |
| mnasiadka | ERROR: Requirement for package docker excludes a version not excluded in the global list. | 11:50 |
| mnasiadka | Local settings : {'<6'} | 11:50 |
| mnasiadka | Global settings: set() | 11:50 |
| mnasiadka | Unexpected : {'<6'} | 11:50 |
| mnasiadka | yoctozepto: I just love it | 11:51 |
| kevko | mnasiadka: oh sure, but i don't know how proxysql can broke something if it is enabled: no | 11:53 |
| kevko | mnasiadka: can u send me a log ? | 11:53 |
| frickler | kevko: any of the recent runs like https://zuul.opendev.org/t/openstack/build/c1454e526ef34d1ea0587f6a2a9085f1 | 12:03 |
| mnasiadka | yoctozepto, frickler: are we updating global-requirements or backporting the fix (I don't think anybody on Wallaby+ should be using older Docker than we require)? | 12:15 |
| kevko | thanks | 12:17 |
| yoctozepto | mnasiadka: makes sense in practice | 12:17 |
| kevko | "couldn't resolve module/action 'sysctl'. This often indicates a misspelling, missing collection, or incorrect module path.\n\nThe error appears to be in | 12:36 |
| kevko | ^^ bad ansible or what ? | 12:36 |
| mnasiadka | only ansible-core installed | 12:38 |
| mnasiadka | kolla-ansible needs full ansible package | 12:38 |
| kevko | hmm ok, thanks | 12:45 |
| kevko | btw ..can we add this to requirements and not parse from bash as it is now ? | 12:46 |
| kevko | or - include installation to install-deps .. | 12:46 |
| kevko | because i have to support several installations of openstack in our CI and i had a script to install right version ...but from ansible -> ansible-base + ansible -> ansible-core + ansible my script is working ..well bad ;D | 12:47 |
| opendevreview | Merged openstack/kolla-ansible master: Support configuring HAProxy services as active/passive https://review.opendev.org/c/openstack/kolla-ansible/+/854977 | 12:51 |
| opendevreview | Merged openstack/kolla-ansible master: Use active_passive parameter for existing services https://review.opendev.org/c/openstack/kolla-ansible/+/855019 | 12:56 |
| opendevreview | Merged openstack/kolla-ansible master: Switch Prometheus Alertmanager to active/passive https://review.opendev.org/c/openstack/kolla-ansible/+/854816 | 12:56 |
| frickler | mnasiadka: I don't think we should ignore stable policy, no matter how sensible that me be in this special case. another option would be to actually fix our version check, I had almost done that before it occured to me that it is easier to simply drop it. let me dig that out | 13:01 |
| mnasiadka | frickler: requirements-check won't allow us pinning docker, unless we raise a change to requirements - that's probably going to take some time | 13:02 |
| mnasiadka | so that's why I'm thinking of alternatives | 13:02 |
| frickler | mnasiadka: yes, with what I propose, we won't need that | 13:03 |
| frickler | give me a minute or two | 13:03 |
| mnasiadka | sure | 13:03 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix creation of prometheus user and grant permissions https://review.opendev.org/c/openstack/kolla-ansible/+/855206 | 13:05 |
| kevko | this should fix prometheus -> https://review.opendev.org/c/openstack/kolla-ansible/+/855206 | 13:05 |
| kevko | mnasiadka: ^^ | 13:06 |
| yoctozepto | kevko: found another bug in the same file | 13:17 |
| kevko | yoctozepto: hmm, why ? it should run only once ... | 13:18 |
| yoctozepto | kevko: yes, but then we must not filter on the inventory_hostname as ansible can pick some other sone | 13:18 |
| yoctozepto | s/sone/one | 13:18 |
| frickler | meh, importlib.metadata was added in py38 only, can't use that for legacy distros either. then some nasty try-clause it will be | 13:21 |
| kevko | i am not sure if understand .. | 13:23 |
| kevko | sometimes i think we should provide docker container for whole kolla-ansible ..some tiny one :P | 13:23 |
| yoctozepto | kevko: run_once is decided first, then the "when" is consulted | 13:23 |
| kevko | yoctozepto: aaaa, understand ...and run_once is preferred i suppose | 13:27 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla stable/yoga: mariadb: Fix RPM repository URLs https://review.opendev.org/c/openstack/kolla/+/854546 | 13:28 |
| frickler | kevko: we (osism) have https://github.com/osism/container-image-kolla-ansible/ , happy to support if someone wants to make it an upstream project | 13:29 |
| kevko | frickler: thank you, i will check | 13:33 |
| wuchunyang | EFK CI task failed. failed: [localhost] (item=localhost) => {"action": "mysql_user", "ansible_loop_var": "item", "changed": false, "item": {"key": "0", "value": {"hosts": ["localhost"]}}, "msg": "unable to connect to database, check login_user and login_password are correct or /var/lib/ansible/.my.cnf has the credentials. Exception message: (1045, \"Access denied for user 'root_shard_0'@'bj-openstack' (using password: YES)\") | 13:33 |
| wuchunyang | "} | 13:33 |
| kevko | hmm | 13:34 |
| kevko | proxysql_enabled: true ? | 13:34 |
| frickler | wuchunyang: yes, that's what we were talking about earlier | 13:34 |
| kevko | wuchunyang: do you have proxysql enabled right ? | 13:35 |
| wuchunyang | No， i try it now. | 13:36 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Fix creation of prometheus user and grant permissions https://review.opendev.org/c/openstack/kolla-ansible/+/855206 | 13:37 |
| frickler | wuchunyang: iiuc https://review.opendev.org/c/openstack/kolla-ansible/+/855206 should be the fix. but need to run the actual job on it to verify | 13:38 |
| kevko | wuchunyang: did you apply this patch ? https://review.opendev.org/c/openstack/kolla-ansible/+/855206 ? | 13:38 |
| yoctozepto | let's wait for Zuul results but I believe it's the right solution | 13:39 |
| wuchunyang | No, i have not applied it yet. | 13:39 |
| kevko | wuchunyang: so apply that patch and it will work | 13:39 |
| frickler | mnasiadka: https://zuul.opendev.org/t/openstack/build/19408cdf26764436970ddf06d19f5826 ... I'll update once again | 13:39 |
| wuchunyang | thanks. | 13:39 |
| kevko | btw, did someone try proxysql after merge :) ? | 13:39 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla stable/yoga: mariadb: Fix RPM repository URLs https://review.opendev.org/c/openstack/kolla/+/854546 | 13:40 |
| wuchunyang | I can test proxysql. | 13:40 |
| kevko | wuchunyang: you can ..it's merged | 13:41 |
| kevko | wuchunyang: just enable proxysql in globals and run deploy with tags mariadb,loadbalancer | 13:42 |
| frickler | INFO:kolla.common.utils.horizon:ImportError: cannot import name 'force_src' from 'django.utils.encoding' (/usr/lib/python3/dist-packages/django/utils/encoding.py) | 13:43 |
| wuchunyang | It pass now without proxysql, i would try enable proxysql next. | 13:43 |
| frickler | yoctozepto: mnasiadka: ^^ failure for ubuntu-binary on yoga, is that a known thing? | 13:43 |
| yoctozepto | frickler: I doubt it | 13:44 |
| yoctozepto | I mean, probably know to some unlucky end users <sweat-smile> | 13:44 |
| yoctozepto | s/know/known | 13:44 |
| kevko | frickler: distro ? | 13:54 |
| wuchunyang | It passed in my local, i would + w, if no objection. | 14:13 |
| yoctozepto | let's wait for zuul nonetheless | 14:15 |
| wuchunyang | okay No problem. | 14:17 |
| wuchunyang | yoctozepto: hi i have a question. why the kolla project can run ci tasks that defined in kolla-ansible project? | 14:23 |
| yoctozepto | wuchunyang: because of how zuul behaves - it allows us to reuse jobs from other repos; similarly, the bifrost repo tests with kolla-ansible | 14:25 |
| wuchunyang | how do we use this ? we just need claim queue field in zuul project dict? I saw this field in both kolla and kolla-ansible repo. | 14:30 |
| yoctozepto | wuchunyang: no, queue is irrelevant for this to work; queue is for automatic behaviour with depends-on and approvals | 14:38 |
| yoctozepto | one just needs to reference the job name, that's it | 14:39 |
| yoctozepto | you can test trove changes with kolla if you want | 14:39 |
| wuchunyang | yes, haha, this is what i want . | 14:39 |
| yoctozepto | are you planning to drop devstack support? ;p | 14:40 |
| opendevreview | Mark Goddard proposed openstack/kayobe stable/yoga: Fix IP allocation limit in overcloud inventory discover https://review.opendev.org/c/openstack/kayobe/+/855143 | 14:40 |
| opendevreview | Mark Goddard proposed openstack/kayobe stable/xena: Fix IP allocation limit in overcloud inventory discover https://review.opendev.org/c/openstack/kayobe/+/855144 | 14:40 |
| wuchunyang | currently, not .. | 14:40 |
| opendevreview | Mark Goddard proposed openstack/kayobe stable/wallaby: Fix IP allocation limit in overcloud inventory discover https://review.opendev.org/c/openstack/kayobe/+/855145 | 14:40 |
| wuchunyang | devstack is still working now. | 14:40 |
| yoctozepto | nah, nah, just kidding, it's still the recommended tool and what not | 14:41 |
| opendevreview | Mark Goddard proposed openstack/kayobe stable/yoga: Fix use of become_user by installing acl package https://review.opendev.org/c/openstack/kayobe/+/855147 | 14:41 |
| opendevreview | Mark Goddard proposed openstack/kayobe stable/xena: Fix use of become_user by installing acl package https://review.opendev.org/c/openstack/kayobe/+/855148 | 14:41 |
| opendevreview | Mark Goddard proposed openstack/kayobe stable/wallaby: Fix use of become_user by installing acl package https://review.opendev.org/c/openstack/kayobe/+/855149 | 14:41 |
| wuchunyang | i would like to add a test like octavia in kolla, but trove-guest needs access Internet, that's the problem i don't know how to resolve. | 14:42 |
| yoctozepto | what does trove-guest need to do with the internet? | 14:43 |
| * yoctozepto has no idea how trove works | 14:43 | |
| wuchunyang | trove-guest uses docker to pull database image(e.g. mysql:8.0) now. | 14:44 |
| wuchunyang | and I can't download the docker image in the image building by DIB. | 14:45 |
| yoctozepto | hmm, well, it's going to be wasteful but we can set up internet access in there; btw, can the registry url be set with trove? | 14:47 |
| yoctozepto | this way we could use the mirror and make it much less wasteful | 14:47 |
| wuchunyang | maybe， i should try to use a local registry in host, and trove-guest access this registry by mgmt network. | 14:48 |
| yoctozepto | and yes, that would be best to be honest | 14:49 |
| wuchunyang | by default , Do we will run docker registry in host ? | 14:49 |
| yoctozepto | no, normally only on image build but it can be overridden easily | 14:51 |
| wuchunyang | can the registry url be set with trove? yes, trove supports local registry. | 14:51 |
| yoctozepto | well, then it saves us from the need to enable internet access there | 14:51 |
| yoctozepto | just use the local registry to proxy the required images | 14:52 |
| yoctozepto | and there we use the infra proxy to optimise the traffic | 14:52 |
| wuchunyang | yes, if we have a local registry, i would be better. | 14:52 |
| wuchunyang | s/i/it/ | 14:52 |
| wuchunyang | I would try to add those functions in kolla. And i need you help with CI. thanks in advance. | 14:54 |
| yoctozepto | happy to help, just let me know where exactly | 14:56 |
| wuchunyang | that's no problem. I need sleep now, good night. | 14:59 |
| yoctozepto | good night | 15:00 |
| opendevreview | Dr. Jens Harbott proposed openstack/kolla stable/yoga: mariadb: Fix RPM repository URLs https://review.opendev.org/c/openstack/kolla/+/854546 | 16:06 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Remove deprecated sysctl knobs https://review.opendev.org/c/openstack/kolla-ansible/+/855259 | 16:27 |
| opendevreview | Merged openstack/kolla-ansible master: Fix creation of prometheus user and grant permissions https://review.opendev.org/c/openstack/kolla-ansible/+/855206 | 17:18 |
| opendevreview | Verification of a change to openstack/kayobe stable/yoga failed: Remove unused parameter from seed container example https://review.opendev.org/c/openstack/kayobe/+/854799 | 17:22 |
| opendevreview | Verification of a change to openstack/kayobe stable/xena failed: Remove unused parameter from seed container example https://review.opendev.org/c/openstack/kayobe/+/854800 | 19:32 |
| opendevreview | Merged openstack/kolla stable/yoga: mariadb: Fix RPM repository URLs https://review.opendev.org/c/openstack/kolla/+/854546 | 19:35 |
| opendevreview | Merged openstack/kolla-ansible master: [security] Make Ironic tftpd run as nobody https://review.opendev.org/c/openstack/kolla-ansible/+/854841 | 19:35 |
| opendevreview | Merged openstack/kolla-ansible master: Fix haproxy prechecks after deployment https://review.opendev.org/c/openstack/kolla-ansible/+/855071 | 19:36 |
| opendevreview | Merged openstack/kolla-ansible master: keystone: Remove redundant 'when' https://review.opendev.org/c/openstack/kolla-ansible/+/852739 | 19:36 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/yoga: [security] Make Ironic tftpd run as nobody https://review.opendev.org/c/openstack/kolla-ansible/+/855235 | 19:36 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/xena: [security] Make Ironic tftpd run as nobody https://review.opendev.org/c/openstack/kolla-ansible/+/855236 | 19:37 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/xena: [security] Make Ironic tftpd run as nobody https://review.opendev.org/c/openstack/kolla-ansible/+/855236 | 19:38 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: [security] Make Ironic tftpd run as nobody https://review.opendev.org/c/openstack/kolla-ansible/+/855237 | 19:38 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/victoria: [security] Make Ironic tftpd run as nobody https://review.opendev.org/c/openstack/kolla-ansible/+/855238 | 19:38 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: [security] Make Ironic tftpd run as nobody https://review.opendev.org/c/openstack/kolla-ansible/+/855239 | 19:39 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/yoga: keystone: Remove redundant 'when' https://review.opendev.org/c/openstack/kolla-ansible/+/855240 | 19:41 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/xena: keystone: Remove redundant 'when' https://review.opendev.org/c/openstack/kolla-ansible/+/855241 | 19:41 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: keystone: Remove redundant 'when' https://review.opendev.org/c/openstack/kolla-ansible/+/855242 | 19:41 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/victoria: keystone: Remove redundant 'when' https://review.opendev.org/c/openstack/kolla-ansible/+/855243 | 19:41 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: keystone: Remove redundant 'when' https://review.opendev.org/c/openstack/kolla-ansible/+/855244 | 19:41 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/yoga: Fix haproxy prechecks after deployment https://review.opendev.org/c/openstack/kolla-ansible/+/855245 | 19:42 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/xena: Fix haproxy prechecks after deployment https://review.opendev.org/c/openstack/kolla-ansible/+/855246 | 19:42 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Fix haproxy prechecks after deployment https://review.opendev.org/c/openstack/kolla-ansible/+/855247 | 19:42 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Fix haproxy prechecks after deployment https://review.opendev.org/c/openstack/kolla-ansible/+/855247 | 19:43 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Fix haproxy prechecks after deployment https://review.opendev.org/c/openstack/kolla-ansible/+/855247 | 19:43 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/victoria: Fix haproxy prechecks after deployment https://review.opendev.org/c/openstack/kolla-ansible/+/855248 | 19:43 |
| opendevreview | Radosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Fix haproxy prechecks after deployment https://review.opendev.org/c/openstack/kolla-ansible/+/855249 | 19:44 |
| opendevreview | Merged openstack/kayobe stable/wallaby: Remove unused parameter from seed container example https://review.opendev.org/c/openstack/kayobe/+/854801 | 20:01 |
| opendevreview | Merged openstack/kayobe stable/victoria: Remove unused parameter from seed container example https://review.opendev.org/c/openstack/kayobe/+/854802 | 20:01 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!