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

Sentinel-2 pair picking can exceed the maximum lambda runtime of 900 s (15 min) #78

Closed
jhkennedy opened this issue May 8, 2024 · 1 comment · Fixed by #79
Closed
Labels
bug Something isn't working

Comments

@jhkennedy
Copy link
Contributor

jhkennedy commented May 8, 2024

In its-live-monitoring-test, processing the sentinel2-valid-2,json integration test message (shown below) pulls so many potential secondary pairs that the lambda is timing out even when setting the maximum timeout of 15 minutes.

{
  "name": "S2B_MSIL1C_20240430T142739_N0510_R139_T24VUR_20240430T162937",
  "id": "bb62af19-0e23-4f90-87be-a1e936a117db",
  "path": "products/2024/4/30/S2B_MSIL1C_20240430T142739_N0510_R139_T24VUR_20240430T162937",
  "timestamp": "2024-04-30T14:27:39.024Z",
  "datatakeIdentifier": "GS2B_20240430T142739_037345_N05.10",
  "sciHubIngestion": "2024-04-30T17:19:38.004Z",
  "s3Ingestion": "2024-04-30T18:14:57.807Z",
  "tiles": [
    {
      "path": "tiles/24/V/UR/2024/4/30/0",
      "timestamp": "2024-04-30T14:34:42.769Z",
      "utmZone": 24,
      "latitudeBand": "V",
      "gridSquare": "UR",
      "datastrip": {
        "id": "S2B_OPER_MSI_L1C_DS_2BPS_20240430T162937_S20240430T143400_N05.10",
        "path": "products/2024/4/30/S2B_MSIL1C_20240430T142739_N0510_R139_T24VUR_20240430T162937/datastrip/0"
      }
    }
  ],
  "datastrips": [
    {
      "id": "S2B_OPER_MSI_L1C_DS_2BPS_20240430T162937_S20240430T143400_N05.10",
      "path": "products/2024/4/30/S2B_MSIL1C_20240430T142739_N0510_R139_T24VUR_20240430T162937/datastrip/0"
    }
  ]
}

The test stack is much more verbose, setting the logging level to DEBUG -- it's possible that setting logging to INFO would keep us under the timeout...

But I think that this is currently a blocking issue.

@jhkennedy jhkennedy added the bug Something isn't working label May 8, 2024
@jhkennedy jhkennedy transferred this issue from ASFHyP3/hyp3-autorift May 8, 2024
@jhkennedy
Copy link
Contributor Author

Switching from DEBUG to LOG logging level in the test environment brought this down to just over 5 minutes, and sentinel2-valid-1.json ran in 1.5 minutes.

I will leave this open to monitor over time as we'll probably want to address it, but this is not a blocking issue anymore. We can monitor and determine how often this is a real problem.

Another strategy would be to reduce the search bounding box so that it doesn't overlap with neighbor tiles, significantly reducing the number of results to check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant