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

Refactor worker to derive its own account for each host #1464

Merged
merged 52 commits into from
Aug 28, 2024
Merged

Refactor worker to derive its own account for each host #1464

merged 52 commits into from
Aug 28, 2024

Conversation

ChrisSchinnerl
Copy link
Member

@ChrisSchinnerl ChrisSchinnerl commented Aug 20, 2024 •
edited
Loading

Closes #1462

@ChrisSchinnerl ChrisSchinnerl self-assigned this Aug 20, 2024
Base automatically changed from chris/rhp3-client to dev August 20, 2024 16:37
@ChrisSchinnerl ChrisSchinnerl force-pushed the chris/accounts branch 2 times, most recently from a382ef2 to 45b2a76 Compare August 21, 2024 09:09
@ChrisSchinnerl ChrisSchinnerl changed the base branch from dev to v2 August 21, 2024 09:23
@ChrisSchinnerl ChrisSchinnerl mentioned this pull request Aug 22, 2024
Closed
@ChrisSchinnerl
Add contract formation endpoint to bus (#1463)
460771d 
This PR moves contract formation to the bus and gets rid of routes that
became redundant because of the move. I want to draw attention to the
route I added which is `POST /contracts`, I originally had `POST
/rhp/form` but I think now is a good time to reconsider having `rhp` in
the route. There's really no need to expose that to someone using the
API.

Fixes #1456
@ChrisSchinnerl
Merge branch 'dev' into v2
6b53638
@ChrisSchinnerl
bus: make sure formed contracts are added to the worker cache
1a36a8b
@ChrisSchinnerl
worker: AccountMgr
7638e49
@ChrisSchinnerl
bus: remove accounts
e208dfd
@ChrisSchinnerl
bus: remove AccountManager
fc177fc
@ChrisSchinnerl
worker: fix TestDownloaderStopped
349e1ce
@ChrisSchinnerl
worker: update locking
e891bb7
@ChrisSchinnerl
worker: fix TestUploadSingleSectorSlowHosts
0845ed2
@ChrisSchinnerl
bus: fix 'owner'
e97d007
@ChrisSchinnerl
bus: remove unused request type
6e7d234
@ChrisSchinnerl
worker: deep-copy in Account
65299ee
@ChrisSchinnerl
worker: convert helper
96275cb
@ChrisSchinnerl
worker: TestEphemeralAccounts
42cad74
@ChrisSchinnerl
worker: add accounts_test.go
5857876
@ChrisSchinnerl ChrisSchinnerl changed the base branch from v2 to chris/form-contract-cache August 22, 2024 08:59
peterjan
peterjan reviewed Aug 26, 2024
View reviewed changes
bus/client/accounts.go Outdated Show resolved Hide resolved
Base automatically changed from v2 to dev August 26, 2024 14:14
peterjan
peterjan reviewed Aug 26, 2024
View reviewed changes
api/bus.go Outdated Show resolved Hide resolved
peterjan
peterjan reviewed Aug 28, 2024
View reviewed changes
stores/sql/main.go Outdated Show resolved Hide resolved
@ChrisSchinnerl ChrisSchinnerl merged commit b9ec530 into dev Aug 28, 2024
7 checks passed
@ChrisSchinnerl ChrisSchinnerl deleted the chris/accounts branch August 28, 2024 08:58
@ChrisSchinnerl
Copy link
Member Author

@alexfreska this PR requires updating the UI. reset drift is now under worker and not bus anymore.

@alexfreska alexfreska mentioned this pull request Sep 11, 2024
Draft
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peterjan peterjan peterjan approved these changes

Assignees

@ChrisSchinnerl ChrisSchinnerl

Labels
None yet
Projects
Sia
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Give every worker its own ephemeral account
2 participants
@ChrisSchinnerl @peterjan