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

Use correct minio-java version in test run phase #187

Merged
merged 1 commit into from
Oct 20, 2017

Conversation

nitisht
Copy link
Contributor

@nitisht nitisht commented Oct 20, 2017

run/core/minio-java/run.sh script should use the minio-java version
downloaded during build phase to run minio-java tests

fixes #186

@nitisht nitisht changed the title Update minio-java version automatically Update minio-java version Oct 20, 2017
Copy link
Member

@balamurugana balamurugana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right fix is

  • Modify line 36 to java -cp "/mint/run/core/minio-java:." FunctionalTest \
  • Remove lines 19 and 18

MINIO_JAVA_VERSION="3.0.7"
MINIO_JAVA_PATH="/mint/run/core/minio-java"
MINIO_JAVA_PATH="$MINT_RUN_CORE_DIR/minio-java"
MINIO_JAVA_VERSION=$(curl -s https://api.github.com/repos/minio/minio-java/releases/latest | jq -r .tag_name)
Copy link
Member

@balamurugana balamurugana Oct 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fix won't work if docker image has lower version of jar files and higher version available in github

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docker image with get the jar file using same mechanism in install.sh, so how it will have lower version?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While running docker image, you fetch version information from github which would return different version than the one available inside the image.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah..yes.good point..will update

@@ -15,8 +15,13 @@
# limitations under the License.
#

MINIO_JAVA_VERSION="3.0.7"
MINIO_JAVA_PATH="/mint/run/core/minio-java"
MINIO_JAVA_PATH="$MINT_RUN_CORE_DIR/minio-java"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MINT_RUN_CORE_DIR is not at all available globally.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, will remove this

@nitisht
Copy link
Contributor Author

nitisht commented Oct 20, 2017

Modify line 36 to java -cp "/mint/run/core/minio-java:." FunctionalTest \

Agreed, this should work. Will change to this

@nitisht
Copy link
Contributor Author

nitisht commented Oct 20, 2017

@balamurugana

Modify line 36 to java -cp "/mint/run/core/minio-java:." FunctionalTest \

This is not working, path to exact jar file is needed.

@nitisht
Copy link
Contributor Author

nitisht commented Oct 20, 2017

Updated with different approach @balamurugana please check.

@nitisht nitisht force-pushed the minio-java-failure branch 2 times, most recently from e1e24c6 to 61048a9 Compare October 20, 2017 18:20
run/core/minio-java/run.sh script should use the minio-java version
downloaded during build phase to run minio-java tests

fixes minio#186
@nitisht nitisht changed the title Update minio-java version Use correct minio-java version in test run phase Oct 20, 2017
Copy link
Member

@balamurugana balamurugana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nitisht
Copy link
Contributor Author

nitisht commented Oct 20, 2017

Tested with this build, works fine

Running with
SERVER_ENDPOINT: 127.0.0.1:9000
ACCESS_KEY:      minio
SECRET_KEY:      ***REDACTED***
ENABLE_HTTPS:    0
SERVER_REGION:   us-east-1
MINT_DATA_DIR:   /mint/data
MINT_MODE:       core
To get intermittent logs, 'sudo docker cp 8136b886d1f3:/mint/log /tmp/mint-logs'
Running aws-sdk-php tests ... done in 41 seconds
Running aws-sdk-ruby tests ... done in 1 seconds
Running awscli tests ... done in 22 seconds
Running mc tests ... done in 4 seconds
Running minio-go tests ... done in 21 seconds
Running minio-java tests ... done in 6 seconds
Running minio-js tests ... done in 23 seconds
Running minio-py tests ... done in 10 seconds
Running s3cmd tests ... done in 7 seconds
Finished running all tests.
To get logs, run 'sudo docker cp 8136b886d1f3:/mint/log /tmp/mint-logs'

@nitisht nitisht merged commit d39e142 into minio:master Oct 20, 2017
@nitisht nitisht deleted the minio-java-failure branch October 20, 2017 18:50
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.

Mint fails with latest Docker image
3 participants