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

salt-ssh throws exception when minionfs is enabled. #50351

Open
russellballestrini opened this issue Nov 2, 2018 · 5 comments
Open

salt-ssh throws exception when minionfs is enabled. #50351

russellballestrini opened this issue Nov 2, 2018 · 5 comments
Labels
Bug broken, incorrect, or confusing behavior severity-low 4th level, cosemtic problems, work around exists
Milestone

Comments

@russellballestrini
Copy link
Contributor

Description of Issue/Question

Setup

salt-ssh throws the following exception when minionfs is enabled on salt master::

ERROR   ] An Exception occurred while executing state.highstate: [Errno 2] No such file or directory: '/var/tmp/.root_812c13_salt/running_data/var/cache/salt/minion/minions'
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/salt/client/ssh/__init__.py", line 1156, in run_wfunc
    result = self.wfuncs[self.fun](*self.args, **self.kwargs)
  File "/usr/lib/python2.7/dist-packages/salt/client/ssh/wrapper/state.py", line 705, in highstate
    roster_grains)
  File "/usr/lib/python2.7/dist-packages/salt/client/ssh/state.py", line 229, in prep_trans_tar
    files = file_client.cache_dir(name, saltenv, cachedir=cachedir)
  File "/usr/lib/python2.7/dist-packages/salt/fileclient.py", line 240, in cache_dir
    for fn_ in self.file_list(saltenv):
  File "/usr/lib/python2.7/dist-packages/salt/fileclient.py", line 1283, in file_list
    return salt.utils.data.decode(self.channel.send(load)) if six.PY2 \
  File "/usr/lib/python2.7/dist-packages/salt/fileserver/__init__.py", line 871, in send
    return getattr(self.fs, cmd)(load)
  File "/usr/lib/python2.7/dist-packages/salt/utils/decorators/__init__.py", line 594, in wrapped
    **salt.utils.data.decode_dict(kwargs)
  File "/usr/lib/python2.7/dist-packages/salt/fileserver/__init__.py", line 752, in file_list
    ret.update(self.servers[fstr](load))
  File "/usr/lib/python2.7/dist-packages/salt/fileserver/minionfs.py", line 253, in file_list
    minion_dirs = os.listdir(minions_cache_dir)
OSError: [Errno 2] No such file or directory: '/var/tmp/.root_812c13_salt/running_data/var/cache/salt/minion/minions'
[DEBUG   ] LazyLoaded nested.output
[TRACE   ] data = {u'mx1.example.net': u"An Exception occurred while executing state.highstate: [Errno 2] No such file or directory: '/var/tmp/.root_812c13_salt/running_data/var/cache/salt/minion/minions'"}

Steps to Reproduce Issue

enable minionfs on salt-master and try to run state.highstate using salt-ssh

Versions Report

$ salt-ssh --version
salt-ssh 2018.3.3 (Oxygen)

$ salt --version
salt 2018.3.3 (Oxygen)
@dwoz
Copy link
Contributor

dwoz commented Nov 2, 2018

@russellballestrini Thank you for reporting this issue. I have been able to confirm the issue by using this state.

pushfile:
  module.run:
     - name: cp.push
     - path: /home/admin/foo

@dwoz dwoz added this to the Approved milestone Nov 2, 2018
@dwoz dwoz added P4 Priority 4 Bug broken, incorrect, or confusing behavior severity-low 4th level, cosemtic problems, work around exists labels Nov 2, 2018
@russellballestrini
Copy link
Contributor Author

Anyone have any hints on where to hunt for this defect / regression?

It's starting to block me.

@stale
Copy link

stale bot commented Jan 9, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Jan 9, 2020
@russellballestrini
Copy link
Contributor Author

It's still broken.

@stale
Copy link

stale bot commented Jan 9, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Jan 9, 2020
@sagetherage sagetherage removed the P4 Priority 4 label Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior severity-low 4th level, cosemtic problems, work around exists
Projects
None yet
Development

No branches or pull requests

3 participants