-
Notifications
You must be signed in to change notification settings - Fork 2
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
ref_lid_collection error when ingesting data sets #57
Comments
{ "update" : {"_id" : "urn:nasa:pds:epoxi_mri::1.0" } }
{ "doc" : {"ops:Tracking_Meta/ops:archive_status" : "archived"} } EDIT: TIL about ndJSON |
General guess, or is there a specific release or related issue you're thinking of? |
Bug appears to be triggering due to the relevant bundle containing only one collection. The response returned by the API request during {
"_index": "registry",
"_type": "_doc",
"_id": "urn:nasa:pds:epoxi_mri::1.0",
"_version": 2,
"_seq_no": 27012,
"_primary_term": 3,
"found": true,
"_source": {
"ref_lid_collection": "urn:nasa:pds:epoxi_mri:hartley2_photometry"
}
} with @jordanpadams this appears to be the fault of the API. Will follow up when I identify the relevant next steps. |
Confirmed that the same request for a bundle with multiple collections does not have this issue and correctly returns an array of strings. {
"_index": "registry",
"_type": "_doc",
"_id": "urn:nasa:pds:dart_teleobs::1.0",
"_version": 1,
"_seq_no": 24385,
"_primary_term": 3,
"found": true,
"_source": {
"ref_lidvid_collection": [
"urn:nasa:pds:dart_teleobs:data_ldtcal::1.0",
"urn:nasa:pds:dart_teleobs:data_ldtddp::1.0",
"urn:nasa:pds:dart_teleobs:data_ldtraw::1.0",
"urn:nasa:pds:dart_teleobs:document_ldt::1.0"
],
"ref_lid_collection": [
"urn:nasa:pds:dart_teleobs:data_ldtcal",
"urn:nasa:pds:dart_teleobs:data_ldtddp",
"urn:nasa:pds:dart_teleobs:data_ldtraw",
"urn:nasa:pds:dart_teleobs:document_ldt"
]
}
} |
@jordanpadams @al-niessner @jimmie @nutjob4life is it |
@alexdunnjpl Harvest is the tool for doing this. we discussed merging registry-mgr and harvest into that registry-loader repo, but it has not happened. |
Thanks Jordan - should be able to continue with this now. |
Oops! Nevermind. deleted previous comment because I was definitely wrong. Hopefully the harvest fix will make it work. |
Closing as this is a |
π Describe the bug
π To Reproduce
Steps to reproduce the behavior:
π΅οΈ Expected behavior
Change archive status for the collection
π Version of Software Used
π©Ί Test Data / Additional context
πScreenshots
π₯ System Info
π¦ Related requirements
βοΈ Engineering Details
Initial guess for this error is because the first product ingested only included 1 collection reference, so harvest/registry manager created the schema as a string value instead of an array.
Duplicate of NASA-PDS/registry#118 but keeping open to ensure traceability.
The text was updated successfully, but these errors were encountered: