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

New use_superseded option needs added to configuration documentation #50348

Open
doesitblend opened this issue Nov 1, 2018 · 12 comments
Open
Labels
Confirmed Salt engineer has confirmed bug/feature - often including a MCVE doc-request net new docs requested Documentation Relates to Salt documentation severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around time-estimate-sprint ZD The issue is related to a Zendesk customer support ticket.

Comments

@doesitblend
Copy link
Collaborator

doesitblend commented Nov 1, 2018

Description of Issue/Question

The use_superseded option is not mentioned in either the master config documentation or the minion config documentation.

Specifically, this documentation only mentions to adding this option to the minion configuration inferring that is the only place it is needed.

Versions Report

2018.3.3

@doesitblend
Copy link
Collaborator Author

This issue may be related to #44882

@doesitblend doesitblend added the ZD The issue is related to a Zendesk customer support ticket. label Nov 1, 2018
@doesitblend
Copy link
Collaborator Author

ZD-2936

@dwoz dwoz added this to the Approved milestone Nov 2, 2018
@dwoz dwoz added severity-low 4th level, cosemtic problems, work around exists Documentation Relates to Salt documentation labels Nov 2, 2018
@rallytime
Copy link
Contributor

@isbm I think use_superseded was your addition. Would you be willing to add documentation for this?

@cr1st1p
Copy link

cr1st1p commented Jan 26, 2019

#51349 Module.run doc also needs to be improved.

@devurandom
Copy link

While you are at it, could you please also add documentation on how to set use_superseded for salt-ssh?

@stale
Copy link

stale bot commented Jan 9, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Jan 9, 2020
@mattayes
Copy link

bump

@stale
Copy link

stale bot commented Jan 10, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Jan 10, 2020
@cr1st1p
Copy link

cr1st1p commented Jan 12, 2020

See also #53504 : Keeping old behavior always on needs to happen, or somehow solve situation with mixed style (think 3rd party saltstack formulas)
And when that kind of situation is solved, document it.

@stale
Copy link

stale bot commented Feb 11, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Feb 11, 2020
@sagetherage sagetherage added the doc-request net new docs requested label Feb 11, 2020
@stale
Copy link

stale bot commented Feb 11, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Feb 11, 2020
@sagetherage sagetherage added the Confirmed Salt engineer has confirmed bug/feature - often including a MCVE label Feb 11, 2020
@sagetherage sagetherage assigned cmcmarrow and unassigned isbm Feb 11, 2020
@max-arnold
Copy link
Contributor

max-arnold commented Mar 5, 2020

For salt-ssh the trick is to use the following config option:

# /etc/salt/master.d/ssh.conf
ssh_minion_opts:
  use_superseded:
    - module.run

The same can be done in a roster file through the minion_opts option:

MINION:
  host: minion.example.com
  minion_opts:
    use_superseded:
      - module.run

The hint is here: https://docs.saltstack.com/en/latest/ref/configuration/master.html#ssh-minion-opts

@sagetherage sagetherage added severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around and removed CS-S3 severity-low 4th level, cosemtic problems, work around exists labels May 7, 2021
@sagetherage sagetherage added the Phosphorus v3005.0 Release code name and version label May 18, 2021
@sagetherage sagetherage modified the milestones: Approved, Phosphorus May 18, 2021
@Ch3LL Ch3LL removed the Phosphorus v3005.0 Release code name and version label Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Confirmed Salt engineer has confirmed bug/feature - often including a MCVE doc-request net new docs requested Documentation Relates to Salt documentation severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around time-estimate-sprint ZD The issue is related to a Zendesk customer support ticket.
Projects
None yet
Development

No branches or pull requests