IRC logs for #buildstream for Friday, 2017-10-20

*** tristan has quit IRC04:18
*** tristan has joined #buildstream04:27
gitlab-br-botpush on buildstream@sam/fix-docs (by Tristan Van Berkom): 4 commits (last: Fix keyboardinterrupts caused by subprocesses) https://gitlab.com/BuildStream/buildstream/commit/23c1e284ea748942e7067454aae2b279a1ab0e6805:07
gitlab-br-botbuildstream: merge request (sam/fix-docs->master: .gitlab-ci.yml: Fix documentation build) #116 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/11605:07
gitlab-br-botbuildstream: merge request (102-run-ci-as-non-root-user->master: Resolve "Run CI as non-root user") #104 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/10405:10
gitlab-br-botpush on buildstream@102-run-ci-as-non-root-user (by Tristan Van Berkom): 2 commits (last: Add warnings when staging to /buildstream/build) https://gitlab.com/BuildStream/buildstream/commit/10847a66dc0748696829d5fd18ad17043c2c687805:10
gitlab-br-botpush on buildstream@master (by Tristan Van Berkom): 1 commit (last: .gitlab-ci.yml: Fix documentation build) https://gitlab.com/BuildStream/buildstream/commit/8cdacab742d0749da45ba2b807f13c59f0bfd18f05:11
gitlab-br-botbuildstream: merge request (sam/fix-docs->master: .gitlab-ci.yml: Fix documentation build) #116 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/11605:11
gitlab-br-botbuildstream: merge request (102-run-ci-as-non-root-user->master: Resolve "Run CI as non-root user") #104 changed state ("opened"): https://gitlab.com/BuildStream/buildstream/merge_requests/10405:12
gitlab-br-botpush on buildstream@102-run-ci-as-non-root-user (by Tristan Van Berkom): 2 commits (last: .gitlab-ci.yml: Fix documentation build) https://gitlab.com/BuildStream/buildstream/commit/8cdacab742d0749da45ba2b807f13c59f0bfd18f05:12
*** adds68 has quit IRC07:57
*** adds68 has joined #buildstream07:57
*** jonathanmaw has joined #buildstream08:37
gitlab-br-botpush on buildstream@master (by Tristan Van Berkom): 1 commit (last: .gitlab-ci.yml: Drop root privileges for some tests) https://gitlab.com/BuildStream/buildstream/commit/324a3cbd69026fa565bbeb174a308456f1ef8fa608:54
gitlab-br-botbuildstream: issue #102 ("Run CI as non-root user") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/10208:54
gitlab-br-botbuildstream: merge request (102-run-ci-as-non-root-user->master: Resolve "Run CI as non-root user") #104 changed state ("merged"): https://gitlab.com/BuildStream/buildstream/merge_requests/10408:54
tristanjuergbi, any thoughts on https://gitlab.com/BuildStream/buildstream/issues/105#note_44039467 ?08:58
tristanI'm tempted to immediately set SOURCE_DATE_EPOCH=0 across the board, but reproducible builds says "... MUST not", which is confusing and obnoxious08:59
juergbitaking a look09:01
tristanhttps://reproducible-builds.org/specs/source-date-epoch/09:02
* tristan wonders if they have a mailing list, where do things get contested ?09:02
* tristan subscribes to https://lists.reproducible-builds.org/listinfo/rb-general09:03
tristanmight be interesting ML for others in this channel btw09:03
juergbitristan: i assume the 'MUST NOT' refers to build systems of individual modules09:04
tristanIt says exporting to subprocesses, if it is not present09:04
juergbii.e., they should allow top-level build systems to set it and the module-level build systems should use the set value09:04
tristanAh that could make more sense09:05
juergbihttps://wiki.debian.org/ReproducibleBuilds/TimestampsProposal#Setting_the_variable09:05
tristanonly set if not already in the env09:05
juergbiDebian sets the variable09:05
juergbii'm not a big fan of timestamp 0, not sure whether we can come up with something better, though09:06
tristanbaserock was using november 11, 2011 for some things09:06
tristanI suppose all it does is avoid weirdos (like ninja) making freaking weird assumptions about timestamps09:06
tristanjuergbi, it would be more *correct* to dynamically set it to last mtime of Source implementation, so that... meh, not such a big bonus ?09:07
juergbidebian uses the timestamp of the last entry in debian/changelog, which is much nicer09:07
tristanmaybe something like `cat --version` is going to say "Built on Jan 1, 1970" ?09:08
tristannicer but... like what gain do you get from it ?09:08
*** givascu has joined #buildstream09:08
tristanWe'd have to have all sources find the last mtime, which eh, seems mostly pointless09:08
juergbigit doesn't store mtimes, so for git it would also be the commit timestamp09:09
tristanthat's what an mtime would be for a git source, yeah09:09
tristanI mean, last commit is last modification time isnt it ?09:09
tristanan every source would need to provide one09:09
tristanin some way09:09
tristanall so that... modules can know when they were built ?09:10
juergbii thought you meant filesystem mtime with mtime09:10
tristanyeah I gather :)09:10
tristanI mean abstract Source.mtime() return value09:11
juergbii'm not sure whether there would be sufficient real benefits09:11
juergbii just really dislike completely fake timestamps09:11
tristanOk, and adding SOURCE_DATE_EPOCH=0 means cache key breakage if we ever change that09:11
tristannot bad, but persia doesnt like those :)09:12
juergbiavoiding timestamp 0 to avoid breaking some odd software may be sensible, though09:12
juergbii.e., like baserock09:12
tristanjuergbi, lets set it to something fake, november 11th 2011 lets say... and if one day we want to do better, we will09:12
tristanmakes sense I think ?09:13
juergbiok09:13
* tristan struggles to ask `date` to tell him the real value of human readable "Nov 11 2011 UTC"09:13
tristanor UTC doesnt matter here I guess09:13
*** ssam2 has joined #buildstream09:14
tristan132093720009:14
tristan2011-11-11 = 132093720009:14
gitlab-br-botpush on buildstream@master (by Tristan Van Berkom): 1 commit (last: data/projectconfig.yaml: Fixes #105 - Set SOURCE_DATE_EPOCH to constant value) https://gitlab.com/BuildStream/buildstream/commit/ea3c76b1b272f7699c6ef32b164881fc7cf591a609:17
gitlab-br-botbuildstream: issue #105 ("Consider SOURCE_DATE_EPOCH effort") changed state ("closed") https://gitlab.com/BuildStream/buildstream/issues/10509:17
*** tlater has joined #buildstream09:27
tristantlater, fyi, I'm a bit nervous about having landed the running CI as regular user thing09:32
tristantlater, the pytest job failed, and then passed on a retry, and... I'm not exactly sure why09:32
tristan:-/09:32
* tlater will have a look09:33
tristanPerhaps it relied on the chown thing having run once, or not, in the cache ?09:33
tlatertristan: I don't think so, considering I ran it on a fresh instance of the project before pushing to origin09:33
tristanyeah I really dont know09:35
tristantlater, we might get some failing pipelines, and it might be related to that09:35
tristanit certainly was not a network timeout, failed somewhere in the push/pull frontend tests09:36
* tristan gotta run though :-/09:36
*** tristan has quit IRC09:40
*** ssam2 has quit IRC10:25
*** ssam2 has joined #buildstream10:41
gitlab-br-botpush on buildstream@tracking-changes (by Tristan Maat): 1 commit (last: Issue #113: Split tracking and saving in `bst build`) https://gitlab.com/BuildStream/buildstream/commit/c54ec0bc9140c7955e54b137a1678f35faa8c5bd11:54
*** givascu has quit IRC13:06
gitlab-br-botpush on buildstream@tracking-changes (by Tristan Maat): 1 commit (last: Issue #113: Split tracking and saving in `bst build`) https://gitlab.com/BuildStream/buildstream/commit/ae7374b3a41531aca0c91da3b144e4d39cefdeda13:22
gitlab-br-botpush on buildstream@tracking-changes (by Tristan Maat): 1 commit (last: Issue #113: Split tracking and saving in `bst build`) https://gitlab.com/BuildStream/buildstream/commit/2b42f782f8589be70fa46a6d9219a15f9e7d074b14:07
*** gitlab-br-bot has quit IRC15:05
ssam2bot is being migrated to a new host15:05
*** gitlab-br-bot has joined #buildstream15:08
*** brlogger has joined #buildstream15:35
*** brlogger` has joined #buildstream15:38
ssam2two loggers! oh dear15:39
*** brlogger has joined #buildstream15:41
*** brlogger has joined #buildstream15:46
ssam2go away brlogger!15:47
*** brlogger has joined #buildstream16:09
*** brlogger has quit IRC16:17
*** jonathanmaw has quit IRC16:26
*** jude has quit IRC16:28
*** jude has joined #buildstream16:45
*** xjuan has joined #buildstream16:56
*** tristan has joined #buildstream17:03
tristansorry I broke tests :)17:04
*** tiagogomes has quit IRC17:11
tristanfixed17:17
ssam2the gitlab bot isn't working, i'll try and fix that early next week17:19
tlaterAww, poor gitlab-br-bot17:21
ssam2i didn't realise it needs to be able to receive http requests for the webhooks to work17:21
tlatertristan: Awesome, now my CI won't fail anymore ;)17:22
*** gitlab-br-bot has quit IRC17:22
*** tlater has quit IRC17:25
*** ssam2 has quit IRC17:25
*** bochecha has quit IRC21:49
*** xjuan has quit IRC22:43

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