…thentication (feast-dev#4384)

* allow feast snowflake to read in byte string for private-key authentication

Signed-off-by: Artur <artur.kolakowski@medely.com>

* Update type hint for  to use Union instead of | syntax

Signed-off-by: Artur <artur.kolakowski@medely.com>

* Update type hint for private_key to use Union instead of | syntax

Signed-off-by: Artur <artur.kolakowski@medely.com>

* Update type hint in parse_private_key_path

Signed-off-by: Artur <artur.kolakowski@medely.com>

* added private_key_content in Snowflake configs to support key-pair auth by reading in byte string

Signed-off-by: Artur <artur.kolakowski@medely.com>

* fix incompatible linting types

Signed-off-by: Artur <artur.kolakowski@medely.com>

* remove unused Union import

Signed-off-by: Artur <artur.kolakowski@medely.com>

* fix formating

Signed-off-by: Artur <artur.kolakowski@medely.com>

---------

Signed-off-by: Artur <artur.kolakowski@medely.com>
Co-authored-by: Artur <artur.kolakowski@medely.com>

…erns and feature transformations (feast-dev#4400)

* merging changes

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* merging

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Adding some clarity.

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Update write-patterns.md

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* chore: Update feature-transformetion.md (feast-dev#4405)

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated readme

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated summary and readme

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated docs

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Updated readme

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated more

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated transformation

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated urls

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* refactoring and renaming architecture-and-components to components and architecture

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated urls

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* moving files from architecture-and-components to components/

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* had a typo in components

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* Cleaned everything up

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

---------

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

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 <dan.baron@starlingbank.com>

* fix: fixing linting and formatting issues in tests

Signed-off-by: Dan Baron <dan.baron@starlingbank.com>

* fix: removing unnecessary Path object construction

Signed-off-by: Dan Baron <dan.baron@starlingbank.com>

---------

Signed-off-by: Dan Baron <dan.baron@starlingbank.com>

…4416)

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* fix: Remove pytz.

Signed-off-by: Shuchu Han <shuchu.han@gmail.com>

* fix: Keep the pytz.UTC part in dask.py

Signed-off-by: Shuchu Han <shuchu.han@gmail.com>

---------

Signed-off-by: Shuchu Han <shuchu.han@gmail.com>

feast-dev#4420)

Update SUMMARY.md

…ML (feast-dev#4423)

* chore: fixed README template to be consistent with current README

Signed-off-by: dandawg <12484302+dandawg@users.noreply.github.com>

* docs: markdown links consistency with html

Signed-off-by: dandawg <12484302+dandawg@users.noreply.github.com>

---------

Signed-off-by: dandawg <12484302+dandawg@users.noreply.github.com>

…v#4424)

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>

* initial commit

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fixed linting issues (but 1)

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* deleted AuthzedResource and moved types to the Permission class

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* using pytest.mark.parametrize tests

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* moved decorator to decorator module

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* parametrized decision tests

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Added matcher and action modules. Added global assert_permissions function

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fixed linting error

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Managing with_subclasses flag and overriding it in case it's an abstract class like DataSource

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Permission includes a single Policy

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* completed docstrings for permissions package

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fixed inter issues

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Changed roles matching rule from "all" to "any"

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Introducing permission framework and authorization manager in user guide (to be continued after the code is consolidated)

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* removed test code

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* hiding sensitive data (false positive, anyway)

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Added filter_only flag to assert_permissions and returning a list of filtered resources instead of PermissionError

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added the option to return the single resource, or None

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* separate validating functions: assert_permission and filtered_resources

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Applied review comments

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry
- Fixes to code
- Made test case broader

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry
- Fixed incorrectly recognized linter error

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry
- Added test
- Fixed missing property to permission
- Changed code following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry
- Fixes to code
- Made test case broader

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry
- Fixed incorrectly recognized linter error

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry
- Added test
- Fixed delete and apply permission

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* replaced aggregated actions with aliases for QUERY and WRITE and ALL

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Updated user guide

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Updated enum in proto

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry
- Fixed test errors following refactor
- Added test

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Store and Manage permissions in the Registry
- Removed redundant property
- Added tags filter option to list_permissions

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Added permission assert check for registry server, offline server, online server functions

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fix linter after rebase

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* CLI command "feast permissions list"
- Changes following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* CLI command "feast permissions list"
- Changes following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added the documents reference for permissions for online, offline, registry server endpoints.

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>

* Incorporating code review comments - renaming type from k8 to kubernetes.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

---------

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* definition and integration of auth manager in feast offline and online servers

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* typo

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* duplicated if

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* renamed functions with long name

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* using User class instead of RoleManager (completely removed)

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Feed SecurityManager with Registry instance to fetch the actual permissions

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fixed linter

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* review comments

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fixed broken IT

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Adding registry server (UT to be completed)

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix linter

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* passing auth manager type from config

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* used auth config to set auth manager type

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* inject the user details

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* created decorator function and applied to arrow function for injecting the user detail:wq

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* code review fixes including the unit test and integration test as suggested

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Implementation of oidc client authentication. (feast-dev#40)

* Adding initial draft code to manage the oidc client authentication.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

* Adding initial draft code to manage the oidc client authentication.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

* Incorporating code review comments.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

---------

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Added authentication header for client grpc calls

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Made changes following code review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Made changes following code review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Made changes following code review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Made changes following code review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Made changes following code review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Made changes following code review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Made changes following code review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added auth configuration for arrow flight client

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Made changes following code review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix linter

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Propagating auth config to token parser in server init

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* adding headers and client_secret to token request

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* working E2E test of authenticated registy server

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* renamed test

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fixed broken test

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix rebase issues

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix rebase issues

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Adding the auth client documentations and unit testing for auth client code.
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Adding the auth client documentations and unit testing for auth client code.
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Incorporating code review comments.
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Incorporating code review comments.
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Introducing permission framework and authorization manager in user guide (to be continued after the code is consolidated)

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* CLI command "feast permissions list"
- Added missing dependency

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Client module-grpc
- Added missing auth header for calls to remote registry

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fix auth tests with permissions
- Made changes to enforcer ana security manager permission checking logic

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fix auth tests with permissions
- Made changes following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fix auth tests with permissions
- Made changes following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fix auth tests with permissions
- Made changes following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

* Adding missed dependency and regenerated the requirements files.
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

* Addinig missing changes from the original PR.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

---------

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added check and list-roles subcommands

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* typo

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added comment in cli_utils to remind the original function from which this logic was derived

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Moved the common fixtures to the root conftest.py or auth_permissions_util.py

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Adding missed dependency and regenerated the requirements files.
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fixed bug in GetPermission API

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Permission CRUD test

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Added feast-rbac example

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Added support to read the token from enviroment variable to run from local

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fix the header for arrow fligth

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix the header issue

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added permissions apply file

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* set the user in the grpc server

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added roles and updated permission with all roles

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* updated chart to include the service account

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* created client example with roles and updated installation/cleanup script

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* rebased with master

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

* Adding missed dependency and regenerated the requirements files.
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

* Addinig missing changes from the original PR.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

---------

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fixed DecisionStrategy not persisted

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fixed DecisionStrategy not persisted

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fixed DecisionStrategy not persisted
- Implemented review comments

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Revert "Fix decision strategy not saved"

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Dropped global decision strategy

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* updated rbac demo example

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Initial Draft version to the tests with remote offline server with OIDC authentication permissions. Happy path only.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Abstracting the specific code for Offline Permissions by creating new class for PermissionsEnvironment.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Formatting the python files using make format-python.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Created the grpc client auth header interceptor and removed the manual injection of the header.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Created the grpc client auth header interceptor and removed the manual injection of the header.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

* Adding missed dependency and regenerated the requirements files.
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

* Addinig missing changes from the original PR.

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>

---------

Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Adding the extra writer permission to fix the integration test issue with offline server.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Try to fix java integration test - ModuleNotFoundError: No module named 'feast.permissions.server'

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix java integration test - ModuleNotFoundError: No module named 'jwt'

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix java integration test - ModuleNotFoundError: No module named 'kubernetes'

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Adding missing permissions for offline store test cases - classes FileSource, FeatureService classes. (feast-dev#64)

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Updating the offline integration test permissions.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* updated test.py file for rbac-example

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix the DeleteFeatureView function to handle stream feature view type

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Incorporating the code review comments from Francisco on upstream PR.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Update docs/getting-started/concepts/permission.md

Co-authored-by: Francisco Arceo <farceo@redhat.com>
Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Update docs/getting-started/concepts/permission.md

Co-authored-by: Francisco Arceo <farceo@redhat.com>
Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Update docs/getting-started/concepts/permission.md

Co-authored-by: Francisco Arceo <farceo@redhat.com>
Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Small fixes (feast-dev#71)

* Improved permission denial log

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>

* Added leeway option to accept tokens released in the past (up to 10")

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>

---------

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* commented/removed oidc tests to verify  integration test
commented/removed test_auth_permission.py file

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Making number of workers back to 8 and enabled the test_remote_online_store_read

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Making number of workers to 4.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Reverting number of workers from 8 to 4.
Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Reverting number of workers from 8 to 4. Reverting the marker @pytest.mark.xdist_group(name="keycloak")
Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Reverting number of workers from 8 to 4 for make target test-python-integration-local

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <19699092+lokeshrangineni@users.noreply.github.com>

* Fixing the failing integration test cases by setting the header in binary format.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* 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 <19699092+lokeshrangineni@users.noreply.github.com>

* Incorproating code review comments from Daniel.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

---------

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* removed with_subclasses option (it's the default and unique behavior)

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* a full, minimal, reproducible example of the RBAC feature

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* 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 <tmihalac@redhat.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fixed the registry apply function assertation

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* removed the examples

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Integrated comment

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>

* removed the firebase depdency and fix the doc conflicts

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Introducing permission framework and authorization manager in user guide (to be continued after the code is consolidated)

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Permission resources miss the created_timestamp and last_updated_timestamp fields

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* remove error incase if user has no roles assinged incase unthorized user

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* renamed READ action to DESCRIBE

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>

* Specified authorization manager and authorization configuration

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>

* fix the linter and remove subclass from doc

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* addressed the pr reivew comments

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Incorporating code review comment and this file is not needed.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Addressed the review comments on the PR

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Reducing the markers from 8 to 4 to see if it fixes the issues with memory.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* addresses feedback on rbac doc

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* rename action name from QUERY to READ

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix the doc to replace query with read

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

---------

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Abdul Hameed <ahameed@redhat.com>
Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>
Signed-off-by: Lokesh Rangineni <lokeshforjava@gmail.com>
Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Co-authored-by: Theodor Mihalache <tmihalac@redhat.com>
Co-authored-by: Abdul Hameed <ahameed@redhat.com>
Co-authored-by: lokeshrangineni <lokeshforjava@gmail.com>
Co-authored-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>
Co-authored-by: Francisco Arceo <farceo@redhat.com>

…east-dev#4430)

* fix the rbac docs links

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* fix: links to the RBAC documentation under Concepts and Components sections

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

---------

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* reorganize registry docs

Signed-off-by: tokoko <togurgenidze@gmail.com>

* remove commented out text

Signed-off-by: tokoko <togurgenidze@gmail.com>

* changes in registry.md

Signed-off-by: tokoko <togurgenidze@gmail.com>

---------

Signed-off-by: tokoko <togurgenidze@gmail.com>
Co-authored-by: tokoko <togurgenidze@gmail.com>

build: force the protobuf version in the build system so that it is compatible with the runtime dependency

Signed-off-by: Yang, Bo <bo.yang@protonbase.io>

fix typos

Signed-off-by: Brijesh Vora <brijesh.vora@sailpoint.com>

…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 <19699092+lokeshrangineni@users.noreply.github.com>

* 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 <19699092+lokeshrangineni@users.noreply.github.com>

* 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 <19699092+lokeshrangineni@users.noreply.github.com>

* 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 <19699092+lokeshrangineni@users.noreply.github.com>

* Fixing the issue with pre-commit hook template. Accidentally this was reverted in previous rebase and reverting it now.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

---------

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* mark keycloak tests with xdist_group

Signed-off-by: tokoko <togurgenidze@gmail.com>

* apply changes to test-python-integration

Signed-off-by: tokoko <togurgenidze@gmail.com>

---------

Signed-off-by: tokoko <togurgenidze@gmail.com>
Co-authored-by: tokoko <togurgenidze@gmail.com>

fix: locate feature_store.yaml from __file__

Signed-off-by: Yang, Bo <bo.yang@protonbase.io>

Signed-off-by: Yang, Bo <bo.yang@protonbase.io>

…new and the old instance (feast-dev#4449)

* Validating permission to update an existing request on both the new and the old instance

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>

* Reviewed update permission logic as per comment, added UT

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

---------

Signed-off-by: Daniele Martinoli <86618610+dmartinol@users.noreply.github.com>
Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

…east-dev#4465)

introduced base class FeastError for all Feast exceptions, with initial methods to map the grpc and HTTP status code

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

Intra server communication

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

…east-dev#4467)

Bumps [notebook](https://github.com/jupyter/notebook) from 7.2.1 to 7.2.2.
- [Release notes](https://github.com/jupyter/notebook/releases)
- [Changelog](https://github.com/jupyter/notebook/blob/@jupyter-notebook/tree@7.2.2/CHANGELOG.md)
- [Commits](https://github.com/jupyter/notebook/compare/@jupyter-notebook/tree@7.2.1...@jupyter-notebook/tree@7.2.2)

---
updated-dependencies:
- dependency-name: notebook
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

feast-dev#4468)

Bumps [jupyterlab](https://github.com/jupyterlab/jupyterlab) from 4.2.3 to 4.2.5.
- [Release notes](https://github.com/jupyterlab/jupyterlab/releases)
- [Changelog](https://github.com/jupyterlab/jupyterlab/blob/@jupyterlab/lsp@4.2.5/CHANGELOG.md)
- [Commits](https://github.com/jupyterlab/jupyterlab/compare/@jupyterlab/lsp@4.2.3...@jupyterlab/lsp@4.2.5)

---
updated-dependencies:
- dependency-name: jupyterlab
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…gine (feast-dev#4456)

* Check for snowflake functions over stage

Signed-off-by: Ben Stuart <ben@benstuart.ie>

* Refactor stage_list to function_list

Signed-off-by: Ben Stuart <ben@benstuart.ie>

---------

Signed-off-by: Ben Stuart <ben@benstuart.ie>

* initial commit targetting grpc registry server

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* refactor: Introduced base class FeastError for all Feast exceptions (feast-dev#4465)

introduced base class FeastError for all Feast exceptions, with initial methods to map the grpc and HTTP status code

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* initial commit targetting grpc registry server

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* fixed merge error

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* initial commit targetting grpc registry server

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* fixed merge error

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* integrated comment

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* moved imports as per comment

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

---------

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

…east-dev#4472)

build: explicit protobuf build version; consistent build/setup deps

Right now if one downloads `feast-0.40.1-py2.py3-none-any.whl` from
PyPi it contains:
```
$ grep 'Protobuf Python Version' feast/protos/feast/registry/RegistryServer_pb2.py
```
Which is outside
```
$ grep 'protobuf<' feast-0.40.1.dist-info/METADATA
Requires-Dist: protobuf<5.0.0,>=4.24.0
```
Leading to runtime errors (feast-dev#4437).  This was mitigated by feast-dev#4438.  This
change tightens this up further by:
 * Deleting the Makefile command that was trying to do this
 unsuccessfully.
 * Aligns the setup/build requirements
 * Sets the version of protobuf to match the *minimum* of the range.
 There is no guarantee that protos generated by `4.X` will work with `4.(X-1)`.

Signed-off-by: Chris Burroughs <chris.burroughs@gmail.com>

…datory onl… (feast-dev#4460)

* squashed the last 23 commits and make username, password,  client_Secret fields are required for oidc client configuration

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* squashed the last 23 commits and make username, password,  client_Secret fields are required for oidc client configuration

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Fixing the failing tests.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Fixing the Integration test failures.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Fixing the Integration test failures.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Removing the unnecessary configuration not needed after recent change.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Added client_secret also to calculate the oidc_client type calculation.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

---------

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Updated API docs with Permission types and functions

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* Updated API docs with Permission types and functions

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

---------

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* Added Online Store rest client errors handler

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* Added Online Store rest client errors handler
- Small refactor to from_error_detail and FeastErrors
- Fixed tests

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* Added Online Store rest client errors handler
- Fixed linter

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

---------

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

feast-dev#4487)

feat: Add cli list/describe for SavedDatasets, StreamFeatureViews, & ValidationReferences

Signed-off-by: Tommy Hughes <tohughes@redhat.com>

…ments (feast-dev#4483)

chore: Bump cryptography in /sdk/python/requirements

Bumps [cryptography](https://github.com/pyca/cryptography) from 42.0.8 to 43.0.1.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@42.0.8...43.0.1)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…ast-dev#4469)

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] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Signed-off-by: tokoko <togurgenidze@gmail.com>

* feat: Added Project object to Feast Objects

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Extend FeastError and fixed integration tests

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Small optimization to test_modify_feature_views_success test

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Added Project object to template and quick start

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

---------

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>

* not sure why fix linting

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* not sure why fix linting

Signed-off-by: cmuhao <sduxuhao@gmail.com>

---------

Signed-off-by: cmuhao <sduxuhao@gmail.com>

…v#4497)

chore: refactor fixtures in test_universal_registry

Signed-off-by: tokoko <togurgenidze@gmail.com>

)

* add new registry method for working with any fv type

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: different project for each test in test_universal_registry

Signed-off-by: tokoko <togurgenidze@gmail.com>

* revert project names to project in test_universal_registry

Signed-off-by: tokoko <togurgenidze@gmail.com>

* remove print statements from test_universal_registry

Signed-off-by: tokoko <togurgenidze@gmail.com>

---------

Signed-off-by: tokoko <togurgenidze@gmail.com>

…kflows (feast-dev#4482)

chore: Bump actions/download-artifact in /.github/workflows

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 2 to 4.1.7.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v2...v4.1.7)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…use cases (feast-dev#4494)

fix: Resovled merge conflicts associated to new changes

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>

Signed-off-by: Shuchu Han <shuchu.han@gmail.com>

* fix: Disable active_timer When registry_ttl_sec is 0

Signed-off-by: Jiwon Park <phil.park@sktelecom.com>

* feat: Add delete mark

Signed-off-by: Jiwon Park <phil.park@sktelecom.com>

---------

Signed-off-by: Jiwon Park <phil.park@sktelecom.com>

…to Feast code. (feast-dev#4500)

Signed-off-by: Shuchu Han <shuchu.han@gmail.com>

* added rbac examples

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* Fixed the intra service communication

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* remove the rbac-local example

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* updated readme with suggestions from code review

Co-authored-by: Francisco Arceo <farceo@redhat.com>

---------

Signed-off-by: Abdul Hameed <ahameed@redhat.com>
Co-authored-by: Francisco Arceo <farceo@redhat.com>

…ent_m… (feast-dev#4505)

* Refactor auth_client_manager_factory.py in function get_auth_client_manager

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* Refactor auth_client_manager_factory.py in function get_auth_client_manager
-Added test

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* Refactor auth_client_manager_factory.py in function get_auth_client_manager
-updated test following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* Refactor auth_client_manager_factory.py in function get_auth_client_manager
-fixed linter

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

---------

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

…st-dev#4501)

A full, minimal, reproducible example of the RBAC feature

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* Adding the missed documentation -
* OIDC Token requirement or assumptions
* Added `feast permissions check` cli command documentation.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Adding the missed documentation -
* OIDC Token requirement or assumptions
* Added `feast permissions check` cli command documentation.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

* Fixed code review comments.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

---------

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

Alternative to feast-dev#4461

Signed-off-by: Chris Burroughs <chris.burroughs@gmail.com>

…#4519)

* Removed the k8s dependency from required dependencies

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* updated requirments lock files

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

---------

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

…st-dev#4525)

tslib is a runtime dependency, so it should be in dependencies. This
ensures the specified version is installed also when the package is used
as a module.

Signed-off-by: Harri Lehtola <peruukki@hotmail.com>

* fix: Added Offline Store Arrow client errors handler

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* Added more tests

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

---------

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* fix: Removed protobuf as a required dependency

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Removed install-protoc-dependencies target

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

---------

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>

…ssion flow (feast-dev#4531)

* Logger setting for feature servers

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* added/updated logger for permission flow

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* set defualt logger level for feature servers as WARNING

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

---------

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

…tations (feast-dev#4537)

* fix: Removed protobuf as a required dependency

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Removed install-protoc-dependencies target

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Ran lock python dependencies to correct dependencies

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Adding protobuf<5 as a required dependency due to snowflake limitations

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

---------

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>

…dev#4538)

minor: Added indexes to sql tables to optimize query execution

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>

…ev#4536)

* Add docs example of how to use tags with feature views

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* Add docs example of how to use tags with feature views
- changes following review

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

---------

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

…dev#4544)

* fix: Fix for SQL registry initialization fails feast-dev#4543

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Removed combined_sql_fixtures

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Added protobuf dependency to pyproject.toml

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

---------

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>

* update entity retrieval and add duckdb

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix typo

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix typo

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix typo

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix typo

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix typo

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix typo

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix typo

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix typo

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix lint

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix test

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix test

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix test

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix test

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix test

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix test

Signed-off-by: cmuhao <sduxuhao@gmail.com>

* fix test

Signed-off-by: cmuhao <sduxuhao@gmail.com>

---------

Signed-off-by: cmuhao <sduxuhao@gmail.com>

…them (feast-dev#4545)

…e Store (feast-dev#4509)

Fixes validator field access for 'billing_project_id' in BigQuery Offline Store

Signed-off-by: gesche <26026582+jgesche@users.noreply.github.com>

…ocally (feast-dev#4516)

* Added examples to deployed Feast remote server components podman container locally

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

* removed the script and used composed files

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

---------

Signed-off-by: Abdul Hameed <ahameed@redhat.com>

…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 <farceo@redhat.com>

* updated

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

---------

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

…dev#4550)

* fix: Deleting data from feast_metadata when we delete project

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

* fix: Deleting for snowflake

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>

---------

Signed-off-by: Bhargav Dodla <bdodla@expediagroup.com>
Co-authored-by: Bhargav Dodla <bdodla@expediagroup.com>

…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] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: commit generated python files to repo

Signed-off-by: tokoko <togurgenidze@gmail.com>

* merge from master

Signed-off-by: tokoko <togurgenidze@gmail.com>

* chore: remove protos from gitignore

Signed-off-by: tokoko <togurgenidze@gmail.com>

---------

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: grpc import error

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: loosen protobuf build requirement

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: pin grpcio-tools version

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: revert build-system in pyproject

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: add manual install of setuptools and grpcio-tools

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: remove incorrect pixi call

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: add in-function imports in errors.py

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: formatting

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: merge changes from master

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: add line endings

Signed-off-by: tokoko <togurgenidze@gmail.com>

* fix: add line endings

Signed-off-by: tokoko <togurgenidze@gmail.com>

* chore: remove proto generation from make commands

Signed-off-by: tokoko <togurgenidze@gmail.com>

---------

Signed-off-by: tokoko <togurgenidze@gmail.com>

…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 <peruukki@hotmail.com>

…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 <peruukki@hotmail.com>

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] <support@github.com>
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] <support@github.com>
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] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…ments (feast-dev#4557)

…atch_source and entities (feast-dev#4530)

* feat: Updating protos for Projections to include more info

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* adding unit test

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* adding type checking where batch source is already serialized into protobuf

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* almost got everything working and type validation behaving

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* cleaned up and have tests behaving

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removed comment

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* updated FeatureViewProjection batch_source serialization

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* trying to debug a test

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* handling snowflake issue, cant confirm why it is happening so just going to put a workaround

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* linter

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* trying to handle it correctly

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* handling the else case for from_feature_view_definition

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* adding print

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* adding test of issue

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* think i got everything working now

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removing print

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

---------

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

…v#4528)

* Simple deployment on Kind

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* updated notebooks output

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* added missing README

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* typo

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

---------

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

Fixing the master branch build failure.

Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com>

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

…nd post apply() (feast-dev#4571)

* chore: Adding docs, community, and examples to ignore during PR tests

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* chore: Updating test to confirm feature view apply behavior for entity columns

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

* removing changes from other PR

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

---------

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

…east-dev#4572)

Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>

…ent security concerns (feast-dev#4549)

* Test workflow changes: on from pull_request_target to pull_request

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

* fix: Removed usage of pull_request_target as much as possible to prevent security concerns

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>

---------

Signed-off-by: Theodor Mihalache <tmihalac@redhat.com>