Skip to content

Ignore publish version code hash if hot-reloading is active#8004

Merged
dfangl merged 1 commit intomasterfrom
hot-reloading-ignore-hash
Mar 29, 2023
Merged

Ignore publish version code hash if hot-reloading is active#8004
dfangl merged 1 commit intomasterfrom
hot-reloading-ignore-hash

Conversation

@dfangl
Copy link
Member

@dfangl dfangl commented Mar 29, 2023

Motivation

Some tools, for example serverless, seem to send the local code hash with them when using publish version. This comparison will fail, since hot-reloading code does not have a proper hash (but a placeholder string).

Related comment: localstack/serverless-localstack#215 (comment)

Changes

  • Skip code hash verification if code is hot reloading + test
  • Sneaked in a opensearch test update to make sure the cluster is properly terminated after the test

…e for opensearch test to avoid resource leak
@dfangl dfangl requested a review from dominikschubert as a code owner March 29, 2023 14:46
Copy link
Member

@dominikschubert dominikschubert left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@github-actions
Copy link

LocalStack integration with Pro

1 868 tests  +1   1 677 ✔️ +2   1h 41m 2s ⏱️ + 18m 22s
       1 suites ±0      191 💤  - 1 
       1 files   ±0          0 ±0 

Results for commit 355c2bd. ± Comparison against base commit 6dd1c41.

@dfangl dfangl merged commit 7666017 into master Mar 29, 2023
@dfangl dfangl deleted the hot-reloading-ignore-hash branch March 29, 2023 16:50
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.

2 participants