Skip to content

feat: use river for bootstrap#1863

Merged
kian99 merged 4 commits intov3from
feature/use-river-for-bootstrap
Feb 10, 2026
Merged

feat: use river for bootstrap#1863
kian99 merged 4 commits intov3from
feature/use-river-for-bootstrap

Conversation

@kian99
Copy link
Contributor

@kian99 kian99 commented Feb 10, 2026

Description

This PR is a culmination of 3 previous PRs: #1852, #1858, and #1861. We use River for the bootstrap/destroy controller background job and remove our homegrown job tracker.

Fixes JUJU-9148

Engineering checklist

  • Documentation updated
  • Covered by unit tests
  • Covered by integration tests

kian99 and others added 4 commits February 9, 2026 09:06
* feat: add bootstrap and destroy-controller river jobs

* chore: update godocs and workerParams struct

* refactor: tweak WaitForJobCompletion logic
* feat: update JIMM's bootstrap logic to queue river jobs

* chore: use TryLock with jujuCLILock to return early
@kian99 kian99 requested a review from a team as a code owner February 10, 2026 13:28
@kian99 kian99 merged commit bb4eb17 into v3 Feb 10, 2026
8 checks passed
@kian99 kian99 deleted the feature/use-river-for-bootstrap branch February 11, 2026 07:56
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.

3 participants