test_system.TestKMSIntegration.test_bucket_w_default_kms_key_name failed #190

Activity

1. IlyaFaer

   added

   api: storageIssues related to the googleapis/python-storage API.Issues related to the googleapis/python-storage API.

   triage meI really want to be triaged.I really want to be triaged.

   on Jun 23, 2020
2. IlyaFaer

   changed the title [-]test_system.TestKMSIntegration testMethod=test_bucket_w_default_kms_key_name failed[/-] [+]test_system.TestKMSIntegration.test_bucket_w_default_kms_key_name failed[/+] on Jun 23, 2020
3. yoshi-automation

   added

   🚨This issue needs some love.This issue needs some love.

   on Jun 28, 2020
4. frankyn

   added

   type: cleanupAn internal cleanup or hygiene concern.An internal cleanup or hygiene concern.

   and removed

   🚨This issue needs some love.This issue needs some love.

   triage meI really want to be triaged.I really want to be triaged.

   on Jun 29, 2020
5. frankyn commented on Jun 29, 2020

   

   frankyn

   on Jun 29, 2020

   Contributor

   More actions

   def test_bucket_w_default_kms_key_name(self):
           BLOB_NAME = "default-kms-key-name"
           OVERRIDE_BLOB_NAME = "override-default-kms-key-name"
           ALT_BLOB_NAME = "alt-default-kms-key-name"
           CLEARTEXT_BLOB_NAME = "cleartext"
   
           file_data = self.FILES["simple"]
   
           with open(file_data["path"], "rb") as _file_data:
               contents = _file_data.read()
   
           kms_key_name = self._kms_key_name()
           self.bucket.default_kms_key_name = kms_key_name
           self.bucket.patch()
           self.assertEqual(self.bucket.default_kms_key_name, kms_key_name)
   
           defaulted_blob = self.bucket.blob(BLOB_NAME)
           defaulted_blob.upload_from_filename(file_data["path"])
           self.case_blobs_to_delete.append(defaulted_blob)
   
           self.assertEqual(defaulted_blob.download_as_string(), contents)
           # We don't know the current version of the key.
           self.assertTrue(defaulted_blob.kms_key_name.startswith(kms_key_name))
   
           alt_kms_key_name = self._kms_key_name(self.ALT_KEY_NAME)
   
           override_blob = self.bucket.blob(
               OVERRIDE_BLOB_NAME, kms_key_name=alt_kms_key_name
           )
           override_blob.upload_from_filename(file_data["path"])
           self.case_blobs_to_delete.append(override_blob)
   
           self.assertEqual(override_blob.download_as_string(), contents)
           # We don't know the current version of the key. 
           self.assertTrue(override_blob.kms_key_name.startswith(alt_kms_key_name))
   
           self.bucket.default_kms_key_name = alt_kms_key_name
           self.bucket.patch()
   
           # this may be eventually consistent for bucket default_kms_key_name to take effect. 
           # Per: https://cloud.google.com/storage/docs/consistency
           # "For buckets, while metadata updates are strongly consistent for read-after-metadata-update operations, the resulting configuration changes may take time to propagate."
           # {
           alt_blob = self.bucket.blob(ALT_BLOB_NAME)
           alt_blob.upload_from_filename(file_data["path"])
           self.case_blobs_to_delete.append(alt_blob)
   
           self.assertEqual(alt_blob.download_as_string(), contents)
           # We don't know the current version of the key.
          self.assertTrue(alt_blob.kms_key_name.startswith(alt_kms_key_name))
           # } 
6. frankyn

   assigned

   IlyaFaer

   on Jun 29, 2020
7. andrewsg

   closed this as completedin #198on Jul 1, 2020