Skip to content

Commit

Permalink
[chore] Move memorylimiter dependencies under one module (open-teleme…
Browse files Browse the repository at this point in the history
…try#11419)

This PR only moves all the internal code used by internal/memorylimiter
under the same directory, next PR will extract that in a different
module to limit dependencies.

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
  • Loading branch information
bogdandrutu authored Oct 12, 2024
1 parent be5fc23 commit 35d4156
Show file tree
Hide file tree
Showing 44 changed files with 14 additions and 14 deletions.
2 changes: 1 addition & 1 deletion extension/memorylimiterextension/memorylimiter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@ import (
"go.uber.org/zap"

"go.opentelemetry.io/collector/component"
"go.opentelemetry.io/collector/internal/iruntime"
"go.opentelemetry.io/collector/internal/memorylimiter"
"go.opentelemetry.io/collector/internal/memorylimiter/iruntime"
)

func TestMemoryPressureResponse(t *testing.T) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

//go:build linux

package cgroups // import "go.opentelemetry.io/collector/internal/cgroups"
package cgroups // import "go.opentelemetry.io/collector/internal/memorylimiter/cgroups"

import (
"bufio"
Expand Down
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

//go:build linux

package cgroups // import "go.opentelemetry.io/collector/internal/cgroups"
package cgroups // import "go.opentelemetry.io/collector/internal/memorylimiter/cgroups"
import (
"bufio"
"io"
Expand Down
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -26,4 +26,4 @@
// Package cgroups provides utilities to access Linux control group (CGroups)
// parameters (total memory, for example) for a given process.
// The original implementation is taken from https://github.com/uber-go/automaxprocs
package cgroups // import "go.opentelemetry.io/collector/internal/cgroups"
package cgroups // import "go.opentelemetry.io/collector/internal/memorylimiter/cgroups"
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

//go:build linux

package cgroups // import "go.opentelemetry.io/collector/internal/cgroups"
package cgroups // import "go.opentelemetry.io/collector/internal/memorylimiter/cgroups"

import "fmt"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

//go:build linux

package cgroups // import "go.opentelemetry.io/collector/internal/cgroups"
package cgroups // import "go.opentelemetry.io/collector/internal/memorylimiter/cgroups"

import (
"bufio"
Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

//go:build linux

package cgroups // import "go.opentelemetry.io/collector/internal/cgroups"
package cgroups // import "go.opentelemetry.io/collector/internal/memorylimiter/cgroups"

import (
"bufio"
Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// Copyright The OpenTelemetry Authors
// SPDX-License-Identifier: Apache-2.0

package iruntime // import "go.opentelemetry.io/collector/internal/iruntime"
package iruntime // import "go.opentelemetry.io/collector/internal/memorylimiter/iruntime"

import (
"github.com/shirou/gopsutil/v4/mem"
Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@

//go:build linux

package iruntime // import "go.opentelemetry.io/collector/internal/iruntime"
package iruntime // import "go.opentelemetry.io/collector/internal/memorylimiter/iruntime"

import "go.opentelemetry.io/collector/internal/cgroups"
import "go.opentelemetry.io/collector/internal/memorylimiter/cgroups"

// unlimitedMemorySize defines the bytes size when memory limit is not set
// for the container and process with cgroups
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

//go:build !linux

package iruntime // import "go.opentelemetry.io/collector/internal/iruntime"
package iruntime // import "go.opentelemetry.io/collector/internal/memorylimiter/iruntime"

// TotalMemory returns total available memory for non-linux platforms.
func TotalMemory() (uint64, error) {
Expand Down
2 changes: 1 addition & 1 deletion internal/memorylimiter/memorylimiter.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ import (
"go.uber.org/zap"

"go.opentelemetry.io/collector/component"
"go.opentelemetry.io/collector/internal/iruntime"
"go.opentelemetry.io/collector/internal/memorylimiter/iruntime"
)

const (
Expand Down
2 changes: 1 addition & 1 deletion internal/memorylimiter/memorylimiter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import (
"github.com/stretchr/testify/require"
"go.uber.org/zap"

"go.opentelemetry.io/collector/internal/iruntime"
"go.opentelemetry.io/collector/internal/memorylimiter/iruntime"
)

// TestMemoryPressureResponse manipulates results from querying memory and
Expand Down
2 changes: 1 addition & 1 deletion processor/memorylimiterprocessor/memorylimiter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ import (
"go.opentelemetry.io/collector/component"
"go.opentelemetry.io/collector/component/componenttest"
"go.opentelemetry.io/collector/consumer/consumertest"
"go.opentelemetry.io/collector/internal/iruntime"
"go.opentelemetry.io/collector/internal/memorylimiter"
"go.opentelemetry.io/collector/internal/memorylimiter/iruntime"
"go.opentelemetry.io/collector/pdata/plog"
"go.opentelemetry.io/collector/pdata/pmetric"
"go.opentelemetry.io/collector/pdata/ptrace"
Expand Down

0 comments on commit 35d4156

Please sign in to comment.