Leverage 4318b27 and no longer treat instance tags as JSON, but as dictionary types which could be key=value or dicts or JSON or references.

Revert 4833c2fd

Corrected docs ("monitor" -> "monitoring" and dissociated monitoring from instance_tags in examples)

The code to determine the number of running instances could blow up
if the response from AWS did not actually contain any data. This code
makes it a bit smarter in handling, so that it will wait for a valid
response regardless of the wait condition.

Fixes #3980

No idea if I'm rebasing properly or not.  This is my first attempt.

Adds some additional docs about the module parameters for s3
and ec2 modules.

Add some details in the docs about how the id parameter of the ec2
module behaves.

Remove the reference to the unused "termination_list" parameter
in the ec2 module. The instance_ids parameter is the one that contains
the list of instance ids to be terminated.

In the ec2 module, if an id is specified, check if there have been
any changes. If not, return changed=False

Fixes #3746

Pass in the `instances` output of the ec2 module
to terminate a list of instances that were previously provisioned.

Useful for automated testing.