sanjibansg commented on code in PR #12701:
URL: https://github.com/apache/arrow/pull/12701#discussion_r852600099
##########
python/pyarrow/tests/conftest.py:
##########
@@ -311,3 +313,23 @@ def s3_server(s3_connection):
finally:
if proc is not None:
proc.kill()
+
+
[email protected](scope='session')
+def limited_s3_user(request, s3_server):
Review Comment:
Sorry, this fixture thing was a little confusing at first. I thought by
decorating it as a fixture it should set up the s3 user properly with the
required permissions. But, that might be the reason of the failing CI. Updated
the code where it is no longer a fixture, and moved it to util.py as well.
Thanks for the suggestion!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
