Skip to content

mod: replace gopkg.in/natefinch/lumberjack.v2 with elastic fork#51950

Merged
efd6 merged 1 commit into
elastic:mainfrom
efd6:49092-lumberjack
Jul 15, 2026
Merged

mod: replace gopkg.in/natefinch/lumberjack.v2 with elastic fork#51950
efd6 merged 1 commit into
elastic:mainfrom
efd6:49092-lumberjack

Conversation

@efd6

@efd6 efd6 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

See title.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.

Disruptive User Impact

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

@efd6 efd6 self-assigned this Jul 15, 2026
@efd6 efd6 added Filebeat Filebeat Team:Security-Service Integrations Security Service Integrations Team backport-8.19 Automated backport to the 8.19 branch backport-9.4 backport-9.5 labels Jul 15, 2026
@botelastic botelastic Bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Jul 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
  • /test : Run the Buildkite pipeline.

@github-actions

Copy link
Copy Markdown
Contributor

TL;DR

The Filebeat Go integration job did not reach Go integration tests. It failed while Docker was building the generated filebeat-oss and filebeat-oss-wolfi images because Chainguard returned HTTP 500 for the anonymous token request to cgr.dev. This is an external registry/infrastructure failure, not an error introduced by the lumberjack dependency replacement.

Remediation

  • Retry the Buildkite job; the log shows the same cgr.dev/chainguard/wolfi-base:latest token failure for both image builds.
  • If the retry fails again, investigate Chainguard registry availability or the Buildkite runner access to (cgr.dev/redacted) before changing PR code.
Investigation details

Root Cause

The job command was cd filebeat followed by mage goIntegTest, but packaging failed before tests could execute. The generated Dockerfile uses FROM --platform=linux/amd64 cgr.dev/chainguard/wolfi-base AS home; both the regular and Wolfi Filebeat image builds failed during base-image metadata resolution.

Evidence

Verification

No Go tests ran because the image build failed before test execution.

Follow-up

A successful retry should confirm whether this was transient registry availability.


What is this? | From workflow: PR Buildkite Detective

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

@efd6

efd6 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

/test

@efd6 efd6 marked this pull request as ready for review July 15, 2026 04:15
@efd6 efd6 requested review from a team as code owners July 15, 2026 04:15
@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@mergify

mergify Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@efd6 efd6 merged commit 3de2668 into elastic:main Jul 15, 2026
222 of 225 checks passed
efd6 added a commit that referenced this pull request Jul 15, 2026
… with elastic fork (#51955)

* mod: replace gopkg.in/natefinch/lumberjack.v2 with elastic fork (#51950)

(cherry picked from commit 3de2668)

# Conflicts:
#	go.mod
#	go.sum
#	x-pack/filebeat/input/entityanalytics/provider/okta/conf.go

* resolve conflicts

---------

Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
efd6 added a commit that referenced this pull request Jul 15, 2026
…) (#51957)

(cherry picked from commit 3de2668)

Co-authored-by: Dan Kortschak <dan.kortschak@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-8.19 Automated backport to the 8.19 branch backport-9.4 backport-9.5 Filebeat Filebeat Team:Security-Service Integrations Security Service Integrations Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mod: replace github.com/natefinch/lumberjack with elastic fork x-pack/filebeat/input: tracer leaks goroutines via lumberjack.Logger.Close

2 participants