| *** gtristan has joined #baserock | 06:41 | |
| *** toscalix has joined #baserock | 08:54 | |
| *** fay_ has joined #baserock | 08:59 | |
| *** locallycompact has joined #baserock | 09:14 | |
| *** ctbruce has joined #baserock | 09:40 | |
| *** jonathanmaw has joined #baserock | 09:45 | |
| *** ctbruce has quit IRC | 09:53 | |
| *** ssam2 has joined #baserock | 10:01 | |
| *** ChanServ sets mode: +v ssam2 | 10:01 | |
| *** franred has joined #baserock | 10:02 | |
| *** edcragg has joined #baserock | 10:12 | |
| franred | ssam2, https://gerrit.baserock.org/#/c/1737/ is the only patch of this series which stops for the series to be merged. Are you still against it after pedroalvarez explanations? AFAICR, this patch-series fixes the installation in jetson boards. | 10:16 |
|---|---|---|
| ssam2 | franred: i'm looking at it now | 10:17 |
| ssam2 | i'm still reluctant to merge code to the deployment process that i can't understand | 10:18 |
| ssam2 | am trying to add some comments to make it clear what's going on | 10:18 |
| ssam2 | pedro *believes* to understand it, and nobody else has commented at all about how they think it works, so i'm not confident it's maintainable as-is | 10:18 |
| pedroalvarez | yes, and I've never tested it | 10:19 |
| ssam2 | will try to submit a new version with more comments and docstrings | 10:19 |
| pedroalvarez | and I'd like to before it's merged | 10:19 |
| paulsherwood | radiofree: ^^ ? | 10:19 |
| radiofree | even i don't understand how it works | 10:27 |
| radiofree | to be serious, i'll comment on it later | 10:27 |
| radiofree | basically you maintain btrfs filesystem structure on the boot partition to help system-version-manager out | 10:27 |
| paulsherwood | lol | 10:28 |
| radiofree | system-version-manager expects kernel and dtb to be in /systems/version/kernel and /system/version/dtb | 10:28 |
| paulsherwood | if radiofree doesn't understand it, what hope is there for the rest of us? | 10:28 |
| radiofree | so the layout doesn't magically change, you're maintaining the layout on the boot partition | 10:28 |
| radiofree | ssam2: http://git.baserock.org/cgit/baserock/baserock/tbdiff.git/commit/?id=61f116bc5367e8ba26e8fd0f48aa71e644cb9b1e | 10:30 |
| radiofree | system-version-manager already supports this, the jetson flashing script does the same thing (though manually using cp....), this just allows you to build a single image you can dd directly | 10:30 |
| radiofree | tested on arm and x86 btw | 10:31 |
| radiofree | x86 was "normal" single partition image, deploy/upgrade still works, and deploy/upgrade to x86 partitioned system | 10:31 |
| * radiofree will respond in gerrit at some point | 10:33 | |
| radiofree | very busy atm | 10:33 |
| paulsherwood | radiofree: thanks a lot for the info | 10:33 |
| ssam2 | ok, ta. will try and sprinkle some comments and docstrings into the patch | 10:36 |
| *** fay_ is now known as faybrocklebank | 10:38 | |
| *** gtristan has quit IRC | 10:41 | |
| *** ctbruce has joined #baserock | 10:54 | |
| *** gtristan has joined #baserock | 11:09 | |
| pedroalvarez | wooo! genivi systems are being upgraded to weston1.9 ! | 11:12 |
| pedroalvarez | https://gerrit.baserock.org/#/q/status:open+project:baserock/baserock/definitions+branch:master+topic:baserock/pedroalvarez/genivi-weston-1.9-review | 11:12 |
| radiofree | excellent! | 11:25 |
| paulsherwood | +1 | 11:26 |
| radiofree | so have they ported the gdp-hmi-controller to use the new input api? | 11:26 |
| pedroalvarez | radiofree: jonathanmaw has already provided a version that works | 11:29 |
| pedroalvarez | I'll upgrade to the new gdp-hmi as soon as he thinks it's finished | 11:30 |
| radiofree | nice | 12:10 |
| paulsherwood | hmmm .... as of 227b22d91893417b1c3633a8372f63a4489a68d8 i'm seeing http://paste.baserock.org/xinobequce | 12:48 |
| paulsherwood | jjardon: ^^ | 12:49 |
| pedroalvarez | that's probably caused by 84d3b08d9caf6183a2d1b677c335b00123638a46 | 12:51 |
| pedroalvarez | I thought the patch was tested anyway.. | 12:52 |
| paulsherwood | well, i'm happy to be part of testing :) | 12:53 |
| paulsherwood | am hoping for pre-merge ci soon | 12:54 |
| pedroalvarez | could we add the word "fast" to your hopes? :) | 12:55 |
| * SotK wonders what the plan for pre-merge ci is these days | 12:55 | |
| * richard_maw assumes concourse | 12:55 | |
| jjardon | paulsherwood: sorry, feel free to revert it (this is one of the reason I wanted a way to build the gnome strata without python2 at all) | 12:56 |
| jjardon | paulsherwood: or wait to the afternoon and you will get a fix | 12:57 |
| jjardon | pedroalvarez: 1.9 is old already :) I will send patches to upgrade stock wayland/weston to 1.10 today | 12:59 |
| pedroalvarez | hahahahah | 13:00 |
| pedroalvarez | I don't think I will try to upgrade just now | 13:02 |
| pedroalvarez | I was planning to merge weston and weston-genivi somehow.. but it looks like weston will be always ahead | 13:02 |
| jjardon | yeah, there are some new dependencies as well: wayland-protocols and now libinput depend on libwacom | 13:03 |
| *** richard_maw has left #baserock | 13:03 | |
| jjardon | paulsherwood: if you upload you current cache you will get a fix sooner :) | 13:09 |
| paulsherwood | jjardon: i'll wait for fixes :) | 13:12 |
| paulsherwood | jjardon: i always do :) | 13:25 |
| paulsherwood | (upload cache) | 13:26 |
| radiofree | Can't we just merge Weston and weston-genivi now | 13:44 |
| *** gtristan has quit IRC | 13:44 | |
| pedroalvarez | radiofree: right now yes, but if we upgrade weston I don't know if genivi sytems will continue to work | 13:50 |
| paulsherwood | pedroalvarez: better to ask for forgiveness than wait for permission :) | 13:53 |
| pedroalvarez | oh well :) | 13:55 |
| radiofree | pedroalvarez: wayland-ivi-extension is pretty pro-upstream weston, i don't see it being a problem | 13:55 |
| radiofree | nicer to merge both and upgrade to 1.10 | 13:55 |
| pedroalvarez | I could test a patch for that today | 13:56 |
| *** gtristan has joined #baserock | 13:56 | |
| pedroalvarez | tomorrow I'll have less hardware for testing :( | 13:56 |
| * pedroalvarez continues investigating: | 13:58 | |
| pedroalvarez | File "/src/definitions/extensions/writeexts.py", line 475, in create_versioned_layout | 13:58 |
| pedroalvarez | version_label, os.path.join(mountpoint, 'systems', 'default')) | 13:58 |
| pedroalvarez | OSError: [Errno 1] Operation not permitted | 13:58 |
| pedroalvarez | os.symlink failing | 13:59 |
| pedroalvarez | context: testing extlinux-partition-support patch | 14:00 |
| *** faybrocklebank has quit IRC | 14:04 | |
| radiofree | bugger | 14:10 |
| radiofree | operation not permitted | 14:12 |
| pedroalvarez | I can't see the problem | 14:12 |
| radiofree | is this with morph? | 14:13 |
| pedroalvarez | yes | 14:13 |
| pedroalvarez | it shouldn't matter, i think | 14:13 |
| pedroalvarez | trying this partitioning: http://paste.baserock.org/imodojimoq | 14:14 |
| * radiofree only tested this with ybd, but it shouldnt matter? | 14:14 | |
| pedroalvarez | the code used for the deployments should be the same | 14:17 |
| *** faybrocklebank has joined #baserock | 14:19 | |
| pedroalvarez | I don't understand... version_root folder is created just before trying to create the default symlink | 14:21 |
| pedroalvarez | but creating the symlink fails | 14:22 |
| franred | pedroalvarez, permission problems? | 14:23 |
| *** brlogger has joined #baserock | 14:38 | |
| *** bwh has joined #baserock | 14:39 | |
| pedroalvarez | I don't understand what's going on... :/ | 14:48 |
| paulsherwood | pedroalvarez: welcome to my world :) | 14:55 |
| tiagogomes_ | can I reproduce it easily? | 14:57 |
| locallycompact | paulsherwood, please give me a clue here. | 15:06 |
| *** ctbruce has quit IRC | 15:06 | |
| locallycompact | https://github.com/locallycompact/ybd/commit/d1138c0bb706116159495024604fc636b1c48d9d | 15:06 |
| *** ctbruce has joined #baserock | 15:06 | |
| radiofree | does it work in ybd? | 15:06 |
| locallycompact | gives back | 15:06 |
| radiofree | pedroalvarez: ^ | 15:06 |
| locallycompact | https://paste.fedoraproject.org/329249/56412811/ | 15:07 |
| radiofree | paulsherwood: can you try it with ybd? I don't have an x86 vm to hand | 15:07 |
| paulsherwood | locallycompact: about what? | 15:07 |
| paulsherwood | radiofree: what am i trying precisely? does it need btrfs? | 15:07 |
| locallycompact | where the side effect is | 15:08 |
| pedroalvarez | tiagogomes_: branch baserock/pedroalvarez/testing-partitioning | 15:08 |
| radiofree | was that for paulsherwood? | 15:09 |
| pedroalvarez | a base system should be fast to build using cache | 15:09 |
| radiofree | oh | 15:09 |
| radiofree | vfat | 15:09 |
| radiofree | there's your issue | 15:09 |
| pedroalvarez | radiofree: haven't tested in ybd | 15:09 |
| radiofree | why vfat? | 15:09 |
| pedroalvarez | hhh | 15:09 |
| pedroalvarez | arhghasdflakshdflkajsghdaeg | 15:09 |
| pedroalvarez | why not!?!?! | 15:10 |
| paulsherwood | locallycompact: is str(path) the same as path[2:] ? what about leading / ? | 15:10 |
| * pedroalvarez <- facepalms | 15:10 | |
| pedroalvarez | radiofree: I copied-pasted from another example, missed that bit | 15:10 |
| locallycompact | os.walk gives back paths prepended with ./. fs.walkfiles gives back unicode paths without, that's why the 2: is not there and the str() is | 15:11 |
| paulsherwood | ok | 15:11 |
| paulsherwood | locallycompact: i don't like the raise, by the way | 15:12 |
| * paulsherwood continues to think | 15:12 | |
| locallycompact | It was hiding errors from me | 15:13 |
| paulsherwood | ok | 15:13 |
| paulsherwood | why not sort() ? | 15:13 |
| locallycompact | no reason | 15:15 |
| locallycompact | it doesn't quite have the same meaning on that generator though | 15:15 |
| paulsherwood | i'd be surprised if there wasn't a reason for it originally | 15:15 |
| paulsherwood | jjardon: still getting the zeitgeist error | 15:16 |
| *** tiagogomes___ has joined #baserock | 15:25 | |
| *** Zara_ has joined #baserock | 15:26 | |
| *** tiagogomes_ has quit IRC | 15:26 | |
| *** Zara has quit IRC | 15:27 | |
| *** gary_perkins has quit IRC | 15:27 | |
| *** jonathanmaw has quit IRC | 15:27 | |
| *** ssam2 has quit IRC | 15:27 | |
| *** jonathanmaw has joined #baserock | 15:27 | |
| *** gary_perkins has joined #baserock | 15:27 | |
| *** ssam2 has joined #baserock | 15:27 | |
| *** ChanServ sets mode: +v ssam2 | 15:27 | |
| *** Zara_ is now known as Zara | 15:30 | |
| *** richard_maw has joined #baserock | 15:38 | |
| *** vgrade has joined #baserock | 15:43 | |
| jjardon | https://www.irccloud.com/pastebin/eJbtq8qI/ | 15:44 |
| jjardon | paulsherwood: ^ sounds familiar? | 15:44 |
| pedroalvarez | surprise! | 15:45 |
| jjardon | :) | 15:45 |
| rdale | it looks like the docutils stratum is called 'docutils.morph' | 15:45 |
| paulsherwood | eek | 15:46 |
| jjardon | paulsherwood: it fails only in master, latest tag builds | 15:46 |
| * paulsherwood blames locallycompact, then :) | 15:47 | |
| paulsherwood | jjardon: you mean master ybd, or definitions? | 15:47 |
| jjardon | paulsherwood: ybd | 15:47 |
| jjardon | (Im using master of definitions as well) | 15:47 |
| radiofree | pedroalvarez: it would be possible to support vfat i guess | 15:50 |
| paulsherwood | jjardon: ack, please stick with tag | 15:50 |
| radiofree | you'd have to copy the kernel and dtb out of "factory" into a folder called default, rather than symlink | 15:51 |
| pedroalvarez | radiofree: to be honest I'm not worried about that right now | 15:51 |
| radiofree | and then make sure system-version-manager did that (instead of updating the symlink, copy out the kernel and dtb from VERSION) | 15:51 |
| * radiofree also doesn't care about supporting fat32 | 15:51 | |
| pedroalvarez | maybe just raising an error there saying "symlinks in fat? u crazy?" is enough | 15:52 |
| radiofree | it's kind of pointless without that mechanism though | 15:53 |
| radiofree | the whole point of this (in my opinion) is to allow upgrades to systems where the bootloader can't read from btrfs | 15:53 |
| pedroalvarez | but can read from ext | 15:53 |
| radiofree | or fat | 15:54 |
| radiofree | if we're not supporting fat, we should error if /boot is fat | 15:54 |
| radiofree | the only benefit of using fat32, that i can think of, is that you could mount the boot partition on a windows laptop and fix things (and/or deploy testing kernels manually)... but i don't care about that (i can mount ext fine on linux) | 15:56 |
| jjardon | paulsherwood: My understanding is that this should not be needed because transitive dependencies but, can you try this? | 15:56 |
| jjardon | https://www.irccloud.com/pastebin/dIU5h5tL/ | 15:56 |
| paulsherwood | trying | 15:58 |
| *** ctbruce has quit IRC | 15:58 | |
| *** ctbruce has joined #baserock | 15:59 | |
| pedroalvarez | partitoned image upgrade failed, and i don't have logs | 16:01 |
| pedroalvarez | and no brain left after debugging why symlinks didn't work on fat... | 16:02 |
| pedroalvarez | brain is back, base system doesn't have python 2, system-version-manager fails | 16:03 |
| edcragg | radiofree: hmm, i agree, except i have seen one case where the primary bootloader expects a fat partition to read uboot and kernel from | 16:03 |
| radiofree | damn you whoever removed python2 from base system! | 16:05 |
| richard_maw | but python3 is the future!!11 | 16:06 |
| *** richard_maw has left #baserock | 16:06 | |
| CTtpollard | I have a feeling the minnowboard expects boot to be fat32 | 16:07 |
| radiofree | ok so it sounds like fat32 support is needed | 16:07 |
| rjek | is it EFI? | 16:09 |
| *** richard_maw has joined #baserock | 16:10 | |
| CTtpollard | I'm just double checking now | 16:10 |
| ssam2 | it should be simple to make system-version-manager work with either python | 16:11 |
| richard_maw | hahaahahahahahah! | 16:11 |
| ssam2 | are you laughing as a way of motivating me to do it? it's really not so hard for simple programs | 16:12 |
| ssam2 | system-version-manager doesn't depend on cliapp or any such thing | 16:12 |
| pedroalvarez | It's my fault for writting that code.. | 16:12 |
| richard_maw | ssam2: I spent a couple of hours trying before I had to get on with project work, our assumption that strings are sequences of bytes (and they are, since we only really care about file paths) is pretty ingrained | 16:13 |
| ssam2 | cool, do you have a branch somewhere? | 16:13 |
| * pedroalvarez upgrades again, hoping that the upgrade uses system-version-manager from the new system | 16:13 | |
| richard_maw | no, it was coding by proxy with radiofree | 16:13 |
| richard_maw | changing our strings to be b'foo' got much of the way there, but the last bit before I gave up was mkdtemp, which I have since learned how to make it give a proper bytes string | 16:14 |
| radiofree | there were multiple other failures depending on what you were using in system-version-manager | 16:15 |
| radiofree | e.g "list" or "set-default" etc.. | 16:15 |
| richard_maw | td = tempfile.mkdtemp(dir=tempfile.gettempdirb()) | 16:17 |
| richard_maw | note: gettempdirb() is a comparatively modern invention (only python 3.5!!!1) | 16:18 |
| CTtpollard | radiofree, rjek: getting conflicting info, but from my local image I have here boot is fat16 EFI | 16:21 |
| CTtpollard | I've seen OpenWRT images for it use fat32 though | 16:22 |
| CTtpollard | so yeh, variants of vfat for minnowboard boot partitions | 16:25 |
| ssam2 | actually, if updating system-version-manager to work with Python3 is non-trivial, rewriting it in C might be a better use of effort | 16:27 |
| ssam2 | the Python implementation was never intended to be permanent | 16:27 |
| radiofree | what is the bootloader on the minnowboard? | 16:28 |
| CTtpollard | radiofree: uefi & grub | 16:30 |
| pedroalvarez | no luck with python2 either | 16:38 |
| pedroalvarez | and I'm really tired of testing this patch :/ | 16:38 |
| *** ctbruce has quit IRC | 16:51 | |
| * ssam2 creates https://storyboard.baserock.org/#!/story/78 | 16:51 | |
| ssam2 | for rewriting system-version-manager in C | 16:51 |
| ssam2 | rdale: the more I use json-ld the more I like it | 17:14 |
| rdale | have you been using it then? | 17:14 |
| ssam2 | I thought this blog post by the author was very good: http://manu.sporny.org/2014/json-ld-origins-2/ | 17:14 |
| ssam2 | yes, although not directly for Baserock stuff | 17:15 |
| rdale | right, he does excellent videos | 17:15 |
| rdale | your owl grammar converts to json-ld with the rdflib json-ld plugin | 17:15 |
| ssam2 | i've been writing json-ld documents as yaml though :-) | 17:16 |
| ssam2 | which isn't ideal because you have to quote keys that start with @ | 17:16 |
| ssam2 | but it's more ideal than writing JSON and having to remember where I should and shouldn't put a comma | 17:16 |
| pedroalvarez | argh! BOOT_DEVICE wasn't set! | 17:16 |
| rdale | you can alias keywords with '@' | 17:16 |
| ssam2 | using YAML aliases? is that easier than quoting them? | 17:17 |
| ssam2 | pedroalvarez: sounds like there's another warning missing ! | 17:17 |
| rdale | json-ld alias's in the context | 17:17 |
| ssam2 | oh, right, that's cool | 17:17 |
| ssam2 | every time I try and do something I end up thinking "oh, cool, they thought of that" | 17:18 |
| ssam2 | and I especially like: "Many W3C specifications are so cryptic that they require the sacrifice of your sanity and a secret W3C decoder ring to read. I never understood why these documents were so difficult to read, and after years of study on the matter, I think I found the answer. It turns out that most specification editors are just crap at writing. " | 17:18 |
| rdale | yes, it's simple but allows you to do complex things which is difficult to design | 17:20 |
| pedroalvarez | I think, if BOOT_DEVICE is not set, used to not fail | 17:21 |
| pedroalvarez | but yes, this needs a warning or something | 17:21 |
| pedroalvarez | but, yay! patches work | 17:23 |
| franred | pedroalvarez, :) | 17:25 |
| *** locallycompact has quit IRC | 17:26 | |
| *** franred has quit IRC | 17:26 | |
| *** locallycompact has joined #baserock | 17:29 | |
| *** locallycompact has quit IRC | 17:35 | |
| * pedroalvarez merges things | 17:43 | |
| *** jonathanmaw has quit IRC | 17:48 | |
| pedroalvarez | jjardon: thanks for reviewing https://gerrit.baserock.org/#/c/1806/ | 17:52 |
| pedroalvarez | it raises an error now | 17:52 |
| *** CTtpollard has quit IRC | 17:56 | |
| *** edcragg has quit IRC | 18:15 | |
| *** rdale has quit IRC | 18:29 | |
| *** ssam2 has quit IRC | 18:29 | |
| *** toscalix has quit IRC | 19:11 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!