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 does not use custom utils #53666

Closed
clinta opened this issue Jun 30, 2019 · 3 comments · Fixed by #65735
Closed

salt-ssh does not use custom utils #53666

clinta opened this issue Jun 30, 2019 · 3 comments · Fixed by #65735
Labels
Feature new functionality including changes to functionality and code refactors, etc. Salt-SSH
Milestone

Comments

@clinta
Copy link
Contributor

clinta commented Jun 30, 2019

Description of Issue

Custom utils in _utils do not work with salt-ssh

Setup

_modules/foo.py:

import salt.utils.foo

def foo():
    return salt.utils.foo.foo()

_utils/foo.py:

def foo():
    return "foo"

Steps to Reproduce Issue

$ salt-ssh --regen-thin minion foo.foo          

minion:
    ----------
    retcode:
        255
    stderr:
        'foo' __virtual__ returned False: No module named foo
    stdout:

Versions Report

$ salt-ssh --versions-report
Salt Version:
           Salt: 2019.2.0
 
Dependency Versions:
           cffi: Not Installed
       cherrypy: Not Installed
       dateutil: Not Installed
      docker-py: Not Installed
          gitdb: Not Installed
      gitpython: Not Installed
          ioflo: Not Installed
         Jinja2: 2.10.1
        libgit2: Not Installed
        libnacl: Not Installed
       M2Crypto: 0.34.0
           Mako: Not Installed
   msgpack-pure: Not Installed
 msgpack-python: 0.6.1
   mysql-python: Not Installed
      pycparser: Not Installed
       pycrypto: 3.8.2
   pycryptodome: Not Installed
         pygit2: Not Installed
         Python: 2.7.16 (default, Mar 11 2019, 18:59:25)
   python-gnupg: Not Installed
         PyYAML: 5.1
          PyZMQ: 18.0.1
           RAET: Not Installed
          smmap: Not Installed
        timelib: Not Installed
        Tornado: 5.1.1
            ZMQ: 4.3.1
 
System Versions:
           dist:   
         locale: UTF-8
        machine: x86_64
        release: 5.1.15-arch1-1-ARCH
         system: Linux
        version: Not Installed
@Akm0d Akm0d added v2019.2.0 unsupported version Feature new functionality including changes to functionality and code refactors, etc. Salt-SSH labels Jul 2, 2019
@Akm0d Akm0d added this to the Approved milestone Jul 2, 2019
@max-arnold
Copy link
Contributor

Related: #46841, #32500

@stale
Copy link

stale bot commented Jan 8, 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 8, 2020
@sagetherage sagetherage added the Confirmed Salt engineer has confirmed bug/feature - often including a MCVE label Jan 9, 2020
@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 v2019.2.0 unsupported version Confirmed Salt engineer has confirmed bug/feature - often including a MCVE labels May 18, 2020
@github-project-automation github-project-automation bot moved this to Done in Salt-SSH Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature new functionality including changes to functionality and code refactors, etc. Salt-SSH
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants