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

Create schemas and permissions in more situations #2838

Closed
wants to merge 1 commit into from

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Sep 2, 2017

On trusty, this is needed for all mysql/mariadb builds.
Fixes #2837

On trusty, this is needed for all mysql/mariadb builds.
Fixes doctrine#2837
Copy link
Contributor

@Majkl578 Majkl578 left a comment

Choose a reason for hiding this comment

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

LGTM, we don't really want non-determinism on CI.
Good enough until we migrate DBAL to Build Stages.

@greg0ire
Copy link
Member Author

greg0ire commented Sep 9, 2017

Restarting the build because I've had a new issue locally even with that change

@greg0ire greg0ire closed this Sep 9, 2017
@greg0ire greg0ire reopened this Sep 9, 2017
@greg0ire
Copy link
Member Author

greg0ire commented Sep 9, 2017

Ok still green :)

@lcobucci
Copy link
Member

lcobucci commented Sep 9, 2017

@greg0ire @Majkl578 I've handled the this using stages on #2849 (with some other improvements).

@greg0ire
Copy link
Member Author

greg0ire commented Sep 9, 2017

I know, but your PR is big and still red, so not mergeable right now. I just noticed that Travis is not required for the build to pass, so this is not as critical as I think. However this situation is not great, because people might assume they didn't break the build when they actually did. Right now they can't know for sure unless they check the output of the logs on each commit.

@Majkl578
Copy link
Contributor

Majkl578 commented Sep 10, 2017

Handled in #2849, sorry for extra work.

@Majkl578 Majkl578 closed this Sep 10, 2017
@greg0ire
Copy link
Member Author

Not a big deal :)

@greg0ire greg0ire deleted the fix_trusty_build branch September 10, 2017 22:03
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants