-
Notifications
You must be signed in to change notification settings - Fork 206
refactor: kv cache manager repo #570
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.
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
Changes from all commits
1c15fd2
2683a46
2facce6
5755ebd
824c89b
58f6a4b
b89abce
a26d4b1
2e43da1
4e44531
3a9a6d9
9c820cf
4edcd32
2a5a77e
b63ef84
31eefb9
9c943e7
a50eb9d
b9fd43d
d7653bf
05ba63c
2e59a41
cd74019
7861c04
c5f12b0
886cb07
1848a6a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| # Git | ||
| .git | ||
| .gitignore | ||
|
|
||
| # Build artifacts | ||
| bin | ||
| build | ||
|
|
||
| # IDE and OS files | ||
| .idea | ||
| .vscode | ||
| *.DS_Store | ||
|
|
||
| # Local virtual environments | ||
| venv | ||
|
|
||
| # Python cache files | ||
| __pycache__ | ||
|
|
||
| # Docker files | ||
| Dockerfile | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,22 +12,27 @@ jobs: | |
| check-changes: | ||
| runs-on: ubuntu-latest | ||
| outputs: | ||
| docs: ${{ steps.filter.outputs.docs }} | ||
| src: ${{ steps.filter.outputs.src }} | ||
| steps: | ||
| - name: Checkout source | ||
| uses: actions/checkout@v6 | ||
| - uses: dorny/paths-filter@v3 | ||
| id: filter | ||
| with: | ||
| filters: | | ||
| docs: | ||
| - 'README.md' | ||
| - 'docs/**' | ||
| src: | ||
| - '**/*.go' | ||
| - '**/*.py' | ||
| lint-and-test: | ||
| needs: check-changes | ||
| if: ${{ needs.check-changes.outputs.docs == 'false' }} | ||
| if: ${{ needs.check-changes.outputs.src == 'true' }} | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Free Disk Space (Ubuntu) | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Q: can you briefly explain why this is needed and was not needed before?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Because now the disk use for dependencies is larger - and it now runs out of space. This was discussed with Greg at some point, we can increase capacity by upgrading tier, but this approach was preferred. It is temporary in all cases.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. since this is the first step, this clears up disk space of packages that are installed (and hopefully not used) before anything from inference-scheduler is actually run? |
||
| uses: jlumbroso/free-disk-space@main | ||
| with: | ||
| tool-cache: false | ||
|
|
||
| - name: Checkout source | ||
| uses: actions/checkout@v6 | ||
|
|
||
|
|
@@ -43,9 +48,6 @@ jobs: | |
| go-version: "${{ env.GO_VERSION }}" | ||
| cache-dependency-path: ./go.sum | ||
|
|
||
| - name: Install dependencies | ||
| run: sudo make install-dependencies | ||
|
|
||
| - name: Configure CGO for Python | ||
| run: | | ||
| PYTHON_INCLUDE=$(python3 -c "import sysconfig; print(sysconfig.get_path('include'))") | ||
|
|
@@ -57,13 +59,16 @@ jobs: | |
| - name: Set PKG_CONFIG_PATH | ||
| run: echo "PKG_CONFIG_PATH=/usr/lib/pkgconfig" >> $GITHUB_ENV | ||
|
|
||
| - name: go mod tidy | ||
| run: go mod tidy | ||
| - name: Install dependencies | ||
| run: | | ||
| go mod tidy | ||
| sudo -E env "PATH=$PATH" make install-dependencies | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Q: any limitation in running |
||
| sudo -E env "PATH=$PATH" make install-python-deps | ||
|
|
||
| - name: Run lint checks | ||
| uses: golangci/golangci-lint-action@v9 | ||
| with: | ||
| version: 'v2.1.6' | ||
| version: "v2.1.6" | ||
| args: "--config=./.golangci.yml" | ||
| env: | ||
| CGO_ENABLED: ${{ env.CGO_ENABLED }} | ||
|
|
@@ -74,10 +79,8 @@ jobs: | |
|
|
||
| - name: Run make build | ||
| shell: bash | ||
| run: | | ||
| make build | ||
| run: make build | ||
|
|
||
| - name: Run make test | ||
| shell: bash | ||
| run: | | ||
| make test | ||
| run: make test | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -11,6 +11,8 @@ | |
| main | ||
| bin/ | ||
|
|
||
| *debug_bin* | ||
|
|
||
| # Test binary, built with `go test -c` | ||
| *.test | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: newline missing