Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need an ArchiveFileCopy-listing command #248

Closed
ketiltrout opened this issue Nov 22, 2024 · 0 comments · Fixed by #261
Closed

Need an ArchiveFileCopy-listing command #248

ketiltrout opened this issue Nov 22, 2024 · 0 comments · Fixed by #261

Comments

@ketiltrout
Copy link
Member

The file list command (#247) lists ArchiveFiles, but I think there's a need for an ArchiveFileCopy-listing command as well.

The distinction is: file list only ever lists a file at most once, where this command would potentially list each copy of a file.

In #202 I propose file find for this.

It would allow listing, say, every known corrupt file, something along the lines of:

  $ alpenhorn file find --corrupt
  
  File       Node
  --------   ------
  acq/file1  node1
  acq/file1  node2
  acq/file4  node1
  acq/file8  node5
This was referenced Nov 22, 2024
ketiltrout added a commit that referenced this issue Nov 30, 2024
After "file list", this was less involved that I feared it would be.

I think there's potential to add more ways of searching for files, but
this is good for now. There are always going to be limits to how much
selection can be done with the CLI. For very complex queries, users are
going to have to access the ORM directly and make their own queries.

This is the last command I wanted to implement from the command list in
#202, unless someone can come up with an argument to promote one of the
proposed commands I suggested, which otherwise will get left on the
back-burner.

After all these command PRs are merged there is still going to be one or
more clean-up passes to do on the CLI before I'm willing to leave it
alone for the time-being.

Closes #248
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant