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

feat: add UAE Spain and Zurich to the AWS regions #11524

Merged
merged 1 commit into from Nov 17, 2022
Merged

feat: add UAE Spain and Zurich to the AWS regions #11524

merged 1 commit into from Nov 17, 2022

Conversation

umutuzgur
Copy link
Contributor

@umutuzgur umutuzgur commented Nov 17, 2022

Closes: #11522

@umutuzgur umutuzgur changed the title add UAE Spain and Zurich to the AWS regions feat: add UAE Spain and Zurich to the AWS regions Nov 17, 2022
@codecov
Copy link

codecov bot commented Nov 17, 2022

Codecov Report

Base: 85.63% // Head: 85.63% // No change to project coverage 👍

Coverage data is based on head (7fdb9bb) compared to base (20d79a2).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #11524   +/-   ##
=======================================
  Coverage   85.63%   85.63%           
=======================================
  Files         314      314           
  Lines       13133    13133           
=======================================
  Hits        11246    11246           
  Misses       1887     1887           
Impacted Files Coverage Δ
lib/plugins/aws/provider.js 94.70% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@umutuzgur
Copy link
Contributor Author

Although pushing changes like this make me feel like an opensource contributor again 😂 , it would be great to automate this 😬

Copy link
Contributor

@medikoo medikoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @umutuzgur !

How do you think, it can be automated?

@medikoo medikoo merged commit 935d8e9 into serverless:main Nov 17, 2022
@umutuzgur
Copy link
Contributor Author

I was thinking about a github actions cron job that runs everyday, parses an endpoints where the DCs are located from AWS which I assume they have something for this, compares it with the ones in this file and creates a new PR if it detects a drift

@umutuzgur umutuzgur deleted the patch-1 branch November 17, 2022 14:13
@medikoo
Copy link
Contributor

medikoo commented Nov 17, 2022

@umutuzgur do you think creating such PR and involved maintenance work of auto-update logic is worth it in comparison to manual update of the list we practise currently?

@umutuzgur
Copy link
Contributor Author

@medikoo Maybe not. I don't really know how often people create issues regarding the unsupported regions. I would say it is not worth it since AWS has 2-3 new DCs every year but it might be just nice to get rid of a recurring situation once and for all and not think about it again.

@medikoo
Copy link
Contributor

medikoo commented Nov 17, 2022

@umutuzgur if you're into, you may prepare a PR that will automate that, but I assume it's complexity will be big, and I would be reluctant to take it assuming. we have to do such update as today maybe once a year.

It seems that less cost and friction is when this is handled manually

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support AWS - UAE Region
2 participants