Project

General

Profile

Actions

Bug #16727

closed

[FUSE] [cgofuse] Refresh signatures / reload collection instead of using expired blob signatures

Added by Tom Clegg over 3 years ago. Updated about 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Story points:
-
Release relationship:
Auto

Subtasks 1 (0 open1 closed)

Task #18652: Review 16727-refresh-signaturesResolvedLucas Di Pentima01/27/2022Actions

Related issues

Related to Arvados Epics - Idea #16082: Port client tools to GoNewActions
Related to Arvados - Idea #12308: [FUSE] Golang-based fuse driverIn ProgressTom CleggActions
Related to Arvados Epics - Idea #17849: FUSE driver v2NewActions
Has duplicate Arvados - Bug #18391: [go] FileSystem object does not refresh signatures resulting in read errorsResolvedActions
Actions #1

Updated by Tom Clegg over 3 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Tom Clegg over 3 years ago

  • Related to Idea #16082: Port client tools to Go added
Actions #3

Updated by Tom Clegg over 3 years ago

  • Related to Idea #12308: [FUSE] Golang-based fuse driver added
Actions #4

Updated by Peter Amstutz over 2 years ago

Actions #5

Updated by Peter Amstutz over 2 years ago

  • Target version deleted (To Be Groomed)
Actions #6

Updated by Peter Amstutz over 2 years ago

  • Has duplicate Bug #18391: [go] FileSystem object does not refresh signatures resulting in read errors added
Actions #7

Updated by Peter Amstutz over 2 years ago

  • Status changed from In Progress to New
Actions #8

Updated by Peter Amstutz about 2 years ago

  • Release set to 46
  • Target version set to 2022-02-02 sprint
Actions #10

Updated by Tom Clegg about 2 years ago

  • Status changed from New to In Progress
Actions #11

Updated by Lucas Di Pentima about 2 years ago

Just one question:

  • Is collectionFileSystem.guessSignatureTTL just an accessory struct member to be able to test it? If that's the case, why assigning ttl's value to it? I think that may need some clarifying comment.

The rest LGTM

Actions #12

Updated by Tom Clegg about 2 years ago

16727-refresh-signatures @ 9f442d51f9ee07be3b25e461847bae58fe1cd126

Added some comments about guessSignatureTTL -- is that better?

Actions #13

Updated by Lucas Di Pentima about 2 years ago

Thanks, I understand better now. LGTM.

Actions #14

Updated by Tom Clegg about 2 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados-private:commit:arvados|4cf58c672f91a0e4fec3ea3d4a1934c01b1fe214.

Actions

Also available in: Atom PDF