Friday, 2019-03-29

*** slaweq has joined #openstack-infra00:00
*** tosky has quit IRC00:05
*** slaweq has quit IRC00:05
*** lseki has quit IRC00:08
*** wolverineav has quit IRC00:10
*** ianychoi has quit IRC00:17
*** ianychoi has joined #openstack-infra00:18
*** jamesmcarthur has joined #openstack-infra00:18
*** weshay has quit IRC00:19
*** mhu has joined #openstack-infra00:19
*** weshay has joined #openstack-infra00:20
*** sthussey has quit IRC00:20
*** wolverineav has joined #openstack-infra00:33
*** wolverineav has quit IRC00:38
*** wolverineav has joined #openstack-infra00:39
*** markvoelker has joined #openstack-infra00:40
*** hongbin has joined #openstack-infra00:40
*** hamzy has joined #openstack-infra00:44
*** stakeda has joined #openstack-infra00:46
*** wolverineav has quit IRC00:46
*** jistr has quit IRC01:00
*** jistr has joined #openstack-infra01:00
dmsimardseeing node failures on opensuse things01:02
fungiyeah, if it's for opensuse-423 it's currently trying to build, this one's made it 25 minutes so far without failing01:04
fungibut not sure how much longer it has to go01:05
ianwfungi: looks like it usually runs ~ 55 minutes http://grafana.openstack.org/d/NcFtwKriz/nodepool-dib-status?orgId=101:05
*** yolanda has quit IRC01:07
dmsimardianw: those are some neat graphs01:07
ianwamazing what you can find in graphite when you check what firewall ports are open :)01:08
dmsimardfungi: I saw the last set was about 3 hours old, I'll try 423 when it lands :D01:09
dmsimardhow do we typically troubleshoot node failures btw ?01:09
dmsimardI'm not finding anything super helpful in the zuul debug logs -- I thought about checking the nodepool launchers but there's multiple of them01:10
*** slaweq has joined #openstack-infra01:11
ianwyeah really got to find the logs on the launcher01:14
ianwand if they don't reveal anything ... prepare for a long session of debugging and cross your fingers systemd isn't involved :)01:14
fungiin this case it'll be because we have no images for 423. i deleted them since they were all booting broken nodes anyway01:14
*** yolanda has joined #openstack-infra01:15
fungiand the current rebuild attempt is going on 37 minutes now01:15
fungiso... fingers crossed01:15
dmsimardianw: this is what the docs currently have on node_failure: https://zuul-ci.org/docs/zuul/user/jobs.html?highlight=node_failure#build-status01:15
*** slaweq has quit IRC01:16
dmsimardfungi: worst case I'll submit another patch to set the job I forgot as non-voting. Was hoping not to have to do that haha :p01:16
fungiyeah, we're getting close to having them working again, i hope01:17
*** whoami-rajat has joined #openstack-infra01:19
*** ricolin has joined #openstack-infra01:21
openstackgerritKendall Nelson proposed openstack-infra/project-config master: Trove: use storyboard  https://review.openstack.org/64832001:23
*** wolverineav has joined #openstack-infra01:48
*** wolverineav has quit IRC01:51
*** jamesmcarthur has quit IRC01:51
*** jamesmcarthur has joined #openstack-infra01:51
*** wolverineav has joined #openstack-infra01:53
*** rlandy has quit IRC01:55
*** jamesmcarthur has quit IRC01:56
*** jamesmcarthur has joined #openstack-infra02:02
*** wolverineav has quit IRC02:03
*** jamesmcarthur has quit IRC02:06
fungi| opensuse-423-0000036173         | opensuse-423         | nb02    | qcow2,raw,vhd | ready | 00:00:38:37 |02:07
fungidmsimard: ianw: ^02:07
*** slaweq has joined #openstack-infra02:11
*** slaweq has quit IRC02:16
*** bhavikdbavishi has joined #openstack-infra02:18
*** jamesmcarthur has joined #openstack-infra02:22
pabelangerianw: any thoughts? http://paste.openstack.org/show/748559/02:26
pabelangerianw: I'm not able to build fedora-29 minimal using latest dib02:26
*** diablo_rojo has quit IRC02:28
pabelangerianw: I guess we should first check if enable, and if not enable it02:28
pabelangerI'll have to dig more into it in the morning02:29
*** ramishra has joined #openstack-infra02:43
*** bhavikdbavishi has quit IRC02:45
ianwpabelanger: hrm, theres a change on that02:49
ianwhttps://review.openstack.org/#/c/634105/ -- change between dbus-daemon to dbus-broker.  i wonder if something else has changed?02:49
mnaserI'm almost sure ovh-gra1 has some I/o issues..02:53
mnaserI'm getting a lot of timeouts02:53
*** jamesmcarthur has quit IRC02:54
*** psachin has joined #openstack-infra02:56
clarkbamorin is our contact at ovh02:57
clarkbI know they were looking at io slowness in the context of etcd at one point02:57
clarkbmichal dulko wasworking with them according to email and found ~1.5% of fsyncs took longer than a second03:00
*** jamesmcarthur has joined #openstack-infra03:02
*** slaweq has joined #openstack-infra03:12
*** slaweq has quit IRC03:16
*** jamesmcarthur has quit IRC03:18
*** jamesmcarthur has joined #openstack-infra03:19
*** jamesmcarthur has quit IRC03:21
*** ykarel has joined #openstack-infra03:22
*** otherwiseguy has quit IRC03:24
*** weshay has quit IRC03:24
*** weshay has joined #openstack-infra03:24
*** pabelanger has quit IRC03:24
*** weshay has quit IRC03:24
*** diablo_rojo has joined #openstack-infra03:25
*** weshay has joined #openstack-infra03:25
*** bhavikdbavishi has joined #openstack-infra03:30
mnaserit's really hurting our jobs I think :\03:32
*** otherwiseguy has joined #openstack-infra03:32
mnasercloudnull: shall we drop stable/ocata at this point? at the time we discussed leaving it for now because of some rax requirements (odyssey4me brought it up).. is it still relevant or?03:43
mnaserhttps://review.openstack.org/#/q/projects:openstack/openstack-ansible+is:open+branch:stable/ocata looks like we have some trivial patches we can just abandon03:43
*** hongbin has quit IRC03:52
*** raukadah is now known as chandankumar03:53
*** udesale has joined #openstack-infra03:56
*** wolverineav has joined #openstack-infra04:03
*** wolverineav has quit IRC04:08
*** amansi26 has joined #openstack-infra04:08
mnaserodyssey4me: any reason behind why we can't run horizon on metal here? https://review.openstack.org/#/c/648595/04:08
*** eernst has joined #openstack-infra04:09
*** eernst has quit IRC04:11
mnaserjeez, I'm starting to use the wrong channels, oops.04:21
mnaseranyhow, infra-root: just saw this Download error on http://mirror.dfw.rax.openstack.org/pypi/simple/pbr/: timed out -- Some packages may not be found!04:21
mnaseralso, infra-root: http://mirror.sjc1.vexxhost.openstack.org/wheel/ubuntu-18.04-x86_64/ is there any reason 'pip' is missing there? it seems to be in all the other distro/os04:24
ianwmnaser: that's a proxy url ... so issues at the other end?04:25
mnaserianw: the first or the latter?04:25
ianwmnaser: the wheels are built from requirements.txt ... so it's not there?04:26
mnaserok, I guess at some point we removed pip from there perhaps04:26
mnaserI'll go try figure out what's going wrong04:26
openstackgerritIan Wienand proposed openstack-infra/zone-opendev.org master: Add DS record for acme.opendev.org  https://review.openstack.org/64860704:31
*** ykarel has quit IRC04:47
openstackgerritIan Wienand proposed openstack-infra/zone-opendev.org master: Add DS record for acme.opendev.org  https://review.openstack.org/64860704:50
openstackgerritIan Wienand proposed openstack-infra/zone-opendev.org master: Fix trailing dots for acme-challenge records  https://review.openstack.org/64860804:50
ianwi wonder how many hours of humanity has been lost to trailing .'s in bind config04:50
*** ykarel has joined #openstack-infra05:02
*** slaweq has joined #openstack-infra05:11
*** slaweq has quit IRC05:15
*** diablo_rojo has quit IRC05:20
*** jamesmcarthur has joined #openstack-infra05:22
*** jamesmcarthur has quit IRC05:27
*** kjackal has joined #openstack-infra05:37
*** kjackal has quit IRC05:53
*** yboaron_ has quit IRC06:02
*** e0ne has joined #openstack-infra06:06
*** janki has joined #openstack-infra06:10
*** e0ne has quit IRC06:10
*** slaweq has joined #openstack-infra06:37
*** quiquell|off is now known as quiquell06:37
*** rcernin has quit IRC06:38
*** udesale has quit IRC06:41
*** udesale has joined #openstack-infra06:41
*** slaweq has quit IRC06:42
*** kopecmartin|off is now known as kopecmartin07:07
*** slaweq has joined #openstack-infra07:08
*** dpawlik has joined #openstack-infra07:17
quiquellHello we are facing issues accessing docker registry mirrors at zuul07:22
quiquellhttp://logs.openstack.org/19/647419/6/check/tripleo-ci-centos-7-standalone-upgrade/c5dd168/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz#_2019-03-28_13_44_3107:23
quiquellInspecting with skopeo works07:23
quiquellkopeo inspect docker://docker.io/tripleostein/centos-binary-cinder-api:fbb797b891c6f2530b18ee904a93c35b9e7490c1_2c625b4907:23
quiquellskopeo inspect docker://docker.io/tripleostein/centos-binary-cinder-api:fbb797b891c6f2530b18ee904a93c35b9e7490c1_2c625b4907:23
*** yamamoto has joined #openstack-infra07:23
quiquellianw: any idea on this ^ ?07:23
quiquellykarel: ^07:24
*** quiquell is now known as quiquell|brb07:25
*** apetrich has joined #openstack-infra07:30
*** e0ne has joined #openstack-infra07:32
ianwquiquell: hrm, i'm pretty sure that's coming from the remote end07:36
*** slaweq has quit IRC07:36
*** pgaxatte has joined #openstack-infra07:36
*** jaosorior has quit IRC07:37
ianwyeah07:38
ianwhttps://registry-1.docker.io/v2/tripleostein/centos-binary-cinder-api/manifests/fbb797b891c6f2530b18ee904a93c35b9e7490c1_2c625b4907:38
ianwso maybe something up with the registry-1 bit?07:39
*** rpittau|afk is now known as rpittau07:39
openstackgerritDirk Mueller proposed openstack-infra/system-config master: Skip mirroring live and jeos images for opensuse  https://review.openstack.org/64863207:40
dirkinfra-root: looks like the opensuse mirror still hasn't updated. any idea what the issue of today is?07:41
*** ykarel is now known as ykarel|lunch07:42
dirkinfra-root: ah, nevermind. CTRL-R was missing (and maybe more coffee..)07:42
iceyAJaeger: looks like sphinx's parsing has gotten a lot stricter, putting ellipses in JSON is no longer acceptable either :-/07:44
AJaegericey: what fun ;(07:44
iceyAJaeger: looks like some `rm`s and some new `\` may resolve this though - hopefully have a new review up soon07:45
AJaegericey: thanks. We could also disable highlighting for that one if it's too tricky07:45
iceyAJaeger: we'd have to disable highlighting for a lot more than jsut that one07:45
iceyI'm onto the third file it doesn't like now07:46
icey ./source/identity/federated-keystone.rst ./source/identity/policies.rst ./source/instance-management/security-services-for-instances.rst so far07:46
*** slaweq has joined #openstack-infra07:48
*** slaweq has quit IRC07:52
iceyAJaeger: not too bad, although I'm not sure how I feel about having to make those changes: https://review.openstack.org/648634 - some of them feel hacky07:52
quiquell|brbianw: humm thanks, maybe we are missing something else at creating the tripleostein namespace07:55
quiquell|brbykarel|lunch: ^ do you know something we can be missing ?07:55
*** quiquell|brb is now known as quiquell08:06
*** tkajinam has quit IRC08:07
*** ccamacho has joined #openstack-infra08:07
*** ginopc has joined #openstack-infra08:08
*** tosky has joined #openstack-infra08:15
*** yboaron_ has joined #openstack-infra08:15
*** yamamoto has quit IRC08:20
*** yamamoto has joined #openstack-infra08:22
*** rcernin has joined #openstack-infra08:22
*** helenaAM has joined #openstack-infra08:24
*** e0ne has quit IRC08:31
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add job and integration tests for the upload-git-mirror role  https://review.openstack.org/64343508:31
*** jangutter has quit IRC08:32
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Get executor job params  https://review.openstack.org/60707808:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Separate out executor server from runner  https://review.openstack.org/60707908:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: runner: implement prep-workspace  https://review.openstack.org/60708208:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: runner: add configuration schema  https://review.openstack.org/64067208:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: runner: add execute sub-command  https://review.openstack.org/63094408:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: add command line interface  https://review.openstack.org/64477008:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: runner: add job parameters listing  https://review.openstack.org/64479508:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Add depends-on support to frozen jobs API  https://review.openstack.org/63902208:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: add support for depends-on  https://review.openstack.org/63206408:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Move repository preparation into common class  https://review.openstack.org/64864208:35
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Separate out executor concerns from AnsibleJob  https://review.openstack.org/64864308:35
AJaegericey:08:36
AJaegericey: let me look and see whether I can make it less hacky - thanks08:37
iceyAJaeger: thanks! I'm annoyed in some way about all three of the files - although the new `\` in the shell json bit is probably the most irritating to me08:37
*** gfidente has joined #openstack-infra08:38
*** jpena|off is now known as jpena08:40
*** yamamoto has quit IRC08:49
*** ykarel|lunch is now known as ykarel08:49
*** yamamoto has joined #openstack-infra08:49
openstackgerritMerged openstack-infra/zuul-jobs master: Add a role to mirror a git repository to a remote git server  https://review.openstack.org/64339408:51
openstackgerritMerged openstack-infra/zuul-jobs master: Add a job that runs the upload-git-mirror role  https://review.openstack.org/64823008:51
openstackgerritMerged openstack-infra/zuul-jobs master: Fix typo in upload-git-mirror README  https://review.openstack.org/64842308:51
*** jpich has joined #openstack-infra08:52
*** bhavikdbavishi has quit IRC08:53
fricklercorvus: jhesketh: thanks for merging my cacti patch, seems to have worked well and created e.g. http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=66569&rra_id=all08:54
*** yboaron_ has quit IRC08:56
*** rcernin has quit IRC08:57
*** udesale has quit IRC08:57
*** udesale has joined #openstack-infra08:58
openstackgerritHannu Hartikainen proposed openstack-infra/git-review master: Push with --no-follow-tags  https://review.openstack.org/64864608:59
*** e0ne has joined #openstack-infra09:00
*** lucasagomes has joined #openstack-infra09:02
*** lucasagomes has quit IRC09:03
*** lucasagomes has joined #openstack-infra09:03
*** stakeda has quit IRC09:05
*** udesale has quit IRC09:07
*** udesale has joined #openstack-infra09:10
*** pcaruana has joined #openstack-infra09:15
*** udesale has quit IRC09:15
ykarelquiquell, no idea09:22
fricklerdirk: fyi, there's another error in the tumbleweed sync now in addition to the broken tmp dirs that still exist, but it is successfully ignored with your patch now: http://paste.openstack.org/show/748572/09:24
ykarelquiquell, so just checked the patch and i can see job moved , so stein containers are working09:27
ykarelhttp://logs.openstack.org/19/647419/8/check/tripleo-ci-centos-7-standalone-upgrade/281fb53/logs/undercloud/var/log/extra/docker/docker_allinfo.log.txt.gz09:28
ykarelthe job failed at validate-services09:28
quiquellok so maybe there were not there09:28
quiquellok then as expected09:28
quiquell:-)09:28
quiquellnice09:28
*** priteau has joined #openstack-infra09:33
*** electrofelix has joined #openstack-infra09:37
*** dtantsur|afk is now known as dtantsur09:43
*** e0ne has quit IRC09:49
*** yamamoto has quit IRC09:52
*** e0ne has joined #openstack-infra09:54
openstackgerritrenminmin proposed openstack/diskimage-builder master: Add DIB_LOCAL_IMAGE to use ubuntu local cloud image  https://review.openstack.org/64388509:59
*** yamamoto has joined #openstack-infra10:00
*** wolverineav has joined #openstack-infra10:00
*** amansi26 has quit IRC10:01
*** wolverineav has quit IRC10:05
*** yamamoto has quit IRC10:11
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Separate out executor concerns from AnsibleJob  https://review.openstack.org/64864310:11
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: runner: implement prep-workspace  https://review.openstack.org/60708210:11
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: runner: add configuration schema  https://review.openstack.org/64067210:11
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: runner: add execute sub-command  https://review.openstack.org/63094410:11
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: add command line interface  https://review.openstack.org/64477010:11
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: runner: add job parameters listing  https://review.openstack.org/64479510:12
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Add depends-on support to frozen jobs API  https://review.openstack.org/63902210:12
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: add support for depends-on  https://review.openstack.org/63206410:12
*** yamamoto has joined #openstack-infra10:13
openstackgerritHannu Hartikainen proposed openstack-infra/git-review master: Push with --no-follow-tags  https://review.openstack.org/64864610:28
*** Lucas_Gray has joined #openstack-infra10:31
*** ricolin has quit IRC10:33
openstackgerritLuigi Toscano proposed openstack-infra/zuul-jobs master: stage-output: fix dotfiles handling  https://review.openstack.org/64865810:34
*** Lucas_Gray has quit IRC10:48
*** Lucas_Gray has joined #openstack-infra10:50
*** e0ne has quit IRC10:50
*** Lucas_Gray has quit IRC10:50
*** Lucas_Gray has joined #openstack-infra10:51
*** jaosorior has joined #openstack-infra10:55
*** slaweq has joined #openstack-infra11:00
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: trigger: add job filter event  https://review.openstack.org/63990511:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add initial driver and event  https://review.openstack.org/55515311:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add web route and rpclistener  https://review.openstack.org/55483911:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add build button to trigger job  https://review.openstack.org/63571611:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add support for parameterized trigger  https://review.openstack.org/64448411:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add build modal with a parameter form  https://review.openstack.org/64448511:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add support for checkbox and list parameters  https://review.openstack.org/64866111:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add triggers information to pipeline list  https://review.openstack.org/63767011:08
*** jaosorior has quit IRC11:09
*** slaweq has quit IRC11:09
*** slaweq has joined #openstack-infra11:14
*** whoami-rajat has quit IRC11:42
*** bhavikdbavishi has joined #openstack-infra11:45
*** fatema_ has joined #openstack-infra11:53
*** e0ne has joined #openstack-infra11:54
*** wolverineav has joined #openstack-infra12:01
*** e0ne has quit IRC12:02
*** wolverineav has quit IRC12:06
*** weshay is now known as weshay|rover12:06
*** jamesmcarthur has joined #openstack-infra12:18
redrobotGood morning Infra friends!12:19
*** janki has quit IRC12:20
redrobotI'm trying to debug a failing gate in Barbican.  The problem seems to be that get_python_exec_prefix in DevStack is evaluating to the wrong path12:20
redrobothttp://git.openstack.org/cgit/openstack-dev/devstack/tree/inc/python#n4112:20
*** trown|outtypewww is now known as trown12:20
redrobotIt appears to be looking for /usr/local/bin/keystone-manage12:20
redrobotbut in Fedora, it is being installed to /usr/bin/keystone-manage12:21
*** EmilienM is now known as EvilienM12:21
fungiredrobot: did this behavior change recently? on debian-based distros pip installs into /usr/local while pip on red hat distros installs into /usr12:22
fungiredrobot: you also may have more luck asking in #openstack-qa since this looks like a devstack behavior and the qa team are the maintainers of devstack12:23
*** kgiusti has joined #openstack-infra12:26
*** kgiusti has quit IRC12:29
redrobotfungi, yeah, our gates just started failing last week.  I'll check #openstack-qa, thanks!12:32
*** markvoelker has quit IRC12:32
*** jpena is now known as jpena|lunch12:32
*** jamesmcarthur has quit IRC12:33
*** e0ne has joined #openstack-infra12:36
zigofungi: Hello there!12:37
zigofungi: Is the mirroring done this time?12:37
fungizigo: yes, it finished yesterday but it was probably very late in your day by that time12:44
zigofungi: Ok, so I can ask for the review of the Debian Buster image then? :)12:44
fungiyes!12:47
*** jtomasek has joined #openstack-infra12:47
fungiwhat was the review number on that one again?12:47
*** pabelanger has joined #openstack-infra12:49
*** jamesmcarthur has joined #openstack-infra12:50
*** aaronsheffield has joined #openstack-infra12:54
*** lseki has joined #openstack-infra12:55
*** quiquell is now known as quiquell|lunch12:56
*** jaosorior has joined #openstack-infra12:56
*** kgiusti has joined #openstack-infra12:57
*** efried is now known as fried_rice12:58
*** e0ne has quit IRC13:02
funginevermind, found it once i got a bit of coffee in me ;)13:02
fungiianw: if you're not done yet for the week, have a moment to look back over 645574 again?13:04
*** yamamoto has quit IRC13:07
*** eharney has joined #openstack-infra13:07
*** yamamoto has joined #openstack-infra13:09
zigofungi: https://review.openstack.org/#/c/645574/13:10
zigoYeah.13:11
zigofungi: Sorry, was busy with colleagues.13:11
*** e0ne has joined #openstack-infra13:11
fungino need to apologize!13:11
zigoI've done quite some bits for packaging Stein already.13:11
zigokeystone, glance, cinder, neutron, nova, all done ... :P13:11
fungiwe're all busy, so totally understand13:11
zigo(plus all other 120+ dependencies)13:11
fungiyeah, that's a massive pile of work13:11
*** e0ne has quit IRC13:13
*** lpetrut has joined #openstack-infra13:15
odyssey4mehmm, is it possible that some of the pypi mirrors are a bit broken? I'm seeing the same task pass in some regions, but fail in others - eg: http://logs.openstack.org/66/648566/4/check/openstack-ansible-deploy-aio_lxc-centos-7/e6a1d27/logs/ara-report/result/dfc3e494-f38b-4219-b5d0-c7c6fa921573/ vs http://logs.openstack.org/66/648566/4/check/openstack-ansible-deploy-aio_proxy-ubuntu-bionic/c9b3de9/logs/ara-report/resul13:23
odyssey4met/e286f165-6b72-4e44-9e3e-832d9c130c04/13:23
odyssey4meugh, http://logs.openstack.org/66/648566/4/check/openstack-ansible-deploy-aio_proxy-ubuntu-bionic/c9b3de9/logs/ara-report/result/e286f165-6b72-4e44-9e3e-832d9c130c04/13:23
*** rlandy has joined #openstack-infra13:26
openstackgerritMarkus Hosch proposed openstack-infra/zuul master: Evaluate CODEOWNERS settings during canMerge check  https://review.openstack.org/64455713:26
fungiodyssey4me: we don't mirror pypi any longer, we just have caching web proxies. pip shouldn't be falling back on the wheel mirrors we build unless the proxy was unable to fetch https://files.pythonhosted.org/packages/d8/f3/413bab4ff08e1fc4828dfc59996d721917df8e8583ea85385d51125dceff/pip-19.0.3-py2.py3-none-any.whl or the index from pypi pointing to it13:28
fungicould point to issues with the proxies or with the fastly cdn pypi fronts their service with13:29
*** jaosorior has quit IRC13:30
fungii seem to be able to retrieve http://mirror.iad.rax.openstack.org/pypi/packages/d8/f3/413bab4ff08e1fc4828dfc59996d721917df8e8583ea85385d51125dceff/pip-19.0.3-py2.py3-none-any.whl at the moment so i don't think it's constant13:30
frickleralso bionic seems to try to look for a distro specific wheel, centos installs the generic one13:31
fungiwell, the generic one is what pypi (via our cache) provides, while the platform-specific wheels we only build from the global requirements list (which pip isn't in anyway) so that shouldn't exist regardless13:32
odyssey4meyeah, I find it a bit odd that it's failing with a 404 rather than just realising it's not on that mirror13:33
*** jtomasek has quit IRC13:34
*** jtomasek has joined #openstack-infra13:34
*** jpena|lunch is now known as jpena13:35
*** jtomasek has quit IRC13:36
cloudnullmnaser +1 on dropping ocata from the gates.13:37
*** priteau has quit IRC13:37
*** whoami-rajat has joined #openstack-infra13:38
*** e0ne has joined #openstack-infra13:42
fungiodyssey4me: and you say it's not happening consistently?13:43
*** jaosorior has joined #openstack-infra13:43
odyssey4mefungi well, for https://review.openstack.org/648566 it does appear to be consistent13:44
odyssey4meit looks like it's bionic only, but that's likely because pip.conf on bionic has the extra wheel links configured - whereas centos does not13:44
fungihttp://logs.openstack.org/66/648566/4/check/openstack-ansible-deploy-aio_lxc-ubuntu-bionic/dfb9540/logs/etc/host/pip.conf.txt.gz13:45
fungithat's only a fallback though13:46
odyssey4mehmm, I don't think pip works that way13:46
fungiwell, sorry, "fallback" is a poor term here13:46
odyssey4meit queries all indexes13:46
fungiyes13:46
fungiand it should be finding pip at http://mirror.dfw.rax.openstack.org/pypi/simple13:46
fungiso the fact that it doesn't find pip at http://mirror.dfw.rax.openstack.org/wheel/ubuntu-18.04-x86_64 ought to be irrelevant to it13:47
odyssey4meright, so in this case pip doesn't find it in the index and it 404's... that seems like a bug to me13:47
odyssey4meyeah, as long as it finds it in any one of its indexes, it should be fine13:47
*** bnemec is now known as beekneemech13:47
fungithere are plenty of other packages we install which aren't in the built wheel set there too, and pip doesn't normally choke13:48
*** stephenfin is now known as finucannot13:48
fungii have a feeling it's hiding an earlier failure to hit the http://mirror.dfw.rax.openstack.org/pypi/simple index13:48
fungiso the error about not finding pip at http://mirror.dfw.rax.openstack.org/wheel/ubuntu-18.04-x86_64 is likely a red herring13:48
fungioh, indeed, http://mirror.dfw.rax.openstack.org/simple/pip/13:49
fungithat's what pip in the simple index is linking to and it's a 40413:49
fungiit should instead link to http://mirror.dfw.rax.openstack.org/pypi/simple/pip/13:50
fungiwhich has content13:50
*** psachin has quit IRC13:52
fungithis seems to be the case for all entries in the index13:53
odyssey4meaha, so this is perhaps a content rewrite issue in the reverse proxy?13:53
fungior could be pypi recently changed how packages are linked in the simple index13:53
mordredoh good13:53
fungicurrently the simple index contains entries like <a href="/simple/pip/">pip</a>13:55
*** yamamoto has quit IRC13:55
fungipossible that until recently they did something like <a href="pip/">pip</a> instead13:55
mordredfungi: ugh. so it's anchored now13:56
openstackgerritLuigi Toscano proposed openstack-infra/zuul-jobs master: stage-output: fix dotfiles handling  https://review.openstack.org/64865813:56
fungiyes, i just fetched https://pypi.org/simple/ to confirm it13:56
mordred:(13:56
fungihaving to stick our pypi proxy on a separate tcp port would be unfun13:57
mnaserwhy not have two proxies, one listening at /simple ?13:57
* mnaser will gladly do what's necessary as this blocks osa13:58
mnaserso point directions and I can get at it13:58
*** yamamoto has joined #openstack-infra13:59
fungiwell, also possible it's always been this way and pip checks simple/$packagename if the url for it in simple/index.html returns a 40413:59
*** ginopc has quit IRC14:00
fungiso this might not be the actual problem14:00
*** yamamoto has quit IRC14:00
fungibut yeah, instead of a separate socket we could rewrite requests for ^/simple/.* to /pypi/simple/$114:00
mordredI would expect that potentially there would be much more breakage if all of the links are broken14:01
fungiand just remember that we can't ever have a separate tree called /simple for some other proxy purpose14:01
*** jaosorior has quit IRC14:01
*** quiquell|lunch is now known as quiquell14:02
*** jaosorior has joined #openstack-infra14:02
*** wolverineav has joined #openstack-infra14:02
mnasermordred: maybe most projects are just using the infra built wheels14:02
mnaserand we're the only weirdos consuming something like 'pip' which is probably not part of requirements14:03
frickleralso devstack is still capping pip<10, this may only be affecting latest pip14:03
mordredexcellent. more variables14:04
*** bauzas is now known as DaddyBear14:09
*** wolverineav has quit IRC14:09
funginext steps are probably to either attempt to reproduce with the same pip.conf and --verbose enabled (or sniffing communication to the mirror via tcpdump), or checking the apache access log on a mirror server to see what nodes are trying to hit14:09
*** DaddyBear is now known as PapaOurs14:10
mnaserfungi: perhaps we can get a hold on an openstack-ansible job and we can run a recheck and once it fails we can come back and sync up?14:12
mnaserwhich is probably the most automated and efficient way to getting us a broken environment to do our testing14:13
openstackgerritChris Dent proposed openstack-infra/project-config master: Stop nova-specs announcing in #openstack-placement  https://review.openstack.org/64869814:13
*** cmurphy is now known as cmurpheus14:18
*** cmurpheus is now known as cmorpheus14:18
*** yamamoto has joined #openstack-infra14:18
fungithat's an option14:19
fungii'll set one for change 648566 job openstack-ansible-deploy-aio_proxy-ubuntu-bionic14:20
mnaserfungi: "openstack-ansible-deploy-aio_lxc-ubuntu-bionic" is bit better14:20
mnaserit has less magic going on, that one has a squid proxy and tries to test we work in proxied environments14:20
fungik, will do14:21
fungiokay, autohold is set so when that job fails from odyssey4me's 13:42z recheck we should be able to poke around on it14:23
*** yamamoto has quit IRC14:24
openstackgerritMalek Karray proposed openstack-infra/storyboard-webclient master: Converting a search query into a browse  https://review.openstack.org/64442414:24
*** yamamoto has joined #openstack-infra14:25
*** yamamoto has quit IRC14:25
*** yamamoto has joined #openstack-infra14:26
*** yamamoto has quit IRC14:30
mnaserfungi: the job failed at 14:24 UTC and you put the hold in at 14:23 UTC14:31
mnaserdid we catch it or was it too late? :P14:31
fungiokay, i've got 158.69.65.235 now held from that build but i need to disappear to run errands in a few before some guests arrive for the weekend14:31
fungiwho wants ssh keys added to it?14:31
mnaserhttps://github.com/mnaser.keys14:31
mnaserand ill then try to orchestrate cause ill be around all day14:32
fungiadded14:32
mnaserI'm in, thank you sir14:32
fungiexcellent14:32
fungithanks!14:32
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Re-enable voting for opensuse15, opensuse423 and opensuse-tumbleweed  https://review.openstack.org/64848214:36
clarkbfwiw our mirrors/caches are publicly accessible in part to enable this sort of debugging14:36
clarkbyou should be able to pip install pip locally against them to test14:36
clarkb--index-url and --trusted-hostname or similar14:37
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: web: add tenant and project scoped, JWT-protected actions  https://review.openstack.org/57690714:39
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Allow operator to generate auth tokens through the CLI  https://review.openstack.org/63619714:39
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Zuul CLI: allow access via REST  https://review.openstack.org/63631514:39
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Add Authorization Rules configuration  https://review.openstack.org/63985514:39
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] Web: plug the authorization engine  https://review.openstack.org/64088414:40
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: Zuul Web: add /api/user/actions endpoint  https://review.openstack.org/64109914:40
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: authentication config: add optional token_expiry  https://review.openstack.org/64240814:40
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: [WIP] admin REST API: docker-compose PoC, frontend  https://review.openstack.org/64353614:40
AJaegerdmsimard: want to abandon https://review.openstack.org/648494 now?14:41
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool master: WIP: Add test demonstrating image build leak  https://review.openstack.org/64870614:47
fungiclarkb: yeah, that was my first suggestion, but i wasn't going to be around to actually set up and try that14:48
fungiso if folks want to have a node held to rule out something extra funky in the ansible aio setup, who am i to argue14:49
mnaserI think we might be hitting a pip bug?14:53
mnaserhttp://paste.openstack.org/show/748588/14:54
mnaserit seems to actually find it fine, it might be doing its own introspection and finds the right path14:54
mnaserthe problem is that when it fails to find it in the second index... it exits?14:54
*** jamesmcarthur has quit IRC14:54
mnasercc odyssey4me ^14:54
clarkbwhat version is listed in that constraints file?14:56
clarkbcould it be that version isnt in the first index so not finding it in the second is an error?14:56
mnaserclarkb: ok that's interesting -- the command I ran is -- /openstack/venvs/wheel-builder/bin/pip2 install -U --constraint /var/www/repo/os-releases/19.0.0.0b2.dev48/utility-19.0.0.0b2.dev48-source-constraints.txt --find-links /var/www/repo/os-releases/19.0.0.0b2.dev48/ubuntu-18.04-x86_64/ --log /var/log/python_venv_build.log pip setuptools wheel -vvvv14:57
mnaserlet me double check14:57
mnaser /var/www/repo/os-releases/19.0.0.0b2.dev48/utility-19.0.0.0b2.dev48-source-constraints.txt contains --constraint https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt?h=1785672e676e025db75a33fae370e42f1c70bec814:57
mnaserwhich doesn't have pip there14:57
mnaserso im assuming with -U it's trying to get the latest and greatest14:57
*** jcoufal has joined #openstack-infra14:57
mnaserand it is trying to seek out all indexes to find the latest version (I guess)14:58
*** e0ne has quit IRC14:58
fungiokay, heading out for a little while but should be back in an hour or so15:01
*** e0ne has joined #openstack-infra15:07
*** jamesmcarthur has joined #openstack-infra15:08
*** smarcet has joined #openstack-infra15:11
smarcetfungi:: morning15:11
smarcetfungi: yesterday i received by email an error from the former openstackid production instance15:12
smarcetfungi: i think that we could take it down15:12
*** e0ne has quit IRC15:12
smarcetfungi: not sure why got hitted15:12
*** eharney has quit IRC15:12
*** jamesmcarthur has quit IRC15:13
odyssey4memnaser fungi this is slightly orthoganal, but why are we serving the infra built wheels using an index, instead of just putting them in an auto-index folder on the web server and using find-links?15:15
odyssey4methat would mean the wheels can just be in a flat folder - not need to make an index15:16
*** dpawlik has quit IRC15:16
* mnaser doesn't know enough about the inner workings those pieces15:17
*** jamesmcarthur has joined #openstack-infra15:18
*** slaweq has quit IRC15:19
odyssey4methe advantage of using find-links are threefold - one is that the implementation of the local wheel serving simpler, another is that pip doesn't expect anything fancy from the thing serving the wheels and another is that anything found in find-links is preferred automatically above the indexes15:19
*** jaosorior has quit IRC15:21
clarkbodyssey4me: I don't think we want that last behavior fwiw15:21
clarkbwe want the packages to be resolved properly: use the newest valid version then prefer wheels if available15:21
mnaserI think newer versions of pip don't hit this bug because they just gracefully ignore things when it can't be found15:21
clarkbmnaser: what version of pip are you using there?15:22
odyssey4meclarkb it will only happen if the links and indexes have the same version - if the index has a newer version, that still wins15:22
clarkbodyssey4me: ok so that behavior already exists in our setup15:22
mnaserclarkb: 9.0.1 in here15:22
mnaserit doesn't seem to happen in centos which I think ships newer centos (via rod)15:22
mnaserrod**15:22
mnaserRDO*15:22
*** kgiusti has quit IRC15:23
clarkbmnaser: are you downgrading to 9.0.1?15:23
mnaserno, we're running -U pip setuptools wheel15:23
clarkbwhere does 9.0.1 come from then? (I just checked recently and our images do start with latest pip)15:24
clarkbat least bionic and xenial15:24
*** dhellmann_ has joined #openstack-infra15:24
mnaserI think 9.0.1 comes from the fact we install python-setuptools15:24
mnasererr15:24
mnaserpython-virtualenv15:24
*** gouthamr has quit IRC15:24
*** dmellado has quit IRC15:24
mnaserand then create a virtualenv which in turn uses whatever pip it decides to use in there15:24
*** dhellmann has quit IRC15:24
*** dulek has quit IRC15:24
mnaser(this happens inside the OSA containers, perhaps odyssey4me can correct me if im wrong)15:24
*** ruijing has quit IRC15:25
odyssey4meyeah, we install the distro package for virtualenv, then use pip to upgrade pip15:25
*** dmellado has joined #openstack-infra15:26
*** kgiusti has joined #openstack-infra15:26
*** roman_g has quit IRC15:26
odyssey4meclarkb the issue here is that an index and find-links are treated differently - if find-links was being used, pip wouldn't care if the package wasn't there15:26
clarkbodyssey4me: as for how we end up with that index setup it is how pip wheel caches them on disk and we just copy them over iirc15:27
clarkbodyssey4me: its never cared about the indexes before15:27
clarkbwe've done this for years15:27
clarkband its been fine15:27
mnaserclarkb: the thing is, pip is missing from ubuntu-18.0415:27
*** dulek has joined #openstack-infra15:27
mnaserbut it is in 16.04 and centos-715:27
odyssey4meyeah, it is odd15:27
*** dhellmann_ is now known as dhellmann15:27
odyssey4memnaser actually pip is not on centos - only virtualenv ;)15:27
*** gouthamr has joined #openstack-infra15:27
*** slaweq has joined #openstack-infra15:28
clarkbmnaser: we install it via get-pip15:28
mnaserodyssey4me, clarkb: by this I mean the 3 packages we want: pip, setuptools and wheel all have wheels in the mirrors for centos and ubuntu-16.0415:28
fricklerto be more specific http://mirror.bhs1.ovh.openstack.org/wheel/centos-7-x86_64/pip/ has 9.0.1, so does 16.04, but nothing for 18.0415:28
odyssey4meoh, that's odd15:28
mnaserwe could work around this just by having pip/setuptools/wheel there at least.15:28
mnaserit will just find 9.0.1 there and pick 19.0.3 from pypi15:29
clarkbas was mentioned before I think we removed themfrom the wheel build list15:29
odyssey4mebbiab conf call15:29
clarkbthe total number of pacakges we build wheels for is quite tiny and I know we have things like nodepool and zuul consuming package that don't exist there15:29
fricklerso maybe just having an empty http://mirror.bhs1.ovh.openstack.org/wheel/ubuntu-18.04-x86_64/pip/ dir would solve the issue15:30
*** chandankumar is now known as raukadah15:30
mnaserif someone wants to mkdir that, I can test it.15:30
*** smarcet has quit IRC15:30
*** gmann is now known as gmann_afk15:32
*** slaweq has quit IRC15:32
toskyanother issue found in zuul-jobs/stage-output: https://review.openstack.org/#/c/648658/15:32
*** kopecmartin is now known as kopecmartin|off15:32
clarkbmnaser: is your pip config using the wheels as an extra-index-url or an index-url?15:33
mnaserclarkb: our pip config inside our containers is cp'd from the host's /etc/pip.conf (the one that infra ships)15:33
mnaserhttps://www.irccloud.com/pastebin/sYzfWSal/15:33
mnaserwhich is that ^15:33
odyssey4meclarkb the pip config comes from the host, which is provided in the image - so it's using http://logs.openstack.org/66/648566/4/check/openstack-ansible-deploy-aio_lxc-ubuntu-bionic/dfb9540/logs/etc/host/pip.conf.txt.gz15:33
*** sshnaidm|off has quit IRC15:34
clarkbok I am unable to reproduce the behaviro you have locally15:34
clarkbgiev me a second to paste up what I get locally15:34
openstackgerritMalek Karray proposed openstack-infra/storyboard-webclient master: Converting a search query into a browse  https://review.openstack.org/64442415:35
clarkbneed to find the right number v's to use without making this output huge15:35
odyssey4mewell, so - let me find that bug I was looking at earlier - I think the issue might be with the pip in the package provided by ubuntu15:37
odyssey4mehttps://bugs.debian.org/cgi-bin/bugreport.cgi?bug=83776415:37
openstackDebian bug 837764 in src:python-pip "python-pip: Using `--extra-index-url` results in `HTTPError: 404 Client Error: NOT FOUND`" [Important,Open]15:37
clarkbhttp://paste.openstack.org/show/748593/ is how pip 9.0.1 behaves when asking it to pip install -U against those two indexes with the wheels as an extrai ndex url15:37
clarkbusing pip install pip 9.0.1 not debuntu packages15:38
clarkbit correctly finds the 404 and ignores that in the extra index url15:38
clarkbwow so debian has patched pip that causes this to break15:38
clarkbI don't think this is something we can fix for you in the mirror15:38
clarkbany idea which debian patch is referred to there?15:39
odyssey4mewell, this is a really old bug15:40
*** ykarel has quit IRC15:40
mnaserok I can replicate this15:40
mnaserdocker run -it --rm ubuntu:18.04 /bin/bash15:40
mnaserapt-get update15:40
mnaserapt-get install curl python-virtualenv15:40
mnasercurl http://logs.openstack.org/66/648566/4/check/openstack-ansible-deploy-aio_lxc-ubuntu-bionic/dfb9540/logs/etc/host/pip.conf.txt.gz -o /etc/pip.conf15:40
mnaservirtualenv --no-download venv15:40
mnaser./venv/bin/pip2 install -U pip15:40
mnaserdamnit, sorry.15:40
mnaserhttps://www.irccloud.com/pastebin/A3Mxhv5F/15:41
mnaserthat will fail15:41
*** knikolla is now known as nikrin15:41
clarkbright seems to be a bug in the distro packaging15:41
clarkbwhat if you pip install virtualenv instead of using the distro packaging15:41
mordredyeah. I've become a big fan of not touching distro packages for these things15:42
clarkbapt-get install python-pip && pip install virtualenv && virtualenv venv && venv/bin/pip do things15:42
mnasercentos doesn't ship pip15:42
mordredinstall pip via get-pip and then things are clean from there ... but I'm also me15:42
*** slaweq has joined #openstack-infra15:42
clarkbmnaser: get-pip.py works on centos15:42
mnaserso our best 'path to success' is get virtualenv which we have available across all systems15:42
clarkbalso I thought centos did pcakge pip15:42
mnaserand then go from there15:42
mnaserpython-pip is shipped through RDO only15:42
*** ricolin has joined #openstack-infra15:43
mordredmnaser: yeah. that's the thing don't install pip from distro at all - just use get-pip because otherwise madness starts happening really quickly15:43
clarkblooks like it is in epel15:43
clarkbso ya either use epel or get-pip15:43
mordredbut - that's jsut me running my mouth15:43
mnaserok, well, I'll ask odyssey4me if he's okay with us maybe switching to get-pip15:43
odyssey4meclarkb the trouble is that centos doesn't have a pip package, so we'd have to revert to using get-pip, and that increases complexity somewhat15:43
clarkbbut I don't think we can be responsible for broken distro packaging of pip15:43
mordredI'm rather annoyed that distro packaging has patched and broken pip15:43
odyssey4mewhen we started using the distro packages, we got to remove several hundreds of lines of code from all over the place - that's really not a place we'd like to return to15:44
mnasernot really much we can do I guess thanks to distros15:45
mordrednod. maybe we can get the debian package unbroken?15:45
* mnaser still thinks we should build our own distro one day15:45
mnaserI mean this _only_ affects our gate15:45
mordredzigo is around these days - I don't know how much magic he's got15:45
mnaserwe don't make use of extra-index-urls these days15:45
mnaserat all15:45
mnaserso the only thing is if we somehow got maybe an empty pip/wheel folder, it might think that there's nothing there and it'll be happy afterwards15:46
clarkbmnaser: you could remove the extra index url when installing pip15:46
mnaserclarkb: I tried to look for command line options to 'un-enable' the extra index url and couldn't find any15:46
clarkblike make your installation line pip install --index-url $somespecific source15:46
zigoo/15:46
clarkbmnaser: I think if you set a specific index-url on the command line only command line indexes are taken15:46
*** eharney has joined #openstack-infra15:46
mnaserclarkb: yeah but then we get into the mess of where we get the index-url from, we don't have this ever set so it takes the python default15:47
mnaserbut in our gates it takes the value from /etc/pip.conf so we don't have to do magical introspection15:47
zigomordred: So, pip install is broken in Ubuntu?15:47
mnaserzigo: my repro https://www.irccloud.com/pastebin/A3Mxhv5F/15:47
clarkbmnaser: you could set it to pypi15:47
*** lpetrut has quit IRC15:47
clarkbthen just hope that is reliable enough for a small number of package installs15:48
mnaserkinda defeats our purpose of leveraging our mirrors? :\15:48
*** sshnaidm|off has joined #openstack-infra15:48
odyssey4memnaser we could work around it by having our AIO bootstrap remove that line, but that would be unfortunate because we'd lose out on using the pre-built wheels15:48
clarkbzigo: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=837764 is the debian bug15:48
openstackDebian bug 837764 in src:python-pip "python-pip: Using `--extra-index-url` results in `HTTPError: 404 Client Error: NOT FOUND`" [Important,Open]15:48
* mnaser doesn't want to be a pain but really wonders if creating an empty pip and wheel folder would help get us through15:48
mordredmnaser: yeah - but you only need to do it for installing pip step, right?15:48
mordredlike, once your virtualenv has newer pip, you should be able to not do --index-url=pypi?15:49
clarkbmnaser: if you want to do that convince requirements team to add those packages back to constraints and we'll install them15:49
mnaserclarkb: I don't want to add them as constraint, just an empty folder to work around it, I think that might do the trick15:49
clarkbmnaser: but I don't want people to depend on the infra team manually mkdiring everytime we discover debian is silly15:49
openstackgerritMalek Karray proposed openstack-infra/storyboard-webclient master: Enabling suggestions on all user fields  https://review.openstack.org/64852515:49
odyssey4memordred in theory, yes - however that would involve putting workaround code in just for the purpose of openstack CI, which is not ideal15:49
mnaserok, fair15:50
zigoWhy not simply apt-get install python-pip ?15:50
clarkbmnaser: the issue is you'll need this for any such package15:50
zigoWhat's wrong with it?15:50
clarkbzigo: its broken as per the bug above15:50
*** slaweq has quit IRC15:50
*** psachin has joined #openstack-infra15:50
clarkbzigo: it doesn't handle extra-index-urls properly supposedly due to a local debian patch15:50
mnaserclarkb: hmm.. ok, that's valid15:50
mnaserright, if it was those 2 only that makes sense but it _could_ be more.15:50
odyssey4mezigo in this case we are installing python-virtualenv, creating a virtualenv, then trying to upgrade pip inside that virtualenv... but the pi inside the virtualenv is not treating extra indexes properly15:50
odyssey4me*the pip15:50
zigoclarkb: Yeah, but there's a workaround ... :P15:51
zigo(ie: reinstall pip with pip ...)15:52
clarkbzigo: there are a few (whcih we've suggested above), but ideally debian's package would also be fixed15:52
odyssey4methat is exactly what we're trying to do15:52
mordredyeah. I'd love to get the debian package to not be broken - even though I don't use it - just because it's the right thing to do and it's broken due to a local patch15:52
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Add logs spec  https://review.openstack.org/64871415:52
mordredbut I doubt that'll be a _short-term_ fix for anybody15:52
zigoclarkb: If I understand the bug report, it's a missing patch?15:52
zigoclarkb: Buster is still in freeze, so I could address the bug and fix it ...15:53
clarkbzigo: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=837764#15 says that the fix is to install upstream pip so that the debian patch is overwritten15:53
openstackDebian bug 837764 in src:python-pip "python-pip: Using `--extra-index-url` results in `HTTPError: 404 Client Error: NOT FOUND`" [Important,Open]15:53
clarkbunfortunately they haven't linked to the debian patch from what I can see so not sure what patch that is15:53
mnaserI wonder if there's a command line option like --no-extra-index-url15:53
* mnaser try15:53
mnasernope, doesn't exist15:54
mnasereven running it with `--isolated` seems to fail because it still seems to care about `pip.conf`15:54
zigoclarkb: The bug is about version 8.1.2-2, Buster has version 18.1-4 ...15:54
zigoclarkb: Could you test with the new version?15:54
clarkbmnaser: even if youset a --index-url?15:55
zigoCan you give me a working example of this pip command with extra index?15:56
zigoSo I can try to reproduce and fix ...15:56
mnaser /openstack/venvs/wheel-builder/bin/pip2 install -vvv -U --index-url '' pip => 1 location(s) to search for versions of pip: * http://mirror.bhs1.ovh.openstack.org/wheel/ubuntu-18.04-x86_64/pip/15:56
clarkbmnaser: same thing if you give --index-url a value?15:57
*** yamamoto has joined #openstack-infra15:57
mnaserclarkb: same for `/openstack/venvs/wheel-builder/bin/pip2 install -vvv -U --index-url https://pypi.org/simple/ pip`15:57
clarkbzigo: using pypi provided pip you can pip install --extra-index-url $url_that_will_404 $pacakge15:57
clarkbzigo: one example you can use is starting from older pip (say 9.0.1) `pip install --extra-index-url http://mirror.dfw.rax.openstack.org/wheel/ubuntu-18.04-x86_64 -U pip15:58
*** trown is now known as trown|lunch15:58
clarkbzigo: that extra index url does not have a pip directory so it will 404. The expectation is that pip will use the pypi package (default index-url) and ignore the 404 on the extra-index-url. The debian packaged pip errors on the 40415:59
*** quiquell is now known as quiquell|off15:59
openstackgerritJames E. Blair proposed openstack-infra/zuul-sphinx master: Update to new docs jobs  https://review.openstack.org/64871815:59
zigoclarkb: What's the version of python-pip that you guys are using? The one in Bionic?15:59
mnaserzigo: correct15:59
zigo9.0.1-2.3~ubuntu1 ?15:59
mnaserzigo: we use virtualenv to get pip though, if that makes sense15:59
mnaserzigo: apt-get install python-virtualenv; virtualenv venv; venv/bin/pip2 install --extra-index-url http://mirror.dfw.rax.openstack.org/wheel/ubuntu-18.04-x86_64 -U pip16:00
mnaserand that should fail16:00
zigomnaser: Ok, I'll try. This means using the system's pip though... :P16:00
mnaserzigo: well yeah, we want to upgrade to non-system pip inside a venv (which is pretty neat and clean imho, we don't clobber things)16:01
*** yamamoto has quit IRC16:01
zigomnaser: There's really no reason why you'd want to do that, you only want a pip that works ... :P16:01
* zigo starts an ubuntu image in his internal cloud16:02
clarkbreading the patches in the debian packages I don't see an obvious culprit fwiw16:02
clarkbI wonder if it is actually a patch in requests or similar. Like the exception type is different somehow making pip not catch it properly16:02
zigoclarkb: Where may I download the Bionic qcow that you are using in infra, so I can use the same image?16:02
mnaserzigo: if you feel like using docker, I have a very small repro16:03
zigomnaser: I don't ! :)16:03
mnaserhttps://www.irccloud.com/pastebin/A3Mxhv5F/16:03
zigoI prefer a VM.16:03
mnaseryou can run the same in a VM :)16:03
mnaserjust ignore the first line and you'll get the exact failure16:03
zigoI don't want to do that, I want to fix the python-pip package of Ubuntu.16:04
*** PapaOurs is now known as bauzas16:04
odyssey4mezigo you won't need the infra image - any stock ubuntu 18 will do16:04
clarkbya since this is ubuntu's packaging our image shouldn't matter16:04
mnaserhttps://www.irccloud.com/pastebin/dK7bLzIN/16:05
mnaserzigo: ^ that fails in ubuntu 18.0416:05
zigoThis ? https://cloud-images.ubuntu.com/bionic/20190327/bionic-server-cloudimg-amd64.img16:05
mnasershould fail in a VM with no problems16:06
odyssey4mezigo and this is only with installing python-virtualenv/virtualenv via ubuntu packages, then creating a virtualenv, then trying to update pip in the virtualenv... and the reason we do that is to ensure that regardless of distribution preferences, all our virtualenv use the same pip package so we know what to expect from it16:06
mnaserfail with no problems, ha. but that's a repro16:06
*** ricolin has quit IRC16:07
clarkbHTTPError is the correct exception, then in pip.index.HTMLPage.get_page the try except basically catches HTTPError and says error ignoring and moves on16:10
clarkbthat is how upstream pip handles this properly16:11
clarkbnow to find debuntu's index.py16:11
clarkbhttps://salsa.debian.org/python-team/modules/python-pip/blob/master/src/pip/_internal/index.py#L76-79 upstreams handling of that should go through there16:14
clarkbhttp://paste.openstack.org/show/748593/ and my paste using upstream pip shows that message16:14
*** e0ne has joined #openstack-infra16:14
*** e0ne has quit IRC16:14
clarkbmy hunch is that that doesn't get raised as requests.HTTPError and is instead debubuntu.somewhere.requests.HTTPError16:18
zigomnaser: What you pasted worked for me, it's *after* that upgrading pip broke stuff...16:19
zigoOh, ok, got the error...16:21
zigoSo what patch is to be removed then?16:22
zigoThere's quite a few in the Debian package.16:23
clarkbzigo: I haven't sorted that out and unfortunately that bug doesn't say anything more specific16:23
clarkbhowever, my hunch is that it has to do with the handling of unvendoring code in pip intersecting with debian somehow16:23
*** tosky has quit IRC16:23
zigoI can dicotomy search it ... :P16:23
clarkbbecause it should catch that requests.HTTPError and ignore it after logging it, but that isn't happening16:23
zigoOh ok.16:24
zigoWell, I'm not surprised.16:24
zigoBad upstream... :P16:24
clarkbif that object isn't actually requests.HTTPError but isntead debian.path.requests.HTTPError or requests.somewhere.else.HTTPError we would see this I think16:24
openstackgerritMerged openstack-infra/zuul-sphinx master: Update to new docs jobs  https://review.openstack.org/64871816:24
mnaserclarkb, zigo, mordred: I've managed to workaround it fro now in a very naughty way16:26
*** diablo_rojo has joined #openstack-infra16:26
mnaserPIP_CONFIG_FILE=/dev/null pip2 install -U pip16:26
zigomnaser: Come on, tell, don't tease ! :)16:26
zigoAh ...16:26
zigoWell, indeed, the pip.conf was the wrong bits...16:26
clarkbzigo: its not16:26
zigoWithout it, it does work.16:26
mnaser--isolated doesn't work because /etc/pip.conf is always used16:26
clarkbthe pip.conf triggers the broken path in the debian package16:26
clarkbbut the pip.conf is valid16:27
zigoOk.16:27
*** slaweq has joined #openstack-infra16:28
mnaser:O16:30
*** rlandy is now known as rlandy|brb16:30
zigoI'm trying to build the latest python-pip package from Buster, just to see what happens...16:31
zigoSame stuff ...16:32
*** roman_g has joined #openstack-infra16:33
*** pgaxatte has quit IRC16:36
clarkbzigo: I think we want to inspect what exception is actually being thrown at https://salsa.debian.org/python-team/modules/python-pip/blob/master/src/pip/_internal/index.py#L164-175 if you are able to put a breakpoint there or add some debugging printfs16:36
clarkbzigo: maybe add an except Exception as exc: #exception debugging stuff here16:36
*** Lucas_Gray has quit IRC16:37
clarkbzigo: it should be handled at line 164 but I believe it is falling through, but I don't have a good debuntu install to test on currently16:37
*** Lucas_Gray has joined #openstack-infra16:38
corvusexcept Exception as exc: print('Got exception %s' % repr(exc))16:38
corvusputting that before line 174 should do the trick16:38
openstackgerritHervé Beraud proposed openstack-dev/pbr master: Fix parsing on egg names with dashes from git URLs  https://review.openstack.org/64872716:38
openstackgerritFabien Boucher proposed openstack-infra/zuul master: WIP - Pagure driver  https://review.openstack.org/60440416:39
*** slaweq has quit IRC16:43
*** wolverineav has joined #openstack-infra16:44
*** fried_rice is now known as fried_rolls16:45
*** helenaAM has quit IRC16:45
* zigo got to go16:48
zigosorry, I tried...16:48
*** rossella_s has quit IRC16:48
*** rlandy|brb is now known as rlandy16:49
*** lucasagomes has quit IRC16:49
*** Lucas_Gray has quit IRC16:50
*** bhavikdbavishi has quit IRC16:51
fungicatching up from lunch scrollback, but pretty sure find-links is going away in pip or already has in newer versions16:51
*** bhavikdbavishi has joined #openstack-infra16:54
fungithe reason there are ubuntu-16.04 pip et cetera mirrored, i think, is that it was added briefly to global requirements when someone was trying to pin it in an inadvisable manner16:54
openstackgerritHervé Beraud proposed openstack-dev/pbr master: Fix parsing on egg names with dashes from git URLs  https://review.openstack.org/64872716:54
mnaserinfra-root: held vm 158.69.65.235 can be dropped16:54
*** jpich has quit IRC16:55
*** gfidente is now known as gfidente|afk16:55
*** rpittau is now known as rpittau|afk16:57
*** ykarel has joined #openstack-infra16:59
clarkbI'm spinning up an ubuntu container now to do the package edit ninja style17:00
*** iurygregory has quit IRC17:02
clarkbGot exception HTTPError(u'404 Client Error: Not Found for url: http://mirror.dfw.rax.openstack.org/wheel/ubuntu-18.04-x86_64/pip/',) confirmed it is falling through17:02
*** ykarel has quit IRC17:03
fungiand now i'm caught up on scrollback. exciting developments!17:04
clarkb(<class 'pip._vendor.requests.exceptions.HTTPError'>, HTTPError(u'404 Client Error: Not Found for url: http://mirror.dfw.rax.openstack.org/wheel/ubuntu-18.04-x86_64/pip/',), <traceback object at 0x7fae017bb200>)17:05
fungithanks mnaser, i'm releasing that held node now17:05
clarkbis the full exc_info17:05
clarkbok I've fixed it17:06
clarkbit was a pathing issue17:06
*** dtantsur is now known as dtantsur|afk17:06
clarkbthe fix is actually in pip itself17:07
clarkbwhich means I've got to sort out their PR process again :/ I'll work on that17:07
fungiyou can actually just fork the repo on salsa.debian.org and then create a merge request to the original with your commit17:13
fungiit's (unfortunately) eerily similar to a github workflow17:13
clarkbfungi: the fix ahs to go to pip upstream17:14
fungioh17:14
clarkbI'll get a PR up shortly and share it here :)17:14
fungiwell then, it's eerily similar to a salsa (gitlab) workflow ;)17:14
*** owalsh has quit IRC17:15
*** owalsh has joined #openstack-infra17:17
clarkbhttps://github.com/pypa/pip/pull/636717:24
clarkbmnaser: odyssey4me zigo ^ fyi17:24
clarkbit is possible that debuntu may want to carry that patch in their packaging if upstream doesn't grab it quickly17:24
clarkbnote the failing tests on that PR seem to be related to pytest failing to load a plugin17:27
clarkbI don't think it is related to my PR17:27
corvusyeah, i was about to say apparently all your.... rocketships? crashed..?17:28
clarkbthe plugin it failed to load is rerunfailures plugin17:29
corvusyou can fix that by pre-loading the "irony" plugin17:29
clarkbits a good thing I wasn't drinking tea just now17:29
clarkbit would be all over my monitor17:29
clarkbI'm checking master right now to see if this is "expected"17:31
clarkbyes master has the same issue17:31
corvusclarkb: https://github.com/pypa/pip/pull/6365 and https://github.com/pypa/pip/pull/6366 are relevant17:33
clarkbah the plugin requires a newer version of pytest17:33
clarkbyup hopefully they get that fixed and we can merge other fixes like mine :)17:34
corvusclarkb: so all you need to do is just Depends-On: #636517:34
clarkbcorvus: if only all CI worked like zuul17:34
fungis/worked like/were replaced by/17:34
clarkbin any case my PR should fix this should debuntu pull it in at some point17:34
clarkband now I can feel like I did my good deed of the day fixing a 3 year old bug that both sides rejected as not their problem17:35
*** jpena is now known as jpena|off17:37
AJaegerconfig-core, there's quite a backlog of reviews, could you spend some time reviewing, please?17:43
*** slaweq has joined #openstack-infra17:44
AJaegeralso, please review this infra-manual change https://review.openstack.org/#/c/630981/17:44
AJaegerconfig-core, I'm conferencing next week - please don't depend on me ;)17:45
*** jistr is now known as jistr|off17:45
*** jistr|off is now known as jistr17:45
*** psachin has quit IRC17:47
*** kjackal has joined #openstack-infra17:48
*** finucannot is now known as stephenfin17:53
*** yolanda has quit IRC17:53
clarkbfungi: is there a tldr on the opensuse problem?17:54
clarkbAJaeger: have fun in tenesee17:54
clarkb(also I can't type or spell)17:54
clarkbAJaeger: and ya I expect today is mostly going to be a day of reviews for me.17:55
*** wolverineav has quit IRC18:00
*** jamesmcarthur has quit IRC18:01
*** ramishra has quit IRC18:01
*** wolverineav has joined #openstack-infra18:02
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool master: WIP: Fix for image build leaks  https://review.openstack.org/64759918:02
*** jamesmcarthur has joined #openstack-infra18:03
*** kjackal has quit IRC18:07
*** kjackal has joined #openstack-infra18:07
*** sshnaidm|off has quit IRC18:08
*** trown|lunch is now known as trown18:08
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Match tag items against containing branches  https://review.openstack.org/57855718:10
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Use implied branch matcher for implied branches  https://review.openstack.org/64027218:10
fungiclarkb: no real tl;dr because there were several problems18:10
fungibiggest was that the master suse package repositories were, at one point, published empty18:11
fungiso we faithfully mirrored that18:11
fungiand as a result the base install carried a version of unbound with outdated/expired root signing keys so was filtering out important dns responses18:12
*** jamesmcarthur_ has joined #openstack-infra18:12
*** adriancz has quit IRC18:13
fungithey got the packages back in place, but had some temp directories in the tumbleweed package trees which were not world-readable so rsync was trying to descend into them and failed18:13
fungibecause all the suse mirrors share a volume, we weren't vos releasing the mirrors which were repaired18:13
fungiso we merged some (partly broken) temporary workarounds to ignore rsync errors for tumbleweed18:14
*** jamesmcarthur has quit IRC18:14
fungionce the mirrors were back on track, image builds for 42.3 were failing (not sure why, i lacked sufficient time to dig into the logs) but eventually completed later on18:15
*** iurygregory has joined #openstack-infra18:17
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Complete zuul-sphinx removal from openstack tenant  https://review.openstack.org/64875218:18
*** jamesmcarthur_ has quit IRC18:20
*** jamesmcarthur has joined #openstack-infra18:21
*** sshnaidm|off has joined #openstack-infra18:21
*** jamesmcarthur has quit IRC18:26
*** electrofelix has quit IRC18:29
*** roman_g has quit IRC18:34
clarkbfungi: opensuse is expected to work at this point though? (maybe tumbleweed is still broken?)18:34
*** slaweq has quit IRC18:36
fungii think they all should work18:38
*** pcaruana has quit IRC18:38
fungitumbleweed is probably just lacking some unreadable temporary directories which yast or whatever they use these days won't be looking for anyway18:39
clarkbzypper and yast2 :P18:39
* fungi shows his age apparently18:39
clarkbI just did a big zypper update since my computer was off while out of town. I should reboot for that at some point18:40
*** kmalloc is now known as needscoffee18:41
fungioh, and not long ago i approved the change to turn the opensuse jobs back on for openstack-zuul-jobs18:42
clarkbthanks!18:43
fungiif you want a simple but exciting review to ease your way back off the island, check out 64875218:45
*** raukadah has quit IRC18:45
clarkbwill do. I'm almost through the two devstack stacks I was asked to review18:47
*** wolverineav has quit IRC18:47
*** wolverineav has joined #openstack-infra18:49
clarkbfungi: is that the first repo to use the new zuul tenant?18:50
clarkbthe new zuul "zuul" tenant :)18:50
*** wolverineav has quit IRC18:50
*** wolverineav has joined #openstack-infra18:53
*** tbachman has joined #openstack-infra18:57
fungisort of18:57
fungidepends on what you mean "in"18:57
fungihttp://zuul.opendev.org/t/zuul/projects18:57
fungiit will be the first repo *only* in the "zuul" zuul tenant once that change merges18:58
openstackgerritMerged openstack-infra/infra-manual master: Improve the details about settings i18n for new repos  https://review.openstack.org/63098118:58
fungithere are some config and job projects which are in "zuul" but also in other tenants18:58
*** wolverineav has quit IRC18:58
openstackgerritMerged openstack-infra/project-config master: Complete zuul-sphinx removal from openstack tenant  https://review.openstack.org/64875219:00
corvusi think the next steps there are to add tag docs jobs (zuul-sphinx never had them, but other projects do, so let's just add them to zuul-sphinx to work out the issues), then pypi publication19:09
corvusi think we can rework most of the pypi jobs to be in zuul-jobs now.  dhellmann wanted to keep the playbooks together, and since the secrets required that some of them be in project-config, all of them are.  however, the new pass-to-parent feature should let us keep the secrets in p-c while the playbooks are in zuul-jobs.19:11
corvusso i'll take a stab at making zuul-jobs / opendev versions of those and using those for the zuul tenant.  if that works, we can backport to openstack.  it should make them easier to maintain.19:12
corvusof course, we can backport to openstack *after* the release.  :)19:12
*** fried_rolls is now known as fried_rice19:14
*** gmann_afk is now known as gmann19:19
AJaegerclarkb: thanks, will do my best ;)19:19
*** dpawlik has joined #openstack-infra19:20
AJaegercorvus: could you look at the errors in https://review.openstack.org/643525  - -2 due to " The project "openstack-infra/zuul-sphinx" was not found."19:21
AJaeger, please?19:21
AJaegerso, right now openstack-infra/project-config is broken ;(19:21
AJaegerah, I see you merged something already, corvus ... thanks!19:22
corvusAJaeger: it should be fixed as of 19:00, i rechecked that change19:22
AJaegercorvus: yeah, found it in backscroll now after more carefull reading , sorry19:23
*** sthussey has joined #openstack-infra19:33
*** tbachman has quit IRC19:34
fungihave the electrician working on some switches on the same circuit as my cable modem, so may be offline for a little while19:37
openstackgerritMerged openstack-infra/project-config master: Remove unneeded storyboard-webclient post job  https://review.openstack.org/64352519:41
openstackgerritMerged openstack-infra/project-config master: Set variables for storyboard-webclient release job  https://review.openstack.org/64354519:41
*** bhavikdbavishi has quit IRC19:42
*** tosky has joined #openstack-infra19:48
*** tbachman has joined #openstack-infra19:54
*** wolverineav has joined #openstack-infra19:57
*** wolverineav has quit IRC19:57
*** wolverineav has joined #openstack-infra19:57
*** dpawlik has quit IRC19:57
*** slaweq has joined #openstack-infra19:58
*** yolanda has joined #openstack-infra19:58
*** wolverineav has quit IRC20:01
*** wolverineav has joined #openstack-infra20:02
*** kgiusti has quit IRC20:04
*** slaweq has quit IRC20:05
*** wolverineav has quit IRC20:07
*** dpawlik has joined #openstack-infra20:09
*** slaweq has joined #openstack-infra20:11
*** dpawlik has quit IRC20:14
*** slaweq has quit IRC20:16
*** wolverineav has joined #openstack-infra20:17
*** jtomasek has joined #openstack-infra20:17
*** owalsh has quit IRC20:25
*** wolverineav has quit IRC20:26
*** jtomasek has quit IRC20:26
*** wolverineav has joined #openstack-infra20:27
clarkbcorvus: I unWIPed  https://review.openstack.org/#/c/644428/1 if you want to give that a review now (handles the lack of metadata in zuul return)20:30
*** wolverineav has quit IRC20:32
*** wolverineav has joined #openstack-infra20:32
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Allow user to set ara_report_executable  https://review.openstack.org/64877320:32
*** wolverineav has quit IRC20:34
openstackgerritJames E. Blair proposed opendev/base-jobs master: Add opendev-publish-tox-docs  https://review.openstack.org/64877520:34
*** wolverineav has joined #openstack-infra20:35
corvusclarkb: thanks!  we're likely to start running into that soon (opendev and zuul-sphinx are now running docs jobs which produce artifacts with no metadata)20:35
*** raissa has joined #openstack-infra20:37
*** mrhillsman is now known as mrhillsman_away420:40
openstackgerritJames E. Blair proposed opendev/base-jobs master: Add opendev-publish-tox-docs  https://review.openstack.org/64877520:41
*** mrhillsman_away4 is now known as mrhillsman_wkend20:41
openstackgerritJames E. Blair proposed opendev/base-jobs master: Add opendev-publish-tox-docs  https://review.openstack.org/64877520:43
openstackgerritMerged openstack-infra/zuul-jobs master: Handle no metadata in zuul artifact items  https://review.openstack.org/64442820:43
*** jamesmcarthur has joined #openstack-infra20:45
*** iurygregory has quit IRC20:45
openstackgerritJames E. Blair proposed openstack-infra/zuul-sphinx master: Publish docs on release  https://review.openstack.org/64877920:57
*** jcoufal has quit IRC20:58
*** trown is now known as trown|outtypewww21:00
openstackgerritKendall Nelson proposed openstack-infra/irc-meetings master: First Contact SIG update  https://review.openstack.org/64878221:05
*** owalsh has joined #openstack-infra21:06
*** wolverineav has quit IRC21:08
*** wolverineav has joined #openstack-infra21:10
*** slaweq has joined #openstack-infra21:11
*** wolverineav has quit IRC21:12
*** wolverineav has joined #openstack-infra21:15
dmsimardShould the post pipeline be sending comments back to gerrit on status ? https://opendev.org/openstack-infra/project-config/src/branch/master/zuul.d/pipelines.yaml#L11221:15
*** slaweq has quit IRC21:15
dmsimardFor context, https://review.openstack.org/#/c/648445/ has triggered a post job http://zuul.openstack.org/builds?job_name=ara-upload-git-mirror21:16
clarkbprometheanfire: tonyb: requirements type question. If neutron imports code added in neutron-lib 1.25.0 (code doesn't exist in earlier releases) does requirements global-requirements need to set the req to >=1.25.0 ?21:16
dmsimardthe job failed but I'm in the process of troubleshooting it21:16
clarkbdmsimard: no post jobs are not sent back to gerrit as they don't directly map to a change (could be tags etc)21:17
dmsimardanyway, Zuul didn't comment on gerrit about the failure21:17
dmsimardclarkb: what if there is a change, though ?21:17
clarkbdmsimard: the event type isn't a change which is the issue21:17
dmsimardit's from the merge ?21:18
clarkbdmsimard: the promote pipeline which does use change merged events could comment back (but I'm not sure it does)21:18
clarkbdmsimard: ya but the gerrit event is "this ref updated" there is no change info21:18
clarkband the ref may not be tied to a change21:18
clarkbits a weird spot for zuul to try and infer things21:18
dmsimardhmmm21:18
dmsimardzuul knows if it's going to be merging something, it knows the ref and could trigger post from there instead of the event that comes afterwards ?21:19
* dmsimard hand waving21:19
clarkbdmsimard: if zuul was updated to do the merges then yes (its been on the raod map for a while but still not done yet)21:19
*** owalsh_ has joined #openstack-infra21:19
dmsimardoh so it's actually gerrit that merges ?21:19
clarkbyes21:20
clarkbzuul pushes the gerrit "submit" button21:20
clarkb(same thing with github driver too fwiw)21:20
*** wolverineav has quit IRC21:20
dmsimardthe comment Zuul leaves is "Change has been successfully merged by Zuul" haha21:21
clarkbgerrit creates that comment21:21
dmsimardanyway, it makes sense21:21
clarkbif I clicked the button it would say Change has been successfully merged by Clark Boylan21:21
*** owalsh has quit IRC21:21
dmsimardso when it clicks the button21:21
dmsimardit knows which change it's clicking the button on21:21
clarkbyes, but not what the resulting ref will be21:22
dmsimardbecause the merge isn't done yet21:22
dmsimardI get it now21:22
dmsimard:D21:22
clarkb(because timestamp is part of the sha1 calculation)21:22
clarkbin the fast forward case it wouldn't change, but in the merge case there are things to lookup21:22
clarkbbut as you say if zuul started to do the merges this could potentially change21:23
*** owalsh has joined #openstack-infra21:23
clarkband that has been planned for a while, just needs to be done21:23
dmsimardso then no more button clicking ?21:23
clarkbyup, zuul would merge locally then (force?) push into the git repo21:24
*** owalsh_ has quit IRC21:24
mordredclarkb: I'm marvelling at your pip PR21:26
clarkbmordred: its a good one21:28
pabelangerdmsimard: clarkb: promote pipeline will report back to gerrit on success / failure, am using that today on github also21:29
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64685521:30
*** wolverineav has joined #openstack-infra21:31
*** jamesmcarthur has quit IRC21:31
*** jamesmcarthur has joined #openstack-infra21:32
dmsimardanyway, what I was trying to troubleshoot is why it seems my secret isn't making it to my job.. I'm a bit confused21:33
dmsimardbut I think it would be because the secret can't be used until after the merge and yet post runs after so the secret wasn't available yet ? it's seeking in an untrusted path http://logs.openstack.org/4f/4f96908789b93d1bfc00047e2e3b39d85ab76e3d/post/ara-upload-git-mirror/f8d0260/ara-report/result/0ffa8903-4dc2-4bd7-834b-0514b07899b8/21:33
pabelangerwhere is git_mirror_credentials defined?21:39
*** josephrsandoval has joined #openstack-infra21:39
pabelangerpost pipline, should have access to a secret21:40
dmsimardpabelanger: https://opendev.org/openstack/ara/commit/4f96908789b93d1bfc00047e2e3b39d85ab76e3d21:40
dmsimardmaybe I have the names and stuff mixed up21:40
dmsimardI'll re-read docs :D21:40
zigoclarkb: I'll try to get this patch into Debian ASAP, before Buster is frozen.21:42
zigoThanks for it.21:42
zigoclarkb: We can try to get it into Ubuntu Bionic as well, maybe.21:42
zigojamespage: coreycb: ^21:42
zigoThe patch is: https://github.com/pypa/pip/pull/6367/commits/f8292a304deebcf0e4cda2e40caa226c70030f1121:42
*** jamesmcarthur has quit IRC21:43
*** jamesmcarthur has joined #openstack-infra21:43
pabelangerdmsimard: I think you have a syntax issue, I believe you want: http://paste.openstack.org/show/748610/21:43
pabelangerhttps://zuul-ci.org/docs/zuul/user/config.html#attr-job.secrets21:44
dmsimardoh I had it backwards ?21:44
pabelangerdmsimard: no, secrets is a list21:44
dmsimardoh that makes more sense21:44
dmsimardinteresting that it didn't trip zuul syntax error21:45
dmsimardI didn't provide a list21:45
pabelangeryah, not sure why21:45
*** owalsh has quit IRC21:45
dmsimardpabelanger: I'll merge a fix, thanks :D21:48
*** yamamoto has joined #openstack-infra21:49
*** jamesmcarthur has quit IRC21:49
*** jamesmcarthur has joined #openstack-infra21:50
*** josephrsandoval has quit IRC21:50
mnaseranyone ever seen something where wheels don't get cached so pip installs in the gate are always pulling down requirements every single time?21:50
mnaserI have a job here that's taking quite sometime and im pretty sure it would be much faster if it just cached them instead of redownloading them21:51
*** yamamoto has quit IRC21:53
pabelangermnaser: possible that our wheels mirrors don't have said wheel?21:54
pabelangerhowever, I think we now proxy pypi, don't we?21:54
corvusdmsimard, pabelanger: 'secrets' is, like many zuul job attrs, a singleton-or-list.21:54
tbachmanI have an infra-related question — this gerrit: https://review.openstack.org/#/c/647475/ passed on the 25th, but then when we went to merge yesterday, it’s as though the artifacts themselves weren’t there. Did something significant change recently wrt the way buidls are being done the last couple of days?21:55
pabelangerah, cool. didn't realize that21:55
tbachmanthe closest thing I could find was a change from git to https, but my first attempt to change those in our repos didn’t seem to have any effect21:56
* tbachman will accept any response, including “go away you infra n00b”21:56
corvustbachman: this looks like an error: http://logs.openstack.org/75/647475/1/check/openstack-tox-pep8/6c806c4/job-output.txt.gz#_2019-03-29_00_17_47_98308021:57
tbachmanright - but what’s weird is we have no idea why that would have disappeared. Its as though the packages themselves aren’t present21:57
tbachmanwhich is what led me to the git vs. https21:58
tbachmanour ocata/pike builds were okay - it was only starting with queens21:58
pabelangerdmsimard: Oh, I think you need to set pass-to-parent: true, right? because it is the parent that is using the secret, not the job in ara21:59
mnaserpabelanger: the logs show the wheel being downloaded only enough22:00
pabelangermnaser: have a log I can look at?22:00
mnaserpabelanger: http://logs.openstack.org/10/648710/1/check/openstack-ansible-deploy-aio_metal-ubuntu-bionic/74f0461/logs/ara-report/ if you open tasks under setup-openstack.yml and sort by duration, you'll see a lot of long "python_venv_build : Install python packages into the venv"22:01
mnaserwhen you click on it to view the in detail log, you see it's downloading it22:01
mnasernow if you click on a bunch you'll notice for an example PyMySQL is always being downloaded22:01
*** rlandy has quit IRC22:02
pabelangermnaser: Hmm, this doesn't look right22:03
pabelangerThe repository located at mirror.gra1.ovh.openstack.org is not a trusted or secure host and is being ignored. If this repository is available via HTTPS we recommend you use HTTPS instead, otherwise you may silence this warning and allow it anyway with '--trusted-host mirror.gra1.ovh.openstack.org'22:03
mnaserpabelanger: oh that's interesting, I wonder if our usage of --trusted-host is overriding the pip.conf22:06
mnaserhttp://logs.openstack.org/10/648710/1/check/openstack-ansible-deploy-aio_metal-ubuntu-bionic/74f0461/logs/etc/host/pip.conf.txt.gz22:07
mnaserI mean it's there.22:07
dmsimardpabelanger: ah, it might be that too. I wanted to try using it without because I was paranoid about the wording used22:07
pabelangermnaser: not sure, maybe clarkb knows more22:10
*** slaweq has joined #openstack-infra22:11
*** owalsh has joined #openstack-infra22:15
*** slaweq has quit IRC22:16
prometheanfireclarkb: neutron itself needs to update it's minimum in requirements.txt22:19
*** jamesmcarthur has quit IRC22:19
prometheanfireclarkb: global-reqs in the reqs project doesn't have minimums22:20
clarkbprometheanfire: thanks22:21
clarkbtrustedhosts should mean its ok to do http instead of https to that host22:21
*** raissa has quit IRC22:24
*** ijw has joined #openstack-infra22:31
*** ijw has left #openstack-infra22:31
*** ijw has joined #openstack-infra22:31
ijw#join openstack-infra22:32
ijwgrr22:32
*** ijw has left #openstack-infra22:32
*** ijw has joined #openstack-infra22:32
*** rkukura_ has joined #openstack-infra22:36
ijwApologies for asking, but can someone +A https://review.openstack.org/#/c/648553/ ?  It removes a legacy check job for networking-vpp that has a fault in it anyway, and I was planning to move it over to .zuul.yaml as I fix it, but the sooner it's gone the less time it wastes.22:36
*** rkukura has quit IRC22:38
*** rkukura_ is now known as rkukura22:38
openstackgerritMerged openstack-infra/storyboard-webclient master: Stars should be next to the name of the entity  https://review.openstack.org/64315322:41
clarkbijw: done, is there a change to openstack-infra/openstack-zuul-jobs to remove the job definition from that repo?22:44
*** wolverineav has quit IRC22:51
rpiosodiablo_rojo: Thanks for the email about the final PTG schedule. It appears the Baremetal SIG will be dynamically scheduled during the PTG. Is my understanding correct? Is holding it during the Forum still being considered? My present travel arrangements are for attending only the PTG.22:53
openstackgerritMerged openstack-infra/project-config master: Remove legacy-networking-vpp-smoke  https://review.openstack.org/64855322:55
diablo_rojorpioso, I am not sure what they have planned for the Forum if anything. You would want to talk to hogepodge about that.22:57
hogepodgeI was hoping we would get a space, but I think we were too late.22:58
hogepodgerpioso: but you can plan on us doing something during the PTG22:58
rpiosodiablo_rojo, hogepodge: Thank you! Hope you have an enjoyable weekend :)22:59
*** gfidente|afk is now known as gfidente22:59
*** wolverineav has joined #openstack-infra23:01
*** sthussey has quit IRC23:01
openstackgerritMerged openstack-infra/storyboard-webclient master: Don't create a tarball in the post job  https://review.openstack.org/64770623:02
ijwclarkb: https://review.openstack.org/#/c/648549/ (just kicked its recheck)23:03
*** aaronsheffield has quit IRC23:03
clarkbijw: looks like it needs a rebase?23:04
ijwWill deal, give me 523:04
clarkbno rush. And thanks23:04
ijwI saw yesterday someone was rewriting URLs all over, so it's a fair bet that caught it23:04
diablo_rojorpioso, you too!23:05
clarkbya we are trying to move away from the git protocol urls in favor of http(s) so that we can switch to gitea as our git server23:05
openstackgerritIan Wells proposed openstack-infra/openstack-zuul-jobs master: Remove legacy-networking-vpp-smoke  https://review.openstack.org/64854923:07
ijwNo, I saw and it entirely makes sense; it was just my luck to pick the one moment when a broad-brush change came along (in a file I was deleting, no less).  Anyway, merge conflict resolved and Zuul is thinking23:10
clarkb+2 thanks again23:13
*** gfidente has quit IRC23:13
*** diablo_rojo has quit IRC23:14
*** rh-jelabarre has quit IRC23:34
*** wolverineav has quit IRC23:45
*** whoami-rajat has quit IRC23:47
*** dklyle has quit IRC23:48
*** david-lyle has joined #openstack-infra23:48
*** tbachman has quit IRC23:50
*** needscoffee is now known as kmalloc23:53
tonybclarkb: neutron need to set it's minimum in it's requirements to >=1.25.0 and the matching lower-constraint23:53
clarkbtonyb: yup it looks like they've done that. Reading requirements more some entries do have lower bounds is that just legacy info?23:54
tonybclarkb: We shoudln't be tracking lowers globally anymore23:55
tonybclarkb: I'll lookinto it on Monday if you leave me a pointer23:55
clarkboh you are right I think I read a != as a >=23:55
tonybahh cool23:56
tonyb[tony@thor requirements]$ git grep '>=' global-requirements.txt23:56
tonybglobal-requirements.txt:Django<2.1;python_version>='3.0'  # BSD23:56
tonybglobal-requirements.txt:dnspython3!=1.13.0,!=1.14.0;python_version>='3.0'  # http://www.dnspython.org/LICENSE23:56
tonybglobal-requirements.txt:mypy;python_version>='3.4'  # MIT23:56
tonybOh no that's in the marker my bad23:56
tonybnothign to see here23:56
* tonyb goes to the park23:56

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