Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add ca-west-1 region #703

Merged

Conversation

jgrammen-agilitypr
Copy link
Contributor

Add ca-west-1 region to region list in aws.rb
This is an attempt to fix the missing region (based on comments in #701 ), my live testing suggests it works, but I am inexperienced with ruby programming.

fixes #701

Add ca-west-1 region to region list in aws.rb
Copy link
Member

@geemus geemus left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@geemus
Copy link
Member

geemus commented Mar 12, 2024

Note: I'm not sure what's up with the actions/tests. The "expected" stuff is all the old versions, which I thought I recently disabled in testing to focus on non-EOL rubies. In any event, I don't think we need to block on this here.

@geemus geemus merged commit 9dd7c7d into fog:master Mar 12, 2024
8 checks passed
@geemus
Copy link
Member

geemus commented Mar 12, 2024

Released in v3.21.1.

@jgrammen-agilitypr
Copy link
Contributor Author

@geemus thank you for the new version. 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing AWS Region - ca-west-1 (Calgary)
2 participants