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

add support for cookie changes #37

Merged
merged 1 commit into from
May 28, 2016
Merged

add support for cookie changes #37

merged 1 commit into from
May 28, 2016

Conversation

amitmurthy
Copy link
Contributor

@amitmurthy
Copy link
Contributor Author

@andreasnoack could you please test this on any cluster you may have access to and merge?

@andreasnoack
Copy link
Member

I've tested with SGE and Slurm which are the managers I've access to. It works so I'll merge. If there are issues with other managers then people can open specific issues. Thanks for fixing this.

@andreasnoack andreasnoack merged commit 5a192c6 into master May 28, 2016
@andreasnoack andreasnoack deleted the amitm/cookie branch May 28, 2016 17:36
@bjarthur
Copy link
Collaborator

bjarthur commented Jun 2, 2016

does NOT work for me! gives an 'invalid version string" ERROR:

julia> addprocs_sge(3)
job id is 50808835, waiting for job to start ....................................
Worker 5 terminated.
Worker 6 terminated.ERROR (unhandled task failure): ArgumentError: invalid version string: 

ERROR (unhandled task failure): ArgumentError: invalid version string: 
    From worker 6:  Master process (id 1) could not connect within 60.0 seconds.
    From worker 6:  exiting.
Worker 7 terminated.
ERROR (unhandled task failure): ArgumentError: invalid version string: 
 in VersionNumber(::String) at ./version.jl:91
 in process_hdr(::TCPSocket, ::Bool) at ./multi.jl:1054
 in message_handler_loop(::TCPSocket, ::TCPSocket, ::Bool) at ./multi.jl:994
 in process_tcp_streams(::TCPSocket, ::TCPSocket, ::Bool) at ./multi.jl:985
 in (::Base.##299#300{TCPSocket,TCPSocket,Bool})() at ./event.jl:46

@amitmurthy
Copy link
Contributor Author

What are the versions of Julia on the master and the workers? Are they the same or different?

@bjarthur
Copy link
Collaborator

bjarthur commented Jun 2, 2016

last night's nightly on both. Version 0.5.0-dev+4445

@bjarthur
Copy link
Collaborator

bjarthur commented Jun 2, 2016

nevermind. works fine when running julia on a compute node. problem was that i forgot to qlogin and was on the login node. not sure why that would make a difference. but it works.

interesting though that the "invalid version string" seems to be what is thrown when there is a timeout error.

@amitmurthy
Copy link
Contributor Author

The wrong error message needs to be fixed. Good catch.

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

Successfully merging this pull request may close these issues.

3 participants