Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove jCIFS and j-Interop #8523

Merged
merged 6 commits into from
Jan 19, 2025
Merged

Conversation

jtnord
Copy link
Member

@jtnord jtnord commented Sep 25, 2023

jinterop existed soley so we can install Jenkins as a service via the Jenkins UI.

This functionality should not be used, and the windows installer should be used instead.

Removed the jcifs library as it is outdated, no longer maintained and was used only to support launching windows agents via DCOM which no longer works with supported windows versions.

TODO

See https://groups.google.com/g/jenkinsci-dev/c/Hj-p1QCntw8

Testing done

Proposed changelog entries

  • Remove the ability to install as a Windows service from a running Jenkins install. Users wanting this functionality should instead download and install the MSI package.

Proposed upgrade guidelines

N/A

Submitter checklist

Preview Give feedback

Desired reviewers

@mention

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

Maintainer checklist

Preview Give feedback

jinterop existed soley so we can install Jenkins as a service via the
Jenkins UI.

This functionality should not be used, and the windows installer should
be used instead.

Removed the jcifs library as it is outdated, no longer maintained and
was used only to support launching windows agents via DCOM which no
longer works with supported windows versions.
@KalleOlaviNiemitalo
Copy link

Perhaps additional things to remove:

  • core/src/main/resources/hudson/lifecycle/WindowsInstallerLink/*.properties
  • core/pom.xml portion copying jenkins.exe from winsw, unless that is still needed for the MSI package

    jenkins/core/pom.xml

    Lines 638 to 646 in c0f6fbe

    <artifactItem>
    <groupId>com.sun.winsw</groupId>
    <artifactId>winsw</artifactId>
    <version>2.9.0</version>
    <classifier>net4</classifier>
    <type>exe</type>
    <outputDirectory>${project.build.outputDirectory}/windows-service</outputDirectory>
    <destFileName>jenkins.exe</destFileName>
    </artifactItem>

@jtnord
Copy link
Member Author

jtnord commented Sep 27, 2023

Perhaps additional things to remove:

  • core/src/main/resources/hudson/lifecycle/WindowsInstallerLink/*.properties

👍

  • core/pom.xml portion copying jenkins.exe from winsw, unless that is still needed for the MSI package

it is used by the WindowsServiceLifecyle to update the exe if Jenkins is updated.
Whilst I question the ability to update Jenkins from within Jenkins if you installed with the MSI (to me if you install with a package you should always upgrade with the package) the functionality does not rely on jcifs or j-interpo so I would say that should live in another PR. Perhaps this is a semi left over from the ability that you could create a service outside the MSI, however it does support more than that.

@abhishekmaity
Copy link
Contributor

/label work-in-progress

@comment-ops-bot comment-ops-bot bot added the work-in-progress The PR is under active development, not ready to the final review label Sep 28, 2023
@github-actions github-actions bot added the unresolved-merge-conflict There is a merge conflict with the target branch. label Apr 22, 2024
Copy link
Contributor

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

@github-actions github-actions bot removed the unresolved-merge-conflict There is a merge conflict with the target branch. label Apr 22, 2024
@jtnord jtnord added the removed This PR removes a feature or a public API label Apr 23, 2024
@jtnord jtnord self-assigned this May 31, 2024
@github-actions github-actions bot added the unresolved-merge-conflict There is a merge conflict with the target branch. label Aug 14, 2024
Copy link
Contributor

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

@github-actions github-actions bot removed the unresolved-merge-conflict There is a merge conflict with the target branch. label Aug 14, 2024
@github-actions github-actions bot added the unresolved-merge-conflict There is a merge conflict with the target branch. label Sep 3, 2024
Copy link
Contributor

github-actions bot commented Sep 3, 2024

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

@github-actions github-actions bot removed the unresolved-merge-conflict There is a merge conflict with the target branch. label Jan 18, 2025
basil added a commit to basil/bom that referenced this pull request Jan 18, 2025
basil added a commit to basil/acceptance-test-harness that referenced this pull request Jan 18, 2025
@basil basil removed the work-in-progress The PR is under active development, not ready to the final review label Jan 18, 2025
@basil basil added the ath-successful This PR has successfully passed the full acceptance-test-harness suite label Jan 18, 2025
@basil basil marked this pull request as ready for review January 18, 2025 17:33
Copy link
Member

@basil basil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only consumer that still remains relevant is publish-over-cifs, but that publishes its own copy of jcifs-ng that should be unaffected by this removal. (The WMI Windows Agents plugin repository has been archived since last August.)


This PR is now ready for merge. We will merge it after approximately 24 hours if there is no negative feedback. Please see the merge process documentation for more information about the merge process. Thanks!

@basil basil added the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label Jan 18, 2025
@basil basil changed the title remove jcifs and j-interop Remove jcifs and j-interop Jan 18, 2025
@basil basil changed the title Remove jcifs and j-interop Remove JCIFS and j-Interop Jan 18, 2025
@basil basil changed the title Remove JCIFS and j-Interop Remove jCIFS and j-Interop Jan 18, 2025
@basil basil added the pct-successful This PR has successfully passed the full plugin-compatibility-test suite label Jan 18, 2025
@basil basil merged commit 40ae390 into jenkinsci:master Jan 19, 2025
16 checks passed
@jtnord jtnord deleted the remove-jcifs-and-jinterop branch January 20, 2025 18:28
@MarkEWaite MarkEWaite added the upgrade-guide-needed This changes might be breaking in rare circumstances, an entry in the LTS upgrade guide is needed label Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ath-successful This PR has successfully passed the full acceptance-test-harness suite pct-successful This PR has successfully passed the full plugin-compatibility-test suite ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback removed This PR removes a feature or a public API upgrade-guide-needed This changes might be breaking in rare circumstances, an entry in the LTS upgrade guide is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants