mbox series

[00/18] New CI image and fixes

Message ID 20240717-docker-image-v1-0-7c7fc6251e7a@flygoat.com
Headers show
Series New CI image and fixes | expand

Message

Jiaxun Yang July 17, 2024, 2:29 p.m. UTC 
Hi all,

This series build a new CI image based on Ubuntu focal with LoongArch64
support, fixed various python scripts for python 3.12, fixed various
problems popped up when testing againt latest software.

Last two commits are for demonstration purpose and not for commit
into repo.

CI runs passed at azure [1].

Thanks
[1]: https://flygoat.visualstudio.com/u-boot/_build/results?buildId=64&view=results

Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
---
Jiaxun Yang (18):
      binman: Replace pkg_resources with importlib.resources
      py: Replace distutils.core with setuptools
      doc/sphinx: Remove usage of six
      py: Bump pylint version and clear warnings
      binman: Workaround lz4 cli padding in test cases
      tests/test_event_dump: Relax match rule for output
      lib/charset & efi: Fix possible unaligned accesses
      CI: Ensure pip install is always performed in venv
      CI: Dockerfile: Set global git name & email config
      CI: Dockerfile: Bump base OS version
      CI: Dockerfile: Bump GRUB to 2.12
      CI: Dockerfile: Bump QEMU to 9.0.1
      CI: Dockerfile: Bump fiptool to 2.10.4
      CI: Dockerfile: Bump genimage to 17
      CI: Dockerfile: Use kernel.org upstream for trace-cmd
      CI: Dockerfile: Bump coreboot to 24.05
      CI: Dockerfile: Add LoongArch64 support
      [NFC] Use Jiaxun's CI Image

 .azure-pipelines.yml                            | 22 ++++----
 .gitlab-ci.yml                                  | 19 ++++---
 doc/develop/python_cq.rst                       |  4 +-
 doc/sphinx/kfigure.py                           |  3 +-
 lib/charset.c                                   | 21 ++++----
 lib/efi_loader/efi_device_path.c                | 11 +---
 test/py/tests/test_event_dump.py                | 10 ++--
 test/py/tests/test_ums.py                       |  1 +
 test/py/tests/test_usb.py                       |  1 +
 tools/binman/control.py                         | 18 +++----
 tools/binman/etype/fdtmap.py                    |  1 +
 tools/binman/etype/fit.py                       |  1 +
 tools/binman/etype/image_header.py              |  1 +
 tools/binman/etype/pre_load.py                  |  2 +
 tools/binman/etype/ti_board_config.py           |  1 +
 tools/binman/etype/x509_cert.py                 |  1 +
 tools/binman/ftest.py                           |  8 ++-
 tools/binman/setup.py                           |  2 +-
 tools/binman/state.py                           |  1 +
 tools/binman/test/184_compress_section_size.dts |  1 +
 tools/buildman/builder.py                       |  2 +
 tools/buildman/requirements.txt                 |  1 +
 tools/docker/Dockerfile                         | 68 +++++++++++++++----------
 tools/dtoc/setup.py                             |  2 +-
 tools/microcode-tool.py                         |  1 +
 tools/patman/test_checkpatch.py                 |  2 +
 tools/qconfig.py                                |  1 +
 27 files changed, 123 insertions(+), 83 deletions(-)
---
base-commit: 902d8ee94ce29a088a325da6e69eeb1a25f1fec7
change-id: 20240610-docker-image-868126a1a929

Best regards,

Comments

Tom Rini July 18, 2024, 2:36 p.m. UTC | #1 
On Wed, Jul 17, 2024 at 10:29:11PM +0800, Jiaxun Yang wrote:

> Hi all,
> 
> This series build a new CI image based on Ubuntu focal with LoongArch64
> support, fixed various python scripts for python 3.12, fixed various
> problems popped up when testing againt latest software.
> 
> Last two commits are for demonstration purpose and not for commit
> into repo.
> 
> CI runs passed at azure [1].
> 
> Thanks
> [1]: https://flygoat.visualstudio.com/u-boot/_build/results?buildId=64&view=results
> 
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>

Overall this looks good but please split this up as:

> ---
> Jiaxun Yang (18):
>       binman: Replace pkg_resources with importlib.resources
>       py: Replace distutils.core with setuptools
>       doc/sphinx: Remove usage of six
>       py: Bump pylint version and clear warnings
>       binman: Workaround lz4 cli padding in test cases
>       tests/test_event_dump: Relax match rule for output
>       lib/charset & efi: Fix possible unaligned accesses

The binman patches (and change py: Replace.. to tool: Replace..).
The doc/sphinx patch, standalone.
Break the pylint patch up in to chunks fixing various tools, and then
standalone bump the version.
The dump test change by itself.
The unaligned access change by itself.


>       CI: Ensure pip install is always performed in venv
>       CI: Dockerfile: Set global git name & email config
>       CI: Dockerfile: Bump base OS version
>       CI: Dockerfile: Bump GRUB to 2.12
>       CI: Dockerfile: Bump QEMU to 9.0.1
>       CI: Dockerfile: Bump fiptool to 2.10.4
>       CI: Dockerfile: Bump genimage to 17
>       CI: Dockerfile: Use kernel.org upstream for trace-cmd
>       CI: Dockerfile: Bump coreboot to 24.05
>       CI: Dockerfile: Add LoongArch64 support

These are their own series.

>       [NFC] Use Jiaxun's CI Image

Just don't send this part next time.

Thanks!