| pedroalvarez | Hm.. ci.morph broken (see mason) | 07:38 |
|---|---|---|
| pedroalvarez | And no more morph edit | 07:38 |
| paulsherwood | erk | 07:38 |
| pedroalvarez | :/ I will have to fix the license check in definitions.git | 07:38 |
| paulsherwood | can i help? | 07:39 |
| pedroalvarez | I'll send a quick patch for ci.morph that you can review :) | 07:40 |
| paulsherwood | ok | 07:41 |
| tlsa | hi | 07:49 |
| pedroalvarez | hey tlsa :) | 07:50 |
| tlsa | I thew this together last night: http://www.smoothartist.com/temp/ciat.html | 07:50 |
| tlsa | it's not actually monitoring what's happening on the system | 07:50 |
| pedroalvarez | This should put mason green: https://gerrit.baserock.org/#/c/1138 | 07:50 |
| pedroalvarez | tlsa: hey, that's nice | 07:50 |
| tlsa | just a demo of what the UI could look like | 07:50 |
| pedroalvarez | I guessed that (it's not live :P) | 07:51 |
| tlsa | :) | 07:51 |
| pedroalvarez | have you found the way to get all the information needed from buildbot? | 07:52 |
| paulsherwood | tlsa: fab. do you hate rounded corners? | 07:52 |
| tlsa | it produces json, which could be parsed and used to generate the page | 07:52 |
| paulsherwood | pedroalvarez: merged | 07:53 |
| pedroalvarez | paulsherwood: ta! | 07:53 |
| paulsherwood | i like the font choice... reminds me of Blake's Seven :) | 07:53 |
| pedroalvarez | heh | 07:54 |
| tlsa | paulsherwood: now with rounded corners | 07:54 |
| paulsherwood | :-) | 07:58 |
| paulsherwood | maybe a teesy bit smaller rounds? | 07:58 |
| tlsa | meh, it's triavially changed, the important thing is somone would need to hook it up so it shows whats currently happening | 07:59 |
| paulsherwood | ack. | 08:00 |
| paulsherwood | i wonder how best to show the parallel tracks. buildbot waterfall is actually quite confusing for me | 08:00 |
| paulsherwood | maybe it could just be lots of those sets-of-four blocks. provide an identifier/label for each | 08:02 |
| tlsa | I was going to do something like: | 08:02 |
| tlsa | [Firehose]----[Build x86]----[Deploy]----[Test] | 08:03 |
| tlsa | \ | 08:03 |
| tlsa | `-[Build arm] | 08:03 |
| paulsherwood | ah, ok. and if n targets, n of those branches? | 08:03 |
| tlsa | but I did not have enough time to spend on it, and am on something else now | 08:03 |
| paulsherwood | ack | 08:03 |
| paulsherwood | i like the ideas | 08:03 |
| tlsa | Well, I'd hae collapsed it to just show one "Build" block unless you click on it to "open" it, or unless the ones under "build" have mixed pass/fail states | 08:04 |
| paulsherwood | even better | 08:06 |
| paulsherwood | that would be pretty cool, actually. | 08:06 |
| * pedroalvarez imagines a UI full of boxes and lines, representing the complexity of CIAT but with some buttons to filter per architecture, firehose sets, and more things to see simpler views of it | 08:07 | |
| paulsherwood | :-) | 08:08 |
| pedroalvarez | more things are broken from the gnome system patch | 08:12 |
| pedroalvarez | I guess jjardon will want to lorry m4-common | 08:13 |
| * SotK tries to make sense of BuildBot's JSON API | 09:16 | |
| tlsa | SotK: You can get whether they're idle or not from: http://ciat.baserock.org:8010/json/builders/?as_text=1 | 09:20 |
| tlsa | and you can get the number of the last cached build | 09:20 |
| tlsa | so for "2. x86_64 Build", that's 24 | 09:20 |
| tlsa | so you can construct this URL from that: http://ciat.baserock.org:8010/json/builders/2.%20x86_64%20Build/builds/24?as_text=1 | 09:22 |
| tlsa | and from there you can extract whether or not it was "successful" | 09:22 |
| SotK | tlsa: brilliant, thanks! | 09:30 |
| SotK | Is there a way to figure out how the pipeline is arranged beyond guessing based on the names? | 09:58 |
| * SotK has to stop for a bit now, I shoved some javascript into tlsa's demo UI from earlier: http://217.44.72.233/ | 10:25 | |
| paulsherwood | cool :-) | 10:27 |
| pedroalvarez | :D | 10:33 |
| tlsa | SotK: nice :) | 10:34 |
| pedroalvarez | tlsa: now that you are around, would you be so kind of giving me access to the ciat machine? :) | 10:34 |
| tlsa | pedroalvarez: the machine on aws? | 10:35 |
| pedroalvarez | tlsa: yes, please :) | 10:35 |
| tlsa | let me see | 10:36 |
| * paulsherwood thinks that maybe kbas could serve git repos to other ybd instances too, save everything hitting g.b.o | 10:43 | |
| *** zoli__ has joined #baserock | 11:05 | |
| pedroalvarez | thanks tlsa! | 11:06 |
| tlsa | SotK: that won't set the "active", class yet, if I read it right | 11:06 |
| tlsa | pedroalvarez: yw | 11:06 |
| *** SotK_ has joined #baserock | 11:34 | |
| tlsa | hi SotK_ | 11:34 |
| SotK_ | tlsa: correct, it'll be simple to do though | 11:34 |
| * SotK_ had to go out before he could do it | 11:35 | |
| tlsa | :) | 11:35 |
| tlsa | you can remove the ",400" from the font URL | 11:36 |
| tlsa | I decided not to use that weight for the body text; it looked nasty | 11:36 |
| *** SotK_ has quit IRC | 11:41 | |
| pedroalvarez | jjardon: thanks for fixing m4-common, although it may fail because it has .gitmodules | 12:30 |
| jjardon | hi | 12:39 |
| jjardon | I wonder if it would be better to improve morph/ybd to make the translation automatically instead us patching all the repos that have .gitmodules | 12:40 |
| jjardon | Any idea if this has been discussed before? | 12:40 |
| paulsherwood | i don't think it has, jjardon | 12:44 |
| paulsherwood | you're suggesting replace http://foo.bar/submodule-name with upstream:submodule-name by default ? | 12:46 |
| * jjardon just realize some of the gnome chunks are probably using gitmodules as well | 12:46 | |
| jjardon | paulsherwood: yes | 12:46 |
| paulsherwood | seems sensible to me | 12:48 |
| pedroalvarez | I remember some discussions to add some functionality to morph to solve this problem | 13:21 |
| jjardon | mmm, Im getting a weird error here: http://paste.baserock.org/mihanucoti | 13:26 |
| jjardon | any idea what I can be doing wrong? | 13:26 |
| jjardon | mmm, it happen with current master as well | 13:27 |
| jjardon | oh, this is probably related with some of the recent changes in morph | 13:29 |
| SotK | If a build is in progress, should the colour of the box indicate the success/failure of the previous build? | 14:08 |
| tlsa | it should always show the result of the previous build | 14:08 |
| pedroalvarez | jjardon, paulsherwood: regarding replacing url's, we can't do that for all of them, given that we haven't been consitent when creating lorries | 14:08 |
| tlsa | and if there's an active build in progress, set the active class | 14:08 |
| pedroalvarez | jjardon, paulsherwood: although we could use lorries.git to get that info I guess | 14:09 |
| pedroalvarez | tlsa: so, if building, and last build successful, green and the in-progress indicator? | 14:10 |
| tlsa | yep | 14:10 |
| tlsa | and if last build failed, and its now doing a new build: red + flashing | 14:10 |
| SotK | tlsa: done :) | 14:14 |
| SotK | (not the failed part actually, is there an example of a failed build somewhere?) | 14:14 |
| tlsa | let me see | 14:15 |
| tlsa | http://52.19.1.31:8010/json/builders/2.%20x86_64%20Build/builds/23?as_text=1 | 14:16 |
| tlsa | has "failed" | 14:16 |
| SotK | thanks | 14:19 |
| tlsa | if the builder has no history (say its just been restarted), then neither the pass nor fail class should be set :) | 14:20 |
| pedroalvarez | there is another status I believe: interrupted | 14:20 |
| tlsa | (that should show the box as gray, rather than red/green) | 14:20 |
| tlsa | pedroalvarez: that would show as gray too | 14:21 |
| * pedroalvarez nods | 14:21 | |
| tlsa | antway, noce work SotK :) | 14:21 |
| tlsa | *anyway | 14:21 |
| pedroalvarez | indeed, looking good | 14:21 |
| tlsa | SotK: does it keep refreshing automatically? | 14:22 |
| SotK | tlsa: not yet | 14:22 |
| SotK | any opinons on a sensible refresh rate? | 14:22 |
| tlsa | 1 min? | 14:23 |
| SotK | seems sensible | 14:26 |
| tlsa | 'fail': !step.lastBuild.success maybe needs to be 'fail': !step.lastBuild.failed | 14:27 |
| SotK | grr, I updated angularjs and now the lastBuild isn't working properly | 14:57 |
| tlsa | SotK: if you update to the CSS from http://www.smoothartist.com/temp/ciat.html it should improve the spacing a bit | 15:23 |
| SotK | tlsa: thanks! | 15:28 |
| SotK | should be auto-refreshing the data once per minute now :) | 15:41 |
| *** pedroalvarez has quit IRC | 18:03 | |
| *** pedroalvarez has joined #baserock | 18:04 | |
| *** ChanServ sets mode: +v pedroalvarez | 18:04 | |
| pedroalvarez | tlsa: so, was something like this your idea to add support to specify users? http://paste.baserock.org/izudiracif | 19:03 |
| *** _longines has quit IRC | 19:16 | |
| pedroalvarez | I've decided to do it that way, although the wait_for_ssh also needs an user to do the actual check (which is ssh in and run true) | 19:39 |
| pedroalvarez | I guess that can be replaced with something else | 19:39 |
| pedroalvarez | maybe check that there is something in the port 22 | 19:41 |
| tlsa | hi | 20:25 |
| *** zoli__ has quit IRC | 21:34 | |
| *** zoli__ has joined #baserock | 22:01 | |
| *** zoli__ has quit IRC | 22:05 | |
| *** zoli__ has joined #baserock | 22:06 | |
| *** zoli__ has quit IRC | 22:53 | |
| *** zoli__ has joined #baserock | 23:49 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!