about summary refs log tree commit diff
path: root/nixos/tests
Commit message (Collapse)AuthorAge
...
* | | nixos/tests/installer.nix: shorten test name to fit length limitSymphorien Gibol2018-11-17
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | previously one of the tests would fail with boot-after-install: starting vm Path length (109) is longer than maximum supported length (108) and will be truncated at /nix/store/0ingn8cwwnl84i374hcl6nafsm2c5m2p-perl-5.28.0/lib/perl5/5.28.0/x86_64-linux-thread-multi/Socket.pm line 872, <__ANONIO__> line 268. boot-after-install# qemu-system-x86_64: -monitor unix:./monitor: Failed to connect socket ./monitor: No such file or directory error: QEMU died prematurely QEMU died prematurely vde_switch: EOF on stdin, cleaning up and exiting cleaning up builder for '/nix/store/zbpxwwxwy7f6z5z3kg4nf5mjqsywzjvx-vm-test-run-installer-filesystemEncryptedWithKeyfile.drv' failed with exit code 4 Shortening the test name fixes the issue.
* | Merge pull request #50356 from mayflower/gitlab-smtp-fixLinus Heckemann2018-11-15
|\ \ | | | | | | gitlab: fix smtp setting
| * | gitlab: fix smtp settingRobin Gloster2018-11-14
| |/ | | | | | | fixes #50163
* | nixos/tests/all-tests.nix: Fix incron test pathSarah Brofeldt2018-11-14
| |
* | nodePackages.statsd: removeJörg Thalheim2018-11-14
|/ | | | The package/service is broken. Upstream is dead
* Merge pull request #49840 from markuskowa/fix-pgBackupSamuel Dionne-Riel2018-11-14
|\ | | | | nixos/postgresqlBackup: set to umask to 0077
| * nixos/postgresqlBackup: set to umask to 0077Markus Kowalewski2018-11-06
| | | | | | | | | | * Ensure that the backup file is only readable by the owner * Add file permission test to tests
* | Merge pull request #50226 from danieldk/handbrake-1.1.2Renaud2018-11-13
|\ \ | | | | | | handbrake: 1.1.0 -> 1.1.2
| * | tests: handbrake: test transcoding to MKV and MP4.Daniël de Kok2018-11-12
| | |
* | | nixos/tests/networking: Fix syntax erroraszlig2018-11-12
|/ / | | | | | | | | | | | | Introduced by 6c68fbd4e1f8beac39cb1f499ff90c78256262d6. Signed-off-by: aszlig <aszlig@nix.build> Cc: @Ekleog
* | meta.tests: rename into passthru.testsLéo Gaspard2018-11-11
| | | | | | | | | | | | | | Nix currently rejects derivations in `meta` values. This works around that limitation by using `passthru` instead. Closes https://github.com/NixOS/nixpkgs/issues/50230
* | tests: disable some broken tests and/or restrict to x86_64Léo Gaspard2018-11-11
| |
* | tests: split into a separate `all-tests.nix` fileLéo Gaspard2018-11-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will make the list much easier to re-use, eg. for `nixosTests` The drawback is that this approaches makes the ``` nix-build release.nix -A tests.opensmtpd.x86_64-linux ``` command about twice as slow (3s to 6s): it now has to evaluate `nixpkgs` once for each architecture, instead of just having the hardcoded list of tests that allowed to say “ok just evaluate for x86_64-linux”. On the other hand, complete evaluation of `release.nix` should be much faster because we no longer import `nixpkgs` for each test: testing with the following command went from 30s to 18s, and that's just for a few tests. ``` time nix-instantiate --eval --strict nixos/release.nix -A tests.nat ``` I initially wanted to test on the whole `release.nix`, but there are too many broken tests and it takes too long to eval them all, especially compared to the fact that the current implementation breaks some setup. Given developers can just `nix-build nixos/tests/my-test.nix`, it sounds like an overall win.
* | tests: refactor to carry the package set as an argumentLéo Gaspard2018-11-11
| | | | | | | | | | This way, the package set will be possible to pass without re-importing all the time
* | nixos/incron: added nixos test to ensure expected behaviourAaron Andersen2018-11-10
| |
* | Merge pull request #49809 from griff/rspamd-workersFranz Pletz2018-11-09
|\ \ | | | | | | nixos/rspamd: Multiple workers, extraConfig priority & postfix integration
| * | nixos/rspamd: Add options for postfix integrationBrian Olsen2018-11-09
| | | | | | | | | | | | | | | | | | | | | The `rmilter` module has options for configuring `postfix` to use it but since that module is deprecated because rspamd now has a builtin worker that supports the milter protocol this commit adds similar `postfix` integration options directly to the `rspamd` module.
| * | nixos/rspamd: Put extraConfig in included filesBrian Olsen2018-11-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | The lines stored in `extraConfig` and `worker.<name?>.extraConfig` should take precedent over values from included files but in order to do this in rspamd UCL they need to be stored in a file that then gets included with a high priority. This commit uses the overrides option to store the value of the two `extraConfig` options in `extra-config.inc` and `worker-<name?>.inc` respectively.
| * | nixos/rspamd: Support multiple workersBrian Olsen2018-11-06
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the workers option for rspamd was originally implemented it was based on a flawed understanding of how workers are configured in rspamd. This meant that while rspamd supports configuring multiple workers of the same type, so that different controller workers could have different passwords, the NixOS module did not support this because it would write an invalid configuration file if you tried. Specifically a configuration like the one below: ``` workers.controller = {}; workers.controller2 = { type = "controller"; }; ``` Would result in a rspamd configuration of: ``` worker { type = "controller"; count = 1; .include "$CONFDIR/worker-controller.inc" } worker "controller2" { type = "controller"; count = 1; } ``` While to get multiple controller workers it should instead be: ``` worker "controller" { type = "controller"; count = 1; .include "$CONFDIR/worker-controller.inc" } worker "controller" { type = "controller"; count = 1; } ```
* | Merge pull request #49814 from aanderse/giteaRenaud2018-11-08
|\ \ | | | | | | nixos/gitea: fix mysql issue, add mysql socket auth, and add a nixos test
| * | nixos/gitea: add a nixos test to ensure the initial database migration ↵Aaron Andersen2018-11-08
| |/ | | | | | | succeeds so the application can start
* | nixos/tests/hydra: set the project visibleAntoine Eiche2018-11-08
| | | | | | | | | | If projects are not visible, the are not taken into account by search queries and it's quite hard to debug!
* | Merge pull request #44439 from Ekleog/meta-testsTimo Kaufmann2018-11-07
|\ \ | | | | | | [RFC] Use `meta.tests` to link from packages to the tests that test them
| * | dovecot, opensmtpd: add link to test in `meta.tests`Léo Gaspard2018-10-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rationale --------- Currently, tests are hard to discover. For instance, someone updating `dovecot` might not notice that the interaction of `dovecot` with `opensmtpd` is handled in the `opensmtpd.nix` test. And even for someone updating `opensmtpd`, it requires manual work to go check in `nixos/tests` whether there is actually a test, especially given not so many packages in `nixpkgs` have tests and this is thus most of the time useless. Finally, for the reviewer, it is much easier to check that the “Tested via one or more NixOS test(s)” has been checked if the file modified already includes the list of relevant tests. Implementation -------------- Currently, this commit only adds the metadata in the package. Each element of the `meta.tests` attribute is a derivation that, when it builds successfully, means the test has passed (ie. following the same convention as NixOS tests). Future Work ----------- In the future, the tools could be made aware of this `meta.tests` attribute, and for instance a `--with-tests` could be added to `nix-build` so that it also builds all the tests. Or a `--without-tests` to build without all the tests. @Profpatsch described in his NixCon talk such systems. Another thing that would help in the future would be the possibility to reasonably easily have cross-derivation nix tests without the whole NixOS VM stack. @7c6f434c already proposed such a system. This RFC currently handles none of these concerns. Only the addition of `meta.tests` as metadata to be used by maintainers to remember to run relevant tests.
* | | Merge pull request #49792 from griff/rspamd-multifile-enableFranz Pletz2018-11-06
|\ \ \ | | | | | | | | nixos/rspamd: Fix enable for locals and overrides
| * | | nixos/rspamd: Fix enable for locals and overridesBrian Olsen2018-11-05
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | When implementing #49620 I included an enable option for both the locals and overrides options but the code writing the files didn't actually look at enable and so would write the file regardless of its value. I also set the type to loaOf which should have been attrsOf since the code was not written to handle the options being lists. This fixes both of those issues.
* | | Merge pull request #49516 from johanot/kubedns-to-corednsSarah Brofeldt2018-11-06
|\ \ \ | |/ / |/| | nixos/kubernetes: KubeDNS -> CoreDNS
| * | nixos/kubernetes: Replace KubeDNS with CoreDNSJohan Thomsen2018-10-31
| | |
* | | Merge pull request #49283 from aanderse/solrAndreas Rammhold2018-11-04
|\ \ \ | | | | | | | | solr: 4.10.3 -> 7.5.0, refactor service to reflect major changes in version bump
| * | | solr: 4.10.3 -> 7.5.0, refactor service to reflect major changes in version ↵Aaron Andersen2018-11-03
| | | | | | | | | | | | | | | | bump, NixOS test included
* | | | Merge pull request #49620 from griff/rspamd-multifileJoachim F2018-11-03
|\ \ \ \ | |/ / / |/| | | nixos/rspamd: Add support for included files
| * | | nixos/rspamd: Add support for included filesBrian Olsen2018-11-02
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default rspamd will look for multiple files in /etc/rspamd/local.d and /etc/rspamd/override.d to be included in subsections of the merged final config for rspamd. Most of the config snippets in the official rspamd documentation are made to these files and so it makes sense for NixOS to support them and this is what this commit does. As part of rspamd 1.8.1 support was added for having custom Lua rules stored in $LOCAL_CONFDIR/rspamd.local.lua which means that it is now possible for NixOS to support such rules and so this commit also adds support for this to the rspamd module.
* | | Merge pull request #49658 from mayflower/gitlab-refactorFlorian Klink2018-11-03
|\ \ \ | | | | | | | | gitlab: refactor and fix test
| * | | gitlab: refactor and fix testRobin Gloster2018-11-02
| | | |
* | | | nixos/dockerPreloader: preload docker images (#49379)lewo2018-11-03
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This module permits to preload Docker image in a VM in order to reduce OIs on file copies. This module has to be only used in testing environments, when the test requires several Docker images such as in Kubernetes tests. In this case, `virtualisation.dockerPreloader.images` can replace the `services.kubernetes.kubelet.seedDockerImages` options. The idea is to populate the /var/lib/docker directory by mounting qcow files (we uses qcow file to avoid permission issues) that contain images. For each image specified in config.virtualisation.dockerPreloader.images: 1. The image is loaded by Docker in a VM 2. The resulting /var/lib/docker is written to a QCOW file This set of QCOW files can then be used to populate the /var/lib/docker: 1. Each QCOW is mounted in the VM 2. Symlink are created from these mount points to /var/lib/docker 3. A /var/lib/docker/image/overlay2/repositories.json file is generated 4. The docker daemon is started.
* | | nixos/tests/chromium: Fix sandbox info matchingaszlig2018-11-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As reported by @andir, the regular expressions that match the sandbox output are no longer matching in the recent Chromium bump as of bb03fbc2c8b97ef5a4f4205eee51029155164b8a. Instead of a boolean field that determines whether namespace sandboxes are on, the namespace sandbox is now an enum within "Layer 1 Sandbox". I've modified the regular expressions accordingly and also ran the test for the stable branch, which now succeeds. Signed-off-by: aszlig <aszlig@nix.build> Issue: https://github.com/NixOS/nixpkgs/issues/49442 Cc: @bendlas, @andir
* | | chromium tests: inherit timeout from the packageVladimír Čunát2018-11-01
|/ / | | | | | | /cc #49442. It should decrease the waste of resources due to abortions.
* | Merge pull request #49348 from markuskowa/mod-slurm-upgradexeji2018-10-31
|\ \ | | | | | | nixos/slurm: add slurmdbd, run daemons as user
| * | nixos/slurm: add maintainer to module and testMarkus Kowalewski2018-10-30
| | |
| * | nixos/slurm: node/partitionName option -> listMarkus Kowalewski2018-10-30
| | | | | | | | | | | | | | | | | | | | | Make the node and partitionname options lists. There can be more than paratition or set of nodes. Add changes to release notes
| * | nixos/slurm: add slurmdbd to moduleMarkus Kowalewski2018-10-30
| | | | | | | | | | | | | | | * New options "services.slurm.dbdserver.[enable,config]" * Add slurmdbd to test slurm.nix
* | | tests/docs: remove remnants of old allowPing default (#49198)Robin Gloster2018-10-30
|/ / | | | | | | | | This has been defaulting to true since 16.03, we don't need this code anymore, also the note in the documentation has been obsolete for quite a while.
* | postgresql_11: init at 11.0Alyssa Ross2018-10-30
| |
* | postgresql*: use underscores in version numbersAlyssa Ross2018-10-30
| |
* | postgresql10: rename from postgresql100Alyssa Ross2018-10-30
| |
* | nixos/tests/rsyslogd: Fix evalTuomas Tynkkynen2018-10-30
| |
* | nixos/tests/home-assistant: no longer ignore "Timer got out of sync" errorRobert Schütz2018-10-29
| | | | | | | | That error message was removed in https://github.com/home-assistant/home-assistant/pull/17398.
* | Merge pull request #49355 from Mic92/sddmJörg Thalheim2018-10-28
|\ \ | |/ |/| nixos/plasma5: disable ocr tests
| * nixos/plasma5: disable ocr testsJörg Thalheim2018-10-28
| | | | | | | | This is brittle and breaks the test
* | Merge pull request #48901 from Ekleog/opensmtpd-6.4.0obadz2018-10-28
|\ \ | |/ |/| opensmtpd: 6.0.3p1 -> 6.4.0