Skip to content

Conversation

@bergundy
Copy link
Member

@bergundy bergundy commented Jan 15, 2026

What changed?

Reapplied #9010.

The original PR that introduced these timeouts did not populate the operation-timeout header or set the call context timeout correctly. This PR fixes the logic.

@bergundy bergundy requested review from a team as code owners January 15, 2026 00:45
@bergundy bergundy force-pushed the fix-nexus-new-timeouts-logic branch 2 times, most recently from 758b03d to 9d4a35c Compare January 15, 2026 18:42
@bergundy bergundy force-pushed the fix-nexus-new-timeouts-logic branch from 880c153 to 2f4c0eb Compare January 15, 2026 23:56
@bergundy bergundy changed the title Fix Nexus new timeouts logic Nexus new timeouts (with fixes) Jan 15, 2026
@bergundy bergundy changed the title Nexus new timeouts (with fixes) Nexus schedule to start and start to close timeouts (with fixes) Jan 15, 2026
@bergundy bergundy merged commit 58449b9 into temporalio:main Jan 16, 2026
69 checks passed
@bergundy bergundy deleted the fix-nexus-new-timeouts-logic branch January 16, 2026 01:09
simvlad added a commit that referenced this pull request Jan 17, 2026
simvlad added a commit that referenced this pull request Jan 17, 2026
…s) (#9033) (#9072)

This reverts commit 58449b9.

## What changed?
This reverts Nexus schedule to start and start to close timeouts (with
fixes) (#9033)

## Why?
Causes failures in nightly pipeline

## How did you test it?
- [X] built
- [ ] run locally and tested manually
- [ ] covered by existing tests
- [ ] added new unit test(s)
- [ ] added new functional test(s)
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