fix tests

Author: rockspore

src/python/grpcio/grpc/_cython/_cygrpc/credentials.pyx.pxi

@@ -204,10 +204,10 @@ cdef class SSLChannelCredentials(ChannelCredentials):
     cdef grpc_tls_certificate_provider* c_tls_certificate_provider
 
     c_tls_credentials_options = grpc_tls_credentials_options_create()
-    if self._pem_root_certificates is not None:
-      c_pem_root_certificates = self._pem_root_certificates
-    else:
+    if self._pem_root_certificates is None:
       c_pem_root_certificates = NULL
+    else:
+      c_pem_root_certificates = self._pem_root_certificates
     if self._private_key or self._certificate_chain:
       c_tls_identity_pairs = grpc_tls_identity_pairs_create()
       if self._private_key:
@@ -223,6 +223,10 @@ cdef class SSLChannelCredentials(ChannelCredentials):
       c_tls_certificate_provider = grpc_tls_certificate_provider_static_data_create(c_pem_root_certificates, c_tls_identity_pairs)
       grpc_tls_credentials_options_set_certificate_provider(c_tls_credentials_options, c_tls_certificate_provider)
       grpc_tls_certificate_provider_release(c_tls_certificate_provider)
+      if c_pem_root_certificates != NULL:
+        grpc_tls_credentials_options_watch_root_certs(c_tls_credentials_options)
+      if c_tls_identity_pairs != NULL:
+        grpc_tls_credentials_options_watch_identity_key_cert_pairs(c_tls_credentials_options)
     with nogil:
       return grpc_tls_credentials_create(c_tls_credentials_options)
 

src/python/grpcio/grpc/_cython/_cygrpc/grpc.pxi

@@ -589,33 +589,39 @@ cdef extern from "grpc/credentials.h":
     # We don't care about the internals (and in fact don't know them)
     pass
 
-  grpc_tls_credentials_options *grpc_tls_credentials_options_create()
+  grpc_tls_credentials_options *grpc_tls_credentials_options_create() nogil
 
   ctypedef struct grpc_tls_certificate_provider:
     # We don't care about the internals (and in fact don't know them)
     pass
 
-  void grpc_tls_credentials_options_set_certificate_provider(grpc_tls_credentials_options* options, grpc_tls_certificate_provider* provider)
+  void grpc_tls_credentials_options_set_certificate_provider(grpc_tls_credentials_options* options, grpc_tls_certificate_provider* provider) nogil
 
   ctypedef struct grpc_tls_identity_pairs:
     # We don't care about the internals (and in fact don't know them)
     pass
 
-  grpc_tls_identity_pairs* grpc_tls_identity_pairs_create();
+  grpc_tls_identity_pairs* grpc_tls_identity_pairs_create() nogil
 
   void grpc_tls_identity_pairs_add_pair(grpc_tls_identity_pairs* pairs,
                                               const char* private_key,
-                                              const char* cert_chain);
+                                              const char* cert_chain) nogil
 
   grpc_tls_certificate_provider* grpc_tls_certificate_provider_static_data_create(
-    const char* root_certificate, grpc_tls_identity_pairs* pem_key_cert_pairs);
+    const char* root_certificate, grpc_tls_identity_pairs* pem_key_cert_pairs) nogil
 
   void grpc_tls_credentials_options_set_certificate_provider(
     grpc_tls_credentials_options* options,
-    grpc_tls_certificate_provider* provider);
+    grpc_tls_certificate_provider* provider) nogil
+  
+  void grpc_tls_credentials_options_watch_root_certs(
+    grpc_tls_credentials_options* options) nogil
+
+  void grpc_tls_credentials_options_watch_identity_key_cert_pairs(
+    grpc_tls_credentials_options* options) nogil
 
   void grpc_tls_certificate_provider_release(
-    grpc_tls_certificate_provider* provider);
+    grpc_tls_certificate_provider* provider) nogil
 
   ctypedef struct grpc_channel_credentials:
     # We don't care about the internals (and in fact don't know them)