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

8266 no processor available #220

Merged
merged 5 commits into from Jul 6, 2012

Conversation

joshmoore
Copy link
Member

These are not necessarily fixes for the 8266 issue but:

  • the timeouts should help to prevent race conditions on startup, reducing the number of times that the python processes are dead in the water on startup;
  • and the improved error message (and new exception type) should help to figure out just what's going on.

Though it is unlikely to completely fix the issues with
"No processor available" adding a wait on startup may
help some instances of Processors not properly connecting
to the main Blitz server.

The 10 seconds is configurable via the "OMERO_STARTUP_WAIT"
environment variable with the default being 10000 milliseconds.
As we add more server-specific environment variables,
it makes sense to gather these in a single location
like we do with configuration properties in etc/omero.properties

These files should be configured by users and then
run in the current shell before starting OMERO.
Currently, it's unclear from the error message whether
no processor was available due to no servers being found
or whether all of the servers refused to execute the given
script. This is now included in the error message.
@joshmoore
Copy link
Member Author

These commits are of a nature that they could have been on the sprint19 branch (already merged into develop) but there were enough of them that I thought a separate branch better.

joshmoore added a commit that referenced this pull request Jul 6, 2012
@joshmoore joshmoore merged commit 0d65d6f into ome:develop Jul 6, 2012
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.

None yet

1 participant