Thursday, 2020-03-12

clarkb(thinking out loud more though I don't think it matters if the pyyaml used to parse the test matrix is python3 though or if ansible runs under python3)00:00
clarkbso we may still be able to python3 those bits00:00
ianwyeah ... all our hosts *have* python300:00
openstackgerritIan Wienand proposed openstack/devstack-gate master: Convert testmatrix run to venv  https://review.opendev.org/71259000:05
fungipython3 venv can be used to install python3 virtualenv which can then be used to create a python2 env00:07
fungithis is how i create python2.7 envs00:07
fungipython3 -m venv foo;foo/bin/pip install virtualenv;foo/bin/virtualenv -p python2.7 bar00:08
fungithough you could also given the right circumstances shorten that to `pip install --user virtualenv;virtualenv -p python2.7 bar`00:09
fungi(python3 -m pip ... as needed)00:10
fungiultimately i feel like jumping through the venv module is still cleaner, assuming it's installed (which we've decided it will be)00:10
clarkbfungi: ya that shorter version won't work in ourproposed new state00:11
fungii figured00:11
clarkb(no pip)00:11
fungiright, basically i use venv to have a working pip to get a working virtualenv to make a working python2 env00:12
fungiand the intermediate foo venv can just be deleted afterward00:13
*** prometheanfire has joined #openstack-infra00:13
*** tetsuro has joined #openstack-infra00:13
*** prometheanfire has quit IRC00:15
ianwit's like the python version of alice in wonderland00:17
ianwi hope you do this on a container in a vm behind dvr with about 15 layers of tunnels and bridges00:19
fungi'twas brillig and the slithy toves did gyre and gimble in the wabe00:20
fungithough i guess jabberwocky was in through the looking glass00:21
funginot wonderland00:21
fungihow doth the little crocodile improve his shining tail?00:22
fungithat's more like it00:22
fungi(ftr i prefer looking glass, of the two)00:23
fungiif i taught math theory, i'd make that required reading, along with flatland00:23
*** ijw has joined #openstack-infra00:26
*** ijw has quit IRC00:30
*** nicolasbock has quit IRC00:30
*** ijw has joined #openstack-infra00:30
*** nicolasbock has joined #openstack-infra00:30
*** nicolasbock has quit IRC00:43
*** openstackstatus has joined #openstack-infra00:44
*** ChanServ sets mode: +v openstackstatus00:44
*** prometheanfire has joined #openstack-infra01:06
*** hongbin has joined #openstack-infra01:12
openstackgerritTrinh Nguyen proposed opendev/irc-meetings master: Remove Searchlight meeting  https://review.opendev.org/71259501:15
*** dangtrinhnt has joined #openstack-infra01:16
*** diablo_rojo_phon has joined #openstack-infra01:22
*** dangtrinhnt has quit IRC01:31
*** dangtrinhnt has joined #openstack-infra01:32
*** yamamoto has joined #openstack-infra01:35
*** yamamoto has quit IRC01:36
*** yamamoto has joined #openstack-infra01:36
*** gyee has quit IRC01:51
*** dangtrinhnt has quit IRC01:51
*** sgw has joined #openstack-infra02:11
sgwHi Infra team, what's the process to get someone added to allow them to do merge commits?  I would like to add Shuicheng Lin to the list of people allowed to do merge commit in StarlingX repos02:12
clarkbsgw: to click the approval button in gerrit?02:13
*** diablo_rojo has quit IRC02:13
sgwNo to do git review pushes02:14
*** xinranwang has joined #openstack-infra02:14
sgwgit review -t centos8-rebase remote: Processing changes: refs: 1, done To ssh://slin14@review.opendev.org:29418/starlingx/integ.git ! [remote rejected] HEAD -> refs/publish/f/centos8/centos8-rebase (you are not allowed to upload merges) error: failed to push some refs to 'ssh://slin14@review.opendev.org:29418/starlingx/integ.git'02:14
clarkboh I see actual merge commits02:14
clarkbone sec02:14
*** rlandy|bbl is now known as rlandy02:15
sgwyup, I have permissions, but want to have shuicheng do it in the future02:15
clarkbsgw: currently it looks like starlingx repos have acls that allow the starglinx-release team to push merge commits02:16
clarkbhttps://review.opendev.org/#/admin/projects/starlingx/helm-charts,access random example I pulled up02:16
clarkbyou can add Shuicheng Lin to starglinx-release if appropriate. If not appropriate the acls could be updated to allow another different group to push the merge commit02:17
sgwOk, thanks, I think adding him there mades the most sense02:17
*** jamesmcarthur has joined #openstack-infra02:18
sgwOh **&*^, I am still having that dang Web interface failure I had before, I need to go back to the CLI again02:19
clarkbhrm that is really weird. I wonder if it is some mismatch between web account and ssh account02:20
sgwI can do reviews, I jst can't add people to groups02:20
clarkbsgw: when its not 7pm I'd be happy to look into that more02:20
*** yamamoto has quit IRC02:20
sgwAgreed, just dealing with a small backlog issue.02:20
sgwwe can talk again tomorrow02:21
sgwthanks for your help now02:21
fungisgw: if you go to https://review.opendev.org/#/settings/ what account id number does it show you?02:22
fungialso does the username it shows on that page correspond to the one you're using with ssh?02:23
sgwyup, they are the same sgw-starlingx02:24
sgwis the '-' a problem?02:24
fungishouldn't be02:24
fungiokay, so account 2860702:25
sgwyup, that's what seems to be the acct id02:26
sgwI was able to do ssh add just now.  This can wait for tomorrow if we want to try again, we talked weeks about about this.02:26
fungii've double-checked, the preferred_email associated with that account is unique too (there is a bit of a bug with gerrit's openid account autocreation where it can get confused if multiple accounts get created for the same email)02:27
fungiso yeah, not the usual suspects at least02:27
fungiwhat error do you get when trying to add people to groups using the webui?02:27
sgwCode Review - Error02:28
sgwThe page you requested was not found, or you do not have permission to view this page.02:28
fungihuh02:29
fungialso i've double-checked the Lin Shuicheng account and it looks fine too02:29
sgwYup went through this the last time also, when I found that the SSH CLI worked, we just stopped looking at the WebUI failure.02:31
fungithat comes up after you select a user and try to hit the "add" button i guess?02:31
sgwcorrect02:31
fungihave you tried a different browser? grasping at straws obviously02:31
fungitried turning it off and on again? ;)02:31
fungithis is definitely one i haven't heard of before your case02:32
sgwDifferent OSes, Browsers, on VPN/off VPN02:32
fungistrange02:32
sgwLogged out and in again, Incognetio mode, ...02:32
sgwI am just cursed!02:33
fungii'm double-checking gerrit's logs, but they have a tendency to be kinda useless02:33
fungiopaque java backtraces spanning hundreds of lines02:34
fungiand full of benign "errors" looking very similar02:34
*** yamamoto has joined #openstack-infra02:35
sgwthanks for looking again.02:35
fungiyeah, sorry i'm coming up short again too02:37
sgwForunately the CLI works, I just have to search how to use it which is easy enough!02:39
fungiit's at least thoroughly documented https://review.opendev.org/Documentation/cmd-index.html02:43
fungialso has some fairly useful context help if you pass --help on incomplete commands02:43
fungiand is flexible about how you refer to specific users (display name, email address, account id, username...)02:44
fungiyou can also refer to groups by name or id number02:44
LiangFangAJaeger: https://review.opendev.org/#/c/711878/ Brian has reviewed and gave +1, thanks02:50
*** jamesmcarthur has quit IRC02:56
*** jamesmcarthur has joined #openstack-infra02:58
*** hongbin has quit IRC03:03
*** jamesmcarthur has quit IRC03:03
*** jamesmcarthur has joined #openstack-infra03:06
*** rkukura has quit IRC03:10
*** rkukura has joined #openstack-infra03:12
*** yamamoto has quit IRC03:19
*** yamamoto has joined #openstack-infra03:24
*** KeithMnemonic1 has joined #openstack-infra03:28
*** KeithMnemonic has quit IRC03:31
*** ricolin_ has joined #openstack-infra03:31
*** ricolin_ has quit IRC03:33
*** ricolin_ has joined #openstack-infra03:34
*** jamesmcarthur has quit IRC03:34
*** jamesmcarthur has joined #openstack-infra03:35
*** psachin has joined #openstack-infra03:36
*** jamesmcarthur has quit IRC03:40
*** tetsuro has quit IRC03:41
*** yamamoto has quit IRC03:44
*** rlandy has quit IRC03:44
*** tetsuro has joined #openstack-infra03:45
*** yamamoto has joined #openstack-infra03:45
*** ykarel|away is now known as ykarel03:48
*** jamesmcarthur has joined #openstack-infra03:56
*** dave-mccowan has quit IRC03:58
*** apetrich has quit IRC04:00
*** bdodd has joined #openstack-infra04:00
*** KeithMnemonic1 has quit IRC04:06
*** tobiash has quit IRC04:16
*** udesale has joined #openstack-infra04:17
*** tobiash has joined #openstack-infra04:17
*** jamesmcarthur has quit IRC04:23
*** jamesmcarthur has joined #openstack-infra04:27
*** jamesmcarthur has quit IRC04:32
*** jamesmcarthur has joined #openstack-infra04:33
*** factor has joined #openstack-infra04:34
*** tobiash has quit IRC04:43
*** tobiash has joined #openstack-infra04:46
*** Martouf has quit IRC05:14
*** raukadah is now known as chandankumar05:16
*** dSrinivas has joined #openstack-infra05:18
dSrinivasHi All, i am getting this error http://paste.openstack.org/show/790571/  from March 3 in nova-compute logs in our CI FC HBA's are Online and Connectivity exists between vm and Storage05:18
*** tetsuro_ has joined #openstack-infra05:23
*** zxiiro has quit IRC05:25
*** tetsuro has quit IRC05:27
*** jamesmcarthur has quit IRC05:30
*** lbragstad__ has joined #openstack-infra05:33
*** lbragstad_ has quit IRC05:35
*** evrardjp has quit IRC05:35
*** evrardjp has joined #openstack-infra05:36
*** strigazi has quit IRC05:49
*** strigazi has joined #openstack-infra05:51
ianwclarkb: few things in flight @ https://review.opendev.org/#/q/topic:cleanup-test-image-python ... i feel like that might be enough that devstack doesn't blow up instantly on a plain node.  todo is tox tests05:59
*** xinranwang has quit IRC06:04
*** prometheanfire has quit IRC06:26
*** prometheanfire has joined #openstack-infra06:28
*** prometheanfire has quit IRC06:36
*** prometheanfire has joined #openstack-infra06:37
AJaegerdSrinivas: better ask the nova team on #openstack-nova06:39
*** yamamoto has quit IRC06:40
*** yamamoto has joined #openstack-infra06:42
*** yamamoto has quit IRC06:50
*** lmiccini has joined #openstack-infra06:53
*** yamamoto has joined #openstack-infra06:55
*** LiangFang has quit IRC07:02
*** strigazi has quit IRC07:03
*** smcginnis has quit IRC07:03
*** dpawlik has joined #openstack-infra07:04
*** smcginnis has joined #openstack-infra07:04
*** Liang__ has joined #openstack-infra07:04
dSrinivasAJaeger: Thank you07:08
*** prometheanfire has quit IRC07:10
*** roman_g has quit IRC07:11
*** prometheanfire has joined #openstack-infra07:16
*** prometheanfire has quit IRC07:17
*** rpittau|afk is now known as rpittau07:17
*** prometheanfire has joined #openstack-infra07:20
*** pgaxatte has joined #openstack-infra07:24
*** tetsuro has joined #openstack-infra07:25
AJaegerconfig-core, please review https://review.opendev.org/712550 to remove a duplicate role.07:27
*** tetsuro_ has quit IRC07:29
openstackgerritAndreas Jaeger proposed openstack/openstack-zuul-jobs master: Use a fake zuul_return and an .ansible-lint file  https://review.opendev.org/71254607:29
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: Use a fake zuul_return and an .ansible-lint file  https://review.opendev.org/71254707:30
AJaegerconfig-core, those two as well (if they pass now), please - allow linting without Zuul installed ^07:34
*** tetsuro_ has joined #openstack-infra07:35
*** tetsuro has quit IRC07:37
*** matt_kosut has quit IRC07:46
*** matt_kosut has joined #openstack-infra07:46
*** matt_kosut has quit IRC07:51
AJaegermordred: 712547 needs some more work to make linting happy, 712546 passes now07:56
*** hashar has joined #openstack-infra07:58
*** slaweq has joined #openstack-infra08:02
*** apetrich has joined #openstack-infra08:03
*** amoralej|off is now known as amoralej08:06
*** tesseract has joined #openstack-infra08:08
*** matt_kosut has joined #openstack-infra08:10
*** jcapitao has joined #openstack-infra08:15
*** dchen has quit IRC08:17
*** tosky has joined #openstack-infra08:18
*** ramishra has quit IRC08:37
*** ramishra has joined #openstack-infra08:39
*** tetsuro_ has quit IRC08:44
*** jpena|off is now known as jpena08:45
*** tetsuro has joined #openstack-infra08:47
*** pkopec has joined #openstack-infra08:51
*** rpittau is now known as rpittau|bbl08:52
*** tetsuro_ has joined #openstack-infra08:53
*** tetsuro has quit IRC08:57
*** ralonsoh has joined #openstack-infra08:58
*** xek has joined #openstack-infra08:58
*** matt_kos_ has joined #openstack-infra09:07
*** matt_kosut has quit IRC09:08
*** matt_kos_ has quit IRC09:12
*** dtantsur|afk is now known as dtantsur09:13
*** matt_kosut has joined #openstack-infra09:13
*** tetsuro has joined #openstack-infra09:14
*** tetsuro_ has quit IRC09:17
*** ykarel is now known as ykarel|lunch09:19
*** tetsuro_ has joined #openstack-infra09:20
*** tetsuro has quit IRC09:23
*** priteau has joined #openstack-infra09:25
*** sshnaidm|afk is now known as sshnaidm09:39
*** derekh has joined #openstack-infra09:41
openstackgerritJens Harbott (frickler) proposed opendev/system-config master: Add goaccess reporting to more static.o.o sites  https://review.opendev.org/71257609:43
openstackgerritFabien Boucher proposed zuul/zuul master: Declare support for Python3.8  https://review.opendev.org/71248009:45
openstackgerritMerged openstack/devstack-gate master: Fix virtualenv detection  https://review.opendev.org/71258709:49
*** ociuhandu has joined #openstack-infra09:59
openstackgerritMerged opendev/irc-meetings master: Remove Trove team meeting  https://review.opendev.org/71245109:59
openstackgerritMerged opendev/irc-meetings master: Remove Searchlight meeting  https://review.opendev.org/71259510:05
openstackgerritMerged opendev/irc-meetings master: Remove Doc team meeting  https://review.opendev.org/71245310:05
lifelessfungi: :)10:09
lifelessmordred: :)10:09
openstackgerritMerged opendev/system-config master: Remove old 404 checker job  https://review.opendev.org/71257510:16
*** matt_kosut has quit IRC10:18
*** matt_kosut has joined #openstack-infra10:18
*** ykarel|lunch is now known as ykarel10:19
*** ociuhandu has quit IRC10:25
*** ociuhandu has joined #openstack-infra10:28
*** dSrinivas has quit IRC10:38
*** tetsuro_ has quit IRC10:39
openstackgerritMerged opendev/system-config master: Add goaccess reporting to more static.o.o sites  https://review.opendev.org/71257610:49
*** ociuhandu has quit IRC10:49
dpawlikHi. Is possible to add new Fedora releases to mirror.mtl01 ?10:50
*** ociuhandu has joined #openstack-infra10:50
*** ociuhandu has quit IRC10:53
dpawlikcc fungi ^^10:53
*** nicolasbock has joined #openstack-infra10:56
*** ociuhandu has joined #openstack-infra10:58
*** matt_kosut has quit IRC10:59
*** jamesdenton has joined #openstack-infra11:01
*** jcapitao is now known as jcapitao_afk11:02
*** ociuhandu has quit IRC11:03
*** derekh has quit IRC11:10
openstackgerritMerged opendev/irc-meetings master: Remove Upgrades SIG meeting  https://review.opendev.org/71245211:14
*** nicolasbock has quit IRC11:15
openstackgerritMerged opendev/irc-meetings master: Remove Networking VPP team meeting  https://review.opendev.org/71244811:15
openstackgerritMerged opendev/irc-meetings master: Remove Group-Based policy team meeting  https://review.opendev.org/71245711:15
*** sshnaidm has quit IRC11:18
*** sshnaidm has joined #openstack-infra11:19
*** derekh has joined #openstack-infra11:20
*** ociuhandu has joined #openstack-infra11:20
openstackgerritFabien Boucher proposed zuul/zuul master: Declare support for Python3.8  https://review.opendev.org/71248011:20
*** yamamoto has quit IRC11:22
*** sshnaidm has quit IRC11:22
*** yamamoto has joined #openstack-infra11:24
*** ociuhandu has quit IRC11:27
*** yamamoto has quit IRC11:27
*** ociuhandu has joined #openstack-infra11:27
*** tetsuro has joined #openstack-infra11:28
openstackgerritThierry Carrez proposed opendev/irc-meetings master: Consolidate meeting channels  https://review.opendev.org/71265611:29
*** sshnaidm has joined #openstack-infra11:30
*** sshnaidm has quit IRC11:30
openstackgerritThierry Carrez proposed opendev/irc-meetings master: Consolidate meeting channels  https://review.opendev.org/71265611:33
*** matt_kosut has joined #openstack-infra11:35
*** nicolasbock has joined #openstack-infra11:39
*** matt_kosut has quit IRC11:40
*** nicolasbock has quit IRC11:42
*** nicolasbock has joined #openstack-infra11:43
*** matt_kosut has joined #openstack-infra11:46
*** jpena is now known as jpena|lunch11:49
*** nicolasbock has quit IRC11:51
*** eharney has joined #openstack-infra11:52
*** sshnaidm has joined #openstack-infra11:52
*** sshnaidm has quit IRC11:53
*** sshnaidm has joined #openstack-infra11:54
*** nicolasbock has joined #openstack-infra11:54
*** ociuhandu has quit IRC11:54
*** ociuhandu has joined #openstack-infra11:55
*** ociuhandu has quit IRC11:55
*** ociuhandu has joined #openstack-infra11:55
*** nicolasbock has quit IRC11:56
*** nicolasbock has joined #openstack-infra11:57
*** yamamoto has joined #openstack-infra11:59
*** rlandy has joined #openstack-infra12:01
*** TViernion has quit IRC12:04
*** yamamoto has quit IRC12:05
*** yamamoto has joined #openstack-infra12:14
*** udesale_ has joined #openstack-infra12:14
*** yamamoto has quit IRC12:15
openstackgerritAlbin Vass proposed zuul/nodepool master: Enable setting label and instance name separately  https://review.opendev.org/71266612:15
*** TViernion has joined #openstack-infra12:16
*** udesale has quit IRC12:17
*** lastmikoi has quit IRC12:20
*** gfidente has joined #openstack-infra12:21
*** tetsuro has quit IRC12:22
*** rh-jelabarre has joined #openstack-infra12:24
*** lastmikoi has joined #openstack-infra12:25
*** rcernin has quit IRC12:25
*** tkajinam has quit IRC12:27
*** jpena|lunch is now known as jpena12:32
*** Lucas_Gray has joined #openstack-infra12:43
*** ahosam has joined #openstack-infra12:44
*** dciabrin_ is now known as dciabrin12:45
*** jcapitao_afk is now known as jcapitao12:46
*** psachin has quit IRC12:47
*** Lucas_Gray has quit IRC12:49
*** ociuhandu has quit IRC12:50
*** ociuhandu has joined #openstack-infra12:51
*** yamamoto has joined #openstack-infra12:52
*** Lucas_Gray has joined #openstack-infra12:53
*** eharney has quit IRC12:53
*** nightmare_unreal has joined #openstack-infra12:56
nightmare_unrealhow can one create a cell on devstack1 and attach it to devstack2 ?12:57
*** ociuhandu has quit IRC12:57
*** yamamoto has quit IRC12:57
*** ociuhandu has joined #openstack-infra13:00
*** rpittau|bbl is now known as rpittau13:02
*** Lucas_Gray has quit IRC13:10
*** psachin has joined #openstack-infra13:12
*** ianychoi has quit IRC13:13
*** amoralej is now known as amoralej|lunch13:15
*** ociuhandu has quit IRC13:15
*** ociuhandu has joined #openstack-infra13:16
*** nightmare_unreal has quit IRC13:19
*** ociuhandu has quit IRC13:21
*** ociuhandu has joined #openstack-infra13:22
*** yamamoto has joined #openstack-infra13:24
*** derekh has quit IRC13:29
*** ociuhandu has quit IRC13:30
mgoddardHi. I am seeing in the centos-7 nodepool image that creating a virtualenv now defaults to using python 3. Previously it was python 2. This is breaking some jobs - kayobe & tenks13:30
mgoddardAre there any recent changes that could have caused this?13:30
*** ijw has quit IRC13:31
*** ociuhandu has joined #openstack-infra13:32
openstackgerritAlbin Vass proposed zuul/nodepool master: Enable setting label and instance name separately  https://review.opendev.org/71266613:32
openstackgerritAlbin Vass proposed zuul/nodepool master: Enable setting label and instance name separately  https://review.opendev.org/71266613:33
fungidpawlik: newer than fedora 30 you mean?13:33
dpawlikfungi, can be fedora 31 :)13:34
dpawlik32 is not released yet, so maybe after a while I will come back :)13:34
fungijust seeing we have 29 and 30 mirrored at the moment13:35
*** dtantsur is now known as dtantsur|brb13:36
*** eharney has joined #openstack-infra13:38
dpawlikfungi, yes. It is possible to add new or someone else is doing that?13:38
fungidpawlik: https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/mirror-update/files/fedora-mirror-update#L4913:38
fungialso need to update line 74 of that file similarly13:39
*** yamamoto has quit IRC13:39
dpawlikfungi, thank you !13:40
fungilooks like we have plenty of available quota for the fedora mirror volume so that shouldn't pose an issue13:40
fungimgoddard: i think since all our platforms have python3 available now, there was a recent change to start using it to install virtualenv consistently across all platforms. there's work underway to stop preinstalling it at all so that it gets installed at job runtime, so that projects have a bit more control over how it gets there, but i think that's still being tested in a separate node type to work13:42
fungiout the kinks13:42
cloudnullmgoddard it looks like https://opendev.org/opendev/base-jobs/src/branch/master/playbooks/base/pre.yaml#L42-L49 is the issue we're seeing on those c7 nodes, and I think all we need is to have python{2,3}-devel installed on the base image. is there a good place to amend the package list to ensure that is done?13:43
cloudnull-cc fungi13:43
fungii don't recall if that was changed more recently on centos-7 than on ubuntu-xenial, but i recall having some discussions about it back around november13:43
fungimgoddard: cloudnull: have a link to a failing build?13:44
cloudnullhttps://zuul.opendev.org/t/openstack/build/bf343e1ad1684bfb902fb0703085e92213:44
fungicurious to see, if the job includes a capture of the installed package list, what packages end up installed13:44
*** amoralej|lunch is now known as amoralej13:46
openstackgerritDaniel Pawlik proposed opendev/system-config master: Added new Fedora release to mirroring  https://review.opendev.org/71268013:46
yoctozeptofungi, cloudnull, mgoddard: moving here from kolla13:47
yoctozeptoso images suddenly have python3 with pip313:47
yoctozeptoand the base job does us bad favor installing virtualenv favoring pip3 if available13:48
fungiso, better cross-platform support there would be to invoke `virtualenv -p python2.7 ...`13:49
*** dklyle has quit IRC13:49
*** dklyle has joined #openstack-infra13:49
yoctozeptofungi: hmm, mgoddard tried that13:49
yoctozeptohttps://2ae0518c3adaf13d29a6-33cd363c3f72485dda255154bdda0fc8.ssl.cf5.rackcdn.com/712673/1/check/kayobe-overcloud-centos/cdc6674/primary/ansible/overcloud-deploy13:50
fungithough i haven't been following the python cleanup work closely so i don't know when that changed for centos-7 (i think it changed for ubuntu-xenial a while back)13:50
yoctozepto    "msg": "Error while linking: [Errno 2] No such file or directory",13:50
yoctozepto    "path": "/home/zuul/kolla-venv/lib/python2.7/site-packages/selinux"13:50
yoctozeptoargh13:50
yoctozeptomgoddard: ^13:50
yoctozeptofungi: could you help us trace where/how centos image changed?13:50
*** ijw has joined #openstack-infra13:51
fungii'll see if anything obvious landed in diskimage-builder or project-config recently related to the python cleanup work13:51
yoctozeptofungi: thanks13:52
fungimy first guess would be https://review.opendev.org/70751313:53
yoctozeptofungi: yeah, desc sounds like it13:54
yoctozepto(thought not the subject)13:54
yoctozeptothough*13:54
fungior possibly https://review.opendev.org/71222513:55
yoctozeptofungi: yeah, all in the gerrit topic together13:56
*** ijw has quit IRC13:56
yoctozeptofungi: this way we started getting python3+pip313:56
fungimore broadly this is implementing bits of https://docs.opendev.org/opendev/infra-specs/latest/specs/cleanup-test-node-python.html13:56
yoctozeptofungi: hmm, then base job assumptions changed considerably13:57
yoctozeptowe can't be replacing virtualenv with virtualenv-for-py313:57
yoctozeptoit's actually not straightforward thinking13:57
fungiwell, we're not trying to replace virtualenv with virtualenv-for-py313:58
fungiwe're trying to get away from preinstalling virtualenv and pip at all13:58
fungiand then letting them get installed flexibly at job runtime13:59
yoctozeptofungi: well, there must be pip3 installed in the image if base job detects it and does replace-virtualenv dance13:59
fungiat the moment i think the base job is supposed to be handling this, but it's possible the logic used to install virtualenv on centos-7 isn't correct13:59
yoctozeptofungi: https://opendev.org/opendev/base-jobs/src/branch/master/playbooks/base/pre.yaml#L43-L4913:59
yoctozeptofungi: it's part of dirty hack13:59
*** jistr is now known as jistr|mtg13:59
fungiand yes, i wonder if it's because centos-7 incorporates pip3 at this point14:00
yoctozeptofungi: well, it *is*14:00
yoctozeptofungi: do you think we still need that pin?14:00
yoctozeptofungi: if not, the situation should clarify if we remove it14:01
yoctozeptofungi: not necessarily improve though :-)14:01
yoctozeptofungi: at least we won't be doing something possibly dumb14:02
*** derekh has joined #openstack-infra14:03
fungigit blame says it was added by https://review.opendev.org/71085114:03
*** yamamoto has joined #openstack-infra14:03
fungiin service of working around https://review.opendev.org/71085114:04
fungier https://gitlab.com/python-devs/importlib_resources/issues/8314:04
fungilooks like the fix for that merged a week ago14:05
fungiseeing if there's been a release since14:05
fungiimportlib-resources 1.2.0 should have it now14:06
yoctozeptofungi: yeah, proposing WIP14:06
fungioh, but was also updated by https://review.opendev.org/710967 to work around https://github.com/pypa/virtualenv/issues/167014:08
AJaegerdpawlik: to use Fedora 31, you also need nodepool images. ianw is working on this, this needs some larger changes (new nodepool host)14:08
openstackgerritRadosÅ‚aw Piliszek proposed opendev/base-jobs master: WIP: Stop pinning virtualenv/importlib  https://review.opendev.org/71268914:08
yoctozeptofungi: and I can make my change depend on it and it should pick it up, right?14:09
dpawlikAJaeger, ack. Thanks for info14:10
fricklerclarkb: triggered a periodic run with 712576 merged14:10
yoctozeptodoing https://review.opendev.org/71264414:11
*** ociuhandu has quit IRC14:11
fungiif i'm reading that virtualenv issue correctly, the cascade of regressions which came after it should all be settled out as of 20.0.814:11
yoctozeptofungi: thanks, that makes it likely to work14:12
fungiyoctozepto: unfortunately changes to opendev/base-jobs can not be tested speculatively, for safety reasons14:12
yoctozeptofungi: ah14:12
frickleryoctozepto: no, you cannot test that via depends-on. you would need to patch the base-test job, get that merged and then have a job run on that14:12
yoctozeptofrickler: :-(14:12
fungiwe have a comment in the file explaining how we can make the alteration to the base-test job first and then reparent an affected job on that14:12
* yoctozepto sad like never before14:12
yoctozeptofungi: where?14:13
*** eharney has quit IRC14:13
mgoddardI'm going to try installing virtualenv via pip2 as a workaround14:14
yoctozeptomgoddard: +1, for kayobe14:15
fungiyoctozepto: oh, sorry, in jobs.yaml here: https://opendev.org/opendev/base-jobs/src/branch/master/zuul.d/jobs.yaml#L5-L2314:15
mgoddardcorrect, and tenks14:15
yoctozeptomgoddard: best do in pre, it should be right14:15
yoctozeptocloudnull: ^ use mgoddard's idea for workaround in tripleo14:16
fricklerclarkb: seems this works fine so far. two small issues: a) it would be great if we could somehow include the name of the site within the report, currently it is hidden at the end of the long log url14:17
yoctozeptofungi: well, base-test does not have this workaround14:18
yoctozeptofungi: so I can skip to step 4 :D14:18
fricklerb) the daily curve seems to indicate a dip for the first and last day, since we only process partial data there. maybe we should consider doing log rotation more 0:00z-ish and then don't include the current log file in the report14:18
fungiyoctozepto: frickler: yes, i expect we can just propose reverts of 710851 and 710967 if a run reparented on base-test works14:22
*** ijw has joined #openstack-infra14:22
*** auristor has quit IRC14:23
mgoddardyoctozepto, fungi: another data point: this was working yesterday: http://zuul.openstack.org/builds?job_name=tenks-deploy-teardown-centos714:24
yoctozeptomgoddard, fungi: yeah, it matches the time when centos images rebuilt with better python3 coverage14:26
*** ijw has quit IRC14:27
fungimgoddard: looks like that failed while installing libvirt-python under python3.6, so if it was previously using 2.7 that might explain it14:28
mgoddardright14:28
fungii need to step away for a few minutes and finish my morning routine, but will be back asap14:29
*** pkopec has quit IRC14:29
*** yamamoto has quit IRC14:29
fricklerclarkb: also, there seems to be a very long sequence of "waiting for logger" msgs in the job log, is there a way to avoid those?14:30
*** dave-mccowan has joined #openstack-infra14:31
frickleryoctozepto: fungi: you'd have to run the base-test based jobs against a number of specific job+stable branch combinations, iirc one issue was only seen with stable/rocky, the other on newer stable branches14:33
clarkbfrickler: I'll look into annotating the report with the sitename. The jobs have the waiting on logger message because we dont start a zuul console log daemon on static.opendev.org, I think this might actually be desireable in this case, but am open to changing that14:36
clarkb(there is a role we can run to start it)14:37
yoctozeptofrickler: it would break centos/train for its python2, then again iirc ubuntu/rocky has had py3.5 so you might be right there14:37
yoctozeptofrickler: you think testing it with devstack/rocky would make sense?14:38
*** auristor has joined #openstack-infra14:38
yoctozeptofrickler: argh, devstack sets parent to multinode from zuul-jobs, bum14:40
*** ricolin_ has quit IRC14:41
*** ricolin_ has joined #openstack-infra14:41
*** ociuhandu has joined #openstack-infra14:42
yoctozeptowell, at most it will break multinode things but should pass some things14:43
yoctozeptoI hope14:43
*** gyee has joined #openstack-infra14:43
clarkbthinking out loud here, can we simply shim in a pip = pip -p python2 redirect on the jobs for centos714:44
yoctozeptoclarkb: the base job does pip314:45
yoctozeptolike in *pip3*14:45
fricklerclarkb: I'm just watching things on static.o.o, seems goaccess uses files in /tmp and not in-memory as the man page claims. taking >30mins now on docs.o.o14:45
yoctozeptothe pip3 :D14:45
fricklerclarkb: also the various reports seem blocked on each other, likely due to the /tmp files having non-unique names14:46
clarkbfrickler: oh hrm, it definitely said it uses memory by default14:46
clarkbfrickler: ok so we may need to sequence stuff or figure out how to force using memory14:46
clarkbyoctozepto: well its fine to use pip3 if thats ehat you want. I thought the issue is jobs using `pip` getting pip3?14:47
clarkbyoctozepto: an easy fix for that is to make `pip` pip214:47
*** ociuhandu has quit IRC14:47
*** ricolin_ has quit IRC14:48
yoctozeptoclarkb: no, the issue is that base job is rewriting virtualenv to mean virtualenv for py314:48
yoctozeptoclarkb: when the assumption is otherwise14:48
yoctozeptoclarkb: dirty hack bit us14:48
yoctozepto(like they always do)14:48
clarkbyoctozepto: can you link to that?14:50
clarkbfrickler: ok we can set --keep-db-files explicitly false and then set --db-path to unique path per site14:51
clarkbthen it should delete files on exit and avoid collisions (I dont understand why memory is not used)14:51
fricklerclarkb: the other idea would be to keep the files and look into setting up the iterated logfile processing properly14:52
yoctozeptoclarkb: sure, it's the part being reverted here https://review.opendev.org/71268914:52
*** psachin has quit IRC14:52
fricklerclarkb: that might allow us to report over a longer range and also avoid reprocessing logs each day, if that turns up to take significant time14:53
*** ijw has joined #openstack-infra14:53
clarkbfrickler: yes, the tricky thing with that I found is if you process a log line twice it gets recorded as multiple requests14:53
fricklerclarkb: yeah, so we'd just feed it the *.1 file once each day14:54
clarkbwe'd have to be very careful about input data if we do that and I was hoping to start simple and improve frlm there14:54
clarkbfrickler: ya14:54
fricklerclarkb: could even trigger that by logrotate instead of a periodic job14:54
clarkbfrickler: oh thats an interesting idea, then zuul could just collect it14:55
clarkbyoctozepto: ah ok I've been trying to revert that all week :)14:55
yoctozeptoclarkb: late by just one day! :D14:55
yoctozeptoclarkb: ianw's cleanup re glean hit us14:56
yoctozeptoas fungi helped us to establish14:56
clarkbyoctozepto: https://review.opendev.org/#/c/712107/14:56
EmilienMmordred: hey, where can I find some infos about the container build/push things you're doing?14:56
yoctozeptoclarkb: hah, I missed that14:56
*** ociuhandu has joined #openstack-infra14:57
AJaegerclarkb: for sequencing: do we need a semaphore?14:58
mordredEmilienM: hi there! one sec, lemme get you a link and some pointers14:58
clarkbAJaeger: yes, but uf we do the --db-path we dont need to sequence unless wprried about other resource consumption there14:58
AJaegerclarkb: ok, understood14:58
*** ijw has quit IRC14:58
clarkbAJaeger: frickler I think first step is set those goaccess args and maybe add a sempahore then look into only processing the .1 log file each day14:59
*** Lucas_Gray has joined #openstack-infra15:00
mordredEmilienM: https://zuul-ci.org/docs/zuul-jobs/docker-jobs.html is the general documentation. you can see an example of it in use with some base jobs here: https://opendev.org/opendev/system-config/src/branch/master/.zuul.yaml#L24-L49 ... and then scroll down a bit and you can see jobs building specific container images with their interdependencies expressed15:00
fricklerclarkb: did you ever run a test against docs.o.o logs? the job is at 45mins now, at least it only seems to use one out of 8 cpus15:01
clarkbfrickler: I did not. It was primarily tested against zuul and also bad input (to check behavior when given bad input)15:01
*** ociuhandu has quit IRC15:02
*** yamamoto has joined #openstack-infra15:02
mordredEmilienM: the tl;dr is that we have job constructs that can set up buildset registries so that images can be built as-if other images have been published - and we can pull images built in gate and promote those images to be published to a registry in the promote pipeline instead of needing to rebuild after gate for publication15:02
mordredEmilienM: the buildset registry is also shared between jobs in the buildset, so you can build images in one job and then test them with other jobs15:03
clarkbfrickler: if you think we should revert the broader application and then sort out these problems that is fine by me15:03
clarkbbut still trying to catch up on all the things this morning15:03
EmilienMmordred: excellent. I'll look at all of it15:03
mordredEmilienM: there's a LOT there :) feel free to follow up of course - or if you want to schedule a time to sync up more synchronously just let me know15:05
corvusEmilienM, mordred: i'm happy to help out too15:05
mordredEmilienM: fwiw - the registry support groks support for multi-registry (like quay.io or whatnot) - and we also have support for configuring k8s and openshift to use the buildset registry15:05
*** yamamoto has quit IRC15:07
fricklerclarkb: I'm thinking I'll give it another 30 mins and if it doesn't finish until then, to kill it and rerun manually with in-memory. that should still leave enough room until the next automatic periodic trigger15:07
EmilienMcorvus, mordred: ok, nice. There is a lot to digest, I'll take a look and come back to you15:08
clarkbinfra-root 712107 would be good to land to fix yoctozepto's problem15:08
yoctozeptoconfirmed - fixing and no pin no longer needed from testing so far15:09
yoctozepto"no pin needed any longer"*15:09
clarkbya I did a bit of testing removing the pins myslef as long as our images are up to date (they were when I pushed that change) I thibk we are safe to proceed15:10
*** pkopec has joined #openstack-infra15:10
yoctozeptoclarkb: yeah, they changed a bit with that glean cleanup so was probably worth testing again15:11
*** Lucas_Gray has quit IRC15:14
*** ijw has joined #openstack-infra15:16
fricklerclarkb: oh, the jobs already timed-out long ago, but still left the goaccess processes running. so maybe we should indeed revert the added sites and I can do some more testing then tomorrow15:16
yoctozeptofrickler: commented, not worth waiting15:16
*** Lucas_Gray has joined #openstack-infra15:16
yoctozeptofrickler: devstack happily replaced virtualenv by itself15:17
yoctozeptootoh, I wonder how virtualenv 20.0.10 went in there? ;o15:18
*** hashar has quit IRC15:18
clarkbfrickler: revert sounds good15:18
*** Goneri has joined #openstack-infra15:18
clarkbfrickler: then maybe start with setting paths to see if they timed out due to internal conflicts15:18
clarkband go from there15:18
fricklerdirk: cmurphy: these SSL errors seem to fail devstack runs since a couple of days, does that look familiar to you? https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8ac/712634/1/check/devstack-platform-opensuse-15/8ac29df/job-output.txt15:26
openstackgerritJens Harbott (frickler) proposed opendev/system-config master: Revert "Add goaccess reporting to more static.o.o sites"  https://review.opendev.org/71271215:27
fricklerclarkb: ^^15:28
clarkbfrickler: +2 we can give it a minute or two for others to review but should be safe to single core approve that one15:29
openstackgerritMerged opendev/base-jobs master: Cleanup workaround for importlib-resources bad version  https://review.opendev.org/71210715:29
AJaegerfrickler, clarkb, for goaccess: Want to keep the jobs and just remove from periodic pipeline instead?15:29
AJaegerYour decision what's easier moving forward...15:30
cmurphyfrickler: not familiar to me, is that only on opensuse?15:30
clarkbAJaeger: I dont think we can trigger them without the pipeline config so probably doesnt help much to keep the jobs15:30
*** Lucas_Gray has quit IRC15:30
cmurphywonder why it is trying to download from https://pypi.org/simple/nose/ and not the mirror15:30
*** Lucas_Gray has joined #openstack-infra15:32
fricklercmurphy: that may be part of the issue, yes. I haven't seen this elsewhere, was just looking why the opensuse jobs seem to be failing persistently15:32
*** jistr|mtg is now known as jistr15:33
*** ykarel is now known as ykarel|away15:34
openstackgerritMonty Taylor proposed zuul/zuul-jobs master: Support buildkit backend when building docker images  https://review.opendev.org/71271615:35
openstackgerritDaniel Pawlik proposed zuul/zuul-jobs master: Add phoronix-test-suite job  https://review.opendev.org/67908215:39
openstackgerritMerged opendev/irc-meetings master: Remove Glance Bug Squad meeting  https://review.opendev.org/71245515:40
openstackgerritMonty Taylor proposed zuul/zuul master: Use buildkit to mount instead of copy /output  https://review.opendev.org/71271715:41
AJaegermordred: want to add a fake zuul_console to https://review.opendev.org/#/c/712547/ as well?15:44
fricklerclarkb: looking at the goaccess manpage again, the "configuration" section seems to list compile-time options, not run-time. so likely ubuntu compiles it to use /tmp, we'd have to build our own pkg to use in-memory. the jobs did finish successfully though after about an hour, so maybe as a first step we can just increase the job timeout to 2h and see what happens over the weekend15:44
clarkbfrickler: aha15:44
clarkbfrickler: increasing the timeout wfm15:45
AJaegermordred, clarkb, frickler, do we need the test anywhere for mirror-info, see frickler's question on https://review.opendev.org/#/c/712550/ ?15:45
fricklerclarkb: o.k. I'll propose a patch15:45
mordredAJaeger: yah - will add zuul_console - thanks15:45
*** rlandy is now known as rlandy|afk15:46
AJaegernice15:46
clarkbAJaeger: frickler the role in ozj was unused if tested. Wecan add a test back into base-jobs, but for now it seems to be working. Not sure if we want to wait for need or proactively set up a test15:47
* AJaeger sticks to his +2 for now...15:49
*** eharney has joined #openstack-infra15:49
openstackgerritJens Harbott (frickler) proposed opendev/system-config master: Increase timeout for goaccess report jobs  https://review.opendev.org/71271815:49
fricklerclarkb: ^^15:51
clarkb+215:51
openstackgerritJames E. Blair proposed zuul/zuul master: Add TLS support for ZooKeeper  https://review.opendev.org/71253115:51
openstackgerritMichal Arbet proposed openstack/project-config master: Add xstatic-** projects for vitrage-dashboard  https://review.opendev.org/70413315:54
*** dtantsur|brb is now known as dtantsur15:56
*** diablo_rojo has joined #openstack-infra15:58
clarkbyoctozepto: if it isn't too late in your day it would be good if you can confirm that base-jobs cleanup makes jobs starting after that merged happy16:01
yoctozeptoclarkb: sure, the base-test helped so rather should as they are almost the same now; will let you know, already running several previously blocked changes16:04
*** diablo_rojo has quit IRC16:06
*** jamesmcarthur has joined #openstack-infra16:06
*** roman_g has joined #openstack-infra16:07
*** diablo_rojo has joined #openstack-infra16:10
openstackgerritJames E. Blair proposed zuul/zuul master: Add TLS support for ZooKeeper  https://review.opendev.org/71253116:11
*** pgaxatte has quit IRC16:11
*** pgaxatte has joined #openstack-infra16:12
clarkbsgw: fungi: was brainstorming a bit on things that could possibly cause that behavior. Assuming the web user and the ssh user are the same (which I think we've double checked twice now) I would expect the db to be fully coherent. One idea is that this could be an api bug of some sort16:12
*** pgaxatte has quit IRC16:13
clarkbI think last time we ruled out browser plugins by using different browsers and incognito mode (if we can confirm that that may be helpful)16:13
clarkbbut given that different browsers exhibit the same behavior and everything works fine via ssh that really makes me think something is up with the http api16:14
*** ociuhandu has joined #openstack-infra16:15
openstackgerritAlbin Vass proposed zuul/nodepool master: Enable setting label and instance name separately  https://review.opendev.org/71266616:15
*** ijw has quit IRC16:19
*** ociuhandu has quit IRC16:19
*** yamamoto has joined #openstack-infra16:20
*** jamesmcarthur has quit IRC16:20
*** ijw has joined #openstack-infra16:24
*** jamesmcarthur has joined #openstack-infra16:24
*** dpawlik has quit IRC16:28
*** dpawlik has joined #openstack-infra16:28
*** ijw has quit IRC16:29
*** gshippey has joined #openstack-infra16:30
openstackgerritMerged openstack/openstack-zuul-jobs master: Remove mirror-info role and tests  https://review.opendev.org/71255016:31
openstackgerritMerged openstack/openstack-zuul-jobs master: Use a fake zuul_return and an .ansible-lint file  https://review.opendev.org/71254616:31
zbrAJaeger: ^ thanks, quite good since there is no whitelisting support in ansible-lint16:33
openstackgerritJames E. Blair proposed zuul/zuul master: Add TLS support for ZooKeeper  https://review.opendev.org/71253116:36
mordredzbr: you saw that we still had to use find yes? the ansible-lint finding roles and playbooks worked VERY badly16:36
mordredbut I'm very happy with the addition of the config file - I think it makes things much more readable16:37
*** hashar has joined #openstack-infra16:37
mordred(just mentionoing in case you happen to interact with the upstream in some manner)16:37
yoctozeptoclarkb: all jobs that were failing previously - now passing - as expected16:39
clarkbexcellent, thank you for confirming16:39
*** jcapitao is now known as jcapitao_afk16:40
*** tesseract has quit IRC16:40
openstackgerritMonty Taylor proposed zuul/zuul-jobs master: Support buildkit backend when building docker images  https://review.opendev.org/71271616:42
openstackgerritMerged opendev/system-config master: Increase timeout for goaccess report jobs  https://review.opendev.org/71271816:46
zbrmordred: do you have a bug files regarding detection failure? i could look at it over the weekend.16:46
zbrbecause the linter maintenance is still in limbo, i was considering forking it in order to fix some outstanding bugs, but i still hope that aspect will be sorted soon.16:48
zbrlast success was to enable zuul on it16:48
openstackgerritJames E. Blair proposed zuul/nodepool master: Add ZooKeeper TLS support  https://review.opendev.org/71273316:48
*** udesale_ has quit IRC16:49
*** jamesmcarthur has quit IRC16:54
*** ijw has joined #openstack-infra16:56
*** jamesmcarthur has joined #openstack-infra16:56
openstackgerritMichal Arbet proposed openstack/project-config master: Add xstatic-** projects for vitrage-dashboard  https://review.opendev.org/70413316:56
*** ijw has quit IRC16:58
*** ijw has joined #openstack-infra16:59
openstackgerritMonty Taylor proposed zuul/zuul master: Remove stretch-backports from docker build  https://review.opendev.org/71273716:59
openstackgerritMonty Taylor proposed zuul/zuul master: Install javascript tools in docker via bindep  https://review.opendev.org/71273816:59
openstackgerritMichal Arbet proposed openstack/project-config master: Add xstatic-** projects for vitrage-dashboard  https://review.opendev.org/70413316:59
mordredzbr: I don't - it was the EOD yesterday and I was out of brain pellets - I'll see if I can do a better job today17:00
mordredzbr: it did a good job of ignoring non-ansible files - so that's an improvement17:00
mordredzbr: the main issue was that it skipped MANY ansible files17:01
mordredso it was failing open17:01
*** priteau has quit IRC17:01
zbrmordred: new logic is to ignore task files from inside roles because it tests when when testing the role itself.17:02
zbrbut i am sure that the logic is still flawed, also got https://github.com/ansible/ansible-lint/pull/655 which was not reviewed.17:04
mordredzbr: it was not clear that it was doing anything with the role itself ... but I'll take another peek at it17:04
mordredianw: ^^17:04
zbrmordred: it would be very useful if you, or others would get a bit active on its issues, it may help me raise awareness and start a more active maintenance process for it.17:05
zbrgithub tickets, but using #ansible-galaxy channel is also a good idea, the linter being under the protection of galaxy team.17:06
mordredzbr: nod17:06
zbrmordred: i am sure is not beyond salvation, but it needs a kick ;)17:07
*** sshnaidm is now known as sshnaidm|afk17:07
*** jcapitao_afk is now known as jcapitao17:12
* mordred afks for a few minutes17:15
fungiclarkb: sgw: given that the logging in gerrit isn't much help for this, it might make sense to make a note to revisit it after we upgrade17:18
clarkbfungi: ya, the only other proactive idea I had was to try and capture the http requests and responses17:19
*** jamesmcarthur has quit IRC17:19
*** rlandy|afk is now known as rlandy17:19
sgwMorning guys. I think we tried the capture last time also, there really anything useful if I recall correctly.  Let me know after the upgrade happens.17:24
fungiclarkb: which is not easy given it's all https17:24
AJaegerconfig-core, a few open reviews: https://review.opendev.org/711114 https://review.opendev.org/711878 https://review.opendev.org/712586 https://review.opendev.org/712048 https://review.opendev.org/#/c/708642/17:25
AJaegerplease put on your review queue17:25
clarkbfungi: ya easiest method is probably for sgw to do it in the browser then export that (somehow)17:25
*** rpittau is now known as rpittau|afk17:27
*** ahosam has quit IRC17:28
*** ahosam has joined #openstack-infra17:29
*** auristor has quit IRC17:30
*** hashar has quit IRC17:33
*** dtantsur is now known as dtantsur|afk17:34
*** evrardjp has quit IRC17:35
*** evrardjp has joined #openstack-infra17:36
*** kozhukalov has joined #openstack-infra17:46
*** jamesmcarthur has joined #openstack-infra17:47
*** smarcet has joined #openstack-infra17:58
smarcetfungi: afternoon when u have some time please review https://review.opendev.org/#/c/712511/ :) thx u!17:58
*** derekh has quit IRC18:01
*** Lucas_Gray has quit IRC18:05
*** auristor has joined #openstack-infra18:05
*** paladox has quit IRC18:07
*** paladox has joined #openstack-infra18:09
*** jcapitao has quit IRC18:20
*** pkopec has quit IRC18:21
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add TLS configuration to ZooKeeper service  https://review.opendev.org/71275918:24
*** ahosam has quit IRC18:26
*** ralonsoh has quit IRC18:29
openstackgerritMerged opendev/puppet-openstackid master: Added python packages install support  https://review.opendev.org/71251118:33
*** gfidente is now known as gfidente|afk18:41
openstackgerritMonty Taylor proposed zuul/zuul-jobs master: Support buildkit backend when building docker images  https://review.opendev.org/71271618:42
*** jpena is now known as jpena|off18:51
*** gmann is now known as gmann_lunch19:00
*** andrewbonney has quit IRC19:02
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: install-kubernetes: install lz4 packages  https://review.opendev.org/71277119:04
*** gfidente|afk has quit IRC19:05
*** kozhukalov has quit IRC19:13
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add TLS configuration to ZooKeeper service  https://review.opendev.org/71275919:13
*** jamesmcarthur has quit IRC19:15
openstackgerritMonty Taylor proposed zuul/zuul master: Remove stretch-backports from docker build  https://review.opendev.org/71273719:18
openstackgerritMonty Taylor proposed zuul/zuul master: Be explicit about source of base images  https://review.opendev.org/71277219:18
*** amoralej is now known as amoralej|off19:24
openstackgerritMerged zuul/zuul-jobs master: install-kubernetes: install lz4 packages  https://review.opendev.org/71277119:27
openstackgerritDavid Shrewsbury proposed zuul/nodepool master: Add options to CLI info command  https://review.opendev.org/71253919:33
openstackgerritDavid Shrewsbury proposed zuul/nodepool master: WIP: Support uploads in 'info' CLI command  https://review.opendev.org/71277519:33
*** jamesmcarthur has joined #openstack-infra19:35
*** smarcet has quit IRC19:36
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add TLS configuration to ZooKeeper service  https://review.opendev.org/71275919:41
openstackgerritMonty Taylor proposed openstack/diskimage-builder master: Remove hacking from requirements  https://review.opendev.org/71277819:45
openstackgerritMonty Taylor proposed openstack/diskimage-builder master: Remove hacking from requirements  https://review.opendev.org/71277819:46
*** smarcet has joined #openstack-infra19:49
openstackgerritMonty Taylor proposed zuul/nodepool master: Stop using hacking for anything  https://review.opendev.org/71278119:50
*** trident has quit IRC19:51
openstackgerritJames E. Blair proposed zuul/zuul master: Add TLS support for ZooKeeper  https://review.opendev.org/71253119:53
openstackgerritMonty Taylor proposed openstack/diskimage-builder master: Remove hacking from requirements  https://review.opendev.org/71277819:54
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add TLS configuration to ZooKeeper service  https://review.opendev.org/71275919:57
*** trident has joined #openstack-infra19:59
*** gmann_lunch is now known as gmann20:01
*** jamesdenton has quit IRC20:03
openstackgerritMonty Taylor proposed zuul/nodepool master: Stop using hacking for anything  https://review.opendev.org/71278120:03
*** jamesdenton has joined #openstack-infra20:03
*** Goneri has quit IRC20:18
*** rh-jelabarre has quit IRC20:22
*** smarcet has quit IRC20:24
*** smarcet has joined #openstack-infra20:27
*** rh-jelabarre has joined #openstack-infra20:29
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add TLS configuration to ZooKeeper service  https://review.opendev.org/71275920:34
*** hashar has joined #openstack-infra20:37
*** smarcet has quit IRC20:38
*** ociuhandu has joined #openstack-infra20:42
fungigonna step away for ~45 minutes to get a walk in while the weather's cooperative20:48
*** ociuhandu has quit IRC21:02
*** ociuhandu has joined #openstack-infra21:03
*** gshippey has quit IRC21:07
openstackgerritIan Wienand proposed opendev/system-config master: fedora-mirror-update: remove Fedora 29 mirrors  https://review.opendev.org/71279721:07
*** ociuhandu has quit IRC21:08
ianwdpawlik: ^ it sounded like you're using the mirror outside infra?  i don't want to break you unnecessarily, but in general we're not thinking of external users when we commit changes either so it's going to be fragile generally21:09
clarkbianw: https://review.opendev.org/712778 would be a good one for you to look at (its causing nodepool to run hacking)21:10
ianwclarkb: huh, it was there before just capped.  so it's an issue with the new version?21:11
ianw(of hacking)21:11
openstackgerritMonty Taylor proposed openstack/diskimage-builder master: Remove hacking from requirements  https://review.opendev.org/71277821:12
clarkbianw: ya I think nodepool was old hacking clean, but then new hacking doesn't respect the ignore H rule or something and causes nodepool to run hacking anyway21:12
clarkbianw: because nodepool is installing flake8 and running it and dib pulls in hacking transitively (nodepool doesn't intend to run hacking at all but this does an end around that)21:12
mordredyeah - or more to the point there is a hacking test that prints stuff to stdout that for some reason just started showing up where it didn't before21:13
mordredso if you run tox -epep8 in nodepol it is ... unpleasant21:13
mordredand that's even with H in the ignores list21:13
clarkbmordred: maybe te ignores means run everything then filter rather than filter then run21:13
mordredbut - that was really just the thing that caused me to call down the rabbithole21:14
mordredclarkb: I am pretty certain that is the case21:14
ianwclarkb/mordred : i feel like we did just this before -- put hacking into test-requirements.txt and then reverted it https://review.opendev.org/#/c/682257/21:15
openstackgerritJames E. Blair proposed zuul/zuul master: Add TLS support for ZooKeeper  https://review.opendev.org/71253121:15
clarkbianw: ya one suggestion I had was that instead of deping on hacking could dep on flake8 instead21:15
clarkbbut mordred felt that was still not appropriate21:16
mordredyeah. I think it's worth breaking people21:16
mordredor worth making dib-lint its own package21:16
mordredor something21:16
mordredbut if I install diskimage-builder to build images I don't expect to get linter tools21:16
mordredI think depending on flake8 and not hacking would be an acceptable compromise though21:17
mordredif everybody else feels strongly that dib-lint is an important part of what dib is shipping21:17
ianwhrm, i think my thinking was it's kind of ilke warnings in a compiler; you might want to run it on your elements21:17
clarkbit does help provide useful advice for producing maintainable elements21:17
clarkb(things like how to set up debug tracing)21:18
mordredyeah - I can see it being a useful tool21:18
ianwi'm not strongly of that opinion, however.  i think the immediate issue was project-config broke (or will break with that)21:18
mordredk. how about we start with just depending on flake821:18
clarkbya I suggested mordred test with project-config too since we use the linter there21:18
mordredsince that won't break anything21:18
mordredthen we can come back to the other question as a followon?21:19
clarkbya that makes sense21:19
clarkbwe can take the halfway step to remove hacking from things while not breaking existing users21:19
clarkbthen sort out how to transition users if ncessary21:19
ianwi think that seems very reasonable; i think hacking has really just been copy-pasted over the years, rather than being the actual requirement21:19
*** stampeder has joined #openstack-infra21:20
openstackgerritMonty Taylor proposed openstack/diskimage-builder master: Remove hacking from requirements  https://review.opendev.org/71277821:21
mordredclarkb: feel like re-reviewing https://review.opendev.org/#/c/712772/ and https://review.opendev.org/#/c/712772/ - I removed the buildkit stuff from teh stack21:24
ianwcool; i can point-release with that ... assuming there's not some consistent issue with the triple-o job :/  if there is we can force-merge21:24
openstackgerritMonty Taylor proposed zuul/nodepool master: Be explicit about base image source  https://review.opendev.org/71279821:25
mordredianw: fingers crossed21:25
clarkbmordred: those are different changes but I'll review them anyway :)21:25
clarkblooking for the tab where I had the other stack21:26
mordredclarkb: one of them is new :)21:26
*** rh-jelabarre has quit IRC21:26
mordredclarkb: also the nodepool one there I just uploaded21:26
mordredyay thanks!21:26
*** Martouf has joined #openstack-infra21:26
mordredShrews: ^^ nodepool if you're bored21:26
ianwbtw i just approved the nb01.opendev.org deployment so will be watching what ansible gets up to with that soon21:27
*** rh-jelabarre has joined #openstack-infra21:27
ianwmordred: i don't think podman-compose will update the containers, right?21:27
clarkbianw: exciting. fyi we (mostly frickler) discovered that goaccess is compile with using /tmp for data rather than memory on ubuntu this makes it a bit slower than expected (those safer for large data sets I suppose. To address this we bumped the job timeout to 2 hours21:27
clarkbdocs.openstack.org took about an hour I guess21:28
mordredianw: it actually should - I believe our stuff does pull-always yeah?21:28
mordredianw: but - let's watch and see :)21:28
ianwmordred: i ... don't know.  we'll find out soon enough21:28
ianwclarkb: cool, that seems long; as long as it doesn't kill the webserver in the process ;)21:29
clarkbianw: its single core so should be relatively safe21:30
clarkb(and by writing to disk avoids using much memory)21:30
melwittdoes anyone know how I can know which log files are indexed by our logstash? for example, are the mysql error logs indexed?21:30
clarkbI expect its fine and we can probably nice it to be extra sure21:30
clarkbmelwitt: one sec21:30
clarkbmelwitt: https://opendev.org/opendev/base-jobs/src/branch/master/roles/submit-logstash-jobs/defaults/main.yaml is the list21:32
melwittah thank you21:32
*** rlandy is now known as rlandy|afk21:32
clarkbdoesn't look like mysql error logs are in there21:34
clarkbbut you might be able to get a canary log from something sqlalchemy leaves in service logs ?21:35
openstackgerritMerged opendev/system-config master: Add nb01.opendev.org  https://review.opendev.org/69311921:36
openstackgerritMerged opendev/system-config master: Added new Fedora release to mirroring  https://review.opendev.org/71268021:36
*** stampeder has quit IRC21:37
*** hashar has quit IRC21:40
openstackgerritJames E. Blair proposed zuul/zuul master: Add TLS support for ZooKeeper  https://review.opendev.org/71253121:40
melwittclarkb: thanks. I'm back looking at the cell timeout gate bug, was just seeing if I could do a larger scale search to check if there's a 100% fail pattern in the error log. unfortunately not something that shows up in service logs, but I can just keep looking by hand. I was just gonna use logstash if I could to get even more result :)21:41
*** ociuhandu has joined #openstack-infra21:41
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add TLS configuration to ZooKeeper service  https://review.opendev.org/71275921:43
*** xek has quit IRC21:45
mordredianw: looking forward to our new nb01 overlords21:46
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: ensure-tox: use python3 by default  https://review.opendev.org/71280421:50
*** amoralej|off is now known as amoralej22:00
*** ociuhandu has quit IRC22:03
*** ociuhandu has joined #openstack-infra22:04
*** rcernin has joined #openstack-infra22:04
*** rcernin has quit IRC22:04
*** rcernin has joined #openstack-infra22:05
*** ociuhandu has quit IRC22:09
ianwcron is failing on the old nb01 when it runs git gc on the dib_cache/source-repositories directoires with : fatal: bad object refs/remotes/origin/HEAD22:22
ianwunfortunately, it doesn't say what repo22:22
ianwbut i guess it means we're probably distributing one bad repo currently ... running manually to see what's up22:22
fungiianw: i take it that was you generating a sudo error on afs01.ord.o.o?22:22
ianwfungi: yes, sorry something else i noticed.  we had a bad sudo file on there, from ages ago when i deployed a corrupt one22:23
fungioops. cool, thanks for fixing22:23
ianwthis is the problem with tailing ansible.log, you notice all sorts of things :)22:23
fungiand yeah, it looked like the cryptic error you get when someone tries to sudo and the sudoers is corrupt22:23
ianwyep, went back in via root on bridge and cleared it22:24
openstackgerritMerged zuul/nodepool master: Be explicit about base image source  https://review.opendev.org/71279822:25
*** factor has quit IRC22:27
*** slaweq has quit IRC22:28
*** jamesmcarthur has quit IRC22:30
*** nicolasbock has quit IRC22:43
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add missing zuul service volumes  https://review.opendev.org/71281122:44
*** dave-mccowan has quit IRC22:50
*** rlandy|afk is now known as rlandy22:55
*** tkajinam has joined #openstack-infra22:56
*** matt_kosut has quit IRC22:59
*** Goneri has joined #openstack-infra23:08
openstackgerritJames E. Blair proposed zuul/zuul-operator master: Use explicit provides/requires for container jobs  https://review.opendev.org/71281623:21
*** jamesmcarthur has joined #openstack-infra23:22
openstackgerritJames E. Blair proposed zuul/zuul-operator master: Update to dhall lang v14  https://review.opendev.org/71064923:23
openstackgerritJames E. Blair proposed zuul/zuul-operator master: Add TLS configuration to ZooKeeper service  https://review.opendev.org/71275923:23
openstackgerritJames E. Blair proposed zuul/zuul master: WIP: use ZK TLS in quickstart  https://review.opendev.org/71281723:27
*** jamesmcarthur has quit IRC23:27
ianwPLAY [Configure nodepool builders] *********************************************23:28
ianwskipping: no hosts matched23:28
ianwhrm ...23:28
*** amoralej is now known as amoralej|off23:29
ianwargh, no it's some sort of distutils error23:34
ianwmordred: http://paste.openstack.org/show/790653/ does this look familiar?23:34
ianwFAILED! => {"changed": true, "cmd": ["python3", "/var/lib/get-pip.py"], ... "ModuleNotFoundError: No module named 'distutils.util'"],23:35
fungihttps://github.com/pypa/get-pip/issues/4323:36
fungineed python3-distutils installed on debian and its derivatives, i guess23:36
fungiit's separated out into its own package23:37
*** jamesmcarthur has joined #openstack-infra23:37
ianwfungi: sigh, thanks23:37
fungiany time23:37
openstackgerritIan Wienand proposed opendev/system-config master: pip3: Add python3-distutils  https://review.opendev.org/71281823:39
ianwi guess that's on gate hosts, which is why it didn't show up in CI23:39
ianw(for now)23:39
ianwin fact, we could try system-config on the plain nodes ...23:40
*** dchen has joined #openstack-infra23:40
mordredianw: yes. python3-distutil23:41
mordredianw: have I mentioned how much I love all of this? :)23:41
openstackgerritIan Wienand proposed opendev/system-config master: [dnm] test with plain nodes  https://review.opendev.org/71281923:43
ianwlet's see what explosions that brings23:43
*** dchen has quit IRC23:43
mordredianw: I think that's a GREAT idea23:45
*** dchen has joined #openstack-infra23:46
*** jamesmcarthur has quit IRC23:49
*** threestrands has joined #openstack-infra23:50

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