Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing Go Pipeline #145

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Fixing Go Pipeline #145

wants to merge 3 commits into from

Conversation

niclasheun
Copy link
Contributor

  • This PR is for testing -

@niclasheun
Copy link
Contributor Author

With these changes it works in 7/8 times...

@niclasheun
Copy link
Contributor Author

@rappm Nimm das mal als starting point. Das funktioniert immerhin schon in 80%+ der Worfklow runs.

Schau mal ob du die Runs linearisieren kannst, also dass die Tests nicht parallel laufen. Oder Container wiederverwendet werden können etc.

Für mich am vielversprechensten wäre, dass man eine Main-Test-Suite hat die nur einen Postgres Container startet. Alle anderen Test Suits sind dann Sub-Suites (also erben von der Main-Test-Suite) und anstelle eines eignen Containers, bekommen die vlt einfach nur eine eigene DB (und nicht gleich einen eigenen Container). Also das wäre eine Helper Function, die sich auf den existierenden Container verbindet, ein "CREATEDB" im exisitierenden Container durchführt (zB mit einem Random namen) und dann die Connection zu dieser DB zurückgibt.

Damit würde man ganz von den failenden Container Creation weg kommen.

@niclasheun niclasheun marked this pull request as draft February 7, 2025 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

1 participant