search

rust-vmm / rust-vmm-ci

Apache License 2.0
18 stars 33 forks source link

fix bug in coverage test #74

Closed andreeaflorescu closed 2 years ago

andreeaflorescu commented 3 years ago

Make sure that there are no spaces in the JSON field that defines the features, otherwise we can encounter failures when running kcov.

Example of failure: https://buildkite.com/rust-vmm/mshv-ci/builds/7#cd3a2709-ad84-484a-9fef-ab41da00b7c8

ahmedaabouzied commented 2 years ago

I'll start working on this.

andreeaflorescu commented 2 years ago

We should also check that it is a valid json if that is possible. Do you think you could also add a check for that @ahmedaabouzied?

ahmedaabouzied commented 2 years ago

Sure @andreeaflorescu , I'll try to