Skip to content

Trigger unit tests for docker images upload workflow#3329

Draft
xibinliu wants to merge 1 commit intomainfrom
xibin/ci2
Draft

Trigger unit tests for docker images upload workflow#3329
xibinliu wants to merge 1 commit intomainfrom
xibin/ci2

Conversation

@xibinliu
Copy link
Copy Markdown
Collaborator

@xibinliu xibinliu commented Mar 6, 2026

Description

  • images will only be tagged to the current date when unit tests pass
  • Images will only be tagged to "latest" when unit tests pass

Tests

See this manually triggered workflow run

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

Comment thread .github/workflows/UploadDockerImages.yml Outdated
Comment thread .github/workflows/UploadDockerImages.yml Outdated
@xibinliu xibinliu mentioned this pull request Mar 6, 2026
4 tasks
@xibinliu xibinliu force-pushed the xibin/ci2 branch 2 times, most recently from ffd00f3 to ea56990 Compare March 6, 2026 21:33
context: .
file: ${{ inputs.dockerfile }}
tags: gcr.io/tpu-prod-env-multipod/${{ inputs.image_name }}:latest
tags: gcr.io/tpu-prod-env-multipod/${{ inputs.image_name }}:${{ inputs.image_date }}-build-${{ github.run_id }}
Copy link
Copy Markdown
Collaborator Author

@xibinliu xibinliu Mar 6, 2026

Choose a reason for hiding this comment

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

@SurbhiJainUSC we need to discuss what tags do we want to add to the images before and after testing.

Current:

  • before tests: ${{ github.run_id }}
  • after tests: "latest", and "<image_date>"

@xibinliu xibinliu force-pushed the xibin/ci2 branch 2 times, most recently from 3b2089d to 8ca7650 Compare March 7, 2026 00:01
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@xibinliu xibinliu force-pushed the xibin/ci2 branch 2 times, most recently from be50e5b to 71076a6 Compare March 12, 2026 18:42
Comment thread .github/workflows/test_and_tag_docker_image.yml Outdated
Comment thread .github/workflows/UploadDockerImages.yml Outdated
Comment thread .github/workflows/build_and_push_docker_image.yml
@xibinliu xibinliu force-pushed the xibin/ci branch 3 times, most recently from ea0f4ee to 5563359 Compare March 19, 2026 22:04
@xibinliu xibinliu force-pushed the xibin/ci2 branch 5 times, most recently from 2def6c9 to dbe5b32 Compare March 21, 2026 00:04
Comment thread .github/workflows/UploadDockerImages.yml Outdated
Comment thread .github/workflows/UploadDockerImages.yml Outdated
Comment thread .github/workflows/build_and_push_docker_image.yml Outdated
@xibinliu xibinliu force-pushed the xibin/ci branch 10 times, most recently from df3c1cf to 77686b6 Compare April 1, 2026 17:52
@xibinliu xibinliu force-pushed the xibin/ci2 branch 8 times, most recently from 984e151 to 5e7770b Compare April 2, 2026 15:51
Base automatically changed from xibin/ci to main April 2, 2026 17:12
@xibinliu xibinliu force-pushed the xibin/ci2 branch 6 times, most recently from 845fce9 to b74e21b Compare April 3, 2026 21:14
- images will only be tagged to "latest" when tests pass
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