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
test 
Signed-off-by: Kevin Zhang <kzhang@tecton.ai>
  • Loading branch information
@kevjumba
kevjumba committed Jun 30, 2022
commit c24fee015edf43369c71a4d3a1ccf90a362f7c0c
7 changes: 2 additions & 5 deletions .github/workflows/build_wheels.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- 'v*.*.*'
workflow_call:


jobs:
get-version:
runs-on: ubuntu-latest
Expand Down Expand Up @@ -83,8 +82,8 @@ jobs:
make install-protoc-dependencies
make install-go-proto-dependencies
make install-go-ci-dependencies
git status
git restore go.mod go.sum
# git status
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 are we removing this? And if we are it should be in a different PR

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.

Oh Let me change that.

# git restore go.mod go.sum
CIBW_BEFORE_TEST: "cd {project} && git status"
# py3.10 on MacOS does not work with Go so we have to install separately. Issue is tracked here: https://github.com/feast/dev/feast/issues/2881.
- name: Build py310 specific wheels for macos
Expand Down Expand Up @@ -146,8 +145,6 @@ jobs:
matrix:
os: [ubuntu-latest, macos-10.15 ]
python-version: [ "3.7", "3.8", "3.9", "3.10"]
# os: [macos-10.15 ]
# python-version: [ "3.10"]
from-source: [ True, False ]
env:
# this script is for testing servers
Expand Down