-
Notifications
You must be signed in to change notification settings - Fork 105
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
IPAM tests cannot be run concurrently #1382
Comments
What if we set up the IPAM & pool before the test (and just use it if it already exists). I think that way we should get around those limitations |
That's a good idea. We originally considered just creating a permanent one and using that since they are free, but decided not to because it didn't affect the test speed. The concurrency limitation makes it worth it I think. |
#1404 looks like @danielrbradley setup a test IPAM, but I need to confirm where is the infra that's managing it. If it is permanent we can rewrite the tests to get it. For now I will skip the tests. |
What happened?
It looks like we have a limit of 1 ipam existing at a time, which means that we can only run that test with a concurrency of 1 across all workflow runs.
We could:
main
workflow after PRs have been mergedExample
N/A
Output of
pulumi about
N/A
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).
The text was updated successfully, but these errors were encountered: