Conversation
Added granule-invalidator Terraform and deployment mechanism Added the `deploy` dir to the create-release-artifacts.sh script so task Terraform is included in release
for more information, see https://pre-commit.ci
Updated package.json based on updates to scripts
…mulus into mikedorfman/CUMULUS-4385
for more information, see https://pre-commit.ci
…mulus into mikedorfman/CUMULUS-4385
for more information, see https://pre-commit.ci
…mulus into mikedorfman/CUMULUS-4384
Updated various issues while troubleshooting related to granule-invalidator
for more information, see https://pre-commit.ci
Added a private_api_lambda_arn variable for the ingest module
…mulus into mikedorfman/CUMULUS-4384
| "collectionId": f"{collection}___{version}", | ||
| } | ||
| for invalid_granule in invalid_granules | ||
| invalid_granule["granuleId"] for invalid_granule in invalid_granules |
There was a problem hiding this comment.
This change was based on recent Cumulus updates which accept only a list of granule IDs in a bulk delete.
| filename = "${path.module}/../dist/lambda.zip" | ||
| source_code_hash = filebase64sha256("${path.module}/../dist/lambda.zip") | ||
| handler = "main.handler" | ||
| filename = "${path.module}/../dist/final/lambda.zip" | ||
| source_code_hash = filebase64sha256("${path.module}/../dist/final/lambda.zip") | ||
| handler = "main.lambda_handler" |
There was a problem hiding this comment.
Packaging updates result in a different path for the file
| "Type": "Choice", | ||
| "Choices": [ | ||
| { | ||
| "Variable": "$deletion_count", |
There was a problem hiding this comment.
| "Variable": "$deletion_count", | |
| "Variable": "$.deletion_count", |
Small missing "." to help with the reference
There was a problem hiding this comment.
Ah, this is actually correct without the . - I've used a StepFunction variable to assign a value to deletion_count. Unlike referencing input, vars are referenced with just the $ at the beginning.
| "Type": "Choice", | ||
| "Choices": [ | ||
| { | ||
| "Variable": "$deletion_count", |
There was a problem hiding this comment.
| "Variable": "$deletion_count", | |
| "Variable": "$.deletion_count", |
There was a problem hiding this comment.
Ah, this is actually correct without the . - I've used a StepFunction variable to assign a value to deletion_count. Unlike referencing input, vars are referenced with just the $ at the beginning.
| expect(workflowExecution.status).toEqual('completed'); | ||
| }); | ||
|
|
||
| it('ingestDateDate rolloff configuration is honored', async () => { |
There was a problem hiding this comment.
Did you intend for "ingestDateDate", or just "ingestDate"
There was a problem hiding this comment.
Definitely me stuttering - I'll fix this here and in a few other spots.
| } | ||
| )).toBeRejectedWithError(CumulusApiClientError, /404/); | ||
|
|
||
| const afteringestDateDateTimeCutoffGranule = await getGranule( |
There was a problem hiding this comment.
Maybe its intentional, I might be missing something
2 participants
Summary: Summary of changes
Addresses CUMULUS-4384: Add workflow to example deployment for rolling archive
Changes
Fixed the following:
package.shwas not appropriately packaging dependencies. Reverted back touv pip installwith a target platform specified.PRIVATE_API_LAMBDA_ARNenv var in the lambda and appropriate terraform vars in cumulus/archive, cumulus/ingest and the task Terraform to provide this arn.PR Checklist
📝 Note:
For most pull requests, please Squash and merge to maintain a clean and readable commit history.