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

supervisord mod_watch does not accept sfun #23192

Closed
joejulian opened this issue Apr 29, 2015 · 2 comments
Closed

supervisord mod_watch does not accept sfun #23192

joejulian opened this issue Apr 29, 2015 · 2 comments
Labels
Bug broken, incorrect, or confusing behavior fixed-pls-verify fix is linked, bug author to confirm fix P2 Priority 2 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 State-Module
Milestone
Approved

Comments

@joejulian
Copy link
Contributor

supervisord's mod_watch does not accept sfun resulting in a warning, 'sfun' and '__reqs__' are invalid keyword arguments for supervisord.mod_watch. If you were trying to pass additional data to be used in a template context, please populate 'context' with 'key: value' pairs. Your approach will work until Salt Lithium is out. Please update your state files.
joejulian pushed a commit to joejulian/salt that referenced this issue Apr 29, 2015
supervisord.mod_watch should accept sfun
efb59f9 
Fixes saltstack#23192
@joejulian joejulian mentioned this issue Apr 29, 2015
Merged
@jfindlay jfindlay added Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around State-Module fixed-pls-verify fix is linked, bug author to confirm fix P2 Priority 2 labels May 1, 2015
@jfindlay jfindlay added this to the Approved milestone May 1, 2015
@jfindlay
Copy link
Contributor

jfindlay commented May 1, 2015

@joejulian, since your pull request has been merged, feel free to close this.

@joejulian
Copy link
Contributor Author

verified

@jfindlay jfindlay added the Platform Relates to OS, containers, platform-based utilities like FS, system based apps label May 26, 2015
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 fixed-pls-verify fix is linked, bug author to confirm fix P2 Priority 2 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 State-Module
Projects
None yet
Milestone
Approved
Development

No branches or pull requests
2 participants
@jfindlay @joejulian