Bug fixes for GCP modules (#64827)

lib/ansible/modules/cloud/google/gcp_mlengine_model.py

@@ -70,7 +70,7 @@ options:
       name:
         description:
         - The name specified for the version when it was created.
-        required: false
+        required: true
         type: str
   regions:
     description:
@@ -214,7 +214,7 @@ def main():
             state=dict(default='present', choices=['present', 'absent'], type='str'),
             name=dict(required=True, type='str'),
             description=dict(type='str'),
-            default_version=dict(type='dict', options=dict(name=dict(type='str'))),
+            default_version=dict(type='dict', options=dict(name=dict(required=True, type='str'))),
             regions=dict(type='list', elements='str'),
             online_prediction_logging=dict(type='bool'),
             online_prediction_console_logging=dict(type='bool'),

lib/ansible/modules/cloud/google/gcp_pubsub_subscription.py

@@ -175,7 +175,7 @@ options:
     - A subscription is considered active as long as any connected subscriber is successfully
       consuming messages from the subscription or is issuing operations on the subscription.
       If expirationPolicy is not set, a default policy with ttl of 31 days will be
-      used. If it is set but left empty, the resource never expires. The minimum allowed
+      used. If it is set but ttl is "", the resource never expires. The minimum allowed
       value for expirationPolicy.ttl is 1 day.
     required: false
     type: dict
@@ -188,7 +188,7 @@ options:
         - If ttl is not set, the associated resource never expires.
         - A duration in seconds with up to nine fractional digits, terminated by 's'.
         - Example - "3.5s".
-        required: false
+        required: true
         type: str
   project:
     description:
@@ -379,7 +379,7 @@ expirationPolicy:
   - A subscription is considered active as long as any connected subscriber is successfully
     consuming messages from the subscription or is issuing operations on the subscription.
     If expirationPolicy is not set, a default policy with ttl of 31 days will be used.
-    If it is set but left empty, the resource never expires. The minimum allowed value
+    If it is set but ttl is "", the resource never expires. The minimum allowed value
     for expirationPolicy.ttl is 1 day.
   returned: success
   type: complex
@@ -428,7 +428,7 @@ def main():
             ack_deadline_seconds=dict(type='int'),
             message_retention_duration=dict(default='604800s', type='str'),
             retain_acked_messages=dict(type='bool'),
-            expiration_policy=dict(type='dict', options=dict(ttl=dict(type='str'))),
+            expiration_policy=dict(type='dict', options=dict(ttl=dict(required=True, type='str'))),
         )
     )
 

lib/ansible/modules/cloud/google/gcp_pubsub_subscription_info.py

@@ -221,7 +221,7 @@ resources:
       - A subscription is considered active as long as any connected subscriber is
         successfully consuming messages from the subscription or is issuing operations
         on the subscription. If expirationPolicy is not set, a default policy with
-        ttl of 31 days will be used. If it is set but left empty, the resource never
+        ttl of 31 days will be used. If it is set but ttl is "", the resource never
         expires. The minimum allowed value for expirationPolicy.ttl is 1 day.
       returned: success
       type: complex