Skip to content

Comments

[pull] master from ruby:master#797

Merged
pull[bot] merged 6 commits intoturkdevops:masterfrom
ruby:master
Feb 21, 2026
Merged

[pull] master from ruby:master#797
pull[bot] merged 6 commits intoturkdevops:masterfrom
ruby:master

Conversation

@pull
Copy link

@pull pull bot commented Feb 21, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

byroot and others added 6 commits February 21, 2026 05:12
This test has been failing intermittently on multiple CI platforms
(Debian 13, RHEL 9/10, Ubuntu aarch64) with heap_allocated_pages
growing from 100 to 130, or major_gc_count incrementing by 1.

Failing run: https://rubyci.s3.amazonaws.com/ubuntu/ruby-master/log/20260220T001003Z.fail.html.gz

The test uses assert_separately, so test ordering is not a factor.
In failing runs sweep_slots (27,386) lands within ~1,200 of
min_free_slots (28,652) after warmup.

I've been unable to reliably replicate this.

However, when looking at the failures messages all failing runs
total_freed_pages is 0: pages are not being freed and reallocated in a
cycle. This is distinct from [Bug #18929] where pages cycled between
allocation and the tomb heap.

Since total_freed_pages == 0 already guards against the thrashing
from [Bug #18929], relax heap_allocated_pages to assert_in_epsilon
(50% tolerance) and drop major_gc_count which follows from the same
marginal threshold.
Use keyword arguments and extract repeated regexps as constant.
@pull pull bot locked and limited conversation to collaborators Feb 21, 2026
@pull pull bot added the ⤵️ pull label Feb 21, 2026
@pull pull bot merged commit a0d9b69 into turkdevops:master Feb 21, 2026
1 of 2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants