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.

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: Reset the module global cache on every new interpretation to avoid using outdated modules #1291

Merged
merged 1 commit into from Sep 12, 2023

Conversation

gbouv
Copy link
Contributor

@gbouv gbouv commented Sep 12, 2023

Description:

Using a single module cache lead to issues like this one b/c the interpreter was pulling the outdated cache.

Fixes #1226

Is this change user facing?

NO

References (if applicable):

Copy link
Contributor

@h4ck3rk3y h4ck3rk3y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gbouv gbouv merged commit 81c5462 into main Sep 12, 2023
29 checks passed
@gbouv gbouv deleted the gbouv/new-module-cache branch September 12, 2023 08:55
h4ck3rk3y pushed a commit that referenced this pull request Sep 14, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.82.23](0.82.22...0.82.23)
(2023-09-14)


### Features

* folks can delete enclaves from the frontend
([#1250](#1250))
([ee11b7c](ee11b7c))
* The current enclave plan is now persisted to the enclave DB every
times the execution finishes
([#1280](#1280))
([33d867e](33d867e))


### Bug Fixes

* follow logs
([#1298](#1298))
([9b0bcb7](9b0bcb7))
* Reset the module global cache on every new interpretation to avoid
using outdated modules
([#1291](#1291))
([81c5462](81c5462))
* return after stream err
([#1301](#1301))
([f40559b](f40559b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

updating image of add_service doesn't change the running service
2 participants