Skip to content

Enable ARM64 E2E tests in US Government - #3680

Merged
maddieford merged 2 commits into
Azure:developfrom
maddieford:arm64_fairfax
Sep 17, 2026
Merged

maddieford merged 2 commits into
Azure:developfrom
maddieford:arm64_fairfax

Conversation

@maddieford

Copy link
Copy Markdown
Contributor

Description

There is ARM64 capacity in Fairfax cloud now. This PR updates the image definitions for the endorsed-arm64 images in image.yml to reflect the availability in Gov cloud. The PR also updates the sku used for the arm64 images to the latest gen arm64 sku which is available in both our Public and Fairfax test subs

Issue #

PR information

  • Ensure development PR is based on the develop branch.
  • If applicable, the PR references the bug/issue that it fixes in the description.
  • New Unit tests were added for the changes made

Quality of Code and Contribution Guidelines


Distro maintenance information, if applicable

  • This is a contribution from a distro maintainer
  • The changes in this PR have been taken as a downstream patch (Note: it is not recommended to patch the agent without upstream review and approval)

Use Standard_D2ps_v6 for ARM64 test images, enable the images available in Government cloud, and add Oracle Linux 10 ARM64 to the endorsed set.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 15, 2026 22:44
- "rhel_95_arm64"
- "ubuntu_2204_arm64"
- "ubuntu_2404_arm64"
- "oracle_10_arm64"

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.

Adding oracle 10 arm64 to the endorsed-arm64 list since it is available now

- "Standard_D2ps_v6"
locations:
AzureUSGovernment: []
AzureUSGovernment: [] # TODO: This image is not available in Gov cloud. Update this once the image is available in Gov cloud.

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.

azurelinux4 arm64 isn't available in Gov

# The VM size chosen by LISA for ARM64 VMs has recently produced VM that do not start, so we explicitly use
# Standard_B2pls_v2 (which was the size LISA was using previously). Consider removing this explicit size
# if/when the issue with the size selected by LISA is fixed.
# Standard_D2ps_v6. Consider removing this explicit size if/when the issue with the size selected by LISA is fixed.

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.

I confirmed with the ARM64 PM that this sku is the latest gen arm64 sku

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.

Standard_B2pls_v2 wasn't available in fairfax

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Debian ARM64 entries conflict with the stated SKU update.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This PR updates ARM64 E2E image definitions for Azure Government availability, adds Oracle Linux 10 ARM64 coverage, and updates VM sizing.

Changes:

  • Enables supported ARM64 images in Gov cloud.
  • Adds Oracle Linux 10 ARM64.
  • Updates selected ARM64 VM sizes to Standard_D2ps_v6.
File summaries
File Summary
tests_e2e/test_suites/images.yml Updates ARM64 image availability, cloud exclusions, test membership, and VM sizing. Debian 12 and 13 ARM64 entries still use Standard_B2pls_v2, requiring update or clarification.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread tests_e2e/test_suites/images.yml
@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.07%. Comparing base (8646bf3) to head (158c90c).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #3680   +/-   ##
========================================
  Coverage    74.07%   74.07%           
========================================
  Files          123      123           
  Lines        18871    18871           
  Branches      2530     2530           
========================================
  Hits         13979    13979           
  Misses        4289     4289           
  Partials       603      603           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI review requested due to automatic review settings September 15, 2026 22:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Azure Linux 4 ARM64 remains excluded from Azure Government and requires clarification or adjustment.

Review details

Suppressed comments (1)

tests_e2e/test_suites/images.yml:200

  • Azure Linux 4 ARM64 is still explicitly unavailable in AzureUSGovernment. The combinator treats an empty location list as a skip (agent_test_suite_combinator.py:580-584), so every endorsed-arm64 suite continues to omit this image in Gov despite this PR changing its VM size. If this image is intended to be part of the enablement, remove this restriction; otherwise remove it from endorsed-arm64 or clearly document the exception.
         AzureUSGovernment: []   # TODO: This image is not available in Gov cloud. Update this once the image is available in Gov cloud.
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@maddieford
maddieford merged commit c425b19 into Azure:develop Sep 17, 2026
14 checks passed
@maddieford
maddieford deleted the arm64_fairfax branch September 17, 2026 14:19
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.

3 participants