-
Notifications
You must be signed in to change notification settings - Fork 11
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
MNT Output more info for debugging failures #105
MNT Output more info for debugging failures #105
Conversation
.github/workflows/ci.yml
Outdated
cat << EOF > .env | ||
SS_DATABASE_CLASS="SQLite3Database" | ||
SS_DATABASE_USERNAME="root" | ||
SS_DATABASE_PASSWORD="" | ||
SS_SQLITE_DATABASE_PATH=":memory:" | ||
EOF | ||
elif [[ "${{ matrix.db }}" =~ mariadb ]]; then | ||
echo "mariadb version": | ||
mysql --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.
Is it correct?
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.
I think it should be mariadb --version
, isn't it?
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.
Oops not correct, but not for the reason you're thinking. The mysql
cli will work fine here.
I need to tell it which port to look at (same for the mysql databases) so it gives us the correct 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.
Fixed
98bebe3
to
65fed72
Compare
There's a few things that could be causing the mariadb failure, such as missing PHP mods, misconfigured ini, breaking changes in mariadb if that updated, etc.
This PR gives a little more info about what's installed in CI to help rule out possibilities.
Issue