[PR #789] [MERGED] Close docker client #1607

Closed
opened 2026-03-01 21:51:53 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/nektos/act/pull/789
Author: @ChristopherHX
Created: 8/30/2021
Status: Merged
Merged: 10/24/2021
Merged by: @mergify[bot]

Base: masterHead: fix/docker_fd_leak


📝 Commits (7)

  • b8781e6 Close docker client!
  • c60cbed Fix nil dereference
  • f4faa2f Update run_context.go
  • 41ea4f4 Bump
  • 2a996ee Merge branch 'master' into fix/docker_fd_leak
  • bede76b Merge branch 'master' into fix/docker_fd_leak
  • 7a88802 Merge branch 'master' into fix/docker_fd_leak

📊 Changes

7 files changed (+24 additions, -3 deletions)

View changed files

📝 pkg/container/docker_build.go (+1 -0)
📝 pkg/container/docker_images.go (+1 -0)
📝 pkg/container/docker_pull.go (+1 -0)
📝 pkg/container/docker_run.go (+11 -0)
📝 pkg/container/docker_volume.go (+2 -0)
📝 pkg/runner/run_context.go (+6 -1)
📝 pkg/runner/step_context.go (+2 -2)

📄 Description

Please stop leaking unix sockets, it breaks my github-act-runner program.

Reference: https://github.com/ChristopherHX/github-act-runner/issues/41


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/nektos/act/pull/789 **Author:** [@ChristopherHX](https://github.com/ChristopherHX) **Created:** 8/30/2021 **Status:** ✅ Merged **Merged:** 10/24/2021 **Merged by:** [@mergify[bot]](https://github.com/apps/mergify) **Base:** `master` ← **Head:** `fix/docker_fd_leak` --- ### 📝 Commits (7) - [`b8781e6`](https://github.com/nektos/act/commit/b8781e67326e42590f7be332ec01ff7ca2673c1c) Close docker client! - [`c60cbed`](https://github.com/nektos/act/commit/c60cbed978d5676022257b9a7df5156da2a91bcd) Fix nil dereference - [`f4faa2f`](https://github.com/nektos/act/commit/f4faa2f045b1f6a7cb93340d6c51e6e20820e6c7) Update run_context.go - [`41ea4f4`](https://github.com/nektos/act/commit/41ea4f4f15aa4cd6b423cdc7f22ab508c6aef020) Bump - [`2a996ee`](https://github.com/nektos/act/commit/2a996ee39cb1f573810c3404bb1497c28bfb2349) Merge branch 'master' into fix/docker_fd_leak - [`bede76b`](https://github.com/nektos/act/commit/bede76b7594a14c3db03e9fa1212743441070931) Merge branch 'master' into fix/docker_fd_leak - [`7a88802`](https://github.com/nektos/act/commit/7a888022f98dfef22403c9a8be6958fe4faa6d30) Merge branch 'master' into fix/docker_fd_leak ### 📊 Changes **7 files changed** (+24 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `pkg/container/docker_build.go` (+1 -0) 📝 `pkg/container/docker_images.go` (+1 -0) 📝 `pkg/container/docker_pull.go` (+1 -0) 📝 `pkg/container/docker_run.go` (+11 -0) 📝 `pkg/container/docker_volume.go` (+2 -0) 📝 `pkg/runner/run_context.go` (+6 -1) 📝 `pkg/runner/step_context.go` (+2 -2) </details> ### 📄 Description Please stop leaking unix sockets, [it breaks my github-act-runner program](https://github.com/ChristopherHX/github-act-runner). Reference: https://github.com/ChristopherHX/github-act-runner/issues/41 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-01 21:51:53 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/act#1607
No description provided.