[Fuego] fuego-core patches (migration to testspecs and more)

Daniel Sangorrin daniel.sangorrin at toshiba.co.jp
Thu Mar 30 01:04:28 UTC 2017


Hi,

This bag of patches includes bug fixes and changes in functionality.

[PATCH 01/16] indentation: fix indentation for dhrystone parser
[PATCH 02/16] abort: fix the abort function
  -> without this fix failed jobs were looping until the timeout.
     Seems like a Jenkins' bug.

[PATCH 03/16] shell e flag: remove any e flag from fuego
  -> As Tim mentioned during the AGL web meeting the e flag is
     not the way to catch errors so I removed it.

[PATCH 04/16] parser: remove error message when no matches
[PATCH 05/16] benchmark: use the same pattern as in functional
[PATCH 06/16] board file: remove conf from the path to the board file
  -> fuego-ro/boards/ instead of fuego-ro/conf/boards

[PATCH 07/16] batch: remove batch_testplan reference
  -> I will implement an add-batch subcommand in the future and
     fix reports.sh. For now remove this dead code.

[PATCH 08/16] testplans: use testspecs instead of testplans
  -> This is the MOST IMPORTANT patch in this series. It removes
     the need to have a testplan or have your test in testplan_default.
     Actually testplans are only needed when adding jobs, or
     when creating a batch job. The rest can be done with just
     the test spec. This also allowed me to simplify ovgen.py
     a little bit more.

[PATCH 09/16] distrib: DISTRIB must be defined in the board file
  -> I didnt update all of the boards, because I want to completely
     rewrite the board files.

[PATCH 10/16] user_checks: add to commands so they execute under
   -> thanks for the tip

[PATCH 11/16] ftc:rm-jobs: remove limitation of 4 words
[PATCH 12/16] description setter: put log if the test fails
  -> Tim, you were right about this.

[PATCH 13/16] ftc: remove testplans and use testspecs
  -> Maybe i should have rebased this and squash it with 08/16 sorry

[PATCH 14/16] ftc test: remove distrib and update paths
[PATCH 15/16] fail_check_cases should not abort the job
  -> this might need more review

[PATCH 16/16] a bit of cleaning and style fixes

Thanks,
Daniel



More information about the Fuego mailing list