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.

Sign up for GitHub

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

Jump to bottom

user.list_users does not work on smartos #34038

Closed
Ch3LL opened this issue Jun 15, 2016 · 2 comments
Closed

user.list_users does not work on smartos #34038

Ch3LL opened this issue Jun 15, 2016 · 2 comments
Labels
Bug broken, incorrect, or confusing behavior Confirmed Salt engineer has confirmed bug/feature - often including a MCVE Execution-Module fixed-pls-verify fix is linked, bug author to confirm fix P4 Priority 4 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
Approved

Comments

@Ch3LL
Copy link
Contributor

Ch3LL commented Jun 15, 2016

Description of Issue/Question

The excecution module user.list_users is not available in smartos and is not in solaris_user.py.

Setup

  1. Minion on smartos

Steps to Reproduce Issue

[root@d860d7c7-1cb1-4b6a-9c83-5642a46b46c0 /opt/salt]# /opt/salt/bin/salt '*' user.list_users
d860d7c7-1cb1-4b6a-9c83-5642a46b46c0:
    'user.list_users' is not available.
ERROR: Minions returned with non-zero exit code

As shown above user.list_users is not able to run on smartos

ping @sjorge

Versions Report

Tested on 2016.3.0
@Ch3LL Ch3LL added Bug broken, incorrect, or confusing behavior P4 Priority 4 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around Execution-Module Platform Relates to OS, containers, platform-based utilities like FS, system based apps Confirmed Salt engineer has confirmed bug/feature - often including a MCVE labels Jun 15, 2016
@Ch3LL Ch3LL added this to the Approved milestone Jun 15, 2016
@sjorge
Copy link
Contributor

sjorge commented Jun 15, 2016 •
edited
Loading

At first glance it looks like the one from useradd can just be copied to solaris_user.
Enabled my build host again, it should be online later tonight. I won't be home though so I will try and take a look later this week.

[root@cronos /salt/dynmod/_modules]# salt-call user.list_users
local:
    - _pkgsrc
    - adm
    - admin
    - bin
    - cyrus
    - daemon
    - dladm
    - listen
    - netadm
    - netcfg
    - noaccess
    - nobody
    - nobody4
    - nuucp
    - postfix
    - root
    - sjorge
    - smmsp
    - svctag
    - sys
    - unknown
    - upnp
    - uucp
    - xvm
    - zfssnap

Will push PR shortly

@sjorge sjorge mentioned this issue Jun 15, 2016
Merged
@rallytime
Copy link
Contributor

@Ch3LL Should be fixed by #34042

@rallytime rallytime added the fixed-pls-verify fix is linked, bug author to confirm fix label Jun 15, 2016
gitebra pushed a commit to gitebra/salt that referenced this issue Jun 16, 2016
@jojohans
Merge remote-tracking branch 'upstream/develop' into develop
e83b4bb 
* upstream/develop: (42 commits)
  Typo fix (saltstack#34066)
  states.disk: rewrite unit tests
  states.disk.status: validate percent values
  states.disk: add documentation
  _active_mounts_openbsd: unbreak output for special filesystems (saltstack#34057)
  Use the develop version of the disk state
  fix saltstack#34038 (saltstack#34042)
  Merge pull request saltstack#34025 from rallytime/merge-2016.3
  Update saltmod tests to use master __opts__
  Documentation to describe orchestration support for masterless minions
  Support `salt.state` orchestration formats on masterless minions
  Import the orchestrate runner as an execution module function
  Updated latest release to 2016.3.1 Clean up installation instructions code-block type updates Add link to jinja tutorial
  fix launch config creation params (saltstack#34014)
  Always make chanes to minion config if set (saltstack#34021)
  `states.postgres_privileges` expects a real list, not a comma-separated string for `privileges`
  one clause to set OS grain from CPE_NAME
  fix redis_return's clean_old_jobs.
  Adds new Understanding Jinja topic, and fixes several Jinja doc issues. Removes the "Full list of builtin ..." from each module reference list, leaving just the module type for scanability.
  Call `sys.exit()` instead of `exit()`
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug broken, incorrect, or confusing behavior Confirmed Salt engineer has confirmed bug/feature - often including a MCVE Execution-Module fixed-pls-verify fix is linked, bug author to confirm fix P4 Priority 4 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
Milestone
Approved
Development

No branches or pull requests
3 participants
@sjorge @rallytime @Ch3LL