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.

Sign up for GitHub

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

Jump to bottom

Use docker containers in workflows #8082

Closed
Tracked by #8418
ttmchiou opened this issue May 2, 2024 · 0 comments
Closed
Tracked by #8418

Use docker containers in workflows #8082

ttmchiou opened this issue May 2, 2024 · 0 comments
Assignees
@dimitri-tenstorrent @ttmchiou
Labels
docker feature infra-ci infrastructure and/or CI changes P1

Comments

@ttmchiou
Copy link
Contributor

ttmchiou commented May 2, 2024

No description provided.

@ttmchiou ttmchiou added infra-ci infrastructure and/or CI changes feature-request External feature request labels May 2, 2024
@ttmchiou ttmchiou self-assigned this May 2, 2024
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Test commit to build docker container on builder BMs
c115c79
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Test commit to build docker container on builder BMs
4d087bb
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Remove docker build push action GH action
0e4598d
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: fix path
fdd8fb4
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: use correct shell script
8999037
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: DROP! remove -it for test
759c73a
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Drop remove loading --device
00345b1
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Drop remove loading --device
cd9279c
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Added config env to docker
5bed359
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Preserve dockerfile metadata by using multistage build
1ffd5b8
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Test docker buildx steps and uploading docker image
ed08b3d
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Test docker buildx steps and uploading docker image
9d355b5
ttmchiou added a commit that referenced this issue May 3, 2024
@ttmchiou
#8082: Test docker buildx steps and uploading docker image
a2a7111
@jliangTT jliangTT added feature and removed feature-request External feature request labels May 3, 2024
ttmchiou added a commit that referenced this issue May 6, 2024
@ttmchiou
#8082: DROP! remove -it for test
5f10218
ttmchiou added a commit that referenced this issue May 6, 2024
@ttmchiou
#8082: Drop remove loading --device
924bd21
ttmchiou added a commit that referenced this issue May 6, 2024
@ttmchiou
#8082: Added config env to docker
2cdc037
ttmchiou added a commit that referenced this issue May 6, 2024
@ttmchiou
#8082: Preserve dockerfile metadata by using multistage build
12c0774
ttmchiou added a commit that referenced this issue May 6, 2024
@ttmchiou
#8082: Test docker buildx steps and uploading docker image
7b2742d
ttmchiou added a commit that referenced this issue May 6, 2024
@ttmchiou
#8082: install cargo from apt instead
8983115
ttmchiou added a commit that referenced this issue May 6, 2024
@ttmchiou
#8082: try multi stage docker
c2e0433
ttmchiou added a commit that referenced this issue May 7, 2024
@ttmchiou
#8082: DROP! remove -it for test
7930ef7
ttmchiou added a commit that referenced this issue May 7, 2024
@ttmchiou
#8082: Drop remove loading --device
cf57780
ttmchiou added a commit that referenced this issue May 7, 2024
@ttmchiou
#8082: Added config env to docker
e23c529
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Fix the issue with the docker image not being pulled and the …
615685d 
…runners using an old image

 Use the branch tagging strategy for the docker images
 Change the branch name to main before the release
 Remove Python user as that interferes with IRD
 Undo most changes in Docker container
 Change the folder since we are not setting the workdir anymore
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: PR review
0b32a7c
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Run tests on an isolated machine
24038eb
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Add the most important line in the PR
af29c82
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Change all the jobs that use the wheel to now run on the diff…
55fe836 
…erent tag
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Removed incorrectly - we do not want to rebuild all the time.
bbe5c45
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Do not limit the machines to the vms assigned to the test
6e8cf1c
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Build wheels, docker images with wheels, and execute Python t…
56e6c1e 
…ests in All Post Commit against the wheel

 #8082: Add Graphviz and make Docker branch dependent upon requirements
 #8082: Allow the wheel to rely on the precompiled C++ packages
 #11257 as it is blocking the PR
 #8082: Move the ttnn cpp unit-tests to post-commit cpp tests
 #11132: Remove the yml file
 #8082: Move the Docker Image plus Wheel to the future release branch
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Do nothing with the hosted folder! Copy the source
53b71e6
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Split the combined C++ and Python tests into two separate ones
62a8c6a
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Delete the release image file from this PR
cb9a57c
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Fix the issue with the docker image not being pulled and the …
a48d981 
…runners using an old image

 Use the branch tagging strategy for the docker images
 Change the branch name to main before the release
 Remove Python user as that interferes with IRD
 Undo most changes in Docker container
 Change the folder since we are not setting the workdir anymore
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: PR review
e10b89a
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Run tests on an isolated machine
6c20a69
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Add the most important line in the PR
4b13613
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Change all the jobs that use the wheel to now run on the diff…
fca5fde 
…erent tag
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Removed incorrectly - we do not want to rebuild all the time.
4ee7cf6
dimitri-tenstorrent added a commit that referenced this issue Sep 6, 2024
@dimitri-tenstorrent
#8082: Do not limit the machines to the vms assigned to the test
79c5ad3
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Build wheels, docker images with wheels, and execute Python t…
082ea2c 
…ests in All Post Commit against the wheel

 #8082: Add Graphviz and make Docker branch dependent upon requirements
 #8082: Allow the wheel to rely on the precompiled C++ packages
 #11257 as it is blocking the PR
 #8082: Move the ttnn cpp unit-tests to post-commit cpp tests
 #11132: Remove the yml file
 #8082: Move the Docker Image plus Wheel to the future release branch
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Do nothing with the hosted folder! Copy the source
6c60c89
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Split the combined C++ and Python tests into two separate ones
62b12af
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Delete the release image file from this PR
054f1e7
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Fix the issue with the docker image not being pulled and the …
dc874c3 
…runners using an old image

 Use the branch tagging strategy for the docker images
 Change the branch name to main before the release
 Remove Python user as that interferes with IRD
 Undo most changes in Docker container
 Change the folder since we are not setting the workdir anymore
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: PR review
debaeb4
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Run tests on an isolated machine
a0e2dc7
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Add the most important line in the PR
f3ae27d
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Change all the jobs that use the wheel to now run on the diff…
f69bfee 
…erent tag
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Removed incorrectly - we do not want to rebuild all the time.
ee38aae
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent @TT-billteng
#8082: Do not limit the machines to the vms assigned to the test
cccf07d
TT-billteng pushed a commit that referenced this issue Sep 7, 2024
@dimitri-tenstorrent
#11132 Run Post-Commit Python Tests against wheels: Try #3 (#12325)
48b4555 
* #8082: Build wheels, docker images with wheels, and execute Python tests in All Post Commit against the wheel
 #8082: Add Graphviz and make Docker branch dependent upon requirements
 #8082: Allow the wheel to rely on the precompiled C++ packages
 #11257 as it is blocking the PR
 #8082: Move the ttnn cpp unit-tests to post-commit cpp tests
 #11132: Remove the yml file
 #8082: Move the Docker Image plus Wheel to the future release branch

* #8082: Do nothing with the hosted folder! Copy the source

* #8082: Split the combined C++ and Python tests into two separate ones

* #8082: Delete the release image file from this PR

* #8082: Fix the issue with the docker image not being pulled and the runners using an old image
 Use the branch tagging strategy for the docker images
 Change the branch name to main before the release
 Remove Python user as that interferes with IRD
 Undo most changes in Docker container
 Change the folder since we are not setting the workdir anymore

* #8082: PR review

* #8082: Run tests on an isolated machine

* #8082: Add the most important line in the PR

* #8082: Change all the jobs that use the wheel to now run on the different tag

* #8082: Removed incorrectly - we do not want to rebuild all the time.

* #8082: Do not limit the machines to the vms assigned to the test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dimitri-tenstorrent dimitri-tenstorrent

@ttmchiou ttmchiou

Labels
docker feature infra-ci infrastructure and/or CI changes P1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Mchiou/8082 use docker containers in workflows tenstorrent/tt-metal
4 participants
@dimitri-tenstorrent @TT-billteng @jliangTT @ttmchiou