Skip to content

Remove parallel testing#174

Closed
kegsay wants to merge 4 commits intomainfrom
kegan/parallel
Closed

Remove parallel testing#174
kegsay wants to merge 4 commits intomainfrom
kegan/parallel

Conversation

@kegsay
Copy link
Copy Markdown
Member

@kegsay kegsay commented May 7, 2025

Dirty servers seem to use the same namespace, so can cause container conflicts when you run >1 test suite at the same time:

CreateDirtyDeployment failed: CreateDirtyServer: Failed to deploy image homeserver : Error response from daemon: Conflict. The container name "/complement_crypto_dirty_hs1" is already in use by container "ef141595c2188aa4d0ece3aa5f74fdb43a7ba276cf95a00076ad387881d7b24a". You have to remove (or rename) that container to be able to reuse that name.

We should be namespacing dirty servers correctly, but for now we can quickly fix this by removing concurrency.

Dirty servers seem to use the same namespace, so can cause container
conflicts:
```
CreateDirtyDeployment failed: CreateDirtyServer: Failed to deploy image homeserver : Error response from daemon: Conflict. The container name "/complement_crypto_dirty_hs1" is already in use by container "ef141595c2188aa4d0ece3aa5f74fdb43a7ba276cf95a00076ad387881d7b24a". You have to remove (or rename) that container to be able to reuse that name.
```

We should be namespacing dirty servers correctly, but for now we can
quickly fix this by removing concurrency.
Copy link
Copy Markdown
Member

@Hywan Hywan left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix.

@kegsay
Copy link
Copy Markdown
Member Author

kegsay commented May 8, 2025

This was not the fix we were looking for. #175

@kegsay kegsay closed this May 8, 2025
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