US142337 Upgrade serverless to 2.x

[mostlyfabulous]

US142338: [Telemetry Service] Upgrade serverless to 2.x to resume deployment

serverless 1.x and Node 16.x don't work well together and serverless can't deploy properly as a result.

[AntonBazhal]

This fork should be deleted. All changes we made here had been eventually merged into the original repo, so there is no reason to continue using this.

[mostlyfabulous]

@AntonBazhal It seems there's only one important difference between the Brightspace fork and the original: ACloudGuru/serverless-plugin-aws-alerts@master...Brightspace:serverless-plugin-aws-alerts:main#diff-bfe9874d239014961b1ae4e89875a6155667db834a410aaaa2ebe3cf89820556R164

[AntonBazhal]

@AntonBazhal It seems there's only one important difference between the Brightspace fork and the original: ACloudGuru/serverless-plugin-aws-alerts@master...Brightspace:serverless-plugin-aws-alerts:maindiff-bfe9874d239014961b1ae4e89875a6155667db834a410aaaa2ebe3cf89820556R164

I'm not sure I follow. Are you talking about evaluateLowSampleCountPercentile?

[mostlyfabulous]

Yes, between our fork the main repo, I think the main repo doesn't accept that option

[AntonBazhal]

Yes, between our fork the main repo, I think the main repo doesn't accept that option

It does https://github.com/ACloudGuru/serverless-plugin-aws-alerts#using-percentile-statistic-for-a-metric

[mostlyfabulous]

It does https://github.com/ACloudGuru/serverless-plugin-aws-alerts#using-percentile-statistic-for-a-metric

Ah okay should we make this repo read only then?

[AntonBazhal]

It does https://github.com/ACloudGuru/serverless-plugin-aws-alerts#using-percentile-statistic-for-a-metric

Ah okay should we make this repo read only then?

I think it's a good idea.