Skip to content

Commit

Permalink
remove TE build from all workflows and use JAX build instead
Browse files Browse the repository at this point in the history
  • Loading branch information
yhtang committed Jul 19, 2023
1 parent 503ad91 commit 4514fca
Show file tree
Hide file tree
Showing 8 changed files with 10 additions and 180 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/_build_pax.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
type: string
description: 'Base docker image that provides JAX'
required: false
default: ghcr.io/nvidia/jax-te:latest
default: ghcr.io/nvidia/jax:latest
BUILD_DATE:
type: string
description: "Build date in YYYY-MM-DD format"
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/_build_t5x.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
type: string
description: 'Base docker image that provides JAX'
required: false
default: ghcr.io/nvidia/jax-te:latest
default: ghcr.io/nvidia/jax:latest
BUILD_DATE:
type: string
description: "Build date in YYYY-MM-DD format"
Expand Down
90 changes: 0 additions & 90 deletions .github/workflows/_build_te.yaml

This file was deleted.

2 changes: 1 addition & 1 deletion .github/workflows/_test_te.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ on:
type: string
description: 'JAX-TE image build by NVIDIA/JAX-Toolbox'
required: true
default: 'ghcr.io/nvidia/jax-te:latest'
default: 'ghcr.io/nvidia/jax:latest'
outputs:
UNIT_TEST_ARTIFACT_NAME:
description: 'Name of the unit test artifact for downstream workflows'
Expand Down
17 changes: 3 additions & 14 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -119,16 +119,6 @@ jobs:
REF_XLA: ${{ needs.metadata.outputs.REF_XLA }}
secrets: inherit

build-te:
needs: [metadata, build-jax]
uses: ./.github/workflows/_build_te.yaml
with:
BUILD_DATE: ${{ needs.metadata.outputs.BUILD_DATE }}
BASE_IMAGE: ${{ needs.build-jax.outputs.DOCKER_TAGS }}
REPO_TE: ${{ needs.metadata.outputs.REPO_TE }}
REF_TE: ${{ needs.metadata.outputs.REF_TE }}
secrets: inherit

build-t5x:
needs: [metadata, build-jax]
uses: ./.github/workflows/_build_t5x.yaml
Expand Down Expand Up @@ -170,7 +160,7 @@ jobs:
secrets: inherit

build-summary:
needs: [build-base, build-jax, build-te, build-t5x, build-pax, build-rosetta-t5x, build-rosetta-pax]
needs: [build-base, build-jax, build-t5x, build-pax, build-rosetta-t5x, build-rosetta-pax]
runs-on: ubuntu-22.04
steps:
- name: Generate job summary for container build
Expand All @@ -183,7 +173,6 @@ jobs:
| ------------ | -------------------------------------------------- |
| Base | ${{ needs.build-base.outputs.DOCKER_TAGS }} |
| JAX | ${{ needs.build-jax.outputs.DOCKER_TAGS }} |
| JAX-TE | ${{ needs.build-te.outputs.DOCKER_TAGS }} |
| T5X | ${{ needs.build-t5x.outputs.DOCKER_TAGS }} |
| PAX | ${{ needs.build-pax.outputs.DOCKER_TAGS }} |
| ROSETTA(t5x) | ${{ needs.build-rosetta-t5x.outputs.DOCKER_TAGS }} |
Expand All @@ -198,10 +187,10 @@ jobs:
secrets: inherit

test-te:
needs: build-te
needs: [build-jax, test-jax]
uses: ./.github/workflows/_test_te.yaml
with:
JAX_TE_IMAGE: ${{ needs.build-te.outputs.DOCKER_TAGS }}
JAX_TE_IMAGE: ${{ needs.build-jax.outputs.DOCKER_TAGS }}
secrets: inherit

test-t5x:
Expand Down
64 changes: 0 additions & 64 deletions .github/workflows/nightly-te-build.yaml

This file was deleted.

6 changes: 3 additions & 3 deletions .github/workflows/nightly-te-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: Nightly Transformer Engine test

on:
workflow_run:
workflows: [Nightly Transformer Engine build]
workflows: [Nightly JAX build]
types: [completed]
branches: [main]
workflow_dispatch:
Expand All @@ -11,7 +11,7 @@ on:
type: string
description: 'JAX-TE image build by NVIDIA/JAX-Toolbox'
required: true
default: 'ghcr.io/nvidia/jax-te:latest'
default: 'ghcr.io/nvidia/jax:latest'
PUBLISH:
type: boolean
description: Update status badge?
Expand All @@ -24,7 +24,7 @@ permissions:
packages: write # to upload container

env:
DEFAULT_JAX_TE_IMAGE: 'ghcr.io/nvidia/jax-te:latest'
DEFAULT_JAX_TE_IMAGE: 'ghcr.io/nvidia/jax:latest'

jobs:

Expand Down
7 changes: 1 addition & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,9 @@
| Image | Build | Test |
| ---------------------------------------------------- | ------------------------------------------ | -------------------------------------- |
| [![container-badge-base]][container-link-base] | [![build-badge-base]][workflow-base] | n/a |
| [![container-badge-jax]][container-link-jax] | [![build-badge-jax]][workflow-jax] | [![test-badge-jax]][workflow-jax-unit] |
| [![container-badge-jax]][container-link-jax] | [![build-badge-jax]][workflow-jax] | [![test-badge-jax]][workflow-jax-unit] <br> [![unit-test-badge-te]][workflow-te-test] <br> [![integration-test-badge-te]][workflow-te-test] |
| [![container-badge-t5x]][container-link-t5x] | [![build-badge-t5x]][workflow-t5x] | [![test-badge-t5x]][workflow-t5x-perf] |
| [![container-badge-pax]][container-link-pax] | [![build-badge-pax]][workflow-pax] | [![test-badge-pax]][workflow-pax-perf] |
| [![container-badge-te]][container-link-te] | [![build-badge-te]][workflow-te] | [![unit-test-badge-te]][workflow-te-test] <br> [![integration-test-badge-te]][workflow-te-test] |
| [![container-badge-rosetta-t5x]][container-link-rosetta-t5x] | [![build-badge-rosetta-t5x]][workflow-rosetta-t5x] | [![test-badge-rosetta-t5x]][workflow-rosetta-t5x] |
| [![container-badge-rosetta-pax]][container-link-rosetta-pax] | [![build-badge-rosetta-pax]][workflow-rosetta-pax] | [![test-badge-rosetta-pax]][workflow-rosetta-pax] |

Expand All @@ -16,13 +15,11 @@
[container-badge-pax]: https://img.shields.io/static/v1?label=&message=PAX&color=gray&logo=docker
[container-badge-rosetta-t5x]: https://img.shields.io/static/v1?label=&message=ROSETTA(T5X)&color=gray&logo=docker
[container-badge-rosetta-pax]: https://img.shields.io/static/v1?label=&message=ROSETTA(PAX)&color=gray&logo=docker
[container-badge-te]: https://img.shields.io/static/v1?label=&message=TE&color=gray&logo=docker

[container-link-base]: https://github.com/NVIDIA/JAX-Toolbox/pkgs/container/jax-toolbox
[container-link-jax]: https://github.com/NVIDIA/JAX-Toolbox/pkgs/container/jax
[container-link-t5x]: https://github.com/NVIDIA/JAX-Toolbox/pkgs/container/t5x
[container-link-pax]: https://github.com/NVIDIA/JAX-Toolbox/pkgs/container/pax
[container-link-te]: https://github.com/NVIDIA/JAX-Toolbox/pkgs/container/jax-te
[container-link-rosetta-t5x]: https://github.com/NVIDIA/JAX-Toolbox/pkgs/container/rosetta-t5x
[container-link-rosetta-pax]: https://github.com/NVIDIA/JAX-Toolbox/pkgs/container/rosetta-pax

Expand All @@ -32,15 +29,13 @@
[build-badge-pax]: https://img.shields.io/github/actions/workflow/status/NVIDIA/JAX-Toolbox/nightly-pax-build.yaml?branch=main&label=nightly&logo=github-actions&logoColor=dddddd
[build-badge-rosetta-t5x]: https://img.shields.io/endpoint?url=https%3A%2F%2Fgist.githubusercontent.com%2Fnvjax%2F913c2af68649fe568e9711c2dabb23ae%2Fraw%2Frosetta-t5x-build-status.json&logo=github-actions&logoColor=dddddd
[build-badge-rosetta-pax]: https://img.shields.io/endpoint?url=https%3A%2F%2Fgist.githubusercontent.com%2Fnvjax%2F913c2af68649fe568e9711c2dabb23ae%2Fraw%2Frosetta-pax-build-status.json&logo=github-actions&logoColor=dddddd
[build-badge-te]: https://img.shields.io/github/actions/workflow/status/NVIDIA/JAX-Toolbox/nightly-te-build.yaml?branch=main&label=nightly&logo=github-actions&logoColor=dddddd

[workflow-base]: https://github.com/NVIDIA/JAX-Toolbox/actions/workflows/weekly-base-build.yaml
[workflow-jax]: https://github.com/NVIDIA/JAX-Toolbox/actions/workflows/nightly-jax-build.yaml
[workflow-t5x]: https://github.com/NVIDIA/JAX-Toolbox/actions/workflows/nightly-t5x-build.yaml
[workflow-pax]: https://github.com/NVIDIA/JAX-Toolbox/actions/workflows/nightly-pax-build.yaml
[workflow-rosetta-t5x]: https://github.com/NVIDIA/JAX-Toolbox/actions/workflows/nightly-rosetta-t5x-build.yaml
[workflow-rosetta-pax]: https://github.com/NVIDIA/JAX-Toolbox/actions/workflows/nightly-rosetta-pax-build.yaml
[workflow-te]: https://github.com/NVIDIA/JAX-Toolbox/actions/workflows/nightly-te-build.yaml

[test-badge-jax]: https://img.shields.io/endpoint?url=https%3A%2F%2Fgist.githubusercontent.com%2Fnvjax%2F913c2af68649fe568e9711c2dabb23ae%2Fraw%2Fjax-unit-test-status.json&logo=nvidia
[test-badge-t5x]: https://img.shields.io/github/actions/workflow/status/NVIDIA/JAX-Toolbox/nightly-t5x-test-mgmn.yaml?branch=main&label=A100%20MGMN&logo=nvidia
Expand Down

0 comments on commit 4514fca

Please sign in to comment.