Skip to content
This repository has been archived by the owner on Oct 16, 2024. It is now read-only.

added DirectoryTimedRotatingFileHandler, removed sentry workaround, impr... #197

Merged
merged 2 commits into from
Jul 8, 2014

Conversation

mkurek
Copy link
Contributor

@mkurek mkurek commented Jul 7, 2014

added DirectoryTimedRotatingFileHandler, removed sentry workaround, improved default loggers and config

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.76%) when pulling 32d245f on mkurek:feature/log-separation into 02c6840 on allegro:develop.

@@ -32,7 +32,7 @@ def update_assets_parts(data, date):


@plugin.register(chain='pricing', requires=['assets'])
def parts(**kwargs):
def assets_parts(**kwargs):
"""Updates the devices from Ralph Assets."""
date = kwargs['today']
count = sum(update_assets_parts(data, date) for data in get_asset_parts())
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can turn off this plugin at now. Exactly its not work correctly, saved device part don't have parent and some mistakes like this. What do you think? maybe just remove this plugin. It is only 40 lines of code contains imports and blank lines.

@kula1922
Copy link
Contributor

kula1922 commented Jul 8, 2014

LGTM

kula1922 added a commit that referenced this pull request Jul 8, 2014
added DirectoryTimedRotatingFileHandler, removed sentry workaround, impr...
@kula1922 kula1922 merged commit 2b24b49 into allegro:develop Jul 8, 2014
@mkurek mkurek deleted the feature/log-separation branch July 11, 2014 08:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants