Thursday, 2018-05-03

openstackgerritPaul Belanger proposed openstack-infra/openstack-zuul-jobs master: Remove fedora-26 nodesets  https://review.openstack.org/56591400:00
*** wendar has quit IRC00:07
ianwok, initial backup of review01 all done00:11
ianwcore/signal.c: In function ‘uwsgi_register_signal’:00:15
ianwin my day, gcc outputted plain old ascii, and that's the way we liked it00:16
pabelanger:)00:16
ianwwe wore an onion on our belt, which was the style at the time ...00:16
*** lbragstad has quit IRC00:17
openstackgerritPaul Belanger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy-ubuntu-xenial-3-node  https://review.openstack.org/56591800:18
*** dingyichen has joined #openstack-infra00:20
ianwit might be worth figuring out how to get uwsgi doing linux any wheels.  it's not something i've looked at before00:21
*** wendar has joined #openstack-infra00:23
*** mriedem_inlaws has quit IRC00:23
clarkbaiui the basic idea is you build them against a really minimal rhel5 like base set of libs00:24
clarkband they have a docker image they publish to build thrm in00:25
pabelangerclarkb: you might be looking a ze02 which is old format, ze02_openstack_org is right one00:25
pabelangeralso, do you mind a review on https://review.openstack.org/564506/ to fix gerrit periodic jobs00:25
ianwclarkb: right, i'm assuming someone has plugged this into github releases/travis-ci etc00:26
ianwfungi: when around, https://review.openstack.org/#/c/541791/ is a little fix for git-review and later gerrits.  we still can't really test later gerrits with git-review because of https://review.openstack.org/#/c/552288/ but the change is probably worth review00:29
ianwhttps://github.com/joerick/cibuildwheel looks promising00:30
*** yamamoto has joined #openstack-infra00:36
ianwand to wrap it up ... https://github.com/unbit/uwsgi/blob/54f6d87aea896eff3ae3410758faed69fb170690/.travis.yml#L37 is why it's all too hard i guess.  too many libraries.  if only there was a concise and portable way those libraries could be listed, perhaps in a formatted text file with the project00:36
pabelangerIRK00:39
pabelangerIKR*00:39
pabelangerspeaking of wheels, I just pushed up https://github.com/Infinidat/munch/pull/32 to have munch start producing them00:39
*** mriedem has joined #openstack-infra00:41
*** yamamoto has quit IRC00:41
*** camunoz has quit IRC00:44
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: WIP: Add logo and move menu to top right  https://review.openstack.org/56590900:45
*** r-daneel has joined #openstack-infra00:46
ianwsudo $apxs -i -c mod_proxy_uwsgi.c <--- ok, so that's my confusion over uwsgi ... we're actually building the apache module directly.  i knew there was something odd00:46
*** r-daneel_ has joined #openstack-infra00:49
*** r-daneel has quit IRC00:50
*** r-daneel_ is now known as r-daneel00:50
pabelangerclarkb: corvus: fungi: following up here, I am not 100% but I think the job results table on review is now missing: https://review.openstack.org/#/c/565550/00:53
pabelangeri am not exactly sure where to look to confirm that00:53
clarkbit loads js in the header content for the website00:55
clarkbI would start by opening your browser debugger and see if that js is loaded00:55
pabelangerhmm, see a syntax error00:57
pabelanger Syntax error, unrecognized expression: :visible:00:57
pabelangerlets see what the googles says00:58
pabelangerlibjs-jquery                     1.11.3+dfsg-400:59
pabelangeris now installed00:59
pabelangermaybe we're now using removed function00:59
tristanCpabelanger: for this version, you need to have ':visible' without the trailing ':'01:02
clarkbhuh01:02
clarkbits in system-config iirc if you want to push a patch tristanC01:02
clarkbor pabelanger01:03
pabelangerI'm trying to see if I can test that in browser01:03
pabelangerbut have no idea what I am doing :)01:03
tristanCclarkb: but the question is why the previous server had a different jquery installed?01:03
*** sai_ has quit IRC01:03
clarkbtristanC: probably because it was an older version of ubuntu01:04
pabelangerya, trusty was 1.7.2+dfsg-2ubuntu101:05
*** threestrands has joined #openstack-infra01:05
*** sai_ has joined #openstack-infra01:06
pabelangerI can fix system-config, but unsure how to test it01:06
openstackgerritTristan Cacqueray proposed openstack-infra/system-config master: hideci: fix compatibility with jquery>=v1.11  https://review.openstack.org/56592401:06
*** r-daneel has quit IRC01:06
pabelangertristanC: thanks +201:06
*** yamahata has quit IRC01:08
clarkb+2 as well but nit approving as I cant watch it this evening01:08
clarkbneed to make dinner01:08
pabelangerianw: maybe if you have time^ otherwise, I'll have time first thing in the morning01:09
ianwpabelanger: oh, yep, i'll watch it when it goes through01:10
pabelangergreat01:12
*** aeng has joined #openstack-infra01:13
*** zhangfei has joined #openstack-infra01:18
*** Guest10014 is now known as jlvillal01:19
*** namnh has joined #openstack-infra01:31
*** namnh has quit IRC01:31
*** rosmaita has quit IRC01:32
*** Nil_ has quit IRC01:34
*** yamamoto has joined #openstack-infra01:38
*** dpawlik has joined #openstack-infra01:44
*** yamamoto has quit IRC01:44
openstackgerritMerged openstack-infra/system-config master: hideci: fix compatibility with jquery>=v1.11  https://review.openstack.org/56592401:47
*** dpawlik has quit IRC01:48
*** yamamoto has joined #openstack-infra01:50
*** r-daneel has joined #openstack-infra01:53
*** gcb has joined #openstack-infra02:00
*** hongbin has joined #openstack-infra02:12
*** annp has joined #openstack-infra02:12
tristanCianw: not sure why, but gerrit may still be referencing the old hidejs script, perhaps a gerrit restart would fix this02:15
*** psachin has joined #openstack-infra02:17
*** lbragstad has joined #openstack-infra02:24
*** dchen has joined #openstack-infra02:35
mnasertristanC: I think we need to wait for puppet to do a pass on infra which happens every hour or so to update that file02:44
tristanCmnaser: the file has been updated, but for somereason gerrit is adding the md5sum of the previous version02:51
*** mriedem has quit IRC02:52
*** spsurya has joined #openstack-infra02:54
ianwtristanC: oh, hmmm, i think i know something about that, but i can't remember what that is :)  let me check my logs03:02
ianwJun 17 12:33:54 <jhesketh>      ianw: right, sometimes that doesn't work and we don't know why03:05
ianwJun 17 12:34:07 <jhesketh>      ianw: the solution is to just touch the header03:05
*** honza has quit IRC03:06
tristanCianw: do you want a noop review on the GerritHeader or can you touch it through ssh?03:09
*** dpawlik has joined #openstack-infra03:10
openstackgerritTristan Cacqueray proposed openstack-infra/system-config master: hideci: touch the GerritSiteHeader  https://review.openstack.org/56594403:10
ianwtristanC: ok, just did that :)  i think it's looking happier03:12
*** dpawlik has quit IRC03:14
ianwheh, we alreay have the reload i suggested in the puppet03:19
*** nicolasbock has quit IRC03:21
*** ramishra has joined #openstack-infra03:32
*** hamzy has joined #openstack-infra03:34
*** eernst has joined #openstack-infra03:34
*** udesale has joined #openstack-infra03:36
*** Gorian has quit IRC03:36
*** namnh has joined #openstack-infra03:38
*** Gorian has joined #openstack-infra03:41
*** zhangfei has quit IRC03:41
*** hongbin has quit IRC03:42
dmsimardpabelanger: just got a sudo failure for gerrit wanting to do a touch on hideci.js ?03:44
dmsimardon review0103:44
*** lpetrut has joined #openstack-infra03:44
*** jogo has quit IRC03:45
*** janki has joined #openstack-infra03:47
*** jogo has joined #openstack-infra03:50
*** jogo has joined #openstack-infra03:50
*** eernst has quit IRC03:52
*** zhangfei has joined #openstack-infra03:53
*** eernst has joined #openstack-infra03:57
*** jchhatbar has joined #openstack-infra04:00
*** germs has quit IRC04:02
*** janki has quit IRC04:03
openstackgerritMerged openstack-infra/zuul master: Update documentation URLs  https://review.openstack.org/56591004:11
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Don't create db accounts unencrypted  https://review.openstack.org/56594804:15
*** kjackal has joined #openstack-infra04:17
*** lpetrut has quit IRC04:18
*** eernst has quit IRC04:20
*** lpetrut has joined #openstack-infra04:21
*** ykarel has joined #openstack-infra04:27
*** jpena|off has quit IRC04:29
*** ssbarnea has quit IRC04:29
*** weshay has quit IRC04:29
*** rasca has quit IRC04:30
*** rnoriega_ has quit IRC04:31
*** haleyb_ has quit IRC04:31
*** betherly_ has quit IRC04:31
*** dmanchad has quit IRC04:31
*** radez has quit IRC04:31
*** jpena|off has joined #openstack-infra04:31
*** pabelanger has quit IRC04:31
*** nhicher has quit IRC04:31
*** ltomasbo has quit IRC04:31
*** rook has quit IRC04:31
*** ssbarnea has joined #openstack-infra04:32
*** pabelanger has joined #openstack-infra04:32
*** pgadiya has joined #openstack-infra04:32
*** nhicher has joined #openstack-infra04:32
*** rnoriega has joined #openstack-infra04:33
*** radez has joined #openstack-infra04:33
*** dchen has quit IRC04:34
*** pgadiya has quit IRC04:34
*** haleyb has joined #openstack-infra04:34
*** rook has joined #openstack-infra04:34
*** rook is now known as Guest9498304:34
*** weshay has joined #openstack-infra04:35
*** betherly_ has joined #openstack-infra04:35
*** rasca has joined #openstack-infra04:36
*** dchen has joined #openstack-infra04:36
*** dmanchad has joined #openstack-infra04:38
*** ianychoi_ has joined #openstack-infra04:38
*** gema has quit IRC04:40
*** gema has joined #openstack-infra04:41
*** ianychoi has quit IRC04:41
*** gyee has quit IRC04:41
*** lpetrut has quit IRC04:43
*** links has joined #openstack-infra04:46
ianwdmsimard: ahhh, that would explain it!04:55
*** sree_ has joined #openstack-infra04:56
*** sree_ is now known as Guest4188804:57
ianwdmsimard: oh, no it wouldn't.  yeah that was me04:57
ianwi was thinking somehow the puppet call doing that might have been running as gerrit204:57
*** Guest41888 has quit IRC04:57
*** dhajare has joined #openstack-infra04:59
*** e0ne has joined #openstack-infra05:03
*** e0ne has quit IRC05:03
*** aeng has quit IRC05:04
*** yamahata has joined #openstack-infra05:06
*** markvoelker has quit IRC05:08
*** dchen has quit IRC05:11
*** dpawlik has joined #openstack-infra05:11
*** dpawlik has quit IRC05:15
*** dchen has joined #openstack-infra05:19
*** dingyichen has quit IRC05:20
*** dchen has quit IRC05:20
*** dingyichen has joined #openstack-infra05:20
*** Douhet has quit IRC05:26
*** armaan has joined #openstack-infra05:28
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Add tox-py36 jobs  https://review.openstack.org/56588105:31
*** sree_ has joined #openstack-infra05:34
*** sree_ is now known as Guest2225805:34
*** wolverineav has joined #openstack-infra05:34
*** iranzo has joined #openstack-infra05:36
*** quiquell|off is now known as quiquell|rover05:45
*** quiquell|rover is now known as quiquell05:46
*** pcichy has joined #openstack-infra05:51
*** wolverineav has quit IRC05:52
*** jtomasek has quit IRC05:59
openstackgerritOpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/56596606:02
*** dchen has joined #openstack-infra06:06
*** dingyichen has quit IRC06:08
*** markvoelker has joined #openstack-infra06:09
*** jtomasek has joined #openstack-infra06:11
*** Douhet has joined #openstack-infra06:14
*** andreas_s has joined #openstack-infra06:16
openstackgerritMerged openstack-infra/zuul master: Don't create db accounts unencrypted  https://review.openstack.org/56594806:18
*** jtomasek has quit IRC06:18
*** jtomasek has joined #openstack-infra06:19
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/zuul master: Do not fail so early on missing required-projects  https://review.openstack.org/56597306:19
*** dchen has quit IRC06:21
*** dingyichen has joined #openstack-infra06:21
openstackgerritMerged openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/56596606:21
*** kjackal has quit IRC06:23
*** aojea has joined #openstack-infra06:25
*** kjackal has joined #openstack-infra06:29
*** lpetrut has joined #openstack-infra06:29
*** dhajare has quit IRC06:37
*** markvoelker has quit IRC06:44
*** dhajare has joined #openstack-infra06:51
*** eumel8 has joined #openstack-infra06:52
*** pcaruana has joined #openstack-infra06:53
*** rcernin has quit IRC06:58
*** threestrands has quit IRC06:58
*** ccamacho has joined #openstack-infra06:58
*** ianychoi_ is now known as ianychoi07:00
openstackgerritTobias Henkel proposed openstack-infra/zuul master: WIP: test required project not found error  https://review.openstack.org/56597907:07
*** sshnaidm|rover has quit IRC07:08
*** dpawlik has joined #openstack-infra07:12
*** quiquell is now known as quiquell|afk07:12
*** jbadiapa has joined #openstack-infra07:16
*** dpawlik has quit IRC07:16
*** dhajare has quit IRC07:17
*** tesseract has joined #openstack-infra07:18
*** kjackal has quit IRC07:25
*** kjackal has joined #openstack-infra07:25
*** florianf has joined #openstack-infra07:28
*** tosky has joined #openstack-infra07:29
*** quiquell|afk is now known as quiquell07:29
*** ssbarnea_ has joined #openstack-infra07:30
*** ssbarnea_ has quit IRC07:31
*** Hal has joined #openstack-infra07:35
*** jpena|off is now known as jpena07:35
*** Hal is now known as Guest9437507:35
*** ykarel is now known as ykarel|lunch07:38
*** markvoelker has joined #openstack-infra07:41
*** alexchadin has joined #openstack-infra07:43
*** ssbarnea_ has joined #openstack-infra07:44
*** zhangfei has quit IRC07:45
*** apetrich_ has joined #openstack-infra07:50
*** amoralej|off is now known as amoralej07:50
*** dhajare has joined #openstack-infra07:54
*** jpich has joined #openstack-infra07:55
*** sshnaidm|rover has joined #openstack-infra07:55
*** electrofelix has joined #openstack-infra07:57
*** zhangfei has joined #openstack-infra07:58
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Don't validate references for excluded config items  https://review.openstack.org/56597307:58
*** hs365 has joined #openstack-infra08:05
*** ramishra_ has joined #openstack-infra08:06
*** ramishra has quit IRC08:07
*** shardy has joined #openstack-infra08:08
*** hs366 has quit IRC08:09
*** gfidente has joined #openstack-infra08:13
*** gfidente has joined #openstack-infra08:13
*** markvoelker has quit IRC08:13
*** yamahata has quit IRC08:14
*** wolverineav has joined #openstack-infra08:15
*** lucas-afk is now known as lucasagomes08:17
*** ykarel|lunch is now known as ykarel08:25
*** wolverineav has quit IRC08:27
*** alexchad_ has joined #openstack-infra08:27
*** wolverineav has joined #openstack-infra08:28
*** alexchadin has quit IRC08:31
*** e0ne has joined #openstack-infra08:31
*** wolverineav has quit IRC08:32
*** derekh has joined #openstack-infra08:36
*** hs365 has quit IRC08:39
*** hs367 has joined #openstack-infra08:39
*** dingyichen has quit IRC08:42
*** links has quit IRC08:43
*** links has joined #openstack-infra08:44
*** Guest22258 has quit IRC08:45
*** hemna_ has quit IRC08:46
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Only emit parent-change-enqueued if needed  https://review.openstack.org/56319408:48
*** namnh has quit IRC08:48
*** namnh has joined #openstack-infra08:48
*** slaweq has quit IRC08:56
*** slaweq has joined #openstack-infra08:57
*** jrist has quit IRC08:57
*** dbecker has joined #openstack-infra08:58
*** jrist has joined #openstack-infra09:01
openstackgerritMerged openstack-infra/zuul-jobs master: Add tox-py36 job  https://review.openstack.org/56588409:03
*** markvoelker has joined #openstack-infra09:10
*** apetrich_ has quit IRC09:11
*** dpawlik has joined #openstack-infra09:13
*** dpawlik has quit IRC09:17
openstackgerritMerged openstack-infra/zuul master: Trivial: Update pypi url to new url  https://review.openstack.org/56324809:27
*** zoli is now known as zoli|lunch09:30
*** khappone has joined #openstack-infra09:32
*** khappone_ has quit IRC09:36
*** pbourke has joined #openstack-infra09:42
openstackgerritTobias Urdin proposed openstack-infra/system-config master: Add puppetlabs apt puppet5 pool  https://review.openstack.org/56586709:43
*** markvoelker has quit IRC09:44
*** yolanda_ has joined #openstack-infra09:49
*** yolanda has quit IRC09:51
*** yolanda has joined #openstack-infra09:56
*** yolanda_ has quit IRC09:59
*** ramishra_ has quit IRC10:02
*** yolanda has quit IRC10:02
*** ramishra has joined #openstack-infra10:03
*** alexchad_ has quit IRC10:07
*** Guest94375 has quit IRC10:10
*** namnh has quit IRC10:11
*** pcichy has quit IRC10:12
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Extend github testing using app auth  https://review.openstack.org/56329710:12
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Test parent-change-enqueued with github  https://review.openstack.org/56324210:12
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Optionally record created github clients in tests  https://review.openstack.org/56340210:12
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Limit search scope of getChangesDependingOn to tenant  https://review.openstack.org/56319610:13
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Remove getPullBySha from FakeGithubConnection  https://review.openstack.org/56341610:13
*** kjackal has quit IRC10:14
*** yolanda has joined #openstack-infra10:15
*** yamamoto has quit IRC10:21
*** yolanda has quit IRC10:25
*** verdurin has joined #openstack-infra10:28
*** nicolasbock has joined #openstack-infra10:31
*** edmondsw has joined #openstack-infra10:31
*** edmondsw has quit IRC10:36
*** yamamoto has joined #openstack-infra10:37
*** yolanda has joined #openstack-infra10:39
*** annp has quit IRC10:40
*** markvoelker has joined #openstack-infra10:41
*** alexchadin has joined #openstack-infra10:44
*** alexchadin has quit IRC10:49
*** dtantsur|afk is now known as dtantsur10:50
*** alexchadin has joined #openstack-infra10:54
*** jpena is now known as jpena|lunch10:58
*** slaweq has quit IRC10:58
*** apetrich_ has joined #openstack-infra11:00
*** apetrich has quit IRC11:05
*** persia has joined #openstack-infra11:06
openstackgerritLuke Hinds proposed openstack-infra/project-config master: Ends project gating for Bandit  https://review.openstack.org/56602811:09
*** lucasagomes is now known as lucas-hungry11:10
*** apetrich_ has quit IRC11:10
*** slaweq has joined #openstack-infra11:10
*** zoli|lunch is now known as zoli11:13
*** markvoelker has quit IRC11:14
*** slaweq has quit IRC11:15
*** zhangfei has quit IRC11:18
*** jcoufal has joined #openstack-infra11:22
*** kjackal has joined #openstack-infra11:28
*** hs367 has quit IRC11:28
*** panda|off is now known as panda|lunch11:33
*** kjackal has quit IRC11:36
*** kjackal has joined #openstack-infra11:36
*** ldnunes has joined #openstack-infra11:39
*** yamamoto has quit IRC11:48
*** rosmaita has joined #openstack-infra11:48
*** eharney has quit IRC11:49
*** jamesmcarthur has joined #openstack-infra11:52
*** salv-orlando has joined #openstack-infra11:53
*** amoralej is now known as amoralej|off11:53
*** amoralej|off is now known as amoralej|lunch11:54
*** yamamoto has joined #openstack-infra11:55
*** yamamoto_ has joined #openstack-infra11:56
*** quiquell is now known as quiquell|lunch11:56
*** yamamoto_ has quit IRC11:57
*** salv-orlando has quit IRC11:58
*** yamamoto_ has joined #openstack-infra11:59
*** yamamoto has quit IRC12:00
*** armaan has quit IRC12:00
*** armaan has joined #openstack-infra12:01
*** wolverineav has joined #openstack-infra12:01
*** yamamoto_ has quit IRC12:02
*** kjackal has quit IRC12:03
*** sshnaidm|rover is now known as sshnaidm|rover|a12:08
*** sshnaidm|rover|a is now known as sshnaidm|rover12:09
*** gcb has quit IRC12:09
*** tpsilva has joined #openstack-infra12:11
*** jpena|lunch is now known as jpena12:11
*** markvoelker has joined #openstack-infra12:11
*** yamamoto has joined #openstack-infra12:13
*** quiquell|lunch is now known as quiquell12:16
*** edmondsw has joined #openstack-infra12:16
*** lucas-hungry is now known as lucasagomes12:16
*** panda|lunch is now known as panda12:17
*** apetrich has joined #openstack-infra12:19
*** markvoelker has quit IRC12:20
*** markvoelker has joined #openstack-infra12:20
*** yamamoto has quit IRC12:20
*** yamamoto has joined #openstack-infra12:21
*** yamamoto has quit IRC12:26
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Improve test output on assertHistory fail  https://review.openstack.org/56604812:28
*** njohnston has quit IRC12:29
*** njohnston has joined #openstack-infra12:31
*** rlandy has joined #openstack-infra12:34
openstackgerritFatih Degirmenci proposed openstack-infra/zuul master: Add CentOS 7 environment setup instructions  https://review.openstack.org/56494812:39
*** Hal has joined #openstack-infra12:52
*** kgiusti has joined #openstack-infra12:52
*** Hal is now known as Guest5908212:52
*** armaan has quit IRC12:54
openstackgerritLuke Hinds proposed openstack-infra/project-config master: Ends project gating for Bandit  https://review.openstack.org/56602812:55
*** nicolasbock has quit IRC12:57
*** yolanda has quit IRC12:57
*** beagles is now known as beagles_mtg12:57
*** psachin has quit IRC12:59
*** kjackal has joined #openstack-infra13:00
*** jchhatbar has quit IRC13:02
*** bobh has joined #openstack-infra13:05
*** honza has joined #openstack-infra13:06
*** honza is now known as Guest9522413:07
*** zoli is now known as zoli|afk13:07
*** Guest95224 is now known as honza_13:07
*** zoli|afk is now known as zoli13:07
*** slaweq has joined #openstack-infra13:10
*** yamamoto has joined #openstack-infra13:11
*** yolanda has joined #openstack-infra13:11
*** dpawlik has joined #openstack-infra13:12
*** nicolasbock has joined #openstack-infra13:14
*** Goneri has joined #openstack-infra13:14
*** slaweq has quit IRC13:16
*** dpawlik has quit IRC13:17
*** yamamoto has quit IRC13:17
*** yamamoto has joined #openstack-infra13:17
*** slaweq has joined #openstack-infra13:19
*** mriedem has joined #openstack-infra13:20
*** dhill_ has joined #openstack-infra13:20
*** yamamoto has quit IRC13:22
*** amoralej|lunch is now known as amoralej13:26
*** ykarel is now known as ykarel|away13:27
*** eumel8 has quit IRC13:28
*** VW has joined #openstack-infra13:30
*** agopi has quit IRC13:31
*** eharney has joined #openstack-infra13:32
*** ykarel|away has quit IRC13:32
*** dayou_ has joined #openstack-infra13:44
*** pcichy has joined #openstack-infra13:46
*** ssbarnea_ has quit IRC13:59
*** rajinir has joined #openstack-infra14:00
*** annp has joined #openstack-infra14:02
*** jamesmcarthur has quit IRC14:02
*** agopi has joined #openstack-infra14:02
*** esberglu has joined #openstack-infra14:06
*** hemna_ has joined #openstack-infra14:06
*** eernst has joined #openstack-infra14:07
*** andreas_s has quit IRC14:10
*** beagles_mtg is now known as beagles14:13
*** andreas_s has joined #openstack-infra14:14
esbergluHi! PowerVM CI has been having connection issues between jenkins and nodepool nodes.14:16
esbergluSeeing errors like here https://wiki.jenkins.io/display/JENKINS/Remoting+issue14:16
esbergluThis started happening after upgrading our cloud from ocata to queens14:17
esbergluAnyone here run into this before or have any ideas?14:18
*** yolanda has quit IRC14:19
*** yamamoto has joined #openstack-infra14:20
*** honza_ is now known as honza14:21
*** rkukura has joined #openstack-infra14:22
*** apetrich has quit IRC14:23
*** esberglu has quit IRC14:23
pabelangeryou may want to try in #openstack-third-party-ci to see if any others have seen this issue, we haven't been running jenkins for a while now.14:24
*** hongbin has joined #openstack-infra14:24
*** sshnaidm|rover is now known as sshnaidm|afk14:26
*** mikal has quit IRC14:27
fungii was about to suggest it could be related to the gerrit ip address change, but they're already gone14:31
fungiand they don't appear to have popped into #openstack-third-party-ci yet either14:32
*** esberglu has joined #openstack-infra14:34
*** pabelanger has quit IRC14:35
*** pabelanger has joined #openstack-infra14:35
*** esberglu_ has joined #openstack-infra14:36
*** r-daneel has quit IRC14:36
*** dayou_ has quit IRC14:37
*** germs has joined #openstack-infra14:38
*** germs has quit IRC14:38
*** germs has joined #openstack-infra14:38
*** kaisers has quit IRC14:38
*** germs has quit IRC14:39
*** kaisers has joined #openstack-infra14:39
*** germs has joined #openstack-infra14:39
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Add promote pipeline  https://review.openstack.org/56608914:40
*** esberglu has quit IRC14:40
fungiooh!14:40
*** esberglu_ is now known as esberglu14:40
rcarrillocruzo_O14:42
rcarrillocruzi'm curious about ^14:42
*** kiennt26 has joined #openstack-infra14:43
*** yamamoto has quit IRC14:46
*** _d34dh0r53_ is now known as d34dh0r5314:46
fungihrm, looking back more carefully at what esberglu said, it was between jenkins and slaves, not jenkins and gerrit, so not going to be related to the gerrit maintenance yesterday anyway14:48
*** alexchadin has quit IRC14:49
*** mikal has joined #openstack-infra14:50
mnaserhey infra, is there a document about where the pre-cached files in images are located?14:53
*** yolanda has joined #openstack-infra14:53
mnaserhitting timeouts downloading cirros so want to know where it's located, we're checking /home/zuul/cache/files/cirros-0.4.0-x86_64-disk.img presently14:53
*** afazekas_ has quit IRC14:54
*** Nil_ has joined #openstack-infra14:58
*** afazekas has joined #openstack-infra14:58
*** r-daneel has joined #openstack-infra14:59
openstackgerritMerged openstack-infra/project-config master: Ends project gating for Bandit  https://review.openstack.org/56602815:00
*** yamahata has joined #openstack-infra15:00
*** links has quit IRC15:06
pabelangermnaser: /opt/cache/files/15:07
pabelangermnaser: you can see our build logs: http://nb01.openstack.org/centos-7-0000009028.log15:07
*** jamesmcarthur has joined #openstack-infra15:09
mnaserpabelanger: oh, sweet, thank you15:11
*** dhajare has quit IRC15:12
mnaserpabelanger: https://review.openstack.org/566095 Switch to using the cached version of CirrOS    thanks ;)15:13
*** dpawlik has joined #openstack-infra15:13
*** slaweq has quit IRC15:17
*** dpawlik has quit IRC15:17
*** annp has quit IRC15:20
*** janki has joined #openstack-infra15:24
*** ccamacho has quit IRC15:24
*** ccamacho1 has joined #openstack-infra15:24
*** andreas_s has quit IRC15:27
*** andreas_s has joined #openstack-infra15:28
*** pblaho has quit IRC15:31
*** apetrich has joined #openstack-infra15:33
*** ccamacho1 has quit IRC15:35
*** andreas_s has quit IRC15:37
*** e0ne has quit IRC15:40
mwhahahahey folks, can someone add the tripleo-core to the tripleo-ha-utils group in gerrit for me? we added the repo but we forgot to ask to have someone added to it15:40
*** slaweq has joined #openstack-infra15:40
*** felipemonteiro__ has joined #openstack-infra15:41
*** andreas_s has joined #openstack-infra15:42
*** ssbarnea_ has joined #openstack-infra15:43
*** yamamoto has joined #openstack-infra15:44
*** felipemonteiro_ has quit IRC15:45
*** auristor1 has quit IRC15:47
*** gyee has joined #openstack-infra15:47
*** auristor has joined #openstack-infra15:49
*** pcaruana has quit IRC15:50
*** bdodd has joined #openstack-infra15:52
clarkbmwhahaha: looking15:54
openstackgerritPaul Belanger proposed openstack-infra/nodepool master: WIP: Add fedora mirror to nodepool dsvm jobs  https://review.openstack.org/56610215:55
clarkbmwhahaha: done15:55
mwhahahaclarkb: thanks15:55
*** yamamoto has quit IRC15:55
*** hamzy has quit IRC15:56
*** andreas_s has quit IRC15:56
*** kiennt26 has quit IRC15:57
*** panda is now known as panda|off15:57
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Add logo and move menu to top right  https://review.openstack.org/56590916:01
*** andreas_s has joined #openstack-infra16:01
*** quiquell is now known as quiquell|off16:02
*** yolanda has quit IRC16:06
*** andreas_s has quit IRC16:10
openstackgerritMerged openstack-infra/zuul master: Don't validate references for excluded config items  https://review.openstack.org/56597316:11
*** yamahata has quit IRC16:12
openstackgerritMerged openstack-infra/zuul-website master: Replace background image  https://review.openstack.org/56590816:15
*** felipemonteiro__ has quit IRC16:17
*** felipemonteiro_ has joined #openstack-infra16:17
weshaycorvus, thanks for fixing up zuul w/ regards to the python-tripleoclient :)16:17
*** pbourke has quit IRC16:17
weshayrepo16:17
*** pbourke has joined #openstack-infra16:19
clarkbpabelanger: as a followup to your question about java 8 and memory re review.o.o yesterday. Melody reports what appears to be really good memory consumption after a day of use. mean use is 10GB and max is 20GB16:20
clarkbpabelanger: we should keep an eye on it but that looks good to start16:20
*** andreas_s has joined #openstack-infra16:20
*** lpetrut_ has joined #openstack-infra16:21
pabelangerclarkb: do you remember the numbers on java 7?16:22
clarkbpabelanger: not for ~18 hours after a restart but steady state was basically max at the limit of assigned heap space ~48GB with gc going up and down not too far below that16:23
clarkbpabelanger: checking what the numbers look like in a week or two will probably be a good indicator16:23
pabelangeryah, lets hope it holds out!16:23
corvusweshay: np, thanks for finding the bug :)16:24
*** lpetrut has quit IRC16:25
*** lpetrut_ has quit IRC16:26
clarkbpabelanger: iirc melody shoves all this data in /tmp so we lost the old content when we moved servers. We can try to extract the old data if we really need it but I don't think we do (since new server isn't acting improperly)16:27
*** ramishra has quit IRC16:27
*** tesseract has quit IRC16:28
*** hongbin has quit IRC16:28
*** hongbin has joined #openstack-infra16:29
*** slaweq has quit IRC16:29
*** slaweq has joined #openstack-infra16:29
*** slaweq has quit IRC16:30
pabelangerYah, rolling forward seems like a fine idea16:30
*** andreas_s has quit IRC16:35
*** jd_ has quit IRC16:38
openstackgerritPaul Belanger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy-ubuntu-xenial-3-node  https://review.openstack.org/56591816:44
pabelangerone step forward to removing another legacy node from ozj16:45
*** andreas_s has joined #openstack-infra16:49
*** yamahata has joined #openstack-infra16:50
*** notmyname has quit IRC16:50
*** jamesmcarthur has quit IRC16:50
*** thedac has quit IRC16:51
clarkbcorvus: does our use of github project zuul config exclusions mean we should prioritize a scheduler restart for 565973?16:51
clarkbcorvus: I think we've largely avoided trouble by those projects not having zuul configs of their own at this point?16:51
*** notmyname has joined #openstack-infra16:51
*** pfalleno1 has quit IRC16:51
*** pfallenop has joined #openstack-infra16:52
*** yolanda has joined #openstack-infra16:53
*** jamesmcarthur has joined #openstack-infra16:54
*** rpioso is now known as rpioso|eat16:55
*** andreas_s has quit IRC16:55
clarkbmaybe that means it isn't urgent for us16:56
*** thedac has joined #openstack-infra16:56
*** jamesmcarthur has quit IRC16:57
*** tesseract has joined #openstack-infra16:57
*** derekh has quit IRC16:59
*** yolanda_ has joined #openstack-infra17:01
*** jpena is now known as jpena|off17:01
*** dtantsur is now known as dtantsur|afk17:02
*** shardy has quit IRC17:03
*** hamzy has joined #openstack-infra17:03
clarkbinfra-root I'd appreciate your thoughts on https://review.openstack.org/#/c/564338/ mordred may find that particularly interesting. Basicalyl remove localhost from default kick.sh host list to avoid inadverdent restoration of puppeting17:03
*** yolanda has quit IRC17:04
*** yolanda has joined #openstack-infra17:05
openstackgerritPaul Belanger proposed openstack-infra/nodepool master: WIP: Add fedora mirror to nodepool dsvm jobs  https://review.openstack.org/56610217:05
*** lucasagomes is now known as lucas-afk17:08
*** yolanda_ has quit IRC17:08
mordredclarkb: ++17:09
*** jpich has quit IRC17:10
*** slaweq has joined #openstack-infra17:11
pabelangerwfm17:11
corvusclarkb: let's say we should definitely do it by eod friday :)17:12
corvus(restart)17:13
*** dpawlik has joined #openstack-infra17:14
*** ssbarnea_ has quit IRC17:14
clarkbcorvus: ok17:15
*** slaweq has quit IRC17:16
*** e0ne has joined #openstack-infra17:16
*** trown is now known as trown|lunch17:17
*** dpawlik has quit IRC17:18
*** e0ne has quit IRC17:18
*** dhajare has joined #openstack-infra17:22
*** janki has quit IRC17:24
*** jd_ has joined #openstack-infra17:25
*** tesseract has quit IRC17:25
*** jamesmcarthur has joined #openstack-infra17:26
*** jamesmcarthur has quit IRC17:34
*** e0ne has joined #openstack-infra17:37
*** shaner has quit IRC17:41
*** florianf has quit IRC17:41
*** shaner has joined #openstack-infra17:42
*** spsurya has quit IRC17:42
*** VW_ has joined #openstack-infra17:46
*** shaner has quit IRC17:48
*** VW has quit IRC17:49
*** VW_ has quit IRC17:50
*** VW has joined #openstack-infra17:50
*** udesale has quit IRC17:50
openstackgerritMerged openstack-infra/project-config master: Adding networking-ansible to the project list  https://review.openstack.org/56537017:50
*** Guest94983 is now known as rook17:54
*** jd_ has quit IRC17:56
*** jd_ has joined #openstack-infra17:56
*** zoli is now known as zoli|goe17:57
*** zoli|goe is now known as zoli|gone17:57
*** zoli|gone is now known as zoli17:58
*** kjackal has quit IRC18:07
*** kjackal has joined #openstack-infra18:07
*** SumitNaiksatam has joined #openstack-infra18:08
*** shaner has joined #openstack-infra18:10
*** VW has quit IRC18:15
*** kjackal has quit IRC18:15
*** shaner has quit IRC18:15
*** VW has joined #openstack-infra18:16
*** VW_ has joined #openstack-infra18:17
*** VW has quit IRC18:17
*** SumitNaiksatam has quit IRC18:18
fricklerouch, sounds like spectre isn't over yet https://www.heise.de/ct/artikel/Exclusive-Spectre-NG-Multiple-new-Intel-CPU-flaws-revealed-several-serious-4040648.html18:19
clarkbfrickler: that picture makes me think I'm better off not knowing :)18:19
NgOh good. Days of IRC highlights for me from that one ;)18:20
NgThanks Intel!18:20
*** yolanda has quit IRC18:20
*** aojea has quit IRC18:21
*** VW_ has quit IRC18:22
*** shaner has joined #openstack-infra18:22
fricklerNg: sorry, didn't even notice your nick before18:22
Ngnp, I’m used to it ;)18:22
NgAt least rsyslog mostly replaces syslog-ng18:23
anteayaNg,  have a told you before you selected a horrible nick18:23
Nganteaya: 14 year old me has no regrets18:23
anteayaha ha ha18:23
*** auristor has quit IRC18:24
*** jamesmcarthur has joined #openstack-infra18:26
fungii don't recall you being especially spectre-like anyway18:27
*** auristor has joined #openstack-infra18:27
dhellmannconfig-core: can I get another review on this patch to create the constellations repo? I don't mind waiting to set up the publishing jobs until pkovar gives a +1 on the second patch but I would like to move ahead with setting up the review team while we have some momentum on it. https://review.openstack.org/#/c/565498/18:29
*** trown|lunch is now known as trown18:29
clarkboh good I've already reviewed it18:30
dhellmannyes, thanks, clarkb18:30
clarkbfwiw I've been trying to make semi regular passes through project-config and ozj reviews so that AJaeger_away doesn't have to do it all18:30
mnaserdhellmann: +A and we can leave the one above of it once we decide on ownership of the group18:30
dhellmannmnaser : perfect, thanks!18:31
*** jamesmcarthur has quit IRC18:31
mnaserill make some project config passes i guess18:31
dhellmannthat's all "decided" we just need to document it properly with pkovar's +1 on that second patch18:31
mnaserdhellmann: perfect, sounds good18:31
mnaserdhellmann: feel free to ping me when we're at that stage18:31
dhellmanncool, thanks18:31
*** dhajare has quit IRC18:33
openstackgerritAnita Kuno proposed openstack-infra/system-config master: This patch impliments the survey spec: I3c389596373b94459a32a4e540d514a2941acbb1  https://review.openstack.org/55797918:33
*** rpioso|eat is now known as rpioso18:34
*** pcrews has quit IRC18:34
anteayafungi but he is very next generation18:34
*** auristor has quit IRC18:35
*** yolanda has joined #openstack-infra18:35
*** pcrews has joined #openstack-infra18:36
openstackgerritMerged openstack-infra/project-config master: create constellations repository  https://review.openstack.org/56549818:37
*** felipemonteiro__ has joined #openstack-infra18:39
*** amoralej is now known as amoralej|off18:42
*** felipemonteiro_ has quit IRC18:42
*** iranzo has quit IRC18:42
*** auristor has joined #openstack-infra18:43
openstackgerritClark Boylan proposed openstack-infra/nodepool master: Fix test patching of clouds.yaml file locations  https://review.openstack.org/56613818:47
*** auristor has quit IRC18:53
*** shaner has joined #openstack-infra18:53
*** rkukura has quit IRC18:54
*** eharney has quit IRC18:54
*** jamesmcarthur has joined #openstack-infra18:57
*** jamesmcarthur has quit IRC19:01
mordredlook it's an Ng19:01
Nghey mordred :)19:02
*** hashar has joined #openstack-infra19:04
*** openstackgerrit has quit IRC19:05
mnaserhmm19:06
mnaseropenstackgerrit died?19:06
clarkbcertainly looks like closed connection and not a channel part19:07
clarkbpabelanger: ^ is that related to your gerrit stuff?19:07
pabelangerI haven't done anything with gerrit today19:13
pabelangerlet me look19:13
pabelangermnaser: clarkb: I still see gerritbot running on review01.o.o19:15
*** dpawlik has joined #openstack-infra19:15
pabelangermaybe just left channel because of limit?19:16
pabelangerI see it in #tripleo19:16
pabelangerI think it will join back when a new review is pushed up19:16
*** VW has joined #openstack-infra19:18
*** openstackgerrit has joined #openstack-infra19:19
openstackgerritPaul Belanger proposed openstack-infra/project-config master: DNM - testing gerritbot  https://review.openstack.org/56614619:19
pabelangermnaser: clarkb: yah, that looks to be it19:19
*** dpawlik has quit IRC19:19
clarkbhuh ut shows as connection closed? maybe that is the bot supplied message19:20
*** felipemonteiro_ has joined #openstack-infra19:21
*** felipemonteiro__ has quit IRC19:21
*** slaweq has joined #openstack-infra19:21
*** slaweq has quit IRC19:22
*** slaweq has joined #openstack-infra19:22
*** jamesmcarthur has joined #openstack-infra19:23
*** jamesmcarthur has quit IRC19:24
*** jamesmcarthur has joined #openstack-infra19:24
*** ssbarnea_ has joined #openstack-infra19:29
*** zzzeek_ has quit IRC19:35
*** zzzeek has joined #openstack-infra19:37
*** jamesmcarthur has quit IRC19:38
*** stevebaker has quit IRC19:47
*** pcichy has quit IRC19:49
*** stevebaker has joined #openstack-infra19:49
*** eharney has joined #openstack-infra19:49
openstackgerritClark Boylan proposed openstack-infra/nodepool master: Fix test patching of clouds.yaml file locations  https://review.openstack.org/56613819:50
*** gfidente has quit IRC19:52
*** camunoz has joined #openstack-infra19:52
*** ssbarnea_ has quit IRC19:54
*** ssbarnea_ has joined #openstack-infra19:55
dhellmannit looks like the openstack/constellations repo was created, but gerrit doesn't know about it. That's not how I remember things going before, did something go wrong?20:05
dhellmannhttps://review.openstack.org/#/admin/projects/?filter=openstack%252Fconstellations20:05
dhellmannthe new group doesn't exist, either, but I can clone the empty repo20:05
clarkbdhellmann: where are you cloning from?20:08
dhellmannhttps://git.openstack.org/openstack/constellations20:08
dhellmanngit says "warning: You appear to have cloned an empty repository." and it is and should be20:08
clarkbwe set up the repo on the git mirrors first so that gerrit can replicate into them. I'm guessing the gerrit side of the creation failed. I'll take a look20:09
dhellmannclarkb : the usual answer here is that I wasn't patient enough for some job to finish20:09
dhellmannI don't know what job to be looking for though :-)20:09
*** VW_ has joined #openstack-infra20:10
clarkbpuppet drives this. And it appears to be quite broken20:10
clarkbpabelanger: ^ I think this is fallout from our migration of the server20:10
dhellmannoh, right, I forgot about that20:11
clarkbhttp://paste.openstack.org/show/720321/ couple of issues there20:11
*** VW has quit IRC20:13
*** e0ne has quit IRC20:15
clarkbinfra-root ^ I'm a bit confused by that. The manage_projects logs show that its cloning things into /opt/lib/jeepyb for its cache but if I look in that dir there isn't any content20:17
clarkbalso I think paramiko in jeepyb is leaking ssh connections. I'm going to start by fixing that one as its more clear to me20:19
*** wolverineav has quit IRC20:19
*** wolverineav has joined #openstack-infra20:20
*** agopi is now known as agopi|out20:23
pabelangerlooking20:23
*** wolverineav has quit IRC20:24
pabelanger2018-05-03 06:36:19,746: jeepyb.utils - INFO - Executing command: git clone ssh://review01.openstack.org:29418/openstack-infra/gerritbot /opt/lib/jeepyb/openstack-infra/gerritbot20:24
pabelangerI wonder if that needs to be review.o.o20:24
*** auristor has joined #openstack-infra20:25
clarkbI think at least part of the problem is our project caceh file is out of date20:26
clarkbbecause it was in /opt so didn't get copied in the volume move20:26
clarkbso we're rebuilding taht from scratch20:26
pabelangerAh, I see20:26
clarkband in doing so we're leaking enough resources that things go boom20:27
clarkbI still don't understand the fsck failures. We didn't have them thena ll of a sedden they started. Maybe the clones over ssh are at fault?20:27
*** agopi|out has quit IRC20:27
clarkbI'm only a little concerned that the bad cache data might cause us to overwrite a repo20:27
pabelangeryah, I think we fix paramiko first, we most not be closing the sessing on __del__, which was needed 2.0+20:28
pabelangerwe had that issue on nodepool a while back20:29
*** VW_ has quit IRC20:29
corvusyeah sounds like that one again20:29
clarkbya we aren't I'm working on that patch now20:29
*** VW has joined #openstack-infra20:30
*** kgiusti has left #openstack-infra20:30
*** weshay is now known as weshay|rover20:30
*** VW has quit IRC20:30
*** VW has joined #openstack-infra20:31
corvusclarkb: i too think we need to answer the question "are we about to (or have we already) pushed old project data to gerrit?"20:31
corvusi'm looking at code/logs trying to figure that out20:32
clarkbcorvus: thanks, my initial reading was that we didn't20:32
clarkbbut we should double check20:32
corvusclarkb: do you know why we didn't?20:32
clarkbcorvus: ya we do a listProjects and then only create projects if not in the list projects20:32
clarkband I think we only push data if we've created the project20:33
clarkbso as long as ls-projects returns an accurate and compelte list I think we are ok20:33
corvusclarkb: but the congress-tempest-plugin failed only 1 line before "push_to_gerrit"20:33
clarkboh the shutil rm tree there may explain why those dirs are not on disk20:34
corvusclarkb: oh, you reckon make_local_copy will have returned None for the push_string for that, so we won't do the push20:34
clarkbmaybe?20:35
clarkbI'm not sure I'm a fan of how complicated it is to figure out if jeepyb will overwrite data :/20:35
*** eernst has quit IRC20:36
clarkbbut also those clones should be from gerrit so would be a short window where it may delete things20:36
*** eernst has joined #openstack-infra20:36
clarkbbasically would require us to clone, zuul to merge a patch, then for us to push back over as if that patch wasn't merged20:36
corvusthe log has no 'git push'es other than to refs/meta/config20:36
clarkbcool20:37
*** kjackal has joined #openstack-infra20:37
corvusand yeah... apparently we really do delete the repo at the end of the process20:39
corvus(in the finally clause in manage_projects.py)20:39
corvusso that explains the empty dirs20:39
openstackgerritClark Boylan proposed openstack-infra/gerritlib master: Close ssh connections when done with them  https://review.openstack.org/56615720:39
clarkbI think ^ may be the simplest fix for the ssh problems20:40
clarkbuntested thoguh. I guess I'll test it locally20:40
corvusclarkb: i think the only remaining question is what about the fsck?  a local clone of congress-tempest-plugin and git fsck works20:40
clarkbcorvus: maybe the clone is failing due to the ssh connection limit20:41
clarkbbut we aren't raising there and so it continues20:41
corvusclarkb: plausible20:42
corvusclarkb: yeah, i don't think we check the output of the clone command20:42
corvusclarkb: i'm satisfied that the ssh connection fix is the only one needed20:43
clarkbgerritlib is not python3 safe in case anyone was wondering20:44
clarkbbut it created an error and netstat says my connection went to TIME_WAIT so I think that means the explciit close is working20:45
clarkbI'll redo with python2 and double check20:45
clarkbya it seems to work locally20:47
pabelangerwill we need to tag a new release for jeepby to pick it up?20:48
clarkbpabelanger: new gerritlib release then update the lib on review.o.o20:48
pabelangerkk20:48
clarkbthen probably manually invoke manage-projects against constellations and that congress project and make sure both sets of fails are happy then run it against the entire list?20:48
clarkbcorvus: your ssh fix is only needed satisfaction implies you don't think we'll affect gerrit side state inappropriately?20:49
pabelangerclarkb: +1 to order20:50
clarkbwe might also need to clear out the leaked ssh connections20:50
openstackgerritMonty Taylor proposed openstack-infra/nodepool master: Use openstacksdk instead of os-client-config  https://review.openstack.org/56615820:50
openstackgerritMonty Taylor proposed openstack-infra/nodepool master: Remove use of six  https://review.openstack.org/56615920:50
clarkbthough those may go away when the process quits20:50
pabelangerclarkb: I'll have to duck out here in a few minutes, but will be back in a few hours to help if needed20:51
clarkbok20:51
*** andreas_s has joined #openstack-infra20:53
*** hamzy has quit IRC20:53
openstackgerritMerged openstack-infra/gerritlib master: Close ssh connections when done with them  https://review.openstack.org/56615720:54
clarkblooks like we would want an 0.8.1 release20:54
clarkbanyone else want to give that the greenlight and I'll go find my key?20:55
clarkb(or want to do additional testing first?)20:55
mordredclarkb: 0.8.1 release sounds good to me20:56
*** andreas_s has quit IRC20:57
clarkbI'm also going to put review01 back into the emergency file now so that we don't have new projects or project updates interfere with untangling this20:58
clarkbthat is done20:58
corvusclarkb: correct, i think we're safe to resume operations with that fix21:00
corvusclarkb: agree regarding release21:00
clarkb0.8.1 tag pushed.21:03
*** agopi|out has joined #openstack-infra21:03
clarkbI'll work on updating gerritlib and running manage projects one project at a time for a couple projects to build confidence in it then I guess run it against the entire set21:04
clarkb(would appreciate extra eyeballs on that process while I do it)21:04
*** trown is now known as trown|outtypewww21:06
*** VW_ has joined #openstack-infra21:07
clarkbgoing to run pip install -U /opt/jeepyb since that apepars to be how we would normally update gerritlib21:10
*** VW has quit IRC21:11
clarkbunrelated but apparently the kernel runs a charger_manager process thing21:11
clarkbsuper useful in the cloud21:11
*** wolverineav has joined #openstack-infra21:12
*** VW_ has quit IRC21:13
*** dtruong_ has quit IRC21:13
ianwpabelanger: if you get a sec want to look at puppetlabs puppet5 mirror in https://review.openstack.org/#/c/565867/2 ... i can babysit a run of it to be sure21:13
*** Goneri has quit IRC21:14
*** ldnunes has quit IRC21:15
*** edmondsw has quit IRC21:15
clarkbpip doesn/t see 0.8.1 yet so that isn't working. Also noticed that becuse we weren't installing newer gerritlib it upgraded paramiko to >2.0.0 when gerritlib has < 2.0.0 in its requirements21:16
*** dpawlik has joined #openstack-infra21:16
clarkbwhihc may be source of the problem21:16
*** edmondsw has joined #openstack-infra21:16
*** dpawlik has quit IRC21:21
*** edmondsw has quit IRC21:21
clarkbok if I passed --no-cache-dir it found the newer gerritlib21:21
clarkbI'm going to run `/usr/local/bin/manage-projects -v -l /var/log/manage_projects.log openstack/congress-tempest-plugin` momentarily21:22
clarkbok that seemed to work. Going to run the aboe command but swapping project out for constellations21:26
*** esberglu has quit IRC21:26
clarkbdhellmann: ok I think that sorted out your repo21:27
clarkbinfra-root you ready for me to run it without a project restriction and let it update teh cache info for all the projects?21:27
dhellmannclarkb : it does seem to have. Can you add me to https://review.openstack.org/#/admin/groups/1904,members when things settle down, please21:28
clarkbdhellmann: ya I actually found a bug though21:28
clarkbdhellmann: http://git.openstack.org/cgit/openstack/constellations/tree/.gitreview#n2 should be review.o.o not review01.o.o21:28
clarkbdhellmann: you'll probably want to make that your first commit. Sorry21:28
dhellmannnp, I can do that as soon as I have permission to land something :-)21:29
dhellmannhttps://review.openstack.org/56616921:30
clarkbdhellmann: done21:30
*** jtomasek has quit IRC21:30
clarkbI'm going to sort that out before running it against the whole list in case ther are other new projects that need that fix21:30
dhellmannclarkb ty21:31
* fungi is back... isp at the house has been offline for >1.5 hours already but wireless modem is working in a pinch21:33
clarkbfungi: I'll have a patch for you to review momentarily21:33
openstackgerritClark Boylan proposed openstack-infra/system-config master: Fix the gerrit-host used by jeepyb config  https://review.openstack.org/56617021:35
clarkbinfra-root ^ that should fix jeepyb manage_projects writing the wrong gerrit host value in .gitreview files21:36
clarkbwe'll want that in before doing any more manage projects runs21:36
*** esberglu has joined #openstack-infra21:39
fungicool, caught up with scrollback and reviewing now. exciting hour i missed!21:39
clarkbI guess I'll have to manually apply that change once it merges since I have the host in the emergency file21:44
clarkbopenstack/networking-ansible is another new project yet to be created. I will test ^ on that to make sure its happy and makes a correct .gitreview21:51
*** yamamoto has joined #openstack-infra21:53
tonybI know it's not really an infra question, but are y'all aware of any problems with login.ubuntu.com ?  I'm trying to login (duh) and I keep getting "Bad bot, go away! Request aborted."21:55
* tonyb is not a bot!21:55
*** jcoufal has quit IRC21:55
*** wolverineav has quit IRC21:55
clarkbtonyb: you are the first person to report anything. Try incognito mode maybe?21:56
clarkbif it persists you may need to ask canonical/launchpad directly21:56
*** kjackal has quit IRC21:56
tonybclarkb: okay same deal, so I'll ask canonical21:57
fungitonyb: i highly recomment #launchpad21:57
*** yamamoto has quit IRC21:57
tonybfungi: thanks21:59
fungirecommend too21:59
tonybIs this right? http://git.openstack.org/cgit/openstack/constellations/tree/.gitreview#n222:04
tonybmy clone/update all the things script didn't like it22:05
clarkbtonyb: no, see scrollback :) tldr the move of the gerrit server updated the config in a bad way and I just fixed it above22:05
clarkbor rather think it is fixed and new projects shouldn't have that problem I will confirm on openstack/networking-ansible once it merges22:05
tonybclarkb: Oh okay. Oh look there it is sorry22:06
*** eharney has quit IRC22:06
clarkbtonyb: dhellmann has a patch up to fix it in that particular project22:06
tonybclarkb: cool.  I'll just rm -rf my clone and let it do the right thiing on the next run22:06
dhellmannI approved it, so it should merge22:06
dhellmannalthough I guess we need zuul to know about that repo first?22:06
clarkbdhellmann: ya you may need to get that in first22:06
dhellmannlet me split the patch that adds the publishing job out so we can get the no-op jobs22:07
*** rcernin has joined #openstack-infra22:07
*** hashar has quit IRC22:08
openstackgerritMerged openstack-infra/system-config master: Fix the gerrit-host used by jeepyb config  https://review.openstack.org/56617022:08
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: add basic jobs for constellations repository  https://review.openstack.org/56549922:09
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: publish constellations  https://review.openstack.org/56617622:09
dhellmannclarkb , tonyb : ^^22:09
*** mriedem is now known as mrjazzercise22:09
dhellmannclarkb : it seems I'm missing something there, but I don't know what22:10
dhellmannwhich is the "main configuration file"?22:10
clarkbdhellmann: I think because gerrit failed to create the project when we tried to tell zuul about it zuul also didn't get the memo22:10
dhellmannaha22:10
clarkbcorvus: ^ can you confirm and is the proper action there to reload the zuul-scheduler?22:10
*** bobh has quit IRC22:11
clarkblet me create the new ansible project before we do that though22:11
*** eernst has quit IRC22:11
clarkbok that one had an upstream so wouldn't test making a .gitreview22:14
clarkbbut it seems to have worked22:15
fungiso we're probably in the clear now?22:15
fungi(for approving more new projects i mean)22:15
clarkbfungi: no we need to do a full run of manage projects and make sure it doesn't push anything unexpected22:15
fungioh, got it22:16
clarkbthen we have to remove review01 from the emergency file22:16
clarkbthen we should make sure zuul is aware of these new projects22:16
fungiseems like the next step is a safe one anyway22:16
clarkbfungi: you are round if I run the full manage projects now?22:16
*** eernst has joined #openstack-infra22:16
clarkbfungi: ya corvus reviewed the code and doesn't expect the lack fo a cache to cause us problems22:16
fungii am (working from a wireless modem but around)22:16
* fungi shakes fist at flaky island isp22:16
clarkbok I'm going to run it and it will log to the normal log file if you want to tail that22:16
clarkbI did get /usr/local/lib/python2.7/dist-packages/paramiko/client.py:685: UserWarning: Unknown ssh-rsa host key for [review.openstack.org]:2941822:17
clarkbdid pabelanger's update of the ssh host keys merge?22:17
fungii thought so... checking22:17
clarkbI'll wait on the main run22:17
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Add community page  https://review.openstack.org/56617922:17
clarkbfungi: basically we want to do a foreground run to make sure that the ssh connection leak is gone and that we don't push anything unexpected22:18
corvusclarkb: yes, a full scheduler reload (sighup) should take care ofi t22:19
corvusclarkb: want me to do that now?22:19
clarkbcorvus: I think it is safe to do now ya (the two known outstanding projects are in22:19
* dhellmann read sighup as "sigh up"22:20
corvusnow i'll never read it any other way22:20
clarkbfungi: looks like the ssh host keys were for zuul22:21
clarkbfungi: gerrit doesn't seem to have gotten that. I think I'll manually accept the key for now?22:21
dhellmannfunny, it it had been all caps I would have probably just matched the pattern without really reading it :-)22:21
fungihttps://review.openstack.org/565864 hasn't merged, but that only seems to cover review.o.o known_hosts entries for zuul servers?22:22
corvusif you don't yell the signals in all caps, the app might not hear them22:22
clarkbfungi: ya that22:22
fungiclarkb: i suppose accept it for now, but it would be good to figure out if we've puppeted that file22:23
*** slaweq has quit IRC22:23
clarkbhrm seems like its accepted at least from here22:23
fungiclarkb: oh!22:23
clarkblet me try rerun manage_projects against a single node and see if I get the same warning22:23
fungithis is a connection to localhost22:24
fungii be we just never puppeted it at all22:24
clarkbfungi: yes but not via the name localhost22:24
clarkbya wouldn't surprise me22:24
fungiright, but it's not an address change it's complaining about22:24
clarkbI'm going to rerun against constellations which should be a noop but it should still ssh to determine that22:24
fungii think it must have been manually accepted on previous gerrit servers and so lost when we moved to a new one22:24
fungicool22:25
clarkbya no warning this time. I think because the ip and key matched just the name chagned it may have auto accepted?22:25
corvusdhellmann, clarkb: 2018-05-03 22:25:29,062 INFO zuul.Scheduler: Full reconfiguration complete22:25
corvusdhellmann: should be good to recheck22:25
* dhellmann crosses his fingers22:26
clarkbfungi: corvus you good for me to run manage_projects against all the projects now?22:26
corvusclarkb: ++22:26
clarkbok I'll have ti write to the normal log file so you can easily watch it22:26
clarkbit is running22:27
fungisounds good, but fair warning, i have to move my wireless modem to a different machine if i want to be able to ssh into openstack servers (only takes a few minutes so happy to if/when needed)22:27
clarkblooks like it is updating acls in puppet-* repos22:28
fungii should work out a solution for just plugging the wireless modem into my openbsd firewall here at the house, but worried christine would try to netflix through it then ;)22:28
clarkbI'm not sure what we changed there22:28
clarkbI guess I can check the git log22:28
corvusclarkb: i thought it was doing that to all the repos earlier22:29
clarkbcorvus: it was22:29
clarkbcorvus: it may just be nooping too?22:29
corvusyeah maybe?22:29
clarkbhttps://git.openstack.org/cgit/openstack-infra/puppet-redis/commit/project.config?h=refs/meta/config&id=682b315f13a06f4eae591ed00573a653e53afef922:30
fungii take it no change to the last modified timestamp of any files in the git repo on disk?22:30
clarkbfungi: its definitely updating the git repo meta/config ref22:30
*** rlandy is now known as rlandy|bbl22:30
fungiahh22:30
clarkbthat diff looks ok to me22:31
corvusyeah i just saw the same in the bashate config22:31
corvushttps://git.openstack.org/cgit/openstack-dev/bashate/commit/project.config?h=refs/meta/config&id=8e4b4a9852d7dcce78261a2ecf691e808f3df2e422:31
corvusi guess we're killing all tabs for some reason22:31
fungioh, right, gerrit helpfully indents the configuration for you22:31
corvuswhat's with allowSuperproject?22:31
fungiand is adding that superproject entry for the openstack/openstack repo22:31
fungiwhich we maybe ought to look into ditching22:32
corvusis that going to break openstack/openstack?22:32
clarkbhttps://git.openstack.org/cgit/openstack-infra/puppet-redis/commit/project.config?h=refs/meta/config&id=e10bb73871ee339d338e2da8af5d6499d4f2da8722:32
clarkblooks like gerrit added it itself22:32
corvusfungi: it's removing that entry, right?22:32
clarkbcorvus: yes it is removing the entry22:32
clarkbcorvus: it may break openstack/openstack22:32
fungijeepyb is removing it i believe, yes22:32
clarkbI think we want to manage that directly either way so not the end of the world22:32
fungithough openstack/openstack has been fairly broken for years i thought22:32
clarkbwe'll just have to add it back in if necessary22:32
corvusso it was added july 2017, and now it's being removed (presumably because there's no cache)22:32
fungiwhen we determined that there was no good way to extend it to cover additional repositoires22:33
corvusfungi: i thought it's just been plugging along22:33
fungioh, has it? how have we been adding new openstack deliverables to it? last time i tried, the change sat there indefinitely because it's not a reviewable project22:34
corvusthough, to be fair, it looks like it only has project-config?22:34
corvusfungi: well, at one point it was the release manager's job to add things22:34
clarkbcorvus: ya I think because there is no cache we don't have the data to say this project is up to date which means check directly and since there is a delta between what gerrit has and what we have in config its pushing what we have in config over what gerrit has22:34
corvusfungi: if they have abdicated, i'm unaware22:34
fungibest i could figure out we had to amend the submodules list in openstack/openstack which we don't have a good way to do22:34
*** slaweq has joined #openstack-infra22:35
corvushttps://review.openstack.org/#/admin/projects/openstack/openstack,access22:35
fungibut maybe the way is something other than code review?22:35
corvusi thought it was code review22:35
corvusmy guess is that no one cares, and the people who know about it think it's broken.  and regardless of whether they were before, they're about to be right.  :)22:36
clarkbhttps://git.openstack.org/cgit/openstack/openstack/commit/?id=ae3bc0ff7f7141d6f95da57a982fcfaeb3828f94 looks like its updating things from more than project-config22:36
corvus(personally, i still think it's a neat idea, and perhaps it would make sense to put project submodules in there grouped by constellation -- thereby solving the "what goes in there?" problem)22:38
clarkbcorvus: and if we do that explicitly add the allow super project config to the canonical config22:38
fungimmm, yeah looks like openstack/openstack has the noop-jobs template so should in theory be reviewable22:38
clarkbit is a bit annoying that gerrit would silentyl rewrite that for you behind the scenes22:38
corvusat any rate, i reckon someone should fix it or kill it :)22:38
dmsimardinfra-root: oi, ara 0.15.0 is tagged. I'll babysit executors and logs.o.o.22:38
corvusclarkb: yeah, i suspect we never noticed that, and therefore, probably any project that *used* to be in openstack/openstack silently got dropped if/when someone updated its acl file22:39
clarkbother than that I've not seen anything in the logs indicating weirdness. Alsocinder is done and a spot check of that repo manually looks happy22:39
clarkbcorvus: ya22:39
fungihttps://review.openstack.org/344585 appears to be the most recent attempt to update openstack/openstack22:41
fungibut yeah, the fact that adding submodule configuration in one repo updates the config of another repo to add a superproject section runs sort of at odds with how we've got manage-projects designed22:42
* mordred suggested we consider updating openstack/openstack as part of the release process to the release team recently, but it was more of a "hey, maybe we shoudl think about this"22:42
clarkb`grep push /var/log/manage_projects.log  | grep -v meta` shows only networking-ansible and constellations which is good22:42
dhellmannmordred : I'm not against the idea, I would just like to know that someone actually uses it before we go to the trouble.22:43
mordredyah - totally22:43
*** felipemonteiro__ has joined #openstack-infra22:43
dhellmannfor some reason I thought we were updating that repo every time something in the integrated gate merged22:44
*** felipemonteiro__ has quit IRC22:44
*** felipemonteiro__ has joined #openstack-infra22:44
mordredyah - that's a thing that goes on - but managing the submodules that are in it is another story22:45
fungithe integrated gate queue is sort of a nebulous thing given that it's partly defined by job matching22:45
dhellmannyeah22:45
mordredand it's not really things in the integrated gate - the last time we updated the submodules list in that repo it was "every official openstack repo"22:45
mordredso everything in the integrated gate - plus many other things22:46
fungiit would make more sense to define openstack/openstack based on governance metadata or release metadata22:46
mordredyup22:46
fungiso all official deliverables with this set of governance tags, or all official deliverables with these release models22:46
* fungi waves kermit arms22:47
mordredthat was the most recent idea we discussed - make a thing that was driven by governance metadata that updated the submodules in the repo as things changed - and potentially do something with release metadata around release time so that we can cut a rocky branch in openstack/openstack that contains the set of projects that were official for rocky22:47
mordredfungi: yup22:47
dhellmannwe should probably also work into the definition how we think it's useful for someone to consume it22:47
*** felipemonteiro_ has quit IRC22:47
clarkbmy extra paranoia has me rereading jeepyb and make_local_copy also seems to do the right thing if the project is in projects list22:47
mordreddhellmann: the most recent use I found for openstack/openstack was as a replacement git repo for the cncf thing that showed activity dashboard for various cloud related things22:48
mordredthe repo they used for openstack was "openstack/nova" - which is obviously incorrect22:48
dhellmannthey needed a mono-repo for that?22:48
dhellmannah22:48
clarkband gerrit ls-projects seems to return all 1911 repos without needing paging so I think we are ok there22:48
mordredtheir tool was, in the state it was in, designed to think of the world only in terms of mono-repos22:48
mordredI don't think we need to design systems here around tools other people use to track stats that are not meaningful22:49
dhellmannnot too surprising22:49
clarkbit is a little scary to me that we rely on ls-projects and not a direct check if $project exists22:49
clarkbbut maybe ls-projects is good enough22:49
mordredbut it's a use22:49
clarkbdoes anyone know if ls-projects can eventually paginate?22:50
mordredclarkb: I do not know22:50
corvusclarkb: i don't know for sure, but because it's not returning structured data, i wouldn't *expect* it to22:50
clarkbcorvus: thats a good point22:51
corvusclarkb: there are... however, 'start' and 'limit' options.  so i guess it's theoretically possible.22:51
clarkbhttps://review.openstack.org/Documentation/cmd-ls-projects.html doesn't seem to hae start?22:53
corvusweird:  ssh review gerrit ls-projects --help22:54
corvus --start (-S) CNT                       : number of projects to skip22:54
clarkbhuh22:54
clarkbthe --limit in the docs at least is realtive to the beginning22:54
clarkbso not proper paging if that is the case22:54
clarkblooking at the list of api commands over ssh ls-projects is likely most appropriate for checking this info too22:56
clarkbso jeepby probably doesn't need an update to use something else22:56
clarkbwe are into the openstack/m's22:57
clarkbwe should probably run it one more time after this run to make sure it noops as expected so that it will end up under the puppet run timeout22:58
clarkbthen we can reenable puppet22:58
*** threestrands has joined #openstack-infra22:59
*** tpsilva has quit IRC23:00
fungiif we really, really, really get worried that ls-projects might return an incomplete list at some length in the future, we could add a double-check for a specific project name only when it's not returned in the list23:01
fungithat way we at least don't query gerrit thousands of times23:01
clarkbya23:01
fungior take it as a sign to rewrite jeepyb to use the rest api23:03
clarkbit also helps that we auto clean out upstreams now23:03
clarkbsince usptream probably poses the most dangerous state instruction to jeepyb in this situation (as its basically saying go get your code from over there)23:03
clarkbits doing noav now23:04
*** felipemonteiro__ has quit IRC23:04
*** rpioso is now known as rpioso|afk23:05
*** hongbin has quit IRC23:06
corvusif we use the rest api, we will certainly need to paginate23:07
clarkbnoav takes a long time to clone I expect it will take a long time to fsck too23:07
clarkb(which is why we have the cache)23:08
*** gouthamr has quit IRC23:08
clarkbnova looked ok23:09
fungiyeah, the rest api would need pagination, but at least then it's known and explicit23:09
*** rajinir has quit IRC23:09
clarkbI'm going to step away for a couple minutes while this is going I've not seen anything weird. let me know if you do23:09
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Switch to Open Sans font  https://review.openstack.org/56618523:10
openstackgerritJames E. Blair proposed openstack-infra/zuul-website master: Change tagline to dark blue  https://review.openstack.org/56618623:10
*** gouthamr has joined #openstack-infra23:10
*** slaweq has quit IRC23:13
*** dpawlik has joined #openstack-infra23:14
clarkbwe are about 3/4 of the way through the list at this point23:17
*** dpawlik has quit IRC23:18
*** gouthamr has quit IRC23:18
*** dmellado has quit IRC23:19
clarkbdhellmann: I've approved the constellations job add and can recheck the gitreview fix if necessary23:19
*** stevebaker has quit IRC23:20
corvusclarkb, fungi: can you take a look at https://review.openstack.org/566089 when you have a sec?23:21
corvusi'd like to make a version of a docs job which, rather than rebuilding in post, uses the artifact already built in gate.23:22
clarkbcorvus: neat, basically copying the draft content to the afs fs?23:22
corvusclarkb: well, i figured we'd stash it on logs.o.o or tarballs.o.o, indexed by change number23:23
clarkbah23:23
*** masber has joined #openstack-infra23:23
openstackgerritMerged openstack-infra/project-config master: add basic jobs for constellations repository  https://review.openstack.org/56549923:23
corvusbasically, if you just drop the build id from the log url, you get a gate artifact repository23:24
corvus(last build wins; though we might need to clear out the previous build before uploading)23:25
fungicorvus: cool, i looked at 566089 earlier but out of context it wasn't clear to me whether demonstration meant you were ready to merge it. sorry i didn't follow up asking for clarification23:27
clarkbless than 200 hundred projects to go now23:28
corvusfungi: sorry for vagueness, yeah, i mostly meant to indicate that i wanted to set this up with one job to try out the concept and have no current plans to make any sweeping changes23:29
clarkboh you know what I'm a derp23:30
clarkbthe constellations change to add pep8 won't pass either23:30
* clarkb decides to focus on the one thing at a time, manage_projects23:30
clarkbtil there is a project called whereto23:31
fungiyou can thank dhellmann for that one too ;)23:33
fungi(i think?)23:33
openstackgerritMerged openstack-infra/project-config master: Add promote pipeline  https://review.openstack.org/56608923:33
clarkbok its done23:33
clarkbthe only pushes logged were for the two new projects when i ran the script manually23:34
clarkbI'm going to rerun it now after double checking the cache file23:34
fungisounds good23:35
clarkbok the one weird thing I see is some projects looks like those that were able to run with the half broken ssh and the ones I just created have created-in-github set to false23:36
clarkbbut at least for constellations it exists on github23:36
clarkblooks like we only set that flag to true if we've actually created the repo in github23:38
clarkbI'm going to guess that the exceptions borked that state but since it is actually in github we won't flip that flag23:38
clarkbthe side effect of this is we will run github things against these projects every time we run manage projects23:39
clarkbmordred: corvus fungi dare I manually edit the cache file to set created-in-github to true?23:39
fungiif this is intended to be a persistent cache, then yeah either than or try to copy in the old one from now defunct review.o.o23:40
fungis/than/that/23:40
corvuslooking23:40
corvusat this point, copying won't work, it'll have the same problem23:40
clarkbya especialyl for the new projects23:40
corvusclarkb: agree with your assesment.  i think fix is to set created in the cache object inside create_update_github_project23:42
corvusclarkb: and also, yes, for now manually edit the cache23:42
clarkbok I'm going to manually edit the cache file then I'll run against a small number of edited projects before doing another full run23:43
clarkb/opt/lib/jeepyb/project.cache.bak is a backup of existing cache state23:43
clarkbwe can diff against that to see if I did anything silly23:43
clarkbok I did a vim substition, 96 lines replaced. Diff lgtm anyone else want to read it before I rerun a couple of these?23:46
corvusclarkb: why so many?23:47
corvusi thought it was just the 2 new projects23:47
*** eernst has quit IRC23:47
clarkbcorvus: two were from the new projects. The others were the half successful runs when ssh was half working I think23:47
clarkbcorvus: its the first ~94 projects in the list23:47
corvusclarkb: okay; spot checking a bunch of those, all of them seem like they should have the flag set.  so, lgtm.23:48
clarkbok I'm going to check the two new projects first then will do an openstack-dev project and an openstack-infra project before rerunning everything23:48
clarkbhrm it flipped the value back for constellations23:49
clarkbthis may have actually been a long standing bug23:50
clarkbI think those flipped because they ran more than once23:50
clarkbya that is what happened I think23:50
clarkbI'll still run against the other ones to confirm that behavior and get comfy with it then I'll do a global run and we can fix this separately23:51
*** gouthamr has joined #openstack-infra23:51
*** dmellado has joined #openstack-infra23:51
clarkbya they all do that so fairly convinced its a long standing bug23:53
clarkbI'm going to go ahead and do a second global run now to make sure its largely a noop23:53
clarkband it looks clean23:54
clarkbI've got to go get the grill started now and make dinner23:54
clarkbI'm happy to keep review01 in the emergency file until morning when we can watch it unless ianw wants to remove it23:54
corvusclarkb: why didn't all the projects get it set to false?23:55
clarkbcorvus: they appear to all be false now?23:56
corvusclarkb: shouldn't they all have been false before your vim replace?23:56
clarkbbut previously there was no cache data for the projects and they were crashing due to the ssh problem before they could get updated info which would set them to true, then the next run would set them to false23:56
clarkbcorvus: because that run was the first run for most of those projects according to the cache they all got set to true after that first one23:56
clarkb(they weren't in the cache previously basically)23:56
corvusclarkb: how did they get set to true?  i don't see a path that sets created to true unless either it is actually created, or gerrit is added to the team23:57
clarkbcorvus: it was logging adding gerrit to the team23:58
clarkbso I'm guessing that is related23:58
corvushrm.  wonder why gerrit wasn't in the team?23:58
clarkbbecause the cache was empty23:58
corvusclarkb: oh i see now23:58
corvuswell23:58
corvusclarkb: it checks github before adding it23:59
clarkbit does seem to check against whether or not gerrit is in the team for actually adding gerrit23:59
clarkbbut not for setting created to true23:59
clarkbbut that log line is what was emitted so curious23:59
corvusso i agree that if the cache is false, it will update the cache, but it shouldn't log adding the team... exactly23:59
*** yamamoto has joined #openstack-infra23:59

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