Rework of more_time feature

In the past this feature has basically checked returns, pinged minions if they were still running the job, then returning once the job was no longer run remotely. This has caused some issues with race conditions if the minion or master are busy (and the time between minion done, and master recieved gets large).

This changes the behavior of the "wait" mechanism. The intent of this feature is to wait for the job return from all minions the job went to. To accomplish this we fire the job, then check the returns. If the initial returns are what we wanted/expected we break. If it isn't we will ping all targets and see if its still running. If more minions return, we merge them into the list of minions we are waiting for. We will timeout the job once all minions have stopped running the job and TIMEOUT seconds have passed (since the slowest minion)
Pylint cleanup
Initial interaction with F5 Load Balancers
Start pinging minions after TIMEOUT, instead of immediately
Add a timeout to get_event_iter_returns, the old implementation won't timeout if the event buss is relatively busy
Typo fix
salt.states.file.copy now copies file user/group ownership and permissions from source
Reordering ret statement
Rework of more_time feature

In the past this feature has basically checked returns, pinged minions if they were still running the job, then returning once the job was no longer run remotely. This has caused some issues with race conditions if the minion or master are busy (and the time between minion done, and master recieved gets large).

This changes the behavior of the "wait" mechanism. The intent of this feature is to wait for the job return from all minions the job went to. To accomplish this we fire the job, then check the returns. If the initial returns are what we wanted/expected we break. If it isn't we will ping all targets and see if its still running. If more minions return, we merge them into the list of minions we are waiting for. We will timeout the job once all minions have stopped running the job and TIMEOUT seconds have passed (since the slowest minion)

Conflicts:

	salt/client/__init__.py
Pylint cleanup
Start pinging minions after TIMEOUT, instead of immediately
Add a timeout to get_event_iter_returns, the old implementation won't timeout if the event buss is relatively busy
Typo fix
Remove unused refresh argument

Closes #16118
Fix - Removed the line break (\n) character at the end of the pillar data that was inserted by the python open() function; Added a booleen variable called raw_data, which if True will return to the old behavior and add a line break at the end of the pillar data
avoid creating unparseable lxc config param: lxc.network.mac
Adjusted the documentation for file tree following the addition of the line break fix
remove unused fscache-files from #15230, removed in favour of #15382
lxc module: don't reinvent the wheel -- use lxc-attach built-in args for env and use pipes.quote instead of manual quoting
Ensure that mount.mounted modifies entries in fstab on changes

mount.mounted compared entries in fstab only by their fs_spec to determine if an
entry that is about to be added was already present and that the original one
should therefore be modified. It is, however, necessary to compare both fs_spec
and fs_file fields to allow users to mount new filesystems or block devices to
the same mountpoint.

This fixes #15968 which had already been fixed in #9573 but was re-introduced in
c8c6112 due to, what I believe, not rebasing that PR against develop.