Skip to content

fix: Fix grpc and update protobuf #2894

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

Merged
feast-ci-bot merged 31 commits into from
Jun 30, 2022
Merged

fix: Fix grpc and update protobuf #2894

Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
dd52cbc
Test
kevjumba Jun 29, 2022
2f6fbc2
Fix
kevjumba Jun 29, 2022
9fa6cd5
Fix workflow
kevjumba Jun 29, 2022
b7b2c11
Fix workflow
kevjumba Jun 29, 2022
7c0e972
Test
kevjumba Jun 29, 2022
ad47069
Fix
kevjumba Jun 29, 2022
160c18e
Fix
kevjumba Jun 29, 2022
f782488
Fix
kevjumba Jun 29, 2022
985d672
test
kevjumba Jun 29, 2022
2fa19af
Fix
kevjumba Jun 29, 2022
fa91bea
Fix
kevjumba Jun 29, 2022
8edd89d
Fix
kevjumba Jun 29, 2022
d5f2000
Fix
kevjumba Jun 29, 2022
c24fee0
test
kevjumba Jun 29, 2022
0c7cb35
Fix
kevjumba Jun 29, 2022
9027a59
Fix protobuf pin version and add path variable
kevjumba Jun 29, 2022
1edb15c
Revert setup
kevjumba Jun 29, 2022
25b090d
Update 3.8
kevjumba Jun 29, 2022
61e1aec
Fix
kevjumba Jun 29, 2022
f75d9ab
Update 3.10
kevjumba Jun 29, 2022
3b39aaa
update
kevjumba Jun 29, 2022
278cda7
Fix
kevjumba Jun 29, 2022
3d63de0
Fix
kevjumba Jun 29, 2022
698e335
Fix
kevjumba Jun 29, 2022
2b8cbe9
Fix test
kevjumba Jun 29, 2022
4f5ec88
Fix
kevjumba Jun 30, 2022
c7c9ff1
Update go with grpc 1.47
kevjumba Jun 30, 2022
ca1fd4b
Fix
kevjumba Jun 30, 2022
d875c61
Fix
kevjumba Jun 30, 2022
c6d2ef7
Fix
kevjumba Jun 30, 2022
7a7a473
Fix
kevjumba Jun 30, 2022
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
Prev Previous commit
Next Next commit
Fix 
Signed-off-by: Kevin Zhang <kzhang@tecton.ai>
  • Loading branch information
@kevjumba
kevjumba committed Jun 30, 2022
commit ad4706948fa678ed8598c7c00d53dc01103d4193
6 changes: 2 additions & 4 deletions .github/workflows/build_wheels.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,8 @@ jobs:
verify-python-wheels:
runs-on: ${{ matrix.os }}
needs: [build-python-wheel, build-source-distribution, get-version]
Copy link
Copy Markdown
Member

@achals achals Jun 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this change needed?

Copy link
Copy Markdown
Collaborator Author

@kevjumba kevjumba Jun 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to give the job access the release version which means we must add it to needs

env:
VERSION_WITHOUT_PREFIX: ${{ needs.get-version.outputs.version_without_prefix }}
strategy:
matrix:
# os: [ubuntu-latest, macos-10.15 ]
Expand Down Expand Up @@ -202,8 +204,6 @@ jobs:
run: brew install coreutils
# Validate that the feast version installed is not development and is the correct version of the tag we ran it off of.
- name: Validate Feast Version
env:
VERSION_WITHOUT_PREFIX: ${{ needs.get-version.outputs.version_without_prefix }}
run: |
VERSION_REGEX='[0-9]+\.[0-9]+\.[0-9]+'
OUTPUT_REGEX='^Feast SDK Version: "$VERSION_REGEX"$'
Expand All @@ -217,8 +217,6 @@ jobs:
exit 1
fi
- name: Smoke test
env:
VERSION_WITHOUT_PREFIX: ${{ needs.get-version.outputs.version_without_prefix }}
run: |
feast init test_repo
cd test_repo/
Expand Down