Tuesday, 2020-01-28

*** threestrands has joined #openstack-infra00:04
clarkbfungi: does gertty use gerritlib?00:06
clarkb(I think its all http so no)00:06
openstackgerritMerged opendev/zone-opendev.org master: Add static01.opendev.org records  https://review.opendev.org/70446900:07
fungiit does not00:07
fungicorvus observed that the gerrit rest api is plenty well-defined that wrapping it in anything fancier than the requests library is somewhat pointless00:08
fungithough i think several of us wound up at the same conclusion00:09
fungiwhen doing the rewrite of openstack's election tooling a while back it was faster and easier to read the rest api docs than to figure out some additional python library00:10
clarkbya gerritlib is ssh only too00:10
paladoxpygerrit2 is rest api :)00:11
*** smarcet has quit IRC00:12
clarkbright I want ssh :)00:12
openstackgerritClark Boylan proposed opendev/gerritlib master: Use newer paramiko  https://review.opendev.org/70447100:13
openstackgerritClark Boylan proposed opendev/gerritlib master: Convert gerrit output to utf8 unicode  https://review.opendev.org/70447200:13
clarkbfungi: ^ the first one bumping paramiko should fix status upgrade00:14
fungicool, will take a look00:14
clarkbthe second was something I noticed when testing the change. I ran listProjects against review-dev and old code only worked with python2, new code seems to wrok with 2 and 300:14
clarkbianw: ^ you too00:14
paladoxclarkb https://pypi.org/project/pygerrit/ supports ssh :)00:14
clarkbpaladox: ya iirc it came years after gerritlib00:14
paladoxah00:14
clarkbso we were already using this lib for a long time in various places00:15
clarkbit would probably make sense to convert, but updating paramiko is much easier right now00:15
fungithere have been a bunch of active gerritlib reviews recently, so we may want to take a stroll through those before blindly approving new patches for it00:16
clarkbfungi: ya I skimmed it and couldn't find any that bumped paramiko00:16
clarkband https://review.opendev.org/#/c/412536/6/gerritlib/gerrit.py for example has the python3 problem (out.split's arg is a unicode string but out is a bytestring and it goes boom)00:17
clarkbI don't think any of the changes there address these problems00:17
fungiaah, the only recently-active ones are pelix's 412535 and 41253600:18
fungitristanC's 412757 is also borderline recently-active00:19
clarkbpaladox: http://codesearch.openstack.org/?q=gerritlib&i=nope&files=&repos= gives a rough idea of what would need converting. It isn't all that much, but does tend to be things that are neglected :/00:19
paladoxheh00:19
clarkbgermqtt, elastic-recheck, gerritbot and jeepyb00:19
fungii mostly mean look through them in terms of approving them if we can and think they're reasonable so we're not just approving each others' changes and ignoring other contributors ;)00:20
clarkbthats fair, but those repos are pretty dead due to lack of contributors helping :/00:20
clarkbit would be really really nice if the QA team could help with e-r more00:21
clarkbgmann: ^00:21
clarkbI think the mqtt stuff was on my list of services to consider cleaning up due to lack of use00:21
clarkbthe other three should probably use pygerrit assuming it is still maintained00:21
*** rlandy is now known as rlandy|afk00:24
fungiit looks like pelix's two changes and tristanC's change to gerritlib just need one more +2 anyway00:25
fungii had apparently already reviewed some of them00:26
fungii approved 412757 and i already had a +2 on 412535 and 412536 (the parent change needs a second +2 but the child change already has 2x +200:28
clarkb757 is a bit odd, looking at fd internals00:31
clarkbit would probably be better to call read() until you get back '', then split what you got00:32
clarkbfungi: 757 conflicts with 535 fwiw00:32
openstackgerritIan Wienand proposed opendev/system-config master: Add static01.opendev.org  https://review.opendev.org/70447600:34
fungiyeah, i hadn't seen that trick with readline()'s _pos and _realpos attributes before00:35
ianw^ with that in i think we'll be able to test governance/security with host overrides00:35
ianwbefore we flip the dns switch00:35
clarkbfungi: I've +2'd 535 but not approved as 757 will conflict with it00:39
fungii removed my votes from 757 following your comments in here00:40
clarkbianw: +200:40
clarkbfungi: ah in that case feel free to approve the other two00:40
fungiianw: in playbooks/roles/letsencrypt-create-certs/handlers/main.yaml how do the repeated roles/letsencrypt-create-certs/handlers/restart_apache.yaml entries get scoped to different vhosts?00:40
*** jhesketh has joined #openstack-infra00:41
clarkbfungi: they should get run multiple times00:41
clarkbfungi: but since its a graceful restart I'm not too worried about it. Also only happens every 2 months00:41
fungimore just wondering why it's the same role included multiple times under different names with no conditional00:41
clarkbfungi: re gerritlib if we land those changes we should plan to do a release tomorrow?00:41
fungiclarkb: i think so, yes00:42
clarkbfungi: the handler name is the conditional, since we trigger the specific handler by name only when that cert is updated00:42
clarkbfungi: its like a function look up table00:42
fungioh, so that "letsencrypt updated static01-opendev-org-main" name gets referenced explicitly elsewhere?00:42
clarkbfungi: yes, by the code that updates the contents on disk for static01-opendev-org-main (we basically call the update code for each entry at https://review.opendev.org/#/c/704476/1/playbooks/host_vars/static01.opendev.org.yaml it decides if it needs to do anything and if so triggers the handler via that lookup table)00:43
*** mattw4 has quit IRC00:43
fungii guess that happens in a future patch not yet written00:43
*** hwoarang has quit IRC00:43
clarkbno thats already written00:43
clarkbits just variable based on what is in the host/group file00:44
clarkbso not explicit in the code00:44
clarkbianw: can you review https://review.opendev.org/#/c/704471/ as that is another fix necessary for status.o.o00:44
clarkbthat and a gerritlib release including it00:44
openstackgerritMerged opendev/system-config master: Flot sources moved, update our minimization to accomodate  https://review.opendev.org/70446700:46
*** hwoarang has joined #openstack-infra00:46
clarkbre yui minimization is the concern that it might produce code that isn't valid in modern browsers?00:47
clarkbit doesn't do the fancy mapping files to make sense of what is what which I guess is another downside00:47
openstackgerritJulia Kreger proposed openstack/diskimage-builder master: Add handling for no activate_this.py in venv  https://review.opendev.org/70447800:48
clarkbTheJulia: ianw ^ does that inappropriately detect the no venv case as a venv case?00:49
*** smarcet has joined #openstack-infra00:50
clarkbI guess it probably doesn't matter if that happens00:50
TheJuliaclarkb: no, venv lacks actiate_this.py which virtualenv for python2 provided00:51
TheJuliapython3.3 introduced the other variables in sys as handled by default as well00:51
clarkbTheJulia: one more comment left inline00:55
openstackgerritJulia Kreger proposed openstack/diskimage-builder master: Add handling for no activate_this.py in venv  https://review.opendev.org/70447800:58
clarkbupon further investigation I think my initial concern is still valid00:58
clarkbbecause we'll end up with /usr/local/bin in sys.path which isn't quite right00:58
clarkb(probably won't break anything in most cases though)00:58
clarkboh except we are guarded at the top00:59
clarkbI am up to speed now, thanks00:59
ianwTheJulia: ahhh, yes i think i hit this and meant to do something about it :/01:00
ianwi think that seems about right, thanks01:01
TheJuliaianw: it shows up nicely in bifrost's build jobs when we switched over to venv becuase virutalenv seemed to implode on us.01:01
ianwTheJulia: is there a depends-on i can monitor just to be sure?01:02
TheJulianope, but should be easy to give a spin. will double check real quick01:03
TheJuliaianw: https://review.opendev.org/70448001:05
ianwi wonder if, since we moved dib-run-parts into lib/, if we actually need all that any more.  one to put in the todo list i guess01:06
TheJuliathe dibipa job should explode nicely if it doesn't work01:06
ianwcool, thanks, will keep an eye01:06
*** signed8b_ has joined #openstack-infra01:07
*** signed8bit has quit IRC01:11
TheJuliaianw: we might not be installing from git....  I feel like that might have changed since we didn't feel the need to pull in latest items  :\01:14
TheJuliaheh, wrong patch01:15
openstackgerritsebastian marcet proposed opendev/puppet-openstackid master: Fixed broken deployment  https://review.opendev.org/70448201:16
*** gyee has quit IRC01:17
*** jamesmcarthur has joined #openstack-infra01:18
clarkbfungi: were you going to approve those gerritlib changes?01:21
clarkb535 and 53601:22
TheJuliaianw: yeah... some recent changes appear to have nuked out the git install01:22
fungiclarkb: yep, just about to circle back around to them01:23
fungigit sidetracked refreshing my memory of how the letsencrypt handlers are mapped01:24
TheJuliaianw: https://review.opendev.org/704480 updated to swap install orders so it doesn't get downgraded to the latest release01:25
*** KeithMnemonic1 has joined #openstack-infra01:25
*** KeithMnemonic2 has joined #openstack-infra01:26
*** yamamoto has joined #openstack-infra01:26
*** KeithMnemonic has quit IRC01:29
*** KeithMnemonic1 has quit IRC01:30
openstackgerritMerged opendev/gerritlib master: Reuse same paramiko connection recreating on errors  https://review.opendev.org/41253501:32
openstackgerritMerged opendev/gerritlib master: Support querying for single group info  https://review.opendev.org/41253601:32
clarkbianw: I've got to pop out for dinner soon but review on https://review.opendev.org/#/c/704471/ would be excellent if possible before tomorrow01:33
clarkbI'll try to make a gerritlib release in the morning if that lands01:33
openstackgerritMerged opendev/system-config master: Add static01.opendev.org  https://review.opendev.org/70447601:37
*** jamesmcarthur has quit IRC01:43
*** factor has joined #openstack-infra01:46
*** diablo_rojo has joined #openstack-infra01:59
openstackgerritMerged opendev/gerritlib master: Use newer paramiko  https://review.opendev.org/70447102:00
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: WIP: add CONTRIBUTING guidelines  https://review.opendev.org/70448402:02
*** jamesmcarthur has joined #openstack-infra02:07
openstackgerritsebastian marcet proposed opendev/puppet-openstackid master: Fixed broken deployment  https://review.opendev.org/70448202:15
*** jamesmcarthur has quit IRC02:19
*** yamamoto has quit IRC02:19
*** yamamoto has joined #openstack-infra02:20
openstackgerritMerged opendev/gerritlib master: Convert gerrit output to utf8 unicode  https://review.opendev.org/70447202:28
*** roman_g has quit IRC02:34
*** yamamoto has quit IRC02:39
*** gfidente has quit IRC02:40
ianwhrm, static.opendev got certs but didn't seem to get a webserver02:42
*** yamamoto has joined #openstack-infra02:42
*** yamamoto has quit IRC02:42
ianwfatal: [mirror01.regionone.linaro-us.opendev.org]: UNREACHABLE! => {"changed": false, "msg": "Data could not be sent to remote host \"2604:1380:4111:3e54:f816:3eff:fea4:899d\"02:43
ianwthat seems bad too, but one thing at a time02:43
*** yamamoto has joined #openstack-infra02:43
*** jamesmcarthur has joined #openstack-infra02:44
*** jamesmcarthur has quit IRC02:45
ianwoh i've forgotten to add it to the cron rub02:46
ianwrun02:46
*** jamesmcarthur has joined #openstack-infra02:50
*** yamamoto has quit IRC02:52
*** diablo_rojo has quit IRC02:53
*** jamesmcarthur_ has joined #openstack-infra02:54
*** jamesmcarthur has quit IRC02:54
openstackgerritIan Wienand proposed opendev/system-config master: Add service-static.yaml to cron run  https://review.opendev.org/70449102:54
*** jamesmcarthur_ has quit IRC03:03
*** jamesmcarthur has joined #openstack-infra03:03
*** factor has quit IRC03:05
*** factor has joined #openstack-infra03:06
*** jamesmcarthur has quit IRC03:08
*** rkukura has quit IRC03:13
ianwkevinz: so, related to ^^ it seems the linaro-us mirror has gone down and i can't see anything useful; filed https://storyboard.openstack.org/#!/story/2007195 task #3849103:16
ianwi'm also pretty sure that along with other things going on in CN it's new year, which i imagine means holidays too so not sure on latency for a fix (no rush, as such, we still have the other comms issues to fix up too)03:18
ianwi'll leave it for now.  my comms eqiupment has arrived so offline for a bit to plug it all in03:19
*** yamamoto has joined #openstack-infra03:29
*** jamesmcarthur has joined #openstack-infra03:34
*** jamesmcarthur has quit IRC03:41
*** zxiiro has quit IRC03:41
*** diablo_rojo has joined #openstack-infra04:01
*** rcernin has quit IRC04:18
*** rcernin has joined #openstack-infra04:18
*** armax has quit IRC04:24
*** smarcet has quit IRC04:27
*** goldyfruit_ has quit IRC04:27
*** goldyfruit_ has joined #openstack-infra04:27
*** jamesmcarthur has joined #openstack-infra04:37
*** jamesmcarthur has quit IRC04:41
*** rlandy|afk has quit IRC04:48
*** ykarel|away is now known as ykarel04:49
*** smarcet has joined #openstack-infra04:55
*** smarcet has left #openstack-infra05:10
*** jamesmcarthur has joined #openstack-infra05:12
*** goldyfruit_ has quit IRC05:13
*** goldyfruit_ has joined #openstack-infra05:13
*** jamesmcarthur has quit IRC05:18
*** udesale has joined #openstack-infra05:25
*** raukadah is now known as chkumar|rover05:34
mordredianw: in the service-static playbook - those a2ensite commands have creates: on them - wouldn't that prevent the notify from firing if the file exists but the contents change? (I have no idea of the answer- wondering if you do)05:43
openstackgerritMonty Taylor proposed opendev/system-config master: Add service-static.yaml to cron run  https://review.opendev.org/70449105:46
mordredianw: also - ^^ fixed a copy-pasta05:46
*** toabctl has quit IRC06:05
*** factor has quit IRC06:05
*** threestrands has quit IRC06:08
*** lmiccini has joined #openstack-infra06:09
*** toabctl has joined #openstack-infra06:12
*** jamesmcarthur has joined #openstack-infra06:13
*** rkukura has joined #openstack-infra06:16
*** lpetrut has joined #openstack-infra06:16
*** jamesmcarthur has quit IRC06:18
*** jtomasek has joined #openstack-infra06:35
*** psachin has joined #openstack-infra06:39
openstackgerritMerged opendev/system-config master: Add service-static.yaml to cron run  https://review.opendev.org/70449106:43
*** sshnaidm|afk is now known as sshnaidm06:48
openstackgerritMonty Taylor proposed opendev/system-config master: Add Apache to Ansible for Gerrit  https://review.opendev.org/70449806:48
*** ahosam has quit IRC06:50
*** jamesmcarthur has joined #openstack-infra06:51
mordredianw: have we gotten anywhere yet with letsencrypt for openstack.org domains?06:54
*** jamesmcarthur has quit IRC06:56
*** pgaxatte has joined #openstack-infra07:18
*** jtomasek has quit IRC07:24
*** ociuhandu has joined #openstack-infra07:30
*** dpawlik has joined #openstack-infra07:34
*** ykarel is now known as ykarel|lunch07:35
*** ociuhandu has quit IRC07:35
*** yamamoto has quit IRC07:37
*** slaweq has joined #openstack-infra07:39
*** xek has joined #openstack-infra07:45
*** jamesmcarthur has joined #openstack-infra07:52
*** jamesmcarthur has quit IRC07:57
*** aedc has quit IRC08:04
*** rpittau|afk is now known as rpittau08:07
*** jtomasek has joined #openstack-infra08:07
*** yamamoto has joined #openstack-infra08:12
*** yamamoto has quit IRC08:12
*** yamamoto has joined #openstack-infra08:12
*** tosky has joined #openstack-infra08:13
AJaegerconfig-core, could you review https://review.opendev.org/704306 and https://review.opendev.org/704321, please?08:15
*** tesseract has joined #openstack-infra08:16
*** dchen has quit IRC08:20
*** iurygregory has joined #openstack-infra08:24
*** jamesmcarthur has joined #openstack-infra08:27
*** ykarel|lunch is now known as ykarel08:31
*** jamesmcarthur has quit IRC08:32
*** tkajinam has quit IRC08:34
openstackgerritTobias Henkel proposed zuul/zuul master: Handle draft pull requests in canMerge  https://review.opendev.org/70415008:37
openstackgerritJan Kubovy proposed zuul/zuul master: Enforce sql connections for scheduler and web  https://review.opendev.org/63047208:37
*** ralonsoh has joined #openstack-infra08:38
openstackgerritMerged openstack/project-config master: check-release-approval job improvements  https://review.opendev.org/70430608:39
openstackgerritMerged openstack/project-config master: Updated x/networking-mlnx project for pypi and neutron  https://review.opendev.org/70432108:39
*** kjackal has joined #openstack-infra08:40
*** jpena|off is now known as jpena08:52
*** ccamacho has joined #openstack-infra08:57
*** tetsuro has joined #openstack-infra08:58
*** priteau has joined #openstack-infra08:59
openstackgerritMerged zuul/nodepool master: Remove duplicate log info from OpenStack driver  https://review.opendev.org/70437909:00
*** hashar has joined #openstack-infra09:03
*** lucasagomes has joined #openstack-infra09:11
*** lucasagomes has quit IRC09:11
*** lucasagomes has joined #openstack-infra09:13
openstackgerritAntoine Musso proposed zuul/zuul master: Add client_id to RPC client  https://review.opendev.org/70420109:16
openstackgerritMiklos Vajna proposed opendev/git-review master: Make it possible to specify who is notified  https://review.opendev.org/70286109:25
openstackgerritMerged zuul/zuul master: Be more specific with remaining time  https://review.opendev.org/70419109:32
ianwmordred: thanks09:42
ianwmordred: yes, we redirect the _acme-challenge.name.openstack.org CNAME to acme.opendev.org and use the "usual" configuration09:43
ianwthat's a point about changing the content of the file and creates: ... i will have to investigate09:44
*** yamamoto has quit IRC09:44
ianwit looks like we still have issues with the openafs roles :/ --> http://paste.openstack.org/show/788863/09:46
*** yamamoto has joined #openstack-infra09:46
ianwno hang on ... Cache size (50000000) must be less than 95% of partition size (which is 38447165). Lower cache size09:47
openstackgerritMerged zuul/zuul master: Update screenshot for gerrit profile in quick-start  https://review.opendev.org/70422209:48
*** yamamoto has quit IRC09:50
*** gfidente has joined #openstack-infra09:53
openstackgerritIan Wienand proposed opendev/system-config master: openafs-client: add option for OpenAFS cache location  https://review.opendev.org/70452310:05
mordredianw: cool - so we could go ahead and LE review-dev if we wanted to at this point10:06
* mordred is mostly pondering the "best" way to deal with review-dev and review currently doing ssl differently for apache - don't really want to make the ansible overly clever10:08
*** ociuhandu has joined #openstack-infra10:08
ianwmordred: yep, just gotta log in to rax and point the openstack.org challenge domains by hand once10:08
mordredbut the current review.o.o cert doesn't expire til mar 26 - and I think we don't get a TON of win unless we LE both of them10:08
mordrednod10:08
ianwit should be pretty straight forward to get the LE certs on the host; can be done asynchronously with actually using them of course10:09
mordredianw: oh - that's a good point10:10
*** Lucas_Gray has joined #openstack-infra10:10
*** electrofelix has joined #openstack-infra10:10
ianwthe "dummy" handler is to just touch a file when the certs are updated, i think there's an example for graphite in there10:11
mordredmaybe I'll work on doing that for both next - that way I can have the ansible role just point to the LE certs like you did with static10:11
openstackgerritMerged zuul/zuul master: Fix path in quick-start localtest  https://review.opendev.org/70438510:11
ianwmordred: should i ask why you're awake?10:12
mordredianw: I'm in Thailand :)10:12
mordredianw: but - today's my last day on this side of the world, I start the long trip back stateside tonight10:12
ianwheh, where is the world is Mordred Sandiego10:13
mordredcome Monday we'll be moved in to our new place in New Orleans and I'll be more predictably located and available10:13
mordredianw: yah. everyone's favorite game10:13
ianwall those places sound fun :)10:14
ianwwell, as long as you're above sea level in New Orleans10:14
mordredianw: I mean ... we'll do our best ...10:15
ianwi guess you have a lot of scuba gear anyway :)10:16
mordredthat's all we need, right?10:18
*** ociuhandu has quit IRC10:21
*** Lucas_Gray has quit IRC10:32
*** Lucas_Gray has joined #openstack-infra10:40
*** aedc has joined #openstack-infra10:55
*** ociuhandu has joined #openstack-infra10:55
*** Lucas_Gray has quit IRC11:00
*** Lucas_Gray has joined #openstack-infra11:06
*** dtantsur|afk is now known as dtantsur11:07
*** kjackal has quit IRC11:07
*** aedc has quit IRC11:12
*** rpittau is now known as rpittau|bbl11:18
*** tetsuro has quit IRC11:19
*** rm_work has quit IRC11:21
*** rm_work has joined #openstack-infra11:21
*** hashar has quit IRC11:35
*** yamamoto has joined #openstack-infra11:42
*** yamamoto has quit IRC11:43
*** kjackal has joined #openstack-infra11:44
*** lpetrut has quit IRC11:50
*** Lucas_Gray has quit IRC11:50
*** hashar has joined #openstack-infra11:58
*** Lucas_Gray has joined #openstack-infra12:01
*** Lucas_Gray has quit IRC12:06
*** Lucas_Gray has joined #openstack-infra12:09
*** yamamoto has joined #openstack-infra12:18
*** yamamoto has quit IRC12:19
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: Link the policy to the CONTRIBUTING.rst file  https://review.opendev.org/70448412:22
*** philroche has quit IRC12:22
*** philroche has joined #openstack-infra12:23
*** jpena is now known as jpena|lunch12:23
*** rcernin has quit IRC12:29
*** ociuhandu has quit IRC12:29
*** ociuhandu has joined #openstack-infra12:30
*** ociuhandu has quit IRC12:35
*** Lucas_Gray has quit IRC12:39
*** Lucas_Gray has joined #openstack-infra12:40
*** mwhahaha has quit IRC12:40
*** mwhahaha has joined #openstack-infra12:41
*** yamamoto has joined #openstack-infra12:51
*** aedc has joined #openstack-infra12:51
slaweqhi infra team, I have a question about launchpad12:54
slaweqI would like to add new tag to registered tags list for neutron12:54
slaweqdo You know how can I do this in LP maybe? Any guide?12:54
*** yamamoto has quit IRC12:57
*** rlandy has joined #openstack-infra12:59
*** rh-jelabarre has joined #openstack-infra13:01
*** portdirect has quit IRC13:01
*** portdirect has joined #openstack-infra13:02
*** rpittau|bbl is now known as rpittau13:04
*** psachin has quit IRC13:06
*** ykarel is now known as ykarel|mtg13:06
*** ociuhandu has joined #openstack-infra13:10
*** jamesmcarthur has joined #openstack-infra13:11
*** jamesmcarthur has quit IRC13:13
*** jamesmcarthur has joined #openstack-infra13:13
*** eharney has joined #openstack-infra13:17
*** jpena|lunch is now known as jpena13:20
*** dtantsur is now known as dtantsur|lunch13:20
*** ociuhandu has quit IRC13:20
*** udesale_ has joined #openstack-infra13:22
*** udesale has quit IRC13:25
*** Darcidride has joined #openstack-infra13:27
*** jamesmcarthur has quit IRC13:35
*** psachin has joined #openstack-infra13:38
*** jamesmcarthur has joined #openstack-infra13:39
*** jamesmcarthur has quit IRC13:45
*** jamesmcarthur has joined #openstack-infra13:47
*** Goneri has joined #openstack-infra13:48
*** aaronsheffield has joined #openstack-infra13:50
*** ociuhandu has joined #openstack-infra13:53
*** jaicaa has quit IRC13:54
*** signed8b_ has quit IRC13:56
*** signed8bit has joined #openstack-infra13:57
*** ociuhandu has quit IRC14:00
*** tkajinam has joined #openstack-infra14:00
*** AJaeger has quit IRC14:02
openstackgerritAntoine Musso proposed zuul/zuul master: Speed up test_inventory  https://review.opendev.org/70457014:03
*** jaicaa has joined #openstack-infra14:06
*** ykarel|mtg is now known as ykarel14:09
*** sdoran has quit IRC14:09
*** sdoran has joined #openstack-infra14:10
*** redrobot_ has joined #openstack-infra14:10
*** redrobot_ is now known as redrobot14:11
*** dtantsur|lunch is now known as dtantsur14:12
openstackgerritFabien Boucher proposed opendev/system-config master: DNM: Test ara-report artifact  https://review.opendev.org/69768314:13
*** ykarel is now known as ykarel|away14:18
*** AJaeger has joined #openstack-infra14:19
*** yamamoto has joined #openstack-infra14:20
*** Lucas_Gray has quit IRC14:23
*** Wryhder has joined #openstack-infra14:23
*** Wryhder is now known as Lucas_Gray14:24
*** pcaruana has quit IRC14:24
*** jamesden_ has joined #openstack-infra14:28
*** hamzy__ is now known as hamzy14:28
*** jamesdenton has quit IRC14:29
*** pkopec has joined #openstack-infra14:31
*** tobias-urdin has joined #openstack-infra14:36
openstackgerritJan Kubovy proposed zuul/zuul master: Enforce sql connections for scheduler and web  https://review.opendev.org/63047214:40
*** smarcet has joined #openstack-infra14:43
*** Lucas_Gray has quit IRC14:47
*** Lucas_Gray has joined #openstack-infra14:49
*** Lucas_Gray has quit IRC14:53
*** Lucas_Gray has joined #openstack-infra14:56
*** Wryhder has joined #openstack-infra14:58
*** jtomasek has quit IRC15:00
*** Lucas_Gray has quit IRC15:01
*** Wryhder has quit IRC15:01
*** tkajinam has quit IRC15:02
clarkbslaweq: tags?15:02
clarkbinfra-root I'd like to make a gerritlib release today. I think I'll dot 0.8.2 on c234e11b513e0a636d487140728e9830ca1753b9 and avoid tagging 945d2088a7dc5c433020542f53b4381765fc7830 as 0.9.0 just yet since I've realized that gerritlibs unittest framework is a nop15:03
clarkbmy concern there is that the type change might affect jeepyb. Once I've got status.o.o up I'll look at maybe adopting zuul's gerrit docker-compose stuff to do an integration job of gerritlib, jeepyb and gerrit15:04
clarkbinfra-root ^ if that plan sounds good to you I'll push the tag after the board meeting15:05
*** Lucas_Gray has joined #openstack-infra15:06
*** jamesmcarthur has quit IRC15:07
openstackgerritSorin Sbarnea proposed opendev/git-review master: Adopt flake8+black  https://review.opendev.org/70405115:07
*** jamesden_ is now known as jamesdenton15:07
*** psachin has quit IRC15:07
*** michael-beaver has joined #openstack-infra15:08
slaweqclarkb: ok, I found it https://bugs.launchpad.net/neutron/+manage-official-tags :)15:08
*** pcaruana has joined #openstack-infra15:11
fungiclarkb: sgtm15:12
*** jamesden_ has joined #openstack-infra15:15
corvusclarkb: what's gerritlib used for?15:15
corvusand did i miss the part where we changed our minds on flake8?15:16
corvuser black15:16
*** jamesdenton has quit IRC15:18
*** ociuhandu has joined #openstack-infra15:18
corvusclarkb: iiuc, gerritlib is used by some processes on status.o.o so is a blocker for updates there?15:20
corvus(there's a lot of scrollback, i'm try to tease out what's relevant)15:21
clarkbcorvus: yes, elastic-recheck uses gerritlib and that pulls in pycrypto which fails to build. We should be using newer paramiko which uses cyptography and has wheels instead (and is more secure and up to date)15:21
clarkbcorvus: we've updated gerritlib to bump paramiko and rudimentary testing doing listProjects against review-dev works15:21
clarkbcorvus: there is a followon change that attempts to make gerritlib work with python3 that I think deserves more testing before adding it to a release15:22
openstackgerritAntoine Musso proposed zuul/zuul master: Log duration of AnsibleJob.execute()  https://review.opendev.org/70458915:22
clarkbcorvus: jeepyb also uses gerritlib so trying to keep the delta small15:22
*** hashar has quit IRC15:22
corvusclarkb: thanks.  sgtm.15:23
clarkbre black I don't think there is a plan. zbr seems to just be pusing the change. I've reviewed it as I'm curious to see what that looks like in practice15:23
*** jamesden_ is now known as jamesdenton15:23
*** Lucas_Gray has quit IRC15:24
*** Wryhder has joined #openstack-infra15:24
*** ociuhandu has quit IRC15:24
corvusas you know since we've had this conversation many times, i'm strongly opposed.15:24
*** Wryhder is now known as Lucas_Gray15:25
*** ociuhandu has joined #openstack-infra15:25
*** Lucas_Gray has quit IRC15:27
*** Lucas_Gray has joined #openstack-infra15:29
corvusmy understanding is that we have a consensus on code style and linters which we have arrived at after much discussion.  if we're going to re-open the discussion, let's do so deliberately.  otherwise, we should let people know that the status quo is intentional so we all don't wast time going over ground we've gone over before.15:31
clarkbyes, I think any changes should be agreed upon globally and applied globally so that we don't have to perform gymnastics when writing code for various repos15:33
zbrcorvus: clarkb: feel free to -2 the black experiment, i would have raised it as draft/private if possible.15:33
corvuszbr: ok, i've -2d that.  i think it's fine to push up changes like that for experiment/show-and-tell.  i just want to make sure we're on the same page about that and no one is wasting effort.  :)15:35
*** ociuhandu has quit IRC15:37
*** jamesmcarthur has joined #openstack-infra15:37
zbrcorvus: would be better if I put something like "POC:" prefix on it?15:46
corvuszbr: yeah15:47
*** diablo_rojo has quit IRC15:47
*** Goneri has quit IRC15:47
corvushelps everyone know what the intention is15:47
openstackgerritSorin Sbarnea proposed opendev/git-review master: POC: Adopt flake8+black  https://review.opendev.org/70405115:47
*** Lucas_Gray has quit IRC15:48
zbrat some point we will have to discuss about pre-commit tool, i know that infra was against it, but nova already adopted it, i doubt we could block its use.15:50
zbrbetter to think more about implications and if there is something to make it easier to use15:50
fungii don't think we're particularly against projects using any tools they want to use15:51
*** jamesmcarthur has quit IRC15:52
fungimany of the core reviewers for our infra team projects were against using the pre-commit tool in the projects they're reviewing15:52
fungijust because the infra team doesn't prefer a tool, that doesn't mean other projects shouldn't use it if their reviewers/maintainers have an interest in doing so15:53
zbrif I rememeber, the controversial part was always clone implementation, which on zuul would have impose some load issues, right?15:53
fungiwe pointed out that it would likely be unstable for that reason15:54
fungiit only clones itself and its plugins though, right?15:54
zbrthe number of linter/hooks repos is quite limited, probably less than 8 for entire openstack so far.15:54
zbrwhich means, we can mirror them.15:54
zbrin fact this was my attempt when I proposed the new project yesterday.15:55
fungiif it can clone from file paths, projects could probably configure zuul to provide those directly on the filesystem when requested15:55
zbrfungi: that needs to work for devs too! no way to hardcode paths there.15:56
fungiyeah, i didn't mean hard-code them15:56
fungidev's aren't going to use our ci mirrors either15:56
*** jamesmcarthur has joined #openstack-infra15:57
fungibut that's the closest we come to "mirroring" git repositories for projects we're not hosting15:57
zbrpre-commit uses its own cache, quite well managed, but we cannot used it until we unable caching in zuul.15:57
fungiso could the job prepopulate pre-commit's repository cache?15:57
zbrinteresting question...15:58
clarkbI think I have a local docker-compose bootstrap of gerrit working based on zuul quickstart and from there have managed to manually install jeepyb and gerritlib from source then run manage projects against the gerrit15:59
openstackgerritAntoine Musso proposed zuul/zuul master: Log duration of AnsibleJob.execute()  https://review.opendev.org/70458916:00
clarkbnow to encode that into a job16:00
*** chkumar|rover is now known as raukadah16:02
zbrfungi: take a look at how cache looks like http://paste.openstack.org/show/788885/16:04
*** armax has joined #openstack-infra16:05
zbrit has a db, and these folders, they are reused between projects when (repo,version) touple matches16:05
fungiyeah, that's not too bad. looks like sqlite i guess?16:06
zbryep16:06
zbreach folder contains a git sparse clone and one/more virtualenvs.16:07
fungiand the repo dirs in the cache seem to be randomly-generated suffixes referenced from the cache table16:07
zbrnot sure if is random, or hash based, i would use a hash myself.16:07
fungiahh, so the cache embeds virtualenvs inside it16:07
zbryep, because once run, it will instantly run the tool16:08
*** ociuhandu has joined #openstack-infra16:08
zbrone of the reasons is so fast16:08
zbrbut this does not make the cache portable between different machines16:08
corvusis nova using pre-commit in the gate?16:08
*** jackedin has joined #openstack-infra16:09
fungii guess the suffix *could* be some sort of base64 encoding (i see upper and lower case, digits and _ which is 63 at least)16:09
corvusbecause if it isn't, then it isn't really an infra issue; but if it is (or wants to), then that's a big change that the larger community needs to agree to.  updating this document is how that would happen: https://governance.openstack.org/tc/reference/pti/python.html16:12
*** ociuhandu has quit IRC16:12
*** psachin has joined #openstack-infra16:12
fungias far as i can tell they merged a configuration for the tool and some brief documentation on how to run it locally16:15
*** zxiiro has joined #openstack-infra16:15
*** aedc has quit IRC16:15
corvusthat's cool16:15
corvusand it's certainly the case that figuring out how to run it stably in the gate is a pre-requisite to poposing a change to the pti16:16
fungiyeah, it seems to have exposed challenges such as the one mordred fixed for them in https://review.opendev.org/67405716:18
*** adriancz has quit IRC16:18
*** adriancz has joined #openstack-infra16:18
fungiapparently always downloading whatever you want can be at odds with more than just network stability16:19
corvusbut i just wanted to clarify that there is a project policy around how to run tests in the gate, and it's not the case that infra plays "catch-up" to what projects are doing -- we work together deliberately on changes like this.16:20
*** gyee has joined #openstack-infra16:20
fungiyeah, in the case of openstack's project testing interface, that was carefully built in collaboration with infra so that we could arrive at something stable and sustainable16:21
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462316:21
*** udesale_ has quit IRC16:22
clarkbconsidering ^ is so close I think I may hold off on a release until I get that stable and am happy with it just as another layer of sanity checking16:22
fungicorvus: granted, nova's already in a risky state of pti compliance doing things like https://opendev.org/openstack/nova/src/branch/master/tox.ini#L4616:24
zbron nova alone i see it pointing to two external repos, one is the main hooks repo and another one is of more personal nature, and adds a very tiny check.16:24
zbri was considering that if we create our own fork of hooks we could avoid that external dependency, even add our own extra checks inside.16:24
zbri do see this approach much safer than just pointing to outside repos.16:25
fungiseems like they could avoid that flake8 wrapper script for the "pep8" tox testenv and just make a separate testenv for the current purpose it claims to have (checking only files touched in the current patch)16:26
clarkbis there a generic "I want to run ansible from docker container" image?16:29
clarkbI'm using the zuul-executor above since that is what quickstart does (and it needs taht image anyway so makes sense there) but am wondering if there is a better option in the context of my new job which isn't running a zuul16:30
corvusclarkb: i'm not sure; maybe ansible/ansible-runner ?16:31
clarkbthanks. I'll add that to the list of things to test once the base version is running. Also need to run a python3 version too16:34
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462316:38
*** pgaxatte has quit IRC16:45
*** jpena is now known as jpena|brb16:45
*** rfolco is now known as rfolco|bbl16:46
smarcetfungi: clarkb: afternoon guys, when u have some time please review https://review.opendev.org/#/c/704482/ and https://review.opendev.org/#/c/704329/ thx u ! :)16:51
*** tesseract has quit IRC17:01
*** aedc has joined #openstack-infra17:02
*** iurygregory has quit IRC17:04
*** aedc has quit IRC17:08
*** aedc has joined #openstack-infra17:09
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462317:10
clarkbmanage-projects will just retry in a loop if auth fails bceause it can't read its ssh key17:11
clarkbthat was an interesting one to debug17:11
*** aedc has quit IRC17:11
clarkbsmarcet: in that change you remove the database config? does that mean the db is no longer used?17:12
smarcetclarkb: the DB config removed there is the CMS DB, IDP was refactored to be a AUTHZ/AUTH Microservice so will have its own store of users , all data from member table ( from CMS side) will be move to the IDP new store, and will be managed from there :)17:14
*** lucasagomes has quit IRC17:15
*** psachin has quit IRC17:15
fungismarcet: on the permissions change, it looks from the commit message like you're saying you want the www-data user to be able to run the artisan command but you're only setting it executable by owner/group and i don't see where ownership of the file is set to www-data17:17
smarcetfungi: yes the ownership is set on line 51 of that file i think17:19
fungichown -R $FILE_OWNER:$FILE_GROUP $target_dir17:19
fungiaha, so $FILE_OWNER and $FILE_GROUP are being set elsewhere outside the script, thankls17:19
smarcetwell its on the same function i think17:20
smarcetsite_init and site_update17:20
funginothing in that script sets values for FILE_OWNER and $FILE_GROUP so it must be in another script which is calling those functions17:21
*** artom has quit IRC17:22
fungianyway, thanks for pointing out the chown -R, i missed it was also being done for all of $target_dir17:22
*** artom has joined #openstack-infra17:22
*** jpena|brb is now known as jpena17:24
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462317:24
*** hashar has joined #openstack-infra17:25
*** artom has quit IRC17:27
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462317:27
openstackgerritsebastian marcet proposed opendev/puppet-openstackid master: Refactored Deployment Script Functions  https://review.opendev.org/70432917:28
*** artom has joined #openstack-infra17:29
*** lmiccini has quit IRC17:30
*** otherwiseguy has joined #openstack-infra17:33
fungiheads up! there's a beta out for virtualenv 20, so we should probably brace for possible behavior changes when it releases: https://discuss.python.org/t/virtualenv-20-0-0-beta1-is-available/3077/417:42
*** ociuhandu has joined #openstack-infra17:42
fungiunder python3 it's basically just using the venv module now17:42
*** jrosser has quit IRC17:42
fungior basically the same as just using the venv module now17:43
*** jrosser has joined #openstack-infra17:43
*** igordc has joined #openstack-infra17:46
*** ociuhandu_ has joined #openstack-infra17:46
*** ociuhandu has quit IRC17:47
zbrcorvus: can you look at https://review.opendev.org/#/q/topic:mac-support+(status:open+OR+status:merged) ?17:48
openstackgerritTobias Henkel proposed zuul/zuul master: DNM: experiment with pygit2  https://review.opendev.org/70464417:50
*** ociuhandu_ has quit IRC17:51
*** gfidente has quit IRC17:52
openstackgerritClark Boylan proposed opendev/jeepyb master: Don't use the db in manage-projects  https://review.opendev.org/70464517:52
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462317:53
clarkbI've discovered that jeepyb's use of the db makes things difficult17:53
*** KeithMnemonic2 has quit IRC17:53
clarkbwhat I have realized though is that once we have a basic integration test in place it will make rewriting jeepyb safely much easier17:54
clarkbstill work pushing on17:54
clarkbthat said I'm fairly confident in the paramiko changes now since ssh isn't what is breaking the testing17:55
clarkbits the layer on top of ssh17:55
clarkbI want to see how this last run goes then I'll likely push a 0.8.2 tag as proposed earlier17:55
*** nickv1985 has quit IRC17:57
*** nickv1985 has joined #openstack-infra17:58
*** dustinc has quit IRC17:58
*** dustinc has joined #openstack-infra17:58
*** electrofelix has quit IRC17:59
*** derekh has quit IRC18:00
fungi#status log a reboot migration of static.openstack.org is scheduled for 2020-01-30 at 20:15 UTC per provider ticket #200126-ord-000031618:00
openstackstatusfungi: finished logging18:00
*** hashar has quit IRC18:01
fungi#status log a reboot migration of wiki.openstack.org is scheduled for 2020-01-31 at 02:16 UTC per provider ticket #200127-ord-000005618:01
openstackstatusfungi: finished logging18:01
*** priteau has quit IRC18:01
*** diablo_rojo has joined #openstack-infra18:02
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462318:07
openstackgerritSagi Shnaidman proposed openstack/project-config master: WIP: don't require CLA for contributors of Ansible Openstack modules  https://review.opendev.org/70464818:07
clarkbI think it might actually pass this time \o/18:07
sshnaidmmordred, mnaser take a look ^18:07
*** dtantsur is now known as dtantsur|afk18:07
*** pkopec has quit IRC18:08
*** Shrews has quit IRC18:09
*** Shrews has joined #openstack-infra18:09
*** armax has quit IRC18:10
*** rkukura has quit IRC18:14
*** smarcet has left #openstack-infra18:20
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462318:20
clarkbit passed but there was a small issue in the jeepyb git commits18:21
*** pkopec has joined #openstack-infra18:25
*** jpena is now known as jpena|off18:32
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462318:34
clarkbthis will go green!18:34
*** rpittau is now known as rpittau|afk18:40
*** jdelaros1 has quit IRC18:41
*** jdelaros1 has joined #openstack-infra18:41
*** armax has joined #openstack-infra18:42
openstackgerritClark Boylan proposed opendev/jeepyb master: Log errors when committing and pushing acls  https://review.opendev.org/70466018:47
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462318:47
*** yamamoto has quit IRC18:48
*** yamamoto has joined #openstack-infra18:51
*** pkopec has quit IRC18:57
openstackgerritClark Boylan proposed opendev/gerritlib master: Add integration testing with jeepyb and gerrit  https://review.opendev.org/70462319:01
*** yamamoto has quit IRC19:05
openstackgerritTobias Henkel proposed zuul/zuul master: DNM: experiment with pygit2  https://review.opendev.org/70464419:06
*** yamamoto has joined #openstack-infra19:06
*** yamamoto has quit IRC19:06
*** yamamoto has joined #openstack-infra19:07
openstackgerritMattias Jernberg proposed opendev/git-review master: Allow the default of notopic to be configurable  https://review.opendev.org/69744819:10
*** yamamoto has quit IRC19:14
*** rfolco|bbl has quit IRC19:19
openstackgerritJan Kubovy proposed zuul/zuul master: Enforce sql connections for scheduler and web  https://review.opendev.org/63047219:19
*** jamesmcarthur has quit IRC19:21
openstackgerritJeremy Stanley proposed zuul/zuul master: Flesh out the glossary significantly  https://review.opendev.org/70439119:24
ianwfungi: virtualenv-20 looks ... fun19:25
fungiyup19:26
openstackgerritJan Kubovy proposed zuul/zuul master: Enforce sql connections for scheduler and web  https://review.opendev.org/63047219:30
*** rkukura has joined #openstack-infra19:41
openstackgerritClark Boylan proposed opendev/jeepyb master: Run gerritlib and Gerrit integration test  https://review.opendev.org/70467119:42
*** hashar has joined #openstack-infra19:42
osmanlicilegigreetings. can someone help me with logging into wiki? I'm getting error "an invalid token was found" for a long time.19:43
clarkbosmanlicilegi: at what point in the login process does that happen?19:44
openstackgerritDaniel Lublin proposed opendev/git-review master: Allow choosing which field to use as author when naming branch  https://review.opendev.org/44457419:46
osmanlicilegiclarkb: after I login over ubuntu one, wiki asks me to choose a nickname although I have one. when I select a nickname and click login, it says: An error occurred: an invalid token was found.19:49
*** rchanter has joined #openstack-infra19:50
*** ociuhandu has joined #openstack-infra19:50
clarkbosmanlicilegi: ok, we are finishing up our weekly team meeting then I expect someone can go take a look at the logs19:50
osmanlicilegiclarkb: ah, ok. thx!19:52
fungiosmanlicilegi: i've seen a handful of new users get that error when trying to create accounts and can't figure out what causes it. it seems to be consistent for that fraction of users, so it could be something to do with their openids served by login.ubuntu.com19:54
*** ociuhandu has quit IRC19:54
clarkbI've pushed gerritlib 0.8.2 tag19:57
clarkbinfra-root I also expect the stack ending at https://review.opendev.org/704671 is ready for review if interested in some of the testing related to getting that 0.8.2 tag out19:57
clarkbI'm hoping that makes it easier for us to modify our gerrit management tools as we look towards self service or project managed gerrit19:57
paladox\o/19:58
paladoxclarkb we have a task on self service :P19:59
clarkbpaladox: related to that it would help us test a swap out of gerrit python library19:59
clarkbpaladox: so that is somethign we can look at as a next step from that test job19:59
paladoxoh19:59
clarkbthough we likely need to cover more of the cases that use gerritlib first19:59
hasharhi there, is there a way to list all repositories that might trigger a specific job ( tox-linters )20:01
fungizbr: redirecting here from the meeting, if we track 404 responses and add redirects, we build a solution which solves popular broken links in a variety of different search engines rather than just fixing them for one search engine (or having to independently manage accounts and relationships with the countless web search engines users may rely on)20:02
hasharand if so a way to trigger the job against those repositories with a proposed config change applied ( https://review.opendev.org/#/c/704414/1/zuul.d/python-jobs.yaml )20:02
clarkbhashar: you can probably crawl the zuul api to find all uses of that job, but I don't think such a tool exists already. And I don't think there is a way to adhoc trigger it against all projects (you might want #zuul though?)20:02
corvusclarkb, hashar: i think that's accurate.  you could probably use codesearch to find a bunch manually -- enough to get a sense of whether it works.20:03
zbrfungi: sounds like a brave challenge, only if we would have endless time for a DIY that would avoid spending 5 minutes on something that already exists. by the time we have it up and running, it will already be too late to fix the original problem.20:04
hasharah the codesearch20:04
osmanlicilegifungi: so you mean there's no alternate solution for this issue?20:04
corvuszbr: your tone is not constructive20:05
corvuszbr: you have made the point that we should consider users and avoid breaking urls (with which, i believe, everyone agrees and many have been working diligently on)20:05
fungiosmanlicilegi: none that i've found yet, other than to try a different login.ubuntu.com openid. something causes the error to eventually cease for some of the users who have reported it, but i'm not sure if that's because of something which changed in the openid provider or with their specific openid account20:05
corvuszbr: you have also made the point that we should use google site tools to do that (with which many people disagree and have suggested alternatives).20:05
zbrcorvus: i should clarify: i am trying to find a practical solution.20:05
hasharclarkb: corvus: thank you :)20:06
zbri will not go deepend into that.20:06
osmanlicilegifungi: got it, thanks.20:06
fungizbr: your underlying assertion seems to be that open tools are not "practical" and that we should give up on them and use proprietary ones instead. i reject that premise20:06
clarkbosmanlicilegi: does your launchpad openid url have "weird" characters in it? I've seen other openid imlpementation not properly handle them in the past20:07
fungiand now i need to go run some errands20:07
clarkbthings like spaces20:07
clarkbosmanlicilegi: if you right clean on your lp user page and view source then grep for openid you should see your openid values20:07
fungiand grab early dinner. i hope to not be more than an hour or so20:07
clarkb(Not sure if there is a better/easier way to get those urls)20:07
clarkb*if you right click on your20:08
osmanlicilegiclarkb: only latin chars20:08
fungii was able to reproduce the problem myself at one point by creating a new launchpad account/openid and then trying to authenticate to wiki.o.o to autocreate an account. i picked apart all the available logging for it, but nothing jumped out at me indicating what was going wrong with the openid response20:10
funginot every new launchpad openid seems to cause this either, i seem to have gotten "lucky" with that particular test account20:12
osmanlicilegimaybe someone from canonical should examine it20:15
fungiit's not clear to me whether the error is coming from the mediawiki openid consumer implementation or the ubuntu one openid provider20:16
fungii just reproduced it with my test account again, so it does seem to still be affected at least20:16
openstackgerritAndrii Ostapenko proposed zuul/zuul-jobs master: Fix periodic image build jobs  https://review.opendev.org/70468020:19
fungianyway, i have to pop out, will be back as soon as i can20:20
*** rfolco has joined #openstack-infra20:24
*** armax has quit IRC20:30
*** armax has joined #openstack-infra20:32
*** diablo_rojo has quit IRC20:34
openstackgerritMerged opendev/system-config master: openafs-client: add option for OpenAFS cache location  https://review.opendev.org/70452320:35
*** Darcidride has quit IRC20:45
*** harlowja has joined #openstack-infra20:48
*** jamesmcarthur has joined #openstack-infra20:51
*** ralonsoh has quit IRC20:57
*** Lucas_Gray has joined #openstack-infra21:07
*** jamesmcarthur has quit IRC21:17
*** kjackal has quit IRC21:17
*** jamesmcarthur has joined #openstack-infra21:18
*** slaweq has quit IRC21:23
*** slaweq has joined #openstack-infra21:29
ianwinfra-root / fungi in particular: if you'd like to override governance.openstack.org & security.openstack.org to 23.253.245.150 and make sure it looks the same, and is letsencrypted, that would be great21:29
*** slaweq has quit IRC21:33
clarkbianw: looking now. Can you do similar for 104.130.141.217 as status.openstack.org no https to check though and it appears not all e-r graphs are generating21:42
*** slaweq has joined #openstack-infra21:43
clarkbianw: I can confirm both names have LE certs. I am able to browse most of the content but https://governance.openstack.org/election/ is forbidden21:43
ianwclarkb: yeah, i see nothing on the e-r page, but otherwise all content seems to be there for me on status.o.o @ 104...21:45
clarkbI'm manually running the e-r graph generation in the foreground to try and generate errors21:46
ianwAlias "/election/" "${AFS_ROOT}/election/" ... hrm21:46
clarkbah that gets published from another repo to another location? might need to add that one in before switching governance over?21:47
*** slaweq has quit IRC21:47
*** jackedin has quit IRC21:48
ianwyep for sure, let me dig21:48
clarkbor change it to be a url redirect instead?21:48
ianwno, i think it's not publishing21:48
clarkbah21:48
*** yamamoto has joined #openstack-infra21:50
fungiianw: back and testing now21:51
fungithere are something like four repos which publish stuff under governance.o.o21:53
ianwi think the elections is the only we've missed ... sigs tc uc should be there21:54
fungiyep, tested those and they seem to be fine21:54
fungii've got +2 on openstack/elections if we need to fix any publishing jobs in it21:55
*** yamamoto has quit IRC21:55
*** dpawlik has quit IRC21:55
ianwi always have to teach myself how it works from scratch, i don't have ajaeger's innate knowledge :)21:55
clarkbfungi: the changes to gerritlib to manage connections separately broke the watcher thread :/21:57
clarkbfungi: http://paste.openstack.org/show/788900/21:57
fungithat's from elastic-recheck?21:58
fungii guess we can revert and tag a patch release21:58
clarkbfungi: yes, I don't think its urgent since most of what e-r does is the periodic bug updates.21:58
clarkbI think I see the issue too21:58
*** rcernin has joined #openstack-infra22:00
openstackgerritClark Boylan proposed opendev/gerritlib master: Pass Gerrit not GerritConnection to GerritWatcher  https://review.opendev.org/70469322:01
*** jamesmcarthur has quit IRC22:01
clarkbfungi: ^ I think that will fix it, but not in a great spot to debug as I am trying to figure out why e-r isn't working in generating its graphs22:01
clarkbif someone wants to help push that through the newly added test job could be used to set up a stream watcher22:01
*** jamesmcarthur has joined #openstack-infra22:03
fungiianw: security.o.o from the new server lgtm22:03
clarkbits probably sufficient to set up a local watcher against review.o.o22:05
clarkband see that it works rather than crashing22:05
*** bdodd has quit IRC22:05
fungishould we try editing the copy on the replacement status.o.o?22:06
clarkbI can try that once I've got e-r graphs sorted out. I'm going to rerun graph generation with the verbose flag22:06
fungiokay, not wanting to interrupt your troubleshooting there22:06
ianwopenstack/transparency-policy is one i hadn't considered yet22:08
*** bdodd has joined #openstack-infra22:09
clarkbfungi: ok realized that my first patch is buggy as username and port etc all come from gerrit.connection now22:09
fungiianw: that hasn't had any changes since long before the move to zuul v3, so getting it to republish might be a challenge. where was it publishing to?22:10
fungiwe could just copy the content over into afs22:10
ianwfungi: not sure yet, but i don't think https://www.openstack.org/legal/transparency-policy/ is showing a "live" copy of it from us22:11
fungiahh, yeah, the osf is probably maintaining that anyway22:11
clarkbauthentication via openid is broken on review-dev beacuse we redirect back to review.opendev.org22:12
fungiianw: so it wasn't published to governance.o.o? what brought it to your attention?22:12
ianwfungi: https://opendev.org/openstack/project-config/src/branch/master/zuul.d/jobs.yaml#L592 ... but it looks like the job is never used22:13
fungiahh22:13
fungiso cruft22:14
openstackgerritClark Boylan proposed opendev/gerritlib master: Pass Gerrit not GerritConnection to GerritWatcher  https://review.opendev.org/70469322:14
clarkbfungi: ^ that seems to work using a simple startWatching then getEvent against review-dev22:14
openstackgerritAntoine Musso proposed zuul/zuul master: ansible manager: only failed if last ansible failed  https://review.opendev.org/70469922:15
openstackgerritAntoine Musso proposed zuul/zuul master: zuul-manage-ansible: add validate option  https://review.opendev.org/70470122:16
openstackgerritIan Wienand proposed openstack/project-config master: Add openstack/election AFS promotion job  https://review.opendev.org/70470222:18
clarkb'2020-01-28 22:18:29  DEBUG    [ergraph        ] Starting query for bug 1708704' <- seems like e-r is spending a lot of time on that query22:21
openstackbug 1708704 in OpenStack-Gate "yum client: [Errno 256] No more mirrors to try" [Undecided,New] https://launchpad.net/bugs/170870422:21
clarkbI suspect that query makes elasticsearch sad22:25
ianwfungi: that -> https://review.opendev.org/704703 Add promote-governance-election job <- should be the pair to 70470222:25
ianwi'll make a (wip) change to cleanup while i'm here22:25
fungithanks22:25
clarkbok it is OOMing22:26
clarkband it isn't processing any other bug queries22:26
fungiouch22:26
*** diablo_rojo has joined #openstack-infra22:27
clarkbI'm trying to rerun it without that query file in place22:27
clarkbit seems to be moving down its query list now22:30
clarkbif it manages to complete without OOMing I think we can remove that query for now until we sort out why it is so memory hungry22:31
clarkbwhile that is running I'm going to try and get a bike ride in as the sun just came out22:32
clarkbback in a bit22:32
*** jamesmcarthur has quit IRC22:34
fungiit's the e-r python process which was oom'ing yah?22:36
*** jamesmcarthur has joined #openstack-infra22:36
fungimaybe that query just  returns too many hits22:36
clarkbyes22:41
fungiapplying the query from https://opendev.org/opendev/elastic-recheck/src/branch/master/queries/1708704.yaml in kibana over the last 7 days takes quite a while to return22:45
*** jamesmcarthur has quit IRC22:45
fungithe loglines it returns don't look like they contain that message substring either22:46
*** rfolco has quit IRC22:46
fungithough it's still churning so i don't entirely trust the matched lines being displayed22:47
openstackgerritMerged openstack/project-config master: Add openstack/election AFS promotion job  https://review.opendev.org/70470222:48
*** rh-jelabarre has quit IRC22:50
*** jamesmcarthur has joined #openstack-infra22:50
*** signed8bit has quit IRC22:52
fungiclarkb: oh!22:53
fungitry it (on a small range) and expand one of the resulting hits22:53
fungiit seems to be all tripleo-ci jobs encoding thousands of lines as a single log entry somehow22:53
fungilooks like maybe embedded ansible output22:54
fungiso something like an entire ansible run to deploy openstack crammed into one logline in elasticsearch22:55
*** hashar has quit IRC22:56
fungiit wouldn't take too many of those to explode the process memory in elastic-recheck22:56
*** jamesmcarthur has quit IRC23:03
*** tkajinam has joined #openstack-infra23:06
*** slaweq has joined #openstack-infra23:11
*** slaweq has quit IRC23:16
*** dchen has joined #openstack-infra23:24
*** aaronsheffield has quit IRC23:29
*** rlandy is now known as rlandy|bbl23:32
clarkbfungi: that would do it, thanks for the investigating23:44
fungiwell, i'm unfortunately not coming up with great suggestions for how to fix that console log though23:44
clarkbweshay|ruck: zbr ^ is that something you might know about?23:44
clarkbfungi: I think we either stop indexing those files or fix the indexing of them to not treat the whole file as a single event23:44
fungiwell, that's the job's console log i think?23:45
clarkbfwiw my manual removal of that query allowed all those graphs to populate23:45
clarkbhrm console logs should parse properly /me looks more closely23:45
fungior those entries were tagged "console" anyway23:45
fungii didn't think to check the file path it returned for them23:45
clarkbfungi: ianw care to rereview 104.130.141.217 as http://status.openstack.org ? if it loosk good now I'll go ahead and update dns then can work to remove the old host tomorrow assuming nothing new pops up23:47
fungisure thing23:47
*** michael-beaver has quit IRC23:47
clarkbalso ianw corvus I'm reasonably happy with https://review.opendev.org/#/c/704693/ now after testing it locally against review-dev.o.o23:48
clarkbwe'll need to make a new release of gerritlib (maybe tomorrow) and that should fix e-r's irc bot23:48
clarkbfungi: https://7ae6145b803acbd97c06-6efb80aba20f3eb99d58ca8635d16bb1.ssl.cf2.rackcdn.com/704345/1/check/tripleo-ci-centos-7-containers-multinode-train/07ab6f6/logs/undercloud/var/log/extra/logstash.txt it is that file23:50
*** smarcet has joined #openstack-infra23:50
fungioh neat23:50
clarkbI think ti may be breaking parsing because timestamps and/or hostname data isn't there in the prefix of the lines23:50
fungii love that it's specifically named "logstash.txt"23:50
fungiso anyway, yeah, likely needs ingestion rules tweaked23:50
clarkb2020-01-28 22:16:25.546979 | Job console starting... <- good lines from job-output.txt23:50
clarkb2020-01-27 16:58:32 INFO  |     "INFO:tripleo_common.image.image_uploader:    xusr: true",  <- line from logstash.txt23:51
weshay|ruckclarkb, ah crud.. let me check out a recent change.. sorry23:51
clarkbweshay|ruck: thanks. let me pull up the parsing rules too as that might be a useful reference23:51
fungiianw: https://governance.openstack.org/election/ is rendering correctly from 23.253.245.150 since that job addition merged23:51
clarkbweshay|ruck: https://opendev.org/openstack/logstash-filters/src/branch/master/filters/openstack-filters.conf23:51
weshay|ruckthanks23:52
clarkbfungi: ianw agreed  Ithink governance.o.o via new webserver lgtm23:52
fungiclarkb: no https for status.o.o?23:53
fungilooks like it's listening on 443/tcp but not doing ssl there23:53
clarkbfungi: I bet that is the default vhost23:54
clarkbI don't think we ever actually configured ssl there23:54
fungiokay, just making sure i'm hitting the right vm23:55
fungiapache access log on that server shows my ip address making requests, so seems i am23:56
clarkbI've added the status01.openstack.org dns records. Will add status.o.o cname once you think you are happy with it23:56
fungiDelay in Elastic Search: Indexing behind by 29 hours23:57
fungiyeesh23:57
clarkbya that was  separate issue23:57
clarkbthough I wonder if the giant tripleo docs played a part in that23:57
fungialso doesn't seem to be rendering graphs for me at http://status.openstack.org/elastic-recheck/23:57
clarkb(we had a corrupt index shard, so I sacrificed it to the ether)23:57
clarkbwe were about 72 hours behind yesterday23:57
fungiouch23:58
clarkbfungi: hrm it renders for me. Anything in the js console?23:58
clarkbiirc it loads json file and renders it with js23:58
*** rkukura has quit IRC23:58
fungiwas just looking at page source, and yeah, seems that's all being done with linked scripts23:59
fungii miss when people used to make web sites with html as something more than a wrapper for javascript23:59
clarkbok I am able to reproduce after a hard refresh23:59
clarkbI bet I had the old js stuff cached and now the newer js stuff isn't working against our html :/23:59
clarkbso we fixed the puppetry and installed js that doesn't work with our html I bet23:59

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