Skip to content

NIT-2962: Clean one-off script containers - #147

Merged
Tristan-Wilson merged 2 commits into
masterfrom
pmikolajczyk/nit-2962-run-rm
Sep 11, 2025
Merged

Tristan-Wilson merged 2 commits into
masterfrom
pmikolajczyk/nit-2962-run-rm

Conversation

@pmikolajczyk41

Copy link
Copy Markdown
Member

Previously, after running some scripts, docker space was filled with exited containers. This has been already reported by the docker itself like:

$ ./test-node.bash script send-l2
Using NITRO_CONTRACTS_BRANCH: v3.1.0
Using TOKEN_BRIDGE_BRANCH: v1.2.2
WARN[0000] Found orphan containers ([
  nitro-testnode-scripts-run-ec50076a11b4 
  nitro-testnode-scripts-run-138a9430b88a 
  nitro-testnode-scripts-run-4fa182918ce3 
  nitro-testnode-scripts-run-28cf6eaa8940 
  nitro-testnode-scripts-run-6d8ab3c09c91 
  nitro-testnode-scripts-run-5187961b90db 
  nitro-testnode-scripts-run-205892e27840 
  nitro-testnode-scripts-run-ab7eb9ba0083
]) for this project. If you removed or renamed this service in your compose file, you can run this command with the --remove-orphans flag to clean it up. 

Now we simply remove the script container after it exits.

@cla-bot cla-bot Bot added the s label Sep 11, 2025
@pmikolajczyk41
pmikolajczyk41 changed the base branch from release to master September 11, 2025 07:28

@Tristan-Wilson Tristan-Wilson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good find

@Tristan-Wilson

Copy link
Copy Markdown
Member

I just enabled the option to have auto-merge for this repo, but it doesn't seem like it will get picked up yet for this PR. Assigning back to @pmikolajczyk41 to merge once tests pass.

@Tristan-Wilson
Tristan-Wilson merged commit 81a225b into master Sep 11, 2025
26 checks passed
@pmikolajczyk41
pmikolajczyk41 deleted the pmikolajczyk/nit-2962-run-rm branch September 11, 2025 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants