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

Test cloud transitions with subfolders #539

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

petrutlucian94
Copy link

RGW cloud sync operations were unable to handle subfolders due to the fact that slashes were url encoded, causing mismatches between the source and destination object names.

We're updating the cloud transiton test, which exercises a similar code path. We'll ensure that subfolders are transferred properly.

RGW cloud sync operations were unable to handle subfolders due to
the fact that slashes were url encoded, causing mismatches between
the source and destination object names.

We're updating the cloud transiton test, which exercises a similar
code path. We'll ensure that subfolders are transferred properly.

Signed-off-by: Lucian Petrut <lpetrut@cloudbasesolutions.com>
@petrutlucian94
Copy link
Author

Test result:

py run-test: commands[0] | pytest s3tests_boto3/functional/test_s3.py::test_lifecycle_cloud_transition
======================================== test session starts =========================================
platform linux -- Python 3.10.12, pytest-7.4.4, pluggy-1.3.0
cachedir: .tox/py/.pytest_cache
rootdir: /mnt/data/workspace/s3-tests
configfile: pytest.ini
collected 1 item

s3tests_boto3/functional/test_s3.py

.                                                          [100%]

=================================== 1 passed in 241.14s (0:04:01) ====================================
______________________________________________ summary _______________________________________________
  py: commands succeeded
  congratulations :)

@petrutlucian94
Copy link
Author

@cbodley hi, I was wondering if you're still interested in this simple test that covers subfolders.

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.

None yet

1 participant