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

[BUG] salt.wait_for_event does not accept argument "pillar" #61092

Closed
agraul opened this issue Oct 21, 2021 · 0 comments · Fixed by #61093
Closed

[BUG] salt.wait_for_event does not accept argument "pillar" #61092

agraul opened this issue Oct 21, 2021 · 0 comments · Fixed by #61093
Assignees
Labels
Bug broken, incorrect, or confusing behavior severity-high 2nd top severity, seen by most users, causes major problems
Milestone

Comments

@agraul
Copy link
Contributor

agraul commented Oct 21, 2021

Description

It is not possible to use salt.wait_for_event together with salt-run state.orchestrate_single because the latter passes a pillar argument to the former. The former does not accept **kwargs and pillar is not in the function signature.

Setup

  • VM (KVM CentOS 8 with latest Salt from repo.saltproject.io/#rhel)

Steps to Reproduce the behavior

[root@upstream-master ~]# salt-run state.orchestrate_single fun=salt.wait_for_event name='/salt/minion/*/start' id_list=minion1                                                                                                             
                                                                                                                                                                                                                                              
Passed invalid arguments: 'pillar' is an invalid keyword argument for 'salt.wait_for_event'                                                                                                                                                   
                                                                                                                                                                                                                                              
Usage:                                                                                                                                                                                                                                        
                                                                                                                                                                                                                                              
    Execute a single state orchestration routine                                                                                                                                                                                              
                                                                                                                                                                                                                                              
    .. versionadded:: 2015.5.0                                                                                                                                                                                                                
                                                                                                                                                                                                                                              
    CLI Example:                                                                                                                                                                                                                              
                                                                                                                                                                                                                                              
    .. code-block:: bash                                                                                                                                                                                                                      
                                                                                                                                                                                                                                              
        salt-run state.orchestrate_single fun=salt.wheel name=key.list_all         

Expected behavior
salt-run state.orchestrate_single fun=salt.wait_for_event name='/salt/minion/*/start' id_list=minion1 runs salt.wait_for_event without an error.

Versions Report

salt --versions-report (Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)
[root@upstream-master ~]# salt --versions-report
Salt Version:
          Salt: 3003.3
 
Dependency Versions:
          cffi: 1.11.5
      cherrypy: Not Installed
      dateutil: 2.6.1
     docker-py: Not Installed
         gitdb: Not Installed
     gitpython: Not Installed
        Jinja2: 2.10.1
       libgit2: Not Installed
      M2Crypto: 0.35.2
          Mako: Not Installed
       msgpack: 0.6.2
  msgpack-pure: Not Installed
  mysql-python: Not Installed
     pycparser: 2.14
      pycrypto: Not Installed
  pycryptodome: Not Installed
        pygit2: Not Installed
        Python: 3.6.8 (default, Apr 16 2020, 01:36:27)
  python-gnupg: Not Installed
        PyYAML: 3.12
         PyZMQ: 19.0.0
         smmap: Not Installed
       timelib: Not Installed
       Tornado: 4.5.3
           ZMQ: 4.3.4
 
System Versions:
          dist: centos 8 Core
        locale: UTF-8
       machine: x86_64
       release: 4.18.0-193.6.3.el8_2.x86_64
        system: Linux
       version: CentOS Linux 8 Core
@agraul agraul added Bug broken, incorrect, or confusing behavior needs-triage labels Oct 21, 2021
@OrangeDog OrangeDog added severity-high 2nd top severity, seen by most users, causes major problems Bug broken, incorrect, or confusing behavior and removed Bug broken, incorrect, or confusing behavior needs-triage labels Oct 26, 2021
@OrangeDog OrangeDog added this to the Approved milestone Oct 26, 2021
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 severity-high 2nd top severity, seen by most users, causes major problems
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants