-
-
Notifications
You must be signed in to change notification settings - Fork 635
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_plugin code broken due to new -remoting cli change. #602
Comments
Also seeing this issue. Its probably worth mentioning that its not possible to install an older version of Jenkins because the apt repository only has the latest version. |
Related to #599 |
+1 Major issue for us. |
Does anybody know of a workaround for this issue? I've tried downgrading the jenkins cookbook to < 5.0.0, which doesn't seem to have any effect, and I've tried the following as suggested in the OP:
which just gives me:
Platform is RHEL 7.3. |
Blocked on upgrading past Jenkins 2.53 based off of this issue. |
@Poohblah Look at #603 for a fix |
Thanks for the PR folks. Please check to see if cookbook version 5.0.1 resolves this issue for you. |
5.0.1 actually causes this issue for me. I've had to pin my cookbooks to 5.0.0 for now. |
@oberones What version of jenkins are you on? |
@daften I'm using 2.32.1, which does not have the remoting option available in the cli. |
@oberones have you tried putting executor['protocol'] to nil by overriding it in a wrapper cookbook, environment, ...? That should disable the -remoting option |
5.0.1 seems to fix the issues with |
@daften I haven't tried that but I bet it will work. |
Version 5.0.1 does not fix this for me. I am unable to use the SSH protocol because Jenkins by default starts with SSH disabled so the SSH protocol causes errors when attempting to do anything. I am trying to use the http protocol as the remoting protocol is deprecated and causes warnings. Using the http protocol causes the same errors. I've attempted to set the protocol to nil. This gives me:
|
If I manually add
... this works fine. so it seems like Jenkins does not see a direct filepath without I've created two issues around this: |
@stefhen can you update the title of this issue to reflect the problem that you are seeing rather than ascribing to remoting option being added? I think there are some conflicting reports being added to this and it's hard to track down what is broken/fixed for folks. I think the file:// issue that was pulled in today will fix your issue but I don't want to assume. Based on the change, then I can start asking folks who still have problems to separate out their issues rather than trying to solve everything about plugin installation in this issue (because plugin installation is a big problem that isn't going to be solved in one big change). Thanks! |
Hi @iennae This was actually fixed in |
Thanks for updating @stefhen. For folks still having issues, please verify that a different issue hasn't already been created. If not, create a new issue so that we can make sure to track the specific changes required. Plugin installation is complex and we are working to do a rehaul on this completely. |
Even with See my comment here: |
Cookbook version
5.0.0
Chef-client version
Chef: 12.19.36
Platform Details
Running in AWS.
Scenario:
Plugin installation fails, appears to be due to the cli update that isn't based on remoting, announced today here: https://jenkins.io/blog/2017/04/26/security-advisory/
Steps to Reproduce:
Run
jenkins_plugin
code and get the error above.Example:
Expected Result:
Plugin should be installed.
Actual Result:
Plugin install fails with error above instead of being installed.
It appears the
-remoting
option needs to be passed to the cli options injenkins_plugin
for this to now work:java -jar /home/rightscale/.chef/local-mode-cache/cache/jenkins-cli.jar -remoting -s http://localhost:8080 install-plugin /home/rightscale/.chef/local-mode-cache/cache/swarm-3.4.plugin -name swarm Installing a plugin from local file: /home/rightscale/.chef/local-mode-cache/cache/swarm-3.4.plugin
The text was updated successfully, but these errors were encountered: