Project

General

Profile

Actions

Bug #7207

closed

[SDKs] Remove outdated `arv keep less` and `arv keep check` subcommands

Added by Joshua Randall over 10 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
-
Category:
SDKs
Target version:
-
Story points:
0.5

Description

Not sure what they are supposed to do (I was just test running all subcommands to make sure I had manually installed the CLI correctly), but I can't seem to figure out how to get `arv keep less` and `arv keep check` to work.

They both fail in "check_subcommands()" at line 105:

elsif ['less', 'check'].index @sub then
  1. wh* shims
    exec `which wh#{@sub}`.strip, *remaining_opts

Not sure what the 'wh* shims' are, but I don't have them and am not sure where to find them.

Actions #1

Updated by Brett Smith over 10 years ago

  • Target version set to Arvados Future Sprints
Actions #2

Updated by Brett Smith over 10 years ago

  • Subject changed from `arv keep less` and `arv keep check` (wh*) subcommands don't exist? to [SDKs] `arv keep less` and `arv keep check` (wh*) subcommands don't exist?
  • Category set to SDKs
Actions #3

Updated by Brett Smith over 10 years ago

We don't maintain the actual commands anymore, so these subcommands in the arv wrapper are outdated references that should be removed.

arv keep less does exactly what you think: it's just arv keep get ... | less.

arv keep check went through a collection manifest, and sent a HEAD request to every Keep service on every block in it to confirm that all of the data was still stored in Keep. Our Data Manager component is going to do similar work at a higher level, though we understand that users might still be interested in manually running this kind of tool.

Actions #4

Updated by Brett Smith over 10 years ago

  • Subject changed from [SDKs] `arv keep less` and `arv keep check` (wh*) subcommands don't exist? to [SDKs] Remove outdated `arv keep less` and `arv keep check` subcommands
  • Story points set to 0.5
Actions #5

Updated by Brett Smith over 10 years ago

  • Target version deleted (Arvados Future Sprints)
Actions #6

Updated by Brett Smith over 10 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:3cbdb14acde0092ae4f33e41a4bf10c6a1fce052.

Actions

Also available in: Atom PDF