Repository: ansible/ansible-modules-extras

Issue:
efs and efs_facts modules documentation malformed
##### ISSUE TYPE
 - Documentation Report

##### COMPONENT NAME
efs
efs_facts

##### ANSIBLE VERSION
<!--- Paste verbatim output from “ansible --version” between quotes below -->
```
ansible 2.2.0.0
  config file =
  configured module search path = Default w/o overrides
```

##### CONFIGURATION


##### OS / ENVIRONMENT
N/A

##### SUMMARY
Table of options is malformed due to use of | operator in the description of a couple parameters.

##### STEPS TO REPRODUCE


<!--- Paste example playbooks or commands between quotes below -->
```
EXAMPLE CODE:

    tags:
        description:
            - |
                List of tags of Amazon EFS. Should be defined as dictionary
                In case of 'present' state with list of tags and existing EFS (matched by 'name'), tags of EFS will be replaced with provided data.
        required: false
        default: None

```

##### EXPECTED RESULTS
Table would be properly formatted

##### ACTUAL RESULTS
Table comes out malformed with raw HTML in the output. 



Discussion:

Resolving patch:
diff --git a/cloud/amazon/efs.py b/cloud/amazon/efs.py
--- a/cloud/amazon/efs.py
+++ b/cloud/amazon/efs.py
@@ -50,26 +50,23 @@
         choices: ['general_purpose', 'max_io']
     tags:
         description:
-            - |
-                List of tags of Amazon EFS. Should be defined as dictionary
-                In case of 'present' state with list of tags and existing EFS (matched by 'name'), tags of EFS will be replaced with provided data.
+            - "List of tags of Amazon EFS. Should be defined as dictionary
+              In case of 'present' state with list of tags and existing EFS (matched by 'name'), tags of EFS will be replaced with provided data."
         required: false
         default: None
     targets:
         description:
-            - |
-                List of mounted targets. It should be a list of dictionaries, every dictionary should include next attributes:
-                    - subnet_id - Mandatory. The ID of the subnet to add the mount target in.
-                    - ip_address - Optional. A valid IPv4 address within the address range of the specified subnet.
-                    - security_groups - Optional. List of security group IDs, of the form "sg-xxxxxxxx". These must be for the same VPC as subnet specified
-                This data may be modified for existing EFS using state 'present' and new list of mount targets.
+            - "List of mounted targets. It should be a list of dictionaries, every dictionary should include next attributes:
+                   - subnet_id - Mandatory. The ID of the subnet to add the mount target in.
+                   - ip_address - Optional. A valid IPv4 address within the address range of the specified subnet.
+                   - security_groups - Optional. List of security group IDs, of the form 'sg-xxxxxxxx'. These must be for the same VPC as subnet specified
+               This data may be modified for existing EFS using state 'present' and new list of mount targets."
         required: false
         default: None
     wait:
         description:
-            - |
-                In case of 'present' state should wait for EFS 'available' life cycle state (of course, if current state not 'deleting' or 'deleted')
-                In case of 'absent' state should wait for EFS 'deleted' life cycle state
+            - "In case of 'present' state should wait for EFS 'available' life cycle state (of course, if current state not 'deleting' or 'deleted')
+               In case of 'absent' state should wait for EFS 'deleted' life cycle state"
         required: false
         default: "no"
         choices: ["yes", "no"]
diff --git a/cloud/amazon/efs_facts.py b/cloud/amazon/efs_facts.py
--- a/cloud/amazon/efs_facts.py
+++ b/cloud/amazon/efs_facts.py
@@ -37,17 +37,15 @@
         default: None
     tags:
         description:
-            - |
-                List of tags of Amazon EFS. Should be defined as dictionary
+            - List of tags of Amazon EFS. Should be defined as dictionary
         required: false
         default: None
     targets:
         description:
-            - |
-                List of mounted targets. It should be a list of dictionaries, every dictionary should include next attributes:
+            - "List of mounted targets. It should be a list of dictionaries, every dictionary should include next attributes:
                     - SubnetId - Mandatory. The ID of the subnet to add the mount target in.
                     - IpAddress - Optional. A valid IPv4 address within the address range of the specified subnet.
-                    - SecurityGroups - Optional. List of security group IDs, of the form "sg-xxxxxxxx". These must be for the same VPC as subnet specified.
+                    - SecurityGroups - Optional. List of security group IDs, of the form 'sg-xxxxxxxx'. These must be for the same VPC as subnet specified."
         required: false
         default: None
 extends_documentation_fragment:


Test patch: