Skip to content

"global name 'self' is not defined" in diamond/utils/scheduler.py #761

Closed

Description

@mchx-mgruberman

We're currently seeing the following exception from collector using the latest diamond, likely the version we auto-deployed from the master branch after Wed Dec 29 03:03:46 UTC 2021 (the last timestamp in /var/log/diamond/archive.log).

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

followed by an exit(1), detected from strace:

exit_group(1)

Per our chef install log, it looks like this broke when we installed the 0a9e17d commit:

[2021-12-10T01:46:08+00:00] INFO: git[/usr/local/share/diamond_src] updated to revision e866d24071b11c42b369ef0869e504d5b5b153d3
[2021-12-15T02:46:26+00:00] INFO: git[/usr/local/share/diamond_src] updated to revision be0a43ec00dd006484c39d1bb6845db0f19d72d6
[2021-12-22T22:18:06+00:00] INFO: git[/usr/local/share/diamond_src] updated to revision 774284f92e89af0fb1879e8a38f64d7ff2618cf2   << still working
[2021-12-29T02:57:49+00:00] INFO: git[/usr/local/share/diamond_src] updated to revision 0a9e17dbcfc66be8b2c37479b1a51ac7b16c4fd8   << stacktrace
[2021-12-31T03:33:54+00:00] INFO: git[/usr/local/share/diamond_src] updated to revision 7000e16cfdf4508ed9291fc4b3800592557b2431

Activity

  1. mattrobenolt commented on Jan 6, 2022

    @mattrobenolt
    Member

    It looks like, unfortuantely 9 days ago, this was merged haphazardly: #698

    And doesn't work. cc @shortdudey123 @DStape

  2. mattrobenolt commented on Jan 6, 2022

    @mattrobenolt
    Member

    I left a comment on what I think the issue is: https://github.com/python-diamond/Diamond/pull/698/files#r779903486

    But as I mentioned, I have 0 capacity to confirm this. I haven't used or touched Diamond in many years.

  3. shortdudey123 commented on Jan 15, 2022

    @shortdudey123
    Member

    This should have been solved by #760
    Reopen if not
    Closing . . .

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions