search

serverless / serverless-plugin-log-retention

Control the retention of your serverless function's cloudwatch logs.
MIT License
20 stars 16 forks source link

Serverless v2 Validation Failed #7

Open baizhenyu opened 3 years ago

baizhenyu commented 3 years ago

When deploy with serverless v2, I got the following warning:

Serverless:   at 'functions.timeZoneCollector': unrecognized property 'logRetentionInDays'

I believe this caused by introduction of schema validation in v2: https://www.serverless.com/framework/docs/providers/aws/guide/plugins/#extending-validation-schema

Can you add schema validation to this?

joquijada commented 3 years ago

PR created, #8

FedeMadoery commented 2 years ago

Any news about it? I saw the PR ready to go :railway_track:

joquijada commented 2 years ago

Any news about it? I saw the PR ready to go 🛤️

Sadly no. There's no one with write access apparently available to merge. As a workaround I copied this module into my local and packaged it along with my dependent app until this PR gets merged. At any rate we might be moving to SST (Serverless Stack) which will render this unnecessary for us, however that won't happen overnight so it would be night to continue to get support on all serverless framework plugins like this one. Thanks, and good luck.

Cheers, Jose.

ArtificerEntertainment commented 2 years ago

To better support the users of this software, I will be transitioning ownership and maintenance of this repo to Serverless. The details should be sorted out within a couple of weeks.