Introduce `HistoricalBuild` interface #9674

Vlatombe · 2024-08-28T08:23:46Z

This allows to display Run-like objects in build history without having concrete Run present in the Jenkins instance.

Related soft downstream Whitelist method moved from Run#getFullDisplayName to HistoricalBuild#getFullDisplayName script-security-plugin#576

See JENKINS-XXXXX.

Testing done

Proposed changelog entries

N/A

Proposed upgrade guidelines

N/A

Submitter checklist

Give feedback

The Jira issue, if it exists, is well-described.
The changelog entries and upgrade guidelines are appropriate for the audience affected by the change (users or developers, depending on the change) and are in the imperative mood (see examples). Fill in the Proposed upgrade guidelines section only if there are breaking changes or changes that may require extra steps from users during upgrade.
There is automated testing or an explanation as to why this change has no tests.
New public classes, fields, and methods are annotated with @Restricted or have @since TODO Javadocs, as appropriate.
New deprecations are annotated with @Deprecated(since = "TODO") or @Deprecated(forRemoval = true, since = "TODO"), if applicable.
New or substantially changed JavaScript is not defined inline and does not call eval to ease future introduction of Content Security Policy (CSP) directives (see documentation).
For dependency updates, there are links to external changelogs and, if possible, full differentials.
For new APIs and extension points, there is a link to at least one consumer.
Options

Desired reviewers

@mention

Before the changes are marked as ready-for-merge:

Maintainer checklist

Give feedback

There are at least two (2) approvals for the pull request and no outstanding requests for change.
Conversations in the pull request are over, or it is explicit that a reviewer is not blocking the change.
Changelog entries in the pull request title and/or Proposed changelog entries are accurate, human-readable, and in the imperative mood.
Proper changelog labels are set so that the changelog can be generated automatically.
If the change needs additional upgrade steps from users, the upgrade-guide-needed label is set and there is a Proposed upgrade guidelines section in the pull request title (see example).
If it would make sense to backport the change to LTS, a Jira issue must exist, be a Bug or Improvement, and be labeled as lts-candidate to be considered (see query).
Options

This allows to display Run-like objects in build history without having concrete Run present in the Jenkins instance.

Vlatombe · 2024-08-28T08:24:19Z

core/src/main/java/hudson/model/Run.java

-     * @deprecated truncated description based on the {@link #TRUNCATED_DESCRIPTION_LIMIT} setting.
-     */
-    @Deprecated
-    public @CheckForNull String getTruncatedDescription() {


Moved to HistoricalBuild

Vlatombe · 2024-08-28T08:24:33Z

core/src/main/java/hudson/model/Run.java

-    public @NonNull String getBuildStatusUrl() {
-        return getIconColor().getImage();
-    }
-
-    public String getBuildStatusIconClassName() {
-        return getIconColor().getIconClassName();
-    }


Moved to HistoricalBuild

Vlatombe · 2024-08-28T08:36:02Z

core/src/main/java/hudson/model/Run.java

-    /**
-     * Target size limit for truncated {@link #description}s in the Build History Widget.
-     * This is applied to the raw, unformatted description. Especially complex formatting
-     * like hyperlinks can result in much less text being shown than this might imply.
-     * Negative values will disable truncation, {@code 0} will enforce empty strings.
-     * @since 2.223
-     */
-    private static /* non-final for Groovy */ int TRUNCATED_DESCRIPTION_LIMIT = SystemProperties.getInteger("historyWidget.descriptionLimit", 100);


Inlined in HistoricalBuild#getTruncatedDescription

Vlatombe · 2024-08-28T08:41:23Z

core/src/main/resources/hudson/widgets/BuildHistoryWidget/entries.jelly

This file was actually dead code since 078d3c7

jglick · 2024-08-28T12:49:19Z

core/src/main/resources/hudson/widgets/BuildHistoryWidget/entries.jelly

jglick · 2024-08-28T12:51:02Z

core/src/main/java/hudson/model/Run.java

    /**
     * Gets the subset of {@link #getActions()} that consists of {@link BuildBadgeAction}s.
     */
+    @Override


Should Javadoc be pulled up then?

Since getActions() doesn't exist on HistoricalBuild, I left it there.

jglick · 2024-08-28T12:51:38Z

core/src/main/java/hudson/model/Run.java

-     * @return
-     *      String like "job/foo/32/" with trailing slash but no leading slash.
-     */
-    // I really messed this up. I'm hoping to fix this some time


lol that ship has sailed

core/src/main/java/jenkins/widgets/HistoryPageFilter.java

Co-authored-by: Jesse Glick <[email protected]>

github-actions · 2024-09-03T22:01:37Z

Please take a moment and address the merge conflicts of your pull request. Thanks!

Vlatombe · 2024-09-09T14:58:45Z

/label ready-for-merge

This PR is now ready for merge, after ~24 hours, we will merge it if there's no negative feedback.

Thanks!

@dwnusbaum

commit c5128f4 Author: Jan Faracik <[email protected]> Date: Thu Sep 12 14:39:02 2024 +0100 Use notice for views lacking jobs commit bb7a30a Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Date: Thu Sep 12 09:02:10 2024 +0100 Update dependency org.jenkins-ci.plugins:script-security to v1361 (jenkinsci#9719) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> commit 5fc92c6 Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Date: Thu Sep 12 09:01:59 2024 +0100 Update jenkins/ath Docker tag to v5992 (jenkinsci#9720) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> commit 36c155e Author: Alexander Brandes <[email protected]> Date: Thu Sep 12 10:01:35 2024 +0200 Automate the since updater (jenkinsci#7240) Co-authored-by: Tim Jacomb <[email protected]> commit 65f374c Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Date: Wed Sep 11 05:13:32 2024 -0600 Update dependency org.jenkins-ci.plugins:junit to v1300 (jenkinsci#9716) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> commit 35d7677 Author: Vincent Latombe <[email protected]> Date: Wed Sep 11 04:32:36 2024 +0200 Introduce `HistoricalBuild` interface (jenkinsci#9674) * Introduce HistoricalBuild interface This allows to display Run-like objects in build history without having concrete Run present in the Jenkins instance. * Remove BuildHistoryWidget#entries.jelly * JDK17 level Co-authored-by: Jesse Glick <[email protected]> --------- Co-authored-by: Jesse Glick <[email protected]> commit 78e132d Author: Basil Crow <[email protected]> Date: Tue Sep 10 19:32:04 2024 -0700 Build tests without forking the compiler (jenkinsci#9717) commit 66a7810 Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Date: Tue Sep 10 20:31:40 2024 -0600 Update dependency org.jenkins-ci.plugins:cloudbees-folder to v6.951.v5f91d88d76b_b_ (jenkinsci#9715) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> commit 53f3203 Author: Daniel Beck <[email protected]> Date: Tue Sep 10 22:52:20 2024 +0200 [JENKINS-73709] Build without forking the compiler (jenkinsci#9709) Co-authored-by: Daniel Beck <[email protected]> commit 905df3d Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Date: Tue Sep 10 09:40:46 2024 -0700 Update dependency org.jenkins-ci.plugins:junit to v1299 (jenkinsci#9714) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> commit 11c405b Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Date: Tue Sep 10 09:40:08 2024 -0700 Update dependency org.apache.maven.plugins:maven-jarsigner-plugin to v3.1.0 (jenkinsci#9712) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> commit eb6be17 Author: Jenkins Release Bot <[email protected]> Date: Tue Sep 10 14:10:20 2024 +0000 [maven-release-plugin] prepare for next development iteration commit 561231f Author: Jenkins Release Bot <[email protected]> Date: Tue Sep 10 14:09:56 2024 +0000 [maven-release-plugin] prepare release jenkins-2.476 commit 0c49380 Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Date: Mon Sep 9 16:47:09 2024 -0700 Update eslint monorepo to v9.10.0 (jenkinsci#9710) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> commit 1b8c198 Author: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Date: Mon Sep 9 11:19:46 2024 -0700 Update dependency io.jenkins.plugins:font-awesome-api to v6.6.0-2 (jenkinsci#9708) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> commit 85d6358 Author: Basil Crow <[email protected]> Date: Mon Sep 9 08:14:48 2024 -0700 Compatibility for `ChainedServletFilter` (jenkinsci#9696) commit 4334aa0 Author: Juan Jesús Alejo Sillero <[email protected]> Date: Mon Sep 9 17:13:51 2024 +0200 fix(i18n): correct typo in Spanish translation for 'Preparation' (jenkinsci#9702) commit 2431b7c Author: Markus Winter <[email protected]> Date: Mon Sep 9 17:13:20 2024 +0200 [JENKINS-72988] validate displayname against items in the same ItemGroup (jenkinsci#9152) Co-authored-by: Kris Stern <[email protected]> commit bfcb874 Merge: b56e5d7 2f6d77c Author: Vincent Latombe <[email protected]> Date: Mon Sep 9 16:57:03 2024 +0200 Merge pull request jenkinsci#9645 from Vlatombe/with-console-url commit b56e5d7 Merge: c05d756 6b88bfc Author: Vincent Latombe <[email protected]> Date: Mon Sep 9 11:21:50 2024 +0200 Merge pull request jenkinsci#9673 from Vlatombe/computerListener.onIdle commit 2f6d77c Merge: 49618a0 307f680 Author: Vincent Latombe <[email protected]> Date: Wed Sep 4 10:48:53 2024 +0200 Merge branch 'master' into with-console-url commit 6b88bfc Author: Vincent Latombe <[email protected]> Date: Wed Aug 28 16:22:14 2024 +0200 Spotless commit ed93ad0 Author: Vincent Latombe <[email protected]> Date: Wed Aug 28 16:12:09 2024 +0200 Revert "Call listener synchronously, but outside synchronized" This reverts commit 75242cf. commit 75242cf Author: Vincent Latombe <[email protected]> Date: Wed Aug 28 15:47:51 2024 +0200 Call listener synchronously, but outside synchronized commit 9823b8e Author: Vincent Latombe <[email protected]> Date: Wed Aug 28 15:40:43 2024 +0200 Execute ComputerListener#onIdle in another thread. commit 64667af Author: Vincent Latombe <[email protected]> Date: Wed Aug 28 09:54:25 2024 +0200 Implement ComputerListener#onIdle Introduces a new computer listener fired whenever a Computer becomes idle. commit 49618a0 Author: Vincent Latombe <[email protected]> Date: Fri Aug 23 08:14:22 2024 +0200 Updating the test to note the behavioural change commit 2d30cfc Author: Vincent Latombe <[email protected]> Date: Thu Aug 22 17:20:38 2024 +0200 Fix review commit 44602bb Author: Vincent Latombe <[email protected]> Date: Thu Aug 22 17:12:59 2024 +0200 Typo spotted by @dwnusbaum Co-authored-by: Devin Nusbaum <[email protected]> commit 39d30c8 Author: Vincent Latombe <[email protected]> Date: Tue Aug 13 15:04:12 2024 +0200 Add WithConsoleUrl interface This simplifies the logic of `Functions#getConsoleUrl` and makes it pluggable.

Introduce HistoricalBuild interface

91dc9fa

This allows to display Run-like objects in build history without having concrete Run present in the Jenkins instance.

Vlatombe added the developer Changes which impact plugin developers label Aug 28, 2024

Vlatombe requested a review from jglick August 28, 2024 08:23

Vlatombe commented Aug 28, 2024

View reviewed changes

Vlatombe marked this pull request as draft August 28, 2024 08:25

Remove BuildHistoryWidget#entries.jelly

54ba6cd

Vlatombe commented Aug 28, 2024

View reviewed changes

Vlatombe mentioned this pull request Aug 28, 2024

Whitelist method moved from Run#getFullDisplayName to HistoricalBuild#getFullDisplayName jenkinsci/script-security-plugin#576

Merged

6 tasks

Vlatombe added the skip-changelog Should not be shown in the changelog label Aug 28, 2024

Vlatombe marked this pull request as ready for review August 28, 2024 09:44

Vlatombe requested a review from a team August 28, 2024 11:06

jglick approved these changes Aug 28, 2024

View reviewed changes

JDK17 level

e45a7d3

Co-authored-by: Jesse Glick <[email protected]>

alecharp approved these changes Aug 30, 2024

View reviewed changes

github-actions bot added the unresolved-merge-conflict There is a merge conflict with the target branch. label Sep 3, 2024

Merge branch 'master' into historicalbuild

63647d7

github-actions bot removed the unresolved-merge-conflict There is a merge conflict with the target branch. label Sep 5, 2024

Merge branch 'master' into historicalbuild

7b28457

comment-ops-bot bot added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label Sep 9, 2024

jglick approved these changes Sep 9, 2024

View reviewed changes

MarkEWaite merged commit 35d7677 into jenkinsci:master Sep 11, 2024
16 checks passed

Vlatombe deleted the historicalbuild branch September 11, 2024 08:42

MarkEWaite mentioned this pull request Sep 16, 2024

Fix JenkinsWhitelistTest on 2.477 incremental jenkinsci/script-security-plugin#578

Merged

6 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Introduce `HistoricalBuild` interface #9674

Introduce `HistoricalBuild` interface #9674

Vlatombe commented Aug 28, 2024 •

edited

Loading

Submitter checklist

Maintainer checklist

Vlatombe Aug 28, 2024

Vlatombe Aug 28, 2024

Vlatombe Aug 28, 2024

Vlatombe Aug 28, 2024 •

edited

Loading

jglick Aug 28, 2024

jglick Aug 28, 2024

jglick Aug 28, 2024

Vlatombe Aug 28, 2024

jglick Aug 28, 2024

github-actions bot commented Sep 3, 2024

Vlatombe commented Sep 9, 2024

Introduce HistoricalBuild interface #9674

Introduce HistoricalBuild interface #9674

Conversation

Vlatombe commented Aug 28, 2024 • edited Loading

Testing done

Proposed changelog entries

Proposed upgrade guidelines

Submitter checklist

Desired reviewers

Maintainer checklist

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Vlatombe Aug 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Sep 3, 2024

Vlatombe commented Sep 9, 2024

Introduce `HistoricalBuild` interface #9674

Introduce `HistoricalBuild` interface #9674

Vlatombe commented Aug 28, 2024 •

edited

Loading

Vlatombe Aug 28, 2024 •

edited

Loading