-
Notifications
You must be signed in to change notification settings - Fork 32
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
make test failing #32
Labels
Comments
Could you use MySQL 5.7 for now? and raise supporting MariaDB as an enhancement? We can discuss we should support both or not on the issue. |
I can, but before doing that, can someone confirm that 'make test' works fine with mysql other than yourself to ensure its not environmental? |
Just confirmed works fine with mysql, closing. $ make test
./tools/test.sh
? github.com/Juniper/contrail [no test files]
? github.com/Juniper/contrail/cmd/contrail [no test files]
? github.com/Juniper/contrail/cmd/contrailutil [no test files]
ok github.com/Juniper/contrail/pkg/apisrv 1.110s coverage: 60.0% of statements
? github.com/Juniper/contrail/pkg/apisrv/keystone [no test files]
? github.com/Juniper/contrail/pkg/cmd/contrail [no test files]
? github.com/Juniper/contrail/pkg/cmd/contrailutil [no test files]
ok github.com/Juniper/contrail/pkg/common 1.026s coverage: 1.8% of statements
? github.com/Juniper/contrail/pkg/generated/api [no test files]
ok github.com/Juniper/contrail/pkg/generated/db 1.311s coverage: 73.2% of statements
ok github.com/Juniper/contrail/pkg/generated/models 1.104s coverage: 27.3% of statements |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Note I am running with mariadb rather than mysql.
$ mysql --version mysql Ver 15.1 Distrib 10.2.11-MariaDB, for osx10.12 (x86_64) using readline 5.1
Output from make test
The text was updated successfully, but these errors were encountered: