Skip to content
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

Ensure the correct args are passed to jid_dir #1007

Merged
merged 1 commit into from
Mar 27, 2012

Conversation

fatbox
Copy link

@fatbox fatbox commented Mar 27, 2012

In 1e8e75d one of the calls to jid_dir didn't get all the necessary arguments.

This leads to the following error:

12:36:32,338 [salt.master    ][INFO    ] AES payload received with command minion_publish
Process MWorker-6:
Traceback (most recent call last):
  File "/usr/lib/python2.6/multiprocessing/process.py", line 232, in _bootstrap
    self.run()
  File "/usr/lib/python2.6/dist-packages/salt/master.py", line 368, in run
    self.__bind()
  File "/usr/lib/python2.6/dist-packages/salt/master.py", line 317, in __bind
    ret = self.serial.dumps(self._handle_payload(payload))
  File "/usr/lib/python2.6/dist-packages/salt/master.py", line 335, in _handle_payload
    'clear': self._handle_clear}[key](load)
  File "/usr/lib/python2.6/dist-packages/salt/master.py", line 362, in _handle_aes
    return self.aes_funcs.run_func(data['cmd'], data)
  File "/usr/lib/python2.6/dist-packages/salt/master.py", line 770, in run_func
    ret = getattr(self, func)(load)
  File "/usr/lib/python2.6/dist-packages/salt/master.py", line 700, in minion_publish
    salt.utils.jid_dir(jid),
TypeError: jid_dir() takes exactly 3 arguments (1 given)

SEJeff added a commit that referenced this pull request Mar 27, 2012
Ensure the correct args are passed to jid_dir
@SEJeff SEJeff merged commit 3a245aa into saltstack:develop Mar 27, 2012
@SEJeff
Copy link
Contributor

SEJeff commented Mar 27, 2012

Thanks much!

@thatch45
Copy link
Contributor

Thanks @fatbox, I hope I did not miss any others!

@fatbox
Copy link
Author

fatbox commented Mar 27, 2012

@thatch45 nope, that was the only one. I looked :)

@thatch45
Copy link
Contributor

Thanks!

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.

3 participants