This is an automated email from the ASF dual-hosted git repository.

shunping pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new cbc31be7bbc Reuse secret instead of creating every time (#36955)
cbc31be7bbc is described below

commit cbc31be7bbc770231dfbc76ebdbc5f6e64db6b65
Author: Danny McCormick <[email protected]>
AuthorDate: Tue Dec 2 09:43:21 2025 -0500

    Reuse secret instead of creating every time (#36955)
    
    * Reuse secret instead of creating every time
    
    * Reuse secret instead of creating every time
    
    * lint
    
    * lint
---
 sdks/python/apache_beam/transforms/util_test.py | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/sdks/python/apache_beam/transforms/util_test.py 
b/sdks/python/apache_beam/transforms/util_test.py
index 34e251fad1c..3dc089134a3 100644
--- a/sdks/python/apache_beam/transforms/util_test.py
+++ b/sdks/python/apache_beam/transforms/util_test.py
@@ -28,7 +28,6 @@ import logging
 import math
 import random
 import re
-import sys
 import time
 import unittest
 import warnings
@@ -320,9 +319,7 @@ class GroupByEncryptedKeyTest(unittest.TestCase):
   def setUpClass(cls):
     if secretmanager is not None:
       cls.project_id = 'apache-beam-testing'
-      py_version = f'_py{sys.version_info.major}{sys.version_info.minor}'
-      secret_postfix = datetime.now().strftime('%m%d_%H%M%S') + py_version
-      cls.secret_id = 'gbek_util_secret_tests_' + secret_postfix
+      cls.secret_id = 'gbek_util_secret_tests'
       cls.client = secretmanager.SecretManagerServiceClient()
       cls.project_path = f'projects/{cls.project_id}'
       cls.secret_path = f'{cls.project_path}/secrets/{cls.secret_id}'
@@ -350,11 +347,6 @@ class GroupByEncryptedKeyTest(unittest.TestCase):
       cls.gcp_secret = GcpSecret(version_name)
       cls.secret_option = f'type:GcpSecret;version_name:{version_name}'
 
-  @classmethod
-  def tearDownClass(cls):
-    if secretmanager is not None:
-      cls.client.delete_secret(request={'name': cls.secret_path})
-
   def test_gbek_fake_secret_manager_roundtrips(self):
     fakeSecret = FakeSecret()
 

Reply via email to