Arvados

Some comments & questions:

• Do we need more testing on this command?
• There’re more references to “pipeline” on argument descriptions, and code comments.
• It seems there’s more pipeline related code:
  • Line 273: migrate_components_filters()
  • Line 410: migrate_jobspec()
  • Line 452: copy_git_repos()
  • Line 754: copy_docker_images() (it looks for a components key, so it seems pipeline-specific)
• Is the whilelisted portable_data_hash field necessary? Isn’t the PDH computed from others?

16138-fix-arv-copy @ 20e09ab0726d7f0edb4333277c3403f995b960ce

I've tested it manually with 2 arvbox instances, it worked OK.
There're some more pipeline references that may need cleaning, other than that, it LGTM.

$ grep -in pipeline ./sdk/python/build/lib/arvados/commands/arv_copy.py
103:        help='The name of the destination git repository. Required when copying a pipeline recursively.')
106:        help='The UUID of the project at the destination to which the pipeline should be copied.')
121:        description='Copy a pipeline instance, template, workflow, or collection from one Arvados instance to another.',
471:            # as the root item (usually a pipeline) being copied.
710:#    It returns a string such as 'Collection', 'PipelineInstance', etc.