| bbezak | Hi, could you please take a look frickler, kevko https://review.opendev.org/q/topic:%22ironic-rbac%22 | 09:40 |
|---|---|---|
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Revert "CI: retry smoke tests and instance creation" https://review.opendev.org/c/openstack/kolla-ansible/+/904729 | 10:39 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: stable-only: Drop upgrade testing https://review.opendev.org/c/openstack/kolla-ansible/+/908250 | 10:39 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: Keystone: Add service role https://review.opendev.org/c/openstack/kolla-ansible/+/909084 | 10:39 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: Update keystone service user passwords https://review.opendev.org/c/openstack/kolla-ansible/+/908273 | 10:39 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Update keystone service user passwords https://review.opendev.org/c/openstack/kolla-ansible/+/908271 | 11:18 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Update keystone service user passwords https://review.opendev.org/c/openstack/kolla-ansible/+/908272 | 11:18 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.1: Fix mariadb role when used with check mode https://review.opendev.org/c/openstack/kolla-ansible/+/909096 | 11:18 |
| opendevreview | Merged openstack/kolla-ansible stable/zed: Fix mariadb role when used with check mode https://review.opendev.org/c/openstack/kolla-ansible/+/909097 | 11:18 |
| bbezak | thx kevko | 11:19 |
| kevko | bbezak: np | 11:24 |
| opendevreview | Merged openstack/kolla-ansible stable/2023.2: Fix mariadb role when used with check mode https://review.opendev.org/c/openstack/kolla-ansible/+/909095 | 11:26 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Revert "Pin ansible-core at 2.15.6" https://review.opendev.org/c/openstack/kayobe/+/908276 | 11:44 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Revert "Pin ansible-core at 2.15.6" https://review.opendev.org/c/openstack/kayobe/+/908276 | 11:45 |
| opendevreview | Merged openstack/kolla-ansible master: Ironic: enable elevated access for project scoped service role https://review.opendev.org/c/openstack/kolla-ansible/+/908007 | 12:40 |
| opendevreview | Merged openstack/kolla-ansible master: Add service role to ironic service users https://review.opendev.org/c/openstack/kolla-ansible/+/908580 | 12:40 |
| opendevreview | Merged openstack/kolla-ansible master: Template system scoped admin-openrc and clouds.yml files https://review.opendev.org/c/openstack/kolla-ansible/+/908168 | 12:40 |
| opendevreview | Merged openstack/kolla-ansible master: Revert "Disable new defaults and scope for Ironic (RBAC)" https://review.opendev.org/c/openstack/kolla-ansible/+/907274 | 12:43 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Do not remove inventory file if placed in /etc/kolla https://review.opendev.org/c/openstack/kolla-ansible/+/908429 | 12:47 |
| mnasiadka | kevko: https://review.opendev.org/c/openstack/kolla-ansible/+/908405 - have some time to have a look? ;-) | 12:58 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: cinder: start using active-active for rbd https://review.opendev.org/c/openstack/kolla-ansible/+/763011 | 13:15 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: cinder: start using active-active for rbd https://review.opendev.org/c/openstack/kolla-ansible/+/763011 | 13:16 |
| kevko | mnasiadka: yep | 13:36 |
| kevko | guys, from time to time i'm seeing this in my testing pipelines | 13:36 |
| kevko | blablabla ....."msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": -13}[0m | 13:36 |
| mnasiadka | kevko: broken pipe? | 13:38 |
| mnasiadka | seems the python module was killed with SIGPIPE | 13:39 |
| mnasiadka | (that's rc 13) | 13:39 |
| kevko | but why :( ...it's only from time to time ..and hard to debug :D | 13:40 |
| kevko | nobody see this also ? | 13:40 |
| mnasiadka | not really | 13:40 |
| mnasiadka | any specific module, or different modules? | 13:41 |
| kevko | mnasiadka: I didn't notice, because I usually start pipeline again and it goes through for the second attempt | 13:42 |
| frickler | I think this is the known issue with ssh pipelining timeout? had the same thing in opendev things. if a task takes exactly as long as the timeout is, a race happens | 13:42 |
| kevko | frickler: so is it configurable I suppose ? | 13:42 |
| frickler | yes, but that only helps in making it happen less often. let me try to find a patch | 13:43 |
| kevko | frickler: this one ? ServerAliveInterval ServerAliveCountMax ? | 13:44 |
| frickler | no. https://review.opendev.org/c/opendev/system-config/+/902436 | 13:45 |
| frickler | seems we could do the same in tests/templates/ansible.cfg.j2 | 13:46 |
| kevko | frickler: oh, thanks ...i will try it | 13:46 |
| mnasiadka | frickler: https://review.opendev.org/c/openstack/kolla-ansible/+/902699/1/tests/templates/ansible.cfg.j2 - we have that | 14:00 |
| SvenKieske | yes, I have seen this error btw sometimes in the past (I can't link to anything sadly, it's been some time ago | 14:10 |
| kevko | mnasiadka: so i will put this also to my inventory template | 14:10 |
| frickler | mnasiadka: ah, thx, I was somehow looking in a stable/zed checkout accidentally | 14:22 |
| frickler | which I guess means we might want to backport that to stable branches? | 14:23 |
| opendevreview | Merged openstack/kayobe stable/2023.2: Fix wipe-disks role to work on util-linux >= 2.37 https://review.opendev.org/c/openstack/kayobe/+/909089 | 14:41 |
| opendevreview | Merged openstack/kayobe stable/2023.1: Fix wipe-disks role to work on util-linux >= 2.37 https://review.opendev.org/c/openstack/kayobe/+/909090 | 14:41 |
| opendevreview | Merged openstack/kayobe stable/zed: Fix wipe-disks role to work on util-linux >= 2.37 https://review.opendev.org/c/openstack/kayobe/+/909091 | 14:41 |
| kevko | mnasiadka: btw - regarding cinder service role .. shouldn't we add some test to verify it's working ? | 14:43 |
| mnasiadka | frickler: would make sense | 14:43 |
| mnasiadka | kevko: it's tested in cephadm jobs - we have cinder there | 14:44 |
| kevko | mnasiadka: i remeber that I was getting different code via Rest api if i was manipulating with attachments .. | 14:44 |
| mnasiadka | and do all sorts of weird tests with it | 14:44 |
| kevko | mnasiadka: (in rally/tempest) | 14:44 |
| mnasiadka | well then it's time to switch to tempest :D | 14:44 |
| kevko | mnasiadka: it is ! i would like to see it working :) | 14:47 |
| SvenKieske | yes, tempest would be nice | 14:47 |
| kevko | or better said ..i think rally is better for our usage | 14:48 |
| kevko | simpler configuration and docker container upstream prepared | 14:48 |
| kevko | *image | 14:48 |
| mnasiadka | which downloads tempest when you run tests | 14:56 |
| kevko | frickler: mnasiadka: i have this in my pipeline ... | 14:56 |
| kevko | ssh_args = -o ControlMaster=auto -o ControlPersist=60s | 14:56 |
| kevko | do you think 60sec is low value ? | 14:57 |
| mnasiadka | so basically just another interface to tempest | 14:57 |
| kevko | mnasiadka: yea - but it is containerized, ready, it has very nice commands for reports which can be printed on the end ..or export to html ...etc...etc..etc | 14:57 |
| kevko | mnasiadka: so on the end you can just open report and see what happened | 14:58 |
| kevko | mnasiadka: tempest itself can't do this | 14:58 |
| mnasiadka | come on, junit2html can do this | 14:58 |
| mnasiadka | devstack runs tempest using tox, I hoped we could reuse what all projects CI jobs do today to run tempest | 14:59 |
| kevko | mnasiadka: why i should implement something more ? :D | 14:59 |
| mnasiadka | instead of reinventing the wheel | 14:59 |
| mattcrees | Afternoon everyone, I've got a bugfix in a-c-k looking for a second reviewer if anyone fancies it: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/907107 | 15:26 |
| SvenKieske | public service announcement: most distros really switched some time (years) ago all stuff from "/bin" to "/usr/bin/", that means, "/bin/" is just a symlink to "usr/bin", thx! | 15:32 |
| SvenKieske | see also: https://www.freedesktop.org/wiki/Software/systemd/TheCaseForTheUsrMerge/ | 15:34 |
| SvenKieske | can we merge this oneline fix? https://review.opendev.org/c/openstack/kolla-ansible/+/907495 (cores needed) | 15:45 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Fix 'cinder-backup' service when Swift with TLS enabled https://review.opendev.org/c/openstack/kolla-ansible/+/907495 | 15:58 |
| SvenKieske | something wrong with gerrit? I can't edit (save) a draft message on a WIP change, I'm getting a gerrit error message. | 16:09 |
| SvenKieske | weird gate broken as well, never seen this error before: https://review.opendev.org/c/openstack/kolla-ansible/+/907495 | 16:22 |
| SvenKieske | ah I guess that is because of the Depends-On? Change 909242 in project openstack/cinder does not share a change queue with 907495 in project openstack/kolla-ansible | 16:22 |
| frickler | in this case, that's what you get from depending on change-ids instead of review urls, because they also match all cherry-picks | 16:42 |
| opendevreview | Merged openstack/ansible-collection-kolla master: Fix: include missing docker log-opts in config https://review.opendev.org/c/openstack/ansible-collection-kolla/+/907107 | 16:43 |
| opendevreview | Matt Crees proposed openstack/ansible-collection-kolla stable/2023.2: Fix: include missing docker log-opts in config https://review.opendev.org/c/openstack/ansible-collection-kolla/+/909252 | 16:47 |
| opendevreview | Matt Crees proposed openstack/ansible-collection-kolla stable/2023.1: Fix: include missing docker log-opts in config https://review.opendev.org/c/openstack/ansible-collection-kolla/+/909253 | 16:47 |
| opendevreview | Matt Crees proposed openstack/ansible-collection-kolla stable/zed: Fix: include missing docker log-opts in config https://review.opendev.org/c/openstack/ansible-collection-kolla/+/909254 | 16:48 |
| opendevreview | Ivan Halomi proposed openstack/kolla-ansible master: Refactor of docker worker https://review.opendev.org/c/openstack/kolla-ansible/+/908295 | 16:49 |
| opendevreview | Merged openstack/kayobe master: Make OVS container build regex independent to OVN https://review.opendev.org/c/openstack/kayobe/+/907721 | 17:13 |
| opendevreview | Pierre Riteau proposed openstack/kayobe master: Rephrase kolla_build_neutron_ovs release note https://review.opendev.org/c/openstack/kayobe/+/909473 | 17:34 |
| *** NeilHanlon is now known as nhanlon | 19:00 | |
| *** nhanlon is now known as NeilHanlon | 19:02 | |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Fix 'cinder-backup' service when Swift with TLS enabled https://review.opendev.org/c/openstack/kolla-ansible/+/907495 | 20:07 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Fix 'cinder-backup' service when Swift with TLS enabled https://review.opendev.org/c/openstack/kolla-ansible/+/907495 | 20:13 |
| opendevreview | Uwe Jäger proposed openstack/kolla-ansible master: Allow overriding of Skyline logos https://review.opendev.org/c/openstack/kolla-ansible/+/909481 | 21:19 |
| opendevreview | Uwe Jäger proposed openstack/kolla-ansible master: Add more services to nginx.conf https://review.opendev.org/c/openstack/kolla-ansible/+/909482 | 21:21 |
| SvenKieske | mhm, wondering how much we want to support the ansible --limit option? stumbling upon some bugs here and there when using --limit | 21:35 |
| SvenKieske | e.g. nova-status upgrade check runs on the first host of the inventory, but only registers the variable, so if I limit the execution to a different host during upgrade the subsequent tasks do fail because other hosts do not have access to a registered variable on a different host: https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/nova/tasks/upgrade.yml#L20C3-L20C11 | 21:37 |
| SvenKieske | we could set this as a fact instead, I'd volunteer to change this and write a bugreport if there is consensus that we want to support the --limit option, at least for nova upgrades | 21:38 |
| SvenKieske | I filed a bug already, to keep track of it, we can still close it, or I just post a documentation patch which closes it: https://bugs.launchpad.net/kolla-ansible/+bug/2054348 | 21:46 |
| SvenKieske | also added it to the whiteboard | 21:48 |
| opendevreview | Sven Kieske proposed openstack/kolla-ansible master: Fix 'cinder-backup' service when Swift with TLS enabled https://review.opendev.org/c/openstack/kolla-ansible/+/907495 | 21:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!