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

More descriptive error messages #23

Open
amikhalev opened this issue Feb 8, 2015 · 12 comments
Open

More descriptive error messages #23

amikhalev opened this issue Feb 8, 2015 · 12 comments
Labels

Comments

@amikhalev
Copy link
Contributor

Some errors we have encountered that are annoying to debug

  • Missing config key
  • Wrong type for config node
  • Missing command
@amikhalev amikhalev added the bug label Feb 8, 2015
@PlasmaPower
Copy link
Contributor

Proposal: add an "Expecting" log level. This would be for when something is expecting something else, and will crash when it does not exist. Examples include config and Kremlin.

@amikhalev
Copy link
Contributor Author

So we are going to clutter the log with a bunch of those messages?

@PlasmaPower
Copy link
Contributor

They can be hidden in TatorTool by default

@amikhalev
Copy link
Contributor Author

There are better solutions than that

@PlasmaPower
Copy link
Contributor

I think my solution solves other problems too, and generally helps with debugging.

@PlasmaPower
Copy link
Contributor

Also, doing anything with YAML will be a pain.

@amikhalev
Copy link
Contributor Author

@PlasmaPower Bump

@PlasmaPower
Copy link
Contributor

@amikhalev Why don't you do this? I barely know where to begin.

@zachjs
Copy link
Member

zachjs commented Feb 25, 2015

Status?

@PlasmaPower
Copy link
Contributor

Missing command: done
All things yaml: not done

@amikhalev
Copy link
Contributor Author

@PlasmaPower Another thing that would help is better stack traces, but I'm not sure if that is possible

@PlasmaPower
Copy link
Contributor

@amikhalev For one thing, we could preload /lib/libSegFault.so for better seg fault messages.

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

No branches or pull requests

3 participants