Repository navigation
hotfix: migrate EPIC data endpoints from dtn-eic to dtn2304 - #139
Conversation
Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>
Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>
Co-authored-by: wdconinc <4656391+wdconinc@users.noreply.github.com>
wdconinc
changed the title
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The updated sed mapping in docs/_data/list_S3.sh emits a duplicated EPIC path segment, so regenerated manifests would not match the migrated endpoint layout.
Review effort: Balanced
Findings: 1
Open (1)
What changed in this PR
This hotfix migrates all documented EPIC dataset access endpoints from the retired dtn-eic.jlab.org host (implicit port 1094) to dtn2304.jlab.org:8443, and reworks the XRootD path layout accordingly (/volatile/eic/EPIC/... → /jlab-osdf-ro/eic/EPIC/volatile/... and /work/eic2/EPIC/... → /jlab-osdf-ro/eic/EPIC/work/eic2/...). The change is documentation-focused (generated index.md manifests, blog-style train summaries, and the FAQ) plus the generator script that produces those manifests, keeping published access instructions pointing at the live endpoint.
Changes:
- Replaced the host/port in every ROOT/XRootD URL across
docs/RECO/*,docs/FULL/*, anddocs/_posts/*manifests. - Updated the FAQ
xrdfs/ls/TFile::Open/xrdcpexamples to the new host and path layout, and fixed the "constructs" typo in the generator script. - Updated the
list_S3.shgenerator'ssedmapping to emit the new endpoint/path form (contains a path-duplication defect, noted below).
| File | Description |
|---|---|
docs/RECO/*/index.md (incl. *_manifest.txt, transfer_manifest.py, alternative-geometries-img-ecal) |
Migrated volatile/work/eic2 ROOT URL prefixes to the new host/path form |
docs/FULL/*/index.md |
Migrated work/eic2 ROOT URL prefixes to the new host/path form |
docs/_posts/2023-*/2024-01-31-train-summary.md |
Updated work/eic2 ROOT URL prefixes in campaign summaries |
docs/_documentation/faq.md |
Updated interactive examples (host, port, paths) and trimmed a trailing blank line |
docs/_data/list_S3.sh |
Updated generator sed mapping and fixed "contructs"→"constructs"; mapping now duplicates the EPIC path segment |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This hotfix updates EPIC dataset access references from
dtn-eictodtn2304, including the required path migration from/volatile/eic/EPICto/jlab-osdf-ro/eic/EPIC/volatile. Scope is repository-wide where these endpoint forms are documented or generated.Endpoint migration across docs
dtn-eichost references withdtn2304.jlab.org:8443.root://dtn2304.jlab.org:8443//jlab-osdf-ro/eic/EPIC/volatile/...root://dtn2304.jlab.org:8443//jlab-osdf-ro/eic/EPIC/work/eic2/...FAQ command/path alignment
xrdfs/lsexamples to usedtn2304:8443and/jlab-osdf-ro/eic/EPIC/volatile/....Generator script alignment
docs/_data/list_S3.shoutput URL mapping so generated index content uses the new endpoint/path convention.