Thursday, 2018-06-07

openstackgerritMerged openstack-infra/zuul master: Fix zuul-web sql connections  https://review.openstack.org/57221000:04
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Handle websocket client hangups better  https://review.openstack.org/57303400:22
*** threestrands has quit IRC00:25
*** threestrands has joined #zuul00:28
*** D3VIATION has joined #zuul00:34
openstackgerritMerged openstack-infra/zuul master: Handle websocket client hangups better  https://review.openstack.org/57303401:04
openstackgerritIan Wienand proposed openstack-infra/nodepool master: webapp: fix browser return  https://review.openstack.org/57305301:17
openstackgerritIan Wienand proposed openstack-infra/nodepool master: webapp: fix browser return  https://review.openstack.org/57305301:20
*** threestrands has quit IRC01:35
*** threestrands has joined #zuul01:35
*** threestrands has quit IRC01:36
*** threestrands has joined #zuul01:37
*** threestrands has quit IRC01:37
*** threestrands has joined #zuul01:37
*** threestrands has quit IRC01:38
*** threestrands has joined #zuul01:38
*** threestrands has quit IRC01:38
*** threestrands has joined #zuul01:38
*** threestrands has quit IRC01:39
*** threestrands has joined #zuul01:40
*** threestrands has quit IRC01:40
*** threestrands has joined #zuul01:40
*** threestrands has quit IRC01:41
*** D3VIATION has quit IRC01:43
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: Ensure cherrypy listen port is int type  https://review.openstack.org/57306001:47
*** threestrands has joined #zuul02:11
*** threestrands has quit IRC02:11
*** threestrands has joined #zuul02:11
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/job/{job_name} route  https://review.openstack.org/55097802:25
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/projects and /{tenant}/project/{project} routes  https://review.openstack.org/55097902:25
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/pipelines route  https://review.openstack.org/54152102:25
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/labels route  https://review.openstack.org/55397902:25
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/nodes route  https://review.openstack.org/55399802:25
*** rlandy|rover|bbl is now known as rlandy|rover02:35
*** rlandy|rover has quit IRC02:39
*** elyezer has quit IRC02:51
*** elyezer has joined #zuul02:52
*** D3VIATION has joined #zuul02:57
*** threestrands has quit IRC03:13
tobiashcorvus: thanks03:58
openstackgerritkaka proposed openstack-infra/zuul-website master: Trivial: update url to new url  https://review.openstack.org/57307904:01
openstackgerritMerged openstack-infra/zuul master: Ensure cherrypy listen port is int type  https://review.openstack.org/57306004:13
*** Rohaan has joined #zuul05:13
*** D3VIATION has quit IRC05:42
openstackgerritkaka proposed openstack-infra/zuul-website master: Trivial: update url to new url  https://review.openstack.org/57307905:44
*** pcaruana has joined #zuul06:35
*** hashar has joined #zuul06:41
*** jesusaur has quit IRC06:44
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/job.html page to display job details  https://review.openstack.org/53554506:54
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/projects.html web page  https://review.openstack.org/53787006:54
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/labels.html web page  https://review.openstack.org/55398006:54
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/nodes.html web page  https://review.openstack.org/55399906:54
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add jobs graph rendering  https://review.openstack.org/53786906:54
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add project pipeline rendering  https://review.openstack.org/53787106:54
tristanCmordred: ^ rebase of all the new route pages on top of the angular6 patch06:55
tristanCmordred: beside the routing complexicity, my main concern is that the status controller is currently polled whatever the page you are browsing06:56
*** jesusaur has joined #zuul06:56
tristanCmordred: otherwise, the new architecture is lgtm, thanks!06:56
*** gtema has joined #zuul07:20
*** swest has quit IRC07:29
*** swest has joined #zuul07:30
*** jpena|off is now known as jpena07:36
gtematobiash: are you planning to held a talk about Zuul in Berlin? or anyone else?08:09
tobiashgtema: yes08:10
gtemaok, I wanted to have one either (about Zuul at OpenTelekomCloud). Wanted to check that there are no similar talks. Should we/ can we join?08:10
tobiashgtema: maybe we should discuss that with the foundation people?08:14
gtematobiash: shure08:15
gtemaunder foundation you mean whom?08:15
tobiashgtema: my contact is Allison Price08:16
gtemaok, would you check that? I will try to also trigger some checks from my side, not sure however, that there are contacts in foundation08:17
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/job/{job_name} route  https://review.openstack.org/55097808:20
tobiashgtema: I'll write a mail later08:26
gtematobiash: thanks08:27
*** electrofelix has joined #zuul08:35
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/projects and /{tenant}/project/{project} routes  https://review.openstack.org/55097908:45
openstackgerritIan Wienand proposed openstack-infra/nodepool master: webapp: fix browser return  https://review.openstack.org/57305308:47
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/pipelines route  https://review.openstack.org/54152109:38
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/labels route  https://review.openstack.org/55397909:57
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/nodes route  https://review.openstack.org/55399810:02
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/job.html page to display job details  https://review.openstack.org/53554510:13
openstackgerritMerged openstack-infra/nodepool master: Add opensuse-150 to nodepool dsvm  https://review.openstack.org/57241610:41
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/projects.html web page  https://review.openstack.org/53787010:47
*** eliqiao has joined #zuul10:53
eliqiao Eli Qiao hello guys, I have setup zuul service, but the Web-based log streaming failed and said : "Failure from finger client: [Errno -2] Name or service not known" , anyone know how to resolve that?10:54
gtemaeliqiao - retry again. I have it similar when the job is assigned, but the remote job is not yet properly started10:55
eliqiaoissue is same every time,.10:55
gtemado you have it written in the console tab or somewhere in the logs?10:56
eliqiaoI can find console log on server10:57
eliqiaohttps://www.irccloud.com/pastebin/7BYdZOF4/10:57
gtemawhere do you see your failure message? what is your /var/tmp/zuul-console (which process is writing there)?11:00
eliqiao@gtema, on web11:07
gtemawhat's in your executor logs?11:08
gtemato tell the truth I am not 100% sure, that finger is really required for logs11:08
eliqiaoshould I open debug mode for zuul-executor? I can find any useful log information for now.11:10
eliqiaohttps://www.irccloud.com/pastebin/HGMcWdMG/11:11
gtemayes, look in debug. But before check that your finger service is running, and executor is looking on the correct host/port: http://paste.openstack.org/show/722871/11:12
eliqiaoGot some log message from zuul-web service11:13
eliqiaohttps://www.irccloud.com/pastebin/PnLnlGGp/11:13
eliqiao looks something wrong ?11:15
eliqiaohttps://www.irccloud.com/pastebin/vfrO3uVN/11:15
eliqiaohttps://www.irccloud.com/pastebin/TWdlmXXw/11:17
gtemalook in web-debug log. It seems you have multiple problems11:18
tobiasheliqiao: the executor needs to know his hostname11:18
eliqiaowhat service create fingergw.socket?11:19
gtemadepends on how you install zuul11:19
gtemagenerally zuul-fingergw11:19
*** jpena is now known as jpena|off11:20
gtemadoes your [web] section contain listen_address?11:22
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/labels.html web page  https://review.openstack.org/55398011:22
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/nodes.html web page  https://review.openstack.org/55399911:24
tobiasheliqiao: if the executor cannot resolve its hostname automatically you have to define the hostname/ipaddress in the executor config: https://zuul-ci.org/docs/zuul/admin/components.html#attr-executor.hostname11:24
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add jobs graph rendering  https://review.openstack.org/53786911:37
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: dashboard: add project pipeline rendering  https://review.openstack.org/53787111:43
tristanCi now hope we are not going to replace cherrypy or angular6 anytime soon :-)11:44
gtemalol11:45
*** jpena|off is now known as jpena12:01
eliqiao@gtema @tobiash thanks @tristanC to provide the solution : "hostnamectl set-hostname managesf.sftests.com", this should fix "hostname -f"12:04
eliqiaothank you guys.12:04
tobiash:)12:11
*** Rohaan___ has joined #zuul12:29
*** Rohaan has quit IRC12:29
*** Rohaan has joined #zuul12:39
*** Rohaan___ has quit IRC12:39
*** elyezer has quit IRC12:57
*** elyezer has joined #zuul13:03
*** myoung|off is now known as myoung13:05
pabelangereliqiao: yah, you need to setup hostname in /etc/hosts for fingergw13:08
*** Rohaan has quit IRC13:09
Shrewsthat sounds like something we should document13:09
*** D3VIATION has joined #zuul13:12
*** openstackgerrit has quit IRC13:34
*** gtema has quit IRC13:44
*** rlandy has joined #zuul13:46
*** rlandy is now known as rlandy|rover13:47
mordredtristanC: I think replacing frameworks every few weeks keeps us nimble :)14:09
*** D3VIATION has quit IRC14:16
corvusShrews: would you relieve 535781 of your -2 please?14:20
Shrewsgladly14:20
* Shrews flips the sign bit14:21
*** gtema has joined #zuul14:22
tobiashjust learned that deleting tags on the remote can break zuul permanently14:25
tobiashI think we need to prune tags on fetch14:25
Shrewscorvus: i'm sure you're aware but just making sure we don't forget https://review.openstack.org/535839 and https://review.openstack.org/54669814:33
Shrews(the 2nd looks like the tip of the stack)14:34
corvusShrews: oh thanks i forgot14:37
tobiashcorvus: I'm going home now and be back in probably 1-2 hours is assistance is needed14:39
tobiashs/is/if14:39
corvustobiash: thanks!14:41
*** openstackgerrit has joined #zuul15:02
openstackgerritMerged openstack-infra/zuul master: Move tmpdir into work root  https://review.openstack.org/54669815:02
openstackgerritMerged openstack-infra/zuul master: Update to Ansible 2.4  https://review.openstack.org/53578115:02
openstackgerritMerged openstack-infra/zuul master: Update to Ansible 2.5  https://review.openstack.org/56266815:02
openstackgerritMerged openstack-infra/zuul master: Disable action and lookup plugins from 2.4  https://review.openstack.org/53583915:02
*** acozine1 has joined #zuul15:03
-openstackstatus- NOTICE: Zuul update for Ansible 2.5 in progress. Scheduler crashed as unexpected side effect of pip upgrade. Will be back and running shortly.15:32
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Delete stale jobdirs at startup  https://review.openstack.org/57330215:56
-openstackstatus- NOTICE: The zuul upgrade to ansible 2.5 is complete and zuul is running again. Changes uploaded or approved between 15:25 and 15:45 will need to be rechecked. Please report any problems in #openstack-infra16:11
corvusthe next release will be 3.1.0 due to the ansible upgrade.  do we also want to include https://review.openstack.org/571745 in it, or should that be 3.2.0?16:16
Shrewsi need to review that still16:17
tobiashcorvus: depends on the time schedule16:18
tobiashcorvus: I have yet another security fix in the pipeline16:18
corvustobiash, SpamapS: also, i think you're aware of https://review.openstack.org/571745  (we talked about it for a while on zuul-discuss)  but i wanted to point it out to make sure it didn't slip past you.16:18
corvusoh hey what do you know there's a story16:19
tobiashcorvus: yes, sorry didn't have time yet to look at this16:19
tobiashcorvus: I'm planning to work on that security issue tomorrow16:21
corvustobiash: thanks16:21
tobiashI did at least a quick analysis and have a rough idea how to fix that16:22
tobiashhopefully that works out16:22
clarkbcorvus: I can't decide if it is more user friendly to stagger those or not16:22
openstackgerritClark Boylan proposed openstack-infra/zuul master: Add timestamps to multiline debug message logs  https://review.openstack.org/57284516:23
SpamapScorvus: I'll peek at it16:23
clarkbI'd like to get ^ in just for logging consistency16:23
corvustobiash: do you think it's possible ara might be affected?16:23
*** myoung is now known as myoung|biaf16:23
tobiashcorvus: I don't know how ara works, but at least this affects both text and json log16:24
corvusdmsimard: can you take a look at https://storyboard.openstack.org/#!/story/2002177  and evaluate if ara might be affected?16:25
corvusdmsimard: it's a private story for a security vulnerability; you'll need to be logged in16:26
*** myoung|biaf is now known as myoung|lunch16:32
*** acozine1 has quit IRC17:07
dmsimardLooking17:12
dmsimardtobiash: is there a reproducer playbook ?17:17
*** jpena is now known as jpena|off17:17
tobiashdmsimard: not yet17:18
dmsimardcorvus: FWIW I've reported this a long time ago: https://bugzilla.redhat.com/show_bug.cgi?id=CVE-2017-747317:18
dmsimardtobiash: ok I'll try to reproduce.17:19
dmsimardtobiash: is this with the latest ansible ?17:20
tobiashyes, we run 2.5.something17:20
corvusdmsimard: if you find ara is affected can we do coordinated disclosure? :)17:22
dmsimardI'd very much appreciate if Ansible could properly sanitize the data before it's sent to the callback, the burden is in the wrong yard -- but sure17:23
dmsimardI'll add a comment to the bug soon.17:23
tobiashdmsimard: added a bit more info to the story17:26
dmsimardtobiash: added a comment17:28
*** myoung|lunch is now known as myoung17:32
tobiashdmsimard: commented17:34
tobiashdmsimard: ara is its own callback right?17:34
dmsimardtobiash: right17:34
*** openstackgerrit has quit IRC17:34
dmsimardlikely have a reproducer soon17:38
*** acozine1 has joined #zuul18:01
*** gtema has quit IRC18:19
dmsimardtobiash, corvus, fungi: reproducer in story18:23
fungithanks dmsimard!18:32
tobiashcorvus: this makes me think if we should think about trying to filter secrets from logs within zuul as second line of defence18:41
tobiashwe know secrets in advance so maybe that's possible18:41
tobiashthis probably won't work 100% but maybe safe us from common mistakes even if someone misses no_log accidentally18:43
*** mugsie has quit IRC19:19
*** mugsie has joined #zuul19:21
*** mugsie has quit IRC19:21
*** mugsie has joined #zuul19:21
*** mugsie has quit IRC19:21
*** mugsie has joined #zuul19:21
*** mugsie has quit IRC19:22
*** Guest68045 has joined #zuul19:24
*** Guest68045 has quit IRC19:24
*** mugsie_ has joined #zuul19:24
*** mugsie_ has quit IRC19:27
*** mugsie_ has joined #zuul19:27
*** mugsie_ is now known as _mugsie19:27
*** _mugsie is now known as mugsie_19:28
dmsimardcorvus: to answer your question, yes ara is also impacted19:31
corvusdmsimard, tobiash: how about we use that story to coordinate fixes to both, and then when ready, propose them to gerrit at the same time?19:53
dmsimardtobiash is eod, but that sounds like a good idea to me. Let's wait and see what upstream has to say about it19:55
*** pcaruana has quit IRC20:39
*** openstackgerrit has joined #zuul20:52
openstackgerritJames E. Blair proposed openstack-infra/zuul master: Accumulate errors in context managers  https://review.openstack.org/57343820:52
corvusfbo_: ^ i think we can reduce some of the extra except clauses in the load-broken-config change, but i don't want to delay that one any more than necessary, so i proposed a followup20:55
*** myoung is now known as myoung|off21:14
mordredcorvus: any reason at this point to hold off on landing fbo_'s patch?21:18
corvusmordred: nope, i think it's gtg21:19
mordredwoot21:19
dmsimardansible 2.6 is already in rc1...21:36
openstackgerritMerged openstack-infra/zuul master: Make Zuul able to start with a broken config  https://review.openstack.org/53551121:38
dmsimardcorvus, fungi, tobiash: added a comment on the story21:40
corvusdmsimard: ack thx.21:46
*** acozine1 has quit IRC21:52
corvustristanC, mordred: fyi see my comment on https://review.openstack.org/55387321:53
corvusmordred: i'd love for one of you to weigh in on https://review.openstack.org/567864 if you have a chance... i'm wondering if that should just be fixed in zuul_stream instead of command (unless, obviously, it's fixed upstream in command)22:00
*** hashar has quit IRC22:00
corvusmordred: any of you will do :)22:00
*** tranzemc has joined #zuul22:22
tranzemchi there, how dashboard use zuul to display building process/status like cicd ??22:23
tranzemcwhat dashboard ?22:24
openstackgerritMerged openstack-infra/zuul master: Remove getRepoPermission from FakeGithubConnection  https://review.openstack.org/57042822:28

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