-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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 drone #5724
Fix drone #5724
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5724 +/- ##
=============================================
+ Coverage 35.35% 53.86% +18.51%
Complexity 22755 22755
=============================================
Files 1405 1405
Lines 86741 86711 -30
Branches 1328 1328
=============================================
+ Hits 30665 46705 +16040
+ Misses 56076 40006 -16070
|
@danxuliu @icewind1991 I needed to disable the acceptance and object storage tests temporarily because they rely on the fact, that the service (selenium and fake-s3) on localhost, but actually with drone 0.7 this is not the case anymore (because they matched with the docker-compose behaviour). Now we need to use the hostnames |
autotest.sh
Outdated
@@ -46,6 +46,8 @@ function print_syntax { | |||
echo -e "\nIf no arguments are specified, all tests will be run with all database configs" >&2 | |||
} | |||
|
|||
pwd |
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.
is this change intended?
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.
no :P
Fix service container host name check current folder fix redis for integration test Fix more hostnames Signed-off-by: Morris Jobke <[email protected]>
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.
drone seems happy 👍
Signed-off-by: Morris Jobke <[email protected]>
No description provided.