Skip to content

Comments

fix: create named mongo db test container#79

Merged
FlandiaYingman merged 1 commit intomainfrom
fix/mongo-db-docker
Feb 21, 2026
Merged

fix: create named mongo db test container#79
FlandiaYingman merged 1 commit intomainfrom
fix/mongo-db-docker

Conversation

@MisterCommand
Copy link
Member

Line 77-78 — Added a step that force-removes any existing container named mongodb-actions before the MongoDB action runs (|| true ensures the step doesn't fail if the container doesn't exist).

Line 83 — Added mongodb-container-name: mongodb-actions to the supercharge/mongodb-github-action step so the container is created with the explicit name mongodb-actions.

Copilot AI review requested due to automatic review settings February 20, 2026 12:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses container persistence issues in the GitHub Actions workflow by adding explicit container naming and cleanup for MongoDB. The changes ensure that MongoDB containers from previous (potentially cancelled) workflow runs don't interfere with new runs, which is particularly important for self-hosted runners where containers may persist between runs.

Changes:

  • Added pre-cleanup step to remove any existing MongoDB container before starting a new one
  • Added explicit container naming to the MongoDB GitHub Action for predictable container management

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MisterCommand MisterCommand changed the title fix: create named container fix: create named mongo db test container Feb 20, 2026
@FlandiaYingman FlandiaYingman merged commit effa8e2 into main Feb 21, 2026
16 checks passed
@FlandiaYingman FlandiaYingman deleted the fix/mongo-db-docker branch February 21, 2026 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants