Project

General

Profile

Actions

Feature #21978

closed

Clarify replace_files API docs

Added by Tom Clegg about 2 months ago. Updated about 1 month ago.

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

Description

From #21701#note-17:

The documentation says a valid source is:

<PDH>/<path> where <PDH> is the portable data hash of a collection on the cluster

I assume "on the cluster" means I can't refer to collections on other clusters in a federation. But I think you have to be pretty deep in Arvados, and read very carefully, to recognize that. If I followed right, should we specifically call this out under the "Usage restrictions" section?

Does this limitation also apply to PDHs in the manifest_text? If so, that seems worth calling out too.

Under the update method header, the description for replace_files reads:

Delete and replace files and directories using content from other collections

With these changes this seems to narrow. The argument can do more than "delete and replace files and directories," and those operations can work on multiple data sources, not just "other collections." It would be nice to broaden this. I also recognize that it's a challenge to do this without making it almost uselessly generic. "Update this collection's contents by referencing collection manifests" maybe?


Subtasks 1 (0 open1 closed)

Task #21979: Review 21978-replace-files-docsResolvedTom Clegg07/22/2024Actions

Related issues

Related to Arvados - Feature #21701: replace_files requests can refer to "current version" and "provided manifest_text" as sourcesResolvedTom CleggActions
Actions

Also available in: Atom PDF