gitlab-br-bot | push on infrastructure@master (by Pedro Alvarez Piedehierro): 2 commits (last: baserock_bots: configure Gitlab bot for Baserock too) https://gitlab.com/baserock/infrastructure/commit/65bf07f0fe6ec48dc8425c3d42813c18ded31d92 | 00:02 |
---|---|---|
gitlab-br-bot | infrastructure: merge request (pedro/gitlab-bot-multichannel->master: baserock_bots: configure Gitlab bot for Baserock too) #2 changed state ("merged"): https://gitlab.com/baserock/infrastructure/merge_requests/2 | 00:02 |
gitlab-br-bot | infrastructure: Pedro Alvarez Piedehierro deleted branch pedro/gitlab-bot-multichannel | 00:02 |
gitlab-br-bot | push on definitions@jjardon/bst2 (by Javier Jardón): 1 commit (last: Add debug) https://gitlab.com/baserock/definitions/commit/d2334bf036075fdd84fa6fa00482343323d06a32 | 00:12 |
gitlab-br-bot | push on definitions@jjardon/bst2 (by Javier Jardón): 1 commit (last: Set cache folder) https://gitlab.com/baserock/definitions/commit/8206a69214b7d16f468f4f41446dd15e3c46a7c9 | 00:47 |
*** gtristan has quit IRC | 04:07 | |
*** toscalix has joined #baserock | 07:20 | |
*** bruce has joined #baserock | 07:46 | |
*** bruce is now known as Guest70876 | 07:46 | |
*** paulwaters_ has joined #baserock | 07:57 | |
*** gtristan has joined #baserock | 08:01 | |
*** gtristan has quit IRC | 08:10 | |
*** gtristan has joined #baserock | 08:15 | |
*** jonathanmaw has joined #baserock | 08:38 | |
*** ssam2 has joined #baserock | 08:45 | |
*** ChanServ sets mode: +v ssam2 | 08:45 | |
*** jonathanmaw has quit IRC | 08:55 | |
*** tiagogomes has joined #baserock | 08:59 | |
*** jonathanmaw has joined #baserock | 09:04 | |
*** jonathanmaw_ has joined #baserock | 09:05 | |
* jjardon updates https://gitlab.com/baserock/definitions/issues/9 | 09:28 | |
jjardon | ironfoot: can you change the bot ot not show every push? I think notification for MR and issues is enough | 09:29 |
gtristan | jjardon, I think we can case it to only mention commits on master, for instance | 09:30 |
gtristan | whether they are MRs or whatever | 09:30 |
ironfoot | gtristan: I think I agree with jjardon, only MR and issues. | 09:30 |
ironfoot | I thinik than filtering pushes to master is going to be tricky | 09:30 |
gtristan | what is the difference ? | 09:30 |
gtristan | difficult really ? ok I thought the bot already made the string | 09:31 |
gtristan | from project@branch | 09:31 |
* ironfoot -> meeting | 09:31 | |
gtristan | anyway note that the commit notifications are only once per push/merge (not one per commit in a pushed branch) | 09:32 |
gtristan | so I think it's equivalent, in the case that you only accept pushes to master through MRs | 09:33 |
gtristan | ironfoot, anyway btw, I will modify the bot to support that, cause I want the same functionality for the gitlab-bst bot | 09:44 |
ironfoot | gtristan: note, I've been hacking over the weekend on this bot, and have already done some modifications, so that it supports different networks/projects/channels | 09:53 |
gtristan | :) | 09:54 |
gtristan | ironfoot, is there a git repo ? | 09:54 |
ironfoot | I suggested only getting MR notifications given that: | 09:55 |
ironfoot | - every push goes through MR | 09:55 |
ironfoot | - when MR is merged it will trigger also a push event | 09:55 |
ironfoot | - when MR is merged it will trigger a push to delete branch | 09:55 |
ironfoot | gtristan: https://gitlab.com/palvarez89/gitlab-to-irc | 09:55 |
gtristan | cool | 09:56 |
gtristan | Yeah I get that, but "every push goes through MR" is the key thing why it works for baserock | 09:56 |
gtristan | so for buildstream I would prefer just notifying everything that lands on [branch, list] | 09:57 |
ironfoot | gtristan: so, here is the current config for the bot: https://gitlab.com/baserock/infrastructure/blob/master/baserock_bots/config.js | 09:58 |
gtristan | yup, I can see the changes you made :) | 09:59 |
ironfoot | i'm *not* proud, took me a while to understand the difficulty of the problem | 10:00 |
ironfoot | i had fun, though :) | 10:00 |
SotK | huh, is https://gitlab.com/baserock/infrastructure private? | 10:02 |
ironfoot | eeerm.. it shouldnt :/ | 10:03 |
SotK | I get "You need to sign in or sign up before continuing." | 10:04 |
ironfoot | jjardon: ^^ still set to internal | 10:04 |
*** locallycompact has joined #baserock | 11:35 | |
*** gtristan has quit IRC | 13:48 | |
*** gtristan has joined #baserock | 14:10 | |
jjardon | ironfoot: sorry, forgot to click the "save" button | 15:16 |
jjardon | SotK: try again please | 15:16 |
SotK | I can see it now :) | 15:17 |
ironfoot | :) | 15:17 |
jjardon | paulsher1ood: would a ybd patch like this accepted upstream: http://paste.baserock.org/qulujeyejo ? | 15:17 |
SotK | thanks jjardon | 15:17 |
ironfoot | I believe that gitlab project configuration can be managed from Ansible, that might help us a lot to have all configured | 15:18 |
*** Guest70876 has quit IRC | 15:25 | |
*** bruce has joined #baserock | 15:25 | |
*** bruce is now known as Guest29591 | 15:26 | |
*** jonathanmaw_ has quit IRC | 15:30 | |
*** jonathanmaw has quit IRC | 15:30 | |
*** Guest29591 is now known as ctbruce | 15:32 | |
paulsherwood | jjardon: yes, i don't see why not | 15:35 |
jjardon | ironfoot: I looked around and seems its not possible to set you as owner; if you find out let me know | 15:39 |
jjardon | ironfoot: that would be really cool | 15:40 |
* jjardon creates https://gitlab.com/baserock/ybd/merge_requests/358 | 15:45 | |
* paulsherwood approves it | 15:46 | |
*** toscalix has quit IRC | 15:47 | |
*** ctbruce has quit IRC | 16:17 | |
*** paulwaters_ has joined #baserock | 16:47 | |
*** ssam2_ has joined #baserock | 16:59 | |
*** ssam2 has quit IRC | 16:59 | |
*** paulwaters_ has quit IRC | 17:04 | |
*** locallycompact has quit IRC | 17:52 | |
*** ssam2_ has quit IRC | 18:14 | |
*** gitlab-br-bot has quit IRC | 18:27 | |
*** gitlab-br-bot has joined #baserock | 18:27 | |
gitlab-br-bot | infrastructure: merge request (pedro/no-push-notifications->master: Pedro/no push notifications) #3 changed state ("opened"): https://gitlab.com/baserock/infrastructure/merge_requests/3 | 18:30 |
gitlab-br-bot | infrastructure: merge request (pedro/no-push-notifications->master: Pedro/no push notifications) #3 changed state ("merged"): https://gitlab.com/baserock/infrastructure/merge_requests/3 | 18:32 |
ironfoot | push notifications removed | 18:32 |
*** gtristan has quit IRC | 19:33 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!