-
Notifications
You must be signed in to change notification settings - Fork 4
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
add tmp_upath_factory fixture #8
Conversation
24b13e4
to
738bed7
Compare
self.proc = subprocess.Popen( | ||
shlex.split("moto_server s3 -p %s" % self.port) | ||
shlex.split( | ||
f"moto_server s3 -p {self.port}", | ||
), | ||
stdout=subprocess.PIPE, | ||
stderr=subprocess.PIPE, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we are seriously polluting your machine ports, which will turn ugly. Take a look at how we do it in dvc - we spawn one container with moto (and azurite in the future, for example) to simulate real s3 and we use a random single port without polluting.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we at least don't spam lots of servers - this will become minor 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added an optional flag (--moto-server
) to set a custom port. In #13 I introduce a docker fixture which can be used to spawn docker containers. I will introduce some proper port management after that.
da36b78
to
786ce5d
Compare
a206a50
to
06595c6
Compare
06595c6
to
9360d7b
Compare
9360d7b
to
474bbea
Compare
No description provided.