IRC logs for #buildstream for Thursday, 2017-07-27

*** jude has joined #buildstream08:12
*** tristan has quit IRC08:29
*** tlater has joined #buildstream08:30
*** jonathanmaw has joined #buildstream08:41
gitlab-br-botpush on buildstream@master (by Jürg Billeter): 1 commit (last: _scheduler/job.py: Ignore SIGTSTP error for already exited process) https://gitlab.com/BuildStream/buildstream/commit/7bd590319df360d0aac84eff72f9dabd8b38953309:22
*** tristan has joined #buildstream09:27
juergbitristan: do we need to do something to get the automated gnome-modulesets conversion use latest jhbuild2bst?09:38
tristanjuergbi, it selfupdates :)09:38
juergbii don't see any recent updates, though09:39
juergbinothing since the semi-manual triggered one yesterday afternoon09:39
juergbialso nothing in tracker09:39
juergbi*tracked09:39
tristanhmmm09:39
tristanjuergbi, are you checking the gnome7 logs, or are you checking the commit history ?09:40
tristanlooks like something got screwed there, lemme pop in and take a look...09:40
tristancrap this again09:41
juergbii simply checked git commit history09:42
tristanso sometimes (and this happened on gnome builders as well), they all get blocked on a lock file and keep queueing09:42
tristanlooks like one process never exited09:42
tristanSo git history will not change if something didnt change09:42
tristan(empty commit)09:42
tristanBut I suppose you were expecting a change :)09:43
* tristan restarted it anyway09:43
juergbiyes i was expecting changes :)09:43
tristanlets wait for the cron to trigger09:43
tristanlast time I manually launched one, maybe that was the issue09:43
juergbiok09:43
tristanthat's gonna rebuild everything :)09:44
tristanshould be 4 and a half minutes...09:45
gitlab-br-botpush on buildstream@cross_platform (by Tristan Maat): 1 commit (last: element.py: Remove SandboxBwrap import) https://gitlab.com/BuildStream/buildstream/commit/66b2f1f086685f1cc51b0ad966f05189bcbb70d609:57
juergbiyes, it's fine now10:02
juergbitristan: is "net-attempts" (default: 3) ok as name for the scheduler config option for network retries or do you have a better proposal?10:05
juergbior shall we go for something like "net-retries" where 0 would be the minimum value as retry excludes the first attempt10:05
gitlab-br-botpush on buildstream@cross_platform (by Tristan Maat): 1 commit (last: _artifactcache/__init__.py: Remove OSTreeCache import) https://gitlab.com/BuildStream/buildstream/commit/9402db7421903b00ad3b938f34f5a989656b3e7f10:05
tristanUmmm10:18
tristanI think I like 'retries' better, both for the inclusion of valid value 0, and also because it sounds like I would understand it better than 'attempts'10:19
tristanor understand it faster10:19
tristanalso maybe 'network-retries' ?10:19
tristanno need to abbreviate that ?10:19
juergbifine by me10:19
gitlab-br-botpush on buildstream@network-retries (by Jürg Billeter): 2 commits (last: _scheduler/job.py: Fix respawn) https://gitlab.com/BuildStream/buildstream/commit/855b63003ee15eb8cdd42f174a9193ee7b942ef510:59
gitlab-br-botbuildstream: merge request (network-retries->master: Network retries) #67 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/6711:00
ironfoot<juergbi> rm refs/heads/'gnu-toolchain/core-linux-pam/af9a8fd2d234e497bed8f863b672768ef3b612be4ebee38f73b90c8198ea937711:01
ironfootjuergbi: I've done that11:01
* ironfoot triggers failing pipeline https://gitlab.com/baserock/definitions/-/jobs/2490683911:02
gitlab-br-botpush on buildstream@jonathan/fix-dpkg-docs (by Jonathan Maw): 1 commit (last: Fix dpkg_build and dpkg_deploy documentation using wrong script names) https://gitlab.com/BuildStream/buildstream/commit/3f91671502de2a8efafa1d7e5a959a5744c6e82f11:04
gitlab-br-botbuildstream: merge request (jonathan/fix-dpkg-docs->master: Fix dpkg_build and dpkg_deploy documentation using wrong script names) #68 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/6811:05
gitlab-br-botbuildstream: issue #30 ("Configurable retries for network related tasks") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/3011:08
gitlab-br-botbuildstream: merge request (network-retries->master: Network retries) #67 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/6711:08
gitlab-br-botpush on buildstream@master (by Tristan Van Berkom): 2 commits (last: _scheduler/job.py: Fix respawn) https://gitlab.com/BuildStream/buildstream/commit/855b63003ee15eb8cdd42f174a9193ee7b942ef511:08
gitlab-br-botbuildstream: Jürg Billeter deleted branch network-retries11:08
tristanironfoot, looks like different failure now ?11:39
tristanxml-catalog11:39
gitlab-br-botpush on buildstream@pushreceive (by Jürg Billeter): 2 commits (last: _artifactcache/pushreceive.py: Add handshake after sending objects) https://gitlab.com/BuildStream/buildstream/commit/330615135053957b34a250ea919ecddeb67e82b311:40
tristanironfoot, not sure what's up with that but suspect has something to do with missing DESTDIR in environment ?11:40
gitlab-br-botbuildstream: merge request (pushreceive->master: Artifact push robustness improvements) #69 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/6911:41
tristanironfoot, you will recall back when we started this conversion, we agreed it was more desirable to make the scripts mention DESTDIR="%{install-root}" explicitly rather than have an env var11:41
tristanI suspect that needs to be handled for the xml-catalog post-install case11:42
ironfoothm.. so that is not happening automatically in defs2bst?11:44
ironfootthat may break lot of things11:45
tristanironfoot, you were there :)11:47
tristanyou know that code !11:47
ironfoot:P11:47
ironfootsilly past-me11:48
tristanironfoot, we talked about that, so yeah it would be more reliable to do "exactly what YBD did", but we considered this was a low amount of cases which required it and it would be better to require explicit make DESTDIR="%{install-root}" install across the board11:48
tristanironfoot, the converter takes care of the majority of cases, though11:49
ironfootso, is the ideal solution to explicitly put DESTDIR=$DESTDIR in all the places needed? like in xml-catalog post-install script?11:50
tristanironfoot, DESTDIR="%{install-root}" ./post-install.sh11:51
tristanin this case11:51
tristanI dont think there are many11:51
ironfootgiven that this is being auto-converted from definitions.git, i was thinking of fixing it in the .morph files11:51
tristanI see11:51
tristanironfoot, you could alternatively add an exception in defs2bst11:52
tristanironfoot, or checkout what defs2bst is doing first, and coax the morph file to be something it will convert well11:52
tristanironfoot, I think there is a clause where we shove PREFIX and DESTDIR in the environment, your DESTDIR=$DESTDIR thing might trigger that11:52
ironfootyeah, I'd have to check that first11:53
ironfootunfortunately, little time :(11:53
* ironfoot adds a note in the relevant issue12:00
gitlab-br-botpush on buildstream@jonathan/fix-dpkg-docs (by Jonathan Maw): 1 commit (last: Fix dpkg_build and dpkg_deploy documentation using wrong script names) https://gitlab.com/BuildStream/buildstream/commit/e15ca6cf49bc3f8ef79338322299978620ef6f6d12:08
gitlab-br-botbuildstream: merge request (jonathan/fix-dpkg-docs->master: Fix dpkg_build and dpkg_deploy documentation using wrong script names) #68 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/6812:08
gitlab-br-botpush on buildstream@master (by Jonathan Maw): 1 commit (last: Fix dpkg_build and dpkg_deploy documentation using wrong script names) https://gitlab.com/BuildStream/buildstream/commit/e15ca6cf49bc3f8ef79338322299978620ef6f6d12:24
gitlab-br-botbuildstream: merge request (jonathan/fix-dpkg-docs->master: Fix dpkg_build and dpkg_deploy documentation using wrong script names) #68 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/6812:24
gitlab-br-botbuildstream: Jonathan Maw deleted branch jonathan/fix-dpkg-docs12:24
gitlab-br-botpush on buildstream@jonathan/dpkg_no_debiandir (by Jonathan Maw): 1 commit (last: dpkg_build.yaml: Remove useless debiandir from artifacts) https://gitlab.com/BuildStream/buildstream/commit/a56d9efba5a0095234ee5c26b7bb80c3eaa1195813:48
gitlab-br-botbuildstream: merge request (jonathan/dpkg_no_debiandir->master: dpkg_build.yaml: Remove useless debiandir from artifacts) #70 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/7013:54
gitlab-br-botbuildstream: merge request (jonathan/dpkg_no_debiandir->master: dpkg_build.yaml: Remove useless debiandir from artifacts) #70 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/7014:04
gitlab-br-botpush on buildstream@master (by Tristan Van Berkom): 1 commit (last: dpkg_build.yaml: Remove useless debiandir from artifacts) https://gitlab.com/BuildStream/buildstream/commit/a56d9efba5a0095234ee5c26b7bb80c3eaa1195814:04
gitlab-br-botbuildstream: Jonathan Maw deleted branch jonathan/dpkg_no_debiandir14:04
gitlab-br-botpush on buildstream@pushreceive (by Jürg Billeter): 2 commits (last: _artifactcache/pushreceive.py: Add handshake after sending objects) https://gitlab.com/BuildStream/buildstream/commit/f44da295a33f09a9294d6aa0b7782361cd7d288c14:10
gitlab-br-botbuildstream: merge request (pushreceive->master: Artifact push robustness improvements) #69 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/6914:10
*** tiagogomes has quit IRC14:10
gitlab-br-botbuildstream: merge request (pushreceive->master: Artifact push robustness improvements) #69 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/6914:25
gitlab-br-botpush on buildstream@master (by Jürg Billeter): 2 commits (last: _artifactcache/pushreceive.py: Add handshake after sending objects) https://gitlab.com/BuildStream/buildstream/commit/f44da295a33f09a9294d6aa0b7782361cd7d288c14:25
gitlab-br-botbuildstream: Jürg Billeter deleted branch pushreceive14:25
tristanironfoot, can you upgrade buildstream on the baserock artifact server again ?14:53
gitlab-br-botpush on buildstream@master (by Tristan Van Berkom): 1 commit (last: Updated man pages.) https://gitlab.com/BuildStream/buildstream/commit/332fed488a8b8af995dbd346654b157b75846adf15:05
ironfoottristan: should be done15:10
tristanironfoot, great, there will be an error if old versions of buildstream try to push to that server now15:10
tristanso any builders, need to also be updated15:11
ironfootgitlab jobs will use latest15:11
tristanyep15:12
gitlab-br-botbuildstream: issue #32 ("bst build does not handle elements with Consistency.INCONSISTENT") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/3215:15
gitlab-br-botpush on buildstream@cross_platform (by Tristan Maat): 1 commit (last: WIP: tarcache.py: Speed up tar cache extraction) https://gitlab.com/BuildStream/buildstream/commit/5892da9afcb200f6e12a8f25bce262b7c2cb9ae415:28
*** tristan has quit IRC16:17
gitlab-br-botpush on buildstream@cross_platform (by Tristan Maat): 1 commit (last: tarcache.py: Speed up tar cache extraction) https://gitlab.com/BuildStream/buildstream/commit/323679b3c8f8d4f240e6a58f4c378e815e04dd0f16:21
gitlab-br-botpush on buildstream@cross_platform (by Tristan Maat): 1 commit (last: tarcache.py: Speed up tar cache extraction) https://gitlab.com/BuildStream/buildstream/commit/70e8b24e38f0b313b7fea7b77eac746aa25eb80216:29
*** tristan has joined #buildstream16:41
*** tlater has quit IRC17:09
*** jonathanmaw has quit IRC17:33

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