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
Add minio & rclone to our functional tests
All checks were successful
continuous-integration/drone/pr Build is passing
continuous-integration/drone/push Build is passing
ef4d6e782a
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
Some checks are pending
continuous-integration/drone/pr Build is passing
continuous-integration/drone/push Build is passing
ci/woodpecker/pr/debug
Required

Pull request closed

Sign in to join this conversation.
No reviewers
lx
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Deuxfleurs/garage#65
No description provided.