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

[configuration] print better error messages on loading #306

Merged
merged 1 commit into from
Mar 16, 2017

Conversation

mikz
Copy link
Contributor

@mikz mikz commented Mar 16, 2017

fixes https://issues.jboss.org/browse/THREESCALE-80

  • print reason why configuration could not be downloaded
  • print more debugging information

example log:

2017/03/16 08:08:15 [error] 30795#0: [lua] configuration_loader.lua:80: init(): boot could not get configuration (exit 1)
2017/03/16 08:08:16 [warn] 30816#0: *2 [lua] remote_v2.lua:58: call(): failed to get list of services: invalid status, context: ngx.timer
./src/configuration_loader.lua:25: missing configuration

2017/03/16 08:08:15 [emerg] 30795#0: [lua] configuration_loader.lua:101: init(): failed to load configuration, exiting

@octobot octobot added the T-obux label Mar 16, 2017
@mikz mikz force-pushed the loading-configuration-log branch 2 times, most recently from 700adac to 5889a46 Compare March 16, 2017 07:26
* print reason why configuration could not be downloaded
* print more debugging information
* don't duplicatate error messages
@mikz mikz force-pushed the loading-configuration-log branch from 5889a46 to c944a88 Compare March 16, 2017 07:40
@mikz mikz merged commit 4934fb8 into master Mar 16, 2017
@mikz mikz deleted the loading-configuration-log branch March 16, 2017 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants