Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add ARM64 GitHub Actions matrix build support#4352

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
Copilot wants to merge2 commits intomain
base:main
Choose a base branch
Loading
fromcopilot/fix-3ecf5681-f475-4cf5-b695-363f04d4e8fa

Conversation

Copilot
Copy link
Contributor

@CopilotCopilotAI commentedJul 18, 2025
edited
Loading

Summary

This PR adds ARM64 architecture support to the GitHub Actions build matrix to ensure the Application Insights Java Agent works correctly on ARM64 platforms.

Changes

🔧 Matrix Expansion

Added ARM64 runners to the build matrix in.github/workflows/build-common.yml:

  • ubuntu-latest-arm64 - Linux ARM64 runner
  • windows-latest-arm64 - Windows ARM64 runner

🔄 Condition Update

Updated the Windows-specific condition from:

if:matrix.os == 'windows-2022'

to:

if:startsWith(matrix.os, 'windows')

This ensures the "Support long paths" step runs on bothwindows-2022 andwindows-latest-arm64 runners.

Impact

Build Matrix Expansion

  • Before: 10 jobs (2 OS × 5 Java versions × 1 VM)
  • After: 20 jobs (4 OS × 5 Java versions × 1 VM)

Platform Coverage

The agent will now be tested on:

  • ✅ Windows x64 (windows-2022)
  • ✅ Linux x64 (ubuntu-latest)
  • Linux ARM64 (ubuntu-latest-arm64) 🆕
  • Windows ARM64 (windows-latest-arm64) 🆕

Java Version Coverage

All Java versions (8, 11, 17, 21, 23) will be tested on ARM64 platforms.

Automatic Inheritance

Other workflows that usebuild-common.yml automatically inherit ARM64 support:

  • build.yml (PR and main branch builds)
  • build-daily.yml (daily builds)
  • build-daily-no-build-cache.yml (daily builds without cache)

Testing

  • YAML syntax validated
  • Matrix configuration verified
  • Windows condition logic tested for all runners
  • No breaking changes to existing workflows

This ensures the Application Insights Java Agent maintains compatibility and performance across both x64 and ARM64 architectures on Windows and Linux platforms.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Clickhere to start the survey.

Co-authored-by: trask <218610+trask@users.noreply.github.com>
@CopilotCopilotAI changed the title[WIP] add arm64 GitHub action matrix buildAdd ARM64 GitHub Actions matrix build supportJul 18, 2025
Copilot finished work on behalf oftraskJuly 18, 2025 20:30
@CopilotCopilotAI requested a review fromtraskJuly 18, 2025 20:30
@trask
Copy link
Member

Not sure about this, it seems arm64 runners are more limited and so slows the overall build down

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@trasktraskAwaiting requested review from trasktrask is a code owner

@harsimarharsimarAwaiting requested review from harsimarharsimar will be requested when the pull request is marked ready for reviewharsimar is a code owner

@jeanbisuttijeanbisuttiAwaiting requested review from jeanbisuttijeanbisutti will be requested when the pull request is marked ready for reviewjeanbisutti is a code owner

@rajkumar-rangarajrajkumar-rangarajAwaiting requested review from rajkumar-rangarajrajkumar-rangaraj will be requested when the pull request is marked ready for reviewrajkumar-rangaraj is a code owner

@ramthiramthiAwaiting requested review from ramthiramthi will be requested when the pull request is marked ready for reviewramthi is a code owner

@xiang17xiang17Awaiting requested review from xiang17xiang17 will be requested when the pull request is marked ready for reviewxiang17 is a code owner

At least 1 approving review is required to merge this pull request.

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@Copilot@trask

[8]ページ先頭

©2009-2025 Movatter.jp