Skip to content

[CI] Add aspeed-arm64 build target for AST2700 BMC platform#25963

Merged
yxieca merged 3 commits intosonic-net:masterfrom
Gfrom2016:add_aspeed_build_target
Mar 17, 2026
Merged

[CI] Add aspeed-arm64 build target for AST2700 BMC platform#25963
yxieca merged 3 commits intosonic-net:masterfrom
Gfrom2016:add_aspeed_build_target

Conversation

@Gfrom2016
Copy link
Copy Markdown
Contributor

@Gfrom2016 Gfrom2016 commented Mar 9, 2026

Why I did it

Add a new CI build target aspeed-arm64 to enable building SONiC images for the Aspeed AST2700 BMC platform (arm64).

The Aspeed AST2700 platform support was added in PR #24898 but no CI build target existed to produce the sonic-aspeed-arm64.bin image. This PR adds the pipeline definition so the aspeed image is built alongside other arm64 targets.

Work item tracking
  • Microsoft ADO (number only): 36717394

How I did it

Added a new build target entry aspeed-arm64 in azure-pipelines.yml under the arm64 build stage, specifying PLATFORM_NAME: aspeed and PLATFORM_ARCH: arm64, using the existing sonicso1ES-arm64 pool.

How to verify it

  1. The aspeed-arm64 pipeline job runs successfully and produces sonic-aspeed-arm64.bin
  2. Install the image on an Aspeed AST2700 board (EVB or NextHop B27) using:
    sudo sonic-installer install --skip-package-migration sonic-aspeed-arm64.bin
    
  3. After reboot, verify all 7 expected containers are running:
    docker ps -a
    
    Expected: database, sysmgr, restapi, gnmi, telemetry, lldp, pmon

Tested branch (Please provide the tested image version)

  • master — SONiC-OS-master-25963.1055394-49231a76b (Build ID: 1055394)
    • Verified on NextHop B27 board (arm64-nexthop_b27-r0): all 7 containers running ✅

Which release branch to backport (provide reason below if selected)

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Description for the changelog

Add aspeed-arm64 build target to CI pipeline for Aspeed AST2700 BMC platform image builds.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: zitingguo [email protected]

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@Gfrom2016 Gfrom2016 changed the title Add new target aspeed-arm64 [CI] Add aspeed-arm64 build target for AST2700 BMC platform Mar 10, 2026
@Gfrom2016 Gfrom2016 marked this pull request as ready for review March 10, 2026 06:41
@Gfrom2016 Gfrom2016 requested a review from lguohan as a code owner March 10, 2026 06:41
Copilot AI review requested due to automatic review settings March 10, 2026 06:41
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Azure Pipelines build job group so CI produces the Aspeed AST2700 (BMC) arm64 installer image (sonic-aspeed-arm64.bin) alongside existing arm64 targets.

Changes:

  • Introduced a new aspeed-arm64 job group in the Build stage.
  • Configured the job to run on the existing sonicso1ES-arm64 pool with PLATFORM_NAME=aspeed and PLATFORM_ARCH=arm64.

You can also share your feedback on Copilot code review. Take the survey.

@Gfrom2016
Copy link
Copy Markdown
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Gfrom2016 and others added 3 commits March 12, 2026 09:52
BMC is a minimal embedded platform that does not need REST API.
Remove DOCKER_RESTAPI from the dockers list and add it to
DISABLED_PACKAGES_LOCAL to skip building the package entirely.

Co-authored-by: Copilot <[email protected]>
Signed-off-by: zitingguo <[email protected]>
@Gfrom2016 Gfrom2016 force-pushed the add_aspeed_build_target branch from 80d0d57 to b85f86d Compare March 12, 2026 02:18
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

$(DOCKER_EVENTD) $(DOCKER_DASH_HA) $(DOCKER_STP)
$(DOCKER_EVENTD) $(DOCKER_DASH_HA) $(DOCKER_STP) \
$(DOCKER_RESTAPI)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor issue: is this change still needed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is still needed. The DISABLED_PACKAGES_LOCAL + filter-out prevents these docker images from being built at all — removing them from _DOCKERS only prevents inclusion in the final image, but they'd still be compiled, wasting CI time on the arm64 runner. This follows the same pattern used by Pensando and NVIDIA Bluefield platforms.

The `` line is a debug print showing which packages are filtered — happy to remove it if you prefer cleaner output.

@yxieca
Copy link
Copy Markdown
Contributor

yxieca commented Mar 12, 2026

/azpw ms_conflict

1 similar comment
@StormLiangMS
Copy link
Copy Markdown
Contributor

/azpw ms_conflict

@yxieca yxieca merged commit 7ae9fbc into sonic-net:master Mar 17, 2026
21 checks passed
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.

6 participants