Merge branch 'main' into add-aggregation-selector-to-otlp-metric-expo…

…rter
open-telemetry · Jan 18, 2024 · d689f47 · d689f47
2 parents efe9a55 + 2a3c264
commit d689f47
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -20,6 +20,7 @@ For experimental package changes, see the [experimental CHANGELOG](experimental/
 ### :house: (Internal)
 
 * chore(opentelemetry-context-zone-peer-dep): support zone.js ^v0.13.0 [#4320](https://github.com/open-telemetry/opentelemetry-js/pull/4320)
+* refactor(core): drop unnecessary assignment of HOSTNAME [#4421](https://github.com/open-telemetry/opentelemetry-js/pull/4421) @pichlermarc
 
 ## 1.20.0
 

diff --git a/packages/opentelemetry-core/src/platform/node/environment.ts b/packages/opentelemetry-core/src/platform/node/environment.ts
@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 
-import * as os from 'os';
 import {
   DEFAULT_ENVIRONMENT,
   ENVIRONMENT,
@@ -27,11 +26,5 @@ import {
  */
 export function getEnv(): Required<ENVIRONMENT> {
   const processEnv = parseEnvironment(process.env as RAW_ENVIRONMENT);
-  return Object.assign(
-    {
-      HOSTNAME: os.hostname(),
-    },
-    DEFAULT_ENVIRONMENT,
-    processEnv
-  );
+  return Object.assign({}, DEFAULT_ENVIRONMENT, processEnv);
 }