Skip to content

docs: Example to install feast on local computer using Kind #4528

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

Jump to bottom
Merged
franciscojavierarceo merged 7 commits into from
Sep 23, 2024
Merged

docs: Example to install feast on local computer using Kind #4528

franciscojavierarceo merged 7 commits into from
Sep 23, 2024

Conversation

@dmartinol
Copy link
Contributor

@dmartinol dmartinol commented Sep 17, 2024 •
edited
Loading

What this PR does / why we need it:

Provide a reference implementation of Feast project installed on local Kind cluster and validated with a remote client.

  • Prerequisites are mentioned and reference installation commands provided for MacOS.
  • A postgres template is used to initialize the project.
  • All the commands are executed from individual notebooks, which also report the output of the latest run.
  • Hints are provided to customize the provided code for a personalized experience.
  • A dedicated Job is used to apply the feature store definitions (e.g. feast apply) and materializations, overcoming issues: Remote materialization #4526 and Remote apply #4529

Note: the execution of the client application is still affected by #4392 until we release the new version

Which issue(s) this PR fixes:

Fixes #4527

@dmartinol
Simple deployment on Kind
9468258 
Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
@dmartinol
updated notebooks output
e76953f 
Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
@dmartinol
Copy link
Contributor Author

dmartinol commented Sep 17, 2024

@tokoko @franciscojavierarceo

@dmartinol dmartinol mentioned this pull request Sep 17, 2024
Closed
@lokeshrangineni
Copy link
Contributor

lokeshrangineni commented Sep 17, 2024 •
edited
Loading

A brief readme.md description about the example at https://github.com/dmartinol/feast/tree/install_kind/examples/kind-quickstart will help before looking at all the jupyter notebooks and figure it out. something similar to this PR description or section in pre-req.

@dmartinol
added missing README
b9ae8b7 
Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
@dmartinol
Copy link
Contributor Author

dmartinol commented Sep 17, 2024

A brief readme.md description about the example at https://github.com/dmartinol/feast/tree/install_kind/examples/kind-quickstart will help before looking at all the jupyter notebooks and figure it out. something similar to this PR description or section in pre-req.

Thanks for noticing, it reained untracked in my local repo 😲

lokeshrangineni
lokeshrangineni reviewed Sep 17, 2024
View reviewed changes
dmartinol and others added 2 commits September 18, 2024 08:44
@dmartinol
typo
388931a 
Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
@dmartinol
Merge branch 'feast-dev:master' into install_kind
7af6a8b
@dmartinol
Copy link
Contributor Author

dmartinol commented Sep 23, 2024

@franciscojavierarceo could pls you TAL?

franciscojavierarceo
franciscojavierarceo reviewed Sep 23, 2024
View reviewed changes
examples/kind-quickstart/02-Client.ipynb
"cell_type": "markdown",
"metadata": {},
"source": [
"Note If you see the following error, it is likely due to the [issue #4392](https://github.com/feast-dev/feast/issues/4392):\n",
Copy link
Member

@franciscojavierarceo franciscojavierarceo Sep 23, 2024

Choose a reason for hiding this comment

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

Looks like you're showing this error on purpose then, eh?

Copy link
Contributor Author

@dmartinol dmartinol Sep 24, 2024

Choose a reason for hiding this comment

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

Yep, the issue has been fixed after this PR, I should have updated it.

franciscojavierarceo
franciscojavierarceo approved these changes Sep 23, 2024
View reviewed changes
Copy link
Member

@franciscojavierarceo franciscojavierarceo left a comment

Choose a reason for hiding this comment

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

lgtm!

@franciscojavierarceo franciscojavierarceo merged commit e675cbd into feast-dev:master Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lokeshrangineni lokeshrangineni lokeshrangineni left review comments

@franciscojavierarceo franciscojavierarceo franciscojavierarceo approved these changes

Assignees

No one assigned

Labels

ok-to-test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add example of deployment of Feast in local Kubernetes environment

4 participants

@dmartinol @lokeshrangineni @franciscojavierarceo @jeremyary