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

Add cirrus lambdas #73

Merged
merged 18 commits into from
Aug 20, 2024
Merged

Add cirrus lambdas #73

merged 18 commits into from
Aug 20, 2024

Conversation

jkeifer
Copy link
Member

@jkeifer jkeifer commented Aug 15, 2024

Related issue(s)

  • n/a

Proposed Changes

  1. Add lambdas to cirrus deployment
  2. Remove unneeded cirrus IAM roles
  3. Remove deprecated cirrus failed SNS topic
  4. Rename stac-server update-lambdas.sh script to update-stac-server-lambdas.bash

Of the three lambdas with triggers (api, process, update-state), only the latter two have the triggers added here. The api is triggered off an API Gateway and that configuration has been omitted simply due to the level of effort required. That API Gateway configuration will need to be completed via future work.

Testing

This change was validated by the following observations:

  1. Deploy and check configurations of new lambdas
  2. Try running test payloads into lambdas
  3. Validate triggers have been created (where applicable)

Note that fully testing the lambdas is not currently possible due to the incomplete cirrus deployment. It might be worth having terraform optionally create S3 buckets and any other missing resources to allow the flop deployment to be testable.

Checklist

  • I have deployed and validated this change
  • Changelog
    • I have added my changes to the changelog
    • No changelog entry is necessary
  • README migration
    • I have added any migration steps to the Readme
    • No migration is necessary

@jkeifer jkeifer force-pushed the jak/cirrus-lambdas branch 2 times, most recently from 04da35a to 2e533d4 Compare August 19, 2024 19:30
@jkeifer jkeifer marked this pull request as ready for review August 20, 2024 00:03
@jkeifer jkeifer self-assigned this Aug 20, 2024
@hectormachin hectormachin merged commit 1639ab0 into main Aug 20, 2024
4 checks passed
@hectormachin hectormachin deleted the jak/cirrus-lambdas branch August 20, 2024 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants