Skip to content

Commit 2ac4906

Browse files
authored
chore: Consolidate wheels/source builds & artifacts for release (feast-dev#5119)
consolidate wheels/source builds & artifacts for release Signed-off-by: Tommy Hughes <[email protected]>
1 parent 24d5db6 commit 2ac4906

File tree

1 file changed

+6
-50
lines changed

1 file changed

+6
-50
lines changed

.github/workflows/build_wheels.yml

+6-50
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ on:
2929

3030
jobs:
3131
build-python-wheel:
32-
name: Build wheels
32+
name: Build wheels and source
3333
runs-on: ubuntu-latest
3434
steps:
3535
- uses: actions/checkout@v4
@@ -50,66 +50,26 @@ jobs:
5050
with:
5151
custom_version: ${{ github.event.inputs.custom_version }}
5252
token: ${{ github.event.inputs.token }}
53-
- name: Build wheels
53+
- name: Checkout version and install dependencies
5454
env:
5555
VERSION: ${{ steps.get-version.outputs.release_version }}
5656
PYPI_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
5757
run: |
5858
git fetch --tags
5959
git checkout ${VERSION}
6060
python -m pip install build
61-
python -m build --wheel --outdir wheelhouse/
61+
- name: Build feast
62+
run: python -m build
6263
- uses: actions/upload-artifact@v4
6364
with:
6465
name: python-wheels
65-
path: ./wheelhouse/*.whl
66-
67-
build-source-distribution:
68-
name: Build source distribution
69-
runs-on: macos-13
70-
steps:
71-
- uses: actions/checkout@v4
72-
- name: Setup Python
73-
id: setup-python
74-
uses: actions/setup-python@v5
75-
with:
76-
python-version: "3.11"
77-
architecture: x64
78-
- name: Setup Node
79-
uses: actions/setup-node@v3
80-
with:
81-
node-version-file: './ui/.nvmrc'
82-
registry-url: 'https://registry.npmjs.org'
83-
- id: get-version
84-
uses: ./.github/actions/get-semantic-release-version
85-
with:
86-
custom_version: ${{ github.event.inputs.custom_version }}
87-
token: ${{ github.event.inputs.token }}
88-
- name: Build and install dependencies
89-
env:
90-
VERSION: ${{ steps.get-version.outputs.release_version }}
91-
# There's a `git restore` in here because `make install-go-ci-dependencies` is actually messing up go.mod & go.sum.
92-
run: |
93-
git fetch --tags
94-
git checkout ${VERSION}
95-
pip install -U pip setuptools wheel twine
96-
make build-ui
97-
git status
98-
git restore go.mod go.sum
99-
git restore sdk/python/feast/ui/yarn.lock
100-
- name: Build
101-
run: |
102-
python3 setup.py sdist
103-
- uses: actions/upload-artifact@v4
104-
with:
105-
name: source-distribution
10666
path: dist/*
10767

10868
# We add this step so the docker images can be built as part of the pre-release verification steps.
10969
build-docker-images:
11070
name: Build Docker images
11171
runs-on: ubuntu-latest
112-
needs: [ build-python-wheel, build-source-distribution ]
72+
needs: [ build-python-wheel ]
11373
strategy:
11474
matrix:
11575
component: [ feature-server-dev, feature-server-java, feature-transformation-server, feast-operator ]
@@ -137,7 +97,7 @@ jobs:
13797
verify-python-wheels:
13898
name: Verify Python wheels
13999
runs-on: ${{ matrix.os }}
140-
needs: [ build-python-wheel, build-source-distribution ]
100+
needs: [ build-python-wheel ]
141101
strategy:
142102
matrix:
143103
os: [ ubuntu-latest, macos-13 ]
@@ -168,10 +128,6 @@ jobs:
168128
with:
169129
name: python-wheels
170130
path: dist
171-
- uses: actions/[email protected]
172-
with:
173-
name: source-distribution
174-
path: dist
175131
- name: Install OS X dependencies
176132
if: matrix.os == 'macos-13'
177133
run: brew install coreutils

0 commit comments

Comments
 (0)