fix: Return Resource.K8sContainer when autodetecting resource type (#641)

wildhemp · web-flow · commit 6103560c1c59 · 2021-09-16T10:23:40.000-07:00
According to https://cloud.google.com/stackdriver/docs/solutions/gke/migration#resource-changes container is legacy the new type is k8s_container.
diff --git a/google-cloud-logging/src/main/java/com/google/cloud/logging/MonitoredResourceUtil.java b/google-cloud-logging/src/main/java/com/google/cloud/logging/MonitoredResourceUtil.java
@@ -226,7 +226,7 @@ private static Resource getAutoDetectedResourceType() {
       return Resource.GaeAppFlex;
     }
     if (System.getenv("KUBERNETES_SERVICE_HOST") != null) {
-      return Resource.Container;
+      return Resource.K8sContainer;
     }
     if (ServiceOptions.getAppEngineAppId() != null) {
       return Resource.GaeAppStandard;

Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,7 @@ private static Resource getAutoDetectedResourceType() {`
`226`	`226`	`return Resource.GaeAppFlex;`
`227`	`227`	`}`
`228`	`228`	`if (System.getenv("KUBERNETES_SERVICE_HOST") != null) {`
`229`		`- return Resource.Container;`
	`229`	`+ return Resource.K8sContainer;`
`230`	`230`	`}`
`231`	`231`	`if (ServiceOptions.getAppEngineAppId() != null) {`
`232`	`232`	`return Resource.GaeAppStandard;`