Add PE worker log group and make sure desired count gets set back to 0

cisagov / crossfeed

External monitoring for organization assets

https://docs.crossfeed.cyber.dhs.gov

Creative Commons Zero v1.0 Universal

366 stars 54 forks source link

Add PE worker log group and make sure desired count gets set back to 0 #2393

Closed aloftus23 closed 10 months ago

aloftus23 commented 10 months ago

🗣 Description

The pe-staging-worker log group isn't created automatically so need to add this to the terraform.

Another oversight is when the desired count is set to 5, we need to make sure that when all the messages have gone through, we reset the desired count to 0 so that no more fargate tasks run.

aloftus23 commented 10 months ago

Looks great. My only feedback is the comment on line 188 of infrastructure/pe_worker.tf isn't very clear.

Good catch I removed. That was a comment I copied over that isn't relevant here