| *** paulw has joined #baserock | 06:31 | |
| *** rdale has quit IRC | 07:21 | |
| *** rdale has joined #baserock | 07:21 | |
| *** petefoth has quit IRC | 08:02 | |
| *** petefoth has joined #baserock | 08:02 | |
| *** bashrc has joined #baserock | 08:03 | |
| * tiagogomes notes that openssh 7.0 was been released http://lwn.net/Articles/654129/ | 08:17 | |
| rjek | Handful of security-related changes, but nothing burning | 08:24 |
|---|---|---|
| *** edcragg has joined #baserock | 08:40 | |
| *** jonathanmaw has joined #baserock | 08:41 | |
| * nowster goes mad. | 10:52 | |
| richard_maw | what's up? | 10:52 |
| nowster | what the hell happens here: | 10:52 |
| nowster | artifacts: | 10:52 |
| nowster | gcc-libs: build-essential-minimal | 10:52 |
| nowster | we're within build-essential, and making gcc | 10:53 |
| richard_maw | that's saying that the gcc-libs chunk artifact should be put in build-essential-minimal | 10:53 |
| nowster | WHAT THE ****! | 10:53 |
| nowster | so what's the products table at the top of build-essential doing? | 10:54 |
| nowster | and how do they conflict? | 10:54 |
| richard_maw | the one at the top is pattern matching, so you can say that every chunk artifact ending with -devel goes into a different stratum artifact | 10:55 |
| richard_maw | then IIRC the entries in the chunks list override anything given above | 10:56 |
| richard_maw | since it's a more specific assignment | 10:56 |
| nowster | yes, but why have a reverse mapping in the chunks table? | 10:56 |
| nowster | it's the wrong way round for a start | 10:56 |
| richard_maw | is it? chunk artifacts can only go into one stratum | 10:57 |
| nowster | having the same thing defined in two places is just wrong | 10:58 |
| richard_maw | if you had it the other way around you'd need to make it a mapping from stratum artifact to list of chunk artifacts | 10:58 |
| nowster | do it once | 10:58 |
| * richard_maw checks definitions for context | 10:58 | |
| nowster | so build-essential defines matching rules and then extra mappings | 10:58 |
| richard_maw | yep | 10:59 |
| richard_maw | it was a trade-off between verbosity and the need for a more specific assignment | 11:00 |
| richard_maw | I haven't yet gotten to the discussions about reworking the format, but making artifact splitting easier to understand was on the list of things I wanted to do | 11:00 |
| richard_maw | I'm sorry for all unnecessary stress my design has caused you. I tried to get more feedback at the time. | 11:03 |
| * nowster needs some more coffee. | 11:04 | |
| * richard_maw goes to grab some lunch | 11:05 | |
| *** CTtpollard has quit IRC | 12:37 | |
| *** CTtpollard has joined #baserock | 12:52 | |
| *** petefoth has quit IRC | 14:19 | |
| tiagogomes | what people think about splitting morph logging output in two files: one with the logging of the last morph command executed, and another with the the log of the previous commands | 14:43 |
| tiagogomes | the reason is, everytime I need to look at the morph.log, I need to remove it and re-run morph, as the file is too big | 14:44 |
| richard_maw | so one file for all logs, and another for the latest command? | 14:54 |
| tiagogomes | yep | 14:55 |
| richard_maw | just to clarify, the log of everything will also have the last command in it? | 14:56 |
| tiagogomes | is that relevant? Although if not I'd need to add startup code to copy what is on current to the all-logs file | 14:59 |
| richard_maw | yeah, that was what I was worried about, I'd rather it went into both at the same time | 14:59 |
| nowster | bug in strata/build-essential/glibc.morph... | 15:00 |
| nowster | - artifact: glibc-bins | 15:00 |
| nowster | include: | 15:00 |
| nowster | - (usr/)?s?bin/.* | 15:00 |
| nowster | - (usr/)?libexec/getconf/.* | 15:00 |
| nowster | - (usr/)?lib/libSegFault\.so(\.\d+)*$ | 15:00 |
| nowster | should be | 15:00 |
| nowster | - artifact: glibc-bins | 15:00 |
| nowster | include: | 15:00 |
| nowster | - (usr/)?s?bin/.* | 15:00 |
| nowster | - (usr/)?libexec/getconf/.* | 15:00 |
| nowster | - (usr/)?lib(32|64)?/libSegFault\.so(\.\d+)*$ | 15:00 |
| pedroalvarez | that change makes sense | 15:01 |
| tiagogomes | richard_maw, that the easiest way to implement it. But it will make a bit more difficult if you want to compare the log of the last command with the log of the second to last command. Not sure how useful that would be though | 15:03 |
| *** persia_ has quit IRC | 15:06 | |
| *** persia_ has joined #baserock | 15:09 | |
| *** persia_ has joined #baserock | 15:09 | |
| *** paulw has quit IRC | 15:10 | |
| tiagogomes | nowster can you send a patch? :) | 15:11 |
| nowster | wilco | 15:11 |
| nowster | gerrit! | 15:15 |
| tiagogomes | did anyone profiled morph? | 16:05 |
| *** jonathanmaw has quit IRC | 16:07 | |
| paulsherwood | tiagogomes: have you looked at ybd log output? i've tried to make it as helpful as possible | 16:07 |
| paulsherwood | tiagogomes: what do you mean by profile? | 16:07 |
| tiagogomes | paulsherwood I haven't | 16:07 |
| paulsherwood | i did build time for morph at various times | 16:08 |
| tiagogomes | paulsherwood run morph in a code profiler and see which lines/functions consumed more CPU | 16:08 |
| paulsherwood | ah, noty. | 16:08 |
| tiagogomes | hi, I am thinking in further improving morph debugging after the https://gerrit.baserock.org/#/q/topic:tiagogomes/improve-debugging series. This will possibly change code the was modified on https://gerrit.baserock.org/#/c/1025/ . Should I abandon that patch or submit another separately? if it didn't have already a +1 I'drop it | 16:17 |
| tiagogomes | Also, https://gerrit.baserock.org/#/c/1023/ could be merged | 16:18 |
| richard_maw | tiagogomes: https://gerrit.baserock.org/#/c/1023/ could indeed be merged, though it's currently suffering from path conflicts | 16:20 |
| tiagogomes | ok, I'll rebase it | 16:21 |
| richard_maw | I'd prefer if we merged https://gerrit.baserock.org/#/c/1025/ first… | 16:22 |
| richard_maw | tiagogomes: I'll give it another read through, and if I still like it I'll +2 | 16:22 |
| tiagogomes | richard_maw probably https://gerrit.baserock.org/#/c/1024/ needs to be merged first before than https://gerrit.baserock.org/#/c/1025/ | 16:23 |
| tiagogomes | otherwise at least the chroot scripts will be broken | 16:24 |
| richard_maw | tiagogomes: reviewing | 16:26 |
| *** persia_ has quit IRC | 16:42 | |
| *** persia has quit IRC | 16:42 | |
| richard_maw | tiagogomes: reviewed: you have a fd leak in https://gerrit.baserock.org/#/c/1024/ | 16:44 |
| richard_maw | though it would take a huge number of concurrent morph builds to trigger a gc fail because of it | 16:44 |
| paulsherwood | can morph actually run concurrently? | 16:45 |
| *** bashrc has quit IRC | 16:47 | |
| richard_maw | it used to be possible, provided morph didn't share a cachedir or tempdir | 16:49 |
| tiagogomes | richard_maw ok, I'll look into it tomorrow. I may just add a `if fd != None: os.close(fd)` to the except statement | 16:50 |
| tiagogomes | anyway, the current code is an improvement over the other code, as staging areas created when morph crashed will now be cleared with `morph gc` | 16:51 |
| richard_maw | tiagogomes: you'd have to nullify fd first, but if you widen the range of handled exceptions, that should be enough | 16:52 |
| tiagogomes | richard_maw that was the plan :) | 16:52 |
| richard_maw | tiagogomes: oh yes, it's definitely an improvement | 16:52 |
| richard_maw | grand! | 16:52 |
| *** tiagogomes has quit IRC | 17:01 | |
| nowster | does anyone know why the difference between autoconf-tarball in build-essentials and autoconf in core? | 17:08 |
| nowster | actually, they're both in core | 17:09 |
| nowster | ah, right... bootstrapping | 17:09 |
| nowster | ignore me! :) | 17:09 |
| *** persia has joined #baserock | 17:24 | |
| *** persia has quit IRC | 17:24 | |
| *** persia has joined #baserock | 17:24 | |
| * SotK thinks about poking around at storyboard-on-baserock | 18:45 | |
| SotK | Did anyone ever get anywhere looking at theat? | 18:45 |
| SotK | (I think persia may have looked at it once upon a time, though I may be wrong) | 18:45 |
| SotK | s/theat/that/ | 18:45 |
| *** edcragg has quit IRC | 18:51 | |
| * SotK is delighted to discover that apache and mod_wsgi appear to already be in definitions | 18:53 | |
| *** paulw has joined #baserock | 20:45 | |
| *** paulw has quit IRC | 21:15 | |
| SotK | Sadly, mostly all I've discovered is that its going to need a big reorganisation of the openstack-* strata | 21:16 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!