Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [eclipse-dev] [jdt-dev] Automate check to warn about freeze onGerrit

Thanks Mickael and Roland for taking this up. We were discussing the same last week.
 
We will need some refinement and least manual intervention as we go along but it's a great start.
 
Thanks & Regards,
Sarika
 
----- Original message -----
From: Mickael Istria <mistria@xxxxxxxxxx>
Sent by: eclipse-dev-bounces@xxxxxxxxxxx
To: carsten.hammer@xxxxxxxxxxx, "General development mailing list of the Eclipse project." <eclipse-dev@xxxxxxxxxxx>
Cc:
Subject: [EXTERNAL] Re: [eclipse-dev] [jdt-dev] Automate check to warn about freeze on Gerrit
Date: Tue, Dec 8, 2020 3:51 AM
 
 
 
On Mon, Dec 7, 2020 at 11:04 PM <carsten.hammer@xxxxxxxxxxx> wrote:
Hi Mickael,
Interesting idea but I think it is not the right point in time. I think block rules are there to reduce temporarily the rights to eg. push - so exactly when you try to merge is imho the best point in time to create a info message to tell about the reason for the unsuccessful merge.
https://gerrit-review.googlesource.com/Documentation/access-control.html#block
 
The thing is that we need committers to be allowed to ignore those messages and merge anyway. Some fixes may be necessary during the freeze period; and we cannot know in advance who's going to submit this fix to tweak the access rules in advance accordingly. This has to be a hint, not a blocker IMO.
Moreover, from this doc, I don't see how this is something that the Platform committers can easily play with. It seems deep inside Gerrit configuration. Also I don't see how we could have a rule to dynamically changing permissions.
 
I see no reason to annoy a user with warnings when he does not plan to merge anyway.
 
If a user doesn't plan to merge, they should probably use Gerrit Drafts or WIP changes, which are excluded from such checks.
 
At the freeze time window only some special users should have the right to merge that know about the freeze anyway.
 
That's not the reality of the project; we basically trust committers to do the right thing, and they agree with it when becoming committers. Here it's more about giving a hint to those committers, nothing about restraining who can do what.
 
However when it is just a info like what is possible implementing a gerrit plugin without putting a -1 on a gerrit it might be a nice feature.
 
So if this job were not voting but just printing a comment, it would be good enough in your opinion? That's totally doable in the job configuration...
_______________________________________________
eclipse-dev mailing list
eclipse-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/eclipse-dev
 


Back to the top