Include LINC API as a supported instance type for DANDI CLI

[aaronkanzer]

Relates to #1517
Follow-up of #1519 -- used a different fork since tags were not copied over in the prior fork in #1519

Cc @kabilar

Steps to properly test:

1. pip uninstall dandi -y to remove DANDI CLI locally (can use a venv too if desired)
2. pip install -e git+https://github.com/lincbrain/dandi-cli.git@ak-linc#egg=dandi -- reference my forked branch here as the dandi CLI version
3. Made a Dandiset on LINC: https://lincbrain.org/dandiset/000045
4. Run dandi download https://lincbrain.org/dandiset/000045/draft -- the CLI should prompt for an API value here
5. I copied over an NWB into the 000045 folder
6. Run dandi upload -i linc
7. Observe files present in https://lincbrain.org/dandiset/000045/draft/files?location=

Prior to running any steps, I ran dandi instances to see that the proper values were listed

@jwodder @yarikoptic -- mind a brief re-review here?

I think we should be fine here -- it seems that sometimes the CI tests for Ubuntu can be flaky -- you'd know better than I if this is a common issue over time -- I know sometimes GitHub runners can be unpredictable with longer, parallel jobs

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.65%. Comparing base (5066cca) to head (284ead1).
Report is 23 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1527      +/-   ##
==========================================
- Coverage   88.57%   87.65%   -0.92%     
==========================================
  Files          78       78              
  Lines       10589    10591       +2     
==========================================
- Hits         9379     9284      -95     
- Misses       1210     1307      +97     

Flag	Coverage Δ
unittests	87.65% <100.00%> (-0.92%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kabilar]

Thank you, @aaronkanzer. This looks good to me. I tested these changes using your instructions with https://lincbrain.org/dandiset/000044/draft. I can verify that I was only able to download and upload when using the correct lincbrain.org API key.

Let's go ahead and include updates to the docs in this pull request. See notes here: #1517 (comment).

[yarikoptic]

unfortunately we have some "manually" curated dumps of --help for commands and thus need to extend with this new instances the docs/source/cmdline/instances.rst file. Please add listing there too

[aaronkanzer]

unfortunately we have some "manually" curated dumps of --help for commands and thus need to extend with this new instances the docs/source/cmdline/instances.rst file. Please add listing there too

Updated!

[aaronkanzer]

@yarikoptic @jwodder -- did this not get fully released to PyPI? Just noticing that there is no new patch version at https://pypi.org/project/dandi/#history

[jwodder]

@aaronkanzer The PR didn't have the "release" label, so no release was triggered.

[aaronkanzer]

@jwodder -- would the preferred route to release then be that I push a trivial change with those labels then?

[yarikoptic]

merged

• download: more consistent and exhaustive logging, new DANDI_DEVEL_AGGRESSIVE_RETRY mode, respect (?) Retry-After #1509

with release... release should come shortly

[github-actions]

🚀 PR was released in 0.64.0 🚀