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

Copy+paste errors in postgres docs #51213

Open
OrangeDog opened this issue Jan 17, 2019 · 4 comments
Open

Copy+paste errors in postgres docs #51213

OrangeDog opened this issue Jan 17, 2019 · 4 comments
Labels
Bug broken, incorrect, or confusing behavior Confirmed Salt engineer has confirmed bug/feature - often including a MCVE Documentation Relates to Salt documentation severity-low 4th level, cosemtic problems, work around exists time-estimate-sprint
Milestone
Approved

Comments

@OrangeDog
Copy link
Contributor

In particluar, the maintainance_db parameter is documented incorrectly for most states (having been copied from postgres_language), but the whole set probably needs a look.

https://github.com/saltstack/salt/blob/b3c1be2/salt/states/postgres_privileges.py#L143
@dwoz
Copy link
Contributor

dwoz commented Jan 17, 2019

@OrangeDog Thanks for reporting this. Are you able to make a PR with the corrections?

@dwoz dwoz added Documentation Relates to Salt documentation Bug broken, incorrect, or confusing behavior P4 Priority 4 severity-low 4th level, cosemtic problems, work around exists labels Jan 17, 2019
@dwoz dwoz added this to the Approved milestone Jan 17, 2019
@OrangeDog
Copy link
Contributor Author

No, because I don't know what the correct documentation should be.

@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
@waynew waynew added the Confirmed Salt engineer has confirmed bug/feature - often including a MCVE label Jan 9, 2020
@stale
Copy link

stale bot commented Jan 9, 2020

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

@stale stale bot removed the stale label Jan 9, 2020
@sagetherage sagetherage removed the P4 Priority 4 label Jun 3, 2020
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 Documentation Relates to Salt documentation severity-low 4th level, cosemtic problems, work around exists time-estimate-sprint
Projects
None yet
Milestone
Approved
Development

No branches or pull requests
5 participants
@waynew @OrangeDog @dwoz @sagetherage @barbaricyawps