Merge pull request #12851 from rallytime/file_append_docs

Arg definitions added to states.file.append
fixed tests to add auto field in SafeKeep
try using name instead of eid do remote key
fixed logic bug in check_minion_cache
Merge remote-tracking branch 'upstream/develop' into sam_raet_39
Merge pull request #12847 from basepi/archiveextracteddocs12498

Add note about ``if_missing`` to archive.extracted
Fixed bug in ip_addrs
On OS X there is no 'label' field
Cleaned up logic so more readable
Merge pull request #12856 from SmithSamuelM/sam_raet_39

Fixed  a couple of bugs from latest develop. Added some changes for Raet changes
Populate 'Changes' on docker.installed container creation
removed debug line
Merge pull request #12858 from SmithSamuelM/sam_raet_39

removed debug line
Move mine_interval example to a minion conf file

mine_interval has no effect when added in a pillar
Change default output_loglevel to debug
Remove output_loglevel='debug'

With the default loglevel now set to debug, setting it to debug when
invoking cmd.* is redundant.
Fixed rST syntax that was causing the PDF not to build
Merge pull request #12863 from whiteinge/doc-rst-latex-fixes

Fixed rST syntax that was causing the PDF not to build
Added note when using the loader to run modules that contact the master

Closes #12732
Merge pull request #12864 from whiteinge/doc-loader-note-keyerror-master_uri

Added note when using the loader to run modules that contact the master
Merge pull request #12860 from terminalmage/change-output-loglevel

Change default output loglevel to debug
Merge pull request #12846 from basepi/serviceenablefreebsd12836

Ensure that freebsd service rc.conf ends in newline before appending
Merge pull request #12841 from rogst/develop

Added module and state for adding new firewall rules