-
Notifications
You must be signed in to change notification settings - Fork 172
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
Added logging to the profiler #237
Conversation
|
Traceback (most recent call last):
File "/home/travis/build/glastopf/glastopf/glastopf/testing/test_honeypot.py", line 57, in test_honeypot_mongo
self.glastopf = GlastopfHoneypot(work_dir=self.tmpdir, config=config_file)
File "/home/travis/build/glastopf/glastopf/glastopf/glastopf.py", line 83, in __init__
self.profiler_available = conf_parser.getboolean("profiler", "enabled")
File "/usr/lib/python2.7/ConfigParser.py", line 368, in getboolean
v = self.get(section, option)
File "/usr/lib/python2.7/ConfigParser.py", line 607, in get
raise NoSectionError(section)
NoSectionError: No section: 'profiler' |
|
@sookyp can you rebase? |
Removed leftovers Added profiler section
rebased. but some tests are failing |
rather verbose, but it works the failing test_honeypot.py fails also on master on my computer |
yep, there is an error:
|
I found this http://stackoverflow.com/questions/24306205/file-not-found-error-when-launching-a-subprocess but solution comes with BIG warning https://docs.python.org/2/library/subprocess.html#frequently-used-arguments |
finaly in master f1842ba |
Attempting to fix issue #26. Besides stdout logging, it also stores/retrieves the comments in a similar way like the former glastopf/modules/reporting/log_postgresql.py