COPRS / rs-issues

This repository contains all the issues of the COPRS project (Scrum tickets, ivv bugs, epics ...)
2 stars 2 forks source link

[BUG] S3-SL2-NTC fails with exit code 129 on S3A products #1012

Closed eroan-marie closed 1 year ago

eroan-marie commented 1 year ago

Environment:

Traceability:

Current Behavior:

After adding some missing parameters in the stream-parameters.properties :

app.preparation-worker.s3-type-adapter.optionalOutputs.S3A_SL2=SL_2_LST___
app.preparation-worker.s3-type-adapter.optionalOutputs.S3B_SL2=SL_2_LST___
app.housekeep.s3-type-adapter.optionalOutputs.S3A_SL2=SL_2_LST___
app.housekeep.s3-type-adapter.optionalOutputs.S3B_SL2=SL_2_LST___

app.preparation-worker.s3-type-adapter.mpcSearch.S3A_SL2.productTypes=SL_1_RBT___
app.preparation-worker.s3-type-adapter.mpcSearch.S3B_SL2.productTypes=SL_1_RBT___
app.housekeep.s3-type-adapter.mpcSearch.S3A_SL2.productTypes=SL_1_RBT___
app.housekeep.s3-type-adapter.mpcSearch.S3B_SL2.productTypes=SL_1_RBT___

S3B products work just fine but S3A products fail with exit code 129 saying "Job order: no supported output product (WST, WCT, LST) specified"

Expected Behavior:

After the addition, S3A and S3B should work

Steps To Reproduce:

After adding the missing parameters in the stream-parameters.properties, try ingesting S3A_SL1 products. See error in the logs below.

Test execution artefacts (i.e. logs, screenshots…) Tip: You can attach images or log files by dragging & dropping, selecting or pasting them. s3-sl2-ntc.log

Whenever possible, first analysis of the root cause

Bug Generic Definition of Ready (DoR)

Bug Generic Definition of Done (DoD)

LAQU156 commented 1 year ago

IVV_CCB_2023_w26 : Moved into "Accepted Werum" for implementation, Priority blocking, to be fixed phase 1

Woljtek commented 1 year ago

Please have a look to the OPS PR to retrieve the good configuration=> Preint s3 sl2 ntc #196

LAQU156 commented 1 year ago

Werum_CCB_2023_w26 : Moved into "Product Backlog" to update the configuration

w-jka commented 1 year ago

@Woljtek The PR looks ok to me, there are a lot more changes in there than I expected, however the configuration of the optional outputs is correct. We updated the default configuration to include this as well.

vgava-ads commented 1 year ago

Delivered in Sentinel-1 v1.14.0 (refer to https://github.com/COPRS/processing-sentinel-1/releases) and in Sentinel-3 v1.14.0 (refer to https://github.com/COPRS/processing-sentinel-3/releases)

pcuq-ads commented 1 year ago

SYS_CCB_w27 : This issue is fixed with the update of configuration on COPRS/rs-config/OPS. The version 1.14.0 embeds the good configuration. The ticket can be closed.