search

snabblab / snabblab-nixos

NixOS configuration for the Snabb Lab
59 stars 17 forks source link

dpdk-1.8.0: disable since it doesn't work #79

Closed domenkozar closed 7 years ago

domenkozar commented 8 years ago

cc @lukego

lukego commented 7 years ago

Sorry for slow reply...

I don't think it makes sense to remove a test because it fails. Just now the test case is conveying useful information i.e. that our test setup fails with DPDK 1.8. If we remove the test then we have less information i.e. we lose track of whether we are compatible with DPDK 1.8 or not. So passing is best but failing is better than no test at all.

I also suspect there is some trivial reason that this fails. Historically we have tested with older DPDKs and SnabbBot did so until very recently. I have a vague recollection that older DPDKs are sensitive to specific QEMU command-line options. Is the problem due to some difference between how SnabbBot runs QEMU vs how Hydra/Nix does?

lukego commented 7 years ago

Just a related note on the telecom industry:

It is common for vendors to freeze all of their dependencies when developing a product and then not update them unless absolutely necessary. We should be prepared to interoperate with 5+ year old versions of Linux, DPDK, FreeBSD, etc in the wild. Having solid interoperability across the whole software ecosystem is important for making Snabb easy to deploy.

domenkozar commented 7 years ago

Understood :)