Expected Behavior
We have started using feast (v0.40.0) to create a feature registry on S3. In a script to fetch data from the offline store the feature store is instantiated with configuration from the default config file. Afterwards, data can be fetched in the script. The registry is supposed to be changed only by dedicated users.
Current Behavior
The script fails with AccessDenied error because it tries to change the registry on S3 but there are no sufficient access credentials configured.
Essentially, feast fails at
which was added in
#4475 .
I do not understand the complete logic but why do we make a commit to the registry when we only want to use the existing registry? Should we have a flag here to disable it in cases where we only want to read the registry, like fetching data or feast plan?
Steps to reproduce
Call fs = FeatureStore() with existing feature store config and S3 registry without configuring PutObject policies for current user.
Specifications
- Version: v0.41.3
- Platform: MacOs
- Subsystem:
Possible Solution
Expected Behavior
We have started using feast (v0.40.0) to create a feature registry on S3. In a script to fetch data from the offline store the feature store is instantiated with configuration from the default config file. Afterwards, data can be fetched in the script. The registry is supposed to be changed only by dedicated users.
Current Behavior
The script fails with AccessDenied error because it tries to change the registry on S3 but there are no sufficient access credentials configured.
Essentially, feast fails at
feast/sdk/python/feast/infra/registry/registry.py
Line 232 in 4a6b663
I do not understand the complete logic but why do we make a commit to the registry when we only want to use the existing registry? Should we have a flag here to disable it in cases where we only want to read the registry, like fetching data or
feast plan?Steps to reproduce
Call
fs = FeatureStore()with existing feature store config and S3 registry without configuring PutObject policies for current user.Specifications
Possible Solution