parallax | Morning | 07:03 |
---|---|---|
*** amoralej|off is now known as amoralej | 07:11 | |
Fl1nt | howdy everyone! | 09:07 |
kevko | morning | 09:20 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Restore libvirtd cgroupfs mount https://review.opendev.org/c/openstack/kolla-ansible/+/806555 | 09:36 |
priteau | Good morning | 09:42 |
opendevreview | Pierre Riteau proposed openstack/kayobe stable/wallaby: Fix documentation links for routed control plane networks https://review.opendev.org/c/openstack/kayobe/+/806518 | 09:50 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Test instance health after upgrade https://review.opendev.org/c/openstack/kolla-ansible/+/806331 | 09:53 |
priteau | yoctozepto: Are you aware of breakage on ubuntu/xena? We are seeing this in Kayobe CI when building container images | 10:21 |
priteau | GPG error: http://ubuntu-cloud.archive.canonical.com/ubuntu focal-updates/xena InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 5EDB1B62EC4926EA | 10:21 |
yoctozepto | priteau: not aware; is it running kolla there? or..? | 10:22 |
priteau | It runs "kayobe seed container image build ^base", which calls kolla-build | 10:23 |
priteau | It could be related to https://review.opendev.org/c/openstack/kolla/+/784923 | 10:27 |
yoctozepto | hmm, so does kayobe try to customise repos? | 10:32 |
priteau | Nope | 10:35 |
priteau | The patch linked above used to include this key | 10:35 |
priteau | {# 391A9AA2147192839E9DB0315EDB1B62EC4926EA -- Canonical Cloud Archive Signing Key <ftpmaster@canonical.com> #} | 10:35 |
priteau | It can be installed with the ubuntu-cloud-keyring package | 10:35 |
priteau | I don't know why we would see this only in kayobe and not in kolla CI. Does kolla check certificates? | 10:36 |
priteau | s/kolla/kolla CI/ | 10:36 |
opendevreview | Pierre Riteau proposed openstack/kolla master: Install UCA GPG key https://review.opendev.org/c/openstack/kolla/+/806560 | 10:43 |
priteau | Trying this as a fix ^^ | 10:43 |
ignaziocassano | Hello All, I have just update kolla wallaby with images released 37 hours ago but hacluster stopped working with them: | 10:57 |
priteau | base_ubuntu_package_pre applies to debian too :/ | 10:57 |
ignaziocassano | Hello All, I have just update kolla wallaby with images released 37 hours ago but hacluster stopped working with them: TASK [hacluster : Ensure stonith is disabled] fails | 10:58 |
opendevreview | Pierre Riteau proposed openstack/kolla master: Install UCA GPG key https://review.opendev.org/c/openstack/kolla/+/806560 | 10:58 |
ignaziocassano | sorry 27 hours ago | 10:58 |
ignaziocassano | The firs controller reports the following in pacemaker.log :https://paste.openstack.org/show/808400/ | 11:02 |
opendevreview | Pierre Riteau proposed openstack/kolla master: Install UCA GPG key https://review.opendev.org/c/openstack/kolla/+/806560 | 11:16 |
*** Flint is now known as Fl1nt | 11:31 | |
Fl1nt | test | 11:33 |
yoctozepto | ignaziocassano: is corosync running? | 11:54 |
ignaziocassano | yoctozepto: yes, it is. Now I am trying to deployng again but bevore I stoppeed and removed hacluster containers and volumes | 11:59 |
ignaziocassano | yoctozepto: nou hacluster container is running again but only on first controller. Is it right ? | 12:02 |
yoctozepto | ignaziocassano: should be on all the controllers | 12:03 |
ignaziocassano | yoctozepto: Yes, I presume it should | 12:04 |
yoctozepto | priteau: I think we avoided ignoring keys but let me check how we have done that now | 12:04 |
ignaziocassano | yoctozepto: Yes, I presume it should | 12:05 |
ignaziocassano | yoctozepto: I runed again deploy but with tags hacluster and now is running on all controllers | 12:06 |
ignaziocassano | yoctozepto: it is strange but now it is working fine | 12:08 |
ignaziocassano | yoctozepto: so I stopped and removed hacluster containers and columes. I runned deploy and then I runned deply again with -t hacluster :-) | 12:10 |
ignaziocassano | yoctozepto: now crm cluster status reports well | 12:11 |
yoctozepto | it should be enough to deploy it once; we test this in CI; the play always runs against all the controllers | 12:16 |
ignaziocassano | yoctozepto: probably I am impatient and I must wait hacluster containers appaer on the other controllers. Anycase the update left hacluster in error and I had to remove its container and volume and deploy again | 12:28 |
yoctozepto | ignaziocassano: they appear after the role gets applied, no need to wait :D | 12:28 |
yoctozepto | ignaziocassano: perhaps there is some race condition? I can't tell; it always succeeded for me so far | 12:29 |
kevko | priteau: hmm, I gave you +1 to UCA GPG key ..but instead of installing deb package, shouldn't be key just added to base_apt_keys dict ? | 12:32 |
*** amoralej is now known as amoralej|lunch | 12:34 | |
yoctozepto | kevko: I am glad you are doing thorough reviews :-) | 12:40 |
kevko | :D | 12:40 |
kevko | maybe you can check proxysql patches :D :D :D | 12:43 |
kevko | yoctozepto: but yeah, trying to do the best | 12:43 |
priteau | yoctozepto: It's another option. I thought since we have a package, it will stay up to date even if the key changes. | 12:45 |
yoctozepto | cores: there is a permanent failure in our multinode upgrade jobs in xena (master) due to cinder failing on OSM | 12:49 |
kevko | priteau: well, it was reworked because to be one key -> one repo .. if i am correct ..installing that package causes that key will be trusted globally | 12:50 |
yoctozepto | priteau: yeah, in kolla we go with trusted=yes for base repos so we prevented this issue for non-base repos but still can hit for base ones; pity | 12:51 |
yoctozepto | kevko, priteau: we need to fix the key anyhow; glad kayobe tests this and we catch this early | 12:51 |
priteau | happy to include the key directly if this is preferred? | 12:52 |
kevko | priteau: i think you can add similar block as debian has already for ubuntu also (debian using extrepo ..but it's only used to configure and then purged package), so similar block can be for ubuntu | 12:52 |
kevko | priteau: it's on line 277 master | 12:52 |
yoctozepto | priteau: I think your patch makes sense | 12:53 |
yoctozepto | just can be done better | 12:54 |
yoctozepto | commenting inline now | 12:54 |
yoctozepto | commented | 12:55 |
kevko | let me try something | 12:55 |
*** amoralej|lunch is now known as amoralej | 12:58 | |
opendevreview | Pierre Riteau proposed openstack/kolla master: Install UCA GPG key https://review.opendev.org/c/openstack/kolla/+/806560 | 13:10 |
parallax | I'm just browsing through package repositories configured in Kolla, curious on crmsh.repo | 13:32 |
parallax | Any chance this is still used, eg. by Masakari ? | 13:32 |
yoctozepto | parallax: no, crmsh is not; it was added at some point by the previous folk who was about to add pacemaker and family | 14:06 |
yoctozepto | but we do not use it in KA | 14:06 |
parallax | yoctozepto: drop it ? | 14:07 |
yoctozepto | parallax: I would accept such a patch :-) | 14:07 |
parallax | good | 14:08 |
kevko | who is developing elasticsearch-py ? elasticsearch ? | 14:24 |
Fl1nt | kevko seems so: https://elasticsearch-py.readthedocs.io/en/7.x/ | 14:26 |
yoctozepto | kevko: evil folks | 14:26 |
kevko | hmm, reviewing this https://review.opendev.org/c/openstack/kolla/+/805990 | 14:26 |
kevko | it looks like they stopped supporting oss version ? | 14:27 |
yoctozepto | kevko: yeah | 14:27 |
yoctozepto | that's why I called them evil | 14:27 |
yoctozepto | ;p | 14:27 |
yoctozepto | I forgot to +2 that, I had only a nit comment there | 14:27 |
* parallax can follow up on that to not waste yoctozepto's precious +2 | 14:29 | |
parallax | but I guess we're lacking cores on duty today :) | 14:30 |
kevko | hmm, but what does it mean ? will they fix it for OSS or it is their intention to disable it? | 14:33 |
kevko | i mean disable oss functionality .. | 14:34 |
yoctozepto | parallax: yeah, we truly are | 14:34 |
yoctozepto | kevko: no idea, amazon forked their own version, we might want to switch to that at some point | 14:35 |
parallax | it's similar to this: https://github.com/logstash-plugins/logstash-output-elasticsearch/pull/1005 - notice the thumbs down on this one from the community | 14:38 |
kevko | sadly :( | 14:41 |
kevko | they (oss responsible guys) should fork elasticsearch-py library and removing this sh*** from code :D | 14:43 |
kevko | btw, someone for review this one line ? :P https://review.opendev.org/c/openstack/kolla-ansible/+/775627 ? | 14:53 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: [WIP] Build overcloud host image directly with DIB https://review.opendev.org/c/openstack/kayobe/+/772609 | 15:14 |
*** amoralej is now known as amoralej|off | 15:40 | |
priteau | yoctozepto: I proposed a kayobe patch (just above) with Depends-On on my kolla UCA patch and it passes kayobe-seed-ubuntu-focal which was failing before | 15:48 |
opendevreview | Merged openstack/kolla master: haproxy: Use haproxy 2.2.x https://review.opendev.org/c/openstack/kolla/+/804513 | 17:52 |
dmsimard | FYI, ansiblefest agenda is out: https://www.ansible.com/ansiblefest -> https://events.ansiblefest.redhat.com/widget/redhat/ansible21/sessioncatalog, contributor summit registrations are also open: https://ansiblecs202109.eventbrite.com/ | 18:00 |
yoctozepto | priteau: ++ | 18:10 |
opendevreview | Merged openstack/kolla master: Install UCA GPG key https://review.opendev.org/c/openstack/kolla/+/806560 | 18:24 |
opendevreview | Merged openstack/kolla stable/ussuri: Unify curl options https://review.opendev.org/c/openstack/kolla/+/805644 | 18:24 |
opendevreview | Merged openstack/kolla stable/victoria: Unify curl options https://review.opendev.org/c/openstack/kolla/+/805643 | 18:25 |
opendevreview | Merged openstack/kolla master: docs: add feature freeze to release management https://review.opendev.org/c/openstack/kolla/+/806013 | 18:25 |
opendevreview | James Kirsch proposed openstack/kolla-ansible master: Support for keystone scoped authorization https://review.opendev.org/c/openstack/kolla-ansible/+/692179 | 19:19 |
opendevreview | Maksim Malchuk proposed openstack/kolla master: Unify curl use (customisation options) https://review.opendev.org/c/openstack/kolla/+/805640 | 20:15 |
opendevreview | Maksim Malchuk proposed openstack/kolla master: Unify curl use (customisation options) https://review.opendev.org/c/openstack/kolla/+/805640 | 20:25 |
opendevreview | James Kirsch proposed openstack/kolla-ansible master: Support for keystone scoped authorization https://review.opendev.org/c/openstack/kolla-ansible/+/692179 | 22:11 |
opendevreview | James Kirsch proposed openstack/kolla-ansible master: Support for keystone scoped authorization https://review.opendev.org/c/openstack/kolla-ansible/+/692179 | 23:39 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!