Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Graviton to Github CI OpenCL CTS planned testing #664

Open
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

alan-forbes-cp
Copy link
Collaborator

@alan-forbes-cp alan-forbes-cp commented Jan 29, 2025

Overview

Add aarch64 Graviton to Github CI OpenCL CTS planned testing

Reason for change

Replace current qemu-based workflow

Description of change

  • Add new graviton dockerfile.
  • Update publish job in docker workflow for graviton
  • Add new runs-on: and image: config to OpenCL CTS job for graviton.
  • Remove qemu setting from OpenCL CTS aarch64 run job
  • Host target lists and image config were refactored as they had become difficult to read.
  • Dockerfiles and corresponding image names have been updated to reflect the arch.

Anything else we should know?

  • Test results are the same as qemu (and Gitlab)
  • Note that some temporary changes which were added to limit the number of jobs that run during development will be removed prior to final merge.
  • Similarly the new docker image for aarch64 OpenCL CTS will be pointed at uxlfoundation rather than the current development registry.

@alan-forbes-cp alan-forbes-cp changed the title DRAFT: add cp-graviton Add Graviton to Github CI planned testing Feb 3, 2025
@alan-forbes-cp alan-forbes-cp changed the title Add Graviton to Github CI planned testing Add Graviton to Github CI OpenCL CTS planned testing Feb 3, 2025
@alan-forbes-cp alan-forbes-cp marked this pull request as ready for review February 3, 2025 17:40
@alan-forbes-cp alan-forbes-cp requested a review from a team as a code owner February 3, 2025 17:40
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.

2 participants