| *** gtristan has joined #baserock | 05:46 | |
| *** fay has joined #baserock | 06:48 | |
| *** fay is now known as faybrocklebank | 06:48 | |
| *** rdale has joined #baserock | 07:21 | |
| *** ctbruce has joined #baserock | 07:32 | |
| *** toscalix has joined #baserock | 08:00 | |
| *** tiagogomes has joined #baserock | 08:57 | |
| *** CTtpollard has quit IRC | 08:58 | |
| *** CTtpollard has joined #baserock | 09:01 | |
| *** cphang has quit IRC | 09:05 | |
| *** jonathanmaw has joined #baserock | 09:09 | |
| *** locallycompact has joined #baserock | 09:17 | |
| *** cphang has joined #baserock | 09:31 | |
| *** tiagogomes has quit IRC | 09:35 | |
| *** benbrown_ has quit IRC | 09:39 | |
| *** benbrown_ has joined #baserock | 09:43 | |
| *** tiagogomes has joined #baserock | 09:48 | |
| *** jmacs has quit IRC | 09:56 | |
| *** jmac has joined #baserock | 09:59 | |
| *** jmac is now known as jmacs | 10:03 | |
| pedroalvarez | cphang: btw, what version of definitions were you going to use to upgrade? | 10:17 |
|---|---|---|
| pedroalvarez | (or update) | 10:17 |
| pedroalvarez | following the instructions in the wiki it says to `git clone git://git.baserock.org/baserock/baserock/definitions --branch baserock-16.13` | 10:18 |
| cphang | Ehh the default, whatever is in git.baserock.org/baserock/baserock/definitions I didn't specify a specific branch | 10:19 |
| pedroalvarez | great | 10:19 |
| pedroalvarez | I was going to suggest using that (master branch) | 10:19 |
| cphang | Sure. 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 possible | 10:20 |
| pedroalvarez | that's fine | 10:20 |
| pedroalvarez | we have never been good at docs | 10:21 |
| cphang | Np :), I'm currently building&deploying with virtualbox to see if I have any differences in behaviour. | 10:21 |
| pedroalvarez | it should be the same behaviour | 10:24 |
| cphang | pedroalvarez: It should yeah. did you reproduce the errors that I had, or did your upgrade go okay? | 10:42 |
| pedroalvarez | haven't had the chance yet, will let you know | 10:43 |
| cphang | Okay 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 |
| pedroalvarez | no no, master always :) | 10:48 |
| pedroalvarez | cphang: 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 |
| pedroalvarez | shouldn't ** | 10:49 |
| cphang | yep I did | 10:49 |
| pedroalvarez | cphang: good news, reproduced | 11:05 |
| pedroalvarez | bad news for everybody else :) | 11:06 |
| pedroalvarez | (or just me) | 11:06 |
| cphang | pedroalvarez: Eeep, well at least it's consistent! Let me know if there's anything else I can do :) | 12:21 |
| cphang | Wait 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 |
| pedroalvarez | hrm, they shouldn't ? | 12:42 |
| cphang | Unless there's another version? http://wiki.baserock.org/guides/no-frills/ | 12:45 |
| pedroalvarez | so, what we are doing here, is using your current system to build a different one, and then switch to that one | 12:51 |
| pedroalvarez | you can of course do that with a base system, but the goal here is that you have a devel system to do development | 12:52 |
| cphang | Aah okay. Thanks for the clarification | 12:56 |
| pedroalvarez | I think I found the bug | 13:18 |
| pedroalvarez | hehe, I introduced it | 13:19 |
| SotK | what was it? | 13:19 |
| pedroalvarez | being v1 factory/orig and vu factory/run and v2 newsystem/orig | 13:20 |
| pedroalvarez | and vt newsystem/run | 13:20 |
| pedroalvarez | this line http://git.baserock.org/cgit/baserock/baserock/tbdiff.git/tree/baserock-system-config-sync/baserock-system-config-sync?id=ca449ed8c6b02d63936e191b36bbef0ac5127699#n111 | 13:20 |
| pedroalvarez | so, for something that exists in v1 and in vu but not in v2, we were then copying the file from v2 | 13:21 |
| pedroalvarez | good job pedro | 13:21 |
| * paulsherwood agrees... good job tracking this down | 13:22 | |
| cphang | Aaah hence the complaints about not being able to find the relevant files. Makes sense :) | 13:22 |
| pedroalvarez | heh :) | 13:23 |
| SotK | oops :) | 13:23 |
| pedroalvarez | cphang: indeed, the code maybe makes less sense | 13:23 |
| pedroalvarez | hm.. I'm still not sure about the fix, though | 13:25 |
| pedroalvarez | the current logic in place (this is just for symlinks) is: | 13:25 |
| pedroalvarez | is the link in v1 and v2 the same? (pointing to the same place?) then use v2 | 13:25 |
| pedroalvarez | this 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 config | 13:26 |
| cphang | What 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 |
| pedroalvarez | I think it's better to fix the error so that you can move forward | 13:29 |
| cphang | Of course. | 13:30 |
| pedroalvarez | i'm not sure about what the code should do in this case | 13:48 |
| pedroalvarez | just not create any symlink in vt? | 13:48 |
| pedroalvarez | SotK: any opinion? | 13:48 |
| pedroalvarez | or 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#n142 | 13:49 |
| *** gtristan has quit IRC | 13:50 | |
| SotK | I would say the latter to be consistent | 13:51 |
| SotK | though, 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 best | 13:55 |
| SotK | (but still doing cp -a "$v2" "$vt" if v2 does exist) | 13:56 |
| pedroalvarez | heheh | 13:59 |
| pedroalvarez | thinking about upgrades is only possible if you think about specific cases | 13:59 |
| pedroalvarez | if 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 |
| pedroalvarez | maybe there is a reason for that ..? | 14:00 |
| pedroalvarez | anyway, this would be the fix: https://gerrit.baserock.org/#/c/2301/ | 14:13 |
| pedroalvarez | thanks benbrown_ :) | 14:19 |
| pedroalvarez | any other taker? | 14:31 |
| pedroalvarez | cphang: you can also have a look :) | 14:31 |
| franred | pedroalvarez, done | 14:34 |
| cphang | pedroalvarez: Will do, thanks :) | 14:35 |
| pedroalvarez | thanks :) | 14:35 |
| tiagogomes | v1, v2, vu, vt sound so 2013 … | 14:41 |
| benbrown_ | tiagogomes: :Þ | 14:43 |
| pedroalvarez | aaaaand https://gerrit.baserock.org/#/c/2302/ | 14:43 |
| franred | these variable names .... | 14:43 |
| franred | tiagogomes, ^^ | 14:43 |
| pedroalvarez | thanks all for reviewing! :) | 14:44 |
| franred | pedroalvarez, would be amazing if gerrit could link definitions and link them to ctgit | 14:44 |
| franred | g.b.o. I meant | 14:45 |
| pedroalvarez | what do you mean | 14:45 |
| franred | pedroalvarez, merged | 14:45 |
| pedroalvarez | thanks | 14:46 |
| pedroalvarez | you mean like in here? http://git.baserock.org/cgit/baserock/baserock/definitions.git/tree/strata/foundation.morph | 14:46 |
| franred | pedroalvarez, yep | 14:46 |
| pedroalvarez | yeh, not sure how complex would be that | 14:46 |
| pedroalvarez | And the last patch of the day: | 14:51 |
| pedroalvarez | https://gerrit.baserock.org/2303 Upgrade to OpenSSL 1.0.2i | 14:51 |
| pedroalvarez | this one is a security update :) | 14:53 |
| franred | pedroalvarez, merged | 14:56 |
| cphang | pedroalvarez: 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 |
| pedroalvarez | yeah, code could be improved in very different ways | 14:57 |
| *** gtristan has joined #baserock | 15:08 | |
| pedroalvarez | cphang: if you update your checkout of definitions now you will be able to build and upgrade the system | 15:15 |
| pedroalvarez | note: this is going to take a while, given that some artifacts are not cached yet | 15:15 |
| cphang | Okay great thanks :) | 15:16 |
| pedroalvarez | yw :) | 15:16 |
| *** ctbruce has quit IRC | 16:12 | |
| *** jonathanmaw has quit IRC | 16:49 | |
| *** locallycompact has quit IRC | 17:00 | |
| *** tiagogomes has quit IRC | 17:22 | |
| *** toscalix has quit IRC | 18:29 | |
| *** trn has quit IRC | 20:16 | |
| *** gtristan has quit IRC | 20:20 | |
| *** faybrocklebank has quit IRC | 21:36 | |
| *** _longines has quit IRC | 21:37 | |
| *** leeming has quit IRC | 21:37 | |
| *** inara has quit IRC | 21:37 | |
| *** paulsherwood has quit IRC | 21:37 | |
| *** ChanServ has quit IRC | 21:37 | |
| *** SotK has quit IRC | 21:37 | |
| *** pedroalvarez has quit IRC | 21:37 | |
| *** richard_maw has quit IRC | 21:37 | |
| *** juergbi has quit IRC | 21:37 | |
| *** anahuelamo_ has quit IRC | 21:37 | |
| *** JPohlmann has quit IRC | 21:37 | |
| *** edcragg has quit IRC | 21:37 | |
| *** gary_perkins has quit IRC | 21:37 | |
| *** cphang has quit IRC | 21:37 | |
| *** CTtpollard has quit IRC | 21:37 | |
| *** franred has quit IRC | 21:37 | |
| *** zoli_ has quit IRC | 21:37 | |
| *** jjardon_matrix has quit IRC | 21:37 | |
| *** jjardon has quit IRC | 21:37 | |
| *** rdale has quit IRC | 21:37 | |
| *** vgrade has quit IRC | 21:37 | |
| *** jmacs has quit IRC | 21:37 | |
| *** dabukalam has quit IRC | 21:37 | |
| *** persia has quit IRC | 21:37 | |
| *** lachlanmackenzie has quit IRC | 21:37 | |
| *** persia_ has quit IRC | 21:38 | |
| *** benbrown_ has quit IRC | 21:38 | |
| *** cosm has quit IRC | 21:38 | |
| *** bjdooks_ has quit IRC | 21:38 | |
| *** wdutch has quit IRC | 21:38 | |
| *** tardyp has quit IRC | 21:38 | |
| *** cyndis has quit IRC | 21:38 | |
| *** perryl has quit IRC | 21:38 | |
| *** jmacs has joined #baserock | 21:39 | |
| *** cphang has joined #baserock | 21:39 | |
| *** CTtpollard has joined #baserock | 21:39 | |
| *** rdale has joined #baserock | 21:39 | |
| *** vgrade has joined #baserock | 21:39 | |
| *** lachlanmackenzie has joined #baserock | 21:39 | |
| *** dabukalam has joined #baserock | 21:39 | |
| *** franred has joined #baserock | 21:39 | |
| *** jjardon_matrix has joined #baserock | 21:39 | |
| *** anahuelamo_ has joined #baserock | 21:39 | |
| *** JPohlmann has joined #baserock | 21:39 | |
| *** zoli_ has joined #baserock | 21:39 | |
| *** paulsherwood has joined #baserock | 21:39 | |
| *** ChanServ has joined #baserock | 21:39 | |
| *** edcragg has joined #baserock | 21:39 | |
| *** jjardon has joined #baserock | 21:39 | |
| *** pedroalvarez has joined #baserock | 21:39 | |
| *** SotK has joined #baserock | 21:39 | |
| *** richard_maw has joined #baserock | 21:39 | |
| *** juergbi has joined #baserock | 21:39 | |
| *** gary_perkins has joined #baserock | 21:39 | |
| *** persia has joined #baserock | 21:39 | |
| *** persia_ has joined #baserock | 21:39 | |
| *** niven.freenode.net sets mode: +ovv ChanServ pedroalvarez richard_maw | 21:39 | |
| *** jjardon has quit IRC | 21:40 | |
| *** wdutch has joined #baserock | 21:40 | |
| *** tardyp has joined #baserock | 21:40 | |
| *** cyndis has joined #baserock | 21:40 | |
| *** perryl has joined #baserock | 21:40 | |
| *** faybrocklebank has joined #baserock | 21:41 | |
| *** _longines has joined #baserock | 21:41 | |
| *** leeming has joined #baserock | 21:41 | |
| *** inara has joined #baserock | 21:41 | |
| *** benbrown_ has joined #baserock | 21:42 | |
| *** cosm has joined #baserock | 21:42 | |
| *** bjdooks_ has joined #baserock | 21:42 | |
| *** trn has joined #baserock | 22:00 | |
| *** jjardon has joined #baserock | 22:00 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!