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 configuration for CloudWatch Log for choosing LogGroupClass #12278

Open
1 task done
davidsoendoro opened this issue Nov 28, 2023 · 5 comments
Open
1 task done

Add configuration for CloudWatch Log for choosing LogGroupClass #12278

davidsoendoro opened this issue Nov 28, 2023 · 5 comments

Comments

@davidsoendoro
Copy link

Is there an existing issue for this?

  • I have searched existing issues, it hasn't been reported yet

Use case description

AWS CloudWatch Log has just released a new option for log class at the 26th of November 2023. The new option is to use Infrequent Access log type on storing the logs in CloudWatch. This option is suitable for those who are not using features of CloudWatch Logs other than storing the logs as it enables 50% lower cost for data ingestion. Full announcement of the release is available here

Proposed solution (optional)

Add an additional configuration in the AWS function

functions:
  myCloudWatchLog:
    handler: myCloudWatchLog.handler
    events:
      - cloudwatchLog:
          logGroup: '/aws/lambda/hello'
          filter: '{$.userIdentity.type = Root}'
          logClass: 'Infrequent Access'

Which value will be used in the CloudFormation LogGroupClass as described here. Example:

{
  "Type" : "AWS::Logs::LogGroup",
  "Properties" : {
      "DataProtectionPolicy" : Json,
      "KmsKeyId" : String,
      "LogGroupClass" : String,
      "LogGroupName" : String,
      "RetentionInDays" : Integer,
      "Tags" : [ Tag, ... ]
    }
}
darcoli pushed a commit to darcoli/serverless that referenced this issue Nov 29, 2023
darcoli pushed a commit to darcoli/serverless that referenced this issue Nov 29, 2023
darcoli pushed a commit to darcoli/serverless that referenced this issue Nov 29, 2023
darcoli pushed a commit to darcoli/serverless that referenced this issue Nov 29, 2023
darcoli pushed a commit to darcoli/serverless that referenced this issue Nov 29, 2023
darcoli pushed a commit to darcoli/serverless that referenced this issue Nov 29, 2023
@JaredAAT
Copy link

JaredAAT commented Dec 7, 2023

related: #12264

@davidsoendoro
Copy link
Author

related: #12264

Jared I don’t think it’s related, it’s talking about some other logging feature.

@darcoli
Copy link

darcoli commented Dec 18, 2023

@davidsoendoro please see this comment and confirm whether you agree on the mentioned approach

darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 3, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 3, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 3, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 4, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 4, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 4, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 4, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 4, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 4, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 4, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 4, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 5, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 5, 2024
@SadmiB
Copy link

SadmiB commented Jan 10, 2024

I'm interested in the same feature. AWS introduced CloudWatch logs Infrequent Access storage class, offering less cost than the standard one. However, it seems the serverless framework still hasn't adapted the feature yet.

it would be nice to have it.

darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 19, 2024
darcoli pushed a commit to darcoli/serverless that referenced this issue Jan 19, 2024
@affanshahab
Copy link

Any update on this yet?

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

No branches or pull requests

5 participants