*** gtristan has quit IRC | 02:02 | |
*** gtristan has joined #baserock | 02:37 | |
*** gtristan has quit IRC | 04:21 | |
*** gtristan has joined #baserock | 04:33 | |
*** gtristan has quit IRC | 06:53 | |
*** paulw has joined #baserock | 07:59 | |
paulw | morning | 07:59 |
---|---|---|
*** paulw has quit IRC | 07:59 | |
*** toscalix has joined #baserock | 08:25 | |
*** ctbruce has joined #baserock | 08:30 | |
*** gtristan has joined #baserock | 09:13 | |
*** rdale has joined #baserock | 09:27 | |
*** gtristan has quit IRC | 09:57 | |
*** jonathanmaw has joined #baserock | 09:58 | |
*** gtristan has joined #baserock | 10:10 | |
*** locallycompact has joined #baserock | 10:36 | |
*** CTtpollard has joined #baserock | 10:38 | |
locallycompact | jjardon, I had to rebase everything, can you reapprove https://gitlab.com/baserock/ybd/merge_requests | 11:30 |
jjardon | locallycompact: all of it? :) let me check | 11:30 |
locallycompact | I wish it would let you rebase without two approvals in the browser | 11:31 |
jjardon | gtristan: Hi! about https://gitlab.com/baserock/ybd/merge_requests/279 ; you have to resolve all the discussions to allow the branch to be merged | 12:05 |
gtristan | "resolve a discussion" | 12:15 |
gtristan | interesting | 12:16 |
gtristan | jjardon, resolved discussions | 12:17 |
* gtristan likes merge requests and pipelines, and dislikes bug tracking and discussion in gitlab - I'm concerned mostly that discussions on potentially multiple merge requests do not happen on the actual issue they intend to address | 12:18 | |
jjardon | gtristan: Add "Fixes #<issue number>" to the merge request and merge issue and merge request will be linked | 12:31 |
jjardon | locallycompact: I guess we will need a new release of ybd after https://gitlab.com/baserock/spec/merge_requests/3 is merged? | 12:32 |
locallycompact | Why? | 12:32 |
jjardon | locallycompact: mmm ybd already supports that spec change, so I guess it doesnt matter | 12:36 |
locallycompact | yup | 12:36 |
locallycompact | Why do we hardlink for one thing and copy for the other? Why not just copy regardless? https://gitlab.com/baserock/ybd/merge_requests/284/diffs#b9337e5d93d45463b7b6d3e30c2a792702cad348_83_84 | 12:41 |
*** lachlan has quit IRC | 12:42 | |
locallycompact | I will see if it makes a difference | 12:42 |
*** tiagogomes has quit IRC | 12:45 | |
*** tiagogomes has joined #baserock | 12:46 | |
*** lachlan_ has joined #baserock | 12:48 | |
*** gtristan has quit IRC | 13:15 | |
*** tardyp has quit IRC | 13:26 | |
*** tardyp has joined #baserock | 13:26 | |
locallycompact | here is some ci for the spec https://gitlab.com/baserock/spec/builds/7312246 | 14:15 |
locallycompact | we should fix these rogue unpetrify-ref not being strings | 14:15 |
locallycompact | https://gitlab.com/baserock/definitions/merge_requests/21 | 14:28 |
*** tiagogomes has quit IRC | 14:42 | |
*** tiagogomes has joined #baserock | 14:45 | |
jjardon | locallycompact: nice, seems its failing though? | 14:48 |
locallycompact | bc master definitions doesn't pass validation | 14:49 |
locallycompact | merge that other one | 14:49 |
locallycompact | https://gitlab.com/baserock/definitions/merge_requests/21 | 14:49 |
jjardon | cool | 14:49 |
*** zoli_ has left #baserock | 14:50 | |
locallycompact | probably needs to be calling the spec from definitions rather than the other way around, or both. at least is something | 14:50 |
jjardon | yeah; can you add the same check in definitions ci, please? probably before doing anything else | 14:52 |
locallycompact | that needs to be in first so the script is there | 14:52 |
locallycompact | where to run, first or last? | 14:53 |
locallycompact | first? | 14:53 |
jjardon | first | 14:53 |
jjardon | can anyone else review https://gitlab.com/baserock/spec/merge_requests/5 and https://gitlab.com/baserock/definitions/merge_requests/21 ? | 14:56 |
locallycompact | jjardon, done, but needs the branch name scrubbed before merge https://gitlab.com/baserock/definitions/commits/lc/validate-ci | 15:08 |
*** gtristan has joined #baserock | 15:08 | |
jjardon | locallycompact: done, send the MR please | 15:10 |
jjardon | sorry, this is still needed: https://gitlab.com/baserock/spec/merge_requests/5 | 15:13 |
*** jonathanmaw has quit IRC | 15:24 | |
*** jonathanmaw has joined #baserock | 15:29 | |
locallycompact | https://gitlab.com/baserock/definitions/merge_requests/22 | 15:30 |
jjardon | great, thanks! | 15:32 |
jjardon | definitions tested against the spec before merging; welcome to the future! :) | 15:32 |
locallycompact | it's almost like as if we have our shit together | 15:34 |
*** ctbruce has quit IRC | 15:57 | |
*** ctbruce has joined #baserock | 16:06 | |
*** lachlan_ has quit IRC | 16:07 | |
*** lachlan_ has joined #baserock | 16:08 | |
*** lachlan_ has quit IRC | 16:12 | |
*** 7ITAAX0NR has joined #baserock | 16:22 | |
locallycompact | jjardon, benbrown_ https://gitlab.com/baserock/spec/merge_requests/6 | 16:31 |
locallycompact | renders here http://baserock.gitlab.io/spec/ | 16:31 |
jjardon | cool, we have https://docs.baserock.org already though; does anyone know where is the source of it? | 16:34 |
jjardon | I think we can merge your branch, we can always move the other stuff later on | 16:38 |
benbrown_ | locallycompact: what magic is involved to get it to render from artifacts? | 16:38 |
benbrown_ | or is that not what it's doing | 16:38 |
pedroalvarez | the source of https://docs.baserock.org/ is the spec.git repo | 16:39 |
pedroalvarez | or was | 16:39 |
pedroalvarez | (now all the repos have been moved, but the infra hasn't been updated) | 16:39 |
benbrown_ | oh, 'pages:' | 16:39 |
locallycompact | benbrown_, yeah just that pages field | 16:39 |
locallycompact | magic thing | 16:39 |
locallycompact | I will see if it goes better | 16:39 |
benbrown_ | locallycompact: will that default to master only builds? | 16:39 |
locallycompact | not currently | 16:39 |
benbrown_ | locallycompact: in terms of rendering | 16:40 |
benbrown_ | or just whatever the most recent build produces | 16:40 |
benbrown_ | in which case, can has 'only:\nbaserock/spec'? | 16:40 |
locallycompact | only master? | 16:41 |
benbrown_ | or rather, 'only:\nmaster@baserock/spec' | 16:41 |
benbrown_ | yh | 16:41 |
benbrown_ | with a '-# | 16:41 |
locallycompact | yea | 16:41 |
locallycompact | jjardon, http://baserock.gitlab.io/spec/ | 16:59 |
jjardon | O_o, awesome! | 17:00 |
locallycompact | cool merge it https://gitlab.com/baserock/spec/merge_requests/6 | 17:06 |
locallycompact | I will fix all the information | 17:07 |
jjardon | +1 | 17:12 |
jjardon | needs one more approval though; benbrown_ ? | 17:12 |
benbrown_ | done | 17:13 |
jjardon | cheers | 17:13 |
benbrown_ | when accepting MR's (post approval), do we click 'remove source branch' | 17:13 |
benbrown_ | or leave it to the owner? | 17:13 |
locallycompact | what does your heart say | 17:14 |
benbrown_ | nuke all the stale branches | 17:14 |
locallycompact | then destroy | 17:15 |
jjardon | locallycompact: how can we change docs.baserock.org to point to http://baserock.gitlab.io/spec/ ? | 17:20 |
locallycompact | I don't know where that is hosted | 17:20 |
jjardon | maybe pedroalvarez knows? or gary_perkins ? | 17:24 |
pedroalvarez | you want to change the dns record to create a CNAME? Or maybe change the configuration in our haproxy to point there? or just continue hosting it? | 17:26 |
* jjardon doesnt understand pedroalvarez :) locallycompact what do we want? | 17:39 | |
locallycompact | koala bears | 17:39 |
locallycompact | what is any of that stuff | 17:40 |
pedroalvarez | will that url be accesible using a different url? | 17:40 |
locallycompact | ? | 17:40 |
pedroalvarez | does gitlab pages have CNAMEs support? | 17:41 |
pedroalvarez | I think it doesnt | 17:41 |
locallycompact | this doesn't mean anything to me sorry | 17:42 |
pedroalvarez | where is that spec page configured? | 17:42 |
locallycompact | https://gitlab.com/baserock/spec/blob/master/.gitlab-ci.yml#L13 | 17:43 |
pedroalvarez | I'd like to see https://gitlab.com/baserock/spec/pages | 17:44 |
pedroalvarez | 404 | 17:44 |
pedroalvarez | so, there, it looks like you can add a new domain? | 17:45 |
pedroalvarez | and some certificates I guess for TLS | 17:46 |
pedroalvarez | i'm not sure we would like to upload those there... | 17:46 |
pedroalvarez | (the certificates we use for baserock.org infra, I mean) | 17:47 |
pedroalvarez | but it looks optional | 17:47 |
pedroalvarez | can anyone give me access to that one? I guess is because i'm only user of the spec repo, and not owner or master, or whatever it's called | 17:49 |
locallycompact | jjardon, ^ | 17:49 |
pedroalvarez | I keep seeing this in gitlab docs: "Note: Custom CNAMEs with TLS support were introduced in GitLab EE 8.5." | 17:51 |
jjardon | pedroalvarez: what is you username in gitlab? | 17:51 |
pedroalvarez | palvarez89 iirc | 17:51 |
pedroalvarez | i'm in the baserock group | 17:51 |
jjardon | ok, got it | 17:52 |
jjardon | pedroalvarez: you should have superpowers now | 17:52 |
pedroalvarez | meh, gitlab suggests me to use an A record and a hardcoded IP | 17:54 |
*** ctbruce has quit IRC | 17:54 | |
pedroalvarez | http://spec.baserock.org/ | 18:01 |
locallycompact | sick | 18:01 |
locallycompact | pedroalvarez, thanks! | 18:02 |
pedroalvarez | (I haven't used docs.baserock.org just in case I broke things) | 18:02 |
pedroalvarez | It can be readjusted whenever we agree names etc | 18:02 |
jjardon | pedroalvarez: thanks! | 18:02 |
pedroalvarez | np | 18:02 |
*** jonathanmaw has quit IRC | 18:03 | |
*** locallycompact has quit IRC | 18:40 | |
*** rdale has quit IRC | 19:25 | |
*** locallycompact has joined #baserock | 19:35 | |
*** gtristan has quit IRC | 20:00 | |
*** 7ITAAX0NR has quit IRC | 20:20 | |
*** toscalix has quit IRC | 21:39 | |
*** toscalix has joined #baserock | 21:45 | |
*** toscalix has quit IRC | 21:45 | |
*** locallycompact has quit IRC | 23:11 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!