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

wget timeout? #1

Open
cabynum opened this issue Mar 31, 2014 · 16 comments
Open

wget timeout? #1

cabynum opened this issue Mar 31, 2014 · 16 comments

Comments

@cabynum
Copy link

cabynum commented Mar 31, 2014

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
Copy link
Contributor

Same issue here, would love to use this plugin.

@dueyfinster
Copy link
Contributor

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

@runthebusiness
Copy link
Owner

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
Copy link
Contributor

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
Copy link
Owner

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 [email protected]:

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//issues/1#issuecomment-39512963
.

@dueyfinster
Copy link
Contributor

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
Copy link
Owner

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 [email protected]:

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//issues/1#issuecomment-39710331
.

@runthebusiness
Copy link
Owner

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
Copy link
Owner

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
Copy link
Owner

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
Copy link
Contributor

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. wget timeout? #1

@runthebusiness
Copy link
Owner

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 [email protected]:

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

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

@BennyHallett
Copy link

+1

I'm experiencing the same issue.

@runthebusiness
Copy link
Owner

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 [email protected]
wrote:

+1

I'm experiencing the same issue.


Reply to this email directly or view it on GitHub
#1 (comment)
.

@BennyHallett
Copy link

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
Copy link
Owner

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 [email protected]
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
#1 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants