Skip to content

mysql service: Make initialDatabases.schema attribute optional#25950

Merged
fpletz merged 1 commit intoNixOS:masterfrom
florianjacob:mysql-optional-schema
Aug 31, 2017
Merged

mysql service: Make initialDatabases.schema attribute optional#25950
fpletz merged 1 commit intoNixOS:masterfrom
florianjacob:mysql-optional-schema

Conversation

@florianjacob
Copy link
Contributor

Motivation for this change

the services.mysql.initialDatabases list does not allow to create empty initial databases, e.g. for web apps like piwik or nextcloud that create their own initial schema via their web installer.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@florianjacob
Copy link
Contributor Author

Related to #6963 , implementing a small part of it using the already existing initialDatabases option.

@florianjacob
Copy link
Contributor Author

From the git log of the file, as the bot seems to be inactive: @joachifm maybe you could take a look, or recommend someone else to ask? 😊

@joachifm
Copy link
Contributor

cc @grahamc

@fpletz fpletz added this to the 17.09 milestone Aug 31, 2017
@fpletz fpletz added the 8.has: module (update) This PR changes an existing module in `nixos/` label Aug 31, 2017
@fpletz
Copy link
Member

fpletz commented Aug 31, 2017

MySQL tests are still green. Thanks!

@fpletz fpletz merged commit d22c1c0 into NixOS:master Aug 31, 2017
@florianjacob florianjacob deleted the mysql-optional-schema branch August 31, 2017 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: module (update) This PR changes an existing module in `nixos/`

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants