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 fails on import msgpack - 2015.8 #30887

Closed
anlutro opened this issue Feb 4, 2016 · 2 comments
Closed

salt-ssh fails on import msgpack - 2015.8 #30887

anlutro opened this issue Feb 4, 2016 · 2 comments
Labels
Bug broken, incorrect, or confusing behavior Core relates to code central or existential to Salt fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 Regression The issue is a bug that breaks functionality known to work in previous releases. severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone

Comments

@anlutro
Copy link
Contributor

anlutro commented Feb 4, 2016

It seems like msgpack has been changed from an optional to a required dependency in a recent 2015.8 commit. Output from running salt-ssh against a brand new Debian 8 VM:

    stderr:
        [CRITICAL] Unable to import msgpack or msgpack_pure python modules
        [ERROR   ] An un-handled exception was caught by salt's global exception handler:
        ImportError: No module named msgpack
        Traceback (most recent call last):
          File "/tmp/.vagrant_6a32f1_salt/salt-call", line 4, in <module>
            salt_call()
          File "/tmp/.vagrant_6a32f1_salt/salt/scripts.py", line 329, in salt_call
            import salt.cli.call
          File "/tmp/.vagrant_6a32f1_salt/salt/cli/call.py", line 6, in <module>
            from salt.utils import parsers
          File "/tmp/.vagrant_6a32f1_salt/salt/utils/parsers.py", line 26, in <module>
            import salt.config as config
          File "/tmp/.vagrant_6a32f1_salt/salt/config.py", line 40, in <module>
            import salt.utils.sdb
          File "/tmp/.vagrant_6a32f1_salt/salt/utils/sdb.py", line 9, in <module>
            import salt.loader
          File "/tmp/.vagrant_6a32f1_salt/salt/loader.py", line 30, in <module>
            import salt.utils.event
          File "/tmp/.vagrant_6a32f1_salt/salt/utils/event.py", line 85, in <module>
            import salt.utils.cache
          File "/tmp/.vagrant_6a32f1_salt/salt/utils/cache.py", line 8, in <module>
            import msgpack
        ImportError: No module named msgpack
        Traceback (most recent call last):
          File "/tmp/.vagrant_6a32f1_salt/salt-call", line 4, in <module>
            salt_call()
          File "/tmp/.vagrant_6a32f1_salt/salt/scripts.py", line 329, in salt_call
            import salt.cli.call
          File "/tmp/.vagrant_6a32f1_salt/salt/cli/call.py", line 6, in <module>
            from salt.utils import parsers
          File "/tmp/.vagrant_6a32f1_salt/salt/utils/parsers.py", line 26, in <module>
            import salt.config as config
          File "/tmp/.vagrant_6a32f1_salt/salt/config.py", line 40, in <module>
            import salt.utils.sdb
          File "/tmp/.vagrant_6a32f1_salt/salt/utils/sdb.py", line 9, in <module>
            import salt.loader
          File "/tmp/.vagrant_6a32f1_salt/salt/loader.py", line 30, in <module>
            import salt.utils.event
          File "/tmp/.vagrant_6a32f1_salt/salt/utils/event.py", line 85, in <module>
            import salt.utils.cache
          File "/tmp/.vagrant_6a32f1_salt/salt/utils/cache.py", line 8, in <module>
            import msgpack
        ImportError: No module named msgpack
@anlutro
Copy link
Contributor Author

anlutro commented Feb 4, 2016

This is probably the offending commit: 02d8ff6

@jfindlay jfindlay added Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around Regression The issue is a bug that breaks functionality known to work in previous releases. Core relates to code central or existential to Salt P3 Priority 3 fixed-pls-verify fix is linked, bug author to confirm fix labels Feb 4, 2016
@jfindlay jfindlay added this to the Approved milestone Feb 4, 2016
@anlutro
Copy link
Contributor Author

anlutro commented Feb 5, 2016

I guess I can confirm this as fixed :)

@anlutro anlutro closed this as completed Feb 5, 2016
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 Core relates to code central or existential to Salt fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 Regression The issue is a bug that breaks functionality known to work in previous releases. severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Development

No branches or pull requests

2 participants