Friday, 2022-03-25

opendevreviewwangxiyuan proposed openstack/ansible-collection-kolla master: Correct docker package on openEuler  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/83032002:12
opendevreviewwangxiyuan proposed openstack/kolla-ansible master: Add openEuler Distro support  https://review.opendev.org/c/openstack/kolla-ansible/+/83011503:32
opendevreviewwangxiyuan proposed openstack/kolla-ansible master: Add openEuler Distro support  https://review.opendev.org/c/openstack/kolla-ansible/+/83011503:52
opendevreviewwangxiyuan proposed openstack/kolla-ansible master: Add openEuler Distro support  https://review.opendev.org/c/openstack/kolla-ansible/+/83011506:34
opendevreviewJan Horstmann proposed openstack/kolla-ansible master: Change grafana provisioning.yaml indentation  https://review.opendev.org/c/openstack/kolla-ansible/+/83482608:43
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/wallaby: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kolla-ansible/+/83518208:54
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/victoria: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kolla-ansible/+/83518308:55
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/ussuri: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kolla-ansible/+/83518408:55
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/train: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kolla-ansible/+/83518508:56
opendevreviewPierre Riteau proposed openstack/kayobe stable/victoria: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kayobe/+/83518608:59
opendevreviewPierre Riteau proposed openstack/kayobe stable/wallaby: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kayobe/+/83518709:01
opendevreviewPierre Riteau proposed openstack/kayobe stable/wallaby: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kayobe/+/83518709:08
*** mgoddard- is now known as mgoddard09:11
opendevreviewMaksim Malchuk proposed openstack/kayobe stable/xena: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kayobe/+/83519109:19
opendevreviewMerged openstack/kolla-ansible stable/xena: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/kolla-ansible/+/83512609:28
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/train: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kolla-ansible/+/83518509:39
fricklermgoddard: mnasiadka: yoctozepto: ^^ not sure how far we want to backport the jinja fix vs. doing the pinning, wdyt?09:40
jingvarwhat is a reason to place files like grafana.yml in etc/kayobe/ ? As I  understand it is a part of Kolla09:41
yoctozeptofrickler: me and priteau have established that before xena we need to pin before the u-c was for <309:41
yoctozepto(and it's not a heart emoticon)09:41
yoctozeptos/before/because09:41
priteaujingvar: it's a historical reason. There was no support for loading dashboards at the time.09:43
priteauYou can use the native kolla-ansible support now09:44
jingvarWhich files can I remove from root kayobe 09:45
priteauWhat do you mean?09:46
jingvarmonasca.yml09:46
jingvarnova.yml09:46
priteauYou can remove all the .yml files if you are not using them, the defaults live inside the kayobe source code09:47
jingvarwhat do openstack component  files in kayobe-config/etc/kayobe?09:48
jingvarI don't see nova.yml in the Kayobe doc09:50
priteauLook into the file, there are comments. nova.yml is only used for the nova-ironic integration09:51
sorin-mihai_i'm trying to connect kolla to ceph, i seem to be having issues because of the ceph.conf and keyring files. can anyone show me some working examples?09:51
jingvarAs I understad all changes around openstack services can be done via kayobe-config/etc/kayobe/kolla/09:51
opendevreviewMerged openstack/kolla-ansible master: re-add rabbitmq config for clustering interface  https://review.opendev.org/c/openstack/kolla-ansible/+/75857609:52
priteaujingvar: The service config itself, yes. But for example the setting in nova.yml is used to generate the ansible inventory which controls container placement.09:52
jingvarprobably I've missed it in the Doc?09:53
priteauNot everything is documented09:54
priteauThese are niche settings09:54
jingvaryep, I'am writing myown documentation, and trying to describe where what is09:56
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/xena: re-add rabbitmq config for clustering interface  https://review.opendev.org/c/openstack/kolla-ansible/+/83513409:58
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/wallaby: re-add rabbitmq config for clustering interface  https://review.opendev.org/c/openstack/kolla-ansible/+/83513509:58
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/victoria: re-add rabbitmq config for clustering interface  https://review.opendev.org/c/openstack/kolla-ansible/+/83513609:58
hrwfrickler: there is no support behind train10:10
frickleryoctozepto: pinning jinja to the heart, I like that ;)10:11
yoctozeptofrickler: :-)10:11
opendevreviewRafael Weingartner proposed openstack/kolla-ansible master: Customize the authentication error timeout page in modOIDC  https://review.opendev.org/c/openstack/kolla-ansible/+/83280610:26
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Add Neutron ssh key  https://review.opendev.org/c/openstack/kolla-ansible/+/83520010:39
priteauMy patches is failing requirements-check :/10:40
fricklerI wonder if kolla-ansible should really be tied to openstack requirements, does it make sense? cause we explicitly install everything openstack-y in its dedicated env10:41
priteaukayobe installs kolla-ansible using openstack upper constraints10:42
mnasiadkafrickler: got a minute for https://review.opendev.org/c/openstack/kolla-ansible/+/802301 ?10:43
priteauAnd it is saving us of this breakage (at least of the kolla-ansible part)10:43
sorin-mihai_can someone please point me to the CI where the tests for hci with ceph are configured? i feel like there's something missing in ceph.conf or keyrings, so i need to double check the format of those files10:43
priteaufrickler: we may need to support both old and new versions10:46
fricklermnasiadka: done10:46
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: haproxy: support single external frontend  https://review.opendev.org/c/openstack/kolla-ansible/+/82339510:52
opendevreviewMerged openstack/kolla-ansible master: Change grafana provisioning.yaml indentation  https://review.opendev.org/c/openstack/kolla-ansible/+/83482611:36
opendevreviewMerged openstack/kolla-ansible stable/xena: designate: fix external backend deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/83505011:48
opendevreviewMerged openstack/kolla-ansible stable/wallaby: designate: fix external backend deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/83505111:48
opendevreviewMerged openstack/kolla-ansible stable/victoria: designate: fix external backend deployment  https://review.opendev.org/c/openstack/kolla-ansible/+/83505211:51
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Add Neutron ssh key  https://review.opendev.org/c/openstack/kolla-ansible/+/83520012:03
opendevreviewMerged openstack/kolla-ansible master: designate: Allow to disable notifications  https://review.opendev.org/c/openstack/kolla-ansible/+/80230112:06
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: haproxy: support single external frontend  https://review.opendev.org/c/openstack/kolla-ansible/+/82339512:14
opendevreviewVerification of a change to openstack/ansible-collection-kolla master failed: baremetal: refactor docker_sdk installation to a separate role  https://review.opendev.org/c/openstack/ansible-collection-kolla/+/82101612:49
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/train: re-add rabbitmq config for clustering interface  https://review.opendev.org/c/openstack/kolla-ansible/+/83514112:57
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/ussuri: re-add rabbitmq config for clustering interface  https://review.opendev.org/c/openstack/kolla-ansible/+/83514212:59
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: re-add rabbitmq config for clustering interface  https://review.opendev.org/c/openstack/kolla-ansible/+/83514212:59
yoctozeptofrickler: I agree, we should be more independent13:01
yoctozeptothat said, we still need a similar mechanism13:01
yoctozeptomaybe use pipenv13:01
opendevreviewVerification of a change to openstack/kayobe stable/wallaby failed: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kayobe/+/83518713:11
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Add Neutron ssh key  https://review.opendev.org/c/openstack/kolla-ansible/+/83520013:36
opendevreviewJuan Pablo Suazo proposed openstack/kolla-ansible master: Adds FluentD configuration for OpenvSwitch log matching  https://review.opendev.org/c/openstack/kolla-ansible/+/83455513:36
opendevreviewJuan Pablo Suazo proposed openstack/kolla-ansible master: Adds FluentD configuration for OpenvSwitch log matching  https://review.opendev.org/c/openstack/kolla-ansible/+/83455514:01
SvenKieskeI know this is beating a dead horse, but: why do train changes hardfail on openstack-tox-py27 ?? afaik train is a python3 only release?14:39
SvenKiesketalking about https://review.opendev.org/c/openstack/kolla-ansible/+/835141/ specifically; which is expected to fail on python 2, because iirc the used python module just exists since python 3.x14:40
yoctozeptoSvenKieske: nope, train still supported py2.7 - on centos 714:47
yoctozeptoSvenKieske: just make the string in tests a unicode string and it should be happy I guess14:49
yoctozepto(and migrate to ussuri to help py2.7 in openstack die)14:49
yoctozepto;-)14:49
SvenKieskeyoctozepto: yeah, migrationtests are already completed, just one more complete redeploy of train, than upgrade to ussuri and hopefully victoria after that..15:16
SvenKieskewill fix that error though15:16
opendevreviewSven Kieske proposed openstack/kolla-ansible stable/train: re-add rabbitmq config for clustering interface  https://review.opendev.org/c/openstack/kolla-ansible/+/83514115:28
jingvarhow to override docker_storage_driver for computes, for example. 15:43
jingvarvai inventory/group_vars/compute/  does'nt work15:48
mnasiadkaas long as you're not setting that in globals - it should15:49
jingvarI did'nt set it globals, it was rendered as default - layer215:52
SvenKieskedepends on your release, in older releases ripgrep tells me it's defined in "ansible/roles/baremetal/defaults/main.yml" as empty. I don't know from the top of my head which variable get's loaded first.. role defaults before or after group vars?15:53
jingvarxena15:53
jingvarinventory/group_vars/computes/docker_storage_driver:docker_storage_driver: fuse-overlayfs15:54
jingvarwill try via ansible/group_vars/computes/15:54
SvenKieskeyeah, it's set to "" in the above mentioned role in xena release15:55
SvenKieskelet's look at: https://docs.ansible.com/ansible/2.9/user_guide/playbooks_variables.html#understanding-variable-precedence15:55
SvenKieskeseems fine, inventory group vars have higher precedence than role defaults15:56
jingvarI understand, but why other vars works as expected15:57
SvenKieskehonestly: I don't know. I wish I could already use xena :) what is your endgoal btw? I never interacted with the docker_storage_driver until now15:58
jingvarI have a host group where I have to use fuse-overlayfs15:59
jingvarI have separated provision controllers and the group, via running kayobe overcloud host configure -l compute_ucs -kl computes -e docker_storage_driver=fuse-overlayfs 16:01
jingvarit is ugly, I want to properly configure kayobe to avoid this16:02
SvenKieskewe're also not using kayobe, just plain kolla-ansible with our own overrides and some thousands lines of gitlab CI automation..so really no clue, sorry, but some folks here are pretty knowledgeable about kayobe I guess :)16:02
SvenKieskemaybe the culprit is in the kayobe codebase then, though? I just grepped through kolla-ansible, maybe something in kayobe interferes with variable settings? just a wild guess though.16:04
jingvarwe were loved in idea to use one place and tool for deployement16:04
jingvarthere ara some not clear dependencies16:05
priteaujingvar: If I remember correctly, per-group or per-host docker storage config is not supported because it is a kolla globals.yml16:23
jingvaryes, but it works around ansible16:24
jingvarwhat is a reason to do that strict behaviour16:25
SvenKieskeI guess just nobody got around to implement it in the playbooks/roles on a per group or host basis, patches welcome I guess? :/ :)16:27
SvenKieskefwiw that's how I ended up here in IRC :D scratch your own itch. the positive side is, that it's ansible (most of the time) and mostly easy to fix, if you can spare the time.16:28
jingvaras usual we need a patch  yesterday :)16:30
SvenKieskethen better be fast :D have a nice weekend @all, I'm out for now16:31
opendevreviewPierre Riteau proposed openstack/kayobe stable/xena: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/kayobe/+/83527916:47
opendevreviewMerged openstack/kayobe stable/wallaby: Pin Jinja2<3.1.0 to avoid contextfilter removal  https://review.opendev.org/c/openstack/kayobe/+/83518717:13
opendevreviewMaksim Malchuk proposed openstack/kayobe stable/xena: Ubuntu: add support for Apt configuration  https://review.opendev.org/c/openstack/kayobe/+/83492917:38
opendevreviewDavid Moreau Simard proposed openstack/kolla-ansible master: DNM: Test ara 1.5.8rc1 with kolla-ansible  https://review.opendev.org/c/openstack/kolla-ansible/+/72090219:02
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/kayobe/+/83509319:08
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/kayobe/+/83509319:32
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/kayobe/+/83509319:43
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/kayobe/+/83509320:05
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/kayobe/+/83509320:15
opendevreviewPierre Riteau proposed openstack/kayobe master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/kayobe/+/83509320:44
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/xena: Fix prometheus without prometheus-libvirt-exporter group  https://review.opendev.org/c/openstack/kolla-ansible/+/83532121:21

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!