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

Write documentation about git lfs not properly pulling on self-hosted CI runners and tell all submodule owners #2723

Closed
tt-rkim opened this issue Sep 18, 2023 · 0 comments · Fixed by #16722
Closed

Write documentation about git lfs not properly pulling on self-hosted CI runners and tell all submodule owners #2723

tt-rkim opened this issue Sep 18, 2023 · 0 comments · Fixed by #16722
Assignees
@tt-rkim
Labels
docs Improvements or additions to documentation infra-ci infrastructure and/or CI changes P3

Comments

@tt-rkim
Copy link
Collaborator

tt-rkim commented Sep 18, 2023

No description provided.

@tt-rkim tt-rkim added the infra-ci infrastructure and/or CI changes label Sep 18, 2023
@tt-rkim tt-rkim self-assigned this Sep 18, 2023
@tt-rkim tt-rkim mentioned this issue Sep 20, 2023
Closed
@tt-rkim tt-rkim added docs Improvements or additions to documentation P3 labels Dec 20, 2023
@blozano-tt blozano-tt mentioned this issue Jan 14, 2025
Merged
6 tasks
TT-billteng added a commit that referenced this issue Jan 15, 2025
@TT-billteng
remove references to LFS (#16722)
76ce6c7 
### Tickets
Closes #2723
Closes #4435

### Problem description
We get charged $$$$ for LFS usage, LFS clone has introduced intermittent
failures in CI, and it adds complexity to our install process.

### What's changed
Goodbye LFS

### Checklist
- [x] Post commit CI passes
- [ ] Blackhole Post commit (if applicable)
- [ ] Model regression CI testing passes (if applicable)
- [ ] Device performance regression CI testing passes (if applicable)
- [ ] **(For models and ops writers)** Full [new
models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tt-rkim tt-rkim

Labels
docs Improvements or additions to documentation infra-ci infrastructure and/or CI changes P3
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

remove references to LFS tenstorrent/tt-metal
1 participant
@tt-rkim