Fix deprecation warnings for SQLite #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
…thentication (feast-dev#4384) * allow feast snowflake to read in byte string for private-key authentication Signed-off-by: Artur <[email protected]> * Update type hint for to use Union instead of | syntax Signed-off-by: Artur <[email protected]> * Update type hint for private_key to use Union instead of | syntax Signed-off-by: Artur <[email protected]> * Update type hint in parse_private_key_path Signed-off-by: Artur <[email protected]> * added private_key_content in Snowflake configs to support key-pair auth by reading in byte string Signed-off-by: Artur <[email protected]> * fix incompatible linting types Signed-off-by: Artur <[email protected]> * remove unused Union import Signed-off-by: Artur <[email protected]> * fix formating Signed-off-by: Artur <[email protected]> --------- Signed-off-by: Artur <[email protected]> Co-authored-by: Artur <[email protected]>
…erns and feature transformations (feast-dev#4400) * merging changes Signed-off-by: Francisco Javier Arceo <[email protected]> * merging Signed-off-by: Francisco Javier Arceo <[email protected]> * updated Signed-off-by: Francisco Javier Arceo <[email protected]> * Update write-patterns.md Signed-off-by: Francisco Javier Arceo <[email protected]> * Update write-patterns.md Signed-off-by: Francisco Javier Arceo <[email protected]> * Update write-patterns.md Signed-off-by: Francisco Javier Arceo <[email protected]> * Update write-patterns.md Adding some clarity. Signed-off-by: Francisco Javier Arceo <[email protected]> * Update write-patterns.md Signed-off-by: Francisco Javier Arceo <[email protected]> * chore: Update feature-transformetion.md (feast-dev#4405) Signed-off-by: Francisco Javier Arceo <[email protected]> * updated readme Signed-off-by: Francisco Javier Arceo <[email protected]> * updated summary and readme Signed-off-by: Francisco Javier Arceo <[email protected]> * updated docs Signed-off-by: Francisco Javier Arceo <[email protected]> * Updated readme Signed-off-by: Francisco Javier Arceo <[email protected]> * updated more Signed-off-by: Francisco Javier Arceo <[email protected]> * updated transformation Signed-off-by: Francisco Javier Arceo <[email protected]> * updated urls Signed-off-by: Francisco Javier Arceo <[email protected]> * refactoring and renaming architecture-and-components to components and architecture Signed-off-by: Francisco Javier Arceo <[email protected]> * updated urls Signed-off-by: Francisco Javier Arceo <[email protected]> * moving files from architecture-and-components to components/ Signed-off-by: Francisco Javier Arceo <[email protected]> * had a typo in components Signed-off-by: Francisco Javier Arceo <[email protected]> * Cleaned everything up Signed-off-by: Francisco Javier Arceo <[email protected]> --------- Signed-off-by: Francisco Javier Arceo <[email protected]>
Update README.md
Update language.md
* Create ADOPTERS.md * Update ADOPTERS.md
…re-yaml overrides (feast-dev#4413) * fix: using repo_config parameter in teardown to allow for feature-store-yaml overrides Signed-off-by: Dan Baron <[email protected]> * fix: fixing linting and formatting issues in tests Signed-off-by: Dan Baron <[email protected]> * fix: removing unnecessary Path object construction Signed-off-by: Dan Baron <[email protected]> --------- Signed-off-by: Dan Baron <[email protected]>
…4416) Signed-off-by: Francisco Javier Arceo <[email protected]>
* fix: Remove pytz. Signed-off-by: Shuchu Han <[email protected]> * fix: Keep the pytz.UTC part in dask.py Signed-off-by: Shuchu Han <[email protected]> --------- Signed-off-by: Shuchu Han <[email protected]>
Update SUMMARY.md
…ML (feast-dev#4423) * chore: fixed README template to be consistent with current README Signed-off-by: dandawg <[email protected]> * docs: markdown links consistency with html Signed-off-by: dandawg <[email protected]> --------- Signed-off-by: dandawg <[email protected]>
…v#4424) Signed-off-by: Tommy Hughes <[email protected]>
Signed-off-by: Bhargav Dodla <[email protected]> Co-authored-by: Bhargav Dodla <[email protected]>
* initial commit Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fixed linting issues (but 1) Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * deleted AuthzedResource and moved types to the Permission class Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * using pytest.mark.parametrize tests Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * moved decorator to decorator module Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * parametrized decision tests Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Added matcher and action modules. Added global assert_permissions function Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fixed linting error Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Managing with_subclasses flag and overriding it in case it's an abstract class like DataSource Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Permission includes a single Policy Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * completed docstrings for permissions package Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fixed inter issues Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Changed roles matching rule from "all" to "any" Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Introducing permission framework and authorization manager in user guide (to be continued after the code is consolidated) Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * removed test code Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * hiding sensitive data (false positive, anyway) Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Added filter_only flag to assert_permissions and returning a list of filtered resources instead of PermissionError Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * added the option to return the single resource, or None Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * separate validating functions: assert_permission and filtered_resources Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Applied review comments Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry - Fixes to code - Made test case broader Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry - Fixed incorrectly recognized linter error Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry - Added test - Fixed missing property to permission - Changed code following review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry - Fixes to code - Made test case broader Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry - Fixed incorrectly recognized linter error Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry - Added test - Fixed delete and apply permission Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * replaced aggregated actions with aliases for QUERY and WRITE and ALL Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Updated user guide Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Updated enum in proto Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry - Fixed test errors following refactor - Added test Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Store and Manage permissions in the Registry - Removed redundant property - Added tags filter option to list_permissions Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Added permission assert check for registry server, offline server, online server functions Signed-off-by: Abdul Hameed <[email protected]> * Fix linter after rebase Signed-off-by: Abdul Hameed <[email protected]> * CLI command "feast permissions list" Added cli command permissions Added tags parameter to list_validation_references and list_saved_datasets in registry Added list_validation_references and list_saved_datasets apis to feature_store Added missing tags parameters to registry_server methods Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * CLI command "feast permissions list" - Changes following review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * CLI command "feast permissions list" - Changes following review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * added the documents reference for permissions for online, offline, registry server endpoints. Signed-off-by: Abdul Hameed <[email protected]> * Incorporating code review comments to parse the auth block from the f… (feast-dev#36) * Incorporating code review comments to parse the auth block from the feature_store.yaml file. Signed-off-by: Lokesh Rangineni <[email protected]> * Incorporating code review comments - renaming type from k8 to kubernetes. Signed-off-by: Lokesh Rangineni <[email protected]> --------- Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * definition and integration of auth manager in feast offline and online servers Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * typo Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * duplicated if Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * renamed functions with long name Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * using User class instead of RoleManager (completely removed) Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Feed SecurityManager with Registry instance to fetch the actual permissions Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fixed linter Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * review comments Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fixed broken IT Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Adding registry server (UT to be completed) Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fix linter Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * passing auth manager type from config Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * used auth config to set auth manager type Signed-off-by: Abdul Hameed <[email protected]> * inject the user details Signed-off-by: Abdul Hameed <[email protected]> * created decorator function and applied to arrow function for injecting the user detail:wq Signed-off-by: Abdul Hameed <[email protected]> * code review fixes including the unit test and integration test as suggested Signed-off-by: Abdul Hameed <[email protected]> * Implementation of oidc client authentication. (feast-dev#40) * Adding initial draft code to manage the oidc client authentication. Signed-off-by: Lokesh Rangineni <[email protected]> * Adding initial draft code to manage the oidc client authentication. Signed-off-by: Lokesh Rangineni <[email protected]> * Incorporating code review comments. Signed-off-by: Lokesh Rangineni <[email protected]> --------- Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Added authentication header for client grpc calls Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Made changes following code review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Made changes following code review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Made changes following code review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Made changes following code review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Made changes following code review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Made changes following code review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Made changes following code review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * added auth configuration for arrow flight client Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Made changes following code review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fix linter Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Propagating auth config to token parser in server init Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * adding headers and client_secret to token request Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * working E2E test of authenticated registy server Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * renamed test Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fixed broken test Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fix rebase issues Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fix rebase issues Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Adding the auth client documentations and unit testing for auth client code. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Adding the auth client documentations and unit testing for auth client code. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Incorporating code review comments. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Incorporating code review comments. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Introducing permission framework and authorization manager in user guide (to be continued after the code is consolidated) Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * CLI command "feast permissions list" - Added missing dependency Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Client module-grpc - Added missing auth header for calls to remote registry Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fix auth tests with permissions - Made changes to enforcer ana security manager permission checking logic Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fix auth tests with permissions - Made changes following review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fix auth tests with permissions - Made changes following review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fix auth tests with permissions - Made changes following review Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Moved the common fixtures to the root conftest.py or auth_permissions_util.py (feast-dev#54) * Moved the common fixtures to the root conftest.py or auth_permissions_util.py Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> * Adding missed dependency and regenerated the requirements files. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> * Addinig missing changes from the original PR. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> --------- Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * added check and list-roles subcommands Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * typo Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * added comment in cli_utils to remind the original function from which this logic was derived Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * 1) Updating the existing integration test with auth permissions configurations. 2) Refactored the common code and moved to the util class and common conftest.py file. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Moved the common fixtures to the root conftest.py or auth_permissions_util.py Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Adding missed dependency and regenerated the requirements files. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * 1) Updating the existing integration test with auth permissions configurations. 2) Refactored the common code and moved to the util class and common conftest.py file. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * 1) Fixing an issue with the way getting markers after changing the fixture scope to module. Now looking up the markers coming from the entire module run. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fixed bug in GetPermission API Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Permission CRUD test Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Added feast-rbac example Signed-off-by: Abdul Hameed <[email protected]> * Added support to read the token from enviroment variable to run from local Signed-off-by: Abdul Hameed <[email protected]> * Fix the header for arrow fligth Signed-off-by: Abdul Hameed <[email protected]> * fix the header issue Signed-off-by: Abdul Hameed <[email protected]> * added permissions apply file Signed-off-by: Abdul Hameed <[email protected]> * set the user in the grpc server Signed-off-by: Abdul Hameed <[email protected]> * added roles and updated permission with all roles Signed-off-by: Abdul Hameed <[email protected]> * updated chart to include the service account Signed-off-by: Abdul Hameed <[email protected]> * created client example with roles and updated installation/cleanup script Signed-off-by: Abdul Hameed <[email protected]> * rebased with master Signed-off-by: Abdul Hameed <[email protected]> * Moved the common fixtures to the root conftest.py or auth_permissions_util.py (feast-dev#54) * Moved the common fixtures to the root conftest.py or auth_permissions_util.py Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> * Adding missed dependency and regenerated the requirements files. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> * Addinig missing changes from the original PR. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> --------- Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fixed DecisionStrategy not persisted Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fixed DecisionStrategy not persisted Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fixed DecisionStrategy not persisted - Implemented review comments Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Revert "Fix decision strategy not saved" Signed-off-by: Abdul Hameed <[email protected]> * Dropped global decision strategy Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * updated rbac demo example Signed-off-by: Abdul Hameed <[email protected]> * Adding permissions directly instead of from the common place for the online read integration tests. Cleaned up some minor changes to fix the unpredictable issue with the feature server process. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Initial Draft version to the tests with remote offline server with OIDC authentication permissions. Happy path only. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Abstracting the specific code for Offline Permissions by creating new class for PermissionsEnvironment. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Formatting the python files using make format-python. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Separated the permissions for online, offline and registry servers. moved the fixtures scope accordingly as we can't reuse the permissions for all the test cases. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Separated the permissions for online, offline and registry servers. moved the fixtures scope accordingly as we can't reuse the permissions for all the test cases. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Created the grpc client auth header interceptor and removed the manual injection of the header. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Created the grpc client auth header interceptor and removed the manual injection of the header. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fix: java to proto failing - changed java_outer_classname for Permission.proto and Policy.proto - removed experimental optional from permission proto Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * CLI command "feast permissions list" Added cli command permissions Added tags parameter to list_validation_references and list_saved_datasets in registry Added list_validation_references and list_saved_datasets apis to feature_store Added missing tags parameters to registry_server methods Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Moved the common fixtures to the root conftest.py or auth_permissions_util.py (feast-dev#54) * Moved the common fixtures to the root conftest.py or auth_permissions_util.py Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> * Adding missed dependency and regenerated the requirements files. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> * Addinig missing changes from the original PR. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> --------- Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fix: java to proto failing - changed java_outer_classname for Permission.proto and Policy.proto - removed experimental optional from permission proto Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Adding the extra writer permission to fix the integration test issue with offline server. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Try to fix java integration test - ModuleNotFoundError: No module named 'feast.permissions.server' Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fix java integration test - ModuleNotFoundError: No module named 'jwt' Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * fix java integration test - ModuleNotFoundError: No module named 'kubernetes' Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Adding missing permissions for offline store test cases - classes FileSource, FeatureService classes. (feast-dev#64) Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Updating the offline integration test permissions. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * updated test.py file for rbac-example Signed-off-by: Abdul Hameed <[email protected]> * fix the DeleteFeatureView function to handle stream feature view type Signed-off-by: Abdul Hameed <[email protected]> * Updating permissions of the integration test cases to address code review comments and also check if the online_read integration test fixes. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Incorporating the code review comments from Francisco on upstream PR. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Update docs/getting-started/concepts/permission.md Co-authored-by: Francisco Arceo <[email protected]> Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Update docs/getting-started/concepts/permission.md Co-authored-by: Francisco Arceo <[email protected]> Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Update docs/getting-started/concepts/permission.md Co-authored-by: Francisco Arceo <[email protected]> Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Small fixes (feast-dev#71) * Improved permission denial log Signed-off-by: Daniele Martinoli <[email protected]> * Added leeway option to accept tokens released in the past (up to 10") Signed-off-by: Daniele Martinoli <[email protected]> --------- Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * commented/removed oidc tests to verify integration test commented/removed test_auth_permission.py file Signed-off-by: Abdul Hameed <[email protected]> * Enabling the keycloak related integration tests and also initializing the keycloak only once in the entire run. Reduced the number of works and increased the duration as well. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Making number of workers back to 8 and enabled the test_remote_online_store_read Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Making number of workers to 4. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Incorporating the code review comments from Tornike to use @pytest.mark.xdist_group(name="keycloak"). Reverting number of markers from 4 to 8 for the make file target test-python-integration-local. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Reverting number of workers from 8 to 4. Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Reverting number of workers from 8 to 4. Reverting the marker @pytest.mark.xdist_group(name="keycloak") Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Reverting number of workers from 8 to 4 for make target test-python-integration-local Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Added the arrow flight interceptor to inject the auth header. (feast-dev#68) * * Added the arrow flight interceptor to inject the auth header. * Injecting grpc interceptor if it is needed when auth type is not NO_AUTH. Signed-off-by: Lokesh Rangineni <[email protected]> * Fixing the failing integration test cases by setting the header in binary format. Signed-off-by: Lokesh Rangineni <[email protected]> * Refactored method and moved to factory class to incorporate code review comment. Fixed lint error by removing the type of port. and other minor changes. Signed-off-by: Lokesh Rangineni <[email protected]> * Incorproating code review comments from Daniel. Signed-off-by: Lokesh Rangineni <[email protected]> --------- Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * removed with_subclasses option (it's the default and unique behavior) Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * a full, minimal, reproducible example of the RBAC feature Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Add missing required_tags to permission object and cli info - Add missing required_tags to permission object - added required_tags to cli info Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Fixed the registry apply function assertation Signed-off-by: Abdul Hameed <[email protected]> * removed the examples Signed-off-by: Abdul Hameed <[email protected]> * Integrated comment Signed-off-by: Daniele Martinoli <[email protected]> * removed the firebase depdency and fix the doc conflicts Signed-off-by: Abdul Hameed <[email protected]> * Introducing permission framework and authorization manager in user guide (to be continued after the code is consolidated) Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> * Permission resources miss the created_timestamp and last_updated_timestamp fields Signed-off-by: Theodor Mihalache <[email protected]> * remove error incase if user has no roles assinged incase unthorized user Signed-off-by: Abdul Hameed <[email protected]> * renamed READ action to DESCRIBE Signed-off-by: Daniele Martinoli <[email protected]> * Specified authorization manager and authorization configuration Signed-off-by: Daniele Martinoli <[email protected]> * fix the linter and remove subclass from doc Signed-off-by: Abdul Hameed <[email protected]> * addressed the pr reivew comments Signed-off-by: Abdul Hameed <[email protected]> * Incorporating code review comment and this file is not needed. Signed-off-by: Lokesh Rangineni <[email protected]> * Addressed the review comments on the PR Signed-off-by: Abdul Hameed <[email protected]> * Reducing the markers from 8 to 4 to see if it fixes the issues with memory. Signed-off-by: Lokesh Rangineni <[email protected]> * addresses feedback on rbac doc Signed-off-by: Abdul Hameed <[email protected]> * rename action name from QUERY to READ Signed-off-by: Abdul Hameed <[email protected]> * fix the doc to replace query with read Signed-off-by: Abdul Hameed <[email protected]> --------- Signed-off-by: Daniele Martinoli <[email protected]> Signed-off-by: Abdul Hameed <[email protected]> Signed-off-by: Theodor Mihalache <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Signed-off-by: Lokesh Rangineni <[email protected]> Co-authored-by: Theodor Mihalache <[email protected]> Co-authored-by: Abdul Hameed <[email protected]> Co-authored-by: lokeshrangineni <[email protected]> Co-authored-by: Lokesh Rangineni <[email protected]> Co-authored-by: Francisco Arceo <[email protected]>
…east-dev#4430) * fix the rbac docs links Signed-off-by: Abdul Hameed <[email protected]> * fix: links to the RBAC documentation under Concepts and Components sections Signed-off-by: Abdul Hameed <[email protected]> --------- Signed-off-by: Abdul Hameed <[email protected]>
* reorganize registry docs Signed-off-by: tokoko <[email protected]> * remove commented out text Signed-off-by: tokoko <[email protected]> * changes in registry.md Signed-off-by: tokoko <[email protected]> --------- Signed-off-by: tokoko <[email protected]> Co-authored-by: tokoko <[email protected]>
build: force the protobuf version in the build system so that it is compatible with the runtime dependency Signed-off-by: Yang, Bo <[email protected]>
fix typos Signed-off-by: Brijesh Vora <[email protected]>
…ast-dev#4429) * refactoring the permissions side server side code to get the OIDC end points from the discovery URL. Also removing the auth_server_url config from oidc auth config. Signed-off-by: Lokesh Rangineni <[email protected]> * refactoring the permissions side server side code to get the OIDC end points from the discovery URL. Also removing the auth_server_url config from oidc auth config. Signed-off-by: Lokesh Rangineni <[email protected]> * refactoring the permissions side server side code to get the OIDC end points from the discovery URL. Also removing the auth_server_url config from oidc auth config. Signed-off-by: Lokesh Rangineni <[email protected]> * refactoring the permissions side server side code to get the OIDC end points from the discovery URL. Also removing the auth_server_url config from oidc auth config. Signed-off-by: Lokesh Rangineni <[email protected]> * Fixing the issue with pre-commit hook template. Accidentally this was reverted in previous rebase and reverting it now. Signed-off-by: Lokesh Rangineni <[email protected]> --------- Signed-off-by: Lokesh Rangineni <[email protected]>
* mark keycloak tests with xdist_group Signed-off-by: tokoko <[email protected]> * apply changes to test-python-integration Signed-off-by: tokoko <[email protected]> --------- Signed-off-by: tokoko <[email protected]> Co-authored-by: tokoko <[email protected]>
fix: locate feature_store.yaml from __file__ Signed-off-by: Yang, Bo <[email protected]>
Signed-off-by: Yang, Bo <[email protected]>
…tations (feast-dev#4537) * fix: Removed protobuf as a required dependency Signed-off-by: Bhargav Dodla <[email protected]> * fix: Removed install-protoc-dependencies target Signed-off-by: Bhargav Dodla <[email protected]> * fix: Ran lock python dependencies to correct dependencies Signed-off-by: Bhargav Dodla <[email protected]> * fix: Adding protobuf<5 as a required dependency due to snowflake limitations Signed-off-by: Bhargav Dodla <[email protected]> --------- Signed-off-by: Bhargav Dodla <[email protected]> Co-authored-by: Bhargav Dodla <[email protected]>
…dev#4538) minor: Added indexes to sql tables to optimize query execution Signed-off-by: Bhargav Dodla <[email protected]> Co-authored-by: Bhargav Dodla <[email protected]>
…ev#4536) * Add docs example of how to use tags with feature views Signed-off-by: Theodor Mihalache <[email protected]> * Add docs example of how to use tags with feature views - changes following review Signed-off-by: Theodor Mihalache <[email protected]> --------- Signed-off-by: Theodor Mihalache <[email protected]>
…dev#4544) * fix: Fix for SQL registry initialization fails feast-dev#4543 Signed-off-by: Bhargav Dodla <[email protected]> * fix: Removed combined_sql_fixtures Signed-off-by: Bhargav Dodla <[email protected]> * fix: Added protobuf dependency to pyproject.toml Signed-off-by: Bhargav Dodla <[email protected]> --------- Signed-off-by: Bhargav Dodla <[email protected]> Co-authored-by: Bhargav Dodla <[email protected]>
* update entity retrieval and add duckdb Signed-off-by: cmuhao <[email protected]> * lint Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix typo Signed-off-by: cmuhao <[email protected]> * fix typo Signed-off-by: cmuhao <[email protected]> * fix typo Signed-off-by: cmuhao <[email protected]> * fix typo Signed-off-by: cmuhao <[email protected]> * fix typo Signed-off-by: cmuhao <[email protected]> * fix typo Signed-off-by: cmuhao <[email protected]> * fix typo Signed-off-by: cmuhao <[email protected]> * fix typo Signed-off-by: cmuhao <[email protected]> * fix lint Signed-off-by: cmuhao <[email protected]> * fix test Signed-off-by: cmuhao <[email protected]> * fix test Signed-off-by: cmuhao <[email protected]> * fix test Signed-off-by: cmuhao <[email protected]> * fix test Signed-off-by: cmuhao <[email protected]> * fix test Signed-off-by: cmuhao <[email protected]> * fix test Signed-off-by: cmuhao <[email protected]> * fix test Signed-off-by: cmuhao <[email protected]> --------- Signed-off-by: cmuhao <[email protected]>
…e Store (feast-dev#4509) Fixes validator field access for 'billing_project_id' in BigQuery Offline Store Signed-off-by: gesche <[email protected]>
…ocally (feast-dev#4516) * Added examples to deployed Feast remote server components podman container locally Signed-off-by: Abdul Hameed <[email protected]> * removed the script and used composed files Signed-off-by: Abdul Hameed <[email protected]> --------- Signed-off-by: Abdul Hameed <[email protected]>
…on after apply() (feast-dev#4553) * chore: Adding unit test to test feature view dummy entity serialization after apply() Signed-off-by: Francisco Javier Arceo <[email protected]> * updated Signed-off-by: Francisco Javier Arceo <[email protected]> --------- Signed-off-by: Francisco Javier Arceo <[email protected]>
…dev#4550) * fix: Deleting data from feast_metadata when we delete project Signed-off-by: Bhargav Dodla <[email protected]> * fix: Deleting for snowflake Signed-off-by: Bhargav Dodla <[email protected]> --------- Signed-off-by: Bhargav Dodla <[email protected]> Co-authored-by: Bhargav Dodla <[email protected]>
…and `dependencies` (feast-dev#4554)
…o 3.0.0 (feast-dev#3752) chore: Bump gopkg.in/yaml.v3 Bumps gopkg.in/yaml.v3 from 3.0.0-20200313102051-9f266ea9e77c to 3.0.0. --- updated-dependencies: - dependency-name: gopkg.in/yaml.v3 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore: commit generated python files to repo Signed-off-by: tokoko <[email protected]> * merge from master Signed-off-by: tokoko <[email protected]> * chore: remove protos from gitignore Signed-off-by: tokoko <[email protected]> --------- Signed-off-by: tokoko <[email protected]>
* fix: grpc import error Signed-off-by: tokoko <[email protected]> * fix: loosen protobuf build requirement Signed-off-by: tokoko <[email protected]> * fix: pin grpcio-tools version Signed-off-by: tokoko <[email protected]> * fix: revert build-system in pyproject Signed-off-by: tokoko <[email protected]> * fix: add manual install of setuptools and grpcio-tools Signed-off-by: tokoko <[email protected]> * fix: remove incorrect pixi call Signed-off-by: tokoko <[email protected]> * fix: add in-function imports in errors.py Signed-off-by: tokoko <[email protected]> * fix: formatting Signed-off-by: tokoko <[email protected]> * fix: merge changes from master Signed-off-by: tokoko <[email protected]> * fix: add line endings Signed-off-by: tokoko <[email protected]> * fix: add line endings Signed-off-by: tokoko <[email protected]> * chore: remove proto generation from make commands Signed-off-by: tokoko <[email protected]> --------- Signed-off-by: tokoko <[email protected]>
…st UI (feast-dev#4556) As noted in feast-dev#3794, Feast UI is not compatible with latest react-router-dom versions, more precisely from 6.4.0 onwards. Limit react-router-dom version to a compatible range to avoid the runtime errors mentioned in the issue when installing peer dependencies without specifying exact versions. After setting the restricted versions, `yarn install` updated `react-router-dom` to the latest compatible version 6.3.0. It should be a minor upgrade (https://github.com/remix-run/react-router/blob/main/CHANGELOG.md#v630), and I didn't notice anything not working in the UI after it. Signed-off-by: Harri Lehtola <[email protected]>
…dev#4555) I initially created a `project-list.json` file, then found out it should be `projects-list.json`. Then spotted another tiny typo, so fixed it too. Signed-off-by: Harri Lehtola <[email protected]>
Bumps [webpack](https://github.com/webpack/webpack) from 5.76.1 to 5.94.0. - [Release notes](https://github.com/webpack/webpack/releases) - [Commits](webpack/webpack@v5.76.1...v5.94.0) --- updated-dependencies: - dependency-name: webpack dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…east-dev#4448) Bumps [micromatch](https://github.com/micromatch/micromatch) from 4.0.5 to 4.0.8. - [Release notes](https://github.com/micromatch/micromatch/releases) - [Changelog](https://github.com/micromatch/micromatch/blob/4.0.8/CHANGELOG.md) - [Commits](micromatch/micromatch@4.0.5...4.0.8) --- updated-dependencies: - dependency-name: micromatch dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ast-dev#4522) Bumps [express](https://github.com/expressjs/express) from 4.19.2 to 4.21.0. - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/4.21.0/History.md) - [Commits](expressjs/express@4.19.2...4.21.0) --- updated-dependencies: - dependency-name: express dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…atch_source and entities (feast-dev#4530) * feat: Updating protos for Projections to include more info Signed-off-by: Francisco Javier Arceo <[email protected]> * adding unit test Signed-off-by: Francisco Javier Arceo <[email protected]> * adding type checking where batch source is already serialized into protobuf Signed-off-by: Francisco Javier Arceo <[email protected]> * almost got everything working and type validation behaving Signed-off-by: Francisco Javier Arceo <[email protected]> * cleaned up and have tests behaving Signed-off-by: Francisco Javier Arceo <[email protected]> * removed comment Signed-off-by: Francisco Javier Arceo <[email protected]> * updated FeatureViewProjection batch_source serialization Signed-off-by: Francisco Javier Arceo <[email protected]> * trying to debug a test Signed-off-by: Francisco Javier Arceo <[email protected]> * handling snowflake issue, cant confirm why it is happening so just going to put a workaround Signed-off-by: Francisco Javier Arceo <[email protected]> * linter Signed-off-by: Francisco Javier Arceo <[email protected]> * trying to handle it correctly Signed-off-by: Francisco Javier Arceo <[email protected]> * handling the else case for from_feature_view_definition Signed-off-by: Francisco Javier Arceo <[email protected]> * adding print Signed-off-by: Francisco Javier Arceo <[email protected]> * adding test of issue Signed-off-by: Francisco Javier Arceo <[email protected]> * think i got everything working now Signed-off-by: Francisco Javier Arceo <[email protected]> * removing print Signed-off-by: Francisco Javier Arceo <[email protected]> --------- Signed-off-by: Francisco Javier Arceo <[email protected]>
…v#4528) * Simple deployment on Kind Signed-off-by: Daniele Martinoli <[email protected]> * updated notebooks output Signed-off-by: Daniele Martinoli <[email protected]> * added missing README Signed-off-by: Daniele Martinoli <[email protected]> * typo Signed-off-by: Daniele Martinoli <[email protected]> --------- Signed-off-by: Daniele Martinoli <[email protected]>
Fixing the master branch build failure. Signed-off-by: Lokesh Rangineni <[email protected]>
Signed-off-by: Francisco Javier Arceo <[email protected]>
…nd post apply() (feast-dev#4571) * chore: Adding docs, community, and examples to ignore during PR tests Signed-off-by: Francisco Javier Arceo <[email protected]> * chore: Updating test to confirm feature view apply behavior for entity columns Signed-off-by: Francisco Javier Arceo <[email protected]> * removing changes from other PR Signed-off-by: Francisco Javier Arceo <[email protected]> --------- Signed-off-by: Francisco Javier Arceo <[email protected]>
…east-dev#4572) Signed-off-by: Francisco Javier Arceo <[email protected]>
…ent security concerns (feast-dev#4549) * Test workflow changes: on from pull_request_target to pull_request Signed-off-by: Theodor Mihalache <[email protected]> * fix: Removed usage of pull_request_target as much as possible to prevent security concerns Signed-off-by: Theodor Mihalache <[email protected]> --------- Signed-off-by: Theodor Mihalache <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request addresses the deprecation warnings related to the default datetime adapter and timestamp converter in SQLite, ensuring compatibility with Python 3.12 and beyond. Link to Devin run: https://preview.devin.ai/devin/1f77bad8f80e4b769dff96e2d4eec68f
If you have any feedback, you can leave comments in the PR and I'll address them in the app!