Skip to content

Commit 4e8ed14

Browse files
authored
update node api (#102)
1 parent c90ccac commit 4e8ed14

File tree

11 files changed

+15
-16
lines changed

11 files changed

+15
-16
lines changed

.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
- name: Set up Go
4040
uses: actions/setup-go@v4
4141
with:
42-
go-version: '1.21'
42+
go-version: '1.22'
4343

4444
- name: Build
4545
run: go build

gateway/local_apis.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ func getFileStatus(c *gin.Context) (interface{}, error) {
8383
var notFinalized bool
8484

8585
for _, client := range allClients {
86-
info, err := client.GetFileInfo(context.Background(), root)
86+
info, err := client.GetFileInfo(context.Background(), root, false)
8787
if err != nil {
8888
return nil, err
8989
}

indexer/api.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ func (api *IndexerApi) GetFileLocations(ctx context.Context, root string) (locat
6666
var txSeq uint64
6767
found := false
6868
for _, client := range trustedClients {
69-
info, err := client.GetFileInfo(ctx, hash)
69+
info, err := client.GetFileInfo(ctx, hash, true)
7070
if err != nil || info == nil {
7171
continue
7272
}

indexer/gateway/controller.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ func (ctrl *RestController) getAvailableFileLocations(ctx context.Context, cid C
5151
// find corresponding tx sequence
5252
hash := eth_common.HexToHash(cid.Root)
5353
for _, client := range ctrl.nodeManager.TrustedClients() {
54-
info, err := client.GetFileInfo(ctx, hash)
54+
info, err := client.GetFileInfo(ctx, hash, true)
5555
if err == nil && info != nil {
5656
return ctrl.fileLocationCache.GetFileLocations(ctx, info.Tx.Seq)
5757
}
@@ -130,7 +130,7 @@ func getOverallFileInfo(ctx context.Context, clients []*node.ZgsClient, cid Cid)
130130
if cid.TxSeq != nil {
131131
info, err = client.GetFileInfoByTxSeq(ctx, *cid.TxSeq)
132132
} else {
133-
info, err = client.GetFileInfo(ctx, rootHash)
133+
info, err = client.GetFileInfo(ctx, rootHash, true)
134134
}
135135

136136
if err != nil {

node/client_zgs.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,8 @@ func (c *ZgsClient) CheckFileFinalized(ctx context.Context, txSeqOrRoot TxSeqOrR
5858
}
5959

6060
// GetFileInfo Call zgs_getFileInfo RPC to get the information of a file by file data root from the node.
61-
func (c *ZgsClient) GetFileInfo(ctx context.Context, root common.Hash) (*FileInfo, error) {
62-
return providers.CallContext[*FileInfo](c, ctx, "zgs_getFileInfo", root)
61+
func (c *ZgsClient) GetFileInfo(ctx context.Context, root common.Hash, needAvailable bool) (*FileInfo, error) {
62+
return providers.CallContext[*FileInfo](c, ctx, "zgs_getFileInfo", root, needAvailable)
6363
}
6464

6565
// GetFileInfoByTxSeq Call zgs_getFileInfoByTxSeq RPC to get the information of a file by file sequence id from the node.

tests/go_tests/indexer_test/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ func runTest() error {
8989

9090
client0 := node.MustNewZgsClient(zgsNodeUrls[0])
9191
for {
92-
info, err := client0.GetFileInfo(ctx, root)
92+
info, err := client0.GetFileInfo(ctx, root, true)
9393
if err != nil {
9494
return errors.WithMessage(err, "failed to get file info")
9595
}

tests/go_tests/segment_upload_test/main.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ waitLoop:
115115
for tryN, maxTries := 0, 15; tryN < maxTries; tryN++ {
116116
time.Sleep(time.Second)
117117
for _, client := range zgsClients {
118-
info, err = client.GetFileInfo(ctx, tree.Root())
118+
info, err = client.GetFileInfo(ctx, tree.Root(), true)
119119
if err != nil {
120120
return errors.WithMessage(err, "failed to get file info")
121121
}
@@ -145,7 +145,7 @@ checkLoop:
145145
for tryN, maxTries := 0, 15; tryN < maxTries; tryN++ {
146146
time.Sleep(time.Second)
147147
for _, client := range zgsClients {
148-
info, err = client.GetFileInfo(ctx, tree.Root())
148+
info, err = client.GetFileInfo(ctx, tree.Root(), true)
149149
if err != nil {
150150
return errors.WithMessage(err, "failed to get file info")
151151
}

tests/test_framework/zgs_node.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ def zgs_download_segment_decoded(self, data_root: str, start_chunk_index: int, e
9797
return None if encodedSegment is None else base64.b64decode(encodedSegment)
9898

9999
def zgs_get_file_info(self, data_root):
100-
return self.rpc.zgs_getFileInfo([data_root])
100+
return self.rpc.zgs_getFileInfo([data_root, True])
101101

102102
def zgs_get_file_info_by_tx_seq(self, tx_seq):
103103
return self.rpc.zgs_getFileInfoByTxSeq([tx_seq])

tests/utility/build_binary.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ def build_cli(dir: str) -> BuildBinaryResult:
9090
dir=dir,
9191
binary_name=CLIENT_BINARY,
9292
github_url="https://github.com/0glabs/0g-storage-client.git",
93-
git_rev=CLI_GIT_REV,
9493
build_cmd="go build",
9594
compiled_relative_path=[],
9695
)

transfer/downloader.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ func (downloader *Downloader) Download(ctx context.Context, root, filename strin
117117
func (downloader *Downloader) queryFile(ctx context.Context, root common.Hash) (info *node.FileInfo, err error) {
118118
// do not require file finalized
119119
for _, v := range downloader.clients {
120-
info, err = v.GetFileInfo(ctx, root)
120+
info, err = v.GetFileInfo(ctx, root, true)
121121
if err != nil {
122122
return nil, err
123123
}

0 commit comments

Comments
 (0)