Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
53 changes: 53 additions & 0 deletions .github/.testcoverage.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,53 @@
# (mandatory)
# Path to coverage profile file (output of `go test -coverprofile` command).
#
# For cases where there are many coverage profiles, such as when running
# unit tests and integration tests separately, you can combine all those
# profiles into one. In this case, the profile should have a comma-separated list
# of profile files, e.g., 'cover_unit.out,cover_integration.out'.
profile: coverage.out

# Holds coverage thresholds percentages, values should be in range [0-100].
threshold:
# (optional; default 0)
# Minimum coverage percentage required for individual files.
# file: 70

# (optional; default 0)
# Minimum coverage percentage required for each package.
# package: 80

# (optional; default 0)
# Minimum overall project coverage percentage required.
total: 80

# Holds regexp rules which will override thresholds for matched files or packages
# using their paths.
#
# First rule from this list that matches file or package is going to apply
# new threshold to it. If project has multiple rules that match same path,
# override rules should be listed in order from specific to more general rules.
# override:
# # Increase coverage threshold to 100% for `foo` package
# # (default is 80, as configured above in this example).
# - path: ^pkg/lib/foo$
# threshold: 100

# Holds regexp rules which will exclude matched files or packages
# from coverage statistics.
exclude:
# Exclude files or packages matching their paths
paths:
- zz_generated.deepcopy.go
- openapi_generated.go
- testing
- pkg\\/client

# File name of go-test-coverage breakdown file, which can be used to
# analyze coverage difference.
breakdown-file-name: ''

diff:
# File name of go-test-coverage breakdown file which will be used to
# report coverage difference.
base-breakdown-file-name: ''
2 changes: 1 addition & 1 deletion .github/workflows/automated-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
# Bump Versions
make bump-version
./hack/generate-install.sh $RELEASE_TAG
make poetry-lock
make uv-lock

Comment thread
mholder6 marked this conversation as resolved.
# Update Release Branch and Push Tag
git diff
Expand Down
47 changes: 25 additions & 22 deletions .github/workflows/e2e-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -314,8 +314,8 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

Comment thread
mholder6 marked this conversation as resolved.
- name: Install KServe
run: |
Expand Down Expand Up @@ -394,8 +394,8 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
Expand Down Expand Up @@ -468,11 +468,13 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
# export VIRTUAL_ENV=python/kserve/.venv
# source $VIRTUAL_ENV/bin/activate
./test/scripts/gh-actions/setup-kserve.sh

kubectl get pods -n kserve
Expand Down Expand Up @@ -553,8 +555,8 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
Expand Down Expand Up @@ -622,8 +624,9 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
./test/scripts/gh-actions/setup-kserve.sh
Expand Down Expand Up @@ -672,8 +675,8 @@ jobs:
- name: Download base images
uses: ./.github/actions/base-download

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install Kserve from helm
run: |
Expand Down Expand Up @@ -761,8 +764,8 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
Expand Down Expand Up @@ -868,8 +871,8 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
Expand Down Expand Up @@ -936,8 +939,8 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
Expand Down Expand Up @@ -995,8 +998,8 @@ jobs:
with:
directory: ./tmp

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
Expand Down Expand Up @@ -1098,8 +1101,8 @@ jobs:
minikube ssh -n minikube-m02 -- sudo mkdir -p -m=777 /models
minikube ssh -n minikube-m03 -- sudo mkdir -p -m=777 /models

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

- name: Install KServe
run: |
Expand Down
43 changes: 26 additions & 17 deletions .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,11 @@ on:
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true

permissions:
contents: write
pull-requests: write

jobs:
test:
name: Test
Expand Down Expand Up @@ -151,33 +153,40 @@ jobs:
- name: post coverage report
if: github.event_name == 'pull_request'
uses: thollander/actions-comment-pull-request@v3
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
comment-tag: coverage-report
pr-number: ${{ github.event.pull_request.number }}
message: |
## 📊 Go Test Coverage Report
- name: post coverage report
Comment thread
mholder6 marked this conversation as resolved.
# this has evalated permission to post back the coverage, only restricted to this step.
if: github.event_name == 'pull_request_target'
uses: thollander/actions-comment-pull-request@v3
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
comment-tag: coverage-report
pr-number: ${{ github.event.pull_request.number }}
message: |
## 📊 Go Test Coverage Report

${{
steps.current-coverage.outputs.coverage > steps.master-coverage.outputs.coverage
&& '✅ **Overall code coverage increased.**'
|| steps.current-coverage.outputs.coverage < steps.master-coverage.outputs.coverage
&& '❌ **Overall code coverage decreased.**'
|| 'ℹ️ **Overall code coverage unchanged.**'
}}
**🔍 Coverage Summary**
- **Pull Request Coverage:** `${{ steps.current-coverage.outputs.coverage }}%`
- **Main Branch Coverage:** `${{ steps.master-coverage.outputs.coverage }}%`

**🔍 Coverage Summary**
- **Pull Request Coverage:** `${{ steps.current-coverage.outputs.coverage }}%`
- **Main Branch Coverage:** `${{ steps.master-coverage.outputs.coverage }}%`
<details>
<summary>📄 Click to expand full coverage breakdown</summary>

<details>
<summary>📄 Click to expand full coverage breakdown</summary>
```
${{ steps.full_coverage_report.outputs.report }}
```

```
${{ steps.full_coverage_report.outputs.report }}
```
</details>
- name: Rename and upload master coverage
if: github.ref_name == 'master'
run: mv coverage.out master-coverage.out

</details>
</details>
- name: Rename and upload master coverage
Comment thread
mholder6 marked this conversation as resolved.
if: github.ref_name == 'master'
run: mv coverage.out master-coverage.out
Expand All @@ -188,4 +197,4 @@ jobs:
with:
name: master-coverage.out
path: master-coverage.out
if-no-files-found: error
if-no-files-found: error
14 changes: 8 additions & 6 deletions .github/workflows/python-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,19 +23,21 @@ jobs:
with:
python-version: "3.11"

- name: Install Poetry and version plugin
run: ./test/scripts/gh-actions/setup-poetry.sh
- name: Install UV
run: ./test/scripts/gh-actions/setup-uv.sh

Comment thread
mholder6 marked this conversation as resolved.
- name: KServe - Build and publish
env:
POETRY_PYPI_TOKEN_PYPI: ${{ secrets.PYPI_TOKEN }}
UV_PUBLISH_TOKEN: ${{ secrets.PYPI_TOKEN }}
run: |
cd python/kserve
poetry publish --build
uv build
uv publish

- name: KServe Storage - Build and publish
env:
POETRY_PYPI_TOKEN_PYPI: ${{ secrets.PYPI_TOKEN }}
UV_PUBLISH_TOKEN: ${{ secrets.PYPI_TOKEN }}
run: |
cd python/storage
poetry publish --build
uv build
uv publish
Loading