Google Git
Sign in
chromium / chromiumos / platform / fw-testing-configs / 89507fe8aee2bd0947d59717d7c0a4481429e4b9
commit89507fe8aee2bd0947d59717d7c0a4481429e4b9[log] [tgz]
authorUsha P <usha.p@intel.com>Thu Oct 20 15:15:07 2022
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>Mon Oct 24 03:14:21 2022
treea6961b60896d7caf853a293e39d88328ef5aa121
parentfca9d86ce327d3f69129c57aa09e16afdf9a443e [diff]
nissa: Set cbmem_preserved_by_ap_reset value to false.

cbmem_preserved_by_ap_reset value needs to be set true for the ARM boards as per the below CL.
https://chromium-review.googlesource.com/c/chromiumos/platform/fw-testing-configs/+/3705099

nissa is expected to have this value set to false.

BUG=b:254631984
BRANCH=main
TEST=Verify firmware_MiniDiag FAFT test passing on nissa boards.

Signed-off-by: Usha P <usha.p@intel.com>
Change-Id: I41b8d1089313a968826914e2cee0a1966a8288ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/fw-testing-configs/+/3966013
Reviewed-by: Reka Norman <rekanorman@chromium.org>
Commit-Queue: Reka Norman <rekanorman@chromium.org>
2 files changed
tree: a6961b60896d7caf853a293e39d88328ef5aa121
  1. .gitignore
  2. .style.yapf
  3. arkham.json
  4. asuka.json
  5. asurada.json
  6. atlas.json
  7. auron.json
  8. banjo.json
  9. banon.json
  10. betty.json
  11. bob.json
  12. brain.json
  13. brask.json
  14. brya.json
  15. buddy.json
  16. candy.json
  17. caroline.json
  18. cave.json
  19. celes.json
  20. chell.json
  21. cherry.json
  22. cheza.json
  23. consolidate.py
  24. consolidate_unittest.py
  25. CONSOLIDATED.json
  26. coral.json
  27. corsola.json
  28. cyan.json
  29. dedede.json
  30. DEFAULTS.json
  31. draco.json
  32. dragonegg.json
  33. drallion.json
  34. edgar.json
  35. elm.json
  36. endeavour.json
  37. enguarde.json
  38. eve.json
  39. expresso.json
  40. fievel.json
  41. fizz.json
  42. gale.json
  43. gandof.json
  44. glados.json
  45. gnawty.json
  46. gru.json
  47. grunt.json
  48. guado.json
  49. guybrush.json
  50. hana.json
  51. hatch.json
  52. heli.json
  53. herobrine.json
  54. jacuzzi.json
  55. jecht.json
  56. jetstream.json
  57. kalista.json
  58. keeby.json
  59. kefka.json
  60. kevin.json
  61. kevin64.json
  62. kevin_tpm2.json
  63. kip.json
  64. kukui.json
  65. kunimitsu.json
  66. lars.json
  67. lulu.json
  68. mickey.json
  69. mistral.json
  70. nami.json
  71. nasher.json
  72. nautilus.json
  73. ninja.json
  74. nissa.json
  75. nocturne.json
  76. oak.json
  77. octopus.json
  78. orco.json
  79. OWNERS
  80. paine.json
  81. pinky.json
  82. platform_json.py
  83. platform_json_unittest.py
  84. poppy.json
  85. PRESUBMIT.cfg
  86. PRESUBMIT.py
  87. puff.json
  88. pyro.json
  89. query_by_field.py
  90. query_by_field_unittest.py
  91. rambi.json
  92. rammus.json
  93. README.md
  94. reef.json
  95. reef_uni.json
  96. reks.json
  97. relm.json
  98. rikku.json
  99. samus.json
  100. sand.json
  101. sarien.json
  102. scarlet.json
  103. sentry.json
  104. setzer.json
  105. shotzo.json
  106. skyrim.json
  107. slippy.json
  108. snappy.json
  109. soraka.json
  110. storm.json
  111. strago.json
  112. strongbad.json
  113. sumo.json
  114. swanky.json
  115. terra.json
  116. tidus.json
  117. tiger.json
  118. trogdor.json
  119. ultima.json
  120. veyron.json
  121. volteer.json
  122. whirlwind.json
  123. winky.json
  124. wizpig.json
  125. yuna.json
  126. zork.json
README.md

CrOS fw-testing-configs: User’s Guide

go/cros-fw-testing-configs-guide

Background

End-to-end firmware testing in ChromeOS relies on board-specific configuration files. Previously, those files were stored in Autotest. Now, they have been moved into a separate repository, fw-testing-configs, so that other testing frameworks (e.g. Tast, SerialTest) can access them too.

The design document for the new repository is at go/cros-fw-testing-configs.

Working With Configs

File Locations & Names

All the config files are located at the top directory of the fw-testing-configs repository.

The fw-testing-configs repository currently has two checkouts in the ChromeOS manifest: one inside of Autotest, and one inside of tast-tests.

There is one config file for each platform, named as ${PLATFORM}.json: for example, octopus.json. There are also one special config file, DEFAULTS.json and CONSOLIDATED.json. DEFAULTS.json contains default values and documentation for each attribute. CONSOLIDATED.json is a generated file, containing the config data from all the other JSON files.

File Contents

Each config file should contain a single object whose fields override the values specified in DEFAULTS.json. Any fields which do not have a corresponding value in DEFAULTS.json will be ignored.

There are a few special fields to be aware of:

  • platform (required): A string which should exactly match the config file’s basename (minus the .json extension).
  • parent (optional): A string which can contain the name of a parent platform, whose fields will be inherited with lower precedence. See “Inheritance” below. Example: asuka.json
  • models (optional): An object which can contain the names of any models, whose fields will be inherited with higher precedence. See “Inheritance” below. Example: octopus.json

Inheritance

There is an inheritance model in these config files. In all cases, the most specific configuration is used:

[Model > ] Platform [ > Parent [...] ] > DEFAULTS

For each attribute that exists in DEFAULTS.json (besides the documentation attributes):

  • If that attribute is given a value in the platform’s models[${MODEL}], then that is the value that is used.
  • Otherwise, if that attribute is given a value in the ${PLATFORM}.json, then that is the value that is used.
  • Otherwise, if ${PLATFORM}.json specifies a parent configuration via the parent attribute, then that is the value that is used.
  • Parent configurations can specify parents as well, and the inheritance recurses until a config file does not have a parent attribute.
  • Finally, for any attribute which was not otherwise given a value, the value from DEFAULTS.json is used.

Workflows

Consolidate before uploading

CONSOLIDATED.json is a generated script containing the data from all other JSON files. This allows Tast tests to access the config data. Thus, it is important to keep CONSOLIDATED.json up-to-date.

When you make a change to fw-testing-configs JSON, before you upload, be sure to run ./consolidate.py (with no command-line args). This will update CONSOLIDATED.json. Amend your change to include the update CONSOLIDATED.json, and then re-upload.

A preupload hook will verify that your CONSOLIDATED.json is up-to-date.

Modifying Tests and Configs

fw-testing-configs is a separate repository from both Autotest and tast-tests. Thus, edits to fw-testing-configs require a separate CL from edits to Autotest and tast-tests. If you are accustomed to working with config files directly in Autotest, this is a slightly different workflow.

When you are editing config files, please be sure to run git commands from within the fw-testing-configs checkout. If you run git add . or repo upload --cbr . from within autotest/server/cros/faft/, then your changes to the config files will not be captured.

If you are modifying both test files and config files, then you will need at least two CL’s: one for the test change, and one for the fw-testing-configs change. Consider also whether your change will impact the other testing repositories (Autotest, Tast, SerialTest): you might need to run tests or make changes there, too.

When submitting multiple CL’s like that, please use the Cq-Depend syntax as appropriate. If you don‘t use Cq-Depend, you risk the name of a config attribute changing while tests are still looking for the old name, and then tests will break. The reverse is a risk, too. You might need both CL’s to depend on each other. Note that this while this precaution mitigates risk, it does not fully eliminate risk: only one of the CL’s might get uploaded to the lab machines running Autotest/Tast, or one CL may be reverted without the other.

Accessing Configs During Tests

In Autotest, configs are loaded during FirmwareTest.initialize. Config values can be accessed via self.faft_config.${ATTRIBUTE}, such as self.faft_config.chrome_ec.

In Tast, configs can be created via firmware.NewConfig. In order to conform with Go’s style, attribute names are modified to MixedCaps. Config values can be accessed via cfg[${ATTRIBUTE}], such as cfg[ChromeEC].

Utility Scripts

This repo offers a few handy-dandy utility scripts.

consolidate.py, as described above, combines all the individual ${PLATFORM}.json files into one file, CONSOLIDATED.json:

> consolidate.py

platform_json.py outputs the final configuration for a single platform. --model or -m will include a model-specific override. --field or -f will only print a single field value. --condense-output will trim unnecessary whitespace, making it easier to consume via other scripts.

> platform_json.py octopus
> platform_json.py octopus -m robo360
> platform_json.py octopus -f has_lid
> platform_json.py octopus -m robo360 -f has_lid --condense-output

query_by_field.py reports a list of all platforms and models matching certain field conditions. Multiple conditions can be specified. See usage info (query_by_field.py -h) for details about supported operators. Note that you might need to escape some of the operators (like \!) for Bash to parse them.

> query_by_field.py has_lid=false
> query_by_field.py "firmware_screen<=8"
> query_by_field.py chrome_ec=true ec_capability\!:x86

Each of these scripts also supports --help (or -h) for usage info.