Visitar URL original
Adding missing permissions for offline store test cases - classes Fil… · feast-dev/feast@0a3a3ba · GitHub
Skip to content

Commit 0a3a3ba

Browse files
lokeshrangineniredhatHameed
authored andcommitted
Adding missing permissions for offline store test cases - classes FileSource, FeatureService classes. (#64)
Signed-off-by: Lokesh Rangineni <19699092+lokeshrangineni@users.noreply.github.com> Signed-off-by: Abdul Hameed <ahameed@redhat.com>
1 parent a95c234 commit 0a3a3ba

1 file changed

Lines changed: 21 additions & 5 deletions

File tree

‎sdk/python/tests/integration/feature_repos/repo_configuration.py‎

Lines changed: 21 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,13 @@
2222
from feast.constants import FULL_REPO_CONFIGS_MODULE_ENV_NAME
2323
from feast.data_source import DataSource
2424
from feast.errors import FeastModuleImportError
25+
from feast.feature_service import FeatureService
2526
from feast.infra.feature_servers.base_config import (
2627
BaseFeatureServerConfig,
2728
FeatureLoggingConfig,
2829
)
2930
from feast.infra.feature_servers.local_process.config import LocalFeatureServerConfig
31+
from feast.infra.offline_stores.file_source import FileSource
3032
from feast.permissions.action import AuthzedAction
3133
from feast.permissions.auth_model import OidcAuthConfig
3234
from feast.permissions.permission import Permission
@@ -488,35 +490,49 @@ def setup(self):
488490
Permission(
489491
name="offline_permissions_perm",
490492
types=Permission,
491-
policy=RoleBasedPolicy(roles=["reader", "writer"]),
493+
policy=RoleBasedPolicy(roles=["writer"]),
492494
actions=[AuthzedAction.QUERY_OFFLINE, AuthzedAction.WRITE_OFFLINE],
493495
),
494496
Permission(
495497
name="offline_entities_perm",
496498
types=Entity,
497499
with_subclasses=False,
498-
policy=RoleBasedPolicy(roles=["reader", "writer"]),
500+
policy=RoleBasedPolicy(roles=["writer"]),
499501
actions=[AuthzedAction.QUERY_OFFLINE, AuthzedAction.WRITE_OFFLINE],
500502
),
501503
Permission(
502504
name="offline_fv_perm",
503505
types=FeatureView,
504506
with_subclasses=False,
505-
policy=RoleBasedPolicy(roles=["reader", "writer"]),
507+
policy=RoleBasedPolicy(roles=["writer"]),
506508
actions=[AuthzedAction.QUERY_OFFLINE, AuthzedAction.WRITE_OFFLINE],
507509
),
508510
Permission(
509511
name="offline_odfv_perm",
510512
types=OnDemandFeatureView,
511513
with_subclasses=False,
512-
policy=RoleBasedPolicy(roles=["reader", "writer"]),
514+
policy=RoleBasedPolicy(roles=["writer"]),
513515
actions=[AuthzedAction.QUERY_OFFLINE, AuthzedAction.WRITE_OFFLINE],
514516
),
515517
Permission(
516518
name="offline_sfv_perm",
517519
types=StreamFeatureView,
518520
with_subclasses=False,
519-
policy=RoleBasedPolicy(roles=["reader", "writer"]),
521+
policy=RoleBasedPolicy(roles=["writer"]),
522+
actions=[AuthzedAction.QUERY_OFFLINE, AuthzedAction.WRITE_OFFLINE],
523+
),
524+
Permission(
525+
name="offline_fs_perm",
526+
types=FeatureService,
527+
with_subclasses=False,
528+
policy=RoleBasedPolicy(roles=["writer"]),
529+
actions=[AuthzedAction.QUERY_OFFLINE, AuthzedAction.WRITE_OFFLINE],
530+
),
531+
Permission(
532+
name="offline_filesource_perm",
533+
types=FileSource,
534+
with_subclasses=False,
535+
policy=RoleBasedPolicy(roles=["writer"]),
520536
actions=[AuthzedAction.QUERY_OFFLINE, AuthzedAction.WRITE_OFFLINE],
521537
),
522538
]

0 commit comments

Comments
 (0)