Skip to content

ci: add round trip validation job#425

Open
micgor32 wants to merge 1 commit into
mainfrom
ci-roundtrip
Open

ci: add round trip validation job#425
micgor32 wants to merge 1 commit into
mainfrom
ci-roundtrip

Conversation

@micgor32
Copy link
Copy Markdown
Contributor

@micgor32 micgor32 commented May 4, 2026

Add CI job that validates that the binaries exported from a given image are identical to the ones generated based on the config from the same image.

Note: this fails for BPM of BG 1.0 because of missmatch in IBB segments:

 00000000: 5f5f 4143 4250 5f5f 1001 0000 0000 0400  __ACBP__........
 00000010: 5f5f 4942 4253 5f5f 1000 000f 0300 0000  __IBBS__........
 00000020: 0000 d1fe 0000 0000 0000 d9fe 0000 0000  ................
 00000030: 0000 0000 0000 0000 0000 0000 0000 0000  ................
 00000040: 0000 0000 0000 0000 1000 0000 0000 0000  ................
 00000050: 0000 0000 0000 0000 0000 0000 0000 0000  ................
 00000060: 0000 0000 0000 0000 0000 0000 f0ff ffff  ................
-00000070: 0b00 2000 74e3 29a8 0414 52e2 0ddd e3e2  .. .t.)...R.....
-00000080: da09 8224 723b 3648 f2d9 de54 c944 215b  ...$r;6H...T.D![
-00000090: 5be1 fd92 0200 0000 0000 00f0 ff80 810f  [...............
+00000070: 0b00 2000 9ae8 c524 bc22 6c26 9258 5990  .. ....$."l&.XY.
+00000080: 9a92 227c 70c1 98f7 a9cc d348 5faf 88a0  .."|p......H_...
+00000090: 14eb aef2 0200 0000 0000 00f0 ff80 810f  ................

This will be fixed with the recent changes in Fiano (see #424), the missmatch comes from the image size calculation not being consistent if the image size does not match the bios flash area size calculated from IFD (which is the case in HPE Proliant Gen10 fw image used here for the test).

Add CI job that validates that the binaries exported from a given image
are identical to the ones generated based on the config from the same
image.

Signed-off-by: Michal Gorlas <michal.gorlas@9elements.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant