runthebusiness / puppet-eclipse

This class installs eclipse, eclipse plug ins and allows for management of projects and workspaces via puppet.
5 stars 5 forks source link

wget timeout? #1

Open cabynum opened 10 years ago

cabynum commented 10 years ago

I seem to get a "corrupted" eclipse package each time when trying to use this module. The error (unfortunately I'm not currently looking at it) seem to say something along the lines of "this isn't a gzip package..". I'm thinking that the complete archive is not being downloaded, possibly due to the time allotted for downloading. Is there a way to increase the timeout? Have you all seen this issue? Thanks for any assistance.

dueyfinster commented 10 years ago

Same issue here, would love to use this plugin.

dueyfinster commented 10 years ago

Looks like this could be the issue: https://ask.puppetlabs.com/question/1308/command-exceeded-timeout/

runthebusiness commented 10 years ago

Hi Guys

My guess would be that the url used in the module has moved. They keep moving it so I haven't been updating the module for it to keep up enough.

Will try to look into it more soon.

Currently we are using this:

eclipse{'eclipse': downloadurl=>'http://www.eclipse.org/downloads/download.php?file=/eclipse/downloads/drops/R-3.7.2-201202080800/eclipse-SDK-3.7.2-linux-gtk-x86_64.tar.gz', downloadfile=>'eclipse-SDK-3.7.2-linux-gtk-x86_64.tar.gz' }

Let me know if that works.

Thanks very much for your interest in the module.

All the best.

Will Ferrer

dueyfinster commented 10 years ago

Hi Will,

Cheers for writing it! No the URL still works, can still download Eclipse (I thought that was the issue also). It's a pure timeout issue I think (I'm trying it on Centos). I got it working after using a file provider (with puppet:// url) in init.pp instead - I'm not sure how to integrate it properly (new to puppet) so I simply replaced the wget call. I would think a slight change to $method and the if statement could call a file provider and leave the wget option also. I could post it (my init.pp) up as a gist if you'd like?

Thanks again, Neil

runthebusiness commented 10 years ago

Hi Neil

So it sounds like it needs an extended time out, and a method that is not just wget as a work around. Thanks very much for the report on it. I will take a gist, a pull request if you manage to make the needed changes, or even just an email with the code from you and see about fixing the module some time next week.

I hope you have a great weekend.

All the best.

Will Ferrer

On Thu, Apr 3, 2014 at 3:29 PM, Neil Grogan notifications@github.comwrote:

Hi Will,

Cheers for writing it! No the URL still works, can still download Eclipse (I thought that was the issue also). It's a pure timeout issue I think (I'm trying it on Centos). I got it working after using a file provider (with puppet:// url) in init.pp instead - I'm not sure how to integrate it properly (new to puppet) so I simply replaced the wget call. I would think a slight change to $method and the if statement could call a file provider and leave the wget option also. I could post it (my init.pp) up as a gist if you'd like?

Thanks again, Neil

Reply to this email directly or view it on GitHubhttps://github.com/runthebusiness/puppet-eclipse/issues/1#issuecomment-39512963 .

dueyfinster commented 10 years ago

Hi Will,

Posted it here: https://gist.github.com/dueyfinster/10017186

Sorry I'd do a full pull request, but I'm not sure I'd get the code in the correct state for it to be just an automatic merge.

_Neil

runthebusiness commented 10 years ago

Hi Neil

Thanks very much.

I will be looking at this and incorporating it soon.

I hope you had a great weekend.

All the best.

Will Ferrer

On Mon, Apr 7, 2014 at 2:22 AM, Neil Grogan notifications@github.comwrote:

Hi Will,

Posted it here: https://gist.github.com/dueyfinster/10017186

Sorry I'd do a full pull request, but I'm not sure I'd get the code in the correct state for it to be just an automatic merge.

_Neil

Reply to this email directly or view it on GitHubhttps://github.com/runthebusiness/puppet-eclipse/issues/1#issuecomment-39710331 .

runthebusiness commented 10 years ago

Hi Guys

I merged the pull requests, and also added several new features based on the gist that Neil sent and the puppet exec time out issue post I received as well.

The changes are in branch v1.1.0.b

I was wondering if you could maybe check out the branch for me, give it a test and see if it's working for you.

I don't have good environment for testing these changes at the moment but I can get one set up next week.

If it has problems I will try to spend more time on it next week to sort out the issues.

Thank you very much for your contribution and help with the project.

All the best.

Will Ferrer

runthebusiness commented 10 years ago

Hello again

I just updated the new branch after doing an audit of the changes I had made rather hastily earlier this week.

The next step for me will be setting up a new testing environment so I can be sure the new features / fixes are functioning.

If either of you end up testing the new branch with the features and fixes that arose from our conversations please let me know what the results of the tests are.

I hope this message finds you well.

All the best.

Will Ferrer

runthebusiness commented 10 years ago

Hi Neil

I also just added you to the contributing authors in the read me file.

Thanks again for your assistance on the project.

All the best.

Will Ferrer

dueyfinster commented 10 years ago

Hi Will,

So sorry for late reply, up to my eyes this week! Thanks so much for merging.

Regards, Neil

On Thu, 17 Apr 2014, at 23:15, runthebusiness wrote:

Hi Neil

I also just added you to the contributing authors in the read me file.

Thanks again for your assistance on the project.

All the best.

Will Ferrer

Reply to this email directly or [1]view it on GitHub. [166151__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcxMzM5MjExNy wiZGF0YSI6eyJpZCI6Mjg5MDAzODV9fQ==--19ddee40a9d3fc960911a19a04ccc390ffa d6441.gif]

References

  1. https://github.com/runthebusiness/puppet-eclipse/issues/1?utm_campaign=website&utm_source=sendgrid.com&utm_medium=email#issuecomment-40769180
runthebusiness commented 10 years ago

Hi Neil

Thanks again for your contribution. No worries about the last response. let me know if you get a chance to test the new branch I made with the other changes from the gist and etc/

All the best.

Will Ferrer

On Tue, Apr 22, 2014 at 5:40 AM, Neil Grogan notifications@github.comwrote:

Hi Will,

So sorry for late reply, up to my eyes this week! Thanks so much for merging.

Regards, Neil

On Thu, 17 Apr 2014, at 23:15, runthebusiness wrote:

Hi Neil

I also just added you to the contributing authors in the read me file.

Thanks again for your assistance on the project.

All the best.

Will Ferrer

Reply to this email directly or [1]view it on GitHub. [166151__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcxMzM5MjExNy wiZGF0YSI6eyJpZCI6Mjg5MDAzODV9fQ==--19ddee40a9d3fc960911a19a04ccc390ffa d6441.gif]

References

1. https://github.com/runthebusiness/puppet-eclipse/issues/1?utm_campaign=website&utm_source=sendgrid.com&utm_medium=email#issuecomment-40769180

Reply to this email directly or view it on GitHubhttps://github.com/runthebusiness/puppet-eclipse/issues/1#issuecomment-41034458 .

BennyHallett commented 10 years ago

+1

I'm experiencing the same issue.

runthebusiness commented 10 years ago

Hi Benny

I have a new branch of the project that I think resolves this. I have been to distracted to test the branch however. If you get a chance to test it could you let me know?

https://github.com/runthebusiness/puppet-eclipse/tree/v1.1.0.b

Thanks very much for the input.

All the best.

Will

On Tue, Jun 17, 2014 at 1:49 PM, Benny Hallett notifications@github.com wrote:

+1

I'm experiencing the same issue.

— Reply to this email directly or view it on GitHub https://github.com/runthebusiness/puppet-eclipse/issues/1#issuecomment-46363188 .

BennyHallett commented 10 years ago

Hey Will,

Sorry, I meant to update you earlier. I did notice that you had created a branch for this change. I've tried it out and it works great.

Thanks heaps!

runthebusiness commented 10 years ago

Hi Benny

I will merge the branch into live if it worked well for you. Thanks so much for the test and the update.

I hope this message finds you well.

All the best.

Will Ferrer

On Thu, Jun 19, 2014 at 3:40 PM, Benny Hallett notifications@github.com wrote:

Hey Will,

Sorry, I meant to update you earlier. I did notice that you had created a branch for this change. I've tried it out and it works great.

Thanks heaps!

— Reply to this email directly or view it on GitHub https://github.com/runthebusiness/puppet-eclipse/issues/1#issuecomment-46626151 .