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 on Gerrit

On 14 Dec 2020, at 08:48, Mickael Istria <mistria@xxxxxxxxxx> wrote:



On Mon, Dec 14, 2020 at 9:36 AM Sravan K Lakkimsetti <sravankumarl@xxxxxxxxxx> wrote:

Yes That is correct. In my opinion getting a +1 from platform bot before the actual gerrit job is confusing. Is it possible not to give +1 but give -1 in case of freeze period?


OK, I'm changing that.

I saw a comment on a recent change:


Build Started https://ci.eclipse.org/platform/job/Report-freeze-dates-to-Gerrit/148/ Checking whether Eclipse project is currently in stabilitzation/code-freeze period
Checking whether Eclipse project is currently in stabilitzation/code-freeze period
hecking whether Eclipse project is currently in stabilitzation/code-freeze period

Build Successful
https://ci.eclipse.org/platform/job/Report-freeze-dates-to-Gerrit/148/ : Eclipse project is currently not in a code-freeze period. You can merge today if the change meets all other merge criteria.

Can we have the bot not comment if we’re not in a change?
https://ci.eclipse.org/platform/job/Report-freeze-dates-to-Gerrit/148/ : Eclipse project is currently not in a code-freeze period. You can merge today if the change meets all other merge criteria.

Can we have the bot not comment if we’re not in a change freeze, which will be most of the time? We don’t really need to know that the ‘check freeze’ bot has started, nor if it isn’t a freeze … I’d argue that we only really need the comment to be added iff we’re in a freeze.

Alex

Back to the top