-
Notifications
You must be signed in to change notification settings - Fork 50
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
Add test framework for mint #27
Conversation
- Tests for mc and general cleanup - clean up for minio-py
removing logging in python sdk
- Added documentation (#6)
apps/init/initCheck.go
Outdated
@@ -24,9 +24,6 @@ import ( | |||
) | |||
|
|||
func main() { | |||
// obtain endpoint from env. | |||
// it is set using `-e` when starting the container | |||
// check README.md for instructions. | |||
endpoint := os.Getenv("S3_ADDRESS") |
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.
Remove S3_ADDRESS
run.sh
Outdated
# S3_ADDRESS is passed on as env variables while starting the docker container. | ||
# see README.md for info on options. | ||
# Note: https://play.minio.io hosts publicly available Minio server. | ||
if [ -z "$S3_ADDRESS" ]; then |
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.
Remove S3_ADDRESS
@@ -0,0 +1,69 @@ | |||
#!/usr/bin/env bash |
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.
Avoid binary files..
} | ||
|
||
# invoke the script | ||
main "$@" |
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.
add a newline.
README.md
Outdated
The project will be published in Docker hub after further more testing. Till then the docker image has to be built locally and run. | ||
|
||
### Build | ||
|
||
```sh | ||
$ git clone https://github.com/minio/mint.git | ||
$ cd mint | ||
$ docker build -t minio/mint:alpha . | ||
$ docker build -t minio/mint:mint . |
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.
Keep the image as latest
minio/mint:latest
Dockerfile
Outdated
go get -u github.com/minio/minio-go && \ | ||
chmod +x run.sh | ||
|
||
COPY . /mint |
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.
Keep two dockerfiles one more Dockerfile and another for release. Dockerfile.release
Dockerfile
Outdated
default-jre \ | ||
default-jdk \ | ||
git \ | ||
golang-go \ |
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.
go-1.6 version should be moved to go1.7
324d647
to
10566c4
Compare
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.
everything else looks good.. one final comment.
Dockerfile
Outdated
mv go /usr/local && \ | ||
rm -rf go && \ | ||
rm go1.7.4.linux-amd64.tar.gz | ||
COPY . /mint |
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.
Are we going to add a new Dockerfile.release which only does git clone
version?
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.
yes, we will send an update shortly
Fix mc scripts to return back to main script on failure
Rewrite mint to create a framework for running SDK tests on Docker.
mc, minio-py, minio-go, minio-java, minio-js sdk tests have currently been added to the framework.
This is w.r.t issue #26