Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 0 additions & 6 deletions dev-tools/mage/build.go
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@
// environment.
func GolangCrossBuild(params BuildArgs) error {
if os.Getenv("GOLANG_CROSSBUILD") != "1" {
return errors.New("Use the crossBuild target. golangCrossBuild can " +

Check failure on line 171 in dev-tools/mage/build.go

View workflow job for this annotation

GitHub Actions / lint (ubuntu-latest)

ST1005: error strings should not end with punctuation or newlines (staticcheck)
"only be executed within the golang-crossbuild docker environment.")
}

Expand Down Expand Up @@ -198,12 +198,6 @@
return Build(params)
}

func BuildOTel() error {
args := DefaultBuildArgs()
args.ExtraFlags = append(args.ExtraFlags, "-tags", "otelbeat")
return Build(args)
}

// Build invokes "go build" to produce a binary.
func Build(params BuildArgs) error {
fmt.Println(">> build: Building", params.Name)
Expand Down Expand Up @@ -298,7 +292,7 @@
},
StringFileInfo: goversioninfo.StringFileInfo{
CompanyName: BeatVendor,
ProductName: strings.Title(BeatName),

Check failure on line 295 in dev-tools/mage/build.go

View workflow job for this annotation

GitHub Actions / lint (ubuntu-latest)

SA1019: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead. (staticcheck)
ProductVersion: version,
FileVersion: version,
FileDescription: BeatDescription,
Expand Down
7 changes: 0 additions & 7 deletions dev-tools/mage/gotest.go
Original file line number Diff line number Diff line change
Expand Up @@ -503,13 +503,6 @@ func BuildSystemTestBinary() error {
return BuildSystemTestGoBinary(DefaultTestBinaryArgs())
}

// BuildSystemTestOTelBinary builds beat binary that includes otel.
func BuildSystemTestOTelBinary() error {
args := DefaultTestBinaryArgs()
args.ExtraFlags = []string{"-tags", "otelbeat"}
return BuildSystemTestGoBinary(args)
}

// BuildSystemTestGoBinary build a binary for testing that is instrumented for
// testing and measuring code coverage. The binary is only instrumented for
// coverage when TEST_COVERAGE=true (default is false).
Expand Down
37 changes: 0 additions & 37 deletions libbeat/otelbeat/helper.go

This file was deleted.

15 changes: 0 additions & 15 deletions x-pack/filebeat/cmd/otelcmd_disabled.go

This file was deleted.

16 changes: 0 additions & 16 deletions x-pack/filebeat/cmd/otelcmd_enabled.go

This file was deleted.

1 change: 0 additions & 1 deletion x-pack/filebeat/cmd/root.go
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ func Filebeat() *cmd.BeatsRootCmd {
command.PersistentPreRun = func(cmd *cobra.Command, args []string) {
management.ConfigTransform.SetTransform(filebeatCfg)
}
addOTelCommand(command)
return command
}

Expand Down
13 changes: 3 additions & 10 deletions x-pack/filebeat/magefile.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,14 +43,9 @@ func Build() error {
return devtools.Build(devtools.DefaultBuildArgs())
}

// BuildOTel builds the Beat binary with OTel sub command
func BuildOTel() error {
return devtools.BuildOTel()
}

// BuildSystemTestBinary builds a binary instrumented for use with Python system tests.
func BuildSystemTestBinary() error {
return devtools.BuildSystemTestOTelBinary()
return devtools.BuildSystemTestBinary()
}

// GolangCrossBuild builds the Beat binary inside of the golang-builder.
Expand Down Expand Up @@ -173,8 +168,7 @@ func IntegTest() {

// GoIntegTest starts the docker containers and executes the Go integration tests.
func GoIntegTest(ctx context.Context) error {
// build integration test binary with otel sub command
devtools.BuildSystemTestOTelBinary()
devtools.BuildSystemTestBinary()
args := devtools.DefaultGoTestIntegrationFromHostArgs(ctx)
// ES_USER must be admin in order for the Go Integration tests to function because they require
// indices:data/read/search
Expand All @@ -185,8 +179,7 @@ func GoIntegTest(ctx context.Context) error {

// GoFIPSOnlyIntegTest starts the docker containers and executes the Go integration tests with GODEBUG=fips140=only set.
func GoFIPSOnlyIntegTest(ctx context.Context) error {
// build integration test binary with otel sub command
devtools.BuildSystemTestOTelBinary()
devtools.BuildSystemTestBinary()
args := devtools.DefaultGoTestIntegrationFromHostArgs(ctx)
// ES_USER must be admin in order for the Go Integration tests to function because they require
// indices:data/read/search
Expand Down
72 changes: 0 additions & 72 deletions x-pack/libbeat/common/otelbeat/inspect.go

This file was deleted.

165 changes: 0 additions & 165 deletions x-pack/libbeat/common/otelbeat/otel.go

This file was deleted.

15 changes: 0 additions & 15 deletions x-pack/metricbeat/cmd/otelcmd_disabled.go

This file was deleted.

Loading
Loading