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

Timeouts for get_next_session not reported correctly #126

Closed
tisba opened this issue Sep 3, 2015 · 4 comments
Closed

Timeouts for get_next_session not reported correctly #126

tisba opened this issue Sep 3, 2015 · 4 comments
Milestone

Comments

@tisba
Copy link
Collaborator

tisba commented Sep 3, 2015

Although ts_launcher:do_launch/1 should catch this properly (in

[{'EXIT', {timeout, _ }}] ->
), I still saw this recently:

=INFO REPORT==== 21-May-2015::09:08:53 ===
         ts_launcher:(3:<0.177.0>) get_next_session failed for unexpected reason [{'EXIT',
                                                                                   {timeout,
                                                                                    {gen_server,
                                                                                     call,
                                                                                     [{global,
                                                                                       ts_config_server},
                                                                                      {get_next_session,
                                                                                       "ip-172-31-13-176",
                                                                                       1}]}}}], abort !

I'd like to provide a patch, but I do not understand why this can happen ([{'EXIT', {timeout, _ }}] seem to be correct to match the error reported above. I'm also not sure how to reproduce this reliably.

/cc @nniclausse

@tisba
Copy link
Collaborator Author

tisba commented Sep 3, 2015

I also saw this error, where I'm actually not sure what it actually means:

=INFO REPORT==== 21-May-2015::13:37:20 ===
         ts_launcher:(3:<0.484.0>) get_next_session failed for unexpected reason [{'EXIT',
                                                                                   {noproc,
                                                                                    {gen_server,
                                                                                     call,
                                                                                     [{global,
                                                                                       ts_config_server},
                                                                                      {get_next_session,
                                                                                       "ip-172-31-0-122",
                                                                                       1}]}}}], abort !

@tisba
Copy link
Collaborator Author

tisba commented Sep 3, 2015

Wow, nice! Could you please explain what the other error means, @nniclausse?

@nniclausse
Copy link
Contributor

i think it means that the ts_config_server does not exist from the ts_launcher point of view. ts_config_server is a globally registered process and tsung forces a global:sync before starting, so it should not happend in theory :) Now i don't know how the global registration stuff works when either the controller or one of the client nodes gets overloaded

@tisba
Copy link
Collaborator Author

tisba commented Sep 3, 2015

Thanks. The cluster was quite overloaded, so I guess this is could be the reason. I was just looking through the logs to look for issues and thought I report anything I find.

@nniclausse nniclausse added this to the v1.6.1 milestone Aug 17, 2017
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

2 participants