IRC logs for #buildstream for Thursday, 2019-10-24

*** narispo has quit IRC00:16
*** narispo has joined #buildstream00:16
*** narispo has quit IRC00:19
*** narispo has joined #buildstream00:19
*** narispo has quit IRC00:32
*** narispo has joined #buildstream00:33
*** narispo has quit IRC02:23
*** narispo has joined #buildstream02:28
*** narispo has quit IRC02:41
*** narispo has joined #buildstream02:41
*** narispo has quit IRC04:03
*** narispo has joined #buildstream04:04
*** narispo has quit IRC04:46
*** narispo has joined #buildstream04:48
*** narispo has quit IRC05:08
*** narispo has joined #buildstream05:09
*** narispo has quit IRC05:28
*** narispo has joined #buildstream05:28
*** narispo has quit IRC05:33
*** narispo has joined #buildstream05:34
*** akvilebirgelyte has joined #buildstream07:41
*** traveltissues has joined #buildstream07:55
*** bochecha has joined #buildstream08:08
*** ikerperez has joined #buildstream08:12
*** traveltissues has quit IRC08:32
*** tiagogomes has joined #buildstream08:38
*** adds68 has joined #buildstream08:42
*** santi has joined #buildstream08:50
*** qinusty has joined #buildstream08:50
*** jonathanmaw has joined #buildstream09:04
*** rdale has quit IRC09:19
*** rdale has joined #buildstream09:22
*** lachlan has joined #buildstream09:30
juergbitest suite with buildbox-run: 1401 passed, 14 skipped, 4 xfailed, 32 warnings in 294.22s (0:04:54)10:40
tlater[m]juergbi: Do you happen to know why we need to `__update_cache_keys()` after `__update_source-state()`?10:49
* tlater[m] feels like those two would be mostly independent10:49
juergbitlater[m]: the source keys are part of the element cache key calculation10:49
tlater[m]Ah10:49
tlater[m]That's a good point10:49
benschubertjuergbi: using the bwrap thing around it?11:07
juergbibenschubert: yes11:07
benschubertawesome!11:07
juergbiso not testing userchroot itself yet but all of the rest11:07
juergbibenschubert: btw: userchroot complains if it's owned by group root11:09
juergbiwhat group is it commonly assigned to?11:09
benschubertjuergbi: did you setgid it? It should not, it should only be setuid :)11:09
juergbino, I used your docker file and got that error message11:10
benschubertoh interesting11:10
benschubertI didn't get that11:10
benschubertare you calling it with user root?11:10
juergbiah, I might have as I was just testing from docker11:10
juergbithat explains it. the error message is a bit confusing, though11:11
benschubertagreed11:11
benschuberttheir error messages are really bad :)11:11
*** traveltissues has joined #buildstream11:12
juergbibenschubert: have you had a chance to test freedesktop-sdk bst show with the FetchTree branch? #112611:59
gitlab-br-botIssue #1126: Cache resolution is slow https://gitlab.com/BuildStream/buildstream/issues/112611:59
*** santi has quit IRC12:13
*** santi has joined #buildstream12:24
benschubertsadly not, I'll do this by the end of the week :)12:35
juergbita12:43
gitlab-br-bottlater opened (was WIP) MR !1660 (tlater/annihilate_update_state->master: Remove update_state) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/166013:03
*** traveltissues has quit IRC13:09
*** phildawson_ has quit IRC13:44
*** phildawson_ has joined #buildstream13:44
*** lachlan has quit IRC13:47
*** lachlan has joined #buildstream13:55
* tpollard realises that Resolved Key Summary is no longer shown after bst build13:59
tpollardseemingly for a long long time14:01
tlater[m]juergbi: Looks like we don't need to initialize there, I wonder how I got that idea14:01
tlater[m]Looks like the if was just a noop14:01
tlater[m]Good catch, ta!14:01
juergbitlater[m]: great. I think there are multiple rdep initialize state cases. is this true for all?14:02
tlater[m]Two of them so far14:02
tlater[m]I think14:03
tlater[m]Halfway through testing the second14:03
tlater[m]I think the others will still need to initialize, simply because the elements are actually initialized in those cases.14:03
gitlab-br-bottpollard opened issue #1180 (Resolved Key Summary is not displayed in the end summary) on buildstream https://gitlab.com/BuildStream/buildstream/issues/118014:09
benschuberttpollard: isn't that normally only when new keys are discovered? That is when a track is involved?14:09
tpollardor if the source is changed?14:14
benschubertthe source changed?14:14
tpollardmaybe I've got some odd behaviour going on here....14:16
gitlab-br-bottpollard closed issue #1180 (Resolved Key Summary is not displayed in the end summary) on buildstream https://gitlab.com/BuildStream/buildstream/issues/118014:17
benschubertI'm not sure what's the correct behavior there, but it seems sensible to me not to show it if nothing has changed (no --track)14:19
tpollardyeh I think we must have actively decided that since 1.1 at some point14:20
tpollardmakes sense to me14:20
gitlab-br-botaevri opened (was WIP) MR !1657 (aevri/enable_spawn_ci_5->master: job pickling: also pickle global state in node.pyx) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/165714:56
*** bochecha has quit IRC15:15
*** lachlan has quit IRC15:17
tlater[m]juergbi: Do you expect there to be any other things that interact with the old workspace handling?15:28
* tlater[m] has looked through, and tested removing the occasional workspace reference, but it doesn't look like it15:28
juergbitlater[m]: I've noted the 3 things that I saw so far, but I will have to do some more reviewing15:31
tlater[m]Ok15:31
tlater[m]Thanks a lot for looking through this :)15:31
*** santi has quit IRC15:41
gitlab-br-botjuergbi opened MR !1667 (juerg/casd-connect->master: cascache.py: Defer attempt to connect to casd until socket file exists) on buildstream https://gitlab.com/BuildStream/buildstream/merge_requests/166715:57
*** santi has joined #buildstream16:03
*** lachlan has joined #buildstream16:04
juergbitlater[m]: that link in the commit just shows me the whole diff. you probably wanted to point to a line but that doesn't seem to work16:05
tlater[m]Oh, grmbl16:05
tlater[m]juergbi: This link should work: https://gitlab.com/BuildStream/buildstream/merge_requests/1645/diffs#d10b1e972922f297b08c3c672d861e73ef4e75f1_50_26516:06
tlater[m]If not, casserver.py:26516:06
juergbistill didn't work, even tried in firefox and chrome16:06
juergbiok16:06
juergbimaybe because the file was automatically collapsed16:07
* tlater[m] thinks it's probably because that diff is collapsed and gitlab screws up16:07
tpollardthe best feature....16:07
juergbitlater[m]: resolve_digest() is independent of refs, though, isn't it?16:07
tlater[m]snappish16:07
tlater[m]juergbi: I was mostly leaving that for reference, tbf, once I replace that with FetchTree it's a moot point16:08
juergbitrue16:08
tlater[m]No, it uses object_path16:09
tlater[m]I could have duplicated the logic, but I didn't want to :)16:09
*** santi has quit IRC17:01
*** lachlan has quit IRC17:07
*** narispo has quit IRC17:08
*** narispo has joined #buildstream17:08
*** tiagogomes has quit IRC17:15
*** jonathanmaw has quit IRC17:51
*** narispo has quit IRC18:22
*** narispo has joined #buildstream18:24
*** slaf has quit IRC18:27
*** slaf_ has joined #buildstream18:27
*** slaf has joined #buildstream18:27
*** slaf has joined #buildstream18:27
*** slaf has joined #buildstream18:27
*** slaf has joined #buildstream18:28
*** slaf has joined #buildstream18:28
*** slaf has joined #buildstream18:28
*** slaf has joined #buildstream18:28
*** narispo has quit IRC21:53
*** narispo has joined #buildstream21:53
*** slaf has quit IRC22:13
*** slaf has joined #buildstream22:13
*** slaf has joined #buildstream22:13
*** slaf has joined #buildstream22:13
*** slaf has joined #buildstream22:14
*** slaf has joined #buildstream22:14
*** slaf has joined #buildstream22:14
*** slaf has joined #buildstream22:14
*** slaf has joined #buildstream22:15
*** slaf has joined #buildstream22:15
*** slaf has joined #buildstream22:15
*** slaf has joined #buildstream22:15
*** slaf has joined #buildstream22:16
*** slaf has joined #buildstream22:16
*** slaf has joined #buildstream22:16
*** slaf has joined #buildstream22:17
*** slaf has joined #buildstream22:17
*** slaf has joined #buildstream22:17
*** slaf has joined #buildstream22:18
*** slaf has joined #buildstream22:18
*** slaf has joined #buildstream22:18
*** slaf has joined #buildstream22:18
*** slaf has joined #buildstream22:19
*** narispo has quit IRC22:24
*** narispo has joined #buildstream22:24
*** narispo has quit IRC23:25
*** narispo has joined #buildstream23:25
*** narispo has quit IRC23:48
*** narispo has joined #buildstream23:48
*** narispo has quit IRC23:51
*** narispo has joined #buildstream23:51
*** narispo has quit IRC23:57
*** narispo has joined #buildstream23:57
*** narispo has quit IRC23:59
*** narispo has joined #buildstream23:59

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