Skip to content

[BUG] saltmod.state doesn't respect "roster_file" argument #56649

Open

Description

@brianthelion

Description
#51158 added support for the roster argument, but this is only a partial solution. roster_file must be supported as well for use-cases that need to configure the roster at state execution time.

Setup
N/A

Steps to Reproduce the behavior
N/A

Expected behavior
See #51158.

Screenshots
See #51158.

Versions Report

salt --versions-report
Salt Version:
           Salt: 3000.1
 
Dependency Versions:
           cffi: 1.14.0
       cherrypy: Not Installed
       dateutil: 2.8.1
      docker-py: Not Installed
          gitdb: Not Installed
      gitpython: Not Installed
         Jinja2: 2.11.1
        libgit2: Not Installed
       M2Crypto: Not Installed
           Mako: Not Installed
   msgpack-pure: Not Installed
 msgpack-python: 0.6.2
   mysql-python: Not Installed
      pycparser: 2.20
       pycrypto: 2.6.1
   pycryptodome: Not Installed
         pygit2: Not Installed
         Python: 3.6.9 (default, Nov  7 2019, 10:44:02)
   python-gnupg: Not Installed
         PyYAML: 5.3.1
          PyZMQ: 19.0.0
          smmap: Not Installed
        timelib: Not Installed
        Tornado: 4.5.3
            ZMQ: 4.3.2
 
System Versions:
           dist: Ubuntu 18.04 bionic
         locale: UTF-8
        machine: x86_64
        release: 4.15.0-96-generic
         system: Linux
        version: Ubuntu 18.04 bionic

Additional context
N/A

Activity

  1. added
    Featurenew functionality including changes to functionality and code refactors, etc.
    on Apr 15, 2020
  2. added this to the Approved milestone on Apr 15, 2020
  3. reopened this on Apr 15, 2020
  4. added
    bugbroken, incorrect, or confusing behavior
    severity-low4th level, cosemtic problems, work around exists
    and removed
    Featurenew functionality including changes to functionality and code refactors, etc.
    on Oct 23, 2026
  5. va000612 commented on Nov 5, 2026

    @va000612

    I would like to work on this.
    PR has been raised with all testing details covered: #70686

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    State-Modulebugbroken, incorrect, or confusing behaviorseverity-low4th level, cosemtic problems, work around exists

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions