| *** xjuan has quit IRC | 00:45 | |
| *** tristan has joined #buildstream | 05:25 | |
| paulsher1ood | robjh: or borrow a moonshot sled, or use packet.net or other arm cloud | 06:46 |
|---|---|---|
| paulsher1ood | the pi route will likely be quite slow to build | 06:47 |
| paulsher1ood | in case it's not obvious, the reason bst projects are normally native (not cross) is that it avoids having to carry differences in impelementation higher up the stack. normally all arch-specific or device-specific patches can stay in the 'bsp' | 06:48 |
| *** iker has joined #buildstream | 07:07 | |
| *** tristan has quit IRC | 07:16 | |
| *** toscalix has joined #buildstream | 07:49 | |
| qinusty | https://xkcd.com/2044/ slight relevance | 08:01 |
| *** tristan has joined #buildstream | 08:11 | |
| *** bochecha has joined #buildstream | 08:25 | |
| *** rdale has joined #buildstream | 08:30 | |
| *** WSalmon_ has joined #buildstream | 09:02 | |
| *** toscalix_ has joined #buildstream | 09:03 | |
| *** WSalmon has quit IRC | 09:03 | |
| *** jonathanmaw has joined #buildstream | 09:04 | |
| *** CTtpollard has joined #buildstream | 09:04 | |
| *** tpollard has quit IRC | 09:05 | |
| *** abderrahim has quit IRC | 09:12 | |
| *** abderrahim has joined #buildstream | 09:13 | |
| tlater[m] | Oh, we have pytest-timeout now? | 09:22 |
| tlater[m] | Nice, that should help debugging :) | 09:22 |
| gitlab-br-bot | buildstream: merge request (Qinusty/skipped-rework->master: Add SkipError for indicating a skipped activity) #765 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/765 | 09:26 |
| *** lachlan has joined #buildstream | 09:27 | |
| *** toscalix_ has quit IRC | 09:36 | |
| tlater[m] | Would someone mind giving https://gitlab.com/BuildStream/buildstream-docker-images/merge_requests/53 a quick review? Sam was nice enough to do so already, so it's close to merging. | 09:37 |
| *** lachlan has quit IRC | 09:41 | |
| *** lachlan has joined #buildstream | 09:52 | |
| *** toscalix_ has joined #buildstream | 09:53 | |
| tlater[m] | jjardon: I see you recently updated some of the docker image dependencies | 09:56 |
| tlater[m] | Is there any reason you added bubblewrap and fuse to the artifact server dependencies? To my knowledge we need neither for just a cache | 09:57 |
| gitlab-br-bot | buildstream: merge request (Qinusty/skipped-rework->master: Add SkipError for indicating a skipped activity) #765 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/765 | 10:02 |
| Nexus | tlater[m]: i'll have a scan | 10:03 |
| qinusty | Can I get a review of https://gitlab.com/BuildStream/buildstream/merge_requests/765? This is up for a backport consideration also. (1.2.1 maybe?) | 10:04 |
| tlater[m] | Thanks Nexus :) | 10:04 |
| qinusty | During the work I came across a comment I've noted here https://gitlab.com/BuildStream/buildstream/merge_requests/765#note_100063057. Perhaps someone knows something regarding the origin of this comment? | 10:04 |
| tiagogomes | qinusty git blame? | 10:05 |
| * Nexus always recommends resolvinf git issues using `git push <remote_name> :<branch_name>` | 10:06 | |
| qinusty | Indeed. Looking at the commit which the comment was added. It's most likely an interpretation issue. But perhaps it needs clarifying. | 10:06 |
| Nexus | *results may vary* | 10:06 |
| *** lachlan has quit IRC | 10:09 | |
| *** lachlan has joined #buildstream | 10:14 | |
| jjardon | tlater[m]: mmm, probably an overuse of sed? I should not have touched the artifact-server folder. Let me check | 10:15 |
| *** toscalix_ has quit IRC | 10:19 | |
| *** lachlan has quit IRC | 10:20 | |
| *** lachlan has joined #buildstream | 10:21 | |
| qinusty | tristan, re: accessing protected members outside of class scope. Is this not checked in pylint? | 10:29 |
| tristan | qinusty, It's not; because we currently cannot follow the standard to the letter | 10:35 |
| qinusty | Ah okay, so we /could/ enable it | 10:36 |
| tristan | We have functions that are API private (not a part of stable public facing API), but need to be called by BuildStream internals | 10:36 |
| tristan | My last email the list illustrates a possible avenue to allow such linting, at the bottom of the email | 10:36 |
| tristan | I actually considered having ElementController, SourceController, etc... at the beginning | 10:37 |
| tristan | But thought it was just overkill | 10:37 |
| qinusty | Seems like a reasonable idea. We only really have those two public API surfaces | 10:37 |
| jjardon | tlater[m]: looking at the commit, I left there what are the current hard dependecies of buildstream. If the artifact cache have different dependencies maybe we should document those somewhere (and then use the artifact-cache installation commands to create the docker image)? | 10:40 |
| tlater[m] | jjardon: We have this page: https://buildstream.gitlab.io/buildstream/install_artifacts.html | 10:41 |
| tristan | qinusty, we have Sandbox also, which is not publicly subclassable but has the same problems | 10:41 |
| tlater[m] | It does in fact not list the dependencies | 10:41 |
| tlater[m] | I'll write an issue about this, seeing as I'm trying to debug something else right now. I do think the dependencies are different, but it seems that we've neglected to document this. | 10:42 |
| jjardon | tlater[m]: seems we already use "BST_ARTIFACTS_ONLY=1 pip3 install ." to create the docker image, so that is done | 10:42 |
| jjardon | tlater[m]: yeah, let's open an issue so we do not forget | 10:42 |
| *** tristan has quit IRC | 10:45 | |
| *** lachlan has quit IRC | 10:46 | |
| *** lachlan has joined #buildstream | 10:48 | |
| gitlab-br-bot | buildstream: issue #648 ("Dependencies for a pure artifact server are undocumented") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/648 | 10:49 |
| *** lachlan has quit IRC | 11:01 | |
| *** toscalix_ has joined #buildstream | 11:01 | |
| *** lachlan has joined #buildstream | 11:05 | |
| tlater[m] | qinusty: Looks like you're the person to talk to about website contents in absence of toscalix_ - is there an overview of topics that we want to be covered? | 11:10 |
| qinusty | The feature page lists features which are to be included. Some are not detailed. tlater[m], | 11:12 |
| qinusty | If you can elaborate on any of these, feel free. | 11:13 |
| *** lachlan has quit IRC | 11:13 | |
| tiagogomes | tlater[m] I think there is an issue created for every page which describes the content he wanted there | 11:19 |
| tlater[m] | I've seen the issues, they're just a bit hard to navigate | 11:20 |
| tlater[m] | Lots of empty ticks, which is scary, heh | 11:20 |
| tlater[m] | But "features page" is a good pointer, thanks qinusty | 11:20 |
| qinusty | Indeed, I think toscalix is around | 11:21 |
| *** jonathanmaw_ has joined #buildstream | 11:21 | |
| *** jonathanmaw has quit IRC | 11:21 | |
| toscalix_ | tlater[m]: was talking to qinusty about it | 11:30 |
| toscalix_ | he will talk to you | 11:31 |
| toscalix_ | about the contents left t be done: feature page and known issues page | 11:31 |
| *** lachlan has joined #buildstream | 11:31 | |
| toscalix_ | buildstream in detail can wait | 11:31 |
| toscalix_ | we will do it the coming days | 11:31 |
| gitlab-br-bot | buildstream: issue #649 ("Artifact cache file can get stale") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/649 | 11:32 |
| gitlab-br-bot | buildstream: merge request (tiagogomes/issue-573->master: Reduce IO overhead caused by artifact cache size calculation) #671 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/671 | 11:36 |
| adds68 | When you bst shell on a failed element, does that checkout the log files that were generated when it failed? | 11:39 |
| tlater[m] | Do we have a spell check run planned for the website as well? | 11:40 |
| tlater[m] | adds68: Unless this has changed, log files are simply kept in .cache and never touched by buildstream. | 11:40 |
| adds68 | tlater[m], so when you enter the shell on a failed build that is a clean sandbox? | 11:41 |
| adds68 | tlater[m], as for example i can see a config.log, which has errors, so i'm wondering if these errors are what have been generated when bst has tried to build it | 11:42 |
| tlater[m] | adds68: Ah, no, that is not a clean sandbox | 11:42 |
| tlater[m] | I thought you meant the log files buildstream generates - the log files generated by the build tool will be in the sandbox | 11:43 |
| tlater[m] | You see the dirty, failed build in that sandbox. | 11:43 |
| adds68 | tlater[m], ta :) | 11:43 |
| tlater[m] | qinusty: I assume you only want detail to be added to the little features table on the features page. | 11:52 |
| *** toscalix_ has quit IRC | 11:58 | |
| *** alatiera_ has joined #buildstream | 12:13 | |
| tlater[m] | qinusty: Is this sort of what you were looking for? https://gitlab.com/BuildStream/website/commit/ab0b9d50340ef0691a71d9c110b39847bd955b1d | 12:29 |
| gitlab-br-bot | buildstream: merge request (tiagogomes/issue-573->master: Reduce IO overhead caused by artifact cache size calculation) #671 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/671 | 12:30 |
| tlater[m] | juergbi: Sorry to prod, but do you intend to write the section on CAS on that page? | 12:42 |
| * tlater[m] is quite happy to take on writing that if you're too busy this week | 12:46 | |
| juergbi | tlater[m]: I would certainly not mind passing this on to you | 12:48 |
| tlater[m] | juergbi: I'll get you to review, then :) | 12:49 |
| juergbi | thanks a lot | 12:49 |
| *** rdale has quit IRC | 12:53 | |
| gitlab-br-bot | buildstream: merge request (Qinusty/skipped-rework->master: Add SkipError for indicating a skipped activity) #765 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/765 | 12:54 |
| gitlab-br-bot | buildstream: merge request (Qinusty/skipped-rework->master: Add SkipError for indicating a skipped activity) #765 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/765 | 12:54 |
| gitlab-br-bot | buildstream: merge request (Qinusty/skipped-rework->master: Add SkipError for indicating a skipped activity) #765 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/765 | 12:54 |
| qinusty | Apologies for spam, gitlab autoclose issue detection is not quite as predictable as I had hoped... And it doesn't show in preview | 12:55 |
| gitlab-br-bot | buildstream: merge request (richardmaw/builddir-sockets->master: Fix: While caching build artifact: "Cannot extract [path to socket file] into staging-area. Unsupported type.") #783 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/783 | 12:57 |
| gitlab-br-bot | buildstream: merge request (tiagogomes/issue-573->master: Reduce IO overhead caused by artifact cache size calculation) #671 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/671 | 12:58 |
| robjh | the other i managed to build a kernel using the tlater/image-authoring branch. today im trying to replicate the success on aarch64 using a moonshot sled. But it fails with an error suggesting that its trying to run a shell binary for x86; | 13:20 |
| robjh | sed -i 's|#!/bin/bash|#!/bin/ash|' scripts/config | 13:20 |
| robjh | bwrap: execvp sh: Exec format error | 13:20 |
| robjh | Ive tired changing all references to x86{,_64} to aarch64. ive tried changing the url specified in elements/contents/alpine.bst to the aarch64 download. And ive tried removing the cache in ~/.cache/buildstream | 13:20 |
| tlater[m] | This is during the linux kernel compilation... | 13:21 |
| robjh | yep | 13:21 |
| tlater[m] | robjh: One complexity of this process is that there are actually two base images in use | 13:22 |
| robjh | at the start, as its setting up its config file | 13:22 |
| robjh | tlater[m], i didnt realise that | 13:22 |
| tlater[m] | One to build the image, and one to place on the image | 13:22 |
| tlater[m] | The alpine element is only placed on the image, so setting it to aarch64 is irrelevant | 13:22 |
| tlater[m] | You'll need to change the docker image | 13:22 |
| robjh | i see. `find . -name *docker*` isnt giving me any results | 13:23 |
| * tlater[m] checks the branch and sees if he can fix that | 13:23 | |
| tlater[m] | I think it was bluntly named base.bst or something | 13:23 |
| * tlater[m] is happy it's working well on x86 though 😆 | 13:24 | |
| robjh | i see that file. but im unsure of what to change in there :x | 13:24 |
| robjh | im happy youre happy :) | 13:25 |
| tlater[m] | It's going to be a learning experience for both of us then! I didn't intend for this to run on anything but x86, btw, I'm not sure if our image plugin will support this. | 13:25 |
| tlater[m] | robjh: Alright, so this is not currently possible with the docker image | 13:27 |
| tlater[m] | We don't build an aarch64 version of that particular image (presumably because it explicitly says x86image) | 13:28 |
| robjh | is it possible to specify a different docker image? (an arm based one) | 13:28 |
| tlater[m] | Yes, but you will first need to create such a docker image | 13:28 |
| robjh | i see that it's using buildstream/image-tools. where can i find that? | 13:29 |
| tlater[m] | I'm also unsure if you can use a local docker image, so you may need to build one and push it to dockerhub... | 13:30 |
| tlater[m] | Let me link you to our docker image repo | 13:30 |
| robjh | docker on arm is a thing, i'd be surprised if there isnt a slimmed down image out there already | 13:31 |
| tlater[m] | robjh: This is the source for this: https://gitlab.com/BuildStream/buildstream-docker-images/tree/master/image-tools | 13:31 |
| robjh | cool, thank you | 13:31 |
| tlater[m] | robjh: If you can get that to build on arm, our CI supports arm builds. Presumably enabling that should just be a small change to the CI config there. It will automatically upload to dockerhub. | 13:32 |
| robjh | let me have a look | 13:32 |
| tlater[m] | After that you'll just need to change the tag used by `base.bst` | 13:32 |
| tlater[m] | tiagogomes: On the known issues page... Do you have any bright ideas for displaying a (closed) thing next to issue links? | 13:36 |
| tlater[m] | I think it can be done quite easily with some embedded javascript, but obviously that won't work if javascript is disabled. Not sure how big of an issue that is. | 13:37 |
| tiagogomes | Use gitlab API? | 13:37 |
| tlater[m] | Yep, it's just one ajax call from what I can tell. | 13:38 |
| tiagogomes | And an pelican extension to use it | 13:38 |
| tiagogomes | As is being done for releases | 13:38 |
| tlater[m] | Right, I haven't looked into pelican extensions yet. | 13:38 |
| *** lachlan has quit IRC | 13:40 | |
| Nexus | juergbi: Would you be able to look at the latest commit on https://gitlab.com/BuildStream/buildstream/merge_requests/726 for me pls? | 13:48 |
| tiagogomes | Me neither. I only know they exist and valentind used one to full up the releases page | 13:48 |
| tiagogomes | s/full/fill | 13:48 |
| tlater[m] | tiagogomes: Looking further into it, this seems to be a markdown extension | 13:49 |
| juergbi | Nexus: I wrote a couple of comments ten minutes ago, you saw those already? | 13:49 |
| tlater[m] | So one way to do this would be to mimick gitlab and allow people to link to issues with buildstream#256 | 13:49 |
| valentind | tlater[m], What do you need as extension? | 13:49 |
| tlater[m] | valentind: We want to be able to see a little (closed) message next to links to buildstream issues when they are closed | 13:50 |
| tlater[m] | I'm thinking about the best way to do this... | 13:50 |
| tlater[m] | I don't think a markdown extension is a bad idea, but I think transforming buildstream#issue_num -> #issue_num is probably better. | 13:50 |
| valentind | Is there a markdown extension that already exist for that? | 13:51 |
| tlater[m] | Good question | 13:53 |
| * tlater[m] checks | 13:53 | |
| valentind | Otherwise, making a markdown extension should be quick. | 13:54 |
| tlater[m] | Yeah, I've had a look through your table extension | 13:55 |
| tlater[m] | It doesn't seem too hard | 13:55 |
| valentind | And then use that: https://docs.gitlab.com/ee/api/issues.html | 13:56 |
| Nexus | juergbi: i hadn't refreshed :) mb | 13:56 |
| tlater[m] | GET /projects/:id/issues/:issue_iid is all we need, actually | 13:56 |
| *** lachlan has joined #buildstream | 13:57 | |
| tlater[m] | We could even do fun things such as display the number of comments, or last update date | 13:57 |
| tlater[m] | Perhaps in a mouseover thing | 13:57 |
| *** WSalmon_ has quit IRC | 13:58 | |
| valentind | I would put the title of the issue as anchor title. | 13:58 |
| valentind | So that the mouseover shows the title of the issue. | 13:58 |
| *** WSalmon has joined #buildstream | 13:58 | |
| tiagogomes | I wouldn't lose time making it very fancy. The existence of a known issues page is quite debatable | 13:59 |
| tlater[m] | That is probably better, yes | 13:59 |
| *** toscalix_ has joined #buildstream | 14:00 | |
| Nexus | juergbi: please can you quickly confirm or correct my question on the MR? :) | 14:00 |
| tlater[m] | Looks like gitlab's own extensions are all vue-specific | 14:00 |
| tiagogomes | ~~~~~~~ tristan can you review https://gitlab.com/BuildStream/buildstream/merge_requests/671 before it becomes bit rot again :) commit-by-commit review is recommended | 14:02 |
| *** ikerperez has joined #buildstream | 14:04 | |
| *** ikerperez has quit IRC | 14:04 | |
| juergbi | Nexus: replied | 14:13 |
| Nexus | juergbi: thanks :D | 14:13 |
| *** rdale has joined #buildstream | 14:15 | |
| *** lachlan has quit IRC | 14:16 | |
| *** toscalix_ has quit IRC | 14:17 | |
| *** lachlan has joined #buildstream | 14:23 | |
| *** lachlan has quit IRC | 14:26 | |
| toscalix | added sept. 30th as fosdem 2019 dev room announcement date to the calendar | 14:37 |
| gitlab-br-bot | buildstream: merge request (tiagogomes/issue-287-backport->bst-1.2: Backport of !678 (Add validation of configuration variables) to 1.2 branch.) #789 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/789 | 14:40 |
| gitlab-br-bot | buildstream: issue #650 ("RFE: Add plugin to generate snaps") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/650 | 14:41 |
| gitlab-br-bot | buildstream: merge request (Qinusty/skipped-rework->master: Add SkipError for indicating a skipped activity) #765 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/765 | 14:44 |
| *** lachlan has joined #buildstream | 14:45 | |
| toscalix | I also added the lightning talks deadline | 14:46 |
| toscalix | If you want to receive alerts, go to the calendar and add yourself to the event. | 14:47 |
| gitlab-br-bot | buildstream: merge request (richardmaw/test-config-fixes->master: Fix tests that attempt to access the home directory) #780 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/780 | 14:47 |
| toscalix | Link to calendar: https://calendar.google.com/calendar/embed?src=codethink.co.uk_mpgah0uj538hnbsf4l7b4rchts%40group.calendar.google.com&ctz=Europe%2FMadrid | 14:49 |
| gitlab-br-bot | buildstream: merge request (jjardon/pycodestyle->master: WIP: Use pycodestyle instead pep8 python module) #638 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/638 | 14:49 |
| gitlab-br-bot | buildstream: merge request (jjardon/pycodestyle->master: WIP: Use pycodestyle instead pep8 python module) #638 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/638 | 14:50 |
| qinusty | https://gitlab.com/BuildStream/buildstream/merge_requests/765 is ready for review. Rebased to latest master. | 14:52 |
| gitlab-br-bot | buildstream: merge request (jjardon/pycodestyle->master: WIP: Use pycodestyle instead pep8 python module) #638 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/638 | 14:52 |
| *** lachlan has quit IRC | 15:02 | |
| gitlab-br-bot | buildstream: merge request (tpollard/494->master: WIP: Don't pull artifact buildtrees by default) #786 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/786 | 15:05 |
| *** lachlan has joined #buildstream | 15:16 | |
| gitlab-br-bot | buildstream: merge request (mac_fixes->master: WIP: Implement compatibility fixes for MacOSX and WSL Blocks #411 and #412") #726 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/726 | 15:20 |
| *** lachlan has quit IRC | 15:25 | |
| qinusty | I'm not going to get through all of these commits today tiagogomes, but I've dropped a few things I picked up on | 15:38 |
| *** lachlan has joined #buildstream | 15:41 | |
| tiagogomes | qinusty thanks, what do you mean by "This can be removed."? | 15:44 |
| qinusty | We do not need 'pass' when there is another line of code there. tiagogomes | 15:45 |
| tiagogomes | ah, indeed | 15:46 |
| gitlab-br-bot | buildstream: merge request (jonathan/pickle-yaml->master: WIP: Add a cache of parsed and provenanced yaml) #787 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/787 | 15:52 |
| *** lachlan has quit IRC | 16:39 | |
| *** toscalix has quit IRC | 16:49 | |
| *** jonathanmaw_ has quit IRC | 17:05 | |
| *** lachlan has joined #buildstream | 17:17 | |
| *** lachlan has quit IRC | 17:23 | |
| *** lachlan has joined #buildstream | 17:39 | |
| *** rdale has quit IRC | 18:16 | |
| gitlab-br-bot | buildstream: merge request (Qinusty/skipped-rework->master: Add SkipError for indicating a skipped activity) #765 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/765 | 18:19 |
| *** tristan has joined #buildstream | 18:24 | |
| *** xjuan has joined #buildstream | 18:27 | |
| *** lachlan has quit IRC | 18:28 | |
| *** lachlan has joined #buildstream | 18:42 | |
| *** xjuan has quit IRC | 19:01 | |
| *** lachlan has quit IRC | 19:22 | |
| gitlab-br-bot | buildstream: merge request (valentindavid/cargo_plugin->master: WIP: Add source plugin for cargo.) #790 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/790 | 20:14 |
| *** bochecha has quit IRC | 20:20 | |
| *** xjuan has joined #buildstream | 20:53 | |
| gitlab-br-bot | buildstream: merge request (mac_fixes->master: WIP: Implement compatibility fixes for MacOSX and WSL Blocks #411 and #412") #726 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/726 | 23:04 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!