Salt: reload_modules is not documented for the pkg state

[mitar]

It seems that reload_modules exists also for pkg.installed, but it is not documented. While it is documented for cmd.run.

[mitar]

cc @kostko

[ssgward]

Thanks for the issue report.

[ltsampros]

I think this bug can be closed. reload_modules is documented for most versions here: https://docs.saltstack.com/en/2014.7/ref/states/

[rallytime]

Yes, I think so. The reload documentation is located here and here on the 2015.8 branch.

@mitar I think the documentation you're referring to in your first comment is the note at the bottom of the cmd.run state function. Do I have that right? Are you asking that the same note be applied to something like the pkg.installed state? The reload_modules is a global requisite and can be applied to any state. From the second doc link:

reload_modules is a boolean option recognized by salt on all available states which forces salt to reload its modules once a given state finishes.

I think we now have quite a bit of documentation about this option, however, I don't want to close this without your input. Let me know what you think. Thanks!

[mitar]

Yes, I understand that this is a global option, but such a note would be very useful for pkg.installed as well. It is probably one of the main cases where one would like/need to use reload_modules.

[rallytime]

Ok we can do that. I just wanted to make sure that was what you meant.

[rallytime]

@mitar Ok, I have added that note to the the pkg.installed docs. Thanks for the clarification!

[mitar]

Thanks!!!

[mitar]

Thanks!