Skip to content
Commit 7797a07f authored by Ryan Roberts's avatar Ryan Roberts
Browse files

cli: Ensure "plan exec" remains compatible with old containers



Commit ac88ccb ("cli: Remove RESULTCLASS summary field") removed the
RESULTCLASS.summary field, and as a result this was removed from the
results.csv file passed back from the container to the test runner. This
was a breaking change and was intentional; The plan was to rebuild the
containers.

But it turns out we can't easily rebuild the speedometer container
because the version of Chromium that the old version was using is no
longer available. Changing the browser version will change the benchmark
performance so a new container needs to be properly versioned.

So let's just allow the old container to work with the new code. We
allow "summary" but ignore it.

Signed-off-by: Ryan Roberts's avatarRyan Roberts <ryan.roberts@arm.com>
parent 033c9d16
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment