Thursday, 2021-07-29

opendevreviewDavid Moreau Simard proposed openstack/kolla-ansible master: DNM: Test ara 1.5.7rc3 with kolla-ansible  https://review.opendev.org/c/openstack/kolla-ansible/+/72090201:56
*** rpittau|afk is now known as rpittau07:10
opendevreviewGaël THEROND proposed openstack/kolla stable/train: Improve offline build scenario.  https://review.opendev.org/c/openstack/kolla/+/80235607:15
opendevreviewGaël THEROND proposed openstack/kolla stable/ussuri: Improve offline build scenario.  https://review.opendev.org/c/openstack/kolla/+/80235507:18
opendevreviewGaël THEROND proposed openstack/kolla stable/victoria: Improve offline build scenario.  https://review.opendev.org/c/openstack/kolla/+/80235707:20
opendevreviewGaël THEROND proposed openstack/kolla master: Fix various exporter tiny issues such as:  https://review.opendev.org/c/openstack/kolla/+/80049907:56
opendevreviewSkylar Tristan Kelty proposed openstack/kolla-ansible master: Update Manila deploy steps for Wallaby  https://review.opendev.org/c/openstack/kolla-ansible/+/80274307:58
kevkomorning :) 08:03
parallaxHi08:07
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: test-requirements: remove not needed entries  https://review.opendev.org/c/openstack/kolla/+/80276908:16
kevkohaerwu: did you check that reviews regarding users/groups ? 08:19
Fl1ntGood morning folks!08:28
haerwukevko: will do today08:36
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/wallaby: manila: add glance section in manila-share.conf  https://review.opendev.org/c/openstack/kolla-ansible/+/80278308:37
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/victoria: manila: add glance section in manila-share.conf  https://review.opendev.org/c/openstack/kolla-ansible/+/80278408:38
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/ussuri: manila: add glance section in manila-share.conf  https://review.opendev.org/c/openstack/kolla-ansible/+/80278508:38
opendevreviewWill Szumski proposed openstack/kayobe master: Use merge_configs and merge_yaml to generate custom config  https://review.opendev.org/c/openstack/kayobe/+/78274908:43
kevkohaerwu: thank you, because it will help me to apply our local patch for proxysql (i'm creating proxysql user with new UUID and new users in upstream will cause inconsistency ..so I have to bump local proxysql patch)08:46
kevko*bump UUID for proxysql user ..08:47
kevkohaerwu: which reminds me .. can u also check proxysql image review from me ? I think it is in good shape and we are using from stein .. and nothing prevents it from being merged to kolla git 08:48
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/ussuri: docs: Add information on tuning Ansible  https://review.opendev.org/c/openstack/kolla-ansible/+/80271208:52
opendevreviewPierre Riteau proposed openstack/kayobe stable/wallaby: Remove stale DIB_DISABLE_KERNEL_CLEANUP references  https://review.opendev.org/c/openstack/kayobe/+/80278608:52
opendevreviewPierre Riteau proposed openstack/kayobe stable/victoria: Remove stale DIB_DISABLE_KERNEL_CLEANUP references  https://review.opendev.org/c/openstack/kayobe/+/80278708:53
opendevreviewPierre Riteau proposed openstack/kayobe stable/ussuri: Remove stale DIB_DISABLE_KERNEL_CLEANUP references  https://review.opendev.org/c/openstack/kayobe/+/80284408:54
opendevreviewPierre Riteau proposed openstack/kayobe stable/wallaby: Use stream8 images with molecule  https://review.opendev.org/c/openstack/kayobe/+/80278808:55
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: sync requirements with openstack/requirements  https://review.opendev.org/c/openstack/kolla/+/80284508:56
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/victoria: Use ansible_facts to reference facts  https://review.opendev.org/c/openstack/kolla-ansible/+/80271108:56
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/ussuri: Use ansible_facts to reference facts  https://review.opendev.org/c/openstack/kolla-ansible/+/80271308:57
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/ussuri: Use ansible_facts to reference facts  https://review.opendev.org/c/openstack/kolla-ansible/+/80271308:57
opendevreviewMark Goddard proposed openstack/kolla-ansible stable/ussuri: Use ansible_facts to reference facts  https://review.opendev.org/c/openstack/kolla-ansible/+/80271309:03
haerwukevko: group/home set reviewed and commented09:09
haerwukevko: would be nice to drop [WIP] from them09:09
kevkohaerwu: going to process :) 09:19
haerwukevko: proxysql is funny set.09:21
haerwukevko: kolla one is zuul+1, k-a part is zuul-109:21
haerwumake it pass zuul please09:22
opendevreviewMichal Arbet proposed openstack/kolla master: Fix missing pacemaker-cli-utils in Debian hacluster images  https://review.opendev.org/c/openstack/kolla/+/79978209:39
kevkohaerwu: roger that 09:39
kevkohaerwu: why funny ? :D 09:40
haerwukevko: hard to tell09:43
haerwuhad such feeling09:44
kevko:/09:44
kevkohaerwu: i can give you access to kolla-ansible deployed stack and you can check it online09:44
kevkoclean stack for test09:44
yoctozeptohaerwu: reviewed and offered more drops, u interested?09:45
yoctozeptohttps://review.opendev.org/c/openstack/kolla/+/80276909:45
haerwunoticed09:46
haerwuyoctozepto: and prettytable is dependency of something else09:48
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: test-requirements: remove not needed entries  https://review.opendev.org/c/openstack/kolla/+/80276909:54
haerwuyoctozepto: review please09:54
opendevreviewMerged openstack/kolla-ansible master: [CI] Slim down Masakari job  https://review.opendev.org/c/openstack/kolla-ansible/+/80215509:57
opendevreviewMerged openstack/kolla-ansible stable/victoria: docs: Add information on tuning Ansible  https://review.opendev.org/c/openstack/kolla-ansible/+/80271009:57
opendevreviewMerged openstack/kolla-ansible stable/wallaby: Use ansible_facts to reference facts  https://review.opendev.org/c/openstack/kolla-ansible/+/80270410:05
opendevreviewSkylar Tristan Kelty proposed openstack/kolla-ansible master: Update Manila deploy steps for Wallaby  https://review.opendev.org/c/openstack/kolla-ansible/+/80274310:06
Fl1ntyoctozepto, hi, where did you found out the pip error in here: https://review.opendev.org/c/openstack/kolla/+/802356 ? I can't manage to understand how is zuul error files working ^^10:08
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: Upgrade pip and friends in source images  https://review.opendev.org/c/openstack/kolla/+/77064710:15
opendevreviewMark Goddard proposed openstack/kolla-ansible master: service-ks-register: ensure all user roles exist  https://review.opendev.org/c/openstack/kolla-ansible/+/77445110:20
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Support creating roles and assigning to service users  https://review.opendev.org/c/openstack/kolla-ansible/+/77445210:20
opendevreviewMichal Arbet proposed openstack/kolla master: Config: allow to set group for user  https://review.opendev.org/c/openstack/kolla/+/80266310:22
opendevreviewMichal Arbet proposed openstack/kolla master: Config: Set default home to /var/lib/{user}  https://review.opendev.org/c/openstack/kolla/+/80268510:22
opendevreviewMichal Arbet proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones  https://review.opendev.org/c/openstack/kolla/+/80267610:22
opendevreviewMichal Arbet proposed openstack/kolla master: Add some system users to fix user uuid inconsistencies  https://review.opendev.org/c/openstack/kolla/+/80267110:22
yoctozeptoFl1nt: I just clicked on the first centos job10:35
yoctozeptoit's a breakage in CI, you can try to fix it in another change if you feel up to the task10:35
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/wallaby: [CI] Slim down Masakari job  https://review.opendev.org/c/openstack/kolla-ansible/+/80287010:38
haerwuINFO:kolla.common.utils.neutron-base:.[91m  DEPRECATION: A future pip version will change local packages to be built in-place without first copying to a temporary directory. We recommend you use --use-feature=in-tree-build to test your packages with this new behavior before it becomes the default.10:41
haerwuINFO:kolla.common.utils.neutron-base:   pip 21.3 will remove support for this functionality. You can find discussion regarding this at https://github.com/pypa/pip/issues/7555.10:41
haerwuspeaking of deprecations ;D10:41
opendevreviewMerged openstack/kolla stable/victoria: Improve offline build scenario.  https://review.opendev.org/c/openstack/kolla/+/80235710:43
opendevreviewMerged openstack/kolla master: Add missing default policy files for debian-binary-horizon  https://review.opendev.org/c/openstack/kolla/+/79821210:47
haerwuanyway all our images build fine with --use-feature=in-tree-build used so we are save10:53
opendevreviewPierre Riteau proposed openstack/kayobe master: Document variables to configure connectivity check  https://review.opendev.org/c/openstack/kayobe/+/80286210:58
opendevreviewMichal Arbet proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones  https://review.opendev.org/c/openstack/kolla/+/80267610:59
opendevreviewMichal Arbet proposed openstack/kolla master: Add some system users to fix user uuid inconsistencies  https://review.opendev.org/c/openstack/kolla/+/80267110:59
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support multiple inventories passed to kolla-ansible  https://review.opendev.org/c/openstack/kolla-ansible/+/80286311:04
opendevreviewWill Szumski proposed openstack/kayobe master: Support merging kolla group_vars when using multiple inventories  https://review.opendev.org/c/openstack/kayobe/+/80286411:08
opendevreviewMerged openstack/kolla stable/ussuri: Improve offline build scenario.  https://review.opendev.org/c/openstack/kolla/+/80235511:19
opendevreviewMerged openstack/kolla stable/wallaby: Add boto3 as s3 dependency for Glance container  https://review.opendev.org/c/openstack/kolla/+/80275511:19
opendevreviewWill Szumski proposed openstack/kayobe master: Kayobe environment dependencies  https://review.opendev.org/c/openstack/kayobe/+/80286511:23
opendevreviewMerged openstack/kolla stable/victoria: Add boto3 as s3 dependency for Glance container  https://review.opendev.org/c/openstack/kolla/+/80275611:33
opendevreviewMerged openstack/kolla stable/ussuri: Add boto3 as s3 dependency for Glance container  https://review.opendev.org/c/openstack/kolla/+/80275711:33
opendevreviewMerged openstack/kolla master: Remove tempest image  https://review.opendev.org/c/openstack/kolla/+/79459511:33
opendevreviewVerification of a change to openstack/kolla master failed: monasca-thresh: Allow topology check and removal in storm  https://review.opendev.org/c/openstack/kolla/+/79275111:33
opendevreviewIlya Popov proposed openstack/kolla-ansible master: Extra var ironic_enable_keystone_integration added.  https://review.opendev.org/c/openstack/kolla-ansible/+/79366411:40
haerwuyoctozepto, mgoddard, kevko: https://review.opendev.org/c/openstack/kolla/+/784923 - apt keys done right11:40
haerwuI would love to get rid of that cleanup from queue11:40
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: Update code to Jinja 3.0 style  https://review.opendev.org/c/openstack/kolla/+/79826311:41
kevkohaerwu: interisting .. apt-transport-https no more needed ? 11:42
kevkoor is it installed somewhere else ? 11:42
haerwukevko: it was transitional in buster already iirc11:42
kevkoah ok11:42
haerwu" This is a dummy transitional package - https support has been moved into the apt package in 1.5. It can be safely removed. "11:43
kevkolooks good to me 11:46
haerwu+! :D11:48
*** Guest2351 is now known as kmasterson12:01
Fl1ntJust got an issue with one of my controller node having horizon acting weirdly because it didn't perform a manage.py compress BUT that doesn't make sens as controller 01 and 03 don't get this issue as all horizon containers are USSURI 10.2.0 and others are working.12:08
Fl1ntI did the compress manually on the container and kind of fixed the issue.12:09
Fl1ntbut still don't get why one is not working where two others do ^^12:13
Fl1ntfor now I've just stopped controller02 horizon and it works again :D12:13
priteauWe've occasionally seen issues like this, destroying and redeploying the container can resolve it12:14
priteauYou might want to restart memcached too if you don't mind loosing sessions12:14
Fl1ntyeah I don't mind it's seasons holyday time :D12:17
Fl1ntthx I'll do :D12:17
sydhi all, is this also a support channel?12:18
haerwusyd: ask and wait?12:22
sydhad a question with regards to the stable binary builds (specifically horizon & karbor), the current Victoria builds are erroring with  line 121: ENABLE_KARBOR: unbound variable12:23
sydbut I'm also under the impression karbor is deprecated in kolla?12:23
sydI don't have karbor enabled in globals12:25
priteauAre you sure you're not mixing wallaby in there?12:25
haerwusyd: 1. which images you are using?12:27
sydarg, it looks like I may be actually12:27
Fl1ntsyd, Karbor is deprecated in Openstack itself and even no longer used: https://opendev.org/openstack/karbor/src/branch/master/README.rst12:27
sydkeystone-manage --version returns 18.0.0 so I presume I'm still getting wallaby somehow12:28
Fl1ntTeam, is there a reason we're deprecating tempest and rally containers? Lack of support?12:28
haerwuFl1nt: tempest is not something you deploy but use for testing12:28
haerwuFl1nt: so kind of out scope for us while giving several issues with their approach to branches12:29
haerwuFl1nt: iirc tempest is 'we are master only kind of'12:29
sydis openstack_release: "victoria" in globals enough or have I missed something else required for a non current deployment?12:31
priteauCheck which version of kolla-ansible is installed12:37
sydohhh, I see what I've done. thank you!12:39
haerwumixed victoria images with wallaby k-a?12:41
sydI believe so yeah12:43
haerwuhappens12:43
Fl1nthaerwu, Sorry but this isn't our case ^^ We do deploy it and use it regularly as a service of the platform, because we use it to analyse stability and maturity of the platform even in between release deployment.12:47
sydthanks priteau, I was under the impression I was using a symlinked version, you've saved me tearing out my hair :)12:48
haerwuFl1nt: understood13:01
Fl1ntso this is something that we're kind of worried about :D But honestly not a blocking point.13:02
haerwuFl1nt: there are probably several other ways of getting tempest running13:03
Fl1ntwell, yes, we can spawn it within the platform or on a satellite HW.13:04
haerwuand with >222 images supported I am surprised that we still keep adding new ones instead of dropping less popular ones.13:04
haerwuprobably half of images are 'sure, we have but do not expect support' levelbut as they do not generate build/ci issues we ignore13:05
Fl1ntWell that's Openstack ^^ You've got a bunch of projects and a bunch of operators that deploy a bunch of heterogeneous deployments ^^13:09
Fl1ntso as a major lifecycle management software you need to support it widely :D13:13
Fl1ntwell, not you, but we :D13:13
opendevreviewGaël THEROND proposed openstack/kolla-ansible stable/ussuri: Add missing elasticsearch cloudkitty storage and prometheus collector backend support.  https://review.opendev.org/c/openstack/kolla-ansible/+/80287913:32
opendevreviewGaël THEROND proposed openstack/kolla-ansible stable/victoria: Add missing elasticsearch cloudkitty storage and prometheus collector backend support.  https://review.opendev.org/c/openstack/kolla-ansible/+/80288013:34
opendevreviewGaël THEROND proposed openstack/kolla-ansible stable/wallaby: Add missing elasticsearch cloudkitty storage and prometheus collector backend support.  https://review.opendev.org/c/openstack/kolla-ansible/+/80288113:34
opendevreviewGaël THEROND proposed openstack/kolla-ansible stable/wallaby: Add missing elasticsearch cloudkitty storage and prometheus collector backend support.  https://review.opendev.org/c/openstack/kolla-ansible/+/80288113:35
opendevreviewMerged openstack/kolla-ansible stable/victoria: Use ansible_facts to reference facts  https://review.opendev.org/c/openstack/kolla-ansible/+/80271114:05
opendevreviewMerged openstack/kolla-ansible stable/ussuri: docs: Add information on tuning Ansible  https://review.opendev.org/c/openstack/kolla-ansible/+/80271214:05
opendevreviewMerged openstack/kolla-ansible stable/ussuri: Use ansible_facts to reference facts  https://review.opendev.org/c/openstack/kolla-ansible/+/80271314:05
opendevreviewMarcin Juszkiewicz proposed openstack/kolla master: Update code to Jinja 3.0 style  https://review.opendev.org/c/openstack/kolla/+/79826314:19
haerwurebased again14:19
kevkoFl1nt: I have same issue with horizon ! 14:20
Fl1ntusing Ussuri ?14:21
kevkoFl1nt: i've met this while testing wallaby 14:21
kevkoFl1nt: what image did you use ? 14:21
Fl1ntI'm on ussuri since yesterday based on binary 10.2.0 CentOS 814:22
kevkoFl1nt: i was wondering if this can be issue ... https://github.com/django-compressor/django-compressor/issues/169 , https://stackoverflow.com/questions/52840617/offlinegenerationerror-key-not-in-manifest-json14:23
kevkoFl1nt: i had this in log You have offline compression enabled but key "b68ba4d2116295c56138e439f1b03a03" is missing from offline manifest. You may need to run "python manage.py compress".14:24
Fl1ntooooh, that's exactly that kevko, nice catch!14:24
Fl1ntyes, that's the same issue that I had.14:25
kevkodiff between first and second container was different and compressed one less thing ..]14:27
kevkobut I can't find a log ...don't know where I stored it :( 14:27
haerwuFl1nt: 'binary 10.2.0' - you built them or fetch from external repo?14:30
Fl1ntI build it14:30
haerwuok, just wanted to be sure as external images using numbered tags are iirc older than named tag ones14:31
haerwulocally I use MMDD-BUILDNUMBER (0729-1) ones14:32
kevkoFl1nt: root@controller1:/home/ubuntu# diff -u /tmp/horizon_0 /tmp/horizon_1 14:33
kevko--- /tmp/horizon_02021-07-29 14:32:42.515598554 +000014:33
kevko+++ /tmp/horizon_12021-07-29 14:32:18.631219731 +000014:33
kevko@@ -1,2 +1,2 @@14:33
kevko Compressing... done14:33
kevko-Compressed 7 block(s) from 12 template(s) for 0 context(s).14:33
kevko+Compressed 8 block(s) from 12 template(s) for 0 context(s).14:33
Fl1ntwoot ?14:34
Fl1ntWe build everything, we're the offline guys that bother everyone :p14:34
kevkoFl1nt: same container ..same settings ... different LOG ..compressed 7 vs 8 blocks 14:34
Fl1ntis that your runtime logs or build logs ?14:36
kevkokolla_extend_start 14:36
kevkothat stuff with md5sums and collect compress ...docker logs horizon 14:36
Fl1ntoh ok, container logs, so runtime restart :D14:37
Fl1ntI've just done the priteau suggestion and removed then redeployed the container, now it works fine ^^14:39
Fl1ntGuessing its somehow related to django compressing AngularJS bundles.14:39
kevkoFl1nt: yeah, I also fixed by remove container and redeploy it 14:42
kevkoFl1nt: but it's only workaround :( 14:43
kevkoanother reason to fix docker healthcheck for horizon in wallaby and upper ...14:43
Fl1ntexact :D14:43
kevkoi have patch prepared :) 14:43
kevkobut i have to test it with all tls layers enabled 14:43
Fl1ntOn my own it's not really a big deal but yeah reliability of the platform is always appreciated :D14:45
Fl1ntOnce I'll get back all previously pushed patchs cherry picked to ussuri I'll upload the last one a.k.a SAML2 SSO :D14:47
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support multiple inventories passed to kolla-ansible  https://review.opendev.org/c/openstack/kolla-ansible/+/80286315:02
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support multiple inventories  https://review.opendev.org/c/openstack/kolla-ansible/+/80286315:22
opendevreviewWill Szumski proposed openstack/kolla-ansible master: Support multiple inventories  https://review.opendev.org/c/openstack/kolla-ansible/+/80286315:25
opendevreviewMerged openstack/kolla-ansible master: Revert "cephadm: Set auth_allow_insecure_global_id_reclaim to true"  https://review.opendev.org/c/openstack/kolla-ansible/+/79545715:27
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Revert "cephadm: Set auth_allow_insecure_global_id_reclaim to true"  https://review.opendev.org/c/openstack/kolla-ansible/+/80288915:45
opendevreviewMerged openstack/kolla-ansible master: cyborg: add missing become for api-paste.ini  https://review.opendev.org/c/openstack/kolla-ansible/+/80243015:54
*** rpittau is now known as rpittau|afk16:03
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/wallaby: cyborg: add missing become for api-paste.ini  https://review.opendev.org/c/openstack/kolla-ansible/+/80293116:11
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/victoria: cyborg: add missing become for api-paste.ini  https://review.opendev.org/c/openstack/kolla-ansible/+/80293216:11
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: cyborg: add missing become for api-paste.ini  https://review.opendev.org/c/openstack/kolla-ansible/+/80293316:12
opendevreviewMaksim Malchuk proposed openstack/kayobe master: Add support multi-segments for ip-allocation  https://review.opendev.org/c/openstack/kayobe/+/79984517:09
opendevreviewRadosław Piliszek proposed openstack/kolla master: Remove patrole from version-check  https://review.opendev.org/c/openstack/kolla/+/80292618:43
opendevreviewDavid Moreau Simard proposed openstack/kolla-ansible master: DNM: Test ara 1.5.7rc4 with kolla-ansible  https://review.opendev.org/c/openstack/kolla-ansible/+/72090221:41

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