Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Include etag with rest of bundle metadata #4544

Closed
tsandall opened this issue Apr 4, 2022 · 0 comments · Fixed by #4578
Closed

Include etag with rest of bundle metadata #4544

tsandall opened this issue Apr 4, 2022 · 0 comments · Fixed by #4578
Assignees
@ashutosh-narkar
Labels
distribution Issues related to the bundle plugin

Comments

@tsandall
Copy link
Member

tsandall commented Apr 4, 2022

Currently the etag from the HTTP response of activated bundles is not retained once the bundle activates. As a result, when the disk-store is used, OPA will re-download and activate a bundle following a process restart, even if the store contents are up-to-date. We could avoid this potentially expensive operation by including the etag in the persisted bundle metadata and initializing the downloader accordingly on startup.
@tsandall tsandall added the distribution Issues related to the bundle plugin label Apr 4, 2022
@ashutosh-narkar ashutosh-narkar self-assigned this Apr 7, 2022
@ashutosh-narkar ashutosh-narkar mentioned this issue Apr 13, 2022
Merged
ashutosh-narkar added a commit to ashutosh-narkar/opa that referenced this issue Apr 13, 2022
@ashutosh-narkar
Persist activated bundle etag to store
406a304 
Currently etag from the HTTP response of activated bundles is not
persisted to store. Hence if OPA restarts and an activated bundle
loaded from the disk store is up-to-date, OPA may still download
the same version of the bundle and activate it. With this change,
OPA should include the right etag in the bundle download request
thereby avoiding unnecessary bundle download and activation.

Fixes: open-policy-agent#4544

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
ashutosh-narkar added a commit that referenced this issue Apr 13, 2022
@ashutosh-narkar
Persist activated bundle etag to store
ccba4a6 
Currently etag from the HTTP response of activated bundles is not
persisted to store. Hence if OPA restarts and an activated bundle
loaded from the disk store is up-to-date, OPA may still download
the same version of the bundle and activate it. With this change,
OPA should include the right etag in the bundle download request
thereby avoiding unnecessary bundle download and activation.

Fixes: #4544

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
rokkiter pushed a commit to rokkiter/opa that referenced this issue Apr 18, 2022
@ashutosh-narkar @rokkiter
Persist activated bundle etag to store
4a12289 
Currently etag from the HTTP response of activated bundles is not
persisted to store. Hence if OPA restarts and an activated bundle
loaded from the disk store is up-to-date, OPA may still download
the same version of the bundle and activate it. With this change,
OPA should include the right etag in the bundle download request
thereby avoiding unnecessary bundle download and activation.

Fixes: open-policy-agent#4544

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ashutosh-narkar ashutosh-narkar

Labels
distribution Issues related to the bundle plugin
Projects
Open Policy Agent
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Persist activated bundle etag to store ashutosh-narkar/opa
2 participants
@tsandall @ashutosh-narkar