Tuesday, 2019-05-28

*** eernst has quit IRC00:13
*** rcernin has quit IRC00:17
*** rcernin has joined #openstack-infra00:18
*** hwoarang has quit IRC00:30
*** hwoarang has joined #openstack-infra00:31
*** smarcet has joined #openstack-infra00:37
*** hwoarang has quit IRC00:41
*** hwoarang has joined #openstack-infra00:42
*** rcernin has quit IRC00:49
*** rcernin has joined #openstack-infra00:50
*** hwoarang has quit IRC01:00
*** hwoarang has joined #openstack-infra01:00
*** markvoelker has quit IRC01:05
*** rfolco has quit IRC01:06
*** whoami-rajat has joined #openstack-infra01:06
*** hongbin has joined #openstack-infra01:10
*** lseki has quit IRC01:11
*** bobh has joined #openstack-infra01:12
*** ricolin has joined #openstack-infra01:13
*** hwoarang has quit IRC01:13
*** hwoarang has joined #openstack-infra01:14
*** bobh has quit IRC01:23
*** sreejithp has joined #openstack-infra01:27
*** sreejithp has quit IRC01:31
*** hwoarang has quit IRC01:33
*** hwoarang has joined #openstack-infra01:33
*** smarcet has quit IRC01:43
*** hwoarang has quit IRC01:47
*** hwoarang has joined #openstack-infra01:47
*** gregoryo has joined #openstack-infra01:58
*** hwoarang has quit IRC01:58
*** hwoarang has joined #openstack-infra01:59
*** hwoarang has quit IRC02:04
*** hwoarang has joined #openstack-infra02:04
openstackgerritTony Breeds proposed opendev/storyboard master: Change the dist name to 'opendev-storyboard'  https://review.opendev.org/66166302:05
*** udesale has quit IRC02:15
openstackgerritMerged openstack/reviewstats master: Adds tox docs job  https://review.opendev.org/47064202:19
openstackgerritMerged openstack/reviewstats master: Add cinder-tempest-plugin repository for Cinder  https://review.opendev.org/65667502:25
openstackgerritMerged openstack/reviewstats master: Add cinderlib repository for Cinder  https://review.opendev.org/65668202:29
*** gmann has quit IRC02:31
*** hwoarang has quit IRC02:32
*** hwoarang has joined #openstack-infra02:32
*** hwoarang has quit IRC02:48
*** hwoarang has joined #openstack-infra02:49
*** ktsuyuzaki is now known as kota_03:01
*** hwoarang has quit IRC03:03
*** hwoarang has joined #openstack-infra03:03
*** markvoelker has joined #openstack-infra03:06
*** jamesdenton has quit IRC03:20
*** psachin has joined #openstack-infra03:21
*** hwoarang has quit IRC03:21
*** hwoarang has joined #openstack-infra03:21
*** markvoelker has quit IRC03:40
*** RobertNordix has quit IRC03:55
*** imacdonn has quit IRC04:20
*** imacdonn has joined #openstack-infra04:21
*** pcaruana has joined #openstack-infra04:22
*** ykarel|afk has joined #openstack-infra04:31
*** markvoelker has joined #openstack-infra04:37
*** hwoarang has quit IRC04:38
*** psachin has quit IRC04:39
*** psachin has joined #openstack-infra04:39
*** hwoarang has joined #openstack-infra04:40
*** dchen has joined #openstack-infra04:46
*** hongbin has quit IRC04:49
*** janki has joined #openstack-infra04:51
*** raukadah is now known as chandankumar04:52
*** virendra has joined #openstack-infra04:56
virendraHi, I am getting error as below04:58
virendraERROR: could not install deps [-chttps://releases.openstack.org/constraints/upper/master, -r/opt/stack/new/tempest/requirements.txt]; v = InvocationError('/opt/stack/new/tempest/.tox/tempest/bin/pip install -chttps://releases.openstack.org/constraints/upper/master -r/opt/stack/new/tempest/requirements.txt (see /opt/stack/new/tempest/.tox/tempest/log/full-1.log)04:58
virendrafor 3rd party CI devstack installation for command "tox -r --notest -efull"04:59
virendraif anyone can help that will very helpful04:59
*** YaminiU has joined #openstack-infra05:05
*** hwoarang has quit IRC05:05
YaminiUHi Team, we were using the new review.opendev.org url for our CI system over the past 24 hours it is failing. we are seeing this error. 2019-05-28 04:54:16,032 DEBUG zuul.AnsibleJob: [build: a247ab84ca2142b18d8ec4570eb652c4] Sending result: {"result": "ERROR", "error_detail": "Failed to update project review.opendev.org/openstack/nova"} 2019-05-28 04:54:16,036 DEBUG zuul.ExecutorServer: Sent SIGTERM to SSH Agent, {'SSH_AUTH05:06
*** hwoarang has joined #openstack-infra05:06
YaminiUdid something change which is causing this failure. manually also that link no longer works05:06
YaminiUCan someone please help in this regard05:07
*** ykarel|afk is now known as ykarel05:10
*** markvoelker has quit IRC05:10
*** ykarel_ has joined #openstack-infra05:13
tonybvirendra: Are the logs somewhere?05:13
virendraHi @tonyb, please find log "http://54.201.44.218/60/660460/2/check/3par-fc-driver-master-client-pip-ssa02-dsvm/4dfc718/logs/devstacklog.txt.gz"05:15
*** ykarel has quit IRC05:15
*** ykarel__ has joined #openstack-infra05:16
*** hwoarang has quit IRC05:16
*** hwoarang has joined #openstack-infra05:17
tonybvirendra: "error('Tunnel connection failed: 503 Connect failed'"05:17
tonybvirendra: is your proxy server able to get https://releases.openstack.org/constraints/upper/master ?05:17
*** ykarel_ has quit IRC05:19
YaminiUHi Team, we were using the new review.opendev.org url for our CI system over the past 24 hours it is failing. we are seeing this error. 2019-05-28 04:54:16,032 DEBUG zuul.AnsibleJob: [build: a247ab84ca2142b18d8ec4570eb652c4] Sending result: {"result": "ERROR", "error_detail": "Failed to update project review.opendev.org/openstack/nova"} 2019-05-28 04:54:16,036 DEBUG zuul.ExecutorServer: Sent SIGTERM to SSH Agent, {'SSH_AUTH05:20
YaminiUthis url is not working05:21
YaminiUhas anyone faced this issue05:21
YaminiUhow to resolve it05:21
*** hwoarang has quit IRC05:22
*** hwoarang has joined #openstack-infra05:22
tonybYaminiU: The usful part of the error has been snipped :(05:27
*** jchhatbar has joined #openstack-infra05:27
YaminiUok i will give paste url05:27
tonybYaminiU: but at a guess I'd say you ssh creds are messed up05:27
YaminiUoh ok05:27
tonybYaminiU: but I have no access to anything to verify that05:28
*** janki has quit IRC05:30
YaminiUhttp://paste.openstack.org/show/752137/05:34
YaminiUhi tonyb please find the paste link05:34
YaminiUwith the error05:34
*** hwoarang has quit IRC05:35
*** hwoarang has joined #openstack-infra05:35
*** kjackal has joined #openstack-infra05:35
*** yboaron_ has joined #openstack-infra05:36
tonybYaminiU: Thanks, I may have been wrong about the root cause, sadly I don't have any idea how to debug that further :(05:36
YaminiUok thanks Tonub05:36
*** quiquell has joined #openstack-infra05:36
YaminiUtonyb05:36
YaminiUhi tonyb can you please check this paste05:41
YaminiUhttp://paste.openstack.org/show/752138/05:41
virendratonyb: https://releases.openstack.org/constraints/upper/master not reachable through proxy, let me pass it through.05:46
tonybvirendra: okay at least that's a somewhat well defined problem05:47
YaminiUhow do we increase the timeout for the clone from 300secs05:54
virendratonyb: yes, thank You!05:58
*** lpetrut has joined #openstack-infra06:01
tonybYaminiU: https://zuul-ci.org/docs/zuul/admin/components.html#attr-merger.git_timeout ?06:02
*** kopecmartin|off is now known as kopecmartin06:04
*** e0ne has joined #openstack-infra06:06
openstackgerritTristan Cacqueray proposed zuul/nodepool master: openstack: document the key-name parameter  https://review.opendev.org/66167706:07
*** markvoelker has joined #openstack-infra06:07
*** hwoarang has quit IRC06:11
*** hwoarang has joined #openstack-infra06:12
*** ykarel__ is now known as ykarel|afk06:14
*** ccamacho has joined #openstack-infra06:18
*** slaweq has joined #openstack-infra06:22
*** e0ne has quit IRC06:25
*** dpawlik has joined #openstack-infra06:32
*** yolanda has quit IRC06:38
*** markvoelker has quit IRC06:40
AJaegerconfig-core, a small cleanup for review, please: https://review.opendev.org/66161006:41
*** ramishra has joined #openstack-infra06:48
*** ykarel|afk is now known as ykarel06:55
*** gregoryo has quit IRC06:56
YaminiUhi team seeing the below error and CI jobs are fialing07:00
YaminiUhttp://paste.openstack.org/show/752141/07:00
YaminiU@tonyb after increasing timeout am seeing this error07:00
YaminiUseems new07:00
openstackgerritIan Wienand proposed opendev/system-config master: Replace skip with errors=ignore  https://review.opendev.org/66054107:01
*** rpittau|afk is now known as rpittau07:02
openstackgerritLucian Petrut proposed openstack/project-config master: Rename x/compute-hyperv to openstack/compute-hyperv  https://review.opendev.org/66168207:07
*** takamatsu has joined #openstack-infra07:07
*** jhesketh has quit IRC07:11
*** ricolin_ has joined #openstack-infra07:12
*** tesseract has joined #openstack-infra07:12
*** takamatsu has quit IRC07:13
tonyb[m]https://usercontent.irccloud-cdn.com/file/KAEjmkeu/irccloudcapture5579368549190068292.jpg07:14
*** ricolin has quit IRC07:15
*** ginopc has joined #openstack-infra07:15
tonyb[m]I did 5k in almost exactly 30mins the rest was a cool down walk07:17
tonyb[m]Lol wrong channel :-( #egg on face07:17
*** yamamoto has joined #openstack-infra07:18
*** yboaron_ has quit IRC07:22
*** parasitid has joined #openstack-infra07:23
parasitidhi there07:23
parasitidi have an issue with mirrors on nodepools07:24
parasitidthe openstack loci project07:24
evrardjptonyb[m]: hahaha07:24
parasitidis based on stuff available on the mirrors07:24
parasitid(eg http://mirror.iad.rax.openstack.org/opensuse/update/leap/)07:24
evrardjpparasitid: oh I am just looking at this07:25
parasitidand the loci project targets the opensuse/leap:15 docker container07:25
parasitidbut it has been updated 6 days ago and now the "15" docker tag points to the "15.1" version07:25
*** yamamoto has quit IRC07:25
parasitidwhich isnt available on the mirrors07:26
parasitidevrardjp: ok thanks07:26
evrardjpparasitid: there are multiple things that break loci right now, but they are indeed all waiting for a change in infra mirrors07:26
parasitidi wonder if i should propose a patchset in  loci to explicitly target the 15.0 version07:27
evrardjphttps://review.opendev.org/#/c/660126/ just merged07:27
evrardjpparasitid: please don't07:27
parasitidok07:27
parasitidthanks07:27
evrardjpwe should just make it work in infra07:27
evrardjpthat would be great if there is someone that can give us a view of why the patch above merged, but the mirrors (like rax) aren't updated yet -- maybe there is a problem with the script?07:28
*** apetrich has joined #openstack-infra07:29
*** jhesketh has joined #openstack-infra07:29
*** happyhemant has joined #openstack-infra07:31
*** logan- has quit IRC07:33
*** dchen has quit IRC07:34
*** tosky has joined #openstack-infra07:35
*** RobertNordix has joined #openstack-infra07:35
*** logan- has joined #openstack-infra07:36
*** markvoelker has joined #openstack-infra07:37
ianwevrardjp: like that *just* merged?07:47
ianwsorry, i thought i'd reviewed that.  anyway, it won't apply till the next mirror update pulse07:47
*** xek_ has joined #openstack-infra07:48
ianwwhich you can sort of watch here -> http://grafana.openstack.org/d/ACtl1JSmz/afs?orgId=107:48
*** rossella_s has joined #openstack-infra07:49
ianwbut now i notice opensuse hasn't released07:50
ianwevrardjp: http://paste.openstack.org/show/752142/07:50
ianwis that our problem, or the remote end?07:50
evrardjpthat's a very good question :p07:52
evrardjplet me do my homework there then I guess :)07:52
evrardjpand thanks for pointing me to the watch part07:52
*** jpena|off is now known as jpena07:52
ianwevrardjp: sorry, i've got to run off ... unfortunately the logs aren't exactly exposed so stuff like that you really need an infra-root to see.  but yeah, at least on the granfana page it should stamp if it *is* updating07:58
*** Emine has joined #openstack-infra07:58
*** ykarel is now known as ykarel|lunch08:00
*** ginopc has quit IRC08:03
evrardjpianw: thanks already08:07
evrardjpI am in meetings so I couldn't give fast feedback there :(08:07
evrardjpI will try to understand the full thing and get back here, asking the infra-roots :)08:07
*** markvoelker has quit IRC08:10
*** lucasagomes has joined #openstack-infra08:11
*** gmann has joined #openstack-infra08:13
*** yamamoto has joined #openstack-infra08:14
*** YaminiU has quit IRC08:23
*** pkopec has joined #openstack-infra08:24
*** ralonsoh has joined #openstack-infra08:26
gmannclarkb: replied on this. legacy jobs testing of latest stable should be enough - https://review.opendev.org/#/c/647912/308:32
*** yboaron_ has joined #openstack-infra08:34
*** piotrowskim has joined #openstack-infra08:40
*** yboaron_ has quit IRC08:40
*** yboaron_ has joined #openstack-infra08:41
*** gfidente has joined #openstack-infra08:52
*** derekh has joined #openstack-infra08:52
*** ginopc has joined #openstack-infra08:56
*** priteau has joined #openstack-infra08:57
*** e0ne has joined #openstack-infra09:00
*** rascasoft has joined #openstack-infra09:01
openstackgerritMerged openstack/project-config master: Cleanup in-tree removed jobs  https://review.opendev.org/66161009:02
*** rascasoft has quit IRC09:03
*** rascasoft has joined #openstack-infra09:03
*** rascasoft has quit IRC09:04
aspiersfungi, corvus: https://gitenterprise.me/2019/05/20/gerrit-v3-0-is-here/09:06
aspiersit requires migrating through all the intermediate releases though09:07
*** markvoelker has joined #openstack-infra09:07
*** tkajinam has quit IRC09:13
*** rascasoft has joined #openstack-infra09:14
*** jesusaur has quit IRC09:17
*** jchhatba_ has joined #openstack-infra09:18
*** jesusaur has joined #openstack-infra09:20
*** jchhatbar has quit IRC09:21
*** rossella_s has quit IRC09:22
*** yoctozepto has joined #openstack-infra09:25
*** sreejithp has joined #openstack-infra09:28
*** sreejithp has quit IRC09:33
*** ykarel|lunch is now known as ykarel09:33
*** markvoelker has quit IRC09:40
*** rossella_s has joined #openstack-infra09:46
*** ricolin_ has quit IRC10:02
*** dpawlik has quit IRC10:06
*** ramishra has quit IRC10:11
*** dpawlik has joined #openstack-infra10:18
*** emine__ has joined #openstack-infra10:20
*** Emine has quit IRC10:22
*** dpawlik has quit IRC10:22
*** ramishra has joined #openstack-infra10:25
*** dpawlik has joined #openstack-infra10:35
*** markvoelker has joined #openstack-infra10:37
*** dpawlik has quit IRC10:40
*** virendra has quit IRC10:49
*** yamamoto has quit IRC10:50
*** dpawlik has joined #openstack-infra11:02
*** yamamoto has joined #openstack-infra11:04
*** markvoelker has quit IRC11:11
*** ginopc has quit IRC11:11
*** dave-mccowan has joined #openstack-infra11:13
*** psachin has quit IRC11:14
*** yamamoto has quit IRC11:14
*** yamamoto has joined #openstack-infra11:20
*** yamamoto has quit IRC11:21
*** AJaeger has quit IRC11:24
openstackgerritTobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check  https://review.opendev.org/64455711:27
*** yamamoto has joined #openstack-infra11:32
*** jpena is now known as jpena|lunch11:33
*** yamamoto has quit IRC11:34
*** AJaeger has joined #openstack-infra11:36
openstackgerritAndreas Jaeger proposed openstack/project-config master: Move x/monitorstack jobs in-tree  https://review.opendev.org/66173311:42
AJaegerconfig-core, I forgot to add one repo, could you review ^, please?11:42
*** yamamoto has joined #openstack-infra11:44
aspiersAJaeger: if the same fix applies over multiple repos, is it best practice to reuse the Change-Id?11:44
*** yamamoto has quit IRC11:48
openstackgerritNir Magnezi proposed openstack/diskimage-builder master: Add version-less RHEL element for RHEL7 and RHEL8  https://review.opendev.org/64373111:50
*** lucasagomes has quit IRC11:51
AJaegeraspiers: it's nice but not needed. I would require same topic on the other hand11:51
*** lucasagomes has joined #openstack-infra11:51
AJaegeraspiers: if you want to do it, go for it - but I wouldn't enforce it.11:51
aspiersAJaeger: OK thanks!11:51
*** rh-jelabarre has joined #openstack-infra11:51
openstackgerritAdam Spiers proposed opendev/git-review master: Add missing upper-constraints.txt to deps in docs testenv  https://review.opendev.org/66173611:55
*** factor has joined #openstack-infra11:57
*** psachin has joined #openstack-infra11:57
*** rfolco has joined #openstack-infra11:59
*** electrofelix has joined #openstack-infra12:02
*** markvoelker has joined #openstack-infra12:06
*** yamamoto has joined #openstack-infra12:08
*** rf0lc0 has joined #openstack-infra12:09
*** rfolco has quit IRC12:10
*** otherwiseguy_ has joined #openstack-infra12:10
*** rfarr has joined #openstack-infra12:12
*** priteau has quit IRC12:13
*** priteau has joined #openstack-infra12:14
*** yamamoto has quit IRC12:17
*** rlandy has joined #openstack-infra12:26
*** _erlon_ has joined #openstack-infra12:26
*** derekh has quit IRC12:27
*** chason has joined #openstack-infra12:30
*** rossella_s has quit IRC12:32
*** rf0lc0 has quit IRC12:34
*** rfolco has joined #openstack-infra12:34
*** rf0lc0 has joined #openstack-infra12:35
*** rfolco has quit IRC12:38
openstackgerritAdam Spiers proposed opendev/git-review master: Build docs using Python 3  https://review.opendev.org/66175012:48
evrardjpis there an infra-root which can help me on the oudated mirror.opensuse? ianw gave me a first hint, but I would need some extra help.12:48
fricklerevrardjp: dirk: seems the opensuse mirror update job is failing and thus not updating the mirror, this is the latest error: http://paste.openstack.org/show/752150/12:52
fungiwe can try removing those from the local tree and see if it's able to sync now, though if they're still present in the remote tree then it will just continue doing that12:55
fungipretty sure i determined that error was about the directories on the remote end anyway, though it's been months since i last looked into it12:56
*** derekh has joined #openstack-infra12:57
fungilooking back through my notes, last time it was tumbleweed which we couldn't update due to stale .~tmp~ dirs12:58
*** igordc has joined #openstack-infra12:59
evrardjpfrickler: dirk is on holiday12:59
evrardjpbut it seems the mirror is correct, and has the right files13:00
*** dave-mccowan has quit IRC13:00
evrardjpso I am thinking those `.tmp` files are an artifact.13:00
fungiif you're browsing the mirror through a webserver it will likely hide anything starting with a .13:00
evrardjpfungi: would it make sense to add .tmp to excludes then?13:00
evrardjpfungi: I just checked with rsync to list files, it doesn't list it13:01
evrardjpbut my rsync foo is low13:01
fungievrardjp: i can't remember if we tried that before, but to start with i'll delete the local ones and see if they reappear13:01
evrardjpfungi: that would be awesome13:01
*** yamamoto has joined #openstack-infra13:02
openstackgerritAdam Spiers proposed opendev/git-review master: Change StoryBoard URL to be a permalink  https://review.opendev.org/66145113:06
fungievrardjp: same error. looks like the mirror we're being directed to is mirror.wdc1.us.leaseweb.net so maybe there's more than one and this one is broken?13:07
evrardjpwell13:07
openstackgerritAdam Spiers proposed opendev/git-review master: Fix complimentary typo  https://review.opendev.org/66145313:07
evrardjpleaseweb is used only for product and updates13:07
evrardjpthe OBS repos are supposed to be sync on provo-mirror13:08
evrardjpI thought the error was obs only13:08
*** jchhatba_ has quit IRC13:08
evrardjpdo you mean there is another issue somewhere?13:08
*** zul has joined #openstack-infra13:08
*** jchhatba_ has joined #openstack-infra13:08
fungiahh, we're not getting it out of a round-robin or dns glb, it looks like we set it explicitly13:09
*** ekultails has joined #openstack-infra13:09
evrardjpoh interesting I just re-did a rsync on that folder, and I get a .~tmp~ now13:09
fungiand an i/o error out of rsync?13:10
evrardjprsync saying permission denied on said ~tmp~ file13:11
fungiyep13:12
fungiso same error basically13:12
evrardjpworks if I have the exclude.13:12
evrardjp(ofc)13:12
evrardjplet me add this to the mirroring, if you don't mind.13:13
evrardjpAnd see how it goes13:13
evrardjpit can be an artifact from our side of the mirroring, but I don't see a valid reason to sync those files13:13
evrardjp(identically I don't see a reason to NOT clean up on our side at the same time)13:13
evrardjpfungi: you notes about tumbleweed, are they about obs repos?13:14
evrardjpor anything mirrored out of opensuse.org directly?13:14
*** mriedem has joined #openstack-infra13:15
fungichecking13:15
*** jpena|lunch is now known as jpena13:16
fungilooks like we were getting /afs/.openstack.org/mirror/opensuse/tumbleweed/repo/oss/i686/.~tmp~ and similar for other architectures13:16
fungiat the time dirk decided it was okay if tumbleweed mirroring went stale from time to time and so we adjusted the script to just ignore tumbleweed mirror errors and still mirror the other distributions in the script13:17
*** rfarr has quit IRC13:18
*** beekneemech is now known as bnemec13:18
fungiand yeah, i was misreading the server motd in the log. we're using mirror.wdc1.us.leaseweb.net for other (working) stuff there, but the errors are coming from a server which announces itself as stage.opensuse.org13:19
openstackgerritJean-Philippe Evrard proposed opendev/system-config master: Exclude tmp files artifacts from syncing  https://review.opendev.org/66175213:20
*** priteau has quit IRC13:20
*** aaronsheffield has joined #openstack-infra13:20
evrardjpfungi: ok let me check on the suse side about stage.opensuse.org13:20
evrardjpbut where does this come from though?13:20
*** tosky__ has joined #openstack-infra13:21
evrardjpI don't see any reference to stage from the mirror script13:21
*** tosky is now known as Guest7207313:21
*** tosky__ is now known as tosky13:21
evrardjpfungi: on the patch just above I prevented the syncing from those files all over the place ^13:21
evrardjpfungi: oh I guess the motd of the mirror on provo?13:22
*** yamamoto has quit IRC13:23
fungievrardjp: in the motd when the script connects to provo-mirror.opensuse.org, this is what it says: http://paste.openstack.org/show/752156/13:23
fungiso, yes13:23
evrardjpyes got it :)13:23
*** yamamoto has joined #openstack-infra13:24
fungii can hand-apply 661752 and see if it works around this13:24
evrardjpthat would be awesome to see if this works13:24
evrardjpbecause the files will still be there13:24
*** psachin has quit IRC13:24
evrardjpas I said, my rsync foo is bad :p13:24
rosmaitafrickler: could you take a look at https://review.opendev.org/#/c/652711/2 when you have a few minutes? I think I addressed your concerns.  Thanks!13:26
*** sreejithp has joined #openstack-infra13:26
fungievrardjp: i've removed the local .~tmp~ dirs in afs and started a copy of the opensuse-mirror-update script patched with 66175213:28
evrardjpso it ... seems to work?13:29
evrardjplet's see13:29
evrardjpI will still clarify on the stage./provo-mirror. side13:29
fungilooks like it's copying stuff from there now. already finished /afs/.openstack.org/mirror/opensuse/repositories/Cloud:/OpenStack:/Rocky/openSUSE_Leap_42.3/13:30
evrardjpAs long as I am getting 15.1 mirrored, I am just fine :D13:30
evrardjpI am surprised this blocks the rest of the mirroring though, as obs seem lower in prio than the rest13:31
evrardjpto me at least :p13:31
evrardjpbut at the end I prefer knowing if something is wrong13:31
*** rf0lc0 is now known as rfolco13:32
fungilots of package names with lp151 in them now13:32
*** sreejithp has quit IRC13:32
evrardjp"in them"?13:33
fungiin the version strings13:33
funginoarch/cassandra-kit-3.11.3-lp151.2.1.noarch.rpm13:33
fungiet cetera13:33
evrardjpok, not in 42.3 or something13:33
evrardjpsounds good13:33
evrardjpsounds great, even!13:33
fungiand yeah, the script short-circuits if an error is encountered mirroring any of those except tumbleweed, but perhaps the sequence should be rearranged13:34
evrardjpI like to know when something is wrong though13:35
evrardjpI think "the system works"13:35
rosmaitafungi: when you have a few minutes, let me know what you think of my last comment on https://review.opendev.org/#/c/655451/1  ... thanks!13:36
fungiyou bet13:36
AJaegerfungi, if you're reviewing, please look at https://review.opendev.org/661733 as well - thanks13:38
*** ginopc has joined #openstack-infra13:39
*** DinaBelova has joined #openstack-infra13:42
fungirosmaita: the devstack change mentions that setting periodic_interval "may have unintentional adverse effects" but i can't find where any of those are enumerated13:42
rosmaitafungi: there's a reference in the bug to another bug that explains: https://bugs.launchpad.net/cinder/+bug/182374813:44
openstackLaunchpad bug 1823748 in Cinder "periodic_interval used in too many places" [Undecided,Fix released] - Assigned to Brian Rosmaita (brian-rosmaita)13:45
*** priteau has joined #openstack-infra13:46
*** rf0lc0 has joined #openstack-infra13:46
fungiaha, thanks13:46
*** rfolco has quit IRC13:48
corvusttx: can you take a look at https://review.opendev.org/66138513:50
fricklerAJaeger: neutron-functional job is failing on ds-gate because tox is running in the ds-gate repo instead of neutron. not sure what changed, this doesn't seem to have been broken for long. http://logs.openstack.org/51/655451/1/check/neutron-functional/d195967/ara-report/result/1d6232f6-e3f7-45b2-bb64-9da99f3fbbd0/13:50
fricklerrosmaita: added to my list, thanks for the reminder13:50
rosmaitafrickler: thanks, appreciate it13:51
*** Goneri has joined #openstack-infra13:51
AJaegerfrickler: strange, does not ring a bell13:53
*** jchhatba_ has quit IRC13:55
AJaegerfrickler: https://opendev.org/openstack/neutron/src/branch/master/.zuul.yaml#L137 sets it - according to git blame no recent changes13:57
*** rf0lc0 is now known as rfolco13:57
AJaegerfrickler: oh, you say the chdir is wrong? That is really strange ;(13:58
AJaegerfrickler: better ask corvus ^13:59
AJaegerfrickler: if you have time, could you review  https://review.opendev.org/661733 , please?13:59
corvuso/14:00
fricklerAJaeger: corvus: actually I think this change by slaweq may be the culprit, maybe one of you can help fixing it https://review.opendev.org/61974214:02
*** rpittau is now known as rpittau|afk14:08
slaweqfrickler: what I broke this time? :)14:10
corvusfrickler, slaweq: i think if you set 'zuul_work_dir: "{{ devstack_base_dir }}/neutron" that would do the trick.  see https://zuul-ci.org/docs/zuul-jobs/python-roles.html#rolevar-tox.zuul_work_dir14:10
*** yamamoto has quit IRC14:11
*** yamamoto has joined #openstack-infra14:14
*** yamamoto has quit IRC14:14
*** yamamoto has joined #openstack-infra14:15
openstackgerritMerged openstack/project-config master: Move x/monitorstack jobs in-tree  https://review.opendev.org/66173314:15
mordredinfra-root: I'm here, but have a stack of meetings/phone calls - so will be slow to respond14:17
openstackgerritMerged opendev/system-config master: Exclude tmp files artifacts from syncing  https://review.opendev.org/66175214:18
*** yamamoto has quit IRC14:19
*** yamamoto has joined #openstack-infra14:20
*** yamamoto has quit IRC14:20
*** yamamoto has joined #openstack-infra14:20
corvusmordred: ack; i'm going to try to work on that image job testing project today14:24
corvusand i assume we'll switch the load balancer algo when clarkb comes online14:24
mordredcorvus: ++14:25
*** yamamoto has quit IRC14:25
*** chandankumar is now known as raukadah14:25
mordredduring the moments where I'm not in a meeting I'm going to continue working on the status.o.o transition14:25
mordredspeaking of: https://review.opendev.org/#/c/661385/1 <-- ttx do you have any opinion on this?14:26
corvusmordred: yeah, i +2d your second patch and will be happy to +2 that if ttx okays it14:27
*** dpawlik has quit IRC14:29
*** lpetrut has quit IRC14:30
*** kjackal has quit IRC14:33
*** kjackal has joined #openstack-infra14:33
*** quiquell has quit IRC14:34
*** ykarel has quit IRC14:35
openstackgerritTim Burke proposed openstack/project-config master: Rename x/pyeclib and x/liberasurecode to openstack/*  https://review.opendev.org/65715514:39
*** lseki has joined #openstack-infra14:40
corvusmordred: https://review.opendev.org/640027 has 2x2; not sure if we want to +W it today14:43
openstackgerritTobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check  https://review.opendev.org/64455714:44
clarkbcorvus: mordred I'm probably 40 minutes from keyboard, but ya want to change to source balance method, send out meeting agenda (late) , then start making sure we areready for friday renames14:44
mordredclarkb: you're a friday rename14:44
corvusclarkb: ack.  i should be around in a supporting role for the lb change14:45
corvuswe have data at http://grafana.openstack.org/d/jlskTlZZz/opendev-load-balancer?orgId=114:45
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132714:49
*** ykarel has joined #openstack-infra14:53
*** lpetrut has joined #openstack-infra14:59
*** dklyle has joined #openstack-infra14:59
*** udesale has joined #openstack-infra15:00
*** e0ne has quit IRC15:05
*** yboaron_ has quit IRC15:07
* fungi is also happy to help with the haproxy stuff today15:09
*** roman_g has quit IRC15:11
*** ykarel is now known as ykarel|away15:13
clarkbis there a change yet for the balance method swap?15:15
corvusnot that i'm aware of15:15
clarkbk writing one15:15
*** michael-beaver has joined #openstack-infra15:15
*** tosky has quit IRC15:16
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132715:17
*** hamzy has quit IRC15:18
openstackgerritClark Boylan proposed opendev/system-config master: Switch git lb to source balance method  https://review.opendev.org/66178315:18
clarkbthere we go15:19
*** rfarr has joined #openstack-infra15:19
openstackgerritSlawek Kaplonski proposed zuul/zuul-jobs master: Add role to fetch journal log from test node  https://review.opendev.org/64373315:22
*** rfarr_ has joined #openstack-infra15:23
mtreinishrandom doc8 question, is there a way to tell it to ignore line length for tables?15:24
fungiAJaeger: ^ you may know15:24
clarkbfungi: https://review.opendev.org/#/c/661783/ is the change to swap balance methods15:25
fungiyep, saw it, thanks!15:25
*** rfarr has quit IRC15:26
openstackgerritSlawek Kaplonski proposed zuul/zuul-jobs master: Add role to fetch journal log from test node  https://review.opendev.org/64373315:26
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132715:36
clarkbmeeting agenda sent. It includes a link to the open project-rename changes. infra-root if you can review those today and tomorrow that would be great. Particularly the one from mordred for opendev15:37
*** ykarel|away has quit IRC15:38
*** lpetrut has quit IRC15:38
clarkbhttps://review.opendev.org/#/q/topic:project-rename+status:open is the link15:38
AJaegermtreinish: only for tables? Not that I know off15:38
AJaegermtreinish: have a change for me to review?15:38
clarkbAJaeger: also ^ you are our project creator review maestro. Please let us know if you have any concerns too. I'll be reviewing them myself this morning I hope15:39
fungii find it interesting that the http/https frontend connection rate doesn't really seem to fluctuate between weekend and weekday loads (current sessions goes up a bit as does session time)15:39
AJaegerclarkb: will go over them...15:40
clarkbfungi: I'm guessing a good chunk of it is bots15:41
corvus> 50% of the rename changes have zuul-115:42
fungiclarkb: yeah, i mean zuul caches aggressively but i expected third-party ci systems to show more prominently15:42
corvus= 50% if you limit to p-c15:42
clarkbcorvus: I think that is due to updatings in zuul's main.yaml that won't take effect until after the changes merge?15:43
clarkbI want to say last time we did renames like this we force merged to get around that?15:43
corvuswe could split the changes15:44
clarkbya I think we discussed this a bunch ~6 months ago and decided it was easiset to just force merge then we don't have to do a dance of changes merging before and after gerrit downtime. fungi do you recall?15:45
fungihonestly i was trying and failing to remember15:45
fungibest i can hope for is we find a maintenance plan or transcript of the last rename before the opendev transition15:46
corvusi think the big question about the opendev change is whether zone-zuul-ci.org should be in zuul/ or opendev/15:46
AJaegerI'm giving +2 if those are correct as endresult - if we want to split, I'll remove my votes...15:46
mordredcorvus: ++15:46
corvusi'm fine with not splitting15:47
mordredcorvus: I can make arguments in both directions, so I believe I'm in support of either15:47
fungicorvus: my argument for opendev is that we want to have control over making sure it runs the jobs which test for broken configuration, snice that could knock the silent master offline if someone approved a broken bind zone config15:48
corvusyeah, i also don't have a strong opinion.15:48
fungigranted, it's a weak risk15:48
clarkbfungi: fwiw we have approved and merged broken zones recently (also quickly cleaned up)15:48
fungiyep!15:48
corvusfungi: yes -- though we can still do that with a config-project project stanza15:48
*** hamzy has joined #openstack-infra15:49
mordredI think I lean slightly towards opendev, because it's an operational repo that drives the configuration of services the opendev admins are responsible for15:49
fungialso ianw has added a test that bind can parse the zonefiles now15:50
corvusit's clear to me that it's a "shared jurisdiction" repo, so should always be a collaboration between opendev and zuul; so for me the question is more of where is its "home"?  is it part of opendev shared with zuul, or part of zuul shared with opendev?15:50
openstackgerritMatt McEuen proposed openstack/project-config master: Add granular ACLs for Airship Projects  https://review.opendev.org/66046115:51
fungianother litmus test: are there likely to be records where a zuul maintainer is likely to know what should be in that record when an opendev sysadmin won't? how about the other way around?15:51
mordredI *think* I think it's part of opendev shared with zuul - because the primary data contained in it is information that stems from other operational things (like, the IP address for a given service is an IP address managed by opendev, and verifying that a given IP maps to a given server is a thing that opendev folks would be doing)15:52
mordredI think I may have said something similar to fungi except with more useless words15:52
corvusfungi: i suspect not, as long as zuul continues to be hosted on opendev.  i think the point at which that changes would be if zuul were to pack up and leave opendev, at which point, i rather think the name of the repo *in opendev* doesn't matter anymore.  :)15:52
clarkbmordred: fungi: that makes sense to me for why it should be in opendev/15:53
fungiyeah, if zuul departs from opendev it will almost certanily take its dns hosting with it15:53
corvussounds like we're all leaning toward opendev/  so mordred, want to respin that?15:53
mordredcorvus: sure15:53
fungiif we decide it's safe, we can always add folks from related projects as core reviewers on a zone repo15:54
corvusyep15:54
fungiso that record changes can be made without opendev sysadmins holding that up15:54
fungi(say, if we're short on review bandwidth and there's an urgent change which needs to go through)15:55
AJaegerdo we really want to retire opendev-website? Then let's do the usual retirement steps (remove content etc) as well15:56
clarkbre finding previous rename notes, the last set of projects we would've renamed included shade -> sdk stuff15:56
clarkbmordred: ^ do you recall what the specific renames were there or did we not end up doing them with shade/sdk beacuse of the code merge?15:57
*** kopecmartin is now known as kopecmartin|off15:58
* clarkb is doing wiki page history forensics. Not as easy as with git15:58
fungimanual bisection with the start/end radio buttons is what i usually end up doing15:59
mordredclarkb: I don't remember anything specific15:59
openstackgerritMonty Taylor proposed openstack/project-config master: Rename opendev projects we missed  https://review.opendev.org/65727715:59
mordredhow's that look now?15:59
fungiclarkb: granted, it helps a lot if people get in the habit of leaving commentary metadata for any nontrivial wiki edit15:59
AJaegermordred: please move the two retirement repos out of that change15:59
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132716:00
AJaegermordred: those retirement is not done correctly and can be done independent16:00
clarkbfungi: ya thankfully it adds the section header as the metadata entry if you don't add something else16:00
fungiassuming people click the section edit link rather than the top-level page edit button16:00
AJaegermordred: -1 for the retirement - that is indeed not done properly16:01
*** Goneri has quit IRC16:01
clarkbhttps://etherpad.openstack.org/p/project-renames-2018-08-03 found em16:02
clarkbor at least a set of renames16:02
openstackgerritSorin Sbarnea proposed opendev/git-review master: Corrected git-review project organization  https://review.opendev.org/66180016:02
clarkband ya force merge changes seems to be on the list16:02
clarkband we do that before we stop gerrit so that they can replicate16:03
clarkbthese are good notes :)16:03
openstackgerritMonty Taylor proposed openstack/project-config master: Rename opendev projects we missed  https://review.opendev.org/65727716:03
*** lucasagomes has quit IRC16:03
mordredAJaeger: fixed - thanks!16:03
AJaegerclarkb: happy now with https://review.opendev.org/#/c/660461 ? could you review, please16:03
clarkbAJaeger: looking16:03
AJaegermordred: thanks. Want to send the retirement change as well?16:04
mordredsure16:04
clarkbAJaeger: done16:04
openstackgerritSorin Sbarnea proposed opendev/git-review master: Assure python3 is preferred by tox  https://review.opendev.org/66180216:05
AJaegermordred: commmented - needs a bit more work16:06
clarkbzbr: re ^ I thought tox defaulted to using the python version it was installed under for those envs that don't specify. Does putting python3 before python2 change that?16:06
AJaegerthanks, clarkb16:06
*** tesseract has quit IRC16:06
openstackgerritMonty Taylor proposed openstack/project-config master: Retire pbrx and opendev-website  https://review.opendev.org/66180316:07
zbrclarkb: yep, tox preferrence is based on the order mentioned in envlist. confirmed that. it was suggested to me by the maintainers some time ago16:07
mordredAJaeger: ah - poop. thanks16:07
fungizbr: as a substitute for setting defaultpython?16:08
zbrit is big ugly because this behavior is not self-describingf16:08
AJaegermordred: we retire repos in several steps: 1) send email; 2) add noop-jobs 3) remoe all content; 4) your change to rmeove it from gerrit and zuul16:08
clarkbzbr: neat I did not know about that feature16:08
zbrfungi: yep. alternative to basepython, and it also has additional feature.16:08
AJaegermordred: so, your change needs a bit more splitting... The infra-manual has full details16:09
AJaegerbbl16:09
zbrfungi: basepython=python3 may pick an unsupported python version on some platforms (like py35), but if you use preference list, you can tell tox to prefer "37,36,35,27"16:09
*** ykarel|away has joined #openstack-infra16:09
fungizbr: and tox 3.1.0 is the version which added that behavior?16:09
zbrfungi: nope, this feature was added very long time ago (i do not remember), 3.1 added the ignore_basepython_conflict16:10
*** ricolin_ has joined #openstack-infra16:10
fungiahh, okay16:10
zbrfungi: i adopted this approach recently as it gives a decent developer experience as I want to run linters using newest supported python version.16:12
zbras we know flake8 on py27 fails to pick lots of tests which are found under py3* versions. i guess soon we will see more tools dropping old interpreters, so picking this approach is likely to be better.16:13
fungizbr: so if i have a python3-only project, i don't actually need to set testenv.basepython=python3 as long as i have py3 (and no py27) in my tox.envlist? neat!16:13
openstackgerritMonty Taylor proposed openstack/project-config master: Rename opendev projects we missed  https://review.opendev.org/65727716:14
openstackgerritMonty Taylor proposed openstack/project-config master: Retire pbrx and opendev-website  https://review.opendev.org/66180316:14
zbrfungi: yep, try it. but remember to clean the envs. does decides that only for new envs. it will not recreate exiting ones.16:14
fungisure, or pass -r16:15
mordredAJaeger: nod. I think I got the rename patch more correct now - I'll work on the other steps for the retirement16:15
fungizbr: wow, this is a rather opaque magic, too bad it's not documented16:16
fungibut it's allowing me to clean up some personal projects, so... thanks for the pointer!16:16
zbrfungi: i agree with you, i am considering updating the docs to mention this. very useful feature which is very well hidden.16:16
openstackgerritJens Harbott (frickler) proposed openstack/project-config master: Drop neutron-functional job from devstack-gate  https://review.opendev.org/66180616:17
fricklerAJaeger: slaweq: corvus: after thinking a bit longer, I think this is the correct solution ^^. we could still fix the job if it needs to be run against other repos16:17
corvusfrickler: wfm16:17
corvusi think it was mostly there to test hooks, etc16:18
fungiand we still have a job which directly tests hooks anyway (with dummy hooks), if memory serves16:18
*** gyee has joined #openstack-infra16:20
mwhahahadid the yum-puppetlabs mirror sync change recently? it appears that we no longer have the el/ folder16:23
* mwhahaha tries to remember where that is configured16:25
openstackgerritMerged openstack/project-config master: Add granular ACLs for Airship Projects  https://review.opendev.org/66046116:26
*** emine__ has quit IRC16:27
fungimwhahaha: https://opendev.org/opendev/system-config/src/branch/master/modules/openstack_project/files/mirror/yum-puppetlabs-mirror-update.sh16:27
*** prometheanfire has quit IRC16:28
fungimwhahaha: last update was https://review.opendev.org/656912 which merged 2019-05-09 (several weeks ago)16:28
mwhahahayea there seems to be a desync between the rsync version and the published version16:29
mwhahahahttp://mirror.ord.rax.openstack.org/yum-puppetlabs/ vs http://yum.puppetlabs.com/16:29
*** prometheanfire has joined #openstack-infra16:29
mwhahahalooks like the rsync split off the older stuff to http://release-archives.puppet.com/yum16:29
fungii wonder if their breaking of puppet 4 deb repos has halted our mirroring of the rest16:29
mwhahahawhich is the bits that have broken the stable/queens puppet jobs16:30
fungichecking logs on the mirror-update server16:30
mwhahahacause we need puppet 4 :/16:30
AJaegerconfig-core, pleaes review another job removal: https://review.opendev.org/66154216:30
fungimwhahaha: have you been following https://tickets.puppetlabs.com/browse/CPR-680 ?16:31
fungiand subsequent https://tickets.puppetlabs.com/browse/CPR-68516:32
mwhahahano hadn't see them16:32
mordredmwhahaha: it looks like they deleted puppet4 for rpm just like they deleted puppet4 for deb - and I don't see puppet4 in release-archives16:33
*** ramishra has quit IRC16:33
mwhahahathey are there16:33
mordredthey are?16:33
mwhahahathey were pc116:33
mwhahahaso the pc1 stuff was ~puppet 416:33
fungilooks like they put it all back temporarily a week ago yesterdau16:33
mwhahahahttps://release-archives.puppet.com/apt/16:33
mordredwow. so puppetlabs-release-pc1-el-5.noarch.rpm is actually puppet4?16:33
mwhahahayea16:34
mwhahahahttps://review.opendev.org/#/c/661787/1/functions16:34
mordredwow16:34
mwhahahathat's teh logic we had for 4 vs 516:34
mwhahahaPC1 = 416:34
mwhahahathe problem we currently have is that the infra mirrors don't have the older stuff anymore cause it looks like they dropped it from the rsync mirror16:35
mordredyah.16:35
fungilast update before the weekend seems to be that they'll remove it again soon... "We will likely remove the release packages from the subdirectories before the archives are ready. I'll send an update next week when we have a better sense of the timeline." (from the google groups thread linked in the second cpr)16:35
mwhahahai guess we could just switch the puppet openstack stuff to use upstream mirrors for now, we don't have that many jobs anymore16:35
mwhahahaor patches at least for teh puppet 4 bits16:35
mwhahahathe 5/6 seem to be ok still16:35
* mwhahaha will patch later16:36
fungimwhahaha: aha, i see the problem. out of quota on that afs volume16:36
fungirsync: close failed on "/afs/.openstack.org/mirror/yum-puppetlabs/sles/11/PC1/x86_64/.puppet-agent-1.10.12-1.sles11.x86_64.rpm.EDZHao": Disk quota exceeded (122)16:37
mwhahahaoh well that'd do it too :D16:37
*** ginopc has quit IRC16:38
fungimwhahaha: first occurrence of that quota exceeded error was 2019-04-26 00:20:01 utc, for the record16:39
fungiso i'm guessing it's just over a month stale now16:39
clarkbits just going to break again when they undo the undo right?16:39
clarkbI mean puppet-4 specifically16:39
openstackgerritMerged openstack/project-config master: Drop neutron-functional job from devstack-gate  https://review.opendev.org/66180616:39
clarkbso you probably want to be addressing that as well16:40
fungiyeah, unless what they said in that google groups thread is a typo, they're removing it again *before* they get similar package repos added on the new archive site16:40
clarkbfwiw we install the puppet-agent pc1 package directly and because they vendor everything they need into /opt/puppetlabs with that package it just works for us16:40
zbrfungi: it seems that my assumption about tox and preference list was wrong, it does not work. it seems that I was only lucky to get the desired behavior by chance. i will see if I can fix tox to do it.16:40
fungizbr: interesting, because it seems to be happening for me in local testing16:41
zbrlets see what we get on https://github.com/tox-dev/tox/pull/1329 ---16:41
zbrone way or another we should find a solution, maybe we will endup with a new config option, one that is more self-describing.16:42
fungizbr: oh, actually i think it's working on my local py3-only projects because i'm running a tox which is installed under python316:43
zbrfungi: yeah. same happening with me.16:44
fungialso after some correction of my test case, i see that it's using the python3.7 my tox was installed with even though py38 is first in my list16:44
clarkbfungi: ya that was the behavior I knew about16:45
clarkbbalance method change is in the gate now16:47
mtreinishAJaeger: sry missed your response earlier, it's on: ff16:47
mtreinishAJaeger:  https://github.com/Qiskit/qiskit/pull/307 (wrong paste buffer don't know where ff came from)16:48
*** dpawlik has joined #openstack-infra16:48
mtreinishthe travis failure is doc8 complaining about line length on the table: https://travis-ci.com/Qiskit/qiskit/jobs/203592013#L602 trying to figure out how to proceed there, because I don't think you can line wrap that16:49
*** kjackal has quit IRC16:49
*** armax has quit IRC16:50
fungimtreinish: in that particular example, the first column heading could likely be wrapped onto a second line16:50
* clarkb finds breakfast16:51
* fungi assumes that's valid rst anyway16:52
mtreinishfungi: heh perhaps, I didn't try that. I'll give it a go. But that'll make it kinda hard to read though16:53
mtreinishalso how do I format the width for the columns if they overlap16:54
mtreinishoh, nm I misread what you said16:54
fungii meant wrap the text within the cell16:54
mtreinishcolmn not rwo16:54
mtreinishthat is valid rst16:54
fungior find a way to effectively abbreviate that first column heading, though i'm sure you've already made it as brief as possible16:55
mtreinishyeah I could change metapackage to meta, but wrapping that probably would be the easiest16:55
fungibut yeah, it's the classic "my headings are much longer than the data below them" problem16:56
mtreinishfungi: :/ line wrapping the column header is still 14 characters too long...16:59
*** mattw4 has joined #openstack-infra17:00
*** derekh has quit IRC17:00
*** ricolin_ has quit IRC17:00
zbrfungi: clarkb please check https://github.com/tox-dev/tox/issues/1330 and comment on it, it may be important to state the importance of this feature to tox maintainers, before they close the feature request due to "deterministic" excuse.17:04
*** Goneri has joined #openstack-infra17:04
openstackgerritMerged opendev/system-config master: Switch git lb to source balance method  https://review.opendev.org/66178317:06
clarkband now we watch and wait17:07
fungiso... the /afs/.openstack.org/mirror/yum-puppetlabs volume has a 100gb quota currently and is using all of it. i wonder if we're mirroring far too much there17:08
clarkbzbr: I'm probably the wrong person to comment on that as my platforms all have a single python2 and single python3 version17:08
clarkbso I'm happy with setting basepython to python317:08
clarkbfungi: or maybe we aren't deleting the stuff they deleted?17:08
clarkbso have duplicate type datA?17:08
fungiwe're passing --delete17:11
clarkbre haproxy update we last ran ansible there at about 16:55UTC so I expect in about 20-30 minutes we'll get our balance method updated17:12
*** Weifan has joined #openstack-infra17:12
clarkbzbr: looks like they have some good ideas so we'll probably get something that works17:14
clarkb(re your github tox bug)17:14
zbrclarkb: if you happen to run centos 7 you may be not so happy because on it default python3 is py34 which was dropped by many python projects.17:14
AJaegermtreinish: use a list-table, see https://docs.openstack.org/doc-contrib-guide/rst-conv/tables.html#list-tables17:16
zbrclarkb: glad to read the last comment17:17
*** irclogbot_3 has quit IRC17:17
*** RobertNordix has quit IRC17:17
*** irclogbot_3 has joined #openstack-infra17:18
fungiaccording to du the breakdown of top-level directories in the yum-puppetlabs mirror is 36gb for el, 27gb for puppet5, 16gb for puppet6, 15gb for fedora, 2gb for sles and 2gb for puppet-enterprise-tools17:18
mtreinishAJaeger: ok cool, thanks. That should do the trick17:19
mtreinishI also just switched from a grid table to a simple table and it fit in the character limit too17:20
mtreinishah, but it didn't format the line wrap for the first column header properly. So list table it is17:21
fungiwe could probably stand to exclude a lot of stuff in this yum-puppetlabs mirror... el/5 and el/6 together use 26gb of space17:24
fungiif you add puppet5/el/5 and puppet5/el/6 that's another 13gb17:25
*** sreejithp has joined #openstack-infra17:25
*** armax has joined #openstack-infra17:25
*** jpena is now known as jpena|off17:26
clarkbfungi: we (infra) never used those mirrors so no worries removing stuff from our side17:27
clarkbwould come down to puppet-openstack and tripleos needs I think17:27
fungii doubt they're running centos5 or centos6 jobs17:28
rpiosohodgepodge: When you have a chance, please review https://review.opendev.org/#/c/660534/17:29
clarkbfungi: good point17:30
fungipuppet6/el/5 and puppet6/el/6 are smaller, but also pointless17:31
fungi5gb together17:31
fungialso fedora/f20 through... fedora/27?17:32
rpiosohodgepodge: Please be aware that dtantsur, fungi, TheJulia, and I had a discussion about it in #openstack-ironic this past Friday. It starts at http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2019-05-24.log.html#t2019-05-24T14:23:2817:32
clarkbcorvus: fungi mordred I think we may have to manually restart haproxy? I don't think the service was updated after the template was updated (though I'm trying to find where the file is on disk to confirm it actually updated)17:33
rpioso^ dtantsur|afk17:33
fungiclarkb: oh, probably... though i'm about to pop out to get some groceries so will probably disappear for 30-45 minutes17:34
*** igordc has quit IRC17:34
*** ykarel|away has quit IRC17:34
*** ykarel|away has joined #openstack-infra17:35
clarkbyup the file is updated17:35
clarkbnow to see if admin socket will tell me what balance method it is using17:35
fungiokay, bad timing but have to coordinate with my wife's schedule for shopping so stepping out for the next few minutes but should be back well before the meeting17:43
clarkbthe way you gracefully restart haproxy is to sigusr1 it which tells the running haproxy to keep running until existing connections stop. You are expected to already have a new haproxy running17:47
clarkbso I think our best bet here is to docker-compose restart and take a small service restart hit17:47
*** electrofelix has quit IRC17:49
clarkbI may as well pop out for a bit too if others are afk. Back in a bit well before the meeting17:49
*** igordc has joined #openstack-infra17:55
*** igordc has quit IRC17:55
*** igordc has joined #openstack-infra17:55
*** priteau has quit IRC17:56
*** Weifan has quit IRC17:57
*** Weifan has joined #openstack-infra17:57
corvusclarkb: was in meeting; maybe after meeting would be best17:59
*** Weifan has quit IRC18:01
*** ykarel|away has quit IRC18:01
clarkbk18:06
*** efried has quit IRC18:17
*** efried has joined #openstack-infra18:18
*** rfarr_ has quit IRC18:27
*** jesusaur has quit IRC18:30
* rpioso realizes he incorrectly entered hogepodge's nic18:30
openstackgerritTobias Henkel proposed zuul/zuul master: Add script to pre-populate private keys  https://review.opendev.org/66182418:31
rpiosohogepodge: When you have a chance, please review https://review.opendev.org/#/c/660534/18:32
*** imacdonn has quit IRC18:33
rpiosohogepodge: Please be aware that dtantsur|afk, fungi, TheJulia, and I had a discussion about it in #openstack-ironic this past Friday. It starts at http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2019-05-24.log.html#t2019-05-24T14:23:2818:33
*** rlandy is now known as rlandy|brb18:33
clarkbrpioso: I think hogepodge is travelling so it may be a day or two18:33
rpiosoclarkb: Thank you!18:34
fungirpioso: out of curiosity, why are you thinking he'll have an interest in that?18:34
fungii probably missed an ml thread somewhere with the tie-in18:35
rpiosofungi: I believe he's involved with the OpenStack Marketplace.18:35
rpiosoAt least, so I've been told.18:35
fungiahh, yeah he may have been helping on some of the marketplace ideas18:36
rpioso:-)18:36
fungiJpMaxMan, jamesmacarthur, smarcet et al are the ones managing the implementation for it though18:37
fungiand pretty sure they're all aware of the earlier plan thingee was driving to standardize driver matrices in individual service projects18:37
fungibut we should definitely loop them back in once work resumes on that18:38
*** rfarr has joined #openstack-infra18:39
rpiosofungi: Thank you for that info. I wasn't previously aware of their involvement.18:39
*** hwoarang has quit IRC18:41
*** hwoarang has joined #openstack-infra18:41
rpiosofungi: Do you have any insight when it might resume?18:41
*** Weifan has joined #openstack-infra18:42
openstackgerritMatt McEuen proposed openstack/project-config master: Rename request: airship-in-a-bottle  https://review.opendev.org/65600818:44
AJaegerfungi, could I trouble you with reviewing a small cleanup, please? https://review.opendev.org/#/c/661542/18:45
AJaegerteam, do we want to switch rax.dfw mirror to opendev? See https://review.opendev.org/661187 by ianw18:45
fungirpioso: i think it needs a critical mass of people interested in collaborating on a standard, with input from maintainers of multiple openstack services18:46
fungithis could be a target for an openstack cycle goal, maybe in the "u" (unnamed) cycle18:46
rpiosofungi: When the time comes, I'm interested in contributing some not so critical mass ;-)18:47
*** Weifan has quit IRC18:47
*** lpetrut has joined #openstack-infra18:47
dirkfungi: thanks for the notification about the mirror failure. For how long is it failing already? I was assuming those ~tmp~ dirs are temporary18:48
fungidirk: i'm not sure how long it was failing for, evrardjp may have a better idea, or i can try to dig it out of the logs when i get a moment18:49
clarkbpasting yaml into etherpad is not a fun time18:50
fungirpioso: cool, getting some voices from the openstack tc behind such an effort could help. maybe could be organized as a "pop-up team" https://review.opendev.org/66135618:50
dirkfungi: also note that we're trying to get rid of mirroring alltogether via https://review.opendev.org/#/c/658855/18:50
evrardjpdirk: almost 2 days, as far as I can see. Go back to vacationing though ;)18:50
dirkUnfortunately reviews on that one are missing18:51
*** kjackal has joined #openstack-infra18:51
*** lpetrut has quit IRC18:51
evrardjpdirk: that would be good too. I needed to go forward to unblock some gates though.18:51
*** raissa has joined #openstack-infra18:53
*** rlandy|brb is now known as rlandy18:53
dirkevrardjp: I am still vacationing.. when we get the lxc 42.3 jobs removed we could stop mirroring that repo18:54
dirkI wonder if the ~tmp~ means that somebody deleted the repository18:54
dirkI'll check18:54
clarkbAs a friendly reminder we will have our weekly infra team meeting in a few minutes over in #openstack-meeting18:57
openstackgerritMerged zuul/zuul-jobs master: validate-zone-db : add job and make more generic  https://review.opendev.org/66113818:58
*** raissa has quit IRC18:58
rpiosofungi: A pop-up team sounds like a reasonable structure for that effort.19:00
fungibasically the pop-up team could drive an initial pilot in a few projects and then could dissolve once a cycle goal to repeat that pattern in all openstack projects gets approved19:01
fungi(and some folks from the pop-up team might become goal champions on that effort)19:02
*** eernst has joined #openstack-infra19:13
*** eernst has quit IRC19:25
*** takamatsu has joined #openstack-infra19:28
*** pcaruana has quit IRC19:30
openstackgerritTobias Henkel proposed zuul/zuul master: Annotate logs around build completion and cancellation  https://review.opendev.org/66080619:32
openstackgerritTobias Henkel proposed zuul/zuul master: Annotate logs around build states  https://review.opendev.org/66148919:32
openstackgerritTobias Henkel proposed zuul/zuul master: Annotate logs around reporting  https://review.opendev.org/66149019:32
openstackgerritTobias Henkel proposed zuul/zuul master: Annotate logs around finished builds  https://review.opendev.org/66149119:32
*** Weifan has joined #openstack-infra19:34
*** Weifan has quit IRC19:39
openstackgerritAnkita Bansal proposed opendev/storyboard-webclient master: Provide -Assign to self- button for story tasks.  https://review.opendev.org/66183419:40
*** Weifan has joined #openstack-infra19:41
*** kjackal has quit IRC19:41
*** kjackal has joined #openstack-infra19:41
rm_workit looks like we need to have octavia's pyX jobs run with master of octavia-lib (with tox_install_siblings on)... is there an example for setting that up with the base python jobs somewhere?19:42
fungirm_work: infra meeting is underway right now, but i believe the oslo team runs a bunch of source jobs like that19:44
rm_workwondering if i have to remove python-openstack-jobs from our project, and make a whole new set of those jobs with the originals as their parent19:46
rm_workto add the tox_install_siblings thing19:46
rm_worki guess it isn't urgent, i'll keep working on the other bits of my patch that i need to fix first :D19:47
rm_workand see if i can figure this out later19:47
fungiyeah, may want to look into how neutron tests with neutron-lib too?19:50
rm_workyeah, so19:52
rm_workhttps://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L758 seems like it19:52
rm_workwhich does seem like they redo all the jobs19:52
rm_workwhich seems really tedious19:52
rm_workbut i suppose that's fine19:52
fungithey don't redo the jobs, they just parent the normal jobs and override a few parameters19:53
rm_workbut i really just wish there was a way to inject a couple of things into the existing runs without that hassle19:53
funginote those jobs declare no playbooks at all19:53
rm_workright i mean19:53
rm_workfor each job, they have to make an entry19:53
rm_workand parent the original + add stuff19:53
*** ralonsoh has quit IRC19:53
rm_workit's essentially 1:1 for all the jobs19:54
fungihow many do you need it in?19:54
rm_workevery job that runs tox19:54
fungiwow, even things like static code analyzers?19:54
fungidocs builds?19:55
rm_workwell, at least openstack-python-jobs, openstack-python36-jobs, openstack-python37-jobs, openstack-cover-jobs19:55
rm_workMAYBE some doc builds19:55
rm_workprobably not like, release-notes :D19:55
mordredthere was a discussion a while back about making it possible to add a required-projects list to a project pipeline defintion - but I don't believe anyone has even gotten as far as thinking about design, much less work on it19:56
*** imacdonn has joined #openstack-infra19:56
rm_worknot pep8 i guess either19:57
pabelangerfyi, fedora-28 is EOL today: https://fedoraproject.org/wiki/End_of_life19:58
fungirm_work: also, what is the releasability there... do octavia and octavia-lib get released at the same time, or is the lib released earlier? any concerns about testing that octavia works with the released version of octavia-lib at that point, or does its development get frozen until octavia releases?19:59
rm_worklib should be released earlier as it follows lib release rules19:59
fungiand then you avoid merging anything into the lib so you don't accidentally grow a dependence in octavia on some new feature in octavia-lib which won't be released when octavia is?20:00
clarkbpabelanger: ya we're likely to keep it around until centos 8 exists20:00
clarkbpabelanger: it is currently tripleo's python3 test platform20:00
rm_workisn't that standard library release policy?20:01
rm_workfreeze for libs is before freeze for projects20:01
fungirm_work: the alternative is to release the lib much more often and then only test against the released version so you don't have to be diligent about not depending on things in the lib which won't be released when the service releases20:01
clarkbinfra-root ok are we ready to docker-compose restart on opendev.org ? Any better more graceful ideas on how we could do that?20:01
*** takamatsu has quit IRC20:02
rm_workfungi: yeah but what we NEED to be able to do is Depends-On a patch in octavia-lib and have the tests pass20:02
rm_workmaybe I skipped something and this is a case of "back up to what I really need, not what I think I need"20:02
fungiif the change in octavia is backwards-compatible with released octavia-lib then running both jobs should be doable20:02
clarkbI can help drive the haproxy restart if so. But need to find lunch after20:02
corvusclarkb: wfm20:02
clarkbok I'm going to run `docker-compose restart` in /etc/haproxy-docker20:03
fungiclarkb: sounds good, thanks!20:03
openstackgerritMerged openstack-infra/zone-opendev.org master: Add zone-check job  https://review.opendev.org/66088820:03
openstackgerritMerged openstack-infra/zone-zuul-ci.org master: Add zone-check job  https://review.opendev.org/66096720:03
clarkbdone and opendev.org still loads for me20:04
rm_workfungi: well, if i have a patch that is dependent on octavia-lib change, but that change wouldn't be released without the thing that depends on it (it's a feature, no point in adding one without the other), then how does that work? people aren't going to merge and release a lib change that doesn't do anything without a feature that's still not +A, and people don't usually +A patches that are failing all their tests20:04
clarkbdoing some quick tests of puppet-keystone clones20:04
openstackgerritAnkita Bansal proposed opendev/storyboard-webclient master: project_group view: add number of active stories beside repo list  https://review.opendev.org/66183620:04
fungirm_work: "no point" may be a stretch... it does mean that people don't have to upgrade the service and the lib in unison if they're decoupled, but that's up to you20:05
rm_workfungi: for example this patch: https://review.opendev.org/#/c/660239/20:05
rm_workDepends-On a change in lib that enables it20:05
openstackgerritMerged openstack/project-config master: Switch rax.dfw mirror to opendev.org  https://review.opendev.org/66118720:05
clarkbgit clone --mirror https://opendev.org/openstack/puppet-keystone then git --git-dir ./puppet-keystone.git fetch origin --prune works for me now20:06
rm_workI guess we could just let the tests continue to fail and wait until people are "+2 pending lib merge", and then merge/release the lib change, and recheck20:06
clarkbnow to find lunch that start tackling the todo list post meeting. Back in a bit20:06
rm_workjust seems dirty when we should be able to have the normal Depends-On logic do its thing there20:06
corvusclarkb: i can see the discontinuity in the grafana graphs; i don't see any problems.20:06
corvusclarkb: though i can already see the imbalance in sesions20:06
rm_workmaybe it's possible to have this work in such a way that it tests against the normal release UNLESS depends-on is set?20:06
clarkbcorvus: ya I guess that is only really a problem if a single node can't handle what is hashed to it :/20:07
clarkbwe'll have to watch it over time I expect20:07
corvusgitea08=15 gitea01=1.520:07
*** mriedem has quit IRC20:07
fungirm_work: one scheme that works is to run both jobs (with normal released deps and with the lib from source) and then expect the normal version of the job to fail until the lib is released but the source version will succeed in the meantime to demonstrate the feature is viable (and can take advantage of depends-on before the lib implementation merges)20:14
rm_workhmm I guess I could do that and just run like, ONE20:14
rm_workmaybe the py37 one20:14
rm_workthat seems reasonable20:15
fungibut again, whether that's worth it to you probably depends on how tightly or loosely coupled octavia and octavia-lib are20:15
rm_workvery20:16
rm_worki mean, we basically just moved a ton of code directly from octavia to octavia-lib20:16
rm_workand then added a dep20:16
rm_workand again, per my understanding of the way the library-release-policy works, we are REQUIRED to freeze the lib first, release it first, and unfreeze it last20:17
fungiand if octavia-lib is going to intentionally undergo a feature freeze after milestone 3 until octavia releases, then you can probably get away with the source job only20:17
fungiyou may want to check with the release team on whether not running the normal unit test jobs is typical for a project in your situation20:18
*** hamzy has quit IRC20:21
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132720:22
fungirm_work: looking at a random recent neutron change, https://review.opendev.org/660785 shows that they're testing openstack-tox-py36 and not openstack-tox-py36-with-neutron-lib-master20:22
rm_workyeah it seems they only use the source versions as periodics20:23
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132720:24
fungirm_work: agreed... http://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-py36-with-neutron-lib-master20:24
fungiso maybe they defer merging neutron changes which depend on a neutron-lib feature until lib freeze20:25
fungibut again, they or the release team can likely tell you what's typical for this case20:26
fungiit's also possible to add an extra job in a do-not-merge child change to demonstrate the change working with the alternative dependencies, and can use depends-on when doing so20:27
fungispeculative execution of job configuration opens a lot of options20:27
fungi(for most jobs at least, you don't actually have to merge the job to run it)20:28
fungii see people take advantage of that with some frequency20:29
*** sreejithp_ has joined #openstack-infra20:30
*** dpawlik has quit IRC20:31
*** sreejithp has quit IRC20:31
*** hwoarang has quit IRC20:37
*** hwoarang has joined #openstack-infra20:38
rm_workhmmm20:38
rm_workyeah... I just... feel like "Depends-On" should *work*20:38
rm_workbecause it's kinda lame to have to always do that20:38
clarkbinfra-root https://etherpad.openstack.org/p/EkjfrnSIud is my draft gerrit project renaming announcement20:39
clarkbif I can get a couple eyeballs that aren't my own to look at that and ACK it I'll send it on its way20:40
fungirm_work: depends-on should work if you're always installing the dependency from source, yes. what you don't want is to install the dependency from released artifacts most of the time and then install from source when there's a depends-on, because once that change with the depends-on merges all your other changes start failing the job20:41
fungi(at least until the feature you were depending on gets released)20:42
*** takamatsu has joined #openstack-infra20:42
*** ekultails has quit IRC20:42
fungiclarkb: lgtm, thanks!20:43
corvusclarkb: lgtm.  also, maybe we should set up an announce list.20:44
clarkbcorvus: probably not a bad idea20:44
*** xek_ has quit IRC20:44
clarkbI can get a change up for one though I won't wait on that to send this email (as it will take people time to get subscribed)20:45
*** gfidente is now known as gfidente|afk20:45
corvusya20:47
fungiagreed20:48
fungiopendev-announce@lists.opendev.org?20:49
fungiservice-announcements@20:49
*** slaweq has quit IRC20:49
fungioh the bikeshedding we will have20:49
*** mriedem has joined #openstack-infra20:51
mordredfungi: blue@lists.opendev.org20:51
fungii prefer cerulean@20:51
mordredfungi: cobalt@ is also nice20:52
clarkbemails sent20:52
fungipantone-15-4020@20:52
corvushey, no proprietary colors20:53
fungid'oh!20:53
mordredif we call it 9bb7d4@ it's no longer proprietary right?20:54
openstackgerritClaudiu Belu proposed openstack/project-config master: Rename x/compute-hyperv to openstack/compute-hyperv  https://review.opendev.org/66168220:55
clarkbresponded to tripleo/rdo git clone thread asking for some retries20:55
paladoxheh cobalt is what we call our gerrit server :)20:55
fungia company named cobalt used to make servers. i had a few20:55
fungisome were mips-based too20:56
fungi1 rack unit mips server, pretty handy when you need that particular architecture20:56
paladoxheh20:57
openstackgerritClark Boylan proposed opendev/system-config master: Add opendev service-announce list  https://review.opendev.org/66184320:57
clarkbcorvus: fungi ^ you can start bikeshedding there :)20:57
fungione of their nifty features was an integrated lcd display on the front which their bios knew how to drive so you could get the full server post through it by scrolling, and also adjust bios settings20:57
clarkboh sorry it was mordred and fungi bikeshedding :)20:58
*** e0ne has joined #openstack-infra20:58
clarkbcorvus was just making sure we didn't use proprietary bikeshed colors20:58
fungia reasonable concern20:58
fungialso, i figured out why wiki-dev isn't puppeting... it's not in the inventory... its replacement is wiki-dev02 and that *is* puppeting. on to its newest puppet errors now20:58
mordredfungi: that's exciting!20:58
fungiif you call "Package php-apc is not available, but is referred to by another package. This may mean that the package is missing, has been obsoleted, or is only available from another source. E: Package 'php-apc' has no installation candidate" exciting anyway21:01
rm_workfungi: yeah that's why i figured we'd just want to switch it all to from-source, so depends-on would work and it'd be consistent21:01
clarkbinfra-root remote:   https://review.opendev.org/661845 Record historical project renames21:01
*** Goneri has quit IRC21:01
clarkbI ended up putting it in opendev/project-config not openstack/project-config even though oepnstack/project-config has the projects.yaml file currently21:02
clarkbif we think that will creation unnecessary friction getting that data into its final resting place I can put it in the other repo instead21:02
fungione less thing to move, i suppose21:02
fungibut it does mean changes in two places for a rename in the interim21:02
fungis/places/repos/21:02
clarkbya though I think it is always going to be a separate change while we record just the repos.yaml input to the rename playbook and not add it straight ot projects.yaml21:03
clarkbI'll get a similar change up once I stack the rename changes. I wanted to make sure that where aren't any other updates we need to those before stacking them though. Looks like we got a new patchset on at least one of them that needed an update (I don't remember if there was more than that one)21:05
clarkbmaybe I'll give y'all today to review those then stack them tomorrow21:05
clarkbgitea08 seems to be the busiest of the backends over the last hour21:06
clarkband has the longest average session times21:06
clarkbcross checking against cacti cpu, load, and memory seem low to reaosnable21:07
clarkbso I don't think it is sad21:07
fungiokay, looks like in addition to php-apc i also have 8 different international font ttf-.* packages which are either renamed or dropped in xenial... working on that patch now21:08
*** dave-mccowan has joined #openstack-infra21:08
*** slaweq has joined #openstack-infra21:11
*** e0ne has quit IRC21:12
openstackgerritJeremy Stanley proposed opendev/puppet-mediawiki master: More transitional Xenial package name replacements  https://review.opendev.org/66184821:20
fungithere we go... that's the next step for wiki-dev xenialing21:20
* clarkb pulling up all these changes then will work on reviewing them21:20
fungifindnig what new names packages have transitioned to is decidedly harder now that the last release containing transitional dummy packages for the old names has gone eol and packages.ubuntu.com no longer searches it21:21
*** dave-mccowan has quit IRC21:23
*** slaweq has quit IRC21:24
clarkbmordred: did you want to self approve https://review.opendev.org/#/c/640027/ and watch it?21:25
clarkbthen we can get https://review.opendev.org/#/c/640044/ in after21:25
clarkb(I think corvus didn't approve because wanted someone else to watch it go in?)21:25
clarkbI'm about to do a proper review of tobiash's node usage statsd change in zuul so not quite able to watch it myself21:26
fungimanually trying out a potential set of excludes on the yum-puppetlabs-mirror-update script now to see if they work the way i expect, and then change on the way for that21:27
mordredclarkb: yes - although I'm stepping afk for a few minutes - I'll +A it when I get back so I can watch21:29
*** udesale has quit IRC21:31
*** smarcet has joined #openstack-infra21:36
*** tosky has joined #openstack-infra21:36
*** smarcet has quit IRC21:36
*** smarcet has joined #openstack-infra21:38
*** harlowja has joined #openstack-infra21:44
*** kjackal has quit IRC21:45
*** yamamoto has joined #openstack-infra21:45
*** yamamoto has quit IRC21:50
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132721:58
*** hwoarang has quit IRC21:58
*** hwoarang has joined #openstack-infra21:59
*** raissa has joined #openstack-infra22:02
*** raissa has quit IRC22:02
*** raissa has joined #openstack-infra22:03
*** raissa has quit IRC22:03
*** raissa has joined #openstack-infra22:03
*** raissa has quit IRC22:04
*** raissa has joined #openstack-infra22:04
*** raissa has quit IRC22:04
*** raissa has joined #openstack-infra22:05
*** raissa has quit IRC22:05
*** raissa has joined #openstack-infra22:06
*** raissa has quit IRC22:06
*** rcernin has quit IRC22:06
*** smarcet has left #openstack-infra22:10
*** whoami-rajat has quit IRC22:11
*** dave-mccowan has joined #openstack-infra22:12
*** slaweq has joined #openstack-infra22:16
*** pkopec has quit IRC22:19
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132722:20
*** Emine has joined #openstack-infra22:21
*** slaweq has quit IRC22:24
*** emine__ has joined #openstack-infra22:27
*** Emine has quit IRC22:29
*** mriedem has quit IRC22:29
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132722:30
*** dave-mccowan has quit IRC22:31
clarkbcorvus: I filed my expenses :) thank you for the reminder22:38
fungimordred: you probably understand more about pbr's use of pkg_resources than anyone... do you think it's possible to replace its use with importlib_metadata like https://github.com/tox-dev/tox/commit/afe11da does?22:40
clarkbfungi: reading the docs there is a good chance it can actually22:41
clarkbif we can get osc/cliff to use that too we may fix the osc performance problems22:41
clarkbugh I'm gonna get nerdsniped on this22:42
fungisorry, just saw it in the draft changelog for tox completely by chance22:42
fungididn't mean to interrupt!22:43
clarkbno no this is great22:43
clarkbI'm glad they are fixing the super slow startup cost in pkg_resources22:43
fungiyeah, it came up again on the starlingx-discuss ml a couple weeks back about how using pbr for runtime version strings significantly increases startup time, and i had to do a fair amount of debunking22:44
clarkbya the issue goes far deeper than that unfortunately22:45
clarkblike entrypoints + pkg_resources hit it too22:46
clarkbso if you "fix" pbr you don't win in many cases (like osc case) becuase you still ahve the pkg_resources cost incurred for plugin handling22:46
clarkbwhich is what makes this so painful to change (you don't really know if you've helped anything until you've fixed everything)22:46
*** emine__ has quit IRC22:51
ianwhrm, i would have expected that zuul variable change for the mirrors to have rolled out by now22:52
ianwthe latest change in /etc/project-config on the ze's is "Date:   Fri Apr 19 22:21:35 2019 +0000" ... that is a highly suspicious date to have a last log for i think22:58
clarkbfungi: mordred dtroyer remote:   https://review.opendev.org/661856 Replace pkg_resources with importlib-metadata23:00
*** dayou_ has quit IRC23:01
clarkbI have no idea if that is functional but if we can start replacing things we should eventually start to see the performance benefits23:01
clarkbianw: hrm23:01
*** tkajinam has joined #openstack-infra23:02
clarkbianw: https://git.openstack.org/openstack-infra/project-config is the remote there whcih should redirect properly23:02
ianway 28 22:38:34 ze01 puppet-user[4324]: (/Stage[main]/Project_config/Vcsrepo[/etc/project-config]/ensure) Updating to latest '1e6705f722355537e08a0187dfc957949389dc09' revision23:02
ianwmy question now is how does "updating to latest" make that decision23:02
clarkbmaybe try a git remote update and see what that does?23:03
ianw[remote "origin"]23:03
ianw        url = https://git.openstack.org/openstack-infra/project-config23:03
*** dayou has joined #openstack-infra23:03
*** tosky has quit IRC23:04
clarkbso .git/FETCH_HEAD is from today23:04
clarkbbut it isn't checking that out23:04
clarkbso I think the remote update is working at least23:04
clarkbone thing I know is that vcsrepo won't update a remote. But this was only an issue previously where we used git://23:05
clarkbso I manually updated the git:// cases23:05
clarkbhttps:// with redirects being a problem is new to me23:06
*** smarcet has joined #openstack-infra23:06
*** smarcet has left #openstack-infra23:07
ianwright, FETCH_HEAD is @ 5f4aee94c286b0c8641a4fb595f38f590fd2972c which is master top of tree23:08
ianwso what does it do after "git remote update"23:08
clarkbit should checkout FETCH_HEAD I thought23:08
*** slaweq has joined #openstack-infra23:11
clarkbianw: you can probably try updating the remote by hand (like I did with git://) and see if that makes it happier?23:12
ianwyeah, i'm sort of more worried that this plays out on every single server23:13
ianwreview.opendev.org has the same thing, it seems, but that's updated23:13
ianwit's /etc/project-config has the same openstack.org origin23:14
*** dave-mccowan has joined #openstack-infra23:14
*** _erlon_ has quit IRC23:16
clarkbya I know we checked it on the git farm related nodes23:17
clarkbbecause we needed it up to date for zuul main.yaml on schedulwr and review for manage projects and so on23:17
openstackgerritJeremy Stanley proposed opendev/system-config master: Exclude old distros from yum-puppetlabs mirror  https://review.opendev.org/66186023:18
ianwclarkb: here's a difference ... http://paste.openstack.org/show/752197/ review seems to maybe get the HEAD reference externally; while ze's don't23:20
*** rcernin has joined #openstack-infra23:21
fungiinfra-root: mwhahaha: i ran with 661860 manually to get the mirror updated again, but approving quickly would be appreciated so it doesn't regress23:21
clarkbianw ya we ensure review and zuul.o.o and gitea cluster all operate with same project config data23:22
clarkbmaybe that forces the checkout23:22
ianwright; https://opendev.org/opendev/system-config/src/branch/master/playbooks/remote_puppet_git.yaml23:23
*** slaweq has quit IRC23:24
ianwso that's different23:24
ianwi would say the problem is that we've swapped the source=> to vcsrepo, and that has somehow confused it23:25
*** lseki has quit IRC23:31
*** dave-mccowan has quit IRC23:33
ianwhrm, puppet-vcsrepo hasn't appreciably changed in 7+ years; so heaven knows if this might have been fixed upstream etc23:34
ianwthere is certainly a lot more in current code that looks like it handles swapping sources -> https://github.com/puppetlabs/puppetlabs-vcsrepo/blob/master/lib/puppet/provider/vcsrepo/git.rb#L16423:35
mordredclarkb, fungi: ooh neat! (re importlib-metadata)23:36
mordredclarkb: fixing that in python-openstackclient itself should actually fix osc and its plugins - since its plugins should not, unless something is really weird, be doing the pkg_resources things themselves23:37
*** jbadiapa has quit IRC23:37
mordredclarkb: unless, of course, this means ANY use of pkg_resources - because if we do mean that then the __version__ lines in the libraries might wind up being an issue23:38
*** jbadiapa has joined #openstack-infra23:38
mordredclarkb: like: __version__ = pbr.version.VersionInfo('python-novaclient').version_string()23:39
mordredclarkb: which uses pkg_resources to do the VersionInfo23:39
mordredclarkb: but - if we fix pbr - and that patch you made to osc - I think that should actually handle things23:40
*** gyee has quit IRC23:40
clarkbmordred its any useiirc23:40
mordredkk. then yeah - we'll def want to ... oh, this is now very very tricky ...23:41
mordredas of right now we avoid importing libraries in pbr23:41
*** yamamoto has joined #openstack-infra23:41
mordredbecause of bootstrapping issues23:41
mordredI don't remember if pip has fixed its processing of setup_requires so that transitive depends of setup_requires requirements get processed by pip and not by easy_install23:42
mordredso we should check that23:42
mordredif so - then it might be ok to have pbr depend on this particular lib23:42
mordredor - we might want to see if the new PEP stuff about splitting wheel builder plugins is usable enough for us to dive in to making pbr provide one of those23:43
mordredanyway - may moving threads - but potentially exciting if we can solve that pkg_resources cost23:43
clarkbya and really if we can fix one at a time we'll make progress and be future proof23:44
mordred++23:44
*** dave-mccowan has joined #openstack-infra23:46
*** dave-mccowan has quit IRC23:53
*** takamatsu has quit IRC23:56
*** Weifan has quit IRC23:59
*** aaronsheffield has quit IRC23:59

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