test: Raise limit for memory tests #3647
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, two of the memory tests are failing with slightly more than 20M used:
I repeated this a few times - no chance. It always fails the same. It also fails when I run this test locally. And it also fails when I checkout older commits, before the merges I made yesterday and run the tests there.
The memory tests for all commits on main have passed before the PRs were merged. The pipelines on main didn't run through for b598b59 and ee4bfbf - in fact they are still in an odd pending queued state for many hours. The last time that the memory test was run in CI was for 0dc1345 in https://github.com/PostgREST/postgrest/actions/runs/9842378222. All good.
Literally the only change since the last time the memory tests ran successfully is... time. 1-2 days have passed and now the memory tests fail consistently.
That's quite odd, but I can't figure out why. So just bumping the memory limits for those tests for now.