use UNAVAILABLE if CDS resource is not okay

markdroth · Aug 28, 2024 · 0c7bb49 · 0c7bb49
1 parent 5db1d75
commit 0c7bb49
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/core/ext/filters/gcp_authentication/gcp_authentication_filter.cc b/src/core/ext/filters/gcp_authentication/gcp_authentication_filter.cc
@@ -75,7 +75,7 @@ absl::Status GcpAuthenticationFilter::Call::OnClientInitialMetadata(
     // because once that happens, a new instance of this filter will be
     // swapped in for subsequent calls, but *this* call is already tied
     // to this filter instance, which will never see the update.
-    return absl::UnauthenticatedError(
+    return absl::UnavailableError(
         absl::StrCat("GCP authentication filter: CDS resource unavailable for ",
                      cluster_name));
   }

diff --git a/test/core/filters/gcp_authentication_filter_test.cc b/test/core/filters/gcp_authentication_filter_test.cc
@@ -249,7 +249,7 @@ TEST_F(GcpAuthenticationFilterTest, FailsCallIfClusterNotOkayInXdsConfig) {
   service_config_call_data->SetCallAttribute(&xds_cluster_attribute);
   call.Start(call.NewClientMetadata());
   EXPECT_EVENT(Finished(
-      &call, HasMetadataResult(absl::UnauthenticatedError(absl::StrCat(
+      &call, HasMetadataResult(absl::UnavailableError(absl::StrCat(
                  "GCP authentication filter: CDS resource unavailable for ",
                  kClusterName)))));
   Step();