jaredallard/overlay
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions 1

build(lint): properly cleanup container, speed up stopping
All checks were successful
Test / elint (push) Successful in 38s
Details

Browse source
This commit is contained in:
Jared Allard 2025-03-09 16:43:01 -07:00
parent c4274316fa
commit dad55b6193
Signed by: jaredallard
SSH key fingerprint: SHA256:wyRyyv28jBYw8Yp/oABNPUYvbGd6hyZj23XVXEm5G/U
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
.tools/internal/ebuild/manifest.go
View file

@ -93,7 +93,7 @@ func ValidateManifest(stdout, stderr io.Writer, overlayDir, packageName string)
ctx := context.TODO()
bid, err := exec.Command(
"docker", "run", "-d", "--rm", "--entrypoint", "sleep", gentooImage, "infinity",
"docker", "run", "--init", "-d", "--rm", "--entrypoint", "sleep", gentooImage, "infinity",
).Output()
if err != nil {
var execErr *exec.ExitError
@ -104,7 +104,7 @@ func ValidateManifest(stdout, stderr io.Writer, overlayDir, packageName string)
return fmt.Errorf("failed to run container: %w", err)
}
containerID := strings.TrimSpace(string(bid))
defer exec.Command("docker", "stop", containerID) //nolint:errcheck // Why: best effort
defer exec.Command("docker", "stop", containerID).Run() //nolint:errcheck // Why: best effort
lclPkgDir := filepath.Join(overlayDir, packageName)

2
.tools/internal/steps/executor.go
View file

@ -97,7 +97,7 @@ func (e *Executor) Run(ctx context.Context) (*Results, error) {
// TODO(jaredallard): Use the Docker API for this, but for now the CLI
// is much better.
bid, err := exec.Command(
"docker", "run", "-d", "--rm", "--entrypoint", "sleep",
"docker", "run", "--init", "-d", "--rm", "--entrypoint", "sleep",
"ghcr.io/jaredallard/overlay:updater", "infinity",
).Output()
if err != nil {