Skip to content

Conversation

@dennisYehCienet
Copy link
Collaborator

@dennisYehCienet dennisYehCienet commented Dec 15, 2025

Description

This PR introduces placeholders into the YML files to enable the use of different variables for both v6e and v7x during the upload of CI procedures. This applies to both pipeline_jax.yml and main.yml.

FYI, the new procedure covers the pipeline_jax_tpu7x.yml, so the original file has been removed.

Tests

buildkite (test few main.yml and pipeline_jax.yml)

Checklist

Before submitting this PR, please make sure:

  • I have performed a self-review of my code.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have made or will make corresponding changes to any relevant documentation.

@github-actions
Copy link

Description

Start with a short description of what the PR does and how this is a change from
the past.

The rest of the description includes relevant details and context, examples:

  • why is this change being made,
  • the problem being solved and any relevant context,
  • why this is a good solution,
  • some information about the specific implementation,
  • shortcomings of the solution and possible future improvements.

If the change fixes a Github issue, please include a link, e.g.,:
FIXES: #123456

Tests

Please describe how you tested this change, and include any instructions and/or
commands to reproduce.

Checklist

Before submitting this PR, please make sure:

  • I have performed a self-review of my code.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have made or will make corresponding changes to any relevant documentation.

@dennisYehCienet dennisYehCienet changed the title [CI] Enhance uploading the CI procedures for both v6e and v7x. [CI] This PR enhances testing of the CI procedures on both v6e and v7x. Dec 15, 2025
@dennisYehCienet dennisYehCienet added the ready ONLY add when PR is ready to merge/full CI is needed label Dec 16, 2025
@dennisYehCienet dennisYehCienet force-pushed the dennis_v7_test branch 3 times, most recently from 310cd3b to 113c2c8 Compare December 16, 2025 08:13
@CienetStingLin CienetStingLin marked this pull request as ready for review December 16, 2025 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants