From 17c7d6116a3e79bc1179645f271776f513a81b1f Mon Sep 17 00:00:00 2001
From: zhangbailin <zhangbailin@inspur.com>
Date: Mon, 5 Nov 2018 03:03:09 -0500
Subject: [PATCH] Add user_id attribute to the backup response

Add ``user_id`` attribute to the response body of
list backup with detail and show backup detail APIs.

Blueprint: add-user-id-attribute-to-backup-response
Change-Id: I26b9e98af0d1c7230959a392c147a8256bf59638
---
 ...d-user-id-attribute-to-backup-response.rst | 158 ++++++++++++++++++
 1 file changed, 158 insertions(+)
 create mode 100644 specs/stein/add-user-id-attribute-to-backup-response.rst

diff --git a/specs/stein/add-user-id-attribute-to-backup-response.rst b/specs/stein/add-user-id-attribute-to-backup-response.rst
new file mode 100644
index 00000000..d0ebcc19
--- /dev/null
+++ b/specs/stein/add-user-id-attribute-to-backup-response.rst
@@ -0,0 +1,158 @@
+..
+   This work is licensed under a Creative Commons Attribution 3.0 Unported
+ License.
+
+ http://creativecommons.org/licenses/by/3.0/legalcode
+
+========================================
+Add user_id attribute to backup response
+========================================
+https://blueprints.launchpad.net/cinder/+spec/add-user-id-attribute-to-backup-response
+
+This blueprint proposes to add ``user_id`` attribute to
+the response body of list backup with detail and show backup detail APIs.
+
+Problem description
+===================
+
+Currently, there is a ``user_id`` field in the ``backups`` table, these
+fields are very useful for admin to manage the backup file, but this
+is not returned in response body. So it is difficult to manage the resources
+under the project. If there are multiple users under one project, it is
+impossible to distinguish which user the backup file belongs to.
+
+Use Cases
+=========
+
+In large scale environment, lots of backups resources were created in system,
+that we can only see the project to which the backup file belongs, but we
+cannot know which user the backup belongs to it.
+
+In order to better manage the backup files, add ``user_id``
+attribute to the response body of list backup with detail and show backup
+detail APIs, make the backup resources more visible to the users of the
+project.
+
+Proposed change
+===============
+
+This spec proposes that add ``user_id`` attribute to the
+response body of list backup with detail and show backup detail APIs.
+
+Add a new microverion API to add ``user_id`` attribute
+to the response body of list backup with detail and show backup detail APIs:
+
+- List backups with detail GET /v3/{project_id}/backups/detail
+
+- Show backup detail GET /v3/{project_id}/backups/{backup_id}
+
+Alternatives
+------------
+
+The admin/user could get ``user_id`` from the context as a log print, but
+it's difficult to find it out easily, especially when the user wants to find
+a very old backup file.
+
+REST API impact
+---------------
+
+* Add a new microversion in Cinder API.
+
+* List backups with detail::
+
+    GET /v3/{project_id}/backups/detail
+    Response BODY:
+    {
+        "backups": [{
+            ...
+            "user_id": "515ba0dd59f84f25a6a084a45d8d93b2"
+        }]
+    }
+
+* Show backup detail::
+
+    GET /v3/{project_id}/backups/{backup_id}
+    Response BODY:
+    {
+        "backups": [{
+            ...
+            "user_id": "515ba0dd59f84f25a6a084a45d8d93b2"
+        }]
+    }
+
+Calling this method shows a ``user_id`` for volume backup.
+It is intended for admins to use, which is used to display the user to which
+the backup file belongs, and controlled by ``BACKUP_ATTRIBUTES_POLICY``.
+
+Data model impact
+-----------------
+
+None
+
+Security impact
+---------------
+
+None
+
+Notifications impact
+--------------------
+
+None
+
+Other end user impact
+---------------------
+
+None
+
+Performance Impact
+------------------
+
+None
+
+Other deployer impact
+---------------------
+
+None
+
+Developer impact
+----------------
+
+None
+
+Implementation
+==============
+
+Assignee(s)
+-----------
+
+Primary assignee:
+  Brin Zhang <zhangbailin@inspur.com>
+
+Work Items
+----------
+
+* Add a new microversion.
+* Add ``user_id`` to the response body of list backup
+  with detail and show backup detail APIs.
+* Add the related unit tests.
+* Update related list backup with detail and show detail api doc.
+
+Dependencies
+============
+
+None
+
+Testing
+=======
+
+* Unit-tests, tempest and other related test should be implemented
+
+Documentation Impact
+====================
+
+None
+
+References
+==========
+
+None