Add more clients in the test #65

Closed
quentin wants to merge 0 commits from test/more-clients into main
Owner
  • Make Minio works
  • Activate mc and rclone in the CI

Fixes #60 (we are able to do the same set of operation with minio, including delete, than other clients)
Related to #64 (we use a socat TLS proxy to prevent minio from signing payload)

Note: it is possible to only run tests for some clients.
Example, if you want to run tests only for minio:

SKIP_AWS=1 SKIP_S3CMD=1 SKIP_RCLONE=1 ./script/test-smoke.sh

If you do not wish to install socat or minio:

SKIP_HTTPS=1 SKIP_MC=1 ./script/test-smoke.sh
- [x] Make Minio works - [x] Activate `mc` and `rclone` in the CI Fixes #60 (we are able to do the same set of operation with minio, including delete, than other clients) Related to #64 (we use a socat TLS proxy to prevent minio from signing payload) Note: it is possible to only run tests for some clients. Example, if you want to run tests only for minio: ``` SKIP_AWS=1 SKIP_S3CMD=1 SKIP_RCLONE=1 ./script/test-smoke.sh ``` If you do not wish to install socat or minio: ``` SKIP_HTTPS=1 SKIP_MC=1 ./script/test-smoke.sh ```
quentin changed title from [WIP] Add more clients in the test to Add more clients in the test 2021-04-30 20:52:31 +00:00
quentin force-pushed test/more-clients from 513ecf05f1 to 12f85cd36b 2021-04-30 22:46:50 +00:00 Compare
lx requested changes 2021-05-02 11:23:19 +00:00
lx left a comment
Owner

@quentin should this be merged?

If so, can you cleanup the history by squashing everything into one single commit? (or maybe more but only if it makes sense to keep the intermediary steps, at the moment it looks like a lot of noise)

git rebase -i origin/main

@quentin should this be merged? If so, can you cleanup the history by squashing everything into one single commit? (or maybe more but only if it makes sense to keep the intermediary steps, at the moment it looks like a lot of noise) `git rebase -i origin/main`
@ -5,4 +3,0 @@
GARAGE_DEBUG="${REPO_FOLDER}/target/debug/"
GARAGE_RELEASE="${REPO_FOLDER}/target/release/"
PATH="${GARAGE_DEBUG}:${GARAGE_RELEASE}:$PATH"
Owner

Why is this going away here, but not in dev-env-{mc,rclone,...}.sh ?

Why is this going away here, but not in `dev-env-{mc,rclone,...}.sh` ?
quentin marked this conversation as resolved
quentin added 1 commit 2021-05-02 13:04:10 +00:00
continuous-integration/drone/pr Build is passing Details
continuous-integration/drone/push Build is passing Details
ef4d6e782a
Add minio & rclone to our functional tests
It is now possible to configure which clients
you do not want to test with the env variable SKIP_XXX=1,
XXX being the client name. eg. SKIP_S3CMD=1 ./script/test-smoke.sh
Author
Owner

should this be merged?

yes it should!

> should this be merged? yes it should!
quentin requested review from lx 2021-05-02 13:06:16 +00:00
lx closed this pull request 2021-05-02 18:55:10 +00:00
Owner

Closed but actually merged, dunno why Gitea didn't see that.

Closed but actually merged, dunno why Gitea didn't see that.
lx reopened this pull request 2021-05-02 18:57:36 +00:00
lx closed this pull request 2021-05-02 18:57:48 +00:00
All checks were successful
continuous-integration/drone/pr Build is passing
Required
Details
continuous-integration/drone/push Build is passing

Pull request closed

Sign in to join this conversation.
No description provided.