Migrate `renterd` subscriber to `coreutils` types #1098

peterjan · 2024-03-25T10:04:06Z

This PR updates core and coreutils and splits the subscriber logic off into its own package called chain. This PR is the last step in migrating to the coreutils types and the new subscriber pattern.

chain/subscriber.go

internal/test/e2e/cluster_test.go

stores/sql.go

chain/subscriber.go

config/config.go

chain/update.go

stores/sql.go

peterjan · 2024-03-28T17:43:05Z

Marked as DRAFT for the time being, ran into NDF in UpdateElementProof not sure what's up, don't have it locally. Might have to wait until wed because I'm OOO.

chain/subscriber.go

chain/update.go

… store

chain/subscriber.go

… pj/subscription-api

.golangci.yml

autopilot/contractor/contractor.go

bus/bus.go

cmd/renterd/main.go

stores/metrics.go

chain/subscriber.go

… pj/subscription-api

peterjan added 3 commits March 25, 2024 10:37

chain: introduce subscriber

a1ba198

stores: add TestApplyChainUpdate

d2fbe63

bus: fix interface

fb39286

peterjan self-assigned this Mar 25, 2024

peterjan added 3 commits March 25, 2024 20:01

testing: fix TestWalletTransactions

a8560e5

subscriber: commit update on close

9358313

test: fix TestUploadDownloadSpending

8e91afd

peterjan marked this pull request as ready for review March 25, 2024 21:01

peterjan requested a review from ChrisSchinnerl March 25, 2024 21:01

ChrisSchinnerl reviewed Mar 26, 2024

View reviewed changes

ChrisSchinnerl requested a review from n8maninger March 26, 2024 10:50

peterjan added 3 commits March 26, 2024 14:21

stores: update interface

4735104

chain: update subscriber

0e80949

testing: remove TODOs

c6a1898

peterjan requested a review from ChrisSchinnerl March 27, 2024 08:48

ChrisSchinnerl reviewed Mar 27, 2024

View reviewed changes

chain/subscriber.go Outdated Show resolved Hide resolved

chain/subscriber.go Outdated Show resolved Hide resolved

subscriber: get rid of OnReorg

de5402e

ChrisSchinnerl reviewed Mar 27, 2024

View reviewed changes

config/config.go Outdated Show resolved Hide resolved

peterjan added 2 commits March 27, 2024 18:22

subscriber: re-add OnReorg

0b012bc

subscriber: get rid of sync interval entirely

452c39e

peterjan requested a review from ChrisSchinnerl March 28, 2024 08:09

ChrisSchinnerl reviewed Mar 28, 2024

View reviewed changes

chain/update.go Outdated Show resolved Hide resolved

chain/update.go Outdated Show resolved Hide resolved

stores/sql.go Show resolved Hide resolved

chain: handle state elements

ad998b4

peterjan marked this pull request as draft March 28, 2024 17:41

ChrisSchinnerl requested changes Apr 3, 2024

View reviewed changes

chain/subscriber.go Outdated Show resolved Hide resolved

chain/update.go Outdated Show resolved Hide resolved

chain/update.go Outdated Show resolved Hide resolved

chain: get rid of update in favour of exposing a transaction from the…

ba97b55

… store

peterjan commented Apr 5, 2024

View reviewed changes

chain/subscriber.go Show resolved Hide resolved

peterjan added 2 commits April 5, 2024 15:52

stores: remove chain test

1acaabd

worker: undo change

13d5415

peterjan added 4 commits May 3, 2024 16:32

contractor: refresh contract metadata

8fc37a0

all: cleanup PR

85bf378

Merge branch 'its-happening' of github.com:SiaFoundation/renterd into…

893501e

… pj/subscription-api

ci: tmp disable stores test

fae31b0

peterjan changed the title ~~Integrate new subscription api~~ Migrate to coreutils May 3, 2024

chain: update subscriber

5532b8e

peterjan changed the title ~~Migrate to coreutils~~ Migrate to coreutils types May 3, 2024

peterjan changed the title ~~Migrate to coreutils types~~ Migrate consensus to coreutils types May 3, 2024

stores: add logging

7485ceb

peterjan changed the title ~~Migrate consensus to coreutils types~~ Migrate renterd subscriber to coreutils types May 3, 2024

peterjan added 5 commits May 3, 2024 20:25

stors: add logging

42ca8b2

bus: fix store contract update, add sync height

fbb6b5a

lint: disable builtinShadow

5366be3

ci: re-enable store tests

00ffb04

test: block after mining

9671cc0

ChrisSchinnerl reviewed May 6, 2024

View reviewed changes

peterjan added 2 commits May 7, 2024 11:45

test: upgrade host

7c6d1a0

Merge branch 'its-happening' of github.com:SiaFoundation/renterd into…

98940a7

… pj/subscription-api

peterjan force-pushed the pj/subscription-api branch from 450cde6 to db04139 Compare May 7, 2024 13:30

all: implement CR remarks

783f480

peterjan force-pushed the pj/subscription-api branch from db04139 to 783f480 Compare May 7, 2024 13:31

peterjan marked this pull request as ready for review May 7, 2024 13:56

peterjan requested a review from ChrisSchinnerl May 8, 2024 12:20

peterjan added 3 commits May 10, 2024 12:34

testing: always mine and sync

accc3a1

chain: fix chain.db location and pass genesis block id

620c9f7

bus: use maxSyncTime to decide whether the bus is synced or not

4560593

ChrisSchinnerl approved these changes May 14, 2024

View reviewed changes

ChrisSchinnerl merged commit da17f0d into its-happening May 14, 2024
7 checks passed

ChrisSchinnerl deleted the pj/subscription-api branch May 14, 2024 12:34

github-actions bot mentioned this pull request Dec 4, 2024

renterd: v1.1.0 SiaFoundation/desktop#98

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Migrate `renterd` subscriber to `coreutils` types #1098

Migrate `renterd` subscriber to `coreutils` types #1098

peterjan commented Mar 25, 2024 •

edited

Loading

peterjan commented Mar 28, 2024

Migrate renterd subscriber to coreutils types #1098

Migrate renterd subscriber to coreutils types #1098

Conversation

peterjan commented Mar 25, 2024 • edited Loading

peterjan commented Mar 28, 2024

Migrate `renterd` subscriber to `coreutils` types #1098

Migrate `renterd` subscriber to `coreutils` types #1098

peterjan commented Mar 25, 2024 •

edited

Loading