From 97d6ef7c1f32faff037da808459cfb72ca9e0e33 Mon Sep 17 00:00:00 2001
From: Srini Brahmaroutu <srbrahma@us.ibm.com>
Date: Tue, 4 Aug 2020 20:55:55 -0700
Subject: [PATCH] Fixing GRPC API as per discussion in today's meeting (#22)

---
 .../20191125-bucket-provisioning.md           | 24 +++++--------------
 1 file changed, 6 insertions(+), 18 deletions(-)

diff --git a/keps/sig-storage/20191125-bucket-provisioning.md b/keps/sig-storage/20191125-bucket-provisioning.md
index f0b78788a31..37f4152c513 100644
--- a/keps/sig-storage/20191125-bucket-provisioning.md
+++ b/keps/sig-storage/20191125-bucket-provisioning.md
@@ -566,12 +566,9 @@ This call is made to create the bucket in the backend. If the bucket already exi
 
 ```
 message ProvisionerCreateBucketRequest {    
+    // This field is REQUIRED
     string bucket_name = 1;
 
-    string region = 2;
-
-    string zone = 3;
-
     map<string,string> bucket_context = 4;
 
     enum AnonymousBucketAccessMode {
@@ -582,8 +579,6 @@ message ProvisionerCreateBucketRequest {
     }
     
     AnonymousBucketAccessMode anonymous_bucket_access_mode = 5;
-
-    map<string,string> secrets = 6;
 }
 
 message ProvisionerCreateBucketResponse {
@@ -597,12 +592,9 @@ This call is made to delete the bucket in the backend. If the bucket has already
 
 ```
 message ProvisionerDeleteBucketRequest {
+    // This field is REQUIRED
     string bucket_name = 1;
     
-    string region = 2;
-
-    string zone = 3;
-
     map<string,string> bucket_context = 4;    
 }
 
@@ -621,16 +613,14 @@ If the `principal` is empty, then a new service account should be created in the
 
 ```
 message ProvisionerGrantBucketAccessRequest {
+    // This field is REQUIRED
     string bucket_name = 1;
     
-    string region = 2;
-
-    string zone = 3;
-
     map<string,string> bucket_context = 4;  
 
     string principal = 5;
     
+    // This field is REQUIRED
     string access_policy = 6;
 }
 
@@ -651,14 +641,12 @@ This call revokes all access to a particular bucket from a principal.
 
 ```
 message ProvisionerRevokeBucketAccessRequest {
+    // This field is REQUIRED
     string bucket_name = 1;
     
-    string region = 2;
-
-    string zone = 3;
-
     map<string,string> bucket_context = 4;  
 
+    // This field is REQUIRED
     string principal = 5;
 }