Skip to content

Remove call to isDefaultWorkspaceDir - #434

Merged
olamy merged 1 commit into
jenkinsci:masterfrom
MarkEWaite:remove-obsolete-isDefaultWorkspaceDir
Oct 6, 2026
Merged

olamy merged 1 commit into
jenkinsci:masterfrom
MarkEWaite:remove-obsolete-isDefaultWorkspaceDir

Conversation

@MarkEWaite

Copy link
Copy Markdown
Contributor

Remove call to isDefaultWorkspaceDir

Jenkins 2.584 removes the isDefaultWorkspaceDir method. This is the only reference found by GitHub search in all of jenkinsci.

Removed by pull request:

Detected by pull requests:

Upgrades the parent pom because otherwise the plugin cannot be compiled with Jenkins 2.583

Needs to be merged and released before the next plugin BOM release, otherwise it will block the upgrade to Jenkins 2.584.

Testing done

  • Confirmed compilation fails without this change:
    mvn -Djenkins.version=2.584-rc38753.cd27cb_2e2b_98 verify
  • Confirmed compilation succeeds and test passes with this change:
    mvn -Djenkins.version=2.584-rc38753.cd27cb_2e2b_98 -Dtest=JenkinsBuildsAndWorkspacesDirectoriesTest verify

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

Jenkins 2.584 removes the isDefaultWorkspaceDir method.  This is the
only reference found by GitHub search in all of jenkinsci.

Removed by pull request:

* jenkinsci/jenkins#27406

Detected by pull requests:

* jenkinsci/jenkins#27453
* jenkinsci/bom#7496

Upgrades the parent pom because otherwise the plugin cannot be compiled with Jenkins 2.583

Testing done:

* Confirmed compilation fails without this change:
  `mvn -Djenkins.version=2.584-rc38753.cd27cb_2e2b_98 verify`
* Confirmed compilation succeeds and test passes with this change:
  `mvn -Djenkins.version=2.584-rc38753.cd27cb_2e2b_98 -Dtest=JenkinsBuildsAndWorkspacesDirectoriesTest verify`
@MarkEWaite
MarkEWaite requested a review from a team as a code owner September 28, 2026 17:40
@MarkEWaite

MarkEWaite commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor Author

This needs to be labeled developer and merged before the end of this week so that it won't block the plugin BOM from adopting Jenkins 2.584.

@Vlatombe or @olamy could you review, merge, and release this pull request?

@jglick if one of those two are not available, could you help persuade one of the other maintainers to review and merge it?

MarkEWaite added a commit to jenkinsci/bom that referenced this pull request Sep 30, 2026
Jenkins 2.584 fails a Maven plugin test in the plugin BOM because the
test in the Maven plugin uses a method that was marked as restricted
and then was removed in Jenkins 2.584.

Rather than block plugin BOM updates to Jenkins 2.584, let's remove the
Maven plugin from the plugin BOM.

https://plugins.jenkins.io/maven-plugin/#plugin-content-risks advises
against using the plugin.  It has advised against using the plugin for
4 years.

The Maven plugin is not actively maintained.  Its health score continues
to decline.

Fix is proposed for the Maven plugin in pull request:

* jenkinsci/maven-plugin#434

Testing done:

* Confirmed that a plugin with a test dependency on Maven plugin passes tests:
  `LINE=weekly PLUGINS=s3 bash ./local-test.sh`
@jglick

jglick commented Oct 6, 2026

Copy link
Copy Markdown
Member

The last release was cut by Basil AFAICT, and there is no CD. Maybe @timja or someone else with org-wide powers is able to cut releases? Otherwise I guess you can exclude the test from the BOM, but in general if there is no one able or willing to release it then it should not be in the BOM any more.

@timja timja added the developer label Oct 6, 2026
@timja

timja commented Oct 6, 2026

Copy link
Copy Markdown
Member

@olamy / @Vlatombe

@MarkEWaite

Copy link
Copy Markdown
Contributor Author

I guess you can exclude the test from the BOM, but in general if there is no one able or willing to release it then it should not be in the BOM any more.

No worry. The maven plugin has been removed from the plugin BOM with pull request:

The pull request can wait indefinitely.

@olamy
olamy merged commit c638fe9 into jenkinsci:master Oct 6, 2026
17 checks passed
@olamy

olamy commented Oct 7, 2026

Copy link
Copy Markdown
Member

I have deployed a 3.28 release.
Always fun to do this manually. It remembers me when I was a cool kid 🤣

@MarkEWaite
MarkEWaite deleted the remove-obsolete-isDefaultWorkspaceDir branch October 7, 2026 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants