Skip to content

Commit

Permalink
[JENKINS-72252] Warn 12 months and 3 months before end of Java support (
Browse files Browse the repository at this point in the history
jenkinsci#8661)

Show Java version admin monitor at 12 months and 3 months

Daniel Beck described his recommendation to alert users at 12 months
and at 3 months prior to the end of support of a Java version.

He wrote:

> The second warning in particular needs to strike a balance between
> being shown late enough so it's actually relevant for whoever hasn't
> acted yet, while being shown early enough that slightly more elaborate
> environments (difficult to schedule maintenance windows) are informed
> in time. 3 months aligns rather nicely with the LTS schedule where
> we kinda expect folks to do that anyway.
>
> 18/9, or even 12/6 errs too far on the side of those for whom this is
> extreme effort (and who dismissed the first message more appropriate for
> their environment!), while showing everyone else completely irrelevant
> notices they won't care about for many months to come.

jenkinsci/jep#400 (comment) provides
more details.

The Java 8 to Java 11 transition saw a significant change in adoption of
Java 11 once the admin monitor was visible to users.  That was shown
slightly over 12 months before the release that required Java 11.
This change continues that pattern of 12 months warning before end
of support.

jenkinsci/jep#400 (comment) has a
graph that shows the adoption curves for Java 8, Java 11, and Java 17.
  • Loading branch information
MarkEWaite authored Nov 5, 2023
1 parent d329577 commit aeb64c0
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ private static String getId() {

@Override
public boolean isActivated() {
return !disabled && getDeprecationPeriod().toTotalMonths() < 18;
return !disabled && getDeprecationPeriod().toTotalMonths() < 12;
}

@Override
Expand Down Expand Up @@ -156,7 +156,7 @@ private static Period getDeprecationPeriod() {

@NonNull
private static Severity getSeverity() {
return getDeprecationPeriod().toTotalMonths() < 9 ? Severity.DANGER : Severity.WARNING;
return getDeprecationPeriod().toTotalMonths() < 3 ? Severity.DANGER : Severity.WARNING;
}

/**
Expand Down

0 comments on commit aeb64c0

Please sign in to comment.