IRC logs for #buildstream for Friday, 2018-04-20

*** Prince781 has joined #buildstream00:08
gitlab-br-botbuildstream: merge request (jjardon/debian-9->master: WIP: .gitlab-ci.yml: Run test in current Debian stable (stretch)) #425 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/42500:14
*** Prince781 has quit IRC02:18
*** tristan has joined #buildstream04:59
gitlab-br-botbuildstream: merge request (tristan/enable-attribute-defined-outside-init->master: Enable attribute-defined-outside-init pylint eror) #426 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/42605:54
*** tristan has quit IRC06:07
gitlab-br-botbuildstream: merge request (tristan/enable-attribute-defined-outside-init->master: Enable attribute-defined-outside-init pylint eror) #426 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/42606:09
*** tristan has joined #buildstream06:18
gitlab-br-botbuildstream: issue #285 ("Get story straight with private symbols") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/28506:26
*** hase has joined #buildstream06:54
*** hase has quit IRC07:14
*** toscalix has joined #buildstream08:07
*** jonathanmaw has joined #buildstream08:19
paulsherwoodjjardon: ack08:21
*** bethw has joined #buildstream08:26
*** dominic has joined #buildstream08:27
*** noisecell has joined #buildstream09:10
*** aday has joined #buildstream09:16
*** toscalix has quit IRC09:18
*** toscalix has joined #buildstream10:30
gitlab-br-botbuildstream: issue #373 ("Use version checking for optional use of bwrap features") changed state ("opened") https://gitlab.com/BuildStream/buildstream/issues/37310:55
gitlab-br-botbuildstream: merge request (milloni/assert-bwrap-version->master: Assert minimum bwrap version on host) #379 changed state ("closed"): https://gitlab.com/BuildStream/buildstream/merge_requests/37910:56
gitlab-br-botbuildstream: merge request (unused-workspaces->master: _pipeline.py: correct reporting of unused workspace) #363 changed state ("closed"): https://gitlab.com/BuildStream/buildstream/merge_requests/36310:58
gitlab-br-botbuildstream: merge request (soft-arpy-dependency->master: Soften arpy dependency) #342 changed state ("closed"): https://gitlab.com/BuildStream/buildstream/merge_requests/34211:05
Nexustristan: what should i update !372/#311 (caching of build tree + use of cached build tree in workspace) with?11:11
tlaterta for merging !347 tristan :)11:12
tlater!342, even11:13
gitlab-br-botbuildstream: merge request (jmac/performance-ci->master: Add benchmark to CI test and document CI.) #341 changed state ("closed"): https://gitlab.com/BuildStream/buildstream/merge_requests/34111:18
*** mohan43u has quit IRC11:30
gitlab-br-botbuildstream: merge request (tristan/except-hook->master: Handle exceptions in the main process) #427 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/42711:43
gitlab-br-botbuildstream: merge request (jmac/exception-hook->master: Add a handler for otherwise unhandled exceptions in the main BuildStream app) #248 changed state ("closed"): https://gitlab.com/BuildStream/buildstream/merge_requests/24811:44
tristanNexus, I've pretty much outlined the reasons for it being unable to land here: https://gitlab.com/BuildStream/buildstream/issues/311#note_6926779711:46
tristanNexus, I don't have a straight answer for you, rather I could use a proactive clear proposal of what to do from you11:46
* tristan wont be around this night, it's friday and I'm already late :(11:47
*** mohan43u has joined #buildstream11:49
*** solid_black has joined #buildstream11:51
*** tristan has quit IRC11:57
*** solid_black has quit IRC12:26
gitlab-br-botbuildstream: merge request (214-filter-workspacing->master: Make workspace commands on a filter element transparently pass through to their build-dependency) #317 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/31713:12
jennisjonathanmaw, have you recently used the docker image plugin in bst-external?13:47
jonathanmawjennis: not recently, no13:47
jennisI've just tried to build the x86 linux image in the bst-examples repo, and it's being blocked by permissions between the docker image and the sandbox13:48
jennisI've just filed an issue, however, do you have any insight?13:48
jonathanmawjennis: I recall that when I've tried to use docker on my laptop, I often have to do the docker commands as root, I assume because I don't have permissions set up properly to communicate with the docker daemon otherwise13:49
jonathanmawI wonder if that's related.13:49
tlaterjonathanmaw: The docker image plugin doesn't use the docker daemon, iirc13:51
jonathanmaw:/ no idea, then13:51
tlaterjennis' problem seems to be lack of permissions to copy a setuid binary to his sandbox13:51
* tlater hasn't read the plugin's internals closely enough to actually know how to solve that issue, unfortunately13:52
tlaterjennis: Perhaps we'd need to make tar extract as the correct user/without suid here: https://gitlab.com/BuildStream/bst-external/blob/master/bst_external/sources/docker.py#L40913:54
tlaterBut I'd have thought that already happens...13:54
jenniscould this be anything to do with us changing from a tar element to an import element in the examples repo?13:56
tlaterjennis: Well, that's why you found the bug. The tar element was a temporary workaround when the docker image hadn't been upstreamed13:56
*** xjuan has joined #buildstream13:57
jennisOk right, that makes sense13:57
tlaterjennis: Are you blocked on that working? I'd have a look, but it might take a little while...13:58
jennistlater, no13:59
jennisWould've just made my testing a bit easier/quicker13:59
laurenceskullman, i fear that pretty soon your progress may be blocked on  https://gitlab.com/BuildStream/buildstream/issues/311#note_6926779714:02
skullmanlaurence: not in the near term since there's still details I need to get material together to work out (found all the relevant issues, deciding how to tie them all into the work and if more information needs to be put in)14:05
skullmanmedium-term it might cause some issues, but not necessarily, since I'll mostly be working on when things are cached rather than what is cached14:05
laurenceok.14:06
*** Prince781 has joined #buildstream14:08
laurenceskullman, there is also the fact that that issue needs to be resolved in some way, regardless.14:08
laurenceskullman, and I don't anyone yet has a full picture on the solution. So if you're working in the same problem space then any thoughts you have there could be helpful.14:09
tlaterHas anyone currently here had a peek at the scheduler before? I have a few ideas for my refactor, I'd like some opinions on them, if possible.14:13
* tlater wonders if jonathanmaw has dug there, but suspects few others have14:14
skullmanonly vaguely while searching for a bug, not enough to have an opinion on it14:15
jonathanmawtlater: I've been looking inside the FetchQueue recently. I haven't grokked the entire scheduler, though14:16
tlaterHm, I'll stick to some less big bits for now then, ta though, skullman, jonathanmaw14:18
*** Prince781 has quit IRC14:35
gitlab-br-botbuildstream: merge request (135-expire-artifacts-in-local-cache->master: WIP: Resolve "Expire artifacts in local cache") #347 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/34714:37
gitlab-br-botbuildstream: merge request (135-expire-artifacts-in-local-cache->master: WIP: Resolve "Expire artifacts in local cache") #347 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/34714:37
*** Prince781 has joined #buildstream14:47
*** tristan has joined #buildstream15:00
gitlab-br-botbuildstream: merge request (214-filter-workspacing->master: Make workspace commands on a filter element transparently pass through to their build-dependency) #317 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/31715:07
jonathanmaw\o/ after a few hairy fix-ups after rebases, my workspace redirection branch is presentable again!15:09
* jonathanmaw crosses fingers that the CI is happy, too15:10
*** csoriano has quit IRC15:13
*** csoriano has joined #buildstream15:14
*** noisecell has quit IRC15:18
*** toscalix has quit IRC16:14
*** bethw has quit IRC16:34
*** jennis has quit IRC16:56
*** bethw has joined #buildstream17:05
*** dominic has quit IRC17:05
*** jonathanmaw has quit IRC17:08
*** tiago has quit IRC17:10
*** aday has quit IRC17:49
gitlab-br-botbuildstream: merge request (135-expire-artifacts-in-local-cache->master: WIP: Resolve "Expire artifacts in local cache") #347 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/34717:50
gitlab-br-botbuildstream: merge request (135-expire-artifacts-in-local-cache->master: WIP: Resolve "Expire artifacts in local cache") #347 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/34717:51
*** Prince781 has quit IRC17:58
*** Prince781 has joined #buildstream18:00
*** Prince781 has quit IRC18:07
*** bethw has quit IRC18:12
*** Prince781 has joined #buildstream19:57
*** tristan has quit IRC20:38
*** xjuan has quit IRC21:33
*** cs_shadow has quit IRC23:43

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!