-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix for the issue when OPA throws misleading error (storage_not_found_error) #6007
Fix for the issue when OPA throws misleading error (storage_not_found_error) #6007
Conversation
@anderseknert Hi, now the DCO check is passing. |
…_error) message while loading the delta bundle when persist property in config is true. The fix is to prevent the loading of delta bundle when persist is true and give a more clear error message. Fixes open-policy-agent#5959 Signed-off-by: Yogesh Sinha <sinhayogi@gmail.com>
006bc04
to
53feeff
Compare
✅ Deploy Preview for openpolicyagent ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Hmm, the DCO check does not seem to be passing though. Not sure why, as the commit looks correct to me. @srenatus any idea? |
☝️ It's at the bottom for the DCO check, see here |
Huh, why would it expect that signoff? It's a different author 😄 |
Yup no idea. |
Also a different commit. So if push comes to shove, let's "Mark DCO as passed" using the UI. |
Done! Looks good to merge 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution @yogisinha!
18f9ef2
into
open-policy-agent:main
Fix for the issue when OPA throws misleading error (storage_not_found_error) message while loading the delta bundle when persist property in config is true.
The fix is to prevent the loading of delta bundle when persist is true and give a more clear error message.
One Test function included TestReadWithPatchPersistProperty
Fixes #5959
Why the changes in this PR are needed?
a new persist property is created in Reader struct in bundle.go
WithBundlePersistence method is created on Reader
from download , this property is set in Reader
persist property is checked in Read() method to throw the error for delta bundles.