Skip to content

vendor: github.com/containerd/log main, with SetLevel - #7064

Draft
thaJeztah wants to merge 2 commits into
moby:masterfrom
thaJeztah:update_log
Draft

vendor: github.com/containerd/log main, with SetLevel#7064
thaJeztah wants to merge 2 commits into
moby:masterfrom
thaJeztah:update_log

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

Linter incorectly suggests using t.SetEnv, but we need to unset the
env-var;

    level=warning msg="[runner/exclusion_rules] Skipped 0 issues by rules: [Path: \".*\\\\.pb\\\\.go$\", Linters: \"gofmt, goimports\"]"
    cmd/buildctl/build/opt_test.go:89:25: os.Setenv() could be replaced by t.Setenv() in TestParseOpt (usetesting)
     					t.Cleanup(func() { os.Setenv(k, v) })
     					                   ^

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@github-actions github-actions Bot added area/testing area/dependencies Pull requests that update a dependency file area/buildctl labels Aug 23, 2026
@thaJeztah thaJeztah changed the title vendor: github.com/containerd/log main, with SetLevel - #14019 vendor: github.com/containerd/log main, with SetLevel Aug 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/buildctl area/dependencies Pull requests that update a dependency file area/testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant