Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

[Backport release-0.7] fix: lease-acquisition loop can fail with temporary grpc issues#762

Merged
mangelajo merged 1 commit intorelease-0.7from
backport-761-to-release-0.7
Dec 2, 2025
Merged

[Backport release-0.7] fix: lease-acquisition loop can fail with temporary grpc issues#762
mangelajo merged 1 commit intorelease-0.7from
backport-761-to-release-0.7

Conversation

@jumpstarter-backport-bot
Copy link
Copy Markdown

Bot-based backport to release-0.7, triggered by a label in #761.

a lease acquisition loop can fail in the following way very easily if a router
in openshift/k8s is restarted during the lease get operation.

this retry mechanism avoids this issue.

```
2025-11-28T15:22:39.557321889Z for lease: There are 9 approved
2025-11-28T15:22:39.557321889Z exporters, (i.e. ) but all of them are
2025-11-28T15:22:39.557321889Z already leased (7:30:58)
2025-11-28T15:22:44.577566967Z [11/28/25 15:22:44] INFO INFO:jumpstarter.client.lease:Waiting lease.py:355
2025-11-28T15:22:44.577566967Z for lease: There are 9 approved
2025-11-28T15:22:44.577566967Z exporters, (i.e. ) but all of them are
2025-11-28T15:22:44.577566967Z already leased (7:31:03)
2025-11-28T15:22:56.436248186Z Error: grpc error: Socket closed
```

(cherry picked from commit 78ad580)
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Dec 2, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@mangelajo mangelajo enabled auto-merge December 2, 2025 08:33
@mangelajo mangelajo merged commit f95dc68 into release-0.7 Dec 2, 2025
14 checks passed
@mangelajo mangelajo deleted the backport-761-to-release-0.7 branch December 2, 2025 08:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant