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

rpm VirtuozzoLinux not working #34893

Closed
msdogado opened this issue Jul 22, 2016 · 3 comments
Closed

rpm VirtuozzoLinux not working #34893

msdogado opened this issue Jul 22, 2016 · 3 comments
Assignees
Labels
Bug broken, incorrect, or confusing behavior fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone

Comments

@msdogado
Copy link

msdogado commented Jul 22, 2016

Steps to Reproduce Issue

not sure but if you want to add VirtuozzoLinux you will have to fix it aswell

file: /usr/lib/python2.7/dist-packages/salt/modules/rpm.py

 52         return (False, 'The rpm execution module failed to load: failed to detect os or os_family grains.')
 53
 54     enabled = ('amazon', 'xcp', 'xenserver', 'VirtuozzoLinux')
 55
 56     if os_family in ['redhat', 'suse'] or os_grain in enabled:

you have to add VirtuozzoLinux to line 54 to make rpm.py working for Virtuozzo Linux

Versions Report

Salt Version:
           Salt: 2016.3.1

Dependency Versions:
           cffi: Not Installed
       cherrypy: 3.2.2
       dateutil: 1.5
          gitdb: 0.5.4
      gitpython: 0.3.2 RC1
          ioflo: Not Installed
         Jinja2: 2.7.2
        libgit2: Not Installed
        libnacl: Not Installed
       M2Crypto: 0.21.1
           Mako: 0.9.1
   msgpack-pure: Not Installed
 msgpack-python: 0.3.0
   mysql-python: 1.2.3
      pycparser: Not Installed
       pycrypto: 2.6.1
         pygit2: Not Installed
         Python: 2.7.6 (default, Jun 22 2015, 17:58:13)
   python-gnupg: Not Installed
         PyYAML: 3.10
          PyZMQ: 14.0.1
           RAET: Not Installed
          smmap: 0.8.2
        timelib: Not Installed
        Tornado: 4.2.1
            ZMQ: 4.0.4

System Versions:
           dist: Ubuntu 14.04 trusty
        machine: x86_64
        release: 3.13.0-042stab108.7
         system: Linux
@rallytime
Copy link
Contributor

@msdogado Thanks for filing this one too. I've added VirtuozzoLinux to the list of enabled distributions to the rpm execution module. Does that resolve this issue fully for you?

@rallytime rallytime added severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps TEAM Core Bug broken, incorrect, or confusing behavior labels Jul 22, 2016
@rallytime rallytime added this to the C 6 milestone Jul 22, 2016
@rallytime rallytime self-assigned this Jul 22, 2016
@msdogado
Copy link
Author

@rallytime Thanks! As far as I can see right now it fixes the VirtuozzoLinux related issues.

@rallytime
Copy link
Contributor

@msdogado Awesome! I'll close this up, then. :)

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 fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps 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