Saturday, 2019-11-23

openstackgerritPedro Henrique Pereira Martins proposed openstack/kolla-ansible master: Add suport to OpenID Connect Authentication flow  https://review.opendev.org/69543200:22
*** rphillips_ has joined #openstack-kolla00:40
*** rphillips_ is now known as rphillips00:40
rphillipsI have a train install, and I'm trying to add swift. The docker swift images are coming up but exiting with00:45
rphillips+ sudo -E kolla_set_configs00:45
rphillipssudo: no tty present and no askpass program specified00:45
*** igordc has quit IRC00:45
rphillipsanybody know what is up with it?00:45
rphillipsthe base install with nova is working fine.00:46
*** benj_ has quit IRC01:56
*** benj_ has joined #openstack-kolla01:58
*** mrunge_ has joined #openstack-kolla02:03
*** mrunge has quit IRC02:05
openstackgerritMerged openstack/kolla-ansible stable/train: CI: Ignore more file paths when considering ansible runs  https://review.opendev.org/69553502:38
openstackgerritDincer Celik proposed openstack/kolla master: Ensure lsb-release exists on debian/ubuntu  https://review.opendev.org/69563504:18
*** tonythomas has quit IRC04:44
*** msalo has joined #openstack-kolla04:51
*** msalo has quit IRC04:56
*** zhanglong has joined #openstack-kolla06:43
*** zhanglong has quit IRC07:14
*** zhanglong has joined #openstack-kolla07:16
*** msalo has joined #openstack-kolla08:11
*** msalo has quit IRC08:15
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: WIP: Enable repos only when needed  https://review.opendev.org/69546508:24
mnasiadkarphillips: please raise a bug - swift is not something we test in CI08:26
*** zhanglong has quit IRC08:49
*** andrein has quit IRC09:04
*** andrein has joined #openstack-kolla09:04
openstackgerritDincer Celik proposed openstack/kolla master: Ensure lsb-release exists on debian/ubuntu  https://review.opendev.org/69563509:15
yoctozeptomnasiadka: so you mean I should follow up with swift testing, probably worth to test ceph rgw as well (real functioning as replacement of swift)09:31
mnasiadkayoctozepto: don’t test what is deprecated :)09:32
yoctozeptomnasiadka: it is not, only internal is09:33
yoctozeptoyour integration must pass the tests. period. :-)09:33
mnasiadkayoctozepto: and I’m afraid swift role needs a really big ramp up - it’s not in the modern format :)09:33
yoctozeptoyeah, it looks bad09:33
mnasiadkayoctozepto: well, we might skip ceph-ansible - last week Sage told me to go ceph orchestrator way09:33
mnasiadkaceph-ansible might be dead-ish in May09:34
yoctozeptomnasiadka: do whatever is best nowadays09:34
mnasiadkaThat’s the plan09:34
yoctozeptomnasiadka: last time I looked orchestrator used ceph-ansible09:34
yoctozeptoso not sure what that means in fact09:35
mnasiadkayoctozepto: yeah, in Nautilus09:35
hrwhey folks09:35
mnasiadkayoctozepto: but in Octopus not really - they use direct ssh and run python code09:35
yoctozeptoah, not followed then09:35
yoctozeptoif it works fine...09:35
mnasiadkayoctozepto: so I prefer to do something that is easily upgradable...09:35
yoctozeptoyeah, exactly09:36
yoctozeptodo they use containers?09:36
yoctozeptodocker/podman?09:36
yoctozeptohi hrw09:36
hrwdoes someone know how to get to jinja2 template vars from python function registered as jinja2 method?09:36
*** egonzalez has quit IRC09:36
yoctozeptohrw: care to give some example? not sure if it is too early but not following09:37
hrwyoctozepto: https://review.opendev.org/695465 - enable_repos() method09:37
patchbotpatch 695465 - kolla - WIP: Enable repos only when needed - 5 patch sets09:37
yoctozeptohrw: geez, which part?09:37
hrwyoctozepto: now it gets 4 vars: repos + 3 template vars09:37
hrwyoctozepto: methods.py09:37
yoctozeptohrw: ack09:37
hrwI need 4th var now.09:38
*** egonzalez has joined #openstack-kolla09:38
hrwmay need another one09:38
yoctozeptoah, you mean you don't want to pass them around09:38
hrwyep09:38
yoctozeptoyou can use context09:38
hrwso access to template vars or kolla conf is needed09:38
hrwyoctozepto: tell me more09:38
yoctozeptojinja2 context decorator09:38
yoctozeptolook at my ipv6 change09:38
yoctozeptohttps://review.opendev.org/#/c/681573/70/kolla_ansible/kolla_address.py09:40
patchbotpatch 681573 - kolla-ansible - Implement IPv6 support in the control plane (MERGED) - 70 patch sets09:40
yoctozepto@contextfilter09:40
yoctozeptodef kolla_address(context09:40
yoctozeptoand the drama in function body09:40
yoctozeptohrw: ^09:40
hrwyoctozepto: thanks!09:41
yoctozeptohrw: yw!09:41
hrwso all templates vars goes from context?09:42
yoctozeptohrw: as much as you want them09:47
yoctozeptoif they are all shared, you can go all context09:47
yoctozeptofor the most variable keep it in the function09:47
yoctozeptoall context is probably rarely useful09:47
yoctozeptoI did not analyze your use case though09:47
hrwyoctozepto: I have a macro which enables repos. you do call from Dockerfile.j2: {{ macros.enable_extra_repos(['ceph', 'grafana']) }} and it takes care of distro/arch etc09:51
hrwyoctozepto: but I need rpm/deb, yum/dnf, debian/ubuntu, arch...09:51
yoctozeptoso it makes sense to contextify all except repo names09:51
hrwyoctozepto: sure, macro can give all vars but there are more and more...09:51
hrwyes09:51
yoctozeptojust what I mean with variability09:52
yoctozeptoall right, you are all set09:52
hrw@contextfunction09:54
hrwand my patch requires python 3.6+ :D09:55
yoctozeptowhat's the minimum to support in Ussuri? not 3.6? ;p09:56
hrwyep09:56
yoctozeptothen np, go ahead09:56
hrwcommands += f' --enable {repo_list[repo]}'09:56
hrwclean09:57
hrw+                commands += f'echo "{repo_list[repo]}" '09:57
hrw+                commands += f'>/etc/apt/sources.list.d/{repo}.list; '09:57
hrwand if you use Debian (or derived): "apt-mark hold python2" after uninstalling python2 package will mark it as not installable09:58
hrwso unless you install exactly python2 package then it will not get installed (and anything which depend on it)09:58
hrwhttps://review.opendev.org/#/c/695465/ goes nice on x86 :D09:59
patchbotpatch 695465 - kolla - WIP: Enable repos only when needed - 5 patch sets09:59
hrwon monday will merge @contextfunction stuff with !x86 stuff and then it should be ready for detailing10:00
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: WIP: Enable repos only when needed  https://review.opendev.org/69546510:43
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: WIP: Enable repos only when needed  https://review.opendev.org/69546511:23
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: WIP: Enable repos only when needed  https://review.opendev.org/69546511:55
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: test-requirements: bump pyflakes to get f-strings support  https://review.opendev.org/69579611:57
openstackgerritMarcin Juszkiewicz proposed openstack/kolla master: WIP: Enable repos only when needed  https://review.opendev.org/69546511:58
*** spiette has quit IRC14:43
*** mchlumsky has quit IRC14:45
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: CI/Debian: Push images after building  https://review.opendev.org/69424315:05
*** luksky has joined #openstack-kolla15:06
yoctozeptomnasiadka: google does not help me with this new ceph orchestrator - is there any worthy source of information on it? I wanted to know mostly about "why" ceph-ansible is passe15:46
mnasiadkayoctozepto: that is not said in any materials - I’ll ask the ceph-ansible guys on Monday and let you know16:03
yoctozeptomnasiadka: thanks :-)16:04
*** msalo has joined #openstack-kolla16:12
*** pcaruana has quit IRC16:31
*** luksky has quit IRC16:44
*** msalo has quit IRC16:50
*** msalo_ has joined #openstack-kolla16:50
openstackgerritMerged openstack/kolla-ansible stable/stein: Patch to fix RPC selection problem after upgrade  https://review.opendev.org/69504917:01
openstackgerritMerged openstack/kolla-ansible master: Fixes the compatibility issue at HAProxy role with Ansible 2.9  https://review.opendev.org/69405817:02
*** msalo_ has quit IRC17:06
*** msalo has joined #openstack-kolla17:06
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: CI: Cleanups, refactors and testing of Qinling  https://review.opendev.org/69519217:08
*** msalo has quit IRC17:10
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: CI: Cleanups, refactors and testing of Qinling  https://review.opendev.org/69519217:11
*** msalo has joined #openstack-kolla17:11
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: CI: Cleanups, refactors and testing of Qinling  https://review.opendev.org/69519217:17
*** spiette has joined #openstack-kolla17:21
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: CI: Cleanups, refactors and testing of Qinling  https://review.opendev.org/69519217:22
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: CI: Cleanups, refactors and testing of Qinling  https://review.opendev.org/69519217:27
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: CI: Cleanups, refactors and testing of Qinling  https://review.opendev.org/69519217:28
*** dr_feelgood has quit IRC17:56
*** msalo has quit IRC18:00
*** msalo has joined #openstack-kolla18:20
*** msalo has quit IRC18:25
*** numans has quit IRC20:17
*** mchlumsky has joined #openstack-kolla20:20
yoctozeptomgoddard, mnasiadka: tacker drama continued: https://bugs.launchpad.net/tacker/+bug/185371520:26
openstackLaunchpad bug 1853715 in kolla-ansible ussuri "[train] [kolla-ansible] [tacker] Incomplete config results in ERRORs from tacker-server" [High,Triaged] - Assigned to Radosław Piliszek (yoctozepto)20:26
yoctozeptothis time I went all into their source code20:27
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: CI: Cleanups, refactors and testing of Qinling  https://review.opendev.org/69519220:30
openstackgerritRadosław Piliszek proposed openstack/kolla-ansible master: [Train] Fix Tacker config for the local Glance store  https://review.opendev.org/69580420:48
*** goldyfruit_ has quit IRC20:50
*** msalo has joined #openstack-kolla21:48
*** goldyfruit_ has joined #openstack-kolla23:12
*** msalo has quit IRC23:14

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!