IRC logs for #baserock for Thursday, 2016-09-22

*** gtristan has joined #baserock05:46
*** fay has joined #baserock06:48
*** fay is now known as faybrocklebank06:48
*** rdale has joined #baserock07:21
*** ctbruce has joined #baserock07:32
*** toscalix has joined #baserock08:00
*** tiagogomes has joined #baserock08:57
*** CTtpollard has quit IRC08:58
*** CTtpollard has joined #baserock09:01
*** cphang has quit IRC09:05
*** jonathanmaw has joined #baserock09:09
*** locallycompact has joined #baserock09:17
*** cphang has joined #baserock09:31
*** tiagogomes has quit IRC09:35
*** benbrown_ has quit IRC09:39
*** benbrown_ has joined #baserock09:43
*** tiagogomes has joined #baserock09:48
*** jmacs has quit IRC09:56
*** jmac has joined #baserock09:59
*** jmac is now known as jmacs10:03
pedroalvarezcphang: btw, what version of definitions were you going to use to upgrade?10:17
pedroalvarez(or update)10:17
pedroalvarezfollowing the instructions in the wiki it says to `git clone git://git.baserock.org/baserock/baserock/definitions --branch baserock-16.13`10:18
cphangEhh the default, whatever is in git.baserock.org/baserock/baserock/definitions I didn't specify a specific branch10:19
pedroalvarezgreat10:19
pedroalvarezI was going to suggest using that (master branch)10:19
cphangSure. There were some small differences between the 'cut&paste' guide and the more descriptive one, but the 'cut&paste' one seemed to be more current so have defaulted to that where possible10:20
pedroalvarezthat's fine10:20
pedroalvarezwe have never been good at docs10:21
cphangNp :), I'm currently building&deploying with virtualbox to see if I have any differences in behaviour.10:21
pedroalvarezit should be the same behaviour10:24
cphangpedroalvarez: It should yeah. did you reproduce the errors that I had, or did your upgrade go okay?10:42
pedroalvarezhaven't had the chance yet, will let you know10:43
cphangOkay of course, many thanks. Yeh it's just failed using virtualbox. is it worth trying an older version of the definitions, maybe from around when the guide was made?10:47
pedroalvarezno no, master always :)10:48
pedroalvarezcphang: so, did you follow this to use latest morph? http://wiki.baserock.org/using-latest-morph/10:49
pedroalvarez(it should matter, but just looking to reproduce every detail)10:49
pedroalvarezshouldn't **10:49
cphangyep I did10:49
pedroalvarezcphang: good news, reproduced11:05
pedroalvarezbad news for everybody else :)11:06
pedroalvarez(or just me)11:06
cphangpedroalvarez: Eeep, well at least it's consistent! Let me know if there's anything else I can do :)12:21
cphangWait a minute, looking at the last line: I'm building using devel-systen-x86_64-generic.morph? The instructions use base-system-86_64-generic.morph?12:26
pedroalvarezhrm, they shouldn't ?12:42
cphangUnless there's another version? http://wiki.baserock.org/guides/no-frills/12:45
pedroalvarezso, what we are doing here, is using your current system to build a different one, and then switch to that one12:51
pedroalvarezyou can of course do that with a base system, but the goal here is that you have a devel system to do development12:52
cphangAah okay. Thanks for the clarification12:56
pedroalvarezI think I found the bug13:18
pedroalvarezhehe, I introduced it13:19
SotKwhat was it?13:19
pedroalvarezbeing v1  factory/orig and vu factory/run and v2 newsystem/orig13:20
pedroalvarezand vt newsystem/run13:20
pedroalvarezthis line http://git.baserock.org/cgit/baserock/baserock/tbdiff.git/tree/baserock-system-config-sync/baserock-system-config-sync?id=ca449ed8c6b02d63936e191b36bbef0ac5127699#n11113:20
pedroalvarezso, for something that exists in v1 and in vu but not in v2, we were then copying the file from v213:21
pedroalvarezgood job pedro13:21
* paulsherwood agrees... good job tracking this down13:22
cphangAaah hence the complaints about not being able to find the relevant files. Makes sense :)13:22
pedroalvarezheh :)13:23
SotKoops :)13:23
pedroalvarezcphang: indeed, the code maybe makes less sense13:23
pedroalvarezhm.. I'm still not sure about the fix, though13:25
pedroalvarezthe current logic in place (this is just for symlinks) is:13:25
pedroalvarezis the link in v1 and v2 the same? (pointing to the same place?) then use v213:25
pedroalvarezthis makes sense because if the user didn't modify it is because he was happy with the default config, and he will be happy with the new default config13:26
cphangWhat was the baserock build that the quick-start instructions were tested on? Could we use that, or has the merge() code changed substantially since then?13:27
pedroalvarezI think it's better to fix the error so that you can move forward13:29
cphangOf course.13:30
pedroalvarezi'm not sure about what the code should do in this case13:48
pedroalvarezjust not create any symlink in vt?13:48
pedroalvarezSotK: any opinion?13:48
pedroalvarezor maybe vu to be consistent with http://git.baserock.org/cgit/baserock/baserock/tbdiff.git/tree/baserock-system-config-sync/baserock-system-config-sync?id=ca449ed8c6b02d63936e191b36bbef0ac5127699#n14213:49
*** gtristan has quit IRC13:50
SotKI would say the latter to be consistent13:51
SotKthough, if "exists exists none" means that the user didn't change the symlink and the upgrade removes it, then maybe just not symlinking if v2 doesn't exist is best13:55
SotK(but still doing cp -a "$v2" "$vt" if v2 does exist)13:56
pedroalvarezheheh13:59
pedroalvarezthinking about upgrades is only possible if you think about specific cases13:59
pedroalvarezif you are upgrading a system from a version that has a systemd service active by default, and then the new version doesn' have that enabled by default..14:00
pedroalvarezmaybe there is a reason for that ..?14:00
pedroalvarezanyway, this would be the fix: https://gerrit.baserock.org/#/c/2301/14:13
pedroalvarezthanks benbrown_ :)14:19
pedroalvarezany other taker?14:31
pedroalvarezcphang: you can also have a look :)14:31
franredpedroalvarez, done14:34
cphangpedroalvarez: Will do, thanks :)14:35
pedroalvarezthanks :)14:35
tiagogomesv1, v2, vu, vt sound so 2013 …14:41
benbrown_tiagogomes: :Þ14:43
pedroalvarezaaaaand https://gerrit.baserock.org/#/c/2302/14:43
franredthese variable names ....14:43
franredtiagogomes, ^^14:43
pedroalvarezthanks all for reviewing! :)14:44
franredpedroalvarez, would be amazing if gerrit could link definitions and link them to ctgit14:44
franredg.b.o. I meant14:45
pedroalvarezwhat do you mean14:45
franredpedroalvarez, merged14:45
pedroalvarezthanks14:46
pedroalvarezyou mean like in here? http://git.baserock.org/cgit/baserock/baserock/definitions.git/tree/strata/foundation.morph14:46
franredpedroalvarez, yep14:46
pedroalvarezyeh, not sure how complex would be that14:46
pedroalvarezAnd the last patch of the day:14:51
pedroalvarez  https://gerrit.baserock.org/2303 Upgrade to OpenSSL 1.0.2i14:51
pedroalvarezthis one is a security update :)14:53
franredpedroalvarez, merged14:56
cphangpedroalvarez: Might be a silly question, but is it possible/worth it to turn the symlink logic part of merge into case statements like in merge_regular_file, just to emphasise the logic is consistent?14:56
pedroalvarezyeah, code could be improved in very different ways14:57
*** gtristan has joined #baserock15:08
pedroalvarezcphang: if you update your checkout of definitions now you will be able to build and upgrade the system15:15
pedroalvareznote: this is going to take a while, given that some artifacts are not cached yet15:15
cphangOkay great thanks :)15:16
pedroalvarezyw :)15:16
*** ctbruce has quit IRC16:12
*** jonathanmaw has quit IRC16:49
*** locallycompact has quit IRC17:00
*** tiagogomes has quit IRC17:22
*** toscalix has quit IRC18:29
*** trn has quit IRC20:16
*** gtristan has quit IRC20:20
*** faybrocklebank has quit IRC21:36
*** _longines has quit IRC21:37
*** leeming has quit IRC21:37
*** inara has quit IRC21:37
*** paulsherwood has quit IRC21:37
*** ChanServ has quit IRC21:37
*** SotK has quit IRC21:37
*** pedroalvarez has quit IRC21:37
*** richard_maw has quit IRC21:37
*** juergbi has quit IRC21:37
*** anahuelamo_ has quit IRC21:37
*** JPohlmann has quit IRC21:37
*** edcragg has quit IRC21:37
*** gary_perkins has quit IRC21:37
*** cphang has quit IRC21:37
*** CTtpollard has quit IRC21:37
*** franred has quit IRC21:37
*** zoli_ has quit IRC21:37
*** jjardon_matrix has quit IRC21:37
*** jjardon has quit IRC21:37
*** rdale has quit IRC21:37
*** vgrade has quit IRC21:37
*** jmacs has quit IRC21:37
*** dabukalam has quit IRC21:37
*** persia has quit IRC21:37
*** lachlanmackenzie has quit IRC21:37
*** persia_ has quit IRC21:38
*** benbrown_ has quit IRC21:38
*** cosm has quit IRC21:38
*** bjdooks_ has quit IRC21:38
*** wdutch has quit IRC21:38
*** tardyp has quit IRC21:38
*** cyndis has quit IRC21:38
*** perryl has quit IRC21:38
*** jmacs has joined #baserock21:39
*** cphang has joined #baserock21:39
*** CTtpollard has joined #baserock21:39
*** rdale has joined #baserock21:39
*** vgrade has joined #baserock21:39
*** lachlanmackenzie has joined #baserock21:39
*** dabukalam has joined #baserock21:39
*** franred has joined #baserock21:39
*** jjardon_matrix has joined #baserock21:39
*** anahuelamo_ has joined #baserock21:39
*** JPohlmann has joined #baserock21:39
*** zoli_ has joined #baserock21:39
*** paulsherwood has joined #baserock21:39
*** ChanServ has joined #baserock21:39
*** edcragg has joined #baserock21:39
*** jjardon has joined #baserock21:39
*** pedroalvarez has joined #baserock21:39
*** SotK has joined #baserock21:39
*** richard_maw has joined #baserock21:39
*** juergbi has joined #baserock21:39
*** gary_perkins has joined #baserock21:39
*** persia has joined #baserock21:39
*** persia_ has joined #baserock21:39
*** niven.freenode.net sets mode: +ovv ChanServ pedroalvarez richard_maw21:39
*** jjardon has quit IRC21:40
*** wdutch has joined #baserock21:40
*** tardyp has joined #baserock21:40
*** cyndis has joined #baserock21:40
*** perryl has joined #baserock21:40
*** faybrocklebank has joined #baserock21:41
*** _longines has joined #baserock21:41
*** leeming has joined #baserock21:41
*** inara has joined #baserock21:41
*** benbrown_ has joined #baserock21:42
*** cosm has joined #baserock21:42
*** bjdooks_ has joined #baserock21:42
*** trn has joined #baserock22:00
*** jjardon has joined #baserock22:00

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