Skip to content

Commit

Permalink
rename lambda func: copy_harmony_results_to_s3 --> copy_harmony_outpu…
Browse files Browse the repository at this point in the history
…t_to_s3 (#12)
  • Loading branch information
voxparcxls authored Mar 5, 2024
1 parent d328def commit cb70270
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 10 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- `bignbit` Issue #2: [Create github action pipeline to build artifacts](https://github.com/podaac/bignbit/issues/2),
- `bignbit` Issue #3: [Update terraform mock deployment of cumulus module to services accounts](https://github.com/podaac/bignbit/issues/3),


### Changed
- `bignbit` Issue #6: [BIG terraform failing in SWOT venues due to long lambda name](https://github.com/podaac/bignbit/issues/6),


[unreleased]: https://github.com/olivierlacan/keep-a-changelog/compare/v0.0.1...HEAD
Expand Down
2 changes: 1 addition & 1 deletion bignbit/copy_harmony_results_to_s3.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

from bignbit import utils

CUMULUS_LOGGER = CumulusLogger('copy_harmony_results_to_s3')
CUMULUS_LOGGER = CumulusLogger('copy_harmony_output_to_s3')


class CMA(Process):
Expand Down
2 changes: 1 addition & 1 deletion examples/cumulus-tf/step.tf
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,7 @@ resource "aws_sfn_state_machine" "sfn_state_machine" {
},
"Copy Harmony Results to S3":{
"Type":"Task",
"Resource":"${module.bignbit_module.copy_harmony_results_to_s3_arn}",
"Resource":"${module.bignbit_module.copy_harmony_output_to_s3_arn}",
"Parameters":{
"cma":{
"event.$":"$",
Expand Down
10 changes: 5 additions & 5 deletions terraform/lambda_functions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -282,17 +282,17 @@ resource "aws_lambda_function" "get_harmony_job_status" {
tags = var.default_tags
}

resource "aws_lambda_function" "copy_harmony_results_to_s3" {
resource "aws_lambda_function" "copy_harmony_output_to_s3" {
depends_on = [
null_resource.upload_ecr_image
]

package_type = "Image"
image_uri = "${aws_ecr_repository.lambda-image-repo.repository_url}:${local.ecr_image_tag}"
image_config {
command = ["bignbit.copy_harmony_results_to_s3.lambda_handler"]
command = ["bignbit.copy_harmony_output_to_s3.lambda_handler"]
}
function_name = "${local.lambda_resources_name}-copy_harmony_results_to_s3-lambda"
function_name = "${local.lambda_resources_name}-copy_harmony_output_to_s3-lambda"
role = var.lambda_role.arn
timeout = 30
memory_size = 256
Expand Down Expand Up @@ -350,7 +350,7 @@ resource "aws_lambda_function" "apply_opera_treatment" {


resource "aws_lambda_function" "build_image_sets" {

depends_on = [
null_resource.upload_ecr_image
]
Expand All @@ -360,7 +360,7 @@ resource "aws_lambda_function" "build_image_sets" {
image_config {
command = ["bignbit.build_image_sets.lambda_handler"]
}

function_name = local.build_image_sets_function_name
role = var.lambda_role.arn
timeout = 15
Expand Down
4 changes: 2 additions & 2 deletions terraform/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ output "get_harmony_job_status_arn" {
value = aws_lambda_function.get_harmony_job_status.arn
}

output "copy_harmony_results_to_s3_arn"{
value = aws_lambda_function.copy_harmony_results_to_s3.arn
output "copy_harmony_output_to_s3_arn"{
value = aws_lambda_function.copy_harmony_output_to_s3.arn
}

output "apply_opera_treatment_arn"{
Expand Down

0 comments on commit cb70270

Please sign in to comment.