-
Notifications
You must be signed in to change notification settings - Fork 87
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
Update content of "Sparrow Crashes" messages. #309
Conversation
…nto jmh_messages
…nto jmh_messages
…nto jmh_messages
If you use Filebeat as a log aggregator, the messages are not divided by fields. Added a set of filters for Elasticsearch so that a log message is split up into separate fields.
@@ -32,5 +32,8 @@ deploy/settings.yml | |||
deploy/google/app.yaml | |||
!deploy/google/app.yaml.example | |||
|
|||
# Virtual environment | |||
venv/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add these changes to your local .gitignore file. They shouldn't be in .gitignore.
omaha_server/crash/senders.py
Outdated
logger = logging.getLogger('crashes') | ||
|
||
extra.update(tags) | ||
extra['sparrow_version'] = tags['ver'] if 'ver' in tags else 'unknown' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rename sparrow_version to app_version.
omaha_server/crash/senders.py
Outdated
extra['logger_name'] = 'omaha_server' | ||
|
||
# Send message with logger. | ||
logger.info(add_extra_to_log_message("Sparrow Crashes", extra=extra)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use an environment variable to set app specific crash message(e.g. Sparrow crashes).
Could you please squash your changes in 1-2 commits? |
@@ -1,5 +1,5 @@ | |||
[bumpversion] | |||
current_version = 0.6.3 | |||
current_version = 0.6.3.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't change the version. We bump it when preparing a new release.
No description provided.