From b9dd0e22865bb3819b69e8aa9f0e9fb55bbf4639 Mon Sep 17 00:00:00 2001
From: Alex Stephen <alexstephen@google.com>
Date: Wed, 16 Jan 2019 11:16:14 -0800
Subject: [PATCH] New GCP module: gcp_iam_service_account (#50681)

---
 .../cloud/google/gcp_iam_service_account.py   | 277 ++++++++++++++++++
 .../targets/gcp_iam_service_account/aliases   |   2 +
 .../gcp_iam_service_account/defaults/main.yml |   3 +
 .../gcp_iam_service_account/meta/main.yml     |   0
 .../gcp_iam_service_account/tasks/main.yml    | 113 +++++++
 5 files changed, 395 insertions(+)
 create mode 100644 lib/ansible/modules/cloud/google/gcp_iam_service_account.py
 create mode 100644 test/integration/targets/gcp_iam_service_account/aliases
 create mode 100644 test/integration/targets/gcp_iam_service_account/defaults/main.yml
 create mode 100644 test/integration/targets/gcp_iam_service_account/meta/main.yml
 create mode 100644 test/integration/targets/gcp_iam_service_account/tasks/main.yml

diff --git a/lib/ansible/modules/cloud/google/gcp_iam_service_account.py b/lib/ansible/modules/cloud/google/gcp_iam_service_account.py
new file mode 100644
index 0000000000..07e337e632
--- /dev/null
+++ b/lib/ansible/modules/cloud/google/gcp_iam_service_account.py
@@ -0,0 +1,277 @@
+#!/usr/bin/python
+# -*- coding: utf-8 -*-
+#
+# Copyright (C) 2017 Google
+# GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
+# ----------------------------------------------------------------------------
+#
+#     ***     AUTO GENERATED CODE    ***    AUTO GENERATED CODE     ***
+#
+# ----------------------------------------------------------------------------
+#
+#     This file is automatically generated by Magic Modules and manual
+#     changes will be clobbered when the file is regenerated.
+#
+#     Please read more about how to change this file at
+#     https://www.github.com/GoogleCloudPlatform/magic-modules
+#
+# ----------------------------------------------------------------------------
+
+from __future__ import absolute_import, division, print_function
+__metaclass__ = type
+
+################################################################################
+# Documentation
+################################################################################
+
+ANSIBLE_METADATA = {'metadata_version': '1.1',
+                    'status': ["preview"],
+                    'supported_by': 'community'}
+
+DOCUMENTATION = '''
+---
+module: gcp_iam_service_account
+description:
+- A service account in the Identity and Access Management API.
+short_description: Creates a GCP ServiceAccount
+version_added: 2.8
+author: Google Inc. (@googlecloudplatform)
+requirements:
+- python >= 2.6
+- requests >= 2.18.4
+- google-auth >= 1.3.0
+options:
+  state:
+    description:
+    - Whether the given object should exist in GCP
+    choices:
+    - present
+    - absent
+    default: present
+  name:
+    description:
+    - The name of the service account.
+    required: false
+  display_name:
+    description:
+    - User specified description of service account.
+    required: false
+extends_documentation_fragment: gcp
+'''
+
+EXAMPLES = '''
+- name: create a service account
+  gcp_iam_service_account:
+      name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
+
+      '
+      display_name: My Ansible test key
+      project: "test_project"
+      auth_kind: "serviceaccount"
+      service_account_file: "/tmp/auth.pem"
+      state: present
+'''
+
+RETURN = '''
+name:
+  description:
+  - The name of the service account.
+  returned: success
+  type: str
+projectId:
+  description:
+  - Id of the project that owns the service account.
+  returned: success
+  type: str
+uniqueId:
+  description:
+  - Unique and stable id of the service account.
+  returned: success
+  type: str
+email:
+  description:
+  - Email address of the service account.
+  returned: success
+  type: str
+displayName:
+  description:
+  - User specified description of service account.
+  returned: success
+  type: str
+oauth2ClientId:
+  description:
+  - OAuth2 client id for the service account.
+  returned: success
+  type: str
+'''
+
+################################################################################
+# Imports
+################################################################################
+
+from ansible.module_utils.gcp_utils import navigate_hash, GcpSession, GcpModule, GcpRequest, replace_resource_dict
+import json
+
+################################################################################
+# Main
+################################################################################
+
+
+def main():
+    """Main function"""
+
+    module = GcpModule(
+        argument_spec=dict(
+            state=dict(default='present', choices=['present', 'absent'], type='str'),
+            name=dict(type='str'),
+            display_name=dict(type='str')
+        )
+    )
+
+    if not module.params['scopes']:
+        module.params['scopes'] = ['https://www.googleapis.com/auth/iam']
+
+    state = module.params['state']
+
+    fetch = fetch_resource(module, self_link(module))
+    changed = False
+
+    if fetch:
+        if state == 'present':
+            if is_different(module, fetch):
+                update(module, self_link(module))
+                fetch = fetch_resource(module, self_link(module))
+                changed = True
+        else:
+            delete(module, self_link(module))
+            fetch = {}
+            changed = True
+    else:
+        if state == 'present':
+            fetch = create(module, collection(module))
+            changed = True
+        else:
+            fetch = {}
+
+    fetch.update({'changed': changed})
+
+    module.exit_json(**fetch)
+
+
+def create(module, link):
+    auth = GcpSession(module, 'iam')
+    return return_if_object(module, auth.post(link, resource_to_request(module)))
+
+
+def update(module, link):
+    auth = GcpSession(module, 'iam')
+    return return_if_object(module, auth.put(link, resource_to_request(module)))
+
+
+def delete(module, link):
+    auth = GcpSession(module, 'iam')
+    return return_if_object(module, auth.delete(link))
+
+
+def resource_to_request(module):
+    request = {
+        u'name': module.params.get('name'),
+        u'displayName': module.params.get('display_name')
+    }
+    request = encode_request(request, module)
+    return_vals = {}
+    for k, v in request.items():
+        if v or v is False:
+            return_vals[k] = v
+
+    return return_vals
+
+
+def fetch_resource(module, link, allow_not_found=True):
+    auth = GcpSession(module, 'iam')
+    return return_if_object(module, auth.get(link), allow_not_found)
+
+
+def self_link(module):
+    return "https://iam.googleapis.com/v1/projects/{project}/serviceAccounts/{name}".format(**module.params)
+
+
+def collection(module):
+    return "https://iam.googleapis.com/v1/projects/{project}/serviceAccounts".format(**module.params)
+
+
+def return_if_object(module, response, allow_not_found=False):
+    # If not found, return nothing.
+    if allow_not_found and response.status_code == 404:
+        return None
+
+    # If no content, return nothing.
+    if response.status_code == 204:
+        return None
+
+    try:
+        module.raise_for_status(response)
+        result = response.json()
+    except getattr(json.decoder, 'JSONDecodeError', ValueError) as inst:
+        module.fail_json(msg="Invalid JSON response with error: %s" % inst)
+
+    result = decode_response(result, module)
+
+    if navigate_hash(result, ['error', 'errors']):
+        module.fail_json(msg=navigate_hash(result, ['error', 'errors']))
+
+    return result
+
+
+def is_different(module, response):
+    request = resource_to_request(module)
+    response = response_to_hash(module, response)
+    request = decode_response(request, module)
+
+    # Remove all output-only from response.
+    response_vals = {}
+    for k, v in response.items():
+        if k in request:
+            response_vals[k] = v
+
+    request_vals = {}
+    for k, v in request.items():
+        if k in response:
+            request_vals[k] = v
+
+    return GcpRequest(request_vals) != GcpRequest(response_vals)
+
+
+# Remove unnecessary properties from the response.
+# This is for doing comparisons with Ansible's current parameters.
+def response_to_hash(module, response):
+    return {
+        u'name': response.get(u'name'),
+        u'projectId': response.get(u'projectId'),
+        u'uniqueId': response.get(u'uniqueId'),
+        u'email': response.get(u'email'),
+        u'displayName': response.get(u'displayName'),
+        u'oauth2ClientId': response.get(u'oauth2ClientId')
+    }
+
+
+def encode_request(resource_request, module):
+    """Structures the request as accountId + rest of request"""
+    account_id = resource_request['name'].split('@')[0]
+    del resource_request['name']
+    return {
+        'accountId': account_id,
+        'serviceAccount': resource_request
+    }
+
+
+def decode_response(response, module):
+    """Unstructures the request from accountId + rest of request"""
+    if 'name' not in response:
+        return response
+    response['name'] = response['name'].split('/')[-1]
+    return response
+
+
+if __name__ == '__main__':
+    main()
diff --git a/test/integration/targets/gcp_iam_service_account/aliases b/test/integration/targets/gcp_iam_service_account/aliases
new file mode 100644
index 0000000000..9812f019ca
--- /dev/null
+++ b/test/integration/targets/gcp_iam_service_account/aliases
@@ -0,0 +1,2 @@
+cloud/gcp
+unsupported
diff --git a/test/integration/targets/gcp_iam_service_account/defaults/main.yml b/test/integration/targets/gcp_iam_service_account/defaults/main.yml
new file mode 100644
index 0000000000..aa87a2a8e0
--- /dev/null
+++ b/test/integration/targets/gcp_iam_service_account/defaults/main.yml
@@ -0,0 +1,3 @@
+---
+# defaults file
+resource_name: '{{resource_prefix}}'
diff --git a/test/integration/targets/gcp_iam_service_account/meta/main.yml b/test/integration/targets/gcp_iam_service_account/meta/main.yml
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/test/integration/targets/gcp_iam_service_account/tasks/main.yml b/test/integration/targets/gcp_iam_service_account/tasks/main.yml
new file mode 100644
index 0000000000..2e06eb7f5d
--- /dev/null
+++ b/test/integration/targets/gcp_iam_service_account/tasks/main.yml
@@ -0,0 +1,113 @@
+---
+# ----------------------------------------------------------------------------
+#
+#     ***     AUTO GENERATED CODE    ***    AUTO GENERATED CODE     ***
+#
+# ----------------------------------------------------------------------------
+#
+#     This file is automatically generated by Magic Modules and manual
+#     changes will be clobbered when the file is regenerated.
+#
+#     Please read more about how to change this file at
+#     https://www.github.com/GoogleCloudPlatform/magic-modules
+#
+# ----------------------------------------------------------------------------
+# Pre-test setup
+- name: delete a service account
+  gcp_iam_service_account:
+      name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
+
+      '
+      display_name: My Ansible test key
+      project: "{{ gcp_project }}"
+      auth_kind: "{{ gcp_cred_kind }}"
+      service_account_file: "{{ gcp_cred_file }}"
+      state: absent
+#----------------------------------------------------------
+- name: create a service account
+  gcp_iam_service_account:
+      name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
+
+      '
+      display_name: My Ansible test key
+      project: "{{ gcp_project }}"
+      auth_kind: "{{ gcp_cred_kind }}"
+      service_account_file: "{{ gcp_cred_file }}"
+      state: present
+  register: result
+- name: assert changed is true
+  assert:
+    that:
+      - result.changed == true
+- name: verify that service_account was created
+  gcp_iam_service_account_facts:
+      project: "{{ gcp_project }}"
+      auth_kind: "{{ gcp_cred_kind }}"
+      service_account_file: "{{ gcp_cred_file }}"
+      scopes:
+        - https://www.googleapis.com/auth/iam
+  register: results
+- name: verify that command succeeded
+  assert:
+    that:
+      - results['items'] | length >= 1
+# ----------------------------------------------------------------------------
+- name: create a service account that already exists
+  gcp_iam_service_account:
+      name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
+
+      '
+      display_name: My Ansible test key
+      project: "{{ gcp_project }}"
+      auth_kind: "{{ gcp_cred_kind }}"
+      service_account_file: "{{ gcp_cred_file }}"
+      state: present
+  register: result
+- name: assert changed is false
+  assert:
+    that:
+      - result.changed == false
+#----------------------------------------------------------
+- name: delete a service account
+  gcp_iam_service_account:
+      name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
+
+      '
+      display_name: My Ansible test key
+      project: "{{ gcp_project }}"
+      auth_kind: "{{ gcp_cred_kind }}"
+      service_account_file: "{{ gcp_cred_file }}"
+      state: absent
+  register: result
+- name: assert changed is true
+  assert:
+    that:
+      - result.changed == true
+- name: verify that service_account was deleted
+  gcp_iam_service_account_facts:
+      project: "{{ gcp_project }}"
+      auth_kind: "{{ gcp_cred_kind }}"
+      service_account_file: "{{ gcp_cred_file }}"
+      scopes:
+        - https://www.googleapis.com/auth/iam
+  register: results
+- name: verify that command succeeded
+  assert:
+    that:
+      - results['items'] | length == 0
+# ----------------------------------------------------------------------------
+- name: delete a service account that does not exist
+  gcp_iam_service_account:
+      name: '"{{resource_name}}@{{gcp_project}}.google.com.iam.gserviceaccount.com"
+
+      '
+      display_name: My Ansible test key
+      project: "{{ gcp_project }}"
+      auth_kind: "{{ gcp_cred_kind }}"
+      service_account_file: "{{ gcp_cred_file }}"
+      state: absent
+  register: result
+- name: assert changed is false
+  assert:
+    that:
+      - result.changed == false