-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
[JENKINS-66930] remove unsafe classes copied from Apache Ant which have been deprecated in Oct 2019 #5834
[JENKINS-66930] remove unsafe classes copied from Apache Ant which have been deprecated in Oct 2019 #5834
Conversation
…ve been deprecated in Oct 2019 Signed-off-by: Olivier Lamy <[email protected]>
I tried to remove that before, as you can see here: #4272 There are still two plugins which use it: The fstrigger got merged and is already published. We should put this to the upgrade guide. I don't know how often the copy to slave plugin is used. |
Ignore, distribution is suspended (and has been for a long time) so we can pretend it doesn't exist. |
jenkinsci/docker-slaves-plugin@71ed3c8 remains unreleased.
Based on what data is this asserted? Please show your work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am -1 for the change in the current state. hudson.org.apache.tools.tar.TarInputStream
is used in https://github.com/jenkinsci/copy-to-slave-plugin/blob/c0ea34f47941b1647fd05ea31eb69779ce53fab3/src/main/java/com/michelin/cio/hudson/plugins/copytoslave/MyFilePath.java. Even if the class is deprecated, I believe we need to check and cleanup usages in actively used plugins.
https://plugins.jenkins.io/copy-to-slave/ The plugin hasn't been distributed for 3.5 years. |
Shall we archive the github repo? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hudson.org.apache.tools.tar.TarOutputStream
is still used in the currently released version of Docker Slaves. I don't feel strongly that this plugin should remain supported, but if we do decide to drop support for it I think this should be mentioned in the release notes and a deprecation notice added to the plugin in the update center.
I know we only need someone to release docker slave plugin. |
I am not the maintainer of the docker-slaves-plugin. Nevertheless, I would also like to see my cleanups of it being released. |
let's hope someone release the docker-slave-plugin..... |
answer from maintainer: jenkinsci/docker-slaves-plugin@71ed3c8#commitcomment-58330874 |
Nicolas stepped down 2 years ago. At the moment we still consider @ydubreuil to be its (only) maintainer, but he hasn't been around in a while. |
adding |
I don't think we need to wait, we just need to document that there's no compatible release for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with @daniel-beck the code in docker-slaves-plugin
has been merged and waiting for release for over 2 years, if that is not long enough what is the expected minimum amount of time?
added to changelog, feel free to re-word |
This PR is now ready for merge, after ~24 hours, we will merge it if there's no negative feedback. Thanks! |
https://groups.google.com/g/jenkinsci-users/c/fVojVq2Mas0/m/tyW76wgoAwAJ 🙃 |
Misclick, I have no intention of reverting. |
Signed-off-by: Olivier Lamy [email protected]
See JENKINS-66930.
Proposed changelog entries
Proposed upgrade guidelines
In case your plugin use those classes, just migrate to the
org.apache.ant
packageSubmitter checklist
Proposed changelog entries
section only if there are breaking changes or other changes which may require extra steps from users during the upgradeDesired reviewers
@mention
Maintainer checklist
Before the changes are marked as
ready-for-merge
:Proposed changelog entries
are correctupgrade-guide-needed
label is set and there is aProposed upgrade guidelines
section in the PR title. (example)lts-candidate
to be considered (see query).