Skip to content

Fix for issue introduced in #698 - #760

Merged
shortdudey123 merged 2 commits into
python-diamond:masterfrom
pmorgantech:master
Jan 14, 2022
Merged

shortdudey123 merged 2 commits into
python-diamond:masterfrom
pmorgantech:master

Conversation

@pmorgantech

Copy link
Copy Markdown
Contributor

This patch should fix the error introduced with the merge of PR698:

Process MemoryCollector:
Traceback (most recent call last):
File "/usr/lib/python2.7/multiprocessing/process.py", line 267, in _bootstrap
self.run()
File "/usr/lib/python2.7/multiprocessing/process.py", line 114, in run
self._target(*self._args, **self._kwargs)
File "/home/petem/git/Diamond/src/diamond/utils/scheduler.py", line 55, in collector_process
if str_to_bool(self.config['stagger_collection']):
NameError: global name 'self' is not defined

@pmorgantech

Copy link
Copy Markdown
Contributor Author

Accidentally included a fix for issue introduced in PR759. I meant to split these out as separate PRs, but I don't know how to undo it, offhand.

@pmorgantech pmorgantech changed the title Fix for issue introduced in PR698 Fix for issue introduced in #698 Jan 3, 2022
@shortdudey123

Copy link
Copy Markdown
Member

Thank you for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants