nasa / earthdata-search

Earthdata Search is a web application developed by NASA EOSDIS to enable data discovery, search, comparison, visualization, and access across EOSDIS' Earth Science data holdings.
https://search.earthdata.nasa.gov
Other
743 stars 221 forks source link

EDSC-4105: As a user, I need to be able to submit an order through Earthdata Search Client that is over 2000 granules in size, and have all resulting requests be successfully submitted to Harmony for processing #1783

Closed eudoroolivares2016 closed 2 months ago

eudoroolivares2016 commented 2 months ago

Overview

What is the feature?

The issue was that for the particular order on the ticket as well as some subset of orders our memory allocation for the lambda is not sufficient. There is also an issue that the particular collection exposes where the order status was not updating in line with Harmony in cases where the state was complete_with_errors.

What is the Solution?

Increased the lambda memory allocation to the next highest class; updated the logic in the orderStatusItem to account for this potential state coming back from Harmony

What areas of the application does this impact?

Outgoing orders through Harmony via the submitHarmonyOrder lambda as well as the order state on the downloads page when the order is a Harmony order

Testing

Reproduction steps

  1. Create this order on Harmony using these parameters [/search?p=C1256358217-NSIDC_CUAT!C1256358217-NSIDC_CUAT&pg[1][v]=t&pg[1][m]=harmony0&pg[1][cd]=f&pg[1][ets]=t&pg[1][ess]=t&ee=uat&q=C1256358217-NSIDC_CUAT&sb[0]=-112.90495%2C58.01581%2C-11.09534%2C74.58281&tl=1722623855!3!!](https://search.sit.earthdata.nasa.gov/search?p=C1256358217-NSIDC_CUAT!C1256358217-NSIDC_CUAT&pg[1][v]=t&pg[1][m]=harmony0&pg[1][cd]=f&pg[1][ets]=t&pg[1][ess]=t&ee=uat&q=C1256358217-NSIDC_CUAT&sb[0]=-112.90495%2C58.01581%2C-11.09534%2C74.58281&tl=1722623855!3!!)

  2. Ensure that the order updates alongside of the orders on the Harmony workflow page https://harmony.uat.earthdata.nasa.gov/workflow-ui collection in in (UAT) and this ticket was for that

  3. Ensure that when the order(s) complete the fraction updates and we get the Harmony message

Attachments

image image

Checklist

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 83.45%. Comparing base (7d01ab5) to head (d7d3d08).

Files Patch % Lines
.../src/js/components/OrderStatus/OrderStatusItem.jsx 0.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1783 +/- ## ========================================== - Coverage 83.45% 83.45% -0.01% ========================================== Files 700 700 Lines 17503 17504 +1 Branches 4586 4583 -3 ========================================== Hits 14608 14608 - Misses 2698 2750 +52 + Partials 197 146 -51 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.