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

Case-insensitive matching of errors/warnings in admin diagnostics #1117

Merged
merged 2 commits into from May 16, 2013

Conversation

zeb
Copy link
Member

@zeb zeb commented Apr 29, 2013

To test the change (assuming a stopped server):

  • invalidate Ice grid configuration : echo "RaiseAnIceXMLParserExceptionAtStartup" >> $OMERO_HOME/etc/grid/templates.xml
  • run omero admin start : server starts up despite invalid XML syntax
  • run omero admin diagnostics : log files report for master.err should now display positive errors and warnings counts

@bpindelski
Copy link

This definitely enables the display of errors/warnings count from master.err. I wonder if a more general solution would be better: lower the case of each line returned by p.lines() as now we handle each case type separately (what if in the future a component will print Error or eRror?).

@zeb
Copy link
Member Author

zeb commented May 13, 2013

@bpindelski Thanks, updated implementation according to suggested solution.

@joshmoore
Copy link
Member

Travis error is just a timeout as frequently happens. @bpindelski : any comments?

@bpindelski
Copy link

@joshmoore, @zeb Looks better now. Good to merge.

@zeb
Copy link
Member Author

zeb commented May 14, 2013

@bpindelski, @joshmoore Ok. I guess a backport onto dev_4_4 will be required?

@joshmoore
Copy link
Member

Certainly can't hurt, @zeb. That's certainly what you're more likely to be deploying in the short-term.

joshmoore added a commit that referenced this pull request May 16, 2013
…develop

Case-insensitive matching of errors/warnings in admin diagnostics
@joshmoore joshmoore merged commit efdabc1 into ome:develop May 16, 2013
zeb added a commit to zeb/openmicroscopy that referenced this pull request May 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants