-
Notifications
You must be signed in to change notification settings - Fork 136
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
poolmanager: Fix missing access latency and retention policy on pool …
…to pool copy Motivation: Pool manager may as part of a read pool selection trigger pool to pool transfers and staging from tape. A pool persists certain meta data. Pool manager fails to ensure that in particular access latency, retention policy and checksums are included in the request message sent to the pool. The consequence is that files created on behalf of pool manager as part of a stage or pool to pool will lack these attributes. Modification: Adds the missing attributes to the list of required attributes. Further, these attributes are also made required for pool selection. This avoids that the p2p companion has to refetch the missing attributes, but more importantly this ensures that these attributes are available for staging too (staging has no ability to fetch the missing attributes itself). Result: Replicas created by a pool to pool copy will have the correct attributes. The price is that for the common case of not needing to stage or replicate a file, opening a file for read has become more expensive. Future patches may try to optimize this, but correctness is more important. Target: trunk Require-notes: yes Require-book: no Request: 2.14 Request: 2.13 Request: 2.12 Request: 2.11 Request: 2.10 Acked-by: Paul Millar <paul.millar@desy.de> Acked-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de> Patch: https://rb.dcache.org/r/8881/ (cherry picked from commit f8416b5)
- Loading branch information
Showing
4 changed files
with
21 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters