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

*: crash under Linux/ARM64 related to timers: "too many concurrent timer firings" #132596

Closed
rickystewart opened this issue Oct 14, 2024 · 3 comments · Fixed by #132598
Closed

*: crash under Linux/ARM64 related to timers: "too many concurrent timer firings" #132596

rickystewart opened this issue Oct 14, 2024 · 3 comments · Fixed by #132598
Labels
A-build-system branch-master Failures and bugs on the master branch. branch-release-24.3 Used to mark GA and release blockers, technical advisories, and bugs for 24.3 C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. T-dev-inf

Comments

@rickystewart
Copy link
Collaborator

rickystewart commented Oct 14, 2024 •
edited by cockroach-jira-scripts
Loading

This seems to only ever occur on ARM. The timing of the new failures correlates with the recent Go 1.23 upgrade.

#132573
#132578
#132579

Jira issue: CRDB-43172
@rickystewart rickystewart added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) A-build-system T-dev-inf labels Oct 14, 2024
@dt dt mentioned this issue Oct 14, 2024
Closed
@dt dt added the GA-blocker label Oct 14, 2024
@blathers-crl blathers[crl]
Copy link

blathers-crl bot commented Oct 14, 2024

Hi @dt, please add branch-* labels to identify which branch(es) this GA-blocker affects.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@dt dt added branch-master Failures and bugs on the master branch. branch-release-24.3 Used to mark GA and release blockers, technical advisories, and bugs for 24.3 labels Oct 14, 2024
This was referenced Oct 14, 2024
Closed
Closed
@dt dt changed the title *: spurious test failures under Linux/ARM64 related to timers: "too many concurrent timer firings" *: crash under Linux/ARM64 related to timers: "too many concurrent timer firings" Oct 14, 2024
@dt dt added release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. and removed GA-blocker labels Oct 14, 2024
@dt
Copy link
Member

dt commented Oct 14, 2024

I've bumped this to release-blocker since we've seen it outside unit tests and in roachtests as well now: #132525

@dt dt mentioned this issue Oct 14, 2024
Closed
@rickystewart rickystewart mentioned this issue Oct 14, 2024
Merged
This was referenced Oct 14, 2024
Closed
Closed
Closed
Closed
@andrewbaptist andrewbaptist mentioned this issue Oct 14, 2024
Closed
@stevendanna
Copy link
Collaborator

Another possibility: golang/go#69880

@wenyihu6 wenyihu6 mentioned this issue Oct 15, 2024
Closed
@annrpom annrpom mentioned this issue Oct 15, 2024
Closed
This was referenced Oct 15, 2024
Closed
Closed
@dt dt mentioned this issue Oct 15, 2024
Closed
@annrpom annrpom mentioned this issue Oct 15, 2024
Closed
This was referenced Oct 15, 2024
Closed
Closed
@rimadeodhar rimadeodhar mentioned this issue Oct 15, 2024
Closed
@rafiss rafiss mentioned this issue Oct 15, 2024
Closed
This was referenced Oct 15, 2024
Closed
Closed
craig bot pushed a commit that referenced this issue Oct 15, 2024
@rickystewart @jbowens
Merge #132598 #132684
9e7f03e 
132598: patched-go: update `src/runtime/time.go` r=rail a=rickystewart

Cherry-pick [this commit](https://go-review.googlesource.com/c/go/+/617497) to see if it fixes #132596.

Informs #132596

Epic: none
Release note: None

132684: storage: enable FormatFlushableIngestExcises r=itsbilal a=jbowens

Future work will bump the format major version further to FormatColumnarBlocks, but that work is blocked on a regression associated with the recent Pebble version bump.

Epic: none
Release note: none

Co-authored-by: Ricky Stewart <ricky@cockroachlabs.com>
Co-authored-by: Jackson Owens <jackson@cockroachlabs.com>
@mgartner mgartner mentioned this issue Oct 15, 2024
Closed
@craig craig bot closed this as completed in 5177b79 Oct 15, 2024
This was referenced Oct 15, 2024
Closed
Closed
This was referenced Oct 15, 2024
Closed
Closed
This was referenced Oct 15, 2024
Closed
Closed
@rharding6373 rharding6373 mentioned this issue Oct 15, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-build-system branch-master Failures and bugs on the master branch. branch-release-24.3 Used to mark GA and release blockers, technical advisories, and bugs for 24.3 C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. T-dev-inf
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

patched-go: update src/runtime/time.go rickystewart/cockroach
3 participants
@dt @stevendanna @rickystewart