IRC logs for #baserock for Tuesday, 2015-06-23

*** edcragg has quit IRC01:04
*** rdale has quit IRC02:33
*** zoli__ has joined #baserock06:29
*** edcragg has joined #baserock07:34
*** edcragg has quit IRC07:40
*** mariaderidder has joined #baserock07:45
*** gary_perkins has joined #baserock07:51
*** mike has joined #baserock07:52
*** mike is now known as Guest6820007:52
*** persia has quit IRC07:53
*** persia has joined #baserock07:53
*** persia has joined #baserock07:53
*** jonathanmaw has joined #baserock07:55
*** bashrc_ has joined #baserock08:03
* SotK sheepishly sends https://gerrit.baserock.org/914 to finish fixing the mess he made :)08:48
paulsher1oodSotK: was the problem that you didn't have an environment to test the original work in?08:53
SotKmostly yes08:53
paulsher1oodmaybe better to flag that and get help in future08:54
SotKupgrading rawdisks was broken because I forgot that it was possible to upgrade rawdisks when I was testing08:54
SotKpaulsher1ood: probably :/08:54
paulsher1oodSotK: i've done the same, don't worry :)08:54
paulsher1oodSotK: so please can you amend your patch to state what testing has been done, and then reviewers can take a view on whether they need to review, or test and review?08:59
*** ssam2 has joined #baserock09:02
*** ChanServ sets mode: +v ssam209:02
tiagogomes_That would be a nice addition to the cover letter, ah wait....09:02
*** rdale has joined #baserock09:02
*** De|ta_ is now known as De|ta09:03
paulsher1oodis there a cover letter somewhere? am i missing it?09:03
* SotK leaves a comment on the patch09:04
tiagogomes_paulsher1ood no sorry for confusing you. It was a way of saying that gerrit doesn't support cover letters, so you'll have to drop that kind of information in the commit message09:06
*** edcragg has joined #baserock09:13
ssam2or a comment09:13
ssam2SotK: given quite a few .write extensions in definitions.git seem to be broken in the baserock-15.25 tag, could you do a baserock-15.25.1 tag once https://gerrit.baserock.org/#/c/914/ is merged ?09:19
SotKssam2: sure09:19
SotKwait, which are broken in 15.25?09:20
ssam2oh, actually in 15.25 the branch to remove morphlib and cliapp wasn't merged, was it?09:20
SotKcorrect09:20
ssam2so I guess if people are following the instructions on the wiki they won't hit any of those issues09:20
ssam2never mind then!09:20
*** lachlanmackenzie has joined #baserock09:21
ssam2mmm, strata: https://gerrit.baserock.org/#/c/908/09:21
ssam2I guess that diff looks so bad because we still have a different .morph file for each architecture..09:22
pedroalvarezThe distbuild bug of 'building many times the same chunk' is still alive. Does anybody know anything about it?09:49
ssam2no, haven't investigated beyond running `systemctl status mason.service` on a Mason09:50
SotKonly that I've never seen it except on Mason09:50
ssam2which shows that scripts/release-build runs 4 `morph distbuild-morphology` instances in parallel. But distbuild has code to handle that situation and only actual build each chunk once09:50
*** sambishop has quit IRC09:50
*** sambishop has joined #baserock10:05
*** ctgriffiths_ has quit IRC10:05
*** ctgriffiths has joined #baserock10:05
*** a1exhughe5 has joined #baserock10:24
*** ssam2 has quit IRC10:48
*** ssam2 has joined #baserock10:49
*** ChanServ sets mode: +v ssam210:49
*** zoli__ has quit IRC10:56
*** zoli__ has joined #baserock11:25
*** pacon has joined #baserock11:53
*** pacon has quit IRC11:55
*** fay_ has joined #baserock12:27
*** fay__ has quit IRC12:29
pedroalvarezssam2: I'm not really good at tests yet, but I've managed to reproduce the problem without having a mason. I've created a fake stratum with fake 100 fake chunks, and when I kick several parallel builds of a system with this stratum, somem chunks get built twice12:47
ssam2cool12:53
ssam2i've done a patch but I think it doesn't work12:54
pedroalvarezssam2: have you found any cleverer way to test it?12:54
ssam2no12:59
nowsterHi all. Is there any reason why morph might go daft with artefact splitting and make a ${name}-misc chunk containing everything in /${name}.inst/ but not putting anything in, say ${name}-libs or ${$name}-devel chunks?14:12
pedroalvareznowster: there are some default rules for artefact splitting14:14
rjekThis may be of interest https://bits.debian.org/2015/06/reproducible-builds-funded-by-cii.html14:15
nowsterit splits most things but doesn't seem to do "attr" properly14:15
* nowster pulls in most recent morph14:15
nowsterand purges old chunks14:15
nowsterThis is what I'm getting:14:19
nowsterhttp://paste.baserock.org/anotefijiw14:19
tiagogomes_nowster that could happen if the chunk didn't install any library or header files14:20
nowsterhttp://paste.baserock.org/erotopekix14:21
pedroalvarezwell, the "attr.inst/" prefix doesn't look good14:21
ssam2rjek: yes, good news14:22
ssam2nowster: double DESTDIR ?14:22
ssam2what commit of definitions.git are you building?14:22
pedroalvareznowster: I think i fixed that issue in b1fe8c5459e0fc193f656f4a8547c710b855487614:24
nowsterpedroalvarez: of morph?14:24
pedroalvarezdefinitions.git14:25
nowsterright!14:25
pedroalvareznowster: were you hitting a different problem when you decided to start looking at attr artefact?14:26
nowsterI'm working off a branch which is very unlikely ever to be merged back.14:26
nowsterand it's all rdale's fault :)14:30
rdalewell...14:30
nowsterI've tweaked a few things and will see if it still builds wrong14:30
*** jonathanmaw has quit IRC14:31
*** mariaderidder has quit IRC14:34
*** jonathanmaw has joined #baserock14:34
*** mariaderidder has joined #baserock14:34
*** zoli__ has quit IRC14:37
*** zoli__ has joined #baserock14:47
*** zoli__ has quit IRC14:52
pedroalvarezgreat, Sam has fixed distbuild: https://gerrit.baserock.org/#/q/status:open+project:baserock/baserock/morph+branch:master+topic:sam/distbuild-fix-double-builds15:19
pedroalvarezreviews from people familiar with the code, welcome15:19
*** zoli__ has joined #baserock15:20
*** fay_ has quit IRC15:29
*** fay_ has joined #baserock15:30
*** a1exhughe5 has quit IRC15:55
*** fay_ has quit IRC15:58
*** zoli__ has quit IRC16:12
*** Guest68200 has quit IRC16:22
* SotK reviews it16:26
*** Guest68200 has joined #baserock16:37
pedroalvareznearly there :)16:38
pedroalvarezthanks for reviewing!16:38
pedroalvarezon a parallel universe, mason has fiished building16:39
*** mariaderidder has quit IRC16:42
* SotK finishes reviewing it, looks good16:42
SotKthanks ssam2!16:42
* pedroalvarez has a patch to add more feedback to mason16:47
* pedroalvarez tests it https://mason-x86-64.baserock.org/16:55
pedroalvarezlooks like I missed something though16:57
*** jonathanmaw has quit IRC17:02
*** bashrc_ has quit IRC17:02
*** Guest68200 has quit IRC17:04
*** edcragg has quit IRC17:13
*** Guest68200 has joined #baserock17:17
*** ssam2 has quit IRC17:19
*** franred has joined #baserock17:20
*** CTtpollard has quit IRC17:21
*** sambishop has quit IRC17:25
*** lachlanmackenzie has quit IRC17:29
*** Guest68200 has quit IRC17:31
*** sambishop has joined #baserock17:46
*** gary_perkins has quit IRC18:08
*** franred has quit IRC19:00
*** edcragg has joined #baserock19:58
*** flatmush has quit IRC21:32
*** flatmush has joined #baserock21:32
*** franred has joined #baserock22:17
*** franred has quit IRC22:19
*** edcragg has quit IRC22:23
*** pacon has joined #baserock22:33
*** pacon has quit IRC23:13

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