| *** gtristan has quit IRC | 07:14 | |
| *** gtristan has joined #baserock | 07:42 | |
| *** toscalix has joined #baserock | 07:59 | |
| *** cosm has quit IRC | 08:38 | |
| *** bashrc has joined #baserock | 09:01 | |
| *** ctbruce has joined #baserock | 09:08 | |
| ctbruce | a | 09:08 |
|---|---|---|
| *** edcragg has joined #baserock | 09:10 | |
| *** edcragg has quit IRC | 09:27 | |
| *** rdale has joined #baserock | 09:33 | |
| *** jonathanmaw has joined #baserock | 09:41 | |
| *** edcragg has joined #baserock | 09:57 | |
| *** gtristan has quit IRC | 10:01 | |
| *** ssam2 has joined #baserock | 10:10 | |
| *** ChanServ sets mode: +v ssam2 | 10:10 | |
| *** Lachlan1975 has joined #baserock | 10:25 | |
| benbrown_ | richard_maw: I'm actually a little confused by your reply to my ybd patch re remote.origin.mirror, `man git config` doesn't mention that behaviour? | 10:29 |
| benbrown_ | I may have misunderstood what you meant though. | 10:30 |
| benbrown_ | As far as I'm aware, --prune would need to be added to both fetch and remote update to delete refs that no longer exist? | 10:30 |
| richard_maw | mirror mode is a bit magic, I'll see if I can find some documentation to point you at in a bit | 10:31 |
| benbrown_ | richard_maw: hehe, thanks :) | 10:32 |
| *** Lachlan1975 has quit IRC | 10:40 | |
| *** locallycompact has joined #baserock | 10:50 | |
| *** gtristan has joined #baserock | 10:54 | |
| richard_maw | benbrown_: turns out remote.origin.mirror is actually just about push behaviour, which will delete on the remote end | 10:55 |
| richard_maw | benbrown_: so it was entirely redundant in the original config since we never push from our git cache | 10:56 |
| * richard_maw had the behaviour on push confused with the behaviour on fetch, since it would have been logical for it to be symmetric | 10:56 | |
| benbrown_ | richard_maw: Cool, thanks for clearing that up :) | 10:59 |
| *** ctbruce has quit IRC | 12:03 | |
| *** ctbruce has joined #baserock | 12:17 | |
| pedroalvarez | locallycompact: just curious, do you know that we have gerrit available for code review? | 12:20 |
| locallycompact | pedroalvarez, yes, and sorry but once was enough | 12:20 |
| pedroalvarez | haha fair | 12:21 |
| pedroalvarez | patches in the mailing list are welcome too | 12:21 |
| * paulsherwood remembers the days when the mailing list was full of patches | 12:24 | |
| paulsherwood | using gerrit has a side-effect of making the ml seem much quieter than it was | 12:24 |
| paulsherwood | and we lose some feedback that people would give when seeing ml patches | 12:25 |
| * paulsherwood does not see any gerrit patches by default, for example, therefore does not comment on them | 12:26 | |
| pedroalvarez | locallycompact: have you actually used this ansible script? there is something that I'm not sure is syntactically correct | 12:27 |
| *** ctbruce has quit IRC | 12:27 | |
| locallycompact | I have, what is wrong? | 12:27 |
| pedroalvarez | the "get Riemann dash gems" task | 12:28 |
| pedroalvarez | note, I said "I'm not sure" | 12:28 |
| *** rdale_ct has joined #baserock | 12:29 | |
| locallycompact | Oh, is it that you can't do multiple gems? | 12:29 |
| *** rdale has quit IRC | 12:30 | |
| pedroalvarez | well, you can but I've never seen that syntax anywhere. | 12:39 |
| pedroalvarez | I've answered | 12:39 |
| locallycompact | ah, I see | 12:43 |
| locallycompact | tyvm | 12:43 |
| locallycompact | the gems weren't actually working | 12:43 |
| locallycompact | as it was | 12:44 |
| *** gtristan has quit IRC | 14:03 | |
| *** bashrc has quit IRC | 14:15 | |
| *** bashrc has joined #baserock | 14:21 | |
| paulsherwood | JPohlmann: is there a visible demo instance of custard anywhere? (yes, i'm stalking you...) | 14:33 |
| *** gtristan has joined #baserock | 14:40 | |
| *** bashrc has quit IRC | 14:57 | |
| *** bashrc has joined #baserock | 14:57 | |
| locallycompact | Has anyone used AWS to spin up a baserock vm? The launch instance options don't seem to allow for a custom image - just redhat and ubuntu things | 15:13 |
| paulsherwood | iirc it was previously possible to use custom images... but i failed when i tried | 15:13 |
| richard_maw | locallycompact: technically I have, couldn't get it to work though | 15:13 |
| paulsherwood | richard_maw may know more | 15:14 |
| * richard_maw will try to find where he wrote up his steps | 15:14 | |
| paulsherwood | richard_maw: oh, did you succeed, then? | 15:14 |
| richard_maw | paulsherwood: probably not, but without any way to get output from the machine it was impossible to tell | 15:15 |
| paulsherwood | lol | 15:15 |
| paulsherwood | so you couldn't ssh in, i assume | 15:15 |
| richard_maw | locallycompact: I'll PM you the link to where I wrote it up, since it's not recorded in a place for public consumption | 15:16 |
| locallycompact | ok | 15:16 |
| richard_maw | paulsherwood: that, and no serial available | 15:16 |
| paulsherwood | ack | 15:16 |
| richard_maw | there was some evidence the machine claimed the IP address it was allocated over DHCP | 15:16 |
| richard_maw | locallycompact: the gist of the steps to get as far as I did was: 0. make a disk image, 1. import the volume into AWS, 3. convert it into a snapshot 4. register the snapshot as a virtual machine | 15:18 |
| *** CTtpollard has quit IRC | 15:57 | |
| *** jonathanmaw_ has joined #baserock | 15:58 | |
| *** jonathanmaw has quit IRC | 16:01 | |
| *** jonathanmaw_ is now known as jonathanmaw | 16:05 | |
| *** CTtpollard has joined #baserock | 16:07 | |
| *** faybrocklebank has quit IRC | 16:10 | |
| *** ctbruce has joined #baserock | 16:34 | |
| *** tiagogomes has quit IRC | 16:37 | |
| *** edcragg has quit IRC | 16:39 | |
| *** CTtpollard has quit IRC | 16:41 | |
| *** edcragg has joined #baserock | 16:41 | |
| *** CTtpollard has joined #baserock | 16:42 | |
| *** jonathanmaw_ has joined #baserock | 16:44 | |
| *** jonathanmaw has quit IRC | 16:45 | |
| locallycompact | Is the go that's already on baserock lorrying properly? It only goes up to 1.4 | 16:46 |
| pedroalvarez | delta/go.git ? | 16:46 |
| * pedroalvarez goes to have a look | 16:47 | |
| *** ctbruce has quit IRC | 16:49 | |
| *** tiagogomes has joined #baserock | 16:49 | |
| pedroalvarez | locallycompact: there is something weird about it... it's an hg lorry from https://code.google.com/p/go | 16:49 |
| pedroalvarez | but that now links to a github repo o.0 | 16:50 |
| * richard_maw claps | 16:50 | |
| * pedroalvarez too | 16:50 | |
| richard_maw | hand to forehead clapping, rather than hand to hand | 16:50 |
| paulsherwood | ROFL | 16:50 |
| JPohlmann | paulsherwood: There isn't, nope | 16:50 |
| paulsherwood | JPohlmann: ah, ok | 16:51 |
| * paulsherwood can wait :) | 16:51 | |
| JPohlmann | paulsherwood: I could spin up a demo somewhere | 16:51 |
| pedroalvarez | aaaand... latest commit of go in gbo is not present in the github repo | 16:51 |
| pedroalvarez | moar clapping | 16:52 |
| paulsherwood | pedroalvarez: not surprising, their conversion will be different from ours | 16:52 |
| paulsherwood | we'll need a new go-git, i expect | 16:52 |
| JPohlmann | paulsherwood: ...if you remind me over the weekend? I'm pretty busy getting started with customers during the week | 16:52 |
| paulsherwood | JPohlmann: don't worry - i don't want to create work for you. how 'mature' is it? | 16:53 |
| jjardon | Hi! Can I have another review for https://gerrit.baserock.org/#/c/1782/ ? python support is broken for some modules without that patch | 16:54 |
| JPohlmann | paulsherwood: Not very. The UI has a few issues and there is currently no useful data validation. And I've only recently started to look into a decent history feature. | 16:55 |
| JPohlmann | At this point Mustard is more feature-complete. | 16:55 |
| richard_maw | jjardon: I strongly believe that all python3 things shouldn't symlink themselves into where python2 things put themselves | 16:56 |
| richard_maw | $PREFIX/bin/python{,-config} should be for python2 IMO | 16:57 |
| paulsherwood | JPohlmann: ack | 16:57 |
| jjardon | richard_maw: Do you think the bug is in the chunks then? They are expecting python-config to exist | 16:57 |
| richard_maw | jjardon: I think if they are asking for python-config, they want python2 | 16:58 |
| richard_maw | and if they want python, they want python2 | 16:58 |
| jjardon | richard_maw: no, its a python3-compatible chunk; It can be a bug use python-config unconditionally though | 16:59 |
| richard_maw | if it's asking for python-config it should get python2-config, if not, yes, it's a bug in its python3 compatibility | 16:59 |
| jjardon | richard_maw: are you suggesting that python should not exist in a python3-only system? I have no problems with that but I think thats agains one of the python PEPs | 17:00 |
| rjek | Sounds like a packaging/distro issue, not a python issue | 17:00 |
| * richard_maw would have to re-read the pep | 17:01 | |
| richard_maw | but the last time I read one like that, I came to the conclusion that you should not have python if it is not python2 | 17:01 |
| * jjardon rechecks | 17:01 | |
| richard_maw | there was some mumbling about "python" being only for the subset of python that is interpretable identically in both | 17:02 |
| richard_maw | but given the amount of existing software that assumes "python" == "python2", to try that would be antisocial | 17:02 |
| richard_maw | individual applications may decide to play nice with specifying python for the intersection of the two | 17:03 |
| richard_maw | but as a distro packager, unless you have the ability to enforce that only software which conforms to that convention is included | 17:03 |
| richard_maw | you have to say python == python2 | 17:04 |
| * richard_maw steps down from his soapbox | 17:05 | |
| jjardon | richard_maw: ok, I will actually change the patch to not generate a "python" symlink and see what breaks | 17:06 |
| * richard_maw recalls that causing python2 to misbehave, since it assumed that it shouldn't stomp over the /usr/bin/python symlink with python2 if it existed at build time | 17:07 | |
| richard_maw | which was another reason why I didn't like the python -> python3 symlink | 17:08 |
| locallycompact | what does this indicate? (ybd) https://paste.fedoraproject.org/320850/55124100/ | 17:08 |
| paulsherwood | almost certainly, that i'm an idiot. which version of ybd? | 17:10 |
| locallycompact | I think it was 16.01 with some hotfixes, let me try master | 17:10 |
| paulsherwood | locallycompact: to avoid me constantly asking that question, you could paste the whole log :) | 17:10 |
| jjardon | richard_maw: python symlink will be rewriten if python2 is installed: http://git.baserock.org/cgit/baserock/baserock/definitions.git/tree/strata/python2-core/python2.morph#n12 | 17:11 |
| locallycompact | ok | 17:11 |
| richard_maw | jjardon: aye, but if you hadn't made python3 install the python symlink, you wouldn't have needed to add that post-hoc, after much swearing at it breaking things when both python2 and python3 were installed | 17:11 |
| locallycompact | https://paste.fedoraproject.org/320853/12433114/ | 17:12 |
| paulsherwood | locallycompact: hmm. could you try with log-verbose: True, please? | 17:14 |
| jjardon | richard_maw: that line was changed at the same time the python3 transition was made ... if something broke was because some missing python2 dependencies maybe? | 17:17 |
| locallycompact | https://paste.fedoraproject.org/320857/51246371/ | 17:17 |
| paulsherwood | locallycompact: looks like it's not actually doing anything in the go1 sandbox, which is weird. does your go1 definition have a repo: and ref: ? | 17:19 |
| paulsherwood | yes, it does. | 17:20 |
| paulsherwood | what about a build-system? | 17:20 |
| locallycompact | no, should it? | 17:21 |
| jjardon | richard_maw: :) our own python-distutils instructions will break if "python" doesn't exist | 17:21 |
| *** jonathanmaw_ is now known as jonathanmaw | 17:22 | |
| richard_maw | jjardon: yeah, but if you want python3 of those, you should really be explicit | 17:23 |
| paulsherwood | locallycompact: this is still a bug in ybd (should make it more obvious what's happened), but yes, better to specify what build-system you expect it to use, unless you've got a morph file specifying the build-steps explicitly | 17:23 |
| richard_maw | jjardon: since there's a little magic in setup.py, in that it installs the packages for the version of python that invoked the command | 17:23 |
| richard_maw | jjardon: I'd rather you explicitly had it run `python3 setup.py install`, and have a separate chunk with a different name, to prevent confusion when people try to use python modules for the wrong version of python | 17:24 |
| *** jonathanmaw has quit IRC | 17:24 | |
| locallycompact | paulsherwood, I do have a go/go1.4.3.morph with build steps | 17:24 |
| locallycompact | should that be enough? | 17:24 |
| *** jonathanmaw has joined #baserock | 17:24 | |
| jjardon | richard_maw: should we have python3-distutils in DEFAULTS, then ? | 17:24 |
| richard_maw | jjardon: in the absence of parameterised builds, yes | 17:25 |
| jjardon | richard_maw: rigth | 17:25 |
| paulsherwood | locallycompact: yes, it should. i'm confused, then | 17:25 |
| locallycompact | setting build-system: manual in the chunk part of strata morphology seemed to get further | 17:26 |
| pedroalvarez | locallycompact: build-system shouldn't be there if you are also creating a .morph file with the instructions | 17:27 |
| pedroalvarez | it is redundant, and can create conflicts | 17:28 |
| paulsherwood | ??? | 17:28 |
| pedroalvarez | !!! | 17:28 |
| richard_maw | --- | 17:28 |
| richard_maw | ... | 17:28 |
| richard_maw | ^ yaml document | 17:28 |
| locallycompact | {@:O_O:@} | 17:28 |
| locallycompact | ^ mammal document | 17:29 |
| paulsherwood | locallycompact: in any case, would you mind publishing your definitions? i've not seen this issue before | 17:29 |
| *** jonathanmaw has quit IRC | 17:29 | |
| * paulsherwood would like to reproduce, and close the hole | 17:30 | |
| rdale_ct | 'manual' is a set of empty build instructions in the defaults | 17:30 |
| locallycompact | sure let me just do it all once more | 17:30 |
| *** tiagogomes has quit IRC | 17:34 | |
| locallycompact | paulsherwood, https://github.com/locallycompact/definitions/commit/d7c5500f211452b00ca308ab4f03deeaec0d1ed7 | 17:37 |
| paulsherwood | tvm | 17:38 |
| jjardon | mmm, its a bit weird I have to patch morph to support a new build system in DEFAULTS; paulsherwood, does ybd have the same 'problem' ? | 17:46 |
| paulsherwood | i believe ybd should read DEFAULTS from definitions, will fall back to its own copy if none present | 17:47 |
| pedroalvarez | jjardon: uh? isn't morph using them from DEFAULTS? | 17:49 |
| pedroalvarez | I mean, why is patching morph needed? | 17:49 |
| jjardon | pedroalvarez: I think because this line http://git.baserock.org/cgit/baserock/baserock/morph.git/tree/morphlib/defaults.py#n76 | 17:51 |
| jjardon | the build system should exist in http://git.baserock.org/cgit/baserock/baserock/morph.git/tree/morphlib/buildsystem.py#n278 (I can be wrong) | 17:52 |
| pedroalvarez | I can't find anything obvious in the line you posted | 17:55 |
| *** ssam2 has quit IRC | 17:55 | |
| paulsherwood | locallycompact: i'll have to investigate this overnight, sorrty | 17:58 |
| locallycompact | ack | 18:03 |
| *** bashrc has quit IRC | 18:03 | |
| *** locallycompact has quit IRC | 18:05 | |
| rdale_ct | i think the go build problem might be because ybd doesn't set up TMPDIR and it is defaulting to /var/tmp which might not be writable | 18:12 |
| jjardon | I think ybd should use $XDG_CACHE_HOME, not TMPDIR (wich is notmally /tmp, hence a tmpfs filesystem) | 18:17 |
| jjardon | richard_maw: patches to add the new python3-distutils build system sent: https://gerrit.baserock.org/#/q/status:open+project:baserock/baserock/definitions+branch:master+topic:python3-distutils | 18:18 |
| *** edcragg has quit IRC | 18:18 | |
| pedroalvarez | jjardon: nice to see that patching morph wasn't needed | 18:20 |
| jjardon | pedroalvarez: patch coming for morph as well ;) | 18:21 |
| pedroalvarez | doh | 18:21 |
| jjardon | (ybd works without patching btw) | 18:21 |
| pedroalvarez | morph should as well | 18:21 |
| jjardon | pedroalvarez: do the test: try to build the branch Ive just submitted | 18:22 |
| pedroalvarez | that patch shouldn't be merged until we do a release with Morph fix in then | 18:22 |
| pedroalvarez | yeah, I trust you, but it shouldn't fail IMO | 18:22 |
| jjardon | yep, I think its really a bug in morph | 18:23 |
| pedroalvarez | KeyError: 'Unknown build system: python3-distutils' | 18:27 |
| *** toscalix has quit IRC | 18:41 | |
| pedroalvarez | I think I found the error, but it's going to be tricky to fix | 18:53 |
| *** locallycompact has joined #baserock | 19:02 | |
| pedroalvarez | created a story for it: https://storyboard.baserock.org/#!/story/72 | 19:03 |
| paulsherwood | rdale_ct: ybd does setup TMPDIR now - https://github.com/devcurmudgeon/ybd/blob/a3960ba2ef28321aa04f4aed08eaaeee9c34d238/ybd/sandbox.py#L43 | 19:04 |
| paulsherwood | i tried honouring $XDG_CACHE_HOME instead of $base/tmp by default, but it broke for some users | 19:05 |
| rdale_ct | if that TMPDIR change is recent, maybe it isn't in the version of ybd that locallycompact is using | 19:27 |
| locallycompact | I have that change | 19:30 |
| rdale_ct | ah ok | 19:30 |
| rdale_ct | not that then | 19:30 |
| rdale_ct | this won't do anything as a configure command: export GOPATH=$(pwd) as the env variable value will be lost when running the build command | 19:33 |
| locallycompact | yeah I expected as much | 19:34 |
| *** edcragg has joined #baserock | 20:30 | |
| paulsherwood | http://paste.baserock.org/fajuqobono | 20:37 |
| paulsherwood | so i've made ybd explicitly error and raise an exception when it fails to autodetect buildsystem | 20:38 |
| paulsherwood | hopefully that's clearer in future | 20:38 |
| paulsherwood | technically definitions now are supposed to insist on buildsystem being defined, iirc, so ybd is misbehaving by even attempting autodetect :) | 20:39 |
| paulsherwood | locallycompact: i assume you've seen http://git.baserock.org/cgit/delta/go.git/commit/?id=baserock/go1.2.2 | 20:41 |
| paulsherwood | that's how i got pre 1.4 go to work | 20:41 |
| *** cosm has joined #baserock | 20:55 | |
| locallycompact | paulsherwood, I hadn't | 20:57 |
| paulsherwood | ah, ok :) well... maybe it will help :-) | 20:57 |
| locallycompact | paulsherwood, what do I put for build-system, manual? | 20:58 |
| paulsherwood | yup | 20:58 |
| paulsherwood | stretch goal might be to add a go-lang to defaults, including indicators :) | 20:59 |
| paulsherwood | s/go-lang/go-lang buildsystem/ | 20:59 |
| paulsherwood | locallycompact: you remember your error from the other day, when i mentioned about the warnings because missing .unpacked files? did the problem go away after that was fixed | 21:03 |
| paulsherwood | ? | 21:03 |
| locallycompact | Can't remember which one that was right now | 21:04 |
| paulsherwood | nmhttps://github.com/devcurmudgeon/ybd/issues/154 | 21:04 |
| locallycompact | Oh yeah that fixed it | 21:05 |
| paulsherwood | cool | 21:06 |
| *** gtristan has quit IRC | 21:19 | |
| paulsherwood | kbas now serving over 10,000 artifacts :) | 21:31 |
| *** bfletcher has quit IRC | 21:42 | |
| *** zoli_ has quit IRC | 21:42 | |
| *** franred_ has joined #baserock | 21:42 | |
| *** franred has quit IRC | 21:48 | |
| *** trn has quit IRC | 21:48 | |
| *** lachlanmackenzie has quit IRC | 21:48 | |
| *** lachlanmackenzie has joined #baserock | 21:57 | |
| *** trn has joined #baserock | 22:21 | |
| *** zoli_ has joined #baserock | 22:36 | |
| *** cosm has quit IRC | 22:39 | |
| *** bfletcher has joined #baserock | 23:03 | |
| *** edcragg has quit IRC | 23:39 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!