Thursday, 2018-07-19

clarkbShrews: corvus: somethign to think about, but when we retry boots in nodepool we reuse the same nodepool id. This makes it hard to figure out which of the retried boots leaked a neutron port for example since there could be up to three servers all with the same name00:20
clarkbmaybe we can append an iteration value to the instance names? $nodepoolid-0 $nodepoolid-1 $nodepoolid-2 for example00:20
clarkbI want to say we can do this now without creating problems with subnodes since we changed how those work00:21
clarkbI'm getting by with timestamps for now so not sure it is urgent00:21
*** harlowja has quit IRC01:09
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: Use readthedocs webhook to trigger build  https://review.openstack.org/57943401:49
openstackgerritTristan Cacqueray proposed openstack-infra/zuul-jobs master: Add ansible-lint job  https://review.openstack.org/53208302:13
*** SpamapS has quit IRC02:15
ianwcan you set "vars:" for each of your entries in the templates: section?02:15
openstackgerritMerged openstack-infra/zuul master: Remove redhat-rpm-config bindep  https://review.openstack.org/58125202:18
openstackgerritMerged openstack-infra/zuul master: Fix github setup in zuul from scratch  https://review.openstack.org/58125302:18
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: Add role to trigger readthedocs via authenticated endpoint  https://review.openstack.org/57943402:28
tristanCa new regression with master: https://storyboard.openstack.org/#!/story/200302602:40
openstackgerritIan Wienand proposed openstack-infra/zuul-jobs master: Remove trigger-readthedocs job  https://review.openstack.org/58383603:34
*** jimi|ansible has joined #zuul03:50
*** jimi|ansible has joined #zuul03:50
*** harlowja has joined #zuul03:51
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: github: log the project name when getBranch fail  https://review.openstack.org/58384104:21
*** harlowja has quit IRC04:22
*** pcaruana has joined #zuul04:23
*** pcaruana has quit IRC04:34
*** jimi|ansible has quit IRC05:19
*** nchakrab has joined #zuul05:22
*** quiquell|off is now known as quiquell05:26
*** robcresswell has quit IRC05:29
*** fdegir has quit IRC05:29
*** robcresswell has joined #zuul05:29
*** gundalow_ has joined #zuul05:29
*** gundalow_ has quit IRC05:29
*** gundalow_ has joined #zuul05:29
*** fdegir has joined #zuul05:29
*** LinuxJedi_ has joined #zuul05:30
*** portdirect_ has joined #zuul05:30
*** LinuxJedi has quit IRC05:30
*** mwhahaha has quit IRC05:30
*** portdirect has quit IRC05:30
*** LinuxJedi_ is now known as LinuxJedi05:30
*** portdirect_ is now known as portdirect05:30
*** mwhahaha has joined #zuul05:30
*** gundalow has quit IRC05:31
*** dvn has quit IRC05:31
*** jamielennox has quit IRC05:31
*** gundalow_ is now known as gundalow05:31
*** portdirect is now known as Guest7295205:32
*** nchakrab has quit IRC05:43
*** threestrands has quit IRC05:49
*** LinuxJedi has quit IRC05:53
*** LinuxJedi has joined #zuul05:53
*** hashar has joined #zuul06:03
*** dvn has joined #zuul06:04
*** threestrands has joined #zuul06:09
*** threestrands has quit IRC06:09
*** threestrands has joined #zuul06:09
*** nchakrab has joined #zuul06:10
*** jamielennox has joined #zuul06:11
*** gundalow is now known as gundalow__06:13
*** nchakrab_ has joined #zuul06:26
*** gundalow__ is now known as gundalow06:28
*** nchakrab has quit IRC06:30
*** magickris has quit IRC06:32
*** threestrands has quit IRC06:36
*** pcaruana has joined #zuul06:37
*** nchakrab_ has quit IRC06:52
*** openstackgerrit has quit IRC07:04
*** nchakrab has joined #zuul07:23
*** nchakrab has quit IRC07:23
*** nchakrab has joined #zuul07:24
*** electrofelix has joined #zuul08:51
*** sambetts|afk is now known as sambetts09:15
*** nchakrab_ has joined #zuul09:40
*** nchakrab has quit IRC09:43
*** nchakrab_ has quit IRC09:50
*** openstackgerrit has joined #zuul09:50
openstackgerritFabien Boucher proposed openstack-infra/zuul master: Add tenant yaml validation option to zuul client  https://review.openstack.org/57426509:50
*** nchakrab has joined #zuul10:16
*** quiquell is now known as quiquell|lunch11:21
*** GonZo2000 has joined #zuul11:37
*** GonZo2000 has quit IRC11:58
*** jimi|ansible has joined #zuul12:09
*** jimi|ansible has joined #zuul12:09
*** GonZo2000 has joined #zuul12:10
*** GonZo2000 has joined #zuul12:10
*** dkranz has quit IRC12:22
*** nchakrab has quit IRC12:31
*** nchakrab has joined #zuul12:31
*** ssbarnea1 has quit IRC12:36
*** ssbarnea has joined #zuul12:38
*** rlandy has joined #zuul12:40
*** Guest72952 is now known as portdirect12:41
*** quiquell|lunch is now known as quiquell12:45
*** samccann has joined #zuul12:54
-openstackstatus- NOTICE: logs.openstack.org is offline, causing POST_FAILURE results from Zuul. Cause and resolution timeframe currently unknown.12:59
*** ChanServ changes topic to "logs.openstack.org is offline, causing POST_FAILURE results from Zuul. Cause and resolution timeframe currently unknown."12:59
*** dkranz has joined #zuul13:21
*** gundalow is now known as gundalow_____13:33
*** gundalow_____ is now known as gundalow13:34
mordredianw: no - you can't apply vars to a template - although this is potentially a good example of a use case for such a thing13:40
mordredianw: we've also discussed being able to set vars on a project itself - which this also might be a good example of a use case for13:40
*** ChanServ changes topic to "Discussion of the project gating system Zuul | Website: https://zuul-ci.org/ | Docs: https://zuul-ci.org/docs/ | Source: https://git.zuul-ci.org/ | Channel logs: http://eavesdrop.openstack.org/irclogs/%23zuul/ | Weekly updates: https://etherpad.openstack.org/p/zuul-update-email"13:43
-openstackstatus- NOTICE: logs.openstack.org is back on-line. Changes with "POST_FAILURE" job results should be rechecked.13:43
*** gundalow is now known as gundalow___13:45
*** gundalow___ is now known as gundalow13:46
*** nchakrab has quit IRC14:01
*** gundalow has quit IRC14:26
*** gundalow has joined #zuul14:27
*** quiquell is now known as quiquell|off14:32
*** nchakrab has joined #zuul14:33
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: Add a dequeue command to zuul client  https://review.openstack.org/9503514:45
*** pcaruana has quit IRC15:12
*** sshnaidm|rover is now known as sshnaidm|afk15:24
*** GonZo2000 has quit IRC15:33
*** GonZo2000 has joined #zuul15:34
*** GonZo2000 has quit IRC15:54
rcarrillocruzfolks, is there a zuul var to tell the zuul user ?16:00
rcarrillocruzif not, do you introspect that from zuul src_dir or something ? if ever...16:00
clarkbrcarrillocruz: ansible_user: zuul ?16:01
rcarrillocruzohhhhh16:01
rcarrillocruzdidn't think about ansible vars16:01
rcarrillocruzofc, thx!16:01
rcarrillocruzso zuul puts remote_user as ansible_user in the inventory then?16:01
rcarrillocruzthe default_remote_user, can't remmeber the param name from zuul conf files16:02
clarkbI think so16:02
rcarrillocruzand if username is defined on node, i guess it plumbs from there16:02
rcarrillocruzcool16:02
pabelangeransible_user is mostly what I use16:02
pabelangerand ansible_user_dir16:03
*** sshnaidm|afk is now known as sshnaidm|rover16:11
*** nchakrab has quit IRC16:30
*** SpamapS has joined #zuul16:44
*** acozine1 has joined #zuul16:47
*** hashar is now known as hasharAway16:51
clarkbkata has noticed that the details link from zuul on github PRs takes you to the status page rather than showing you logs. We provide log file links in the comments directly instead. I want to say spamaps was trying to update this?16:56
SpamapSYes, it's not too hard.16:57
SpamapSYou just have to rebuild the log schema in the pipeline def16:57
*** elyezer has quit IRC16:57
SpamapSstatus-url: ....16:57
clarkbSpamapS: and point it at the uuid prefix of the log path in our case I'm guessing16:58
*** elyezer has joined #zuul16:58
mhutobiash, SpamapS pabelanger I think https://review.openstack.org/#/c/95035/ is ready (dequeue in CLI), would you mind having a look?17:00
pabelangersure17:00
clarkbSpamapS: will that set the status url when jobs start? I think what we need is a way for it to be the dashboard link when jobs are running and he logs link when jobs are done?17:07
pabelangermhu: left comments17:10
clarkboh we set status-url for each report action so can set it only for success and failure17:11
pabelangerwith new quota management in nodepool, could that be used to check if a provider has enough FIPs, before trying to boot a node?17:30
clarkbpabelanger: yes, though I don't think that is one of the quotas currently checked17:31
clarkbyou'd also have to know if FIPs were required to produce working instacnes17:32
pabelangeryah, today we have cores, instances, ram17:32
pabelangeryah17:33
*** sambetts is now known as sambetts|afk17:33
pabelangerI don't think that will fix this issue we are having downstream, some reason failing to attach FIP to instances it seems17:33
pabelangertrying to first check if that is because no FIPs are available or some other networking issue17:34
clarkbSpamapS: corvus: working out a status-url we don't make the last two characters of the change number available to the item format call from what I can tell so I'm not sure openstack can format a valid url for its log path17:36
clarkbis that something that would be reasonable to add to Change() as number_suffix attribute?17:36
corvuswait back up17:37
corvusthe log url is an opaque string to zuul.  it should not be in the zuul config at all.  it's something that the job gives back to zuul when it's complete.17:38
clarkbcorvus: for specific jobs yes, but we want to have a github status url that goes to eg http://logs.openstack.org/91/91/df28eaf69b0fbb020a64daa6b8f483ed08d95d61/third-party-check/ instead of https://zuul.openstack.org/ once jobs are complete17:38
clarkband that status-url is a setting in the pipeline config17:38
corvusthe log urls are included in the report when the item reports, yeah?17:40
clarkbcorvus: as a distinct comment yes17:40
corvusi think earlier we talked about making the status url a deep link to the status page with the change id17:41
corvusi don't think we've talked about having it be anything related to the logs17:41
clarkbah17:41
corvusif it's just one url, i don't think we have an appropriate single thing to link to yet17:41
clarkbcorvus: https://github.com/kata-containers/proxy/pull/91 scroll to the bottom and see the details link for openstack third party check17:42
clarkbit is a single url link17:42
corvusif we go down the path i wrote about in the email, we'll have 'build' pages in zuul-web.  maybe we can also add a 'buildset' page, and then that would be a good thing to set as the status url.17:42
clarkbok, I was just going off of spamaps comment that there was a reasonable thing to do today setting the status-url17:43
clarkbI could see the logs url example above being a reaonsble stand in for your suggestion until it is implemented17:43
corvusclarkb: i think if you did that, you'd lose the link to the status page17:43
clarkbpossibly, though I can set it for succes/failure only and I think it will convert the link over only after all jobs are done running17:44
clarkbbut I can't slice a string in a format template so would need zuul to set that value for me17:46
corvusclarkb: you can?  which attribute are you looking at?17:46
clarkbchange.number, specifically I need the last two digit of change.number for the first part of the log url17:47
corvusclarkb: no i'm asking how you set the status url on success or failure17:47
corvusi think you may be confusing status-url (a pipeline configuration) with success-url and failure-url (a job configuration)17:48
clarkbI could be17:48
clarkbcorvus: the docs have it listed at pipeline.<reporter>.<github source>.status-url17:49
corvusstatus-url applies to the whole item, and doesn't change based on success/failure.  success-url and failure-url are the log urls for the individual jobs (the things that show up in the report -- the pr comment in the case of github)17:49
clarkbhttps://zuul-ci.org/docs/zuul/admin/drivers/github.html#reporter-configuration17:50
corvusyeah, you can deep link to those: https://zuul-ci.org/docs/zuul/admin/drivers/github.html#attr-pipeline.%3Creporter%3E.%3Cgithub%20source%3E.status-url17:50
clarkbpipeline.success.github.status-url is the path I'm looking at17:50
clarkbsuccess reporter, github github driver, status-url17:51
clarkbwe do set status there already so I think things line up at least as far as the docs go17:51
clarkbthen when all jobs are complete it will report success or failure and set the url to http://logs.openstack.org/91/91/df28eaf69b0fbb020a64daa6b8f483ed08d95d61/third-party-check/17:52
SpamapSIt works great for us.17:52
SpamapSSetting the status url on success, and not commenting, is a nice experience.17:53
clarkbexcept that I need to slice the change.number value17:53
corvusclarkb: so, today, you can either have that link to the status page, which admittedly doesn't make as much sense after the item has reported; or you can do what you're suggesting and link to the future expected location on the log server, which won't make sense until the job reports.  i agree the situation isn't ideal, but until there's a real buildset page in zuul-web, there isn't really anything17:53
corvusappropriate to link to to summarise the report (the report is in the pr comment).  given that otherwise there's no status page link, i think the most appropriate thing is to use the status page link.17:53
clarkbcorvus: ok, but what I've described would be functional right?17:54
corvusclarkb: oh i see what you're saying; using different status-urls in start/success/failure reporters17:54
clarkbcorvus: yes17:54
*** electrofelix has quit IRC17:54
corvusso you could set the status page in start, then something else in success/failure.  yeah that should work.17:54
clarkbyup and my thought is to set it to the url above but that requires having access to the last two digits of the change number because you cannot slice in str.format() aiui17:55
corvusclarkb: i'd like to suggest in #openstack-infra why we shouldn't do that though.  it's a very openstack-infra specific reason.  :)17:55
clarkbif we don't want to do it anyway then probably don't need to modify model.Change17:55
*** sshnaidm|rover is now known as sshnaidm|off17:57
corvusi think the best long-term thing for zuul is to have the buildset page.  i would not recommend using parallel construction of log urls for something like this -- it has served us very poorly in the past and that informed the design in v3 where we very explicitly decided to stop requiring it.  it may work fine in some environments.  i won't suggest that SpamapS shouldn't do it if it's working for him.  :)17:58
corvusi think the buildset page idea will work for everyone17:58
clarkbcorvus: I agree, but would this be an acceptable interim soltion to reduce confusion over why you get a status page without your change on it?17:58
SpamapSI'd love to have a build page to point at17:58
SpamapSthe thing we have now is an nginx index page17:59
clarkbI think part of the problem here is github users don't look for distcint comments, they look at the ci details links17:59
SpamapSYeah, comments are also spammy17:59
corvuswell, github has a new ci reporting system.  zuul doesn't use it yet, but should.  i believe that would let us leave individual job log links in an appropriate non-comment place (the equivalent of gerrit's new reporting system)18:01
corvusi'm not aware of anyone having begun work on implementing that, but that's also likely a very important part of improving this experience18:02
clarkbcorvus: ya the jenkins builds seem to use it on that PR example I gave earlier18:03
clarkbthey are broken out as separate jobs with their own details18:03
clarkb(though that may be old system hackage too)18:03
corvusclarkb: where's the jenkins thing, i don't see it?18:04
clarkbcorvus: jenkins-ci-centos-7-4 and jenkins-ci-fedora-27 and so on in that list18:04
corvusclarkb: the "checks" api is what i was thinking of: https://github.com/kata-containers/proxy/pull/91/checks18:04
clarkbah18:04
corvusclarkb: i really don't see the word 'jenkins' on https://github.com/kata-containers/proxy/pull/9118:05
clarkbif you scroll to the bottom with the some checks were not successful box they are in that and you ahve to scroll in that box18:06
clarkbthat may be a case of jenkins reporting with old api but not aggregating results and reporting jobs individually18:06
corvusclarkb: are you logged in?18:06
corvusi don't see "some checks were not successful"18:07
clarkbyes I am18:07
corvusok.  well, anyway, i'm pretty sure the checks interface will also do what all of us want, and, it looks like nothing on that PR is using it, assuming https://github.com/kata-containers/proxy/pull/91/checks is also empty for your logged-in-self as it is for my non-logged-in-self :)18:08
SpamapSNo I don't think those are the new system.18:08
corvusin fact, if you look at the example images there, it's got all the components of a zuul18:09
corvus-style report18:09
SpamapS"Checks" is the new system, "Statuses" is the old one.18:09
SpamapSand "Checks" can do line-item comments18:10
SpamapShttps://help.github.com/articles/about-status-checks/18:10
corvusso, anyone want to implement that? :)18:10
SpamapSwant, yes18:10
SpamapScan... :(18:10
clarkbSpamapS: re noisy comments I'm the weirdo that expands all comments in gerrit reviews18:12
corvusi mean, at least zuul doesn't just draw a giant green box that takes up 1/4 of the screen.  :)18:14
pabelangerha18:14
corvusanyway, implementing the checks api would be a very welcome improvement if someone wants to do it.18:15
corvusdon't worry about the line comments -- i'm plumbing those through zuul in a separate effort anyway.  so the infrastructure will be there for checks to use.18:15
clarkbI'll admit I lost all patience for the github api ~6 years ago when I made the PR request closer18:16
clarkb(PRs didn't have a delete method you had to delete the issue that mapped to the PR except you can't list them that way and os you just make a leap of faith and it works!)18:17
clarkber not delete, close18:17
tobiashThe checks api is on my agenda but probably not within the next two months18:18
SpamapSclarkb: it's not the expanded comments, it's the emails, that people get all weird about.18:20
SpamapSThe GitHub API is like a box of chocolates. You never know what version you're gonna get.18:21
mordredSpamapS: I'm guessing people are using the emails for conversation and don't like zuul comments showing up there because if conflics with the conversation usecase?18:40
SpamapSYeah, though the failures are helpful as a "hey you have to do something".18:42
SpamapSSo we comment on fail, but not success.18:42
SpamapSMost of the time on success nobody needs to read the logs. But if they do, it's set in the status url.18:42
mordrednod18:43
mordrednewer gerrit also supports replying to emails and incorporating the reply into the comment stream ... and also has the new api for reporting results - so I could see a similar pattern being desired18:45
clarkbthe email based commenting is still rudimentary based on conversations on their mailing list (at least that was my impression of it)18:45
clarkbbut it is a long asked for feature so yay18:46
mordredyah18:46
mordredI mean, I don't want to use it myself- but I know there will be people who will18:46
*** hasharAway is now known as hashar19:03
*** acozine1 has quit IRC19:19
*** myoung is now known as myoung|biab20:00
*** dkranz has quit IRC20:00
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: Add a dequeue command to zuul client  https://review.openstack.org/9503520:19
openstackgerritMatthieu Huin proposed openstack-infra/zuul master: web: add /{tenant}/projects and /{tenant}/project/{project} routes  https://review.openstack.org/55097920:40
*** samccann has quit IRC20:44
*** samccann has joined #zuul20:45
*** samccann has joined #zuul20:46
*** samccann has quit IRC20:49
*** samccann has joined #zuul20:52
*** samccann has quit IRC20:55
*** samccann has joined #zuul20:56
*** acozine1 has joined #zuul21:06
*** samccann has quit IRC21:09
mhutristanC, mordred do you have an ETA on the zuul-web-routes topic ? https://review.openstack.org/#/q/topic:zuul-web-routes+(status:open+OR+status:merged)21:15
*** myoung|biab is now known as myoung21:16
mhuI would like to rebase on these so that I can account for these extra routes in my web api patch21:16
mhubut if the topic needs more work I'll keep the patches independent21:17
*** hashar has quit IRC21:55
*** goern_ has joined #zuul22:13
*** goern has quit IRC22:15
*** goern__ has joined #zuul22:15
*** goern_ has quit IRC22:18
*** harlowja has joined #zuul22:21
rcarrillocruzfolks22:49
rcarrillocruzhttps://livestream.com/accounts/1542415/events/827628122:49
rcarrillocruzzuul in slide, live :D22:50
rcarrillocruzcorvus mordred pabelanger ^22:50
pabelangerwatching22:52
pabelangerrcarrillocruz: where is this happening?22:53
rcarrillocruzhttp://techfieldday.com/22:53
rcarrillocruzsilicon valley, not sure where22:53
pabelangerrcarrillocruz: neat22:55
pabelangerlook like redhat office some place22:56
pabelangersomebody knew of zuul :)22:56
*** acozine1 has quit IRC22:59
*** rlandy is now known as rlandy|bbl22:59
rcarrillocruzheh23:06

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