Saturday, 2021-05-22

openstackgerritClark Boylan proposed opendev/system-config master: Cleanup puppet things from zuul where we don't puppet anymore
openstackgerritClark Boylan proposed opendev/system-config master: Remove system-config-legacy-logstash-filters job
*** DSpider has quit IRC02:35
*** _dvd is now known as dvd03:15
*** dvd is now known as _dvd03:16
*** elod has quit IRC06:11
*** elod has joined #opendev06:13
*** ykarel|away has joined #opendev06:13
*** ykarel|away has quit IRC06:13
*** ykarel|away has joined #opendev06:14
*** ricolin has quit IRC06:50
*** ykarel|away has quit IRC08:10
*** dtantsur has joined #opendev08:23
*** jpenag has joined #opendev08:29
*** CeeMac has quit IRC08:30
*** parallax has quit IRC08:30
*** jrosser has quit IRC08:30
*** paladox has quit IRC08:30
*** jpena|off has quit IRC08:30
*** dtantsur|afk has quit IRC08:30
*** StevenK has quit IRC08:30
*** JayF has quit IRC08:30
*** CeeMac has joined #opendev08:31
*** parallax has joined #opendev08:31
*** paladox has joined #opendev08:31
*** jrosser has joined #opendev08:31
*** StevenK has joined #opendev08:31
*** JayF has joined #opendev08:31
*** gtema has quit IRC08:34
*** dirk has quit IRC08:35
*** reed has quit IRC08:39
*** mordred has quit IRC08:39
*** gtema has joined #opendev08:47
*** dirk has joined #opendev08:49
*** mordred has joined #opendev09:35
*** reed has joined #opendev09:40
*** DSpider has joined #opendev10:34
*** tosky has joined #opendev11:05
*** ykarel|away has joined #opendev11:33
*** slaweq has joined #opendev11:35
*** slaweq has quit IRC12:20
*** ykarel|away has quit IRC12:24
*** ykarel|away has joined #opendev12:25
corvusfungi, clarkb: apparently normal users in gerrit can remove reviewers from changes, which also removes their votes?
fungicorvus: aha, more specifically the "change owner" at least seems to be able to (so not exactly "normal users"). i wonder if this is an implicit change owner permission we have to block13:44
corvus(though when i replied to the change, my vote automatically re-appeared; i manually removed it again)13:44
corvusfungi: yes, correct, sorry13:44
corvusmy first irc transmission after waking up :)13:44
fungithanks for spotting that, i'll try to take a look at the acl docs in a bit once i get a break from the gardening chores13:45
corvusi'm very curious what the user's reasoning is (i've asked)13:48
corvusit's also interesting that they did not remove zuul (which had a -2) as a reviewer13:51
corvusperhaps that's what they were trying to do, but could not because of the -2, and removed us in an attempt to understand why removing zuul wasn't working13:52
*** tosky has quit IRC14:08
*** ykarel|away has quit IRC14:50
*** slaweq has joined #opendev15:05
*** slaweq has quit IRC15:13
*** timburke has quit IRC16:09
*** dmsimard has quit IRC16:38
fungicorvus: sorry, ended up spending longer outside than i anticipated but the todo list grew while i was there... anyway, looks like it's because the votes were positive, and i get the impression from the docs we can't change that, at least not in 3.2: "Change owners can always remove reviewers who have given a zero or positive score (even without having the Remove Reviewer access right assigned)."17:26
fungiso the good news is that if someone pushes a change and you -1 it, they can't delete that (in theory anyway)17:27
fungiworth taking up with upstream gerrit if a new access control to restrict change owners from removing zero or positive votes is desirable, i guess17:29
*** tosky has joined #opendev19:25
*** sshnaidm|pto is now known as sshnaidm20:48
*** dmsimard has joined #opendev20:50
*** tosky has quit IRC23:13
*** DSpider has quit IRC23:45

Generated by 2.17.2 by Marius Gedminas - find it at!