-
Notifications
You must be signed in to change notification settings - Fork 2
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
fix: renable s3 customizations for data connectors #606
Conversation
You can access the deployment of this PR at https://renku-ci-ds-606.dev.renku.ch |
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.
Note, can be solved later: maybe the webdav patches should work like the other patches instead of being invoked as special cases.
Fix looks OK to merge 👍
@@ -600,3 +600,4 @@ async def test_storage_schema(storage_test_client) -> None: | |||
providers = next(p for p in s3["options"] if p["name"] == "provider") | |||
assert providers | |||
assert providers.get("examples") | |||
assert any(e["value"] == "Switch" for e in providers.get("examples")) |
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.
It would probably be better to have one test per patch in the future.
Pull Request Test Coverage Report for Build 12769910826Details
💛 - Coveralls |
Tearing down the temporary RenkuLab deplyoment for this PR. |
this add back s3 and other customisations for the storage schema
/deploy